fix style check errors

Author: mainred

src/aks-agent/azext_aks_agent/_client_factory.py

@@ -4,10 +4,7 @@
 # --------------------------------------------------------------------------------------------
 
 from azure.cli.core.commands.client_factory import get_mgmt_service_client
-from azure.cli.core.commands.parameters import get_resources_in_subscription
-from azure.cli.core.profiles import CustomResourceType, ResourceType
-from azure.mgmt.msi import ManagedServiceIdentityClient
-from knack.util import CLIError
+from azure.cli.core.profiles import CustomResourceType
 
 CUSTOM_MGMT_AKS = CustomResourceType('azext_aks_agent.vendored_sdks.azure_mgmt_containerservice.2025_10_01',
                                      'ContainerServiceClient')

src/aks-agent/azext_aks_agent/agent/aks.py

@@ -46,7 +46,7 @@ def get_aks_credentials(
     return kubeconfig_path
 
 
-def _get_kubeconfig_file_path(
+def _get_kubeconfig_file_path(  # pylint: disable=unused-argument
     resource_group_name: str,
     cluster_name: str,
     subscription_id: str = None

src/aks-agent/azext_aks_agent/agent/k8s/aks_agent_manager.py

@@ -15,7 +15,6 @@
 )
 from azext_aks_agent.agent.k8s.helm_manager import HelmManager
 from azext_aks_agent.agent.llm_config_manager import LLMConfigManager
-from azure.cli.core import get_default_cli
 from azure.cli.core.azclierror import AzCLIError
 from knack.log import get_logger
 from kubernetes import client, config
@@ -219,14 +218,15 @@ def get_agent_pods(self) -> Tuple[bool, Union[List[str], str]]:
             all_pod_items = list(agent_pods.items) + list(mcp_pods.items)
 
             # Create a pod_list-like object with combined items
-            class PodList:
+            class PodList:  # pylint: disable=too-few-public-methods
                 def __init__(self, items):
                     self.items = items
             pod_list = PodList(all_pod_items)
 
             if not pod_list.items:
                 error_msg = (
-                    f"No pods found with label selector '{AGENT_LABEL_SELECTOR}' or '{AKS_MCP_LABEL_SELECTOR}' in namespace '{self.namespace}'. "
+                    f"No pods found with label selector '{AGENT_LABEL_SELECTOR}' or "
+                    f"'{AKS_MCP_LABEL_SELECTOR}' in namespace '{self.namespace}'. "
                     f"This could mean:\n"
                     f"  1. The AKS agent is not deployed in the cluster\n"
                     f"  2. The namespace '{self.namespace}' does not exist\n"
@@ -295,7 +295,8 @@ def __init__(self, items):
             status_summary = "; ".join(status_details) if status_details else "all pods are in unknown state"
 
             error_msg = (
-                f"No running pods found with label selector '{AGENT_LABEL_SELECTOR}' or '{AKS_MCP_LABEL_SELECTOR}' in namespace '{self.namespace}'. "
+                f"No running pods found with label selector '{AGENT_LABEL_SELECTOR}' or "
+                f"'{AKS_MCP_LABEL_SELECTOR}' in namespace '{self.namespace}'. "
                 f"Found {len(pod_list.items)} pod(s) but none are in Running state: {status_summary}. "
                 f"The AKS agent pods may be starting up, failing to start, or experiencing issues."
             )
@@ -401,7 +402,7 @@ def deploy_agent(self, chart_version: Optional[str] = None,
 
         return False, output
 
-    def get_agent_status(self) -> Dict:
+    def get_agent_status(self) -> Dict:  # pylint: disable=too-many-locals
         """
         Get the current status of AKS agent deployment.
 
@@ -491,7 +492,7 @@ def get_agent_status(self) -> Dict:
             all_pods = list(agent_pods.items) + list(mcp_pods.items)
 
             # Create a pods-like object with combined items
-            class PodList:
+            class PodList:  # pylint: disable=too-few-public-methods
                 def __init__(self, items):
                     self.items = items
             pods = PodList(all_pods)
@@ -560,10 +561,9 @@ def check_llm_config_exists(self) -> bool:
                 logger.debug("LLM config secret '%s' not found in namespace '%s'",
                              self.llm_secret_name, self.namespace)
                 return False
-            else:
-                logger.error("Failed to check LLM config existence (API error %s): %s",
-                             e.status, e)
-                raise
+            logger.error("Failed to check LLM config existence (API error %s): %s",
+                         e.status, e)
+            raise
         except Exception as e:  # pylint: disable=broad-exception-caught
             logger.error("Unexpected error checking LLM config existence: %s", e)
             raise AzCLIError(f"Failed to check LLM config existence: {e}")

src/aks-agent/azext_aks_agent/agent/llm_config_manager.py

@@ -4,10 +4,9 @@
 # --------------------------------------------------------------------------------------------
 
 
-from typing import Dict, List, Optional
+from typing import Dict, List
 
 from azext_aks_agent.agent.llm_providers import LLMProvider
-from azure.cli.core.azclierror import AzCLIError
 from knack.log import get_logger
 
 logger = get_logger(__name__)

src/aks-agent/azext_aks_agent/vendored_sdks/__init__.py

@@ -0,0 +1,4 @@
+# --------------------------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All rights reserved.
+# Licensed under the MIT License. See License.txt in the project root for license information.
+# --------------------------------------------------------------------------------------------