mcp-instana 0.2.1__py3-none-any.whl → 0.3.1__py3-none-any.whl

src/application/application_global_alert_config.py

@@ -7,6 +7,11 @@ This module provides application alert configuration tools for Instana monitorin
 import logging
 from typing import Any, Dict, List, Optional, Union
 
+from mcp.types import ToolAnnotations
+
+from src.core.utils import BaseInstanaClient, register_as_tool, with_header_auth
+from src.prompts import mcp
+
 # Import the necessary classes from the SDK
 try:
     from instana_client.api.global_application_alert_configuration_api import (
@@ -18,8 +23,6 @@ except ImportError:
     logger.error("Failed to import application alert configuration API", exc_info=True)
     raise
 
-from src.core.utils import BaseInstanaClient, register_as_tool, with_header_auth
-
 # Configure logger for this module
 logger = logging.getLogger(__name__)
 
@@ -30,12 +33,15 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
         """Initialize the Application Alert MCP tools client."""
         super().__init__(read_token=read_token, base_url=base_url)
 
-    @register_as_tool
+    @register_as_tool(
+        title="Find Active Global Application Alert Configs",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def find_active_global_application_alert_configs(self,
                                             application_id: str,
                                             alert_ids: Optional[List[str]] = None,
-                                            ctx=None, api_client=None) -> Dict[str, Any]:
+                                            ctx=None, api_client=None) -> List[Dict[str, Any]]:
         """
         Get All Global Smart Alert Configuration.
 
@@ -57,7 +63,7 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
 
             # Validate required parameters
             if not application_id:
-                return {"error": "application_id is required"}
+                return [{"error": "application_id is required"}]
 
             # Call the find_active_global_application_alert_configs method from the SDK
             logger.debug(f"Calling find_active_global_application_alert_configs with application_id={application_id}, alert_ids={alert_ids}")
@@ -66,21 +72,33 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
                 alert_ids=alert_ids
             )
 
-            # Convert the result to a dictionary
-            if hasattr(result, 'to_dict'):
-                result_dict = result.to_dict()
+            # Convert the result to a list format
+            if isinstance(result, list):
+                # If it's already a list, convert each item to dict if needed
+                result_list = []
+                for item in result:
+                    if hasattr(item, 'to_dict'):
+                        result_list.append(item.to_dict())
+                    else:
+                        result_list.append(item)
+            elif hasattr(result, 'to_dict'):
+                # If it's a single object, wrap it in a list
+                result_list = [result.to_dict()]
             else:
-                # If it's already a dict or another format, use it as is
-                result_dict = result
+                # If it's already a dict or other format, wrap it in a list
+                result_list = [result] if result else []
 
-            logger.debug(f"Result from find_active_global_application_alert_configs: {result_dict}")
-            return result_dict
+            logger.debug(f"Result from find_active_global_application_alert_configs: {result_list}")
+            return result_list
         except Exception as e:
             logger.error(f"Error in find_active_global_application_alert_configs: {e}", exc_info=True)
-            return {"error": f"Failed to get active global application alert config: {e!s}"}
+            return [{"error": f"Failed to get active global application alert config: {e!s}"}]
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Find Global Application Alert Config Versions",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def find_global_application_alert_config_versions(self,
                                                      id: str,
@@ -128,7 +146,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in find_global_application_alert_config_versions: {e}", exc_info=True)
             return {"error": f"Failed to get global application alert config versions: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Find Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def find_global_application_alert_config(self,
                                             id: Optional[str] = None,
@@ -174,7 +195,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in find_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to get global application alert configs: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Delete Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def delete_global_application_alert_config(self,
                                               id: str,
@@ -215,7 +239,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in delete_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to delete global application alert config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Enable Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def enable_global_application_alert_config(self,
                                               id: str,
@@ -260,7 +287,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in enable_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to enable global application alert config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Disable Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def disable_global_application_alert_config(self,
                                                id: str,
@@ -305,7 +335,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in disable_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to disable global application alert config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Restore Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def restore_global_application_alert_config(self,
                                                id: str,
@@ -355,7 +388,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in restore_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to restore global application alert config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Create Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def create_global_application_alert_config(self,
                                               payload: Union[Dict[str, Any], str],
@@ -497,7 +533,10 @@ class ApplicationGlobalAlertMCPTools(BaseInstanaClient):
             logger.error(f"Error in create_global_application_alert_config: {e}", exc_info=True)
             return {"error": f"Failed to create global application alert config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Update Global Application Alert Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(GlobalApplicationAlertConfigurationApi)
     async def update_global_application_alert_config(self,
                                               id: str,

src/application/application_metrics.py

@@ -8,6 +8,10 @@ import logging
 from datetime import datetime
 from typing import Any, Dict, List, Optional
 
+from mcp.types import ToolAnnotations
+
+from src.prompts import mcp
+
 # Import the necessary classes from the SDK
 try:
     from instana_client.api.application_metrics_api import (
@@ -37,7 +41,10 @@ class ApplicationMetricsMCPTools(BaseInstanaClient):
         """Initialize the Application Metrics MCP tools client."""
         super().__init__(read_token=read_token, base_url=base_url)
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Data Metrics V2",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationMetricsApi)
     async def get_application_data_metrics_v2(self,
                                               metrics: Optional[List[Dict[str, Any]]] = None,
@@ -121,7 +128,10 @@ class ApplicationMetricsMCPTools(BaseInstanaClient):
             logger.error(f"Error in get_application_data_metrics_v2: {e}", exc_info=True)
             return {"error": f"Failed to get application data metrics: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Metrics",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationMetricsApi)
     async def get_application_metrics(self,
                                       application_ids: Optional[List[str]] = None,
@@ -199,7 +209,10 @@ class ApplicationMetricsMCPTools(BaseInstanaClient):
             logger.error(f"Error in get_application_metrics: {e}", exc_info=True)
             return {"error": f"Failed to get application metrics: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Endpoints Metrics",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationMetricsApi)
     async def get_endpoints_metrics(self,
                                     endpoint_ids: Optional[List[str]] = None,
@@ -277,7 +290,10 @@ class ApplicationMetricsMCPTools(BaseInstanaClient):
             logger.error(f"Error in get_endpoints_metrics: {e}", exc_info=True)
             return {"error": f"Failed to get endpoints metrics: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Services Metrics",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationMetricsApi)
     async def get_services_metrics(self,
                                    service_ids: Optional[List[str]] = None,

src/application/application_resources.py

@@ -8,6 +8,10 @@ import logging
 from datetime import datetime
 from typing import Any, Dict, List, Optional
 
+from mcp.types import ToolAnnotations
+
+from src.prompts import mcp
+
 # Import the necessary classes from the SDK
 try:
     from instana_client.api.application_resources_api import (
@@ -32,7 +36,10 @@ class ApplicationResourcesMCPTools(BaseInstanaClient):
         """Initialize the Application Resources MCP tools client."""
         super().__init__(read_token=read_token, base_url=base_url)
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Endpoints",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationResourcesApi)
     async def get_application_endpoints(self,
                                         name_filter: Optional[str] = None,
@@ -99,7 +106,10 @@ class ApplicationResourcesMCPTools(BaseInstanaClient):
             logger.error(f"Error in get_application_endpoints: {e}", exc_info=True)
             return {"error": f"Failed to get application endpoints: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Services",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationResourcesApi)
     async def get_application_services(self,
                                        name_filter: Optional[str] = None,
@@ -203,7 +213,10 @@ class ApplicationResourcesMCPTools(BaseInstanaClient):
             return {"error": f"Failed to get application services: {e!s}"}
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Applications",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationResourcesApi)
     async def get_applications(self,
                                name_filter: Optional[str] = None,
@@ -282,7 +295,10 @@ class ApplicationResourcesMCPTools(BaseInstanaClient):
             return [f"Error: Failed to get applications: {e!s}"]
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Services",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationResourcesApi)
     async def get_services(self,
                            name_filter: Optional[str] = None,

src/application/application_settings.py

@@ -13,9 +13,12 @@ import traceback
 from datetime import datetime
 from typing import Any, Dict, List, Optional, Union
 
-logger = logging.getLogger(__name__)
+from mcp.types import ToolAnnotations
 
 from src.core.utils import BaseInstanaClient, register_as_tool, with_header_auth
+from src.prompts import mcp
+
+logger = logging.getLogger(__name__)
 
 # Import the necessary classes from the SDK
 try:
@@ -68,7 +71,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             raise
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get All Applications Configs",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_all_applications_configs(self,
                              ctx=None,
@@ -85,24 +91,34 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
         """
         try:
             debug_print("Fetching all applications and their settings")
-            result = api_client.get_application_configs()
-            # Convert the result to a list of dictionaries
-            if isinstance(result, list):
-                result_dict = [item.to_dict() if hasattr(item, 'to_dict') else item for item in result]
-            elif hasattr(result, 'to_dict'):
-                result_dict = result.to_dict()
-            else:
-                result_dict = result
-
-            debug_print(f"Result from get_application_configs: {result_dict}")
-            return result_dict
+            # Use raw JSON response to avoid Pydantic validation issues
+            result = api_client.get_application_configs_without_preload_content()
+            import json
+            try:
+                response_text = result.data.decode('utf-8')
+                json_data = json.loads(response_text)
+                # Convert to List[Dict[str, Any]] format
+                if isinstance(json_data, list):
+                    result_dict = json_data
+                else:
+                    # If it's a single object, wrap it in a list
+                    result_dict = [json_data] if json_data else []
+                debug_print("Successfully retrieved application configs data")
+                return result_dict
+            except (json.JSONDecodeError, AttributeError) as json_err:
+                error_message = f"Failed to parse JSON response: {json_err}"
+                debug_print(error_message)
+                return [{"error": error_message}]
 
         except Exception as e:
             debug_print(f"Error in get_application_configs: {e}")
             traceback.print_exc(file=sys.stderr)
             return [{"error": f"Failed to get all applications: {e!s}"}]
 
-    @register_as_tool
+    @register_as_tool(
+        title="Add Application Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def add_application_config(self,
                                payload: Union[Dict[str, Any], str],
@@ -247,7 +263,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in add_application_config: {e}")
             return {"error": f"Failed to add new application config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Delete Application Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def delete_application_config(self,
                                   id: str,
@@ -285,7 +304,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to delete application configuration: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Config",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_application_config(self,
                                   id: str,
@@ -320,7 +342,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to get application configuration: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Update Application Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def update_application_config(
         self,
@@ -473,7 +498,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in put_application_config: {e}")
             return {"error": f"Failed to update existing application config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get All Endpoint Configs",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_all_endpoint_configs(self,
                              ctx=None,
@@ -505,7 +533,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return [{"error": f"Failed to get endpoint configs: {e!s}"}]
 
-    @register_as_tool
+    @register_as_tool(
+        title="Create Endpoint Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def create_endpoint_config(
         self,
@@ -607,7 +638,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in create_endpoint_config: {e}")
             return {"error": f"Failed to create new endpoint config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Delete Endpoint Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def delete_endpoint_config(
         self,
@@ -645,7 +679,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to delete endpoint configs: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Endpoint Config",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_endpoint_config(
         self,
@@ -685,7 +722,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to get endpoint configs: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Update Endpoint Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def update_endpoint_config(
         self,
@@ -811,7 +851,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             return {"error": f"Failed to update existing application config: {e!s}"}
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get All Manual Service Configs",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_all_manual_service_configs(self,
                              ctx=None,
@@ -843,7 +886,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return [{"error": f"Failed to get manual service configs: {e!s}"}]
 
-    @register_as_tool
+    @register_as_tool(
+        title="Add Manual Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def add_manual_service_config(
         self,
@@ -960,7 +1006,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in add_manual_service_config: {e}")
             return {"error": f"Failed to create new manual service config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Delete Manual Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def delete_manual_service_config(
         self,
@@ -998,7 +1047,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to delete manual service configs: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Update Manual Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def update_manual_service_config(
         self,
@@ -1122,7 +1174,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             return {"error": f"Failed to update manual config: {e!s}"}
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Replace All Manual Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def replace_all_manual_service_config(
         self,
@@ -1244,7 +1299,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in replace_all_manual_service_config: {e}")
             return [{"error": f"Failed to replace all manual config: {e!s}"}]
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get All Service Configs",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_all_service_configs(self,
                              ctx=None,
@@ -1276,7 +1334,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return [{"error": f"Failed to get application data metrics: {e}"}]
 
-    @register_as_tool
+    @register_as_tool(
+        title="Add Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def add_service_config(self,
                             payload: Union[Dict[str, Any], str],
@@ -1388,7 +1449,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             logger.error(f"Error in add_service_config: {e}")
             return {"error": f"Failed to add service config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Replace All Service Configs",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def replace_all_service_configs(self,
                             payload: Union[Dict[str, Any], str],
@@ -1498,7 +1562,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             return [{"error": f"Failed to replace all service config: {e!s}"}]
 
 
-    @register_as_tool
+    @register_as_tool(
+        title="Order Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def order_service_config(self,
                                    request_body: List[str],
@@ -1537,7 +1604,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to order service configs: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Delete Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=True)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def delete_service_config(self,
                                 id: str,
@@ -1575,7 +1645,10 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to delete service configuration: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Service Config",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def get_service_config(
         self,
@@ -1615,13 +1688,16 @@ class ApplicationSettingsMCPTools(BaseInstanaClient):
             traceback.print_exc(file=sys.stderr)
             return {"error": f"Failed to get service config: {e!s}"}
 
-    @register_as_tool
+    @register_as_tool(
+        title="Update Service Config",
+        annotations=ToolAnnotations(readOnlyHint=False, destructiveHint=False)
+    )
     @with_header_auth(ApplicationSettingsApi)
     async def update_service_config(self,
                             id: str,
                             payload: Union[Dict[str, Any], str],
                             ctx=None,
-                            api_client=None) -> Dict[str, Any]:
+                            api_client=None) -> Union[Dict[str, Any], List[Dict[str, Any]]]:
         """
         This tool gives is used if one wants to update a particular custom service rule.
         Args:

src/application/application_topology.py

@@ -8,7 +8,10 @@ import logging
 from datetime import datetime
 from typing import Any, Dict, Optional
 
+from mcp.types import ToolAnnotations
+
 from src.core.utils import BaseInstanaClient, register_as_tool
+from src.prompts import mcp
 
 try:
     from instana_client.api.application_topology_api import (
@@ -52,7 +55,10 @@ class ApplicationTopologyMCPTools(BaseInstanaClient):
             logger.error(f"Error initializing ApplicationTopologyMCPTools: {e}", exc_info=True)
             raise
 
-    @register_as_tool
+    @register_as_tool(
+        title="Get Application Topology",
+        annotations=ToolAnnotations(readOnlyHint=True, destructiveHint=False)
+    )
     async def get_application_topology(self,
                               window_size: Optional[int] = None,
                               to_timestamp: Optional[int] = None,
@@ -84,27 +90,29 @@ class ApplicationTopologyMCPTools(BaseInstanaClient):
             if not window_size:
                 window_size = 3600000  # Default to 1 hour in milliseconds
 
-            # Call the API
+            # Call the API with raw JSON response to avoid Pydantic validation issues
             # Note: The SDK expects parameters in camelCase, but we use snake_case in Python
             # The SDK will handle the conversion
-            result = self.topology_api.get_services_map(
+            result = self.topology_api.get_services_map_without_preload_content(
                 window_size=window_size,
                 to=to_timestamp,
                 application_id=application_id,
                 application_boundary_scope=application_boundary_scope
             )
 
-            # Ensure we always return a dictionary
-            if hasattr(result, "to_dict"):
-                result_dict = result.to_dict()
-            elif isinstance(result, dict):
-                result_dict = result
-            else:
-                # Convert to dictionary using __dict__ or as a fallback, create a new dict with string representation
-                result_dict = getattr(result, "__dict__", {"data": str(result)})
-
-            logger.debug("Successfully retrieved service topology data")
-            return result_dict
+            # Parse the JSON response manually
+            import json
+            try:
+                # The result from get_services_map_without_preload_content is a response object
+                # We need to read the response data and parse it as JSON
+                response_text = result.data.decode('utf-8')
+                result_dict = json.loads(response_text)
+                logger.debug("Successfully retrieved service topology data")
+                return result_dict
+            except (json.JSONDecodeError, AttributeError) as json_err:
+                error_message = f"Failed to parse JSON response: {json_err}"
+                logger.error(error_message)
+                return {"error": error_message}
 
         except Exception as e:
             logger.error(f"Error in get_application_topology: {e}", exc_info=True)