Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: update vulnerable dependencies - #1434

Merged
thymikee merged 1 commit into
mainfrom
agent/security-dependabot-fixes
Jul 27, 2026
Merged

fix: update vulnerable dependencies#1434
thymikee merged 1 commit into
mainfrom
agent/security-dependabot-fixes

Conversation

@thymikee

@thymikee thymikee commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

Update the vulnerable transitive dependencies reported by Dependabot:

  • PostCSS to 8.5.18 in the root and example-app workspaces.
  • React Router and React Router DOM to 7.18.1 in the documentation toolchain.
  • brace-expansion to 5.0.8 in the example app.
  • Keep the existing workspace-level override strategy so regenerated lockfiles retain the patched versions.

This PR touches exactly four dependency metadata/lockfile files and does not expand beyond dependency maintenance. No runtime or public CLI behavior changed.

Dependabot alert #54 (React Router RSC CSRF) was verified as not applicable and dismissed with a rationale: the repository uses Rspress declarative mode and has no unstable RSC APIs, Framework Mode, or manual SSR/hydration code.

Validation

  • pnpm check:tooling
  • pnpm --dir website build
  • pnpm audit --audit-level high in examples/test-app — no known vulnerabilities
  • git diff --check

Docs and skills were not updated because this is dependency metadata and lockfile maintenance only.

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-07-27 13:13 UTC

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown

Size Report

Metric Base Current Diff
JS raw 1.86 MB 1.86 MB 0 B
JS gzip 597.0 kB 597.0 kB 0 B
npm tarball 711.8 kB 711.8 kB 0 B
npm unpacked 2.49 MB 2.49 MB 0 B

Startup median (7 runs, lower is better):

Scenario Base Current Diff
CLI --version 17.0 ms 18.8 ms +1.8 ms
CLI --help 34.9 ms 37.1 ms +2.2 ms

Top changed chunks: no changes in the largest emitted chunks.

@thymikee

Copy link
Copy Markdown
Member Author

Reviewed exact head 6f7c50b5. Dependency/security review is clean: root and test-app overrides match their regenerated locks; patched PostCSS, React Router, and brace-expansion versions are present with affected versions absent; the separate React Router RSC advisory is credibly not applicable to the Rspress declarative site. Current checks are green, and no device evidence is needed for dependency metadata only.

P2 — PR body scope disclosure: add the required statement that this touches four dependency metadata/lockfile files and did not expand beyond dependency maintenance. The PR is otherwise ready from this review.

@thymikee
thymikee force-pushed the agent/security-dependabot-fixes branch from 6f7c50b to 2bbca74 Compare July 27, 2026 12:37
@thymikee
thymikee marked this pull request as ready for review July 27, 2026 12:38
@thymikee

Copy link
Copy Markdown
Member Author

Dependency/security review clean at 2bbca749: patched versions satisfy the advisory floors, workspace overrides match both lockfiles, and the only React Router consumer is the green Rspress documentation build. Residual: Dependabot’s default-branch alert state will recalculate after merge. Ready for human review.

@thymikee thymikee added the ready-for-human Valid work that needs human implementation, judgment, or maintainer merge label Jul 27, 2026
@thymikee
thymikee merged commit 6dae319 into main Jul 27, 2026
27 checks passed
@thymikee
thymikee deleted the agent/security-dependabot-fixes branch July 27, 2026 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-human Valid work that needs human implementation, judgment, or maintainer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant