harnery 0.22.0 → 0.22.1

package/dist/core/workflow/validate.d.ts

@@ -9,7 +9,16 @@
  */
 import type { StageSchema } from "./types.js";
 export declare function validateAgainstSchema(value: unknown, schema: StageSchema, path?: string): string[];
-/** Strip accidental markdown code fences, then strict-parse JSON. */
+/**
+ * Parse one schema-gated JSON value.
+ *
+ * Exact JSON and a whole fenced value remain the normal paths. Some harnesses
+ * prepend a short narration despite an explicit JSON-only contract. Accept
+ * that drift only when the reply contains exactly one unambiguous top-level
+ * object or array, the value reaches the end of the reply, and JSON.parse
+ * accepts it. Trailing prose, multiple values, and malformed candidates remain
+ * failures.
+ */
 export declare function parseStageOutput(text: string): {
     value?: unknown;
     error?: string;

package/dist/core/workflow/validate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../../src/core/workflow/validate.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAE9C,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,EAAE,WAAW,EAAE,IAAI,SAAM,GAAG,MAAM,EAAE,CAmG/F;AAED,qEAAqE;AACrE,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG;IAAE,KAAK,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAUlF"}
+{"version":3,"file":"validate.d.ts","sourceRoot":"","sources":["../../../src/core/workflow/validate.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,YAAY,CAAC;AAE9C,wBAAgB,qBAAqB,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,EAAE,WAAW,EAAE,IAAI,SAAM,GAAG,MAAM,EAAE,CAmG/F;AAED;;;;;;;;;GASG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG;IAAE,KAAK,CAAC,EAAE,OAAO,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,CAYlF"}

package/dist/core/workflow/validate.js

@@ -100,7 +100,16 @@ export function validateAgainstSchema(value, schema, path = "$") {
             return [`${path}: unsupported schema type ${String(schema.type)}`];
     }
 }
-/** Strip accidental markdown code fences, then strict-parse JSON. */
+/**
+ * Parse one schema-gated JSON value.
+ *
+ * Exact JSON and a whole fenced value remain the normal paths. Some harnesses
+ * prepend a short narration despite an explicit JSON-only contract. Accept
+ * that drift only when the reply contains exactly one unambiguous top-level
+ * object or array, the value reaches the end of the reply, and JSON.parse
+ * accepts it. Trailing prose, multiple values, and malformed candidates remain
+ * failures.
+ */
 export function parseStageOutput(text) {
     const stripped = text
         .trim()
@@ -110,9 +119,74 @@ export function parseStageOutput(text) {
         return { value: JSON.parse(stripped) };
     }
     catch (err) {
+        const embedded = singleTrailingJsonContainer(stripped);
+        if (embedded !== undefined)
+            return { value: embedded };
         return { error: `not valid JSON: ${err.message}` };
     }
 }
+function singleTrailingJsonContainer(text) {
+    const candidates = [];
+    let start = -1;
+    let stack = [];
+    let inString = false;
+    let escaped = false;
+    for (let index = 0; index < text.length; index++) {
+        const char = text[index];
+        if (start === -1) {
+            if (char === "{" || char === "[") {
+                start = index;
+                stack = [char];
+                inString = false;
+                escaped = false;
+            }
+            continue;
+        }
+        if (inString) {
+            if (escaped) {
+                escaped = false;
+            }
+            else if (char === "\\") {
+                escaped = true;
+            }
+            else if (char === '"') {
+                inString = false;
+            }
+            continue;
+        }
+        if (char === '"') {
+            inString = true;
+            continue;
+        }
+        if (char === "{" || char === "[") {
+            stack.push(char);
+            continue;
+        }
+        if (char !== "}" && char !== "]")
+            continue;
+        const expectedOpen = char === "}" ? "{" : "[";
+        if (stack.at(-1) !== expectedOpen) {
+            start = -1;
+            stack = [];
+            continue;
+        }
+        stack.pop();
+        if (stack.length > 0)
+            continue;
+        const end = index + 1;
+        try {
+            candidates.push({ end, value: JSON.parse(text.slice(start, end)) });
+        }
+        catch {
+            // A balanced brace pair in narration is not a JSON candidate.
+        }
+        start = -1;
+    }
+    if (candidates.length !== 1 || text.slice(candidates[0].end).trim() !== "") {
+        return undefined;
+    }
+    return candidates[0].value;
+}
 function short(value) {
     const s = JSON.stringify(value);
     return s === undefined ? String(value) : s.length > 60 ? `${s.slice(0, 60)}…` : s;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "harnery",
-  "version": "0.22.0",
+  "version": "0.22.1",
   "description": "Multi-agent coordination for AI coding agents - Claude Code, Cursor, and Codex.",
   "license": "MIT",
   "author": "Ryan Kelly",

package/src/core/workflow/validate.ts

@@ -111,7 +111,16 @@ export function validateAgainstSchema(value: unknown, schema: StageSchema, path
   }
 }
 
-/** Strip accidental markdown code fences, then strict-parse JSON. */
+/**
+ * Parse one schema-gated JSON value.
+ *
+ * Exact JSON and a whole fenced value remain the normal paths. Some harnesses
+ * prepend a short narration despite an explicit JSON-only contract. Accept
+ * that drift only when the reply contains exactly one unambiguous top-level
+ * object or array, the value reaches the end of the reply, and JSON.parse
+ * accepts it. Trailing prose, multiple values, and malformed candidates remain
+ * failures.
+ */
 export function parseStageOutput(text: string): { value?: unknown; error?: string } {
   const stripped = text
     .trim()
@@ -120,10 +129,75 @@ export function parseStageOutput(text: string): { value?: unknown; error?: strin
   try {
     return { value: JSON.parse(stripped) };
   } catch (err) {
+    const embedded = singleTrailingJsonContainer(stripped);
+    if (embedded !== undefined) return { value: embedded };
     return { error: `not valid JSON: ${(err as Error).message}` };
   }
 }
 
+function singleTrailingJsonContainer(text: string): unknown | undefined {
+  const candidates: Array<{ end: number; value: unknown }> = [];
+  let start = -1;
+  let stack: string[] = [];
+  let inString = false;
+  let escaped = false;
+
+  for (let index = 0; index < text.length; index++) {
+    const char = text[index]!;
+    if (start === -1) {
+      if (char === "{" || char === "[") {
+        start = index;
+        stack = [char];
+        inString = false;
+        escaped = false;
+      }
+      continue;
+    }
+
+    if (inString) {
+      if (escaped) {
+        escaped = false;
+      } else if (char === "\\") {
+        escaped = true;
+      } else if (char === '"') {
+        inString = false;
+      }
+      continue;
+    }
+    if (char === '"') {
+      inString = true;
+      continue;
+    }
+    if (char === "{" || char === "[") {
+      stack.push(char);
+      continue;
+    }
+    if (char !== "}" && char !== "]") continue;
+
+    const expectedOpen = char === "}" ? "{" : "[";
+    if (stack.at(-1) !== expectedOpen) {
+      start = -1;
+      stack = [];
+      continue;
+    }
+    stack.pop();
+    if (stack.length > 0) continue;
+
+    const end = index + 1;
+    try {
+      candidates.push({ end, value: JSON.parse(text.slice(start, end)) });
+    } catch {
+      // A balanced brace pair in narration is not a JSON candidate.
+    }
+    start = -1;
+  }
+
+  if (candidates.length !== 1 || text.slice(candidates[0]!.end).trim() !== "") {
+    return undefined;
+  }
+  return candidates[0]!.value;
+}
+
 function short(value: unknown): string {
   const s = JSON.stringify(value);
   return s === undefined ? String(value) : s.length > 60 ? `${s.slice(0, 60)}…` : s;