Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add abbreviated lines added/removed to task header #5044

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jrf0110 merged 1 commit into from
Jan 19, 2026
Merged

Add abbreviated lines added/removed to task header #5044

jrf0110 merged 1 commit into from
Jan 19, 2026

Conversation

@jrf0110
Copy link
Contributor

@jrf0110 jrf0110 commented Jan 14, 2026 •
edited
Loading

image image

@changeset-bot
Copy link

changeset-bot bot commented Jan 14, 2026 •
edited
Loading

🦋 Changeset detected

Latest commit: 2ee6e82

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@kiloconnect
Copy link
Contributor

kiloconnect bot commented Jan 14, 2026 •
edited
Loading

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR adds a GitHub-style diff stats display to the task header, showing lines added/removed in real-time. The implementation is well-structured and follows existing patterns in the codebase.

Key Changes:

  • New DiffStatsDisplay component with proportional colored boxes
  • New useTaskDiffStats hook to aggregate diff stats from tool messages
  • Settings toggle to show/hide diff stats (showDiffStats)
  • Full i18n support across all 24 locales
  • Comprehensive test coverage for both component and hook
Files Reviewed (47 files)

Core Changes:

  • .changeset/task-diff-stats-display.md - Changeset
  • packages/types/src/global-settings.ts - Type definition
  • src/core/webview/webviewMessageHandler.ts - Message handler
  • src/shared/ExtensionMessage.ts - Extension state type
  • src/shared/WebviewMessage.ts - Webview message type

UI Components:

  • webview-ui/src/components/kilocode/DiffStatsDisplay.tsx - New component
  • webview-ui/src/components/kilocode/KiloTaskHeader.tsx - Integration
  • webview-ui/src/components/settings/DisplaySettings.tsx - Settings UI
  • webview-ui/src/components/settings/SettingsView.tsx - Settings integration

Hooks & State:

  • webview-ui/src/components/ui/hooks/kilocode/useTaskDiffStats.ts - New hook
  • webview-ui/src/context/ExtensionStateContext.tsx - State context

Tests:

  • webview-ui/src/components/kilocode/__tests__/DiffStatsDisplay.spec.tsx
  • webview-ui/src/components/ui/hooks/__tests__/useTaskDiffStats.spec.ts

i18n (24 locales):

  • webview-ui/src/i18n/locales/*/chat.json - Added "changes" key
  • webview-ui/src/i18n/locales/*/settings.json - Added showDiffStats settings

Notes on Existing Comments

The existing comment on line 71 of DiffStatsDisplay.spec.tsx appears to be outdated - the test correctly expects 0 boxes by default, which matches the component's maxBoxes = 0 default value.

@jrf0110 jrf0110 requested a review from kevinvandijk January 14, 2026 21:40
jrf0110
jrf0110 commented Jan 14, 2026
View reviewed changes
webview-ui/src/components/kilocode/DiffStatsDisplay.tsx
*
* Example: +8 −4 with 4 green boxes, 1 red box, and 0 empty boxes
*/
const DiffStatsDisplay = ({ added, removed, maxBoxes = 0, className }: DiffStatsDisplayProps) => {
Copy link
Contributor Author

@jrf0110 jrf0110 Jan 14, 2026

Choose a reason for hiding this comment

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

Initially we had a UI that showed the diff like a github PR lines added/removed display with the boxes. We decided to not show the boxes but I went ahead and kept the functionality in here in case we want it in the future. maxBoxes = 0 hides the boxes

@jrf0110 jrf0110 enabled auto-merge January 16, 2026 16:27
@jrf0110 jrf0110 merged commit e4ced00 into main Jan 19, 2026
12 checks passed
@jrf0110 jrf0110 deleted the 4122-line-changes branch January 19, 2026 17:52
This was referenced Jan 19, 2026
Merged
Open
Open
Open
Open
Open
Closed
Open
Closed
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kiloconnect kiloconnect[bot] kiloconnect[bot] left review comments

@kevinvandijk kevinvandijk kevinvandijk approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jrf0110 @kevinvandijk