poe-code 3.0.340 → 3.0.342

package/package.json

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

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) {

package/packages/plan-browser/dist/actions.js

@@ -2,9 +2,7 @@ import path from "node:path";
 import { spawnSync as nodeSpawnSync } from "node:child_process";
 import { hasOwnErrorCode } from "./error-codes.js";
 export function resolveEditor(env = process.env) {
-    const editor = getOwnEnvValue(env, "EDITOR")?.trim() ||
-        getOwnEnvValue(env, "VISUAL")?.trim() ||
-        "vi";
+    const editor = getOwnEnvValue(env, "VISUAL")?.trim() || getOwnEnvValue(env, "EDITOR")?.trim() || "vi";
     return editor.length > 0 ? editor : "vi";
 }
 export function editFile(absolutePath, options = {}) {
@@ -66,12 +64,60 @@ function getOwnEnvValue(env, key) {
 function hasErrorCode(error, code) {
     return hasOwnErrorCode(error, code);
 }
+function isCommandWhitespace(character) {
+    return character === " " || character === "\t" || character === "\n" || character === "\r";
+}
 function parseEditorCommand(command) {
-    const parts = command.match(/(?:[^\s"']+|"[^"]*"|'[^']*')+/g) ?? [];
-    return parts.map((part) => {
-        const quote = part[0];
-        return (quote === '"' || quote === "'") && part.at(-1) === quote ? part.slice(1, -1) : part;
-    });
+    const parts = [];
+    let current = "";
+    let quote;
+    let escaping = false;
+    let hasToken = false;
+    for (const character of command) {
+        if (escaping) {
+            current += character;
+            escaping = false;
+            hasToken = true;
+            continue;
+        }
+        if (character === "\\") {
+            escaping = true;
+            hasToken = true;
+            continue;
+        }
+        if (quote !== undefined) {
+            if (character === quote) {
+                quote = undefined;
+            }
+            else {
+                current += character;
+                hasToken = true;
+            }
+            continue;
+        }
+        if (character === "'" || character === '"') {
+            quote = character;
+            hasToken = true;
+            continue;
+        }
+        if (isCommandWhitespace(character)) {
+            if (hasToken) {
+                parts.push(current);
+                current = "";
+                hasToken = false;
+            }
+            continue;
+        }
+        current += character;
+        hasToken = true;
+    }
+    if (escaping) {
+        current += "\\";
+    }
+    if (hasToken) {
+        parts.push(current);
+    }
+    return parts;
 }
 export { archiveSelectedPlan as archivePlan };
 export async function deletePlan(entry, fs) {

package/packages/plan-browser/dist/discovery.js

@@ -1,5 +1,6 @@
 import path from "node:path";
 import * as fsPromises from "node:fs/promises";
+import { parsePlan } from "@poe-code/pipeline";
 import { planConfigScope, readMergedDocumentReadonly, resolveScope } from "@poe-code/poe-code-config";
 import { hasOwnErrorCode } from "./error-codes.js";
 import { readPlanMetadata, splitFrontmatter } from "./format.js";
@@ -42,8 +43,13 @@ function resolveAbsoluteDirectory(dir, cwd, homeDir) {
     }
     return path.isAbsolute(dir) ? dir : path.resolve(cwd, dir);
 }
-function isMarkdownFile(name) {
-    return name.toLowerCase().endsWith(".md");
+function isSupportedPlanFile(name) {
+    const lowerName = name.toLowerCase();
+    return lowerName.endsWith(".md") || lowerName.endsWith(".yaml") || lowerName.endsWith(".yml");
+}
+function isYamlPlanFile(name) {
+    const lowerName = name.toLowerCase();
+    return lowerName.endsWith(".yaml") || lowerName.endsWith(".yml");
 }
 function getPlanTypeLabel(kind) {
     switch (kind) {
@@ -92,6 +98,10 @@ function toPlanKind(value, filePath) {
     throw new Error(`${filePath}: unsupported frontmatter kind ${JSON.stringify(value)}`);
 }
 function classifyPlanKind(content, filePath) {
+    if (isYamlPlanFile(filePath)) {
+        parsePlan(content);
+        return "pipeline";
+    }
     const { data } = splitFrontmatter(content, filePath);
     if (data === undefined) {
         return "plan";
@@ -128,15 +138,31 @@ async function discoverSharedPlans(options) {
     }
     const plans = [];
     for (const name of entries) {
-        if (!isMarkdownFile(name)) {
+        if (!isSupportedPlanFile(name)) {
             continue;
         }
         const absolutePath = path.join(absoluteDir, name);
-        const canonicalPath = await options.fs.realpath(absolutePath);
+        const canonicalPath = await options.fs.realpath(absolutePath).catch((error) => {
+            if (isNotFound(error)) {
+                return undefined;
+            }
+            throw error;
+        });
+        if (canonicalPath === undefined) {
+            continue;
+        }
         if (canonicalPath !== path.resolve(absolutePath)) {
             throw new Error(`Plan file must not be a symbolic link: ${path.join(displayDir, name)}`);
         }
-        const stat = await options.fs.stat(absolutePath);
+        const stat = await options.fs.stat(absolutePath).catch((error) => {
+            if (isNotFound(error)) {
+                return undefined;
+            }
+            throw error;
+        });
+        if (stat === undefined) {
+            continue;
+        }
         if (!stat.isFile()) {
             continue;
         }