refactor: ♻️ Fetch issue if in db

Author: slugb0t

ui/server/services/dashboard/manager.ts

@@ -377,19 +377,13 @@ async function _renderAndUpsertFromDbState(
   let openIssue: IssueRef | null = null;
 
   if (installation?.issue_number) {
-    try {
-      const issues = await provider.listIssues(owner, repo, {
-        creator: provider.getBotLogin(),
-        state: "open",
-      });
-      const existing = issues.find(
-        (i) => i.number === installation.issue_number && i.state === "open",
-      );
-      if (existing) {
-        openIssue = existing;
-      }
-    } catch {
-      // Continue without a cached open-issue reference
+    const fetched = await provider.getIssue(
+      owner,
+      repo,
+      installation.issue_number,
+    );
+    if (fetched?.state === "open") {
+      openIssue = fetched;
     }
   }
 

ui/server/services/providers/github.ts

@@ -398,6 +398,36 @@ export class GitHubRepositoryProvider implements RepositoryProvider {
     }));
   }
 
+  /**
+   * Fetches a single issue by number.
+   * @param owner - The repository owner.
+   * @param repo - The repository name.
+   * @param issueNumber - The issue number to fetch.
+   * @returns The issue reference, or null if the issue does not exist.
+   */
+  async getIssue(
+    owner: string,
+    repo: string,
+    issueNumber: number,
+  ): Promise<IssueRef | null> {
+    try {
+      const { data } = await this.octokit.request(
+        "GET /repos/{owner}/{repo}/issues/{issue_number}",
+        { headers, issue_number: issueNumber, owner, repo },
+      );
+      return {
+        title: data.title,
+        body: data.body ?? null,
+        htmlUrl: data.html_url,
+        number: data.number,
+        state: data.state as "open" | "closed",
+        userLogin: data.user?.login ?? null,
+      };
+    } catch {
+      return null; // 404 — issue was deleted
+    }
+  }
+
   /**
    * Creates a new issue in the repository.
    * @param owner - The repository owner.

ui/server/services/providers/interface.ts

@@ -171,6 +171,12 @@ export interface RepositoryProvider {
 
   // Issues
 
+  getIssue(
+    owner: string,
+    repo: string,
+    issueNumber: number,
+  ): Promise<IssueRef | null>;
+
   listIssues(
     owner: string,
     repo: string,