Skip to content

Clarify guardian timeout guidance#17521

Merged
won-openai merged 1 commit intomainfrom
feat/guardian-timeout-copy-fix
Apr 12, 2026
Merged

Clarify guardian timeout guidance#17521
won-openai merged 1 commit intomainfrom
feat/guardian-timeout-copy-fix

Conversation

@won-openai
Copy link
Copy Markdown
Collaborator

@won-openai won-openai commented Apr 12, 2026

Summary

  • update the guardian timeout guidance to say permission approval review timed out
  • simplify the retry guidance to say retry once or ask the user for guidance or explicit approval

Testing

  • cargo test -p codex-core guardian_timeout_message_distinguishes_timeout_from_policy_denial
  • cargo test -p codex-core guardian_review_decision_maps_to_mcp_tool_decision

@won-openai won-openai merged commit 3895ddd into main Apr 12, 2026
27 of 30 checks passed
@won-openai won-openai deleted the feat/guardian-timeout-copy-fix branch April 12, 2026 09:03
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@xli-oai xli-oai xli-oai approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@won-openai @xli-oai