Skip to content

chore(deps): bump shell-quote and sharp in root npm lockfile - #5361

Merged
cdbartholomew merged 1 commit into
mainfrom
chore/security-daily-20261006-npm-root-r2
Oct 6, 2026
Merged

cdbartholomew merged 1 commit into
mainfrom
chore/security-daily-20261006-npm-root-r2

Conversation

@cdbartholomew

Copy link
Copy Markdown
Contributor

Routine dependency maintenance for the root npm workspace lockfile. It follows up on #5311 for two advisories published after that PR was opened.

Closes Dependabot alerts #1848 #1849

Package Before After Alert
shell-quote 1.10.0 1.12.0 #1848
sharp 0.35.4 0.35.5 (+ @img/sharp-* platform packages, libvips 1.3.3 → 1.3.4) #1849

Both are in-range updates under the existing overrides ("shell-quote": ">=1.9.0", "sharp": ">=0.35.4"), so package.json is unchanged. The lockfile was regenerated with npm 11 (npx npm@11 update sharp shell-quote --package-lock-only), because npm 10.9.x mishandles overrides in workspaces. npm 11 also records libc on the new sharp platform entries.

Verification (npm 10.9.2 npm ci from the committed lockfile)

  • sharp smoke test: create → PNG → resize → WebP succeeds. sharp.versions reports 0.35.5 / vips 8.18.7
  • shell-quote smoke test: parse / quote round-trip
  • npm run build: typescript client, hindsight-control-plane (Next.js), hindsight-all-npm, agent-sdk, and hindsight-docs (docusaurus build) all pass. The control-plane and docs CSS output is byte-identical to before
  • npm test: control-plane 289/289, all-npm, agent-sdk all pass

@cdbartholomew cdbartholomew added security Security / dependency vulnerability remediation p0 Priority: critical labels Oct 6, 2026
@cdbartholomew cdbartholomew self-assigned this Oct 6, 2026
@cdbartholomew

Copy link
Copy Markdown
Contributor Author

Merging over a known, pre-existing build-docs failure. It is unrelated to this lockfile-only change.

build-docs fails at its first step, node hindsight-docs/scripts/check-integrations.mjs, before any dependency is installed:

[integrations] ❌ Changelog pages not linked from their doc page:
  agno — add [View Changelog →](/changelog/integrations/agno)

Why this is pre-existing:

Every other check that ran on this PR passed, including the job(s) that cover the changed lockfile.

@cdbartholomew
cdbartholomew merged commit 9269b88 into main Oct 6, 2026
211 of 214 checks passed
@cdbartholomew
cdbartholomew deleted the chore/security-daily-20261006-npm-root-r2 branch October 6, 2026 16:21
@nicoloboschi nicoloboschi mentioned this pull request Nov 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p0 Priority: critical security Security / dependency vulnerability remediation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant