From 0417791d360d9960ccb946375c9ae66dd013c330 Mon Sep 17 00:00:00 2001 From: JSONbored <49853598+JSONbored@users.noreply.github.com> Date: Sat, 11 Jul 2026 09:40:51 -0700 Subject: [PATCH] fix(ci): build the engine package before the self-host Postgres integration test The Postgres integration test's import graph now reaches @jsonbored/gittensory-engine transitively (#5117 moved local-write action specs there; src/mcp/local-write-tools.ts re-exports them from the engine package). That package's dist/ is gitignored and only exists after an explicit build step, which this workflow never had -- same #ci-engine-build-order class of bug ci.yml's "Build engine package" step already exists to prevent, just missed here. Reproduced locally (removed dist/, confirmed the exact "Failed to resolve entry for package" error; rebuilt, confirmed it resolves cleanly) before landing the fix. --- .github/workflows/selfhost.yml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/.github/workflows/selfhost.yml b/.github/workflows/selfhost.yml index 68a07f3c7..f8ff53ee7 100644 --- a/.github/workflows/selfhost.yml +++ b/.github/workflows/selfhost.yml @@ -74,6 +74,15 @@ jobs: - name: Install deps run: npm ci --ignore-scripts + # Same #ci-engine-build-order fix as ci.yml (see its "Build engine package" step): the Postgres + # integration test's import graph now reaches @jsonbored/gittensory-engine transitively (#5117 moved + # local-write action specs there; src/mcp/local-write-tools.ts re-exports them). That package's + # dist/ is gitignored and only exists after this build step, so the test fails to resolve the + # package's exports without it -- this workflow never needed the engine package built before, so it + # never had this step; it does now. + - name: Build engine package + run: npm run build --workspace @jsonbored/gittensory-engine + - name: Postgres integration test (real PG) run: PG_TEST_URL=postgres://postgres:devpw@localhost:5432/gittensory npx vitest run test/integration/selfhost-pg.test.ts