Repository navigation
feat(sidebar): add title-only session cards - #23512
rio-evrard wants to merge 1 commit into
Conversation
UX Review (Opus 5.5, fork) — ✅ PASS (a BLOCK with no Trigger and no Outcome does not block)UX-level review of UX-Verdict: PASS cannot evaluate: missing the title-line status markers that title-only cards add. The only title-only screenshot shows idle titles, so nobody has seen how a session that needs approval or is running looks. Evidence gaps
Notes
[UX-REVIEWED] 206f141 Description read[DESCRIPTION-READ] 25df14b8cdce4c34d026706d6be1f5258a108a434103f7bc4101eacb4fa892b8 That sha256 names everything this verdict read: the PR description, plus the 6 evidence file(s) the workflow collected from it and handed this review. It covers that evidence because the media strip replaces every attachment URL with the same placeholder, so an edit swapping one attachment for another would leave the description's captured text identical while what this review judged changed. Recompute it with |
First Principles Review (Opus 5.5, fork) — ✅ PASSPremise-level review of First-Principles-Verdict: PASS Nothing blocks. The title-line status ladder repeats glyph and label choices the row resolver already makes. Notes
What this change shipsInventory (9 items) — 8 justifiedIntent: let people reduce sidebar session cards to their titles with one optional switch that is off by default. ADDITION.
[FIRST-PRINCIPLES-REVIEWED] 206f141 Description read[DESCRIPTION-READ] 0a5f657a6abfa4073069cd9446bd2cb4b62f983823049516e65e39a1da34778e That sha256 names the PR description this verdict read. Recompute it with |
Design Review (Opus 5.5, fork) — ✅ PASSDesign-level review of Design-Verdict: PASS The setting is opt-in and off by default, it takes nothing away, and committed screenshots show this revision. Notes
[DESIGN-REVIEWED] 206f141 Description read[DESCRIPTION-READ] d7095a93a8772a5ddd151db67d6ed5856d0c6bb5f20a3837428935dc8380a7ee That sha256 names everything this verdict read: the PR description, plus the 1 evidence file(s) the workflow collected from it and handed this review. It covers that evidence because the media strip replaces every attachment URL with the same placeholder, so an edit swapping one attachment for another would leave the description's captured text identical while what this review judged changed. Recompute it with |
GPT 6.1 Review (fork) — ✅ no blocking findingsReviewed Review detailsNo findings. |
Opus 5.5 Review (fork) — ✅ no blocking findingsReviewed |
37dd33a to
40a3f81
Compare
40a3f81 to
6b8c6fb
Compare
6b8c6fb to
029c8c9
Compare
029c8c9 to
206f141
Compare
Problem / Motivation
Goal: Let people reduce sidebar session cards to their titles with one optional setting.
The sidebar has separate controls for some card details, but no single title-only mode. People who organize many sessions by title still see repeated agent metadata, timestamps, activity previews, and linked-source chips.
Why it matters
A title-only option makes dense session lists faster to scan and keeps unrelated issue or pull-request references out of screenshots and screen shares.
Not a goal
What changed (motivation → approach → change)
A browser-local Title-only session cards switch now appears under Settings → Chat → Sessions. It defaults off, updates the mounted sidebar immediately, and persists with the existing chat display preferences.
When enabled, the shared session row keeps its title and title-line working cue while omitting the repeated agent/timestamp metadata, secondary activity or message line, and linked source chips. A session that needs the user (an owed approval, an unanswered question, a waiting loop, an interrupted turn, or an unread result) keeps one icon in front of its title, with the same glyph and label the full card uses, so it never looks idle. Work the full card names on its status line (a goal loop, a workflow run, a monitor action) lights the title-line live dot, a goal loop whose last turn was interrupted shows the danger loop icon instead of the dot, and a blocked or budget-stopped monitor keeps its warning icon. A collapsed conductor whose hidden sessions need the user shows the needs-you icon, and one whose hidden sessions are running shows the live dot. The same row component supplies list and board views, so both receive the same presentation without changing stored session data or provider refresh behavior.
The generated documentation registry (
src/kiro_crew/docs/settings-registry.generated.json), command-palette registry (website/src/components/commandPalette/settingsRegistry.gen.ts), supported locale catalogs underwebsite/src/i18n, and focused tests describe and lock in the preference. This is broader than #21925, which hides only ordinary message previews while preserving the remaining card status and metadata.Backwards compatibility
Compatible: the new browser-local preference defaults to
false, so existing installations retain detailed session cards.Reader: website/src/pages/ChatSidebar.tsx:ChatSidebar -- chat -- reads the preference and passes it through the shared row shell.
Reader: website/src/pages/chat-sidebar/SessionRow.tsx:SessionRow -- chat -- hides only rendered card details when the preference is enabled; slot payloads and row actions are unchanged.
Reader: website/src/pages/settings/ChatPanel.tsx:ChatPanel -- settings -- writes the preference through the existing
mc-chat-configpath.Tests
ChatSettings.test.tsxcovers the detailed default, stored opt-in, and repair of invalid stored values.ChatSidebar.sourceLinkChip.test.tsxverifies that titles remain while the activity preview, timestamp, and source chips disappear; that a pending approval and an unread result each keep a labelled title-line marker; that a workflow run and a goal loop light the live dot; that a stalled goal loop shows its interrupted marker and no live dot; that a collapsed conductor with a hidden session needing approval shows the needs-you icon, alongside the live dot when the conductor itself is working; and that full cards get none of these extra cues. These tests fail with theSessionRow.tsxchange reverted.231023.Manual verification
Verified in an isolated local dashboard profile:
The screenshots use synthetic sessions and links.
Screenshots / video
Related Issues
Part of #21239
Related overlap: #21925 provides a narrower preview-only setting and intentionally keeps status indicators and other card details visible.
Checklist
Contribution License Agreement
Contributions are accepted under the project's Apache-2.0 license; see the Licensing section of
CONTRIBUTING.md.