Repository navigation
Conversation
There was a problem hiding this comment.
🤖 AI Review
Both independent reviews completed with no findings. Reading the changed code and surrounding implementation confirmed consistent archive paths, copy-before-start ordering, cleanup, and entrypoint behavior. No additional concrete issues or fundamental conflicts with the listed next-branch work were found. Tests were inspected but not run.
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.
CLI benchmark (Lightweight Commands): no changes foundCommit
API calls: unchanged for all 8 commands. Binary size: 134.8 MB, unchanged. 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 |
TL;DR
fixes
supabase startandfunctions servefailing on every restart on Docker Desktop whensupabase/functions/.envholds a multiline valuewhat's missing?
a multiline value such as a quoted PEM key was staged under
supabase/.temp/start-secretsand bind mounted into the edge runtime.stopdeletes that folder and the nextstartrecreates it, and Docker Desktop then fails thedocker cpof the main service withmust be a directory, so only the first start in a project ever workednow fixed by:
the multiline env script and its values now ride in the same
docker cparchive as the main service,so the edge runtime no longer needs that host folder. a
multiline-envfolder left by an older CLI is removed on the next startvisual rep
ref:
supabase startfails with "must be a directory" whensupabase/functions/.envcontains a multi-line value #7603