Skip to content

Gate the jco-browser conformance target in CI; local opt-in - #39

Merged
lann merged 1 commit into
mainfrom
conformance-browser-ci-gate
Jul 26, 2026
Merged

lann merged 1 commit into
mainfrom
conformance-browser-ci-gate

Conversation

@lann

@lann lann commented Jul 26, 2026

Copy link
Copy Markdown
Collaborator

The Actions runner image ships Chrome (≥137, so JSPI is available with no flags), and the browser adapter's findChrome() already probes /usr/bin/google-chrome — so the jco-browser conformance target gates in CI with zero new installs.

  • justfile: conformance gains a _conformance-jco-browser-gate dep that runs the browser target under GitHub Actions, or locally when opted in with CONFORMANCE_BROWSER=1, and skips with a notice otherwise. It runs before the runner pass, so a single classification covers it; the runner's existing warn-on-missing-results behavior handles the local skip.
  • manifests.toml / docs: comments updated to the new gating story (README, AGENTS.md, conformance/README, run-browser.mjs, recipe comments). The jco-browser expected-fail entries (ChaCha20-Poly1305 corpus, deterministic-ECDSA probe) were already written and are unchanged.

Verified locally with CONFORMANCE_BROWSER=1 against a Playwright-cached Chromium: the target runs the full corpus and the runner classifies 0 fail / 0 unexpected-pass with all expected-fails matched. The skip path prints a notice and exits 0.

The Actions runner image ships Chrome (>=137, so JSPI is available), and
the browser adapter's auto-detection already finds it — so the target
gates in CI with zero setup. Locally it stays opt-in: the gate recipe
runs the target under GitHub Actions or when CONFORMANCE_BROWSER=1, and
skips with a notice otherwise (the conformance runner already warns
rather than fails on a missing results file).

The browser target runs before the runner pass, so one classification
covers it; its manifests.toml expectations (ChaCha20-Poly1305 corpus,
deterministic-ECDSA probe) were already written.
@lann
lann merged commit 5f5bbd8 into main Jul 26, 2026
2 checks passed
@lann
lann deleted the conformance-browser-ci-gate branch July 26, 2026 19:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants