From 9f658bb04dd8599ed5fa962625b09704d1419e4e Mon Sep 17 00:00:00 2001 From: Ari Mayer Date: Fri, 31 Jul 2026 14:30:18 -0400 Subject: [PATCH] ci: run the core test suite on pull requests MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit validate.yml runs bun validate and the SDK suite, but the 111 core tests — including the repository-wide data invariants (base_model hygiene, metadata leakage, weights-link coverage) — never execute in CI, so data regressions those tests guard against land silently. Add the missing step and correct the AGENTS.md claim that no test framework exists. Depends on the open-weights metadata fixes landing first so the weights-link invariant passes on dev. Co-Authored-By: Claude Fable 5 --- .github/workflows/validate.yml | 4 ++++ AGENTS.md | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/.github/workflows/validate.yml b/.github/workflows/validate.yml index d96ecdd4f8..f79d86b704 100644 --- a/.github/workflows/validate.yml +++ b/.github/workflows/validate.yml @@ -24,6 +24,10 @@ jobs: - name: Run validation script run: bun validate + - name: Core tests + run: bun test + working-directory: packages/core + - name: SDK tests run: bun run test working-directory: packages/sdk diff --git a/AGENTS.md b/AGENTS.md index cd4f815d3f..5b7e79fc39 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -4,7 +4,7 @@ - **Validate**: `bun validate` - Validates all provider/model configurations - **Build web**: `cd packages/web && bun run build` - Builds the web interface - **Dev server**: `cd packages/web && bun run dev` - Runs development server -- **No test framework** - No dedicated test commands found +- **Test**: `bun test` - Runs the core catalog tests (`packages/core/test`, includes repository-wide data invariants) and SDK unit tests; `cd packages/sdk && bun run test` additionally runs SDK codegen + typecheck ## Code Style - **Runtime**: Bun with TypeScript ESM modules