Repository navigation
Delete the guest Runtime daemon - #562
Merged
Merged
Conversation
Relocate the rules the agent host and launcher share with the guest: the workspace path rule to proto, a Skill's root to agentcapabilities, the export bounds and setup grace to agenthost, and the private-file reader beside executorCredential.
The agent host is the only Runtime: every Harness runs in a view of the
Session's sandbox, and the sandbox serves only Sandbox I/O.
- cli: delete connect, resume, runtime-mcp-exec, status, logout and the
guest suspend control, with the auth, placement and localworkspace
packages. Keep install, start, agent-host, stop, logs and version.
- agent: Runtime is {Info, View}; Register composes the declaration and
registers the kind and its view; discovery takes only the diagnostic
writers.
- adapters: delete the local Executor factories, workspace and tool
environment paths and the HTTP MCP bearer rendering the gateway rule
already forbids in a view.
- dispatch: delete Router.Quiesce and Resume; suspension arrives only as
environment_quiesce and environment_resume frames.
- claude-sdk-adapter and mcode-harness: delete tool_env, runtime MCP
exec and the guest launcher branches.
- Docs and CI: describe the agent-host model in the Harness, Runtime
protocol and Environment contracts, and narrow the native CI steps to
the code that remains.
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
1 participant
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.
Every Session now runs its native Harness on an agent host, but the daemon still carried the guest executor, local workspace, bootstrap and CLI paths. Remove those paths and keep the agent-host command and the self-hosted launcher that supervises Sandbox I/O. Shared rules move to their existing protocol or capability owner; the agent host's registered Harness declarations remain unchanged.
Remove native acceptance fixtures that launch the deleted
connectcommand and update the owning protocol and qualification documentation in English and Chinese. Their public API acceptance scenarios remain required for final cutover qualification; view tests alone do not qualify them.Validation includes Linux build/vet, daemon/internal Darwin and Windows cross-builds, affected Go suites and race tests, adapter Node tests, runtime contract checks and repository hygiene. Independent review found no remaining in-scope defects. Historical real Claude and MiniMax qualification and identical declaration dumps were verified against unchanged adapter/agent-host code; this cleanup does not claim a new full-system live qualification.
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.