Skip to content

chore: sync API types from infrastructure - #5897

Merged
supabase-cli-releaser[bot] merged 2 commits into
developfrom
sync/api-types
Jul 21, 2026
Merged

chore: sync API types from infrastructure#5897
supabase-cli-releaser[bot] merged 2 commits into
developfrom
sync/api-types

Conversation

@supabase-cli-releaser

@supabase-cli-releaser supabase-cli-releaser Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

This PR was automatically created to sync API types from the infrastructure repository.

Changes were detected in the generated API code after syncing with the latest spec from infrastructure.

@blacksmith-sh

This comment has been minimized.

Sign in to view
chatgpt-codex-connector[bot]
chatgpt-codex-connector Bot reviewed Jul 17, 2026
View reviewed changes

@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: 9f1a1e9aa9

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread apps/cli-go/pkg/api/types.gen.go
Comment thread apps/cli-go/pkg/api/types.gen.go Outdated
@avallete
avallete requested a review from a team as a code owner July 20, 2026 16:29
@github-actions

github-actions Bot commented Jul 20, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Supabase CLI preview

npx --yes https://pkg.pr.new/supabase/cli/supabase@4c88ac46b9cfa37a1e70d54200726571028cc081

Preview package for commit 4c88ac4.

@blacksmith-sh

This comment has been minimized.

Sign in to view
@avallete avallete mentioned this pull request Jul 20, 2026
Closed
@supabase-cli-releaser
supabase-cli-releaser Bot added this pull request to the merge queue Jul 21, 2026
Merged via the queue into develop with commit 2b8083c Jul 21, 2026
29 checks passed
@supabase-cli-releaser
supabase-cli-releaser Bot deleted the sync/api-types branch July 21, 2026 13:48
@7ttp 7ttp mentioned this pull request Aug 4, 2026
Merged
pull Bot pushed a commit to oogalieboogalie/cli that referenced this pull request Aug 5, 2026
@7ttp
fix(api): resync sso provider schemas (supabase#6058)
a253ccb 
## TL;DR

`supabase sso list` dies with `SchemaError(Missing key at
["items"][0]["domains"][0]["id"])`

The platform dropped `id` from `saml` and `domains[]` on the sso
provider responses on Jul 21. Go absorbed it same day in supabase#5897
we didn't, since our `openapi.json` still marks `domains[].id` required
and we decode strictly.
`show` and `update`'s preflight hit the same field, `remove` hits
`saml.id` one earlier
they surface one at a time because decoding stops at the first missing
key, which is why it reads like a regression of supabase#5475.

The snapshot never healed because that same spec change tripped the two
`saml/required` `test` ops in
`openapi-overrides.json`, the first sync run after the change, and
**api-package-sync has been red every run since**
so this unblocks that too.
The two overrides go with it: upstream adopted both, there's nothing
left to relax, and keeping them only re-arms the same hard stop next
time.

Resynced the five schemas from the live spec and regenerated
byte-identical to a fresh `pnpm generate`, so the next sync won't fight
it.
Nothing read the dropped fields, and `sso.go-payload.ts` already
mirrored Go's struct without them, so `-o yaml|toml` is unchanged and
`-o json` now drops them the way Go does, even if a project still echoes
them.

Two things the resync pulled in that are worth naming rather than
leaving for a reviewer to spot....
the `attribute_mapping…default` union flipped `oneOf` → `anyOf`
upstream, so the generated `Schema.Union` loses `{ mode: "oneOf" }` —
inert here, the four branches are disjoint so no value can match two.
And the fixture churn: the sso suites were feeding themselves
`saml.id`/`domains[].id` that the API doesn't send,
so they'd have stayed green however stale the schema got. They use the
real payload shape now....

## refs

- closes supabase#6051
- supabase#5897 (the same ten fields removed on the Go side)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chatgpt-codex-connector chatgpt-codex-connector[bot] chatgpt-codex-connector[bot] left review comments
@avallete avallete avallete approved these changes
@supabase-oss supabase-oss supabase-oss approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@avallete @supabase-oss