@bastani/atomic 0.5.30 → 0.5.31-0

package/dist/sdk/providers/claude.d.ts

@@ -271,6 +271,13 @@ export declare class ClaudeSessionWrapper {
      * `claude` CLI binary, not the SDK — so they are silently ignored.
      */
     query(prompt: string, _options?: Partial<SDKOptions>): Promise<SessionMessage[]>;
+    /**
+     * Structured output is only produced by the Agent SDK's `result` message,
+     * which interactive stages don't consume (they drive the `claude` CLI via
+     * tmux, not the SDK). Always `undefined` here — pair `outputFormat` with a
+     * headless stage to read {@link HeadlessClaudeSessionWrapper#lastStructuredOutput}.
+     */
+    get lastStructuredOutput(): unknown;
     /** Noop — for API symmetry with CopilotSession.disconnect(). */
     disconnect(): Promise<void>;
 }
@@ -328,7 +335,17 @@ export declare class HeadlessClaudeSessionWrapper {
      * Claude stages run in parallel (each call gets its own SDK-assigned UUID).
      */
     private _lastSessionId;
+    /**
+     * Validated structured output captured from the most recent `query()`'s
+     * `result` message. Populated only when callers pass
+     * `options.outputFormat = { type: "json_schema", schema }` and the SDK
+     * produced a `subtype: "success"` result with `structured_output` attached.
+     * Remains `undefined` on plain text runs or when the SDK fails validation
+     * (`error_max_structured_output_retries`).
+     */
+    private _lastStructuredOutput;
     get sessionId(): string;
+    get lastStructuredOutput(): unknown;
     query(prompt: string | AsyncIterable<SDKUserMessage>, options?: Partial<SDKOptions>): Promise<SessionMessage[]>;
     disconnect(): Promise<void>;
 }

package/dist/sdk/providers/claude.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"claude.d.ts","sourceRoot":"","sources":["../../../src/sdk/providers/claude.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAGL,KAAK,cAAc,EACnB,KAAK,cAAc,EACnB,KAAK,OAAO,IAAI,UAAU,EAC3B,MAAM,gCAAgC,CAAC;AAgCxC;;;;;;GAMG;AACH,wBAAsB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAiBtE;AAqID,MAAM,WAAW,oBAAoB;IACnC,kDAAkD;IAClD,MAAM,EAAE,MAAM,CAAC;IACf,sIAAsI;IACtI,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,sEAAsE;IACtE,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AACH,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,oBAAoB,GAAG,OAAO,CAAC,MAAM,CAAC,CAqBxF;AAsID;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,eAAe,CAAC,QAAQ,EAAE,cAAc,EAAE,GAAG,OAAO,CAUnE;AAED;;;;;;;;;;;;GAYG;AACH,wBAAsB,cAAc,CAClC,eAAe,EAAE,MAAM,EACvB,KAAK,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI,EACjC,MAAM,EAAE,WAAW,GAClB,OAAO,CAAC,IAAI,CAAC,CAyCf;AAMD;;;;;;GAMG;AACH,wBAAgB,SAAS,IAAI,MAAM,CAElC;AAED;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAE1D;AAED;;;;GAIG;AACH,wBAAgB,QAAQ,IAAI,MAAM,CAEjC;AAED,0EAA0E;AAC1E,wBAAgB,SAAS,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAEzD;AAED;;;;;GAKG;AACH,wBAAgB,UAAU,IAAI,MAAM,CAEnC;AAED,4EAA4E;AAC5E,wBAAgB,WAAW,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAE3D;AAiED;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,eAAe,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAGjF;AAsCD;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH;;GAEG;AACH,wBAAsB,WAAW,CAC/B,eAAe,EAAE,MAAM,EACvB,qBAAqB,EAAE,MAAM,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC,CAiG3B;AAMD,MAAM,WAAW,kBAAkB;IACjC,2CAA2C;IAC3C,MAAM,EAAE,MAAM,CAAC;IACf,yBAAyB;IACzB,MAAM,EAAE,MAAM,CAAC;IACf;;;;OAIG;IACH,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI,CAAC;CACpC;AAED;;;;;;;;;GASG;AACH,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,aAAa,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,CAAC,EACvD,UAAU,EAAE,MAAM,GACjB,MAAM,CAoBR;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,wBAAsB,WAAW,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CA8FxF;AAMD;;;GAGG;AACH,wBAAgB,oBAAoB,CAClC,QAAQ,EAAE,MAAM,EAAE,GAAG,SAAS,EAC9B,MAAM,EAAE,MAAM,EAAE,GACf,MAAM,EAAE,CAMV;AAED;;;GAGG;AACH,qBAAa,mBAAmB;IAC9B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAoD;gBAGvE,MAAM,EAAE,MAAM,EACd,IAAI,GAAE;QAAE,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;QAAC,cAAc,CAAC,EAAE,MAAM,CAAA;KAAO;IAM9D;;;;;;;OAOG;IACG,KAAK,IAAI,OAAO,CAAC,MAAM,CAAC;IAQ9B,yEAAyE;IACnE,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAC5B;AAED;;;GAGG;AACH,qBAAa,oBAAoB;IAC/B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAA2C;gBAG/D,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,EACjB,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI;IAOpC;;;;;;;;OAQG;IACG,KAAK,CACT,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,GAC7B,OAAO,CAAC,cAAc,EAAE,CAAC;IAQ5B,gEAAgE;IAC1D,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;CAClC;AAMD;;;GAGG;AACH,qBAAa,2BAA2B;IACtC;;;;;OAKG;IACG,KAAK,IAAI,OAAO,CAAC,MAAM,CAAC;IAGxB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAC5B;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,wBAAwB,IAAI,MAAM,CAajD;AAED;;;;;;;;;;GAUG;AACH,qBAAa,4BAA4B;IACvC,QAAQ,CAAC,MAAM,MAAM;IACrB;;;;;OAKG;IACH,OAAO,CAAC,cAAc,CAAc;IAEpC,IAAI,SAAS,IAAI,MAAM,CAEtB;IAEK,KAAK,CACT,MAAM,EAAE,MAAM,GAAG,aAAa,CAAC,cAAc,CAAC,EAC9C,OAAO,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC;IAuCtB,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;CAClC;AAQD;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,+DAejC,CAAC"}
+{"version":3,"file":"claude.d.ts","sourceRoot":"","sources":["../../../src/sdk/providers/claude.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,EAGL,KAAK,cAAc,EACnB,KAAK,cAAc,EACnB,KAAK,OAAO,IAAI,UAAU,EAC3B,MAAM,gCAAgC,CAAC;AAgCxC;;;;;;GAMG;AACH,wBAAsB,kBAAkB,CAAC,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAiBtE;AAqID,MAAM,WAAW,oBAAoB;IACnC,kDAAkD;IAClD,MAAM,EAAE,MAAM,CAAC;IACf,sIAAsI;IACtI,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;IACrB,sEAAsE;IACtE,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA4BG;AACH,wBAAsB,mBAAmB,CAAC,OAAO,EAAE,oBAAoB,GAAG,OAAO,CAAC,MAAM,CAAC,CAqBxF;AAsID;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,eAAe,CAAC,QAAQ,EAAE,cAAc,EAAE,GAAG,OAAO,CAUnE;AAED;;;;;;;;;;;;GAYG;AACH,wBAAsB,cAAc,CAClC,eAAe,EAAE,MAAM,EACvB,KAAK,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI,EACjC,MAAM,EAAE,WAAW,GAClB,OAAO,CAAC,IAAI,CAAC,CAyCf;AAMD;;;;;;GAMG;AACH,wBAAgB,SAAS,IAAI,MAAM,CAElC;AAED;;;;GAIG;AACH,wBAAgB,UAAU,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAE1D;AAED;;;;GAIG;AACH,wBAAgB,QAAQ,IAAI,MAAM,CAEjC;AAED,0EAA0E;AAC1E,wBAAgB,SAAS,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAEzD;AAED;;;;;GAKG;AACH,wBAAgB,UAAU,IAAI,MAAM,CAEnC;AAED,4EAA4E;AAC5E,wBAAgB,WAAW,CAAC,eAAe,EAAE,MAAM,GAAG,MAAM,CAE3D;AAiED;;;;GAIG;AACH,wBAAsB,oBAAoB,CAAC,eAAe,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAGjF;AAsCD;;;;;;;;;;;;;;;;;;;;;;;;;;;GA2BG;AACH;;GAEG;AACH,wBAAsB,WAAW,CAC/B,eAAe,EAAE,MAAM,EACvB,qBAAqB,EAAE,MAAM,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC,CAiG3B;AAMD,MAAM,WAAW,kBAAkB;IACjC,2CAA2C;IAC3C,MAAM,EAAE,MAAM,CAAC;IACf,yBAAyB;IACzB,MAAM,EAAE,MAAM,CAAC;IACf;;;;OAIG;IACH,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI,CAAC;CACpC;AAED;;;;;;;;;GASG;AACH,wBAAgB,oBAAoB,CAClC,IAAI,EAAE,aAAa,CAAC;IAAE,IAAI,EAAE,MAAM,CAAC;IAAC,OAAO,EAAE,OAAO,CAAA;CAAE,CAAC,EACvD,UAAU,EAAE,MAAM,GACjB,MAAM,CAoBR;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA+BG;AACH,wBAAsB,WAAW,CAAC,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,cAAc,EAAE,CAAC,CA8FxF;AAMD;;;GAGG;AACH,wBAAgB,oBAAoB,CAClC,QAAQ,EAAE,MAAM,EAAE,GAAG,SAAS,EAC9B,MAAM,EAAE,MAAM,EAAE,GACf,MAAM,EAAE,CAMV;AAED;;;GAGG;AACH,qBAAa,mBAAmB;IAC9B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,OAAO,CAAC,QAAQ,CAAC,IAAI,CAAoD;gBAGvE,MAAM,EAAE,MAAM,EACd,IAAI,GAAE;QAAE,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;QAAC,cAAc,CAAC,EAAE,MAAM,CAAA;KAAO;IAM9D;;;;;;;OAOG;IACG,KAAK,IAAI,OAAO,CAAC,MAAM,CAAC;IAQ9B,yEAAyE;IACnE,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAC5B;AAED;;;GAGG;AACH,qBAAa,oBAAoB;IAC/B,QAAQ,CAAC,MAAM,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,SAAS,EAAE,MAAM,CAAC;IAC3B,OAAO,CAAC,QAAQ,CAAC,KAAK,CAA2C;gBAG/D,MAAM,EAAE,MAAM,EACd,SAAS,EAAE,MAAM,EACjB,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,OAAO,KAAK,IAAI;IAOpC;;;;;;;;OAQG;IACG,KAAK,CACT,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,GAC7B,OAAO,CAAC,cAAc,EAAE,CAAC;IAQ5B;;;;;OAKG;IACH,IAAI,oBAAoB,IAAI,OAAO,CAElC;IAED,gEAAgE;IAC1D,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;CAClC;AAMD;;;GAGG;AACH,qBAAa,2BAA2B;IACtC;;;;;OAKG;IACG,KAAK,IAAI,OAAO,CAAC,MAAM,CAAC;IAGxB,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;CAC5B;AAED;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,wBAAwB,IAAI,MAAM,CAajD;AAED;;;;;;;;;;GAUG;AACH,qBAAa,4BAA4B;IACvC,QAAQ,CAAC,MAAM,MAAM;IACrB;;;;;OAKG;IACH,OAAO,CAAC,cAAc,CAAc;IACpC;;;;;;;OAOG;IACH,OAAO,CAAC,qBAAqB,CAAsB;IAEnD,IAAI,SAAS,IAAI,MAAM,CAEtB;IAED,IAAI,oBAAoB,IAAI,OAAO,CAElC;IAEK,KAAK,CACT,MAAM,EAAE,MAAM,GAAG,aAAa,CAAC,cAAc,CAAC,EAC9C,OAAO,CAAC,EAAE,OAAO,CAAC,UAAU,CAAC,GAC5B,OAAO,CAAC,cAAc,EAAE,CAAC;IA2CtB,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;CAClC;AAQD;;;;;GAKG;AACH,eAAO,MAAM,sBAAsB,+DAejC,CAAC"}

package/dist/sdk/workflows/builtin/ralph/claude/index.d.ts

@@ -7,13 +7,13 @@
  *   - `max_loops` iterations have completed (defaults to {@link DEFAULT_MAX_LOOPS}), OR
  *   - Two parallel reviewer passes both return zero findings.
  *
- * The reviewer stages run the `reviewer` sub-agent in a visible TUI via the
- * `--agent reviewer` chatFlag, then parse the JSON review out of the
- * assistant text with {@link parseReviewResult}. The prompt enumerates the
- * {@link ReviewResultSchema} fields so the model emits matching JSON. We
- * deliberately avoid invoking the Claude Agent SDK's `query()` from inside a
- * non-headless stage — that would spawn a TUI pane that goes unused while
- * the SDK runs in-process (see workflow-creator skill, failure-modes F17).
+ * The reviewer stages run **headless** via the Claude Agent SDK with
+ * `outputFormat: { type: "json_schema", schema: REVIEW_RESULT_JSON_SCHEMA }`,
+ * so the SDK validates {@link ReviewResultSchema} before returning. The
+ * validated object is read from `s.session.lastStructuredOutput` — no text
+ * parsing required. Running the reviewers headless (no tmux pane) keeps the
+ * graph focused on stages the user cares about and lets the SDK enforce the
+ * schema without TUI round-trips.
  *
  * Run: atomic workflow -n ralph -a claude "<your spec>"
  */

package/dist/sdk/workflows/builtin/ralph/claude/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/claude/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;;;;;;;;;;;;AAmCH,wBAyMa"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/claude/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;GAkBG;;;;;;;;;;;;AA+CH,wBAuMa"}

package/dist/sdk/workflows/builtin/ralph/helpers/prompts.d.ts

@@ -45,7 +45,15 @@ export declare const ReviewResultSchema: z.ZodObject<{
     overall_explanation: z.ZodString;
     overall_confidence_score: z.ZodOptional<z.ZodNumber>;
 }, z.core.$strip>;
-/** JSON Schema derived from the Zod schema — used by Claude and OpenCode SDKs. */
+/**
+ * JSON Schema derived from the Zod schema — used by Claude and OpenCode SDKs.
+ *
+ * `target: "openapi-3.0"` drops the `$schema` draft URL that Zod stamps
+ * by default. The Claude Agent SDK's validator silently drops
+ * `structured_output` when that metadata field is present, so we emit
+ * the OpenAPI-flavoured variant which matches the hand-written shape in
+ * the SDK's structured-output guide.
+ */
 export declare const REVIEW_RESULT_JSON_SCHEMA: z.core.ZodStandardJSONSchemaPayload<z.ZodObject<{
     findings: z.ZodArray<z.ZodObject<{
         title: z.ZodString;
@@ -74,10 +82,12 @@ export interface StructuredReviewResult {
 /**
  * Merge two parallel reviewer results into one.
  *
- * Two independent reviewers run the same prompt simultaneously. This function
- * unions their findings and picks the more conservative overall_correctness.
- * When either reviewer's structured output is unavailable, it falls back to
- * text parsing ({@link parseReviewResult}) before merging.
+ * Each SDK enforces {@link ReviewResultSchema} at the provider level (Claude
+ * `outputFormat`, OpenCode `format: json_schema`, Copilot `defineTool`), so a
+ * non-null `structured` is already a validated {@link ReviewResult}. When
+ * either reviewer failed to produce validated output we propagate `null` —
+ * {@link hasActionableFindings} then treats the raw response as actionable so
+ * the loop keeps iterating instead of silently exiting on a missing reviewer.
  */
 export declare function mergeReviewResults(a: StructuredReviewResult, b: StructuredReviewResult): StructuredReviewResult;
 export interface PlannerContext {
@@ -215,16 +225,6 @@ export interface DebuggerContext {
  * planner consumes.
  */
 export declare function buildDebuggerReportPrompt(review: ReviewResult | null, rawReview: string, context: DebuggerContext): string;
-/**
- * Parse the reviewer's JSON output. Tries, in order:
- *   1. Direct JSON.parse on the entire content.
- *   2. The LAST fenced ```json (or unlabelled) code block.
- *   3. The LAST balanced object containing a "findings" key in surrounding prose.
- *
- * Filters out P3 (minor/style) findings — only P0/P1/P2 count as actionable.
- * Returns null when no parse strategy succeeds.
- */
-export declare function parseReviewResult(content: string): ReviewResult | null;
 export declare function filterActionable(parsed: {
     findings: ReviewFinding[];
     overall_correctness: string;

package/dist/sdk/workflows/builtin/ralph/helpers/prompts.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"prompts.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/helpers/prompts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAMxB,8CAA8C;AAC9C,eAAO,MAAM,mBAAmB;;;;;;;;;;;;iBAsC9B,CAAC;AAEH,wDAAwD;AACxD,eAAO,MAAM,kBAAkB;;;;;;;;;;;;;;;;;iBAgB7B,CAAC;AAEH,kFAAkF;AAClF,eAAO,MAAM,yBAAyB;;;;;;;;;;;;;;;;;kBAAqC,CAAC;AAE5E,mEAAmE;AACnE,MAAM,WAAW,sBAAsB;IACrC,oFAAoF;IACpF,UAAU,EAAE,YAAY,GAAG,IAAI,CAAC;IAChC,8DAA8D;IAC9D,GAAG,EAAE,MAAM,CAAC;CACb;AAED;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAChC,CAAC,EAAE,sBAAsB,EACzB,CAAC,EAAE,sBAAsB,GACxB,sBAAsB,CA4CxB;AAMD,MAAM,WAAW,cAAc;IAC7B,0EAA0E;IAC1E,SAAS,EAAE,MAAM,CAAC;IAClB,wEAAwE;IACxE,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;;;GAQG;AACH,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,cAAiC,GACzC,MAAM,CA2GR;AAKD,MAAM,WAAW,mBAAmB;IAClC;;;;;;OAMG;IACH,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,mBAAwB,GAChC,MAAM,CAwIR;AAMD,0EAA0E;AAC1E,MAAM,WAAW,qBAAqB;IACpC,iDAAiD;IACjD,OAAO,EAAE,MAAM,CAAC;IAChB,kDAAkD;IAClD,QAAQ,EAAE,MAAM,CAAC;IACjB,wDAAwD;IACxD,aAAa,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;;;GAQG;AACH,wBAAgB,0BAA0B,IAAI,qBAAqB,CAwGlE;AAMD,oDAAoD;AACpD,MAAM,WAAW,aAAa;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,aAAa,CAAC,EAAE;QACd,SAAS,EAAE,MAAM,CAAC;QAClB,UAAU,EAAE;YAAE,KAAK,EAAE,MAAM,CAAC;YAAC,GAAG,EAAE,MAAM,CAAA;SAAE,CAAC;KAC5C,CAAC;CACH;AAED,mCAAmC;AACnC,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,wBAAwB,CAAC,EAAE,MAAM,CAAC;CACnC;AAED,MAAM,WAAW,aAAa;IAC5B;;;;;OAKG;IACH,SAAS,EAAE;QACT,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,WAAW,EAAE,MAAM,CAAC;QACpB,UAAU,EAAE,MAAM,CAAC;QACnB,MAAM,EAAE,MAAM,EAAE,CAAC;KAClB,CAAC;IACF,2DAA2D;IAC3D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;;OAIG;IACH,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB;;;;;OAKG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,aAAa,GACrB,MAAM,CAgMR;AAMD,MAAM,WAAW,eAAe;IAC9B,8DAA8D;IAC9D,SAAS,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,SAAS,EAAE;QACT,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,WAAW,EAAE,MAAM,CAAC;QACpB,UAAU,EAAE,MAAM,CAAC;QACnB,MAAM,EAAE,MAAM,EAAE,CAAC;KAClB,CAAC;CACH;AAED;;;;;GAKG;AACH,wBAAgB,yBAAyB,CACvC,MAAM,EAAE,YAAY,GAAG,IAAI,EAC3B,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,eAAe,GACvB,MAAM,CAoHR;AAMD;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,GAAG,YAAY,GAAG,IAAI,CAgDtE;AAED,wBAAgB,gBAAgB,CAAC,MAAM,EAAE;IACvC,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,wBAAwB,CAAC,EAAE,MAAM,CAAC;CACnC,GAAG,YAAY,CAUf;AAED;;;;;;;GAOG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAS5D"}
+{"version":3,"file":"prompts.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/helpers/prompts.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAEH,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAMxB,8CAA8C;AAC9C,eAAO,MAAM,mBAAmB;;;;;;;;;;;;iBAsC9B,CAAC;AAEH,wDAAwD;AACxD,eAAO,MAAM,kBAAkB;;;;;;;;;;;;;;;;;iBAgB7B,CAAC;AAEH;;;;;;;;GAQG;AACH,eAAO,MAAM,yBAAyB;;;;;;;;;;;;;;;;;kBAEpC,CAAC;AAEH,mEAAmE;AACnE,MAAM,WAAW,sBAAsB;IACrC,oFAAoF;IACpF,UAAU,EAAE,YAAY,GAAG,IAAI,CAAC;IAChC,8DAA8D;IAC9D,GAAG,EAAE,MAAM,CAAC;CACb;AAED;;;;;;;;;GASG;AACH,wBAAgB,kBAAkB,CAChC,CAAC,EAAE,sBAAsB,EACzB,CAAC,EAAE,sBAAsB,GACxB,sBAAsB,CAoCxB;AAMD,MAAM,WAAW,cAAc;IAC7B,0EAA0E;IAC1E,SAAS,EAAE,MAAM,CAAC;IAClB,wEAAwE;IACxE,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED;;;;;;;;GAQG;AACH,wBAAgB,kBAAkB,CAChC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,cAAiC,GACzC,MAAM,CA2GR;AAKD,MAAM,WAAW,mBAAmB;IAClC;;;;;;OAMG;IACH,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;;;;GASG;AACH,wBAAgB,uBAAuB,CACrC,IAAI,EAAE,MAAM,EACZ,OAAO,GAAE,mBAAwB,GAChC,MAAM,CAwIR;AAMD,0EAA0E;AAC1E,MAAM,WAAW,qBAAqB;IACpC,iDAAiD;IACjD,OAAO,EAAE,MAAM,CAAC;IAChB,kDAAkD;IAClD,QAAQ,EAAE,MAAM,CAAC;IACjB,wDAAwD;IACxD,aAAa,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;;;GAQG;AACH,wBAAgB,0BAA0B,IAAI,qBAAqB,CAwGlE;AAMD,oDAAoD;AACpD,MAAM,WAAW,aAAa;IAC5B,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,aAAa,CAAC,EAAE;QACd,SAAS,EAAE,MAAM,CAAC;QAClB,UAAU,EAAE;YAAE,KAAK,EAAE,MAAM,CAAC;YAAC,GAAG,EAAE,MAAM,CAAA;SAAE,CAAC;KAC5C,CAAC;CACH;AAED,mCAAmC;AACnC,MAAM,WAAW,YAAY;IAC3B,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,wBAAwB,CAAC,EAAE,MAAM,CAAC;CACnC;AAED,MAAM,WAAW,aAAa;IAC5B;;;;;OAKG;IACH,SAAS,EAAE;QACT,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,WAAW,EAAE,MAAM,CAAC;QACpB,UAAU,EAAE,MAAM,CAAC;QACnB,MAAM,EAAE,MAAM,EAAE,CAAC;KAClB,CAAC;IACF,2DAA2D;IAC3D,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB;;;;OAIG;IACH,aAAa,CAAC,EAAE,OAAO,CAAC;IACxB;;;;;OAKG;IACH,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B;AAED;;;;GAIG;AACH,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,aAAa,GACrB,MAAM,CAgMR;AAMD,MAAM,WAAW,eAAe;IAC9B,8DAA8D;IAC9D,SAAS,EAAE,MAAM,CAAC;IAClB;;;OAGG;IACH,SAAS,EAAE;QACT,UAAU,EAAE,MAAM,CAAC;QACnB,QAAQ,EAAE,MAAM,CAAC;QACjB,WAAW,EAAE,MAAM,CAAC;QACpB,UAAU,EAAE,MAAM,CAAC;QACnB,MAAM,EAAE,MAAM,EAAE,CAAC;KAClB,CAAC;CACH;AAED;;;;;GAKG;AACH,wBAAgB,yBAAyB,CACvC,MAAM,EAAE,YAAY,GAAG,IAAI,EAC3B,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,eAAe,GACvB,MAAM,CAoHR;AAMD,wBAAgB,gBAAgB,CAAC,MAAM,EAAE;IACvC,QAAQ,EAAE,aAAa,EAAE,CAAC;IAC1B,mBAAmB,EAAE,MAAM,CAAC;IAC5B,mBAAmB,CAAC,EAAE,MAAM,CAAC;IAC7B,wBAAwB,CAAC,EAAE,MAAM,CAAC;CACnC,GAAG,YAAY,CAUf;AAED;;;;;;;GAOG;AACH,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAS5D"}

package/dist/sdk/workflows/builtin/ralph/opencode/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/opencode/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;;;;;;;;;;;;AAqDH,wBAmMa"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../../../src/sdk/workflows/builtin/ralph/opencode/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;;;;;;;;;;;;AAyDH,wBAmMa"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bastani/atomic",
-  "version": "0.5.30",
+  "version": "0.5.31-0",
   "description": "Configuration management CLI and SDK for coding agents",
   "type": "module",
   "license": "MIT",
@@ -61,7 +61,7 @@
     "typecheck": "bunx tsc --noEmit && bunx tsc -p tests --noEmit",
     "lint": "oxlint --config=oxlint.json src",
     "lint:fix": "oxlint --config=oxlint.json --fix src",
-    "prepare": "prek install -t pre-commit -t pre-push || true"
+    "prepare": "prek install || true"
   },
   "devDependencies": {
     "@j178/prek": "^0.3.10",

package/src/sdk/providers/claude.ts

@@ -1064,6 +1064,16 @@ export class ClaudeSessionWrapper {
     });
   }
 
+  /**
+   * Structured output is only produced by the Agent SDK's `result` message,
+   * which interactive stages don't consume (they drive the `claude` CLI via
+   * tmux, not the SDK). Always `undefined` here — pair `outputFormat` with a
+   * headless stage to read {@link HeadlessClaudeSessionWrapper#lastStructuredOutput}.
+   */
+  get lastStructuredOutput(): unknown {
+    return undefined;
+  }
+
   /** Noop — for API symmetry with CopilotSession.disconnect(). */
   async disconnect(): Promise<void> {}
 }
@@ -1143,11 +1153,24 @@ export class HeadlessClaudeSessionWrapper {
    * Claude stages run in parallel (each call gets its own SDK-assigned UUID).
    */
   private _lastSessionId: string = "";
+  /**
+   * Validated structured output captured from the most recent `query()`'s
+   * `result` message. Populated only when callers pass
+   * `options.outputFormat = { type: "json_schema", schema }` and the SDK
+   * produced a `subtype: "success"` result with `structured_output` attached.
+   * Remains `undefined` on plain text runs or when the SDK fails validation
+   * (`error_max_structured_output_retries`).
+   */
+  private _lastStructuredOutput: unknown = undefined;
 
   get sessionId(): string {
     return this._lastSessionId;
   }
 
+  get lastStructuredOutput(): unknown {
+    return this._lastStructuredOutput;
+  }
+
   async query(
     prompt: string | AsyncIterable<SDKUserMessage>,
     options?: Partial<SDKOptions>,
@@ -1166,12 +1189,15 @@ export class HeadlessClaudeSessionWrapper {
     };
 
     let sdkSessionId = "";
+    let structuredOutput: unknown = undefined;
     try {
       for await (const msg of sdkQuery({ prompt, options: headlessSdkOpts })) {
         if (msg.type === "result") {
-          sdkSessionId = String(
-            (msg as Record<string, unknown>).session_id ?? "",
-          );
+          const record = msg as Record<string, unknown>;
+          sdkSessionId = String(record.session_id ?? "");
+          if (record.subtype === "success" && "structured_output" in record) {
+            structuredOutput = record.structured_output;
+          }
         }
       }
     } catch (err) {
@@ -1187,6 +1213,7 @@ export class HeadlessClaudeSessionWrapper {
       );
     }
     this._lastSessionId = sdkSessionId;
+    this._lastStructuredOutput = structuredOutput;
     return getSessionMessages(sdkSessionId, { dir: process.cwd() });
   }
 

package/src/sdk/workflows/builtin/ralph/claude/index.ts

@@ -7,13 +7,13 @@
  *   - `max_loops` iterations have completed (defaults to {@link DEFAULT_MAX_LOOPS}), OR
  *   - Two parallel reviewer passes both return zero findings.
  *
- * The reviewer stages run the `reviewer` sub-agent in a visible TUI via the
- * `--agent reviewer` chatFlag, then parse the JSON review out of the
- * assistant text with {@link parseReviewResult}. The prompt enumerates the
- * {@link ReviewResultSchema} fields so the model emits matching JSON. We
- * deliberately avoid invoking the Claude Agent SDK's `query()` from inside a
- * non-headless stage — that would spawn a TUI pane that goes unused while
- * the SDK runs in-process (see workflow-creator skill, failure-modes F17).
+ * The reviewer stages run **headless** via the Claude Agent SDK with
+ * `outputFormat: { type: "json_schema", schema: REVIEW_RESULT_JSON_SCHEMA }`,
+ * so the SDK validates {@link ReviewResultSchema} before returning. The
+ * validated object is read from `s.session.lastStructuredOutput` — no text
+ * parsing required. Running the reviewers headless (no tmux pane) keeps the
+ * graph focused on stages the user cares about and lets the SDK enforce the
+ * schema without TUI round-trips.
  *
  * Run: atomic workflow -n ralph -a claude "<your spec>"
  */
@@ -27,8 +27,10 @@ import {
   buildReviewPrompt,
   buildDebuggerReportPrompt,
   extractMarkdownBlock,
-  parseReviewResult,
+  filterActionable,
   mergeReviewResults,
+  REVIEW_RESULT_JSON_SCHEMA,
+  type ReviewResult,
   type StructuredReviewResult,
 } from "../helpers/prompts.ts";
 import { hasActionableFindings } from "../helpers/review.ts";
@@ -42,13 +44,23 @@ const DEFAULT_MAX_LOOPS = 10;
 // timeout is needed.
 
 /**
- * Extract a {@link StructuredReviewResult} from the reviewer TUI's assistant
- * text. {@link parseReviewResult} tolerates surrounding prose and fenced
- * code blocks; the prompt instructs the model to emit JSON matching
- * {@link ReviewResultSchema}.
+ * Turn the SDK's validated structured_output (plus raw transcript text) into a
+ * {@link StructuredReviewResult}. When the SDK failed to validate the schema
+ * (`error_max_structured_output_retries`) `structured_output` is absent and
+ * we propagate `null` so {@link mergeReviewResults} treats the pass as
+ * unknown/actionable.
  */
-function extractReview(rawText: string): StructuredReviewResult {
-  return { structured: parseReviewResult(rawText), raw: rawText };
+function extractReview(
+  structuredOutput: unknown,
+  rawText: string,
+): StructuredReviewResult {
+  if (structuredOutput && typeof structuredOutput === "object") {
+    return {
+      structured: filterActionable(structuredOutput as ReviewResult),
+      raw: rawText,
+    };
+  }
+  return { structured: null, raw: rawText };
 }
 
 export default defineWorkflow({
@@ -179,41 +191,39 @@ export default defineWorkflow({
           patternResult.result,
       ].join("\n\n---\n\n");
 
-      // ── Review (two parallel passes) ────────────────────────────────────
+      // ── Review (two parallel headless passes with schema enforcement) ──
       const reviewPrompt = buildReviewPrompt(prompt, {
         changeset,
         iteration,
         discoveryContext,
       });
 
-      const reviewerChatFlags = [
-        "--agent",
-        "reviewer",
-        "--allow-dangerously-skip-permissions",
-        "--dangerously-skip-permissions",
-      ];
-
-      const [reviewA, reviewB] = await Promise.all([
+      const runReviewer = (name: string) =>
         ctx.stage(
-          { name: `reviewer-${iteration}-a` },
-          { chatFlags: reviewerChatFlags },
+          { name, headless: true },
           {},
-          async (s) => {
-            const result = await s.session.query(reviewPrompt);
-            s.save(s.sessionId);
-            return extractReview(extractAssistantText(result, 0));
-          },
-        ),
-        ctx.stage(
-          { name: `reviewer-${iteration}-b` },
-          { chatFlags: reviewerChatFlags },
           {},
           async (s) => {
-            const result = await s.session.query(reviewPrompt);
+            const result = await s.session.query(reviewPrompt, {
+              agent: "reviewer",
+              permissionMode: "bypassPermissions",
+              allowDangerouslySkipPermissions: true,
+              outputFormat: {
+                type: "json_schema",
+                schema: REVIEW_RESULT_JSON_SCHEMA,
+              },
+            });
             s.save(s.sessionId);
-            return extractReview(extractAssistantText(result, 0));
+            return extractReview(
+              s.session.lastStructuredOutput,
+              extractAssistantText(result, 0),
+            );
           },
-        ),
+        );
+
+      const [reviewA, reviewB] = await Promise.all([
+        runReviewer(`reviewer-${iteration}-a`),
+        runReviewer(`reviewer-${iteration}-b`),
       ]);
 
       const merged = mergeReviewResults(reviewA.result, reviewB.result);

package/src/sdk/workflows/builtin/ralph/helpers/prompts.ts

@@ -78,8 +78,18 @@ export const ReviewResultSchema = z.object({
     .describe("Overall confidence in the review (0.0–1.0)"),
 });
 
-/** JSON Schema derived from the Zod schema — used by Claude and OpenCode SDKs. */
-export const REVIEW_RESULT_JSON_SCHEMA = z.toJSONSchema(ReviewResultSchema);
+/**
+ * JSON Schema derived from the Zod schema — used by Claude and OpenCode SDKs.
+ *
+ * `target: "openapi-3.0"` drops the `$schema` draft URL that Zod stamps
+ * by default. The Claude Agent SDK's validator silently drops
+ * `structured_output` when that metadata field is present, so we emit
+ * the OpenAPI-flavoured variant which matches the hand-written shape in
+ * the SDK's structured-output guide.
+ */
+export const REVIEW_RESULT_JSON_SCHEMA = z.toJSONSchema(ReviewResultSchema, {
+  target: "openapi-3.0",
+});
 
 /** Result from a reviewer stage with structured output support. */
 export interface StructuredReviewResult {
@@ -92,10 +102,12 @@ export interface StructuredReviewResult {
 /**
  * Merge two parallel reviewer results into one.
  *
- * Two independent reviewers run the same prompt simultaneously. This function
- * unions their findings and picks the more conservative overall_correctness.
- * When either reviewer's structured output is unavailable, it falls back to
- * text parsing ({@link parseReviewResult}) before merging.
+ * Each SDK enforces {@link ReviewResultSchema} at the provider level (Claude
+ * `outputFormat`, OpenCode `format: json_schema`, Copilot `defineTool`), so a
+ * non-null `structured` is already a validated {@link ReviewResult}. When
+ * either reviewer failed to produce validated output we propagate `null` —
+ * {@link hasActionableFindings} then treats the raw response as actionable so
+ * the loop keeps iterating instead of silently exiting on a missing reviewer.
  */
 export function mergeReviewResults(
   a: StructuredReviewResult,
@@ -103,38 +115,30 @@ export function mergeReviewResults(
 ): StructuredReviewResult {
   const rawCombined = [a.raw, b.raw].filter(Boolean).join("\n\n---\n\n");
 
-  // Resolve: prefer structured output, fall back to text parsing
-  const parsedA =
-    a.structured ?? (a.raw.trim() ? parseReviewResult(a.raw) : null);
-  const parsedB =
-    b.structured ?? (b.raw.trim() ? parseReviewResult(b.raw) : null);
-
-  if (!parsedA && !parsedB) {
+  // Conservative: any missing structured output → propagate null. Fabricating
+  // a "patch is correct" default here is how the loop previously exited after
+  // a single iteration when one reviewer's output failed SDK validation.
+  if (!a.structured || !b.structured) {
     return { structured: null, raw: rawCombined };
   }
 
-  const findingsA = parsedA?.findings ?? [];
-  const findingsB = parsedB?.findings ?? [];
-
-  const correctnessA = parsedA?.overall_correctness ?? "patch is correct";
-  const correctnessB = parsedB?.overall_correctness ?? "patch is correct";
   const isIncorrect =
-    correctnessA === "patch is incorrect" ||
-    correctnessB === "patch is incorrect";
+    a.structured.overall_correctness === "patch is incorrect" ||
+    b.structured.overall_correctness === "patch is incorrect";
 
   const explanations = [
-    parsedA?.overall_explanation,
-    parsedB?.overall_explanation,
-  ].filter(Boolean) as string[];
+    a.structured.overall_explanation,
+    b.structured.overall_explanation,
+  ].filter((e): e is string => typeof e === "string" && e.length > 0);
 
   const confidences = [
-    parsedA?.overall_confidence_score,
-    parsedB?.overall_confidence_score,
+    a.structured.overall_confidence_score,
+    b.structured.overall_confidence_score,
   ].filter((c): c is number => c !== undefined);
 
   return {
     structured: {
-      findings: [...findingsA, ...findingsB],
+      findings: [...a.structured.findings, ...b.structured.findings],
       overall_correctness: isIncorrect
         ? "patch is incorrect"
         : "patch is correct",
@@ -981,65 +985,6 @@ the "Pitfalls" section entirely if there are none. Begin now.`;
 // PARSING HELPERS
 // ============================================================================
 
-/**
- * Parse the reviewer's JSON output. Tries, in order:
- *   1. Direct JSON.parse on the entire content.
- *   2. The LAST fenced ```json (or unlabelled) code block.
- *   3. The LAST balanced object containing a "findings" key in surrounding prose.
- *
- * Filters out P3 (minor/style) findings — only P0/P1/P2 count as actionable.
- * Returns null when no parse strategy succeeds.
- */
-export function parseReviewResult(content: string): ReviewResult | null {
-  // Strategy 1: direct JSON
-  try {
-    const parsed = JSON.parse(content);
-    if (parsed && parsed.findings && parsed.overall_correctness) {
-      return filterActionable(parsed);
-    }
-  } catch {
-    /* fall through */
-  }
-
-  // Strategy 2: last fenced code block
-  const blockRe = /```(?:json)?\s*\n([\s\S]*?)\n```/g;
-  let lastBlock: string | null = null;
-  let blockMatch: RegExpExecArray | null;
-  while ((blockMatch = blockRe.exec(content)) !== null) {
-    if (blockMatch[1]) lastBlock = blockMatch[1];
-  }
-  if (lastBlock !== null) {
-    try {
-      const parsed = JSON.parse(lastBlock);
-      if (parsed && parsed.findings && parsed.overall_correctness) {
-        return filterActionable(parsed);
-      }
-    } catch {
-      /* fall through */
-    }
-  }
-
-  // Strategy 3: last "{...findings...}" object in surrounding prose
-  const objRe = /\{[\s\S]*?"findings"[\s\S]*?\}/g;
-  let lastObj: string | null = null;
-  let objMatch: RegExpExecArray | null;
-  while ((objMatch = objRe.exec(content)) !== null) {
-    lastObj = objMatch[0];
-  }
-  if (lastObj !== null) {
-    try {
-      const parsed = JSON.parse(lastObj);
-      if (parsed && parsed.findings && parsed.overall_correctness) {
-        return filterActionable(parsed);
-      }
-    } catch {
-      /* nothing more to try */
-    }
-  }
-
-  return null;
-}
-
 export function filterActionable(parsed: {
   findings: ReviewFinding[];
   overall_correctness: string;

package/src/sdk/workflows/builtin/ralph/opencode/index.ts

@@ -46,15 +46,19 @@ function extractResponseText(
 
 /**
  * Extract a {@link StructuredReviewResult} from an OpenCode prompt response.
- * Prefers the SDK's structured_output field; falls back to text extraction.
+ *
+ * The OpenCode SDK places the SDK-validated structured output on the
+ * AssistantMessage as `structured` (see `@opencode-ai/sdk` v2 types.gen.d.ts
+ * — AssistantMessage.structured). Returns `structured: null` whenever the
+ * SDK did not produce a validated object so {@link mergeReviewResults}
+ * treats the pass as unknown/actionable.
  */
 function extractReview(
   data: { info?: Record<string, unknown>; parts: Array<{ type: string; [key: string]: unknown }> },
 ): StructuredReviewResult {
   const raw = extractResponseText(data.parts);
 
-  // The SDK places validated structured output at data.info.structured_output
-  const structuredOutput = data.info?.structured_output;
+  const structuredOutput = data.info?.structured;
   if (structuredOutput && typeof structuredOutput === "object") {
     return {
       structured: filterActionable(structuredOutput as ReviewResult),