Skip to content

ci: put npm's global bin on PATH for the dx CLI - #27

Merged
wittjosiah merged 1 commit into
mainfrom
claude/plugin-excalidraw-pr8-cleanup-hb0ebu
Aug 8, 2026
Merged

ci: put npm's global bin on PATH for the dx CLI#27
wittjosiah merged 1 commit into
mainfrom
claude/plugin-excalidraw-pr8-cleanup-hb0ebu

Conversation

@wittjosiah

@wittjosiah wittjosiah commented Aug 8, 2026

Copy link
Copy Markdown
Member

npm published. @dxos/plugin-tictactoe@0.10.1 is live — dist-tags.latest moved from the 0.0.0 placeholder to 0.10.1 — and tagged. The repository.url fix worked; that half of the pipeline is proven end to end.

The registry publish still didn't run (run 31267297719):

added 1 package in 6s
/home/runner/work/_temp/….sh: line 3: dx: command not found
##[error]Process completed with exit code 127

The install succeeded and the binary was still unresolvable, because node comes from proto and npm's global bin is not a directory the shell already searches.

Change

export PATH="$(npm prefix -g)/bin:${PATH}"
npm install -g "${DX_CLI_PACKAGE}"
dx --version

The same npm answers prefix -g and performs the install, so the location is correct by construction — no second guess about where the binary went, and nothing invented for it to live in.

Verification

Uninstalled the CLI, confirmed dx was genuinely unresolvable, then ran the step's lines extracted from the workflow YAML so the test can't drift from what ships, with proto's node first on PATH as on the runner:

dx before                → none
export PATH=…            → /opt/node22/bin
npm install -g …         → added 1 package in 7s
dx --version             → 0.10.0, exit 0

(An earlier attempt of mine reported success against a stale CLI still on PATH from a previous test; the run above is after removing it.)

Next

npm is append-only, so a plain re-run finds 0.10.1 already published, reports nothing released, and skips the registry step — the trap registry_only was added for in #26. Once this lands I'll dispatch Release with registry_only: true to drive the registry half, with no version burn per attempt.

The registry publish never ran: `npm install -g` reported success and the very
next line failed with `dx: command not found`. node comes from proto, so npm's
global bin is not a directory the shell already searches.

npm reports the prefix it installs into, so the step asks for it rather than
choosing one.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SZ4ZdaPo9erkh5yF4eXh3Z
@wittjosiah
wittjosiah force-pushed the claude/plugin-excalidraw-pr8-cleanup-hb0ebu branch from f975570 to d04d46a Compare August 8, 2026 17:09
@wittjosiah wittjosiah changed the title ci: install the dx CLI into an explicit prefix ci: put npm's global bin on PATH for the dx CLI Aug 8, 2026
@wittjosiah
wittjosiah marked this pull request as ready for review August 8, 2026 17:19
@wittjosiah
wittjosiah added this pull request to the merge queue Aug 8, 2026
Merged via the queue into main with commit 40ab6b6 Aug 8, 2026
3 checks passed
@wittjosiah
wittjosiah deleted the claude/plugin-excalidraw-pr8-cleanup-hb0ebu branch August 8, 2026 17:21
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 participants