@zereight/mcp-gitlab 1.0.69 → 1.0.70

package/build/index.js

@@ -29,7 +29,19 @@ GitLabDiscussionNoteSchema, // Added
 GitLabDiscussionSchema, PaginatedDiscussionsResponseSchema, UpdateMergeRequestNoteSchema, // Added
 CreateMergeRequestNoteSchema, // Added
 ListMergeRequestDiscussionsSchema, UpdateIssueNoteSchema, CreateIssueNoteSchema, ListMergeRequestsSchema, GitLabMilestonesSchema, ListProjectMilestonesSchema, GetProjectMilestoneSchema, CreateProjectMilestoneSchema, EditProjectMilestoneSchema, DeleteProjectMilestoneSchema, GetMilestoneIssuesSchema, GetMilestoneMergeRequestsSchema, PromoteProjectMilestoneSchema, GetMilestoneBurndownEventsSchema, GitLabCompareResultSchema, GetBranchDiffsSchema, ListCommitsSchema, GetCommitSchema, GetCommitDiffSchema, ListMergeRequestDiffsSchema, } from "./schemas.js";
+import { formatBoolean } from "./utils.js";
 import { randomUUID } from "crypto";
+import { pino } from 'pino';
+const logger = pino({
+    level: process.env.LOG_LEVEL || 'info',
+    transport: {
+        target: 'pino-pretty',
+        options: {
+            colorize: true,
+            levelFirst: true,
+        },
+    },
+});
 /**
  * Available transport modes for MCP server
  */
@@ -72,7 +84,7 @@ const USE_MILESTONE = process.env.USE_MILESTONE === "true";
 const USE_PIPELINE = process.env.USE_PIPELINE === "true";
 const SSE = process.env.SSE === "true";
 const STREAMABLE_HTTP = process.env.STREAMABLE_HTTP === "true";
-const HOST = process.env.HOST || '127.0.0.1';
+const HOST = process.env.HOST || '0.0.0.0';
 const PORT = process.env.PORT || 3002;
 // Add proxy configuration
 const HTTP_PROXY = process.env.HTTP_PROXY;
@@ -144,7 +156,7 @@ const createCookieJar = () => {
         return jar;
     }
     catch (error) {
-        console.error("Error loading cookie file:", error);
+        logger.error("Error loading cookie file:", error);
         return null;
     }
 };
@@ -642,7 +654,7 @@ function normalizeGitLabApiUrl(url) {
 const GITLAB_API_URL = normalizeGitLabApiUrl(process.env.GITLAB_API_URL || "");
 const GITLAB_PROJECT_ID = process.env.GITLAB_PROJECT_ID;
 if (!GITLAB_PERSONAL_ACCESS_TOKEN) {
-    console.error("GITLAB_PERSONAL_ACCESS_TOKEN environment variable is not set");
+    logger.error("GITLAB_PERSONAL_ACCESS_TOKEN environment variable is not set");
     process.exit(1);
 }
 /**
@@ -657,8 +669,8 @@ async function handleGitLabError(response) {
         const errorBody = await response.text();
         // Check specifically for Rate Limit error
         if (response.status === 403 && errorBody.includes("User API Key Rate limit exceeded")) {
-            console.error("GitLab API Rate Limit Exceeded:", errorBody);
-            console.log("User API Key Rate limit exceeded. Please try again later.");
+            logger.error("GitLab API Rate Limit Exceeded:", errorBody);
+            logger.error("User API Key Rate limit exceeded. Please try again later.");
             throw new Error(`GitLab API Rate Limit Exceeded: ${errorBody}`);
         }
         else {
@@ -1046,7 +1058,7 @@ async function createMergeRequest(projectId, options) {
             labels: options.labels?.join(","),
             allow_collaboration: options.allow_collaboration,
             draft: options.draft,
-            remove_source_branch: options.remove_source_branch,
+            remove_source_branch: formatBoolean(options.remove_source_branch),
             squash: options.squash,
         }),
     });
@@ -2176,10 +2188,7 @@ async function createPipeline(projectId, ref, variables) {
     const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(getEffectiveProjectId(projectId))}/pipeline`);
     const body = { ref };
     if (variables && variables.length > 0) {
-        body.variables = variables.reduce((acc, { key, value }) => {
-            acc[key] = value;
-            return acc;
-        }, {});
+        body.variables = variables;
     }
     const response = await fetch(url.toString(), {
         method: "POST",
@@ -2256,7 +2265,7 @@ async function getRepositoryTree(options) {
     else {
         headers["Authorization"] = `Bearer ${GITLAB_PERSONAL_ACCESS_TOKEN}`;
     }
-    const response = await fetch(`${GITLAB_API_URL}/projects/${encodeURIComponent(options.project_id)}/repository/tree?${queryParams.toString()}`, {
+    const response = await fetch(`${GITLAB_API_URL}/projects/${encodeURIComponent(getEffectiveProjectId(options.project_id))}/repository/tree?${queryParams.toString()}`, {
         headers,
     });
     if (response.status === 404) {
@@ -2456,7 +2465,7 @@ async function getUser(username) {
         return null;
     }
     catch (error) {
-        console.error(`Error fetching user by username '${username}':`, error);
+        logger.error(`Error fetching user by username '${username}':`, error);
         return null;
     }
 }
@@ -2475,7 +2484,7 @@ async function getUsers(usernames) {
             users[username] = user;
         }
         catch (error) {
-            console.error(`Error processing username '${username}':`, error);
+            logger.error(`Error processing username '${username}':`, error);
             users[username] = null;
         }
     }
@@ -2621,7 +2630,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     };
                 }
                 catch (forkError) {
-                    console.error("Error forking repository:", forkError);
+                    logger.error("Error forking repository:", forkError);
                     let forkErrorMessage = "Failed to fork repository";
                     if (forkError instanceof Error) {
                         forkErrorMessage = `${forkErrorMessage}: ${forkError.message}`;
@@ -2842,7 +2851,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             }
             case "get_project": {
                 const args = GetProjectSchema.parse(request.params.arguments);
-                const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(args.project_id)}`);
+                const url = new URL(`${GITLAB_API_URL}/projects/${encodeURIComponent(getEffectiveProjectId(args.project_id))}`);
                 const response = await fetch(url.toString(), {
                     ...DEFAULT_FETCH_CONFIG,
                 });
@@ -3368,10 +3377,10 @@ async function startSSEServer() {
         });
     });
     app.listen(Number(PORT), HOST, () => {
-        console.log(`GitLab MCP Server running with SSE transport`);
+        logger.info(`GitLab MCP Server running with SSE transport`);
         const colorGreen = "\x1b[32m";
         const colorReset = "\x1b[0m";
-        console.log(`${colorGreen}Endpoint: http://${HOST}:${PORT}/sse${colorReset}`);
+        logger.info(`${colorGreen}Endpoint: http://${HOST}:${PORT}/sse${colorReset}`);
     });
 }
 /**
@@ -3398,14 +3407,14 @@ async function startStreamableHTTPServer() {
                     sessionIdGenerator: () => randomUUID(),
                     onsessioninitialized: (newSessionId) => {
                         streamableTransports[newSessionId] = transport;
-                        console.warn(`Streamable HTTP session initialized: ${newSessionId}`);
+                        logger.warn(`Streamable HTTP session initialized: ${newSessionId}`);
                     }
                 });
                 // Set up cleanup handler when transport closes
                 transport.onclose = () => {
                     const sid = transport.sessionId;
                     if (sid && streamableTransports[sid]) {
-                        console.warn(`Streamable HTTP transport closed for session ${sid}, cleaning up`);
+                        logger.warn(`Streamable HTTP transport closed for session ${sid}, cleaning up`);
                         delete streamableTransports[sid];
                     }
                 };
@@ -3415,7 +3424,7 @@ async function startStreamableHTTPServer() {
             }
         }
         catch (error) {
-            console.error('Streamable HTTP error:', error);
+            logger.error('Streamable HTTP error:', error);
             res.status(500).json({
                 error: 'Internal server error',
                 message: error instanceof Error ? error.message : 'Unknown error'
@@ -3433,8 +3442,8 @@ async function startStreamableHTTPServer() {
     });
     // Start server
     app.listen(Number(PORT), HOST, () => {
-        console.log(`GitLab MCP Server running with Streamable HTTP transport`);
-        console.log(`${colorGreen}Endpoint: http://${HOST}:${PORT}/mcp${colorReset}`);
+        logger.info(`GitLab MCP Server running with Streamable HTTP transport`);
+        logger.info(`${colorGreen}Endpoint: http://${HOST}:${PORT}/mcp${colorReset}`);
     });
 }
 /**
@@ -3442,18 +3451,18 @@ async function startStreamableHTTPServer() {
  * Handle transport-specific initialization logic
  */
 async function initializeServerByTransportMode(mode) {
-    console.log('Initializing server with transport mode:', mode);
+    logger.info('Initializing server with transport mode:', mode);
     switch (mode) {
         case TransportMode.STDIO:
-            console.warn('Starting GitLab MCP Server with stdio transport');
+            logger.warn('Starting GitLab MCP Server with stdio transport');
             await startStdioServer();
             break;
         case TransportMode.SSE:
-            console.warn('Starting GitLab MCP Server with SSE transport');
+            logger.warn('Starting GitLab MCP Server with SSE transport');
             await startSSEServer();
             break;
         case TransportMode.STREAMABLE_HTTP:
-            console.warn('Starting GitLab MCP Server with Streamable HTTP transport');
+            logger.warn('Starting GitLab MCP Server with Streamable HTTP transport');
             await startStreamableHTTPServer();
             break;
         default:
@@ -3472,11 +3481,11 @@ async function runServer() {
         await initializeServerByTransportMode(transportMode);
     }
     catch (error) {
-        console.error("Error initializing server:", error);
+        logger.error("Error initializing server:", error);
         process.exit(1);
     }
 }
 runServer().catch(error => {
-    console.error("Fatal error in main():", error);
+    logger.error("Fatal error in main():", error);
     process.exit(1);
 });

package/build/utils.js

@@ -0,0 +1,9 @@
+export function formatBoolean(value) {
+    if (value == null) {
+        return value;
+    }
+    if (typeof value === 'string') {
+        return value == 'true';
+    }
+    return value;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "1.0.69",
+  "version": "1.0.70",
   "description": "MCP server for using the GitLab API",
   "license": "MIT",
   "author": "zereight",
@@ -23,7 +23,7 @@
     "deploy": "npm publish --access public",
     "generate-tools": "npx ts-node scripts/generate-tools-readme.ts",
     "changelog": "auto-changelog -p",
-    "test": "node test/validate-api.js && npm run test:server",
+    "test": "node test/validate-api.js",
     "test:integration": "node test/validate-api.js",
     "test:server": "npm run build && node build/test/test-all-transport-server.js",
     "lint": "eslint . --ext .ts",
@@ -40,6 +40,8 @@
     "http-proxy-agent": "^7.0.2",
     "https-proxy-agent": "^7.0.6",
     "node-fetch": "^3.3.2",
+    "pino": "^9.7.0",
+    "pino-pretty": "^13.0.0",
     "socks-proxy-agent": "^8.0.5",
     "tough-cookie": "^5.1.2",
     "zod-to-json-schema": "^3.23.5"
@@ -49,11 +51,11 @@
     "@types/node": "^22.13.10",
     "@typescript-eslint/eslint-plugin": "^8.21.0",
     "@typescript-eslint/parser": "^8.21.0",
+    "auto-changelog": "^2.4.0",
     "eslint": "^9.18.0",
     "prettier": "^3.4.2",
     "ts-node": "^10.9.2",
     "typescript": "^5.8.2",
-    "zod": "^3.24.2",
-    "auto-changelog": "^2.4.0"
+    "zod": "^3.24.2"
   }
 }