Repository navigation
fix(stack): connect functions to the database as postgres (CLI-2957) - #7571
Conversation
CLI benchmark (Lightweight Commands): no changes foundCommit
API calls: unchanged for all 7 commands. Binary size: 134.9 MB, up 4.0 KB from 134.9 MB. Details: timing ranges, API calls per command, slowest phasesTiming ranges
API calls per command
Slowest phasesNo command moved beyond noise, so there are no phase deltas. Comment |
There was a problem hiding this comment.
🤖 AI Review
Both independent reviews completed with no findings. Inspection of the checked-out code confirms consistent postgres-role URL rendering and Functions wiring. No actionable issues or fundamental conflicts with the pending next-branch work were identified. Tests were not run during adjudication.
Findings
No issues found.
Stats
Claude findings: 0 · Codex findings: 0 · Confirmed: 0 · Refuted: 0 · Uncertain: 0
Models: claude-opus-5-5 + gpt-6.1-sol · Trigger: auto · Workflow run
This review runs once per PR. A maintainer can request another with a /ai-review comment.
Coly010
left a comment
There was a problem hiding this comment.
LGTM, one small optional ask:
- the PR body says a stack created before this keeps
supabase_adminacrossstack restartuntil its nextstack start- might be worth a line instack/start/SIDE_EFFECTS.mdtoo, so folk hitting it on an older stack can find it without the PR. happy for you to skip it if you think it's too niche
TL;DR
fixes edge functions on the stack connecting to the database as the
supabase_adminsuperuser instead ofpostgreswhats hurting?
on the stack,
SUPABASE_DB_URLinside a function pointed atsupabase_admin, while the legacy backend and deployed functions getpostgresso a function could run superuser only sql locally that fails once deployed
tables a function created were also owned by
supabase_admin, so migrations anddb queryaspostgrescould not alter themnow fixed by?
the stack hands functions a
postgresrole url, onsupabase startand onfunctions servea stack created before this picks it up on its next
stack start, whilestack restartkeeps the saved valueref: