Skip to content

fix(deps): bump the worker's sharp override to 0.35.5 for GHSA-wq5f-xc86-pv6w - #107

Open
WomB0ComB0 wants to merge 1 commit into
mainfrom
fix/worker-sharp-0.35.5
Open

WomB0ComB0 wants to merge 1 commit into
mainfrom
fix/worker-sharp-0.35.5

Conversation

@WomB0ComB0

@WomB0ComB0 WomB0ComB0 commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Summary

Fixes the red scan / osv-scanner check on main. sharp 0.35.4 is affected by GHSA-wq5f-xc86-pv6w (high, a librsvg vulnerability in the bundled libvips), and 0.35.5 is the first patched release. sharp is a dev-only transitive dependency (wrangler -> miniflare), and the existing npm override in worker/package.json holds it at 0.35.4. This PR moves that override to 0.35.5 and regenerates worker/package-lock.json with npm 11.19.0, the version the worker CI job runs under Node 24. In the lockfile, only sharp and its @img/* platform packages change: 27 entries, none added or removed. The Worker still ships zero runtime dependencies, so the deployed bundle is unchanged.

Type of change

  • fix — bug fix
  • build — build system or external dependency

Test plan

  • osv-scanner scan source --recursive ./ (v2.3.5, as the shared job runs it) → No issues found. The same scan of the previous lockfile reports GHSA-wq5f-xc86-pv6w against sharp 0.35.4.
  • npm ci --no-audit --no-fund (npm 11.19.0) → installs sharp 0.35.5
  • npm run typecheck → clean
  • npm run lint (Biome 2.5.2) → clean
  • node worker/test/index.test.mjs → 81 passed, 0 failed
  • npm run build (wrangler deploy --dry-run) → succeeds; bundle 18.87 KiB

Breaking changes

None.

Checklist

  • Commits follow Conventional Commits (commit-msg hook will reject otherwise)
  • Branch name matches the allowed pattern (pre-push hook will reject otherwise)
  • resq pre-commit clean locally (copyright, large files, debug statements, secrets, audit)
  • New code has tests (or a deliberate note on why not): dependency pin only, covered by the existing worker suite and the OSV scan
  • Docs updated (AGENTS.md / README / inline) for user-visible changes: none needed
  • Security-sensitive changes acknowledged (secrets, auth, input validation): clears a dependency advisory

Summary by CodeRabbit

  • Chores
    • No user-facing changes are included in this release. Existing features and behavior remain unchanged.

…c86-pv6w

sharp 0.35.4 bundles a libvips whose librsvg is affected by
GHSA-wq5f-xc86-pv6w (high, CVE-2026-96889); 0.35.5 is the first patched
release. sharp is dev-only here (wrangler -> miniflare) and is held by
the worker's npm override, so the override is what has to move.

worker/package-lock.json was regenerated with npm 11.19.0, the version
the worker CI job runs under Node 24. Only sharp and its @img/* platform
packages change.
@WomB0ComB0
WomB0ComB0 requested a review from a team as a code owner October 19, 2026 02:05
@github-actions github-actions Bot added C-Bug Something isn't working size/XS PR size: extra small (< 10 lines changed) area:worker Changes under worker/ labels Oct 19, 2026
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 94816c76-66e0-4def-88a6-47edc9a0dabe
📥 Commits

Reviewing files that changed from the base of the PR and between 20fa9f4 and 21bb5e2.

⛔ Files ignored due to path filters: 1
  • worker/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing: 1
  • worker/package.json

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

The worker package updates its sharp override from version 0.35.4 to 0.35.5.

Changes: Sharp override

Layer / File(s) Summary
Update sharp override
worker/package.json
The sharp override changes from 0.35.4 to 0.35.5.

Priority: ➖ Normal

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Suggested reviewers: resq-sw

Merge Risk: ⚪ Minimal · up to 21bb5

The dependency update is reflected in the lockfile and does not appear to change the deployed Worker runtime, so no actionable merge risk remains.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely identifies the worker dependency update and its security purpose. It matches the main change.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
get-resq-software 21bb5e2 Oct 19 2026, 02:05 AM

@github-actions

Copy link
Copy Markdown
Contributor

LGTM! This PR updates the sharp package, which is a good security practice. No issues found.

Generated by ai-auditor for #107 · gem25pro · 29.6 AIC · ⌖ 1.2 AIC · ⊞ 12.8K · ◷

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:worker Changes under worker/ C-Bug Something isn't working size/XS PR size: extra small (< 10 lines changed)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant