Android #618 offline-mode UX + Sonos polish + server DRY #81

Merged

bvandeusen merged 7 commits from dev into main 2026-06-04 12:53:59 -04:00

Conversation 0 Commits 7 Files Changed 20

+515 -82

7 Commits

Author	SHA1	Message	Date
bvandeusen	1e17eeda72	feat(android): #618 Phase 5 — confirm offline writes via shell snackbar ... MutationQueue now emits "Saved — will sync when online" on a SharedFlow whenever a user-driven enqueue lands (like toggle, playlist append, request create/cancel, quarantine flag/unflag). Background enqueues (play-offline events, playback-error reports) do not emit — those fire from non-foreground paths where a snackbar would be either dropped (no shell mounted) or jarring (lock-screen toggle). ShellScaffold subscribes via OfflineWriteHintViewModel and routes the hint through its existing snackbar host. Replaces the prior silent- queue UX where a tap on a like / playlist add looked successful but the user couldn't tell whether the server had been hit or the call was deferred for replay.	2026-06-04 12:40:59 -04:00
bvandeusen	1daea79f64	feat(android): #618 Phase 4 — row-level offline-unavailable affordance ... TrackRow now consumes the LocalServerHealth CompositionLocal (provided once at MainActivity from ServerHealthController.state). When the server is Offline or ServerDown and the track id isn't in LocalCachedTrackIds, the row dims to 0.4 alpha and a tap fires a Toast instead of attempting playback. Replaces the silent "tap-and-fail-to-OfflineException" UX with explicit at-a-glance signaling of which rows in a long list will work. Trailing slot (kebab / like / playlist-add) stays interactive so write affordances can route through MutationQueue — Phase 5 gates those at the action level.	2026-06-04 12:37:42 -04:00
bvandeusen	48de720514	feat(android): #618 Phase 2 — search falls back to cached entities when offline ... When ServerHealthController reports Offline or ServerDown, SearchRepository runs Room LIKE queries against cached_artists / cached_albums / cached_tracks instead of hitting /api/search. The screen draws a one-line hint above the results so the user can tell server matches from on-device-only matches. Adds searchByName / searchByTitle DAO methods; LOCAL_SEARCH_LIMIT=20 matches the server's default page size.	2026-06-04 12:35:01 -04:00
bvandeusen	fced6b681e	feat(android): cache-only audio path when ServerHealth != Healthy ... Phase 3 of #618. Wraps the OkHttpDataSource upstream of CacheDataSource with OfflineGatedDataSource. CacheDataSource only consults the upstream factory on a cache miss, so playback of cached audio is unaffected. Offline tap on a non-cached track now throws OfflineException immediately (subclass of IOException for ExoPlayer's PlaybackException to wrap) instead of waiting on a multi-second OkHttp timeout. AudioPrefetcher keeps its own ungated upstream -- writes fail silently when offline, no user-visible impact.	2026-06-04 11:27:11 -04:00
bvandeusen	80a6be25aa	feat(android): ServerHealth tri-state composite + banner distinguishes offline vs server-down ... Phase 1 of #618. VersionCheckController gains reachable: StateFlow<Boolean> from the same /healthz poll (no double polling). ServerHealthController combines connectivity.online + versionCheck.reachable into ServerHealth { Healthy, Offline, ServerDown }. ConnectionErrorBanner now branches on the tri-state, distinguishing 'no Wi-Fi' from 'server unreachable.' Phases 2-5 (local search, cache-only audio source, row-level not-cached affordance, write-affordance gray-out) ship separately as independent slices.	2026-06-04 11:25:22 -04:00
bvandeusen	4d0a0b8e09	fix(android): throttle UPnP extend with 50ms delay per successful AddURIToQueue ... Closes Scribe #611. The 2026-06-04 logcat showed 33 consecutive AddURIToQueue failures clustered at ~10ms intervals once the burst hit offset 39 -- characteristic of Sonos's burst-add rate-limit. 50ms between successful adds adds ~5s to the 100-track background extension but eliminates the burst rejection. Next reproduction with the SOAP fault detail logging (audit commit c5b326c6) will confirm the fault code if any tracks still fail.	2026-06-04 11:03:55 -04:00
bvandeusen	6184c62721	feat(android): MediaSession picks up UPnP state via direct listener invocation ... Closes Scribe #606. Two pieces: MediaMetadata gets durationMs (lock-screen scrubber gets a known total even when wrapped ExoPlayer is paused under UPnP); MinstrelForwardingPlayer keeps an externalListeners registry that mirrors super.addListener so we can directly invoke onIsPlayingChanged / onPlaybackStateChanged / onMediaItemTransition when remoteState mutates. Fires from pollOnce + play()/pause() onSuccess. dedup via lastNotified guards so we don't spam events at 1Hz when nothing changed.	2026-06-04 11:02:51 -04:00