Repository navigation
fix(mcp): preserve exact ownership and verify resource effects - #141
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
Python MCP previously deleted every cached connection at session shutdown, including a sandbox explicitly borrowed from another session with the same credential. Keep creation ownership separate, preserve provider-qualified task handles, and reject explicit CPU/memory requirements that actual platform metadata does not satisfy.
Fixes #139. Fixes #140. Refs Alibaba-Agent-Sandbox/agent-sandbox#46 (remains open).
Changes
provider:id; clients must return it unchanged. Historical bare IDs retain only the documented unique-cache/default-provider scope./tmptest fixture without weakening permissions or runtime behavior.Testing
.agents/evidence/verify/2026-10-26-skill-storage-state-acceptance.mdand desktop evidence.项目网络审计已关闭/502. These are not relabeled passes. Remote final-head Python3.10/3.11/3.12 CI remains required before ready/merge.Documentation and records
Limits
Private test wheels keep0.4.0a6 metadata to match the source checkout; they are not the public0.4.0a6 bits or a new release. No npm/PyPI publication is performed. Desktop uses official optional e2b-desktop compatibility SDK, simulated operator and screenshot timeline, not new methods in the project client or human-login/video certification. Hosted MCP is an owned temporary acceptance service, not a permanent managed production endpoint. No external-VM import or all-provider certification.
Related CLI PR: Alibaba-Agent-Sandbox/agent-sandbox#188. Website separation delivered by https://github.com/Alibaba-Agent-Sandbox/website/pull/65.
Type of Change
Checklist
Definition of Done
bash scripts/check-repo-health.shpasses.Final head d40bd2a: complete CI40269804087 and Docs links40269804106 passed. Remote complete suites establish the normal-environment gate; the four local environmental failures remain retained as failures.