open-standards/.github/workflows/conformance.yml
Transaction Science 23d8d31f65
Some checks failed
smart-byte conformance gate / Validate conformance JSON files (pull_request) Has been cancelled
smart-byte conformance gate / Second implementation reproduces every vector (pull_request) Has been cancelled
smart-byte conformance gate / Spec changes require vector-pack updates (pull_request) Has been cancelled
smart-byte conformance gate / VERSIONS.md and README.md must exist (pull_request) Has been cancelled
joule-code conformance gate / Build + round-trip C1-C4 vectors (pull_request) Has been cancelled
jouleclaw public surface must stay documented / Public-surface audit (jouleclaw-rs) (pull_request) Has been cancelled
no-vanity-metrics / Scan committed Markdown for vanity metrics (pull_request) Has been cancelled
no-vanity-metrics / WAI public text names no comparison (pull_request) Has been cancelled
no-vanity-metrics / WAI public text names no company or product (pull_request) Has been cancelled
standards workspaces build + test / jouleclaw-rs (pull_request) Has been cancelled
standards workspaces build + test / aac-rs (pull_request) Has been cancelled
standards workspaces build + test / arl-rs (pull_request) Has been cancelled
standards workspaces build + test / eoc-rs (pull_request) Has been cancelled
standards workspaces build + test / jcp-rs (pull_request) Has been cancelled
standards workspaces build + test / joule-code-rs (pull_request) Has been cancelled
standards workspaces build + test / joule-ui-rs (pull_request) Has been cancelled
standards workspaces build + test / joulecontract-rs (pull_request) Has been cancelled
standards workspaces build + test / jouledesk-rs (pull_request) Has been cancelled
standards workspaces build + test / joulehook-rs (pull_request) Has been cancelled
standards workspaces build + test / map-rs (pull_request) Has been cancelled
standards workspaces build + test / mesh-rs (pull_request) Has been cancelled
standards workspaces build + test / openpay (pull_request) Has been cancelled
standards workspaces build + test / proof-rs (pull_request) Has been cancelled
standards workspaces build + test / sandbox-rs (pull_request) Has been cancelled
standards workspaces build + test / sense-rs (pull_request) Has been cancelled
standards workspaces build + test / smart-byte-rs (pull_request) Has been cancelled
standards workspaces build + test / macOS energy backends (pull_request) Has been cancelled
wai cross-architecture byte-equality / linux-aarch64 (pull_request) Has been cancelled
wai cross-architecture byte-equality / linux-x86_64 (pull_request) Has been cancelled
wai cross-architecture byte-equality / macos-arm64 (reference) (pull_request) Has been cancelled
WAI conformance / cargo-class corpora · linux (pull_request) Has been cancelled
WAI conformance / cargo-class corpora · linux-arm64 (pull_request) Has been cancelled
WAI conformance / cargo-class corpora · macos (pull_request) Has been cancelled
WAI conformance / v2 vocabulary · manifests validate as stated (pull_request) Has been cancelled
WAI conformance / independent verifiers · claims, JWP, key discovery and integer payloads (Python) (pull_request) Has been cancelled
WAI conformance / cross-arch byte-equality · linux (pull_request) Has been cancelled
WAI conformance / cross-arch byte-equality · linux-arm64 (pull_request) Has been cancelled
WAI conformance / cross-arch byte-equality · macos (pull_request) Has been cancelled
WAI conformance / C2PA manifest emission · signed, read back, validated (pull_request) Has been cancelled
WAI energy receipts / energy receipts · linux (pull_request) Has been cancelled
WAI energy receipts / energy receipts · linux-arm64 (pull_request) Has been cancelled
WAI energy receipts / energy receipts · macos (pull_request) Has been cancelled
wai formal verification (integer core) / kani (pull_request) Has been cancelled
WAI JPEG-AI whole decode (pinned inputs, required) / fetch pinned JPEG-AI inputs → whole decode + pixel hash, required (pull_request) Has been cancelled
WAI neural decoders end-to-end / synthetic fixtures → Rust decode → reference (±4) (pull_request) Has been cancelled
WAI neural sink gates / model-free refusals of the neural sink (pull_request) Has been cancelled
WAI section references / cited sections exist (pull_request) Has been cancelled
WAI mbt2018_mean integer decode (regenerated vectors) / regenerate mbt2018_mean integer vectors → Rust decode, required (pull_request) Has been cancelled
wai-web wasm32 / wai-web wasm32 build · corpus, menu and writers (pull_request) Has been cancelled
ci: one run per branch at a time — no workflow declared a concurrency group
Not one of this repo's 15 workflows declared `concurrency:`, so every push
queued a complete set of runs and nothing superseded anything. A session
pushing four versions of a branch queued four complete sets, and all of them
had to drain. On 2026-10-05 a PR's 21 contexts settled **none in 70 minutes**
behind a backlog of 25 runs, and the backlog grew faster than it cleared —
0.3.31 through 0.3.34 of one branch, four full sets, when only the newest
commit's verdict was wanted.

Every workflow now groups on `${{ github.workflow }}-${{ github.ref }}` with
`cancel-in-progress: true`.

**Verified before being added here, not assumed.** An unparseable workflow
would mean no CI at all, which is a worse failure than a slow queue, and none
of the fleet's existing `concurrency:` users are on this forge — they are
GitHub-hosted, so they prove GitHub parses the key, not Forgejo. Pushed a probe
to `ci-test` on this forge (Forgejo 16.0.5) instead: runs were created, so the
key parses; and a newer push moved two older runs to `cancelled` **while they
were still queued**, which is exactly the case that matters here.

`cancel-in-progress: true` applies to `main` too. That is deliberate: main
advances by merge commits whose tree contains the commits before them, so a
newer run covers an older one's changes.

## The queue's real shape, which is worse than one busy runner

Worth recording because it decides where the next fix goes. The workspace
matrix asks for three labels:

  - **`linux` — 5 workspaces** (jcp-rs, sandbox-rs, smart-byte-rs, proof-rs,
    eoc-rs) → jetson-hub, which the forge reports idle and connected.
  - **`linux-arm64` — 7 workspaces** (openpay, joule-code-rs, map-rs, aac-rs,
    joulecontract-rs, mesh-rs, jouleclaw-rs) → the colima `txarm` VM, **on the
    dev Mac**.
  - **`macos` — 5 workspaces** plus the dedicated `macOS energy backends` job
    → **the dev Mac**.

So **13 of 21 contexts land on the dev Mac** through two runners, while that
same machine was carrying another session's VVC/VTM encodes at load average
132 on 18 CPUs. That is why `jcp-rs` and `eoc-rs` returned quickly — they are
the two on jetson-hub — while `mesh-rs` (linux-arm64) took 25 minutes and the
macOS job never ran at all.

Concurrency groups cut the number of queued runs, which helps every label. They
do not change the fact that one host serves 13 of 21 contexts. The structural
options are to move some `linux-arm64` workspaces onto jetson-hub's `linux`, or
to stand up a second arm64/macOS host — the Mac Studio M3 Ultra is online and
idle — and that is a fleet change rather than a repo one, so it is flagged here
rather than attempted.

No regression guard is added. A check that every workflow declares the key
would itself need a CI job on the runner this change exists to unblock, and an
unwired check is worse than none.

Co-Authored-By: Claude Opus 5 <[email protected]>
2026-10-05 12:01:57 -04:00

202 lines
7.4 KiB
YAML

name: smart-byte conformance gate
# One run per branch at a time. A push supersedes the run before it.
#
# There is one `macos` runner and it is the dev Mac, so every macOS job in this
# repo queues on it. With no concurrency group, a session pushing four versions
# of a branch queued four complete sets of runs and every one of them had to
# drain: on 2026-10-05 a PR's 21 contexts settled none in 70 minutes behind a
# backlog of 25 runs, and the queue grew faster than it cleared. Only the newest
# commit's verdict is wanted, so the older runs are cancelled.
#
# Verified on this forge (Forgejo 16.0.5) in `ci-test` before being added here:
# the key parses, and a newer push moves an older run to `cancelled` even while
# it is still queued, which is the case that matters.
#
# `cancel-in-progress: true` also applies to `main`. That is deliberate and
# safe here: main advances by merge commits whose tree contains the commits
# before them, so the newer run covers the older one's changes.
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: true
on:
pull_request:
paths:
- 'smart-byte/spec/**'
- 'smart-byte/conformance/**'
- '.github/workflows/conformance.yml'
push:
branches:
- main
paths:
- 'smart-byte/spec/**'
- 'smart-byte/conformance/**'
- '.github/workflows/conformance.yml'
jobs:
validate-json:
name: Validate conformance JSON files
runs-on: linux
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Ensure jq is available
run: jq --version
- name: Validate every JSON file under smart-byte/conformance/v*
run: |
set -euo pipefail
found=0
fail=0
while IFS= read -r -d '' f; do
found=$((found+1))
if ! jq empty "$f" >/dev/null 2>&1; then
echo "::error file=$f::invalid JSON"
fail=$((fail+1))
else
echo "ok: $f"
fi
done < <(find smart-byte/conformance -type f -name '*.json' -print0)
if [ "$found" -eq 0 ]; then
echo "::error::no JSON files found under smart-byte/conformance/v*"
exit 1
fi
if [ "$fail" -ne 0 ]; then
echo "::error::$fail JSON file(s) failed validation"
exit 1
fi
echo "validated $found JSON file(s)"
- name: Validate vector schema (top-level array, name field present)
run: |
set -euo pipefail
fail=0
while IFS= read -r -d '' f; do
if ! jq -e 'type == "array"' "$f" >/dev/null; then
echo "::error file=$f::top-level value must be a JSON array"
fail=$((fail+1))
continue
fi
if ! jq -e 'all(.[]; has("name") and has("description"))' "$f" >/dev/null; then
echo "::error file=$f::every vector must have name and description fields"
fail=$((fail+1))
fi
done < <(find smart-byte/conformance -type f -name '*_vectors.json' -print0)
if [ "$fail" -ne 0 ]; then
echo "::error::$fail vector file(s) failed schema validation"
exit 1
fi
- name: Validate vector names are unique within each file
run: |
set -euo pipefail
fail=0
while IFS= read -r -d '' f; do
dupes=$(jq -r '[.[].name] | group_by(.) | map(select(length>1)) | .[][0] // empty' "$f")
if [ -n "$dupes" ]; then
echo "::error file=$f::duplicate vector name(s): $dupes"
fail=$((fail+1))
fi
done < <(find smart-byte/conformance -type f -name '*_vectors.json' -print0)
if [ "$fail" -ne 0 ]; then
exit 1
fi
# The pack's values come from the Rust reference implementation, and
# standards-build checks them against it under `cargo test`. This job checks
# them against a second implementation, written separately in another
# language, so a mistake shared by the generator and its own verifier still
# fails here. Standard library only: nothing to install beyond python3.
crosscheck:
name: Second implementation reproduces every vector
runs-on: linux
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Ensure python3 is available
shell: bash
run: |
if ! command -v python3 >/dev/null 2>&1; then
SUDO=""; [ "$(id -u)" -ne 0 ] && SUDO="sudo"
$SUDO apt-get update -qq
$SUDO apt-get install -y -qq python3
fi
python3 --version
- name: Cross-check v1
run: python3 smart-byte/conformance/crosscheck/verify_v1.py smart-byte/conformance/v1
spec-vector-coupling:
name: Spec changes require vector-pack updates
runs-on: linux
if: github.event_name == 'pull_request'
steps:
- name: Checkout (with full history for diff)
uses: actions/checkout@v4
with:
fetch-depth: 0
- name: Determine the PR base
id: base
run: |
base="${{ github.event.pull_request.base.sha }}"
if [ -z "$base" ]; then
base="origin/main"
fi
echo "base=$base" >> "$GITHUB_OUTPUT"
- name: Enforce coupling between spec and conformance changes
run: |
set -euo pipefail
base="${{ steps.base.outputs.base }}"
head="${{ github.event.pull_request.head.sha }}"
if [ -z "$head" ]; then
head="HEAD"
fi
changed=$(git diff --name-only "$base" "$head")
echo "Files changed in this PR:"
echo "$changed"
spec_touched=0
conformance_touched=0
if echo "$changed" | grep -qE '^smart-byte/spec/'; then
spec_touched=1
fi
if echo "$changed" | grep -qE '^smart-byte/conformance/'; then
conformance_touched=1
fi
if [ "$spec_touched" -eq 1 ] && [ "$conformance_touched" -eq 0 ]; then
echo "::error::This PR modifies smart-byte/spec/ without modifying smart-byte/conformance/."
echo "::error::Per the conformance social contract, every spec change must be accompanied by a vector-pack update."
echo "::error::If the spec change is editorial and the wire format is unchanged, record the editorial reason in smart-byte/conformance/v<N>/regeneration.md and include that file in the PR."
exit 1
fi
echo "coupling check passed (spec_touched=$spec_touched, conformance_touched=$conformance_touched)"
versions-present:
name: VERSIONS.md and README.md must exist
runs-on: linux
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Required pack metadata files
run: |
set -euo pipefail
missing=0
for f in smart-byte/conformance/README.md smart-byte/conformance/VERSIONS.md; do
if [ ! -f "$f" ]; then
echo "::error file=$f::missing required pack metadata file"
missing=$((missing+1))
fi
done
for d in smart-byte/conformance/v*/ ; do
if [ ! -f "$d/regeneration.md" ]; then
echo "::error file=$d/regeneration.md::missing regeneration.md for $d"
missing=$((missing+1))
fi
done
if [ "$missing" -ne 0 ]; then
exit 1
fi