ci: align melonjs prepublishOnly hook with publish workflow (#1498)

obiot · claude · web-flow · commit fe9d160c247b · 2026-06-09T07:55:19.000+08:00
#1497 swapped the publish workflow's build_cmd from `pnpm dist` to `pnpm dist:publish` (drops the redundant `pnpm vitest run` — main.yml gates tests now). But `npm publish` ALSO fires the package's own `prepublishOnly` hook, which still hardcoded `pnpm dist`. So the workflow built cleanly with the new recipe and then `npm publish` re-triggered `pnpm dist` → `pnpm vitest run` → died on missing Playwright browsers. Point `prepublishOnly` at the same `dist:publish` script the workflow uses. Keeps the safety net for local manual `npm publish` (build before shipping) without dragging the broken vitest step back in. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
diff --git a/packages/melonjs/package.json b/packages/melonjs/package.json
@@ -84,7 +84,7 @@
 		"doc": "typedoc src/index.ts --tsconfig tsconfig.build.json --readme DOC_README.md --hideGenerator --name 'melonJS' --navigation.includeCategories true --categorizeByGroup false",
 		"doc:watch": "typedoc src/index.ts --tsconfig tsconfig.build.json --readme DOC_README.md --hideGenerator --name 'melonJS' --navigation.includeCategories true --categorizeByGroup false --watch --skipErrorChecking --preserveWatchOutput --logLevel Error",
 		"serve": "serve docs",
-		"prepublishOnly": "pnpm dist",
+		"prepublishOnly": "pnpm dist:publish",
 		"clean": "tsx scripts/clean.ts",
 		"types": "tsc --project tsconfig.build.json",
 		"test:types": "tsc"
