poe-code 3.0.339 → 3.0.341

package/dist/index.js

@@ -71012,8 +71012,13 @@ function resolveAbsoluteDirectory4(dir, cwd, homeDir) {
   }
   return path89.isAbsolute(dir) ? dir : path89.resolve(cwd, dir);
 }
-function isMarkdownFile2(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) {
@@ -71061,6 +71066,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 } = splitFrontmatter2(content, filePath);
   if (data === void 0) {
     return "plan";
@@ -71096,15 +71105,31 @@ async function discoverSharedPlans(options) {
   }
   const plans = [];
   for (const name of entries) {
-    if (!isMarkdownFile2(name)) {
+    if (!isSupportedPlanFile(name)) {
       continue;
     }
     const absolutePath = path89.join(absoluteDir, name);
-    const canonicalPath = await options.fs.realpath(absolutePath);
+    const canonicalPath = await options.fs.realpath(absolutePath).catch((error3) => {
+      if (isNotFound3(error3)) {
+        return void 0;
+      }
+      throw error3;
+    });
+    if (canonicalPath === void 0) {
+      continue;
+    }
     if (canonicalPath !== path89.resolve(absolutePath)) {
       throw new Error(`Plan file must not be a symbolic link: ${path89.join(displayDir, name)}`);
     }
-    const stat33 = await options.fs.stat(absolutePath);
+    const stat33 = await options.fs.stat(absolutePath).catch((error3) => {
+      if (isNotFound3(error3)) {
+        return void 0;
+      }
+      throw error3;
+    });
+    if (stat33 === void 0) {
+      continue;
+    }
     if (!stat33.isFile()) {
       continue;
     }
@@ -71148,6 +71173,7 @@ async function discoverAllPlans(options) {
 var init_discovery4 = __esm({
   "packages/plan-browser/src/discovery.ts"() {
     "use strict";
+    init_src31();
     init_src9();
     init_error_codes15();
     init_format();
@@ -71158,7 +71184,7 @@ var init_discovery4 = __esm({
 import path90 from "node:path";
 import { spawnSync as nodeSpawnSync2 } from "node:child_process";
 function resolveEditor2(env = process.env) {
-  const editor = getOwnEnvValue2(env, "EDITOR")?.trim() || getOwnEnvValue2(env, "VISUAL")?.trim() || "vi";
+  const editor = getOwnEnvValue2(env, "VISUAL")?.trim() || getOwnEnvValue2(env, "EDITOR")?.trim() || "vi";
   return editor.length > 0 ? editor : "vi";
 }
 function editFile(absolutePath, options = {}) {
@@ -71217,12 +71243,59 @@ function getOwnEnvValue2(env, key2) {
 function hasErrorCode3(error3, code) {
   return hasOwnErrorCode19(error3, code);
 }
+function isCommandWhitespace(character) {
+  return character === " " || character === "	" || 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 !== void 0) {
+      if (character === quote) {
+        quote = void 0;
+      } 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;
 }
 async function deletePlan(entry, fs29) {
   await fs29.unlink(entry.absolutePath);
@@ -79518,7 +79591,7 @@ async function readAutomation(dir, fileName, baseName, baseDirs) {
         path: baseDir
       })),
       {
-        source: "defaults",
+        source: DEFAULTS_SOURCE,
         data: {
           agent: "codex"
         }
@@ -79529,11 +79602,23 @@ async function readAutomation(dir, fileName, baseName, baseDirs) {
     }
   );
   const name = stripPrefix(fileName.slice(0, -3));
-  return {
+  const automation = {
     name,
     prompt: readPrompt(getOwnEntry32(resolved.data, "prompt"), fileName),
     ...readAutomationFields(resolved.data, fileName)
   };
+  Object.defineProperty(automation, AGENT_SOURCE_PROPERTY, {
+    configurable: true,
+    enumerable: false,
+    value: resolved.sources.agent
+  });
+  return automation;
+}
+function usesDefaultAgent(automation) {
+  return getInternalAgentSource(automation) === DEFAULTS_SOURCE;
+}
+function getInternalAgentSource(automation) {
+  return automation[AGENT_SOURCE_PROPERTY];
 }
 function readPrompt(value, fileName) {
   if (typeof value === "string") {
@@ -79546,8 +79631,8 @@ function readPrompt(value, fileName) {
 }
 function readAutomationFields(frontmatter, fileName) {
   const label = readOptionalString3(getOwnEntry32(frontmatter, "label"), "label", fileName);
-  const source = readOptionalString3(getOwnEntry32(frontmatter, "source"), "source", fileName);
-  const agent3 = readOptionalString3(getOwnEntry32(frontmatter, "agent"), "agent", fileName);
+  const source = readOptionalNonBlankString(getOwnEntry32(frontmatter, "source"), "source", fileName);
+  const agent3 = readOptionalNonBlankString(getOwnEntry32(frontmatter, "agent"), "agent", fileName);
   const mcp = readOptionalMcp(getOwnEntry32(frontmatter, "mcp"), fileName);
   const allow = readOptionalStringArray2(getOwnEntry32(frontmatter, "allow"), "allow", fileName);
   const prefix = readOptionalPrefix(getOwnEntry32(frontmatter, "prefix"), fileName);
@@ -79569,6 +79654,20 @@ function readOptionalString3(value, field, fileName) {
   }
   return value;
 }
+function readOptionalNonBlankString(value, field, fileName) {
+  if (value === void 0) {
+    return void 0;
+  }
+  if (typeof value !== "string") {
+    throw new Error(`Automation "${fileName}" has invalid "${field}" frontmatter. Expected a string.`);
+  }
+  if (value.trim().length === 0) {
+    throw new Error(
+      `Automation "${fileName}" has invalid "${field}" frontmatter. Expected a non-empty string.`
+    );
+  }
+  return value;
+}
 function readOptionalPrefix(value, fileName) {
   if (value === void 0) {
     return void 0;
@@ -79642,6 +79741,11 @@ function readOptionalMcp(value, fileName) {
         `Automation "${fileName}" has invalid "mcp.${serverName}.command" frontmatter. Expected a string.`
       );
     }
+    if (command.trim().length === 0) {
+      throw new Error(
+        `Automation "${fileName}" has invalid "mcp.${serverName}.command" frontmatter. Expected a non-empty string.`
+      );
+    }
     const args = getOwnEntry32(serverValue, "args");
     if (args !== void 0 && (!Array.isArray(args) || args.some((item) => typeof item !== "string"))) {
       throw new Error(
@@ -79688,12 +79792,14 @@ function isStringRecord(value) {
 function stripPrefix(name) {
   return name.startsWith(POE_CODE_PREFIX) ? name.slice(POE_CODE_PREFIX.length) : name;
 }
-var VALID_AUTHOR_ASSOCIATIONS, POE_CODE_PREFIX;
+var DEFAULTS_SOURCE, AGENT_SOURCE_PROPERTY, VALID_AUTHOR_ASSOCIATIONS, POE_CODE_PREFIX;
 var init_discover2 = __esm({
   "packages/github-workflows/src/discover.ts"() {
     "use strict";
     init_src5();
     init_errors6();
+    DEFAULTS_SOURCE = "defaults";
+    AGENT_SOURCE_PROPERTY = "__poeCodeAgentSource";
     VALID_AUTHOR_ASSOCIATIONS = /* @__PURE__ */ new Set([
       "COLLABORATOR",
       "CONTRIBUTOR",
@@ -79821,6 +79927,7 @@ function uniqueTruffleHogFindings(findings) {
   return [...unique4.values()];
 }
 function renderTruffleHogFindingsTable(findings, options) {
+  assertPositiveInteger(options.maxFindings, "maxFindings");
   const uniqueFindings = uniqueTruffleHogFindings(findings);
   const rows = [
     "| Detector | Location | Verification |",
@@ -79895,7 +80002,7 @@ async function scanForSecrets(env, cwd, runner, fs29) {
 async function reportAdvisoryResult(env, runner, fs29) {
   const githubToken = requireEnv(env, "GH_TOKEN");
   const headSha = requireEnv(env, "HEAD_SHA");
-  const maxFindings = numberEnv(env, "MAX_FINDINGS");
+  const maxFindings = positiveDecimalIntegerEnv(env, "MAX_FINDINGS");
   const prNumber = requireEnv(env, "PR_NUMBER");
   const repository = requireEnv(env, "REPOSITORY");
   const resultsFile = env.get("TRUFFLEHOG_RESULTS_FILE") ?? DEFAULT_RESULTS_FILE;
@@ -80076,12 +80183,31 @@ function requireEnv(env, name) {
   }
   return value;
 }
-function numberEnv(env, name) {
-  const value = Number(requireEnv(env, name));
-  if (!Number.isFinite(value)) {
-    throw new UserError(`${name} must be a number.`);
+function positiveDecimalIntegerEnv(env, name) {
+  const parsed = parsePositiveDecimalInteger(requireEnv(env, name));
+  if (parsed === void 0) {
+    throw new UserError(`${name} must be a positive integer.`);
+  }
+  return parsed;
+}
+function parsePositiveDecimalInteger(value) {
+  let parsed = 0;
+  for (const char of value) {
+    const digit = char.charCodeAt(0) - "0".charCodeAt(0);
+    if (digit < 0 || digit > 9) {
+      return void 0;
+    }
+    parsed = parsed * 10 + digit;
+    if (!Number.isSafeInteger(parsed)) {
+      return void 0;
+    }
+  }
+  return parsed > 0 ? parsed : void 0;
+}
+function assertPositiveInteger(value, label) {
+  if (!Number.isInteger(value) || value <= 0) {
+    throw new Error(`${label} must be a positive integer.`);
   }
-  return value;
 }
 function parseJsonObject(value) {
   try {
@@ -80156,7 +80282,7 @@ function runPreflightChecks(context) {
 }
 function checkRequiredEnvVar(env, key2) {
   const value = env.get(key2);
-  if (value === void 0 || value === "") {
+  if (value === void 0 || value.trim() === "") {
     throw new UserError(`Missing required environment variable: ${key2}`);
   }
 }
@@ -80973,7 +81099,7 @@ var init_commands2 = __esm({
           await discoverAutomations(await resolveBuiltInPromptsDir(), ...projectPromptDirs(cwd))
         );
         const automation = await loadNamedAutomation(name, cwd);
-        const agent3 = automation.agent ?? params.agent ?? "codex";
+        const agent3 = params.agent !== void 0 && usesDefaultAgent(automation) ? params.agent : automation.agent ?? params.agent ?? "codex";
         const variables = await loadVariables(await resolveBuiltInAssetsDir(), projectGitHubWorkflowsDir(cwd));
         const sharedTemplateContext = { ...variables, ...buildTemplateContext5(env) };
         if (automation.source === void 0) {
@@ -85752,7 +85878,7 @@ async function* runMatrix(opts) {
   const source = await openSource(opts.sourceDir);
   const evalIds = opts.evalIds ?? await listEvals(source);
   const repeats = opts.repeats ?? defaultRepeats;
-  assertPositiveInteger("repeats", repeats);
+  assertPositiveInteger2("repeats", repeats);
   const matrixDir = path135.join(opts.outDir ?? "runs", matrixId);
   await ensureRunArtifactDirectory(source.rootDir, matrixDir);
   for (const evalId of evalIds) {
@@ -85902,7 +86028,7 @@ function assertNonEmpty(name, values) {
     throw new Error(`Eval matrix ${name} must be a non-empty array`);
   }
 }
-function assertPositiveInteger(name, value) {
+function assertPositiveInteger2(name, value) {
   if (!Number.isInteger(value) || value < 1) {
     throw new Error(`Eval matrix ${name} must be a positive integer`);
   }
@@ -138162,7 +138288,7 @@ var init_package2 = __esm({
   "package.json"() {
     package_default2 = {
       name: "poe-code",
-      version: "3.0.339",
+      version: "3.0.341",
       description: "CLI tool to configure Poe API for developer workflows.",
       type: "module",
       main: "./dist/index.js",