Skip to content

fix: freeze public testing API state contracts - #4404

Merged
seratch merged 1 commit into
mainfrom
fix/testing-api-state-contract
Aug 14, 2026
Merged

fix: freeze public testing API state contracts#4404
seratch merged 1 commit into
mainfrom
fix/testing-api-state-contract

Conversation

@seratch

@seratch seratch commented Aug 14, 2026

Copy link
Copy Markdown
Member

This pull request fixes a gap in the prospective v0.21.0 released API contract for the new scripted testing utilities. The contract already covered their modules, exports, and callable signatures, but it did not protect their intended public state surfaces. As a result, supported properties or TypedDict fields could be removed or changed without failing the release-contract check.

The existing release-contract policy and generator now:

  • Capture explicitly selected public properties for the scripted Model, Realtime, Sandbox, and Voice testing utilities.
  • Capture selected field names, requiredness, and declared annotations for ModelStepSpec, SandboxStepSpec, and RealtimeConnectCall.
  • Resolve properties exposed by the public sandbox factory without freezing its private implementation class.
  • Refresh baseline_commit provenance when only a selected TypedDict surface changes.
  • Report missing or incompatible selected surfaces through the existing actionable contract-diff mechanism.

The policy remains deliberately scoped: it does not freeze arbitrary class attributes, private implementation classes, callback internals, or every TypedDict in the repository. This change does not alter runtime behavior, canonical import paths, or existing testing API signatures.

@seratch seratch added this to the 0.21.x milestone Aug 14, 2026

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

Copy link
Copy Markdown

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: 9122aad9c7

ℹ️ 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".

Comment thread integration_tests/_contract_support.py Outdated
@seratch
seratch force-pushed the fix/testing-api-state-contract branch from 9122aad to ec15614 Compare August 14, 2026 01:17
@seratch
seratch merged commit c881417 into main Aug 14, 2026
14 checks passed
@seratch
seratch deleted the fix/testing-api-state-contract branch August 14, 2026 01:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant