[pr-66] fix: make old_line and new_line optional for image diff discussions

🐛 Bug Fix:
- Image files in GitLab MR discussions use x/y coordinates instead of line numbers
- Fixed schema validation error when fetching discussions on image files

📝 Details:
- Changed old_line and new_line from required to optional in GitLabDiscussionNoteSchema
- Allows proper handling of image diff comments that use position_type: 'image'

.github/workflows/docker-publish.yml

@@ -0,0 +1,38 @@
+name: Docker Publish
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  docker:
+    runs-on: ubuntu-latest
+    
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+      
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
+      
+      - name: Login to Docker Hub
+        uses: docker/login-action@v3
+        with:
+          username: ${{ secrets.DOCKERHUB_USERNAME }}
+          password: ${{ secrets.DOCKERHUB_TOKEN }}
+      
+      - name: Extract metadata for Docker
+        id: meta
+        uses: docker/metadata-action@v5
+        with:
+          images: ${{ secrets.DOCKERHUB_USERNAME }}/gitlab-mcp
+          tags: |
+            type=semver,pattern={{version}}
+            latest
+      
+      - name: Build and push Docker image
+        uses: docker/build-push-action@v5
+        with:
+          context: .
+          push: true
+          tags: ${{ steps.meta.outputs.tags }}

.secrets

@@ -0,0 +1,3 @@
+DOCKERHUB_USERNAME=DOCKERHUB_USERNAME
+DOCKERHUB_TOKEN=DOCKERHUB_TOKEN
+GITHUB_TOKEN=DOCKERHUB_TOKEN

event.json

@@ -0,0 +1,6 @@
+{
+    "action": "published",
+    "release": {
+        "tag_name": "v1.0.52"
+    }
+}

schemas.ts

@@ -618,21 +618,21 @@ export const GitLabDiscussionNoteSchema = z.object({
       old_path: z.string(),
       new_path: z.string(),
       position_type: z.enum(["text", "image", "file"]),
-      old_line: z.number().nullable(),
+      old_line: z.number().nullish(), // This is missing for image diffs
-      new_line: z.number().nullable(),
+      new_line: z.number().nullish(), // This is missing for image diffs
       line_range: z
         .object({
           start: z.object({
             line_code: z.string(),
             type: z.enum(["new", "old", "expanded"]),
-            old_line: z.number().nullable(),
+            old_line: z.number().nullish(), // This is missing for image diffs
-            new_line: z.number().nullable(),
+            new_line: z.number().nullish(), // This is missing for image diffs
           }),
           end: z.object({
             line_code: z.string(),
             type: z.enum(["new", "old", "expanded"]),
-            old_line: z.number().nullable(),
+            old_line: z.number().nullish(), // This is missing for image diffs
-            new_line: z.number().nullable(),
+            new_line: z.number().nullish(), // This is missing for image diffs
           }),
         })
         .nullable()