Skip to content

fix: patch js-yaml and prune stale overrides - #502

Merged
Shramkoweb merged 2 commits into
mainfrom
fix/pnpm-overrides-refresh
Jul 19, 2026
Merged

fix: patch js-yaml and prune stale overrides#502
Shramkoweb merged 2 commits into
mainfrom
fix/pnpm-overrides-refresh

Conversation

@Shramkoweb

@Shramkoweb Shramkoweb commented Jul 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • Patch js-yaml advisory 1121859 (<3.15.0, moderate): both parents (gray-matter, @istanbuljs/load-nyc-config) allow ^3.13.1, so an in-range lockfile update to 3.15.0 suffices — no override needed
  • Prune 6 redundant pnpm.overrides (hono, @opentelemetry/core, @opentelemetry/resources, @opentelemetry/sdk-trace-base, js-yaml@4, @babel/core@7) — each verified empirically: removed, re-resolved, resolutions unchanged and audit stayed clean
  • Keep the 2 load-bearing overrides: @hono/node-server ^1.19.13 (@prisma/dev@0.24.3 shipped by latest prisma@7.8.0 pins vulnerable 1.19.11, advisory 1116281) and postcss ^8.5.10 (latest next@16.2.10 pins vulnerable 8.4.31, advisory 1117015). Side fix: the old postcss override froze the direct dep at a stale 8.5.15; it now resolves to a single 8.5.20 matching package.json

Type

  • feat — new feature
  • fix — bug fix
  • docs — docs only
  • style — formatting / whitespace
  • refactor — internal restructuring
  • perf — performance
  • test — tests only
  • build — build system / deps
  • ci — CI config
  • chore — maintenance

Linked spec or plan

None — routine security hygiene pass via the fixing-package-security skill.

Test plan

  • pnpm verify:full exits 0 locally
  • pnpm audit reports 0 vulnerabilities

Breaking changes

None

Summary by CodeRabbit

  • Chores
    • Updated the application framework and development tooling to newer versions.
    • Updated the project’s package manager version.
    • Simplified package version overrides while preserving required compatibility settings.

js-yaml 3.14.2 -> 3.15.0 (advisory 1121859): parents gray-matter and
@istanbuljs/load-nyc-config allow ^3.13.1, so an in-range lockfile
update suffices, no override needed.

Kept (empirically load-bearing, removal reproduces the advisory):
- @hono/node-server ^1.19.13: @prisma/dev@0.24.3 (shipped by latest
  prisma@7.8.0) pins 1.19.11, vulnerable to advisory 1116281
- postcss ^8.5.10: latest next@16.2.10 pins 8.4.31, vulnerable to
  advisory 1117015

Dropped as redundant (removed, re-resolved, audit stayed clean and
resolutions unchanged): hono, @opentelemetry/core, @opentelemetry/
resources, @opentelemetry/sdk-trace-base, js-yaml@4, @babel/core@7.

Side fix: the old postcss override froze the direct dep at a stale
8.5.15; postcss now resolves to a single 8.5.20 matching package.json.
@vercel

vercel Bot commented Jul 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
portfolio Ready Ready Preview Jul 19, 2026 10:58pm

@coderabbitai

coderabbitai Bot commented Jul 19, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: cc3f8f57-f270-4c65-a3c9-1afa13a3cf18

📥 Commits

Reviewing files that changed from the base of the PR and between 1e40e74 and 5b48e0f.

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

📝 Walkthrough

Walkthrough

package.json updates Next.js, development dependency versions, and pnpm. Several pnpm override entries are removed while the @hono/node-server and postcss overrides remain.

Changes

Dependency and tooling updates

Layer / File(s) Summary
Dependency version updates
package.json
Updates Next.js, @types/node, PostCSS, and Tailwind CSS versions.
pnpm version and overrides
package.json
Updates pnpm from 10.34.4 to 10.34.5 and removes several override entries.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: patching js-yaml and removing stale pnpm overrides.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/pnpm-overrides-refresh

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.

@Shramkoweb
Shramkoweb merged commit bfa27e6 into main Jul 19, 2026
12 of 13 checks passed
@Shramkoweb
Shramkoweb deleted the fix/pnpm-overrides-refresh branch July 19, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant