From e019912f29b7e7e4fb39884ce4177d787e1962c0 Mon Sep 17 00:00:00 2001 From: Jose Montes de Oca Date: Mon, 31 Aug 2026 21:04:17 -0400 Subject: [PATCH] ci: run the skill conformance gate on every pull request The gate is about to become a required status check on main. A required check that is path-filtered never reports on a pull request outside the filter, and that pull request then waits forever on a check that will never arrive. The suites read Markdown off disk with no build step, no model key, and no network, so running them unconditionally costs well under a minute. --- .github/workflows/ci-skill.yml | 25 ++++++++----------------- 1 file changed, 8 insertions(+), 17 deletions(-) diff --git a/.github/workflows/ci-skill.yml b/.github/workflows/ci-skill.yml index 33015345..03202d53 100644 --- a/.github/workflows/ci-skill.yml +++ b/.github/workflows/ci-skill.yml @@ -1,25 +1,15 @@ name: CI - Skill conformance -# Runs the OpenProse conformance suites under tests/open-prose/** on every PR -# that touches the skill, the spec, the example corpus, the std/co contract -# libraries, or the gate's own wiring. The suites read Markdown off disk and -# make string assertions: no build step, no model key, no network. +# Runs the OpenProse conformance suites under tests/open-prose/**. The suites +# read Markdown off disk and make string assertions: no build step, no model +# key, no network, well under a minute. +# +# There is deliberately no path filter. This job is the required status check +# on main, and a required check that never reports leaves a pull request +# waiting on it forever — so every pull request runs it, whatever it touches. on: pull_request: - paths: - - "skills/**" - - "spec/**" - - "tests/**" - - "packages/std/**" - - "packages/co/**" - - "README.md" - - "CONTRIBUTING.md" - - "package.json" - - "pnpm-lock.yaml" - - "pnpm-workspace.yaml" - - "vitest.config.ts" - - ".github/workflows/ci-skill.yml" push: branches: [main] @@ -32,6 +22,7 @@ permissions: jobs: skill: + # NOTE: this job name is the required status check on main — do not rename. name: CI - Skill conformance runs-on: ubuntu-latest timeout-minutes: 10