docs: correct Idris2 region-linearity theorem names#115
Merged
Conversation
The recent README/ROADMAP/EXPLAINME rewrite used `regionSafetyTheorem`
and `noGCTheorem` as the names of two theorems in `src/formal/`, but
the actual definitions in `Ephapax/Formal/RegionLinear.idr` are
`regionSafetyExtract` (line 139) and `noGCExtract` (line 151).
EXPLAINME also cited non-existent paths (`NoEscape.idr` / etc) for
each — all four theorems actually live in `RegionLinear.idr`.
Fixes:
README.adoc:242 — regionSafetyTheorem -> regionSafetyExtract
— noGCTheorem -> noGCExtract
ROADMAP.adoc:126 — regionSafetyTheorem -> regionSafetyExtract
ROADMAP.adoc:127 — noGCTheorem -> noGCExtract
EXPLAINME.adoc — theorem-name fixes + all four file paths
corrected to `RegionLinear.idr:<line>`.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
hyperpolymath
added a commit
that referenced
this pull request
May 21, 2026
…122) Closure sweep after PR #116 took preservation from 22 → 12 open goals and PR #121 landed Phase 1 scaffold. Updates the preservation-count claims across docs to reflect 12, and adds a CHANGELOG entry summarising the full 2026-05-20 → 2026-05-21 reduction campaign. ## Files updated | File | What changed | |---|---| | `README.adoc` | Coq formal-foundations paragraph; reduction chain now lists all 8 PRs through #121; clear pointer to ROADMAP closure plan | | `ROADMAP.adoc` | Formal-proof status table row (`12 open goals`); v0.1 blocker entry; v1.0 status; 97.6% → 98.7% reduction figure | | `EXPLAINME.adoc` | Coq theorem table; reduction-story bullets extended through #114, #116, #121 | | `CHANGELOG.md` | New "Proof state" + "Documentation" sections under `[Unreleased]` tracking the full PR chain (#92, #102, #104, #106, #114, #115, #116, #117, #121) + the Idris2 totality campaign (#89–#100) + doc/wiki refresh (#113 + wiki) | ## Companion wiki update `Home.md`, `Proof-status.md`, `What-can-go-wrong.md` updated in the wiki repo with the same 22 → 12 figures + extended reduction story. Pushed separately to wiki master. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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
1 participant
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.
The recent README/ROADMAP/EXPLAINME rewrite used
regionSafetyTheorem/noGCTheoremfor two theorems insrc/formal/, but the actual names areregionSafetyExtract/noGCExtract(inEphapax/Formal/RegionLinear.idrat lines 139 and 151 respectively). EXPLAINME also cited non-existent paths (NoEscape.idr,RegionSafety.idr,NoGC.idr,Orthogonality.idr) — all four theorems actually live inRegionLinear.idr.Fixed in README.adoc, ROADMAP.adoc, EXPLAINME.adoc.
Companion wiki fix (Home.md, Region-calculus.md, Proof-status.md) pushed separately to the wiki repo.
🤖 Generated with Claude Code