Skip to content

Claude/new session up7579 cut - #40

Merged
hyperpolymath merged 3 commits into
mainfrom
claude/new-session-up7579-cut
Oct 14, 2026
Merged

hyperpolymath merged 3 commits into
mainfrom
claude/new-session-up7579-cut

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Closes #

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (would change existing behaviour)
  • 🕳️ Soundness fix (fixes a checker/proof false-negative)
  • 📖 Documentation
  • 🧹 Refactor / tech debt (behaviour-preserving)
  • ⚡ Performance
  • 🔧 Build / CI / tooling

How has this been verified?

Checklist

  • My commits are signed (git commit -S).
  • I ran the project's own checks/tests locally and they pass.
  • New files carry the correct SPDX-License-Identifier (code/config MPL-2.0,
    prose CC-BY-SA-4.0); I did not relicense existing files.
  • Docs are updated, and no public claim now overstates what the code does.
  • I have not introduced a soundness hole (or I have flagged where I might have).

Notes for reviewers

Maintainer decision on MRC-01's open question: adopt the estate's
v4.34.0 floor (F-11) before any rung-3 module lands, as its own change.

The tree at ea1920a was built on both toolchains before the switch: both
completed with no warnings and the same `#eval orbit`, so no source
change is needed. README's build line and STATE.adoc are updated to
match.

Gates (v4.34.0): lake build -> Build completed successfully (14 jobs);
grep -rnw sorry empty; #eval orbit unchanged.

Rung: 3 (preparation; rung stays OPEN)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01VTeVWDEJgdYKsUNr4uhnXX
Marches/Cut.lean (new): a cut of the region DAG is a set of regions
closed under predecessors.
- Cut.edge_crosses_past: in an acyclic graph every edge leaves the
  principal cut of its source; Cut.path_back_crosses_no_cut: an edge on a
  cycle crosses no cut, so Acyclic is load-bearing.
- DiscSys.trace_cross_crosses_cut: every crossing in a well-typed
  composite trace crosses a cut holding the trace's start, not its end.
- DiscSys.trace_mem_back / trace_inside / no_reentry: a trace ending in a
  cut was inside it all along; a trace that leaves a cut never re-enters
  (no_back_cross lifted from one interface to every cut).
- DiscSys.composeCut: compose restricted to a cut prefix is a Discipline
  (prefix-closure); composeCut_trace_embed / composeCut_trace_reflect: its
  traces are exactly the composite's traces that end in the prefix.
- Two-region Interface: trace_cross_crosses_cut, trace_stays_in_B.
- CutExhibit: a concrete twoTropical trace, and a negative control.

Marches/Tropical.lean: TropicalGrade reads TropicalNat as a (min,+)
grade. seq_unitl and alt_idem are the two laws K-CUT-LOSS uses (K-4).
better_reverses_altLe, altLe_one and s3_witness record that marches'
`better` runs against its own induced order (S-3 / W-4 direction).

Every new declaration depends on no axioms (#print axioms) and nothing
re-declares an Agda definition.

Gates: lake build -> Build completed successfully (15 jobs);
grep -rnw sorry empty; #eval orbit unchanged.

Rung: 3 (rung stays OPEN)

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

coderabbitai Bot commented Oct 14, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 32 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: ed7d1f50-8252-4d16-b7ec-418d5fdfbf08
📥 Commits

Reviewing files that changed from the base of the PR and between f591220 and 70d58e2.

📒 Files selected for processing: 6
  • Marches.lean
  • Marches/Cut.lean
  • Marches/Tropical.lean
  • README.adoc
  • STATE.adoc
  • lean-toolchain
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
@hyperpolymath
hyperpolymath merged commit 9d3af1a into main Oct 14, 2026
6 of 7 checks passed
@hyperpolymath
hyperpolymath deleted the claude/new-session-up7579-cut branch October 14, 2026 01:36
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