@zereight/mcp-gitlab 2.0.13 → 2.0.18

package/build/test/utils/mock-gitlab-server.js

@@ -8,22 +8,53 @@ export class MockGitLabServer {
     server = null;
     config;
     requestCount = 0;
+    customRouter;
+    customHandlers = new Map();
     constructor(config) {
         this.config = config;
         this.app = express();
+        this.customRouter = express.Router();
+        // Dynamic dispatcher for custom handlers
+        this.customRouter.use((req, res, next) => {
+            // Create a key from method and path (relative to /api/v4)
+            // req.path is already relative to the mount point
+            const key = `${req.method.toUpperCase()}:${req.path}`;
+            console.log(`[CustomRouter] Checking key: '${key}'`);
+            const handler = this.customHandlers.get(key);
+            if (handler) {
+                console.log(`[MockServer] Custom handler hit: ${key}`);
+                return handler(req, res, next);
+            }
+            else {
+                console.log(`[CustomRouter] No handler found for key: '${key}'. Available keys: ${Array.from(this.customHandlers.keys()).join(', ')}`);
+            }
+            next();
+        });
         this.setupMiddleware();
+        this.app.use('/api/v4', this.customRouter); // Mount router on API path
         this.setupRoutes();
     }
+    addMockHandler(method, path, handler) {
+        // Note: path should be relative to /api/v4
+        const key = `${method.toUpperCase()}:${path}`;
+        console.log(`[MockServer] Adding custom handler: ${key}`);
+        this.customHandlers.set(key, handler);
+    }
+    clearCustomHandlers() {
+        console.log('[MockServer] Clearing custom handlers');
+        this.customHandlers.clear();
+    }
     /**
      * Setup middleware including auth validation
      */
     setupMiddleware() {
-        this.app.use(express.json());
-        // Request counter for rate limiting tests
+        // Request counter for rate limiting tests - Place this FIRST to log everything
         this.app.use((req, res, next) => {
+            console.log(`[MockServer] ${req.method} ${req.originalUrl}`);
             this.requestCount++;
             next();
         });
+        this.app.use(express.json());
         // Artificial delay middleware (for timeout testing)
         if (this.config.responseDelay) {
             this.app.use((req, res, next) => {
@@ -158,21 +189,59 @@ export class MockGitLabServer {
         });
         // GET /api/v4/projects/:projectId/issues - List issues
         this.app.get('/api/v4/projects/:projectId/issues', (req, res) => {
+            const projectId = req.params.projectId;
             res.json([
                 {
                     id: 1,
                     iid: 1,
+                    project_id: projectId,
                     title: 'Test Issue 1',
+                    description: 'Test issue description',
                     state: 'opened',
                     created_at: '2024-01-01T00:00:00Z',
+                    updated_at: '2024-01-02T00:00:00Z',
+                    closed_at: null,
+                    web_url: `https://gitlab.mock/project/${projectId}/issues/1`,
                     author: {
                         id: 1,
                         username: 'test-user',
-                        name: 'Test User'
-                    }
+                        name: 'Test User',
+                        avatar_url: null,
+                        web_url: 'https://gitlab.mock/test-user'
+                    },
+                    assignees: [],
+                    labels: [],
+                    milestone: null
                 }
             ]);
         });
+        // GET /api/v4/projects/:projectId/issues/:issue_iid - Get single issue
+        this.app.get('/api/v4/projects/:projectId/issues/:issue_iid', (req, res) => {
+            const issueIid = parseInt(req.params.issue_iid);
+            const projectId = req.params.projectId;
+            res.json({
+                id: issueIid,
+                iid: issueIid,
+                project_id: projectId,
+                title: `Test Issue ${issueIid}`,
+                description: `Description for issue ${issueIid}`,
+                state: 'opened',
+                created_at: '2024-01-01T00:00:00Z',
+                updated_at: '2024-01-02T00:00:00Z',
+                closed_at: null,
+                web_url: `https://gitlab.mock/project/${projectId}/issues/${issueIid}`,
+                author: {
+                    id: 1,
+                    username: 'test-user',
+                    name: 'Test User',
+                    avatar_url: null,
+                    web_url: 'https://gitlab.mock/test-user'
+                },
+                assignees: [],
+                labels: [],
+                milestone: null
+            });
+        });
         // GET /api/v4/projects - List projects
         this.app.get('/api/v4/projects', (req, res) => {
             res.json([

package/build/test/utils/server-launcher.js

@@ -15,6 +15,7 @@ export var TransportMode;
  * Launch a server with specified configuration
  */
 export async function launchServer(config) {
+    console.log("Launcher: launchServer function entered.");
     const { mode, port = 3002, env = {}, timeout = 3000 } = config;
     // Prepare environment variables based on transport mode
     // Use same configuration pattern as existing validate-api.js
@@ -27,15 +28,11 @@ export async function launchServer(config) {
     if (!GITLAB_TOKEN && !isRemoteAuth) {
         throw new Error('GITLAB_TOKEN_TEST or GITLAB_TOKEN environment variable is required for server testing');
     }
-    if (!TEST_PROJECT_ID && !isRemoteAuth) {
+    if (!TEST_PROJECT_ID && !isRemoteAuth && env.ENABLE_DYNAMIC_API_URL !== 'true') {
         throw new Error('TEST_PROJECT_ID environment variable is required for server testing');
     }
     const serverEnv = {
-        // Add all environment variables from the current process
         ...process.env,
-        GITLAB_API_URL: `${GITLAB_API_URL}/api/v4`,
-        ...(TEST_PROJECT_ID ? { GITLAB_PROJECT_ID: TEST_PROJECT_ID } : {}),
-        GITLAB_READ_ONLY_MODE: 'true', // Use read-only mode for testing
         ...env,
     };
     // Only set GITLAB_PERSONAL_ACCESS_TOKEN if not using remote auth
@@ -57,11 +54,16 @@ export async function launchServer(config) {
             throw new Error(`${TransportMode.STDIO} mode is not supported for server testing, because it uses process communication.`);
     }
     const serverPath = path.resolve(process.cwd(), 'build/index.js');
+    console.log("Launcher: Spawning server process with env:", serverEnv);
+    console.log("Launcher: Spawning server process with env:", serverEnv);
     const serverProcess = spawn('node', [serverPath], {
         env: serverEnv,
         stdio: ['pipe', 'pipe', 'pipe'],
+        shell: false,
         detached: false
     });
+    console.log(`Launcher: Server process spawned with PID: ${serverProcess.pid}`);
+    console.log("Launcher: Server process spawned.");
     // Wait for server to start
     await waitForServerStart(serverProcess, mode, port, timeout);
     const instance = {
@@ -92,8 +94,16 @@ async function waitForServerStart(process, mode, port, timeout) {
         }, timeout);
         let outputBuffer = '';
         const onData = (data) => {
-            const output = data.toString();
-            outputBuffer += output;
+            try {
+                const output = data.toString();
+                console.log(`[Server Output]: ${output}`);
+                outputBuffer += output;
+            }
+            catch (e) {
+                console.error("Error converting server output to string:", e);
+                reject(e);
+                return;
+            }
             // Check for server start messages
             const startMessages = [
                 'Starting GitLab MCP Server with stdio transport',
@@ -105,8 +115,8 @@ async function waitForServerStart(process, mode, port, timeout) {
             const hasStartMessage = startMessages.some(msg => outputBuffer.includes(msg));
             if (hasStartMessage) {
                 clearTimeout(timer);
-                process.stdout?.removeListener('data', onData);
-                process.stderr?.removeListener('data', onData);
+                // process.stdout?.removeListener('data', onData);
+                // process.stderr?.removeListener('data', onData);
                 // Additional wait for HTTP servers to be fully ready
                 if (mode !== TransportMode.STDIO) {
                     setTimeout(resolve, 1000);
@@ -121,10 +131,16 @@ async function waitForServerStart(process, mode, port, timeout) {
             reject(new Error(`Server process error: ${error.message}`));
         };
         const onExit = (code) => {
+            console.log(`[Launcher DEBUG] Process ${process.pid} exited with code ${code}`);
+            // We don't reject here immediately, we wait for 'close' to ensure streams are flushed
+        };
+        const onClose = (code) => {
+            console.log(`[Launcher DEBUG] Process ${process.pid} closed with code ${code}`);
             clearTimeout(timer);
             reject(new Error(`Server process exited with code ${code} before starting`));
         };
         process.stdout?.on('data', onData);
+        process.on('close', onClose);
         process.stderr?.on('data', onData);
         process.on('error', onError);
         process.on('exit', onExit);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "2.0.13",
+  "version": "2.0.18",
   "description": "MCP server for using the GitLab API",
   "license": "MIT",
   "author": "zereight",
@@ -9,6 +9,10 @@
   "files": [
     "build"
   ],
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/zereight/gitlab-mcp.git"
+  },
   "publishConfig": {
     "access": "public"
   },
@@ -35,7 +39,6 @@
     "format:check": "prettier --check \"**/*.{js,ts,json,md}\""
   },
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.10.0",
     "@types/node-fetch": "^2.6.12",
     "express": "^5.1.0",
     "fetch-cookie": "^3.1.0",
@@ -49,6 +52,8 @@
     "pkce-challenge": "^5.0.0",
     "socks-proxy-agent": "^8.0.5",
     "tough-cookie": "^5.1.2",
+    "zod": "^3.24.2",
+    "@modelcontextprotocol/sdk": "^1.24.2",
     "zod-to-json-schema": "3.24.5"
   },
   "devDependencies": {
@@ -62,7 +67,6 @@
     "prettier": "^3.4.2",
     "ts-node": "^10.9.2",
     "tsx": "^4.20.5",
-    "typescript": "^5.8.2",
-    "zod": "^3.24.2"
+    "typescript": "^5.8.2"
   }
 }