Skip to content

fix(deps): close the seven Dependabot alerts - #13

Merged
iamrraj merged 2 commits into
mainfrom
deps/security-2026-08
Aug 3, 2026
Merged

fix(deps): close the seven Dependabot alerts#13
iamrraj merged 2 commits into
mainfrom
deps/security-2026-08

Conversation

@iamrraj

@iamrraj iamrraj commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Closes every alert GitHub currently reports on main: 3 high, 4 moderate.

Package Severity Change Why it matters here
js-yaml high 4.1 β†’ 4.3.1 Merge-key chains force quadratic CPU. Direct dependency, used to parse .github/agent.yml β€” which arrives from a repository, so it is reachable input.
postcss high β†’ 8.5.25 (both manifests) Path traversal in source-map auto-loading.
protobufjs moderate β†’ 7.6.5 Infinite loop parsing .proto options.
react-router-dom moderate Γ—3 6.30 β†’ 7.18.2 The 6.x line has no patch for the alert, so this is the major bump rather than a version pick.

On the react-router major

API-compatible with everything the site uses: BrowserRouter, Routes, Route, Link, NavLink, Navigate, useSearchParams, useLocation, useNavigate, useParams. Verified by typecheck, production build, and a browser pass over every dashboard page β€” nav links still carry filters across pages.

Two things deliberately left alone

react-router 7.18.2 carries GHSA-qwww-vcr4-c8h2 (>=7.12.0 <8.3.0), and 8.3.0 is not published yet. npm offers a "fix" of 7.11.0 β€” which is below 7.18.0 and therefore reintroduces the alert this PR closes. The advisory is an RSC-mode CSRF bypass; this is a client-only SPA with no framework mode, no server actions and no RSC (no react-router.config, nothing imports react-router/rsc). Staying on the latest is the better of the two available positions, and this should be revisited when 8.3.0 ships.

The 21 remaining moderate npm audit findings are all probot's OpenTelemetry and Sentry chain. None are in GitHub's alert list, and the only fix npm offers is probot@14 β€” a major bump of the webhook framework, which does not belong in a dependency patch.

Verification

  • npm test β€” 546 passing
  • npx tsc --noEmit β€” clean, both packages
  • npm run build and vite build β€” clean
  • Browser pass over /dashboard, /runs, /tools, /events, /findings

Raised as a PR rather than a direct push, so the test status check runs before this reaches main.

- js-yaml 4.1 β†’ 4.3.1 (high): merge-key chains force quadratic CPU.
  Direct dependency, used to parse .github/agent.yml β€” which arrives
  from a repository, so it is reachable input.
- postcss β†’ 8.5.25 (high, both manifests): path traversal in source-map
  auto-loading.
- protobufjs β†’ 7.6.5: infinite loop parsing .proto options.
- react-router-dom 6.30 β†’ 7.18.2 (medium Γ—3): the 6.x line has no patch
  for the alert, so this is the major bump rather than a version pick.

The v7 move is API-compatible with what the site uses β€” BrowserRouter,
Routes, Route, Link, NavLink, Navigate, useSearchParams, useLocation,
useNavigate, useParams. Typecheck, build, and a browser pass over the
dashboard all clean; nav links still carry filters across pages.

Two things deliberately not done here:

- react-router 7.18.2 carries GHSA-qwww-vcr4-c8h2 (>=7.12.0 <8.3.0), and
  8.3.0 does not exist yet. npm offers a "fix" of 7.11.0, which is below
  7.18.0 and so reintroduces the alert this commit closes. The advisory
  is an RSC-mode CSRF bypass; this is a client-only SPA with no framework
  mode, no server actions and no RSC β€” there is no react-router.config
  and nothing imports react-router/rsc. Staying on the latest is the
  better of the two available positions.
- The 21 remaining moderate advisories are all probot's OpenTelemetry and
  Sentry chain. None are in GitHub's alert list, and the only fix npm
  offers is probot@14 β€” a major bump of the webhook framework, which does
  not belong in a dependency patch.
@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

πŸ‘€ ShipIT Forge is running a code + security review on this PR… I'll post my review shortly.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

πŸ” ShipIT Forge reviewed this PR β€” πŸ’¬ commented (no blocking issues)

0 finding(s) (0 security). See the review above for inline details and suggested fixes.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ShipIT Forge review

βœ… No blocking issues found. I reviewed the changed files and ran:

  • πŸ›‘οΈ Security checks β€” SSRF, injection (SQL/command/template), broken auth/authz, hardcoded secrets, unsafe deserialization, path traversal, weak crypto.
  • πŸ”§ Code review β€” correctness, error handling, missing tests, clarity.

Nothing to flag. This is a comment, not an approval β€” ShipIT Forge never approves PRs; a human reviewer should approve and merge.

@iamrraj
iamrraj merged commit e40bd82 into main Aug 3, 2026
2 checks passed
@iamrraj
iamrraj deleted the deps/security-2026-08 branch August 3, 2026 13:08
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