Skip to content

feat(ui): show line summaries for multi-file patches - #12725

Merged
marius-kilocode merged 2 commits into
mainfrom
summarize-tool-call-line-changes
Aug 3, 2026
Merged

feat(ui): show line summaries for multi-file patches#12725
marius-kilocode merged 2 commits into
mainfrom
summarize-tool-call-line-changes

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Multi-file patch tool calls currently show only the number of affected files in their collapsed header, while the individual file rows expose their own line counts. This makes the overall size of a patch harder to assess without expanding the tool call.

The header now reuses the per-file diff metadata to show aggregate additions and deletions for the current tool call, while preserving the existing single-file presentation and per-file details. The summary is omitted when there are no line changes.

Before:

Multi-file patch header before aggregate line summary

After:

Multi-file patch header with aggregate line summary

@marius-kilocode
marius-kilocode enabled auto-merge (squash) July 31, 2026 10:11
Comment thread packages/kilo-ui/src/components/message-part.tsx Outdated
@kilo-code-bot

kilo-code-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 file)
  • packages/kilo-ui/src/components/message-part.tsx

The follow-up commit resolves the earlier suggestion: the aggregate DiffChanges now renders through ToolChanges with animate={reveal()}, so it fades in alongside the sibling ToolText. Passing the slot through keeps data-slot="message-part-tool-changes" on the same single wrapper element, so the inline-flex layout and the basic-tool-tool-subtitle + ...::before separator dot still apply despite the span -> div swap, and data-slot is omitted for the other ToolChanges call sites where the prop is undefined.

Previous Review Summary (commit bbf4dff)

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

Previous review (commit bbf4dff)

Status: 1 Issue Found | Recommendation: Optional follow-up, safe to merge

Overview

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

SUGGESTION

File Line Issue
packages/kilo-ui/src/components/message-part.tsx 2772 Aggregate DiffChanges is rendered without the reveal fade used by the adjacent ToolText and by the single-file ToolChanges path
Files Reviewed (3 files)
  • packages/kilo-ui/src/components/message-part.tsx - 1 issue
  • packages/kilo-ui/src/components/basic-tool.css - no issues (separator/flex-shrink handling matches the existing vscode-theme dot pattern)
  • .changeset/bright-files-count.md - no issues

Notes: the some(...) guard correctly suppresses the stray separator dot when nothing changed, and DiffChanges already tolerates missing counts. Existing multi-file apply_patch story fixtures in tool-call-lab.stories.tsx cover the new header, so CI visual baselines should capture it without a new story.

Fix these issues in Kilo Cloud


Reviewed by claude-opus-5 · Input: 36 · Output: 5.3K · Cached: 715.8K

Review guidance: REVIEW.md from base branch main

@marius-kilocode
marius-kilocode disabled auto-merge July 31, 2026 10:20
@marius-kilocode
marius-kilocode enabled auto-merge (squash) July 31, 2026 10:27
@marius-kilocode
marius-kilocode merged commit f239f36 into main Aug 3, 2026
30 checks passed
@marius-kilocode
marius-kilocode deleted the summarize-tool-call-line-changes branch August 3, 2026 10:23
t7tran pushed a commit to t7tran/kilocode that referenced this pull request Aug 14, 2026
* feat(ui): show line summaries for multi-file patches

* fix(ui): animate multi-file patch summaries
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.

2 participants