fix(tui): consume Enter in dialog useKeyboard handlers

[CasualDeveloper]

Issue for this PR

Closes #23389 and #23803

Type of change

• Bug fix
• New feature
• Refactor / code improvement
• Documentation

What does this PR do?

Seven TUI dialogs handled the return key in their useKeyboard callback but never called evt.preventDefault() or evt.stopPropagation(). opentui dispatches keypresses to global useKeyboard listeners first, then to the focused renderable; without consuming the event in the dialog's handler, Enter leaks past the dialog into the focused renderable underneath (typically the main prompt textarea).

With a custom input_newline keybind that includes bare return, confirming the dialog inserts a newline in the main prompt. With default keybinds it manifests as a silent empty-submit or unintended action — e.g. for #23803, pressing Enter on the update-available dialog also submits whatever was already typed in the prompt.

The fix adds the two consume calls to the return branch in:

• ui/dialog-prompt.tsx (session rename, OAuth code, API key, plugin-exposed DialogPrompt)
• ui/dialog-export-options.tsx
• ui/dialog-confirm.tsx (used by the update-available dialog and other generic confirms — Pressing enter on update notice dialog causes prompt to be submitted #23803)
• ui/dialog-alert.tsx
• ui/dialog-help.tsx
• component/dialog-go-upsell.tsx
• component/dialog-session-delete-failed.tsx

Mirrors the consume pattern already used in ui/dialog-select.tsx:205-213 and component/dialog-workspace-unavailable.tsx:27-31. Total diff: +14 LOC across 7 files, no change to dialog action behaviour.

How did you verify your code works?

• bun turbo typecheck from repo root: 13/13 packages pass.
• Reproduced the original DialogPrompt leak locally with "input_newline": "return,shift+return" in tui.json (recordings on linked issue fix(tui): Enter key in DialogPrompt leaks to background prompt (#23390) #23389): pre-fix, confirming session-rename inserted a newline in the main prompt; post-fix, main prompt untouched, dialog action unchanged.
• Audited every TUI file using useKeyboard to confirm no other dialog handlers were missing the consume calls; the 7 above are the complete set.

Screenshots / recordings

Before/after recordings of the underlying bug class (demonstrated via DialogPrompt) are attached to issue #23389. The same root cause and fix shape applies to all 7 dialogs.

Checklist

• I have tested my changes locally
• I have not included unrelated changes in this PR

AI assistance: OpenCode + anthropic/claude-opus-4-7 drafted the analysis and audit; human operator directed the scope and reviewed the fix.

[Copilot]

Pull request overview

Fixes Enter/Return keypress leakage from modal dialogs into the underlying main prompt by consuming the keyboard event in dialog-level handlers.

Changes:

• Consume return in DialogPrompt keyboard handler via evt.preventDefault() + evt.stopPropagation().
• Consume return in DialogExportOptions keyboard handler via evt.preventDefault() + evt.stopPropagation().
• Aligns behavior with the existing consumption pattern used in dialog-select.tsx.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/opencode/src/cli/cmd/tui/ui/dialog-prompt.tsx	Prevents Return from propagating past DialogPrompt confirm and affecting the background prompt.
packages/opencode/src/cli/cmd/tui/ui/dialog-export-options.tsx	Prevents Return from propagating past export confirmation and affecting the background prompt.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[github-actions]

Thanks for updating your PR! It now meets our contributing guidelines. 👍

[github-actions]

Thanks for updating your PR! It now meets our contributing guidelines. 👍

[CasualDeveloper]

yay, ready for maintainer review! :)

[iCrazeiOS]

I found this as I was about to PR the exact same fix. Can confirm this works

[rekram1-node]

/review

[github-actions]

lgtm