From 2b3de23e4550417cf647a89c04b729a6e44bd7cf Mon Sep 17 00:00:00 2001 From: engineer Date: Wed, 3 Jun 2026 14:24:11 -0700 Subject: [PATCH] ci: run typecheck + unit tests on push/PR MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit build.yml only built the APK — the 65 unit tests and typecheck never ran in CI, so regressions in the covered logic (headers/SSE/diagnostics/settings/etc.) could land silently. Add a fast 'test' job (Node 24, native TS test-running) so every push and PR enforces typecheck + npm test before merge. Co-Authored-By: Claude Opus 4.8 --- .github/workflows/build.yml | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 8ebfa77..bc18846 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -8,6 +8,27 @@ on: branches: [main] jobs: + test: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v6 + + - uses: actions/setup-node@v6 + with: + # Node >= 23.6 runs the TypeScript test files natively (type-stripping), + # matching the local toolchain. No build step or extra deps required. + node-version: 24 + cache: npm + + - name: Install dependencies + run: npm install --legacy-peer-deps + + - name: Typecheck + run: npm run typecheck + + - name: Unit tests + run: npm test + build: runs-on: ubuntu-latest env: