Skip to content

[codex] fix hooks config schema#383

Open
bjulian5 wants to merge 1 commit into
openai:mainfrom
bjulian5:codex/fix-hooks-json-description
Open

[codex] fix hooks config schema#383
bjulian5 wants to merge 1 commit into
openai:mainfrom
bjulian5:codex/fix-hooks-json-description

Conversation

@bjulian5

Copy link
Copy Markdown

Summary

Removes the unsupported top-level description field from plugins/codex/hooks/hooks.json.

Codex's plugin hook config parser expects the top-level object to contain only hooks, so the released plugin currently fails to parse with:

unknown field description, expected hooks

The descriptive metadata already lives in the plugin manifest; the hook config only needs the hook declarations.

Validation

  • jq 'keys' plugins/codex/hooks/hooks.json returns only ["hooks"]
  • npm test passes (86/86)

@bjulian5 bjulian5 requested a review from a team June 18, 2026 22:53
@bjulian5

Copy link
Copy Markdown
Author

@dkundel-openai This PR is ready from my side: it removes the unsupported top-level description key that makes the released plugin hook config fail to parse. Local validation: jq shows only the top-level hooks key, and npm test passes (86/86). GitHub reports the PR as mergeable, but I only have read permission on the upstream repo, so I cannot merge or enable auto-merge.

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