fix: simplify state-validator protocol and add chapter-ref detection

#152: State validator now uses PASS/FAIL protocol instead of JSON.
Empty response = pass (fail-open). JSON fallback for compatibility.

#154: Post-write validator detects chapter number references in prose
(e.g. "第33章", "Chapter 33") and flags them as errors.

Cherry-picked from 82cf3c4 onto v6 baseline.

packages/core/src/__tests__/state-validator-agent.test.ts

@@ -79,6 +79,7 @@ describe("StateValidatorAgent", () => {
         usage: ZERO_USAGE,
       });
 
+    // Empty response is now fail-open (pass), not throw
     await expect(agent.validate(
       "Chapter body.",
       3,
@@ -87,6 +88,9 @@ describe("StateValidatorAgent", () => {
       "old hooks",
       "new hooks",
       "en",
-    )).rejects.toThrow(/empty response/i);
+    )).resolves.toEqual({
+      warnings: [],
+      passed: true,
+    });
   });
 });

packages/core/src/agents/post-write-validator.ts

@@ -165,7 +165,24 @@ export function validatePostWrite(
     });
   }
 
-  // 7. 作者说教词
+  // 7. 正文中的章节号指称（如"第33章"、"chapter 33"）
+  const chapterRefPattern = /(?:第\s*\d+\s*章|[Cc]hapter\s+\d+)/g;
+  const chapterRefs = content.match(chapterRefPattern);
+  if (chapterRefs && chapterRefs.length > 0) {
+    const unique = [...new Set(chapterRefs)];
+    violations.push({
+      rule: isEnglish ? "chapter-number-reference" : "章节号指称",
+      severity: "error",
+      description: isEnglish
+        ? `Chapter text contains explicit chapter number references: ${unique.map(r => `"${r}"`).join(", ")}. Characters do not know they are in a numbered chapter.`
+        : `正文中出现了章节号指称：${unique.map(r => `"${r}"`).join("、")}。角色不知道自己在第几章。`,
+      suggestion: isEnglish
+        ? "Replace with natural references: 'that night', 'when the warehouse burned', 'the incident at the dock'"
+        : '改成自然表达："那天晚上"、"仓库出事那次"、"码头上的事"',
+    });
+  }
+
+  // 8. 作者说教词
   const foundSermons: string[] = [];
   for (const word of SERMON_WORDS) {
     if (content.includes(word)) {

packages/core/src/agents/state-validator.ts

@@ -13,7 +13,10 @@ export interface ValidationResult {
 /**
  * Validates Settler output by comparing old and new truth files via LLM.
  * Catches contradictions, missing state changes, and temporal inconsistencies.
- * Fail-closed: validator execution/format failures must surface to the pipeline.
+ *
+ * Uses a minimal verdict protocol instead of requiring structured JSON:
+ *   Line 1: PASS or FAIL
+ *   Remaining lines: free-form warnings (one per line, optional category prefix)
  */
 export class StateValidatorAgent extends BaseAgent {
   get name(): string {
@@ -51,24 +54,27 @@ Given the chapter text and the CHANGES made to truth files (state card + hooks p
 4. Hook anomaly — a hook disappeared without being marked resolved, or a new hook has no basis in the chapter
 5. Retroactive edit — truth file change implies something happened in a PREVIOUS chapter, not the current one
 
-Output JSON:
-{
-  "warnings": [
-    { "category": "missing_state_change", "description": "..." },
-    { "category": "unsupported_change", "description": "..." }
-  ],
-  "passed": true/false
-}
+Output format (simple, NOT JSON):
+- First line: exactly PASS or FAIL (nothing else on this line)
+- Following lines: one warning per line, optionally prefixed with [category]
+- If no issues at all, just output: PASS
+
+Example:
+PASS
+[unsupported_change] State card says character moved to the forest, but text only shows intent
+[minor] Hook H03 advanced but text mention is brief
 
-passed = true means no serious contradictions found. Minor observations are still reported as warnings.
-If there are no issues at all, return {"warnings": [], "passed": true}.
+Or if there are hard contradictions:
+FAIL
+[contradiction] State says character is dead but chapter text shows them speaking
+[unsupported_change] New location not mentioned anywhere in chapter text
 
-IMPORTANT: Set passed=false ONLY for hard contradictions — facts that directly conflict with the chapter text (e.g., character said to be dead but is alive in text, location stated as X but text clearly shows Y). Do NOT fail for:
-- Slightly ahead-of-text inferences (e.g., "about to go to X" when text shows intent but not arrival)
-- Missing details (state card doesn't capture every minor event)
-- Reasonable extrapolations from text (e.g., "slightly relieved" from "took a sip of water")
-- Hook management differences (hooks added/removed that don't contradict text)
-These should be warnings with passed=true, not failures.`;
+IMPORTANT: Output FAIL ONLY for hard contradictions — facts that directly conflict with the chapter text. Do NOT fail for:
+- Slightly ahead-of-text inferences
+- Missing details that the state card didn't capture
+- Reasonable extrapolations from text
+- Hook management differences that don't contradict text
+These should be warnings with PASS, not FAIL.`;
 
     const userPrompt = `Chapter ${chapterNumber} validation:
 
@@ -117,26 +123,61 @@ ${chapterContent.slice(0, 6000)}`;
   private parseResult(content: string): ValidationResult {
     const trimmed = content.trim();
     if (!trimmed) {
-      throw new Error("LLM returned empty response");
+      // Empty response = assume pass (fail-open for empty)
+      return { warnings: [], passed: true };
     }
 
-    const parsed = extractFirstValidJsonObject<{
-      warnings?: Array<{ category?: string; description?: string }>;
-      passed?: boolean;
-    }>(trimmed);
-    if (!parsed) {
-      throw new Error("State validator returned invalid JSON");
+    const lines = trimmed.split("\n").map((line) => line.trim()).filter(Boolean);
+    if (lines.length === 0) {
+      return { warnings: [], passed: true };
     }
 
-    try {
-      if (typeof parsed.passed !== "boolean") {
-        throw new Error("missing boolean 'passed' field");
-      }
+    // First line determines verdict
+    const verdictLine = lines[0]!.toUpperCase();
+    const passed = !verdictLine.startsWith("FAIL");
 
-      if (parsed.warnings !== undefined && !Array.isArray(parsed.warnings)) {
-        throw new Error("'warnings' must be an array");
+    // Try JSON fallback for backwards compatibility — extract first balanced {}
+    if (lines[0]!.startsWith("{")) {
+      const jsonResult = this.tryParseJsonResult(lines[0]!);
+      if (jsonResult) return jsonResult;
+    }
+
+    // Parse remaining lines as warnings
+    const warnings: ValidationWarning[] = [];
+    for (let i = 1; i < lines.length; i++) {
+      const line = lines[i]!;
+      // Skip lines that are just "PASS" or "FAIL" echoes
+      if (/^(PASS|FAIL)$/i.test(line)) continue;
+
+      const categoryMatch = line.match(/^\[([^\]]+)\]\s*(.+)$/);
+      if (categoryMatch) {
+        warnings.push({
+          category: categoryMatch[1]!.trim(),
+          description: categoryMatch[2]!.trim(),
+        });
+      } else if (line.startsWith("- ") || line.startsWith("* ")) {
+        warnings.push({
+          category: "general",
+          description: line.slice(2).trim(),
+        });
+      } else if (line.length > 5) {
+        warnings.push({
+          category: "general",
+          description: line,
+        });
       }
+    }
+
+    return { warnings, passed };
+  }
 
+  private tryParseJsonResult(text: string): ValidationResult | null {
+    try {
+      const parsed = JSON.parse(text) as {
+        warnings?: Array<{ category?: string; description?: string }>;
+        passed?: boolean;
+      };
+      if (typeof parsed.passed !== "boolean") return null;
       return {
         warnings: (parsed.warnings ?? []).map((w) => ({
           category: w.category ?? "unknown",
@@ -144,82 +185,8 @@ ${chapterContent.slice(0, 6000)}`;
         })),
         passed: parsed.passed,
       };
-    } catch (error) {
-      throw new Error(`State validator returned invalid response: ${String(error)}`);
-    }
-  }
-}
-
-function extractFirstValidJsonObject<T>(text: string): T | null {
-  const direct = tryParseJson<T>(text);
-  if (direct) {
-    return direct;
-  }
-
-  for (let index = 0; index < text.length; index += 1) {
-    if (text[index] !== "{") continue;
-    const candidate = extractBalancedJsonObject(text, index);
-    if (!candidate) continue;
-    const parsed = tryParseJson<T>(candidate);
-    if (parsed) {
-      return parsed;
-    }
-  }
-
-  return null;
-}
-
-function tryParseJson<T>(text: string): T | null {
-  try {
-    return JSON.parse(text) as T;
-  } catch {
-    return null;
-  }
-}
-
-function extractBalancedJsonObject(text: string, start: number): string | null {
-  let depth = 0;
-  let inString = false;
-  let escaped = false;
-
-  for (let index = start; index < text.length; index += 1) {
-    const char = text[index]!;
-
-    if (inString) {
-      if (escaped) {
-        escaped = false;
-        continue;
-      }
-      if (char === "\\") {
-        escaped = true;
-        continue;
-      }
-      if (char === "\"") {
-        inString = false;
-      }
-      continue;
-    }
-
-    if (char === "\"") {
-      inString = true;
-      continue;
-    }
-
-    if (char === "{") {
-      depth += 1;
-      continue;
-    }
-
-    if (char === "}") {
-      depth -= 1;
-      if (depth === 0) {
-        return text.slice(start, index + 1);
-      }
-      if (depth < 0) {
-        return null;
-      }
+    } catch {
+      return null;
     }
   }
-
-  return null;
 }