buenosair/lvmh
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

web: tool cards show what ran (command summary + edit/write diffs) #1

Closed
buenosair wants to merge 0 commits from feat/tool-summaries-diffs into master
pull from: feat/tool-summaries-diffs
merge into: :master
Conversation 0 Commits - Files Changed -
buenosair commented 2026-09-01 15:18:35 +02:00
Owner
Copy Link
Copy Source

What

  • Collapsed tool summary shows the primary arg: bash → command, read/write/edit → path (one line, ellipsis).
  • edit tool body renders line-level LCS diff per edits[] oldText/newText with -/+ coloring.
  • write tool body renders content as added lines (capped at 400).
  • Other tools unchanged.

Proof

  • bun run test: 14 files / 286 tests passed (10 new).
  • bun run build clean; daemon untouched (go vet + go test pass).
  • Reviewer pass: fixed JSX indent + capped write render, re-verified green.
## What - Collapsed tool summary shows the primary arg: bash → command, read/write/edit → path (one line, ellipsis). - edit tool body renders line-level LCS diff per edits[] oldText/newText with -/+ coloring. - write tool body renders content as added lines (capped at 400). - Other tools unchanged. ## Proof - `bun run test`: 14 files / 286 tests passed (10 new). - `bun run build` clean; daemon untouched (go vet + go test pass). - Reviewer pass: fixed JSX indent + capped write render, re-verified green.
buenosair added 1 commit 2026-09-01 15:18:36 +02:00
web: tool cards show what ran — summary line (bash command / file path) + edit/write line diffs (LCS, +/- coloring); 286 tests green 3b799c69fb
buenosair closed this pull request 2026-09-01 16:15:17 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
buenosair
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: buenosair/lvmh#1