Skip to content

Commit 7771392

Browse files
claudetuler
authored andcommitted
fix(ci): build the workspace packages before the machine test
The machine workflow's end-to-end test failed at "==> encoding inputs" with ERR_MODULE_NOT_FOUND for @cartesi/codec/dist/index.js. test/machine/{encode-inputs,verify-outputs}.mjs import @cartesi/codec, a workspace package built from source, but nothing built it: `pnpm i` does not build, and run.sh's `pnpm run build` invokes @cartesi/rollup's own tsdown script, which covers only its own dist. The ci workflow is unaffected because `pnpm test` goes through turbo, whose test:run depends on build, which depends on ^build. Build the dependency graph in the workflow, before the test runs, leaving run.sh to assume its packages are already built. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Bp4RNPL7aZprKdXzcuJ98s
1 parent e00fb19 commit 7771392

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

.github/workflows/machine.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,9 @@ jobs:
3838
- name: Install Dependencies
3939
run: pnpm i
4040

41+
- name: Build
42+
run: pnpm build --filter @cartesi/rollup
43+
4144
# libcmt bundles the `cmio` ioctl ABI since machine-guest-tools
4245
# 0.18.0, so the Cartesi Linux headers are no longer needed here.
4346
- name: Install riscv64 cross toolchain

0 commit comments

Comments
 (0)