@zereight/mcp-gitlab 2.1.69 → 2.2.0

package/build/test/no-proxy-test.js

@@ -1,149 +0,0 @@
-/**
- * NO_PROXY Test Suite
- * Tests NO_PROXY pattern matching and proxy bypass functionality
- */
-import { describe, test } from 'node:test';
-import assert from 'node:assert';
-import { GitLabClientPool } from '../gitlab-client-pool.js';
-console.log('🚫 NO_PROXY Test Suite');
-console.log('');
-describe('NO_PROXY Pattern Matching', () => {
-    test('should bypass proxy for exact hostname match', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'gitlab.internal.com',
-        });
-        // Create agent for the NO_PROXY matched host
-        const agent = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4');
-        // The agent should NOT be a proxy agent
-        // It should be a standard HTTPS agent
-        assert.ok(agent, 'Agent should be created');
-        assert.strictEqual(agent.constructor.name, 'Agent', 'Should be a standard Agent, not a proxy agent');
-    });
-    test('should use proxy for non-matching hostname', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'gitlab.internal.com',
-        });
-        // Create agent for a host that should use proxy
-        const agent = pool.getOrCreateAgentForUrl('https://gitlab.external.com/api/v4');
-        // The agent should be a proxy agent
-        assert.ok(agent, 'Agent should be created');
-        assert.notStrictEqual(agent.constructor.name, 'Agent', 'Should be a proxy agent, not standard Agent');
-    });
-    test('should bypass proxy for domain suffix match', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: '.internal.com',
-        });
-        // Test multiple subdomains
-        const agent1 = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4');
-        const agent2 = pool.getOrCreateAgentForUrl('https://api.internal.com/api/v4');
-        const agent3 = pool.getOrCreateAgentForUrl('https://dev.gitlab.internal.com/api/v4');
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'gitlab.internal.com should bypass proxy');
-        assert.strictEqual(agent2.constructor.name, 'Agent', 'api.internal.com should bypass proxy');
-        assert.strictEqual(agent3.constructor.name, 'Agent', 'dev.gitlab.internal.com should bypass proxy');
-    });
-    test('should bypass proxy for localhost', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'localhost,127.0.0.1',
-        });
-        const agent1 = pool.getOrCreateAgentForUrl('http://localhost:8080/api/v4');
-        const agent2 = pool.getOrCreateAgentForUrl('http://127.0.0.1:8080/api/v4');
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'localhost should bypass proxy');
-        assert.strictEqual(agent2.constructor.name, 'Agent', '127.0.0.1 should bypass proxy');
-    });
-    test('should bypass proxy for wildcard pattern', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: '*',
-        });
-        const agent = pool.getOrCreateAgentForUrl('https://gitlab.com/api/v4');
-        assert.strictEqual(agent.constructor.name, 'Agent', 'Wildcard should bypass all proxies');
-    });
-    test('should handle multiple NO_PROXY patterns', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'localhost,.internal.com,192.168.1.1',
-        });
-        const agent1 = pool.getOrCreateAgentForUrl('http://localhost/api/v4');
-        const agent2 = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4');
-        const agent3 = pool.getOrCreateAgentForUrl('http://192.168.1.1/api/v4');
-        const agent4 = pool.getOrCreateAgentForUrl('https://gitlab.com/api/v4');
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'localhost should bypass proxy');
-        assert.strictEqual(agent2.constructor.name, 'Agent', '.internal.com should bypass proxy');
-        assert.strictEqual(agent3.constructor.name, 'Agent', '192.168.1.1 should bypass proxy');
-        assert.notStrictEqual(agent4.constructor.name, 'Agent', 'gitlab.com should use proxy');
-    });
-    test('should handle NO_PROXY with whitespace', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: ' localhost , .internal.com , 192.168.1.1 ',
-        });
-        const agent1 = pool.getOrCreateAgentForUrl('http://localhost/api/v4');
-        const agent2 = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4');
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'Should handle whitespace in NO_PROXY');
-        assert.strictEqual(agent2.constructor.name, 'Agent', 'Should handle whitespace in NO_PROXY');
-    });
-    test('should work without NO_PROXY set', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-        });
-        const agent = pool.getOrCreateAgentForUrl('https://gitlab.com/api/v4');
-        // Should use proxy when NO_PROXY is not set
-        assert.notStrictEqual(agent.constructor.name, 'Agent', 'Should use proxy when NO_PROXY is not set');
-    });
-    test('should work with NO_PROXY set but empty', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: '',
-        });
-        const agent = pool.getOrCreateAgentForUrl('https://gitlab.com/api/v4');
-        // Should use proxy when NO_PROXY is empty
-        assert.notStrictEqual(agent.constructor.name, 'Agent', 'Should use proxy when NO_PROXY is empty');
-    });
-    test('should handle port-specific patterns', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'gitlab.internal.com:443',
-        });
-        const agent1 = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4'); // HTTPS uses port 443 by default
-        const agent2 = pool.getOrCreateAgentForUrl('http://gitlab.internal.com/api/v4'); // HTTP uses port 80 by default
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'Should bypass proxy for matching port');
-        assert.notStrictEqual(agent2.constructor.name, 'Agent', 'Should use proxy for non-matching port');
-    });
-    test('should handle case-insensitive matching', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'GitLab.Internal.COM',
-        });
-        const agent1 = pool.getOrCreateAgentForUrl('https://gitlab.internal.com/api/v4');
-        const agent2 = pool.getOrCreateAgentForUrl('https://GITLAB.INTERNAL.COM/api/v4');
-        assert.strictEqual(agent1.constructor.name, 'Agent', 'Should match case-insensitively (lowercase)');
-        assert.strictEqual(agent2.constructor.name, 'Agent', 'Should match case-insensitively (uppercase)');
-    });
-    test('should re-evaluate NO_PROXY for redirect origins', () => {
-        const pool = new GitLabClientPool({
-            httpProxy: 'http://proxy.example.com:8080',
-            httpsProxy: 'http://proxy.example.com:8080',
-            noProxy: 'gitlab.internal.com',
-        });
-        const proxyToBypass = pool.getDispatcherForOrigin('https://gitlab.external.com/api/v4', 'https://gitlab.internal.com/api/v4');
-        const bypassToProxy = pool.getDispatcherForOrigin('https://gitlab.internal.com/api/v4', 'https://gitlab.external.com/api/v4');
-        assert.strictEqual(proxyToBypass.constructor.name, 'Agent', 'redirect onto NO_PROXY host should bypass');
-        assert.notStrictEqual(bypassToProxy.constructor.name, 'Agent', 'redirect onto proxied host should use proxy');
-    });
-});
-console.log('✅ NO_PROXY tests completed');

package/build/test/nullable-gitlab-response-fields.test.js

@@ -1,37 +0,0 @@
-import assert from "node:assert/strict";
-import { test } from "node:test";
-import { GitLabCiLintResultSchema, GitLabCompareResultSchema, GitLabProtectedBranchAccessLevelSchema, } from "../schemas.js";
-test("role-based protected-branch access levels accept null user_id/group_id", () => {
-    // #575: GitLab returns null for role-based levels, not omitted fields
-    const parsed = GitLabProtectedBranchAccessLevelSchema.parse({
-        access_level: 40,
-        access_level_description: "Maintainers",
-        user_id: null,
-        group_id: null,
-    });
-    assert.equal(parsed.user_id, null);
-    assert.equal(parsed.group_id, null);
-});
-test("branch compare accepts null commit", () => {
-    const parsed = GitLabCompareResultSchema.parse({
-        commit: null,
-        commits: [],
-        diffs: [],
-    });
-    assert.equal(parsed.commit, null);
-});
-test("ci lint result accepts null merged_yaml/includes on an invalid config", () => {
-    // #638: GitLab nulls both fields whenever the config is invalid — i.e. in
-    // exactly the case validate_ci_lint exists to report on.
-    const parsed = GitLabCiLintResultSchema.parse({
-        valid: false,
-        errors: ["image is defined in top-level and `default:` entry"],
-        warnings: [],
-        merged_yaml: null,
-        includes: null,
-    });
-    assert.equal(parsed.valid, false);
-    assert.equal(parsed.merged_yaml, null);
-    assert.equal(parsed.includes, null);
-    assert.deepEqual(parsed.errors, ["image is defined in top-level and `default:` entry"]);
-});

package/build/test/nullish-tool-arguments-schema.test.js

@@ -1,118 +0,0 @@
-import assert from "node:assert/strict";
-import { describe, test } from "node:test";
-import { CreateDraftNoteSchema, CreateLabelSchema, ExecuteGraphQLSchema, GetBranchDiffsSchema, UpdateMergeRequestDiscussionNoteSchema, UpdateMergeRequestSchema, } from "../schemas.js";
-import { sanitizeToolArguments } from "../utils/tool-args.js";
-const PROJECT_ID = "group/project";
-const MERGE_REQUEST_IID = "1";
-function parseAfterSanitize(toolName, raw, parse) {
-    return parse(sanitizeToolArguments(toolName, raw));
-}
-describe("When validating tool arguments after sanitization", () => {
-    describe("with create_draft_note", () => {
-        test("should accept position with null SHAs after sanitize and schema preprocess", () => {
-            const parsed = parseAfterSanitize("create_draft_note", {
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                body: "Test note",
-                position: {
-                    base_sha: null,
-                    head_sha: null,
-                    start_sha: null,
-                    position_type: "text",
-                },
-            }, value => CreateDraftNoteSchema.parse(value));
-            assert.equal(parsed.position, undefined);
-        });
-        test("should accept position with nullable old_line after sanitize", () => {
-            const parsed = parseAfterSanitize("create_draft_note", {
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                body: "Test note",
-                position: {
-                    base_sha: "base",
-                    head_sha: "head",
-                    start_sha: "start",
-                    position_type: "text",
-                    new_path: "file.ts",
-                    old_line: null,
-                    new_line: 10,
-                },
-            }, value => CreateDraftNoteSchema.parse(value));
-            assert.equal(parsed.position?.old_line, null);
-            assert.equal(parsed.position?.new_line, 10);
-        });
-    });
-    describe("with update_merge_request", () => {
-        test("should omit null milestone_id instead of coercing to string null", () => {
-            const parsed = UpdateMergeRequestSchema.parse({
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                title: "keep",
-                milestone_id: null,
-            });
-            assert.equal(parsed.milestone_id, undefined);
-        });
-        test("should keep 0 as unassign sentinel", () => {
-            const parsed = UpdateMergeRequestSchema.parse({
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                milestone_id: 0,
-            });
-            assert.equal(parsed.milestone_id, "0");
-        });
-    });
-    describe("with update_merge_request_discussion_note", () => {
-        test("should accept body plus resolved null after sanitize", () => {
-            const parsed = parseAfterSanitize("update_merge_request_discussion_note", {
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                discussion_id: "d1",
-                note_id: "n1",
-                body: "updated",
-                resolved: null,
-            }, value => UpdateMergeRequestDiscussionNoteSchema.parse(value));
-            assert.equal(parsed.body, "updated");
-            assert.equal(parsed.resolved, undefined);
-        });
-        test("should reject when only resolved null remains after sanitize", () => {
-            assert.throws(() => parseAfterSanitize("update_merge_request_discussion_note", {
-                project_id: PROJECT_ID,
-                merge_request_iid: MERGE_REQUEST_IID,
-                discussion_id: "d1",
-                note_id: "n1",
-                resolved: null,
-            }, value => UpdateMergeRequestDiscussionNoteSchema.parse(value)), /At least one of 'body' or 'resolved'/);
-        });
-    });
-    describe("with get_branch_diffs", () => {
-        test("should ignore unknown commit null field from older clients", () => {
-            const parsed = parseAfterSanitize("get_branch_diffs", {
-                project_id: PROJECT_ID,
-                from: "main",
-                to: "dev",
-                commit: null,
-            }, value => GetBranchDiffsSchema.parse(value));
-            assert.equal(Object.hasOwn(parsed, "commit"), false);
-        });
-    });
-    describe("with execute_graphql", () => {
-        test("should preserve explicit null variable values after sanitize", () => {
-            const parsed = parseAfterSanitize("execute_graphql", {
-                query: "mutation($id: ID) { updateProject(input: { id: $id }) { project { id } } }",
-                variables: { id: null },
-            }, value => ExecuteGraphQLSchema.parse(value));
-            assert.equal(parsed.variables?.id, null);
-        });
-    });
-    describe("with create_label", () => {
-        test("should preserve explicit null priority after sanitize", () => {
-            const parsed = parseAfterSanitize("create_label", {
-                project_id: PROJECT_ID,
-                name: "priority-null",
-                color: "#112233",
-                priority: null,
-            }, value => CreateLabelSchema.parse(value));
-            assert.equal(parsed.priority, null);
-        });
-    });
-});