Add capability of disabling toolsa

Author: zemin-piao

src/spark_history_mcp/config/config.py

@@ -28,7 +28,7 @@ class ServerConfig(BaseSettings):
     emr_cluster_arn: Optional[str] = None  # EMR specific field
     use_proxy: bool = False
     timeout: int = 30  # HTTP request timeout in seconds
-    include_plan_description: bool = False
+    include_plan_description: Optional[bool] = None
 
 
 class McpConfig(BaseSettings):

src/spark_history_mcp/tools/tools.py

@@ -944,9 +944,11 @@ def list_slowest_sql_queries(
     ctx = mcp.get_context()
     client = get_client_or_default(ctx, server, app_id)
 
-    # Use server config if include_plan_description not explicitly provided
-    if include_plan_description is None:
+    # Config takes priority: if config is set (True/False), use it; otherwise default to True
+    if client.config.include_plan_description is not None:
         include_plan_description = client.config.include_plan_description
+    else:
+        include_plan_description = True
 
     all_executions: List[ExecutionData] = []
     offset = 0

tests/unit/test_tools.py

@@ -1127,7 +1127,7 @@ def test_list_slowest_sql_queries_uses_server_config_for_plan_description(
     def test_list_slowest_sql_queries_explicit_override_server_config(
         self, mock_get_client
     ):
-        """Test that explicit include_plan_description parameter overrides server config"""
+        """Test that server config overrides parameter when config is set"""
         # Setup mock client with server config set to False
         mock_client = MagicMock()
         server_config = ServerConfig(
@@ -1150,11 +1150,11 @@ def test_list_slowest_sql_queries_explicit_override_server_config(
         mock_client.get_sql_list.return_value = [sql]
         mock_get_client.return_value = mock_client
 
-        # Call function with explicit include_plan_description=True (should override server config)
+        # Call function with explicit include_plan_description=True (config should override to False)
         result = list_slowest_sql_queries(
             "spark-app-123", include_plan_description=True
         )
 
-        # Verify plan description is included despite server config being False
+        # Verify plan description is NOT included because server config is False
         self.assertEqual(len(result), 1)
-        self.assertEqual(result[0].plan_description, "Sample plan description")
+        self.assertEqual(result[0].plan_description, "")