Thanks to visit codestin.com
Credit goes to github.com

Skip to content

ci: the gate's step 8 says how to check the registry, not how to fix it - #33

Closed
docushell-dev wants to merge 1 commit into
mainfrom
docs/gate-names-the-proxy-remedy
Closed

docushell-dev wants to merge 1 commit into
mainfrom
docs/gate-names-the-proxy-remedy

Conversation

@docushell-dev

Copy link
Copy Markdown
Contributor

ci/sdk-suites.sh already tells a reader whose npm install failed that npm config get registry is the first thing to check, and that an unreachable registry fails the gate rather than silently producing a partial one. Both true — and it stops one step short of the remedy.

Behind a corporate proxy the step fails twice over, and neither message points anywhere useful: ENOTFOUND against an internal mirror with the VPN down, then CERT_HAS_EXPIRED against the same host with it up. Reading either, the natural conclusion is that the mirror is broken and there's nothing to be done.

In fact exporting the proxy for the run fixes it outright — and the gate then passes all eight checks, including the fourteen Node tests that otherwise skip while the run still exits 0. That's the state this repository shipped in until 0.42.1, which is why step 8 refuses to treat a missing dependency as a skip.

Working that out from the error text alone cost an afternoon. The remedy is now in the error text.

Two deliberate choices in the wording:

  • The hosts named are one organisation's, and are labelled as such. What transfers is the shape — export for the run, change nothing persistent.
  • It says explicitly that npm config set is not the advice. A gate that edits your tooling in order to pass is a gate nobody can trust afterwards.

Error-message text only; nothing executes on the passing path. bash -n clean.

I'll add the ci/gate.sh result before merge — an end2end verification run for 0.54.0 is using the machine and I didn't want the two competing, same as #31.

🤖 Generated with Claude Code

`ci/sdk-suites.sh` already tells a reader whose `npm install` failed that `npm config get registry`
is the first thing to check, and that a registry it cannot reach fails the gate rather than
silently producing a partial one. Both true, and it stops one step short of the remedy.

Behind a corporate proxy the step fails TWICE OVER and neither message points anywhere useful:
ENOTFOUND against an internal mirror with the VPN down, then CERT_HAS_EXPIRED against the same host
with it up. Reading either, the natural conclusion is that the mirror is broken and there is
nothing to be done — when in fact exporting the proxy for the run fixes it outright, and the gate
then passes all eight checks including the fourteen Node tests that otherwise skip.

That cost an afternoon to work out from the error text alone. The remedy is now IN the error text.

The hosts named are one organisation's and are labelled as such; what transfers is the shape —
export for the run, change nothing persistent. The message says explicitly that `npm config set`
is NOT the advice, because a gate that edits your tooling in order to pass is a gate nobody can
trust afterwards.

Error-message text only. Nothing executes on the passing path; `bash -n` clean.

Co-Authored-By: Claude Opus 5 <[email protected]>
Signed-off-by: docushell-dev <[email protected]>
@docushell-dev
docushell-dev deleted the docs/gate-names-the-proxy-remedy branch September 8, 2026 12:11
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.

1 participant