Skip to content

fix(website): split npm ci from postinstall to avoid npm crash#47

Merged
heggria merged 2 commits into
mainfrom
fix/website-deploy-split-steps
Jul 6, 2026
Merged

fix(website): split npm ci from postinstall to avoid npm crash#47
heggria merged 2 commits into
mainfrom
fix/website-deploy-split-steps

Conversation

@heggria

@heggria heggria commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Follow-up to #45. The first deploy-website run after the pnpm-migration fix failed:

npm error Exit handler never called!
npm error This is an error with npm itself.

during npm ci (8 minutes, no output, then an internal npm crash — likely the postinstall: fumadocs-mdx handler tripping npm's exit machinery).

Fix

Split the install into three visible steps so a crash is diagnosable and npm's postinstall handler is bypassed:

Step Command
Install dependencies npm ci --ignore-scripts
Generate MDX source npm run prepare-source (fumadocs-mdx, ~10ms)
Build npm run build (next build)

Verified locally

npm ci --ignore-scripts + npm run prepare-source + TASKFLOW_BASE_PATH=/taskflow npm run build all succeed; the build emits static HTML for en + zh-cn.

heggria added 2 commits July 6, 2026 20:22
The first deploy-website run after the pnpm-migration fix failed with
"npm error Exit handler never called!" during `npm ci` (8 min, no output,
then an internal npm crash — likely the postinstall fumadocs-mdx handler
tripping npm's exit machinery).

Split the install into three visible steps so a crash is diagnosable and
npm's postinstall handler is bypassed:
- `npm ci --ignore-scripts` (install only)
- `npm run prepare-source` (fumadocs-mdx, ~10ms, its own step)
- `npm run build` (next build)

Verified locally: ci --ignore-scripts + prepare-source + build all succeed.
@heggria heggria merged commit 381ffb9 into main Jul 6, 2026
6 checks passed
@heggria heggria deleted the fix/website-deploy-split-steps branch July 6, 2026 12:28
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.

1 participant