@zereight/mcp-gitlab 2.1.9 → 2.1.10

package/build/schemas.js

@@ -1404,6 +1404,15 @@ export const GetBranchDiffsSchema = ProjectParamsSchema.extend({
         .describe('Array of regex patterns to exclude files from the diff results. Each pattern is a JavaScript-compatible regular expression that matches file paths to ignore. Examples: ["^vendor/", "^test/mocks/", "\\.spec\\.ts$", "package-lock\\.json"]'),
 });
 export const GetMergeRequestSchema = ProjectParamsSchema.extend({
+    project_id: z
+        .preprocess(value => (value === undefined || value === null ? value : String(value)), z
+        .string({
+        required_error: "project_id is required",
+        invalid_type_error: "project_id is required",
+    })
+        .refine(value => value === "" || value.trim().length > 0, "project_id is required")
+        .transform(value => (value === "" ? value : value.trim())))
+        .describe("Project ID or complete URL-encoded path to project"),
     merge_request_iid: z.coerce.string().optional().describe("The IID of a merge request"),
     source_branch: z.string().optional().describe("Source branch name"),
 });
@@ -1821,7 +1830,8 @@ export const ListProjectsSchema = z
 })
     .merge(PaginationOptionsSchema);
 // Label operation schemas
-export const ListLabelsSchema = z.object({
+export const ListLabelsSchema = z
+    .object({
     project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     with_counts: z
         .coerce.boolean()
@@ -1829,7 +1839,8 @@ export const ListLabelsSchema = z.object({
         .describe("Whether or not to include issue and merge request counts"),
     include_ancestor_groups: z.coerce.boolean().optional().describe("Include ancestor groups"),
     search: z.string().optional().describe("Keyword to filter labels by"),
-});
+})
+    .merge(PaginationOptionsSchema);
 export const GetLabelSchema = z.object({
     project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     label_id: z.coerce.string().describe("The ID or title of a project's label"),

package/build/test/dynamic-routing-tests.js

@@ -417,7 +417,7 @@ async function validateToolCalls(client, mockServer, expectedToken) {
         { name: 'get_merge_request', params: { project_id: '1', merge_request_iid: '1' } },
         { name: 'list_merge_requests', params: { project_id: '1' } },
         { name: 'get_repository_tree', params: { project_id: '1' } },
-        { name: 'list_labels', params: { project_id: '1' } },
+        { name: 'list_labels', params: { project_id: '1', page: 2, per_page: 50 } },
         { name: 'list_pipelines', params: { project_id: '1' } },
         { name: 'list_commits', params: { project_id: '1' } },
     ];
@@ -468,6 +468,10 @@ async function validateToolCalls(client, mockServer, expectedToken) {
             else {
                 assert.strictEqual(req.headers['private-token'], expectedToken);
             }
+            if (tool.name === 'list_labels') {
+                assert.strictEqual(req.query.page, '2');
+                assert.strictEqual(req.query.per_page, '50');
+            }
             res.json(mockResponse);
         });
         const result = await client.callTool(tool.name, tool.params);

package/build/test/schema-tests.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env ts-node
-import { GetFileContentsSchema, GitLabFileContentSchema, GitLabRepositorySchema, CreatePipelineSchema, CreateCommitStatusSchema, ListCommitStatusesSchema, CreateIssueNoteSchema, CreateMergeRequestEmojiReactionSchema, CreateIssueEmojiReactionSchema, DeleteMergeRequestEmojiReactionSchema, DeleteIssueEmojiReactionSchema, CreateWorkItemEmojiReactionSchema, CreateWorkItemNoteEmojiReactionSchema, CreateIssueSchema, ListIssuesSchema, ListMergeRequestsSchema, GitLabMergeRequestSchema, GitLabTreeItemSchema, GetMergeRequestSchema, GetRepositoryTreeSchema } from '../schemas.js';
+import { GetFileContentsSchema, GitLabFileContentSchema, GitLabRepositorySchema, CreatePipelineSchema, CreateCommitStatusSchema, ListCommitStatusesSchema, CreateIssueNoteSchema, CreateMergeRequestEmojiReactionSchema, CreateIssueEmojiReactionSchema, DeleteMergeRequestEmojiReactionSchema, DeleteIssueEmojiReactionSchema, CreateWorkItemEmojiReactionSchema, CreateWorkItemNoteEmojiReactionSchema, CreateIssueSchema, ListIssuesSchema, ListMergeRequestsSchema, ListLabelsSchema, GitLabMergeRequestSchema, GitLabTreeItemSchema, GetMergeRequestSchema, GetRepositoryTreeSchema } from '../schemas.js';
 function runGetFileContentsSchemaTests() {
     console.log('🧪 Testing GetFileContentsSchema...');
     const cases = [
@@ -534,11 +534,36 @@ function runGetMergeRequestSchemaTests() {
             input: { project_id: 'my/project', merge_request_iid: 24 },
             expected: { project_id: 'my/project', merge_request_iid: '24' },
         },
+        {
+            name: 'schema:get_merge_request:coerced-project-id',
+            input: { project_id: 123, merge_request_iid: '42' },
+            expected: { project_id: '123', merge_request_iid: '42' },
+        },
         {
             name: 'schema:get_merge_request:coerced-source-branch',
             input: { project_id: 'my/project', source_branch: 'feature' },
             expected: { project_id: 'my/project', source_branch: 'feature' },
         },
+        {
+            name: 'schema:get_merge_request:reject-missing-project-id-with-merge-request-iid',
+            input: { merge_request_iid: '42' },
+            shouldFail: true,
+        },
+        {
+            name: 'schema:get_merge_request:reject-missing-project-id-with-source-branch',
+            input: { source_branch: 'feature' },
+            shouldFail: true,
+        },
+        {
+            name: 'schema:get_merge_request:allow-empty-project-id-for-default-project',
+            input: { project_id: '', merge_request_iid: '42' },
+            expected: { project_id: '', merge_request_iid: '42' },
+        },
+        {
+            name: 'schema:get_merge_request:reject-whitespace-project-id',
+            input: { project_id: '   ', merge_request_iid: '42' },
+            shouldFail: true,
+        },
     ];
     let passed = 0;
     let failed = 0;
@@ -876,6 +901,52 @@ function runLabelsCoercionSchemaTests() {
     console.log(`\nResults: ${passed} passed, ${failed} failed`);
     return { passed, failed };
 }
+function runListLabelsSchemaTests() {
+    console.log('\n=== List Labels Schema Tests ===');
+    const cases = [
+        {
+            name: 'schema:list_labels:pagination-coercion',
+            input: { project_id: 'my/project', page: '2', per_page: '100' },
+            expected: { project_id: 'my/project', page: 2, per_page: 100 },
+        },
+        {
+            name: 'schema:list_labels:filters-with-pagination',
+            input: { project_id: 'my/project', search: 'backend', with_counts: 'true', page: 3, per_page: 50 },
+            expected: { project_id: 'my/project', search: 'backend', with_counts: true, page: 3, per_page: 50 },
+        },
+    ];
+    let passed = 0;
+    let failed = 0;
+    cases.forEach(testCase => {
+        const result = { name: testCase.name, status: 'failed' };
+        const parsed = ListLabelsSchema.safeParse(testCase.input);
+        if (!parsed.success) {
+            result.error = parsed.error?.message || 'Schema validation failed';
+        }
+        else {
+            const matches = Object.entries(testCase.expected).every(([key, value]) => {
+                const actual = parsed.data[key];
+                return actual === value;
+            });
+            if (matches) {
+                result.status = 'passed';
+            }
+            else {
+                result.error = `Unexpected parsed result: ${JSON.stringify(parsed.data)}`;
+            }
+        }
+        if (result.status === 'passed') {
+            passed++;
+            console.log(`✅ ${result.name}`);
+        }
+        else {
+            failed++;
+            console.log(`❌ ${result.name}: ${result.error}`);
+        }
+    });
+    console.log(`\nResults: ${passed} passed, ${failed} failed`);
+    return { passed, failed };
+}
 function runGitLabTreeItemSchemaTests() {
     console.log('\n=== GitLabTreeItem Schema Tests ===');
     const cases = [
@@ -1008,10 +1079,11 @@ if (import.meta.url === `file://${process.argv[1]}`) {
     const emojiReactionResult = runEmojiReactionSchemaTests();
     const repositorySchemaResult = runGitLabRepositorySchemaTests();
     const labelsCoercionResult = runLabelsCoercionSchemaTests();
+    const listLabelsResult = runListLabelsSchemaTests();
     const treeItemResult = runGitLabTreeItemSchemaTests();
     const repositoryTreeResult = runGetRepositoryTreeSchemaTests();
-    const totalPassed = getFileContentsResult.passed + fileContentResult.passed + createPipelineResult.passed + commitStatusResult.passed + createIssueNoteResult.passed + getMergeRequestResult.passed + gitLabMergeRequestResult.passed + emojiReactionResult.passed + repositorySchemaResult.passed + labelsCoercionResult.passed + treeItemResult.passed + repositoryTreeResult.passed;
-    const totalFailed = getFileContentsResult.failed + fileContentResult.failed + createPipelineResult.failed + commitStatusResult.failed + createIssueNoteResult.failed + getMergeRequestResult.failed + gitLabMergeRequestResult.failed + emojiReactionResult.failed + repositorySchemaResult.failed + labelsCoercionResult.failed + treeItemResult.failed + repositoryTreeResult.failed;
+    const totalPassed = getFileContentsResult.passed + fileContentResult.passed + createPipelineResult.passed + commitStatusResult.passed + createIssueNoteResult.passed + getMergeRequestResult.passed + gitLabMergeRequestResult.passed + emojiReactionResult.passed + repositorySchemaResult.passed + labelsCoercionResult.passed + listLabelsResult.passed + treeItemResult.passed + repositoryTreeResult.passed;
+    const totalFailed = getFileContentsResult.failed + fileContentResult.failed + createPipelineResult.failed + commitStatusResult.failed + createIssueNoteResult.failed + getMergeRequestResult.failed + gitLabMergeRequestResult.failed + emojiReactionResult.failed + repositorySchemaResult.failed + labelsCoercionResult.failed + listLabelsResult.failed + treeItemResult.failed + repositoryTreeResult.failed;
     console.log(`\nTotal Results: ${totalPassed} passed, ${totalFailed} failed`);
     if (totalFailed > 0) {
         process.exit(1);

package/build/test/test-json-schema.js

@@ -124,6 +124,24 @@ function runTests() {
             error: error.message,
         });
     }
+    // Test 7: Effects fields
+    try {
+        const schema = z.object({
+            effectRequired: z.preprocess(value => value, z.string()),
+            effectOptional: z.preprocess(value => value, z.string().optional()),
+        });
+        const result = toJSONSchema(schema);
+        assert(result.required?.includes("effectRequired"), "effectRequired should be in required array");
+        assert(!result.required?.includes("effectOptional"), "effectOptional should NOT be in required array");
+        results.push({ name: "effects fields", status: "passed" });
+    }
+    catch (error) {
+        results.push({
+            name: "effects fields",
+            status: "failed",
+            error: error.message,
+        });
+    }
     // Print results
     const passed = results.filter((r) => r.status === "passed").length;
     const failed = results.filter((r) => r.status === "failed").length;

package/build/utils/schema.js

@@ -6,6 +6,23 @@ import { zodToJsonSchema } from "zod-to-json-schema";
  */
 export const toJSONSchema = (schema) => {
     const jsonSchema = zodToJsonSchema(schema, { $refStrategy: "none" });
+    const isOptionalLikeField = (zodType) => {
+        const def = zodType._def;
+        const typeName = def?.typeName;
+        if (["ZodOptional", "ZodNullable", "ZodDefault", "ZodCatch"].includes(typeName)) {
+            return true;
+        }
+        if (typeName === "ZodEffects") {
+            return isOptionalLikeField(def.schema);
+        }
+        if (typeName === "ZodBranded") {
+            return isOptionalLikeField(def.type);
+        }
+        if (typeName === "ZodPipeline") {
+            return isOptionalLikeField(def.in);
+        }
+        return false;
+    };
     // Extract required fields from Zod schema
     const zodRequiredFields = (() => {
         if (schema instanceof z.ZodObject) {
@@ -13,17 +30,7 @@ export const toJSONSchema = (schema) => {
             const requiredFields = [];
             Object.entries(shape).forEach(([key, fieldDef]) => {
                 const zodType = fieldDef;
-                const typeName = zodType._def?.typeName;
-                // Check if field is wrapped in zod required types
-                const isRequired = [
-                    "ZodOptional",
-                    "ZodNullable",
-                    "ZodDefault",
-                    "ZodEffects",
-                    "ZodCatch",
-                    "ZodBranded",
-                ].includes(typeName);
-                if (!isRequired) {
+                if (!isOptionalLikeField(zodType)) {
                     requiredFields.push(key);
                 }
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "2.1.9",
+  "version": "2.1.10",
   "mcpName": "io.github.zereight/gitlab-mcp",
   "description": "GitLab MCP server for projects, merge requests, issues, pipelines, wiki, releases, and more",
   "keywords": [