Skip to content

ci: fix smoke test for npm 11 (install + run bin directly)#18

Merged
a-essawy merged 1 commit into
mainfrom
fix/smoke-npx-npm11
Jun 1, 2026
Merged

ci: fix smoke test for npm 11 (install + run bin directly)#18
a-essawy merged 1 commit into
mainfrom
fix/smoke-npx-npm11

Conversation

@a-essawy
Copy link
Copy Markdown
Contributor

@a-essawy a-essawy commented Jun 1, 2026

Root cause (surfaced by #14's set +e)

The v1.0.2 release's smoke step failed with exit=127 'sh: 1: rendobar-mcp: not found' on all 5 retries. npx -y @rendobar/mcp@VERSION -- --version fails under npm 11: npx resolves the bin name (rendobar-mcp) instead of the package and can't find it on PATH. Reproduced locally on npm 11.12.1 — Form C (install + node the bin) is the only reliable one.

This blocked Create GitHub Release + MCP registry on every release; set -e masked it until #14.

Fix

Replace npx with a deterministic smoke: install @rendobar/mcp@VERSION into a throwaway dir, run dist/bin.js --version directly with node. No bin-shim / PATH / npx-version dependency; still proves the published artifact installs and runs. Metadata-based propagation wait kept before install.

Recovery context

  • npm 1.0.2 live; GitHub Release v1.0.2 created manually.
  • After merge: dispatch release.yml tag=v1.0.2 to backfill the MCP registry (publish is idempotent), then close Release v1.0.2 failed #13.

`npx -y @rendobar/mcp -- --version` fails under npm 11: npx resolves the bin
name (rendobar-mcp) instead of the package and exits 127 ("rendobar-mcp: not
found"). This blocked Create GitHub Release + MCP registry on every release
(masked until set +e surfaced it). Reproduced locally on npm 11.12.1.

Replace with a deterministic smoke: install the published version into a
throwaway dir and run dist/bin.js directly with node. No bin-shim/PATH or
npx-version dependency; still proves the artifact installs and runs. Keep a
metadata-based propagation wait before installing.
@a-essawy a-essawy enabled auto-merge (squash) June 1, 2026 00:10
@a-essawy a-essawy changed the title fix(ci): smoke test broken on npm 11 — install + run bin directly ci: fix smoke test for npm 11 (install + run bin directly) Jun 1, 2026
@a-essawy a-essawy merged commit 660407b into main Jun 1, 2026
5 checks passed
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.

Release v1.0.2 failed

1 participant