Skip to content

Add Mantle and Pylate targets to lakefile.toml - #17

Open
wkrozowski wants to merge 1 commit into
strata-org:mainfrom
wkrozowski:fix/lakefile-mantle-pylate
Open

wkrozowski wants to merge 1 commit into
strata-org:mainfrom
wkrozowski:fix/lakefile-mantle-pylate

Conversation

@wkrozowski

@wkrozowski wkrozowski commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

Description

Fix CI failure on StrataPython.Mantle.Env with:

error: StrataPython/Mantle/Env.lean:6:0: unknown module prefix 'StrataMantle'

The GitHub mirror excludes lakefile.toml. So when the Mantle and Pylate work landed, its sources (StrataMantle/, StrataMantleTest/, Pylate/, Scripts/pymantle.lean, Scripts/pylate.lean) reached this repo, but the lean_lib/lean_exe declarations that build them did not. No library owns the StrataMantle.* modules here, so Lake never builds them and the import fails.

This PR ports the missing declarations:

  • lean_lib: StrataMantle, StrataMantleTest, Pylate, PylateTests
  • lean_exe: pymantle, pylate, pylateTests
  • defaultTargets matches the internal list
  • StrataPythonFrontEndTestMain.needs adds StrataMantleTest and pylateTests

GitHub-specific parts (the package name, the git/rev require on Strata, and the StrataPythonTestExtra lib) are unchanged.

A follow-up on the mirror side will keep these target declarations from drifting again.

Checklist

  • lake build StrataPython StrataPythonTest succeeds locally (Lean v4.34.0); a plain lake build of all default targets also succeeds
  • New .lean files start with the standard copyright header (no new .lean files)
  • No trailing whitespace and files end with a newline

The mirror from the internal StrataPythonFrontEnd package excludes
lakefile.toml, so the lean_lib/lean_exe declarations added alongside
Mantle (StrataMantle, StrataMantleTest, pymantle) and Pylate (Pylate,
PylateTests, pylate, pylateTests) never reached this repository, even
though their sources did.

Without a lean_lib owning the StrataMantle.* modules, Lake does not
build them and `lake build` fails on StrataPython.Mantle.Env with
"unknown module prefix 'StrataMantle'".

This ports those declarations (and the matching defaultTargets and
StrataPythonFrontEndTestMain needs) from the internal lakefile.
@wkrozowski
wkrozowski requested a review from a team October 28, 2026 12:11
@ssomayyajula

Copy link
Copy Markdown
Contributor

Looks like you need to point the Plausible dependency to the corresponding GitHub URL? Not sure what to make of the test failures yet.

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.

5 participants