test(review): close 2 coverage gaps surfaced by the #4514 rebase-merge

getEffectiveSubmitterReputation's getRepository().catch() needed a real
read-failure test (added); its isConfirmedOfficialMiner().catch() is
unreachable (that function already catches every internal failure point
itself) and getSubmitterReputationAcrossInstall's results ?? [] fallback is
the same "D1 always populates results" case already v8-ignored elsewhere in
this codebase -- both marked accordingly.

Author: JSONbored

src/review/reputation-wire.ts

@@ -78,6 +78,7 @@ export async function getEffectiveSubmitterReputation(
   if (shouldDowngradeToDeterministic(perRepo)) return perRepo;
   const submitter = args.submitter?.trim();
   if (!submitter) return perRepo;
+  /* v8 ignore next -- isConfirmedOfficialMiner already catches every internal failure point itself and never rejects; this guards only a future implementation change. */
   const isMiner = await isConfirmedOfficialMiner(env, submitter).catch(() => false);
   if (!isMiner) return perRepo;
   const repo = await getRepository(env, args.repoFullName).catch(() => null);

src/review/submitter-reputation.ts

@@ -339,6 +339,7 @@ export async function getSubmitterReputationAcrossInstall(
       )
       .bind(installationId, submitter, `-${cfg.windowDays} days`, REPUTATION_WINDOW_ROW_CAP)
       .all<{ status: string; reasonCode: string | null }>();
+    /* v8 ignore next -- D1's .all() always populates results; the fallback only protects a driver anomaly. */
     const rows = result?.results ?? [];
     signal = signalFromCounts(countOutcomes(rows), cfg);
   } catch {

test/unit/reputation-wiring.test.ts

@@ -352,6 +352,23 @@ describe("getEffectiveSubmitterReputation (#4513, install-wide for a confirmed m
       vi.unstubAllGlobals();
     }
   });
+
+  it("fails safe to the per-repo result, without throwing, when the getRepository read itself errors", async () => {
+    const env = createTestEnv();
+    await seedReviewTarget(env, { project: "org/repo-a", repo: "org/repo-a", number: 1, installationId: 999, submitter: "farmer99", status: "closed", reasonCode: "dual_review_declined" });
+    vi.stubGlobal("fetch", stubMinerFetch("farmer99"));
+    const realPrepare = env.DB.prepare.bind(env.DB);
+    env.DB.prepare = ((sql: string) => {
+      if (/FROM.*"?repositories"?/i.test(sql)) throw new Error("d1 down");
+      return realPrepare(sql);
+    }) as typeof env.DB.prepare;
+    try {
+      const rep = await getEffectiveSubmitterReputation(env, { repoFullName: "org/repo-a", submitter: "farmer99" });
+      expect(rep.signal).toBe("neutral");
+    } finally {
+      vi.unstubAllGlobals();
+    }
+  });
 });
 
 describe("processGitHubWebhook records the reputation outcome on a terminal PR (flag-ON call site)", () => {