iterate over all pages until github returns an empty array

__tests__/main.test.ts

@@ -13,10 +13,32 @@ describe('Hide Comments', () => {
     const client = new Client('secrets', 1, 'owner', 'repo')
     const github = nock('https://api.github.com')
       .get(`/repos/owner/repo/issues/1/comments`)
+      .query(true)
       .reply(200, listComment)
+      .get(`/repos/owner/repo/issues/1/comments`)
+      .query(true)
+      .reply(200, [])
 
     const response = await client.SelectComments(`bot`)
 
     expect(response).toStrictEqual(['hide me'])
   })
+
+  it('should paginate properly', async () => {
+    const client = new Client('secrets', 1, 'owner', 'repo')
+    const numPages = 5
+    const github = nock('https://api.github.com')
+      .get(`/repos/owner/repo/issues/1/comments`)
+      .query(true)
+      .times(numPages)
+      .reply(200, listComment)
+      .get(`/repos/owner/repo/issues/1/comments`)
+      .query(true)
+      .reply(200, [])
+
+    const response = await client.SelectComments(`bot`)
+    const expected = new Array(numPages).fill('hide me')
+
+    expect(response).toStrictEqual(expected)
+  })
 })

src/client.ts

@@ -1,3 +1,4 @@
+import * as core from '@actions/core'
 import * as github from '@actions/github'
 
 import {GitHub} from '@actions/github/lib/utils'
@@ -35,18 +36,32 @@ export class Client {
   }
 
   async SelectComments(userName: string): Promise<string[]> {
-    const resp = await this.octokit.rest.issues.listComments({
-      owner: this.owner,
-      repo: this.repo,
-      issue_number: this.issueNumber
-    })
-
     const ids: string[] = []
-    for (const r of resp.data) {
-      if (r.user !== null && r.user.login !== userName) {
-        continue
+
+    // continually page through comments ...
+    // 3k comments is an absurd amount, feels like a pretty decent default limit
+    const maxPages = 100
+    for (let page = 1; page <= maxPages; page++) {
+      const resp = await this.octokit.rest.issues.listComments({
+        owner: this.owner,
+        repo: this.repo,
+        issue_number: this.issueNumber,
+        page
+      })
+
+      // ... until we've read them all
+      if (!resp.data || resp.data.length === 0) {
+        break
+      }
+
+      core.debug(`page ${page} contained ${resp.data.length} entries`)
+
+      for (const r of resp.data) {
+        if (r.user !== null && r.user.login !== userName) {
+          continue
+        }
+        ids.push(r.node_id)
       }
-      ids.push(r.node_id)
     }
     return new Promise<string[]>(resolve => resolve(ids))
   }