feat: allow replacing the default implementations of the cache manager and storage

[AlexV525]

The storage interface has been extracted from the default implementation. Users can implement their own instead of using Get Storage or Shared Preferences.

[OutdatedGuy]

Mostly LGTM, few questions and changes.

[AlexV525]

I've made a couple of changes additionally:

1. The storage interface was extracted with a more general interface that can be used to store any type of data.
2. I've renamed VideoPlayerStorage to VideoPlayerMetadataStorage and made a type alias to deprecate VideoPlayerStorage, but I think we can safely remove it since the class was not exported directly before.

[AlexV525]

@OutdatedGuy friendly ping

[OutdatedGuy]

LGTM! Will once check updated example app and then merge.

Thanks for the contribution!

[OutdatedGuy]

Screen.Recording.2025-10-04.at.12.39.48.PM.mov

@AlexV525 the example app doesn't seems to be working correctly

[AlexV525]

Screen.Recording.2025-10-04.at.12.39.48.PM.mov
@AlexV525 the example app doesn't seems to be working correctly

I'll look into it next week

[OutdatedGuy]

Hi @AlexV525, do you plan on fixing the example app? :)

[AlexV525]

The example fails because the file caches always read from the SharedPreferences as the previous code implemented:

cached_video_player_plus/example/lib/pages/advance_cache_management_page.dart

Line 83 in f5a32f8

final allKeys = await _asyncPrefs.getKeys();

This is a hard-coded behavior that implicitly uses the same storage as the default metadata storage.

I'll update the storage to provide an interface method like FutureOr<List<String>> get keys to get rid of this.

[Copilot]

Pull Request Overview

This PR introduces extensibility to the cached video player by extracting storage into an interface and making the default cache manager and metadata storage implementations replaceable. This allows users to provide custom implementations instead of being locked into SharedPreferences or the default cache manager.

Key Changes:

• Extracted IVideoPlayerMetadataStorage interface from the default implementation, enabling custom storage implementations
• Made static cache manager and metadata storage fields mutable (non-final) to allow runtime replacement
• Added optional constructor and static method parameters for custom cache manager and metadata storage instances

Reviewed Changes

Copilot reviewed 11 out of 15 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
lib/src/i_video_player_metadata_storage.dart	New interface defining the contract for video metadata storage implementations
lib/src/video_player_metadata_storage.dart	Renamed from VideoPlayerStorage and updated to implement the new interface
lib/src/cached_video_player_plus.dart	Added mutable static fields for cache manager and metadata storage, updated constructors and static methods to accept custom implementations
lib/cached_video_player_plus.dart	Exported new interface and implementation classes for public API
example/pubspec.yaml	Reorganized dependencies (moved flutter SDK to top, removed direct shared_preferences dependency)
example/lib/pages/advance_cache_management_page.dart	Demonstrated custom implementations with in-memory metadata storage and custom cache manager toggle functionality
example/lib/main.dart	Minor UI adjustment wrapping title text in FittedBox for better layout
example/pubspec.lock	Updated transitive dependency versions
example/macos/Runner.xcodeproj/project.pbxproj	Updated macOS deployment target to 10.15 and added CLANG configuration
example/ios/Runner.xcodeproj/project.pbxproj	Added CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES setting
example/macos/Podfile	Updated minimum macOS platform version to 10.15
example/macos/Podfile.lock	Updated pod checksums reflecting dependency updates
example/ios/Podfile.lock	Updated Flutter framework checksum
example/macos/.gitignore	Added DerivedData to ignored files
.gitignore	Added devtools_options.yaml to ignored files

Comments suppressed due to low confidence (1)

lib/src/video_player_metadata_storage.dart:12

• The comment references the old class name VideoPlayerStorage but the class has been renamed to VideoPlayerMetadataStorage. The comment should be updated to match the new class name.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[OutdatedGuy]

Lot's of unrelated files changed.
Few nitpicks and bugs in example app

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

lib/src/video_player_metadata_storage.dart:12

• The comment still references the old class name VideoPlayerStorage but should be updated to VideoPlayerMetadataStorage.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[OutdatedGuy]

LGTM!

[OutdatedGuy]

Thanks for the contribution @AlexV525