Repository navigation
governance: AGENTS.md §A.42 item 8 (trace off-main references) and §A.46 (research records and established claims) - #1097
Merged
Conversation
…fore landing Adds item 8 to §A.42. Before a record lands, each reference it makes to content not on main (commit, branch, pull request, comment or path) is traced to the claim it is attached to and classified as provenance, support on main, or support off main (a gap). "Provenance only" is a per-reference finding, not a group label; "proved from", "inherits", "uses" or "preserves" another record's result is support. A gap is closed no later than the record lands, by landing the content or, for a mutable record, by narrowing the claim. Keeping the branch or assigning the gap to an Issue does not close it. An unclosable gap lands only by owner decision, with the unsupported claim named. The enforcement line now covers item 8, applies it to later landings, and leaves frozen records to the forward-only correction of item 6. One lessons-register line records the cause: five evidence landings in PR-AUDIT-1 (#1088) were reviewed for byte identity, CI and verdicts, but their branch-only references were accepted as provenance without a trace, and MR-1's Proposition B2 is proved from a proof held only on #912's kept branch. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
…4, N6) - Item 8 lists the reference kinds as examples (adding Issue and CI run). - It narrows "provenance" to "the claim does not use the referenced content", and says that support on main means the proof, for a result proved elsewhere. - It adds that a claim about commit order or ancestry uses those commits, and copying their files to main does not make them reachable. - An unclosable gap lands only by a recorded owner decision, quoted as item 7 requires, as an exception to item 2 for that claim alone, with the unsupported claim named on main. - The enforcement line now covers byte identity, CI and verdicts. Item 8 applies to every later landing, frozen records included, without changing their text or verdicts, and an earlier gap is closed by landing the content or by correcting the citing record as item 6 allows. Item 8's last sentence, which repeated this, is removed. - The lessons entry no longer says "five": one of the five landings, #1093, named no branch-only commit. The commit message of e95deb1 overstates this in the same way; it is left unchanged. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
- R1: the ancestry sentence now covers a result, control or verdict that depends on commits, such as on their order or ancestry. A pin that only says where content came from stays provenance under item 2. - R2: an unclosable gap's owner decision is quoted, with author and time (item 7), in a record on main that also names the unsupported claim, since a frozen record cannot carry the quote. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
…imit No words changed. Applies the delta review of 1f66557 (finding 1). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
This was referenced Nov 4, 2026
Merged
… claims Adds the owner's preservation-first policy (2026-11-04): main is the authoritative research record. Positive, negative and inconclusive results, failed approaches and their supporting files may stay on main with their provenance, assumptions, verification status and limitations stated. A claim is presented as established only when its evidence meets the verification standard for its kind. Retention promotes nothing. A record's status is stated as independently verified, partially verified, exploratory or known invalid. Unsupported conclusions are corrected where stated while their evidence stays. Removal needs a specific justification, a dependency audit and separate owner authorization. Added to #1097 at the owner's request (one PR for both rule changes); §A.46 is a new section at the end of the file, apart from §A.42. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
- F1: §A.42 item 5 and §A.28 limit what is added to main and are not a ground for removing research evidence already there. - F2: a frozen record's unsupported conclusion is corrected by a new native round (§A.42 item 6). - F3: annotation over deletion applies to research evidence; a manuscript claim that loses its support is handled as §A.27 and §A.30 require. - F4: a record distinguishes the four kinds of result, in the record or, for a frozen record, in a mutable record on main that cites it. - F5: the owner's "provided that" condition is restored in the opening. - F6: item 2 applies to results that do not meet item 1's standard, so a verified negative result can be established (§A.29). - F7, F8, F9: separator line; a resolvable cross-reference; the adoption line narrowed to what is recorded. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
- N1: "whose removal item 5 below governs", so the reference cannot be read as §A.42 item 5. - N2: a status stated for a frozen record that contradicts its own conclusion is a correction under item 4. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
Owner decision (2026-11-04): strengthen the policy so that main does not accumulate unverified data, without deleting the research record. - Item 1: a claim is established only when it can be verified from main and meets its verification standard. A claim landed under an owner exception to §A.42 item 2 is not established (review nit N1 at 8fae35f). - Item 2: research evidence lands when an established claim needs it, or for reproducibility or to support records already on main, with status and limitations stated. Exploratory work nothing on main needs stays on its branch and in its Issue. - Item 4: every unverified claim on main has a stated status, an accountable Issue and a remediation plan; a label alone resolves nothing. - Item 5: remediation lands evidence, re-runs computations or obtains a recorded review; an unverifiable conclusion is narrowed or withdrawn, in a frozen record by the applicable reviewed round, and the evidence stays. - Item 6: removal is a last resort, never an automatic consequence of failed verification, and needs justification, a dependency audit and separate owner authorization. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
The owner's adjustments of 2026-11-04 are quoted in the PR description.
This commit aligns §A.46 with them:
- Removal of research evidence is a last resort, not an *automatic*
consequence of failed verification. A frozen record is removed only by
the applicable reviewed round (§A.39).
- Item 4 applies to claims whose verification is incomplete, and the
four-way status split to their records, not to every record on main.
- Item 5: a gap is fixed by meeting item 1's standard; the three means are
examples, and a review counts only when recorded on main.
- Item 2: evidence "may land"; provenance, assumptions, status and
limitations are stated accurately.
- Item 1 cites §A.42 item 8 unambiguously, and the adoption line drops the
same-day revision clause.
The earlier commit message's summary ("so that main does not accumulate
unverified data") paraphrased the owner's concern rather than quoting the
decision; the PR description quotes it.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
"Work that meets none of these conditions stays on its branch, tracked by its guiding pull request or Issue (§A.43)", in place of the earlier sentence, which used "exploratory" in a different sense from item 4 and was narrower than §A.43. The finding-6 deletion of the previous review was not applied in 5a99420; the sentence is kept, reworded, for the owner to confirm. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
amaybaum
marked this pull request as ready for review
November 4, 2026 15:29
amaybaum
pushed a commit
that referenced
this pull request
Nov 4, 2026
Brings in #1097 (AGENTS.md) and #1098 (history merges) so the branch is up to date for the required checks. No file of this branch changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
amaybaum
pushed a commit
that referenced
this pull request
Nov 4, 2026
Brings in #1097 (AGENTS.md), #1098 (history merges) and #1099 (the missing sources) so the branch is up to date for the required checks. No file of this branch changes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
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.
Requested by the owner (amaybaum, Claude Code session chat, 2026-11-04): "Ok let's update the AGENTS.md file as well, to prevent this from happening again."
It also carries §A.46, moved here from #1101 at the owner's request: "Why do we need 2 separate open PR's for AGENTS.md file changes?", then "Yes please do" (amaybaum, session chat, 2026-11-04). #1101 is closed without merging, and its branch is kept at
c97de3dd.Why. In PR-AUDIT-1 (#1088), evidence landings were reviewed for byte identity, CI and unchanged verdicts. Their references to branch-only content were accepted as "provenance only" without tracing what each one supported. Four of the five landings named branch-only commits: #1089, #1094, #1095 and #1096. #1093 named none. MR-1's Proposition B2 on
main(measurement-record-formation/preregistration.md, "proved on paper from #912's proof") rests on a proof held only on #912's kept branch. The landing records treated keeping that branch, and assigning the gap to an Issue, as enough. §A.42 item 2 already forbids this. The step that was missing was a trace of each reference to its claim.Change. One file,
AGENTS.md, in ten commits:e95deb1cadds §A.42 item 8;6e381b33applies the first review;1f66557dapplies the re-review;16fad497reflows one paragraph;0ad4bd0fadds §A.46;25c7d8c0applies §A.46's review;8fae35faapplies §A.46's delta review (N1, N2);be616ffcrevises §A.46 to the owner's fix-or-remediate adjustments;5a994209applies that revision's review;e10a9ba2rewords item 2's last sentence (delta review nit 1).0ad4bd0fand25c7d8c0are #1101's3a8920a8andc97de3dd, cherry-picked; only the first commit's message changed, to name this PR. At25c7d8c0the §A.46 text equals #1101's head;8fae35fathen applies N1 and N2.What the change adds:
§A.42 item 8, "Trace every off-
mainreference." Before a record lands, each reference it makes to content not onmainis traced to its claim and classified. That includes a commit, branch, PR, Issue, comment, CI run or path, named by id or by number. The classes are:main: the content the claim uses, including the proof of a result proved elsewhere, is onmainand supports the claim as it stands there, with any difference from the cited version stated;main: a gap.The item also states:
maindoes not make them reachable;mainquotes it with author and time (item 7) and names the unsupported claim, and it is then an exception to item 2 for that claim alone.Enforcement line:
Lessons register: one entry recording the cause.
§A.46, "Research records on
mainand established claims" (from #1101, then revised). Owner decisions (amaybaum, session chat, 2026-11-04). First, the preservation-first decision:Then the owner raised a concern:
And then made these adjustments:
What §A.46 says at the head:
mainis the authoritative research record. An incomplete programme may have records onmain; an incomplete verification of a specific claim is a gap to be fixed.mainand meets the standard for its kind. A claim landed under an owner exception (§A.42 item 8) is not established.main.main. Otherwise narrow or withdraw the conclusion and keep the evidence. A frozen record is corrected by the applicable reviewed round.Reviews of §A.46:
8fae35fa: CONFIRMED WITH NITS. Its nit N1 (an owner-exception claim is not established) is applied in item 1.be616ffc: CHANGES REQUIRED, findings 1–8.5a994209applies all of them except the deletion in finding 6, which is kept as above.5a994209: CONFIRMED WITH NITS, nothing blocking. Nit 1 (reword item 2's last sentence) is applied ine10a9ba2.Not in this PR. The read-only compliance check of the nine PRs merged during #1088 is complete, and its findings are with the owner for a decision on remedies. That covers #1016, #1085, #1086, #1087, #1089, #1093, #1094, #1095 and #1096. No record, manuscript or tool changes here. No checker is added: the trace requires reading each claim, so it is a review obligation, like items 1–6.
Checks.
edge_rigidity_probe.py.Review of §A.42 item 8.
e95deb1c, returned CHANGES REQUIRED with three blocking wording fixes (B1–B3) and six nits.6e381b33applies B1–B3, N1–N4 and N6. N5, an optional cross-reference, is not applied.maindoes not make them reachable. The MR-1 compliance check found this case.e95deb1crepeats the "five" overstatement and is left unchanged.6e381b33, returned CHANGES REQUIRED with one blocking fix (R1) and one nit (R2):main.1f66557dapplies R1 and R2 verbatim.1f66557d, confirmed the wording:16fad497reflows that paragraph with no word changed; whitespace-collapsed text is identical to1f66557d.Head
e10a9ba2: CI, then the owner's merge decision. The only change since the confirmed review is the reworded sentence that review proposed.Gates.
5a994209, plus the reviewer's own wording ate10a9ba2.🤖 Generated with Claude Code
https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb