From 61f53c2466f56f6547acdbbce66722a2740cb6c6 Mon Sep 17 00:00:00 2001 From: Copilot Date: Sun, 5 Apr 2026 13:41:43 +0300 Subject: [PATCH] =?UTF-8?q?fix:=20insider=20publish=20workflow=20=E2=80=94?= =?UTF-8?q?=20add=20build=20step=20and=20use=20Node=2022?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The insider publish workflow had two issues causing 80+ test failures: 1. The test job ran on a separate VM from the build job with no artifact transfer. Since dist/ is gitignored, the test job had no compiled output. Vitest failed to resolve workspace package exports (e.g. @bradygaster/squad-sdk entry, squad-cli/commands/rc) because the dist files they point to didn't exist. 2. All three jobs used Node 20, but both packages declare engines.node >= 22.5.0. The CLI's version gate rejected Node 20 at runtime, causing additional failures. Fix: Add 'npm run build' step to the test job, and update all jobs from node-version [20] to [22] to match the engines requirement and the main CI workflow. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .github/workflows/squad-insider-publish.yml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/squad-insider-publish.yml b/.github/workflows/squad-insider-publish.yml index 9903141d4..b5fa557f6 100644 --- a/.github/workflows/squad-insider-publish.yml +++ b/.github/workflows/squad-insider-publish.yml @@ -14,7 +14,7 @@ jobs: timeout-minutes: 10 strategy: matrix: - node-version: [20] + node-version: [22] steps: - uses: actions/checkout@v4 @@ -33,7 +33,7 @@ jobs: timeout-minutes: 15 strategy: matrix: - node-version: [20] + node-version: [22] steps: - uses: actions/checkout@v4 @@ -41,6 +41,9 @@ jobs: with: node-version: ${{ matrix.node-version }} + - name: Build + run: npm run build + - name: Run tests run: npm test @@ -81,7 +84,7 @@ jobs: timeout-minutes: 10 strategy: matrix: - node-version: [20] + node-version: [22] steps: - uses: actions/checkout@v4