Skip to content

feat(review): Add expanded review comment editor#1030

Open
leoreisdias wants to merge 1 commit into
backnotprop:mainfrom
leoreisdias:feat/add-review-comment-expand-modal
Open

feat(review): Add expanded review comment editor#1030
leoreisdias wants to merge 1 commit into
backnotprop:mainfrom
leoreisdias:feat/add-review-comment-expand-modal

Conversation

@leoreisdias

@leoreisdias leoreisdias commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Context

Long agent-generated review comments can be awkward to read or edit in the compact inline review annotation toolbar. The compact composer is still useful for quick comments, but larger findings need a first-class expanded writing space.

What Changed

  • Added an expand control to the code-review annotation toolbar.
  • Added a review-specific expanded comment dialog that edits the same existing toolbar comment state.
  • Preserved the compact toolbar as the default surface, including the original w-80 starting width.
  • Kept textarea vertical resize in the compact composer for a lightweight inline improvement.
  • Left suggested-code expansion on the existing suggestion modal path.

GIF

Jul-09-2026 14-54-08

Review Focus

  • Confirm the expanded dialog uses the existing review annotation state correctly for both add and edit flows.
  • Check the collapse vs close behavior: collapse returns to the compact toolbar, close cancels the annotation like the existing compact close button.
  • Verify the modal does not interfere with the existing suggested-code modal or outside-dismiss behavior.

Test Plan

  • env PATH=/Users/leonardo/.codex/worktrees/91ae/plannotator/node_modules/.bun/typescript@5.8.3/node_modules/.bin:$PATH bun run typecheck
  • bun run build:review
  • bun run build:hook
  • Opened a local plannotator review session and exercised the new expanded comment editor path.

@leoreisdias leoreisdias changed the title Add expanded review comment editor feat(review): Add expanded review comment editor Jul 9, 2026
@leoreisdias leoreisdias marked this pull request as ready for review July 9, 2026 18:00
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