@zereight/mcp-gitlab 2.1.69 → 2.2.0

package/build/test/test-list-merge-requests.js

@@ -1,201 +0,0 @@
-import { describe, test, before, after } from 'node:test';
-import assert from 'node:assert';
-import { spawn } from 'child_process';
-import { MockGitLabServer, findMockServerPort } from './utils/mock-gitlab-server.js';
-const MOCK_TOKEN = 'glpat-mock-token-12345';
-const TEST_PROJECT_ID = '123';
-// Helper to run the MCP tool
-async function callListMergeRequests(args = {}, env) {
-    return new Promise((resolve, reject) => {
-        const proc = spawn('node', ['build/index.js'], {
-            stdio: ['pipe', 'pipe', 'pipe'],
-            env: {
-                ...process.env,
-                ...env,
-                GITLAB_READ_ONLY_MODE: 'true'
-            }
-        });
-        let output = '';
-        let errorOutput = '';
-        proc.stdout?.on('data', d => output += d);
-        proc.stderr?.on('data', d => errorOutput += d);
-        proc.on('close', (code) => {
-            if (code !== 0)
-                return reject(new Error(`Process exited with code ${code}: ${errorOutput}`));
-            // Find the JSON line in stdout
-            const line = output.split('\n').find(l => l.startsWith('{'));
-            if (!line)
-                return reject(new Error('No JSON output found'));
-            try {
-                const response = JSON.parse(line);
-                if (response.error) {
-                    reject(response.error);
-                }
-                else {
-                    // Parse the tool result content
-                    const content = response.result?.content?.[0]?.text;
-                    if (content) {
-                        try {
-                            resolve(JSON.parse(content));
-                        }
-                        catch (e) {
-                            reject(new Error(`Failed to parse tool output JSON: ${content}`));
-                        }
-                    }
-                    else {
-                        // Fallback for direct result (if changed in future) or empty
-                        resolve(response.result);
-                    }
-                }
-            }
-            catch (e) {
-                reject(e);
-            }
-        });
-        proc.stdin?.end(JSON.stringify({
-            jsonrpc: "2.0", id: 1, method: "tools/call",
-            params: { name: "list_merge_requests", arguments: args }
-        }) + '\n');
-    });
-}
-describe('list_merge_requests', () => {
-    let mockGitLab;
-    let mockGitLabUrl;
-    before(async () => {
-        const mockPort = await findMockServerPort();
-        mockGitLab = new MockGitLabServer({
-            port: mockPort,
-            validTokens: [MOCK_TOKEN]
-        });
-        await mockGitLab.start();
-        mockGitLabUrl = mockGitLab.getUrl();
-    });
-    after(async () => {
-        await mockGitLab.stop();
-    });
-    test('lists global merge requests (no project_id)', async () => {
-        const mrs = await callListMergeRequests({}, {
-            GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-            GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN
-        });
-        assert.ok(Array.isArray(mrs), 'Response should be an array');
-        assert.strictEqual(mrs.length, 2, 'Should return 2 mock MRs');
-        // Schema coerces project_id to string
-        assert.strictEqual(String(mrs[0].project_id), '123', 'MR should have correct project_id');
-    });
-    test('lists project-specific merge requests', async () => {
-        const mrs = await callListMergeRequests({ project_id: TEST_PROJECT_ID }, {
-            GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-            GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN
-        });
-        assert.ok(Array.isArray(mrs), 'Response should be an array');
-        assert.strictEqual(mrs.length, 2, 'Should return 2 mock MRs');
-        assert.strictEqual(mrs[0].title, 'Test MR 1');
-    });
-    test('filters global merge requests', async () => {
-        // Note: The mock server returns static data, so filtering won't actually filter the results
-        // unless we implement filtering logic in the mock.
-        // But we can verify the call succeeds.
-        const mrs = await callListMergeRequests({ state: 'opened' }, {
-            GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-            GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN
-        });
-        assert.ok(Array.isArray(mrs), 'Response should be an array');
-        assert.strictEqual(mrs.length, 2, 'Should return 2 mock MRs');
-    });
-    test('omits empty optional filters instead of sending mutually exclusive query parameters', async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler('get', '/merge_requests', (req, res) => {
-            capturedUrl = req.originalUrl;
-            const hasEmptyFilter = Object.values(req.query).some(value => value === '');
-            if (hasEmptyFilter) {
-                res.status(400).json({ message: 'empty filters are invalid' });
-                return;
-            }
-            res.json([]);
-        });
-        try {
-            const mrs = await callListMergeRequests({
-                assignee_id: '',
-                assignee_username: '',
-                author_id: '',
-                author_username: '',
-                reviewer_id: '',
-                reviewer_username: '',
-                milestone: ' ',
-                search: '\t',
-                target_branch: '  ',
-                source_branch: '\n',
-                labels: [],
-                approved_by_usernames: [],
-            }, {
-                GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-                GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN,
-            });
-            assert.deepStrictEqual(mrs, []);
-            assert.strictEqual(capturedUrl, '/api/v4/merge_requests');
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test('forwards approved_by_usernames in bracket-array form', async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler('get', '/merge_requests', (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            await callListMergeRequests({ approved_by_usernames: ['alice', 'bob'] }, {
-                GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-                GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN,
-            });
-            assert.ok(capturedUrl, 'Mock handler should have received a request');
-            assert.match(capturedUrl, /approved_by_usernames%5B%5D=alice/, 'Request URL should contain approved_by_usernames[]=alice (URL-encoded)');
-            assert.match(capturedUrl, /approved_by_usernames%5B%5D=bob/, 'Request URL should contain approved_by_usernames[]=bob (URL-encoded)');
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test('drops blank entries inside array filters', async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler('get', '/merge_requests', (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            await callListMergeRequests({ labels: ['', '  ', 'bug'], approved_by_usernames: ['alice', ''] }, {
-                GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-                GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN,
-            });
-            assert.ok(capturedUrl, 'Mock handler should have received a request');
-            // Parse the query so the assertion does not depend on parameter order
-            const params = new URL(capturedUrl, 'http://localhost').searchParams;
-            assert.deepStrictEqual(params.getAll('labels'), ['bug'], 'Blank array entries should be dropped while non-blank ones are kept');
-            assert.deepStrictEqual(params.getAll('approved_by_usernames[]'), ['alice'], 'Blank array entries should be dropped while non-blank ones are kept');
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test('omits array filters whose entries are all blank', async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler('get', '/merge_requests', (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            const mrs = await callListMergeRequests({ labels: ['', ' '], approved_by_usernames: [''] }, {
-                GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-                GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN,
-            });
-            assert.deepStrictEqual(mrs, []);
-            assert.ok(capturedUrl, 'Mock handler should have received a request');
-            assert.strictEqual(new URL(capturedUrl, 'http://localhost').search, '', 'Array filters with only blank entries should not be serialized');
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-});

package/build/test/test-list-milestones.js

@@ -1,145 +0,0 @@
-import { describe, test, before, after } from "node:test";
-import assert from "node:assert";
-import { spawn } from "child_process";
-import { MockGitLabServer, findMockServerPort } from "./utils/mock-gitlab-server.js";
-const MOCK_TOKEN = "glpat-mock-token-12345";
-const TEST_PROJECT_ID = "123";
-const TEST_GROUP_ID = "my-group";
-// Milestone tools live in the non-default "milestones" toolset
-const TOOLSET_ENV = { GITLAB_TOOLSETS: "milestones" };
-// Helper to run an MCP tool through the stdio server
-async function callTool(toolName, args, env) {
-    return new Promise((resolve, reject) => {
-        const proc = spawn("node", ["build/index.js"], {
-            stdio: ["pipe", "pipe", "pipe"],
-            env: {
-                ...process.env,
-                ...TOOLSET_ENV,
-                ...env,
-                GITLAB_READ_ONLY_MODE: "true",
-            },
-        });
-        let output = "";
-        let errorOutput = "";
-        proc.stdout?.on("data", d => (output += d));
-        proc.stderr?.on("data", d => (errorOutput += d));
-        proc.on("close", code => {
-            if (code !== 0)
-                return reject(new Error(`Process exited with code ${code}: ${errorOutput}`));
-            const line = output.split("\n").find(l => l.startsWith("{"));
-            if (!line)
-                return reject(new Error("No JSON output found"));
-            try {
-                const response = JSON.parse(line);
-                if (response.error) {
-                    reject(new Error(JSON.stringify(response.error)));
-                    return;
-                }
-                const content = response.result?.content?.[0]?.text;
-                if (content) {
-                    try {
-                        resolve(JSON.parse(content));
-                    }
-                    catch {
-                        reject(new Error(`Failed to parse tool output JSON: ${content}`));
-                    }
-                }
-                else {
-                    resolve(response.result);
-                }
-            }
-            catch (e) {
-                reject(e);
-            }
-        });
-        proc.stdin?.end(JSON.stringify({
-            jsonrpc: "2.0",
-            id: 1,
-            method: "tools/call",
-            params: { name: toolName, arguments: args },
-        }) + "\n");
-    });
-}
-describe("milestone list iids filters", () => {
-    let mockGitLab;
-    let mockGitLabUrl;
-    before(async () => {
-        const mockPort = await findMockServerPort();
-        mockGitLab = new MockGitLabServer({ port: mockPort, validTokens: [MOCK_TOKEN] });
-        await mockGitLab.start();
-        mockGitLabUrl = mockGitLab.getUrl();
-    });
-    after(async () => {
-        await mockGitLab.stop();
-    });
-    const env = () => ({
-        GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-        GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN,
-    });
-    test("omits an empty iids filter for list_milestones", async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler("get", `/projects/${TEST_PROJECT_ID}/milestones`, (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            const milestones = await callTool("list_milestones", { project_id: TEST_PROJECT_ID, iids: [] }, env());
-            assert.deepStrictEqual(milestones, []);
-            assert.ok(capturedUrl, "Mock handler should have received a request");
-            assert.strictEqual(new URL(capturedUrl, "http://localhost").search, "", "An empty iids array should not be serialized as `iids=`");
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test("forwards non-empty iids in bracket-array form", async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler("get", `/projects/${TEST_PROJECT_ID}/milestones`, (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            await callTool("list_milestones", { project_id: TEST_PROJECT_ID, iids: [1, 2] }, env());
-            assert.ok(capturedUrl, "Mock handler should have received a request");
-            const params = new URL(capturedUrl, "http://localhost").searchParams;
-            assert.deepStrictEqual(params.getAll("iids[]"), ["1", "2"]);
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test("omits blank milestone filters", async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler("get", `/projects/${TEST_PROJECT_ID}/milestones`, (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            await callTool("list_milestones", { project_id: TEST_PROJECT_ID, iids: [], title: "  ", search: "" }, env());
-            assert.ok(capturedUrl, "Mock handler should have received a request");
-            assert.strictEqual(new URL(capturedUrl, "http://localhost").search, "", "Blank milestone filters should not be serialized");
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-    test("omits an empty iids filter for list_group_milestones", async () => {
-        let capturedUrl;
-        mockGitLab.addMockHandler("get", `/groups/${TEST_GROUP_ID}/milestones`, (req, res) => {
-            capturedUrl = req.originalUrl;
-            res.json([]);
-        });
-        try {
-            const milestones = await callTool("list_group_milestones", { group_id: TEST_GROUP_ID, iids: [], state: "active" }, env());
-            assert.deepStrictEqual(milestones, []);
-            assert.ok(capturedUrl, "Mock handler should have received a request");
-            const params = new URL(capturedUrl, "http://localhost").searchParams;
-            assert.equal(params.has("iids"), false, "An empty iids array should not be serialized as `iids=`");
-            assert.deepStrictEqual(params.getAll("iids[]"), []);
-            assert.deepStrictEqual(params.getAll("state"), ["active"]);
-        }
-        finally {
-            mockGitLab.clearCustomHandlers();
-        }
-    });
-});

package/build/test/test-list-project-members.js

@@ -1,132 +0,0 @@
-import { describe, test, before, after, beforeEach } from 'node:test';
-import assert from 'node:assert';
-import { spawn } from 'child_process';
-import { MockGitLabServer, findMockServerPort } from './utils/mock-gitlab-server.js';
-const MOCK_TOKEN = 'glpat-mock-token-12345';
-const TEST_PROJECT_ID = '123';
-const directMembers = [
-    {
-        id: 1,
-        username: 'direct-user',
-        name: 'Direct User',
-        state: 'active',
-        avatar_url: null,
-        web_url: 'https://gitlab.mock/users/1',
-        access_level: 30,
-        created_at: '2024-01-01T00:00:00Z'
-    }
-];
-const inheritedMembers = [
-    {
-        id: 2,
-        username: 'inherited-user',
-        name: 'Inherited User',
-        state: 'active',
-        avatar_url: null,
-        web_url: 'https://gitlab.mock/users/2',
-        access_level: 20,
-        created_at: '2024-01-02T00:00:00Z'
-    }
-];
-async function callListProjectMembers(args, env) {
-    return new Promise((resolve, reject) => {
-        const proc = spawn('node', ['build/index.js'], {
-            stdio: ['pipe', 'pipe', 'pipe'],
-            env: {
-                ...process.env,
-                ...env,
-                GITLAB_READ_ONLY_MODE: 'true'
-            }
-        });
-        let output = '';
-        let errorOutput = '';
-        proc.stdout?.on('data', d => output += d);
-        proc.stderr?.on('data', d => errorOutput += d);
-        proc.on('close', (code) => {
-            if (code !== 0)
-                return reject(new Error(`Process exited with code ${code}: ${errorOutput}`));
-            const line = output.split('\n').find(l => l.startsWith('{'));
-            if (!line)
-                return reject(new Error('No JSON output found'));
-            try {
-                const response = JSON.parse(line);
-                if (response.error) {
-                    reject(response.error);
-                }
-                else {
-                    const content = response.result?.content?.[0]?.text;
-                    if (content) {
-                        try {
-                            resolve(JSON.parse(content));
-                        }
-                        catch (e) {
-                            reject(new Error(`Failed to parse tool output JSON: ${content}`));
-                        }
-                    }
-                    else {
-                        resolve(response.result);
-                    }
-                }
-            }
-            catch (e) {
-                reject(e);
-            }
-        });
-        proc.stdin?.end(JSON.stringify({
-            jsonrpc: "2.0", id: 1, method: "tools/call",
-            params: { name: "list_project_members", arguments: args }
-        }) + '\n');
-    });
-}
-describe('list_project_members', () => {
-    let mockGitLab;
-    let mockGitLabUrl;
-    let directEndpointHit = false;
-    let inheritedEndpointHit = false;
-    before(async () => {
-        const mockPort = await findMockServerPort();
-        mockGitLab = new MockGitLabServer({
-            port: mockPort,
-            validTokens: [MOCK_TOKEN]
-        });
-        await mockGitLab.start();
-        mockGitLabUrl = mockGitLab.getUrl();
-        mockGitLab.addMockHandler('get', `/projects/${TEST_PROJECT_ID}/members`, (req, res) => {
-            directEndpointHit = true;
-            res.json(directMembers);
-        });
-        mockGitLab.addMockHandler('get', `/projects/${TEST_PROJECT_ID}/members/all`, (req, res) => {
-            inheritedEndpointHit = true;
-            res.json([...directMembers, ...inheritedMembers]);
-        });
-    });
-    beforeEach(() => {
-        directEndpointHit = false;
-        inheritedEndpointHit = false;
-    });
-    after(async () => {
-        await mockGitLab.stop();
-    });
-    test('lists direct project members', async () => {
-        const members = await callListProjectMembers({ project_id: TEST_PROJECT_ID }, {
-            GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-            GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN
-        });
-        assert.ok(Array.isArray(members), 'Response should be an array');
-        assert.strictEqual(members.length, 1, 'Should return direct members');
-        assert.strictEqual(members[0].username, 'direct-user');
-        assert.strictEqual(directEndpointHit, true, 'Direct members endpoint should be called');
-        assert.strictEqual(inheritedEndpointHit, false, 'Inherited members endpoint should not be called');
-    });
-    test('lists project members including inheritance', async () => {
-        const members = await callListProjectMembers({ project_id: TEST_PROJECT_ID, include_inheritance: true }, {
-            GITLAB_API_URL: `${mockGitLabUrl}/api/v4`,
-            GITLAB_PERSONAL_ACCESS_TOKEN: MOCK_TOKEN
-        });
-        assert.ok(Array.isArray(members), 'Response should be an array');
-        assert.strictEqual(members.length, 2, 'Should return inherited members');
-        assert.strictEqual(members[1].username, 'inherited-user');
-        assert.strictEqual(inheritedEndpointHit, true, 'Inherited members endpoint should be called');
-        assert.strictEqual(directEndpointHit, false, 'Direct members endpoint should not be called');
-    });
-});