poe-code 3.0.342 → 3.0.343

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.342",
+  "version": "3.0.343",
   "description": "CLI tool to configure Poe API for developer workflows.",
   "type": "module",
   "main": "./dist/index.js",

package/packages/maestro/dist/config/schema.js

@@ -46,11 +46,12 @@ export function resolveConfig(raw, cwd) {
             intervalMs: readPositiveInteger(getOwnEntry(polling, "interval_ms"), 30_000, "polling.interval_ms")
         },
         workspace: {
-            root: resolvePathValue(readString(getOwnEntry(workspace, "root")) ?? path.join(os.tmpdir(), "poe-code-maestro"), cwd)
+            root: resolvePathValue(readString(getOwnEntry(workspace, "root"), "workspace.root") ??
+                path.join(os.tmpdir(), "poe-code-maestro"), cwd)
         },
         agent: {
-            service: readString(getOwnEntry(agent, "service")) ?? "codex",
-            list: readString(resolveStringValue(getOwnEntry(agent, "list"))),
+            service: readString(getOwnEntry(agent, "service"), "agent.service") ?? "codex",
+            list: readString(resolveStringValue(getOwnEntry(agent, "list")), "agent.list"),
             maxConcurrentAgents: readPositiveInteger(getOwnEntry(agent, "max_concurrent_agents"), 1, "agent.max_concurrent_agents"),
             maxRetryBackoffMs: readNonNegativeInteger(getOwnEntry(agent, "max_retry_backoff_ms"), 300_000, "agent.max_retry_backoff_ms")
         }
@@ -154,25 +155,34 @@ function expandHome(value) {
     }
     return value;
 }
-function readString(value) {
-    if (typeof value !== "string") {
+function readString(value, field) {
+    if (value === undefined) {
         return undefined;
     }
+    if (typeof value !== "string") {
+        throw new Error(`Expected "${field}" to be a string.`);
+    }
     const trimmed = value.trim();
     return trimmed.length > 0 ? value : undefined;
 }
-function readNumber(value, fallback) {
-    return typeof value === "number" && Number.isFinite(value) ? value : fallback;
+function readNumber(value, fallback, field) {
+    if (value === undefined) {
+        return fallback;
+    }
+    if (typeof value !== "number" || !Number.isFinite(value)) {
+        throw new Error(`Expected "${field}" to be a number.`);
+    }
+    return value;
 }
 function readPositiveInteger(value, fallback, field) {
-    const numberValue = readNumber(value, fallback);
+    const numberValue = readNumber(value, fallback, field);
     if (!Number.isInteger(numberValue) || numberValue <= 0) {
         throw new Error(`Expected "${field}" to be a positive integer.`);
     }
     return numberValue;
 }
 function readNonNegativeInteger(value, fallback, field) {
-    const numberValue = readNumber(value, fallback);
+    const numberValue = readNumber(value, fallback, field);
     if (!Number.isInteger(numberValue) || numberValue < 0) {
         throw new Error(`Expected "${field}" to be a non-negative integer.`);
     }

package/packages/maestro/dist/config/validate.js

@@ -8,10 +8,14 @@ export function validateStateDefinitions(value) {
         throw new Error("Workflow config requires at least one state.");
     }
     for (const [name, definition] of entries) {
+        const stateName = String(name);
+        if (stateName.trim().length === 0) {
+            throw new Error("State names must not be empty.");
+        }
         if (!isRecord(definition)) {
-            throw new Error(`State "${String(name)}" must be an object.`);
+            throw new Error(`State "${stateName}" must be an object.`);
         }
-        validateStateDefinition(String(name), definition);
+        validateStateDefinition(stateName, definition);
     }
 }
 export async function validateDispatch(cfg, taskList) {
@@ -86,6 +90,9 @@ function validateStateDefinition(name, definition) {
     if (prompt !== undefined && typeof prompt !== "string") {
         throw new Error(`State "${name}" prompt must be a string.`);
     }
+    if (typeof prompt === "string" && prompt.trim().length === 0) {
+        throw new Error(`State "${name}" prompt must not be empty.`);
+    }
     if (terminal !== undefined && typeof terminal !== "boolean") {
         throw new Error(`State "${name}" terminal must be a boolean.`);
     }

package/packages/maestro/dist/workflow-path.js

@@ -7,5 +7,20 @@ export function resolveWorkflowPath(name, cwd) {
     return path.resolve(cwd, filename);
 }
 function isValidWorkflowName(name) {
-    return name.length > 0 && !name.includes("/") && !name.includes("\\");
+    if (name.length === 0 || name.trim() !== name || name === "." || name === "..") {
+        return false;
+    }
+    for (const character of name) {
+        if (!isWorkflowNameCharacter(character)) {
+            return false;
+        }
+    }
+    return true;
+}
+function isWorkflowNameCharacter(character) {
+    const code = character.charCodeAt(0);
+    const isUppercase = code >= 65 && code <= 90;
+    const isLowercase = code >= 97 && code <= 122;
+    const isDigit = code >= 48 && code <= 57;
+    return isUppercase || isLowercase || isDigit || character === "_" || character === "-";
 }