Skip to content

fix(vscode): improve subagent output and block matrix coverage - #11522

Merged
Drixled merged 9 commits into
mainfrom
fix/vscode-subagent-output
Jul 1, 2026
Merged

fix(vscode): improve subagent output and block matrix coverage#11522
Drixled merged 9 commits into
mainfrom
fix/vscode-subagent-output

Conversation

@Drixled

@Drixled Drixled commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

What changed

  • Polish Explore Agent output to match other tool-call results with the same darker surface, horizontal bleed, and content padding.
  • Fix long subagent results so completed calls open at the first child without clipped text, while preserving the user’s scroll position when an active subagent completes.
  • Keep running subagents following new child activity at the bottom.
  • Extend the Storybook Block Matrix to cover production tool and subagent states and make visual regressions easier to catch.

Before / After

Before After
image Screenshot 2026-06-22 at 19 57 43@2x

Why

Explore Agent results looked inconsistent beside other tool-call output, with extra padding and a missing output surface. Long child-call lists could also open on a partially clipped row. This aligns the visual treatment and gives completed and running subagents the expected scroll behavior without changing shared chat scrolling.

Comment thread packages/kilo-vscode/webview-ui/src/components/chat/TaskToolExpanded.tsx Outdated
@kilo-code-bot

kilo-code-bot Bot commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/labs-tool-call-lab/search-previews-chromium-linux.png
Previous Review Summaries (5 snapshots, latest commit 375ab89)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 375ab89)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-vscode/webview-ui/src/styles/tool-overrides.css

Previous review (commit 9f0c035)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/labs-tool-call-lab/search-previews-chromium-linux.png
  • packages/kilo-vscode/webview-ui/src/styles/tool-overrides.css

Previous review (commit 72df634)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/labs-tool-call-lab/search-previews-chromium-linux.png
  • packages/kilo-vscode/webview-ui/src/styles/tool-overrides.css

Previous review (commit db24fe9)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • packages/kilo-vscode/webview-ui/src/components/chat/TaskToolExpanded.tsx

Previous review (commit 9f1f741)

Status: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
packages/kilo-vscode/webview-ui/src/components/chat/TaskToolExpanded.tsx 105 Completing an already-open subagent unconditionally resets the inner scroll position to the top, which can yank users away from the child call they were inspecting.
Files Reviewed (5 files)
  • .changeset/fix-subagent-output.md
  • packages/kilo-docs/public/img/screenshot-tests/kilo-vscode/visual-regression/labs-tool-call-lab/search-previews-chromium-linux.png
  • packages/kilo-vscode/webview-ui/src/components/chat/TaskToolExpanded.tsx - 1 issue
  • packages/kilo-vscode/webview-ui/src/stories/tool-call-lab.stories.tsx
  • packages/kilo-vscode/webview-ui/src/styles/tool-overrides.css

Reviewed by gpt-5.4-20260305 · Input: 133K · Output: 7.1K · Cached: 229.4K

Review guidance: REVIEW.md from base branch main

@Drixled
Drixled merged commit 5dd37c1 into main Jul 1, 2026
23 checks passed
@Drixled
Drixled deleted the fix/vscode-subagent-output branch July 1, 2026 19:02
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
…utput

fix(vscode): improve subagent output and block matrix coverage
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.

3 participants