player-adapter-contract #8

Merged

dtourolle merged 2 commits from player-adapter-contract into master 2026-07-02 18:02:18 +00:00

Conversation 0 Commits 2 Files Changed 17 +1224 -312

dtourolle commented 2026-07-02 18:02:03 +00:00

Owner

Copy Link

No description provided.

dtourolle added 2 commits 2026-07-02 18:02:03 +00:00

Introduce PlayerAdapter contract; decision logic shared in Rust backend ... a64e1b1fb4

Establish a decoupled player boundary so UI and backend interact with video
through one contract, with the HTML5 (Linux/interim-Android) and native
(ExoPlayer) providers as interchangeable primitive-executor adapters.

- PlayerAdapter interface + AdapterHost callback bag (adapters/types.ts): the
  adapter owns only decision-free element PRIMITIVES (seekElement, reloadSource,
  play/pause, setVolume, selectSubtitle); it never branches on strategy.
- Seek/audio-track DECISIONS stay in Rust (player_seek_video / _switch_audio_track
  return a strategy); the facade dispatches the chosen primitive to the active
  adapter. Both providers share the one decision path — logic lives once, in Rust.
- Facade holds the active adapter; a new ControlCommand PlayerStatusEvent lets
  backend control (lockscreen/remote/sleep) drive the webview <video> element.
- Html5PlayerAdapter resolves the LIVE element via the bridge (fixes play/pause
  silently no-opping when the element was re-bound).
- Do not emit a "stopped" player state on natural end-of-video: it flipped the
  player/mode to idle mid-handoff and suppressed next-episode auto-advance under
  a sleep timer. Jellyfin progress reporting is preserved; the backend's
  on_video_playback_ended owns the transition.
- VideoPlayer net -300 lines (strategy/HLS-reload logic relocated to the adapter).
- Adds 20 adapter unit tests; existing suites stay green (vitest 457, cargo 416).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Use incremental build 37455bc470

dtourolle merged commit 0b5a3aa176 into master 2026-07-02 18:02:18 +00:00

dtourolle referenced this issue from a commit 2026-07-02 18:02:19 +00:00

Merge pull request 'player-adapter-contract' (#8) from player-adapter-contract into master

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

dtourolle

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dtourolle/jellytau#8

No description provided.