Skip to content

refactor(workbench-shell): ♻️ Hide redundant COMMAND block in shell tool detail view - #148

Merged
jorben merged 2 commits into
masterfrom
feat/shell-detail-limit
Apr 28, 2026
Merged

refactor(workbench-shell): ♻️ Hide redundant COMMAND block in shell tool detail view#148
jorben merged 2 commits into
masterfrom
feat/shell-detail-limit

Conversation

@jorben

@jorben jorben commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Hide the COMMAND section in the shell tool's expanded detail view since the command text is already displayed in the tool header line
  • Set showCommandBlock: false in getShellToolPresentation to eliminate duplicate information and save vertical space

Test Plan

  • Expand a shell tool result in the thread — only the OUTPUT section should appear
  • Verify terminal tools (term_write, term_restart) still show their command blocks as before
  • Verify git tools still show their command blocks as before

🤖 Generated with TiyCode

jorben added 2 commits April 28, 2026 13:20
…entation

Added `showCommandBlock: false` to the shell tool presentation configuration
to control visibility of the command block in the runtime thread surface.
@github-actions

Copy link
Copy Markdown

AI Code Review Summary

PR: #148 (refactor(workbench-shell): ♻️ Hide redundant COMMAND block in shell tool detail view)
Preferred language: English

Overall Assessment

No blocking issue was detected in the reviewed diff; keep focused regression testing before merge.

Major Findings by Severity

No major issues identified from the reviewed diff.

Actionable Suggestions

  • Address the highest severity findings first and add targeted tests for changed logic.

Potential Risks

  • Potential hidden risks remain in edge cases not covered by the current diff context.

Test Suggestions

  • Add happy-path + boundary + failure-path tests for touched modules.

File-Level Coverage Notes

  • src/modules/workbench-shell/ui/runtime-thread-surface-tools.tsx: Fine. The addition is safe and aligns with existing CommandOutputToolPresentation usage patterns.

Inline Downgraded Items (processed but not inline)

  • None

Coverage Status

  • Target files: 1
  • Covered files: 1
  • Uncovered files: 0
  • No-patch/binary covered as file-level: 0
  • Findings with unknown confidence (N/A): 0

Uncovered list:

  • None

No-patch covered list:

  • None

Runtime/Budget

  • Rounds used: 1/4
  • Planned batches: 1
  • Executed batches: 1
  • Sub-agent runs: 1
  • Planner calls: 1
  • Reviewer calls: 1
  • Model calls: 2/64
  • Structured-output summary-only degradation: NO

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR review completed.

  • Findings kept: 0
  • Findings with unknown confidence: 0
  • Inline comments attempted: 1
  • Target files: 1
  • Covered files: 1
  • Uncovered files: 0
    See the summary comment for detailed analysis and coverage details.

@@ -344,6 +344,7 @@ function getShellToolPresentation(tool: RuntimeSurfaceToolEntry): CommandOutputT
detailLabel: cwd ? `cwd · ${cwd}` : null,

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review completed for this PR diff. No concrete inline issue was selected after aggregation.

@jorben
jorben merged commit 1f5e966 into master Apr 28, 2026
4 checks passed
@jorben
jorben deleted the feat/shell-detail-limit branch April 28, 2026 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant