mirror of
https://github.com/Memo-2023/mana-monorepo.git
synced 2026-05-14 21:21:10 +02:00
d1d37749f7
The root postinstall was `node scripts/generate-env.mjs || true && pnpm run build:packages || true`. Two failures were being swallowed: 1. shared-auth's build has been broken for a while. shared-types re-exports its submodules with explicit `.ts` extensions (`export * from './theme.ts'`), which only works for downstream consumers that set `allowImportingTsExtensions: true`. shared-auth didn't — tsc emitted TS5097 on every re-export, the build failed, `|| true` hid it, every `pnpm install` appeared clean. 2. The filter `@mana/*` matches everything in the workspace, including `@mana/web` — the full 27-module SvelteKit build. On postinstall this kicked off vite, which OOM-aborted during SW generation. That's the original reason `|| true` was added, judging by shape. Fixes: - Dropped the `.ts` suffix from shared-types/src/index.ts re-exports. shared-types is consumed in bundler-mode tsconfigs everywhere, so no extension is the portable form. shared-types' own `tsc --noEmit` still passes. - Narrowed the filter from `@mana/*` (name-glob, matches apps) to `./packages/*` (path-glob, only workspace packages). Scope drops from 133 → 39 projects; build:packages now runs cleanly in ~15s. - Removed both `|| true` guards. A broken postinstall now fails loudly instead of producing a half-built state nobody notices. Verified: `pnpm install` completes exit 0 in 13s; all 39 packages build green. Closes audit item #37 (postinstall swallows errors). Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| src | ||
| package.json | ||
| tsconfig.json | ||