Skip to content

fix(stack): no docker fallback in native mode - #6006

Draft
7ttp wants to merge 3 commits into
supabase:developfrom
7ttp:fix/stack-native-no-docker-fallback
Draft

fix(stack): no docker fallback in native mode#6006
7ttp wants to merge 3 commits into
supabase:developfrom
7ttp:fix/stack-native-no-docker-fallback

Conversation

@7ttp

@7ttp 7ttp commented Jul 30, 2026

Copy link
Copy Markdown
Member

TL;DR

Fixes mode: "native" silently falling back to Docker when native binary resolution fails.
The fallback was mode blind
a BinaryNotFoundError or DownloadError always resolved to a Docker image, so a 404 on a native release quietly switched the service to Docker and hid whatever was actually broken. Mode is now threaded into resolution: "native" fails with the error, "auto" falls back exactly like before, "docker" untouched...

Since these errors can actually reach the user now, they also got real messages
(which service, which platform, which URL, and a hint to use auto/docker) instead of printing a bare BinaryNotFoundError nothing downstream needed changing since everything already reads message...

refs

@7ttp
7ttp requested a review from a team as a code owner July 30, 2026 13:28
@7ttp 7ttp self-assigned this Jul 30, 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: 510ec2d005

ℹ️ 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 packages/stack/src/StackPreparation.ts
Comment thread packages/stack/docs/architecture.md
@7ttp
7ttp marked this pull request as draft July 31, 2026 10:46
@7ttp 7ttp added the do not merge Approve to apply; do not merge. label Jul 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Approve to apply; do not merge.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

stack: mode: "native" silently falls back to Docker when binary resolution fails

1 participant