chore(deps): security round 4 — postcss, tar - #456
Merged
Conversation
Clears 3 advisories (2 high) that landed after round 3; lockfile-only, no manifest changes: - postcss -> 8.5.25 (alerts #179, #182; advisory floor 8.5.18) in all three lockfiles - tar -> 7.5.22 (alert #180; floor 7.5.21) root + frontend The 2 remaining high alerts are react-router, which requires a 7 -> 8 MAJOR. CLAUDE.md lists React Router among the majors that need explicit founder approval, so it is deliberately NOT included here. Verified: backend lint/build/tests 3541 passed; frontend lint 0 errors, build clean, 906 passed.
Deploying studyhub with
|
| Latest commit: |
20c56e7
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://8a0ee28f.studyhub-2wc.pages.dev |
| Branch Preview URL: | https://chore-security-round4.studyhub-2wc.pages.dev |
Contributor
Reviewer's guide (collapsed on small PRs)Reviewer's GuideLockfile-only dependency upgrades to postcss and tar across all lockfiles, plus a release-log entry documenting the security round and explicitly deferring the react-router major bump. Flow diagram for security dependency round 4 decisionsflowchart TD
A[Identify security advisories] --> B[Check postcss version]
B --> C[Upgrade postcss to 8.5.25 in all lockfiles]
A --> D[Check tar version]
D --> E[Upgrade tar to 7.5.22 in root and frontend lockfiles]
A --> F[Check react-router advisories]
F --> G{Is major bump required?}
G -->|Yes, 7→8 major| H[Defer react-router upgrade for founder sign-off]
H --> I[Document deferral and security round in release-log.md]
C --> I
E --> I
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Contributor
There was a problem hiding this comment.
Hey - I've reviewed your changes and they look great!
Fixed security issues:
- postcss (link)
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
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.
Clears 3 advisories (2 high) that landed after round 3. Lockfile-only — no manifest changes.
Not included: react-router (deliberate)
The 2 remaining high alerts (#176, #177) are
react-router, and the fix is 8.3.0 — a major bump from 7.18.1. CLAUDE.md lists React Router among the majors that require an explicit founder approval, so it is held out of this PR. It touches every route in the app and deserves its own PR with a full route smoke pass. Dependabot has it open as #450/#451.Also open and awaiting the same call: #446 recharts 2→3, #447 @vitejs/plugin-react 5→6, #448 @testing-library/jest-dom 6→7, #449 rollup-plugin-visualizer 6→7 — all majors.
Validation
playwright-smokeremains the known-red baseline (red on main since 2026-06-02)🤖 Generated with Claude Code
Summary by Sourcery
Update dependency lockfiles to address recent security advisories for postcss and tar, and document the changes in the release log.
Enhancements:
Chores: