feat(sdk): add standalone Agent SDK Host baseline - #1720
Merged
Conversation
limityan
force-pushed
the
yanzhn/sdk-host-runtime-contract
branch
2 times, most recently
from
July 24, 2026 04:32
02418cb to
4a1d3bf
Compare
limityan
force-pushed
the
yanzhn/sdk-host-runtime-contract
branch
from
July 24, 2026 05:11
4a1d3bf to
a02da08
Compare
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
bitfun-sdk-hostcomposition root using newline-delimited JSON-RPC over stdioProduct boundary
This PR delivers an internal, standalone SDK Host implementation candidate. Its handshake reports
not_delivered: it is not a public or stable SDK release, and this PR does not publish Python or TypeScript packages or claim full Claude Agent SDK parity.The Headless CLI does not depend on or start the SDK Host. The SDK Host does not depend on the CLI. They select separate delivery profiles and share only the stable Runtime/port layer and product assembly plan.
Architecture and safety
interfaces/sdk-hostis guarded from depending onbitfun-core,terminal-core, concrete services, the SDK Host app, or CLI; the concrete Host may useterminal-coreonly behind the shared RuntimeTerminalPortCI and test isolation
The Linux/macOS/Windows matrix runs the three
terminal-coreshutdown and binding regressions changed by this PR. It does not adopt the unrelated full terminal suite: an earlier broad gate exposed five existing Unixexec.rsfailures in files this PR does not modify. Fixing those command-control tests is intentionally kept out of the SDK Host scope.Standalone SDK Host process smokes are added on all three platforms. The Ubuntu crate-test lane continues to run the complete SDK Host/app contract and transport suites.
Independent whole-PR review
Three isolated reviewers audited the final rebased commit from architecture/lifecycle, product/competitor, and protocol/concurrency perspectives. All actionable findings were fixed, including CLI/Host dependency confusion, request concurrency, settlement/cancellation ownership, output failure handling, transient Tool projection, exact-ID cleanup after task failure, protocol envelope classification, and CI coverage.
Final result: P0 = 0, P1 = 0, P2 = 0.
Exact-head verification
cargo check --locked --workspacebitfun-services-core --no-default-featuresdunceexcludedcargo test --locked -p bitfun-cligit diff --check