@ravshansbox/pi-next-issue 1.0.10 → 1.0.12

package/README.md

@@ -39,7 +39,7 @@ The implementer and the reviewer read the issue title, the body and the comments
 
 Each agent is a pi session in the worktree. It uses the model and thinking level from your pi settings, and loads all of your extensions except this one.
 
-The state of each issue is saved in `.worktrees/<n>.json`. A run holds `.worktrees/run.lock`, which keeps the process id, so that only one pi at a time works on a repository. A second run fails at once. A lock whose process is no longer alive is taken over. If the operating system gives that process id to another process, the lock stays; remove `.worktrees/run.lock` by hand when no run is going. The extension adds `/.worktrees/` to `.git/info/exclude`.
+The state of each issue is saved in `.worktrees/<n>.json`. Each loop removes the saved state of an issue that is closed. A run holds `.worktrees/run.lock`, which keeps the process id, so that only one pi at a time works on a repository. A second run fails at once. A lock whose process is no longer alive is taken over. If the operating system gives that process id to another process, the lock stays; remove `.worktrees/run.lock` by hand when no run is going. The extension adds `/.worktrees/` to `.git/info/exclude`.
 
 ## Config
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ravshansbox/pi-next-issue",
-  "version": "1.0.10",
+  "version": "1.0.12",
   "description": "Pi extension that works through GitHub issues in the background",
   "license": "MIT",
   "type": "module",

package/src/github.ts

@@ -38,7 +38,10 @@ type RawIssue = {
 
 const ISSUE_FIELDS = 'number,title,body,labels,createdAt';
 
-export async function openIssues(repo: Repo, saved: number[]): Promise<Issue[]> {
+export async function openIssues(
+  repo: Repo,
+  saved: number[],
+): Promise<{ open: Issue[]; closed: number[] }> {
   const raw = await gh(repo, [
     'issue',
     'list',
@@ -55,6 +58,7 @@ export async function openIssues(repo: Repo, saved: number[]): Promise<Issue[]>
   ]);
   const listed = JSON.parse(raw) as RawIssue[];
   const known = new Set(listed.map((item) => item.number));
+  const closed: number[] = [];
   for (const number of saved.filter((item) => !known.has(item))) {
     const view = await run(
       'gh',
@@ -67,9 +71,11 @@ export async function openIssues(repo: Repo, saved: number[]): Promise<Issue[]>
     const item = JSON.parse(view.stdout) as RawIssue & { state: string };
     if (item.state === 'OPEN') {
       listed.push(item);
+    } else {
+      closed.push(number);
     }
   }
-  return listed
+  const open = listed
     .sort((a, b) => a.createdAt.localeCompare(b.createdAt))
     .map((item) => ({
       number: item.number,
@@ -77,6 +83,7 @@ export async function openIssues(repo: Repo, saved: number[]): Promise<Issue[]>
       body: item.body ?? '',
       labels: item.labels.map((label) => label.name),
     }));
+  return { open, closed };
 }
 
 export async function comments(repo: Repo, issue: number): Promise<string[]> {
@@ -227,15 +234,24 @@ export async function failedLogs(repo: Repo, pr: number, maxChars: number): Prom
   const failed = (await readChecks(repo, pr)).filter(
     (check) => check.bucket === 'fail' || check.bucket === 'cancel',
   );
-  const share = Math.max(1, Math.floor(maxChars / Math.max(1, failed.length)));
-  const parts: string[] = [];
+  const groups = new Map<string, { names: string[]; runId: string | undefined }>();
   for (const check of failed) {
-    parts.push(`## ${check.name}`);
     const runId = /\/actions\/runs\/(\d+)/.exec(check.link)?.[1];
-    if (runId !== undefined) {
-      const logs = await run('gh', ['run', 'view', runId, '--log-failed', '--repo', repo.slug], {
-        cwd: repo.root,
-      });
+    const key = runId ?? `check:${check.name}`;
+    const group = groups.get(key) ?? { names: [], runId };
+    group.names.push(check.name);
+    groups.set(key, group);
+  }
+  const share = Math.max(1, Math.floor(maxChars / Math.max(1, groups.size)));
+  const parts: string[] = [];
+  for (const group of groups.values()) {
+    parts.push(`## ${group.names.join(', ')}`);
+    if (group.runId !== undefined) {
+      const logs = await run(
+        'gh',
+        ['run', 'view', group.runId, '--log-failed', '--repo', repo.slug],
+        { cwd: repo.root },
+      );
       parts.push((logs.stdout || logs.stderr).slice(-share));
     }
   }

package/src/ports.ts

@@ -10,7 +10,11 @@ export async function createPorts(root: string, config: Config): Promise<Ports>
   const base = await github.defaultBranch(repo);
   const timeoutMs = config.checkTimeoutMinutes * 60_000;
   return {
-    openIssues: async () => github.openIssues(repo, await state.savedIssues(root)),
+    openIssues: async () => {
+      const { open, closed } = await github.openIssues(repo, await state.savedIssues(root));
+      await Promise.all(closed.map((issue) => state.dropState(root, issue)));
+      return open;
+    },
     comments: (issue) => github.comments(repo, issue),
     setLabel: (issue, add, remove) => github.setLabel(repo, issue, add, remove),
     commentOnIssue: (issue, body) => github.commentOnIssue(repo, issue, body),