Skip to content

Add AcceptWithOverrideCommand for approval decision#14565

Draft
qiyaoq-oai wants to merge 12 commits intomainfrom
feat/add-steerable-approval
Draft

Add AcceptWithOverrideCommand for approval decision#14565
qiyaoq-oai wants to merge 12 commits intomainfrom
feat/add-steerable-approval

Conversation

@qiyaoq-oai
Copy link

@qiyaoq-oai qiyaoq-oai commented Mar 13, 2026

Summary

  • Add acceptWithOverrideCommand as a first-class command approval decision across the protocol, app-server schema/docs, and app-server event handling so clients can approve execution with a replacement command.
  • Plumb override commands through the core shell/orchestrator flow so the approved replacement command becomes the effective command for execution, begin/end events, and tool output, while preserving the correct sandbox selection on retry instead of automatically escaping the sandbox.
  • Tighten approval guardrails and coverage by rejecting override commands for network-only approvals

Test

Unit tests

@qiyaoq-oai
Copy link
Author

I have read the CLA Document and I hereby sign the CLA

@etraut-openai etraut-openai added the oai PRs contributed by OpenAI employees label Mar 13, 2026
@qiyaoq-oai qiyaoq-oai force-pushed the feat/add-steerable-approval branch from 191c0d6 to 6e29015 Compare March 13, 2026 19:48
@qiyaoq-oai qiyaoq-oai force-pushed the feat/add-steerable-approval branch from 8791b27 to d347bc8 Compare March 14, 2026 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

oai PRs contributed by OpenAI employees

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants