co-maintainer 0.4.7 → 0.4.8

package/dist/package.json

@@ -1,6 +1,6 @@
 {
     "name": "co-maintainer",
-    "version": "0.4.7",
+    "version": "0.4.8",
     "description": "Analyzes a GitHub repository and writes repository-specific contribution guidance.",
     "license": "MIT",
     "repository": {

package/dist/src/cli/args.js

@@ -52,6 +52,7 @@ export async function parseArgs(args) {
         console.log("Options: --include-codebase --include-pull-requests --include-pull-request-changes");
         console.log("         --include-commit-history --include-how-repo-works");
         console.log("         --max-commits=N --max-pr-months=N --max-pull-request-change-lines=N --max-comment=N");
+        console.log("         --only-request-changed-pr");
         console.log("         --auth=gh|pat --github-pat=... --ai=none|openrouter|hetzner --token=... --low-model=... --high-model=...");
         process.exit(0);
     }
@@ -145,7 +146,8 @@ export async function parseArgs(args) {
         if (arg === "--debug" ||
             arg === "--log-time" ||
             arg === "--review-upstream" ||
-            arg === "--disable-codegraph")
+            arg === "--disable-codegraph" ||
+            arg === "--only-request-changed-pr")
             continue;
         if (arg.startsWith("--") && !known)
             die(`Unknown option: ${arg}`);
@@ -241,6 +243,8 @@ export async function parseArgs(args) {
         includePullRequestChanges: enabled("include-pull-request-changes"),
         includeCommitHistory: enabled("include-commit-history"),
         includeHowRepoWorks: enabled("include-how-repo-works"),
+        onlyRequestChangedPr: rest.includes("--only-request-changed-pr") ||
+            repoConfig.onlyRequestChangedPr === true,
         maxCommits: value("max-commits") ?? repoConfig.maxCommits ?? configDefault.maxCommits,
         maxPrMonths: value("max-pr-months") ??
             repoConfig.maxPrMonths ??

package/dist/src/config.d.ts

@@ -17,6 +17,7 @@ export type RepoConfig = {
     includePullRequestChanges?: boolean;
     includeCommitHistory?: boolean;
     includeHowRepoWorks?: boolean;
+    onlyRequestChangedPr?: boolean;
     /** Five field cron in UTC, checked by `serve` to queue a remake. */
     remakeCron?: string;
 };

package/dist/src/github/collect.js

@@ -10,6 +10,12 @@ function percent(done, total) {
         return "0%";
     return `${Math.round((done / total) * 100)}%`;
 }
+function changesRequested(reviews) {
+    return reviews.some((review) => review.state === "CHANGES_REQUESTED");
+}
+function reviewBodies(reviews) {
+    return reviews.map((review) => String(review.body ?? "")).filter(Boolean);
+}
 function withinPrWindow(updatedAt, months) {
     if (months === undefined || months === 0)
         return true;
@@ -258,13 +264,44 @@ async function pullRequests(client, options, previous, phase, progress) {
             changedFiles: cached?.changedFiles ?? [],
             diff: cached?.diff ?? "",
         };
+        if (typeof cached?.changesRequested === "boolean") {
+            current.changesRequested = cached.changesRequested;
+        }
+        const only = options.onlyRequestChangedPr;
         const discussionUnchanged = cached?.updatedAt === current.updatedAt;
+        const decisionKnown = typeof cached?.changesRequested === "boolean";
         const diffUnchanged = cached?.headSha === current.headSha && Boolean(cached?.diff);
-        const discussionStatus = discussionUnchanged
-            ? "comments/reviews cache"
-            : "download comments/reviews";
+        let dropped = only && discussionUnchanged && cached?.changesRequested === false;
+        const loadComments = async () => {
+            const comments = await client.pages(`repos/${options.repo}/issues/${number}/comments`);
+            current.comments = comments
+                .map((comment) => String(comment.body ?? ""))
+                .filter(Boolean)
+                .slice(0, options.maxComments);
+        };
+        const loadReviews = async () => {
+            const reviews = await client.pages(`repos/${options.repo}/pulls/${number}/reviews`);
+            current.reviews = reviewBodies(reviews);
+            current.changesRequested = changesRequested(reviews);
+        };
+        if (!dropped && only && (!discussionUnchanged || !decisionKnown)) {
+            await loadReviews();
+            if (!current.changesRequested)
+                dropped = true;
+            else if (!discussionUnchanged)
+                await loadComments();
+        }
+        else if (!dropped && !discussionUnchanged) {
+            await loadComments();
+            await loadReviews();
+        }
+        const discussionStatus = dropped
+            ? "dropped"
+            : discussionUnchanged
+                ? "comments/reviews cache"
+                : "download comments/reviews";
         let diffStatus = "diff disabled";
-        if (options.includePullRequestChanges) {
+        if (!dropped && options.includePullRequestChanges) {
             const lines = current.additions + current.deletions;
             diffStatus = diffUnchanged
                 ? "diff cache"
@@ -278,18 +315,10 @@ async function pullRequests(client, options, previous, phase, progress) {
         if (phase) {
             phase.text = `PR #${number} · ${discussionStatus} · ${diffStatus} · ${completed}/${selected.length} · ${percent(completed, selected.length)}`;
         }
-        if (!discussionUnchanged) {
-            const comments = await client.pages(`repos/${options.repo}/issues/${number}/comments`);
-            const reviews = await client.pages(`repos/${options.repo}/pulls/${number}/reviews`);
-            current.comments = comments
-                .map((comment) => String(comment.body ?? ""))
-                .filter(Boolean)
-                .slice(0, options.maxComments);
-            current.reviews = reviews
-                .map((review) => String(review.body ?? ""))
-                .filter(Boolean);
-        }
-        if (options.includePullRequestChanges && !diffUnchanged && detail) {
+        if (!dropped &&
+            options.includePullRequestChanges &&
+            !diffUnchanged &&
+            detail) {
             const lines = current.additions + current.deletions;
             if (options.maxPullRequestChangeLines === undefined ||
                 lines <= options.maxPullRequestChangeLines) {
@@ -311,7 +340,8 @@ async function pullRequests(client, options, previous, phase, progress) {
                 current.diff = "";
             }
         }
-        result[index] = current;
+        if (!dropped)
+            result[index] = current;
         completed++;
         if (phase) {
             phase.text = `pull requests ${completed}/${selected.length} · ${percent(completed, selected.length)}`;
@@ -322,7 +352,11 @@ async function pullRequests(client, options, previous, phase, progress) {
             await save;
         }
     });
-    return result;
+    const kept = result.filter((item) => !!item);
+    if (options.onlyRequestChangedPr) {
+        log("fetch", `only request-changed pr · kept ${kept.length} · dropped ${selected.length - kept.length}`);
+    }
+    return kept;
 }
 async function commits(client, options, meta, phase) {
     const branch = String(meta.default_branch ?? "main");

package/dist/src/knowledge/types.d.ts

@@ -12,6 +12,9 @@ export type PullRequest = {
     deletions: number;
     comments: string[];
     reviews: string[];
+    /** Set once reviews have been read. Missing means an older checkpoint
+     * never stored the review state. */
+    changesRequested?: boolean;
     changedFiles: string[];
     diff: string;
 };

package/dist/src/services/remote_review.js

@@ -44,6 +44,7 @@ function reviewOptionsForRepo(repo, useCodegraph) {
         includePullRequestChanges: true,
         includeCommitHistory: true,
         includeHowRepoWorks: true,
+        onlyRequestChangedPr: false,
     };
 }
 function supersedeActiveRemoteJobs(queueKey, supersededBy) {

package/dist/src/services/review.js

@@ -130,6 +130,7 @@ export function reviewOptions(repo, prNumber) {
         includePullRequestChanges: true,
         includeCommitHistory: true,
         includeHowRepoWorks: true,
+        onlyRequestChangedPr: false,
         useCodegraph: getRepo(repo)?.use_codegraph === 1,
     };
 }

package/dist/src/services/setup.js

@@ -259,6 +259,7 @@ export async function runInitOrRemake(options) {
             includePullRequestChanges: options.includePullRequestChanges,
             includeCommitHistory: options.includeCommitHistory,
             includeHowRepoWorks: options.includeHowRepoWorks,
+            onlyRequestChangedPr: options.onlyRequestChangedPr,
         });
     });
     log("write", `${path} · ${result.changed.length
@@ -306,6 +307,7 @@ export function optionsFromConfig(repo, command) {
         includePullRequestChanges: repoConfig.includePullRequestChanges ?? true,
         includeCommitHistory: repoConfig.includeCommitHistory ?? true,
         includeHowRepoWorks: repoConfig.includeHowRepoWorks ?? true,
+        onlyRequestChangedPr: repoConfig.onlyRequestChangedPr === true,
         maxCommits: repoConfig.maxCommits ?? config.defaults?.maxCommits,
         maxPrMonths: repoConfig.maxPrMonths ?? config.defaults?.maxPrMonths,
         maxPullRequestChangeLines: repoConfig.maxPullRequestChangeLines ??

package/dist/src/types.d.ts

@@ -34,6 +34,9 @@ export type Options = {
     includePullRequestChanges: boolean;
     includeCommitHistory: boolean;
     includeHowRepoWorks: boolean;
+    /** Keep only pull requests in the window that have at least one
+     * `CHANGES_REQUESTED` review. Off by default, so the window is unchanged. */
+    onlyRequestChangedPr: boolean;
     maxCommits?: number;
     maxPrMonths?: number;
     maxPullRequestChangeLines?: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "co-maintainer",
-  "version": "0.4.7",
+  "version": "0.4.8",
   "description": "Analyzes a GitHub repository and writes repository-specific contribution guidance.",
   "license": "MIT",
   "repository": {