Skip to content

Add quick comment to diff viewer (Phase 2)#690

Merged
selfcontained merged 3 commits into
mainfrom
agt_0d2445e454da/agent-e454da
Jun 26, 2026
Merged

Add quick comment to diff viewer (Phase 2)#690
selfcontained merged 3 commits into
mainfrom
agt_0d2445e454da/agent-e454da

Conversation

@selfcontained

Copy link
Copy Markdown
Owner

Summary

  • Add line-level commenting in the Changes tab diff viewer that injects formatted messages into the agent's tmux session
  • Backend: POST /api/v1/agents/:id/diff/comment endpoint that extracts line content from unified diffs and injects via sendAgentPrompt
  • Frontend: gutter click to select lines (with shift-click for ranges), floating comment bubble in the left gutter, inline comment form with "Chat Now" submit
  • Collapsible file tree panel with smooth width animation and sidebar-style toggle icons
  • Click-to-deselect: clicking a selected line (or any line in a multi-line block) clears the selection

Test plan

  • Type check passes (pnpm run check)
  • Web build passes (pnpm run finalize:web)
  • E2E tests pass (171/171)
  • Verified comment injection flow end-to-end on live dev instance with real agent session
  • Verified file tree toggle open/close with smooth animation
  • Verified line selection, shift-click range selection, and click-to-deselect

🤖 Generated with Claude Code

selfcontained and others added 3 commits June 25, 2026 21:41
…ck-to-deselect

- Decouple line selection from comment form — show a floating comment
  bubble in the left gutter instead of immediately opening the form
- Add collapsible file tree panel with smooth width animation and
  sidebar-style toggle icons (PanelLeftClose/PanelLeftOpen)
- Click an already-selected line (or any line in a multi-line block)
  to deselect
- Change submit button label from "Send" to "Chat Now"

Co-Authored-By: Claude Opus 4.6 <[email protected]>
Cap comment length at 10k chars and line range at 500 lines to
prevent oversized tmux pastes.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@selfcontained
selfcontained force-pushed the agt_0d2445e454da/agent-e454da branch from e584d90 to b3516ca Compare June 26, 2026 03:46
@selfcontained
selfcontained merged commit 4bd86ae into main Jun 26, 2026
1 of 2 checks passed
@selfcontained
selfcontained deleted the agt_0d2445e454da/agent-e454da branch June 26, 2026 03:54
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.

1 participant