Skip to content

Conversation

@charley-oai
Copy link
Collaborator

@charley-oai charley-oai commented Jan 27, 2026

Summary

Polishes the request_user_input TUI overlay

Question 1 (unanswered)
Screenshot 2026-01-27 at 1 30 09 PM

Tab to add notes
Screenshot 2026-01-27 at 1 30 25 PM

Question 2 (unanswered)
Screenshot 2026-01-27 at 1 30 55 PM

Ctrl+p or h to go back to q1 (answered)
Screenshot 2026-01-27 at 1 31 27 PM

Unanswered freeform
Screenshot 2026-01-27 at 1 31 42 PM

Key changes

  • Footer tips wrap at tip boundaries (no truncation mid‑tip); footer height scales to wrapped tips.
  • Keep tooltip text as Esc: interrupt in all states.
  • Make the full Tab: add notes tip cyan/bold when applicable; hide notes UI by default.
  • Notes toggling/backspace:
    • Tab opens notes when an option is selected; Tab again clears notes and hides the notes UI.
    • Backspace in options clears the current selection.
    • Backspace in empty notes closes notes and returns to options.
  • Selection/answering behavior:
    • Option questions highlight a default option but are not answered until Enter.
    • Enter no longer auto‑selects when there’s no selection (prevents accidental answers).
    • Notes submission can commit the selected option when present.
    • Freeform questions require Enter with non‑empty text to mark answered; drafts are not submitted unless committed.
  • Unanswered cues:
    • Skipped option questions count as unanswered.
    • Unanswered question titles are highlighted for visibility.
  • Typing/navigation in options:
    • Typing no longer opens notes; notes are Tab‑only.
    • j/k move option selection; h/l switch questions (Ctrl+n/Ctrl+p still work).

Tests

  • Added unit coverage for:
    • tip‑level wrapping
    • focus reset when switching questions with existing drafts
    • backspace clearing selection
    • backspace closing empty notes
    • typing in options does not open notes
    • freeform draft submission gating
    • h/l question navigation in options
  • Updated snapshots, including narrow footer wrap.

Why

These changes make the ask‑user‑question overlay:

  • safer (no silent auto‑selection or accidental freeform submission),
  • clearer (tips wrap cleanly and unanswered states stand out),
  • more ergonomic (Tab explicitly controls notes; backspace acts like undo/close).

Codex author

codex fork 019bfc3c-2c42-7982-9119-fee8b9315c2f

@charley-oai charley-oai force-pushed the ask-user-question-ui-footer-improvements branch from 2a24f03 to c05ed32 Compare January 27, 2026 00:28
@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: c05ed321ec

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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

KeyCode::Enter => {
self.select_current_option();
self.go_next_or_submit();

P2 Badge Allow submitting without auto-selecting option

With the new default of “no selection,” pressing Enter in the options list always calls select_current_option, which clamps a None selection to index 0. That means a user who leaves an options question unanswered (especially on the last question where Enter submits) will unintentionally submit the first option instead of an empty answer, despite the UI showing “No option selected.” Consider skipping select_current_option when Enter is used to submit and no option is currently selected.

ℹ️ 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".

@charley-oai charley-oai force-pushed the ask-user-question-ui-footer-improvements branch from 0de7256 to f86aacd Compare January 27, 2026 01:36
@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: f86aacdd0f

ℹ️ 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".

@charley-oai charley-oai force-pushed the ask-user-question-ui-footer-improvements branch from c238b19 to d554efa Compare January 27, 2026 06:57
@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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

KeyCode::Enter => {
self.select_current_option();
self.go_next_or_submit();

P2 Badge Avoid auto-selecting on Enter when no option chosen

With the new “no selection by default” behavior, hitting Enter in the options focus path still calls select_current_option(), which clamps to index 0 and submits. In a single-question options prompt (no Ctrl+N path), a user cannot submit an unanswered response despite the UI showing “No option selected” and the new empty-answer behavior. This reintroduces the accidental-selection problem and makes skipping impossible in that scenario. Consider only submitting the current selection on Enter (if one exists) or requiring an explicit selection action first.

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

@chatgpt-codex-connector
Copy link
Contributor

Codex Review: Didn't find any major issues. Another round soon, please!

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: 0179dfb4a4

ℹ️ 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".

@charley-oai charley-oai force-pushed the ask-user-question-ui-footer-improvements branch from 0179dfb to ef13a30 Compare January 27, 2026 17:27
@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: ef13a30c85

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: a2fd6a61c7

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: 9bd6abaf72

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: 1d8fd29b18

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

Copy link
Contributor

@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: e5c4303885

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

@charley-oai charley-oai marked this pull request as ready for review January 27, 2026 22:41
Copy link
Contributor

@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: 401917a16f

ℹ️ 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".

@charley-oai
Copy link
Collaborator Author

@codex review this

@charley-oai charley-oai enabled auto-merge (squash) January 27, 2026 22:47
Copy link
Contributor

@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: 8c8cd47a51

ℹ️ 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".

@charley-oai charley-oai disabled auto-merge January 27, 2026 22:57
@charley-oai charley-oai merged commit 19d8f71 into main Jan 27, 2026
51 of 58 checks passed
@charley-oai charley-oai deleted the ask-user-question-ui-footer-improvements branch January 27, 2026 22:57
@github-actions github-actions bot locked and limited conversation to collaborators Jan 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants