poe-code 3.0.341 → 3.0.343

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "poe-code",
-  "version": "3.0.341",
+  "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 === "-";
 }

package/packages/maestro-tui/dist/actions.js

@@ -5,11 +5,16 @@ export function buildOpenSourceAction(options) {
         id: "open-source",
         key: "o",
         label: "Open in $EDITOR",
-        predicate: (ctx) => getTask(options.taskByRowId(), ctx.row.id).sourcePath != null,
+        predicate: (ctx) => getSourcePath(getTask(options.taskByRowId(), ctx.row.id)) !== null,
         handler: async (ctx) => {
             const task = getTask(options.taskByRowId(), ctx.row.id);
+            const sourcePath = getSourcePath(task);
+            if (sourcePath === null) {
+                ctx.toast("No source file available.", "info");
+                return;
+            }
             await ctx.suspendAnd(async () => {
-                editFile(task.sourcePath, { env: options.variables });
+                editFile(sourcePath, { env: options.variables });
             });
             await ctx.refresh();
             ctx.toast(`Edited ${task.qualifiedId}`, "info");
@@ -41,14 +46,25 @@ function getIssueUrl(task) {
     if (typeof url !== "string") {
         return null;
     }
+    const trimmedUrl = url.trim();
+    if (trimmedUrl.length === 0) {
+        return null;
+    }
     try {
-        const parsedUrl = new URL(url);
-        return parsedUrl.protocol === "http:" || parsedUrl.protocol === "https:" ? url : null;
+        const parsedUrl = new URL(trimmedUrl);
+        return parsedUrl.protocol === "http:" || parsedUrl.protocol === "https:" ? trimmedUrl : null;
     }
     catch {
         return null;
     }
 }
+function getSourcePath(task) {
+    const sourcePath = task.sourcePath;
+    if (typeof sourcePath !== "string" || sourcePath.trim().length === 0) {
+        return null;
+    }
+    return sourcePath;
+}
 function getTask(taskByRowId, rowId) {
     const task = taskByRowId.get(rowId);
     if (task === undefined) {

package/packages/maestro-tui/dist/explorer-config.js

@@ -85,6 +85,7 @@ async function renderTaskDetailMarkdown(task, taskList) {
     const eventsMarkdown = await renderEventsMarkdown(task, taskList);
     const description = task.description.length > 0 ? task.description : "_No description._";
     const metadata = stringify(task.metadata).trimEnd();
+    const metadataFence = buildBacktickFence(metadata);
     return [
         `# ${task.name}`,
         "",
@@ -94,9 +95,9 @@ async function renderTaskDetailMarkdown(task, taskList) {
         "",
         "## Metadata",
         "",
-        "```yaml",
+        `${metadataFence}yaml`,
         metadata,
-        "```",
+        metadataFence,
         "",
         "## Next",
         "",
@@ -112,9 +113,45 @@ async function renderEventsMarkdown(task, taskList) {
         return events.map((event) => `- ${event}`).join("\n");
     }
     catch (err) {
-        return `_Could not load events: ${err.message}_`;
+        return `_Could not load events: ${formatThrownValue(err)}_`;
     }
 }
+function buildBacktickFence(content) {
+    return "`".repeat(Math.max(3, longestCharacterRun(content, "`") + 1));
+}
+function longestCharacterRun(content, expected) {
+    let longest = 0;
+    let current = 0;
+    for (const character of content) {
+        if (character === expected) {
+            current += 1;
+            longest = Math.max(longest, current);
+        }
+        else {
+            current = 0;
+        }
+    }
+    return longest;
+}
+function formatThrownValue(value) {
+    if (value instanceof Error && value.message.length > 0) {
+        return value.message;
+    }
+    if (typeof value === "string" && value.length > 0) {
+        return value;
+    }
+    try {
+        const json = JSON.stringify(value);
+        if (json !== undefined && json.length > 0) {
+            return json;
+        }
+    }
+    catch {
+        // Fall back to String below.
+    }
+    const text = String(value);
+    return text.length > 0 ? text : "unknown error";
+}
 function toTaskMap(tasks) {
     const taskByRowId = new Map();
     for (const task of tasks) {