Skip to content

Update pinned API contract snapshot - #2456

Merged
thomasluizon merged 1 commit into
mainfrom
chore/contract-snapshot
Nov 14, 2026
Merged

thomasluizon merged 1 commit into
mainfrom
chore/contract-snapshot

Conversation

@orbit-contract-rebaseline

Copy link
Copy Markdown
Contributor

The orbit-api producer or selected artifact moved. Review the complete source metadata and generated snapshot, then update hand-written Zod types only when the consumed contract changed. API deployment acceptance is separate from generation.

The snapshot is a diff target. It is not imported at runtime.

@sonarqubecloud

Copy link
Copy Markdown

@thomasluizon

Copy link
Copy Markdown
Owner

@pullfrog review

@pullfrog pullfrog 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.

✅ No new issues found.

Reviewed changes

This run reviewed the routine rebaseline. It moves the pinned orbit-api producer SHA without changing the generated snapshot or the selector.

  • Producer pin moves forward. api.spec.commit advances from 1cf3559 to d39bc73. That commit is on orbit-api main, and the old pin is its direct ancestor. The range has two commits: #1112 (staging landing infra) and #1113 (CI). The only src/ change is src/Orbit.Infrastructure/AI/O200kTokenCounter.cs, which is not a contract surface.
  • Selected document is byte-identical. src/Orbit.Api/openapi_v1-r2.json is identical at both producer commits. api.spec.selection.json stays orbit-http-v1-r2 / 1.0, and api.generated.ts is unchanged. Nothing in the consumed contract moved, so the handwritten Zod schemas need no updates and installed Android clients are unaffected.

Pullfrog  | View workflow run | Using claude-opus-5-5 | 𝕏

@thomasluizon
thomasluizon merged commit 1823697 into main Nov 14, 2026
59 checks passed
@thomasluizon
thomasluizon deleted the chore/contract-snapshot branch November 14, 2026 23:59
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.

1 participant