Skip to content

Execute structured Lean imports in isolated REPLs - #31

Open
Deicyde wants to merge 6 commits into
facebookresearch:mainfrom
VivienCabannes:split/04-structured-import-execution
Open

Deicyde wants to merge 6 commits into
facebookresearch:mainfrom
VivienCabannes:split/04-structured-import-execution

Conversation

@Deicyde

@Deicyde Deicyde commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add an optional ordered imports parameter to run_lean_code
  • resolve and revalidate project modules under the request deadline before execution
  • run each nonempty structured-import request in a fresh bounded REPL process and retire it afterward
  • reject mixed or ambiguous source import headers before discovery or process use
  • avoid retries after dispatch and remove process-local environment and proof-state handles from responses
  • preserve the existing path for omitted, null, and empty imports

Depends on #30. The reviewable change is commit 5eb3bc1; earlier commits belong to the dependency stack. This restack preserves the original patch exactly while inheriting #30's repaired response boundary.

Validation

  • range-diff and stable patch ID: unchanged from fe54b7b
  • Python 3.10 cumulative full suite: 798 passed, 5 deselected
  • Python 3.13 cumulative full suite: 797 passed, 1 skipped, 5 deselected
  • focused structured/runtime suite: 293 passed on both Python versions
  • independent restack review: 295 focused tests and exact merge-tree SHIP
  • real Lean suite: 5 passed
  • make lint
  • make check-example
  • git diff --check

Deicyde and others added 5 commits September 18, 2026 16:36
Keep unknown-outcome warnings in public diagnostics. Monitor stdout before and throughout request transmission, and hold back the final frame delimiter so delayed output is rejected before the next command can execute.
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Meta Open Source bot. label Sep 19, 2026
@Deicyde
Deicyde force-pushed the split/04-structured-import-execution branch from fe54b7b to 5eb3bc1 Compare September 20, 2026 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Meta Open Source bot.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant