Repository navigation
Conversation
Keep terminal execution independent from output visibility. Preserve existing behavior until users enable the setting. Refs Zoo-Code-Org#605
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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 |
Generate and compare all four VS Code theme snapshots in the pinned Playwright container. Refs Zoo-Code-Org#605
Review statusThanks for contributing. This comment tracks the review sequence and the next action. Current step: Mark the PR ready. Required CI must pass before CodeRabbit starts. Review-state labels are managed by this workflow; do not edit them manually. |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Related GitHub Issue
Related to #605.
The author has posted
Claimingon the issue. Maintainer assignment is pending.Description
Verbose Inline Terminal output can fill the chat. Add an independent Collapse command output by default setting under Terminal.
This supports Enhanced User Experience. Command execution, output collection, and the selected terminal backend stay unchanged.
This PR implements the independent preference requested in #605. It does not add a last-lines preview or a new exit-code summary.
Test Procedure
Passed locally:
git diff --check.Repeat the focused tests:
Manual review:
Local tools: macOS, Node 24.21.0, pnpm 10.8.1. The repository requests Node 22.23.1.
Pre-Submission Checklist
Visual Snapshots
TerminalSettings.visual.tsxcovers the production settings screen in dark, light, high-contrast, and high-contrast-light themes.The four baselines were generated and compared in the repository-pinned Playwright Docker image.
The temporary container path contains Corepack pnpm shims. Lockfile dependencies include Linux native packages for the container run.
No repository dependency files changed.
Documentation Updates
The setting contains translated help text. No separate documentation change is required for this PR.
Additional Notes
Codex helped implement and test this change. Human review and maintainer assignment are required before marking this draft ready.
The PR keeps existing users on the current display behavior until they enable the preference.