poe-code 3.0.361 → 3.0.362

package/dist/index.js

@@ -26384,7 +26384,7 @@ async function runDocumentWorkflow(options) {
       return;
     }
     let shouldStop = false;
-    for (let iteration = 0; iteration < initialWorkflow.maxIterations; iteration += 1) {
+    for (let iteration = 0; ; iteration += 1) {
       throwIfAborted(options.signal);
       currentWorkflow = iteration === 0 ? initialWorkflow : await readWorkflow2();
       if (iteration >= currentWorkflow.maxIterations) {
@@ -26419,6 +26419,9 @@ async function runDocumentWorkflow(options) {
       if (shouldStop) {
         break;
       }
+      if (iteration + 1 >= currentWorkflow.maxIterations) {
+        break;
+      }
     }
   } catch (error3) {
     pendingError = error3;
@@ -70517,13 +70520,20 @@ function parseFrontmatterData(value) {
       throw new Error('Invalid Ralph frontmatter: "kind" must be "ralph".');
     }
   }
-  const statusValue = parsed ? getOwnEntry27(parsed, "status") : void 0;
-  const parsedStatus = isRecord26(statusValue) ? statusValue : void 0;
-  const state = parsePlanStatus(parsedStatus ? getOwnEntry27(parsedStatus, "state") : void 0) ?? parseLegacyStatus(statusValue) ?? defaults2.status.state;
-  const iteration = parseNonNegativeInteger2(parsedStatus ? getOwnEntry27(parsedStatus, "iteration") : void 0) ?? parseNonNegativeInteger2(parsed ? getOwnEntry27(parsed, "iteration") : void 0) ?? defaults2.status.iteration;
-  const agent3 = parseAgent2(parsed ? getOwnEntry27(parsed, "agent") : void 0);
+  const { state, iteration } = parseStatusFields(parsed, defaults2.status);
+  const agentValue = parsed ? getOwnEntry27(parsed, "agent") : void 0;
+  const agent3 = parseAgent2(agentValue);
+  if (parsed !== void 0 && hasOwnEntry3(parsed, "agent") && agent3 === void 0) {
+    throw new Error(
+      'Invalid Ralph frontmatter: "agent" must be a non-empty string or non-empty string array.'
+    );
+  }
   const extendsValue = parseBoolean2(parsed ? getOwnEntry27(parsed, "extends") : void 0);
-  const iterations = parsePositiveInteger(parsed ? getOwnEntry27(parsed, "iterations") : void 0);
+  const iterationsValue = parsed ? getOwnEntry27(parsed, "iterations") : void 0;
+  const iterations = parsePositiveInteger(iterationsValue);
+  if (parsed !== void 0 && hasOwnEntry3(parsed, "iterations") && iterations === void 0) {
+    throw new Error('Invalid Ralph frontmatter: "iterations" must be a positive integer.');
+  }
   const skills2 = parseSkills4(parsed ? getOwnEntry27(parsed, "skills") : void 0);
   const hooks = parseHooks4(parsed ? getOwnEntry27(parsed, "hooks") : void 0);
   return {
@@ -70538,6 +70548,50 @@ function parseFrontmatterData(value) {
     }
   };
 }
+function parseStatusFields(parsed, defaults2) {
+  if (parsed === void 0) {
+    return defaults2;
+  }
+  const statusValue = getOwnEntry27(parsed, "status");
+  const legacyIterationValue = getOwnEntry27(parsed, "iteration");
+  const hasStatus = hasOwnEntry3(parsed, "status");
+  const hasLegacyIteration = hasOwnEntry3(parsed, "iteration");
+  const parsedStatus = isRecord26(statusValue) ? statusValue : void 0;
+  if (hasLegacyIteration && parseNonNegativeInteger2(legacyIterationValue) === void 0) {
+    throw new Error('Invalid Ralph frontmatter: "iteration" must be a non-negative integer.');
+  }
+  if (parsedStatus !== void 0) {
+    rejectUnknownKeys2(parsedStatus, ["state", "iteration"], "status");
+    const statusStateValue = getOwnEntry27(parsedStatus, "state");
+    const statusIterationValue = getOwnEntry27(parsedStatus, "iteration");
+    const state = parsePlanStatus(statusStateValue);
+    const iteration = parseNonNegativeInteger2(statusIterationValue);
+    if (hasOwnEntry3(parsedStatus, "state") && state === void 0) {
+      throw new Error(
+        'Invalid Ralph frontmatter: "status.state" must be "open", "in_progress", "completed", or "failed".'
+      );
+    }
+    if (hasOwnEntry3(parsedStatus, "iteration") && iteration === void 0) {
+      throw new Error(
+        'Invalid Ralph frontmatter: "status.iteration" must be a non-negative integer.'
+      );
+    }
+    return {
+      state: state ?? defaults2.state,
+      iteration: iteration ?? parseNonNegativeInteger2(legacyIterationValue) ?? defaults2.iteration
+    };
+  }
+  const legacyState = parseLegacyStatus(statusValue);
+  if (hasStatus && legacyState === void 0) {
+    throw new Error(
+      'Invalid Ralph frontmatter: "status" must be "open", "pending", "cancelled", "overbake_abort", "in_progress", or "completed".'
+    );
+  }
+  return {
+    state: legacyState ?? defaults2.state,
+    iteration: parseNonNegativeInteger2(legacyIterationValue) ?? defaults2.iteration
+  };
+}
 function parseAgent2(value) {
   if (typeof value === "string") {
     const trimmed = value.trim();
@@ -70547,7 +70601,7 @@ function parseAgent2(value) {
     return void 0;
   }
   if (value.length === 0) {
-    return [];
+    return void 0;
   }
   const agents = [];
   for (const item of value) {
@@ -70641,8 +70695,11 @@ function parseBoolean2(value) {
 function isRecord26(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
+function hasOwnEntry3(record, key2) {
+  return Object.prototype.hasOwnProperty.call(record, key2);
+}
 function getOwnEntry27(record, key2) {
-  return Object.prototype.hasOwnProperty.call(record, key2) ? record[key2] : void 0;
+  return hasOwnEntry3(record, key2) ? record[key2] : void 0;
 }
 var ralphDocumentSchemaId, DEFAULT_STATUS;
 var init_frontmatter3 = __esm({
@@ -70848,7 +70905,7 @@ async function runRalph(options) {
           await updateFrontmatter2(fs29, absoluteDocPath, "open", iterationsCompleted);
           return;
         }
-        if (iterationNumber === config2.maxIterations) {
+        if (iterationNumber === currentConfig.maxIterations) {
           await updateFrontmatter2(fs29, absoluteDocPath, "completed", iterationsCompleted);
           stopReason = "max_iterations";
           return;
@@ -74136,7 +74193,7 @@ function resolveConfiguredTaskListOptions(cfg) {
   const tasksRecord = tasks;
   const taskType = getOwnEntry31(tasksRecord, "type");
   const taskProject = getOwnEntry31(tasksRecord, "project");
-  if (!hasOwnEntry3(tasksRecord, "path") && !(taskType === "gh-issues" && taskProject === void 0)) {
+  if (!hasOwnEntry4(tasksRecord, "path") && !(taskType === "gh-issues" && taskProject === void 0)) {
     return tasks;
   }
   return {
@@ -74148,11 +74205,11 @@ function resolveConfiguredTaskListOptions(cfg) {
     }
   };
 }
-function hasOwnEntry3(record, key2) {
+function hasOwnEntry4(record, key2) {
   return Object.prototype.hasOwnProperty.call(record, key2);
 }
 function getOwnEntry31(record, key2) {
-  return hasOwnEntry3(record, key2) ? record[key2] : void 0;
+  return hasOwnEntry4(record, key2) ? record[key2] : void 0;
 }
 function deriveStateEvents(stateOrder, terminalStateNames) {
   const terminalStates = new Set(terminalStateNames);
@@ -137933,7 +137990,7 @@ function readTaskListOptions(frontmatter, workflowPath) {
     );
   }
   const resolved = resolveStringValues2(tasks);
-  if (hasOwnEntry4(resolved, "path") && typeof resolved.path === "string") {
+  if (hasOwnEntry5(resolved, "path") && typeof resolved.path === "string") {
     resolved.path = resolveWorkflowRelativePath(resolved.path, workflowPath);
   }
   return resolved;
@@ -138096,11 +138153,11 @@ function resolveWorkflowRelativePath(value, workflowPath) {
 function asRecord6(value) {
   return isPlainRecord6(value) ? value : void 0;
 }
-function hasOwnEntry4(record, key2) {
+function hasOwnEntry5(record, key2) {
   return Object.prototype.hasOwnProperty.call(record, key2);
 }
 function getOwnEntry37(record, key2) {
-  return hasOwnEntry4(record, key2) ? record[key2] : void 0;
+  return hasOwnEntry5(record, key2) ? record[key2] : void 0;
 }
 function isPlainRecord6(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
@@ -138957,7 +139014,7 @@ var init_package2 = __esm({
   "package.json"() {
     package_default2 = {
       name: "poe-code",
-      version: "3.0.361",
+      version: "3.0.362",
       description: "CLI tool to configure Poe API for developer workflows.",
       type: "module",
       main: "./dist/index.js",