fix: don't crash when visiting too many releases with many tracks

lydavid · lydavid · commit 178542931778 · 2026-03-28T17:31:00.000-04:00
because this would save every track's UUID and recording UUID string representation in a parcel, we can easily reach the 1MB limit and get a TransactionTooLargeException
diff --git a/ui/common/src/commonMain/kotlin/ly/david/musicsearch/ui/common/track/TracksByReleasePresenter.kt b/ui/common/src/commonMain/kotlin/ly/david/musicsearch/ui/common/track/TracksByReleasePresenter.kt
@@ -48,7 +48,7 @@ class TracksByReleasePresenter(
         //  tracks is special in that all are cached at once
         //  For other screens, it's possible new remote data appears as the user scrolls,
         //  but then it would just be similar to now where they can click select all again to pick up the new data.
-        val trackIds: List<SelectableId> by rememberSaveable(releaseId) {
+        val trackIds: List<SelectableId> by rememberRetained(releaseId) {
             mutableStateOf(getTracksIdsByRelease(releaseId))
         }
         val lazyListState: LazyListState = rememberLazyListState()

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ class TracksByReleasePresenter(`
`48`	`48`	`// tracks is special in that all are cached at once`
`49`	`49`	`// For other screens, it's possible new remote data appears as the user scrolls,`
`50`	`50`	`// but then it would just be similar to now where they can click select all again to pick up the new data.`
`51`		`- val trackIds: List<SelectableId> by rememberSaveable(releaseId) {`
	`51`	`+ val trackIds: List<SelectableId> by rememberRetained(releaseId) {`
`52`	`52`	`mutableStateOf(getTracksIdsByRelease(releaseId))`
`53`	`53`	`}`
`54`	`54`	`val lazyListState: LazyListState = rememberLazyListState()`