next-issue 2.5.6 → 2.5.8

package/dist/exec.mjs

@@ -1,6 +1,8 @@
+import { AsyncLocalStorage } from 'node:async_hooks';
 import { spawn } from 'node:child_process';
 let observer;
-let defaultTimeoutMs = 10 * 60_000;
+const DEFAULT_TIMEOUT_MS = 10 * 60_000;
+const defaultTimeout = new AsyncLocalStorage();
 const STDERR_TAIL = 200_000;
 const groups = new Set();
 const stoppers = new Set();
@@ -50,12 +52,12 @@ export function track(child) {
 export function setCommandObserver(next) {
     observer = next;
 }
-export function setDefaultTimeout(ms) {
-    defaultTimeoutMs = ms;
+export function withDefaultTimeout(ms, body) {
+    return defaultTimeout.run(ms, body);
 }
 export function run(command, args, options = {}) {
     const started = Date.now();
-    const limit = options.timeoutMs ?? defaultTimeoutMs;
+    const limit = options.timeoutMs ?? defaultTimeout.getStore() ?? DEFAULT_TIMEOUT_MS;
     return new Promise((resolve, reject) => {
         const child = spawn(command, args, {
             cwd: options.cwd,

package/dist/github.mjs

@@ -100,11 +100,25 @@ export function labelArgs(add, remove, current) {
     return args;
 }
 export async function setLabel(repo, kind, number, add, remove) {
-    await ensureLabel(repo, add);
+    await createUnknownLabel(repo, add);
     const current = await labelsOf(repo, kind, number);
-    await gh(repo, [kind, 'edit', String(number), ...labelArgs(add, remove, current)]);
+    const args = [kind, 'edit', String(number), ...labelArgs(add, remove, current)];
+    try {
+        await gh(repo, args);
+    }
+    catch (error) {
+        KNOWN_LABELS.delete(slug(repo));
+        const created = await createUnknownLabel(repo, add);
+        if (created === 'known') {
+            throw error;
+        }
+        if (created instanceof Error) {
+            throw created;
+        }
+        await gh(repo, args);
+    }
 }
-async function ensureLabel(repo, label) {
+async function createUnknownLabel(repo, label) {
     let known = KNOWN_LABELS.get(slug(repo));
     if (known === undefined) {
         const raw = await gh(repo, ['label', 'list', '--limit', '1000', '--json', 'name']);
@@ -112,14 +126,16 @@ async function ensureLabel(repo, label) {
         KNOWN_LABELS.set(slug(repo), known);
     }
     if (known.has(label)) {
-        return;
+        return 'known';
     }
-    const result = await run('gh', ['label', 'create', label, '--repo', slug(repo)], {
-        cwd: repo.root,
-    });
-    if (result.code === 0) {
-        known.add(label);
+    try {
+        await ghOnce(repo, ['label', 'create', label]);
+    }
+    catch (error) {
+        return error instanceof Error ? error : new Error(String(error));
     }
+    known.add(label);
+    return 'created';
 }
 export async function findPr(repo, branch) {
     const raw = await gh(repo, [

package/dist/protocol.mjs

@@ -0,0 +1,37 @@
+export function isNewer(candidate, current) {
+    const [leftCore = '', ...leftRest] = candidate.replace(/\+.*$/, '').split('-');
+    const [rightCore = '', ...rightRest] = current.replace(/\+.*$/, '').split('-');
+    const left = leftCore.split('.').map(Number);
+    const right = rightCore.split('.').map(Number);
+    for (let index = 0; index < Math.max(left.length, right.length); index += 1) {
+        const difference = (left[index] ?? 0) - (right[index] ?? 0);
+        if (difference !== 0) {
+            return difference > 0;
+        }
+    }
+    const leftPrerelease = leftRest.length > 0 ? leftRest.join('-').split('.') : [];
+    const rightPrerelease = rightRest.length > 0 ? rightRest.join('-').split('.') : [];
+    if (leftPrerelease.length === 0 || rightPrerelease.length === 0) {
+        return leftPrerelease.length === 0 && rightPrerelease.length > 0;
+    }
+    for (let index = 0; index < Math.max(leftPrerelease.length, rightPrerelease.length); index += 1) {
+        const leftPart = leftPrerelease[index];
+        const rightPart = rightPrerelease[index];
+        if (leftPart === undefined || rightPart === undefined) {
+            return rightPart === undefined;
+        }
+        if (leftPart === rightPart) {
+            continue;
+        }
+        const leftNumeric = /^\d+$/.test(leftPart);
+        const rightNumeric = /^\d+$/.test(rightPart);
+        if (leftNumeric && rightNumeric) {
+            return Number(leftPart) > Number(rightPart);
+        }
+        if (leftNumeric || rightNumeric) {
+            return rightNumeric;
+        }
+        return leftPart > rightPart;
+    }
+    return false;
+}

package/dist/run.mjs

@@ -2,7 +2,7 @@ import { AsyncLocalStorage } from 'node:async_hooks';
 import { writeFile } from 'node:fs/promises';
 import { join } from 'node:path';
 import { loadConfig } from './config.mjs';
-import { setCommandObserver, setDefaultTimeout } from './exec.mjs';
+import { setCommandObserver, withDefaultTimeout } from './exec.mjs';
 import { detectRepo } from './git.mjs';
 import { defaultBranch } from './github.mjs';
 import { exitCode, runIssues, watchIssues } from './loop.mjs';
@@ -25,7 +25,9 @@ export function recordCommands(recorder, work) {
 }
 export async function runRepo(root, args, controls) {
     const config = await loadConfig(root);
-    setDefaultTimeout(config.commandTimeoutMinutes * 60_000);
+    return withDefaultTimeout(config.commandTimeoutMinutes * 60_000, () => runWithConfig(root, args, controls, config));
+}
+async function runWithConfig(root, args, controls, config) {
     const repo = await detectRepo(root, config.remote);
     const recorder = await Recorder.create(repo.root, args.level, controls.onEvent);
     const pruned = await pruneRuns(repo.root, config.keepRunDays);

package/dist/serve.mjs

@@ -1,5 +1,5 @@
 import { rmSync } from 'node:fs';
-import { chmod, readFile, rm, writeFile } from 'node:fs/promises';
+import { readFile, rm, writeFile } from 'node:fs/promises';
 import { createServer } from 'node:http';
 import { join } from 'node:path';
 import { getAgentDir } from '@earendil-works/pi-coding-agent';
@@ -8,6 +8,7 @@ import { parseArgs } from './args.mjs';
 import { stopChildren } from './exec.mjs';
 import { message } from './observe.mjs';
 import { applyEvent, startProgress } from './progress.mjs';
+import { isNewer } from './protocol.mjs';
 import { runRepo } from './run.mjs';
 import { packageVersion } from './version.mjs';
 export function socketPaths(agentDir = getAgentDir()) {
@@ -16,17 +17,6 @@ export function socketPaths(agentDir = getAgentDir()) {
         lockPath: join(agentDir, 'next-issue.lock'),
     };
 }
-export function isNewer(candidate, current) {
-    const left = candidate.split('.').map(Number);
-    const right = current.split('.').map(Number);
-    for (let index = 0; index < Math.max(left.length, right.length); index += 1) {
-        const difference = (left[index] ?? 0) - (right[index] ?? 0);
-        if (difference !== 0) {
-            return difference > 0;
-        }
-    }
-    return false;
-}
 function alive(pid) {
     try {
         process.kill(pid, 0);
@@ -191,12 +181,17 @@ export async function startWorker(options) {
     });
     await new Promise((resolve, reject) => {
         server.once('error', reject);
-        server.listen(options.socketPath, () => {
-            server.off('error', reject);
-            resolve();
-        });
+        const umask = process.umask(0o177);
+        try {
+            server.listen(options.socketPath, () => {
+                server.off('error', reject);
+                resolve();
+            });
+        }
+        finally {
+            process.umask(umask);
+        }
     });
-    await chmod(options.socketPath, 0o600);
     return { closed, close };
 }
 export async function serve() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "2.5.6",
+  "version": "2.5.8",
   "description": "Autonomous issue harness driven by the pi SDK",
   "keywords": [
     "github",