pr-shepherd 0.22.0 → 0.24.0

package/bin/comments/review-thread-markers.mjs

@@ -0,0 +1,18 @@
+import { markReviewInlineThreads } from "../state/seen-comments.mjs";
+function groupInlineThreadIdsByReview(threads) {
+    const byReview = new Map();
+    for (const thread of threads) {
+        if (thread.reviewId === undefined)
+            continue;
+        const ids = byReview.get(thread.reviewId) ?? new Set();
+        ids.add(thread.id);
+        byReview.set(thread.reviewId, ids);
+    }
+    return new Map([...byReview.entries()].map(([reviewId, ids]) => [
+        reviewId,
+        [...ids].sort((a, b) => a.localeCompare(b)),
+    ]));
+}
+export async function markReviewInlineThreadMarkers(key, threads) {
+    await Promise.allSettled([...groupInlineThreadIdsByReview(threads)].map(([reviewId, inlineThreadIds]) => markReviewInlineThreads(key, reviewId, inlineThreadIds)));
+}

package/bin/comments/review-visibility.mjs

@@ -0,0 +1,14 @@
+import { classifyItem } from "../state/seen-comments.mjs";
+export function classifyReviewsForDisplay(reviews, seenMap) {
+    const visible = [];
+    const toMarkSeen = [];
+    for (const review of reviews) {
+        const cls = classifyItem(review.id, review.body, seenMap);
+        if (cls === "unchanged")
+            continue;
+        const rendered = cls === "edited" ? { ...review, edited: true } : review;
+        visible.push(rendered);
+        toMarkSeen.push(review);
+    }
+    return { visible, toMarkSeen };
+}

package/bin/comments/thread-visibility.mjs

@@ -0,0 +1,60 @@
+import { classifyItem } from "../state/seen-comments.mjs";
+import { threadTranscriptBody } from "../threads/transcript.mjs";
+import { isConfiguredBotAuthor } from "./authors.mjs";
+function withEdited(thread, edited) {
+    return edited ? { ...thread, edited: true } : thread;
+}
+function classifyVisibleThread(thread, seenMap) {
+    const cls = classifyItem(thread.id, threadTranscriptBody(thread), seenMap);
+    if (cls === "unchanged")
+        return null;
+    return withEdited(thread, cls === "edited");
+}
+function classifyFirstLookThread(thread, seenMap, firstLookStatus) {
+    const visible = classifyVisibleThread(thread, seenMap);
+    if (visible === null)
+        return null;
+    return { ...visible, firstLookStatus };
+}
+export function classifyThreadVisibility(threads, seenMap, botUsernames = new Set()) {
+    const unresolvedThreads = threads.filter((t) => !t.isResolved);
+    const activeThreads = unresolvedThreads
+        .filter((t) => !t.isOutdated && !t.isMinimized)
+        .flatMap((t) => {
+        if (isConfiguredBotAuthor(t, botUsernames))
+            return [t];
+        const visible = classifyVisibleThread(t, seenMap);
+        return visible ? [visible] : [];
+    });
+    const resolutionOnlyThreads = unresolvedThreads
+        .filter((t) => t.isOutdated || t.isMinimized)
+        .map((t) => classifyVisibleThread(t, seenMap) ?? t);
+    const firstLookThreads = [
+        ...threads.flatMap((t) => {
+            if (!t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "outdated");
+            return visible ? [visible] : [];
+        }),
+        ...threads.flatMap((t) => {
+            if (!t.isResolved || t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "resolved");
+            return visible ? [visible] : [];
+        }),
+        ...threads.flatMap((t) => {
+            if (!t.isMinimized || t.isResolved || t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "minimized");
+            return visible ? [visible] : [];
+        }),
+    ];
+    const seenMarkerIds = new Set();
+    const toMarkSeen = [...activeThreads, ...resolutionOnlyThreads, ...firstLookThreads].filter((thread) => {
+        if (seenMarkerIds.has(thread.id))
+            return false;
+        seenMarkerIds.add(thread.id);
+        return true;
+    });
+    return { activeThreads, resolutionOnlyThreads, firstLookThreads, toMarkSeen };
+}

package/bin/comments/visible-comments.mjs

@@ -1,11 +1,11 @@
 import { shouldMinimizeAuthor } from "./minimize-policy.mjs";
 import { classifyItem } from "../state/seen-comments.mjs";
-export function classifyVisibleComments(comments, seenMap, minimizeComments) {
+export function classifyVisibleComments(comments, seenMap, minimizeComments, botUsernames = new Set()) {
     const actionable = [];
     const minimizeIds = [];
     const toMarkSeen = [];
     for (const c of comments.filter((comment) => !comment.isMinimized)) {
-        if (shouldMinimizeAuthor(c.authorType, minimizeComments)) {
+        if (shouldMinimizeAuthor(c.authorType, minimizeComments, c.author, botUsernames)) {
             actionable.push(c);
             minimizeIds.push(c.id);
             continue;

package/bin/config/load.mjs

@@ -45,6 +45,12 @@ function parseMinimizeCommentsPolicy(value) {
         return value;
     throw new Error(`Invalid config: iterate.minimizeComments must be one of "all", "bots", "users", or "none", got ${JSON.stringify(value)}`);
 }
+function parseBotUsernames(value) {
+    if (!Array.isArray(value) || !value.every((item) => typeof item === "string")) {
+        throw new Error(`Invalid config: botUsernames must be an array of strings`);
+    }
+    return value;
+}
 const defaults = builtins;
 const configCache = new Map();
 export function loadConfig() {
@@ -60,6 +66,7 @@ export function loadConfig() {
         const raw = readFileSync(rcPath, "utf8");
         const parsed = (parse(raw) ?? {});
         const config = deepMerge(defaults, parsed);
+        config.botUsernames = parseBotUsernames(config.botUsernames);
         config.iterate.minimizeComments = parseMinimizeCommentsPolicy(config.iterate.minimizeComments);
         configCache.set(cwd, config);
         return config;

package/bin/config.json

@@ -1,4 +1,18 @@
 {
+  "botUsernames": [
+    "chatgpt-connector",
+    "openai-codex",
+    "codex",
+    "claude",
+    "copilot-pull-request-reviewer",
+    "gemini-code-assist",
+    "coderabbitai",
+    "greptile-apps",
+    "qodo-merge-pro",
+    "deepsource-io",
+    "sonarqubecloud",
+    "what-the-diff"
+  ],
   "iterate": {
     "fixAttemptsPerThread": 3,
     "stallTimeoutMinutes": 60,

package/bin/github/batch-parser-helpers.mjs

@@ -1,7 +1,6 @@
-export function mapAuthorType(typeName) {
-    if (typeName === "User" || typeName === "Bot")
-        return typeName;
-    return "Unknown";
+import { normalizeAuthorType } from "../comments/authors.mjs";
+export function mapAuthorType(typeName, login) {
+    return normalizeAuthorType(typeName, login);
 }
 export function parseCreatedAt(iso) {
     const ms = new Date(iso).getTime();

package/bin/github/batch-parsers.mjs

@@ -13,8 +13,9 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         const comments = t.comments.nodes.map((c) => ({
             id: c.id,
             isMinimized: c.isMinimized,
+            ...(c.pullRequestReview?.id ? { reviewId: c.pullRequestReview.id } : undefined),
             author: c.author?.login ?? "unknown",
-            authorType: mapAuthorType(c.author?.__typename),
+            authorType: mapAuthorType(c.author?.__typename, c.author?.login),
             body: c.body,
             url: c.url,
             createdAtUnix: parseCreatedAt(c.createdAt),
@@ -27,8 +28,9 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
             path: t.path ?? comment?.path ?? null,
             line: t.line ?? comment?.line ?? null,
             startLine: t.startLine ?? comment?.startLine ?? null,
+            ...(comment?.pullRequestReview?.id ? { reviewId: comment.pullRequestReview.id } : undefined),
             author: comment?.author?.login ?? "unknown",
-            authorType: mapAuthorType(comment?.author?.__typename),
+            authorType: mapAuthorType(comment?.author?.__typename, comment?.author?.login),
             body: comment?.body ?? "",
             url: comment?.url ?? "",
             createdAtUnix: comment ? parseCreatedAt(comment.createdAt) : 0,
@@ -39,7 +41,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         id: c.id,
         isMinimized: c.isMinimized,
         author: c.author?.login ?? "unknown",
-        authorType: mapAuthorType(c.author?.__typename),
+        authorType: mapAuthorType(c.author?.__typename, c.author?.login),
         body: c.body,
         url: c.url,
         createdAtUnix: parseCreatedAt(c.createdAt),
@@ -47,7 +49,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
     const changesRequestedReviews = rawReviewNodes.map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     const reviewSummaries = rawReviewSummaryNodes
@@ -55,7 +57,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         .map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     // APPROVED reviews often have empty bodies (clicking "Approve" without a comment), so
@@ -66,7 +68,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         .map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     const checks = rawCheckNodes.flatMap((node) => {

package/bin/github/client.mjs

@@ -7,9 +7,8 @@
  */
 import { execFile as execFileCb } from "node:child_process";
 import { promisify } from "node:util";
-import { graphql as httpGraphql, rest, GitHubRequestError, } from "./http.mjs";
+import { graphql as httpGraphql, rest } from "./http.mjs";
 import { PR_NUMBER_BY_BRANCH_QUERY, GET_PR_HEAD_SHA_QUERY } from "./queries.mjs";
-export { GitHubRequestError };
 const execFile = promisify(execFileCb);
 // ---------------------------------------------------------------------------
 // GraphQL — thin re-exports so callers don't need to import http.mts directly

package/bin/github/gql/batch-pr.gql

@@ -83,6 +83,9 @@ query BatchPr(
                 __typename
                 login
               }
+              pullRequestReview {
+                id
+              }
               body
               path
               line

package/bin/reporters/agent.mjs

@@ -13,6 +13,7 @@ export function toAgentThread(t) {
     const suggestion = extractSuggestion(t) ?? undefined;
     return {
         id: t.id,
+        ...(t.reviewId !== undefined && { reviewId: t.reviewId }),
         path: t.path,
         line: t.line,
         ...(t.line !== null &&
@@ -22,6 +23,7 @@ export function toAgentThread(t) {
         ...(t.authorType !== undefined && { authorType: t.authorType }),
         body: t.body,
         url: t.url,
+        ...(t.edited === true && { edited: true }),
         ...(t.comments !== undefined && {
             comments: t.comments.map((c) => ({
                 id: c.id,

package/bin/state/seen-comments.mjs

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 import { readFile, writeFile, rename, unlink, mkdir, access, readdir } from "node:fs/promises";
 import { join, dirname } from "node:path";
 import { createHash, randomUUID } from "node:crypto";
@@ -23,8 +24,14 @@ export function classifyItem(id, body, map) {
     const m = map.get(id);
     if (!m)
         return "new";
-    if (typeof m.bodyHash === "string" && m.bodyHash !== hashBody(body))
+    const currentHash = hashBody(body);
+    if (typeof m.previousBodyHash === "string" && m.previousBodyHash === currentHash) {
+        return "unchanged";
+    }
+    if (typeof m.bodyHash === "string" && m.bodyHash !== currentHash)
         return "edited";
+    if (m.bodyHash === undefined && Array.isArray(m.inlineThreadIds))
+        return "new";
     return "unchanged";
 }
 /**
@@ -94,11 +101,31 @@ export async function hasSeen(key, id) {
  * All errors are silently swallowed — the marker is best-effort.
  */
 export async function markSeen(key, id, body) {
+    await writeSeenMarker(key, id, { bodyHash: hashBody(body) });
+}
+export async function markReviewInlineThreads(key, reviewId, inlineThreadIds) {
+    await writeSeenMarker(key, reviewId, {
+        inlineThreadIds: [...new Set(inlineThreadIds)].sort((a, b) => a.localeCompare(b)),
+    });
+}
+/**
+ * Write a marker after Shepherd successfully replies to a review thread.
+ *
+ * `previousBody` suppresses stale GitHub fetches that have not yet included the new reply.
+ * `body` suppresses the expected final transcript once GitHub includes Shepherd's reply.
+ */
+export async function markReplySeen(key, id, previousBody, body, replyBody) {
+    await writeSeenMarker(key, id, {
+        bodyHash: hashBody(body),
+        previousBodyHash: hashBody(previousBody),
+        replyBodyHash: hashBody(replyBody),
+    });
+}
+async function writeSeenMarker(key, id, markerFields) {
     let tmp;
     try {
         const path = resolvePath(key, id);
         await mkdir(dirname(path), { recursive: true });
-        const newHash = hashBody(body);
         let existing = null;
         try {
             const raw = await readFile(path, "utf8");
@@ -107,14 +134,15 @@ export async function markSeen(key, id, body) {
         catch {
             // no existing marker — will create below
         }
-        if (existing !== null && existing.bodyHash === newHash)
+        const unchanged = Object.entries(markerFields).every(([field, value]) => markerFieldEqual(existing?.[field], value));
+        if (existing !== null && unchanged)
             return;
         const seenAt = existing?.seenAt ?? Date.now();
         tmp = `${path}.${randomUUID()}.tmp`;
         // Store `id` in the payload so loadSeenMap can key by the original ID
         // rather than the filename, guarding against case-insensitive filesystems
         // (e.g. macOS APFS) where IDs differing only in case would collide.
-        await writeFile(tmp, JSON.stringify({ seenAt, bodyHash: newHash, id }), "utf8");
+        await writeFile(tmp, JSON.stringify({ ...existing, seenAt, ...markerFields, id }), "utf8");
         await rename(tmp, path);
         tmp = undefined;
     }
@@ -132,6 +160,14 @@ export async function markSeen(key, id, body) {
         }
     }
 }
+function markerFieldEqual(left, right) {
+    if (Object.is(left, right))
+        return true;
+    if (Array.isArray(left) && Array.isArray(right)) {
+        return left.length === right.length && left.every((value, index) => value === right[index]);
+    }
+    return false;
+}
 /** Read the full marker for inspection (returns null on miss or error). */
 export async function readSeenMarker(key, id) {
     try {

package/bin/threads/transcript.mjs

@@ -22,10 +22,10 @@ export function threadComments(thread) {
         },
     ];
 }
-export function threadTranscriptBody(thread) {
-    if (!thread.comments || thread.comments.length === 0)
-        return thread.body;
-    return threadComments(thread)
-        .map((c) => `${c.id}\n${c.body}`)
-        .join("\n\n--- thread comment ---\n\n");
+const THREAD_COMMENT_SEPARATOR = "\n\n--- thread comment ---\n\n";
+export function threadTranscriptBody(thread, appendedBodies = []) {
+    const bodies = thread.comments && thread.comments.length > 0
+        ? threadComments(thread).map((c) => c.body)
+        : [thread.body];
+    return [...bodies, ...appendedBodies].join(THREAD_COMMENT_SEPARATOR);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.22.0",
+  "version": "0.24.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "license": "MIT",
   "author": "Jonathan Ong",
@@ -28,6 +28,7 @@
     "@types/node": "^25.6.0",
     "@vitest/coverage-v8": "^4.1.4",
     "husky": "^9.1.7",
+    "knip": "^5.88.1",
     "oxfmt": "^0.48.0",
     "oxlint": "^1.60.0",
     "typescript": "^6.0.3",
@@ -38,9 +39,12 @@
     "prepare": "node scripts/install-husky.mjs && npm run build",
     "prepublishOnly": "npm run typecheck && npm test && npm run build",
     "typecheck": "tsc --noEmit",
-    "lint": "oxlint src/ plugins/ .agents/plugins/",
-    "format": "oxfmt src/ plugins/ .agents/plugins/ docs/ README.md",
-    "format:check": "oxfmt --check src/ plugins/ .agents/plugins/ docs/ README.md",
+    "knip": "knip",
+    "knip:production": "knip --production",
+    "lint:dead-code": "npm run --silent knip && npm run --silent knip:production",
+    "lint": "oxlint src/ test-helpers/ fixtures/*.mts test-cases/ plugins/ .agents/plugins/ && npm run --silent lint:dead-code",
+    "format": "oxfmt src/ test-helpers/ fixtures/*.mts test-cases/*.mts plugins/ .agents/plugins/ docs/ README.md",
+    "format:check": "oxfmt --check src/ test-helpers/ fixtures/*.mts test-cases/*.mts plugins/ .agents/plugins/ docs/ README.md",
     "test": "vitest run",
     "test:coverage": "vitest run --coverage && node scripts/strip-lcov-branches.mjs",
     "test:watch": "vitest"

package/plugins/pr-shepherd/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.22.0",
+  "version": "0.24.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for Codex.",
   "author": {
     "name": "Jonathan Ong",

package/plugins/pr-shepherd/skills/mark-files-as-viewed/SKILL.md

@@ -0,0 +1,31 @@
+---
+name: mark-files-as-viewed
+description: 'Mark PR changed files as viewed in GitHub with pr-shepherd. Use for requests like "mark tests as viewed" or "mark these files as viewed".'
+user-invocable: true
+argument-hint: "[PR number or URL] [files|tests|--tests|--match REGEX]"
+allowed-tools: ["Bash", "Read", "Grep", "Glob"]
+---
+
+# mark-files-as-viewed
+
+Thin dispatcher for marking GitHub PR changed files as viewed.
+
+## Arguments: $ARGUMENTS
+
+## Steps
+
+1. **Resolve arguments:**
+   - If `$ARGUMENTS` contains a PR number, use it.
+   - If `$ARGUMENTS` contains a GitHub PR URL, extract the number.
+   - Otherwise, infer: `gh pr view --json number --jq .number`
+   - Treat a standalone `tests` argument as `--tests`.
+   - Preserve explicit file paths, `--tests`, and `--match <regex>` arguments.
+   - If no PR found, report an error and stop.
+
+2. **Run `pr-shepherd`:**
+
+   ```bash
+   pr-shepherd mark-files-as-viewed <N> <selectors>
+   ```
+
+   Print the full output.

package/bin/checks/triage.test-support.mjs

@@ -1,60 +0,0 @@
-import { vi, beforeEach } from "vitest";
-// ---------------------------------------------------------------------------
-// Stub fetch globally so http.mts uses our mock.
-// ---------------------------------------------------------------------------
-const mockFetch = vi.fn();
-vi.stubGlobal("fetch", mockFetch);
-import { fetchStartupFailureChecks, triageFailingChecks } from "./triage.mjs";
-import { mergeStartupFailureChecks } from "./startup-failures.mjs";
-const REPO = { owner: "owner", name: "repo" };
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-function makeCheck(overrides = {}) {
-    return {
-        name: "tests",
-        status: "COMPLETED",
-        conclusion: "FAILURE",
-        detailsUrl: "https://github.com/owner/repo/actions/runs/99/jobs/1",
-        event: "pull_request",
-        runId: "run-99",
-        category: "failing",
-        ...overrides,
-    };
-}
-function makeJobsResponse(jobs) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve({ jobs }),
-        text: () => Promise.resolve(JSON.stringify({ jobs })),
-    };
-}
-function makeErrorResponse(status) {
-    return {
-        ok: false,
-        status,
-        headers: new Headers(),
-        text: () => Promise.resolve("error"),
-    };
-}
-function makeWorkflowRunsResponse(runs) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve({ workflow_runs: runs }),
-        text: () => Promise.resolve(JSON.stringify({ workflow_runs: runs })),
-    };
-}
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        mockFetch.mockReset();
-        process.env["GH_TOKEN"] = "test-token";
-    });
-}
-export { REPO, fetchStartupFailureChecks, makeCheck, makeErrorResponse, makeJobsResponse, makeWorkflowRunsResponse, mergeStartupFailureChecks, mockFetch, triageFailingChecks, };

package/bin/cli/iterate-lean.test-support.mjs

@@ -1,3 +0,0 @@
-import { projectIterateLean, projectIterateVerbose } from "./iterate-lean.mjs";
-import { makeIterateResult } from "../cli-parser.iterate-fixtures.mjs";
-export { makeIterateResult, projectIterateLean, projectIterateVerbose };

package/bin/cli-parser.clean.test-support.mjs

@@ -1,44 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-vi.mock("./commands/clean.mts", () => ({
-    runClean: vi.fn(),
-}));
-vi.mock("./commands/resolve.mts", () => ({
-    runResolveFetch: vi.fn(),
-    runResolveMutate: vi.fn(),
-}));
-vi.mock("./commands/log-file.mts", () => ({
-    runLogFile: vi.fn(),
-}));
-vi.mock("./commands/commit-suggestion.mts", () => ({
-    runCommitSuggestion: vi.fn(),
-}));
-vi.mock("./commands/iterate/index.mts", async (importOriginal) => {
-    const actual = await importOriginal();
-    return { ...actual, runIterate: vi.fn() };
-});
-import { main } from "./cli-parser.mjs";
-import { runClean } from "./commands/clean.mjs";
-export const mockRunClean = vi.mocked(runClean);
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let stdoutSpy;
-let stderrSpy;
-export function getStdout() {
-    return stdoutSpy.mock.calls.map((c) => c[0]).join("");
-}
-export function getStderr() {
-    return stderrSpy.mock.calls.map((c) => c[0]).join("");
-}
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        process.exitCode = undefined;
-        stdoutSpy = vi.spyOn(process.stdout, "write").mockImplementation(() => true);
-        stderrSpy = vi.spyOn(process.stderr, "write").mockImplementation(() => true);
-    });
-    afterEach(() => {
-        process.exitCode = undefined;
-        stdoutSpy.mockRestore();
-        stderrSpy.mockRestore();
-    });
-}
-export { main, stderrSpy, stdoutSpy };

package/bin/cli-parser.commit-suggestion.test-support.mjs

@@ -1,65 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-vi.mock("./commands/check.mts", () => ({ runCheck: vi.fn() }));
-vi.mock("./commands/resolve.mts", () => ({
-    runResolveFetch: vi.fn(),
-    runResolveMutate: vi.fn(),
-}));
-vi.mock("./commands/commit-suggestion.mts", () => ({
-    runCommitSuggestion: vi.fn(),
-}));
-vi.mock("./commands/iterate/index.mts", async (importOriginal) => {
-    const actual = await importOriginal();
-    return { ...actual, runIterate: vi.fn() };
-});
-vi.mock("./github/client.mts", () => ({
-    getRepoInfo: vi.fn().mockResolvedValue({ owner: "owner", name: "repo" }),
-}));
-import { main } from "./cli-parser.mjs";
-import { runCommitSuggestion } from "./commands/commit-suggestion.mjs";
-const mockRunCommitSuggestion = vi.mocked(runCommitSuggestion);
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let stdoutSpy;
-let stderrSpy;
-function getStdout() {
-    return stdoutSpy.mock.calls.map((c) => c[0]).join("");
-}
-// ---------------------------------------------------------------------------
-// Fixtures
-// ---------------------------------------------------------------------------
-const SUGGESTION_RESULT = {
-    pr: 42,
-    repo: "owner/repo",
-    threadId: "t1",
-    path: "a.ts",
-    startLine: 5,
-    endLine: 5,
-    author: "alice",
-    patch: "--- a/a.ts\n+++ b/a.ts\n@@ -5,1 +5,1 @@\n-old\n+new\n",
-    commitMessage: "apply fix",
-    commitBody: "Co-authored-by: alice <alice@users.noreply.github.com>",
-    filesToStage: ["a.ts"],
-    postActionInstructions: [
-        "Apply the patch to `a.ts`: run `git apply` with the diff shown above.",
-        "Stage the file: `git add -- a.ts`",
-        'Commit: `git commit -m "apply fix" -m "Co-authored-by: alice <alice@users.noreply.github.com>"`',
-        "Resolve the thread on GitHub: `pr-shepherd resolve 42 --resolve-thread-ids t1`",
-        "Push when ready: `git push` (or `git push --force-with-lease` after rebasing).",
-    ],
-};
-// ---------------------------------------------------------------------------
-// commit-suggestion dispatch
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        process.exitCode = undefined;
-        stdoutSpy = vi.spyOn(process.stdout, "write").mockImplementation(() => true);
-        stderrSpy = vi.spyOn(process.stderr, "write").mockImplementation(() => true);
-    });
-    afterEach(() => {
-        process.exitCode = undefined;
-        stdoutSpy.mockRestore();
-        stderrSpy.mockRestore();
-    });
-}
-export { SUGGESTION_RESULT, getStdout, main, mockRunCommitSuggestion, runCommitSuggestion, stderrSpy, stdoutSpy, };

package/bin/cli-parser.iterate-fix.test-support.mjs

@@ -1,43 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-vi.mock("./commands/check.mts", () => ({ runCheck: vi.fn() }));
-vi.mock("./commands/resolve.mts", () => ({
-    runResolveFetch: vi.fn(),
-    runResolveMutate: vi.fn(),
-}));
-vi.mock("./commands/commit-suggestion.mts", () => ({
-    runCommitSuggestion: vi.fn(),
-}));
-vi.mock("./commands/iterate/index.mts", async (importOriginal) => {
-    const actual = await importOriginal();
-    return { ...actual, runIterate: vi.fn() };
-});
-vi.mock("./github/client.mts", () => ({
-    getRepoInfo: vi.fn().mockResolvedValue({ owner: "owner", name: "repo" }),
-}));
-import { main } from "./cli-parser.mjs";
-import { runIterate } from "./commands/iterate/index.mjs";
-import { makeIterateResult } from "./cli-parser.iterate-fixtures.mjs";
-const mockRunIterate = vi.mocked(runIterate);
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let stdoutSpy;
-let stderrSpy;
-function getStdout() {
-    return stdoutSpy.mock.calls.map((c) => c[0]).join("");
-}
-// ---------------------------------------------------------------------------
-// formatIterateResult — fix_code actions and ## Checks section
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        process.exitCode = undefined;
-        stdoutSpy = vi.spyOn(process.stdout, "write").mockImplementation(() => true);
-        stderrSpy = vi.spyOn(process.stderr, "write").mockImplementation(() => true);
-    });
-    afterEach(() => {
-        process.exitCode = undefined;
-        stdoutSpy.mockRestore();
-        stderrSpy.mockRestore();
-    });
-}
-export { getStdout, main, makeIterateResult, mockRunIterate, runIterate, stderrSpy, stdoutSpy };