Add a Compact transcript setting - #202
Conversation
Settings → General → Compact transcript keeps a work group folded behind the summary line that already titles it, instead of opening it while the turn runs. A busy turn then reads as one row rather than a scrolling wall of steps. The auto-open was the only thing removed: clicking the line still opens the group, an override still sticks, and a step waiting on an approval still opens itself so nothing is approved unseen. A lone call with no headline never formed a group, so it is unaffected. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (7)
Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review. 📝 WalkthroughWalkthroughThe change adds a persisted compact-transcript preference, a General settings toggle, and transcript behavior that folds active work while keeping approval-waiting groups open. ChangesCompact transcript
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Change: Feature Sequence Diagram(s)sequenceDiagram
participant SettingsView
participant appearance
participant useTranscriptCompact
participant AgentTranscript
SettingsView->>appearance: saveTranscriptCompact(value)
appearance->>useTranscriptCompact: dispatch preference change
useTranscriptCompact->>AgentTranscript: update compact state
AgentTranscript->>AgentTranscript: keep active phase collapsed unless approval is waiting
Suggested reviewers: Merge Risk: ⚪ Minimal · up to The compact transcript toggle preserves approval expansion and manual control as intended. No actionable merge risk remains. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
Full details: Docstring CoverageExplanation Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 6 files. (1 skipped: 1 unsupported.)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
What changed
A Compact transcript toggle in Settings → General. When it is on, a work group stays folded behind the summary line that already titles it instead of opening while the turn runs.
Why
A turn that runs a dozen commands pushes the answer off screen, and the steps are rarely what you are reading. The line already says what is happening and how many steps it has taken, so the group opening on top of that is mostly noise. Off by default; nothing changes unless you ask for it.
The auto-open is the only thing the setting removes. Clicking the line still opens the group, a manual override still sticks, and a step waiting on an approval still opens itself, so nothing is ever approved unseen. A lone call with no headline never formed a group, so it is unaffected.
Follows the shape of the existing Anchor prompts to top setting: flag in
appearance.ts, auseTranscript*hook, oneRowinGeneralPage.UI
Same transcript, with the setting off and on:
Running 9 commands+ 10 rowsRunning 9 commandsChecklist
npm run checkNote on the first box:
tsc --noEmitis clean and the tests covering this change pass. Four test files (ModelPicker,SkillsPage,useInboxUnseen,linkedSessionSeen) fail for me on a cleanmainas well, withlocalStorageundefined under happy-dom on Node 26 — unrelated to this change.🤖 Generated with Claude Code
Summary by CodeRabbit
New Features
Bug Fixes