@zereight/mcp-gitlab 1.0.72 → 1.0.74

package/build/customSchemas.js

@@ -0,0 +1,19 @@
+import { z } from "zod";
+import { pino } from 'pino';
+const logger = pino({
+    level: process.env.LOG_LEVEL || 'info',
+    transport: {
+        target: 'pino-pretty',
+        options: {
+            colorize: true,
+            levelFirst: true,
+            destination: 2,
+        },
+    },
+});
+export const flexibleBoolean = z.preprocess((val) => {
+    if (typeof val === 'string') {
+        return val.toLowerCase() === 'true';
+    }
+    return val;
+}, z.boolean());

package/build/index.js

@@ -845,12 +845,12 @@ async function listIssues(projectId, options = {}) {
                         url.searchParams.append(`${key}[]`, label.toString());
                     });
                 }
-                else {
+                else if (value) {
                     url.searchParams.append(`${key}[]`, value.toString());
                 }
             }
             else {
-                url.searchParams.append(key, value.toString());
+                url.searchParams.append(key, String(value));
             }
         }
     });
@@ -879,7 +879,7 @@ async function listMergeRequests(projectId, options = {}) {
                 url.searchParams.append(key, value.join(","));
             }
             else {
-                url.searchParams.append(key, value.toString());
+                url.searchParams.append(key, String(value));
             }
         }
     });
@@ -2617,6 +2617,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         if (GITLAB_AUTH_COOKIE_PATH) {
             await ensureSessionForRequest();
         }
+        logger.info(request.params.name);
         switch (request.params.name) {
             case "fork_repository": {
                 if (GITLAB_PROJECT_ID) {
@@ -3305,6 +3306,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
     }
     catch (error) {
+        logger.debug(request.params);
         if (error instanceof z.ZodError) {
             throw new Error(`Invalid arguments: ${error.errors
                 .map(e => `${e.path.join(".")}: ${e.message}`)

package/build/schemas.js

@@ -1,10 +1,5 @@
 import { z } from "zod";
-const flexibleBoolean = z.preprocess((val) => {
-    if (typeof val === 'string') {
-        return val.toLowerCase() === 'true';
-    }
-    return val;
-}, z.boolean());
+import { flexibleBoolean } from "./customSchemas.js";
 // Base schemas for common types
 export const GitLabAuthorSchema = z.object({
     name: z.string(),
@@ -13,8 +8,8 @@ export const GitLabAuthorSchema = z.object({
 });
 // Pipeline related schemas
 export const GitLabPipelineSchema = z.object({
-    id: z.string().or(z.number()),
-    project_id: z.string().or(z.number()),
+    id: z.coerce.string(),
+    project_id: z.coerce.string(),
     sha: z.string(),
     ref: z.string(),
     status: z.string(),
@@ -28,7 +23,7 @@ export const GitLabPipelineSchema = z.object({
     coverage: z.number().nullable().optional(),
     user: z
         .object({
-        id: z.string().or(z.number()),
+        id: z.coerce.string(),
         name: z.string(),
         username: z.string(),
         avatar_url: z.string().nullable().optional(),
@@ -57,7 +52,7 @@ export const GitLabPipelineSchema = z.object({
 });
 // Pipeline job related schemas
 export const GitLabPipelineJobSchema = z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     status: z.string(),
     stage: z.string(),
     name: z.string(),
@@ -70,7 +65,7 @@ export const GitLabPipelineJobSchema = z.object({
     duration: z.number().nullable().optional(),
     user: z
         .object({
-        id: z.string().or(z.number()),
+        id: z.coerce.string(),
         name: z.string(),
         username: z.string(),
         avatar_url: z.string().nullable().optional(),
@@ -87,8 +82,8 @@ export const GitLabPipelineJobSchema = z.object({
         .optional(),
     pipeline: z
         .object({
-        id: z.string().or(z.number()),
-        project_id: z.string().or(z.number()),
+        id: z.coerce.string(),
+        project_id: z.coerce.string(),
         status: z.string(),
         ref: z.string(),
         sha: z.string(),
@@ -104,7 +99,7 @@ export const PaginationOptionsSchema = z.object({
 });
 // Schema for listing pipelines
 export const ListPipelinesSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     scope: z
         .enum(["running", "pending", "finished", "branches", "tags"])
         .optional()
@@ -145,13 +140,13 @@ export const ListPipelinesSchema = z.object({
 }).merge(PaginationOptionsSchema);
 // Schema for getting a specific pipeline
 export const GetPipelineSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    pipeline_id: z.string().or(z.number().describe("The ID of the pipeline")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    pipeline_id: z.coerce.string().describe("The ID of the pipeline"),
 });
 // Schema for listing jobs in a pipeline
 export const ListPipelineJobsSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    pipeline_id: z.string().or(z.number().describe("The ID of the pipeline")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    pipeline_id: z.coerce.string().describe("The ID of the pipeline"),
     scope: z
         .enum(["created", "pending", "running", "failed", "success", "canceled", "skipped", "manual"])
         .optional()
@@ -160,7 +155,7 @@ export const ListPipelineJobsSchema = z.object({
 }).merge(PaginationOptionsSchema);
 // Schema for creating a new pipeline
 export const CreatePipelineSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     ref: z.string().describe("The branch or tag to run the pipeline on"),
     variables: z
         .array(z.object({
@@ -172,25 +167,22 @@ export const CreatePipelineSchema = z.object({
 });
 // Schema for retrying a pipeline
 export const RetryPipelineSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    pipeline_id: z.string().or(z.number().describe("The ID of the pipeline to retry")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    pipeline_id: z.coerce.string().describe("The ID of the pipeline to retry"),
 });
 // Schema for canceling a pipeline
-export const CancelPipelineSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    pipeline_id: z.string().or(z.number().describe("The ID of the pipeline to cancel")),
-});
+export const CancelPipelineSchema = RetryPipelineSchema;
 // Schema for the input parameters for pipeline job operations
 export const GetPipelineJobOutputSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    job_id: z.string().or(z.number().describe("The ID of the job")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    job_id: z.coerce.string().describe("The ID of the job"),
     limit: z.number().optional().describe("Maximum number of lines to return from the end of the log (default: 1000)"),
     offset: z.number().optional().describe("Number of lines to skip from the end of the log (default: 0)"),
 });
 // User schemas
 export const GitLabUserSchema = z.object({
     username: z.string(), // Changed from login to match GitLab API
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     name: z.string(),
     avatar_url: z.string().nullable(),
     web_url: z.string(), // Changed from html_url to match GitLab API
@@ -199,7 +191,7 @@ export const GetUsersSchema = z.object({
     usernames: z.array(z.string()).describe("Array of usernames to search for"),
 });
 export const GitLabUsersResponseSchema = z.record(z.string(), z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     username: z.string(),
     name: z.string(),
     avatar_url: z.string().nullable(),
@@ -208,15 +200,15 @@ export const GitLabUsersResponseSchema = z.record(z.string(), z.object({
 // Namespace related schemas
 // Base schema for project-related operations
 const ProjectParamsSchema = z.object({
-    project_id: z.string().describe("Project ID or complete URL-encoded path to project"), // Changed from owner/repo to match GitLab API
+    project_id: z.coerce.string().describe("Project ID or complete URL-encoded path to project"), // Changed from owner/repo to match GitLab API
 });
 export const GitLabNamespaceSchema = z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     name: z.string(),
     path: z.string(),
     kind: z.enum(["user", "group"]),
     full_path: z.string(),
-    parent_id: z.string().or(z.number().nullable()),
+    parent_id: z.coerce.string().nullable(),
     avatar_url: z.string().nullable(),
     web_url: z.string(),
     members_count_with_descendants: z.number().optional(),
@@ -237,14 +229,14 @@ export const GitLabNamespaceExistsResponseSchema = z.object({
 // Repository related schemas
 export const GitLabOwnerSchema = z.object({
     username: z.string(), // Changed from login to match GitLab API
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     avatar_url: z.string().nullable(),
     web_url: z.string(), // Changed from html_url to match GitLab API
     name: z.string(), // Added as GitLab includes full name
     state: z.string(), // Added as GitLab includes user state
 });
 export const GitLabRepositorySchema = z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     name: z.string(),
     path_with_namespace: z.string(),
     visibility: z.string().optional(),
@@ -259,7 +251,7 @@ export const GitLabRepositorySchema = z.object({
     default_branch: z.string().optional(),
     namespace: z
         .object({
-        id: z.string().or(z.number()),
+        id: z.coerce.string(),
         name: z.string(),
         path: z.string(),
         kind: z.string(),
@@ -306,7 +298,7 @@ export const GitLabRepositorySchema = z.object({
     shared_runners_enabled: flexibleBoolean.optional(),
     shared_with_groups: z
         .array(z.object({
-        group_id: z.string().or(z.number()),
+        group_id: z.coerce.string(),
         group_name: z.string(),
         group_full_path: z.string(),
         group_access_level: z.number(),
@@ -355,7 +347,7 @@ export const GitLabTreeItemSchema = z.object({
     mode: z.string(),
 });
 export const GetRepositoryTreeSchema = z.object({
-    project_id: z.string().describe("The ID or URL-encoded path of the project"),
+    project_id: z.coerce.string().describe("The ID or URL-encoded path of the project"),
     path: z.string().optional().describe("The path inside the repository"),
     ref: z
         .string()
@@ -402,9 +394,9 @@ export const GitLabReferenceSchema = z.object({
 });
 // Milestones rest api output schemas
 export const GitLabMilestonesSchema = z.object({
-    id: z.string().or(z.number()),
-    iid: z.string().or(z.number()),
-    project_id: z.string().or(z.number()),
+    id: z.coerce.string(),
+    iid: z.coerce.string(),
+    project_id: z.coerce.string(),
     title: z.string(),
     description: z.string().nullable(),
     due_date: z.string().nullable(),
@@ -426,7 +418,7 @@ export const CreateIssueOptionsSchema = z.object({
     title: z.string(),
     description: z.string().optional(), // Changed from body to match GitLab API
     assignee_ids: z.array(z.number()).optional(), // Changed from assignees to match GitLab API
-    milestone_id: z.string().or(z.number().optional()), // Changed from milestone to match GitLab API
+    milestone_id: z.coerce.string().optional(), // Changed from milestone to match GitLab API
     labels: z.array(z.string()).optional(),
 });
 export const GitLabDiffSchema = z.object({
@@ -473,7 +465,7 @@ export const GitLabCompareResultSchema = z.object({
 });
 // Issue related schemas
 export const GitLabLabelSchema = z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     name: z.string(),
     color: z.string(),
     text_color: z.string(),
@@ -487,17 +479,17 @@ export const GitLabLabelSchema = z.object({
     is_project_label: flexibleBoolean.optional(),
 });
 export const GitLabMilestoneSchema = z.object({
-    id: z.string().or(z.number()),
-    iid: z.string().or(z.number()), // Added to match GitLab API
+    id: z.coerce.string(),
+    iid: z.coerce.string(), // Added to match GitLab API
     title: z.string(),
     description: z.string().nullable().default(""),
     state: z.string(),
     web_url: z.string(), // Changed from html_url to match GitLab API
 });
 export const GitLabIssueSchema = z.object({
-    id: z.string().or(z.number()),
-    iid: z.string().or(z.number()), // Added to match GitLab API
-    project_id: z.string().or(z.number()), // Added to match GitLab API
+    id: z.coerce.string(),
+    iid: z.coerce.string(), // Added to match GitLab API
+    project_id: z.coerce.string(), // Added to match GitLab API
     title: z.string(),
     description: z.string().nullable().default(""), // Changed from body to match GitLab API
     state: z.string(),
@@ -531,7 +523,7 @@ export const GitLabIssueSchema = z.object({
 });
 // NEW SCHEMA: For issue with link details (used in listing issue links)
 export const GitLabIssueWithLinkDetailsSchema = GitLabIssueSchema.extend({
-    issue_link_id: z.string().or(z.number()),
+    issue_link_id: z.coerce.string(),
     link_type: z.enum(["relates_to", "blocks", "is_blocked_by"]),
     link_created_at: z.string(),
     link_updated_at: z.string(),
@@ -543,7 +535,7 @@ export const GitLabForkParentSchema = z.object({
     owner: z
         .object({
         username: z.string(), // Changed from login to match GitLab API
-        id: z.string().or(z.number()),
+        id: z.coerce.string(),
         avatar_url: z.string().nullable(),
     })
         .optional(), // Made optional to handle cases where GitLab API doesn't include it
@@ -559,9 +551,9 @@ export const GitLabMergeRequestDiffRefSchema = z.object({
     start_sha: z.string(),
 });
 export const GitLabMergeRequestSchema = z.object({
-    id: z.string().or(z.number()),
-    iid: z.string().or(z.number()),
-    project_id: z.string().or(z.number()),
+    id: z.coerce.string(),
+    iid: z.coerce.string(),
+    project_id: z.coerce.string(),
     title: z.string(),
     description: z.string().nullable(),
     state: z.string(),
@@ -609,7 +601,7 @@ export const LineRangeSchema = z.object({
 }).describe("Line range for multiline comments on GitLab merge request diffs. VALIDATION RULES: 1) line_code is critical for GitLab API success, 2) start/end must have consistent types, 3) line numbers must form valid range, 4) get line_code from GitLab diff API, never generate manually.");
 // Discussion related schemas
 export const GitLabDiscussionNoteSchema = z.object({
-    id: z.string().or(z.number()),
+    id: z.coerce.string(),
     type: z.enum(["DiscussionNote", "DiffNote", "Note"]).nullable(), // Allow null type for regular notes
     body: z.string(),
     attachment: z.any().nullable(), // Can be string or object, handle appropriately
@@ -617,10 +609,10 @@ export const GitLabDiscussionNoteSchema = z.object({
     created_at: z.string(),
     updated_at: z.string(),
     system: flexibleBoolean,
-    noteable_id: z.string().or(z.number()),
+    noteable_id: z.coerce.string(),
     noteable_type: z.enum(["Issue", "MergeRequest", "Snippet", "Commit", "Epic"]),
-    project_id: z.string().or(z.number().optional()), // Optional for group-level discussions like Epics
-    noteable_iid: z.coerce.number().nullable(),
+    project_id: z.coerce.string().optional(),
+    noteable_iid: z.coerce.string().nullable().optional(),
     resolvable: flexibleBoolean.optional(),
     resolved: flexibleBoolean.optional(),
     resolved_by: GitLabUserSchema.nullable().optional(),
@@ -660,7 +652,7 @@ export const PaginatedResponseSchema = z.object({
     pagination: GitLabPaginationSchema.optional(),
 });
 export const GitLabDiscussionSchema = z.object({
-    id: z.string(),
+    id: z.coerce.string(),
     individual_note: flexibleBoolean,
     notes: z.array(GitLabDiscussionNoteSchema),
 });
@@ -670,18 +662,18 @@ export const PaginatedDiscussionsResponseSchema = z.object({
     pagination: GitLabPaginationSchema,
 });
 export const ListIssueDiscussionsSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of the project issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of the project issue"),
 }).merge(PaginationOptionsSchema);
 // Input schema for listing merge request discussions
 export const ListMergeRequestDiscussionsSchema = ProjectParamsSchema.extend({
-    merge_request_iid: z.string().or(z.number().describe("The IID of a merge request")),
+    merge_request_iid: z.coerce.string().describe("The IID of a merge request"),
 }).merge(PaginationOptionsSchema);
 // Input schema for updating a merge request discussion note
 export const UpdateMergeRequestNoteSchema = ProjectParamsSchema.extend({
-    merge_request_iid: z.string().or(z.number().describe("The IID of a merge request")),
-    discussion_id: z.string().describe("The ID of a thread"),
-    note_id: z.string().or(z.number().describe("The ID of a thread note")),
+    merge_request_iid: z.coerce.string().describe("The IID of a merge request"),
+    discussion_id: z.coerce.string().describe("The ID of a thread"),
+    note_id: z.coerce.string().describe("The ID of a thread note"),
     body: z.string().optional().describe("The content of the note or reply"),
     resolved: flexibleBoolean.optional().describe("Resolve or unresolve the note"),
 })
@@ -693,22 +685,22 @@ export const UpdateMergeRequestNoteSchema = ProjectParamsSchema.extend({
 });
 // Input schema for adding a note to an existing merge request discussion
 export const CreateMergeRequestNoteSchema = ProjectParamsSchema.extend({
-    merge_request_iid: z.string().or(z.number().describe("The IID of a merge request")),
-    discussion_id: z.string().describe("The ID of a thread"),
+    merge_request_iid: z.coerce.string().describe("The IID of a merge request"),
+    discussion_id: z.coerce.string().describe("The ID of a thread"),
     body: z.string().describe("The content of the note or reply"),
     created_at: z.string().optional().describe("Date the note was created at (ISO 8601 format)"),
 });
 // Input schema for updating an issue discussion note
 export const UpdateIssueNoteSchema = ProjectParamsSchema.extend({
-    issue_iid: z.string().or(z.number().describe("The IID of an issue")),
-    discussion_id: z.string().describe("The ID of a thread"),
-    note_id: z.string().or(z.number().describe("The ID of a thread note")),
+    issue_iid: z.coerce.string().describe("The IID of an issue"),
+    discussion_id: z.coerce.string().describe("The ID of a thread"),
+    note_id: z.coerce.string().describe("The ID of a thread note"),
     body: z.string().describe("The content of the note or reply"),
 });
 // Input schema for adding a note to an existing issue discussion
 export const CreateIssueNoteSchema = ProjectParamsSchema.extend({
-    issue_iid: z.string().or(z.number().describe("The IID of an issue")),
-    discussion_id: z.string().describe("The ID of a thread"),
+    issue_iid: z.coerce.string().describe("The IID of an issue"),
+    discussion_id: z.coerce.string().describe("The ID of a thread"),
     body: z.string().describe("The content of the note or reply"),
     created_at: z.string().optional().describe("Date the note was created at (ISO 8601 format)"),
 });
@@ -753,7 +745,7 @@ export const CreateIssueSchema = ProjectParamsSchema.extend({
     description: z.string().optional().describe("Issue description"),
     assignee_ids: z.array(z.number()).optional().describe("Array of user IDs to assign"),
     labels: z.array(z.string()).optional().describe("Array of label names"),
-    milestone_id: z.string().or(z.number().optional().describe("Milestone ID to assign")),
+    milestone_id: z.coerce.string().optional().describe("Milestone ID to assign"),
 });
 const MergeRequestOptionsSchema = {
     title: z.string().describe("Merge request title"),
@@ -794,7 +786,7 @@ export const GetBranchDiffsSchema = ProjectParamsSchema.extend({
     excluded_file_patterns: z.array(z.string()).optional().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: [\"^test/mocks/\", \"\\.spec\\.ts$\", \"package-lock\\.json\"]"),
 });
 export const GetMergeRequestSchema = ProjectParamsSchema.extend({
-    merge_request_iid: z.string().or(z.number().optional().describe("The IID of a merge request")),
+    merge_request_iid: z.coerce.string().optional().describe("The IID of a merge request"),
     source_branch: z.string().optional().describe("Source branch name"),
 });
 export const UpdateMergeRequestSchema = GetMergeRequestSchema.extend({
@@ -830,19 +822,19 @@ export const ListMergeRequestDiffsSchema = GetMergeRequestSchema.extend({
     unidiff: flexibleBoolean.optional().describe("Present diffs in the unified diff format. Default is false. Introduced in GitLab 16.5."),
 });
 export const CreateNoteSchema = z.object({
-    project_id: z.string().describe("Project ID or namespace/project_path"),
+    project_id: z.coerce.string().describe("Project ID or namespace/project_path"),
     noteable_type: z
         .enum(["issue", "merge_request"])
         .describe("Type of noteable (issue or merge_request)"),
-    noteable_iid: z.coerce.number().describe("IID of the issue or merge request"),
+    noteable_iid: z.coerce.string().describe("IID of the issue or merge request"),
     body: z.string().describe("Note content"),
 });
 // Issues API operation schemas
 export const ListIssuesSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    assignee_id: z.string().or(z.number().optional().describe("Return issues assigned to the given user ID")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    assignee_id: z.coerce.string().optional().describe("Return issues assigned to the given user ID. user id or none or any"),
     assignee_username: z.array(z.string()).optional().describe("Return issues assigned to the given username"),
-    author_id: z.string().or(z.number().optional().describe("Return issues created by the given user ID")),
+    author_id: z.coerce.string().optional().describe("Return issues created by the given user ID"),
     author_username: z.string().optional().describe("Return issues created by the given username"),
     confidential: flexibleBoolean.optional().describe("Filter confidential or public issues"),
     created_after: z.string().optional().describe("Return issues created after the given time"),
@@ -865,24 +857,20 @@ export const ListIssuesSchema = z.object({
 }).merge(PaginationOptionsSchema);
 // Merge Requests API operation schemas
 export const ListMergeRequestsSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    assignee_id: z
-        .number()
-        .optional()
-        .describe("Returns merge requests assigned to the given user ID"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    assignee_id: z.coerce.string().optional().describe("Return issues assigned to the given user ID. user id or none or any"),
     assignee_username: z
         .string()
         .optional()
         .describe("Returns merge requests assigned to the given username"),
-    author_id: z.string().or(z.number().optional().describe("Returns merge requests created by the given user ID")),
+    author_id: z.coerce.string().optional().describe("Returns merge requests created by the given user ID"),
     author_username: z
         .string()
         .optional()
         .describe("Returns merge requests created by the given username"),
-    reviewer_id: z
-        .number()
+    reviewer_id: z.coerce.string()
         .optional()
-        .describe("Returns merge requests which have the user as a reviewer"),
+        .describe("Returns merge requests which have the user as a reviewer. user id or none or any"),
     reviewer_username: z
         .string()
         .optional()
@@ -934,12 +922,12 @@ export const ListMergeRequestsSchema = z.object({
     with_labels_details: flexibleBoolean.optional().describe("Return more details for each label"),
 }).merge(PaginationOptionsSchema);
 export const GetIssueSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of the project issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of the project issue"),
 });
 export const UpdateIssueSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of the project issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of the project issue"),
     title: z.string().optional().describe("The title of the issue"),
     description: z.string().optional().describe("The description of the issue"),
     assignee_ids: z.array(z.number()).optional().describe("Array of user IDs to assign issue to"),
@@ -947,13 +935,13 @@ export const UpdateIssueSchema = z.object({
     discussion_locked: flexibleBoolean.optional().describe("Flag to lock discussions"),
     due_date: z.string().optional().describe("Date the issue is due (YYYY-MM-DD)"),
     labels: z.array(z.string()).optional().describe("Array of label names"),
-    milestone_id: z.string().or(z.number().optional().describe("Milestone ID to assign")),
+    milestone_id: z.coerce.string().optional().describe("Milestone ID to assign"),
     state_event: z.enum(["close", "reopen"]).optional().describe("Update issue state (close/reopen)"),
     weight: z.number().optional().describe("Weight of the issue (0-9)"),
 });
 export const DeleteIssueSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of the project issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of the project issue"),
 });
 // Issue links related schemas
 export const GitLabIssueLinkSchema = z.object({
@@ -962,28 +950,28 @@ export const GitLabIssueLinkSchema = z.object({
     link_type: z.enum(["relates_to", "blocks", "is_blocked_by"]),
 });
 export const ListIssueLinksSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of a project's issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of a project's issue"),
 });
 export const GetIssueLinkSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of a project's issue")),
-    issue_link_id: z.string().or(z.number().describe("ID of an issue relationship")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of a project's issue"),
+    issue_link_id: z.coerce.string().describe("ID of an issue relationship"),
 });
 export const CreateIssueLinkSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of a project's issue")),
-    target_project_id: z.string().describe("The ID or URL-encoded path of a target project"),
-    target_issue_iid: z.string().or(z.number().describe("The internal ID of a target project's issue")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of a project's issue"),
+    target_project_id: z.coerce.string().describe("The ID or URL-encoded path of a target project"),
+    target_issue_iid: z.coerce.string().describe("The internal ID of a target project's issue"),
     link_type: z
         .enum(["relates_to", "blocks", "is_blocked_by"])
         .optional()
         .describe("The type of the relation, defaults to relates_to"),
 });
 export const DeleteIssueLinkSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    issue_iid: z.string().or(z.number().describe("The internal ID of a project's issue")),
-    issue_link_id: z.string().or(z.number().describe("The ID of an issue relationship")),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
+    issue_iid: z.coerce.string().describe("The internal ID of a project's issue"),
+    issue_link_id: z.coerce.string().describe("The ID of an issue relationship"),
 });
 // Namespace API operation schemas
 export const ListNamespacesSchema = z.object({
@@ -991,14 +979,14 @@ export const ListNamespacesSchema = z.object({
     owned: flexibleBoolean.optional().describe("Filter for namespaces owned by current user"),
 }).merge(PaginationOptionsSchema);
 export const GetNamespaceSchema = z.object({
-    namespace_id: z.string().describe("Namespace ID or full path"),
+    namespace_id: z.coerce.string().describe("Namespace ID or full path"),
 });
 export const VerifyNamespaceSchema = z.object({
     path: z.string().describe("Namespace path to verify"),
 });
 // Project API operation schemas
 export const GetProjectSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
 });
 export const ListProjectsSchema = z.object({
     search: z.string().optional().describe("Search term for projects"),
@@ -1031,7 +1019,7 @@ export const ListProjectsSchema = z.object({
 }).merge(PaginationOptionsSchema);
 // Label operation schemas
 export const ListLabelsSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     with_counts: z
         .boolean()
         .optional()
@@ -1040,12 +1028,12 @@ export const ListLabelsSchema = z.object({
     search: z.string().optional().describe("Keyword to filter labels by"),
 });
 export const GetLabelSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    label_id: z.string().describe("The ID or title of a project's label"),
+    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"),
     include_ancestor_groups: flexibleBoolean.optional().describe("Include ancestor groups"),
 });
 export const CreateLabelSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     name: z.string().describe("The name of the label"),
     color: z
         .string()
@@ -1054,8 +1042,8 @@ export const CreateLabelSchema = z.object({
     priority: z.number().nullable().optional().describe("The priority of the label"),
 });
 export const UpdateLabelSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    label_id: z.string().describe("The ID or title of a project's label"),
+    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"),
     new_name: z.string().optional().describe("The new name of the label"),
     color: z
         .string()
@@ -1065,12 +1053,12 @@ export const UpdateLabelSchema = z.object({
     priority: z.number().nullable().optional().describe("The new priority of the label"),
 });
 export const DeleteLabelSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
-    label_id: z.string().describe("The ID or title of a project's label"),
+    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"),
 });
 // Group projects schema
 export const ListGroupProjectsSchema = z.object({
-    group_id: z.string().describe("Group ID or path"),
+    group_id: z.coerce.string().describe("Group ID or path"),
     include_subgroups: flexibleBoolean.optional().describe("Include projects from subgroups"),
     search: z.string().optional().describe("Search term to filter projects"),
     order_by: z
@@ -1100,28 +1088,28 @@ export const ListGroupProjectsSchema = z.object({
 }).merge(PaginationOptionsSchema);
 // Add wiki operation schemas
 export const ListWikiPagesSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     with_content: flexibleBoolean.optional().describe("Include content of the wiki pages"),
 }).merge(PaginationOptionsSchema);
 export const GetWikiPageSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     slug: z.string().describe("URL-encoded slug of the wiki page"),
 });
 export const CreateWikiPageSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     title: z.string().describe("Title of the wiki page"),
     content: z.string().describe("Content of the wiki page"),
     format: z.string().optional().describe("Content format, e.g., markdown, rdoc"),
 });
 export const UpdateWikiPageSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     slug: z.string().describe("URL-encoded slug of the wiki page"),
     title: z.string().optional().describe("New title of the wiki page"),
     content: z.string().optional().describe("New content of the wiki page"),
     format: z.string().optional().describe("Content format, e.g., markdown, rdoc"),
 });
 export const DeleteWikiPageSchema = z.object({
-    project_id: z.string().describe("Project ID or URL-encoded path"),
+    project_id: z.coerce.string().describe("Project ID or URL-encoded path"),
     slug: z.string().describe("URL-encoded slug of the wiki page"),
 });
 // Define wiki response schemas
@@ -1151,7 +1139,7 @@ export const MergeRequestThreadPositionSchema = z.object({
 });
 // Schema for creating a new merge request thread
 export const CreateMergeRequestThreadSchema = ProjectParamsSchema.extend({
-    merge_request_iid: z.string().or(z.number().describe("The IID of a merge request")),
+    merge_request_iid: z.coerce.string().describe("The IID of a merge request"),
     body: z.string().describe("The content of the thread"),
     position: MergeRequestThreadPositionSchema.optional().describe("Position when creating a diff note"),
     created_at: z.string().optional().describe("Date the thread was created at (ISO 8601 format)"),
@@ -1184,7 +1172,7 @@ export const ListProjectMilestonesSchema = ProjectParamsSchema.extend({
 }).merge(PaginationOptionsSchema);
 // Schema for getting a single milestone
 export const GetProjectMilestoneSchema = ProjectParamsSchema.extend({
-    milestone_id: z.string().or(z.number().describe("The ID of a project milestone")),
+    milestone_id: z.coerce.string().describe("The ID of a project milestone"),
 });
 // Schema for creating a new milestone
 export const CreateProjectMilestoneSchema = ProjectParamsSchema.extend({
@@ -1216,7 +1204,7 @@ export const PromoteProjectMilestoneSchema = GetProjectMilestoneSchema;
 export const GetMilestoneBurndownEventsSchema = GetProjectMilestoneSchema.merge(PaginationOptionsSchema);
 // Add schemas for commit operations
 export const ListCommitsSchema = z.object({
-    project_id: z.string().describe("Project ID or complete URL-encoded path to project"),
+    project_id: z.coerce.string().describe("Project ID or complete URL-encoded path to project"),
     ref_name: z.string().optional().describe("The name of a repository branch, tag or revision range, or if not given the default branch"),
     since: z.string().optional().describe("Only commits after or on this date are returned in ISO 8601 format YYYY-MM-DDTHH:MM:SSZ"),
     until: z.string().optional().describe("Only commits before or on this date are returned in ISO 8601 format YYYY-MM-DDTHH:MM:SSZ"),
@@ -1231,11 +1219,11 @@ export const ListCommitsSchema = z.object({
     per_page: z.number().optional().describe("Number of items per page (max: 100, default: 20)"),
 });
 export const GetCommitSchema = z.object({
-    project_id: z.string().describe("Project ID or complete URL-encoded path to project"),
+    project_id: z.coerce.string().describe("Project ID or complete URL-encoded path to project"),
     sha: z.string().describe("The commit hash or name of a repository branch or tag"),
     stats: flexibleBoolean.optional().describe("Include commit stats"),
 });
 export const GetCommitDiffSchema = z.object({
-    project_id: z.string().describe("Project ID or complete URL-encoded path to project"),
+    project_id: z.coerce.string().describe("Project ID or complete URL-encoded path to project"),
     sha: z.string().describe("The commit hash or name of a repository branch or tag"),
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zereight/mcp-gitlab",
-  "version": "1.0.72",
+  "version": "1.0.74",
   "description": "MCP server for using the GitLab API",
   "license": "MIT",
   "author": "zereight",

package/build/tests/integration.test.js

@@ -1,151 +0,0 @@
-import { describe, it, expect, beforeAll, afterAll } from '@jest/globals';
-import fetch from 'node-fetch';
-// Integration tests that run against real GitLab API (when credentials are provided)
-const GITLAB_API_URL = process.env.GITLAB_API_URL || 'https://gitlab.com';
-const GITLAB_TOKEN = process.env.GITLAB_TOKEN || '';
-const TEST_PROJECT_ID = process.env.TEST_PROJECT_ID || '';
-const skipIntegrationTests = !GITLAB_TOKEN || !TEST_PROJECT_ID;
-describe('GitLab MCP Server Integration Tests', () => {
-    if (skipIntegrationTests) {
-        it('should skip integration tests when credentials are missing', () => {
-            console.log('Skipping integration tests: Missing GITLAB_TOKEN or TEST_PROJECT_ID');
-            expect(true).toBe(true);
-        });
-        return;
-    }
-    let testIssueId = null;
-    let testMRId = null;
-    beforeAll(() => {
-        console.log('Running integration tests with GitLab API');
-    });
-    afterAll(async () => {
-        // Cleanup: Delete test issue and MR if created
-        if (testIssueId) {
-            await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/issues/${testIssueId}`, {
-                method: 'DELETE',
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`
-                }
-            });
-        }
-    });
-    describe('Project Access', () => {
-        it('should fetch project information', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const project = await response.json();
-            expect(project).toHaveProperty('id');
-            expect(project).toHaveProperty('name');
-        });
-    });
-    describe('Issue Operations', () => {
-        it('should create an issue', async () => {
-            const issueData = {
-                title: `Test Issue ${Date.now()}`,
-                description: 'This is a test issue created by MCP integration tests'
-            };
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/issues`, {
-                method: 'POST',
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json',
-                    'Content-Type': 'application/json'
-                },
-                body: JSON.stringify(issueData)
-            });
-            expect(response.ok).toBe(true);
-            const issue = await response.json();
-            expect(issue).toHaveProperty('iid');
-            expect(issue.title).toBe(issueData.title);
-            testIssueId = issue.iid;
-        });
-        it('should list issues', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/issues?state=opened`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const issues = await response.json();
-            expect(Array.isArray(issues)).toBe(true);
-        });
-        it('should add a comment to an issue', async () => {
-            if (!testIssueId) {
-                console.log('Skipping comment test: No test issue created');
-                return;
-            }
-            const commentData = {
-                body: 'Test comment from MCP integration tests'
-            };
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/issues/${testIssueId}/notes`, {
-                method: 'POST',
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json',
-                    'Content-Type': 'application/json'
-                },
-                body: JSON.stringify(commentData)
-            });
-            expect(response.ok).toBe(true);
-            const comment = await response.json();
-            expect(comment.body).toBe(commentData.body);
-        });
-    });
-    describe('Merge Request Operations', () => {
-        it('should list merge requests', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/merge_requests?state=opened`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const mrs = await response.json();
-            expect(Array.isArray(mrs)).toBe(true);
-        });
-    });
-    describe('Repository Operations', () => {
-        it('should fetch repository branches', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/repository/branches`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const branches = await response.json();
-            expect(Array.isArray(branches)).toBe(true);
-            expect(branches.length).toBeGreaterThan(0);
-        });
-        it('should fetch repository commits', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/repository/commits?per_page=5`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const commits = await response.json();
-            expect(Array.isArray(commits)).toBe(true);
-        });
-    });
-    describe('Pipeline Operations', () => {
-        it('should list pipelines', async () => {
-            const response = await fetch(`${GITLAB_API_URL}/api/v4/projects/${encodeURIComponent(TEST_PROJECT_ID)}/pipelines?per_page=5`, {
-                headers: {
-                    'Authorization': `Bearer ${GITLAB_TOKEN}`,
-                    'Accept': 'application/json'
-                }
-            });
-            expect(response.ok).toBe(true);
-            const pipelines = await response.json();
-            expect(Array.isArray(pipelines)).toBe(true);
-        });
-    });
-});

package/build/tests/unit.test.js

@@ -1,122 +0,0 @@
-import { describe, it, expect, beforeEach, afterEach } from '@jest/globals';
-import fetch from 'node-fetch';
-// Mock fetch for unit tests
-jest.mock('node-fetch');
-const mockedFetch = fetch;
-describe('GitLab MCP Server Unit Tests', () => {
-    beforeEach(() => {
-        jest.clearAllMocks();
-    });
-    afterEach(() => {
-        jest.restoreAllMocks();
-    });
-    describe('API URL Construction', () => {
-        it('should use plural resource names in API endpoints', () => {
-            const projectId = 'test/project';
-            const issueIid = 123;
-            // Test issue endpoint
-            const issueUrl = `/api/v4/projects/${encodeURIComponent(projectId)}/issues/${issueIid}`;
-            expect(issueUrl).toContain('/issues/');
-            expect(issueUrl).not.toContain('/issue/');
-            // Test merge request endpoint
-            const mrUrl = `/api/v4/projects/${encodeURIComponent(projectId)}/merge_requests/${issueIid}`;
-            expect(mrUrl).toContain('/merge_requests/');
-            expect(mrUrl).not.toContain('/merge_request/');
-        });
-        it('should properly encode project IDs with special characters', () => {
-            const projectId = 'namespace/project-name';
-            const encoded = encodeURIComponent(projectId);
-            expect(encoded).toBe('namespace%2Fproject-name');
-        });
-    });
-    describe('API Response Handling', () => {
-        it('should handle successful responses', async () => {
-            const mockResponse = {
-                ok: true,
-                status: 200,
-                json: async () => ({ id: 1, title: 'Test Issue' })
-            };
-            mockedFetch.mockResolvedValueOnce(mockResponse);
-            const response = await fetch('https://gitlab.com/api/v4/projects/1/issues/1');
-            const data = await response.json();
-            expect(response.ok).toBe(true);
-            expect(data).toEqual({ id: 1, title: 'Test Issue' });
-        });
-        it('should handle error responses', async () => {
-            const mockResponse = {
-                ok: false,
-                status: 404,
-                statusText: 'Not Found',
-                text: async () => '{"message":"404 Project Not Found"}'
-            };
-            mockedFetch.mockResolvedValueOnce(mockResponse);
-            const response = await fetch('https://gitlab.com/api/v4/projects/999/issues/1');
-            expect(response.ok).toBe(false);
-            expect(response.status).toBe(404);
-        });
-        it('should handle rate limiting', async () => {
-            const mockResponse = {
-                ok: false,
-                status: 429,
-                statusText: 'Too Many Requests',
-                headers: {
-                    get: (name) => name === 'RateLimit-Reset' ? '1234567890' : null
-                }
-            };
-            mockedFetch.mockResolvedValueOnce(mockResponse);
-            const response = await fetch('https://gitlab.com/api/v4/projects/1/issues');
-            expect(response.status).toBe(429);
-        });
-    });
-    describe('Authentication', () => {
-        it('should include Bearer token in Authorization header', () => {
-            const token = 'test-token-123';
-            const headers = {
-                'Authorization': `Bearer ${token}`,
-                'Accept': 'application/json',
-                'Content-Type': 'application/json'
-            };
-            expect(headers.Authorization).toBe('Bearer test-token-123');
-        });
-        it('should handle missing token gracefully', () => {
-            const token = process.env.GITLAB_TOKEN || '';
-            expect(token).toBeDefined();
-        });
-    });
-    describe('Data Validation', () => {
-        it('should validate required fields for issue creation', () => {
-            const validIssue = {
-                title: 'Test Issue',
-                description: 'Test Description'
-            };
-            expect(validIssue.title).toBeTruthy();
-            expect(validIssue.title.length).toBeGreaterThan(0);
-        });
-        it('should validate merge request parameters', () => {
-            const validMR = {
-                source_branch: 'feature-branch',
-                target_branch: 'main',
-                title: 'Test MR'
-            };
-            expect(validMR.source_branch).not.toBe(validMR.target_branch);
-            expect(validMR.title).toBeTruthy();
-        });
-    });
-    describe('Error Handling', () => {
-        it('should handle network errors', async () => {
-            mockedFetch.mockRejectedValueOnce(new Error('Network error'));
-            await expect(fetch('https://gitlab.com/api/v4/projects/1/issues'))
-                .rejects.toThrow('Network error');
-        });
-        it('should handle JSON parsing errors', async () => {
-            const mockResponse = {
-                ok: true,
-                status: 200,
-                json: async () => { throw new Error('Invalid JSON'); }
-            };
-            mockedFetch.mockResolvedValueOnce(mockResponse);
-            const response = await fetch('https://gitlab.com/api/v4/projects/1/issues');
-            await expect(response.json()).rejects.toThrow('Invalid JSON');
-        });
-    });
-});