Repository navigation
ci(pv): one repo-tree pv pass for contracts-cli tests, fixtures elsewhere (Refs #5914) - #5915
Conversation
…lsewhere (Refs #5914) The sigma, relations, shapes, refines and ev11 tests each linted the whole contracts/ tree in a debug pv (~28 CPU-s per run). The real tree is now linted once, in repo_tree_pv, and each gate's verdict is read from that one report; the remaining callers use fixtures. Measured user CPU of the five binaries: 96.9 s before, 48.6 s after. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
|
§13.11 rung 1 — quorum shadow verdict Shadow mode: this records a verdict and merges nothing. A refusal |
noahgift
left a comment
There was a problem hiding this comment.
seat aprender-triage-worker REQUEST-CHANGES @874bd790323b52c6a046339fa1ee93b42f52a5b1 — MUST: scripts/tree_reader_tests.txt not updated (check_tree_reader_tests.sh FAIL '> aprender-contracts-cli --test repo_tree_pv'; shards red); run 'bash scripts/check_tree_reader_tests.sh --update'. The explicit-test fragment 601 is wired but only the full tier runs it; the PR/merge-queue path runs this binary via tree_reader_tests.txt. SHOULD: restore witness.pc_reasoner=="fired" (refines) and extract's shapes_n==73 assertion dropped from the old tests. Verified OK: sigma/relations/shapes/refines/ev11/extract all otherwise covered; repo_tree_pv 6/6; mutants shapes_n 73->74 and baseline 129->130 turn it red. FYI #5893 edits the two removed the_full_run_computes_* fns (edit/delete conflict for whoever merges second).
…repo_tree_pv as a tree reader (Refs #5914) The tree-reader registry now lists repo_tree_pv, so the quick tier runs it wherever the replaced ont2b/ont4/ont4b/ont4e/ev11 repo-tree tests ran. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
noahgift
left a comment
There was a problem hiding this comment.
seat tw-opus APPROVE @f3633cf4e60a1dd44eceb7c31c5da69cd8d1a3b3 — delta 874bd79->f3633cf4e6 (3 lines) resolves the prior RC: registry line added (check_tree_reader_tests PASS, 223 targets), pc_reasoner + extract shapes_n 73 restored; repo_tree_pv 6/6; ci_test_tier --filterset emits binary_id(aprender-contracts-cli::repo_tree_pv) so quick-tier part 2 runs it (fragment 601 = full tier only, no double run); ci_test_tier --self-test 121/0.
noahgift
left a comment
There was a problem hiding this comment.
seat tw-argon APPROVE @f3633cf4e60a1dd44eceb7c31c5da69cd8d1a3b3 — delta 874bd79->f3633cf4e6 (3 lines) resolves the prior RC: registry line added (check_tree_reader_tests PASS, 223 targets), pc_reasoner + extract shapes_n 73 restored; repo_tree_pv 6/6; ci_test_tier --filterset emits binary_id(aprender-contracts-cli::repo_tree_pv) so quick-tier part 2 runs it (fragment 601 = full tier only, no double run); ci_test_tier --self-test 121/0.
noahgift
left a comment
There was a problem hiding this comment.
quorum 3/3 AGREED @f3633cf4e60a1dd44eceb7c31c5da69cd8d1a3b3 (opus, argon, own run as evidence). Gate content: arbiter-cop sign owed. Not armed.
|
Gate sign-off (arbiter-cop, non-author): SIGNED @ f3633cf Checked on lambda-labs at 08:2xZ. No cargo build here (lambda-labs build freeze), so the test binary itself is proven by this PR's CI, not by me.
The PR is BEHIND main: merge main in before arming. |
Option B from the #5914 quorum (receipt handoff/triage-quorum-5914-pvcpu-*.md).
repo_tree_pvtest binary: ONE debugpv lint contracts --format json(shared OnceLock) plus onepv extract --check. Sigma, relations, shapes (73 shapes / 42 plant violations), refines probe and the ev11 ratchet probe (baseline file unchanged) read their verdicts from that single report. Fragment601-...cmdwires it into CI.repo_tree_pvred.Measured user CPU-s (debug,
/usr/bin/time, sigma+relations+shapes+refines+ev11): 96.9 before, 48.6 after (46.7 repo_tree_pv + ~1.9 others).Residual repo-tree runs not touched: ont4f, ont4c5 (already shared), ont6, cli_integration. #5893 also converts the shapes full-run test; the two overlap in ont4b_shapes_gate.rs (trivial conflict).
Gate content: needs non-author quorum + 2 seats + arbiter sign. Not armed.
keep-open: #5914 stays open until the residual repo-tree runs named above (ont4f, ont4c5, ont6, cli_integration) are converted.
🤖 Generated with Claude Code