Repository navigation
fix(deps): let Renovate's npm lock lockfiles that hold npm again - #503
Merged
Merged
Conversation
npm 12 refuses to fetch a package that a URL names (allow-remote=none), yet its --package-lock-only opens a package with bundled dependencies, such as npm, by its registry URL without the exemption a registry tarball gets elsewhere (npm/cli#9800). Renovate's weekly lock file maintenance (#502) therefore failed for package-lock.json and the pivot and nss/6 lockfiles, which hold npm through @semantic-release/npm and the servers' dependencies. Renovate's npm alone now runs with allow-remote=all, merged ahead of each directory's own .npmrc, so the 3-day min-release-age still applies; a test keeps npmrcMerge on. Contributors' npm keeps npm 12's default. Remove both settings once an npm release fixes the issue.
ixuz
enabled auto-merge
November 2, 2026 00:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What and why
Renovate's weekly lock file maintenance (#502) failed its
renovate/artifactscheck withEALLOWREMOTEforpackage-lock.json,e2e/pod/servers/pivot/package-lock.jsonande2e/pod/servers/nss/6/package-lock.json:Those are the lockfiles that hold the
npmpackage. In the root one,@semantic-release/npmbrings it in. npm 12 (packageManager: npm@12.2.1) fetches no package that a URL names (allow-remote=none). Yet--package-lock-only, which Renovate locks with, opens every package with bundled dependencies by its registry URL. It does so without the exemption a registry tarball gets elsewhere. This is an npm bug, npm/cli#9800, and its fix (npm/cli#9819) is in no release yet. A plainnpm installis not affected. Neither is an update that keeps the lockfile, so only lock file maintenance, and any update that movesnpmitself, hit it.renovate.json5now gives Renovate's npmallow-remote=all, withnpmrcMerge: true. Renovate puts that line ahead of each directory's own.npmrcduring its artifact updates, then restores the files. So the 3-daymin-release-ageandignore-scriptsstill apply. Contributors' npm keeps npm 12's default. A test inscripts/renovateConfig.test.tskeepsnpmrcMergeon: without it, Renovate would use itsnpmrcin place of the.npmrcfiles and lock without the 3-day wait. Both settings come out once an npm release fixes the issue.Checked locally:
min-release-agestill reads 3.EALLOWREMOTE.renovate-config-validator --strict(Renovate 44.188.2) accepts the config.Once this is merged, tick the rebase/retry box on #502 so that Renovate runs its artifacts again.
Closes: #505
Checklist
Refs:, notCloses:. Leave this unchecked otherwise.npm run checkpasses (lint, Markdown and spelling included),npm run lint:reusetoo if a file was added, andnpm run crosschecktoo ifns/ordecks/changed. (No file added;ns/anddecks/unchanged.)docs/is listed in docs/docs.ttl. (n/a: the reason is inrenovate.json5's comment, beside the setting; no page describes Renovate's npm settings.)packages/ui/src/i18n/). (n/a: no message changed.)ns/was agreed in an issue first; no publishedns/shapes/*/vN.ttlordecks/*/vN.ttlwas edited; generated files were regenerated, not edited. (n/a:ns/anddecks/unchanged.)!and aBREAKING CHANGE:footer).npm run check:breakingsees a line ofapi/that goes, but not a feature removed, nor a reader that reads less than before.--dry-runfails with EALLOWREMOTE on registry tarballs for packages that declare bundleDependencies npm/cli#9800.)npm run lint:reuse), or a new deck its line inREUSE.toml; third-party content is credited. (n/a: no new file.)npm run dev) and tried the change, and considered the user journeys. (n/a: Renovate configuration only; npm 12's lock run tried by hand instead.)type(scope): subject,!and aBREAKING CHANGE:footer if it breaks something), with an imperative subject and a body that explains why (CONTRIBUTING.md).