Skip to content

feat: add approval override argument parity for RunState resumes#2695

Open
seratch wants to merge 5 commits intomainfrom
feat/approval-override-arguments-parity
Open

feat: add approval override argument parity for RunState resumes#2695
seratch wants to merge 5 commits intomainfrom
feat/approval-override-arguments-parity

Conversation

@seratch
Copy link
Member

@seratch seratch commented Mar 17, 2026

This pull request adds TypeScript-parity approval argument overrides to the Python SDK's HITL resume flow (see openai/openai-agents-js#1098 for TS implementation). It extends RunState.approve() with durable and execution-only override modes, persists the new override bookkeeping in schema 1.7, and updates replay surfaces so resumed tool execution, serialized state, and local canonical history stay aligned.

It also introduces Python equivalents of rewrite-aware and server-managed session capabilities. SQLiteSession and OpenAIResponsesCompactionSession can now rewrite persisted function-call history, OpenAIConversationsSession advertises server-managed history semantics for validation, and the runner now fails fast when a requested override mode is incompatible with the active session or conversation setup.

Finally, this pull request keeps compaction and tracing behavior consistent with the corrected transcript. Session persistence applies pending history mutations even on approval-only save cycles, compaction stays in input mode after local rewrites until a newer response id is observed, and approval overrides emit an audit custom span only when sensitive tracing is enabled. Regression coverage was added across sync resumes, streamed resumes, tracing, and compaction.

@seratch seratch added this to the 0.14.x milestone Mar 17, 2026
@github-actions github-actions bot added enhancement New feature or request feature:sessions labels Mar 17, 2026
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1e8b70a05e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request feature:sessions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant