poe-code 3.0.332 → 3.0.334

package/package.json

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

package/packages/experiment-loop/dist/config/loader.js

@@ -64,7 +64,7 @@ function parseRunConfigData(filePath, document) {
     if (prompt === undefined) {
         throw new Error(`Missing "prompt" field in "${filePath}".`);
     }
-    if (typeof prompt !== "string" || prompt.length === 0) {
+    if (typeof prompt !== "string" || prompt.trim().length === 0) {
         throw new Error(`"prompt" must be a non-empty string in "${filePath}".`);
     }
     return { prompt };

package/packages/experiment-loop/dist/frontmatter/frontmatter.js

@@ -96,7 +96,7 @@ export const experimentDocumentSchema = {
             minimum: 0
         }
     },
-    required: ["kind", "version", "baseline"],
+    required: [],
     additionalProperties: false
 };
 export function parseExperimentFrontmatter(content) {
@@ -132,6 +132,7 @@ export function parseExperimentFrontmatterData(value) {
     const parsed = isRecord(value) ? value : undefined;
     validateFrontmatterFields(parsed);
     validateDocumentKind(parsed ? getOwnEntry(parsed, "kind") : undefined);
+    validateDocumentVersion(parsed ? getOwnEntry(parsed, "version") : undefined);
     const agent = parseAgent(parsed ? getOwnEntry(parsed, "agent") : undefined);
     const extendsValue = parseBoolean(parsed ? getOwnEntry(parsed, "extends") : undefined);
     const metric = parseMetric(parsed ? getOwnEntry(parsed, "metric") : undefined);
@@ -166,71 +167,72 @@ function serializeFrontmatter(frontmatter) {
     };
 }
 function parseMetric(value) {
+    if (value === undefined) {
+        return undefined;
+    }
     if (Array.isArray(value)) {
-        const metrics = value
-            .map((item) => parseMetricDefinition(item))
-            .filter((item) => item !== undefined);
-        return metrics.length === value.length ? metrics : undefined;
+        return value.map((item, index) => parseMetricDefinition(item, `metric[${index}]`));
     }
-    return parseMetricDefinition(value);
+    return parseMetricDefinition(value, "metric");
 }
-function parseMetricDefinition(value) {
-    const parsed = isRecord(value) ? value : undefined;
-    const name = parseString(parsed ? getOwnEntry(parsed, "name") : undefined);
-    const script = parseString(parsed ? getOwnEntry(parsed, "script") : undefined);
-    const direction = parseMetricDirection(parsed ? getOwnEntry(parsed, "direction") : undefined);
-    if (name === undefined || script === undefined || direction === undefined) {
-        return undefined;
+function parseMetricDefinition(value, label) {
+    if (!isRecord(value)) {
+        throw new Error(`${label} must be an object.`);
+    }
+    const name = parseRequiredString(getOwnEntry(value, "name"), `${label}.name`);
+    const script = parseRequiredString(getOwnEntry(value, "script"), `${label}.script`);
+    const direction = parseRequiredMetricDirection(getOwnEntry(value, "direction"), `${label}.direction`);
+    const deltaValue = getOwnEntry(value, "delta");
+    if (deltaValue !== undefined &&
+        (typeof deltaValue !== "number" || !Number.isFinite(deltaValue) || deltaValue < 0)) {
+        throw new Error(`${label}.delta must be a non-negative number.`);
     }
-    const deltaValue = parsed ? getOwnEntry(parsed, "delta") : undefined;
-    const delta = typeof deltaValue === "number" && Number.isFinite(deltaValue) && deltaValue >= 0
-        ? deltaValue
-        : undefined;
     return {
         name,
         script,
         direction,
-        ...(delta !== undefined ? { delta } : {})
+        ...(deltaValue !== undefined ? { delta: deltaValue } : {})
     };
 }
-function parseMetricDirection(value) {
-    return value === "minimize" || value === "maximize" || value === "stable" ? value : undefined;
+function parseRequiredMetricDirection(value, label) {
+    if (value === "minimize" || value === "maximize" || value === "stable") {
+        return value;
+    }
+    throw new Error(`${label} must be one of "minimize", "maximize", or "stable".`);
 }
 function parseBaseline(value) {
     if (value === null || value === undefined) {
         return null;
     }
     if (!isRecord(value)) {
-        return null;
+        throw new Error("baseline must be null or an object.");
     }
-    const baselineEntries = Object.entries(value)
-        .map(([key, entryValue]) => {
+    const baselineEntries = Object.entries(value).map(([key, entryValue]) => {
         if (typeof entryValue !== "number" || !Number.isFinite(entryValue)) {
-            return undefined;
+            throw new Error(`baseline.${key} must be a finite number.`);
         }
         return [key, entryValue];
-    })
-        .filter((entry) => entry !== undefined);
-    return baselineEntries.length === Object.keys(value).length
-        ? Object.fromEntries(baselineEntries)
-        : null;
+    });
+    return Object.fromEntries(baselineEntries);
 }
 function parseAgent(value) {
+    if (value === undefined) {
+        return undefined;
+    }
     if (typeof value === "string") {
-        return parseString(value);
+        return parseRequiredString(value, "agent");
     }
     if (!Array.isArray(value)) {
-        return undefined;
+        throw new Error("agent must be a non-empty string or an array of non-empty strings.");
+    }
+    if (value.length === 0) {
+        throw new Error("agent must contain at least one agent.");
     }
     const agents = [];
-    for (const item of value) {
-        const parsed = parseString(item);
-        if (parsed === undefined) {
-            return undefined;
-        }
-        agents.push(parsed);
+    for (const [index, item] of value.entries()) {
+        agents.push(parseRequiredString(item, `agent[${index}]`));
     }
-    return agents.length > 0 ? agents : undefined;
+    return agents;
 }
 function parseString(value) {
     if (typeof value !== "string") {
@@ -239,6 +241,13 @@ function parseString(value) {
     const trimmed = value.trim();
     return trimmed.length > 0 ? trimmed : undefined;
 }
+function parseRequiredString(value, label) {
+    const parsed = parseString(value);
+    if (parsed === undefined) {
+        throw new Error(`${label} must be a non-empty string.`);
+    }
+    return parsed;
+}
 function parseOptionalNonNegativeInteger(value, label) {
     if (value === undefined) {
         return undefined;
@@ -262,6 +271,11 @@ function validateDocumentKind(value) {
         throw new Error("Experiment document kind must be 'experiment'.");
     }
 }
+function validateDocumentVersion(value) {
+    if (value !== undefined && value !== 1) {
+        throw new Error("Experiment document version must be 1.");
+    }
+}
 function validateFrontmatterFields(value) {
     if (value === undefined) {
         return;