Skip to content

ci: differential oracle lane, test-summary aggregator, copilot-review-posted producer, version 0.10.0 (sq-5reoy) - #102

Merged
jeswr merged 1 commit into
mainfrom
sq-5reoy-ci-release-v0.10.0
Jul 5, 2026
Merged

jeswr merged 1 commit into
mainfrom
sq-5reoy-ci-release-v0.10.0

Conversation

@jeswr

@jeswr jeswr commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

🤖 SPARQ agent — automated CI migration + release prep under maintainer directive sparq-org/sparq#1599 (bead sq-5reoy).

Summary

  • ci.yml — differential job: ports the differential-ieee754 job from sparq's zk-toolchain.yml to the face repo. Builds the standalone Rust harness (differential/), drift-guards the committed oracle (tests/differential_oracle/src/lib.nr against the generator), runs the full oracle suite (24 test functions, ~4500 assertions), and runs the inject-fault self-test to prove the harness is non-vacuous. SHA-pinned actions, two-level nargo pin (noirup tagged release + pinned NARGO_VERSION), Swatinem cache.
  • ci.yml — test-summary aggregator: adds a test-summary job (needs: [test, differential], if: always()) that fails loudly if either predecessor did not succeed. This satisfies the ruleset-11111001 required_status_checks context "test-summary" which was previously phantom-unmet (no workflow produced it), hard-blocking every PR.
  • copilot-review-posted.yml: new producer workflow. Triggers on pull_request_review (submitted) and pull_request (opened/synchronize/reopened). Queries the PR reviews API for copilot-pull-request-reviewer[bot] and POSTs a success/failure commit status with context "copilot-review-posted". Fixes the sq-umj0p gap for noir_IEEE754 — the other required_status_checks context that was phantom-unmet.
  • Nargo.toml: adds version = "0.10.0" — the first versioned release of the sparq_ieee754 lineage (no version field existed in the pre-migration package).

Bootstrap note

This PR itself will be hard-blocked by the ruleset until the copilot-review-posted producer is running and Copilot has reviewed it. Admin bypass is needed for this one bootstrap merge, exactly as surveyed. Once merged, all subsequent PRs will get the producer from the workflow itself.

Test plan

  • CI run for test job green (nargo tests + generated vectors + API surface + lint)
  • CI run for differential job green (drift guard + oracle + self-test)
  • CI run for test-summary job green (aggregator reports success)
  • copilot-review-posted.yml workflow appears in Actions tab
  • After merge: tag v0.10.0 cut on main with release notes

🤖 Generated with Claude Code

…t-review-posted producer + version 0.10.0 (sq-5reoy) [SONNET-4.6]

- ci.yml: add `differential` job (Rust harness drift-guard + full oracle + self-test,
  ported from sparq zk-toolchain.yml differential-ieee754 job; SHA-pinned actions,
  two-level nargo pin, Swatinem cache keyed `ieee754-differential`)
- ci.yml: add `test-summary` aggregator job (needs: [test, differential], if: always(),
  fails if either predecessor did not succeed) — satisfies the ruleset-11111001
  required_status_checks context "test-summary" that was previously phantom-blocked
- copilot-review-posted.yml: producer workflow (pull_request_review + pull_request
  synchronize triggers; statuses: write; queries PR reviews API for
  copilot-pull-request-reviewer[bot]; POSTs success/failure commit status context
  "copilot-review-posted") — fixes sq-umj0p gap for noir_IEEE754
- Nargo.toml: add version = "0.10.0" (first versioned release of the sparq_ieee754
  lineage; no version field existed previously)

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 5, 2026 20:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@jeswr
jeswr merged commit f661893 into main Jul 5, 2026
9 checks passed
@jeswr
jeswr deleted the sq-5reoy-ci-release-v0.10.0 branch July 5, 2026 20:09
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