@gitlab/opencode-gitlab-plugin 1.6.1 → 1.7.0

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.7.0](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.6.2...v1.7.0) (2026-02-24)
+
+
+### ✨ Features
+
+* **award-emoji:** add tools for managing reactions on resources ([77b8ad5](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/77b8ad5656b681c3a6c9395f2a0025126e4c39bb))
+* **discussions:** use GraphQL for discussion resolution ([b6ed342](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/b6ed342cea8f726b14a2850350d217a937406caa))
+
+
+### 🐛 Bug Fixes
+
+* **todos:** handle 404 gracefully when marking todo as done ([2a26763](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/2a267632012d89884ebc3462ac0adafc5ffd9286))
+
+## [1.6.2](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.6.1...v1.6.2) (2026-02-04)
+
+
+### 🐛 Bug Fixes
+
+* resolved filter now correctly filters resolvable discussions only ([59defec](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/59defec6c1c1d1536ea659a536b2bde3e89c95c5))
+
 ## [1.6.1](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.6.0...v1.6.1) (2026-02-04)
 
 

package/dist/index.js

@@ -277,6 +277,23 @@ var SET_AUTO_MERGE_MUTATION = `
     }
   }
 `;
+var RESOLVE_DISCUSSION_MUTATION = `
+  mutation resolveDiscussion($discussionId: DiscussionID!, $resolve: Boolean!) {
+    discussionToggleResolve(input: { id: $discussionId, resolve: $resolve }) {
+      discussion {
+        id
+        resolved
+        resolvedAt
+        resolvedBy {
+          id
+          username
+          name
+        }
+      }
+      errors
+    }
+  }
+`;
 var MergeRequestsClient = class extends GitLabApiClient {
   async getMergeRequest(projectId, mrIid, includeChanges) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -368,6 +385,32 @@ var MergeRequestsClient = class extends GitLabApiClient {
       { resolved: false }
     );
   }
+  /**
+   * Resolve or unresolve a discussion using GraphQL API.
+   * This works for all discussions including outdated ones (resolved: null)
+   * which the REST API cannot handle.
+   *
+   * @param discussionId - The discussion ID (can be short ID or full gid://gitlab/Discussion/ID format)
+   * @param resolve - Whether to resolve (true) or unresolve (false) the discussion
+   */
+  async toggleDiscussionResolved(discussionId, resolve2) {
+    const gid = discussionId.startsWith("gid://") ? discussionId : `gid://gitlab/Discussion/${discussionId}`;
+    const result = await this.fetchGraphQL(RESOLVE_DISCUSSION_MUTATION, {
+      discussionId: gid,
+      resolve: resolve2
+    });
+    if (result.discussionToggleResolve.errors.length > 0) {
+      throw new Error(
+        `Failed to ${resolve2 ? "resolve" : "unresolve"} discussion: ${result.discussionToggleResolve.errors.join(", ")}`
+      );
+    }
+    if (!result.discussionToggleResolve.discussion) {
+      throw new Error(
+        `Failed to ${resolve2 ? "resolve" : "unresolve"} discussion: No discussion returned`
+      );
+    }
+    return result.discussionToggleResolve.discussion;
+  }
   async createMrDiscussion(projectId, mrIid, body, position) {
     const encodedProject = this.encodeProjectId(projectId);
     const requestBody = { body };
@@ -425,6 +468,32 @@ var MergeRequestsClient = class extends GitLabApiClient {
       `/projects/${encodedProject}/merge_requests/${mrIid}/pipelines`
     );
   }
+  /**
+   * Add a reviewer to a merge request without affecting existing reviewers
+   */
+  async addReviewer(projectId, mrIid, reviewerId) {
+    const mr = await this.getMergeRequest(projectId, mrIid);
+    const currentReviewers = mr.reviewers || [];
+    const currentReviewerIds = currentReviewers.map((r) => r.id);
+    if (currentReviewerIds.includes(reviewerId)) {
+      return mr;
+    }
+    const newReviewerIds = [...currentReviewerIds, reviewerId];
+    return this.updateMergeRequest(projectId, mrIid, { reviewer_ids: newReviewerIds });
+  }
+  /**
+   * Remove a reviewer from a merge request without affecting other reviewers
+   */
+  async removeReviewer(projectId, mrIid, reviewerId) {
+    const mr = await this.getMergeRequest(projectId, mrIid);
+    const currentReviewers = mr.reviewers || [];
+    const currentReviewerIds = currentReviewers.map((r) => r.id);
+    if (!currentReviewerIds.includes(reviewerId)) {
+      return mr;
+    }
+    const newReviewerIds = currentReviewerIds.filter((id) => id !== reviewerId);
+    return this.updateMergeRequest(projectId, mrIid, { reviewer_ids: newReviewerIds });
+  }
   async listMergeRequestDiffs(projectId, mrIid, options = {}) {
     const encodedProject = this.encodeProjectId(projectId);
     const params = new URLSearchParams();
@@ -1468,7 +1537,21 @@ var TodosClient = class extends GitLabApiClient {
     return typeMap[type] || type;
   }
   async markTodoAsDone(todoId) {
-    return this.fetch("POST", `/todos/${todoId}/mark_as_done`);
+    try {
+      const result = await this.fetch(
+        "POST",
+        `/todos/${todoId}/mark_as_done`
+      );
+      return { success: true, todo: result };
+    } catch (error) {
+      if (error instanceof Error && error.message.includes("404")) {
+        return {
+          success: false,
+          message: `Todo ${todoId} not found. It may have already been marked as done or does not exist.`
+        };
+      }
+      throw error;
+    }
   }
   async markAllTodosAsDone() {
     return this.fetch("POST", "/todos/mark_as_done");
@@ -1963,6 +2046,63 @@ var AuditClient = class extends GitLabApiClient {
   }
 };
 
+// src/client/award-emoji.ts
+var AwardEmojiClient = class extends GitLabApiClient {
+  /**
+   * Build the base path for award emoji operations
+   */
+  buildAwardEmojiPath(resource) {
+    const encodedProject = this.encodeProjectId(resource.projectId);
+    let basePath;
+    switch (resource.resourceType) {
+      case "merge_request":
+        basePath = `/projects/${encodedProject}/merge_requests/${resource.resourceIid}`;
+        break;
+      case "issue":
+        basePath = `/projects/${encodedProject}/issues/${resource.resourceIid}`;
+        break;
+      case "snippet":
+        basePath = `/projects/${encodedProject}/snippets/${resource.resourceIid}`;
+        break;
+    }
+    if (resource.noteId) {
+      return `${basePath}/notes/${resource.noteId}/award_emoji`;
+    }
+    return `${basePath}/award_emoji`;
+  }
+  /**
+   * List all award emoji on a resource or note
+   */
+  async listAwardEmoji(resource) {
+    const path2 = this.buildAwardEmojiPath(resource);
+    return this.fetch("GET", path2);
+  }
+  /**
+   * Get a single award emoji by ID
+   */
+  async getAwardEmoji(resource, awardId) {
+    const path2 = `${this.buildAwardEmojiPath(resource)}/${awardId}`;
+    return this.fetch("GET", path2);
+  }
+  /**
+   * Add an award emoji (reaction) to a resource or note
+   *
+   * @param resource - The resource to add the emoji to
+   * @param name - The emoji name without colons (e.g., 'thumbsup', 'rocket', 'eyes')
+   */
+  async createAwardEmoji(resource, name) {
+    const path2 = this.buildAwardEmojiPath(resource);
+    return this.fetch("POST", path2, { name });
+  }
+  /**
+   * Remove an award emoji from a resource or note
+   */
+  async deleteAwardEmoji(resource, awardId) {
+    const path2 = `${this.buildAwardEmojiPath(resource)}/${awardId}`;
+    await this.fetch("DELETE", path2);
+  }
+};
+
 // src/client/index.ts
 var UnifiedGitLabClient = class extends GitLabApiClient {
 };
@@ -1992,7 +2132,8 @@ applyMixins(UnifiedGitLabClient, [
   EpicsClient,
   SnippetsClient,
   DiscussionsClient,
-  AuditClient
+  AuditClient,
+  AwardEmojiClient
 ]);
 
 // src/utils.ts
@@ -2203,6 +2344,36 @@ This is useful when you need to process diffs in chunks or when the MR has many
       return JSON.stringify(diffs, null, 2);
     }
   }),
+  gitlab_add_mr_reviewer: tool({
+    description: `Add a reviewer to a merge request without affecting existing reviewers.
+This is safer than gitlab_update_merge_request when you only want to add a single reviewer,
+as it preserves all existing reviewers.`,
+    args: {
+      project_id: z.string().describe("The project ID or URL-encoded path"),
+      mr_iid: z.number().describe("The internal ID of the merge request"),
+      reviewer_id: z.number().describe("The user ID of the reviewer to add")
+    },
+    execute: async (args, _ctx) => {
+      const client = getGitLabClient();
+      const mr = await client.addReviewer(args.project_id, args.mr_iid, args.reviewer_id);
+      return JSON.stringify(mr, null, 2);
+    }
+  }),
+  gitlab_remove_mr_reviewer: tool({
+    description: `Remove a reviewer from a merge request without affecting other reviewers.
+This is safer than gitlab_update_merge_request when you only want to remove a single reviewer,
+as it preserves all other reviewers.`,
+    args: {
+      project_id: z.string().describe("The project ID or URL-encoded path"),
+      mr_iid: z.number().describe("The internal ID of the merge request"),
+      reviewer_id: z.number().describe("The user ID of the reviewer to remove")
+    },
+    execute: async (args, _ctx) => {
+      const client = getGitLabClient();
+      const mr = await client.removeReviewer(args.project_id, args.mr_iid, args.reviewer_id);
+      return JSON.stringify(mr, null, 2);
+    }
+  }),
   gitlab_set_mr_auto_merge: tool({
     description: `Enable auto-merge (MWPS - Merge When Pipeline Succeeds) on a merge request.
 Uses the GitLab GraphQL API mergeRequestAccept mutation with a merge strategy.
@@ -3227,7 +3398,24 @@ Examples:
             return JSON.stringify({ error: 'todo_id is required when action is "one"' }, null, 2);
           }
           const result = await client.markTodoAsDone(args.todo_id);
-          return JSON.stringify(result, null, 2);
+          if (!result.success) {
+            return JSON.stringify(
+              {
+                success: false,
+                message: result.message
+              },
+              null,
+              2
+            );
+          }
+          return JSON.stringify(
+            {
+              success: true,
+              todo: result.todo
+            },
+            null,
+            2
+          );
         }
         case "all": {
           const result = await client.markAllTodosAsDone();
@@ -3388,14 +3576,23 @@ Can update title, description, state, labels, and assignees.`,
 // src/tools/discussions-unified.ts
 import { tool as tool12 } from "@opencode-ai/plugin";
 var z12 = tool12.schema;
+function normalizeBoolean(value) {
+  if (typeof value === "boolean") return value;
+  if (value === "true") return true;
+  if (value === "false") return false;
+  return void 0;
+}
 function filterDiscussionsByResolved(result, resolved) {
-  if (typeof resolved !== "boolean") {
+  const normalizedResolved = normalizeBoolean(resolved);
+  if (normalizedResolved === void 0) {
     return result;
   }
   return {
     discussions: {
       ...result.discussions,
-      nodes: result.discussions.nodes.filter((d) => d.resolved === resolved)
+      nodes: result.discussions.nodes.filter(
+        (d) => d.resolvable && d.resolved === normalizedResolved
+      )
     }
   };
 }
@@ -3458,7 +3655,7 @@ Examples:
       snippet_id: z12.number().optional().describe("Snippet ID (required for snippet)"),
       // Filtering
       resolved: z12.boolean().optional().describe(
-        "Filter by resolved status: true for resolved, false for unresolved (client-side filtering)"
+        "Filter by resolved status: true for resolved, false for unresolved. Only returns resolvable discussions (excludes system notes). Client-side filtering."
       ),
       // Pagination
       first: z12.number().optional().describe("Number of items to return (default: 20)"),
@@ -3669,6 +3866,9 @@ Examples:
     description: `Mark a discussion thread as resolved or unresolve it.
 Only works for resolvable discussions (MRs and issues only).
 
+Uses GraphQL API which handles all discussion types including outdated discussions
+(those with resolved: null) that the REST API cannot handle.
+
 Use after addressing feedback to indicate the discussion is complete.`,
     args: {
       resource_type: z12.enum(["merge_request", "issue"]).describe("Type of resource (only MR and issue discussions can be resolved)"),
@@ -3679,37 +3879,12 @@ Use after addressing feedback to indicate the discussion is complete.`,
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      if (args.action === "resolve") {
-        switch (args.resource_type) {
-          case "merge_request":
-            return JSON.stringify(
-              await client.resolveMrDiscussion(args.project_id, args.iid, args.discussion_id),
-              null,
-              2
-            );
-          case "issue":
-            return JSON.stringify(
-              await client.resolveIssueDiscussion(args.project_id, args.iid, args.discussion_id),
-              null,
-              2
-            );
-        }
-      } else {
-        switch (args.resource_type) {
-          case "merge_request":
-            return JSON.stringify(
-              await client.unresolveMrDiscussion(args.project_id, args.iid, args.discussion_id),
-              null,
-              2
-            );
-          case "issue":
-            return JSON.stringify(
-              await client.unresolveIssueDiscussion(args.project_id, args.iid, args.discussion_id),
-              null,
-              2
-            );
-        }
-      }
+      const resolve2 = args.action === "resolve";
+      return JSON.stringify(
+        await client.toggleDiscussionResolved(args.discussion_id, resolve2),
+        null,
+        2
+      );
     }
   })
 };
@@ -3717,15 +3892,22 @@ Use after addressing feedback to indicate the discussion is complete.`,
 // src/tools/notes-unified.ts
 import { tool as tool13 } from "@opencode-ai/plugin";
 var z13 = tool13.schema;
+function normalizeBoolean2(value) {
+  if (typeof value === "boolean") return value;
+  if (value === "true") return true;
+  if (value === "false") return false;
+  return void 0;
+}
 function filterNotesByResolved(result, resolved) {
-  if (typeof resolved !== "boolean") {
+  const normalizedResolved = normalizeBoolean2(resolved);
+  if (normalizedResolved === void 0) {
     return result;
   }
   return {
     ...result,
     notes: {
       ...result.notes,
-      nodes: result.notes.nodes.filter((n) => n.resolved === resolved)
+      nodes: result.notes.nodes.filter((n) => n.resolvable && n.resolved === normalizedResolved)
     }
   };
 }
@@ -3802,7 +3984,7 @@ Examples:
       snippet_id: z13.number().optional().describe("Snippet ID (required for snippet)"),
       // Filtering
       resolved: z13.boolean().optional().describe(
-        "Filter by resolved status: true for resolved, false for unresolved (client-side filtering)"
+        "Filter by resolved status: true for resolved, false for unresolved. Only returns resolvable notes (excludes system notes). Client-side filtering."
       ),
       // Pagination
       first: z13.number().optional().describe("Number of items to return from the beginning (default: 20, max: 100)"),
@@ -4277,6 +4459,107 @@ Note: Requires administrator access.`,
   })
 };
 
+// src/tools/award-emoji.ts
+import { tool as tool16 } from "@opencode-ai/plugin";
+var z16 = tool16.schema;
+function validateAwardEmojiParams(args) {
+  if (!args.project_id) {
+    throw new Error("project_id is required");
+  }
+  if (args.resource_iid == null) {
+    throw new Error("resource_iid is required");
+  }
+}
+function buildResourceId(args) {
+  return {
+    projectId: args.project_id,
+    resourceType: args.resource_type,
+    resourceIid: args.resource_iid,
+    noteId: args.note_id
+  };
+}
+var awardEmojiTools = {
+  /**
+   * Add a reaction (award emoji) to a resource or note
+   */
+  gitlab_create_award_emoji: tool16({
+    description: `Add a reaction (award emoji) to a merge request, issue, snippet, or a note/comment on these resources.
+
+Common emoji names: thumbsup, thumbsdown, smile, tada, rocket, eyes, heart, +1, -1
+
+To react to a specific comment/note, provide the note_id parameter.
+
+Examples:
+- React to MR: resource_type="merge_request", project_id="group/project", resource_iid=123, name="thumbsup"
+- React to issue comment: resource_type="issue", project_id="group/project", resource_iid=456, note_id=789, name="rocket"`,
+    args: {
+      resource_type: z16.enum(["merge_request", "issue", "snippet"]).describe("Type of resource to add the reaction to"),
+      project_id: z16.string().describe("Project ID or URL-encoded path"),
+      resource_iid: z16.number().describe("Internal ID of the merge request, issue, or snippet"),
+      name: z16.string().describe(
+        'Emoji name without colons (e.g., "thumbsup", "rocket", "eyes", "heart", "tada")'
+      ),
+      note_id: z16.number().optional().describe("Note/comment ID to add the reaction to (if reacting to a specific comment)")
+    },
+    execute: async (args, _ctx) => {
+      validateAwardEmojiParams(args);
+      const client = getGitLabClient();
+      const resource = buildResourceId(args);
+      const result = await client.createAwardEmoji(resource, args.name);
+      return JSON.stringify(result, null, 2);
+    }
+  }),
+  /**
+   * List all reactions on a resource or note
+   */
+  gitlab_list_award_emoji: tool16({
+    description: `List all reactions (award emoji) on a merge request, issue, snippet, or a specific note/comment.
+
+Examples:
+- List reactions on MR: resource_type="merge_request", project_id="group/project", resource_iid=123
+- List reactions on a comment: resource_type="issue", project_id="group/project", resource_iid=456, note_id=789`,
+    args: {
+      resource_type: z16.enum(["merge_request", "issue", "snippet"]).describe("Type of resource"),
+      project_id: z16.string().describe("Project ID or URL-encoded path"),
+      resource_iid: z16.number().describe("Internal ID of the merge request, issue, or snippet"),
+      note_id: z16.number().optional().describe("Note/comment ID to list reactions for (if listing for a specific comment)")
+    },
+    execute: async (args, _ctx) => {
+      validateAwardEmojiParams(args);
+      const client = getGitLabClient();
+      const resource = buildResourceId(args);
+      const result = await client.listAwardEmoji(resource);
+      return JSON.stringify(result, null, 2);
+    }
+  }),
+  /**
+   * Remove a reaction from a resource or note
+   */
+  gitlab_delete_award_emoji: tool16({
+    description: `Remove a reaction (award emoji) from a merge request, issue, snippet, or note/comment.
+
+You need the award_id which can be found using gitlab_list_award_emoji.
+
+Examples:
+- Remove reaction from MR: resource_type="merge_request", project_id="group/project", resource_iid=123, award_id=456
+- Remove from comment: resource_type="issue", ..., note_id=789, award_id=456`,
+    args: {
+      resource_type: z16.enum(["merge_request", "issue", "snippet"]).describe("Type of resource"),
+      project_id: z16.string().describe("Project ID or URL-encoded path"),
+      resource_iid: z16.number().describe("Internal ID of the merge request, issue, or snippet"),
+      award_id: z16.number().describe("ID of the award emoji to remove"),
+      note_id: z16.number().optional().describe("Note/comment ID if removing from a specific comment")
+    },
+    execute: async (args, _ctx) => {
+      validateAwardEmojiParams(args);
+      const client = getGitLabClient();
+      const resource = buildResourceId(args);
+      await client.deleteAwardEmoji(resource, args.award_id);
+      return JSON.stringify({ success: true, message: "Award emoji removed" }, null, 2);
+    }
+  })
+};
+
 // src/index.ts
 var gitlabPlugin = async (_input) => {
   return {
@@ -4312,7 +4595,9 @@ var gitlabPlugin = async (_input) => {
       // Git Tools
       ...gitTools,
       // Audit Tools
-      ...auditTools
+      ...auditTools,
+      // Award Emoji (Reactions) Tools
+      ...awardEmojiTools
     }
   };
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gitlab/opencode-gitlab-plugin",
-  "version": "1.6.1",
+  "version": "1.7.0",
   "description": "GitLab tools plugin for OpenCode - provides GitLab API access for merge requests, issues, pipelines, and more",
   "type": "module",
   "main": "dist/index.js",