@sweny-ai/core 0.1.109 → 0.1.111

package/dist/claude.js

@@ -202,6 +202,11 @@ export class ClaudeClient {
             .join("\n\n");
         const env = this.buildEnv();
         let response = "";
+        // CC-08: when the SDK populates typed structured output (because we passed
+        // an `outputFormat`), prefer it over re-parsing JSON out of the free-text
+        // result. Left undefined when the SDK didn't provide it, so the
+        // tryParseJSON heuristic remains the fallback (e.g. mocks, older results).
+        let structuredOutput;
         // Timeout / abort wiring (back-compat: only armed when requested).
         // A single AbortController drives both an optional caller signal and an
         // optional timeout timer. The SDK aborts the subprocess when this fires.
@@ -225,6 +230,11 @@ export class ClaudeClient {
                     ...(effectiveModel ? { model: effectiveModel } : {}),
                     ...(Object.keys(allMcpServers).length > 0 ? { mcpServers: allMcpServers } : {}),
                     ...(disallowedTools && disallowedTools.length > 0 ? { disallowedTools } : {}),
+                    // CC-08: ask the SDK to produce validated structured output when the
+                    // node declares an output schema. The SDK then returns the parsed
+                    // value on `result.structured_output`, which we prefer over the
+                    // text-scan heuristic below.
+                    ...(outputSchema ? { outputFormat: { type: "json_schema", schema: outputSchema } } : {}),
                 },
             });
             for await (const message of stream) {
@@ -315,12 +325,22 @@ export class ClaudeClient {
                             };
                         }
                         response = resultMsg.result;
+                        // CC-08: capture the SDK's typed structured output when present.
+                        if ("structured_output" in resultMsg && resultMsg.structured_output !== undefined) {
+                            structuredOutput = resultMsg.structured_output;
+                        }
                     }
                     else if ("errors" in resultMsg) {
                         const errors = resultMsg.errors;
+                        // CC-08: surface the structured-output failure subtype explicitly
+                        // so a schema the model can't satisfy fails loudly instead of
+                        // silently producing empty data.
+                        const prefix = resultMsg.subtype === "error_max_structured_output_retries"
+                            ? "Claude could not produce output matching the schema: "
+                            : "";
                         return {
                             status: "failed",
-                            data: { error: errors?.join("\n") ?? "Execution failed" },
+                            data: { error: prefix + (errors?.join("\n") ?? "Execution failed") },
                             toolCalls,
                         };
                     }
@@ -347,9 +367,15 @@ export class ClaudeClient {
             abort?.clear();
             await interruptStream(stream);
         }
+        // CC-08: prefer the SDK's validated structured output when it gave us an
+        // object. Fall back to the free-text JSON heuristic when it's absent
+        // (no schema requested, mocked result, or an older SDK that didn't set it).
+        const parsedData = structuredOutput && typeof structuredOutput === "object" && !Array.isArray(structuredOutput)
+            ? structuredOutput
+            : tryParseJSON(response, outputSchema, this.logger);
         return {
             status: "success",
-            data: { summary: response, ...tryParseJSON(response, outputSchema, this.logger) },
+            data: { summary: response, ...parsedData },
             toolCalls,
         };
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sweny-ai/core",
-  "version": "0.1.109",
+  "version": "0.1.111",
   "type": "module",
   "bin": {
     "sweny": "./dist/cli/main.js"
@@ -61,7 +61,7 @@
     "@clack/prompts": "^1.2.0",
     "chalk": "^5",
     "commander": "^13",
-    "yaml": "^2",
+    "yaml": "^2.9.0",
     "zod": "^3.25.76"
   },
   "devDependencies": {