Stabilize watchlist paging during sync updates

[anod]

Summary

• Fix the duplicate Lazy key crash by making each WatchListPagingSource generation page over a stable app row-id snapshot
• Prevent sync/status updates from moving the same app across OFFSET windows while Paging appends more pages
• Keep recently-discovered ordering valid for row snapshots and add regression coverage for rows moving across page offsets

Crash investigation

Crashlytics logs for the sample event showed manual sync running while the watchlist was paging. During sync, apps including Chrome were marked as viewed, changing app status and reordering the STATUS DESC list between an initial 60-item load and later appends. That can emit the same app row in multiple pages and trigger Compose duplicate Lazy keys.

Validation

• ./gradlew :app:assembleDebug
• ./gradlew :app:testDebugUnitTest
• ./gradlew :app:lintDebug
• ./gradlew ktlintCheck

[Copilot]

Pull request overview

This PR updates the app-list SQL used by watchlist-style paging to ensure each app_list row joins to at most one changelog row per (app_id, version_code), preventing duplicate emitted rows that can trigger Compose Lazy key collisions when legacy/invalid changelog duplicates exist.

Changes:

• Update AppListTable.Queries.createAppsListQuery() to join via a derived “latest changelog row per (app_id, code)” subquery, then join back to changelog by _id.
• Add a regression test that simulates legacy duplicate changelog rows (by dropping the unique index) and asserts the paging result does not duplicate apps.
• Update query-shape assertions in AppListTableQueriesTest to reflect the new join structure.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
app/src/main/java/com/anod/appwatcher/database/AppListTable.kt	Changes the dynamic app-list query to join changelog through a “latest per version” derived table to prevent duplicated app rows.
app/src/test/java/com/anod/appwatcher/database/AppListTableQueriesTest.kt	Updates assertions to validate the new derived-table join shape.
app/src/test/java/com/anod/appwatcher/watchlist/WatchListPagingSourceRoomTest.kt	Adds a regression test covering duplicate changelog versions and a helper to insert legacy-duplicate data.

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 8 changed files in this pull request and generated no new comments.

Suppressed comments (1)

app/src/main/java/com/anod/appwatcher/watchlist/WatchListPagingSource.kt:116

• nextKey calculation still uses loadedDataSize < limit (via calculateKeys), so when snapshot.rows.size is an exact multiple of limit the paging source will advertise an extra append key and then return an empty page at the end. Since you now have the full snapshot size, you can stop precisely when offset + pageRows.size >= snapshot.rows.size to avoid the redundant empty load.

        val (prevKey, nextKey) = calculateKeys(
            key = params.key,
            offset = offset,
            loadSize = params.loadSize,
            loadedDataSize = pageRows.size,