From a4445e39a4ffe37a01a24779ab94d228ce9daf2e Mon Sep 17 00:00:00 2001 From: SToneX Date: Sun, 27 Sep 2026 18:13:04 +0800 Subject: [PATCH] ci: run the catalog validator scripts/catalog_validate.py failed on main unnoticed because CI never ran it; only the unit tests exercised pieces of it. Run it after the import-boundary check, under the same docs-only skip, so a catalog data PR that breaks a rule fails the build. --- .github/workflows/ci.yml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 63a64fee..a0b43bc1 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -61,6 +61,10 @@ jobs: - name: Enforce import boundaries if: steps.docs.outputs.only != 'true' run: uv run --locked lint-imports + - name: Validate the catalog + if: steps.docs.outputs.only != 'true' + # The whole catalog, not the slices the tests exercise: a catalog data PR is gated here. + run: uv run --locked python scripts/catalog_validate.py - name: Run tests if: steps.docs.outputs.only != 'true' # -n auto: the suite is 2,700+ small independent tests — serial execution is what let a