fix(cli): inject S3 and sb key env variables into Studio#5438
Merged
Conversation
Coverage Report for CI Build 26883280758Warning No base build found for commit Coverage: 63.922%Details
Uncovered ChangesNo uncovered changes found. Coverage RegressionsRequires a base build to compare against. How to fix this → Coverage Stats
💛 - Coveralls |
Coly010
approved these changes
Jun 3, 2026
aantti
approved these changes
Jun 3, 2026
jgoux
added a commit
that referenced
this pull request
Jun 4, 2026
- **chore: sync API types from infrastructure (#5417)** - **fix(cli): read Go Windows credentials in legacy TS (#5418)** - **fix(ci): pass release channel to PR smoke workflow (#5419)** - **fix(cli): enable vector buckets by default (#5421)** - **fix(cli): bind a free port for edge-runtime diff containers (#5424)** - **ci(cli): publish pkg.pr.new previews after preview builds (#5420)** - **feat(ci): notify Slack on release failures regardless of channel (#5425)** - **feat(cli): port telemetry (#5422)** - **docs(cli): modernize README and add installer (#5428)** - **fix(docker): check Supabase image updates hourly (#5429)** - **ci(cli): publish pkg.pr.new previews on pull requests (#5427)** - **fix(docker): restore daily Dependabot schedule (#5430)** - **fix(docker): bump the docker-minor group in /apps/cli-go/pkg/config/templates with 5 updates (#5431)** - **fix(cli): read Go Windows credentials via findCredentials (#5423)** - **chore: sync API types from infrastructure (#5434)** - **chore(ci): update Dependabot Go module paths (#5435)** - **ci(release): use app token for release tag pushes (#5432)** - **fix(deps): bump the go-minor group across 2 directories with 7 updates (#5437)** - **fix(docker): bump the docker-minor group in /apps/cli-go/pkg/config/templates with 2 updates (#5436)** - **feat(cli): port link and unlink commands to native TypeScript (#5426)** - **feat(cli): port init (#5433)** - **ci(release): use app token checkout for release pushes (#5439)** - **fix(deps): bump the actions-major group across 1 directory with 14 updates (#5342)** - **fix(docker): bump supabase/studio from 2026.06.01-sha-a4334a2 to 2026.06.03-sha-0bca601 in /apps/cli-go/pkg/config/templates in the docker-minor group (#5441)** - **chore(ci): add CLI preview PR comment (#5440)** - **chore(ci): bump the actions-major group with 2 updates (#5443)** - **chore(ci): use non-releasing actions dependabot prefix (#5442)** - **fix(cli): inject S3 and sb key env variables into Studio (#5438)** - **ci(preview): allow preview package PR comment (#5444)** - **chore(release): add LLM release-notes prompt and approval-based publish pipeline (#5330)** - **chore(ci): fix propose release workflow dispatch (#5447)** - **chore(ci): update workflows to skip CI for release-notes PRs (#5455)** - **feat(cli): port login and logout commands to native TypeScript (#5446)** - **chore(ci): bump aws-actions/configure-aws-credentials from 6.1.2 to 6.1.3 in the actions-major group (#5463)** - **fix(docker): bump supabase/realtime from v2.103.1 to v2.103.2 in /apps/cli-go/pkg/config/templates in the docker-minor group (#5464)** - **fix(cli): persist legacy telemetry opt-out (#5465)** - **fix(cli): restore Go debug output parity (#5467)** --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: supabase-cli-releaser[bot] <246109035+supabase-cli-releaser[bot]@users.noreply.github.com> Co-authored-by: Julien Goux <hi@jgoux.dev> Co-authored-by: Colum Ferry <cferry09@gmail.com> Co-authored-by: Vaibhav <117663341+7ttp@users.noreply.github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: VERSA SYNC STUDIOS <206948228+Versa-Sync-Studios@users.noreply.github.com> Co-authored-by: Andrew Valleteau <avallete@users.noreply.github.com> Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
This wires Studio with the local S3 protocol credentials and the newer opaque Supabase key environment variables in both the Go CLI stack and the TypeScript stack implementation.
The change keeps the existing legacy Studio key variables intact while adding
SUPABASE_PUBLISHABLE_KEY,SUPABASE_SECRET_KEY,S3_PROTOCOL_ACCESS_KEY_ID, andS3_PROTOCOL_ACCESS_KEY_SECRET.Why
Studio expects these values for local S3 key access and newer key handling, but the CLI-managed Studio container only received the legacy key variables before this change.