fix: apply PR lowlighter#1754 working code for recent languages analyzer

Uses exact implementation from oddstr13's PR lowlighter#1754:
- Builds wanted map with repo@ref keys
- Fetches commits via REST API directly
- Tracks before/head SHAs for early exit
- Uses optional chaining to prevent destructuring errors
- Fetches full commit details with file patches

Author: hxreborn

source/plugins/languages/analyzer/recent.mjs

@@ -30,7 +30,7 @@ export class RecentAnalyzer extends Analyzer {
   async patches() {
     //Fetch commits from recent activity
     this.debug(`fetching patches from last ${this.days || ""} days up to ${this.load || "∞"} events`)
-    const events = [], pages = Math.ceil((this.load || Infinity) / 100)
+    const pages = Math.ceil((this.load || Infinity) / 100)
     if (this.context.mode === "repository") {
       try {
         const {data: {default_branch: branch}} = await this.rest.repos.get(this.context)
@@ -42,6 +42,7 @@ export class RecentAnalyzer extends Analyzer {
         this.debug(`failed to get default branch for ${this.context.owner}/${this.context.repo} (${error})`)
       }
     }
+    const events = []
     try {
       for (let page = 1; page <= pages; page++) {
         this.debug(`fetching events page ${page}`)
@@ -58,65 +59,62 @@ export class RecentAnalyzer extends Analyzer {
       this.debug("no more page to load")
     }
     this.debug(`fetched ${events.length} events`)
-    this.results.latest = Math.round((new Date().getTime() - new Date(events.slice(-1).shift()?.created_at).getTime()) / (1000 * 60 * 60 * 24))
-    this.results.commits = events.length
-
-    //Retrieve edited files and filter edited lines (those starting with +/-) from patches
-    this.debug("fetching patches")
 
-    //Extract repository/ref combinations and their commit SHAs from events
-    const commitsByRepo = new Map()
-    for (const {payload, repo: {name}} of events) {
-      const {ref, before, head, commits: eventCommits} = payload
-      if (!ref || !name) continue
-      const key = `${name}:${ref}`
-      if (!commitsByRepo.has(key)) {
-        commitsByRepo.set(key, {repo: name, ref, before, head, shas: new Set()})
-      }
-      const entry = commitsByRepo.get(key)
-      //Add commit SHAs if available in payload
-      if (Array.isArray(eventCommits)) {
-        for (const {sha} of eventCommits) {
-          entry.shas.add(sha)
-        }
-      }
-      entry.head = head //Update with latest head
-    }
+    const wanted = new Map()
+    events.forEach(event => {
+      let key = `${event.repo.name}@${event.payload.ref}`
+      let item = wanted.get(key) ?? { commits: [] }
+      item.repo = event.repo.name
+      item.ref = event.payload.ref
+      item.commits.push(event.payload.before)
+      item.commits.push(event.payload.head)
+      wanted.set(key, item)
+    })
 
-    //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
+    const commits = []
+    for (const item of wanted.values()) {
       try {
-        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)
+        for (let page = 1; page <= pages; page++) {
+          this.debug(`fetching commits page ${page}`)
+          this.debug(`https://api.github.com/repos/${item.repo}/commits?sha=${item.ref}&per_page=20&page=${page}`)
+          commits.push(
+            ...(await this.rest.request(`https://api.github.com/repos/${item.repo}/commits?sha=${item.ref}&per_page=20&page=${page}`)).data
+              .map(x => {
+                item.commits = item.commits.filter(c => c !== x.sha)
+                return x
+              })
+              .filter(({ committer }) => (this.account === "organization") || (this.context.mode === "repository") ? true : !filters.text(committer?.login, [this.login], { debug: false }))
+              .filter(({ commit }) => ((!this.days) || (new Date(commit.committer.date) > new Date(Date.now() - this.days * 24 * 60 * 60 * 1000)))),
+          )
+          if (item.commits.length < 1) {
+            this.debug("found expected commits")
+            break
           }
         }
-      } catch (error) {
-        this.debug(`failed to fetch commits for ${repo}: ${error}`)
+      }
+      catch {
+        this.debug("no more page to load")
       }
     }
 
-    //Fetch full commit details with patches
+    this.debug(`fetched ${commits.length} commits`)
+    this.results.latest = Math.round((new Date().getTime() - new Date(events.slice(-1).shift()?.created_at).getTime()) / (1000 * 60 * 60 * 24))
+    this.results.commits = commits.length
+
+    //Retrieve edited files and filter edited lines (those starting with +/-) from patches
+    this.debug("fetching patches")
     const patches = [
       ...await Promise.allSettled(
-        commitUrls.map(async url => (await this.rest.request(url)).data),
+        commits
+          .filter(({committer}) => filters.text(committer?.email, this.authoring, {debug: false}))
+          .map(commit => commit.url)
+          .map(async commit => (await this.rest.request(commit)).data),
       ),
     ]
       .filter(({status}) => status === "fulfilled")
       .map(({value}) => value)
-      .map(({sha, commit: {message, author}, verification, files}) => ({
+      .filter(({parents}) => parents.length <= 1)
+      .map(({ sha, commit: { message, author }, verification, files }) => ({
         sha,
         name: `${message} (authored by ${author.name} on ${author.date})`,
         verified: verification?.verified ?? null,
@@ -141,7 +139,6 @@ export class RecentAnalyzer extends Analyzer {
           return edition
         }),
       }))
-
     return patches
   }