Repository navigation
Declare each Sandbox Provider's setup in one SetupDeclaration - #844
Merged
Merged
Conversation
This comment has been minimized.
This comment has been minimized.
SaladDay
force-pushed
the
declare-sandbox-provider-setup
branch
2 times, most recently
from
October 23, 2026 20:33
330cecf to
14dab15
Compare
Web follows the declared mode, fields and presets, discovery is typed (field and path in, options out), and the per-provider validators and Web provider knowledge are deleted.
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.
Each Provider authors one
sandbox.SetupDeclarationbesidePolicy(); registration checks it against the Provider's requirements. The E2B adapter derives its allow-lists from it, discovery is typed ({field, path}in,ConfigurationOptionsout), and Web follows the declared mode, fields and presets with no Provider names. The per-Provider validator wrappers, duplicate Web validators and E2B helpers are deleted. The declaration reaches Web only through the generateddeployment-contract.ts. Docs updated in English and Chinese.Need help on this PR? Tag
@codesmith-botwith what you need.