poe-code 3.0.339 → 3.0.340

package/dist/index.js

@@ -79518,7 +79518,7 @@ async function readAutomation(dir, fileName, baseName, baseDirs) {
         path: baseDir
       })),
       {
-        source: "defaults",
+        source: DEFAULTS_SOURCE,
         data: {
           agent: "codex"
         }
@@ -79529,11 +79529,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 +79558,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 +79581,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 +79668,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 +79719,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 +79854,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 +79929,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 +80110,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 +80209,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 +81026,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 +85805,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 +85955,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 +138215,7 @@ var init_package2 = __esm({
   "package.json"() {
     package_default2 = {
       name: "poe-code",
-      version: "3.0.339",
+      version: "3.0.340",
       description: "CLI tool to configure Poe API for developer workflows.",
       type: "module",
       main: "./dist/index.js",