Repository navigation
refactor: separate client installation from local server runtime - #160
Merged
Merged
Conversation
Merged
8 of 11 tasks
anycodes
marked this pull request as ready for review
November 11, 2026 01:05
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
Installing the Python client previously installed the local server runtime even for remote-only use. Move the exact matching server version to the optional
[server]extra; keep standard, asynchronous, Provider and HTTP client entry points independent of that extra. Remove the decorator factory's redundant local execution shortcut while retaining real remote invocation and server registration.Refs #157.
Type of Change
Changes
Package validation now installs the real client wheel without the runtime, then installs its matching extra and tests registration. Bilingual usage, release validation and the command responsibility map are aligned.
Testing
[server]installation/registration and pip checkChecklist
Definition of Done
Draft while complete-suite/E2E gates remain blocked. Evidence:
.agents/evidence/verify/2026-11-10-client-runtime-boundaries.md. No cloud or release result is inferred from local installation.Coordination: Alibaba-Agent-Sandbox/agent-sandbox#244. CLI examples explicitly require the current source CLI; no package release is claimed.
Source integration decision — 2026-11-11
The maintainer explicitly requested that the already-pushed content be merged into main now, ahead of the remaining acceptance work. This supersedes the earlier Draft hold for source integration only. It does not assert completion of unrun cloud acceptance, resolve all coordinated issues, qualify changed template identities, or authorize npm/PyPI releases. Historical evidence remains unchanged.
Current reviewed head:
17f41a16bf72babbe68c5d9ae8b8b860a7dba138. Exact-head CI is green.Normal GitHub merge is used with the expected head SHA. Repository main has no active branch rules or required review requests. Retain open acceptance issues and the documented gaps; source merge is separate from release and full validation.