Skip to content

chore(deps): stack passing Dependabot bumps (@astrojs/preact 6, @primer/react-brand 0.69, @types/node 26)#113

Merged
jldeen merged 1 commit into
mainfrom
jldeen-silver-funicular
Jun 24, 2026
Merged

chore(deps): stack passing Dependabot bumps (@astrojs/preact 6, @primer/react-brand 0.69, @types/node 26)#113
jldeen merged 1 commit into
mainfrom
jldeen-silver-funicular

Conversation

@jldeen

@jldeen jldeen commented Jun 24, 2026

Copy link
Copy Markdown
Collaborator

Stacks the three passing open Dependabot PRs into a single branch with one consolidated package-lock.json:

Verification (local, Node 22 to match CI)

Step Result
npm run check (astro check) ✅ 0 errors
npm run build (astro build)
npm run test (Playwright) ✅ 1 passed

Each bump was also verified individually on its own branch before stacking, then re-verified together on this consolidated branch.

Excluded: #112 (astro 6.4.8 → 7.0.0)

Astro 7 depends on Vite 8 (rolldown). The repo's overrides.vite pin (^7) was the first blocker (rollupOptions.input should not be an html file when building for SSR); bumping it to ^8 gets past that, but Vite 8's built-in postcss-import then fails to resolve Tailwind v4's @import "tailwindcss" (ENOENT … open '…/tailwindcss'). That's a real toolchain migration (Vite 8 / rolldown + Tailwind), not a safe dependency bump, so #112 is left open for dedicated follow-up.

Supersedes #109, #110, #111.

…/node 26

Stacks three passing Dependabot bumps (#110, #109, #111) into one branch
with a single consolidated lockfile. Verified locally with Node 22:
astro check, astro build, and Playwright tests all pass.

Excludes #112 (astro 7): it pulls Vite 8 (rolldown), whose built-in
postcss-import cannot resolve Tailwind v4's @import "tailwindcss",
which needs a dedicated migration.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jldeen jldeen requested a review from Damovisa as a code owner June 24, 2026 20:12
Copilot AI review requested due to automatic review settings June 24, 2026 20:12
GitHub Advanced Security started work on behalf of jldeen June 24, 2026 20:12 View session
GitHub Advanced Security finished work on behalf of jldeen June 24, 2026 20:12

Copilot AI 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.

Pull request overview

This PR consolidates three passing Dependabot dependency bumps into a single branch, updating package.json and producing one unified package-lock.json for the repo’s Node/Astro toolchain.

Changes:

  • Bump @astrojs/preact from ^5.1.5 to ^6.0.0.
  • Bump @primer/react-brand from ^0.68.0 to ^0.69.0.
  • Bump @types/node from ^25.9.3 to ^26.0.0 and update the lockfile accordingly.
Show a summary per file
File Description
package.json Updates the three dependency/devDependency version ranges.
package-lock.json Re-locks dependencies for the consolidated bumps (including transitive updates like Vite and undici-types).

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/2 changed files
  • Comments generated: 1

Comment thread package.json
Comment on lines +14 to +15
"@astrojs/preact": "^6.0.0",
"@primer/react-brand": "^0.69.0",
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.

2 participants