next-issue 2.4.0 → 2.4.2

package/dist/state.mjs

@@ -1,6 +1,6 @@
-import { mkdir, readdir, readFile, rm, writeFile } from "node:fs/promises";
-import { join } from "node:path";
-export const STATE_DIR = ".next-issue";
+import { mkdir, readdir, readFile, rm, writeFile } from 'node:fs/promises';
+import { join } from 'node:path';
+export const STATE_DIR = '.next-issue';
 function dir(repo) {
     return join(repo.root, STATE_DIR);
 }
@@ -11,10 +11,10 @@ export async function readState(repo, issue) {
     const path = file(repo, issue);
     let text;
     try {
-        text = await readFile(path, "utf8");
+        text = await readFile(path, 'utf8');
     }
     catch (error) {
-        if (error.code === "ENOENT") {
+        if (error.code === 'ENOENT') {
             return undefined;
         }
         throw error;
@@ -31,34 +31,34 @@ export async function readState(repo, issue) {
     }
     return parsed;
 }
-const PHASES = ["claimed", "implemented", "review"];
+const PHASES = ['claimed', 'implemented', 'review'];
 function isRecord(value) {
-    return typeof value === "object" && value !== null;
+    return typeof value === 'object' && value !== null;
 }
 function isOptional(value, type) {
     return value === undefined || typeof value === type;
 }
 function isBaseFix(value) {
-    return isRecord(value) && typeof value.sha === "string" && typeof value.head === "string";
+    return isRecord(value) && typeof value.sha === 'string' && typeof value.head === 'string';
 }
 function isReviewRound(value) {
     return (isRecord(value) &&
-        typeof value.round === "number" &&
-        typeof value.fingerprint === "string" &&
-        typeof value.findings === "string" &&
-        isOptional(value.dispute, "string"));
+        typeof value.round === 'number' &&
+        typeof value.fingerprint === 'string' &&
+        typeof value.findings === 'string' &&
+        isOptional(value.dispute, 'string'));
 }
 function isState(value) {
     return (isRecord(value) &&
-        typeof value.issue === "number" &&
+        typeof value.issue === 'number' &&
         PHASES.includes(value.phase) &&
-        typeof value.branch === "string" &&
-        isOptional(value.pr, "number") &&
-        isOptional(value.title, "string") &&
-        typeof value.ciFixes === "number" &&
-        isOptional(value.commitFixes, "number") &&
+        typeof value.branch === 'string' &&
+        isOptional(value.pr, 'number') &&
+        isOptional(value.title, 'string') &&
+        typeof value.ciFixes === 'number' &&
+        isOptional(value.commitFixes, 'number') &&
         (value.baseFix === undefined || isBaseFix(value.baseFix)) &&
-        typeof value.reviewRounds === "number" &&
+        typeof value.reviewRounds === 'number' &&
         Array.isArray(value.reviewLog) &&
         value.reviewLog.every(isReviewRound));
 }
@@ -68,7 +68,7 @@ export async function savedIssues(repo) {
         names = await readdir(dir(repo));
     }
     catch (error) {
-        if (error.code === "ENOENT") {
+        if (error.code === 'ENOENT') {
             return [];
         }
         throw error;

package/dist/summary.mjs

@@ -1,16 +1,16 @@
-const OUTCOMES = ["done", "needs-human", "error", "skipped", "stopped"];
+const OUTCOMES = ['done', 'needs-human', 'error', 'skipped', 'stopped'];
 export function formatSummary(summary) {
-    const tally = OUTCOMES.map((outcome) => `${summary.issues.filter((report) => report.outcome === outcome).length} ${outcome}`).join(", ");
+    const tally = OUTCOMES.map((outcome) => `${summary.issues.filter((report) => report.outcome === outcome).length} ${outcome}`).join(', ');
     const lines = [`run ${summary.runId}`, `issues ${summary.issues.length}: ${tally}`];
     for (const report of summary.issues) {
         lines.push(`  ${issueLine(report)}`);
     }
     const roles = Object.entries(summary.roles)
         .map(([role, totals]) => `${role} ${tokens(totals.total)} in ${totals.calls}`)
-        .join(", ");
-    lines.push(`tokens ${tokens(summary.total.total)}${roles.length > 0 ? ` (${roles})` : ""}`);
+        .join(', ');
+    lines.push(`tokens ${tokens(summary.total.total)}${roles.length > 0 ? ` (${roles})` : ''}`);
     lines.push(`log ${summary.logFile}`);
-    return `${lines.join("\n")}\n`;
+    return `${lines.join('\n')}\n`;
 }
 function issueLine(report) {
     const parts = [`#${report.issue} ${report.outcome.padEnd(11)}`];
@@ -29,12 +29,12 @@ function issueLine(report) {
     if (report.reason !== undefined) {
         parts.push(`(${report.reason})`);
     }
-    return parts.join("  ").trimEnd();
+    return parts.join('  ').trimEnd();
 }
 function duration(ms) {
     const seconds = Math.round(ms / 1000);
     const minutes = Math.floor(seconds / 60);
-    return minutes === 0 ? `${seconds}s` : `${minutes}m${String(seconds % 60).padStart(2, "0")}s`;
+    return minutes === 0 ? `${seconds}s` : `${minutes}m${String(seconds % 60).padStart(2, '0')}s`;
 }
 function tokens(value) {
     if (value < 1000) {

package/dist/verdict.mjs

@@ -1,42 +1,44 @@
 export const VERDICT_SCHEMA = {
-    type: "object",
+    type: 'object',
     properties: {
-        summary: { type: "string", description: "Short summary of the review" },
+        summary: { type: 'string', description: 'Short summary of the review' },
         findings: {
-            type: "array",
-            description: "One entry per problem. Empty when nothing is wrong.",
+            type: 'array',
+            description: 'One entry per problem. Empty when nothing is wrong.',
             items: {
-                type: "object",
+                type: 'object',
                 properties: {
                     severity: {
-                        type: "string",
-                        enum: ["blocking", "minor"],
-                        description: "blocking for a wrong result, a missing part of the issue, a regression or a broken convention; minor for taste and style",
+                        type: 'string',
+                        enum: ['blocking', 'minor'],
+                        description: 'blocking for a wrong result, a missing part of the issue, a regression or a broken convention; minor for taste and style',
                     },
-                    detail: { type: "string", description: "What is wrong and where" },
+                    detail: { type: 'string', description: 'What is wrong and where' },
                 },
-                required: ["severity", "detail"],
+                required: ['severity', 'detail'],
             },
         },
     },
-    required: ["summary", "findings"],
+    required: ['summary', 'findings'],
 };
-const SEVERITIES = ["blocking", "minor"];
+const SEVERITIES = ['blocking', 'minor'];
 export function readVerdict(value) {
-    if (typeof value !== "object" || value === null) {
+    if (typeof value !== 'object' || value === null) {
         return undefined;
     }
     const { summary, findings } = value;
-    if (typeof summary !== "string" || !Array.isArray(findings)) {
+    if (typeof summary !== 'string' || !Array.isArray(findings)) {
         return undefined;
     }
     const checked = [];
     for (const item of findings) {
-        if (typeof item !== "object" || item === null) {
+        if (typeof item !== 'object' || item === null) {
             return undefined;
         }
         const { severity, detail } = item;
-        if (typeof severity !== "string" || !SEVERITIES.includes(severity) || typeof detail !== "string") {
+        if (typeof severity !== 'string' ||
+            !SEVERITIES.includes(severity) ||
+            typeof detail !== 'string') {
             return undefined;
         }
         checked.push({ severity: severity, detail });
@@ -44,27 +46,30 @@ export function readVerdict(value) {
     return { summary, findings: checked };
 }
 export function blockingFindings(verdict) {
-    return verdict.findings.filter((finding) => finding.severity === "blocking");
+    return verdict.findings.filter((finding) => finding.severity === 'blocking');
 }
 export function openFindings(verdict, lastRound) {
     return lastRound ? blockingFindings(verdict) : verdict.findings;
 }
-const LEFT = "The findings above stay as they are: no blocking finding is open and the review budget is spent.";
+const LEFT = 'The findings above stay as they are: no blocking finding is open and the review budget is spent.';
 export function formatVerdict(verdict, open) {
     const approved = open.length === 0;
-    const head = approved ? "Review: approved" : "Review: changes requested";
+    const head = approved ? 'Review: approved' : 'Review: changes requested';
     const lines = verdict.findings.map((finding) => `- **${finding.severity}** ${finding.detail}`);
-    const note = approved && verdict.findings.length > 0 ? LEFT : "";
-    return [`### ${head}`, verdict.summary, lines.join("\n"), note]
+    const note = approved && verdict.findings.length > 0 ? LEFT : '';
+    return [`### ${head}`, verdict.summary, lines.join('\n'), note]
         .filter((part) => part.length > 0)
-        .join("\n\n");
+        .join('\n\n');
 }
 export function formatFindings(findings) {
-    return findings.map((finding) => `- ${finding.detail}`).join("\n");
+    return findings.map((finding) => `- ${finding.detail}`).join('\n');
 }
 export function fingerprint(findings) {
     return findings
-        .map((finding) => finding.detail.toLowerCase().replace(/[^a-z0-9]+/g, " ").trim())
+        .map((finding) => finding.detail
+        .toLowerCase()
+        .replace(/[^a-z0-9]+/g, ' ')
+        .trim())
         .sort()
-        .join("|");
+        .join('|');
 }

package/dist/version.mjs

@@ -1,6 +1,6 @@
-import { readFile } from "node:fs/promises";
-import { join } from "node:path";
+import { readFile } from 'node:fs/promises';
+import { join } from 'node:path';
 export async function packageVersion() {
-    const raw = await readFile(join(import.meta.dirname, "..", "package.json"), "utf8");
+    const raw = await readFile(join(import.meta.dirname, '..', 'package.json'), 'utf8');
     return JSON.parse(raw).version;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "2.4.0",
+  "version": "2.4.2",
   "description": "Autonomous issue harness driven by the pi SDK",
   "keywords": [
     "github",
@@ -34,6 +34,10 @@
     "start": "node src/cli.mts",
     "test": "node --test \"test/**/*.test.mts\"",
     "typecheck": "tsc --noEmit",
+    "lint": "oxlint --deny-warnings",
+    "lint:fix": "oxlint --fix",
+    "format": "oxfmt",
+    "format:check": "oxfmt --check",
     "prepack": "npm run build",
     "prepublishOnly": "npm run typecheck && npm test"
   },
@@ -41,6 +45,8 @@
     "@earendil-works/pi-ai": "^1.5.0",
     "@types/node": "^26.6.6",
     "@types/ws": "^8.18.2",
+    "oxfmt": "^0.74.0",
+    "oxlint": "^1.89.0",
     "typescript": "^7.0.2"
   },
   "dependencies": {