fix: properly fetch commits via REST API for recent languages

The previous fix had a bug where it tried to use listCommits() method
instead of the REST API. Now correctly fetches commit list from each
repo/branch, filters by authoring, and fetches full commit details
with file patches using the standard rest.request() method.

Author: hxreborn

source/plugins/languages/analyzer/recent.mjs

@@ -83,69 +83,65 @@ export class RecentAnalyzer extends Analyzer {
       entry.head = head //Update with latest head
     }
 
-    //Fetch commit details via REST API
-    const patches = []
-    for (const [key, {repo, ref, head, shas}] of commitsByRepo) {
+    //Fetch commits from repos and filter patches
+    const commitUrls = []
+    for (const {repo, ref} of commitsByRepo.values()) {
+      const [owner, repoName] = repo.split("/")
+      const branch = ref.replace("refs/heads/", "")
+      //Fetch commit list for this repo/branch
       try {
-        const [owner, repoName] = repo.split("/")
-        //Fetch commits for this repo/ref combination
-        let foundCommits = 0
-        for (let page = 1; foundCommits < Math.min(100, this.load); page++) {
-          const {data: commits} = await this.rest.repos.listCommits({
-            owner,
-            repo: repoName,
-            sha: ref.replace("refs/heads/", ""),
-            per_page: 20,
-            page,
-          })
-
-          if (!commits.length) break
-
-          for (const commit of commits) {
-            //Filter by authoring
-            if (!commit.author || !filters.text(commit.author?.email || commit.commit.author?.email, this.authoring, {debug: false})) {
-              continue
-            }
-
-            //Skip merge commits
-            if (commit.parents?.length > 1) continue
-
-            foundCommits++
-            patches.push({
-              sha: commit.sha,
-              name: `${commit.commit.message} (authored by ${commit.commit.author.name} on ${commit.commit.author.date})`,
-              verified: commit.verification?.verified ?? null,
-              editions: (commit.files ?? []).map(({filename, patch = ""}) => {
-                const edition = {
-                  path: filename,
-                  added: {lines: 0, bytes: 0},
-                  deleted: {lines: 0, bytes: 0},
-                  patch,
-                }
-                for (const line of patch.split("\n")) {
-                  if ((!/^[-+]/.test(line)) || (!line.trim().length))
-                    continue
-                  if (this.markers.line.test(line)) {
-                    const {op = "+", content = ""} = line.match(this.markers.line)?.groups ?? {}
-                    const size = Buffer.byteLength(content, "utf-8")
-                    edition[{"+": "added", "-": "deleted"}[op]].bytes += size
-                    edition[{"+": "added", "-": "deleted"}[op]].lines++
-                    continue
-                  }
-                }
-                return edition
-              }),
-            })
-
-            if (foundCommits >= this.load) break
+        const {data: commits} = await this.rest.request(`GET /repos/{owner}/{repo}/commits`, {
+          owner,
+          repo: repoName,
+          sha: branch,
+          per_page: 100,
+        })
+        //Filter by authoring and collect URLs
+        for (const commit of commits) {
+          const email = commit.author?.email || commit.commit.author?.email
+          if (email && filters.text(email, this.authoring, {debug: false}) && commit.parents?.length <= 1) {
+            commitUrls.push(commit.url)
           }
         }
-      }
-      catch (error) {
-        this.debug(`failed to fetch commits for ${repo} on ${ref}: ${error}`)
+      } catch (error) {
+        this.debug(`failed to fetch commits for ${repo}: ${error}`)
       }
     }
 
+    //Fetch full commit details with patches
+    const patches = [
+      ...await Promise.allSettled(
+        commitUrls.map(async url => (await this.rest.request(url)).data),
+      ),
+    ]
+      .filter(({status}) => status === "fulfilled")
+      .map(({value}) => value)
+      .map(({sha, commit: {message, author}, verification, files}) => ({
+        sha,
+        name: `${message} (authored by ${author.name} on ${author.date})`,
+        verified: verification?.verified ?? null,
+        editions: files.map(({filename, patch = ""}) => {
+          const edition = {
+            path: filename,
+            added: {lines: 0, bytes: 0},
+            deleted: {lines: 0, bytes: 0},
+            patch,
+          }
+          for (const line of patch.split("\n")) {
+            if ((!/^[-+]/.test(line)) || (!line.trim().length))
+              continue
+            if (this.markers.line.test(line)) {
+              const {op = "+", content = ""} = line.match(this.markers.line)?.groups ?? {}
+              const size = Buffer.byteLength(content, "utf-8")
+              edition[{"+": "added", "-": "deleted"}[op]].bytes += size
+              edition[{"+": "added", "-": "deleted"}[op]].lines++
+              continue
+            }
+          }
+          return edition
+        }),
+      }))
+
     return patches
   }