test: cover pr-outcomes CLI edge branches and fix draft-pr-body fixture

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: RealDiligent

test/unit/mcp-cli-draft-pr-body.test.ts

@@ -142,7 +142,7 @@ describe("loopover_draft_pr_body stdio mirror (#6741)", () => {
     const data = structured(result);
     expect(data).toMatchObject({
       title: "Local branch preflight",
-      repoFullName: "JSONbored/gittensory",
+      repoFullName: "JSONbored/loopover",
       sourceUploadDisabled: true,
     });
     expect(typeof data.markdown).toBe("string");

test/unit/mcp-cli-pr-outcomes.test.ts

@@ -124,6 +124,26 @@ describe("loopover-mcp pr-outcomes CLI (#6747)", () => {
     });
     expect(badLimit.status).toBe(1);
     expect(`${badLimit.stdout}${badLimit.stderr}`).toMatch(/integer between 1 and 100/);
+
+    const bareLimit = runExpectingFailure(["pr-outcomes", "--login", "JSONbored", "--limit", "101"], {
+      LOOPOVER_API_URL: apiUrl,
+      LOOPOVER_TOKEN: "session-token",
+    });
+    expect(bareLimit.status).toBe(1);
+  });
+
+  it("falls back when the API omits summary and prints null pull numbers / empty attributions", async () => {
+    await closeFixtureServer();
+    const sparseUrl = await startFixtureServer({
+      prOutcomes: {
+        summary: "   ",
+        outcomes: [{ repoFullName: "a/b", pullNumber: null, outcome: "merged", attribution: "", deeplink: "https://x", recordedAt: "t" }],
+      },
+    });
+    const env = { LOOPOVER_API_URL: sparseUrl, LOOPOVER_TOKEN: "session-token" };
+    const plain = await runAsync(["pr-outcomes", "--login", "JSONbored"], env);
+    expect(plain).toContain("LoopOver post-merge outcomes for JSONbored.");
+    expect(plain).toContain("a/b#? [merged]");
   });
 
   it("strips ANSI escapes from API-chosen text on the plain-text path but not from --json", async () => {
@@ -142,6 +162,15 @@ describe("loopover-mcp pr-outcomes CLI (#6747)", () => {
     expect(JSON.parse(asJson).summary).toBe("\u001b[31mFAKE PASS\u001b[0m");
   });
 
+  it("ignores a bare --limit flag (no value) and still returns outcomes", async () => {
+    const out = await runAsync(["pr-outcomes", "--login", "JSONbored", "--limit", "--json"], {
+      LOOPOVER_API_URL: apiUrl,
+      LOOPOVER_TOKEN: "session-token",
+    });
+    expect(JSON.parse(out)).toEqual(prOutcomesFixture());
+    expect(capturedRequests.at(-1)?.url).not.toContain("limit=");
+  });
+
   it("documents itself in --help and in the shell-completion command list", () => {
     expect(run(["--help"])).toContain("loopover-mcp pr-outcomes --login <github-login> [--limit N] [--json]");
     expect(run(["pr-outcomes", "--help"])).toContain("Mirrors the loopover_pr_outcome MCP tool");

test/unit/routes-pr-outcomes.test.ts

@@ -64,6 +64,19 @@ describe("GET /v1/contributors/:login/pr-outcomes (#6747)", () => {
     expect(notInt.status).toBe(400);
   });
 
+  it("returns an empty outcomes list when the contributor has no merged-PR deliveries", async () => {
+    const app = createApp();
+    const env = createTestEnv();
+    const response = await app.request("/v1/contributors/miner/pr-outcomes", { headers: apiHeaders(env) }, env);
+    expect(response.status).toBe(200);
+    await expect(response.json()).resolves.toEqual({
+      login: "miner",
+      count: 0,
+      summary: "LoopOver post-merge outcomes for miner: 0 merged PR(s).",
+      outcomes: [],
+    });
+  });
+
   it("rejects unauthenticated callers", async () => {
     const app = createApp();
     const env = createTestEnv();