@codedology/code-forge 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -6,6 +6,16 @@ waits on a second real consumer (plan §10.4, Q17).
 
 ## [Unreleased]
 
+## [0.3.1] — 2026-10-01
+
+### Fixed
+
+- Claude sessions: the `--json-schema` passed to the Claude CLI no longer carries the top-level `$schema` pointer, which
+  the CLI rejected ("no schema with key or ref"), so every Claude reviewer session failed at once and each review came
+  back unavailable. Validation still uses the full schema.
+- Grok sessions: the answer is read from Grok's `structuredOutput`/`text` envelope, so Grok reviewer, author and System
+  2 calls no longer fail schema validation (`invalid-output`).
+
 ## [0.3.0] — 2026-10-01
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codedology/code-forge",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Plan/harden/code/review orchestration CLI and Agent Skill for cross-model, parallel, evidence-gated feature delivery.",
   "repository": {
     "type": "git",

package/skill/scripts/forge

@@ -10,7 +10,7 @@
 #      copied install with no package next to it); `test/skill/shim.test.mjs` asserts the pin
 #      equals package.json's version.
 # The shim holds no logic of its own: every verb, flag and message is the CLI's.
-PINNED_VERSION="0.3.0"
+PINNED_VERSION="0.3.1"
 set -eu
 
 self="$0"

package/src/engines/builders/claude.mjs

@@ -202,7 +202,7 @@ function buildClosedBookArgv(params) {
   argv.push('--no-session-persistence');
   if (systemPromptText !== undefined) argv.push('--system-prompt', systemPromptText);
   argv.push('--output-format', 'json');
-  if (schema !== undefined) argv.push('--json-schema', JSON.stringify(schema));
+  if (schema !== undefined) argv.push('--json-schema', JSON.stringify(stripSchemaMeta(schema)));
   if (maxBudgetUsd !== undefined) argv.push('--max-budget-usd', String(maxBudgetUsd));
   argv.push('--permission-mode', 'dontAsk');
   if (!isFacts) {
@@ -222,3 +222,16 @@ export function buildClaudeArgv(params) {
   assertBaseParams('buildClaudeArgv', params);
   return params.role === 'coder' ? buildCoderArgv(params) : buildClosedBookArgv(params);
 }
+
+/**
+ * The Claude CLI's `--json-schema` validator does not know the draft 2020-12 meta-schema and exits
+ * 1 on a top-level `$schema` naming it ("no schema with key or ref …"). The local Ajv2020 check
+ * keeps the full schema; only the copy handed to the CLI drops the meta-schema pointer.
+ * @param {unknown} schema
+ * @returns {unknown}
+ */
+function stripSchemaMeta(schema) {
+  if (schema === null || typeof schema !== 'object' || Array.isArray(schema) || !('$schema' in schema)) return schema;
+  const { $schema: _meta, ...rest } = /** @type {Record<string, unknown>} */ (schema);
+  return rest;
+}

package/src/session/spawn.mjs

@@ -177,8 +177,9 @@ export function readAnswer(cli, stdout, outText) {
     return { parsed: errorEvent ?? null, text, answer: tryJSON(text), usage: parseUsage('codex', usageEvent ?? null) };
   }
   const parsed = tryJSON(stdout.trim());
-  const text = typeof parsed?.result === 'string' ? parsed.result : stdout;
-  const answer = parsed?.structured_output ?? tryJSON(text);
+  // Claude's envelope carries `result` + `structured_output`; Grok's carries `text` + `structuredOutput`.
+  const text = typeof parsed?.result === 'string' ? parsed.result : typeof parsed?.text === 'string' ? parsed.text : stdout;
+  const answer = parsed?.structured_output ?? parsed?.structuredOutput ?? tryJSON(text);
   return { parsed: parsed ?? null, text, answer, usage: parseUsage(/** @type {"claude"|"grok"} */ (cli), parsed ?? null) };
 }