fix(ENG-2103): pin the antigravity Defender transitive deps - #33
Conversation
All 11 open Dependabot alerts on this repo sit in plugins/security/stackone-defender-antigravity/package-lock.json: that copy ships an empty "overrides" block while plugins/security/stackone-defender already pins the same five packages. Copy those pins across and regenerate the lockfile. Resolves 1 critical (tar <=7.5.18), 6 high (tar, nanoid x2, sharp, adm-zip) and 4 medium (protobufjs x2, tar x2). All transitive, via @huggingface/transformers and onnxruntime-node. @stackone/defender stays at ~0.7.0 — that is a functional version difference from the main plugin, not a security fix, so it is out of scope here. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Pins vulnerable transitive dependencies for the stackone-defender-antigravity plugin using npm overrides, then regenerates the lockfile so resolved package versions are updated and Dependabot alerts are cleared.
Changes:
- Add npm
overridesto pinadm-zip,nanoid,protobufjs,sharp, andtarto patched versions. - Regenerate
package-lock.jsonto reflect the new resolved dependency graph (including updatedsharpplatform packages and related dependency bumps).
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| plugins/security/stackone-defender-antigravity/package.json | Adds overrides to pin vulnerable transitive packages to patched versions. |
| plugins/security/stackone-defender-antigravity/package-lock.json | Updates resolved versions in the lockfile to match the new overrides and refreshed dependency tree. |
Files not reviewed (1)
- plugins/security/stackone-defender-antigravity/package-lock.json: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
There was a problem hiding this comment.
No issues found across 2 files
Requires human review: Pins vulnerable transitive dependencies and regenerates the lockfile, but the sharp 0.35.0 override raises the required Node engine from Node 18/20.3 to >=20.9, a compatibility tradeoff that needs human sign-off.
Re-trigger cubic
Closes all 11 open Dependabot alerts on this repo.
Cause
Every alert is in one file —
plugins/security/stackone-defender-antigravity/package-lock.json. That copy of Defender ships an empty"overrides": {}, whileplugins/security/stackone-defenderalready pins the same five packages and has zero alerts. The fix is to copy those pins across and regenerate the lockfile.Resolved
tartarnanoidsharpadm-zipprotobufjstarAll transitive, pulled in through
@huggingface/transformersandonnxruntime-node.Out of scope, deliberately
@stackone/defenderstays at~0.7.0here versus0.8.2in the main plugin. That is a functional version difference, not a security fix, so bumping it does not belong in a dependency-hygiene PR.Verified
npm testin the antigravity plugin: 12/12 pass.claude plugin validate .passes.Diff is the 5
overrideslines plus the regenerated lockfile; indentation ofpackage.jsonleft untouched.🤖 Generated with Claude Code
https://claude.ai/code/session_01LHqJgp9mMhn3gCfBrkRS3o
Summary by cubic
Pins transitive dependencies in
stackone-defender-antigravityto patched versions, closing all 11 open Dependabot alerts on this repo.The antigravity plugin's lockfile had an empty
overridesblock, leaving vulnerable versions of five packages pulled in through@huggingface/transformersandonnxruntime-node. The same pins already exist in the mainstackone-defenderplugin, so this copies them over and regenerates the lockfile.Dependencies
adm-zip,nanoid,protobufjs,sharp, andtarto patched versions.sharp0.35.0 raises the required Node engine from^18.17.0to>=20.9.0.@stackone/defenderstays at~0.7.0; a version bump there is a functional change, not a security fix.Written for commit 7183f83. Summary will update on new commits.