Skip to content

Fixed permission-prompt choice menu: label, hint, empty-text send, and arrow navigation - #106

Merged
oleg68 merged 1 commit into
nbplugins:mainfrom
oleg68:bugfix/permission-prompt-choice-menu
May 5, 2026
Merged

oleg68 merged 1 commit into
nbplugins:mainfrom
oleg68:bugfix/permission-prompt-choice-menu

Conversation

@oleg68

@oleg68 oleg68 commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • "Yes, and tell Claude what to do next" option in bash permission prompts now shows radio label "Yes" with the hint "and tell Claude what to do next" as a text-field placeholder (not as a gray subtitle), and allows clicking Send with an empty text field to select the option without additional text
  • When the PTY cursor is on a text-input option, digit and TYPE: responses now call detectCurrentMenu() to re-read the actual cursor position from the live screen content, fixing a regression where stale model state caused wrong arrow navigation
  • "Type something." option: uses arrow-key navigation instead of raw digit when PTY cursor is on a text-input option

Test plan

  • Trigger a bash permission prompt ("Yes, and tell Claude what to do next" option)
    • Radio button label = "Yes", text field placeholder = "and tell Claude what to do next", no gray subtitle
    • Send with empty field → selects option
    • Send with typed text → sends text as instruction to Claude
  • Planning menu with "Type something." as default option → click a digit option → arrow navigation, digit does NOT go into text field
  • mvn test passes (728 tests)

🤖 Generated with Claude Code

…d arrow navigation

- "Yes, and tell Claude what to do next": radio label is now "Yes", text field
  shows "and tell Claude what to do next" as placeholder hint (not subtitle),
  Send with empty field submits the option as a plain digit
- When PTY cursor is on a text-input option, digit/TYPE: responses use
  detectCurrentMenu() to re-read actual cursor position from screen content
  (model state may be stale if user navigated directly in PTY)
- "Type something." option: arrow-key navigation used instead of raw digit
  when PTY cursor is on a text-input option

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@oleg68
oleg68 merged commit fc12d17 into nbplugins:main May 5, 2026
3 checks passed
@oleg68
oleg68 deleted the bugfix/permission-prompt-choice-menu branch May 5, 2026 06:45
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