@gitlab/opencode-gitlab-plugin 1.4.0 → 1.5.1

package/CHANGELOG.md

@@ -2,6 +2,20 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.5.1](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.5.0...v1.5.1) (2026-02-02)
+
+
+### 🐛 Bug Fixes
+
+* remove unsupported GraphQL fields causing API errors ([6d6de6f](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/6d6de6fff29609f3fc591a199055a752abe0fdcd))
+
+## [1.5.0](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.4.0...v1.5.0) (2026-02-02)
+
+
+### ✨ Features
+
+* migrate discussions to GraphQL with cursor-based pagination ([1c61895](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/1c61895a8f0146f40d7a29a303cdc49e91280280))
+
 ## [1.4.0](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.3.0...v1.4.0) (2026-02-02)
 
 

package/dist/index.js

@@ -93,7 +93,6 @@ function buildPaginationVariables(options) {
   if (options?.after) variables.after = options.after;
   if (options?.last !== void 0) variables.last = options.last;
   if (options?.before) variables.before = options.before;
-  if (options?.filter) variables.filter = options.filter;
   return variables;
 }
 var NOTES_FRAGMENT = `
@@ -132,6 +131,74 @@ var NOTES_CONNECTION_FRAGMENT = `
   }
 `;
 
+// src/client/discussions-types.ts
+function buildDiscussionsPaginationVariables(options) {
+  const variables = {};
+  if (options?.first !== void 0) {
+    variables.first = options.first;
+  } else if (options?.last == null) {
+    variables.first = 20;
+  }
+  if (options?.after) variables.after = options.after;
+  if (options?.last !== void 0) variables.last = options.last;
+  if (options?.before) variables.before = options.before;
+  return variables;
+}
+var DISCUSSION_NOTE_FRAGMENT = `
+  fragment DiscussionNoteFields on Note {
+    id
+    body
+    bodyHtml
+    createdAt
+    updatedAt
+    system
+    resolvable
+    resolved
+    resolvedAt
+    url
+    author {
+      id
+      username
+      name
+      avatarUrl
+    }
+  }
+`;
+var DISCUSSION_FRAGMENT = `
+  fragment DiscussionFields on Discussion {
+    id
+    replyId
+    createdAt
+    resolved
+    resolvable
+    resolvedAt
+    resolvedBy {
+      id
+      username
+      name
+      avatarUrl
+    }
+    notes {
+      nodes {
+        ...DiscussionNoteFields
+      }
+    }
+  }
+`;
+var DISCUSSIONS_CONNECTION_FRAGMENT = `
+  fragment DiscussionsConnectionFields on DiscussionConnection {
+    pageInfo {
+      hasNextPage
+      hasPreviousPage
+      startCursor
+      endCursor
+    }
+    nodes {
+      ...DiscussionFields
+    }
+  }
+`;
+
 // src/client/merge-requests.ts
 var LIST_MR_NOTES_QUERY = `
   ${NOTES_FRAGMENT}
@@ -143,7 +210,6 @@ var LIST_MR_NOTES_QUERY = `
     $after: String
     $last: Int
     $before: String
-    $filter: WorkItemNotesFilterType
   ) {
     project(fullPath: $projectPath) {
       mergeRequest(iid: $mrIid) {
@@ -152,7 +218,6 @@ var LIST_MR_NOTES_QUERY = `
           after: $after
           last: $last
           before: $before
-          filter: $filter
         ) {
           ...NotesConnectionFields
         }
@@ -160,6 +225,32 @@ var LIST_MR_NOTES_QUERY = `
     }
   }
 `;
+var LIST_MR_DISCUSSIONS_QUERY = `
+  ${DISCUSSION_NOTE_FRAGMENT}
+  ${DISCUSSION_FRAGMENT}
+  ${DISCUSSIONS_CONNECTION_FRAGMENT}
+  query listMrDiscussions(
+    $projectPath: ID!
+    $mrIid: String!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    project(fullPath: $projectPath) {
+      mergeRequest(iid: $mrIid) {
+        discussions(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...DiscussionsConnectionFields
+        }
+      }
+    }
+  }
+`;
 var SET_AUTO_MERGE_MUTATION = `
   mutation setAutoMerge(
     $projectPath: ID!
@@ -218,12 +309,41 @@ var MergeRequestsClient = class extends GitLabApiClient {
       `/projects/${encodedProject}/merge_requests/${mrIid}/changes`
     );
   }
-  async listMrDiscussions(projectId, mrIid) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/merge_requests/${mrIid}/discussions`
-    );
+  /**
+   * List notes on a merge request using GraphQL API with pagination support
+   */
+  async listMrNotes(projectId, mrIid, options) {
+    const variables = {
+      projectPath: projectId,
+      mrIid: String(mrIid),
+      ...buildPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_MR_NOTES_QUERY, variables);
+    const notes = result.project?.mergeRequest?.notes;
+    if (!notes) {
+      throw new Error("Merge request not found or access denied");
+    }
+    return {
+      notes,
+      pageInfo: notes.pageInfo,
+      totalCount: notes.count
+    };
+  }
+  /**
+   * List discussions on a merge request using GraphQL API with pagination support
+   */
+  async listMrDiscussions(projectId, mrIid, options) {
+    const variables = {
+      projectPath: projectId,
+      mrIid: String(mrIid),
+      ...buildDiscussionsPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_MR_DISCUSSIONS_QUERY, variables);
+    const discussions = result.project?.mergeRequest?.discussions;
+    if (!discussions) {
+      throw new Error("Merge request not found or access denied");
+    }
+    return { discussions };
   }
   async getMrDiscussion(projectId, mrIid, discussionId) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -260,26 +380,6 @@ var MergeRequestsClient = class extends GitLabApiClient {
       requestBody
     );
   }
-  /**
-   * List notes on a merge request using GraphQL API with pagination support
-   */
-  async listMrNotes(projectId, mrIid, options) {
-    const variables = {
-      projectPath: projectId,
-      mrIid: String(mrIid),
-      ...buildPaginationVariables(options)
-    };
-    const result = await this.fetchGraphQL(LIST_MR_NOTES_QUERY, variables);
-    const notes = result.project?.mergeRequest?.notes;
-    if (!notes) {
-      throw new Error("Merge request not found or access denied");
-    }
-    return {
-      notes,
-      pageInfo: notes.pageInfo,
-      totalCount: notes.count
-    };
-  }
   async createMrNote(projectId, mrIid, body, discussionId) {
     const encodedProject = this.encodeProjectId(projectId);
     if (discussionId) {
@@ -366,7 +466,6 @@ var LIST_ISSUE_NOTES_QUERY = `
     $after: String
     $last: Int
     $before: String
-    $filter: WorkItemNotesFilterType
   ) {
     project(fullPath: $projectPath) {
       issue(iid: $issueIid) {
@@ -375,7 +474,6 @@ var LIST_ISSUE_NOTES_QUERY = `
           after: $after
           last: $last
           before: $before
-          filter: $filter
         ) {
           ...NotesConnectionFields
         }
@@ -383,6 +481,32 @@ var LIST_ISSUE_NOTES_QUERY = `
     }
   }
 `;
+var LIST_ISSUE_DISCUSSIONS_QUERY = `
+  ${DISCUSSION_NOTE_FRAGMENT}
+  ${DISCUSSION_FRAGMENT}
+  ${DISCUSSIONS_CONNECTION_FRAGMENT}
+  query listIssueDiscussions(
+    $projectPath: ID!
+    $issueIid: String!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    project(fullPath: $projectPath) {
+      issue(iid: $issueIid) {
+        discussions(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...DiscussionsConnectionFields
+        }
+      }
+    }
+  }
+`;
 var IssuesClient = class extends GitLabApiClient {
   async createIssue(projectId, title, options) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -438,12 +562,21 @@ var IssuesClient = class extends GitLabApiClient {
       totalCount: notes.count
     };
   }
-  async listIssueDiscussions(projectId, issueIid) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/issues/${issueIid}/discussions`
-    );
+  /**
+   * List discussions on an issue using GraphQL API with pagination support
+   */
+  async listIssueDiscussions(projectId, issueIid, options) {
+    const variables = {
+      projectPath: projectId,
+      issueIid: String(issueIid),
+      ...buildDiscussionsPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_ISSUE_DISCUSSIONS_QUERY, variables);
+    const discussions = result.project?.issue?.discussions;
+    if (!discussions) {
+      throw new Error("Issue not found or access denied");
+    }
+    return { discussions };
   }
   async getIssueDiscussion(projectId, issueIid, discussionId) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -1264,36 +1397,6 @@ var LIST_TODOS_QUERY = `
             name
             avatarUrl
           }
-          targetEntity {
-            __typename
-            ... on Issue {
-              id
-              title
-              iid
-            }
-            ... on MergeRequest {
-              id
-              title
-              iid
-            }
-            ... on Epic {
-              id
-              title
-              iid
-            }
-            ... on Commit {
-              id
-              title
-            }
-            ... on DesignManagement__Design {
-              id
-            }
-            ... on AlertManagement__Alert {
-              id
-              title
-              iid
-            }
-          }
         }
       }
     }
@@ -1382,7 +1485,7 @@ var LIST_EPIC_NOTES_QUERY = `
   ${NOTES_CONNECTION_FRAGMENT}
   query listEpicNotes(
     $groupPath: ID!
-    $epicIid: ID!
+    $epicIid: String!
     $first: Int
     $after: String
     $last: Int
@@ -1402,6 +1505,32 @@ var LIST_EPIC_NOTES_QUERY = `
     }
   }
 `;
+var LIST_EPIC_DISCUSSIONS_QUERY = `
+  ${DISCUSSION_NOTE_FRAGMENT}
+  ${DISCUSSION_FRAGMENT}
+  ${DISCUSSIONS_CONNECTION_FRAGMENT}
+  query listEpicDiscussions(
+    $groupPath: ID!
+    $epicIid: String!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    group(fullPath: $groupPath) {
+      epic(iid: $epicIid) {
+        discussions(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...DiscussionsConnectionFields
+        }
+      }
+    }
+  }
+`;
 var EpicsClient = class extends GitLabApiClient {
   async getEpic(groupId, epicIid) {
     const encodedGroup = encodeURIComponent(groupId);
@@ -1470,12 +1599,21 @@ var EpicsClient = class extends GitLabApiClient {
       totalCount: notes.count
     };
   }
-  async listEpicDiscussions(groupId, epicIid) {
-    const encodedGroup = encodeURIComponent(groupId);
-    return this.fetch(
-      "GET",
-      `/groups/${encodedGroup}/epics/${epicIid}/discussions`
-    );
+  /**
+   * List discussions on an epic using GraphQL API with pagination support
+   */
+  async listEpicDiscussions(groupId, epicIid, options) {
+    const variables = {
+      groupPath: groupId,
+      epicIid: String(epicIid),
+      ...buildDiscussionsPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_EPIC_DISCUSSIONS_QUERY, variables);
+    const discussions = result.group?.epic?.discussions;
+    if (!discussions) {
+      throw new Error("Epic not found or access denied");
+    }
+    return { discussions };
   }
   async getEpicDiscussion(groupId, epicIid, discussionId) {
     const encodedGroup = encodeURIComponent(groupId);
@@ -1533,21 +1671,32 @@ var LIST_SNIPPET_NOTES_QUERY = `
     }
   }
 `;
-var SnippetsClient = class extends GitLabApiClient {
-  async listSnippetDiscussions(projectId, snippetId) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/snippets/${snippetId}/discussions`
-    );
-  }
-  async getSnippetDiscussion(projectId, snippetId, discussionId) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/snippets/${snippetId}/discussions/${discussionId}`
-    );
+var LIST_SNIPPET_DISCUSSIONS_QUERY = `
+  ${DISCUSSION_NOTE_FRAGMENT}
+  ${DISCUSSION_FRAGMENT}
+  ${DISCUSSIONS_CONNECTION_FRAGMENT}
+  query listSnippetDiscussions(
+    $snippetGid: SnippetID!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    snippets(ids: [$snippetGid]) {
+      nodes {
+        discussions(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...DiscussionsConnectionFields
+        }
+      }
+    }
   }
+`;
+var SnippetsClient = class extends GitLabApiClient {
   /**
    * List notes on a snippet using GraphQL API with pagination support
    */
@@ -1573,6 +1722,33 @@ var SnippetsClient = class extends GitLabApiClient {
       totalCount: notes.count
     };
   }
+  /**
+   * List discussions on a snippet using GraphQL API with pagination support
+   */
+  async listSnippetDiscussions(projectId, snippetId, options) {
+    const variables = {
+      snippetGid: `gid://gitlab/ProjectSnippet/${snippetId}`,
+      ...buildDiscussionsPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_SNIPPET_DISCUSSIONS_QUERY, variables);
+    const discussions = result.snippets.nodes[0]?.discussions || {
+      nodes: [],
+      pageInfo: {
+        hasNextPage: false,
+        hasPreviousPage: false,
+        startCursor: null,
+        endCursor: null
+      }
+    };
+    return { discussions };
+  }
+  async getSnippetDiscussion(projectId, snippetId, discussionId) {
+    const encodedProject = this.encodeProjectId(projectId);
+    return this.fetch(
+      "GET",
+      `/projects/${encodedProject}/snippets/${snippetId}/discussions/${discussionId}`
+    );
+  }
   async createSnippetNote(projectId, snippetId, body, discussionId) {
     const encodedProject = this.encodeProjectId(projectId);
     if (discussionId) {
@@ -1916,11 +2092,20 @@ Returns all discussion threads with nested notes. Each discussion contains a 'no
 Note: For a flattened list of all comments, use gitlab_list_mr_notes instead.`,
     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")
+      mr_iid: z.number().describe("The internal ID of the merge request"),
+      first: z.number().optional().describe("Number of discussions to fetch from the beginning (default: 20)"),
+      after: z.string().optional().describe("Cursor for forward pagination (from pageInfo.endCursor)"),
+      last: z.number().optional().describe("Number of discussions to fetch from the end"),
+      before: z.string().optional().describe("Cursor for backward pagination (from pageInfo.startCursor)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const discussions = await client.listMrDiscussions(args.project_id, args.mr_iid);
+      const discussions = await client.listMrDiscussions(args.project_id, args.mr_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
       return JSON.stringify(discussions, null, 2);
     }
   }),
@@ -1938,8 +2123,7 @@ Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
       first: z.number().optional().describe("Number of items to return from the beginning (default: 20, max: 100)"),
       after: z.string().optional().describe("Cursor for forward pagination - use endCursor from previous response"),
       last: z.number().optional().describe("Number of items to return from the end (for backward pagination)"),
-      before: z.string().optional().describe("Cursor for backward pagination - use startCursor from previous response"),
-      filter: z.enum(["ALL_NOTES", "ONLY_COMMENTS", "ONLY_ACTIVITY"]).optional().describe("Filter notes by type: ALL_NOTES (default), ONLY_COMMENTS, or ONLY_ACTIVITY")
+      before: z.string().optional().describe("Cursor for backward pagination - use startCursor from previous response")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
@@ -1947,8 +2131,7 @@ Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
         first: args.first,
         after: args.after,
         last: args.last,
-        before: args.before,
-        filter: args.filter
+        before: args.before
       });
       return JSON.stringify(result, null, 2);
     }
@@ -2295,8 +2478,7 @@ Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
       first: z2.number().optional().describe("Number of items to return from the beginning (default: 20, max: 100)"),
       after: z2.string().optional().describe("Cursor for forward pagination - use endCursor from previous response"),
       last: z2.number().optional().describe("Number of items to return from the end (for backward pagination)"),
-      before: z2.string().optional().describe("Cursor for backward pagination - use startCursor from previous response"),
-      filter: z2.enum(["ALL_NOTES", "ONLY_COMMENTS", "ONLY_ACTIVITY"]).optional().describe("Filter notes by type: ALL_NOTES (default), ONLY_COMMENTS, or ONLY_ACTIVITY")
+      before: z2.string().optional().describe("Cursor for backward pagination - use startCursor from previous response")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
@@ -2304,8 +2486,7 @@ Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
         first: args.first,
         after: args.after,
         last: args.last,
-        before: args.before,
-        filter: args.filter
+        before: args.before
       });
       return JSON.stringify(result, null, 2);
     }
@@ -2316,11 +2497,20 @@ Returns all discussion threads with nested notes. Each discussion contains a 'no
 Use the discussion 'id' field to reply to a specific thread with gitlab_create_issue_note.`,
     args: {
       project_id: z2.string().describe("The project ID or URL-encoded path"),
-      issue_iid: z2.number().describe("The internal ID of the issue")
+      issue_iid: z2.number().describe("The internal ID of the issue"),
+      first: z2.number().optional().describe("Number of discussions to fetch from the beginning (default: 20)"),
+      after: z2.string().optional().describe("Cursor for forward pagination (from pageInfo.endCursor)"),
+      last: z2.number().optional().describe("Number of discussions to fetch from the end"),
+      before: z2.string().optional().describe("Cursor for backward pagination (from pageInfo.startCursor)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const discussions = await client.listIssueDiscussions(args.project_id, args.issue_iid);
+      const discussions = await client.listIssueDiscussions(args.project_id, args.issue_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
       return JSON.stringify(discussions, null, 2);
     }
   }),
@@ -2593,11 +2783,20 @@ Returns all discussion threads with nested notes. Each discussion contains a 'no
 Use the discussion 'id' field to reply to a specific thread with gitlab_create_epic_note.`,
     args: {
       group_id: z3.string().describe("The group ID or URL-encoded path"),
-      epic_iid: z3.number().describe("The internal ID of the epic")
+      epic_iid: z3.number().describe("The internal ID of the epic"),
+      first: z3.number().optional().describe("Number of discussions to fetch from the beginning (default: 20)"),
+      after: z3.string().optional().describe("Cursor for forward pagination (from pageInfo.endCursor)"),
+      last: z3.number().optional().describe("Number of discussions to fetch from the end"),
+      before: z3.string().optional().describe("Cursor for backward pagination (from pageInfo.startCursor)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const discussions = await client.listEpicDiscussions(args.group_id, args.epic_iid);
+      const discussions = await client.listEpicDiscussions(args.group_id, args.epic_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
       return JSON.stringify(discussions, null, 2);
     }
   }),
@@ -3567,11 +3766,20 @@ Returns all discussion threads with nested notes. Each discussion contains a 'no
 Use the discussion 'id' field to reply to a specific thread with gitlab_create_snippet_note.`,
     args: {
       project_id: z9.string().describe("The project ID or URL-encoded path"),
-      snippet_id: z9.number().describe("The ID of the snippet")
+      snippet_id: z9.number().describe("The ID of the snippet"),
+      first: z9.number().optional().describe("Number of discussions to fetch from the beginning (default: 20)"),
+      after: z9.string().optional().describe("Cursor for forward pagination (from pageInfo.endCursor)"),
+      last: z9.number().optional().describe("Number of discussions to fetch from the end"),
+      before: z9.string().optional().describe("Cursor for backward pagination (from pageInfo.startCursor)")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const discussions = await client.listSnippetDiscussions(args.project_id, args.snippet_id);
+      const discussions = await client.listSnippetDiscussions(args.project_id, args.snippet_id, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
       return JSON.stringify(discussions, null, 2);
     }
   }),

package/package.json

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