Skip to content

refactor: migrate to the new do elaborator - #33

Merged
digama0 merged 4 commits into
digama0:masterfrom
kim-em:agent/migrate-new-do-elaborator
Aug 4, 2026
Merged

refactor: migrate to the new do elaborator#33
digama0 merged 4 commits into
digama0:masterfrom
kim-em:agent/migrate-new-do-elaborator

Conversation

@kim-em

@kim-em kim-em commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Follows #30 and is rebased onto its merged result.

Advances #31. The independent Experimental/ShapeLogRel.lean migration remains pinned to the legacy elaborator and is deliberately left for separate work.

Summary

  • remove the three backward.do.legacy compatibility pins covering TypeChecker, EquivManager, and Verify/TypeChecker/Reduce
  • migrate the verifier proofs to the new do elaborator
  • leave Experimental/ShapeLogRel.lean unchanged to avoid proof-quality regressions and conflicts with ongoing work

Verification

  • lake build
  • lake build Lean4Lean.Experimental
  • lake exe lean4lean Init.Core
  • lake exe lean4lean --fresh Init.System.IO

Coverage note

Removing the TypeChecker.lean pin changes elaborated terms in code paths whose existing .WF theorems still contain sorry (reduceRecursor, reduceProj, inferProj, tryEtaStructCore, and isDefEqUnitLike). The fresh Init.System.IO replay is therefore part of the safety check, rather than relying only on proof coverage.

@kim-em
kim-em force-pushed the agent/migrate-new-do-elaborator branch from 17b0a30 to 114fd7f Compare August 4, 2026 01:23
@digama0

digama0 commented Aug 4, 2026

Copy link
Copy Markdown
Owner

leave ShapeLogRel, the change is basically all regression plus this is going to conflict with other branches

@kim-em

kim-em commented Aug 4, 2026

Copy link
Copy Markdown
Contributor Author

Done in dc76fec: restored Lean4Lean/Experimental/ShapeLogRel.lean to the upstream version, including the legacy elaborator pin. I also updated the PR description to leave that migration in #31. Both lake build and lake build Lean4Lean.Experimental pass.

@digama0
digama0 marked this pull request as ready for review August 4, 2026 03:35
@digama0
digama0 merged commit 408edad into digama0:master Aug 4, 2026
1 check passed
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