pr-shepherd 0.23.0 → 0.24.0

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

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

@@ -1,51 +0,0 @@
-import { vi, beforeEach } from "vitest";
-// ---------------------------------------------------------------------------
-// Stub fetch and child_process globally before any imports.
-// ---------------------------------------------------------------------------
-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: "" }));
-    },
-}));
-import { GitHubRequestError, graphql, graphqlWithRateLimit, rest, restText, _resetTokenCache, } from "./http.mjs";
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-function jsonOk(data) {
-    return {
-        ok: true,
-        status: 200,
-        headers: new Headers({ "content-type": "application/json" }),
-        json: () => Promise.resolve(data),
-        text: () => Promise.resolve(JSON.stringify(data)),
-    };
-}
-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 })),
-    };
-}
-// ---------------------------------------------------------------------------
-// Token resolution
-// ---------------------------------------------------------------------------
-export function registerHooks() {
-    beforeEach(() => {
-        mockFetch.mockReset();
-        mockExecFile.mockReset();
-        _resetTokenCache();
-        delete process.env["GH_TOKEN"];
-        delete process.env["GITHUB_TOKEN"];
-        delete process.env["GITHUB_PERSONAL_ACCESS_TOKEN"];
-    });
-}
-export { GitHubRequestError, _resetTokenCache, gqlOk, graphql, graphqlWithRateLimit, jsonOk, mockExecFile, mockFetch, rest, restText, };

package/bin/state/seen-comments.test-support.mjs

@@ -1,19 +0,0 @@
-import { beforeEach, afterEach } from "vitest";
-import { randomBytes, createHash } from "node:crypto";
-import { rm } from "node:fs/promises";
-export function idToFilename(id) {
-    return createHash("sha256").update(id, "utf8").digest("hex") + ".json";
-}
-export const testKey = { owner: "test-owner", repo: "test-repo", pr: 123 };
-export const testId = "PRRT_kwDOTest123";
-export let testStateDir;
-export function registerHooks() {
-    beforeEach(() => {
-        testStateDir = `${process.env["TMPDIR"] ?? "/tmp"}/shepherd-seen-test-${randomBytes(4).toString("hex")}`;
-        process.env["PR_SHEPHERD_STATE_DIR"] = testStateDir;
-    });
-    afterEach(async () => {
-        delete process.env["PR_SHEPHERD_STATE_DIR"];
-        await rm(testStateDir, { recursive: true, force: true });
-    });
-}

package/bin/suggestions/patch.test-support.mjs

@@ -1,2 +0,0 @@
-import { buildUnifiedDiff } from "./patch.mjs";
-export { buildUnifiedDiff };