Commit ac85c38
committed
refactor(build): consolidate build chain into pnpm build script
The justfile's build recipe and the root package.json's build script
had drifted: the justfile knew it needed to build rolldown's native
binding, rolldown's JS, and vite's types before the @voidzero-dev/*
chain, while package.json's build only ran the @voidzero-dev/* +
vite-plus step. That meant pnpm bootstrap-cli (which calls pnpm build)
silently produced an incomplete tree on fresh checkouts — packages/
core/dist/vite/node/index.d.ts wouldn't exist, and vp check then
failed with TS7016 on every @voidzero-dev/vite-plus-core import.
Move the upstream prerequisites into the pnpm build script and have
the justfile recipe just call pnpm build, so the build sequence has a
single source of truth. CI is unaffected: bootstrap-cli:ci runs only
pnpm install-global-cli, and the per-target build-upstream action
keeps its own chain (it skips build-binding:release because the
binding is pulled from npm, and runs build-ts instead of build because
NAPI bindings are built and cached as a separate step).1 parent 8172f08 commit ac85c38
2 files changed
Lines changed: 2 additions & 8 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
42 | 42 | | |
43 | 43 | | |
44 | 44 | | |
45 | | - | |
46 | | - | |
47 | | - | |
48 | | - | |
49 | | - | |
50 | | - | |
51 | | - | |
| 45 | + | |
52 | 46 | | |
53 | 47 | | |
54 | 48 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
0 commit comments