feat(acp): route session/setTitle to the new schema types#164
feat(acp): route session/setTitle to the new schema types#164daniel-agentee wants to merge 1 commit into
Conversation
34523b3 to
da8a108
Compare
|
Updated this branch for the current schema PR: bumped @codex review |
da8a108 to
34523b3
Compare
Wires the SetSessionTitleRequest / SetSessionTitleResponse types into the agent-side JSON-RPC dispatch tables. Companion to agentclientprotocol/agent-client-protocol#1199, which adds the schema definitions for session/setTitle. This PR is the Rust SDK half of the round trip: once the schema crate ships a release with the new types, ACP-backed agents can implement Agent::set_session_title and have it dispatched here.
34523b3 to
79bb427
Compare
|
@codex review |
|
Refreshed this branch onto current |
|
@codex review |
Summary
Wires the
SetSessionTitleRequest/SetSessionTitleResponsetypes into the agent-side JSON-RPC dispatch tables, so ACP-backed agents can implementAgent::set_session_titleand have it dispatched here.Why
This is the Rust SDK half of
session/setTitle. Once the schema crate ships a release containing those types, this routing makes them callable.What's in this PR
src/agent-client-protocol/src/schema/client_to_agent/requests.rs— addsSetSessionTitleRequest/SetSessionTitleResponseto the imports and registers them withimpl_jsonrpc_request!against"session/setTitle".src/agent-client-protocol/src/schema/enum_impls.rs— addsSetSessionTitleRequestto theClientRequestenum dispatch so the agent-side connection knows to deserialize it as a setTitle request.Total: 7 added lines, 2 files.
Stacking + CI status
Draft. CI is currently red because the types
SetSessionTitleRequest/SetSessionTitleResponsedo not yet exist in any releasedagent-client-protocol-schemaversion on crates.io — they live behind agentclientprotocol/agent-client-protocol#1199. Once that schema PR merges and a release ships (likelyagent-client-protocol-schema = 0.14.0), bump the workspace dep here from=0.13.0to the new version and CI will go green with no other changes.Verified locally against the patched schema crate (cargo build / cargo test --all-features --workspace), all 51 tests pass.
Companion PRs
[patch.crates-io]until the release lands)