Skip to content

refactor(runtime): route CLI interaction responses through SDK - #1589

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/runtime-interaction-sdk
Jul 16, 2026
Merged

refactor(runtime): route CLI interaction responses through SDK#1589
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/runtime-interaction-sdk

Conversation

@limityan

Copy link
Copy Markdown
Collaborator

Summary

  • Add typed Agent Runtime SDK contracts for tool confirmation, rejection, and user-answer responses.
  • Route CLI/TUI/Exec interaction responses and Peer Host confirmation/rejection through the SDK while retaining Core ToolPipeline and user-input ownership.
  • Remove only the replaced compatibility-facade methods and centralize user-facing RuntimeError message conversion.
  • Fix issues found while re-reviewing the previous PR: restore/transcript errors no longer leak internal error-kind prefixes, capability documentation now matches each product surface, and typed user-input errors preserve the existing Desktop string contract.

Scope boundaries

  • No approval-policy or runtime-owner migration.
  • No new Peer Host user-answer command.
  • No Desktop, ACP, remote-control, or HarmonyOS runtime adoption in this PR.
  • The Core compatibility facade remains for fixed-ID session creation, model updates, branching, usage, snapshots, and persistence maintenance.

Validation

  • rustup run nightly cargo check --workspace
  • rustup run nightly cargo test -p bitfun-agent-runtime
  • rustup run nightly cargo test -p bitfun-cli
  • Targeted Core interaction-response and previous-PR error-compatibility tests
  • rustup run nightly cargo fmt --all -- --check
  • Core boundary check and its 15 self-tests
  • Repository hygiene check

The installed stable Rust toolchain is locally inconsistent, so Rust validation was run with the installed nightly toolchain. The workspace check completed successfully; only existing MCP deprecation warnings remain.

@limityan
limityan merged commit 1a000b6 into GCWing:main Jul 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant