opencode-gitlab-plugin 2.4.0 → 2.6.0

package/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [2.6.0](https://gitlab.com/vglafirov/opencode-gitlab-plugin/compare/v2.5.0...v2.6.0) (2026-05-20)
+
+
+### ✨ Features
+
+* **pagination:** expose pagination info in list MR and issue responses ([1c1722e](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/1c1722e62e0e41f00ab5409057bea1e76ec17a8d))
+
+
+### ♻️ Code Refactoring
+
+* **client:** address review feedback on !23 ([2e94e46](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/2e94e46706807bc8797f10884f71fc56ecf967b9)), closes [23#note_3135645102](https://gitlab.com/vglafirov/23/issues/note_3135645102) [23#note_3135645423](https://gitlab.com/vglafirov/23/issues/note_3135645423) [23#note_3135644923](https://gitlab.com/vglafirov/23/issues/note_3135644923)
+
+## [2.5.0](https://gitlab.com/vglafirov/opencode-gitlab-plugin/compare/v2.4.0...v2.5.0) (2026-05-20)
+
+
+### ✨ Features
+
+* **notes:** add gitlab_update_note tool for editing comments ([54a87e7](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/54a87e70d4d276c077ab25619e1d910d6d709575))
+
+
+### 🐛 Bug Fixes
+
+* handle ToolResult union type in mcp-adapter and tests ([a91d5dd](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/a91d5ddc5574d1cf9b7dab2d9624cdddeefd158f))
+* **tests:** update orbit.test.ts for ToolResult type ([29fecec](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/29fecec6a8ba436b3d95b720cf67f162c8c62104))
+
+
+### ♻️ Code Refactoring
+
+* **notes:** address review feedback on !28 ([11e13ba](https://gitlab.com/vglafirov/opencode-gitlab-plugin/commit/11e13ba5a6dafeb77dfb256bde3e9b7b1f5ddd32))
+
 ## [2.4.0](https://gitlab.com/vglafirov/opencode-gitlab-plugin/compare/v2.3.0...v2.4.0) (2026-05-19)
 
 

package/dist/index.js

@@ -26,7 +26,13 @@ var GitLabApiClient = class {
     }
     return projectId;
   }
-  async fetch(method, path2, body) {
+  /**
+   * Core HTTP request used by all REST helpers.
+   *
+   * Returns the raw `Response` after validating the status code so callers can
+   * inspect headers (e.g. for pagination) before parsing the body.
+   */
+  async rawFetch(method, path2, body) {
     const url = `${this.instanceUrl}/api/v4${path2}`;
     const response = await fetch(url, {
       method,
@@ -37,12 +43,53 @@ var GitLabApiClient = class {
       const errorText = await response.text();
       throw new Error(`GitLab API error ${response.status}: ${errorText}`);
     }
+    return response;
+  }
+  async fetch(method, path2, body) {
+    const response = await this.rawFetch(method, path2, body);
     const text = await response.text();
     if (!text) {
       return {};
     }
     return JSON.parse(text);
   }
+  /**
+   * Parse a numeric pagination header, guarding against empty strings.
+   *
+   * GitLab returns `""` (rather than omitting the header) for `X-Next-Page` and
+   * `X-Prev-Page` when there is no next/prev page, and bare `parseInt` without
+   * a radix on an empty string yields `NaN`. Treat empty/missing as `undefined`.
+   */
+  parsePaginationHeader(response, name) {
+    const raw = response.headers.get(name);
+    if (raw == null || raw === "") return void 0;
+    const value = parseInt(raw, 10);
+    return Number.isNaN(value) ? void 0 : value;
+  }
+  /**
+   * Pagination info extracted from GitLab API response headers
+   */
+  extractPaginationInfo(response) {
+    return {
+      total: this.parsePaginationHeader(response, "X-Total"),
+      totalPages: this.parsePaginationHeader(response, "X-Total-Pages"),
+      page: this.parsePaginationHeader(response, "X-Page"),
+      perPage: this.parsePaginationHeader(response, "X-Per-Page"),
+      nextPage: this.parsePaginationHeader(response, "X-Next-Page"),
+      prevPage: this.parsePaginationHeader(response, "X-Prev-Page")
+    };
+  }
+  /**
+   * Fetch with pagination info from response headers.
+   * Use this for list endpoints where knowing if more data exists is important.
+   */
+  async fetchWithPagination(method, path2, body) {
+    const response = await this.rawFetch(method, path2, body);
+    const text = await response.text();
+    const data = text ? JSON.parse(text) : [];
+    const pagination = this.extractPaginationInfo(response);
+    return { data, pagination };
+  }
   async fetchText(method, path2) {
     const url = `${this.instanceUrl}/api/v4${path2}`;
     const response = await fetch(url, {
@@ -306,6 +353,7 @@ var MergeRequestsClient = class extends GitLabApiClient {
   async listMergeRequests(options) {
     const params = new URLSearchParams();
     params.set("per_page", String(options.limit || 20));
+    if (options.page) params.set("page", String(options.page));
     if (options.state) params.set("state", options.state);
     if (options.scope) params.set("scope", options.scope);
     if (options.search) params.set("search", options.search);
@@ -317,7 +365,7 @@ var MergeRequestsClient = class extends GitLabApiClient {
     } else {
       path2 = `/merge_requests?${params}`;
     }
-    return this.fetch("GET", path2);
+    return this.fetchWithPagination("GET", path2);
   }
   async getMrChanges(projectId, mrIid) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -438,6 +486,14 @@ var MergeRequestsClient = class extends GitLabApiClient {
       { body }
     );
   }
+  async updateMrNote(projectId, mrIid, noteId, body) {
+    const encodedProject = this.encodeProjectId(projectId);
+    return this.fetch(
+      "PUT",
+      `/projects/${encodedProject}/merge_requests/${mrIid}/notes/${noteId}`,
+      { body }
+    );
+  }
   async createMergeRequest(projectId, options) {
     const encodedProject = this.encodeProjectId(projectId);
     return this.fetch(
@@ -721,6 +777,7 @@ var IssuesClient = class extends GitLabApiClient {
   async listIssues(options) {
     const params = new URLSearchParams();
     params.set("per_page", String(options.limit || 20));
+    if (options.page) params.set("page", String(options.page));
     if (options.state) params.set("state", options.state);
     if (options.scope) params.set("scope", options.scope);
     if (options.search) params.set("search", options.search);
@@ -733,7 +790,7 @@ var IssuesClient = class extends GitLabApiClient {
     } else {
       path2 = `/issues?${params}`;
     }
-    return this.fetch("GET", path2);
+    return this.fetchWithPagination("GET", path2);
   }
   /**
    * List notes on an issue using GraphQL API with pagination support
@@ -816,6 +873,14 @@ var IssuesClient = class extends GitLabApiClient {
       `/projects/${encodedProject}/issues/${issueIid}/notes/${noteId}`
     );
   }
+  async updateIssueNote(projectId, issueIid, noteId, body) {
+    const encodedProject = this.encodeProjectId(projectId);
+    return this.fetch(
+      "PUT",
+      `/projects/${encodedProject}/issues/${issueIid}/notes/${noteId}`,
+      { body }
+    );
+  }
 };
 
 // src/client/work-items.ts
@@ -2150,6 +2215,14 @@ var EpicsClient = class extends GitLabApiClient {
       `/groups/${encodedGroup}/epics/${epicIid}/notes/${noteId}`
     );
   }
+  async updateEpicNote(groupId, epicIid, noteId, body) {
+    const encodedGroup = encodeURIComponent(groupId);
+    return this.fetch(
+      "PUT",
+      `/groups/${encodedGroup}/epics/${epicIid}/notes/${noteId}`,
+      { body }
+    );
+  }
 };
 
 // src/client/snippets.ts
@@ -2278,6 +2351,14 @@ var SnippetsClient = class extends GitLabApiClient {
       { body }
     );
   }
+  async updateSnippetNote(projectId, snippetId, noteId, body) {
+    const encodedProject = this.encodeProjectId(projectId);
+    return this.fetch(
+      "PUT",
+      `/projects/${encodedProject}/snippets/${snippetId}/notes/${noteId}`,
+      { body }
+    );
+  }
 };
 
 // src/client/discussions.ts
@@ -2615,26 +2696,31 @@ Returns: title, description, state, author, assignees, reviewers, labels, diff s
   }),
   gitlab_list_merge_requests: tool({
     description: `List merge requests for a project or search globally.
-Can filter by state (opened, closed, merged, all), scope (assigned_to_me, created_by_me), and labels.`,
+Can filter by state (opened, closed, merged, all), scope (assigned_to_me, created_by_me), and labels.
+
+IMPORTANT: Response includes pagination info. If nextPage exists, there are more results.
+Always paginate until you get an empty response or no nextPage to ensure you have all data.`,
     args: {
       project_id: z.string().optional().describe("The project ID or path. If not provided, searches globally."),
       state: z.enum(["opened", "closed", "merged", "all"]).optional().describe("Filter by MR state (default: opened)"),
       scope: z.enum(["assigned_to_me", "created_by_me", "all"]).optional().describe("Filter by scope"),
       search: z.string().optional().describe("Search MRs by title or description"),
       labels: z.string().optional().describe("Comma-separated list of labels to filter by"),
-      limit: z.number().optional().describe("Maximum number of results (default: 20)")
+      limit: z.number().optional().describe("Maximum number of results per page (default: 20)"),
+      page: z.number().optional().describe("Page number for pagination (default: 1)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const mrs = await client.listMergeRequests({
+      const result = await client.listMergeRequests({
         projectId: args.project_id,
         state: args.state,
         scope: args.scope,
         search: args.search,
         labels: args.labels,
-        limit: args.limit
+        limit: args.limit,
+        page: args.page
       });
-      return JSON.stringify(mrs, null, 2);
+      return JSON.stringify(result, null, 2);
     }
   }),
   gitlab_get_mr_changes: tool({
@@ -2966,7 +3052,10 @@ Returns: title, description, state, author, assignees, labels, milestone, weight
   }),
   gitlab_list_issues: tool2({
     description: `List issues for a project or search globally.
-Can filter by state, labels, assignee, milestone.`,
+Can filter by state, labels, assignee, milestone.
+
+IMPORTANT: Response includes pagination info. If nextPage exists, there are more results.
+Always paginate until you get an empty response or no nextPage to ensure you have all data.`,
     args: {
       project_id: z2.string().optional().describe("The project ID or path. If not provided, searches globally."),
       state: z2.enum(["opened", "closed", "all"]).optional().describe("Filter by issue state (default: opened)"),
@@ -2974,20 +3063,22 @@ Can filter by state, labels, assignee, milestone.`,
       search: z2.string().optional().describe("Search issues by title or description"),
       labels: z2.string().optional().describe("Comma-separated list of labels to filter by"),
       milestone: z2.string().optional().describe("Filter by milestone title"),
-      limit: z2.number().optional().describe("Maximum number of results (default: 20)")
+      limit: z2.number().optional().describe("Maximum number of results per page (default: 20)"),
+      page: z2.number().optional().describe("Page number for pagination (default: 1)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const issues = await client.listIssues({
+      const result = await client.listIssues({
         projectId: args.project_id,
         state: args.state,
         scope: args.scope,
         search: args.search,
         labels: args.labels,
         milestone: args.milestone,
-        limit: args.limit
+        limit: args.limit,
+        page: args.page
       });
-      return JSON.stringify(issues, null, 2);
+      return JSON.stringify(result, null, 2);
     }
   })
 };
@@ -4489,19 +4580,22 @@ function filterNotesByResolved(result, resolved) {
     }
   };
 }
-var VALID_LIST_CREATE_TYPES = ["merge_request", "issue", "epic", "snippet"];
-var VALID_GET_NOTE_TYPES = ["issue", "epic"];
+var NOTE_RESOURCE_TYPES = ["merge_request", "issue", "epic", "snippet"];
+var GET_NOTE_RESOURCE_TYPES = ["issue", "epic"];
 function validationError3(param, resourceType) {
   return new Error(
     `Missing required parameter: '${param}' is required for resource_type '${resourceType}'`
   );
 }
-function validateListCreateParams(resourceType, args) {
-  if (!VALID_LIST_CREATE_TYPES.includes(resourceType)) {
+function validateNoteParams(resourceType, args, opts) {
+  if (!opts.allowedTypes.includes(resourceType)) {
     throw new Error(
-      `Invalid resource_type '${resourceType}'. Must be one of: ${VALID_LIST_CREATE_TYPES.join(", ")}`
+      `Invalid resource_type '${resourceType}'. Must be one of: ${opts.allowedTypes.join(", ")}`
     );
   }
+  if (opts.requireNoteId && args.note_id == null) {
+    throw validationError3("note_id", resourceType);
+  }
   switch (resourceType) {
     case "merge_request":
     case "issue":
@@ -4518,24 +4612,6 @@ function validateListCreateParams(resourceType, args) {
       break;
   }
 }
-function validateGetNoteParams(resourceType, args) {
-  if (!VALID_GET_NOTE_TYPES.includes(resourceType)) {
-    throw new Error(
-      `Invalid resource_type '${resourceType}'. Must be one of: ${VALID_GET_NOTE_TYPES.join(", ")}`
-    );
-  }
-  if (args.note_id == null) throw validationError3("note_id", resourceType);
-  switch (resourceType) {
-    case "issue":
-      if (!args.project_id) throw validationError3("project_id", resourceType);
-      if (args.iid == null) throw validationError3("iid", resourceType);
-      break;
-    case "epic":
-      if (!args.group_id) throw validationError3("group_id", resourceType);
-      if (args.iid == null) throw validationError3("iid", resourceType);
-      break;
-  }
-}
 var notesUnifiedTools = {
   /**
    * List notes/comments for any GitLab resource type
@@ -4571,7 +4647,7 @@ Examples:
       before: z14.string().optional().describe("Cursor for backward pagination - use startCursor from previous response")
     },
     execute: async (args, _ctx) => {
-      validateListCreateParams(args.resource_type, args);
+      validateNoteParams(args.resource_type, args, { allowedTypes: NOTE_RESOURCE_TYPES });
       const client = getGitLabClient();
       const paginationOptions = {
         first: args.first,
@@ -4625,7 +4701,10 @@ Examples:
       iid: z14.number().describe("Internal ID of the issue or epic")
     },
     execute: async (args, _ctx) => {
-      validateGetNoteParams(args.resource_type, args);
+      validateNoteParams(args.resource_type, args, {
+        allowedTypes: GET_NOTE_RESOURCE_TYPES,
+        requireNoteId: true
+      });
       const client = getGitLabClient();
       switch (args.resource_type) {
         case "issue":
@@ -4669,7 +4748,7 @@ Examples:
       snippet_id: z14.number().optional().describe("Snippet ID (required for snippet)")
     },
     execute: async (args, _ctx) => {
-      validateListCreateParams(args.resource_type, args);
+      validateNoteParams(args.resource_type, args, { allowedTypes: NOTE_RESOURCE_TYPES });
       const client = getGitLabClient();
       switch (args.resource_type) {
         case "merge_request":
@@ -4700,6 +4779,68 @@ Examples:
           throw new Error(`Unsupported resource type: ${args.resource_type}`);
       }
     }
+  }),
+  /**
+   * Update an existing note/comment on any GitLab resource
+   */
+  gitlab_update_note: tool14({
+    description: `Update an existing note/comment on any GitLab resource.
+Modifies the body content of an existing note. You can only update notes you authored.
+
+Examples:
+- MR note: resource_type="merge_request", project_id="group/project", iid=123, note_id=456, body="Updated comment"
+- Issue note: resource_type="issue", project_id="group/project", iid=456, note_id=789, body="Updated comment"
+- Epic note: resource_type="epic", group_id="my-group", iid=1, note_id=456, body="Updated comment"
+- Snippet note: resource_type="snippet", project_id="group/project", snippet_id=789, note_id=123, body="Updated comment"`,
+    args: {
+      resource_type: z14.enum(["merge_request", "issue", "epic", "snippet"]).describe("Type of GitLab resource"),
+      note_id: z14.number().describe("The ID of the note to update"),
+      body: z14.string().describe("The new content for the note (supports Markdown)"),
+      project_id: z14.string().optional().describe("Project ID or path. Required for merge_request, issue, snippet"),
+      group_id: z14.string().optional().describe("Group ID or path. Required for epic"),
+      iid: z14.number().optional().describe("Internal ID of the resource (for merge_request, issue, epic)"),
+      snippet_id: z14.number().optional().describe("Snippet ID (required for snippet)")
+    },
+    execute: async (args, _ctx) => {
+      validateNoteParams(args.resource_type, args, {
+        allowedTypes: NOTE_RESOURCE_TYPES,
+        requireNoteId: true
+      });
+      const client = getGitLabClient();
+      switch (args.resource_type) {
+        case "merge_request":
+          return JSON.stringify(
+            await client.updateMrNote(args.project_id, args.iid, args.note_id, args.body),
+            null,
+            2
+          );
+        case "issue":
+          return JSON.stringify(
+            await client.updateIssueNote(args.project_id, args.iid, args.note_id, args.body),
+            null,
+            2
+          );
+        case "epic":
+          return JSON.stringify(
+            await client.updateEpicNote(args.group_id, args.iid, args.note_id, args.body),
+            null,
+            2
+          );
+        case "snippet":
+          return JSON.stringify(
+            await client.updateSnippetNote(
+              args.project_id,
+              args.snippet_id,
+              args.note_id,
+              args.body
+            ),
+            null,
+            2
+          );
+        default:
+          throw new Error(`Unsupported resource type: ${args.resource_type}`);
+      }
+    }
   })
 };