chore: Remove unused model SuccessResponse

nextcloud_mcp_server/models/__init__.py

@@ -3,7 +3,6 @@
 # Base models
 from .base import (
     BaseResponse,
-    SuccessResponse,
     IdResponse,
     StatusResponse,
 )
@@ -81,7 +80,6 @@ from .webdav import (
 __all__ = [
     # Base models
     "BaseResponse",
-    "SuccessResponse",
     "IdResponse",
     "StatusResponse",
     # Notes models

nextcloud_mcp_server/models/base.py

@@ -1,7 +1,7 @@
 """Base Pydantic models for common response patterns."""
 
 from datetime import datetime, timezone
-from typing import Any, Dict, Optional, Union
+from typing import Optional, Union
 
 from pydantic import BaseModel, Field, field_serializer
 
@@ -35,13 +35,6 @@ class BaseResponse(BaseModel):
         return iso_string
 
 
-class SuccessResponse(BaseResponse):
-    """Generic success response."""
-
-    message: Optional[str] = Field(None, description="Optional success message")
-    data: Optional[Dict[str, Any]] = Field(None, description="Optional response data")
-
-
 class IdResponse(BaseResponse):
     """Response model for operations that return a new ID."""
 

tests/test_models.py

@@ -5,7 +5,7 @@ import json
 import logging
 import re
 
-from nextcloud_mcp_server.models.base import BaseResponse, SuccessResponse
+from nextcloud_mcp_server.models.base import BaseResponse
 
 logger = logging.getLogger(__name__)
 
@@ -17,7 +17,7 @@ def test_timestamp_format_validation():
     seen in MCP inspector. MCP expects RFC3339 format with timezone information.
     """
     # Create a response object
-    response = SuccessResponse(message="Test message")
+    response = BaseResponse()
 
     # Serialize to JSON (mimics what MCP inspector sees)
     json_str = response.model_dump_json()