opencode-gitlab-plugin 2.4.0 → 2.5.0

package/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [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

@@ -438,6 +438,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(
@@ -816,6 +824,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 +2166,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 +2302,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
@@ -4489,19 +4521,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 +4553,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 +4588,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 +4642,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 +4689,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 +4720,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}`);
+      }
+    }
   })
 };