Skip to content

hooks: resolve UserInstructions provider precedence#30140

Draft
anp-oai wants to merge 1 commit into
codex/user-instructions-configfrom
codex/user-instructions-precedence
Draft

hooks: resolve UserInstructions provider precedence#30140
anp-oai wants to merge 1 commit into
codex/user-instructions-configfrom
codex/user-instructions-precedence

Conversation

@anp-oai

@anp-oai anp-oai commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

why

Multiple discovered providers need deterministic precedence, and managed requirements must not silently fall back when their selected provider is unavailable.

what

  • selects the highest-priority active UserInstructions provider
  • gives managed requirements and other managed sources precedence over ordinary config and plugins
  • emits startup warnings for ignored providers
  • fails closed when a managed requirements provider cannot run

manual validation

  • just test -p codex-hooks discovery
  • 14 tests passed

stack, merge in order

  1. hooks: define UserInstructions execution contract #30138 hooks: define UserInstructions execution contract
  2. config: add singleton UserInstructions declarations #30139 config: add singleton UserInstructions declarations
  3. hooks: resolve UserInstructions provider precedence #30140 hooks: resolve UserInstructions provider precedence
  4. core: load hook-backed user instructions #30141 core: load hook-backed user instructions

@anp-oai anp-oai force-pushed the codex/user-instructions-precedence branch from 7b84d42 to 0ba7807 Compare June 25, 2026 23:35
@anp-oai anp-oai force-pushed the codex/user-instructions-config branch from 11189b3 to 1df5a56 Compare June 25, 2026 23:35
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