Skip to content

fix(terminal): stop duplicating agent tool calls above the output - #7433

Merged
waleedlatif1 merged 1 commit into
stagingfrom
fix/agent-tools-output-display
Sep 3, 2026
Merged

fix(terminal): stop duplicating agent tool calls above the output#7433
waleedlatif1 merged 1 commit into
stagingfrom
fix/agent-tools-output-display

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • Drop the AgentStreamToolCallsChrome header from the workflow terminal's output panel — the collapsible "Tools" list rendered above the output while the same calls stayed inline in the agent block's toolCalls field, so every streamed run showed them twice
  • Tool calls read inline in the structured output again, the way they did before the agent-stream work landed
  • The thinking chrome is unchanged, and the tool chrome itself stays for the deployed chat surface, which is its only other consumer

Type of Change

  • Bug fix

Testing

Tested manually. bun run lint, bun run check:audits (45 audits), block-registry check, and bun run type-check all pass.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

🤖 Generated with Claude Code

https://claude.ai/code/session_01CimY1cwvWpQs3RY5MXtdE3

The agent-stream tool chrome rendered a collapsible "Tools" list above the
terminal output panel, while the same calls remained inline in the block's
toolCalls output field. Drop the header so tool calls read inline again.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CimY1cwvWpQs3RY5MXtdE3
@vercel

vercel Bot commented Sep 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
docs Skipped Skipped Sep 3, 2026 3:55pm UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes the workflow terminal’s separate tool-call chrome so agent tool calls appear only in structured output, while preserving the thinking chrome.

  • Removes the unused AgentStreamToolCallsChrome import from the output panel.
  • Simplifies the terminal header condition to render only agent thinking content.

Confidence Score: 5/5

The PR appears safe to merge, with no concrete blocking or independently actionable issues established.

The change is narrowly scoped to removing a redundant terminal presentation of tool calls, and the remaining evidence does not establish that a reachable output state loses required tool-call information.

Important Files Changed

Filename Overview
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/terminal/components/output-panel/output-panel.tsx Removes the duplicate tool-call header renderer while leaving the terminal’s thinking display and structured output paths intact.

Reviews (1): Last reviewed commit: "fix(terminal): stop duplicating agent to..." | Re-trigger Greptile

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 7baba0e into staging Sep 3, 2026
29 checks passed
@waleedlatif1
waleedlatif1 deleted the fix/agent-tools-output-display branch September 3, 2026 15:59
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