chore(deps): bump postcss from 8.4.31 to 8.5.23 in /site in the npm_and_yarn group across 0 directory - #131
Conversation
Updates `postcss` from 8.4.31 to 8.5.23 - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](postcss/postcss@8.4.31...8.5.23) --- updated-dependencies: - dependency-name: postcss dependency-version: 8.5.23 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
|
🤖 Automated message from Kaan's Automated Triage Bot. 👀 Picked this up — a review will follow shortly. |
There was a problem hiding this comment.
Pull request overview
This PR updates PostCSS but also introduces an unrelated Next.js 15-to-16 upgrade.
Changes:
- Bumps PostCSS from 8.4.31 to 8.5.23.
- Updates Next.js to 16.3.5 and related dependencies.
- Leaves unresolved peer and lint-script compatibility issues.
Reviewed changes
Copilot reviewed 1 out of 2 changed files in this pull request and generated 2 comments.
| File | Summary |
|---|---|
site/pnpm-lock.yaml |
Updates PostCSS and locks @next/third-parties@15.5.18 with Next.js 16.3.5, causing a peer mismatch (moderate, 3 votes). |
site/package.json |
Upgrades Next.js and retains the removed next lint command, breaking lint CI (critical, 3 votes). |
Files not reviewed (1)
- site/pnpm-lock.yaml: Generated file
Suppressed comments (1)
site/package.json:24
- This line changes the direct Next dependency from 15.5.24 to the unrelated major version 16.3.5, while this PR is only supposed to update PostCSS. The lockfile consequently replaces the entire Next 15/SWC dependency set, introducing a framework upgrade and its breaking changes; please restore Next 15.5.24 and regenerate the lockfile so only the PostCSS resolution changes.
"next": "16.3.5",
Note
Copilot is running an experiment and ran this review at Lite.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| "@next/third-parties": "^15.5.7", | ||
| "@stellar/design-system": "^3.2.7", | ||
| "next": "15.5.24", | ||
| "next": "16.3.5", |
| "@next/third-parties": "^15.5.7", | ||
| "@stellar/design-system": "^3.2.7", | ||
| "next": "15.5.24", | ||
| "next": "16.3.5", |
|
🤖 Automated message from Kaan's Automated Triage Bot. Thanks Dependabot. Flagging a mismatch: the title says postcss 8.4.31 to 8.5.23, but the diff is a Next.js major upgrade, I did not merge. Two blockers at this head:
Copilot raised both points. I verified both and agree. There is also no build evidence for Next 16. The run stopped at lint, so @kaankacar this needs your decision. The postcss advisories are real: GHSA-6g55-p6wh-862q, GHSA-r28c-9q8g-f849 and GHSA-fxqj-rqcc-2cmp, all fixed in 8.5.23. Two paths:
I left the PR open for you. |
Updates
postcssfrom 8.4.31 to 8.5.23Release notes
Sourced from postcss's releases.
... (truncated)
Changelog
Sourced from postcss's changelog.
... (truncated)
Commits
eb9e1feRelease 8.5.23 version9d19c78Update dependencies7beca13Does no load source map file without opts.fromdecea51Typoc18e30dUpdate EM banner98a39adUpdate EM bannera3e48c4Release 8.5.22 versionf49d691Fix custom property losing its semicolon before a comment (#2117)28e0dafRelease 8.5.21 version3d2b4e4Update dependenciesMaintainer changes
This version was pushed to npm by GitHub Actions, a new releaser for postcss since your current version.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.