Rust SDK API review fixes#1367

Open

SteveSandersonMS wants to merge 22 commits into

mainfrom

stevesandersonms/rust-sdk-api-review-fixes

Commits on May 21, 2026

Phase A: renames (get_messages, disable_resume, destroy)

SteveSandersonMS
and
Copilot
committed
Phase B: MCP tools tri-state Option<Vec<String>>

SteveSandersonMS
and
Copilot
committed
Phase C: drop LogLevel::Info default

SteveSandersonMS
and
Copilot
committed
Phase F: InputOptions -> UiInputOptions

SteveSandersonMS
and
Copilot
committed
Phase D: connection consolidation

SteveSandersonMS
and
Copilot
committed
Phase H + I: multi-client broadcast gating + permission-policy ordering

SteveSandersonMS
and
Copilot
committed
Phase K: docs, examples, scenarios

SteveSandersonMS
and
Copilot
committed
Cross-SDK: derive requestPermission from handler presence on session.create

SteveSandersonMS
and
Copilot
committed
Phase H redo (lib): five optional handler traits + wire-struct split

SteveSandersonMS
and
Copilot
committed
Phase H redo: migrate tests, examples, and scenarios to per-trait handler API

SteveSandersonMS
and
Copilot
committed
Phase H redo: fix lingering subscription.rs doc reference to removed handler types

SteveSandersonMS
and
Copilot
committed
fmt: apply nightly rustfmt with grouped imports config

SteveSandersonMS
and
Copilot
committed
Fix rustdoc broken intra-doc link in permission.rs and redundant link in session.rs

SteveSandersonMS
and
Copilot
committed
Phase H redo cleanup: drop dead request_* fields from public configs

SteveSandersonMS
and
Copilot
committed
Fix mcp-servers scenario for new Option<Vec<String>> tools field

SteveSandersonMS
and
Copilot
committed
Drop redundant explicit link target in types.rs ElicitationHandler doc

SteveSandersonMS
and
Copilot
committed
Fix triple-encoded mojibake in session_test.rs comments

SteveSandersonMS
and
Copilot
committed
Drop removed request_user_input assignment in user-input scenario

SteveSandersonMS
and
Copilot
committed
Unify tool registration: Tool::with_handler + with_tools (drop with_tool_handlers)

SteveSandersonMS
and
Copilot
committed
Rename with_transform to with_system_message_transform; rename transform field to system_message_transform

SteveSandersonMS
and
Copilot
committed
Drop redundant with_tools(declaration-only) calls in multi_client tests

SteveSandersonMS
and
Copilot
committed
Fix README elicitation example to use ElicitationRequest (not ElicitationRequestData)

SteveSandersonMS
and
Copilot
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Rust SDK API review fixes#1367

Rust SDK API review fixes#1367
SteveSandersonMS wants to merge 22 commits into
mainfrom
stevesandersonms/rust-sdk-api-review-fixes

Commits on May 21, 2026

Phase A: renames (get_messages, disable_resume, destroy)

Phase B: MCP tools tri-state Option<Vec<String>>

Phase C: drop LogLevel::Info default

Phase F: InputOptions -> UiInputOptions

Phase D: connection consolidation

Phase H + I: multi-client broadcast gating + permission-policy ordering

Phase K: docs, examples, scenarios

Cross-SDK: derive requestPermission from handler presence on session.create

Phase H redo (lib): five optional handler traits + wire-struct split

Phase H redo: migrate tests, examples, and scenarios to per-trait handler API

Phase H redo: fix lingering subscription.rs doc reference to removed handler types

fmt: apply nightly rustfmt with grouped imports config

Fix rustdoc broken intra-doc link in permission.rs and redundant link in session.rs

Phase H redo cleanup: drop dead request_* fields from public configs

Fix mcp-servers scenario for new Option<Vec<String>> tools field

Drop redundant explicit link target in types.rs ElicitationHandler doc

Fix triple-encoded mojibake in session_test.rs comments

Drop removed request_user_input assignment in user-input scenario

Unify tool registration: Tool::with_handler + with_tools (drop with_tool_handlers)

Rename with_transform to with_system_message_transform; rename transform field to system_message_transform

Drop redundant with_tools(declaration-only) calls in multi_client tests

Fix README elicitation example to use ElicitationRequest (not ElicitationRequestData)