@zereight/mcp-gitlab 1.0.11 → 1.0.12

package/build/index.js

@@ -5,17 +5,56 @@ import { CallToolRequestSchema, ListToolsRequestSchema, } from "@modelcontextpro
 import fetch from "node-fetch";
 import { z } from "zod";
 import { zodToJsonSchema } from "zod-to-json-schema";
+import { fileURLToPath } from "url";
+import { dirname } from "path";
+import fs from "fs";
+import path from "path";
 import { GitLabForkSchema, GitLabReferenceSchema, GitLabRepositorySchema, GitLabIssueSchema, GitLabMergeRequestSchema, GitLabContentSchema, GitLabCreateUpdateFileResponseSchema, GitLabSearchResponseSchema, GitLabTreeSchema, GitLabCommitSchema, CreateOrUpdateFileSchema, SearchRepositoriesSchema, CreateRepositorySchema, GetFileContentsSchema, PushFilesSchema, CreateIssueSchema, CreateMergeRequestSchema, ForkRepositorySchema, CreateBranchSchema, GitLabMergeRequestDiffSchema, GetMergeRequestSchema, GetMergeRequestDiffsSchema, UpdateMergeRequestSchema, CreateNoteSchema, } from "./schemas.js";
+// Read version from package.json
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const packageJsonPath = path.resolve(__dirname, '../package.json');
+let SERVER_VERSION = "unknown";
+try {
+    if (fs.existsSync(packageJsonPath)) {
+        const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf8'));
+        SERVER_VERSION = packageJson.version || SERVER_VERSION;
+    }
+}
+catch (error) {
+    console.error("Warning: Could not read version from package.json:", error);
+}
 const server = new Server({
     name: "better-gitlab-mcp-server",
-    version: "0.0.1",
+    version: SERVER_VERSION,
 }, {
     capabilities: {
         tools: {},
     },
 });
 const GITLAB_PERSONAL_ACCESS_TOKEN = process.env.GITLAB_PERSONAL_ACCESS_TOKEN;
-const GITLAB_API_URL = process.env.GITLAB_API_URL || "https://gitlab.com/api/v4";
+// Smart URL handling for GitLab API
+function normalizeGitLabApiUrl(url) {
+    if (!url) {
+        return "https://gitlab.com/api/v4";
+    }
+    // Remove trailing slash if present
+    let normalizedUrl = url.endsWith('/') ? url.slice(0, -1) : url;
+    // Check if URL already has /api/v4
+    if (!normalizedUrl.endsWith('/api/v4') && !normalizedUrl.endsWith('/api/v4/')) {
+        // Append /api/v4 if not already present
+        normalizedUrl = `${normalizedUrl}/api/v4`;
+    }
+    return normalizedUrl;
+}
+// Use the normalizeGitLabApiUrl function to handle various URL formats
+const GITLAB_API_URL = normalizeGitLabApiUrl(process.env.GITLAB_API_URL || "");
+// Add debug logging for API URL construction
+console.log("=== MCP Server Configuration ===");
+console.log(`GITLAB_API_URL = "${GITLAB_API_URL}"`);
+console.log(`Example project API URL = "${GITLAB_API_URL}/projects/123"`);
+console.log(`Example Notes API URL = "${GITLAB_API_URL}/projects/123/issues/1/notes"`);
+console.log("===============================");
 if (!GITLAB_PERSONAL_ACCESS_TOKEN) {
     console.error("GITLAB_PERSONAL_ACCESS_TOKEN environment variable is not set");
     process.exit(1);
@@ -36,7 +75,7 @@ async function handleGitLabError(response) {
 // 프로젝트 포크 생성
 async function forkProject(projectId, namespace) {
     // API 엔드포인트 URL 생성
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/fork`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/fork`);
     if (namespace) {
         url.searchParams.append("namespace", namespace);
     }
@@ -54,7 +93,7 @@ async function forkProject(projectId, namespace) {
 }
 // 새로운 브랜치 생성
 async function createBranch(projectId, options) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/repository/branches`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/repository/branches`);
     const response = await fetch(url.toString(), {
         method: "POST",
         headers: DEFAULT_HEADERS,
@@ -68,7 +107,7 @@ async function createBranch(projectId, options) {
 }
 // 프로젝트의 기본 브랜치 조회
 async function getDefaultBranchRef(projectId) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}`);
     const response = await fetch(url.toString(), {
         headers: DEFAULT_HEADERS,
     });
@@ -83,7 +122,7 @@ async function getFileContents(projectId, filePath, ref) {
     if (!ref) {
         ref = await getDefaultBranchRef(projectId);
     }
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/repository/files/${encodedPath}`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/repository/files/${encodedPath}`);
     url.searchParams.append("ref", ref);
     const response = await fetch(url.toString(), {
         headers: DEFAULT_HEADERS,
@@ -104,7 +143,7 @@ async function getFileContents(projectId, filePath, ref) {
 }
 // 이슈 생성
 async function createIssue(projectId, options) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/issues`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/issues`);
     const response = await fetch(url.toString(), {
         method: "POST",
         headers: DEFAULT_HEADERS,
@@ -126,7 +165,7 @@ async function createIssue(projectId, options) {
     return GitLabIssueSchema.parse(data);
 }
 async function createMergeRequest(projectId, options) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/merge_requests`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/merge_requests`);
     const response = await fetch(url.toString(), {
         method: "POST",
         headers: {
@@ -156,7 +195,7 @@ async function createMergeRequest(projectId, options) {
 }
 async function createOrUpdateFile(projectId, filePath, content, commitMessage, branch, previousPath) {
     const encodedPath = encodeURIComponent(filePath);
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/repository/files/${encodedPath}`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/repository/files/${encodedPath}`);
     const body = {
         branch,
         content,
@@ -193,7 +232,7 @@ async function createOrUpdateFile(projectId, filePath, content, commitMessage, b
     return GitLabCreateUpdateFileResponseSchema.parse(data);
 }
 async function createTree(projectId, files, ref) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/repository/tree`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/repository/tree`);
     if (ref) {
         url.searchParams.append("ref", ref);
     }
@@ -224,7 +263,7 @@ async function createTree(projectId, files, ref) {
     return GitLabTreeSchema.parse(data);
 }
 async function createCommit(projectId, message, branch, actions) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/repository/commits`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/repository/commits`);
     const response = await fetch(url.toString(), {
         method: "POST",
         headers: {
@@ -255,7 +294,7 @@ async function createCommit(projectId, message, branch, actions) {
     return GitLabCommitSchema.parse(data);
 }
 async function searchProjects(query, page = 1, perPage = 20) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects`);
+    const url = new URL(`${GITLAB_API_URL}/projects`);
     url.searchParams.append("search", query);
     url.searchParams.append("page", page.toString());
     url.searchParams.append("per_page", perPage.toString());
@@ -285,7 +324,7 @@ async function searchProjects(query, page = 1, perPage = 20) {
     });
 }
 async function createRepository(options) {
-    const response = await fetch(`${GITLAB_API_URL}/api/v4/projects`, {
+    const response = await fetch(`${GITLAB_API_URL}/projects`, {
         method: "POST",
         headers: {
             Accept: "application/json",
@@ -310,7 +349,7 @@ async function createRepository(options) {
 }
 // MR 조회 함수
 async function getMergeRequest(projectId, mergeRequestIid) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}`);
     const response = await fetch(url.toString(), {
         headers: DEFAULT_HEADERS,
     });
@@ -319,7 +358,7 @@ async function getMergeRequest(projectId, mergeRequestIid) {
 }
 // MR 변경사항 조회 함수
 async function getMergeRequestDiffs(projectId, mergeRequestIid, view) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}/changes`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}/changes`);
     if (view) {
         url.searchParams.append("view", view);
     }
@@ -332,7 +371,7 @@ async function getMergeRequestDiffs(projectId, mergeRequestIid, view) {
 }
 // MR 업데이트 함수
 async function updateMergeRequest(projectId, mergeRequestIid, options) {
-    const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}`);
+    const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}`);
     const response = await fetch(url.toString(), {
         method: "PUT",
         headers: DEFAULT_HEADERS,
@@ -352,8 +391,11 @@ noteableIid, body) {
         headers: DEFAULT_HEADERS,
         body: JSON.stringify({ body }),
     });
-    await handleGitLabError(response);
-    return await response.json(); // ⚙️ 응답 타입은 GitLab API 문서에 따라 조정 가능, 필요하면 스키마 정의
+    if (!response.ok) {
+        const errorText = await response.text();
+        throw new Error(`GitLab API error: ${response.status} ${response.statusText}\n${errorText}`);
+    }
+    return await response.json();
 }
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
@@ -535,22 +577,12 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 };
             }
             case "create_note": {
-                try {
-                    const args = CreateNoteSchema.parse(request.params.arguments);
-                    const { project_id, noteable_type, noteable_iid, body } = args;
-                    const note = await createNote(project_id, noteable_type, noteable_iid, body);
-                    return {
-                        content: [{ type: "text", text: JSON.stringify(note, null, 2) }],
-                    };
-                }
-                catch (error) {
-                    if (error instanceof z.ZodError) {
-                        throw new Error(`Invalid arguments: ${error.errors
-                            .map((e) => `${e.path.join(".")}: ${e.message}`)
-                            .join(", ")}`);
-                    }
-                    throw error;
-                }
+                const args = CreateNoteSchema.parse(request.params.arguments);
+                const { project_id, noteable_type, noteable_iid, body } = args;
+                const note = await createNote(project_id, noteable_type, noteable_iid, body);
+                return {
+                    content: [{ type: "text", text: JSON.stringify(note, null, 2) }],
+                };
             }
             default:
                 throw new Error(`Unknown tool: ${request.params.name}`);
@@ -566,9 +598,19 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     }
 });
 async function runServer() {
-    const transport = new StdioServerTransport();
-    await server.connect(transport);
-    console.error("GitLab MCP Server running on stdio");
+    try {
+        console.error("========================");
+        console.error(`GitLab MCP Server v${SERVER_VERSION}`);
+        console.error(`API URL: ${GITLAB_API_URL}`);
+        console.error("========================");
+        const transport = new StdioServerTransport();
+        await server.connect(transport);
+        console.error("GitLab MCP Server running on stdio");
+    }
+    catch (error) {
+        console.error("Error initializing server:", error);
+        process.exit(1);
+    }
 }
 runServer().catch((error) => {
     console.error("Fatal error in main():", error);

package/build/test-note.js

@@ -0,0 +1,54 @@
+/**
+ * This test file verifies that the createNote function works correctly
+ * with the fixed endpoint URL construction that uses plural resource names
+ * (issues instead of issue, merge_requests instead of merge_request).
+ */
+import fetch from "node-fetch";
+// GitLab API configuration (replace with actual values when testing)
+const GITLAB_API_URL = process.env.GITLAB_API_URL || "https://gitlab.com";
+const GITLAB_PERSONAL_ACCESS_TOKEN = process.env.GITLAB_TOKEN || "";
+const PROJECT_ID = process.env.PROJECT_ID || "your/project";
+const ISSUE_IID = Number(process.env.ISSUE_IID || "1");
+async function testCreateIssueNote() {
+    try {
+        // Using plural form "issues" in the URL
+        const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(PROJECT_ID)}/issues/${ISSUE_IID}/notes`);
+        const response = await fetch(url.toString(), {
+            method: "POST",
+            headers: {
+                Accept: "application/json",
+                "Content-Type": "application/json",
+                Authorization: `Bearer ${GITLAB_PERSONAL_ACCESS_TOKEN}`,
+            },
+            body: JSON.stringify({ body: "Test note from API - with plural endpoint" }),
+        });
+        if (!response.ok) {
+            const errorBody = await response.text();
+            throw new Error(`GitLab API error: ${response.status} ${response.statusText}\n${errorBody}`);
+        }
+        const data = await response.json();
+        console.log("Successfully created note:");
+        console.log(JSON.stringify(data, null, 2));
+        return true;
+    }
+    catch (error) {
+        console.error("Error creating note:", error);
+        return false;
+    }
+}
+// Only run the test if executed directly
+if (require.main === module) {
+    console.log("Testing note creation with plural 'issues' endpoint...");
+    testCreateIssueNote().then(success => {
+        if (success) {
+            console.log("✅ Test successful!");
+            process.exit(0);
+        }
+        else {
+            console.log("❌ Test failed!");
+            process.exit(1);
+        }
+    });
+}
+// Export for use in other tests
+export { testCreateIssueNote };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "1.0.11",
+  "version": "1.0.12",
   "description": "MCP server for using the GitLab API",
   "license": "MIT",
   "author": "zereight",
@@ -28,6 +28,8 @@
     "zod-to-json-schema": "^3.23.5"
   },
   "devDependencies": {
-    "typescript": "^5.6.2"
+    "@types/node": "^22.13.10",
+    "typescript": "^5.8.2",
+    "zod": "3.21.4"
   }
 }