pr-shepherd 0.22.0 → 0.23.0

package/bin/comments/thread-visibility.mjs

@@ -0,0 +1,60 @@
+import { classifyItem } from "../state/seen-comments.mjs";
+import { threadTranscriptBody } from "../threads/transcript.mjs";
+function withEdited(thread, edited) {
+    return edited ? { ...thread, edited: true } : thread;
+}
+function classifyVisibleThread(thread, seenMap) {
+    const cls = classifyItem(thread.id, threadTranscriptBody(thread), seenMap);
+    if (cls === "unchanged")
+        return null;
+    return withEdited(thread, cls === "edited");
+}
+function classifyFirstLookThread(thread, seenMap, firstLookStatus) {
+    const visible = classifyVisibleThread(thread, seenMap);
+    if (visible === null)
+        return null;
+    return { ...visible, firstLookStatus };
+}
+export function classifyThreadVisibility(threads, seenMap) {
+    const unresolvedThreads = threads.filter((t) => !t.isResolved);
+    const activeThreads = unresolvedThreads
+        .filter((t) => !t.isOutdated && !t.isMinimized)
+        .flatMap((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] : [];
+    });
+    const firstLookThreads = [
+        ...threads.flatMap((t) => {
+            if (!t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "outdated");
+            return visible ? [visible] : [];
+        }),
+        ...threads.flatMap((t) => {
+            if (!t.isResolved || t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "resolved");
+            return visible ? [visible] : [];
+        }),
+        ...threads.flatMap((t) => {
+            if (!t.isMinimized || t.isResolved || t.isOutdated)
+                return [];
+            const visible = classifyFirstLookThread(t, seenMap, "minimized");
+            return visible ? [visible] : [];
+        }),
+    ];
+    const seenMarkerIds = new Set();
+    const toMarkSeen = [...activeThreads, ...resolutionOnlyThreads, ...firstLookThreads].filter((thread) => {
+        if (seenMarkerIds.has(thread.id))
+            return false;
+        seenMarkerIds.add(thread.id);
+        return true;
+    });
+    return { activeThreads, resolutionOnlyThreads, firstLookThreads, toMarkSeen };
+}

package/bin/comments/visible-comments.mjs

@@ -5,7 +5,7 @@ export function classifyVisibleComments(comments, seenMap, minimizeComments) {
     const minimizeIds = [];
     const toMarkSeen = [];
     for (const c of comments.filter((comment) => !comment.isMinimized)) {
-        if (shouldMinimizeAuthor(c.authorType, minimizeComments)) {
+        if (shouldMinimizeAuthor(c.authorType, minimizeComments, c.author)) {
             actionable.push(c);
             minimizeIds.push(c.id);
             continue;

package/bin/github/batch-parser-helpers.mjs

@@ -1,7 +1,6 @@
-export function mapAuthorType(typeName) {
-    if (typeName === "User" || typeName === "Bot")
-        return typeName;
-    return "Unknown";
+import { normalizeAuthorType } from "../comments/authors.mjs";
+export function mapAuthorType(typeName, login) {
+    return normalizeAuthorType(typeName, login);
 }
 export function parseCreatedAt(iso) {
     const ms = new Date(iso).getTime();

package/bin/github/batch-parsers.mjs

@@ -14,7 +14,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
             id: c.id,
             isMinimized: c.isMinimized,
             author: c.author?.login ?? "unknown",
-            authorType: mapAuthorType(c.author?.__typename),
+            authorType: mapAuthorType(c.author?.__typename, c.author?.login),
             body: c.body,
             url: c.url,
             createdAtUnix: parseCreatedAt(c.createdAt),
@@ -28,7 +28,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
             line: t.line ?? comment?.line ?? null,
             startLine: t.startLine ?? comment?.startLine ?? null,
             author: comment?.author?.login ?? "unknown",
-            authorType: mapAuthorType(comment?.author?.__typename),
+            authorType: mapAuthorType(comment?.author?.__typename, comment?.author?.login),
             body: comment?.body ?? "",
             url: comment?.url ?? "",
             createdAtUnix: comment ? parseCreatedAt(comment.createdAt) : 0,
@@ -39,7 +39,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         id: c.id,
         isMinimized: c.isMinimized,
         author: c.author?.login ?? "unknown",
-        authorType: mapAuthorType(c.author?.__typename),
+        authorType: mapAuthorType(c.author?.__typename, c.author?.login),
         body: c.body,
         url: c.url,
         createdAtUnix: parseCreatedAt(c.createdAt),
@@ -47,7 +47,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
     const changesRequestedReviews = rawReviewNodes.map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     const reviewSummaries = rawReviewSummaryNodes
@@ -55,7 +55,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         .map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     // APPROVED reviews often have empty bodies (clicking "Approve" without a comment), so
@@ -66,7 +66,7 @@ export function parseRawPr(raw, rawThreadPages, rawCommentNodes, rawReviewNodes,
         .map((r) => ({
         id: r.id,
         author: r.author?.login ?? "unknown",
-        authorType: mapAuthorType(r.author?.__typename),
+        authorType: mapAuthorType(r.author?.__typename, r.author?.login),
         body: r.body,
     }));
     const checks = rawCheckNodes.flatMap((node) => {

package/bin/reporters/agent.mjs

@@ -22,6 +22,7 @@ export function toAgentThread(t) {
         ...(t.authorType !== undefined && { authorType: t.authorType }),
         body: t.body,
         url: t.url,
+        ...(t.edited === true && { edited: true }),
         ...(t.comments !== undefined && {
             comments: t.comments.map((c) => ({
                 id: c.id,

package/bin/threads/transcript.mjs

@@ -22,10 +22,10 @@ export function threadComments(thread) {
         },
     ];
 }
-export function threadTranscriptBody(thread) {
-    if (!thread.comments || thread.comments.length === 0)
-        return thread.body;
-    return threadComments(thread)
-        .map((c) => `${c.id}\n${c.body}`)
-        .join("\n\n--- thread comment ---\n\n");
+const THREAD_COMMENT_SEPARATOR = "\n\n--- thread comment ---\n\n";
+export function threadTranscriptBody(thread, appendedBodies = []) {
+    const bodies = thread.comments && thread.comments.length > 0
+        ? threadComments(thread).map((c) => c.body)
+        : [thread.body];
+    return [...bodies, ...appendedBodies].join(THREAD_COMMENT_SEPARATOR);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.22.0",
+  "version": "0.23.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "license": "MIT",
   "author": "Jonathan Ong",

package/plugins/pr-shepherd/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.22.0",
+  "version": "0.23.0",
   "description": "Autonomous PR CI monitor and review-comment resolver for Codex.",
   "author": {
     "name": "Jonathan Ong",

package/plugins/pr-shepherd/skills/mark-files-as-viewed/SKILL.md

@@ -0,0 +1,31 @@
+---
+name: mark-files-as-viewed
+description: 'Mark PR changed files as viewed in GitHub with pr-shepherd. Use for requests like "mark tests as viewed" or "mark these files as viewed".'
+user-invocable: true
+argument-hint: "[PR number or URL] [files|tests|--tests|--match REGEX]"
+allowed-tools: ["Bash", "Read", "Grep", "Glob"]
+---
+
+# mark-files-as-viewed
+
+Thin dispatcher for marking GitHub PR changed files as viewed.
+
+## Arguments: $ARGUMENTS
+
+## Steps
+
+1. **Resolve arguments:**
+   - If `$ARGUMENTS` contains a PR number, use it.
+   - If `$ARGUMENTS` contains a GitHub PR URL, extract the number.
+   - Otherwise, infer: `gh pr view --json number --jq .number`
+   - Treat a standalone `tests` argument as `--tests`.
+   - Preserve explicit file paths, `--tests`, and `--match <regex>` arguments.
+   - If no PR found, report an error and stop.
+
+2. **Run `pr-shepherd`:**
+
+   ```bash
+   pr-shepherd mark-files-as-viewed <N> <selectors>
+   ```
+
+   Print the full output.