Skip to content

Update - Upgrade Astro to 7.2 and remove unused session KV binding - #743

Merged
markmead merged 1 commit into
mainfrom
update/astro-7.2
Aug 19, 2026
Merged

Update - Upgrade Astro to 7.2 and remove unused session KV binding#743
markmead merged 1 commit into
mainfrom
update/astro-7.2

Conversation

@markmead

Copy link
Copy Markdown
Owner

Summary

  • Bumps astro 7.0.3 → 7.2.3 and @astrojs/mdx 7.0.0 → 7.0.6 (the newest patch versions available given pnpm's minimumReleaseAge policy — 7.2.4/7.0.7 were published within the last 24h)
  • Fixes an existing peer dependency mismatch: the installed @astrojs/cloudflare@14.2.2 already declares peerDependencies: { astro: "^7.2.0" }, which astro@7.0.3 violated
  • Removes the unused SESSION KV binding from wrangler.jsonc — nothing in src calls Astro.session
  • Regenerates worker-configuration.d.ts accordingly
  • Skips the new experimental.incrementalBuild flag from 7.2 for now — it's still pre-stable, its benefit depends on the build cache persisting across Cloudflare Pages builds (unconfirmed), and the current build is already ~4s for all 88 pages, so there's no bottleneck it would solve

Supersedes #739 (Dependabot's astro 7.1.1 bump).

Test plan

  • pnpm run check — 0 errors
  • pnpm build — clean, 88 pages built
  • pnpm lint — clean
  • pnpm test — 114 passed, 6 skipped, 0 failed

🤖 Generated with Claude Code

astro@7.0.3 was already violating the peer dependency declared by the
installed @astrojs/cloudflare adapter (astro: ^7.2.0). Bumps astro and
@astrojs/mdx, and drops the SESSION KV binding from wrangler.jsonc
since nothing in the codebase uses Astro.session.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

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 updates the site’s build/runtime dependencies to align with the currently installed @astrojs/cloudflare peer requirements, and removes an unused Cloudflare Workers KV binding from the Wrangler configuration (with regenerated Wrangler type output to match).

Changes:

  • Bump astro from 7.0.3 to 7.2.3 and @astrojs/mdx from 7.0.0 to 7.0.6, updating the lockfile accordingly.
  • Remove the unused SESSION KV namespace binding from wrangler.jsonc.
  • Regenerate worker-configuration.d.ts so the Env interface reflects the current Wrangler bindings (dropping SESSION).

Reviewed changes

Copilot reviewed 2 out of 4 changed files in this pull request and generated no comments.

File Description
wrangler.jsonc Removes the unused SESSION KV binding from the Worker configuration.
worker-configuration.d.ts Regenerated Wrangler types; Env no longer includes SESSION and runtime typings are updated.
package.json Updates astro and @astrojs/mdx dependency versions.
pnpm-lock.yaml Updates the dependency graph/lock state to match the new Astro + MDX versions.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@markmead
markmead merged commit 583203f into main Aug 19, 2026
2 checks passed
@markmead
markmead deleted the update/astro-7.2 branch August 19, 2026 17:07
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