(GH-538) Update MCP to use FQTN

This change updates the types and functions in the MCP module to
expect and accept the `FullyQualifiedTypeName` type instead of
strings.

Author: michaeltlombardi

dsc/src/mcp/invoke_dsc_resource.rs

@@ -3,17 +3,15 @@
 
 use crate::mcp::mcp_server::McpServer;
 use dsc_lib::{
-    configure::config_doc::ExecutionKind,
-    dscresources::{
+    DscManager, configure::config_doc::ExecutionKind, dscresources::{
         dscresource::Invoke,
         invoke_result::{
             ExportResult,
             GetResult,
             SetResult,
             TestResult,
         },
-    },
-    DscManager,
+    }, types::FullyQualifiedTypeName
 };
 use rmcp::{ErrorData as McpError, Json, tool, tool_router, handler::server::wrapper::Parameters};
 use rust_i18n::t;
@@ -51,7 +49,7 @@ pub struct InvokeDscResourceRequest {
     #[schemars(description = "The operation to perform on the DSC resource")]
     pub operation: DscOperation,
     #[schemars(description = "The type name of the DSC resource to invoke")]
-    pub resource_type: String,
+    pub resource_type: FullyQualifiedTypeName,
     #[schemars(description = "The properties to pass to the DSC resource as JSON.  Must match the resource JSON schema from `show_dsc_resource` tool.")]
     pub properties_json: String,
 }

dsc/src/mcp/list_dsc_resources.rs

@@ -6,7 +6,7 @@ use dsc_lib::{
     DscManager, discovery::{
         command_discovery::ImportedManifest::Resource,
         discovery_trait::DiscoveryKind,
-    }, dscresources::resource_manifest::Kind, progress::ProgressFormat
+    }, dscresources::resource_manifest::Kind, progress::ProgressFormat, types::FullyQualifiedTypeName
 };
 use rmcp::{ErrorData as McpError, Json, tool, tool_router, handler::server::wrapper::Parameters};
 use rust_i18n::t;
@@ -22,11 +22,11 @@ pub struct ResourceListResult {
 
 #[derive(Serialize, JsonSchema)]
 pub struct ResourceSummary {
-    pub r#type: String,
+    pub r#type: FullyQualifiedTypeName,
     pub kind: Kind,
     pub description: Option<String>,
     #[serde(rename = "requireAdapter")]
-    pub require_adapter: Option<String>,
+    pub require_adapter: Option<FullyQualifiedTypeName>,
 }
 
 #[derive(Deserialize, JsonSchema)]
@@ -67,10 +67,10 @@ impl McpServer {
             for resource in dsc.list_available(&DiscoveryKind::Resource, "*", &adapter_filter, ProgressFormat::None) {
                 if let Resource(resource) = resource {
                     let summary = ResourceSummary {
-                        r#type: resource.type_name.to_string(),
+                        r#type: resource.type_name.clone(),
                         kind: resource.kind.clone(),
                         description: resource.description.clone(),
-                        require_adapter: resource.require_adapter.map(|fqtn| fqtn.to_string()),
+                        require_adapter: resource.require_adapter,
                     };
                     resources.insert(resource.type_name.to_lowercase(), summary);
                 }

dsc/src/mcp/show_dsc_resource.rs

@@ -7,7 +7,7 @@ use dsc_lib::{
     dscresources::{
         dscresource::{Capability, Invoke},
         resource_manifest::Kind
-    },
+    }, types::FullyQualifiedTypeName,
 };
 use rmcp::{ErrorData as McpError, Json, tool, tool_router, handler::server::wrapper::Parameters};
 use rust_i18n::t;
@@ -20,7 +20,7 @@ use tokio::task;
 pub struct DscResource {
     /// The namespaced name of the resource.
     #[serde(rename="type")]
-    pub type_name: String,
+    pub type_name: FullyQualifiedTypeName,
     /// The kind of resource.
     pub kind: Kind,
     /// The version of the resource.
@@ -66,7 +66,7 @@ impl McpServer {
                 Err(_) => None,
             };
             Ok(DscResource {
-                type_name: resource.type_name.to_string(),
+                type_name: resource.type_name.clone(),
                 kind: resource.kind.clone(),
                 version: resource.version.clone(),
                 capabilities: resource.capabilities.clone(),