next-issue 1.0.4 → 1.0.6

package/README.md

@@ -233,7 +233,8 @@ adds `.next-issue/` to `.gitignore`. It stops when the config file is there
 already; `--force` replaces it. The file holds no `setupCommand`, because the
 harness deduces one from the lock file of the project.
 
-A field that is not in the file keeps its default. A file that is not valid
+With no file, every field keeps its default. A field that is not in the file
+keeps its default. A file that cannot be read, a file that is not valid
 JSON, a field with a value of the wrong type and a field name that the harness
 does not know all stop the run, so a typo cannot pass without a word.
 
@@ -325,8 +326,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/config.mjs

@@ -73,8 +73,11 @@ export async function loadConfig(root) {
     try {
         raw = await readFile(join(root, CONFIG_FILE), "utf8");
     }
-    catch {
-        return defaults();
+    catch (error) {
+        if (error.code === "ENOENT") {
+            return defaults();
+        }
+        throw new Error(`${CONFIG_FILE} cannot be read: ${message(error)}`);
     }
     let parsed;
     try {

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.6",
   "description": "Autonomous issue harness driven by the Claude Agent SDK",
   "keywords": [
     "github",