Skip to content

Require complete project-root indexes in module-system packages - #560

Merged
Vilin97 merged 1 commit into
mainfrom
codex/enforce-project-root-index
Oct 1, 2026
Merged

Vilin97 merged 1 commit into
mainfrom
codex/enforce-project-root-index

Conversation

@Vilin97

@Vilin97 Vilin97 commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

Now that #559 migrated every project, make the project-root layout mandatory for module-system packages. The quality gate and local mk_all must reject a flat LeanPool.lean even if every Imports.lean is deleted, and regenerate exactly one complete public aggregate per project. Keep independent legacy non-module fixtures supported; fix the rebase fast-path fixture to use valid TOML.

Validation: 561 Python tests passed (2 unrelated optional skips), including deletion of all aggregates, complete public export compilation, stale/missing/duplicate/private/cross-project imports, collision safety and mechanical rebase conflicts. Ruff, formatting and lake exe mk_all --module --check passed. The complete content migration passed whole-tree static quality checks before merge. No formalization sources, thresholds or branch protection settings change.

@Vilin97
Vilin97 merged commit ec381e1 into main Oct 1, 2026
6 checks passed
@greptile-apps

greptile-apps Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Advisory cross-file review against Lean Pool's supervisory rubric. This is not by itself a significance, source-verification, or merge verdict.

Retrigger

[Medium risk] Enforces stricter module indexing rules in the build system.

The PR appears safe to merge; no actionable regression was identified.

Summary

This PR makes project-root aggregates mandatory for module-system packages while preserving flat indexes for non-module fixtures.

  • Enables the Python index gate to recognize requiresModuleSystem = true even when every project aggregate has been deleted.
  • Gives the rebase fast-path fixture valid TOML so the new configuration check can read it.

Reviews (1) · Last reviewed commit: "Require complete project-root indexes in..."

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.

1 participant