[ephbaum]

📝 What's This PR About?

Type of Change:

• 🏗️ Infrastructure - Build, deployment, or tooling changes

📋 Description

Two commits. The first makes a broken install fail the build instead of silently skipping the type check. The second fixes the broken install on Vercel that the first commit exposed straight away.

For Technical Changes:

• What changed:
  • astro:check is now astro sync && astro-check, replacing astro check. build, check and check:fix call pnpm run astro:check, so the command is defined once.
  • Vercel's installCommand is now rm -rf node_modules && pnpm install --frozen-lockfile, replacing pnpm install.
  • Docs and comments updated to match.
• Why: both are explained below.
• Impact: package.json scripts, vercel.json, and wording in CLAUDE.md, README.md, the ci.yml comment and the session-start hook comment. No change to what gets built.

f4360f7: astro-check instead of astro check

If @astrojs/check fails to import, astro check logs [ERROR] and then exits 0: a non-boolean return falls through to process.exit(0) in astro/dist/cli/index.js. Under CI (GitHub Actions and Vercel both count), ci-info skips the install prompt entirely. So a broken install turns the type-check gate into a silent no-op.

astro-check is the bin that ships with @astrojs/check. It imports the package statically, so the same breakage crashes with ERR_MODULE_NOT_FOUND, names the actual missing package, and exits 1.

The astro sync keeps parity, since astro check syncs before checking. Without it, a fresh checkout with no .astro/ reports 4 false errors.

Second commit: clean-install node_modules on Vercel

The first push failed on Vercel within seconds, and that failure was real. Vercel restores node_modules from its build cache and ran a plain incremental pnpm install.

pnpm records the optional packages it skipped in node_modules/.modules.yaml, and keeps skipping them on later installs even after a lockfile change makes them required:

• before Astro 7.4, @emnapi/[email protected] was optional (wasm32 bindings only), so linux skipped it
• after Astro 7.4, it's a hard dependency of @astrojs/astro2tsx

So cached Vercel deploys came up without it, and @astrojs/check couldn't load. With the old astro check that went unnoticed, which means #107's Vercel builds most likely deployed without type-checking. GitHub CI, which always installs clean, did type-check #107 and reported 0 errors, so nothing bad shipped.

I reproduced it locally by installing the pre-7.4 lockfile, then installing this one on top:

Install over a stale tree	@emnapi/runtime	@astrojs/check imports
pnpm install (old Vercel command)	missing	❌
pnpm install --frozen-lockfile	missing	❌
pnpm install --force	present	✅, but installs every platform's optional binaries (19 @oxc-parser bindings)
rm -rf node_modules && pnpm install --frozen-lockfile	present	✅, only this platform's binaries

🖼️ Visual Changes

• No visual changes

🧪 Testing

• Tested locally with pnpm run dev
• Built successfully with pnpm run build
• Preview deployment looks good
• No console errors
• Mobile responsive (if UI changes)

All runs used CI=true to match GitHub Actions and Vercel:

Scenario	Before (astro check)	After (astro sync && astro-check)
@emnapi/core removed from node_modules	logs [ERROR], exit 0	ERR_MODULE_NOT_FOUND: @emnapi/core, exit 1; build also exits 1
Healthy install	Result (36 files), 0 errors	Result (36 files), 0 errors, exit 0
Type error injected	n/a	ts(2322), Result (37 files): 1 error, exit 1
No .astro/ directory	n/a	0 errors (bare astro-check without sync: 4 false errors)

pnpm run build and pnpm run check pass: 215 pages, lint and format clean. That holds both from a clean install and from the stale tree after running the new Vercel install command.

📚 Documentation

• README.md updated (explain what changed)

• README: the command table, CI section and mermaid diagram describe the new script.

• CLAUDE.md: records why it isn't plain astro check, why the astro sync matters, and why Vercel deletes node_modules before installing.

🔗 Related

• Follow-up to chore(deps): roll up 12 Dependabot bumps (Astro 7.4, satori 0.63, prettier-plugin-astro 1.1) #107
• Upstream, not yet filed:
  • astro check exits 0 when @astrojs/check is missing (Astro CLI)
  • pnpm keeps skipping a previously-optional package after it becomes required, under node-linker=hoisted

✅ Checklist

• Code follows project conventions
• No linting errors
• Preview deployment works correctly
• Ready for review

🤖 Generated with Claude Code

https://claude.ai/code/session_011rvCtaiKiC4ZSvX9yewguJ

[vercel]

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

Project	Deployment	Actions	Updated
ephbaumdotdev	Ready	Preview	Oct 23, 2026 5:01am UTC