Skip to content

[COVAL-7887] Regenerate SDKs and preserve project PATCH skip policies - #48

Merged
callumreid merged 2 commits into
mainfrom
chore/regenerate-sdks
Oct 19, 2026
Merged

callumreid merged 2 commits into
mainfrom
chore/regenerate-sdks

Conversation

@callumreid

@callumreid callumreid commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Regenerate the Python and TypeScript SDKs from coval-ai/docs@d21ed38b3b18341bbec238ba8198186804702152, including review-project skip policy fields, request guidance, simulation-attribute workspace headers, and typed API errors. Python becomes coval-sdk==0.9.2; TypeScript becomes @coval/sdk@0.7.2.

Review found that Python UpdateReviewProjectRequest.from_dict({"display_name":"Renamed"}) emitted omitted fields as explicit nulls, including the new skippable_metric_ids policy. This could reset a policy during an unrelated PATCH. TypeScript deserialization also discarded explicit null for that policy. Deterministic generation patches now preserve omission versus null; regression tests cover omitted, null, empty, and populated values. Both defects reproduced against generated head 680e21c before the fix.

Validation on the final candidate:

  • Two complete code-generation passes from the pinned docs revision produced identical TypeScript and Python trees.
  • TypeScript build and typecheck passed; 29 tests passed; npm package inspection passed; npm audit found zero vulnerabilities.
  • Python: 93 tests passed; wheel and sdist built; twine check passed for both.
  • Clean-installed npm tarball and Python wheel regression smoke tests passed.
  • No database query changes or migrations.

The Monday 09:00 UTC generation run succeeded: https://github.com/coval-ai/coval-examples/actions/runs/39229653020

After an eligible independent approval and merge, successful SDK CI on current main triggers the existing version-tag workflow, then npm/PyPI trusted publishing. Publication remains unverified until those workflows and registry checks complete. No production release has been dispatched manually.

Fixes COVAL-7887.

@callumreid callumreid changed the title chore(sdks): regenerate from latest OpenAPI specs [COVAL-7887] Regenerate SDKs and preserve project PATCH skip policies Oct 19, 2026
@callumreid
callumreid merged commit 3b4e2b7 into main Oct 19, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants