[ef-24] fix: detect missing server.js before spawn + bump to 0.0.1-beta.3#24
Conversation
Replaces the raw Node MODULE_NOT_FOUND crash with an explicit existsSync check in scripts/launch.ts. If .next/standalone/server.js is absent the CLI now prints a clear, actionable error and exits 1 instead of throwing a stack trace. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Free Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughBumps package version to Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Poem
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
npm publish --ignore-scripts bypassed prepublishOnly, meaning the only thing ensuring .next/standalone/server.js was present before publishing was the explicit CI build step — fragile against workflow edits or a manual publish. Dropping --ignore-scripts makes bun run build the package's own invariant. Bun is still installed when npm publish runs. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Summary
existsSynccheck inscripts/launch.tsbefore spawningnode server.js.next/standalone/server.jsis absent, prints a clear actionable error and exits 1 instead of throwing a rawMODULE_NOT_FOUNDstack trace0.0.1-beta.3Test plan
.next/standalone/server.js→.next/standalone/server.js.bak, runfailproofai, confirm friendly error message appears and exit code is 1npm uninstall failproofai— confirm hooks are cleaned up (preuninstall unaffected)🤖 Generated with Claude Code
Summary by CodeRabbit
Chores
Bug Fixes