Skip to content

chore(deps): security round 4 — postcss, tar - #456

Merged
Apexone11 merged 1 commit into
mainfrom
chore/security-round4
Aug 2, 2026
Merged

chore(deps): security round 4 — postcss, tar#456
Apexone11 merged 1 commit into
mainfrom
chore/security-round4

Conversation

@Apexone11

@Apexone11 Apexone11 commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Clears 3 advisories (2 high) that landed after round 3. Lockfile-only — no manifest changes.

Alert Package Advisory floor Now at
#179, #182 (high) postcss 8.5.18 8.5.25 (all 3 lockfiles)
#180 (medium) tar 7.5.21 7.5.22 (root + frontend)

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

  • Backend: lint ✅ · build ✅ · tests ✅ 3541 passed
  • Frontend: lint ✅ 0 errors · build ✅ · 906 passed
  • Release-log entry added (CI gate)
  • playwright-smoke remains 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:

  • Record security round 4 dependency updates and remaining react-router advisories in the v2.3.0 release log entry.

Chores:

  • Upgrade postcss to 8.5.25 and tar to 7.5.22 across all lockfiles to clear three new security advisories.

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.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying studyhub with  Cloudflare Pages  Cloudflare Pages

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

View logs

@sourcery-ai

sourcery-ai Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor
Reviewer's guide (collapsed on small PRs)

Reviewer's Guide

Lockfile-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 decisions

flowchart 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
Loading

File-Level Changes

Change Details Files
Upgrade postcss to 8.5.25 in all lockfiles to clear high-severity advisories.
  • Update resolved version of postcss to 8.5.25 in the root lockfile.
  • Update resolved version of postcss to 8.5.25 in the backend lockfile.
  • Update resolved version of postcss to 8.5.25 in the frontend app lockfile.
  • Adjust transitive dependency graph entries that reference the old postcss version.
package-lock.json
backend/package-lock.json
frontend/studyhub-app/package-lock.json
Upgrade tar to 7.5.22 in the root and frontend lockfiles to address a medium-severity advisory.
  • Update resolved version of tar to 7.5.22 in the root lockfile.
  • Update resolved version of tar to 7.5.22 in the frontend app lockfile.
  • Adjust transitive dependency graph entries that reference the old tar version.
package-lock.json
frontend/studyhub-app/package-lock.json
Document security round 4 dependency updates and the deliberate exclusion of the react-router major bump in the release log.
  • Add a v2.3.0 release-log line summarizing the postcss and tar upgrades and the advisories they clear.
  • Note that react-router’s remaining high alerts require a 7→8 major and are deferred pending founder sign-off.
docs/release-log.md

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hey - I've reviewed your changes and they look great!

Fixed security issues:


Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

@Apexone11
Apexone11 merged commit 5394a33 into main Aug 2, 2026
10 of 11 checks passed
@Apexone11
Apexone11 deleted the chore/security-round4 branch August 2, 2026 04:09
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