Rust SDK API review fixes#1367

Merged

SteveSandersonMS merged 23 commits into

mainfrom

stevesandersonms/rust-sdk-api-review-fixes

May 22, 2026

Commits on May 22, 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
Adapt pre_mcp_tool_call_hook test to Option<Vec<String>> tools field

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 merged 23 commits into
mainfrom
stevesandersonms/rust-sdk-api-review-fixes

Commits on May 22, 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)

Adapt pre_mcp_tool_call_hook test to Option<Vec<String>> tools field