[RSM] Smart Bookmarks list UI

[sztomek]

Description

This is the last PR on the android-side of changes, where we hook up the enrichment flow and surface the results in the UI, on the bookmarks list. When a new bookmark is created, the app now fires off a background enrichment call if the Smart Bookmarks feature flag is enabled. The enrichment runs after the bookmark is saved, so the user sees no delay.
On the display side, the bookmark row now shows the AI-generated title when the user hasn't manually edited it, with a displayTitle property that picks the right one based on timestamp priority. If an AI summary is available, it appears as a secondary line beneath the title.

An upcoming PR will let us open the enhanced bookmarks to see its details.

Testing Instructions

Since the backend changes are not yet deployed, the best way to test the new ui in action is to manually inject data into the database via Android Studio's Database inspector tool.
Select pocketcasts database then run these queries:
1- SELECT uuid, podcast_id FROM podcast_episodes ORDER BY published_date DESC LIMIT 1 -- remember these Ids!
2-

INSERT INTO bookmarks (uuid, podcast_uuid, episode_uuid, time, created_at, title, title_modified, deleted, deleted_modified, ai_title, ai_summary, ai_title_modified, ai_summary_modified, sync_status, clean_title)                                                                                                      
  VALUES                                                                                                                                                                                                                                                                                                                    
    ('aaaa-0001-test-bookmark-01', '<PODCAST_UUID>', '<EPISODE_UUID>', 120, 1715100000000, 'Bookmark', 1715100000000, 0, 1715100000000, 'The future of open-source AI', 'Discussion about why open-weight models are becoming the default choice for enterprise adoption.', 1715100000001, 1715100000001, 0, 'bookmark'),
    ('aaaa-0002-test-bookmark-02', '<PODCAST_UUID>', '<EPISODE_UUID>', 340, 1715100000000, 'Bookmark', 1715100000000, 0, 1715100000000, 'Latency vs throughput tradeoff', 'Why optimizing for low latency often means sacrificing batch throughput, and when that tradeoff makes sense.', 1715100000001, 1715100000001, 0,  
  'bookmark'),                                                                                                                                                                                                                                                                                                              
    ('aaaa-0003-test-bookmark-03', '<PODCAST_UUID>', '<EPISODE_UUID>', 600, 1715100000000, 'Bookmark', 1715100000000, 0, 1715100000000, NULL, NULL, NULL, NULL, 0, 'bookmark');  

don't forget to overwrite the placeholders <PODCAST_UUID> and <EPISODE_UUID> with actual values from the previous query!

Then just navigate to the bookmarks list to see the results

design decision: p1778625860157799/1778602895.425639-slack-C05RR9P9RAT

Screenshots or Screencast

designs	actual

Checklist

• If this is a user-facing change, I have added an entry in CHANGELOG.md
• Ensure the linter passes (./gradlew spotlessApply to automatically apply formatting/linting)
• I have considered whether it makes sense to add tests for my changes
• All strings that need to be localized are in modules/services/localization/src/main/res/values/strings.xml
• Any jetpack compose components I added or changed are covered by compose previews
• I have updated (or requested that someone edit) the spreadsheet to reflect any new or changed analytics.

[dangermattic]

	1 Warning
⚠️	PR is not assigned to a milestone.

Generated by 🚫 Danger

[Copilot]

Pull request overview

This PR completes the Android-side wiring for Smart Bookmarks by triggering an enrichment request after creating a bookmark (when the feature flag is enabled) and surfacing AI-generated titles/summaries in the bookmarks list UI.

Changes:

• Trigger bookmark enrichment after new bookmark creation when Feature.SMART_BOOKMARKS is enabled.
• Add Bookmark.displayTitle to choose between user title and AI title for display.
• Update the bookmark row UI to show displayTitle and an optional AI summary line.

Reviewed changes

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

File	Description
modules/services/repositories/src/main/java/au/com/shiftyjelly/pocketcasts/repositories/bookmark/BookmarkHelper.kt	Triggers enrichment when adding a new bookmark via the helper (e.g., headphones/notification path).
modules/services/model/src/main/java/au/com/shiftyjelly/pocketcasts/models/entity/Bookmark.kt	Introduces displayTitle computed property to select user vs AI title.
modules/services/compose/src/main/java/au/com/shiftyjelly/pocketcasts/compose/bookmark/BookmarkRow.kt	Uses displayTitle and renders aiSummary as a secondary line when present.
modules/features/player/src/main/java/au/com/shiftyjelly/pocketcasts/player/view/bookmark/BookmarkViewModel.kt	Triggers enrichment after saving a newly created bookmark from the player bookmark flow.

[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 6 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comments suppressed due to low confidence (1)

modules/services/compose/src/main/java/au/com/shiftyjelly/pocketcasts/compose/bookmark/BookmarkRow.kt:165

• PR description says that when an AI summary is available it should appear as a secondary line beneath the title in the bookmark row, but this component currently only shows the title + created-at timestamp. If the summary line is intended for the list UI, it needs to be rendered here (or the PR description should be updated to match actual behavior).

                TextH40(
                    text = displayTitle,
                    color = colors.bookmarkRow.primaryText,
                    maxLines = 1,
                    lineHeight = 18.sp,
                )

                TextH70(
                    text = createdAtText,
                    color = colors.bookmarkRow.secondaryText,
                    maxLines = 1,
                    modifier = Modifier.padding(top = 4.dp),
                )

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (1)

modules/services/compose/src/main/java/au/com/shiftyjelly/pocketcasts/compose/bookmark/BookmarkRow.kt:166

• PR description says the bookmarks list should show aiSummary as a secondary line beneath the title when available, but BookmarkRow currently only renders the title and created-at timestamp. If this PR is meant to surface summaries in the list, add a conditional secondary text line for bookmark.aiSummary (and ensure layout/spacing accounts for it).

                val displayTitle = bookmark.displayTitle

                TextH40(
                    text = displayTitle,
                    color = colors.bookmarkRow.primaryText,
                    maxLines = 1,
                    lineHeight = 18.sp,
                )

                TextH70(
                    text = createdAtText,
                    color = colors.bookmarkRow.secondaryText,
                    maxLines = 1,
                    modifier = Modifier.padding(top = 4.dp),
                )

[geekygecko]

In future I wouldn't recommend using 'aaaa-0001-test-bookmark-01' as a test bookmark UUID as my server sync is returning a 400 and is stuck because it expects valid UUIDs.