@gitlab/opencode-gitlab-plugin 1.2.0 → 1.3.0

package/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All notable changes to this project will be documented in this file. See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [1.3.0](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.2.0...v1.3.0) (2026-01-30)
+
+
+### ✨ Features
+
+* **notes:** migrate notes listing to GraphQL API with pagination support ([2ec5d65](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/2ec5d6528b27b86a08f5c747f38762f90eb1f297))
+
+
+### 🐛 Bug Fixes
+
+* address review feedback for notes GraphQL migration ([fcb1a35](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/commit/fcb1a355f3ecbddf2429b0e2fecaeb1f02d29ffe))
+
 ## [1.2.0](https://gitlab.com/gitlab-org/editor-extensions/opencode-gitlab-plugin/compare/v1.1.0...v1.2.0) (2026-01-29)
 
 

package/dist/index.js

@@ -82,7 +82,84 @@ var GitLabApiClient = class {
   }
 };
 
+// src/client/notes-types.ts
+function buildPaginationVariables(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;
+  if (options?.filter) variables.filter = options.filter;
+  return variables;
+}
+var NOTES_FRAGMENT = `
+  fragment NoteFields on Note {
+    id
+    body
+    bodyHtml
+    createdAt
+    updatedAt
+    system
+    internal
+    resolvable
+    resolved
+    resolvedAt
+    url
+    author {
+      id
+      username
+      name
+      avatarUrl
+    }
+  }
+`;
+var NOTES_CONNECTION_FRAGMENT = `
+  fragment NotesConnectionFields on NoteConnection {
+    count
+    pageInfo {
+      hasNextPage
+      hasPreviousPage
+      startCursor
+      endCursor
+    }
+    nodes {
+      ...NoteFields
+    }
+  }
+`;
+
 // src/client/merge-requests.ts
+var LIST_MR_NOTES_QUERY = `
+  ${NOTES_FRAGMENT}
+  ${NOTES_CONNECTION_FRAGMENT}
+  query listMrNotes(
+    $projectPath: ID!
+    $mrIid: String!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+    $filter: WorkItemNotesFilterType
+  ) {
+    project(fullPath: $projectPath) {
+      mergeRequest(iid: $mrIid) {
+        notes(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+          filter: $filter
+        ) {
+          ...NotesConnectionFields
+        }
+      }
+    }
+  }
+`;
 var SET_AUTO_MERGE_MUTATION = `
   mutation setAutoMerge(
     $projectPath: ID!
@@ -183,12 +260,25 @@ var MergeRequestsClient = class extends GitLabApiClient {
       requestBody
     );
   }
-  async listMrNotes(projectId, mrIid) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/merge_requests/${mrIid}/notes`
-    );
+  /**
+   * 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);
@@ -235,10 +325,6 @@ var MergeRequestsClient = class extends GitLabApiClient {
       `/projects/${encodedProject}/merge_requests/${mrIid}/pipelines`
     );
   }
-  /**
-   * List merge request diffs with pagination support
-   * API: GET /projects/:id/merge_requests/:merge_request_iid/diffs
-   */
   async listMergeRequestDiffs(projectId, mrIid, options = {}) {
     const encodedProject = this.encodeProjectId(projectId);
     const params = new URLSearchParams();
@@ -270,6 +356,33 @@ var MergeRequestsClient = class extends GitLabApiClient {
 };
 
 // src/client/issues.ts
+var LIST_ISSUE_NOTES_QUERY = `
+  ${NOTES_FRAGMENT}
+  ${NOTES_CONNECTION_FRAGMENT}
+  query listIssueNotes(
+    $projectPath: ID!
+    $issueIid: String!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+    $filter: WorkItemNotesFilterType
+  ) {
+    project(fullPath: $projectPath) {
+      issue(iid: $issueIid) {
+        notes(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+          filter: $filter
+        ) {
+          ...NotesConnectionFields
+        }
+      }
+    }
+  }
+`;
 var IssuesClient = class extends GitLabApiClient {
   async createIssue(projectId, title, options) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -305,12 +418,25 @@ var IssuesClient = class extends GitLabApiClient {
     }
     return this.fetch("GET", path2);
   }
-  async listIssueNotes(projectId, issueIid) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/issues/${issueIid}/notes`
-    );
+  /**
+   * List notes on an issue using GraphQL API with pagination support
+   */
+  async listIssueNotes(projectId, issueIid, options) {
+    const variables = {
+      projectPath: projectId,
+      issueIid: String(issueIid),
+      ...buildPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_ISSUE_NOTES_QUERY, variables);
+    const notes = result.project?.issue?.notes;
+    if (!notes) {
+      throw new Error("Issue not found or access denied");
+    }
+    return {
+      notes,
+      pageInfo: notes.pageInfo,
+      totalCount: notes.count
+    };
   }
   async listIssueDiscussions(projectId, issueIid) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -357,10 +483,6 @@ var IssuesClient = class extends GitLabApiClient {
       { resolved: false }
     );
   }
-  /**
-   * Get a single note from an issue
-   * API: GET /projects/:id/issues/:issue_iid/notes/:note_id
-   */
   async getIssueNote(projectId, issueIid, noteId) {
     const encodedProject = this.encodeProjectId(projectId);
     return this.fetch(
@@ -1111,6 +1233,31 @@ var TodosClient = class extends GitLabApiClient {
 };
 
 // src/client/epics.ts
+var LIST_EPIC_NOTES_QUERY = `
+  ${NOTES_FRAGMENT}
+  ${NOTES_CONNECTION_FRAGMENT}
+  query listEpicNotes(
+    $groupPath: ID!
+    $epicIid: ID!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    group(fullPath: $groupPath) {
+      epic(iid: $epicIid) {
+        notes(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...NotesConnectionFields
+        }
+      }
+    }
+  }
+`;
 var EpicsClient = class extends GitLabApiClient {
   async getEpic(groupId, epicIid) {
     const encodedGroup = encodeURIComponent(groupId);
@@ -1159,12 +1306,25 @@ var EpicsClient = class extends GitLabApiClient {
       `/groups/${encodedGroup}/epics/${epicIid}/issues/${epicIssueId}`
     );
   }
-  async listEpicNotes(groupId, epicIid) {
-    const encodedGroup = encodeURIComponent(groupId);
-    return this.fetch(
-      "GET",
-      `/groups/${encodedGroup}/epics/${epicIid}/notes`
-    );
+  /**
+   * List notes on an epic using GraphQL API with pagination support
+   */
+  async listEpicNotes(groupId, epicIid, options) {
+    const variables = {
+      groupPath: groupId,
+      epicIid: String(epicIid),
+      ...buildPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_EPIC_NOTES_QUERY, variables);
+    const notes = result.group?.epic?.notes;
+    if (!notes) {
+      throw new Error("Epic not found or access denied");
+    }
+    return {
+      notes,
+      pageInfo: notes.pageInfo,
+      totalCount: notes.count
+    };
   }
   async listEpicDiscussions(groupId, epicIid) {
     const encodedGroup = encodeURIComponent(groupId);
@@ -1195,10 +1355,6 @@ var EpicsClient = class extends GitLabApiClient {
       { body }
     );
   }
-  /**
-   * Get a single note from an epic
-   * API: GET /groups/:id/epics/:epic_iid/notes/:note_id
-   */
   async getEpicNote(groupId, epicIid, noteId) {
     const encodedGroup = encodeURIComponent(groupId);
     return this.fetch(
@@ -1209,6 +1365,30 @@ var EpicsClient = class extends GitLabApiClient {
 };
 
 // src/client/snippets.ts
+var LIST_SNIPPET_NOTES_QUERY = `
+  ${NOTES_FRAGMENT}
+  ${NOTES_CONNECTION_FRAGMENT}
+  query listSnippetNotes(
+    $snippetGid: SnippetID!
+    $first: Int
+    $after: String
+    $last: Int
+    $before: String
+  ) {
+    snippets(ids: [$snippetGid]) {
+      nodes {
+        notes(
+          first: $first
+          after: $after
+          last: $last
+          before: $before
+        ) {
+          ...NotesConnectionFields
+        }
+      }
+    }
+  }
+`;
 var SnippetsClient = class extends GitLabApiClient {
   async listSnippetDiscussions(projectId, snippetId) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -1224,12 +1404,30 @@ var SnippetsClient = class extends GitLabApiClient {
       `/projects/${encodedProject}/snippets/${snippetId}/discussions/${discussionId}`
     );
   }
-  async listSnippetNotes(projectId, snippetId) {
-    const encodedProject = this.encodeProjectId(projectId);
-    return this.fetch(
-      "GET",
-      `/projects/${encodedProject}/snippets/${snippetId}/notes`
-    );
+  /**
+   * List notes on a snippet using GraphQL API with pagination support
+   */
+  async listSnippetNotes(projectId, snippetId, options) {
+    const variables = {
+      snippetGid: `gid://gitlab/ProjectSnippet/${snippetId}`,
+      ...buildPaginationVariables(options)
+    };
+    const result = await this.fetchGraphQL(LIST_SNIPPET_NOTES_QUERY, variables);
+    const notes = result.snippets.nodes[0]?.notes || {
+      nodes: [],
+      pageInfo: {
+        hasNextPage: false,
+        hasPreviousPage: false,
+        startCursor: null,
+        endCursor: null
+      },
+      count: 0
+    };
+    return {
+      notes,
+      pageInfo: notes.pageInfo,
+      totalCount: notes.count
+    };
   }
   async createSnippetNote(projectId, snippetId, body, discussionId) {
     const encodedProject = this.encodeProjectId(projectId);
@@ -1583,17 +1781,32 @@ Note: For a flattened list of all comments, use gitlab_list_mr_notes instead.`,
     }
   }),
   gitlab_list_mr_notes: tool({
-    description: `List all notes/comments on a merge request in a flat structure.
+    description: `List all notes/comments on a merge request using GraphQL API with pagination support.
 Returns all comments including system notes, code review comments, and general discussion.
-This is easier to read than discussions which have nested structure.`,
+This is easier to read than discussions which have nested structure.
+
+The response includes pagination information (pageInfo) with cursors for fetching additional pages.
+Use 'after' with the 'endCursor' from pageInfo to get the next page.
+Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
     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 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")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const notes = await client.listMrNotes(args.project_id, args.mr_iid);
-      return JSON.stringify(notes, null, 2);
+      const result = await client.listMrNotes(args.project_id, args.mr_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before,
+        filter: args.filter
+      });
+      return JSON.stringify(result, null, 2);
     }
   }),
   gitlab_create_mr_note: tool({
@@ -1926,16 +2139,31 @@ Can filter by state, labels, assignee, milestone.`,
     }
   }),
   gitlab_list_issue_notes: tool2({
-    description: `List all notes/comments on an issue.
-Returns all comments including system notes in chronological order.`,
+    description: `List all notes/comments on an issue using GraphQL API with pagination support.
+Returns all comments including system notes in chronological order.
+
+The response includes pagination information (pageInfo) with cursors for fetching additional pages.
+Use 'after' with the 'endCursor' from pageInfo to get the next page.
+Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
     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 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")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const notes = await client.listIssueNotes(args.project_id, args.issue_iid);
-      return JSON.stringify(notes, null, 2);
+      const result = await client.listIssueNotes(args.project_id, args.issue_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before,
+        filter: args.filter
+      });
+      return JSON.stringify(result, null, 2);
     }
   }),
   gitlab_list_issue_discussions: tool2({
@@ -2190,16 +2418,29 @@ Removes the association between the issue and the epic.`,
     }
   }),
   gitlab_list_epic_notes: tool3({
-    description: `List all comments/notes on an epic in a flat structure.
-Returns all comments in chronological order.`,
+    description: `List all comments/notes on an epic using GraphQL API with pagination support.
+Returns all comments in chronological order.
+
+The response includes pagination information (pageInfo) with cursors for fetching additional pages.
+Use 'after' with the 'endCursor' from pageInfo to get the next page.
+Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
     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 items to return from the beginning (default: 20, max: 100)"),
+      after: z3.string().optional().describe("Cursor for forward pagination - use endCursor from previous response"),
+      last: z3.number().optional().describe("Number of items to return from the end (for backward pagination)"),
+      before: z3.string().optional().describe("Cursor for backward pagination - use startCursor from previous response")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const notes = await client.listEpicNotes(args.group_id, args.epic_iid);
-      return JSON.stringify(notes, null, 2);
+      const result = await client.listEpicNotes(args.group_id, args.epic_iid, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
+      return JSON.stringify(result, null, 2);
     }
   }),
   gitlab_list_epic_discussions: tool3({
@@ -3210,16 +3451,29 @@ Use this to get the full context of a specific conversation.`,
     }
   }),
   gitlab_list_snippet_notes: tool9({
-    description: `List all notes/comments on a project snippet in a flat structure.
-Returns all comments including system notes in chronological order.`,
+    description: `List all notes/comments on a project snippet using GraphQL API with pagination support.
+Returns all comments including system notes in chronological order.
+
+The response includes pagination information (pageInfo) with cursors for fetching additional pages.
+Use 'after' with the 'endCursor' from pageInfo to get the next page.
+Use 'before' with the 'startCursor' from pageInfo to get the previous page.`,
     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 items to return from the beginning (default: 20, max: 100)"),
+      after: z9.string().optional().describe("Cursor for forward pagination - use endCursor from previous response"),
+      last: z9.number().optional().describe("Number of items to return from the end (for backward pagination)"),
+      before: z9.string().optional().describe("Cursor for backward pagination - use startCursor from previous response")
     },
     execute: async (args, _ctx) => {
       const client = getGitLabClient();
-      const notes = await client.listSnippetNotes(args.project_id, args.snippet_id);
-      return JSON.stringify(notes, null, 2);
+      const result = await client.listSnippetNotes(args.project_id, args.snippet_id, {
+        first: args.first,
+        after: args.after,
+        last: args.last,
+        before: args.before
+      });
+      return JSON.stringify(result, null, 2);
     }
   }),
   gitlab_create_snippet_note: tool9({

package/package.json

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