Repository navigation
Fix deprecated transitive glob@10.5.0 via gaxios override (bacluc-agent/agent-todo#369) - #5
Open
bacluc-agent wants to merge 1 commit into
Open
bacluc-agent wants to merge 1 commit into
bacluc-agent wants to merge 1 commit into
Conversation
gcp-metadata@8.1.4 exact-pins gaxios 7.1.3, the only gaxios version that carries rimraf@^5.0.1, which pulls the deprecated glob@10.5.0. Overriding gaxios to ^7.3.1 (same major; already resolved by google-auth-library and gtoken in this tree) removes the rimraf/glob@10 subtree from the lockfile. No declared dependency version is changed.
bacluc-agent
force-pushed
the
issue-369-glob-deprecation
branch
from
October 16, 2026 05:41
03e8a2a to
43eb9f8
Compare
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.
https://github.com/bacluc-agent/agent-todo/issues/369
Management summary
npm ciemittednpm warn deprecated glob@10.5.0becausegcp-metadata@8.1.4exact-pinsgaxios@7.1.3, the only gaxios version that carriesrimraf@^5.0.1, which pulls the deprecatedglob@10.5.0. Adding an npmoverridesentry forgaxios^7.3.1(same major; already resolved bygoogle-auth-libraryandgtokenin this tree) makes npm drop therimraf/glob@10subtree from the lockfile, so the warning disappears. No declared dependency version changed (dependencies/devDependenciesuntouched); the lockfile change is deletion-only (0 insertions, 417 deletions), and only this one deprecation class is addressed.Verification evidence
Only the fork run below was triggered explicitly by us via
workflow_dispatch(head43eb9f8a); the upstream run is cited as the pre-existing problem, and automatic push-/PR-triggered workflows are not claimed as our testing.Problem run (pre-existing upstream run, not ours), step 4
Run npm cishows theglob@10.5.0deprecation we fix:https://github.com/BacLuc/opencode-completion-check-command/actions/runs/38853014052/job/116818200659#step:4
Our dispatched run 38857110527 (fork
bacluc-agent/opencode-completion-check-command, job 116830963482, head 43eb9f8), covering every changed path —package-lock.jsonvianpm ci,package.jsonvia typecheck and format:check:Run npm ci— the only deprecation is nownode-domexception@1.0.0;glob@10.5.0is absent:https://github.com/bacluc-agent/opencode-completion-check-command/actions/runs/38857110527/job/116830963482#step:4
Run npm run typecheck— success:https://github.com/bacluc-agent/opencode-completion-check-command/actions/runs/38857110527/job/116830963482#step:5
Run npm test— success:https://github.com/bacluc-agent/opencode-completion-check-command/actions/runs/38857110527/job/116830963482#step:6
Run npm run format:check— success:https://github.com/bacluc-agent/opencode-completion-check-command/actions/runs/38857110527/job/116830963482#step:7
npm ciglob@10.5.0warningglob@10.5.0npm cion fixed treenpm run typechecknpm testnpm run format:checkRationale and out of scope
Why an
overridesentry and not a version bump. Renovate's npm manager extractspackage.jsonoverridesas its own dependency type, so once this entry is merged Renovate keeps it updated like any other dependency (minimumReleaseAge/automerge/labels apply as usual). Two caveats: this repo'srenovate.jsonhas no overrides-specific rule, and Renovate can only update an existing override — it cannot add one, so this initial entry is a one-time manual addition. No declared dependency version (dependencies/devDependencies) is changed.Accepted risk: overriding an exact pin.
gcp-metadata@8.1.4declaresgaxios: "7.1.3"exactly; this override deliberately replaces that exact pin with the same-major7.3.1thatgoogle-auth-library@10.5.0/gtoken@8.0.0already resolve in this tree. Nothing insrc/ortests/importsgaxios, andgoogle-auth-libraryreleases up to11.3.0still requiregaxios@^7.1.4(only the not-yet-used11.3.1moved to^8.3.0), so there is no conflict today. If a future dependency requiresgaxios@8, npm would install7.3.1anyway andnpm lswould still exit 0, so such a bump must ship an override update in the same change.Out of scope for this run (one deprecation class only):
node-domexception@1.0.0is deferred. Every published version of that package is deprecated,fetch-blob@3.2.0andfetch-blob@4.0.0both depend on it, andnode-fetch's stable line is 3.3.2, which pulls it in. There is no clean upgrade path that removes it.projectsyn/pr-label-tag-actionin the Autorelease workflow is deferred (no node24 release of that action exists yet; GitHub force-runs it on node24, so it is warning-only).Changed paths
package.json— addsoverrides(3 insertions, 0 deletions)package-lock.json— 0 insertions, 417 deletions (removes therimraf/glob@10subtree)