From e041acbb21bab3c1e497d56efda616eb09365db1 Mon Sep 17 00:00:00 2001 From: Ruben Fiszel Date: Fri, 17 Jul 2026 13:33:11 +0000 Subject: [PATCH] docs: trim mcp path-param fallback helper comment Co-Authored-By: Claude Opus 4.8 --- backend/windmill-api/src/mcp/utils.rs | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/backend/windmill-api/src/mcp/utils.rs b/backend/windmill-api/src/mcp/utils.rs index 2fe102e670..8c20124066 100644 --- a/backend/windmill-api/src/mcp/utils.rs +++ b/backend/windmill-api/src/mcp/utils.rs @@ -451,13 +451,10 @@ pub fn build_query_string( } } -/// Look up the argument supplied for the path parameter whose original (un-mangled) -/// name is `original_name`. +/// Value of the path parameter whose original (un-mangled) name is `original_name`. /// -/// A body field and a path parameter sharing an original name are exposed to the client -/// under distinct mangled names (`path__body` / `path__path`), and the body field is -/// optional. Callers routinely send only `path__path`, which must still produce a body -/// carrying `path` — the API requires it and rejects the request otherwise. +/// A same-named body field is optional and clients routinely omit it, but the API still +/// requires the key in the body and rejects the request without it. fn same_named_path_param_value<'a>( original_name: &str, args_map: &'a serde_json::Map,