@zereight/mcp-gitlab 2.1.69 → 2.2.0

package/build/test/utils/version-check.test.js

@@ -1,239 +0,0 @@
-import assert from "node:assert/strict";
-import { mkdtempSync, rmSync, writeFileSync } from "node:fs";
-import { tmpdir } from "node:os";
-import { join } from "node:path";
-import { describe, test } from "node:test";
-import { checkForNewVersion, fetchLatestVersion, isNewerVersion } from "../../utils/version-check.js";
-function fetchReturning(status, body) {
-    return async () => new Response(JSON.stringify(body), { status });
-}
-function fetchCapturing(calls, status, body) {
-    return async (url, init) => {
-        const headers = new Headers(init?.headers);
-        calls.push({
-            url: url.toString(),
-            authorization: headers.get("authorization"),
-            redirect: init?.redirect,
-        });
-        return new Response(JSON.stringify(body), { status });
-    };
-}
-function throwingResolver() {
-    throw new Error("npm not found");
-}
-async function withNpmrc(contents, run, globalContents = "") {
-    const configDir = mkdtempSync(join(tmpdir(), "gitlab-mcp-npmrc-"));
-    const userconfigPath = join(configDir, ".npmrc");
-    const globalconfigPath = join(configDir, "global.npmrc");
-    writeFileSync(userconfigPath, contents);
-    writeFileSync(globalconfigPath, globalContents);
-    const previousUserconfig = process.env.NPM_CONFIG_USERCONFIG;
-    const previousGlobalconfig = process.env.NPM_CONFIG_GLOBALCONFIG;
-    process.env.NPM_CONFIG_USERCONFIG = userconfigPath;
-    process.env.NPM_CONFIG_GLOBALCONFIG = globalconfigPath;
-    try {
-        await run();
-    }
-    finally {
-        if (previousUserconfig === undefined)
-            delete process.env.NPM_CONFIG_USERCONFIG;
-        else
-            process.env.NPM_CONFIG_USERCONFIG = previousUserconfig;
-        if (previousGlobalconfig === undefined)
-            delete process.env.NPM_CONFIG_GLOBALCONFIG;
-        else
-            process.env.NPM_CONFIG_GLOBALCONFIG = previousGlobalconfig;
-        rmSync(configDir, { recursive: true, force: true });
-    }
-}
-describe("When isNewerVersion compares versions", () => {
-    test("should detect newer patch, minor, and major versions", () => {
-        assert.equal(isNewerVersion("2.1.30", "2.1.29"), true);
-        assert.equal(isNewerVersion("2.2.0", "2.1.29"), true);
-        assert.equal(isNewerVersion("3.0.0", "2.1.29"), true);
-    });
-    test("should return false for equal or older versions", () => {
-        assert.equal(isNewerVersion("2.1.29", "2.1.29"), false);
-        assert.equal(isNewerVersion("2.1.28", "2.1.29"), false);
-        assert.equal(isNewerVersion("1.9.9", "2.0.0"), false);
-    });
-    test("should return false for malformed versions", () => {
-        assert.equal(isNewerVersion("unknown", "2.1.29"), false);
-        assert.equal(isNewerVersion("2.1.30", "unknown"), false);
-    });
-});
-describe("When checkForNewVersion runs", () => {
-    test("should return the latest version when the registry has a newer one", async () => {
-        const latest = await checkForNewVersion("2.1.29", fetchReturning(200, { version: "2.1.31" }));
-        assert.equal(latest, "2.1.31");
-    });
-    test("should return null when already on the latest version", async () => {
-        const latest = await checkForNewVersion("2.1.31", fetchReturning(200, { version: "2.1.31" }));
-        assert.equal(latest, null);
-    });
-    test("should return null without fetching when current version is unknown", async () => {
-        const calls = [];
-        assert.equal(await checkForNewVersion("unknown", fetchCapturing(calls, 200, {})), null);
-        assert.equal(calls.length, 0);
-    });
-    test("should return null on registry errors instead of throwing", async () => {
-        const failingFetch = async () => {
-            throw new Error("network down");
-        };
-        assert.equal(await checkForNewVersion("2.1.29", failingFetch), null);
-        assert.equal(await checkForNewVersion("2.1.29", fetchReturning(500, {})), null);
-    });
-    test("should return null when the registry returns a non-release version", async () => {
-        const latest = await checkForNewVersion("2.1.29", fetchReturning(200, { version: "2.2.0-beta.1" }));
-        assert.equal(latest, null);
-    });
-});
-describe("When fetchLatestVersion resolves the registry", () => {
-    test("should fetch from the resolved registry instead of a hardcoded one", async () => {
-        const calls = [];
-        const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }), 3000, () => "https://npm.internal.example/");
-        assert.equal(latest, "2.1.31");
-        assert.equal(calls[0]?.url, "https://npm.internal.example/@zereight/mcp-gitlab/latest");
-    });
-    test("should fall back to the public registry when the resolver throws", async () => {
-        const calls = [];
-        const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }), 3000, throwingResolver);
-        assert.equal(latest, null);
-        assert.equal(calls.length, 0);
-    });
-    test("should fall back to the public registry when the resolved value has no hostname", async () => {
-        const calls = [];
-        const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }), 3000, () => "https:///");
-        assert.equal(latest, "2.1.31");
-        assert.equal(calls[0]?.url, "https://registry.npmjs.org/@zereight/mcp-gitlab/latest");
-    });
-    test("should fall back to the public registry when the resolved value has a query string", async () => {
-        const calls = [];
-        const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }), 3000, () => "https://registry.example/npm?tenant=a");
-        assert.equal(latest, "2.1.31");
-        assert.equal(calls[0]?.url, "https://registry.npmjs.org/@zereight/mcp-gitlab/latest");
-    });
-});
-describe("When resolving the registry from npm config", () => {
-    describe("with a scoped @zereight:registry", () => {
-        test("should prefer the scoped registry over the unscoped one", async () => {
-            await withNpmrc("registry=https://unscoped.example/\n@zereight:registry=https://scoped.example/\n", async () => {
-                const calls = [];
-                const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(latest, "2.1.31");
-                assert.equal(calls[0]?.url, "https://scoped.example/@zereight/mcp-gitlab/latest");
-            });
-        });
-    });
-});
-describe("When fetchLatestVersion authenticates against a protected registry", () => {
-    describe("with a scoped _authToken in .npmrc", () => {
-        test("should send a Bearer token only to that HTTPS registry", async () => {
-            await withNpmrc([
-                "registry=https://unscoped.example/",
-                "@zereight:registry=https://scoped.example/",
-                "//scoped.example/:_authToken=scoped-registry-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(latest, "2.1.31");
-                assert.equal(calls[0]?.url, "https://scoped.example/@zereight/mcp-gitlab/latest");
-                assert.equal(calls[0]?.authorization, "Bearer scoped-registry-token");
-            });
-        });
-        test("should refuse to follow redirects when a token is attached", async () => {
-            await withNpmrc([
-                "@zereight:registry=https://scoped.example/",
-                "//scoped.example/:_authToken=scoped-registry-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.redirect, "error");
-            });
-        });
-    });
-    describe("without a matching token", () => {
-        test("should omit Authorization", async () => {
-            await withNpmrc("@zereight:registry=https://scoped.example/\n", async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.authorization, null);
-                assert.equal(calls[0]?.redirect, undefined);
-            });
-        });
-    });
-    describe("when the token is on a parent registry path", () => {
-        test("should fall back to the host-level _authToken", async () => {
-            await withNpmrc([
-                "@zereight:registry=https://scoped.example/npm",
-                "//scoped.example/:_authToken=host-level-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(latest, "2.1.31");
-                assert.equal(calls[0]?.url, "https://scoped.example/npm/@zereight/mcp-gitlab/latest");
-                assert.equal(calls[0]?.authorization, "Bearer host-level-token");
-            });
-        });
-    });
-    describe("when both a path token and a host token exist", () => {
-        test("should prefer the more specific path token", async () => {
-            await withNpmrc([
-                "@zereight:registry=https://scoped.example/npm",
-                "//scoped.example/npm/:_authToken=path-token",
-                "//scoped.example/:_authToken=host-level-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.authorization, "Bearer path-token");
-            });
-        });
-    });
-    describe("when the token is only in the global npmrc", () => {
-        test("should send a Bearer token from globalconfig", async () => {
-            await withNpmrc("@zereight:registry=https://scoped.example/\n", async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.authorization, "Bearer global-registry-token");
-            }, "//scoped.example/:_authToken=global-registry-token\n");
-        });
-    });
-    describe("when the registry path contains underscores", () => {
-        test("should send a Bearer token for an Azure Artifacts-style registry path", async () => {
-            const registry = "https://pkgs.dev.azure.com/org/_packaging/feed/npm/registry";
-            const authKey = "//pkgs.dev.azure.com/org/_packaging/feed/npm/registry/:_authToken";
-            await withNpmrc([`@zereight:registry=${registry}`, `${authKey}=azure-feed-token`].join("\n"), async () => {
-                const calls = [];
-                const latest = await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(latest, "2.1.31");
-                assert.equal(calls[0]?.url, `${registry}/@zereight/mcp-gitlab/latest`);
-                assert.equal(calls[0]?.authorization, "Bearer azure-feed-token");
-            });
-        });
-    });
-    describe("with a token for a different host", () => {
-        test("should omit Authorization", async () => {
-            await withNpmrc([
-                "@zereight:registry=https://scoped.example/",
-                "//other.example/:_authToken=other-host-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.authorization, null);
-            });
-        });
-    });
-    describe("when the registry is HTTP", () => {
-        test("should omit Authorization even if a token is configured", async () => {
-            await withNpmrc([
-                "@zereight:registry=http://insecure.example/",
-                "//insecure.example/:_authToken=http-registry-token",
-            ].join("\n"), async () => {
-                const calls = [];
-                await fetchLatestVersion(fetchCapturing(calls, 200, { version: "2.1.31" }));
-                assert.equal(calls[0]?.url, "http://insecure.example/@zereight/mcp-gitlab/latest");
-                assert.equal(calls[0]?.authorization, null);
-            });
-        });
-    });
-});

package/build/test/utils/wiki-title.test.js

@@ -1,21 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, test } from "node:test";
-import { resolveNestedWikiUpdateTitle } from "../../utils/wiki-title.js";
-describe("When resolveNestedWikiUpdateTitle runs", () => {
-    describe("with nested wiki slugs", () => {
-        test("should prefix leaf titles using the existing hierarchical title", () => {
-            assert.equal(resolveNestedWikiUpdateTitle("00-map/infra-servers", "infra servers v2", "00-map/infra servers"), "00-map/infra servers v2");
-        });
-        test("should prefix leaf titles using the slug parent when the existing title is flat", () => {
-            assert.equal(resolveNestedWikiUpdateTitle("00-map/infra-servers", "infra servers v2", "infra servers"), "00-map/infra servers v2");
-        });
-        test("should keep full hierarchical titles unchanged", () => {
-            assert.equal(resolveNestedWikiUpdateTitle("00-map/infra-servers", "00-map/infra servers v2", "00-map/infra servers"), "00-map/infra servers v2");
-        });
-    });
-    describe("with flat wiki slugs", () => {
-        test("should keep leaf titles unchanged", () => {
-            assert.equal(resolveNestedWikiUpdateTitle("infra-servers", "infra servers v2", "infra servers"), "infra servers v2");
-        });
-    });
-});

package/build/test-note.js

@@ -1,54 +0,0 @@
-/**
- * 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 { encodeGitLabPathSegment } from "./utils/url.js";
-// 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/${encodeGitLabPathSegment(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/build/test-resolve-issue-note.js

@@ -1,127 +0,0 @@
-/**
- * This test file demonstrates the new resolve functionality for issue notes.
- * It shows how to use the update_issue_note tool to resolve or unresolve
- * issue discussion threads.
- */
-import { encodeGitLabPathSegment } from "./utils/url.js";
-// 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");
-const DISCUSSION_ID = process.env.DISCUSSION_ID || "your-discussion-id";
-const NOTE_ID = process.env.NOTE_ID || "your-note-id";
-/**
- * Test resolving an issue note
- */
-async function testResolveIssueNote() {
-    try {
-        const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeGitLabPathSegment(PROJECT_ID)}/issues/${ISSUE_IID}/discussions/${DISCUSSION_ID}/notes/${NOTE_ID}`);
-        const response = await fetch(url.toString(), {
-            method: "PUT",
-            headers: {
-                Accept: "application/json",
-                "Content-Type": "application/json",
-                Authorization: `Bearer ${GITLAB_PERSONAL_ACCESS_TOKEN}`,
-            },
-            body: JSON.stringify({ resolved: true }),
-        });
-        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 resolved issue note:");
-        console.log(JSON.stringify(data, null, 2));
-        return true;
-    }
-    catch (error) {
-        console.error("Error resolving issue note:", error);
-        return false;
-    }
-}
-/**
- * Test unresolving an issue note
- */
-async function testUnresolveIssueNote() {
-    try {
-        const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeGitLabPathSegment(PROJECT_ID)}/issues/${ISSUE_IID}/discussions/${DISCUSSION_ID}/notes/${NOTE_ID}`);
-        const response = await fetch(url.toString(), {
-            method: "PUT",
-            headers: {
-                Accept: "application/json",
-                "Content-Type": "application/json",
-                Authorization: `Bearer ${GITLAB_PERSONAL_ACCESS_TOKEN}`,
-            },
-            body: JSON.stringify({ resolved: false }),
-        });
-        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 unresolved issue note:");
-        console.log(JSON.stringify(data, null, 2));
-        return true;
-    }
-    catch (error) {
-        console.error("Error unresolving issue note:", error);
-        return false;
-    }
-}
-/**
- * Test updating note body (existing functionality should still work)
- */
-async function testUpdateIssueNoteBody() {
-    try {
-        const url = new URL(`${GITLAB_API_URL}/api/v4/projects/${encodeGitLabPathSegment(PROJECT_ID)}/issues/${ISSUE_IID}/discussions/${DISCUSSION_ID}/notes/${NOTE_ID}`);
-        const response = await fetch(url.toString(), {
-            method: "PUT",
-            headers: {
-                Accept: "application/json",
-                "Content-Type": "application/json",
-                Authorization: `Bearer ${GITLAB_PERSONAL_ACCESS_TOKEN}`,
-            },
-            body: JSON.stringify({ body: "Updated note content" }),
-        });
-        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 updated issue note body:");
-        console.log(JSON.stringify(data, null, 2));
-        return true;
-    }
-    catch (error) {
-        console.error("Error updating issue note body:", error);
-        return false;
-    }
-}
-// Only run the test if executed directly
-if (require.main === module) {
-    console.log("Testing issue note resolve functionality...\n");
-    console.log("Note: This is a demonstration test file.");
-    console.log("To run actual tests, set the following environment variables:");
-    console.log("  - GITLAB_API_URL");
-    console.log("  - GITLAB_TOKEN");
-    console.log("  - PROJECT_ID");
-    console.log("  - ISSUE_IID");
-    console.log("  - DISCUSSION_ID");
-    console.log("  - NOTE_ID");
-    console.log("\nExample MCP tool usage:");
-    console.log(`
-  {
-    "name": "update_issue_note",
-    "arguments": {
-      "project_id": "your/project",
-      "issue_iid": "1",
-      "discussion_id": "abc123",
-      "note_id": "456",
-      "resolved": true
-    }
-  }
-  `);
-}
-// Export for use in other tests
-export { testResolveIssueNote, testUnresolveIssueNote, testUpdateIssueNoteBody };