Skip to content

Expose prompt hooks to clients - #26268

Closed
abhinav-oai wants to merge 1 commit into
abhinav/prompt-hooksfrom
abhinav/prompt-hooks-clients
Closed

abhinav-oai wants to merge 1 commit into
abhinav/prompt-hooksfrom
abhinav/prompt-hooks-clients

Conversation

@abhinav-oai

@abhinav-oai abhinav-oai commented Jun 3, 2026 •

Copy link
Copy Markdown
Contributor

Why

Clients need enough prompt-hook metadata to review configured handlers and explain their behavior before users trust or enable them.

What

  • Expose prompt hook definitions, models, and continueOnBlock behavior through config requirements and hooks/list.
  • Document the prompt-hook metadata returned by app-server.
  • Render prompt definitions, model selection, and block behavior in the TUI hook browser.
  • Regenerate app-server schema fixtures and add app-server and TUI snapshot coverage.

Stack

  1. Add prompt-handler execution to codex-hooks #26267
  2. Enable configured prompt hooks in Codex #24634
  3. (this PR) Expose prompt hooks to clients #26268

@abhinav-oai
abhinav-oai force-pushed the abhinav/prompt-hooks branch from 4bc4f2a to 627631d Compare June 4, 2026 00:30
@abhinav-oai
abhinav-oai requested a review from a team as a June 4, 2026 00:30
@abhinav-oai
abhinav-oai force-pushed the abhinav/prompt-hooks-clients branch from bc57d5a to 360aff8 Compare June 4, 2026 00:30
@abhinav-oai
abhinav-oai force-pushed the abhinav/prompt-hooks branch from 627631d to c621ca2 Compare June 4, 2026 00:52
@abhinav-oai
abhinav-oai force-pushed the abhinav/prompt-hooks-clients branch from 360aff8 to d8f4671 Compare June 4, 2026 00:52
@abhinav-oai
abhinav-oai removed the request for review from a team June 4, 2026 01:51
@github-actions

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 14 days. If you plan to continue working on it, feel free to reopen or open a new PR.

@abhinav-oai

Copy link
Copy Markdown
Contributor Author

Moved to the internal stack: https://github.com/openai/codex-internal/pull/1280

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