refactor: move audio player to a separate module (WPB-27633)

[ohassine]

https://wearezeta.atlassian.net/browse/WPB-27633

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Description

• Move audio player to a separate module
• Open audio files in the chat in audio player
• rename video player module from media-player to video-player

Needs releases with:

• GitHub link to other pull request

Testing

Test Coverage (Optional)

• I have added automated test to this contribution

How to Test

Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.

Notes (Optional)

Specify here any other facts that you think are important for this issue.

Attachments (Optional)

Attachments like images, videos, etc. (drag and drop in the text box)

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[pull-request-size]

Ups 🫰🟨

This PR is too big. Please try to break it up into smaller PRs.

[github-actions]

Test Results

2 639 tests   2 639 ✅  1m 34s ⏱️
  336 suites      0 💤
  336 files        0 ❌

Results for commit 256f50e.

♻️ This comment has been updated with latest results.

[AndroidBob]

New ADR(s) in this PR 📚:

13. Shared in-app video player as a core module

Date: 2026-07-16

Status

Proposed

Context

The in-app video player (VideoPlayerScreen, VideoPlayerViewModel, VideoPlaybackState,
VideoViewerNavArgs) currently lives in features:cells and is reachable only from the cells file
browser. It is a self-contained, reusable UI component: it takes a local path / content URL and plays
it via ExoPlayer — it holds no domain logic of its own (assets, download and offline handling already
live in Kalium and in the calling features).

We now want to play videos in-app when a user taps a video message in a chat. Today the chat
(which lives in the app module, ui/home/conversations/…) hands videos off to an external app via
an ACTION_VIEW intent. Reusing the existing player would give a consistent, in-app experience.

The player therefore needs to be consumed by two places: features:cells and app (chat). The
project's dependency law is kalium → core → features → app, with the additional rule that features
must not depend on other features — shared logic goes into core:* or Kalium. That rules out the two
shortcuts:

• Keeping it in features:cells and calling it from chat couples the entire "Wire Cells file storage
  and collaboration" feature to chat, and the screen is bound to the cells navigation graph via
  @WireCellsDestination.
• Creating a features:video-player module is illegal to consume from features:cells (feature →
  feature dependencies are forbidden).

Because the player is shared UI infrastructure used by multiple layers above core, its correct home
is a core module.

Decision

Extract the video player into a new core:video-player module and consume it from both
features:cells and app.

A new dedicated core module is preferred over folding it into the existing core:media module so that
the heavy media dependencies (media3-exoplayer, media3-ui, coil3-video) stay scoped to modules
that actually play video, rather than leaking to everything that already depends on core:media
(currently only PingRinger). The new module can later become the home for the other in-app media
viewers (image viewer, media gallery) if we choose to consolidate them.

The two pieces that are currently cells-specific will be generalized during the move:

• Navigation: replace @WireCellsDestination with a single shared destination registered through
  core:navigation, so the player is navigable from any feature/app graph rather than only the cells
  graph.
• Dependency injection: move the Metro wiring out of CellsViewModelFactory so core:video-player
  provides its own VideoPlayerViewModel.

The work is sequenced extract-then-reuse, so the risky refactor is validated against the existing
caller before chat depends on it:

1. Create core:video-player and move the player files into it, generalizing navigation and DI. No
   user-facing behavior change.
2. Migrate features:cells to navigate to the shared destination and delete its copy of the player.
   This proves parity with the existing flow.
3. Wire chat in app: on tapping a video message, navigate to the shared destination (passing the
   already-resolved local path / content URL) instead of firing the external ACTION_VIEW intent.

Consequences

• A new module core:video-player is added; it is auto-discovered by settings.gradle.kts and built
  with the wire-android-library convention plugin (copied from an existing core module, not from
  features/template). It may depend on core:ui-common, core:navigation, and core:di — all
  core → core, one-directional, no cycle.
• features:cells and app both depend on core:video-player; the kalium → core → features → app
  direction is preserved and no cross-feature dependency is introduced.
• The video player becomes reusable from anywhere, and chat gains an in-app video experience instead
  of delegating to an external app.
• The player's navigation destination is no longer part of the cells graph; callers navigate to the
  shared destination. Any deep links or existing navigation to the cells-scoped destination must be
  updated as part of Phase 2.
• The media3/Coil-video dependencies move to (and are scoped by) the new module.
• Follow-up opportunity (out of scope here): the cells in-app image viewer (CellImageViewerScreen)
  and the app MediaGalleryScreen overlap; core:video-player is a natural future home for a unified
  media viewer, to be handled as a separate ADR/change.

[codecov]

Codecov Report

❌ Patch coverage is 6.57895% with 71 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.36%. Comparing base (969346a) to head (256f50e).

Files with missing lines	Patch %	Lines
...kotlin/com/wire/android/audioplayer/AudioPlayer.kt	4.61%	62 Missing ⚠️
...getypes/multipart/MultipartAttachmentsViewModel.kt	0.00%	1 Missing and 3 partials ⚠️
...conversations/messages/item/MessageClickActions.kt	0.00%	2 Missing ⚠️
...m/wire/android/videoplayer/VideoPlayerViewModel.kt	0.00%	2 Missing ⚠️
...e/android/audioplayer/AudioPlayerViewModelGraph.kt	0.00%	1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (6.57%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #5112      +/-   ##
===========================================
- Coverage    52.55%   52.36%   -0.20%     
===========================================
  Files          740      742       +2     
  Lines        25745    25847     +102     
  Branches      4224     4233       +9     
===========================================
+ Hits         13530    13534       +4     
- Misses       10920    11016      +96     
- Partials      1295     1297       +2     

Files with missing lines	Coverage Δ
...me/conversations/ConversationNavigation3Entries.kt	14.70% <ø> (ø)
...messagetypes/multipart/MultipartAttachmentsView.kt	0.00% <ø> (ø)
.../com/wire/android/ui/sharing/ImportMediaNavArgs.kt	0.00% <ø> (ø)
...com/wire/android/audioplayer/AudioPlaybackState.kt	100.00% <ø> (ø)
...m/wire/android/audioplayer/AudioPlayerViewModel.kt	96.00% <100.00%> (ø)
...com/wire/android/videoplayer/VideoPlaybackState.kt	0.00% <ø> (ø)
...kotlin/com/wire/android/videoplayer/VideoPlayer.kt	0.00% <ø> (ø)
...e/android/videoplayer/VideoPlayerViewModelGraph.kt	0.00% <ø> (ø)
...id/feature/cells/ui/CellsMetroViewModelBindings.kt	0.00% <ø> (ø)
...e/android/audioplayer/AudioPlayerViewModelGraph.kt	0.00% <0.00%> (ø)
... and 4 more

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 969346a...256f50e. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!