Using ARG query to support Aks resource read operations

Directory.Packages.props

@@ -27,7 +27,6 @@
     <PackageVersion Include="Azure.ResourceManager.Authorization" Version="1.1.5" />
     <PackageVersion Include="Azure.ResourceManager.ContainerRegistry" Version="1.3.1" />
     <PackageVersion Include="Azure.ResourceManager.DesktopVirtualization" Version="1.3.2" />
-    <PackageVersion Include="Azure.ResourceManager.ContainerService" Version="1.2.5" />
     <PackageVersion Include="Azure.ResourceManager.EventGrid" Version="1.2.0-beta.2" />
     <PackageVersion Include="Azure.ResourceManager.Kusto" Version="1.6.1" />
     <PackageVersion Include="Azure.ResourceManager.CognitiveServices" Version="1.5.1" />

core/Azure.Mcp.Core/src/Services/Azure/BaseAzureResourceService.cs

@@ -53,7 +53,7 @@ private async Task<TenantResource> GetTenantResourceAsync(Guid? tenantId, Cancel
     /// </summary>
     /// <typeparam name="T">The type to convert each resource to</typeparam>
     /// <param name="resourceType">The Azure resource type to query for (e.g., "Microsoft.Sql/servers/databases")</param>
-    /// <param name="resourceGroup">The resource group name to filter by</param>
+    /// <param name="resourceGroup">The resource group name to filter by (null to query all resource groups)</param>
     /// <param name="subscription">The subscription ID or name</param>
     /// <param name="retryPolicy">Optional retry policy configuration</param>
     /// <param name="converter">Function to convert JsonElement to the target type</param>
@@ -63,23 +63,27 @@ private async Task<TenantResource> GetTenantResourceAsync(Guid? tenantId, Cancel
     /// <returns>List of resources converted to the specified type</returns>
     protected async Task<List<T>> ExecuteResourceQueryAsync<T>(
         string resourceType,
-        string resourceGroup,
+        string? resourceGroup,
         string subscription,
         RetryPolicyOptions? retryPolicy,
         Func<JsonElement, T> converter,
         string? additionalFilter = null,
         int limit = 50,
         CancellationToken cancellationToken = default)
     {
-        ValidateRequiredParameters(resourceType, resourceGroup, subscription);
+        ValidateRequiredParameters(resourceType, subscription);
         ArgumentNullException.ThrowIfNull(converter);
 
         var results = new List<T>();
 
         var subscriptionResource = await _subscriptionService.GetSubscription(subscription, null, retryPolicy);
         var tenantResource = await GetTenantResourceAsync(subscriptionResource.Data.TenantId, cancellationToken);
 
-        var queryFilter = $"Resources | where type =~ '{EscapeKqlString(resourceType)}' and resourceGroup =~ '{EscapeKqlString(resourceGroup)}'";
+        var queryFilter = $"Resources | where type =~ '{EscapeKqlString(resourceType)}'";
+        if (!string.IsNullOrEmpty(resourceGroup))
+        {
+            queryFilter += $" and resourceGroup =~ '{EscapeKqlString(resourceGroup)}'";
+        }
         if (!string.IsNullOrEmpty(additionalFilter))
         {
             queryFilter += $" and {additionalFilter}";
@@ -113,7 +117,7 @@ protected async Task<List<T>> ExecuteResourceQueryAsync<T>(
     /// </summary>
     /// <typeparam name="T">The type to convert the resource to</typeparam>
     /// <param name="resourceType">The Azure resource type to query for (e.g., "Microsoft.Sql/servers/databases")</param>
-    /// <param name="resourceGroup">The resource group name to filter by</param>
+    /// <param name="resourceGroup">The resource group name to filter by (null to query all resource groups)</param>
     /// <param name="subscription">The subscription ID or name</param>
     /// <param name="retryPolicy">Optional retry policy configuration</param>
     /// <param name="converter">Function to convert JsonElement to the target type</param>
@@ -122,20 +126,24 @@ protected async Task<List<T>> ExecuteResourceQueryAsync<T>(
     /// <returns>Single resource converted to the specified type, or null if not found</returns>
     protected async Task<T?> ExecuteSingleResourceQueryAsync<T>(
         string resourceType,
-        string resourceGroup,
+        string? resourceGroup,
         string subscription,
         RetryPolicyOptions? retryPolicy,
         Func<JsonElement, T> converter,
         string? additionalFilter = null,
         CancellationToken cancellationToken = default) where T : class
     {
-        ValidateRequiredParameters(resourceType, resourceGroup, subscription);
+        ValidateRequiredParameters(resourceType, subscription);
         ArgumentNullException.ThrowIfNull(converter);
 
         var subscriptionResource = await _subscriptionService.GetSubscription(subscription, null, retryPolicy);
         var tenantResource = await GetTenantResourceAsync(subscriptionResource.Data.TenantId, cancellationToken);
 
-        var queryFilter = $"Resources | where type =~ '{EscapeKqlString(resourceType)}' and resourceGroup =~ '{EscapeKqlString(resourceGroup)}'";
+        var queryFilter = $"Resources | where type =~ '{EscapeKqlString(resourceType)}'";
+        if (!string.IsNullOrEmpty(resourceGroup))
+        {
+            queryFilter += $" and resourceGroup =~ '{EscapeKqlString(resourceGroup)}'";
+        }
         if (!string.IsNullOrEmpty(additionalFilter))
         {
             queryFilter += $" and {additionalFilter}";

docs/new-command.md

@@ -1955,7 +1955,6 @@ Before submitting:
 - [ ] **AOT compilation verified** with `./eng/scripts/Build-Local.ps1 -BuildNative`
 - [ ] **Clean up unused using statements**: Run `dotnet format --include="tools/Azure.Mcp.Tools.{Toolset}/**/*.cs"` to remove unnecessary imports and ensure consistent formatting
 - [ ] Fix formatting issues with `dotnet format ./AzureMcp.sln` and ensure no warnings
-- [ ] Identify unused properties for Azure Resource with `.\eng\scripts\Check-Unused-ResourceProperties.ps1`
 
 ### Azure SDK Integration
 - [ ] All Azure SDK property names verified and correct