Skip to content

AGENTS.md: Main must be independent of downstream companion papers - #1041

Merged
amaybaum merged 1 commit into
mainfrom
docs/main-proof-independence-r2-20261102
Nov 2, 2026
Merged

amaybaum merged 1 commit into
mainfrom
docs/main-proof-independence-r2-20261102

Conversation

@amaybaum

@amaybaum amaybaum commented Nov 2, 2026

Copy link
Copy Markdown
Owner

This PR adopts the Main proof-independence policy approved by the owner on 2026-11-02 (#1029). It is a separate minimum-change PR, kept apart from the correction units for Main. Merge is not authorized.

It replaces #1040. That PR has the same tree, but its commit message called a paraphrase of the owner's text "verbatim". This commit's message quotes the owner's wording exactly. #1040 is closed unmerged, and its branch is kept.

Change

The PR adds one bullet to AGENTS.md, under "Code Review Rules → Whole-framework scope", directly after the bullet on the whole proof corpus. The text is the owner-approved wording, verbatim and re-wrapped to the section's line width:

papers/Main.md must be independent of downstream companion papers. Main must not reference or cite downstream results, including through indirect labels or unnamed references. Every claim, proof, hypothesis and scope qualification in Main must retain sufficient support within Main, through its own argument or an appropriately cited, independently verifiable theorem or evidence record, with all necessary assumptions explicit. Genuine foundational inputs may be cited as inputs, with their content and assumptions stated. Whether an item is foundational or downstream is determined by its actual logical role, not merely by the paper containing it. Removing a reference without preserving the claim's support is not a correction: unsupported claims must be independently established, narrowed or withdrawn.

The PR changes one file, adding 13 lines and deleting none, in one commit on b26c2ff0. Its tree is 31520449…, identical to #1040's reviewed tree.

Scope

Review of the tree (from #1040)

The independent exact-text review of this tree found:

  • the inserted bullet equals the owner's text exactly;
  • nothing else in AGENTS.md changed;
  • there is no substantive conflict with the rest of AGENTS.md;
  • both strings pinned by the R7-MSP guard are still present;
  • artifact_placement_check passes;
  • the change merges cleanly.

Its only blocker was the commit message, which this PR corrects.

#1036 and #1037 stay paused, and the five standing holds (#1009, #1023, #1011, #1003, #918) are unchanged.


Generated by Claude Code

Adds one bullet to Code Review Rules, "Whole-framework scope", directly
after the bullet on the whole proof corpus. The bullet is the owner's
approved wording (#1029, 2026-11-02), verbatim:

`papers/Main.md` must be independent of downstream companion papers. Main
must not reference or cite downstream results, including through indirect
labels or unnamed references. Every claim, proof, hypothesis and scope
qualification in Main must retain sufficient support within Main, through
its own argument or an appropriately cited, independently verifiable
theorem or evidence record, with all necessary assumptions explicit.
Genuine foundational inputs may be cited as inputs, with their content and
assumptions stated. Whether an item is foundational or downstream is
determined by its actual logical role, not merely by the paper containing
it. Removing a reference without preserving the claim's support is not a
correction: unsupported claims must be independently established, narrowed
or withdrawn.

The bullet states a requirement; Main's present compliance is audited
separately. The neighbouring whole-corpus review bullets are unchanged.
No other file changes.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb

amaybaum commented Nov 2, 2026

Copy link
Copy Markdown
Owner Author

Exact-head review record: cde5ae2a

Verdict: READY-WITH-NITS. The review was independent. Main is at b26c2ff047220da968424db0bda3c9a9d7092994, and the head is cde5ae2a07e7993ecb89f981c01f2f7bf1e4b37b, a single commit on b26c2ff0.

Tree. The tree is 31520449ac91d2075f4fa31367958b1e59057122, identical to #1040's reviewed head ef58c7fd, so every tree-level finding carries over:

  • The diff is to AGENTS.md only: 13 lines added, none deleted.
  • The new bullet sits in "Whole-framework scope", directly after "…not Main alone.".
  • After whitespace collapse, the bullet equals the owner-approved text byte for byte.
  • Deleting the bullet restores the base blob exactly.
  • No substantive conflict with the rest of AGENTS.md. Checked against Whole-framework scope, Claim/evidence boundary, Exactness and controls, §A.25, §A.27, §A.28, the preamble, §A.30, §A.34 and §A.42.
  • The R7-MSP guard's two pinned §A.35 strings are still present, and no other live check reads or hashes AGENTS.md.
  • artifact_placement_check passes.

Commit message. The quoted paragraph equals the owner's text exactly, and the message is otherwise accurate. This was the blocker on #1040.

PR body. It is accurate, including the statement that the bullet is a requirement and not a claim that Main already complies.

Mergeability. git merge-tree --write-tree origin/main cde5ae2a is clean, and its result tree equals the head's tree.

CI at cde5ae2a. Run 41451639275 is green: 25 jobs passed, and 1 (A42 exclusion (${{ matrix.part }})) was skipped by its condition.

Nits.

  • The reviewer listed four optional wording notes on the owner's text. They are non-blocking and were not applied, since the text is the owner's approved wording.
  • A footer nit came from the host's own rewriting of the PR footer, so nothing is needed.

No merge is authorized. Merging needs separate exact-SHA approval from the owner.


Generated by Claude Code

@amaybaum
amaybaum marked this pull request as ready for review November 2, 2026 13:29
@amaybaum
amaybaum merged commit a65fe8a into main Nov 2, 2026
26 checks passed
@amaybaum
amaybaum deleted the docs/main-proof-independence-r2-20261102 branch November 2, 2026 13:29

amaybaum commented Nov 2, 2026

Copy link
Copy Markdown
Owner Author

Exact-head owner-approval merge record — 2026-11-02

Approved and merged the reviewed policy-only PR at head cde5ae2a07e7993ecb89f981c01f2f7bf1e4b37b.

  • Independent review: READY-WITH-NITS, record 6578373246; nonblocking optional wording/footer notes.
  • Exact-head ordinary PR CI: 41451639275, 25 success, 1 expected skip.
  • Base/main just before merging: b26c2ff047220da968424db0bda3c9a9d7092994; exact head unchanged; clean merge.
  • GitHub merge result and independently read back new main: a65fe8a844e774a5737d47b1cb49db9ab0751077.
  • Scope: one new bullet in AGENTS.md (13 added lines only) under Whole-framework scope. No manuscripts, Lean, LEDGER, census, CI or generated documents changed.
  • Ordinary post-merge main verify 41453409888 was queued at the time of this record; pre-merge CI success is not a post-merge conclusion.

Meaning: proof independence from downstream companions is now a normative requirement, not a claim that papers/Main.md already satisfies it. MI-1 on #1029 must still pursue the reviewed scientific-correction-first unit sequence, starting with S1's separately scoped proposal/review. Existing holds on #1009, #1023, #1011, #1003 and #918 unchanged; #1036 and #1037 remain paused. No new scientific proof or physics-premise promotion authorized.

amaybaum commented Nov 2, 2026

Copy link
Copy Markdown
Owner Author

Post-merge verification: a65fe8a8

  • Merge commit: a65fe8a844e774a5737d47b1cb49db9ab0751077, with parents b26c2ff0 (previous main) and cde5ae2a (the reviewed head).
  • Tree: 31520449ac91d2075f4fa31367958b1e59057122, identical to the reviewed head's tree.
  • main CI on a65fe8a8: verify, push event, run 41453409888, is green. 25 jobs passed and 1 (A42 exclusion (${{ matrix.part }})) was skipped by its condition.

The Main proof-independence rule is now in AGENTS.md ("Whole-framework scope"). It states a requirement. Main's compliance remains an open correction obligation, MI-1 on #1029, which runs as units S1, S2, S3, P1, C1, C2 and L1. #1036 and #1037 stay paused, and the five standing holds (#1009, #1023, #1011, #1003, #918) are unchanged.


Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants