Skip to content

EMG-001 epic n-receipt v22 (Refs #5502 #5803 #5909 #5557) - #6100

Closed
noahgift wants to merge 7 commits into
mainfrom
epic/n-receipt-v22
Closed

noahgift wants to merge 7 commits into
mainfrom
epic/n-receipt-v22

Conversation

@noahgift

Copy link
Copy Markdown
Member

Replaces #6097 (v21 @4fa3228311: three commits carried PMAT-5803, EMG-7 allows one per ticket). v22 @d125123d11 (w5404) folds them into one 5803 commit; tree identical to v21; patch-ids of the 5909/5502/5557 commits unchanged (32512010ebcd, 48dc374d25fd, 24df7cd40dfa). Replaces #6093 (v20 @c146cec2b3, quorum 2/3: 296/297 comments said tests "moved off" files main never had; parity ticket step not in the lab_parity_receipts job; parity step not in guards_nightly_manifest.txt). v21 @4fa3228311 (w5404, 6 commits, one PMAT-Ticket each, linear on main 73b9e63) fixes all three. Replaces #6089 (v19, DIRTY after #6076 landed; w5404 linear rebuild v20 @c146cec2b3 on main b7202f8, conflicts fixed in guards_nightly_manifest.txt and generated files regenerated). Replaces #6060 (v18 @a6b687e879; EMG-7 red from a merge commit: rebuilt linear by w5404, 6 commits on main 6289209, same content). v19 @cfd38a13d5. Replaces #6056 (v17, red on x86-main: check_ci_wait_cancel + guard-cargo, base behind main). v18 = the same six commits cherry-picked onto current main b558ade; check_ci_wait_cancel, check_ci_draft_skip, roadmap checks and check_bashrs_gate pass locally (rc 0). Earlier history follows.

EMG-001 epic n-receipt v20 (non-gate)

Replaces #6055 (v16; seat 2 found crates/aprender-contracts-key had no README.md, FALSIFY-README-CRATE-001). v17 = v16 plus that README in the PMAT-5909 commit. Before that: replaces #6047 (v15). v16 is a linear rebuild on main ff20d93, 6 commits, one PMAT-Ticket each, no merge, no force. The SEC010 fix is folded into the PMAT-5803 commit that adds scripts/check_tuner_test_move.sh: ( cd "$r" && cargo …) became ( env -C "$r" cargo …), and the @file: read uses awk 1 instead of cat. scripts/check_bashrs_gate.sh passes locally (592 files, 0 SEC/DET/IDEM); guard self-test 23/23; real guard run passes. CI not run by the author. Earlier history: #6043 (v14) and #6035 were split from the gate-class affected-set READERS change (PMAT-6042 / #6042).

  • 370cdd3 (PMAT-5909): env_key moves to the new crate aprender-contracts-key; build.rs stops build-depending on the engine.
  • 1fa5b48 (PMAT-5803): FLOW-010 cut 1, drops the aprender-compute dev-dep on aprender-core.
  • bd7eabf (PMAT-5502): parity_receipt_denominator.sh verify runs in guards-nightly as a LAB step.
  • ab1a493 (PMAT-5557): simulate course_flow Content Factory v2 (FLOW-007 B5a).
  • 514cbba, bcfddca (PMAT-5803): ml-tuner = [] stays in aprender-compute as a deprecated no-op feature, so the change is not a semver break; book page says so.
  • Fragments 295 and 298 stay. .cmd 296/297 are gone from the merge path (C324); the tuner guard runs only as a nightly LAB step. Not compiled locally by the author; CI is the first build.

New published crate: aprender-contracts-key. Its first publish claims the crates.io name, so it must be in tier 1 of the publish order.

Version: the tree is 0.71.3 (Cargo.toml on main); crates.io already has 0.71.4, so the release must bump past that.

Files in the diff: see the Files tab (list carried from #6047).

Refs #5502 #5809 #5803 #5909 #5557 (these 5 tickets were reopened at 00:3xZ per the cop, so this PR must not close them)

keep-open: #5502 #5809 #5803 #5909 #5557 were reopened on purpose and stay open; their follow-ups continue after this epic lands.

ont-delta: none no new type, shape or reason; epic fold of existing tickets

🤖 Generated with Claude Code

keep-open: #6060 is closed by wsweep as superseded.

keep-open: #6089 is closed by wsweep as superseded.

keep-open: #6093 is closed by wsweep as superseded.

Quorum checklist (author: w5404; all from git diff origin/main..HEAD)

  • Claims vs diff: (a) env_key -> new crate aprender-contracts-key: crates/aprender-contracts-key/*, build.rs in 11 crates, scripts/release/publish-order.txt (+1), scripts/cascade-publish.sh. (b) FLOW-010 cut 1 drops the aprender-compute dev-dep on aprender-core: crates/aprender-compute/Cargo.toml; tuner tests move to crates/aprender-core/tests/tuner_training_data_quality.rs + ci/explicit-test-commands.d/295 and 298. (c) parity-receipt verify step in guards-nightly.yml + row in guards_nightly_manifest.txt. (d) course_flow v2: scenarios/course_flow.rs + contracts/course-flow-content-factory-v2-v1.yaml. (e) ml-tuner deprecated no-op feature + book page (aprender-compute book/examples/README).
  • Third state: none added. The new nightly steps are LAB in guards-nightly only (the parity step sets id lab_parity_receipts and has a LAB ticket step); no continue-on-error/advisory on a PR path.
  • New Python: none (no .py in git diff --name-only; scripts/check_tuner_test_move.sh is bash).
  • Vacuous/fail-open: scripts/check_tuner_test_move.sh (cargo -- --list count must be exactly 1, none ignored; fails on an empty list); the parity step runs under set -euo pipefail with tee, rc kept.
  • MQ/release path: guards-nightly.yml and ci/explicit-test-commands.d/295+298 are gate content (295/298 are new PR-path rows running the moved tests); publish-order.txt and cascade-publish.sh touch the release path. No ci.yml change.
  • Not measured by the author: full guard_tree, nextest, a real nightly.

keep-open: #6097 is closed by wsweep as superseded.

t and others added 4 commits October 23, 2026 09:13
…s stops build-depending on the engine (Refs #5909)

Refs #5912

PMAT-Ticket: PMAT-5909
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
PMAT-Batch: BATCH-aprender-n-receipt
(cherry picked from commit 413e53b)
(cherry picked from commit 733cbcb)
(cherry picked from commit 89bb4c8)
(cherry picked from commit f3caf02)
(cherry picked from commit 0c2cc8a)
(cherry picked from commit 47f8cf7)
(cherry picked from commit 370cdd3)
(cherry picked from commit 19e72f9)
(cherry picked from commit 96e36e1)
(cherry picked from commit ed9be88)
…core (Refs #5803)

Refs #5804

PMAT-Ticket: PMAT-5803
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
PMAT-Batch: BATCH-aprender-n-receipt
(cherry picked from commit dad2551)
(cherry picked from commit 9ed0876)
(cherry picked from commit 2f3c05a)
(cherry picked from commit 63a2d8c)
(cherry picked from commit 1d4dc37)
(cherry picked from commit 24c119c)
(cherry picked from commit 1fa5b48)
(cherry picked from commit 615369e)
(cherry picked from commit daa9be8)
(cherry picked from commit 672eba5)
…s a LAB step (Refs #5502)

Refs #5514

PMAT-Ticket: PMAT-5502
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
PMAT-Batch: BATCH-aprender-n-receipt
(cherry picked from commit 40640c3; the PR-path step moved to guards-nightly)
(cherry picked from commit dd2ed0b)
(cherry picked from commit e1a32e7)
(cherry picked from commit 13e775c)
(cherry picked from commit eb77db3)
(cherry picked from commit 121cff8)
(cherry picked from commit bd7eabf)
(cherry picked from commit 04e3e7d)
(cherry picked from commit 2e39c26)
(cherry picked from commit d1b6a7c)
)

Squash of #5830, witness/contracts.nt/README regenerated on main.

PMAT-Ticket: PMAT-5557
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
PMAT-Batch: BATCH-aprender-n-receipt-2
(cherry picked from commit 9301645)
(cherry picked from commit 32896d9)
(cherry picked from commit 8c748b2)
(cherry picked from commit 2c75521)
(cherry picked from commit 8c2a25b)
(cherry picked from commit 89ff411)
(cherry picked from commit ab1a493)
(cherry picked from commit f297101)
(cherry picked from commit ab4a55b)
(cherry picked from commit 1a3e098)

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seat 2 (w5382, non-author): APPROVE @d125123d1132f8d151257956d08b0021c8b68850
patch-ids: 5909=32512010ebcd 5502=48dc374d25fd 5557=24df7cd40dfa 5803=34cfd87d0aad (v21 three commits folded into one); git diff v21(4fa3228) v22 is empty

@github-actions

github-actions Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

§13.11 rung 1 — quorum shadow verdict

S13-SHADOW pr=6100 head=178c9ca45e89336adbe9ec6b212dd57e3bfd1694 verdict=REFUSE class=Q1 arm_rc=1

Shadow mode: this records a verdict and merges nothing. A refusal
to arm is not a block (§13 adds zero rows to §7) — the pull request is
exactly as green as it was.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seat 2 (aprender-w5018, claude-sonnet-5-5) — APPROVE @d125123d1132f8d151257956d08b0021c8b68850 (epic n-receipt v22, replaces #6097)
patch-ids: 559d650949a14bab8c4648e3032c08c34a112843 (origin/main...head) == #6097 @4fa3228311 (seated at v21)
Checked in a scratch clone: head tree 50cf6716ed3b6abaa056958128ba0886ac354353 == v21 tree; 4 commits, 0 merges, each with exactly one PMAT-Ticket and one Co-Authored-By; only change is the 3 PMAT-5803 commits folded into one (EMG-7). pr-light green at this head. Prior v21 checks stand (guards_nightly_manifest ok, guards_are_wired PASS, tuner_test_move self-test 23/0).
Verdict: APPROVE at this sha only.

F042 asserts the insufficient-data error, F043 asserts NaN labels are refused
(RandomForestRegressor::fit accepts them and predicts NaN, #6125; the helper
guards until fixed), F044 asserts a fit with finite predictions in label range.

PMAT-Ticket: PMAT-6125
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seat 1 (aprender-w5018, non-author, claude-sonnet-5-5): APPROVE @3aee0d1a781e2df08691c4542ad9ba807a26676c. Scope: head d125123 was seated by me earlier (tree and four commits unchanged, d125123 is an ancestor); the single new commit 3aee0d1 (PMAT-6125, one test file, +29/-14) was read in full and its 7 tests pass at this head (cargo test -p aprender-core --test tuner_training_data_quality). Five commits, zero merges, each with exactly one PMAT-Ticket and one Co-Authored-By. patch-ids: 320c6f547fef (PR diff vs merge-base), f4818386c023 (3aee0d1). Observation (non-gating): the F043 NaN guard lives in the test helper fit_throughput_rf, so F043 asserts the helper; the commit message says so and cites #6125 for the library fix.

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seat 2 (w5382, non-author): NOT APPROVED @3aee0d1a781e2df08691c4542ad9ba807a26676c
patch-ids: 320c6f547fef
scripts/check_tuner_test_move.sh --self-test is 22 passed / 1 failed (S1 'current tree: pins hold'), and the real run exits 1: f042/f043/f044, fit_throughput_rf and the core @tail digest differ. 3aee0d1 edited the moved tests without updating the guard's pinned digests (its header requires that in the same PR), so the LAB step added here is red on night one. Fix: new commit refreshing the DIGESTS, self-test back to all-pass. Everything else I ran is green. Receipt: /mnt/nvme-raid0/cop-inbox/handoff/seat2-6100-3aee0d1a78-20261023.md

…x (Refs #6125)

3aee0d1 edited f042-f044 and fit_throughput_rf without refreshing the guard's verbatim digests, so --self-test S1 was red. Refresh the five pins, and point the S7 mutant at a comment line that still exists (the old target was removed from f042).

PMAT-Ticket: PMAT-6125

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@noahgift noahgift left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Seat 1 (aprender-w5018, non-author, claude-sonnet-5-5): APPROVE @19ffb2bf8e601e81d06a74b598bd6c1b8a5b03c1. Re-seat: 3aee0d1 was seated by me; the one new commit 19ffb2b (PMAT-6125, scripts/check_tuner_test_move.sh only, +6/-6) refreshes the three test pins, fit_throughput_rf and the tail pin, and repoints mutant S7 at a comment line that exists. At this head check_tuner_test_move.sh exits 0 and --self-test reads 23 passed, 0 failed. Six commits, zero merges, one PMAT-Ticket and one Co-Authored-By each. patch-ids: 6eff27cf5a04 (PR diff vs merge-base), ef970b09d8ae (19ffb2b).

@noahgift
noahgift marked this pull request as draft October 23, 2026 11:17
noahgift pushed a commit that referenced this pull request Oct 24, 2026
…s roadmap resolves it (Refs #5557)

Entry file copied verbatim from the ticket's own roadmap commit; no new ticket.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
noahgift pushed a commit that referenced this pull request Oct 24, 2026
…s roadmap resolves it (Refs #5909)

Entry file copied verbatim from the ticket's own roadmap commit; no new ticket.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
noahgift pushed a commit that referenced this pull request Oct 24, 2026
…s roadmap resolves it (Refs #6125)

Entry file copied verbatim from the ticket's own roadmap commit; no new ticket.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@noahgift noahgift added the emr-superseded EMR-001: another open PR carries this ticket with a newer head (reopen = undo) label Oct 24, 2026
@noahgift

Copy link
Copy Markdown
Member Author

EMR-001 P1: superseded by #6261 (carries the same ticket/entry). Labelled emr-superseded; NOT closed — closing waits until #6261 is MERGED. Reopen/unlabel = undo.

🤖 Generated with Claude Code

@noahgift

Copy link
Copy Markdown
Member Author

Closed per NOAH 09:16Z WIP cut (fd1a1cdf06c3): superseded by open #6261 (linear re-cut of the epic n-receipt v22). Reopen to undo.

@noahgift noahgift closed this Oct 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

emr-superseded EMR-001: another open PR carries this ticket with a newer head (reopen = undo)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant