feat(execution)!: add fixed compute stdlib - #46
Merged
Conversation
Expose numeric sum, selected JSON helpers, and the pinned math module in every worker. Reserve the fixed language roots across catalog, execution, and worker validation, and keep the model-facing reference synchronized with the runtime. BREAKING CHANGE: Capability names whose top-level segment is a standard Starlark universe name, sum, json, or math now fail Build with ErrInvalidRegistration.
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
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.
Summary
sum(iterable),json.decode/json.encode/json.indent, and the complete pinnedmathmoduleBreaking change
Capability names whose first dotted segment is a standard Starlark universe name,
sum,json, ormathnow failBuildwithErrInvalidRegistration. Nested uses such asstats.sumremain legal. This prevents registered capability namespaces from shadowing the fixed language contract.Validation
mise exec -- go test ./internal/universe ./internal/execution ./internal/catalog ./internal/worker ./mcpserver -count=1mise exec -- moon run root:check --forceFollow-up
#45 tracks recursive depth amplification inside upstream JSON encode/decode. The current worker-process boundary contains a resulting runtime abort to one execution; a complete fix needs to cover both directions and define a separate internal-compute depth contract.
Closes #42
BREAKING CHANGE: Capability names whose top-level segment is a standard Starlark universe name, sum, json, or math now fail Build with ErrInvalidRegistration.