Repository navigation
Retire PR #392 (macOS token-efficiency receipts) with a dated record - #657
Conversation
Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
The independent review of the PR #392 retirement record approved it with minor findings. This commit applies the text fixes: - Attribute the two blocker counts (10 of 17, 31 of 48) to the custody notice and the distinct-revision count (6) to the six receipt-revision tags named in the September 29 review. - Read roadmap row F-2W-6 as the table header gives it: the Mac coordinator as owner, depending on a Mac session. - Anchor the new-target defaults link at both table rows (L78-L79) and note that the file's context-supply prose at L292 predates them: git blame at the verification base attributes it to #591, before the decided rows (#602) and the install plan (#606). - Cite the September 29 review for describing the child/worker inputs as copies of issue bodies, not synthetic fixtures. Co-Authored-By: Claude Opus 5.5 <[email protected]>
Restore main's manifests/evidence.json and replay register_file for the edited decision record, so its files[] row carries the record's new sha256 and byte count. No other registry row changes. Co-Authored-By: Claude Opus 5.5 <[email protected]>
The sentence added for the review's new-target finding began "That file's" right after two sentences about the install plan, so it could be read as naming the plan. Name the decided-defaults file, where line 292 lives. Co-Authored-By: Claude Opus 5.5 <[email protected]>
Restore main's manifests/evidence.json and replay register_file after the record's antecedent fix, so its files[] row carries the record's current sha256 and byte count. No other registry row changes. Co-Authored-By: Claude Opus 5.5 <[email protected]>
…egistry plus the owned rows) Co-Authored-By: Claude Opus 5.5 <[email protected]>
|
Exact-head SOURCE ACCEPT at The record retires #392 after landing and retains its branch, original receipts and unresolved no-purge decision. It preserves the owner-reported 31/48 digest mismatches, 10/17 receipt-revision gaps, unverifiable child result and unknown workflow status. None becomes fresh Mac acceptance. The coordinator must perform the record’s fresh unchanged-head/comment/ownership check before completing custody; an objection still stops disposition. Independent native prospective Git merge against actual main Full originals and pinned references are retained in source manifest SHA256 |
|
Claude session native-agent-stack-0c: landing at the root-ACCEPTed head Observed main Required checks at this head: 8 pass . Unresolved review threads: 0. |
|
Claude session native-agent-stack-0c: post-merge observation. Landed as |
Scope
4ced2923063db6a6dcafa9f25af5ee05a4153c75, main at publication. The record and its registry entry were prepared and first checked on its parent,9b0b8d6d25f9e3fb8f71770500e774170423315e. Main then gained one commit, anAGENTS.mdwording fix with its registry hash row, so main's registry copy was restored and the registration replayed, as the contract's hot-file protocol requires.9b0b8d6d25f9e3fb8f71770500e774170423315e, with the two changes in the working tree. Publication HEAD:f1401d2bd908cbd79a367d31a250dd5e0e433fe5, two commits on the base above. Review-round HEAD:028f15474e738c80bb1484986612aee43cdfb5dc, four further fast-forward commits on the same base (two record edits, each followed by the registry replay); see Review round.lane:foundation. Documentation is foundation-owned; hash registration alone requires no other-lane acknowledgement underdocs/lanes.md, “Labels and PRs.”docs/decisions/2026-10-03-retire-pr-392-mac-token-receipts.mdandmanifests/evidence.json.SOTA sources
register_file; the record is committed first and the registry last.34b878771b3d1e040c74cb957ff8b4118169ece8: only its Scope, SOTA sources, Evidence-class table, Coverage and configuration, Findings and Round 3 sections, plus the comments of September 27 14:23:29Z, September 27 15:13:46Z, September 29 04:46:33Z and September 29 05:15:39Z. No receipt text or withdrawn text is reproduced.c0ca46d1294ea2b286f629aa9550a7cae3be9e6b, and current digest implementation, lines 1012–1019: new recordings hash the published sanitized excerpt. This does not repair old receipt bytes.git blameat the base attributes it to New WSL definitive defaults: one default per slot from a blind decision round in both model families #591, the two table rows to Definitive manifest, next version: the blind GPT round combined with the Claude record (one job per row, 31 definitive, 4 split) #602 and the plan rows to Install plan for the new WSL distribution: 64 rows from the merged manifest, validated on a real throwaway distribution #606.refs/pull/is read-only and removal is a separate operation. Closing Add macOS host receipts for 9 token-efficiency components + jcodemunch-mcp (issue #276) #392 is not a purge.Evidence-class table
source_reviewlocal_integrationgit merge-base --is-ancestor, reference reachability and Python SHA-256 comparison; count command exited 0.source_reviewnative_provenandlocal_integrationlabels are preserved as reported, not promoted to new host acceptance.source_reviewlocal_integrationregister_file, independent base/current JSON comparison and both report--checkcommands.local_integrationlocal_integrationsource_reviewf1401d2bd908cbd79a367d31a250dd5e0e433fe5: approve, eight minor findings, each dispositioned in Review round after one repair round.028f15474e738c80bb1484986612aee43cdfb5dc; the coordinator confirms them green before merge.Local commands run
The first block holds the sandboxed builder's results; the builder ran no git writes. All builder acceptance commands ran at HEAD
9b0b8d6d25f9e3fb8f71770500e774170423315e, with the two working-tree changes present. The second block holds the publication results.The tracked registry whitespace check returned 0 with no diagnostics. The new record's native no-index whitespace comparison returned 1 with no diagnostics because the file differs from the empty input. Independent JSON comparison found exactly one new registry entry, matching the record's SHA-256 and 15,311-byte length. Both generated-report checks passed without rewriting a report.
This required check failed for a reason outside the two owned files, so the builder stopped and handed back. The coordinator must run the guarded hook on the actual staged changes in its authorized environment, repeat both committed-range commands on the final publication head, and retain the independent review and required-check results before merge. No staged-scan pass or committed-range acceptance is claimed by this builder block; the publication block below supplies them.
Publication results, run by the publishing step after the main refresh. Publication HEAD
f1401d2bd908cbd79a367d31a250dd5e0e433fe5on base4ced2923063db6a6dcafa9f25af5ee05a4153c75. The native hooks were active on both commits (no bypass), and TMPDIR was set to a scratch directory outside the repository.These are local structural and integration checks, not upstream host execution or a new model run. The publishing step did not run the independent review or wait for the required checks; Review round below records the review, the repair round and its checks.
Decision record
docs/decisions/2026-10-03-retire-pr-392-mac-token-receipts.mdnames the evidence, alternatives, history-exposure boundary and overturn condition. A Mac owner can use a new PR for fresh selected-component receipts recorded from main at or after #492, withcatalog_revisionset to a commit on main. #276 remains open for that work.Host evidence
Not applicable: this PR adds no host receipt, changes no host receipt and changes no platform status. The historical classifications are quoted as reported metadata only.
Independent review: approve, from one Opus read-only review of the record against its sources at
f1401d2bd908cbd79a367d31a250dd5e0e433fe5, with eight minor findings. One repair round followed; dispositions, checks and residuals are in Review round. Builder self-check is not substituted for this gate.Review round
One independent read-only Opus review of the record against its sources, at
f1401d2bd908cbd79a367d31a250dd5e0e433fe5, returned approve with eight minor findings: five text fixes and three verification gaps, none blocking or should-fix. One repair round followed, as four fast-forward commits on the published branch, each record edit followed by a registry replay.0908b116a8408815819eb71a3b6135581ad1300capplies the five text fixes, and742abc1d556a994795c172baec8bf6b248b80638restores main's registry copy and replays the contract'sregister_filecall. Then5e5f3cfb9959046386387ec8d12d27fcdc160030clarifies one sentence that the first edit added (row 9), and028f15474e738c80bb1484986612aee43cdfb5dcrepeats the registry replay. The record's registry row now reads sha2562c73507dea5faf517e4068b7a6efad57483250e0eb51f3dcba781d4fd922dfe9, 16,044 bytes; the publication block above keeps the values that were true atf1401d2b. Nothing was force-pushed, and #392 is untouched.receipt-revision/*tags in the September 29 04:46 review. Re-derived by a counts-only script: 6 distinct revisions, each the target of exactly onereceipt-revision/*tag. The review's head is an ancestor of the pinned head, and none of the 17 receipt files changed between the two.[new-target]anchored only the ccusage row#L78-L79. The optional clause was added with its source.git blameat the base attributes line 292 to #591, lines 78–79 to #602, and the install-plan lines 1060–1105 to #606. The #591 commit is an ancestor of the #602 commit, which is an ancestor of the #606 commit.gh pr checks 657at 2026-10-03T12:35:25Z, still onf1401d2b, exited 8: 20 passed, 3 skipped, and 2 pending (bootstrap-macos, validate-macos). The checks restart on the new head (residual 1).5e5f3cfb: the sentence now names the decided-defaults file, where line 292 lives.Review-round commands, on final HEAD
028f15474e738c80bb1484986612aee43cdfb5dcand base4ced2923063db6a6dcafa9f25af5ee05a4153c75(main unchanged since publication). The checks after the commits ran on742abc1d556a994795c172baec8bf6b248b80638and again on028f15474e738c80bb1484986612aee43cdfb5dcwith the same results, except the commit count of the range scan; the values shown are from028f15474e738c80bb1484986612aee43cdfb5dc. TMPDIR pointed to a scratch directory outside every repository, the native hooks were active (no bypass), and every exit code was read directly, not through a pipe.Residuals:
028f15474e738c80bb1484986612aee43cdfb5dc. Each body edit firesvalidate.yml'seditedtrigger (line 9). Its concurrency group, keyed by workflow, event name and PR number with cancel-in-progress for pull requests (lines 15–17), cancels the push's in-flight run, so validate re-runs on the same head; a cancelled validate run on this branch is therefore expected, not a failure. The coordinator must confirm every required check green on that head before the squash merge.9b0b8d6d. Main at4ced2923differs from that base only inAGENTS.mdand its registry row, so no cited line moved.Checklist
🤖 Generated with Claude Code