change tool to only return summary information

Author: SteveL-MSFT

dsc/src/mcp/list_dsc_resources.rs

@@ -3,21 +3,27 @@
 
 use crate::mcp::McpServer;
 use dsc_lib::{
-    DscManager,
-    discovery::{
-        command_discovery::ImportedManifest,
+    DscManager, discovery::{
+        command_discovery::ImportedManifest::Resource,
         discovery_trait::DiscoveryKind,
-    },
-    progress::ProgressFormat,
+    }, dscresources::resource_manifest::Kind, progress::ProgressFormat
 };
 use rmcp::{ErrorData as McpError, Json, tool, tool_router};
 use schemars::JsonSchema;
-use serde::{Serialize, Deserialize};
+use serde::Serialize;
+use std::collections::BTreeMap;
 use tokio::task;
 
-#[derive(Serialize, Deserialize, JsonSchema)]
+#[derive(Serialize, JsonSchema)]
 pub struct ResourceListResult {
-    pub resources: Vec<ImportedManifest>,
+    pub resources: Vec<ResourceSummary>,
+}
+
+#[derive(Serialize, JsonSchema)]
+pub struct ResourceSummary {
+    pub r#type: String,
+    pub kind: Kind,
+    pub description: Option<String>,
 }
 
 #[tool_router]
@@ -30,9 +36,9 @@ impl McpServer {
     }
 
     #[tool(
-        description = "List all DSC resources available on the local machine",
+        description = "List summary of all DSC resources available on the local machine",
         annotations(
-            title = "Enumerate all available DSC resources on the local machine",
+            title = "Enumerate all available DSC resources on the local machine returning name, kind, and description.",
             read_only_hint = true,
             destructive_hint = false,
             idempotent_hint = true,
@@ -42,11 +48,18 @@ impl McpServer {
     async fn list_dsc_resources(&self) -> Result<Json<ResourceListResult>, McpError> {
         let result = task::spawn_blocking(move || {
             let mut dsc = DscManager::new();
-            let mut resources = Vec::new();
+            let mut resources = BTreeMap::<String, ResourceSummary>::new();
             for resource in dsc.list_available(&DiscoveryKind::Resource, "*", "", ProgressFormat::None) {
-                resources.push(resource);
+                if let Resource(resource) = resource {
+                    let summary = ResourceSummary {
+                        r#type: resource.type_name.clone(),
+                        kind: resource.kind.clone(),
+                        description: resource.description.clone(),
+                    };
+                    resources.insert(resource.type_name.to_lowercase(), summary);
+                }
             }
-            ResourceListResult { resources }
+            ResourceListResult { resources: resources.into_values().collect() }
         }).await.map_err(|e| McpError::internal_error(e.to_string(), None))?;
 
         Ok(Json(result))

dsc/tests/dsc_mcp.tests.ps1

@@ -89,12 +89,13 @@ Describe 'Tests for MCP server' {
 
         $response = Send-McpRequest -request $mcpRequest
         $response.id | Should -Be 3
-        $resources = dsc resource list | ConvertFrom-Json -Depth 20
+        $resources = dsc resource list | ConvertFrom-Json -Depth 20 | Select-Object type, kind, description -Unique
         $response.result.structuredContent.resources.Count | Should -Be $resources.Count
         for ($i = 0; $i -lt $resources.Count; $i++) {
-            $response.result.structuredContent.resources[$i].Resource.type | Should -BeExactly $resources[$i].type
-            $response.result.structuredContent.resources[$i].Resource.version | Should -BeExactly $resources[$i].version
-            $response.result.structuredContent.resources[$i].Resource.path | Should -BeExactly $resources[$i].path
+            ($response.result.structuredContent.resources[$i].psobject.properties | Measure-Object).Count | Should -Be 3
+            $response.result.structuredContent.resources[$i].type | Should -BeExactly $resources[$i].type -Because ($response.result.structuredContent | ConvertTo-Json -Depth 20 | Out-String)
+            $response.result.structuredContent.resources[$i].kind | Should -BeExactly $resources[$i].kind -Because ($response.result.structuredContent | ConvertTo-Json -Depth 20 | Out-String)
+            $response.result.structuredContent.resources[$i].description | Should -BeExactly $resources[$i].description -Because ($response.result.structuredContent | ConvertTo-Json -Depth 20 | Out-String)
         }
     }
 }