rootly-mcp-server 2.0.12__py3-none-any.whl → 2.0.14__py3-none-any.whl

rootly_mcp_server/server.py

@@ -391,9 +391,10 @@ def create_rootly_mcp_server(
         # Single page mode
         if page_number > 0:
             params = {
-                "page[size]": min(page_size, 5),  # Keep responses very small to avoid errors
+                "page[size]": page_size,  # Use requested page size (already limited to max 20)
                 "page[number]": page_number,
                 "include": "",
+                "fields[incidents]": "id,title,summary,status,severity,created_at,updated_at,url,started_at",
             }
             if query:
                 params["filter[search]"] = query
@@ -409,7 +410,7 @@ def create_rootly_mcp_server(
         # Multi-page mode (page_number = 0)
         all_incidents = []
         current_page = 1
-        effective_page_size = min(page_size, 5)  # Keep responses very small to avoid errors
+        effective_page_size = page_size  # Use requested page size (already limited to max 20)
         max_pages = 10  # Safety limit to prevent infinite loops
 
         try:
@@ -418,6 +419,7 @@ def create_rootly_mcp_server(
                     "page[size]": effective_page_size,
                     "page[number]": current_page,
                     "include": "",
+                    "fields[incidents]": "id,title,summary,status,severity,created_at,updated_at,url,started_at",
                 }
                 if query:
                     params["filter[search]"] = query
@@ -922,6 +924,133 @@ def _filter_openapi_spec(spec: Dict[str, Any], allowed_paths: List[str]) -> Dict
                                 "description": param.get("description", "Parameter value")
                             }
 
+            # Add/modify pagination limits to alerts and incident-related endpoints to prevent infinite loops
+            if method.lower() == "get" and ("alerts" in path.lower() or "incident" in path.lower()):
+                if "parameters" not in operation:
+                    operation["parameters"] = []
+                
+                # Find existing pagination parameters and update them with limits
+                page_size_param = None
+                page_number_param = None
+                
+                for param in operation["parameters"]:
+                    if param.get("name") == "page[size]":
+                        page_size_param = param
+                    elif param.get("name") == "page[number]":
+                        page_number_param = param
+                
+                # Update or add page[size] parameter with limits
+                if page_size_param:
+                    # Update existing parameter with limits
+                    if "schema" not in page_size_param:
+                        page_size_param["schema"] = {}
+                    page_size_param["schema"].update({
+                        "type": "integer",
+                        "default": 10,
+                        "minimum": 1,
+                        "maximum": 20,
+                        "description": "Number of results per page (max: 20)"
+                    })
+                else:
+                    # Add new parameter
+                    operation["parameters"].append({
+                        "name": "page[size]",
+                        "in": "query",
+                        "required": False,
+                        "schema": {
+                            "type": "integer",
+                            "default": 10,
+                            "minimum": 1,
+                            "maximum": 20,
+                            "description": "Number of results per page (max: 20)"
+                        }
+                    })
+                
+                # Update or add page[number] parameter with defaults
+                if page_number_param:
+                    # Update existing parameter 
+                    if "schema" not in page_number_param:
+                        page_number_param["schema"] = {}
+                    page_number_param["schema"].update({
+                        "type": "integer",
+                        "default": 1,
+                        "minimum": 1,
+                        "description": "Page number to retrieve"
+                    })
+                else:
+                    # Add new parameter
+                    operation["parameters"].append({
+                        "name": "page[number]",
+                        "in": "query", 
+                        "required": False,
+                        "schema": {
+                            "type": "integer",
+                            "default": 1,
+                            "minimum": 1,
+                            "description": "Page number to retrieve"
+                        }
+                    })
+                
+                # Add sparse fieldsets for alerts endpoints to reduce payload size
+                if "alert" in path.lower():
+                    # Add fields[alerts] parameter with essential fields only - make it required with default
+                    operation["parameters"].append({
+                        "name": "fields[alerts]",
+                        "in": "query",
+                        "required": True,
+                        "schema": {
+                            "type": "string",
+                            "default": "id,summary,status,started_at,ended_at,short_id,alert_urgency_id,source,noise",
+                            "description": "Comma-separated list of alert fields to include (reduces payload size)"
+                        }
+                    })
+                
+                # Add include parameter for alerts endpoints to minimize relationships
+                if "alert" in path.lower():
+                    # Check if include parameter already exists
+                    include_param_exists = any(param.get("name") == "include" for param in operation["parameters"])
+                    if not include_param_exists:
+                        operation["parameters"].append({
+                            "name": "include",
+                            "in": "query",
+                            "required": True,
+                            "schema": {
+                                "type": "string",
+                                "default": "",
+                                "description": "Related resources to include (empty for minimal payload)"
+                            }
+                        })
+                
+                # Add sparse fieldsets for incidents endpoints to reduce payload size
+                if "incident" in path.lower():
+                    # Add fields[incidents] parameter with essential fields only - make it required with default
+                    operation["parameters"].append({
+                        "name": "fields[incidents]", 
+                        "in": "query",
+                        "required": True,
+                        "schema": {
+                            "type": "string",
+                            "default": "id,title,summary,status,severity,created_at,updated_at,url,started_at",
+                            "description": "Comma-separated list of incident fields to include (reduces payload size)"
+                        }
+                    })
+                
+                # Add include parameter for incidents endpoints to minimize relationships
+                if "incident" in path.lower():
+                    # Check if include parameter already exists
+                    include_param_exists = any(param.get("name") == "include" for param in operation["parameters"])
+                    if not include_param_exists:
+                        operation["parameters"].append({
+                            "name": "include",
+                            "in": "query",
+                            "required": True,
+                            "schema": {
+                                "type": "string", 
+                                "default": "",
+                                "description": "Related resources to include (empty for minimal payload)"
+                            }
+                        })
+
     # Also clean up any remaining broken references in components
     if "components" in filtered_spec and "schemas" in filtered_spec["components"]:
         schemas = filtered_spec["components"]["schemas"]
@@ -935,6 +1064,52 @@ def _filter_openapi_spec(spec: Dict[str, Any], allowed_paths: List[str]) -> Dict
             logger.warning(f"Removing schema with broken references: {schema_name}")
             del schemas[schema_name]
 
+    # Clean up any operation-level references to removed schemas
+    removed_schemas = set()
+    if "components" in filtered_spec and "schemas" in filtered_spec["components"]:
+        removed_schemas = {"new_workflow", "update_workflow", "workflow", "workflow_task", 
+                          "workflow_response", "workflow_list", "new_workflow_task", 
+                          "update_workflow_task", "workflow_task_response", "workflow_task_list"}
+    
+    for path, path_item in filtered_spec.get("paths", {}).items():
+        for method, operation in path_item.items():
+            if method.lower() not in ["get", "post", "put", "delete", "patch"]:
+                continue
+                
+            # Clean request body references
+            if "requestBody" in operation:
+                request_body = operation["requestBody"]
+                if "content" in request_body:
+                    for content_type, content_info in request_body["content"].items():
+                        if "schema" in content_info and "$ref" in content_info["schema"]:
+                            ref_path = content_info["schema"]["$ref"]
+                            schema_name = ref_path.split("/")[-1]
+                            if schema_name in removed_schemas:
+                                # Replace with generic object schema
+                                content_info["schema"] = {
+                                    "type": "object",
+                                    "description": "Request data for this endpoint",
+                                    "additionalProperties": True
+                                }
+                                logger.debug(f"Cleaned broken reference in {method.upper()} {path} request body: {ref_path}")
+            
+            # Clean response references 
+            if "responses" in operation:
+                for status_code, response in operation["responses"].items():
+                    if "content" in response:
+                        for content_type, content_info in response["content"].items():
+                            if "schema" in content_info and "$ref" in content_info["schema"]:
+                                ref_path = content_info["schema"]["$ref"]
+                                schema_name = ref_path.split("/")[-1]
+                                if schema_name in removed_schemas:
+                                    # Replace with generic object schema
+                                    content_info["schema"] = {
+                                        "type": "object",
+                                        "description": "Response data from this endpoint",
+                                        "additionalProperties": True
+                                    }
+                                    logger.debug(f"Cleaned broken reference in {method.upper()} {path} response: {ref_path}")
+
     return filtered_spec
 
 
@@ -946,8 +1121,23 @@ def _has_broken_references(schema_def: Dict[str, Any]) -> bool:
         broken_refs = [
             "incident_trigger_params",
             "new_workflow",
-            "update_workflow",
-            "workflow"
+            "update_workflow", 
+            "workflow",
+            "new_workflow_task",
+            "update_workflow_task",
+            "workflow_task",
+            "workflow_task_response",
+            "workflow_task_list",
+            "workflow_response",
+            "workflow_list",
+            "workflow_custom_field_selection_response",
+            "workflow_custom_field_selection_list",
+            "workflow_form_field_condition_response", 
+            "workflow_form_field_condition_list",
+            "workflow_group_response",
+            "workflow_group_list",
+            "workflow_run_response",
+            "workflow_runs_list"
         ]
         if any(broken_ref in ref_path for broken_ref in broken_refs):
             return True

{rootly_mcp_server-2.0.12.dist-info → rootly_mcp_server-2.0.14.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: rootly-mcp-server
-Version: 2.0.12
+Version: 2.0.14
 Summary: A Model Context Protocol server for Rootly APIs using OpenAPI spec
 Project-URL: Homepage, https://github.com/Rootly-AI-Labs/Rootly-MCP-server
 Project-URL: Issues, https://github.com/Rootly-AI-Labs/Rootly-MCP-server/issues
@@ -28,6 +28,10 @@ Description-Content-Type: text/markdown
 
 # Rootly MCP Server
 
+[![PyPI version](https://badge.fury.io/py/rootly-mcp-server.svg)](https://pypi.org/project/rootly-mcp-server/)
+[![PyPI - Downloads](https://img.shields.io/pypi/dm/rootly-mcp-server)](https://pypi.org/project/rootly-mcp-server/)
+[![Python Version](https://img.shields.io/pypi/pyversions/rootly-mcp-server.svg)](https://pypi.org/project/rootly-mcp-server/)
+
 An MCP server for the [Rootly API](https://docs.rootly.com/api-reference/overview) that integrates seamlessly with MCP-compatible editors like Cursor, Windsurf, and Claude. Resolve production incidents in under a minute without leaving your IDE.
 
 [![Install MCP Server](https://cursor.com/deeplink/mcp-install-dark.svg)](https://cursor.com/install-mcp?name=rootly&config=eyJjb21tYW5kIjoibnB4IC15IG1jcC1yZW1vdGUgaHR0cHM6Ly9tY3Aucm9vdGx5LmNvbS9zc2UgLS1oZWFkZXIgQXV0aG9yaXphdGlvbjoke1JPT1RMWV9BVVRIX0hFQURFUn0iLCJlbnYiOnsiUk9PVExZX0FVVEhfSEVBREVSIjoiQmVhcmVyIDxZT1VSX1JPT1RMWV9BUElfVE9LRU4%2BIn19)

{rootly_mcp_server-2.0.12.dist-info → rootly_mcp_server-2.0.14.dist-info}/RECORD

@@ -1,12 +1,12 @@
 rootly_mcp_server/__init__.py,sha256=rvIuqIyuzgC7b9qSnylrdDP2zPO-7Ou9AoblR6re1co,629
 rootly_mcp_server/__main__.py,sha256=_F4p65_VjnN84RtmEdESVLLH0tO5tL9qBfb2Xdvbj2E,6480
 rootly_mcp_server/client.py,sha256=uit-YijR7OAJtysBoclqnublEDVkFfcb29wSzhpBv44,4686
-rootly_mcp_server/server.py,sha256=5NyGWUOjz1C1kFbAbu2iMNfuKo53_Sq254vF0cEUSHE,41358
+rootly_mcp_server/server.py,sha256=dlqsLnD3QzRtSlUYzK_PUe5zR9rbQRvySRSjmKbCy5w,51476
 rootly_mcp_server/smart_utils.py,sha256=lvGN9ITyJjBkm7ejpYagd8VWodLKnC6FmwECfCOcGwM,22973
 rootly_mcp_server/utils.py,sha256=NyxdcDiFGlV2a8eBO4lKgZg0D7Gxr6xUIB0YyJGgpPA,4165
 rootly_mcp_server/data/__init__.py,sha256=fO8a0bQnRVEoRMHKvhFzj10bhoaw7VsI51czc2MsUm4,143
-rootly_mcp_server-2.0.12.dist-info/METADATA,sha256=UJznyguav4QbdflENbArVS2rGL5f8KZKuA0RLDuN70M,8722
-rootly_mcp_server-2.0.12.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-rootly_mcp_server-2.0.12.dist-info/entry_points.txt,sha256=NE33b8VgigVPGBkboyo6pvN1Vz35HZtLybxMO4Q03PI,70
-rootly_mcp_server-2.0.12.dist-info/licenses/LICENSE,sha256=c9w9ZZGl14r54tsP40oaq5adTVX_HMNHozPIH2ymzmw,11341
-rootly_mcp_server-2.0.12.dist-info/RECORD,,
+rootly_mcp_server-2.0.14.dist-info/METADATA,sha256=XvbzBwNb-XkGeedNz2oZDRebwvBA1cUYSYSrPftB9L4,9078
+rootly_mcp_server-2.0.14.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+rootly_mcp_server-2.0.14.dist-info/entry_points.txt,sha256=NE33b8VgigVPGBkboyo6pvN1Vz35HZtLybxMO4Q03PI,70
+rootly_mcp_server-2.0.14.dist-info/licenses/LICENSE,sha256=c9w9ZZGl14r54tsP40oaq5adTVX_HMNHozPIH2ymzmw,11341
+rootly_mcp_server-2.0.14.dist-info/RECORD,,