Skip to content

docs: mention the send-and-close key - #61

Merged
backnotprop merged 1 commit into
plannotator:mainfrom
mshddev:docs/send-and-close-key
Sep 24, 2026
Merged

backnotprop merged 1 commit into
plannotator:mainfrom
mshddev:docs/send-and-close-key

Conversation

@mshddev

@mshddev mshddev commented Sep 22, 2026

Copy link
Copy Markdown
Contributor

plannotator-tui is adding S to reply review: it sends the review to the agent and closes the window in one key, where today that takes E then q. This updates the one README line that names those keys. E and q are unchanged.

Depends on

What changed

  • README.md: the line under the review key table now says S sends and closes on an agent reply, and that E and q still work.

Nothing else. No action, manifest, binding or version change, because S is typed inside the review window, which this plugin only opens.

Left alone on purpose:

  • docs/windows-full-acceptance.md says q quits the TUI in six places. That stays true, and the checklist tests the ConPTY exit path through q specifically.
  • skills/plannotator-tui/SKILL.md names no keys, and its flow opens a file review, where S does nothing.
  • The manifest descriptions say "send feedback" but name no keys.

Validation

Docs only, so there is nothing to run. Upstream gates S on !is_file_review(), and both folder reviews and Ctrl-clicked files are file reviews, so the line says "agent reply" rather than "review".

🤖 Generated with Claude Code

plannotator-tui adds `S` in reply review: it sends the review to the agent and
closes the window in one key. `E` still sends and `q` still closes. The key
lives in the review window, so no action, manifest or binding changes here.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@backnotprop
backnotprop marked this pull request as ready for review September 24, 2026 23:51
@backnotprop
backnotprop merged commit 5024e83 into plannotator:main Sep 24, 2026
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