efficient-gitlab-mcp-server 2.4.0 → 2.5.0

package/dist/index.js

@@ -54123,6 +54123,27 @@ function registerIssueTools(server, logger2) {
   return tools;
 }
 // src/tools/merge-requests.ts
+var MAX_PATTERN_LENGTH = 200;
+var NESTED_QUANTIFIER_RE = /(\+|\*|\{)\s*(\+|\*|\{)/;
+function safeCompilePatterns(patterns) {
+  return patterns.map((p) => {
+    if (p.length > MAX_PATTERN_LENGTH || NESTED_QUANTIFIER_RE.test(p))
+      return null;
+    try {
+      return new RegExp(p);
+    } catch {
+      return null;
+    }
+  }).filter((re) => re !== null);
+}
+function filterDiffsByPatterns(diffs, patterns) {
+  if (!patterns?.length)
+    return diffs;
+  const regexes = safeCompilePatterns(patterns);
+  if (regexes.length === 0)
+    return diffs;
+  return diffs.filter((d) => !regexes.some((re) => re.test(d.new_path) || (d.old_path ? re.test(d.old_path) : false)));
+}
 var GetMergeRequestSchema = exports_external.object({
   project_id: exports_external.string().describe("Project ID or URL-encoded path"),
   merge_request_iid: exports_external.number().optional().describe("Merge request IID"),
@@ -54177,7 +54198,8 @@ var GetMergeRequestDiffsSchema = exports_external.object({
   project_id: exports_external.string().describe("Project ID or URL-encoded path"),
   merge_request_iid: exports_external.number().describe("Merge request IID"),
   page: exports_external.number().optional().describe("Page number"),
-  per_page: exports_external.number().optional().describe("Results per page")
+  per_page: exports_external.number().optional().describe("Results per page"),
+  excluded_file_patterns: exports_external.array(exports_external.string()).optional().describe('Array of regex patterns to exclude files. Examples: ["^vendor/", "\\.pb\\.go$"]')
 });
 var ListMergeRequestDiscussionsSchema = exports_external.object({
   project_id: exports_external.string().describe("Project ID or URL-encoded path"),
@@ -54465,20 +54487,24 @@ function registerMergeRequestTools(server, logger2) {
   tools.set("merge_merge_request", toolRef5);
   const toolRef6 = server.registerTool("get_merge_request_diffs", {
     title: "Get Merge Request Diffs",
-    description: "Get the changes/diffs of a merge request",
+    description: "Get the changes/diffs of a merge request. Supports excluded_file_patterns filtering using regex.",
     inputSchema: {
       project_id: exports_external.string().describe("Project ID or URL-encoded path"),
       merge_request_iid: exports_external.number().describe("Merge request IID"),
       page: exports_external.number().optional().describe("Page number"),
-      per_page: exports_external.number().optional().describe("Results per page")
+      per_page: exports_external.number().optional().describe("Results per page"),
+      excluded_file_patterns: exports_external.array(exports_external.string()).optional().describe('Array of regex patterns to exclude files. Examples: ["^vendor/", "\\.pb\\.go$"]')
     },
     annotations: { readOnlyHint: true }
   }, async (params) => {
     const args = GetMergeRequestDiffsSchema.parse(params);
     const projectId = encodeProjectId(args.project_id);
     const query = buildQueryString({ page: args.page, per_page: args.per_page });
-    const diffs = await defaultClient.get(`/projects/${projectId}/merge_requests/${args.merge_request_iid}/changes${query}`);
-    return { content: [{ type: "text", text: JSON.stringify(diffs, null, 2) }] };
+    const mr = await defaultClient.get(`/projects/${projectId}/merge_requests/${args.merge_request_iid}/changes${query}`);
+    const changes = filterDiffsByPatterns(mr.changes ?? [], args.excluded_file_patterns);
+    return {
+      content: [{ type: "text", text: JSON.stringify({ ...mr, changes }, null, 2) }]
+    };
   });
   toolRef6.disable();
   tools.set("get_merge_request_diffs", toolRef6);
@@ -54708,24 +54734,7 @@ function registerMergeRequestTools(server, logger2) {
       renamed_file: c.renamed_file,
       deleted_file: c.deleted_file
     }));
-    if (args.excluded_file_patterns && args.excluded_file_patterns.length > 0) {
-      const patterns = [];
-      for (const p of args.excluded_file_patterns) {
-        try {
-          patterns.push(new RegExp(p));
-        } catch {
-          return {
-            content: [
-              {
-                type: "text",
-                text: JSON.stringify({ error: `Invalid regex pattern: ${p}` }, null, 2)
-              }
-            ]
-          };
-        }
-      }
-      files = files.filter((f) => !patterns.some((re) => re.test(f.new_path) || re.test(f.old_path)));
-    }
+    files = filterDiffsByPatterns(files, args.excluded_file_patterns);
     return { content: [{ type: "text", text: JSON.stringify(files, null, 2) }] };
   });
   toolRef18.disable();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "efficient-gitlab-mcp-server",
-  "version": "2.4.0",
+  "version": "2.5.0",
   "mcpName": "io.github.detailobsessed/efficient-gitlab",
   "description": "Production-ready GitLab MCP Server with progressive disclosure pattern",
   "license": "MIT",