Skip to content

Use Auto-review wording for fallback rationale#19168

Merged
won-openai merged 1 commit intomainfrom
maja/nit-wording
Apr 23, 2026
Merged

Use Auto-review wording for fallback rationale#19168
won-openai merged 1 commit intomainfrom
maja/nit-wording

Conversation

@maja-openai
Copy link
Copy Markdown
Contributor

@maja-openai maja-openai commented Apr 23, 2026

Why

PR #18797 currently surfaces fallback rationale text that names Guardian directly.

What changed

  • Updated the bare allow and bare deny fallback rationales in codex-rs/core/src/guardian/prompt.rs from Guardian to Auto-review.
  • Updated the existing bare allow parser test and added explicit bare deny parser coverage.

Verification

  • cargo test -p codex-core parse_guardian_assessment_treats_bare

@maja-openai maja-openai marked this pull request as ready for review April 23, 2026 18:41
@maja-openai maja-openai requested a review from a team as a code owner April 23, 2026 18:41
@won-openai won-openai merged commit cff337e into main Apr 23, 2026
25 checks passed
@won-openai won-openai deleted the maja/nit-wording branch April 23, 2026 18:42
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 23, 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