From 2737149dfbe9d184229b9da0a352418edf01b3d9 Mon Sep 17 00:00:00 2001 From: Jimmy <1fe240cd1a8cf775f6f3060f115e5a303181f3abf28ad4cb0c2515f4a02b36a8@meshllm.communities.buzz.xyz> Date: Tue, 1 Sep 2026 16:45:42 +1000 Subject: [PATCH] fix(buzz-agent): restore Windows MCP env contract Read the shell-resolution allowlist from buzz-model-catalog, where the shared desktop/agent contract moved. This restores the Windows-only build after the direct RMCP registry change. Co-authored-by: Michael Neale Signed-off-by: Jimmy <1fe240cd1a8cf775f6f3060f115e5a303181f3abf28ad4cb0c2515f4a02b36a8@meshllm.communities.buzz.xyz> --- crates/buzz-agent/src/mcp.rs | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/crates/buzz-agent/src/mcp.rs b/crates/buzz-agent/src/mcp.rs index bd3ad0f6f8..a95410223f 100644 --- a/crates/buzz-agent/src/mcp.rs +++ b/crates/buzz-agent/src/mcp.rs @@ -73,10 +73,11 @@ const PASSTHROUGH_ENV_WINDOWS: &[&str] = &["TMP", "TEMP", "USERPROFILE", "APPDAT /// Shell resolver keys are shared with Doctor through the public contract. #[cfg(windows)] fn windows_child_passthrough_env() -> impl Iterator { - PASSTHROUGH_ENV_WINDOWS - .iter() - .copied() - .chain(crate::WINDOWS_SHELL_RESOLUTION_ENV.iter().copied()) + PASSTHROUGH_ENV_WINDOWS.iter().copied().chain( + buzz_model_catalog::WINDOWS_SHELL_RESOLUTION_ENV + .iter() + .copied(), + ) } type Client = RunningService; @@ -1039,7 +1040,7 @@ mod content_tests { ); } let child_env: Vec<_> = windows_child_passthrough_env().collect(); - for var in crate::WINDOWS_SHELL_RESOLUTION_ENV { + for var in buzz_model_catalog::WINDOWS_SHELL_RESOLUTION_ENV { assert!( child_env.contains(var), "{var} must pass through so the MCP shell resolver matches Doctor"