-
Notifications
You must be signed in to change notification settings - Fork 3k
feat(tui): Ctrl+O frozen transcript view and unified tool output rendering #5666
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
65 commits
Select commit
Hold shift + click to select a range
6d5fa82
feat(tui): remove tool group borders and collapse completed tool results
a0b018c
fix(tui): gate collapse on compact mode and fix innerWidth calculation
9948cbe
fix(tui): address review feedback on collapse and visual alignment
ea80b74
feat(tui): unify tool output with semantic summaries
89faa9c
test(tui): add buildToolSummary unit tests and fix stale comment
4146279
fix(tui): address audit findings for unified tool output
3275994
fix(tui): address inline review findings
500200c
fix(tui): remove unused compactMode import in HistoryItemDisplay
089ffaf
ci: trigger re-run with updated merge ref
810c713
docs(tui): design — remove global compact mode, add Ctrl+O transcript…
5058715
Merge commit 'refs/tmp/pr5661' into feat/ctrl-o-detail-expand
3169f52
feat(tui): remove global compact mode toggle (on top of #5661 partiti…
dce2562
Merge remote-tracking branch 'origin/main' into feat/ctrl-o-detail-ex…
69b28e2
feat(tui): Ctrl+O opens a frozen alt-screen transcript full-detail view
1e13262
Merge branch 'main' into feat/ctrl-o-detail-expand
wenshao da01d09
fix(tui): repaint normal buffer when transcript closes (no duplicate …
43fece8
Merge remote-tracking branch 'fork/feat/ctrl-o-detail-expand' into fe…
b7b699e
Merge branch 'main' into feat/ctrl-o-detail-expand
wenshao 5fdf29e
Merge branch 'main' into feat/ctrl-o-detail-expand
af07564
docs(tui): rebase ctrl-o design doc to #5661's type-based partition
c16341f
docs(tui): fix factual nits from cross-audit of the ctrl-o design doc
03dc348
Merge branch 'main' into feat/ctrl-o-detail-expand
0e5ccd6
fix(tui): don't let fullDetail be bypassed by compact early returns
42a6aa8
docs(tui): resolve open design decisions from source evidence
631bc64
docs(tui): align design doc status/scope with current PR (audit follo…
488a9fa
docs(tui): scope mouse click-to-expand out as a follow-up
25361c0
feat(tui): isTTY guard for AlternateScreen + transcript shortcut/i18n…
f2b924a
docs(tui): mark isTTY guard + i18n cleanup as implemented in status t…
0e74be3
fix(i18n): add TranscriptView strings to all locales
fe35052
docs(ctrl-o): add before/after transcript capture evidence
372a3ae
docs(ctrl-o): design §4.9 — full tool detail passthrough in transcript
fef75e9
docs(ctrl-o): adopt plan Y for §4.9 and address transcript-detail audit
c97c447
docs(ctrl-o): tighten §4.9 per second audit (no 2nd truncation, neste…
07392fd
Merge remote-tracking branch 'origin/main' into feat/ctrl-o-detail-ex…
f29b3e5
fix(ctrl-o): address PR review findings on transcript view
12000e3
fix(ctrl-o): second review round — web-shell compactMode + anti-deadl…
9204da8
fix(ctrl-o): regenerate vscode settings schema for re-added ui.compac…
ad71f3d
Merge remote-tracking branch 'origin/main' into feat/ctrl-o-detail-ex…
a12d549
chore(ctrl-o): reset MCP/acp-bridge files to main (drop stale merge d…
ff72770
docs(ctrl-o): update compact-mode docs for transcript model; drop orp…
23281e3
feat(ctrl-o)!: §4.9 full tool-detail passthrough in transcript
e30ff19
fix(ctrl-o): address review — repaint race, suppressOnRestore parity,…
7be4007
test(ctrl-o): cover detailedDisplay resume derivation + message.parts…
4dbf68c
fix(ctrl-o): address review — plain-text detail, shared placeholder c…
b1b59cd
Merge origin/main into feat/ctrl-o-detail-expand
e549d2d
Merge origin/main into feat/ctrl-o-detail-expand
b5096db
fix(tui): guard transcript non-TTY output + clear detailedDisplay on …
c28f29e
test(tui): report a TTY stdout in ScrollableList mouse-scroll tests
255458b
Merge origin/main into feat/ctrl-o-detail-expand
69040bb
fix(tui): address Ctrl+O transcript review — q-guard, callback churn,…
7184565
fix(tui): escape ANSI in transcript detailedDisplay + gate its extrac…
1568db8
fix(tui): gate resume-path detailedDisplay on isCollapsibleTool (matc…
151347d
fix(tui): strip bare C0 control bytes from transcript detailedDisplay…
3b719df
test(tui): memoize HistoryItemDisplay, add ErrorBoundary tests + TAB/…
94e4ba2
refactor(tui): review cleanups — gate sanitize memo, drop dead code, …
d3e7d18
test(tui): remove orphaned no-op CompactModeProvider stubs
b05db46
fix(tui): strip bidi overrides, sanitize error fallbacks, share filters
9e9055d
fix(tui): harden compaction guard to always clear detailedDisplay
2ed723d
fix(core): sanitize mime/uri in getToolResponseDisplayText media plac…
8b492e9
Merge origin/main into feat/ctrl-o-detail-expand
3d93b23
test(tui): report a TTY stdout in BaseSelectionList mouse integration…
85fc9be
docs(core): fix JSDoc placement + note ErrorBoundary fallback is un-t…
d3b39ba
fix(tui): share terminal-sanitize pipeline; guard AlternateScreen writes
67635c6
Merge origin/main into feat/ctrl-o-detail-expand
052047b
Merge origin/main into feat/ctrl-o-detail-expand
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Binary file added
BIN
+349 KB
docs/design/ctrl-o-detail-expand/assets/ctrl-o-transcript-expanded.png
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.