Skip to content

fix(cli): stop cli startup hanging on macos (CLI-2485) - #6783

Merged
avallete merged 2 commits into
developfrom
7ttp/cli-2485-21180-beta72-darwin-arm64-supabase-version-blocks-forever-in
Sep 24, 2026
Merged

avallete merged 2 commits into
developfrom
7ttp/cli-2485-21180-beta72-darwin-arm64-supabase-version-blocks-forever-in

Conversation

@7ttp

@7ttp 7ttp commented Sep 24, 2026

Copy link
Copy Markdown
Member

TL;DR

stops every cli command on macos from waiting on the automounter before it prints anything.

whats broken?

since #6506 every command loads the stack code at startup,
and msgpackr inside it probes for its native addon at the linux build runner's store path baked into the binary

on macos that /home/runner path goes through the automounter on every run, and when the lookup stalls every invocation blocks in open(2)

now fixed by:

setting msgpackr's own MSGPACKR_NATIVE_ACCELERATION_DISABLED
switch at build time for the release and build:binary binaries. the addon never ships in the binary, so msgpackr keeps its js path like it already did wherever that path is missing, and it no longer loads an addon from that path when one exists..

ref:

@7ttp 7ttp self-assigned this Sep 24, 2026
@7ttp
7ttp requested a review from a team as a code owner September 24, 2026 10:02

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 AI Review

The build define is present in both compiled CLI build paths. The only finding is a comment that conflicts with the trusted repository comment policy; no functional defect was identified.

Findings

Severity Location Category Sources Claim
⚪ NIT apps/cli/scripts/build-binary.ts:33 comments codex The comment describes a cross-file match and issue provenance without explaining why this build path needs the define.

Stats

Claude findings: 0 · Codex findings: 1 · Confirmed: 1 · Refuted: 0 · Uncertain: 0


Models: claude-opus-5-5 + gpt-6-sol · Trigger: auto · Workflow run

This review runs once per PR. A maintainer can request another with a /ai-review comment.

Comment thread apps/cli/scripts/build-binary.ts Outdated
Comment thread apps/cli/scripts/build-binary.ts
@avallete
avallete added this pull request to the merge queue Sep 24, 2026
Merged via the queue into develop with commit 7a9221d Sep 24, 2026
37 checks passed
@avallete
avallete deleted the 7ttp/cli-2485-21180-beta72-darwin-arm64-supabase-version-blocks-forever-in branch September 24, 2026 13:39
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.

2.118.0-beta.72 (darwin-arm64): supabase --version blocks forever in open(2), and one stuck process blocks every later invocation

2 participants