bugfix: make tool calls clickable

[Patrick-Erichsen]

Resolves #7768

---

Summary by cubic

Make tool call status text clickable to open the tool’s output in a virtual file. This lets users quickly view full tool output without expanding the UI.

• Bug Fixes
  • Clicking the tool call message opens a "Tool Output" virtual file with rendered output (when available).
  • Applies to both SimpleToolCallUI and ToolCallDisplay; shows pointer/hover states when clickable.
  • Only clickable when the call is done, canceled, or errored; in-progress calls are not clickable.

[github-actions]

⚠️ PR Title Format

Your PR title doesn't follow the conventional commit format, but this won't block your PR from being merged. We recommend using this format for better project organization.

Expected Format:

<type>[optional scope]: <description>

Examples:

• feat: add changelog generation support
• fix: resolve login redirect issue
• docs: update README with new instructions
• chore: update dependencies

Valid Types:

feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert

This helps with:

• 📝 Automatic changelog generation
• 🚀 Automated semantic versioning
• 📊 Better project history tracking

This is a non-blocking warning - your PR can still be merged without fixing this.

[cubic-dev-ai]

3 issues found across 5 files

Prompt for AI agents (all 3 issues)

Understand the root cause of the following 3 issues and fix them.


<file name="gui/src/pages/gui/ToolCallDiv/SimpleToolCallUI.tsx">

<violation number="1" location="gui/src/pages/gui/ToolCallDiv/SimpleToolCallUI.tsx:88">
Click on status text will also trigger the parent container&#39;s onClick due to event bubbling, leading to both actions firing. Stop propagation in the child click handler to avoid unintended toggling.</violation>
</file>

<file name="gui/src/pages/gui/ToolCallDiv/ToolCallDisplay.tsx">

<violation number="1" location="gui/src/pages/gui/ToolCallDiv/ToolCallDisplay.tsx:25">
Guard should require non-empty output. Checking array truthiness allows empty output to open a blank virtual file; use output?.length instead.</violation>

<violation number="2" location="gui/src/pages/gui/ToolCallDiv/ToolCallDisplay.tsx:46">
Attaching onClick leads to a clickable div without keyboard/role support in ToolCallStatusMessage, causing accessibility issues. Prefer a button or add role, tabIndex, and key handlers.</violation>
</file>

_{React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.}

[RomneyDa]

I think SimpleToolCall already has this functionality (see openContextItem), this approach duplicates for SimpleToolCall