Skip to content

fix(tui): render ask_choice custom input inline#2394

Open
XTLine wants to merge 1 commit into
esengine:v1from
XTLine:fix/choice-custom-inline-input
Open

fix(tui): render ask_choice custom input inline#2394
XTLine wants to merge 1 commit into
esengine:v1from
XTLine:fix/choice-custom-inline-input

Conversation

@XTLine
Copy link
Copy Markdown

@XTLine XTLine commented May 31, 2026

What

Render the ask_choice custom answer path directly in the terminal TUI picker when allowCustom is true.

The picker now includes a selectable custom-answer row and an inline text input. Pressing Enter still submits the currently highlighted row, so typed custom text is only used when the custom row is selected.

Why

Fixes #2382. The protocol already carried allowCustom, but the terminal TUI did not expose the custom-answer path in the picker flow.

How to verify

  • npm test -- tests/choice-confirm-render.test.tsx tests/choice.test.ts tests/pause-gate.test.ts
  • npm run typecheck
  • npm run lint
  • npm run verify

Checklist

  • npm run verify passes locally (lint + typecheck + tests + comment-policy gate)
  • No Co-Authored-By: Claude trailer in commits
  • Comments follow CONTRIBUTING.md (no module-essay headers, no incident history)
  • No edits to CHANGELOG.md — release notes are maintainer-written at release time

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d0ebf50562

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/cli/ui/ChoiceConfirm.tsx Outdated
@XTLine XTLine force-pushed the fix/choice-custom-inline-input branch from d0ebf50 to e0602d4 Compare May 31, 2026 07:24
@XTLine
Copy link
Copy Markdown
Author

XTLine commented May 31, 2026

@codex review

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e0602d49fa

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread src/cli/ui/ChoiceConfirm.tsx Outdated
@esengine esengine added the v1 Legacy TypeScript line (0.x) — v1 branch, maintenance only label May 31, 2026
@XTLine XTLine force-pushed the fix/choice-custom-inline-input branch from e0602d4 to e8a5a01 Compare May 31, 2026 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v1 Legacy TypeScript line (0.x) — v1 branch, maintenance only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ask_choice allowCustom not rendered in TUI picker

2 participants