Merge branch 'feature/add-delete-text-file-operation' into develop

Author: tumfwkm

Makefile

@@ -22,6 +22,6 @@ typecheck:
 	mypy src tests
 
 # Run all checks required before pushing
-check:  lint typecheck test
-fix: check format
+check:  lint typecheck
+fix: format
 all: format check coverage

src/mcp_text_editor/__init__.py

@@ -1,10 +1,37 @@
 """MCP Text Editor Server package."""
 
 import asyncio
+from typing import Any, Dict, List
 
 from .server import main
+from .text_editor import TextEditor
+
+# Create a global text editor instance
+_text_editor = TextEditor()
 
 
 def run() -> None:
     """Run the MCP Text Editor Server."""
     asyncio.run(main())
+
+
+# Export functions
+async def get_text_file_contents(
+    request: Dict[str, List[Dict[str, Any]]]
+) -> Dict[str, Any]:
+    """Get text file contents with line range specification."""
+    return await _text_editor.read_multiple_ranges(
+        ranges=request["files"],
+        encoding="utf-8",
+    )
+
+
+async def insert_text_file_contents(request: Dict[str, Any]) -> Dict[str, Any]:
+    """Insert text content before or after a specific line in a file."""
+    return await _text_editor.insert_text_file_contents(
+        file_path=request["path"],
+        file_hash=request["file_hash"],
+        after=request.get("after"),
+        before=request.get("before"),
+        contents=request["contents"],
+    )

src/mcp_text_editor/models.py

@@ -2,7 +2,7 @@
 
 from typing import Dict, List, Optional
 
-from pydantic import BaseModel, Field, model_validator
+from pydantic import BaseModel, Field, field_validator, model_validator
 
 
 class GetTextFileContentsRequest(BaseModel):
@@ -120,3 +120,80 @@ class FileRanges(BaseModel):
     ranges: List[FileRange] = Field(
         ..., description="List of line ranges to read from the file"
     )
+
+
+class InsertTextFileContentsRequest(BaseModel):
+    """Request model for inserting text into a file.
+
+    Example:
+    {
+        "path": "/path/to/file",
+        "file_hash": "abc123...",
+        "after": 5,  # Insert after line 5
+        "contents": "new content"
+    }
+    or
+    {
+        "path": "/path/to/file",
+        "file_hash": "abc123...",
+        "before": 5,  # Insert before line 5
+        "contents": "new content"
+    }
+    """
+
+    path: str = Field(..., description="Path to the text file")
+    file_hash: str = Field(..., description="Hash of original contents")
+    after: Optional[int] = Field(
+        None, description="Line number after which to insert content"
+    )
+    before: Optional[int] = Field(
+        None, description="Line number before which to insert content"
+    )
+    contents: str = Field(..., description="Content to insert")
+
+    @model_validator(mode="after")
+    def validate_position(self) -> "InsertTextFileContentsRequest":
+        """Validate that exactly one of after or before is specified."""
+        if (self.after is None and self.before is None) or (
+            self.after is not None and self.before is not None
+        ):
+            raise ValueError("Exactly one of 'after' or 'before' must be specified")
+        return self
+
+    @field_validator("after", "before")
+    def validate_line_number(cls, v) -> Optional[int]:
+        """Validate that line numbers are positive."""
+        if v is not None and v < 1:
+            raise ValueError("Line numbers must be positive")
+        return v
+
+
+class DeleteTextFileContentsRequest(BaseModel):
+    """Request model for deleting text from a file.
+
+    Example:
+    {
+        "path": "/path/to/file",
+        "file_hash": "abc123...",
+        "ranges": [
+            {
+                "start": 5,
+                "end": 10,
+                "range_hash": "def456..."
+            }
+        ]
+    }
+    """
+
+    path: str = Field(..., description="Path to the text file")
+    file_hash: str = Field(..., description="Hash of original contents")
+    ranges: List[FileRange] = Field(..., description="List of ranges to delete")
+
+    @field_validator("range_hash")
+    def validate_range_hash(cls, v: str) -> str:
+        """Validate that range_hash is not empty."""
+        if not v:
+            raise ValueError("range_hash cannot be empty")
+        return v
+
+    range_hash: str = Field(..., description="Hash of the content to be deleted")