Skip to content

Conversation

Harsh3991
Copy link

@Harsh3991 Harsh3991 commented Oct 13, 2025

…ce for expo-av/expo-video

Enhanced Error Handling and Availability Info for Expo Video Adapters

Improves error messages and adds installation guidance for expo-av/expo-video dependencies.

Brief Summary of Changes

  • Added getAvailabilityInfo() method to all video adapters
  • Enhanced error messages with adapter names and install commands
  • Stored actual error details from module loading failures
  • Added comprehensive test coverage and documentation

What does this PR address?

  • New feature
  • Bug fix
  • Adds more tests

Are tests included?

  • Yes

Reviewer, please note:

  • Non-breaking change - existing code works without modification
  • New optional getAvailabilityInfo() method provides detailed error context
  • Error messages now include specific installation commands (e.g., npx expo install expo-av)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I ran the full test suite before pushing the changes and all the tests pass.

Files Changed:

  • src/adapters/ExpoAVVideoAdapter.tsx - Enhanced error handling
  • src/adapters/ExpoVideoAdapter.tsx - Enhanced error handling
  • src/adapters/FallbackVideoAdapter.tsx - Added availability info
  • src/adapters/types.ts - Added interface for availability info
  • src/adapters/__tests__/ErrorHandling.test.ts - New comprehensive tests
  • ENHANCED_ERROR_HANDLING.md - Documentation and examples

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant