Skip to content

chore(deps): fix dev-dependency security advisories - #282

Merged
STRML merged 1 commit into
masterfrom
chore/dev-dependency-security
Oct 31, 2026
Merged

STRML merged 1 commit into
masterfrom
chore/dev-dependency-security

Conversation

@STRML

@STRML STRML commented Oct 31, 2026 •

Copy link
Copy Markdown
Collaborator

Closes the 48 open Dependabot alerts with lockfile upgrades (yarn up -R), among them brace-expansion 5.0.12, follow-redirects 1.16.1 and qs 6.15.3. All are dev-toolchain packages; dependencies and peerDependencies do not change.

One resolutions entry is needed because no in-range fix exists, with its reason in _resolutionNotes: uuid ^11.1.1, since sockjs (via webpack-dev-server) pins ^8.3.2 and has had no release since.

Check (local, on 8986b14) Result
yarn install --immutable pass
yarn test 6 suites, 502 passed, 2 snapshots (same as master); goldens byte-identical
lint / typecheck / build 0 errors / pass / pass
  • CI green on this head, including the golden browser cross-check

Summary by CodeRabbit

  • Chores
    • Updated an underlying package version to address a reported security advisory. This maintenance change does not alter the app’s functionality.

Lockfile-only upgrades via yarn up -R (all dev-scope): @babel/core 7.29.0->7.29.7 (low), baseline-browser-mapping ->2.11.32, brace-expansion ->1.1.21/2.1.7/5.0.12 (high), browserslist ->4.29.4 (high), compression ->1.8.2 (high), fast-uri ->3.1.9 (high), follow-redirects ->1.16.1 (high), http-proxy-middleware ->2.0.11, js-yaml ->3.15.2/4.3.2 (high), launch-editor ->2.14.2 (high), postcss ->8.5.31 (high), postcss-selector-parser ->7.1.7, proxy-addr ->2.0.8 (critical), qs ->6.15.3 (medium), shell-quote ->1.12.0 (critical), source-map-js ->1.2.2 (high), webpack-dev-server ->5.2.7, websocket-driver ->0.7.5 (critical), ws ->8.22.0 (high).

One resolution added because no in-range fix exists: uuid ^11.1.1 (medium; sockjs pins ^8.3.2, unmaintained; dev-only), noted in package.json _resolutionNotes. All 48 alerts addressed; none left.
@coderabbitai

coderabbitai Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 6862882d-b795-481f-81ad-bc4a2a1f38ef
Commits

Reviewing files that changed from the base of the PR and between 65f8176 and 8986b14.

Files ignored due to path filters: 1
  • yarn.lock is excluded by !**/yarn.lock, !**/*.lock
Files selected for processing: 1
  • package.json

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

Walkthrough

The package configuration adds a Yarn resolution for uuid and records notes about the advisory fix and sockjs pin.

Changes: Dependency resolution

Layer / File(s) Summary
Add dependency resolution pin
package.json
The Yarn configuration pins uuid to ^11.1.1 and records a note about the advisory fix and the sockjs pin through webpack-dev-server.

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

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 8986b

No merge-blocking risk is established for this dependency-resolution change.

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 describes the main change: fixing security advisories in development dependencies.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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 eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. 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

@github-actions

Copy link
Copy Markdown

Packj audit found 1/1 risky dependencies.

Click here for details

Registry Package Version Risks
npm react-draggable 4.5.0

undesirable

. Click for details
  • package is old or abandoned: 493 days old
  • invalid or no author email: no email
  • Triggered by workflow run 52 on commit 7beee270f7759ac8a4ed456ac88afa6fba5d7091

    @STRML
    STRML merged commit ea6bb4f into master Oct 31, 2026
    4 checks passed
    @STRML
    STRML deleted the chore/dev-dependency-security branch October 31, 2026 22:53
    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