pr-shepherd 0.23.0 → 0.25.0

package/bin/comments/thread-visibility.mjs

@@ -1,5 +1,6 @@
 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;
 }
@@ -15,20 +16,19 @@ function classifyFirstLookThread(thread, seenMap, firstLookStatus) {
         return null;
     return { ...visible, firstLookStatus };
 }
-export function classifyThreadVisibility(threads, seenMap) {
+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)
-        .flatMap((t) => {
-        const visible = classifyVisibleThread(t, seenMap);
-        return visible ? [visible] : [];
-    });
+        .map((t) => classifyVisibleThread(t, seenMap) ?? t);
     const firstLookThreads = [
         ...threads.flatMap((t) => {
             if (!t.isOutdated)

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, c.author)) {
+        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,
@@ -12,8 +26,7 @@
     "shaPoll": {
       "intervalMs": 2000,
       "maxAttempts": 10
-    },
-    "fetchReviewSummaries": true
+    }
   },
   "checks": {
     "ciTriggerEvents": ["pull_request", "pull_request_target"]

package/bin/github/batch-parsers.mjs

@@ -13,6 +13,7 @@ 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, c.author?.login),
             body: c.body,
@@ -27,6 +28,7 @@ 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, comment?.author?.login),
             body: comment?.body ?? "",

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 &&

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.23.0",
+  "version": "0.25.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "license": "MIT",
   "author": "Jonathan Ong",
@@ -28,7 +28,8 @@
     "@types/node": "^25.6.0",
     "@vitest/coverage-v8": "^4.1.4",
     "husky": "^9.1.7",
-    "oxfmt": "^0.48.0",
+    "knip": "^6.14.1",
+    "oxfmt": "^0.50.0",
     "oxlint": "^1.60.0",
     "typescript": "^6.0.3",
     "vitest": "^4.1.4"
@@ -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.23.0",
+  "version": "0.25.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for Codex.",
   "author": {
     "name": "Jonathan Ong",

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 };

package/bin/cli-parser.iterate-fixtures.mjs

@@ -1,75 +0,0 @@
-export function makeIterateResult(action = "wait") {
-    const base = {
-        pr: 42,
-        repo: "owner/repo",
-        status: "IN_PROGRESS",
-        state: "OPEN",
-        mergeStateStatus: "BLOCKED",
-        mergeStatus: "BLOCKED",
-        reviewDecision: null,
-        blockingBotReviewInProgress: false,
-        isDraft: false,
-        shouldCancel: false,
-        remainingSeconds: 60,
-        summary: { passing: 0, skipped: 0, filtered: 0, inProgress: 1 },
-        baseBranch: "main",
-        branchProtection: null,
-        checks: [],
-    };
-    if (action === "wait")
-        return { ...base, action: "wait", log: "WAIT: 0 passing, 1 in-progress" };
-    if (action === "mark_ready")
-        return { ...base, action: "mark_ready", markedReady: true, log: "MARKED READY: PR 42" };
-    if (action === "fix_code") {
-        return {
-            ...base,
-            action: "fix_code",
-            fix: {
-                threads: [],
-                resolutionOnlyThreads: [],
-                actionableComments: [],
-                reviewSummaryIds: [],
-                firstLookSummaries: [],
-                editedSummaries: [],
-                surfacedApprovals: [],
-                checks: [],
-                changesRequestedReviews: [],
-                resolveCommand: {
-                    argv: ["pr-shepherd", "resolve", "42"],
-                    requiresHeadSha: true,
-                    requiresDismissMessage: false,
-                    hasMutations: false,
-                },
-                instructions: [
-                    "Stop this iteration — if you pushed new commits, CI needs time before the next tick; otherwise stop before the next tick.",
-                ],
-                firstLookThreads: [],
-                firstLookComments: [],
-                inProgressRunIds: [],
-            },
-            cancelled: [],
-        };
-    }
-    if (action === "cancel")
-        return {
-            ...base,
-            action: "cancel",
-            reason: "ready-delay-elapsed",
-            log: "CANCEL: PR #42 — stopping",
-        };
-    if (action === "escalate") {
-        return {
-            ...base,
-            action: "escalate",
-            escalate: {
-                triggers: [],
-                unresolvedThreads: [],
-                ambiguousComments: [],
-                changesRequestedReviews: [],
-                suggestion: "check manually",
-                humanMessage: "⚠️ /pr-shepherd:pr-shepherd paused — manual intervention required",
-            },
-        };
-    }
-    return { ...base, action: "wait", log: "WAIT: 0 passing, 1 in-progress" };
-}

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

@@ -1,44 +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() };
-});
-import { main } from "./cli-parser.mjs";
-import { runIterate } from "./commands/iterate/index.mjs";
-import { formatIterateResult } from "./cli/iterate-formatter.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("");
-}
-function getStderr() {
-    return stderrSpy.mock.calls.map((c) => c[0]).join("");
-}
-// ---------------------------------------------------------------------------
-// iterate 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 { formatIterateResult, getStderr, getStdout, main, makeIterateResult, mockRunIterate, runIterate, stderrSpy, stdoutSpy, };

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

@@ -1,47 +0,0 @@
-import { readFileSync } from "node:fs";
-import { vi, beforeEach, afterEach } from "vitest";
-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/mark-files-as-viewed.mts", () => ({
-    runMarkFilesAsViewed: 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 { runLogFile } from "./commands/log-file.mjs";
-import { runResolveFetch, runResolveMutate } from "./commands/resolve.mjs";
-import { runMarkFilesAsViewed } from "./commands/mark-files-as-viewed.mjs";
-const mockRunResolveFetch = vi.mocked(runResolveFetch);
-const mockRunResolveMutate = vi.mocked(runResolveMutate);
-const mockRunLogFile = vi.mocked(runLogFile);
-const mockRunMarkFilesAsViewed = vi.mocked(runMarkFilesAsViewed);
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let stdoutSpy;
-let stderrSpy;
-function getStdout() {
-    return stdoutSpy.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 { getStdout, main, mockRunLogFile, mockRunMarkFilesAsViewed, mockRunResolveFetch, mockRunResolveMutate, readFileSync, runLogFile, runResolveFetch, runResolveMutate, stderrSpy, stdoutSpy, };