Update project name to 'gitlab-mcp-server' and enhance project metadata in pyproject.toml; refactor merge request services and schemas for improved input handling and validation; add Dockerfile for containerization.

Author: Adit-999

Dockerfile

@@ -0,0 +1,16 @@
+FROM python:3.13-slim
+
+# Install uv
+COPY --from=ghcr.io/astral-sh/uv:latest /uv /uvx /bin/
+
+# Copy the application into the container
+COPY . /app
+
+# Set working directory
+WORKDIR /app
+
+# Install dependencies (from lockfile for reproducibility)
+RUN uv sync --frozen --no-cache
+
+# Run the MCP server (adjust entrypoint as needed)
+CMD ["uv", "run", "server.py"]

pyproject.toml

@@ -1,5 +1,5 @@
 [project]
-name = "gitlab-mcp"
+name = "gitlab-mcp-server"
 version = "0.2.0"
 description = "GitLab MCP (Model Context Protocol) server for interacting with GitLab API through AI Clients"
 readme = "README.md"
@@ -11,6 +11,21 @@ dependencies = [
     "pydantic-settings>=2.8.1",
     "asyncio>=3.4.3",
 ]
+authors = [{ name = "Adit pal Singh", email = "aditpalsing@gmail.com" }]
+license = { file = "LICENSE" }
+classifiers = [
+    "Programming Language :: Python :: 3.13",
+    "License :: OSI Approved :: MIT License",
+    "Operating System :: OS Independent",
+    "Development Status :: 3 - Alpha",
+    "Intended Audience :: Developers",
+    "Topic :: Software Development :: Libraries :: Application Frameworks",
+]
+keywords = ["gitlab", "mcp", "ai", "server"]
+
+[project.urls]
+Homepage = "https://github.com/Adit-999/gitlab-mcp"
+Documentation = "https://github.com/Adit-999/gitlab-mcp#readme"
 
 [project.optional-dependencies]
 dev = [
@@ -20,18 +35,14 @@ dev = [
     "pytest>=8.0.0",
     "pytest-asyncio>=0.23.5",
 ]
+ruff = ["ruff>=0.11.5"]
 
 [build-system]
 requires = ["hatchling"]
 build-backend = "hatchling.build"
 
 [tool.hatch.build.targets.wheel]
-packages = ["mcp"]
+packages = ["src"]
 
 [tool.pytest.ini_options]
 asyncio_mode = "auto"
-
-[dependency-groups]
-dev = [
-    "ruff>=0.11.5",
-]

server.py

@@ -35,11 +35,8 @@
     get_job_logs,
 )
 from src.services.merge_requests import (
-    apply_multiple_suggestions,
-    apply_suggestion,
     create_merge_request,
     create_merge_request_comment,
-    create_merge_request_thread,
     delete_merge_request,
     get_merge_request,
     list_merge_requests,
@@ -175,18 +172,6 @@
 mcp.tool(
     name="create_merge_request_comment", description="Add a comment to a merge request."
 )(create_merge_request_comment)
-mcp.tool(
-    name="create_merge_request_thread",
-    description="Create a new thread on a merge request.",
-)(create_merge_request_thread)
-mcp.tool(
-    name="apply_suggestion",
-    description="Apply a suggestion in a merge request comment.",
-)(apply_suggestion)
-mcp.tool(
-    name="apply_multiple_suggestions",
-    description="Apply multiple suggestions in a merge request.",
-)(apply_multiple_suggestions)
 
 # Register job tools
 

src/schemas/merge_requests.py

@@ -169,12 +169,54 @@ class CreateMergeRequestCommentInput(BaseModel):
 
 
 class CreateMergeRequestThreadInput(BaseModel):
-    """Input model for creating a thread on a GitLab merge request."""
+    """Input model for creating a thread (suggestion) on a GitLab merge request.
+
+    Attributes:
+        project_path: The path of the project (e.g., 'namespace/project').
+        mr_iid: The internal ID of the merge request.
+        body: The content of the thread (should include suggestion block for suggestions).
+        position_type: The type of position (usually 'text').
+        base_sha: The base commit SHA.
+        start_sha: The start commit SHA.
+        head_sha: The head commit SHA.
+        old_path: The old file path (for changes, required).
+        new_path: The new file path (for changes, required).
+        old_line: The line number in the old file (optional).
+        new_line: The line number in the new file (optional).
+    """
 
     project_path: str
     mr_iid: int
     body: str
-    position: dict[str, Any]
+    position_type: str
+    base_sha: str
+    start_sha: str
+    head_sha: str
+    old_path: str
+    new_path: str
+    old_line: int | None = None
+    new_line: int | None = None
+
+    @classmethod
+    def validate_line(cls, values):
+        if values.get("old_line") is None and values.get("new_line") is None:
+            raise ValueError("At least one of old_line or new_line must be provided.")
+        return values
+
+    def to_position(self) -> dict[str, Any]:
+        position = {
+            "position_type": self.position_type,
+            "base_sha": self.base_sha,
+            "start_sha": self.start_sha,
+            "head_sha": self.head_sha,
+            "old_path": self.old_path,
+            "new_path": self.new_path,
+        }
+        if self.old_line is not None:
+            position["old_line"] = str(self.old_line)
+        if self.new_line is not None:
+            position["new_line"] = str(self.new_line)
+        return position
 
 
 class ApplySuggestionInput(BaseModel):