pr-shepherd 0.23.0 → 0.24.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.23.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.23.0",
+  "version": "0.24.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, };

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

@@ -1,147 +0,0 @@
-import { vi, beforeEach } from "vitest";
-vi.mock("../github/batch.mts", () => ({ fetchPrBatch: vi.fn() }));
-vi.mock("../github/client.mts", () => ({
-    getRepoInfo: vi.fn().mockResolvedValue({ owner: "owner", name: "repo" }),
-    getCurrentPrNumber: vi.fn().mockResolvedValue(42),
-    getMergeableState: vi.fn(),
-}));
-vi.mock("../checks/triage.mts", () => ({
-    triageFailingChecks: vi.fn((checks) => Promise.resolve(checks)),
-    fetchStartupFailureChecks: vi.fn().mockResolvedValue([]),
-}));
-vi.mock("../github/check-annotations.mts", () => ({
-    fetchCheckRunAnnotations: vi.fn().mockResolvedValue([]),
-}));
-vi.mock("../comments/resolve.mts", () => ({
-    autoResolveOutdated: vi.fn().mockResolvedValue({ resolved: [], errors: [] }),
-}));
-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),
-    };
-});
-const { mockLoadConfig } = vi.hoisted(() => ({ mockLoadConfig: vi.fn() }));
-vi.mock("../config/load.mts", () => ({ loadConfig: mockLoadConfig }));
-import { runCheck } from "./check.mjs";
-import { fetchPrBatch } from "../github/batch.mjs";
-import { getCurrentPrNumber, getMergeableState } from "../github/client.mjs";
-import { fetchStartupFailureChecks, triageFailingChecks } from "../checks/triage.mjs";
-import { fetchCheckRunAnnotations } from "../github/check-annotations.mjs";
-import { loadSeenMap, markSeen, hashBody } from "../state/seen-comments.mjs";
-import { autoResolveOutdated } from "../comments/resolve.mjs";
-const mockFetchPrBatch = vi.mocked(fetchPrBatch);
-const mockGetCurrentPrNumber = vi.mocked(getCurrentPrNumber);
-const mockGetMergeableState = vi.mocked(getMergeableState);
-const mockTriageFailingChecks = vi.mocked(triageFailingChecks);
-const mockFetchStartupFailureChecks = vi.mocked(fetchStartupFailureChecks);
-const mockFetchCheckRunAnnotations = vi.mocked(fetchCheckRunAnnotations);
-const mockLoadSeenMap = vi.mocked(loadSeenMap);
-const mockMarkSeen = vi.mocked(markSeen);
-const mockAutoResolveOutdated = vi.mocked(autoResolveOutdated);
-const BASE_OPTS = { format: "text" };
-function defaultConfig() {
-    return {
-        iterate: {
-            fixAttemptsPerThread: 3,
-            stallTimeoutMinutes: 60,
-            minimizeApprovals: false,
-            minimizeComments: "all",
-        },
-        watch: { readyDelayMinutes: 10 },
-        resolve: {
-            shaPoll: { intervalMs: 2000, maxAttempts: 10 },
-            fetchReviewSummaries: true,
-        },
-        checks: {
-            ciTriggerEvents: ["pull_request", "pull_request_target"],
-        },
-        mergeStatus: { blockingReviewerLogins: ["copilot"] },
-        actions: {
-            autoResolveOutdated: true,
-            autoMarkReady: true,
-            commitSuggestions: true,
-        },
-    };
-}
-function makeCheck(overrides = {}) {
-    return {
-        id: null,
-        name: "tests",
-        status: "COMPLETED",
-        conclusion: "SUCCESS",
-        detailsUrl: "",
-        event: "pull_request",
-        runId: null,
-        category: "passed",
-        ...overrides,
-    };
-}
-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: [],
-        branchProtection: null,
-        checks: [makeCheck()],
-        ...overrides,
-    };
-}
-function makeThread(overrides = {}) {
-    return {
-        id: "t1",
-        isResolved: false,
-        isOutdated: false,
-        isMinimized: false,
-        path: "src/foo.mts",
-        line: 10,
-        startLine: null,
-        author: "reviewer",
-        authorType: "Unknown",
-        body: "fix this",
-        url: "",
-        createdAtUnix: 0,
-        ...overrides,
-    };
-}
-function makeComment(overrides = {}) {
-    return {
-        id: "c1",
-        author: "commenter",
-        authorType: "Unknown",
-        body: "comment body",
-        url: "",
-        createdAtUnix: 0,
-        isMinimized: false,
-        ...overrides,
-    };
-}
-// No PR found
-export function registerHooks() {
-    beforeEach(() => {
-        vi.clearAllMocks();
-        mockLoadConfig.mockReturnValue(defaultConfig());
-        mockFetchPrBatch.mockResolvedValue({ data: makeBatchData() });
-        mockGetMergeableState.mockResolvedValue({ mergeable: "MERGEABLE", mergeStateStatus: "CLEAN" });
-        mockFetchStartupFailureChecks.mockResolvedValue([]);
-        mockFetchCheckRunAnnotations.mockResolvedValue([]);
-    });
-}
-export { BASE_OPTS, autoResolveOutdated, defaultConfig, fetchPrBatch, fetchStartupFailureChecks, fetchCheckRunAnnotations, getCurrentPrNumber, getMergeableState, hashBody, loadSeenMap, makeBatchData, makeCheck, makeComment, makeThread, markSeen, mockAutoResolveOutdated, mockFetchPrBatch, mockFetchStartupFailureChecks, mockFetchCheckRunAnnotations, mockGetCurrentPrNumber, mockGetMergeableState, mockLoadConfig, mockLoadSeenMap, mockMarkSeen, mockTriageFailingChecks, runCheck, triageFailingChecks, };

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

@@ -1,47 +0,0 @@
-import { vi, beforeEach, afterEach } from "vitest";
-import { join } from "node:path";
-import { mkdtemp, realpath, rm, mkdir, writeFile, stat } from "node:fs/promises";
-import { tmpdir } from "node:os";
-vi.mock("../github/client.mts", () => ({
-    getRepoInfo: vi.fn().mockResolvedValue({ owner: "acme", name: "widgets" }),
-    getCurrentBranch: vi.fn().mockResolvedValue("feature/test"),
-    getCurrentPrNumber: vi.fn().mockResolvedValue(42),
-    getPrNumberForBranch: vi.fn().mockResolvedValue(42),
-}));
-import { getRepoInfo, getCurrentBranch, getCurrentPrNumber, getPrNumberForBranch, } from "../github/client.mjs";
-export const mockGetRepoInfo = vi.mocked(getRepoInfo);
-export const mockGetCurrentBranch = vi.mocked(getCurrentBranch);
-export const mockGetCurrentPrNumber = vi.mocked(getCurrentPrNumber);
-export const mockGetPrNumberForBranch = vi.mocked(getPrNumberForBranch);
-export let stateDir;
-export function registerHooks() {
-    beforeEach(async () => {
-        const tmpPath = await mkdtemp(join(tmpdir(), "shepherd-clean-test-"));
-        stateDir = await realpath(tmpPath);
-        process.env["PR_SHEPHERD_STATE_DIR"] = stateDir;
-        vi.clearAllMocks();
-        mockGetRepoInfo.mockResolvedValue({ owner: "acme", name: "widgets" });
-        mockGetCurrentBranch.mockResolvedValue("feature/test");
-        mockGetCurrentPrNumber.mockResolvedValue(42);
-        mockGetPrNumberForBranch.mockResolvedValue(42);
-    });
-    afterEach(async () => {
-        delete process.env["PR_SHEPHERD_STATE_DIR"];
-        await rm(stateDir, { recursive: true, force: true });
-    });
-}
-export async function seedPrDir(dir, pr) {
-    const prDir = join(dir, "acme-widgets", String(pr));
-    await mkdir(join(prDir, "seen"), { recursive: true });
-    await writeFile(join(prDir, "fix-attempts.json"), "{}", "utf8");
-    return prDir;
-}
-export async function pathExists(p) {
-    try {
-        await stat(p);
-        return true;
-    }
-    catch {
-        return false;
-    }
-}