Skip to content

Include plugin IDs in hook run summaries#26646

Open
abhinav-oai wants to merge 1 commit into
mainfrom
abhinav/hook-run-plugin-id
Open

Include plugin IDs in hook run summaries#26646
abhinav-oai wants to merge 1 commit into
mainfrom
abhinav/hook-run-plugin-id

Conversation

@abhinav-oai
Copy link
Copy Markdown
Collaborator

@abhinav-oai abhinav-oai commented Jun 5, 2026

Why

Desktop clients only receive the coarse plugin hook source for hook runs. That is not enough to identify which plugin produced a blocked hook, so clients can only open the aggregate plugin settings view.

What

  • Add pluginId to core and app-server v2 hook run summaries.
  • Propagate the plugin identity from discovered handlers into started and completed hook notifications.
  • Regenerate the app-server schemas and document the notification field.

Validation

  • cargo nextest run -p codex-hooks --no-default-features
  • cargo nextest run -p codex-app-server-protocol
  • Verified end to end with a local Codex CLI build and the Codex desktop dev app.

@abhinav-oai abhinav-oai marked this pull request as ready for review June 5, 2026 17:19
@abhinav-oai abhinav-oai requested a review from a team as a code owner June 5, 2026 17:19
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.

2 participants