Skip to content

repair(#1088): correct two outdated provenance references in SUBSTATE-2 ANALYSIS.md - #1107

Merged
amaybaum merged 1 commit into
mainfrom
repair/substate2-analysis-20261104
Nov 4, 2026
Merged

amaybaum merged 1 commit into
mainfrom
repair/substate2-analysis-20261104

Conversation

@amaybaum

@amaybaum amaybaum commented Nov 4, 2026

Copy link
Copy Markdown
Owner

Follow-up 1 of the two kept open under PR-AUDIT-1 (#1088 comment 6640054062).

Owner authorization (amaybaum, Claude Code session chat, 2026-11-04): "ANALYSIS.md — Approved. Authorize a narrowly scoped documentary correction PR. Correct the two outdated references. Verify their replacements against current main. Change no scientific conclusion, proof or verification status. Obtain independent review and CI before presenting the PR for merge approval."

Head

75fd5ab8c16eaefa447871ae5b499f3206a57121, single parent 150c5b95 (current main). One file, two lines changed; the line count is unchanged.

The two corrections in verification/programmes/substratum/recursive-substates/ANALYSIS.md

Line Was Now Checked against main
3 "for issue #1010 / draft PR #1011." "for issue #1010; landed on main by #1096 (#1011 closed unmerged)." ANALYSIS.md was added by 5e8c496f, which entered main through #1096; #1011 is closed and unmerged.
56 "The new OIBridge.RecursiveFiniteFactor Lean candidate is only" "The OIBridge.RecursiveFiniteFactor module (landed by #1096; built and axiom-checked in CI) is only" The module is imported at verification/lean-mathlib/OIBridge.lean:25; the census lists it kernel-only with "landed on main by #1096 … The module builds and its axiom report passes in CI"; main push run 41897206021 at 150c5b95 has Build and Release gate successful.

The rest of the line-56 paragraph, which limits what the module proves ("only the second (finite-family aggregation assuming individual stage factorization) plus a periodic-family wrapper assuming the observation-period identity. It must not be described as a kernel proof of the topological lemma or of a physical OI recurrence."), is unchanged. No conclusion, proof or verification status changes.

Local checks at the head

legacy_records_check.py: 303 records intact. citation_roles_check.py, duplicate_check, staleness_check, artifact_placement_check, lean_manuscript_census, voice_check, claims_check: exit 0.

Still needed before merge

Independent exact-head review, CI at the exact head, and separate owner merge approval of the reviewed SHA.


Generated by Claude Code

Owner-approved follow-up under #1088 (session chat, 2026-11-04).

- :3 "issue #1010 / draft PR #1011" -> landed on main by #1096; #1011
  closed unmerged.
- :56 "The new OIBridge.RecursiveFiniteFactor Lean candidate" -> the module
  landed by #1096, built and axiom-checked in CI (imported at
  OIBridge.lean:25; census kernel-only).

The sentence's scope limits are unchanged. No conclusion, proof or status
changes; the line count is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013T6pfKuv5mi3MXtKjH2tZb
@amaybaum
amaybaum marked this pull request as ready for review November 4, 2026 19:49
@amaybaum
amaybaum merged commit 449b8fa into main Nov 4, 2026
26 checks passed
@amaybaum
amaybaum deleted the repair/substate2-analysis-20261104 branch November 4, 2026 19:49

amaybaum commented Nov 4, 2026 •

Copy link
Copy Markdown
Owner Author

Merged. Merge commit 449b8fa8e732995798b7cbca473cf38b57a037fa, pinned to head 75fd5ab8c16eaefa447871ae5b499f3206a57121.

Owner approval (amaybaum, Claude Code session chat, 2026-11-04): "I approve the normal merge of PR #1107 at exact SHA: 75fd5ab8c16eaefa447871ae5b499f3206a57121".

Immediately before the merge:

  • The remote head was 75fd5ab8 and the base was main 150c5b95; GitHub reported the PR mergeable and clean.
  • Exact-head review: ACCEPT, no blocking findings.
  • CI run 41898792124 at the head concluded success: 25 checks succeeded and 1 was skipped by design; in Mathlib bridge, Build and Release gate succeeded.

After the merge:

  • The parents are 150c5b95 and 75fd5ab8, and the tree (1b749707) equals the head's tree.
  • git diff 150c5b95 449b8fa8 changes only recursive-substates/ANALYSIS.md lines 3 and 56 and equals the reviewed diff.
  • legacy_records_check.py at 449b8fa8: 303 records in 75 closed namespaces, all intact.
  • main CI on 449b8fa8: push run 41900746614 concluded success; 25 checks succeeded and 1 was skipped by design. In the Mathlib bridge job, Build and Release gate both succeeded.

Noted by the review, not changed (out of scope): recursive-substates/README.md:79 still says "candidate" for the module; "draft PR #1011" remains in the opening lines of fixed-scale-skew/ and native-padic-bridge/ PREREGISTRATION.md/REPORT.md and f2-1-coherence-sourcing/preregistration.md:4.


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