Skip to content

[codex] Clean guardian instructions#18934

Merged
dylan-hurd-oai merged 1 commit into
mainfrom
dh--guardian--base-instructions
Apr 22, 2026
Merged

[codex] Clean guardian instructions#18934
dylan-hurd-oai merged 1 commit into
mainfrom
dh--guardian--base-instructions

Conversation

@dylan-hurd-oai

Copy link
Copy Markdown
Collaborator

Summary

  • Keep the guardian policy installed as guardian base instructions.
  • Clear inherited parent developer_instructions for guardian review sessions.
  • Update guardian config tests to assert developer instructions are cleared and policy text is sourced from base instructions.

Why

Guardian review sessions are intended to run under an isolated guardian policy. Because the guardian config is cloned from the parent config, inherited custom or managed developer instructions could otherwise remain active and conflict with guardian review behavior.

Validation

  • just fmt
  • cargo test -p codex-core guardian_review_session_config

@dylan-hurd-oai dylan-hurd-oai marked this pull request as ready for review April 22, 2026 02:53
@dylan-hurd-oai dylan-hurd-oai requested a review from a team as a code owner April 22, 2026 02:53
@dylan-hurd-oai dylan-hurd-oai force-pushed the dh--guardian--base-instructions branch from f6cc531 to 0cc3810 Compare April 22, 2026 04:10
Co-authored-by: Codex <noreply@openai.com>
@dylan-hurd-oai dylan-hurd-oai force-pushed the dh--guardian--base-instructions branch from 0cc3810 to 9352422 Compare April 22, 2026 04:17
@dylan-hurd-oai dylan-hurd-oai changed the title [codex] Clear guardian developer instructions [codex] Clean guardian instructions Apr 22, 2026
@dylan-hurd-oai dylan-hurd-oai merged commit 34800d7 into main Apr 22, 2026
25 checks passed
@dylan-hurd-oai dylan-hurd-oai deleted the dh--guardian--base-instructions branch April 22, 2026 04:48
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 22, 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.

2 participants