Skip to content

fix(cli): align services version sources#5754

Merged
jgoux merged 4 commits into
developfrom
fix/services-postgres-version-reporting
Jul 1, 2026
Merged

fix(cli): align services version sources#5754
jgoux merged 4 commits into
developfrom
fix/services-postgres-version-reporting

Conversation

@jgoux

@jgoux jgoux commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Align next supabase services local rows with the stack version resolution used by supabase start, including linked baselines and local .temp overrides.
  • Keep legacy supabase services on the Go-compatible path: read config.toml, use legacy Postgres image resolution, and preserve the Deno 1 edge-runtime image override.
  • Keep shared service row rendering source-agnostic, with each command handler supplying the image and version source it actually runs.

Context

This prevents false local/linked mismatch warnings when config values or temp pins drift from the runtime source of truth. The next shell now follows stack runtime versions; the legacy shell follows the Go-compatible config and image path until it is retired.

@supabase-oss supabase-oss marked this pull request as ready for review July 1, 2026 14:31
@supabase-oss supabase-oss requested a review from a team as a code owner July 1, 2026 14:31
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

Supabase CLI preview

npx --yes https://pkg.pr.new/supabase/cli/supabase@8be96a4147673c0daed14a87814d830fb3ecf927

Preview package for commit 8be96a4.

@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: c9ccb0db02

ℹ️ 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/src/shared/services/services.shared.ts
Comment thread apps/cli/src/next/commands/services/services.handler.ts Outdated
Comment thread apps/cli/src/legacy/commands/services/services.handler.ts Outdated
Comment thread apps/cli/src/legacy/commands/services/services.integration.test.ts Outdated
Comment thread apps/cli/src/next/commands/services/services.integration.test.ts Outdated
Comment thread apps/cli/src/next/commands/services/services.integration.test.ts Outdated

@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: 63307062a8

ℹ️ 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 apps/cli/src/next/commands/services/services.handler.ts Outdated
Comment thread apps/cli/src/legacy/commands/services/services.handler.ts Outdated
Comment thread apps/cli/src/legacy/commands/services/services.handler.ts Outdated
@jgoux jgoux changed the title fix(cli): report configured postgres service version fix(cli): align services version sources Jul 1, 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: 97fa1fbabd

ℹ️ 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 apps/cli/src/shared/services/services.shared.ts
Comment thread apps/cli/src/legacy/commands/services/services.handler.ts Outdated
Comment thread apps/cli/src/next/commands/services/services.handler.ts
@jgoux jgoux added this pull request to the merge queue Jul 1, 2026
Merged via the queue into develop with commit c328749 Jul 1, 2026
34 checks passed
@jgoux jgoux deleted the fix/services-postgres-version-reporting branch July 1, 2026 21:27
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.

3 participants