Skip to content

fix(cli): avoid auth for local typegen and services - #5553

Merged
jgoux merged 1 commit into
developfrom
julien/cli-1619-v21060-needs-access-token-for-local-type-gen
Jun 12, 2026
Merged

fix(cli): avoid auth for local typegen and services#5553
jgoux merged 1 commit into
developfrom
julien/cli-1619-v21060-needs-access-token-for-local-type-gen

Conversation

@jgoux

Copy link
Copy Markdown
Contributor

Fixes CLI-1619.

This removes eager Management API client construction from command paths that have tokenless behavior:

  • gen types --local and --db-url now use a lean runtime and lazily construct the platform API client only for linked/project-id generation.
  • services now uses a lean runtime so it can always print local service versions, while keeping the linked-version lookup optional when a token is available.
  • Command-wiring regression coverage exercises tokenless gen types --local and services invocations through the actual CLI command layers.

@jgoux
jgoux requested a review from a team as a code ownerJune 11, 2026 15:31
@jgoux
jgouxforce-pushed the julien/cli-1619-v21060-needs-access-token-for-local-type-gen branch from fced43f to f68c021CompareJune 11, 2026 15:36

@chatgpt-codex-connectorchatgpt-codex-connectorBot 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:fced43f38c

ℹ️ 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 threadapps/cli/src/legacy/auth/legacy-platform-api.layer.ts Outdated
@jgoux
jgouxforce-pushed the julien/cli-1619-v21060-needs-access-token-for-local-type-gen branch from f68c021 to 9fcf3c9CompareJune 11, 2026 15:41
@github-actions

github-actionsBot commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

Supabase CLI preview

npx --yes https://pkg.pr.new/supabase@5553

Preview package for commit 7dcdd70.

@chatgpt-codex-connectorchatgpt-codex-connectorBot 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:9fcf3c9e11

ℹ️ 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 threadapps/cli/src/legacy/commands/gen/types/types.layers.ts Outdated
Comment threadapps/cli/src/legacy/commands/gen/types/types.handler.ts Outdated
@jgoux
jgouxforce-pushed the julien/cli-1619-v21060-needs-access-token-for-local-type-gen branch from 9fcf3c9 to 7dcdd70CompareJune 12, 2026 07:11
@jgoux
jgoux added this pull request to the merge queueJun 12, 2026
Merged via the queue into develop with commit 6c80e31Jun 12, 2026
26 checks passed
@jgoux
jgoux deleted the julien/cli-1619-v21060-needs-access-token-for-local-type-gen branch June 12, 2026 07:24
Sign up for freeto 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.

3 participants

@jgoux@avallete@Coly010