Repository navigation
v0.90 cold review: record the review and correct the statements it found - #1674
Merged
Merged
Conversation
Commits the two-round cold review of 80f8676, byte-identical to the frozen copy, with an appended coordinator disposition. Corrects the statements it found: - CHANGELOG [0.90.0] New loud refusals, RQ-90-A32ENTRY, RQ-90-A32CALL: the newly-refused counts were a first-export-per-file sample. They are re-derived over every export of the 268 tracked .wat/.wasm files, v0.89.2 against 0.90.0, and given as separate units: 133 module-level compiles (--all-exports, and the default invocation, which takes the same path); 978 --func-name compiles in 185 files; 107 cortex-r5 --func-name compiles of a calling function in 65 files; 197 files with no target, 212 including cortex-r5. - CHANGELOG Compliance envelope and RQ-90-GATEPLACE: the keep-as-is decision for the wrong-code differentials and the separately requested spec-suite context are now both stated. The tenth context was added to branch protection directly; PR #1670 only moved it into REQUIRED_CONTEXTS. - Six release records whose verified-by says execution-verified now carry verification-track: execution, matching their prose. Refs #1640 Refs #1645 Refs #1573 Refs #1636 Refs #1670 Refs #1662 Refs #1318 Refs #1672 Refs #1673 Co-Authored-By: Claude Opus 5.5 <[email protected]> Claude-Session: https://claude.ai/code/session_01YJK5LZZEkV5smCY1jKn18L
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The v0.90 review record, plus the corrections it calls for. One signed commit.
What it adds
docs/reviews/v0.90-cold-review.mdholds the two-round cold review of 80f8676 with a# v0.90 — coordinator dispositionsection appended.428edbf4…ce94c3. The first 29642 bytes (round 1) givef41f9750…6a85a.diff <frozen> <record> | grep -c '^<'gives 0. A negative control, one byte changed inside the prefix, makes the prefix check fail.wc -cwas re-run after the last edit.Corrected sites (before → after)
1. CHANGELOG [0.90.0], New loud refusals, #1640 bullet.
--all-exports, 179--func-name)".--all-exports, and equally the defaultsynth compile x.wat, which takes the same path), and 978--func-namecompiles in 185 files. That is 197 distinct files. The note says PR v0.90 A32CALL (#1645): self-contained A32 images patch every internal bl and enter A32 code in ARM state #1662's 179 compiled only the first export of each file.2. CHANGELOG, the #1645 bullet.
--func-namecompiles of a calling function, in 65 files. The two refusals together affect 212 distinct files. PR v0.90 A32CALL (#1645): self-contained A32 images patch every internal bl and enter A32 code in ARM state #1662's 48 is labelled a first-export sample.3. RQ-90-A32ENTRY
verified-by, the REACH paragraph.4. RQ-90-A32CALL
verified-by.5. RQ-90-GATEPLACE description.
spec-suite execution (#1636), which v0.90 SPECEXEC (#1636): promote spec-suite execution to a required context #1670 added".PENDING_REQUIRED_CONTEXTSintoREQUIRED_CONTEXTSinscripts/ci_pool_tripwire.py. "nine required status checks" now reads "… of v0.89".6. RQ-90-GATEPLACE
verified-byand the CHANGELOG Compliance envelope.7. Six records: A32BRANCH, A32GUARD, F64STORE, FALCON7, MEMKFIXTURE and NPASHAPE2.
verification-track: manualbesideverified-byprose saying "execution-verified".verification-track: execution, matching the prose and the v0.89.1 and v0.89.2 precedent. No script reads the field.Re-derivation (F1)
Fresh
cargo build --release -p synth-cli --features verifybinaries:bad16d49136ae2e1.git archive v0.89.2, sha256 prefix23dc9ea26da805f5.The census covers the 268 files from
git ls-files '*.wat' '*.wasm'× {no target, cortex-r5} × {default,--all-exports,--func-index 0,--func-namefor every export}. That is 4452 jobs per binary, with 0 harness errors. The export reader agrees with wasmtime on 263 of 263 files.--all-exports--func-name--func-index 0--func-name--all-exportsgive the same result in 1072 of 1072 pairs.--target cortex-r5image ships every internal A32blunpatched (ebfffffe, a branch-and-link to itself) and reports "Internal calls: 1 resolved in place", exit 0 #1645 message, and nothing flips the other way.Gates (at the commit, from the repo root)
claim_check.py claims.yamlstatus_evidence_check.pyverdict_prose_check.pyissue_closure_check.py --release v0.90 --since-tag v0.89.2 --allow-unclosedrelease_file_statistics_scan.py --release v0.90_release.yamllines 32-33 ("this one", "the planned one"). Judgement: both IDIOM.check_py_floor.py(--self-test, then every tracked.py, under Python 3.9.6)run:step of ci.ymlclaim-checkrun whole underbash -o pipefail -eORACLE_EVIDENCE_JSONL; re-run with the workflow env, steps 52 and 53 give rc 0ci_pool_tripwire.py(not--offline)review_record_provenance_check.pyloop_conformance_check.py v0.90.0with HEAD at the commit: verdict DOES-NOT-CONFORM, slots 7, derived 5, attested 1, failures 1. The one failure is step 5's "mcdc ran on commit", which is NOT-DERIVED because the commit was not yet pushed. Each step:--exact-emulation-floor 449389; live runSquash simulation
commit-treebuilds this branch's tree on parent 80f8676, in a clone in scratch. Thenstatus_evidence_check.pyruns on each result. The two runs differ only in the subject:v0.90 cold review: record the review and correct the statements it found (#1674), which is what a squash produces: rc 0. "12 delivery-shaped commits since v0.89.2 — 12 attributed". The subject is not lane-shaped (lower case after the version) and is not a conventional delivery type.v0.90 COLDREVIEW (#1674): …on the identical tree: rc 1,FAIL R10-lane: … names artifact RQ-90-COLDREVIEW, which no release file defines, 13 delivery-shaped.Not in this PR
Nothing is merged, tagged or posted. U2 (274
assert_returnand 177assert_trapundersoftware) is recorded as UNPROVEN, because round 2 says so in the record. A1, the acceptance corpus, is NOT measured: the corpus volume is not mounted on this machine. The disposition lists what is carried to v0.91.Refs #1640
Refs #1645
Refs #1573
Refs #1636
🤖 Generated with Claude Code
https://claude.ai/code/session_01YJK5LZZEkV5smCY1jKn18L