@zereight/mcp-gitlab 2.1.1 → 2.1.3

package/build/schemas.js

@@ -527,7 +527,7 @@ export const GitLabRepositorySchema = z.object({
     http_url_to_repo: z.string().optional(),
     created_at: z.string().optional(),
     last_activity_at: z.string().optional(),
-    default_branch: z.string().optional(),
+    default_branch: z.string().nullable().optional(),
     namespace: z
         .object({
         id: z.coerce.string(),
@@ -621,7 +621,7 @@ export const FileOperationSchema = z.object({
 export const GitLabTreeItemSchema = z.object({
     id: z.string(),
     name: z.string(),
-    type: z.enum(["tree", "blob"]),
+    type: z.enum(["tree", "blob", "commit"]),
     path: z.string(),
     mode: z.string(),
 });
@@ -1219,7 +1219,7 @@ export const CreateIssueSchema = ProjectParamsSchema.extend({
     title: z.string().describe("Issue title"),
     description: z.string().optional().describe("Issue description"),
     assignee_ids: z.array(z.coerce.number()).optional().describe("Array of user IDs to assign"),
-    labels: z.array(z.string()).optional().describe("Array of label names"),
+    labels: coerceStringArray.optional().describe("Array of label names"),
     milestone_id: z.coerce.string().optional().describe("Milestone ID to assign"),
     issue_type: z
         .enum(["issue", "incident", "test_case", "task"])
@@ -1239,7 +1239,7 @@ const MergeRequestOptionsSchema = {
         .array(z.coerce.number())
         .optional()
         .describe("The ID of the users to assign as reviewers of the MR"),
-    labels: z.array(z.string()).optional().describe("Labels for the MR"),
+    labels: coerceStringArray.optional().describe("Labels for the MR"),
     draft: z.coerce.boolean().optional().describe("Create as draft merge request"),
     allow_collaboration: z.coerce.boolean().optional().describe("Allow commits from upstream members"),
     remove_source_branch: z
@@ -1288,7 +1288,7 @@ export const UpdateMergeRequestSchema = GetMergeRequestSchema.extend({
         .array(z.coerce.number())
         .optional()
         .describe("The ID of the users to assign as reviewers of the MR"),
-    labels: z.array(z.string()).optional().describe("Labels for the MR"),
+    labels: coerceStringArray.optional().describe("Labels for the MR"),
     state_event: z
         .enum(["close", "reopen"])
         .optional()
@@ -1475,7 +1475,7 @@ export const ListIssuesSchema = z
     created_after: z.string().optional().describe("Return issues created after the given time"),
     created_before: z.string().optional().describe("Return issues created before the given time"),
     due_date: z.string().optional().describe("Return issues that have the due date"),
-    labels: z.array(z.string()).optional().describe("Array of label names"),
+    labels: coerceStringArray.optional().describe("Array of label names"),
     milestone: z.string().optional().describe("Milestone title"),
     issue_type: z
         .enum(["issue", "incident", "test_case", "task"])
@@ -1546,7 +1546,7 @@ export const ListMergeRequestsSchema = z
         .string()
         .optional()
         .describe("Return merge requests updated before the given time"),
-    labels: z.array(z.string()).optional().describe("Array of label names"),
+    labels: coerceStringArray.optional().describe("Array of label names"),
     milestone: z.string().optional().describe("Milestone title"),
     scope: z
         .enum(["created_by_me", "assigned_to_me", "all"])
@@ -2136,7 +2136,7 @@ export const MyIssuesSchema = z.object({
         .enum(["opened", "closed", "all"])
         .optional()
         .describe("Return issues with a specific state (default: opened)"),
-    labels: z.array(z.string()).optional().describe("Array of label names to filter by"),
+    labels: coerceStringArray.optional().describe("Array of label names to filter by"),
     milestone: z.string().optional().describe("Milestone title to filter by"),
     search: z.string().optional().describe("Search for specific terms in title and description"),
     created_after: z

package/build/test/schema-tests.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env ts-node
-import { GetFileContentsSchema, GitLabFileContentSchema, CreatePipelineSchema, CreateIssueNoteSchema, CreateMergeRequestEmojiReactionSchema, CreateIssueEmojiReactionSchema, DeleteMergeRequestEmojiReactionSchema, DeleteIssueEmojiReactionSchema, CreateWorkItemEmojiReactionSchema, CreateWorkItemNoteEmojiReactionSchema } from '../schemas.js';
+import { GetFileContentsSchema, GitLabFileContentSchema, GitLabRepositorySchema, CreatePipelineSchema, CreateIssueNoteSchema, CreateMergeRequestEmojiReactionSchema, CreateIssueEmojiReactionSchema, DeleteMergeRequestEmojiReactionSchema, DeleteIssueEmojiReactionSchema, CreateWorkItemEmojiReactionSchema, CreateWorkItemNoteEmojiReactionSchema, CreateIssueSchema, ListIssuesSchema, ListMergeRequestsSchema, GitLabTreeItemSchema } from '../schemas.js';
 function runGetFileContentsSchemaTests() {
     console.log('🧪 Testing GetFileContentsSchema...');
     const cases = [
@@ -426,14 +426,216 @@ function runEmojiReactionSchemaTests() {
     console.log(`\nResults: ${passed} passed, ${failed} failed`);
     return { passed, failed };
 }
+function runGitLabRepositorySchemaTests() {
+    console.log('🧪 Testing GitLabRepositorySchema...');
+    const baseProject = {
+        id: '42',
+        name: 'my-project',
+        path_with_namespace: 'group/my-project',
+        description: null,
+    };
+    const cases = [
+        {
+            name: 'schema:repository:default_branch-null',
+            input: { ...baseProject, default_branch: null },
+            shouldFail: false,
+        },
+        {
+            name: 'schema:repository:default_branch-string',
+            input: { ...baseProject, default_branch: 'main' },
+            shouldFail: false,
+        },
+        {
+            name: 'schema:repository:default_branch-omitted',
+            input: { ...baseProject },
+            shouldFail: false,
+        },
+    ];
+    let passed = 0;
+    let failed = 0;
+    cases.forEach(testCase => {
+        const result = { name: testCase.name, status: 'failed' };
+        const parsed = GitLabRepositorySchema.safeParse(testCase.input);
+        if (testCase.shouldFail) {
+            if (parsed.success) {
+                result.error = 'Expected schema validation to fail';
+            }
+            else {
+                result.status = 'passed';
+            }
+        }
+        else if (parsed.success) {
+            result.status = 'passed';
+        }
+        else {
+            result.error = parsed.error?.message || 'Schema validation failed';
+        }
+        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 runLabelsCoercionSchemaTests() {
+    console.log('\n=== Labels Coercion Schema Tests ===');
+    const cases = [
+        {
+            name: 'schema:create_issue:labels-native-array',
+            schema: CreateIssueSchema,
+            input: { project_id: 'my/project', title: 'Test', labels: ['bug', 'enhancement'] },
+            expectedLabels: ['bug', 'enhancement'],
+        },
+        {
+            name: 'schema:create_issue:labels-stringified-array',
+            schema: CreateIssueSchema,
+            input: { project_id: 'my/project', title: 'Test', labels: '["bug","enhancement"]' },
+            expectedLabels: ['bug', 'enhancement'],
+        },
+        {
+            name: 'schema:create_issue:labels-omitted',
+            schema: CreateIssueSchema,
+            input: { project_id: 'my/project', title: 'Test' },
+            expectedLabels: undefined,
+        },
+        {
+            name: 'schema:list_issues:labels-native-array',
+            schema: ListIssuesSchema,
+            input: { project_id: 'my/project', labels: ['bug'] },
+            expectedLabels: ['bug'],
+        },
+        {
+            name: 'schema:list_issues:labels-stringified-array',
+            schema: ListIssuesSchema,
+            input: { project_id: 'my/project', labels: '["bug","enhancement"]' },
+            expectedLabels: ['bug', 'enhancement'],
+        },
+        {
+            name: 'schema:list_merge_requests:labels-native-array',
+            schema: ListMergeRequestsSchema,
+            input: { labels: ['feature'] },
+            expectedLabels: ['feature'],
+        },
+        {
+            name: 'schema:list_merge_requests:labels-stringified-array',
+            schema: ListMergeRequestsSchema,
+            input: { labels: '["feature","bugfix"]' },
+            expectedLabels: ['feature', 'bugfix'],
+        },
+    ];
+    let passed = 0;
+    let failed = 0;
+    function checkLabelResult(testCase, parsed) {
+        const result = { name: testCase.name, status: 'failed' };
+        if (!parsed.success) {
+            result.error = parsed.error?.message || 'Schema validation failed';
+            return result;
+        }
+        const actualLabels = parsed.data['labels'];
+        if (testCase.expectedLabels === undefined) {
+            result.status = actualLabels === undefined ? 'passed' : 'failed';
+            if (actualLabels !== undefined) {
+                result.error = `Expected labels to be undefined, got ${JSON.stringify(actualLabels)}`;
+            }
+            return result;
+        }
+        const match = Array.isArray(actualLabels) &&
+            actualLabels.length === testCase.expectedLabels.length &&
+            testCase.expectedLabels.every((v, i) => actualLabels[i] === v);
+        result.status = match ? 'passed' : 'failed';
+        if (!match) {
+            result.error = `Expected ${JSON.stringify(testCase.expectedLabels)}, got ${JSON.stringify(actualLabels)}`;
+        }
+        return result;
+    }
+    cases.forEach(testCase => {
+        const parsed = testCase.schema.safeParse(testCase.input);
+        let result;
+        if (testCase.shouldFail) {
+            result = { name: testCase.name, status: parsed.success ? 'failed' : 'passed' };
+            if (parsed.success)
+                result.error = 'Expected schema validation to fail';
+        }
+        else {
+            result = checkLabelResult(testCase, parsed);
+        }
+        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 = [
+        {
+            name: 'schema:tree_item:type-blob',
+            input: { id: 'abc123', name: 'README.md', type: 'blob', path: 'README.md', mode: '100644' },
+        },
+        {
+            name: 'schema:tree_item:type-tree',
+            input: { id: 'def456', name: 'src', type: 'tree', path: 'src', mode: '040000' },
+        },
+        {
+            name: 'schema:tree_item:type-commit-submodule',
+            input: { id: 'ghi789', name: 'vendor', type: 'commit', path: 'vendor', mode: '160000' },
+        },
+        {
+            name: 'schema:tree_item:reject-unknown-type',
+            input: { id: 'xyz', name: 'foo', type: 'symlink', path: 'foo', mode: '120000' },
+            shouldFail: true,
+        },
+    ];
+    let passed = 0;
+    let failed = 0;
+    cases.forEach(testCase => {
+        const result = { name: testCase.name, status: 'failed' };
+        const parsed = GitLabTreeItemSchema.safeParse(testCase.input);
+        if (testCase.shouldFail) {
+            result.status = parsed.success ? 'failed' : 'passed';
+            if (parsed.success)
+                result.error = 'Expected schema validation to fail';
+        }
+        else if (parsed.success) {
+            result.status = 'passed';
+        }
+        else {
+            result.error = parsed.error?.message || 'Schema validation failed';
+        }
+        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 };
+}
 if (import.meta.url === `file://${process.argv[1]}`) {
     const getFileContentsResult = runGetFileContentsSchemaTests();
     const fileContentResult = runGitLabFileContentSchemaTests();
     const createPipelineResult = runCreatePipelineSchemaTests();
     const createIssueNoteResult = runCreateIssueNoteSchemaTests();
     const emojiReactionResult = runEmojiReactionSchemaTests();
-    const totalPassed = getFileContentsResult.passed + fileContentResult.passed + createPipelineResult.passed + createIssueNoteResult.passed + emojiReactionResult.passed;
-    const totalFailed = getFileContentsResult.failed + fileContentResult.failed + createPipelineResult.failed + createIssueNoteResult.failed + emojiReactionResult.failed;
+    const repositorySchemaResult = runGitLabRepositorySchemaTests();
+    const labelsCoercionResult = runLabelsCoercionSchemaTests();
+    const treeItemResult = runGitLabTreeItemSchemaTests();
+    const totalPassed = getFileContentsResult.passed + fileContentResult.passed + createPipelineResult.passed + createIssueNoteResult.passed + emojiReactionResult.passed + repositorySchemaResult.passed + labelsCoercionResult.passed + treeItemResult.passed;
+    const totalFailed = getFileContentsResult.failed + fileContentResult.failed + createPipelineResult.failed + createIssueNoteResult.failed + emojiReactionResult.failed + repositorySchemaResult.failed + labelsCoercionResult.failed + treeItemResult.failed;
     console.log(`\nTotal Results: ${totalPassed} passed, ${totalFailed} failed`);
     if (totalFailed > 0) {
         process.exit(1);

package/package.json

@@ -1,6 +1,7 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "2.1.1",
+  "version": "2.1.3",
+  "mcpName": "io.github.zereight/gitlab-mcp",
   "description": "GitLab MCP server for projects, merge requests, issues, pipelines, wiki, releases, and more",
   "keywords": [
     "gitlab",
@@ -24,7 +25,9 @@
   "license": "MIT",
   "author": "zereight",
   "type": "module",
-  "bin": "./build/index.js",
+  "bin": {
+    "mcp-gitlab": "build/index.js"
+  },
   "files": [
     "build"
   ],
@@ -59,6 +62,7 @@
     "lint": "eslint . --ext .ts",
     "lint:fix": "eslint . --ext .ts --fix",
     "release": "bash scripts/release.sh",
+    "release:mcp-registry": "bash scripts/publish_mcp_registry.sh",
     "format": "prettier --write \"**/*.{js,ts,json,md}\"",
     "format:check": "prettier --check \"**/*.{js,ts,json,md}\""
   },