player-adapter-contract #8

2026-07-02T18:02:03Z

dtourolle commented

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

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

🏗️ Build and Test JellyTau / Run Tests (pull_request) Successful in 3m57s

Details

Traceability Validation / Check Requirement Traces (pull_request) Successful in 21s

Details

🏗️ Build and Test JellyTau / Build Android APK (pull_request) Successful in 19m5s

Details

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.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: dtourolle/jellytau#8