Skip to content

feat: enhance Android system media controls - #252

Merged
Quick104 merged 3 commits into
mainfrom
feat/android-now-playing-artwork
Aug 27, 2026
Merged

feat: enhance Android system media controls#252
Quick104 merged 3 commits into
mainfrom
feat/android-now-playing-artwork

Conversation

@Quick104

@Quick104 Quick104 commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • publish wide catalog artwork to Android local-playback media sessions through a Coil-backed Media3 bitmap loader
  • expose active Silo Remote Control sessions in Android system media controls, including transport, seek, next, metadata, and artwork
  • keep the Remote Control service active while playback is engaged and guide users to disable battery optimization
  • use cached or fetched item detail as a backward-compatible artwork fallback while the watch response lacks image fields

Validation

  • full Gradle test suite passed
  • phone debug APK assembled successfully
  • TV debug APK assembled successfully
  • physical Android playback harness passed with advancing playback
  • live Android notification reported a 144x81 bitmap large icon and visibly rendered the backdrop

Follow-up

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 40 seconds.

View limit details

Limit details: You’ve used the included review currently available. Your 95 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: a09438d8-0a6f-4e2d-a15e-e4b35b77fb32

📥 Commits

Reviewing files that changed from the base of the PR and between 4aec538 and 742f62f.

📒 Files selected for processing (17)
  • android-shared/src/androidMain/kotlin/org/siloserver/silo/common/player/SiloMediaSessionBitmapLoader.kt
  • android-shared/src/androidMain/kotlin/org/siloserver/silo/common/player/SiloPlaybackService.kt
  • androidApp/src/androidMain/AndroidManifest.xml
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/SiloApplication.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/cast/RemoteControlBatteryOptimization.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/cast/SiloCastArtworkResolver.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/cast/SiloCastController.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/cast/SiloCastMediaSessionService.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/cast/SiloCastMediaSessionStarter.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/ui/screens/cast/SiloCastArtwork.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/ui/screens/cast/SiloCastRemoteScreen.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/ui/screens/player/MobileVideoPlaybackStarter.kt
  • androidApp/src/androidMain/kotlin/org/siloserver/silo/android/ui/screens/player/PlayerViewModel.kt
  • androidApp/src/androidMain/res/values/strings.xml
  • androidApp/src/androidUnitTest/kotlin/org/siloserver/silo/android/cast/SiloCastMediaSessionStarterTest.kt
  • androidTvApp/src/androidMain/kotlin/org/siloserver/silo/tv/ui/screens/player/TvVideoPlaybackStarter.kt
  • shared/src/commonMain/kotlin/org/siloserver/silo/model/catalog/CatalogModels.kt

Comment @coderabbitai help to get the list of available commands.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4e9afa88e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6aa460f496

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@Quick104
Quick104 merged commit 73d8979 into main Aug 27, 2026
3 checks passed
@Quick104
Quick104 deleted the feat/android-now-playing-artwork branch August 27, 2026 23:01
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