Skip to content

fix: regenerate @astrojs/vercel patch for v9.0.4#388

Merged
0xmaayan merged 1 commit intomainfrom
fix/regenerate-vercel-adapter-patch
Mar 2, 2026
Merged

fix: regenerate @astrojs/vercel patch for v9.0.4#388
0xmaayan merged 1 commit intomainfrom
fix/regenerate-vercel-adapter-patch

Conversation

@0xmaayan
Copy link
Contributor

@0xmaayan 0xmaayan commented Mar 2, 2026

Summary

  • Regenerates the @astrojs/vercel patch to work with v9.0.4 (previously written against v9.0.2)
  • The deps update in Update all deps #382 bumped the adapter version, causing the dist/index.js hunk of the patch to silently fail while the .d.ts hunk still applied — so cspMode: "global" passed type-checking but was ignored at runtime
  • Without the patch, every static page gets its own CSP route entry in Vercel's config.json, exceeding the 5MB deployment body limit

Deploy is failing on Vercel https://vercel.com/aptoslabs/aptos-docs/8iC9BK3YT8LJ1x2SYN8qfHcnk6UG

Test plan

  • Verify Vercel preview deployment succeeds (no "Body exceeded 5mb limit" error)
  • Verify CSP headers are still present on deployed pages

Made with Cursor

The deps update in #382 bumped @astrojs/vercel from 9.0.2 to 9.0.4,
which changed dist/index.js enough that the cspMode patch no longer
applied. The .d.ts hunk still matched so TypeScript accepted the
config, but the runtime JS was unpatched — every static page got its
own CSP route entry in config.json, exceeding Vercel's 5MB body limit.

Regenerated the patch against 9.0.4 so the global cspMode collapses
all per-route CSP headers into a single catch-all route.

Made-with: Cursor
@vercel
Copy link

vercel bot commented Mar 2, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
aptos-docs Building Building Preview, Comment Mar 2, 2026 9:36pm

Request Review

@0xmaayan 0xmaayan merged commit 4eb85aa into main Mar 2, 2026
7 checks passed
@0xmaayan 0xmaayan deleted the fix/regenerate-vercel-adapter-patch branch March 2, 2026 22:20
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