poe-code 3.0.341 → 3.0.343

package/dist/index.js

@@ -73219,10 +73219,14 @@ 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 (!isRecord28(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);
   }
 }
 async function validateDispatch(cfg, taskList) {
@@ -73294,6 +73298,9 @@ function validateStateDefinition(name, definition) {
   if (prompt !== void 0 && 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 !== void 0 && typeof terminal !== "boolean") {
     throw new Error(`State "${name}" terminal must be a boolean.`);
   }
@@ -73410,13 +73417,13 @@ function resolveConfig2(raw, cwd) {
     },
     workspace: {
       root: resolvePathValue(
-        readString8(getOwnEntry30(workspace, "root")) ?? path96.join(os13.tmpdir(), "poe-code-maestro"),
+        readString8(getOwnEntry30(workspace, "root"), "workspace.root") ?? path96.join(os13.tmpdir(), "poe-code-maestro"),
         cwd
       )
     },
     agent: {
-      service: readString8(getOwnEntry30(agent3, "service")) ?? "codex",
-      list: readString8(resolveStringValue(getOwnEntry30(agent3, "list"))),
+      service: readString8(getOwnEntry30(agent3, "service"), "agent.service") ?? "codex",
+      list: readString8(resolveStringValue(getOwnEntry30(agent3, "list")), "agent.list"),
       maxConcurrentAgents: readPositiveInteger(
         getOwnEntry30(agent3, "max_concurrent_agents"),
         1,
@@ -73532,25 +73539,34 @@ function expandHome5(value) {
   }
   return value;
 }
-function readString8(value) {
-  if (typeof value !== "string") {
+function readString8(value, field) {
+  if (value === void 0) {
     return void 0;
   }
+  if (typeof value !== "string") {
+    throw new Error(`Expected "${field}" to be a string.`);
+  }
   const trimmed = value.trim();
   return trimmed.length > 0 ? value : void 0;
 }
-function readNumber3(value, fallback) {
-  return typeof value === "number" && Number.isFinite(value) ? value : fallback;
+function readNumber3(value, fallback, field) {
+  if (value === void 0) {
+    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 numberValue2 = readNumber3(value, fallback);
+  const numberValue2 = readNumber3(value, fallback, field);
   if (!Number.isInteger(numberValue2) || numberValue2 <= 0) {
     throw new Error(`Expected "${field}" to be a positive integer.`);
   }
   return numberValue2;
 }
 function readNonNegativeInteger(value, fallback, field) {
-  const numberValue2 = readNumber3(value, fallback);
+  const numberValue2 = readNumber3(value, fallback, field);
   if (!Number.isInteger(numberValue2) || numberValue2 < 0) {
     throw new Error(`Expected "${field}" to be a non-negative integer.`);
   }
@@ -74542,7 +74558,22 @@ function resolveWorkflowPath2(name, cwd) {
   return path99.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 isDigit5 = code >= 48 && code <= 57;
+  return isUppercase || isLowercase || isDigit5 || character === "_" || character === "-";
 }
 var init_workflow_path = __esm({
   "packages/maestro/src/workflow-path.ts"() {
@@ -91895,11 +91926,16 @@ 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");
@@ -91931,13 +91967,24 @@ 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 === void 0) {
@@ -92033,6 +92080,7 @@ async function renderTaskDetailMarkdown(task, taskList) {
   const eventsMarkdown = await renderEventsMarkdown(task, taskList);
   const description = task.description.length > 0 ? task.description : "_No description._";
   const metadata = stringify7(task.metadata).trimEnd();
+  const metadataFence = buildBacktickFence(metadata);
   return [
     `# ${task.name}`,
     "",
@@ -92042,9 +92090,9 @@ async function renderTaskDetailMarkdown(task, taskList) {
     "",
     "## Metadata",
     "",
-    "```yaml",
+    `${metadataFence}yaml`,
     metadata,
-    "```",
+    metadataFence,
     "",
     "## Next",
     "",
@@ -92059,9 +92107,42 @@ 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 !== void 0 && json.length > 0) {
+      return json;
+    }
+  } catch {
+  }
+  const text5 = String(value);
+  return text5.length > 0 ? text5 : "unknown error";
+}
 function toTaskMap(tasks) {
   const taskByRowId = /* @__PURE__ */ new Map();
   for (const task of tasks) {
@@ -92111,9 +92192,7 @@ var init_explorer_config2 = __esm({
     STATE_ORDER_INDEX = new Map(
       STATE_ORDER.map((state, index) => [state, index])
     );
-    STATE_TONES = new Map(
-      KNOWN_STATES.map(({ state, tone }) => [state, tone])
-    );
+    STATE_TONES = new Map(KNOWN_STATES.map(({ state, tone }) => [state, tone]));
   }
 });
 
@@ -138288,7 +138367,7 @@ var init_package2 = __esm({
   "package.json"() {
     package_default2 = {
       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",