Refactor: Move XmlDocs logic to ResourceClientProvider and ResourceCollectionClientProvider

- Added GetEnhancedFactoryMethodXmlDocs() to ResourceClientProvider for singleton resources
- Added GetEnhancedGetMethodXmlDocs() to ResourceCollectionClientProvider for collection resources
- Updated MockableResourceProvider to use these new methods
- This improves code organization by centralizing documentation logic in the resource providers

Co-authored-by: live1206 <[email protected]>

Author: Copilot

eng/packages/http-client-csharp-mgmt/generator/Azure.Generator.Management/src/Providers/MockableResourceProvider.cs

@@ -239,10 +239,10 @@ private IEnumerable<MethodProvider> BuildMethodsForResource(ResourceClientProvid
                     this);
 
                 // Copy the enhanced XML documentation from the singleton resource's Get method if available
-                var getMethod = resource.Methods.FirstOrDefault(m => m.Signature.Name == "Get");
-                if (getMethod?.XmlDocs?.Summary != null)
+                var enhancedXmlDocs = resource.GetEnhancedFactoryMethodXmlDocs();
+                if (enhancedXmlDocs != null)
                 {
-                    method.XmlDocs?.Update(summary: getMethod.XmlDocs.Summary);
+                    method.XmlDocs?.Update(summary: enhancedXmlDocs);
                 }
 
                 yield return method;
@@ -267,16 +267,16 @@ private IEnumerable<MethodProvider> BuildMethodsForResource(ResourceClientProvid
                 if (getAsyncMethod is not null)
                 {
                     // we should be sure that this would never be null, but this null check here is just ensuring that we never crash
-                    yield return BuildGetMethod(this, getAsyncMethod, collectionMethodSignature, pathParameters, $"Get{resource.ResourceName}Async");
+                    yield return BuildGetMethod(this, getAsyncMethod, collectionMethodSignature, pathParameters, $"Get{resource.ResourceName}Async", collection);
                 }
 
                 if (getMethod is not null)
                 {
                     // we should be sure that this would never be null, but this null check here is just ensuring that we never crash
-                    yield return BuildGetMethod(this, getMethod, collectionMethodSignature, pathParameters, $"Get{resource.ResourceName}");
+                    yield return BuildGetMethod(this, getMethod, collectionMethodSignature, pathParameters, $"Get{resource.ResourceName}", collection);
                 }
 
-                static MethodProvider BuildGetMethod(TypeProvider enclosingType, MethodProvider resourceGetMethod, MethodSignature collectionGetSignature, IReadOnlyList<ParameterProvider> pathParameters, string methodName)
+                static MethodProvider BuildGetMethod(TypeProvider enclosingType, MethodProvider resourceGetMethod, MethodSignature collectionGetSignature, IReadOnlyList<ParameterProvider> pathParameters, string methodName, ResourceCollectionClientProvider collection)
                 {
                     var signature = new MethodSignature(
                         methodName,
@@ -294,9 +294,10 @@ static MethodProvider BuildGetMethod(TypeProvider enclosingType, MethodProvider
                         enclosingType);
 
                     // Copy the enhanced XML documentation from the collection's Get method
-                    if (resourceGetMethod.XmlDocs?.Summary != null)
+                    var enhancedXmlDocs = collection.GetEnhancedGetMethodXmlDocs();
+                    if (enhancedXmlDocs != null)
                     {
-                        method.XmlDocs?.Update(summary: resourceGetMethod.XmlDocs.Summary);
+                        method.XmlDocs?.Update(summary: enhancedXmlDocs);
                     }
 
                     return method;

eng/packages/http-client-csharp-mgmt/generator/Azure.Generator.Management/src/Providers/ResourceClientProvider.cs

@@ -180,6 +180,21 @@ private MethodSignature BuildFactoryMethodSignature()
             }
         }
 
+        /// <summary>
+        /// Gets the enhanced factory method signature with XML documentation from the resource's Get method.
+        /// This should be called after Methods are built to get the enhanced documentation.
+        /// </summary>
+        internal XmlDocSummaryStatement? GetEnhancedFactoryMethodXmlDocs()
+        {
+            if (IsSingleton)
+            {
+                // For singleton resources, get the Get method's XmlDocs
+                var getMethod = Methods.FirstOrDefault(m => m.Signature.Name == "Get");
+                return getMethod?.XmlDocs?.Summary;
+            }
+            return null;
+        }
+
         // TODO: Temporary workaround for recent breaking changes in converting Playwright service.
         // This special-casing will be replaced by a generalized naming strategy in a follow-up PR.
         private string BuildFactoryMethodName()

eng/packages/http-client-csharp-mgmt/generator/Azure.Generator.Management/src/Providers/ResourceCollectionClientProvider.cs

@@ -432,5 +432,16 @@ public bool TryGetPrivateFieldParameter(ParameterProvider parameter, out FieldPr
                 .Value;
             return matchingField != null;
         }
+
+        /// <summary>
+        /// Gets the enhanced XML documentation from the collection's Get method.
+        /// This should be called after Methods are built to get the enhanced documentation.
+        /// </summary>
+        internal XmlDocSummaryStatement? GetEnhancedGetMethodXmlDocs()
+        {
+            // Get the sync Get method's XmlDocs (both async and sync have the same docs)
+            var getMethod = Methods.FirstOrDefault(m => m.Signature.Name == "Get");
+            return getMethod?.XmlDocs?.Summary;
+        }
     }
 }