pr-shepherd 0.23.0 → 0.25.0

package/bin/commands/iterate.fix-code-in-progress.test-support.mjs

@@ -1,127 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-const mockFetch = vi.fn();
-vi.stubGlobal("fetch", mockFetch);
-const { mockExecFile } = vi.hoisted(() => ({ mockExecFile: vi.fn() }));
-vi.mock("node:child_process", () => ({
-    execFile: (cmd, args, optsOrCb, maybeCb) => {
-        const cb = typeof optsOrCb === "function" ? optsOrCb : maybeCb;
-        mockExecFile(cmd, args)
-            .then((result) => cb(null, result))
-            .catch((err) => cb(err, { stdout: "", stderr: err.stderr ?? "" }));
-    },
-}));
-vi.mock("./check.mts", () => ({ runCheck: vi.fn() }));
-vi.mock("./ready-delay.mts", () => ({ updateReadyDelay: vi.fn() }));
-vi.mock("../github/client.mts", () => ({
-    getCurrentPrNumber: vi.fn().mockResolvedValue(42),
-}));
-vi.mock("../state/fix-attempts.mts", () => ({
-    readFixAttempts: vi.fn().mockResolvedValue(null),
-    writeFixAttempts: vi.fn().mockResolvedValue(undefined),
-}));
-vi.mock("../state/iterate-stall.mts", () => ({
-    readStallState: vi.fn().mockResolvedValue(null),
-    writeStallState: vi.fn().mockResolvedValue(undefined),
-}));
-vi.mock("../state/seen-comments.mts", async (importOriginal) => {
-    const actual = await importOriginal();
-    return {
-        ...actual,
-        markSeen: vi.fn().mockResolvedValue(undefined),
-    };
-});
-const { mockLoadConfig } = vi.hoisted(() => ({ mockLoadConfig: vi.fn() }));
-vi.mock("../config/load.mts", () => ({ loadConfig: mockLoadConfig }));
-import { runIterate } from "./iterate/index.mjs";
-import { runCheck } from "./check.mjs";
-import { updateReadyDelay } from "./ready-delay.mjs";
-import { readFixAttempts, writeFixAttempts } from "../state/fix-attempts.mjs";
-import { readStallState, writeStallState } from "../state/iterate-stall.mjs";
-import { markSeen } from "../state/seen-comments.mjs";
-const mockRunCheck = vi.mocked(runCheck);
-const mockUpdateReadyDelay = vi.mocked(updateReadyDelay);
-const mockReadFixAttempts = vi.mocked(readFixAttempts);
-const mockWriteFixAttempts = vi.mocked(writeFixAttempts);
-const mockReadStallState = vi.mocked(readStallState);
-const mockWriteStallState = vi.mocked(writeStallState);
-const mockMarkSeen = vi.mocked(markSeen);
-function makeReport(overrides = {}) {
-    return {
-        pr: 42,
-        nodeId: "PR_kgDOAAA",
-        repo: "owner/repo",
-        status: "READY",
-        baseBranch: "main",
-        branchProtection: null,
-        mergeStatus: {
-            status: "CLEAN",
-            state: "OPEN",
-            isDraft: false,
-            mergeable: "MERGEABLE",
-            reviewDecision: "APPROVED",
-            blockingBotReviewInProgress: false,
-            mergeStateStatus: "CLEAN",
-        },
-        checks: {
-            passing: [],
-            failing: [],
-            inProgress: [],
-            skipped: [],
-            filtered: [],
-            filteredNames: [],
-            blockedByFilteredCheck: false,
-        },
-        threads: {
-            actionable: [],
-            resolutionOnly: [],
-            autoResolved: [],
-            autoResolveErrors: [],
-            firstLook: [],
-        },
-        comments: { actionable: [], firstLook: [] },
-        changesRequestedReviews: [],
-        reviewSummaries: [],
-        firstLookSummaries: [],
-        editedSummaries: [],
-        approvedReviews: [],
-        ...overrides,
-    };
-}
-function makeOpts(overrides = {}) {
-    return {
-        prNumber: 42,
-        format: "json",
-        readyDelaySeconds: 600,
-        ...overrides,
-    };
-}
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockExecFile.mockResolvedValue({ stdout: "abc1234\n", stderr: "" });
-        mockLoadConfig.mockReturnValue({
-            iterate: {
-                fixAttemptsPerThread: 3,
-                stallTimeoutMinutes: 60,
-                minimizeApprovals: false,
-            },
-            watch: { readyDelayMinutes: 10 },
-            resolve: {
-                concurrency: 4,
-                shaPoll: { intervalMs: 2000, maxAttempts: 10 },
-                fetchReviewSummaries: true,
-            },
-            checks: { ciTriggerEvents: ["pull_request"] },
-            mergeStatus: { blockingReviewerLogins: [] },
-            actions: { autoResolveOutdated: false, autoMarkReady: false, commitSuggestions: false },
-        });
-        mockReadFixAttempts.mockResolvedValue(null);
-        mockWriteFixAttempts.mockResolvedValue(undefined);
-        mockReadStallState.mockResolvedValue(null);
-        mockWriteStallState.mockResolvedValue(undefined);
-    });
-    afterEach(() => {
-        vi.restoreAllMocks();
-    });
-}
-export { makeOpts, makeReport, mockExecFile, mockFetch, mockLoadConfig, mockReadFixAttempts, mockReadStallState, mockMarkSeen, mockRunCheck, mockUpdateReadyDelay, mockWriteFixAttempts, mockWriteStallState, readFixAttempts, readStallState, runCheck, runIterate, updateReadyDelay, writeFixAttempts, writeStallState, };

package/bin/commands/poll.test-support.mjs

@@ -1,77 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-vi.mock("./iterate/index.mts", () => ({ runIterate: vi.fn() }));
-import { runIterate } from "./iterate/index.mjs";
-const mockRunIterate = vi.mocked(runIterate);
-function makeWaitResult(overrides = {}) {
-    return {
-        action: "wait",
-        pr: 42,
-        repo: "owner/repo",
-        status: "IN_PROGRESS",
-        state: "OPEN",
-        mergeStateStatus: "BLOCKED",
-        mergeStatus: "BLOCKED",
-        reviewDecision: "REVIEW_REQUIRED",
-        blockingBotReviewInProgress: false,
-        isDraft: false,
-        shouldCancel: false,
-        remainingSeconds: 0,
-        summary: { passing: 2, failing: 0, inProgress: 1, skipped: 0, filtered: 0 },
-        baseBranch: "main",
-        checks: [],
-        log: "WAIT: 2 passing, 1 in-progress",
-        ...overrides,
-    };
-}
-function makeCancelResult() {
-    return {
-        action: "cancel",
-        pr: 42,
-        repo: "owner/repo",
-        status: "READY",
-        state: "MERGED",
-        mergeStateStatus: "CLEAN",
-        mergeStatus: "CLEAN",
-        reviewDecision: "APPROVED",
-        blockingBotReviewInProgress: false,
-        isDraft: false,
-        shouldCancel: true,
-        remainingSeconds: 0,
-        summary: { passing: 3, failing: 0, inProgress: 0, skipped: 0, filtered: 0 },
-        baseBranch: "main",
-        checks: [],
-        reason: "merged",
-        log: "CANCEL: PR #42 is merged — stopping",
-    };
-}
-function makeMarkReadyResult() {
-    return {
-        action: "mark_ready",
-        pr: 42,
-        repo: "owner/repo",
-        status: "READY",
-        state: "OPEN",
-        mergeStateStatus: "CLEAN",
-        mergeStatus: "CLEAN",
-        reviewDecision: "APPROVED",
-        blockingBotReviewInProgress: false,
-        isDraft: false,
-        shouldCancel: false,
-        remainingSeconds: 0,
-        summary: { passing: 3, failing: 0, inProgress: 0, skipped: 0, filtered: 0 },
-        baseBranch: "main",
-        checks: [],
-        markedReady: true,
-        log: "MARKED READY: PR #42 converted from draft to ready for review",
-    };
-}
-function registerPollHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        vi.useFakeTimers();
-    });
-    afterEach(() => {
-        vi.useRealTimers();
-    });
-}
-export { mockRunIterate, makeWaitResult, makeCancelResult, makeMarkReadyResult, registerPollHooks };

package/bin/commands/resolve-instructions.mjs

@@ -1,59 +0,0 @@
-import { buildPrShepherdCommand } from "../cli/runner.mjs";
-import { SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEMS, buildShepherdJournalInstruction, } from "./shepherd-journal.mjs";
-import { buildCommitSuggestionInstruction } from "./commit-suggestion-instruction.mjs";
-/**
- * Build the numbered triage/fix/resolve instruction steps for the agent to follow.
- * Steps are conditionally emitted based on what the fetch returned (mirrors
- * `buildFixInstructions` in `commands/iterate/render.mts`).
- */
-export function buildFetchInstructions(prNumber, result) {
-    const { actionableThreads, resolutionOnlyThreads, firstLookThreads, actionableComments, firstLookComments, changesRequestedReviews, reviewSummaries, commitSuggestionsEnabled, } = result;
-    const firstLookTotal = firstLookThreads.length + firstLookComments.length;
-    const total = actionableThreads.length +
-        resolutionOnlyThreads.length +
-        actionableComments.length +
-        changesRequestedReviews.length +
-        reviewSummaries.length +
-        firstLookTotal;
-    if (total === 0) {
-        return ["No actionable items and no first-look items — end this invocation."];
-    }
-    const hasCodeItems = actionableThreads.length > 0 ||
-        actionableComments.length > 0 ||
-        changesRequestedReviews.length > 0;
-    const hasSuggestions = commitSuggestionsEnabled && actionableThreads.some((t) => t.suggestion != null);
-    const instructions = [];
-    instructions.push(`Classify every item listed above into exactly one of: Fixed / Actionable / Not relevant / Outdated / Acknowledge. Do not silently skip any item. Bot-authored review summaries (authors whose name contains \`[bot]\` or matches \`copilot-pull-request-reviewer\`, \`gemini-code-assist\`) default to Acknowledge with reason "bot summary — no actionable content" unless the body calls out an unaddressed issue.`);
-    if (firstLookTotal > 0) {
-        instructions.push(`Items in \`## First-look items\` are shown so you can acknowledge their current status before acting. If a first-look thread also appears under \`## Review threads to resolve\`, include its ID in \`--resolve-thread-ids\`; otherwise do not pass first-look-only IDs to mutation flags.`);
-    }
-    const editedTotal = actionableComments.filter((c) => c.edited).length +
-        firstLookThreads.filter((t) => t.edited).length +
-        firstLookComments.filter((c) => c.edited).length;
-    if (editedTotal > 0) {
-        instructions.push(`Actionable comments marked \`[edited since first look]\` and first-look bullets tagged \`, edited\` were updated by their author after you previously acknowledged them. Read the updated body before deciding whether any matching \`## Review threads to resolve\` item should be resolved.`);
-    }
-    if (hasSuggestions) {
-        instructions.push(buildCommitSuggestionInstruction(prNumber, "## Actionable Review Threads", true));
-    }
-    if (hasCodeItems) {
-        instructions.push(`Read and edit each file referenced under \`## Actionable Review Threads\`, \`## Actionable PR Comments\`, and \`## Pending CHANGES_REQUESTED reviews\` above. Reclassify each fixed item as Fixed. If an item is too complex to address, leave it as Actionable for the final report.`);
-        instructions.push(`If you applied code edits: commit them with a descriptive message, cancel any stale in-progress runs, then rebase and push per your repository's conventions.`);
-    }
-    if (resolutionOnlyThreads.length > 0) {
-        instructions.push(`Resolve each thread under \`## Review threads to resolve\` with \`--resolve-thread-ids\`. These threads are already outdated or minimized, so no code edit is required for them unless their body reveals separate work you choose to do.`);
-    }
-    const requireShaHint = hasCodeItems
-        ? ` Include \`--require-sha $(git rev-parse HEAD)\` only when you pushed new commits.`
-        : "";
-    const dismissNote = changesRequestedReviews.length > 0
-        ? ` For \`--dismiss-review-ids\`: \`--message\` is required with one specific sentence describing the fix or the reason for not acting (no boilerplate like "address review comments"); omit \`--message\` when not dismissing. Review-summary IDs (\`PRR_…\` from \`## Review summaries\`) go into \`--minimize-comment-ids\`, never \`--dismiss-review-ids\`.`
-        : reviewSummaries.length > 0
-            ? ` Review-summary IDs (\`PRR_…\` from \`## Review summaries\`) go into \`--minimize-comment-ids\`.`
-            : "";
-    const resolveCommand = `${buildPrShepherdCommand(["resolve", String(prNumber)]).text} [--resolve-thread-ids <ids>] [--minimize-comment-ids <ids>] [--dismiss-review-ids <ids> --message "<reason>"]`;
-    instructions.push(`Run \`${resolveCommand}\` with only the non-empty flag subsets. Skip the command entirely if all three ID lists are empty.${requireShaHint}${dismissNote}`);
-    instructions.push(buildShepherdJournalInstruction(prNumber, SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEMS));
-    instructions.push(`Report: echo the CLI's mutation output, then one line per Acknowledged item: \`Acknowledged <id> (@<author>): <reason>\`. If any fetched item was neither resolved nor acknowledged, stop and escalate: "<N> item(s) fetched but not acted on or acknowledged — need human direction before closing".`);
-    return instructions;
-}

package/bin/commands/resolve.test-support.mjs

@@ -1,116 +0,0 @@
-import { vi, beforeEach } from "vitest";
-vi.mock("../github/client.mts", () => ({
-    getRepoInfo: vi.fn().mockResolvedValue({ owner: "owner", name: "repo" }),
-    getCurrentPrNumber: vi.fn().mockResolvedValue(42),
-}));
-vi.mock("../state/seen-comments.mts", async (importOriginal) => {
-    const actual = await importOriginal();
-    return {
-        ...actual,
-        loadSeenMap: vi.fn().mockResolvedValue(new Map()),
-        markSeen: vi.fn().mockResolvedValue(undefined),
-    };
-});
-vi.mock("../github/batch.mts", () => ({
-    fetchPrBatch: vi.fn(),
-}));
-vi.mock("../comments/resolve.mts", () => ({
-    autoResolveOutdated: vi.fn(),
-    applyResolveOptions: vi.fn(),
-}));
-vi.mock("../config/load.mts", () => ({
-    loadConfig: vi.fn().mockReturnValue({
-        resolve: {
-            shaPoll: { intervalMs: 2000, maxAttempts: 10 },
-            fetchReviewSummaries: true,
-        },
-        actions: {
-            autoResolveOutdated: true,
-            autoMarkReady: true,
-            commitSuggestions: true,
-        },
-    }),
-}));
-import { runResolveFetch, runResolveMutate } from "./resolve.mjs";
-import { getCurrentPrNumber } from "../github/client.mjs";
-import { fetchPrBatch } from "../github/batch.mjs";
-import { autoResolveOutdated, applyResolveOptions } from "../comments/resolve.mjs";
-import { loadConfig } from "../config/load.mjs";
-import { loadSeenMap, markSeen, hashBody } from "../state/seen-comments.mjs";
-const mockGetCurrentPrNumber = vi.mocked(getCurrentPrNumber);
-const mockFetchPrBatch = vi.mocked(fetchPrBatch);
-const mockAutoResolveOutdated = vi.mocked(autoResolveOutdated);
-const mockApplyResolveOptions = vi.mocked(applyResolveOptions);
-const mockLoadConfig = vi.mocked(loadConfig);
-const mockLoadSeenMap = vi.mocked(loadSeenMap);
-const mockMarkSeen = vi.mocked(markSeen);
-const BASE_OPTS = { format: "text" };
-function makeBatchData(overrides = {}) {
-    return {
-        nodeId: "PR_kgDOAAA",
-        number: 42,
-        state: "OPEN",
-        isDraft: false,
-        mergeable: "MERGEABLE",
-        mergeStateStatus: "CLEAN",
-        reviewDecision: "APPROVED",
-        headRefOid: "abc123",
-        headRefName: "feature",
-        headRepoWithOwner: "owner/repo",
-        baseRefName: "main",
-        reviewRequests: [],
-        latestReviews: [],
-        reviewThreads: [],
-        comments: [],
-        changesRequestedReviews: [],
-        reviewSummaries: [],
-        approvedReviews: [],
-        checks: [],
-        branchProtection: null,
-        ...overrides,
-    };
-}
-function makeThread(overrides = {}) {
-    return {
-        id: "t-1",
-        isResolved: false,
-        isOutdated: false,
-        isMinimized: false,
-        path: "src/foo.ts",
-        line: 1,
-        startLine: null,
-        author: "alice",
-        authorType: "Unknown",
-        body: "fix this",
-        url: "",
-        createdAtUnix: 1_700_000_000,
-        ...overrides,
-    };
-}
-function makeComment(overrides = {}) {
-    return {
-        id: "c-1",
-        isMinimized: false,
-        author: "bob",
-        authorType: "Unknown",
-        body: "nit",
-        url: "",
-        createdAtUnix: 1_700_000_000,
-        ...overrides,
-    };
-}
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockFetchPrBatch.mockResolvedValue({ data: makeBatchData() });
-        mockAutoResolveOutdated.mockResolvedValue({ resolved: [], errors: [] });
-        mockApplyResolveOptions.mockResolvedValue({
-            repliedThreads: [],
-            resolvedThreads: [],
-            minimizedComments: [],
-            dismissedReviews: [],
-            errors: [],
-        });
-    });
-}
-export { BASE_OPTS, applyResolveOptions, autoResolveOutdated, fetchPrBatch, getCurrentPrNumber, hashBody, loadConfig, loadSeenMap, makeBatchData, makeComment, makeThread, markSeen, mockApplyResolveOptions, mockAutoResolveOutdated, mockFetchPrBatch, mockGetCurrentPrNumber, mockLoadConfig, mockLoadSeenMap, mockMarkSeen, runResolveFetch, runResolveMutate, };

package/bin/commands/shepherd-journal.test-support.mjs

@@ -1,7 +0,0 @@
-import { buildShepherdJournalInstruction, SHEPHERD_JOURNAL_APPEND_HINT, SHEPHERD_JOURNAL_FIRST_LOOK_GUIDANCE, SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEM_HEADINGS, SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEMS, SHEPHERD_JOURNAL_SECTION, SHEPHERD_JOURNAL_SECTION_PATTERN, } from "./shepherd-journal.mjs";
-import { buildFixInstructions } from "./iterate/render.mjs";
-import { buildFetchInstructions } from "./resolve-instructions.mjs";
-function countMentions(text, phrase) {
-    return (text.match(new RegExp(phrase, "g")) ?? []).length;
-}
-export { SHEPHERD_JOURNAL_APPEND_HINT, SHEPHERD_JOURNAL_FIRST_LOOK_GUIDANCE, SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEMS, SHEPHERD_JOURNAL_REFERENCE_GUIDANCE_THREADS_AND_COMMENTS_IN_ITEM_HEADINGS, SHEPHERD_JOURNAL_SECTION, SHEPHERD_JOURNAL_SECTION_PATTERN, buildFetchInstructions, buildFixInstructions, buildShepherdJournalInstruction, countMentions, };

package/bin/comments/outdated.mjs

@@ -1,15 +0,0 @@
-/**
- * Determines which review threads should be auto-resolved as outdated.
- *
- * A thread is eligible for auto-resolution when:
- *   - `isOutdated == true` (GitHub marks these when the diff hunk changed), AND
- *   - `isResolved == false`.
- *
- * GitHub's `isOutdated` flag means the thread's referenced code has changed
- * enough that the comment no longer points to a live diff line. These threads
- * are visually collapsed on GitHub and are safe to resolve programmatically.
- */
-/** Returns the subset of threads that should be auto-resolved as outdated. */
-export function getOutdatedThreads(threads) {
-    return threads.filter((t) => t.isOutdated && !t.isResolved);
-}

package/bin/comments/resolve.test-support.mjs

@@ -1,44 +0,0 @@
-import { vi, beforeEach } from "vitest";
-// ---------------------------------------------------------------------------
-// Mock github/client.mts before any imports.
-// ---------------------------------------------------------------------------
-vi.mock("../github/client.mts", () => ({
-    graphqlWithRateLimit: vi.fn(),
-    getPrHeadSha: vi.fn(),
-}));
-import { applyResolveOptions, autoResolveOutdated } from "./resolve.mjs";
-import { graphqlWithRateLimit, getPrHeadSha } from "../github/client.mjs";
-const mockGraphql = vi.mocked(graphqlWithRateLimit);
-const mockGetPrHeadSha = vi.mocked(getPrHeadSha);
-const REPO = { owner: "owner", name: "repo" };
-/** Build a mock response with the correct nested shape for each alias type (r/m/d). */
-function makeBulkResponse(doc) {
-    const str = typeof doc === "string" ? doc : "";
-    const data = {};
-    for (const match of str.matchAll(/^\s+([a-z]\d+):/gm)) {
-        const alias = match[1];
-        if (alias === undefined)
-            continue;
-        if (alias.startsWith("r"))
-            data[alias] = { thread: { isResolved: true } };
-        else if (alias.startsWith("p"))
-            data[alias] = { comment: { id: `${alias}-comment` } };
-        else if (alias.startsWith("m"))
-            data[alias] = { minimizedComment: { isMinimized: true } };
-        else if (alias.startsWith("d"))
-            data[alias] = { pullRequestReview: { state: "DISMISSED" } };
-        else
-            data[alias] = {};
-    }
-    return { data };
-}
-// ---------------------------------------------------------------------------
-// applyResolveOptions
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockGraphql.mockImplementation(async (doc) => makeBulkResponse(doc));
-    });
-}
-export { REPO, applyResolveOptions, autoResolveOutdated, getPrHeadSha, graphqlWithRateLimit, makeBulkResponse, mockGetPrHeadSha, mockGraphql, };

package/bin/github/batch-parsers.test-support.mjs

@@ -1,66 +0,0 @@
-import { vi, beforeEach } from "vitest";
-vi.mock("./client.mts", () => ({
-    graphql: vi.fn(),
-    graphqlWithRateLimit: vi.fn(),
-}));
-import { fetchPrBatch } from "./batch.mjs";
-import { graphql, graphqlWithRateLimit } from "./client.mjs";
-const mockGraphql = vi.mocked(graphql);
-const mockGraphqlWithRateLimit = vi.mocked(graphqlWithRateLimit);
-const REPO = { owner: "owner", name: "repo" };
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-function makeRawPr(overrides = {}) {
-    return {
-        id: "PR_kgDOAAA",
-        number: 42,
-        state: "OPEN",
-        isDraft: false,
-        mergeable: "MERGEABLE",
-        mergeStateStatus: "CLEAN",
-        reviewDecision: "APPROVED",
-        headRefOid: "abc123",
-        headRefName: "feature",
-        headRepository: { nameWithOwner: "owner/repo" },
-        baseRefName: "main",
-        reviewRequests: { nodes: [] },
-        latestReviews: { nodes: [] },
-        reviewThreads: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        comments: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        changesRequestedReviews: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        reviewSummaries: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        approvedReviews: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        commits: { nodes: [{ commit: { statusCheckRollup: null } }] },
-        ...overrides,
-    };
-}
-function makeResponse(pr = makeRawPr()) {
-    return { data: { repository: { pullRequest: pr } } };
-}
-// ---------------------------------------------------------------------------
-// PR not found
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockGraphql.mockResolvedValue(makeResponse());
-        mockGraphqlWithRateLimit.mockResolvedValue(makeResponse());
-    });
-}
-export { REPO, fetchPrBatch, graphql, graphqlWithRateLimit, makeRawPr, makeResponse, mockGraphql, mockGraphqlWithRateLimit, };

package/bin/github/batch.test-support.mjs

@@ -1,66 +0,0 @@
-import { vi, beforeEach } from "vitest";
-vi.mock("./client.mts", () => ({
-    graphql: vi.fn(),
-    graphqlWithRateLimit: vi.fn(),
-}));
-import { fetchPrBatch } from "./batch.mjs";
-import { graphql, graphqlWithRateLimit } from "./client.mjs";
-const mockGraphql = vi.mocked(graphql);
-const mockGraphqlWithRateLimit = vi.mocked(graphqlWithRateLimit);
-const REPO = { owner: "owner", name: "repo" };
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-function makeRawPr(overrides = {}) {
-    return {
-        id: "PR_kgDOAAA",
-        number: 42,
-        state: "OPEN",
-        isDraft: false,
-        mergeable: "MERGEABLE",
-        mergeStateStatus: "CLEAN",
-        reviewDecision: "APPROVED",
-        headRefOid: "abc123",
-        headRefName: "feature",
-        headRepository: { nameWithOwner: "owner/repo" },
-        baseRefName: "main",
-        reviewRequests: { nodes: [] },
-        latestReviews: { nodes: [] },
-        reviewThreads: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        comments: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        changesRequestedReviews: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        reviewSummaries: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        approvedReviews: {
-            pageInfo: { hasPreviousPage: false, startCursor: null },
-            nodes: [],
-        },
-        commits: { nodes: [{ commit: { statusCheckRollup: null } }] },
-        ...overrides,
-    };
-}
-function makeResponse(pr = makeRawPr()) {
-    return { data: { repository: { pullRequest: pr } } };
-}
-// ---------------------------------------------------------------------------
-// reviewSummaries — COMMENTED reviews surfaced for agent-driven minimize
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockGraphql.mockResolvedValue(makeResponse());
-        mockGraphqlWithRateLimit.mockResolvedValue(makeResponse());
-    });
-}
-export { REPO, fetchPrBatch, graphql, graphqlWithRateLimit, makeRawPr, makeResponse, mockGraphql, mockGraphqlWithRateLimit, };

package/bin/github/client.test-support.mjs

@@ -1,55 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-import { _resetTokenCache } from "./http.mjs";
-export const mockFetch = vi.fn();
-vi.stubGlobal("fetch", mockFetch);
-const { _mockExecFile } = vi.hoisted(() => ({ _mockExecFile: vi.fn() }));
-export const mockExecFile = _mockExecFile;
-vi.mock("node:child_process", () => ({
-    execFile: (cmd, args, optsOrCb, maybeCb) => {
-        const cb = typeof optsOrCb === "function" ? optsOrCb : maybeCb;
-        _mockExecFile(cmd, args)
-            .then((result) => cb(null, result))
-            .catch((err) => cb(err, { stdout: "", stderr: "" }));
-    },
-}));
-export function gqlOk(data) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve({ data }),
-        text: () => Promise.resolve(JSON.stringify({ data })),
-    };
-}
-export function restOk(data) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve(data),
-        text: () => Promise.resolve(JSON.stringify(data)),
-    };
-}
-export function gqlErrors(errors) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve({ data: null, errors }),
-        text: () => Promise.resolve(JSON.stringify({ data: null, errors })),
-    };
-}
-export function registerClientHooks() {
-    beforeEach(() => {
-        mockFetch.mockReset();
-        mockExecFile.mockReset();
-        _resetTokenCache();
-        delete process.env["GITHUB_TOKEN"];
-        delete process.env["GITHUB_PERSONAL_ACCESS_TOKEN"];
-        process.env["GH_TOKEN"] = "test-token";
-    });
-    afterEach(() => {
-        delete process.env["GH_TOKEN"];
-        _resetTokenCache();
-    });
-}