Repository navigation
feat(executor): delegate Lua loading and hooks to execution boundaries - #46
Merged
Merged
Conversation
This was referenced Oct 8, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The SubOS broker in openxlings/xlings#641 needs to load recipes and run hooks in a restricted worker. The current factory and metadata loader execute top-level Lua in the caller before a hook can be isolated.
Add optional execution boundaries for persistent executor operations, recipe metadata loading and index building. A deferred executor performs no local Lua load and delegates every action, retaining the existing cumulative effects and hook context contract. An absent or failed boundary returns an error without falling back to local Lua. Existing overloads retain their behavior for current consumers.
Version: 0.0.61. The client integration and worker transport are in openxlings/xlings#641; this library introduces no JSON or platform transport dependency.
Validation:
mcpp buildwith LLVM 22.1.8 andmcpp test --toolchain gcc@16.1.0 --message-format jsonpassed locally (4 test programs). New cases cover deferred top-level execution, persistent Lua state, cumulative effects, refreshed context, metadata and index builder delegation, and failure without fallback.