Repository navigation
[SDK-757] Clear the remaining 28 Dependabot alerts - #179
Open
devtools-agent[bot] wants to merge 1 commit into
Open
devtools-agent[bot] wants to merge 1 commit into
devtools-agent[bot] wants to merge 1 commit into
Conversation
- Next examples: bump next and eslint-config-next 16.3.4 -> 16.3.8 (critical next/og RCE, SSRF and the other next advisories), which also pulls sharp 0.35.5. - Next examples: override postcss-selector-parser to ^7.1.6 under Tailwind 3; no 6.x release carries the fix. - Root and Next examples: override js-yaml to ^4.1.0 for @istanbuljs/load-nyc-config, removing argparse 1 and sprintf-js, which has no patched release. load-nyc-config only calls js-yaml's load() on a .nycrc.yaml, which behaves the same in js-yaml 4. - All five lockfiles: refresh source-map-js to 1.2.2, and brace-expansion to 5.0.12 in the Next examples. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
AI Agent Review (openai, openai-astra)Review: no blocking issuesThis PR only changes dependencies: the root and two Next.js example manifests, plus five lockfiles. The published What I checked in the checkout
Notes (not findings)
|
brianr
approved these changes
Oct 11, 2026
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.
Description of the change
Clears the 28 open Dependabot alerts that #173 doesn't cover. Together with #173, which handles the 4 react-router alerts, this resolves all 32 open alerts. None of them affect the published
@rollbar/react, whose only runtime dependency istiny-invariant. Every alert is in dev tooling or the example apps.nextnext/ogRCE GHSA-vcvr-r3jv-pc5j, 1 high SSRF)nextjs,nextjs-approuternextandeslint-config-nextpinned 16.3.4 → 16.3.8sharpnextjs,nextjs-approuternextbumpsource-map-jsbrace-expansionnextjs,nextjs-approuterpostcss-selector-parsernextjs,nextjs-approuteroverrides: { "postcss-selector-parser": "^7.1.6" }(6.1.4 → 7.1.6)sprintf-jsnextjs,nextjs-approuteroverrides: { "@istanbuljs/load-nyc-config": { "js-yaml": "^4.1.0" } }Why the two overrides
jest → babel-plugin-istanbul → @istanbuljs/load-nyc-config → js-yaml@3 → argparse@1 → sprintf-js, and Jest 30 has the same chain.load-nyc-configonly callsrequire('js-yaml').load()when it finds a.nycrc.yaml. This repo doesn't have one, andload()behaves the same in js-yaml 4. With the override, argparse 1 and sprintf-js drop out of all three lockfiles.^6, and the only fix is 7.1.6. The only breaking change in 7.0.0 is that inserting nodes during iteration became safe. I checked by running the Tailwind CLI on each example'sglobals.csswith 6.1.4 and with 7.1.6: the generated CSS is byte-identical in both examples.Lockfile hygiene: npm doesn't apply an override to a nested entry that's already in the lockfile, so I pruned the stale entries before reinstalling. I compared package versions in each lockfile before and after, and the only versions that change are the ones listed above. The
.yalc/@rollbar/reactentries in the Next lockfiles are kept exactly as they are onmain.This supersedes Dependabot's #175 (source-map-js). #173 merges cleanly with this branch in either order; I test-merged it locally.
Type of change
Related issues
Checklists
Development
Validated locally on Node 20 / npm 10, the same sequence as CI:
npm run install:all -- ci,lint,lint:examples,typecheck,build:all(includingnext buildfor both Next examples),testandtest:examples. All passed, andprettier --checkis clean on the changed manifests.Code review
🤖 Generated with Claude Code