feat(ui): Channel List — inline Recent discussions section + See all link

[ilmoniemi]

User Story

As a Pyrycode mobile user on the home screen, I want recent discussions surfaced inline below the channels — section header, divider, preview cards, and a "See all discussions" affordance — instead of a single bottom-of-screen pill, so I can glance at recent scratch work without navigating away.

Context

Phase 1's #46 shipped a RecentDiscussionsPill that renders the count at the bottom of ChannelListScreen and navigates to DiscussionListScreen. The canonical Figma treatment (node 15:8) replaces the pill with an inline "Recent discussions" section: divider (Figma 15:85), section header (Figma 15:87), three preview cards (Figma 15:88/15:92/15:96) showing title + 2-line body + workspace · time meta, and a tappable "See all discussions (N) →" affordance (Figma 15:102) that navigates onward.

Acceptance Criteria

• ChannelListScreen renders, below the channels list, in order: a thin horizontal divider (Figma 15:85), a "Recent discussions" section header (Figma 15:87), up to three discussion preview rows for the most-recent discussions, and a "See all discussions (N)" link with a trailing forward-arrow icon (Figma 15:102/15:103)
• Each preview row shows a single-line title, a 2-line (max) ellipsised body, and a <workspace> · <relative-time> meta line — typography/spacing/colours per Figma nodes 15:89–15:99
• Tapping "See all discussions (N)" emits the existing ChannelListEvent.RecentDiscussionsTapped — do not introduce a parallel event
• ChannelListUiState.Loaded and Empty expose the top-3 most-recent discussions alongside the existing count; ChannelListViewModel derives this list from the existing ConversationFilter.Discussions flow
• When the recent-discussions list is empty, the entire section (divider + header + rows + link) is hidden — no orphan header
• Quality bar: RecentDiscussionsPill removed from ChannelListScreen (file deleted if no call sites remain); zero hardcoded colour or dimension literals (everything via MaterialTheme tokens); light + dark previews for ChannelListScreen cover loaded-with-discussions and no-discussions states, and the new preview row composable ships with its own preview

Technical Notes

• Existing files: app/src/main/java/de/pyryco/mobile/ui/conversations/list/ChannelListScreen.kt, app/src/main/java/de/pyryco/mobile/ui/conversations/list/ChannelListViewModel.kt, app/src/main/java/de/pyryco/mobile/ui/conversations/components/RecentDiscussionsPill.kt
• New file expected: DiscussionPreviewRow.kt under ui/conversations/components/ — architect to finalise naming
• Depends on the channels-section/avatar slice landing first (sectioned LazyColumn shape, "Channels" header above)
• The message-content previews shown in the Figma mock are aspirational: the Conversation model carries no last-message field, and ConversationRepository.observeMessages is a paginated per-conversation stream that's expensive to call per row. Render placeholder body text (e.g. conversation name, or a synthesised string from the fake repo) and defer real message-content previews to a follow-up ticket once message access is wired

Size Estimate

S

Figma

https://www.figma.com/design/g2HIq2UyPhslEoHRokQmHG?node-id=15-8

Split from #63.

[ilmoniemi]

🤖 Product Owner — creates structured issues

po agent has completed work on this ticket.

Agent output (click to expand)

The work here is a single quick refinement pass — no todo list needed.

Refined #69 in place. Body changes:

- **Consolidated AC 6–8 into one Quality bar line** matching the sibling #68 precedent (pill removal + no-hardcoded-literals + light/dark previews). AC count: 8 → 6.
- Minor wording tightening on AC 3 ("do not introduce a parallel event" moved inline) and AC 5 ("no orphan header").
- Preserved the human framing throughout: User Story, Context, Technical Notes, Figma node references, "Split from #63" attribution all untouched.

No size change (S correct — ~90 net lines, 3 substantive files, no split-pattern hits). No `security-sensitive` label (pure UI, no auth/network/secrets). Did not add `ready:po` — dispatcher handles that.

Ready for human review. Move to the next column when approved.

[ilmoniemi]

🤖 System Architect — defines Go interfaces, data flows, concurrency patterns

architect agent has completed work on this ticket.

Agent output (click to expand)

Spec committed.

**Summary.** Sized S after edit-fan-out check (RecentDiscussionsPill has 2 same-file call sites, 0 external callers, 2 dedicated strings). No branch-overlap conflicts (`git branch -r` shows no in-flight `feature/*` branches). Spec at `docs/specs/architecture/69-channel-list-recent-discussions-section.md`.

Key design decisions:
- `Loaded`/`Empty` gain `recentDiscussions: List<Conversation>` (top-3, sliced via `.take(3)` against `FakeConversationRepository`'s upstream `lastUsedAt` desc ordering — no new sort contract).
- New `DiscussionPreviewRow.kt` (title + 2-line localised placeholder body + `workspace · time` meta with monospace span for the workspace fragment).
- New `RelativeTime.kt` extracted from `ConversationRow.kt` — only refactor allowed (avoids duplication; behaviour-preserving).
- Recent-discussions section lives inside the `Loaded` `LazyColumn` as a trailing `item` so it scrolls with channels; rendered above the centred empty text in `Empty`.
- "See all discussions (N)" uses `Icons.AutoMirrored.Filled.ArrowForward` (already on classpath via core).
- 2 strings deleted, 6 added; pill file fully removed.
- Test plan: surgical positional-arg updates to 6 existing VM tests, plus 2 new tests for `.take(3)` cap and upstream-ordering preservation.

Branch `feature/69` is one commit ahead; dispatcher will push.

Ready for human review. Move to the next column when approved.