next-issue 1.0.4 → 1.0.5

package/README.md

@@ -325,8 +325,9 @@ A `setupCommand` that fails, or that does not finish in `setupTimeoutMinutes`,
 hands the issue to a person. The wait for the checks has its own limit, so
 `commandTimeoutMinutes` holds for every other `git` and `gh` command. State per
 issue goes to `.next-issue/<issue>.json`, so a new run continues where the last
-run stopped. A state file that is not valid JSON gives the issue an `error`
-outcome that names the file, rather than a silent skip.
+run stopped. A state file that is not valid JSON, or that does not hold a valid
+state, gives the issue an `error` outcome that names the file, rather than a
+silent skip.
 
 ## Retry an issue that waits for a person
 

package/dist/state.mjs

@@ -19,12 +19,48 @@ export async function readState(repo, issue) {
         }
         throw error;
     }
+    let parsed;
     try {
-        return JSON.parse(text);
+        parsed = JSON.parse(text);
     }
     catch {
         throw new Error(`the state file ${path} is not valid JSON`);
     }
+    if (!isState(parsed)) {
+        throw new Error(`the state file ${path} does not hold a valid state`);
+    }
+    return parsed;
+}
+const PHASES = ["claimed", "implemented", "review"];
+function isRecord(value) {
+    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";
+}
+function isReviewRound(value) {
+    return (isRecord(value) &&
+        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" &&
+        PHASES.includes(value.phase) &&
+        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" &&
+        Array.isArray(value.reviewLog) &&
+        value.reviewLog.every(isReviewRound));
 }
 export async function savedIssues(repo) {
     let names;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "next-issue",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Autonomous issue harness driven by the Claude Agent SDK",
   "keywords": [
     "github",