Skip to content

Commit 9b507f5

Browse files
fix(rmcp-macros): replace deprecated *Param type aliases with *Params (#727)
The `#[task_handler]` macro generates code using deprecated type aliases (`PaginatedRequestParam`, `CallToolRequestParam`, `GetTaskInfoParam`, `GetTaskResultParam`, `CancelTaskParam`) that were renamed to `*Params` in rmcp 0.13.0. This causes 5 deprecation warnings for every crate using the macro. Update all references to use the canonical `*Params` names: - `PaginatedRequestParam` → `PaginatedRequestParams` - `CallToolRequestParam` → `CallToolRequestParams` - `GetTaskInfoParam` → `GetTaskInfoParams` - `GetTaskResultParam` → `GetTaskResultParams` - `CancelTaskParam` → `CancelTaskParams` Also fix the corresponding doc examples in `lib.rs`. Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
1 parent 5c5a2e7 commit 9b507f5

2 files changed

Lines changed: 7 additions & 7 deletions

File tree

crates/rmcp-macros/src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ pub fn tool_router(attr: TokenStream, input: TokenStream) -> TokenStream {
144144
/// impl ServerHandler for MyToolHandler {
145145
/// async fn call_tool(
146146
/// &self,
147-
/// request: CallToolRequestParam,
147+
/// request: CallToolRequestParams,
148148
/// context: RequestContext<RoleServer>,
149149
/// ) -> Result<CallToolResult, rmcp::ErrorData> {
150150
/// let tcc = ToolCallContext::new(self, request, context);
@@ -153,7 +153,7 @@ pub fn tool_router(attr: TokenStream, input: TokenStream) -> TokenStream {
153153
///
154154
/// async fn list_tools(
155155
/// &self,
156-
/// _request: Option<PaginatedRequestParam>,
156+
/// _request: Option<PaginatedRequestParams>,
157157
/// _context: RequestContext<RoleServer>,
158158
/// ) -> Result<ListToolsResult, rmcp::ErrorData> {
159159
/// let items = self.tool_router.list_all();

crates/rmcp-macros/src/task_handler.rs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
3333
let list_fn = quote! {
3434
async fn list_tasks(
3535
&self,
36-
_request: Option<rmcp::model::PaginatedRequestParam>,
36+
_request: Option<rmcp::model::PaginatedRequestParams>,
3737
_: rmcp::service::RequestContext<rmcp::RoleServer>,
3838
) -> Result<rmcp::model::ListTasksResult, McpError> {
3939
let running_ids = (#processor).lock().await.list_running();
@@ -61,7 +61,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
6161
let enqueue_fn = quote! {
6262
async fn enqueue_task(
6363
&self,
64-
request: rmcp::model::CallToolRequestParam,
64+
request: rmcp::model::CallToolRequestParams,
6565
context: rmcp::service::RequestContext<rmcp::RoleServer>,
6666
) -> Result<rmcp::model::CreateTaskResult, McpError> {
6767
use rmcp::task_manager::{
@@ -116,7 +116,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
116116
let get_info_fn = quote! {
117117
async fn get_task_info(
118118
&self,
119-
request: rmcp::model::GetTaskInfoParam,
119+
request: rmcp::model::GetTaskInfoParams,
120120
_context: rmcp::service::RequestContext<rmcp::RoleServer>,
121121
) -> Result<rmcp::model::GetTaskResult, McpError> {
122122
use rmcp::task_manager::current_timestamp;
@@ -176,7 +176,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
176176
let get_result_fn = quote! {
177177
async fn get_task_result(
178178
&self,
179-
request: rmcp::model::GetTaskResultParam,
179+
request: rmcp::model::GetTaskResultParams,
180180
_context: rmcp::service::RequestContext<rmcp::RoleServer>,
181181
) -> Result<rmcp::model::GetTaskPayloadResult, McpError> {
182182
use std::time::Duration;
@@ -232,7 +232,7 @@ pub fn task_handler(attr: TokenStream, input: TokenStream) -> syn::Result<TokenS
232232
let cancel_fn = quote! {
233233
async fn cancel_task(
234234
&self,
235-
request: rmcp::model::CancelTaskParam,
235+
request: rmcp::model::CancelTaskParams,
236236
_context: rmcp::service::RequestContext<rmcp::RoleServer>,
237237
) -> Result<rmcp::model::CancelTaskResult, McpError> {
238238
use rmcp::task_manager::current_timestamp;

0 commit comments

Comments
 (0)