Skip to content

fix: run Lake operations in target process - #244

Merged
ejgallego merged 1 commit into
mainfrom
codex/issue-239-lake-env
Aug 31, 2026
Merged

fix: run Lake operations in target process#244
ejgallego merged 1 commit into
mainfrom
codex/issue-239-lake-env

Conversation

@ejgallego

@ejgallego ejgallego commented Aug 30, 2026

Copy link
Copy Markdown
Collaborator

This PR keeps Lake workspace loading and zero-build save trace generation in a target-built process, so MCP servers can safely serve projects whose Lean commit differs from the MCP executable.

The broker exchanges only typed JSON metadata with the bundled target helper while continuing to own the Lean server directly. This preserves the target Lake environment and server arguments, keeps live Lake values from crossing Lean ABIs, and retains atomic save-trace publication. MCP accepts either the complete beam-cli runtime or an explicit command/plugin pair with a sibling helper; standalone same-build daemons continue to use Lake in-process.

@ejgallego
ejgallego force-pushed the codex/issue-239-lake-env branch from 177a169 to 84b7fa1 Compare August 30, 2026 20:21
Base automatically changed from codex/issue-239-diagnostics to main August 30, 2026 20:26
@ejgallego
ejgallego force-pushed the codex/issue-239-lake-env branch 4 times, most recently from 9b7bc2e to bc9f047 Compare August 30, 2026 22:15
@ejgallego ejgallego changed the title fix: retry workspace setup with target Lake fix: run Lake operations in target process Aug 30, 2026
@ejgallego
ejgallego force-pushed the codex/issue-239-lake-env branch from bc9f047 to ff34086 Compare August 30, 2026 22:55
@ejgallego
ejgallego force-pushed the codex/issue-239-lake-env branch from ff34086 to ef5515f Compare August 30, 2026 23:52
@ejgallego
ejgallego marked this pull request as ready for review August 31, 2026 00:22
@ejgallego
ejgallego merged commit d2d82d2 into main Aug 31, 2026
31 checks passed
@ejgallego
ejgallego deleted the codex/issue-239-lake-env branch August 31, 2026 00:22
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