[stack 1/8] ci: harden verification and release compatibility - #1158
[stack 1/8] ci: harden verification and release compatibility#1158sethkarten wants to merge 7 commits into
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit a1d2e83. Configure here.
| Check npm's global bin directory with: | ||
|
|
||
| npm bin -g | ||
| echo "$(npm prefix -g)/bin" |
There was a problem hiding this comment.
PATH hint expands too early
Medium Severity
The PATH recovery hint uses an unquoted heredoc, so $(npm prefix -g) runs while the installer prints the message instead of remaining a command for the user. That replaces the intended npm bin -g substitute with an already-expanded echo "/some/path" line, which makes the “check with” instructions misleading right when the binary is missing from PATH.
Reviewed by Cursor Bugbot for commit a1d2e83. Configure here.


Stack 1/8 — ci: harden verification and release compatibility
Base:
mainReview order: merge only after the preceding stack layer is accepted. This PR is not intended to merge independently out of order.
Stack navigation
Summary
Validation
npm run check; targeted installer/release suites; real npm 12 policy smoke; Python 3.10–3.14 locked runtime suites; release pack smoke.Provenance
upstream/mainusing issue reports and PR descriptions/comments only.Linked-item disposition
Fixed on merge
Independently superseded pull requests
Partial/distinct overlap — remains open
Related reports — no closure claim
/update --extensionsalways fails with "Update exited with code 1"; update-available notice is a false positive on npm >= 11 #738 —/update --extensionsalways fails with "Update exited with code 1"; update-available notice is a false positive on npm >= 11npm bin -gPATH instructions on npm 10+ #749 — Installer gives unsupportednpm bin -gPATH instructions on npm 10+Distinct follow-up — remains open
Reviewer notes
main, to avoid cumulative duplicate diffs.