Repository navigation
fix(sdk): complete Express lifecycle, authentication and invocation - #153
Merged
Merged
Conversation
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.
Description
ProviderSandbox now reaches Express pause/resume through its own documented lifecycle protocol and rebinds returned session credentials, so an already used object remains usable after resume. Private application traffic credentials are retained separately from envd and application credentials. Explicit command argument objects preserve normal business fields such as timeout, name and arguments.
Refs #152. Companion CLI work: Alibaba-Agent-Sandbox/agent-sandbox#230 . Node consumer: Alibaba-Agent-Sandbox/nodejs-sdk#23 .
Type of Change
Changes
Testing
Checklist
Definition of Done
.agents/evidence/verify/2026-11-05-express-sdk-completion.mdIntermittent runtime network E8020 results are retained alongside independent successful replacement/clearing and HTTP204 controls. Actual storage mounts need user-supplied storage fixtures and belong to the companion CLI acceptance; they are not claimed by these SDK lifecycle/application checks. Existing API Keys are reused; no Key is created or renewed.
Independent final review of this exact head found no additional SDK defect. A fresh 36-check run, including actual local HTTP invocation, passed. The pre-existing local malformed-WebSocket HTTP501 still reproduced with proxy variables removed; its assertion and the complete successful remote matrix were preserved. The SDK shared contract can land independently of the companion CLI storage fixture acceptance.
Post-merge CI run 42007870638 completed successfully on attempt2. Attempt1 is retained: one unchanged MCP child-PID marker test read an empty file before its write completed, while the other language jobs passed. Two independent reviewers confirmed the existence-before-content race and unchanged provenance; only failed jobs were rerun once, with no assertion or product changes. Node consumer post-merge CI also passed.