@boyingliu01/xp-gate 0.25.0 → 0.27.0

package/lib/test-alignment.ts

@@ -230,28 +230,124 @@ function countAssertions(content: string): number {
 }
 
 /**
- * Extract JSDoc block preceding a test/it/describe declaration.
- * Handles .each(), .skip(), .todo() variants and backtick template literals.
+ * Given a content string and the index where the test callback arrow/function
+ * begins (the position of `() =>` or `function() {`), scan forward with brace
+ * matching to return the full callback body (including the outer braces).
+ * Skips single/double quoted strings, backtick template literals, and line/block
+ * comments so braces inside them do not throw off matching.
+ */
+function extractCallbackBody(content: string, callbackStart: number): string {
+  // Find the opening brace of the callback body after `() => {` / `function() {`.
+  let i = callbackStart;
+  let inLineComment = false;
+  let inBlockComment = false;
+  let inSingle = false;
+  let inDouble = false;
+  let inTemplate = false;
+  let depth = 0;
+  let started = false;
+  let openIdx = -1;
+
+  while (i < content.length) {
+    const c = content[i];
+    const n = content[i + 1];
+
+    if (inLineComment) {
+      if (c === '\n') inLineComment = false;
+      i++;
+      continue;
+    }
+    if (inBlockComment) {
+      if (c === '*' && n === '/') { inBlockComment = false; i += 2; continue; }
+      i++;
+      continue;
+    }
+    if (inSingle) {
+      if (c === '\\') { i += 2; continue; }
+      if (c === '\'') inSingle = false;
+      i++;
+      continue;
+    }
+    if (inDouble) {
+      if (c === '\\') { i += 2; continue; }
+      if (c === '"') inDouble = false;
+      i++;
+      continue;
+    }
+    if (inTemplate) {
+      if (c === '\\') { i += 2; continue; }
+      if (c === '`') inTemplate = false;
+      // Template `${}` interpolation can contain braces; treat interpolation
+      // content conservatively by not toggling depth on braces inside it.
+      i++;
+      continue;
+    }
+
+    if (c === '/' && n === '/') { inLineComment = true; i += 2; continue; }
+    if (c === '/' && n === '*') { inBlockComment = true; i += 2; continue; }
+    if (c === '\'') { inSingle = true; i++; continue; }
+    if (c === '"') { inDouble = true; i++; continue; }
+    if (c === '`') { inTemplate = true; i++; continue; }
+
+    if (c === '{') {
+      if (!started) { started = true; openIdx = i; depth = 1; }
+      else depth++;
+      i++;
+      continue;
+    }
+    if (c === '}') {
+      depth--;
+      if (depth === 0) {
+        return content.slice(openIdx, i + 1);
+      }
+      i++;
+      continue;
+    }
+
+    i++;
+  }
+
+  // No balanced body found — the declaration had no callback body (e.g. a
+  // `test.todo('name');` line). Return an empty body so it counts 0 assertions.
+  return '';
+}
+
+/**
+ * Extract JSDoc block preceding a test/it/describe declaration, along with the
+ * *function body* of the callback (so assertion counting is per-test, not
+ * per-file). Handles .skip(), .todo(), .only() variants and backtick template
+ * literals.
+ *
+ * Known limitation (pre-existing, not introduced by #392): `test.each([...])`
+ * parameterised forms are NOT matched, because the argument list is a bracketed
+ * array literal and the name string follows it without a comma. This means such
+ * tests are omitted from assertion counting. Coverage determination itself is
+ * driven by `@covers` tags, so this only affects the avg-assertions data-validity
+ * heuristic, never AC coverage. Tracked separately from #392.
  */
 function extractJSDocBeforeTest(content: string): Array<{ jsdoc: string; testBody: string; testName: string }> {
   const results: Array<{ jsdoc: string; testBody: string; testName: string }> = [];
 
-  // Match: /** JSDOC */ (test|it|describe)[.modifier]( [args], )'name', ...
-  // `.each` variants have an extra `()` before the test name string
-  const testPattern = /\/\*\*([\s\S]*?)\*\/(?:[\s\n]*)(?:test|it|describe)(?:\.(?:each|skip|todo|only))?\s*\([^)]*\)\s*,\s*\(\s*(?:['"]([^'"]+)['"]|`([^`]*)`)\s*[,\)]/g;
+  // Match: /** JSDOC */ (test|it|describe)[.modifier]( [args], )'name', ( [params] ) => {
+  // `.each` variants have an extra `()` before the test name string.
+  // Group 4 (bodyStart) points at the callback open paren so we can brace-match
+  // the real function body.
+  const testPattern = /\/\*\*([\s\S]*?)\*\/(?:[\s\n]*)(?:test|it|describe)(?:\.(?:each|skip|todo|only))?\s*\([^)]*\)\s*,\s*\(\s*(?:['"]([^'"]+)['"]|`([^`]*)`)\s*\)?\s*,?\s*(?:\([^)]*\)\s*=>)?\s*/g;
 
   let match: RegExpExecArray | null;
   while ((match = testPattern.exec(content)) !== null) {
     const jsdoc = match[1]!;
     const testName = match[2] || match[3] || '';
     if (testName) {
-      const testBody = content.slice(match.index, match.index + match[0].length);
+      const body = extractCallbackBody(content, match.index + match[0].length);
+      // JSDoc + declaration + function body gives a self-contained test body.
+      const testBody = content.slice(match.index, match.index + match[0].length) + body;
       results.push({ jsdoc, testBody, testName });
     }
   }
 
   // Second pass: match standard test/it/describe without .each
-  const simplePattern = /\/\*\*([\s\S]*?)\*\/(?:[\s\n]*)(?:test|it|describe)(?:\.(?:skip|todo|only))?\s*\(\s*(?:['"]([^'"]+)['"]|`([^`]*)`)\s*[,\)]/g;
+  const simplePattern = /\/\*\*([\s\S]*?)\*\/(?:[\s\n]*)(?:test|it|describe)(?:\.(?:skip|todo|only))?\s*\(\s*(?:['"]([^'"]+)['"]|`([^`]*)`)\s*\)?\s*,?\s*(?:\([^)]*\)\s*=>)?\s*/g;
 
   while ((match = simplePattern.exec(content)) !== null) {
     const jsdoc = match[1]!;
@@ -260,7 +356,8 @@ function extractJSDocBeforeTest(content: string): Array<{ jsdoc: string; testBod
       // Don't duplicate if already found in first pass
       const alreadyFound = results.some(r => r.jsdoc === jsdoc);
       if (!alreadyFound) {
-        const testBody = content.slice(match.index, match.index + match[0].length);
+        const body = extractCallbackBody(content, match.index + match[0].length);
+        const testBody = content.slice(match.index, match.index + match[0].length) + body;
         results.push({ jsdoc, testBody, testName });
       }
     }
@@ -269,6 +366,27 @@ function extractJSDocBeforeTest(content: string): Array<{ jsdoc: string; testBod
   return results;
 }
 
+/**
+ * Extract a Python test function body: from the `def test_xxx():` line up to
+ * (but not including) the next top-level `def ` line or end of file. This keeps
+ * assertion counting scoped to the individual test.
+ */
+function extractPythonTestBody(content: string, defIndex: number): string {
+  const nextDef = content.indexOf('\ndef ', defIndex);
+  const endIdx = nextDef === -1 ? content.length : nextDef;
+  return content.slice(defIndex, endIdx);
+}
+
+/**
+ * Extract a Go test function body via brace matching starting at the opening
+ * brace that follows `func Test_xxx()`. Returns the full `{ ... }` block.
+ */
+function extractGoTestBody(content: string, funcIndex: number): string {
+  const braceIdx = content.indexOf('{', funcIndex);
+  if (braceIdx === -1) return '';
+  return extractCallbackBody(content, braceIdx);
+}
+
 function pushTestWithCovers(
   tests: TestCase[],
   testPath: string,
@@ -276,6 +394,7 @@ function pushTestWithCovers(
   reqId: string,
   intent: string | undefined,
   fileContent: string,
+  bodyContent: string,
   totalAssertionsRef: { value: number },
 ) {
   const coversRegex = /@covers\s+(.+)/g;
@@ -285,7 +404,9 @@ function pushTestWithCovers(
     covers.push(...coversMatch[1]!.split(',').map(s => s.trim()).filter(c => /^AC-[A-Z0-9]+-\d+-\d+$/.test(c)));
   }
 
-  const assertions = countAssertions(fileContent);
+  // Count assertions against the *per-test body*, not the whole file, so
+  // totalAssertions is not inflated by fileCount x perFileCount (#392-#1).
+  const assertions = countAssertions(bodyContent);
   totalAssertionsRef.value += assertions;
 
   tests.push({
@@ -319,7 +440,7 @@ export function parseTestFiles(testPaths: string[]): TestMap {
         // Filter: only valid AC IDs (AC-XXX-NNN-NN pattern)
         const covers = coversRaw.filter(c => /^AC-[A-Z0-9]+-\d+-\d+$/.test(c));
 
-        const assertions = countAssertions(content);
+        const assertions = countAssertions(m.testBody);
         totalAssertionsObj.value += assertions;
 
         tests.push({
@@ -336,7 +457,8 @@ export function parseTestFiles(testPaths: string[]): TestMap {
       const pyTestPattern = /(# @test\s+(REQ-[A-Z0-9]+-[A-Z0-9]+)[\s\S]*?)(?:# @intent\s+(.+?)\n)?def\s+(test_\w+)/g;
       let m: RegExpExecArray | null;
       while ((m = pyTestPattern.exec(content)) !== null) {
-        pushTestWithCovers(tests, testPath, m[4]!, m[2]!, m[3]?.trim(), content, totalAssertionsObj);
+        const body = extractPythonTestBody(content, m.index + m[0].lastIndexOf('def '));
+        pushTestWithCovers(tests, testPath, m[4]!, m[2]!, m[3]?.trim(), content, body, totalAssertionsObj);
       }
     } else if (ext === '.go') {
       const goTestPattern = /(\/\/ @test\s+(REQ-[A-Z0-9]+-[A-Z0-9]+))\s*[\s\S]*?func\s+(Test_\w+)/g;
@@ -344,7 +466,9 @@ export function parseTestFiles(testPaths: string[]): TestMap {
       while ((m = goTestPattern.exec(content)) !== null) {
         const block = content.slice(Math.max(0, m.index - 500), m.index);
         const intentMatch = block.match(/\/\/ @intent\s+(.+)/);
-        pushTestWithCovers(tests, testPath, m[3]!, m[2]!, intentMatch?.[1]?.trim(), content, totalAssertionsObj);
+        const funcStart = m.index + m[0].lastIndexOf('func ');
+        const body = extractGoTestBody(content, funcStart);
+        pushTestWithCovers(tests, testPath, m[3]!, m[2]!, intentMatch?.[1]?.trim(), content, body, totalAssertionsObj);
       }
     }
   }

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Mock layering policy enforcement — Gate M3 of pre-push hook. Ensures integration tests use real implementations for internal dependencies, mock external dependencies, and annotate pending mocks with removal plans. Combines project scope scanning, mock decision engine, and per-file validation into a single pipeline.

package/mutation/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MUTATION KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **Gate M** (incremental mutation testing) + **Gate M2** helpers (test-layer detection used by `src/mock-policy/`). Pre-push quality gate. TypeScript-only; uses Stryker.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/xp-gate",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"

package/plugins/claude-code/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Claude Code. Includes 12 pre-commit gates (Gate 0-11), 8 pre-push gates, Sprint Flow (6 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/claude-code/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/claude-code/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/claude-code/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/dsh/README.md

@@ -185,6 +185,6 @@ npm run build:dsh-plugin       # = node scripts/build-plugin.mjs --platform dsh
 ## 注意
 
 - `lib/` 与 `skills/` 是**构建产物**（已 gitignore），不提交。
-- `scripts/prepack.cjs` 目前硬编码 12 个 skill，**不含 `clipboard-vision`**（仓库有 13 个）。
+- `scripts/prepack.cjs` 的 skill 列表为**动态发现**（读取仓库 `skills/`），不硬编码。
 - `plugins/dsh/package.json` 的 peerDeps 仍 pin `0.1.1-rc.2`，但实测运行在 `0.2.0-rc.2` 上
   `defineTool` / `TOOL_ABORTED` / `HarnessError` 签名未变，无需改写。

package/plugins/dsh/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/dsh-plugin-xp-gate",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "description": "XP-Gate deterministic quality gates + AI workflow skills as a DeepSeek Harness (DSH) native plugin: gate-check / gate-principles / gate-arch tools + 12 SKILL.md skills",
   "type": "module",
   "main": "./lib/index.js",

package/plugins/dsh/scripts/prepack.cjs

@@ -10,8 +10,8 @@
  * plugins/opencode/scripts/prepack.cjs).
  *
  * The skill list is DISCOVERED, not hardcoded: a hardcoded list silently drifts
- * from the repo (it previously shipped 12 of 13, omitting clipboard-vision, and
- * would need editing on every new skill) — see #448.
+ * from the repo (a hardcoded set previously omitted one shipped skill, breaking
+ * parity) — see #448.
  */
 
 const fs = require('fs');

package/plugins/dsh/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/dsh/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/dsh/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/dsh/src/skills.test.ts

@@ -14,10 +14,20 @@ const REPO_SKILLS_DIR = resolve(PLUGIN_ROOT, "..", "..", "skills")
 /**
  * The expected set is the REPO's skill directories, discovered rather than
  * hardcoded. A hardcoded list silently drifts: it previously named 12 skills and
- * omitted clipboard-vision, while prepack shipped 12 of 13 (#448).
+ * omitted one that prepack shipped, breaking parity (#448).
  */
 function repoSkills(): string[] {
-  return readdirSync(REPO_SKILLS_DIR, { withFileTypes: true })
+  // Guarded IO: a missing/unreadable repo skills dir is a test-environment
+  // failure we surface loudly, not a silent empty set (clean-code.missing-error-handling).
+  let entries;
+  try {
+    entries = readdirSync(REPO_SKILLS_DIR, { withFileTypes: true })
+  } catch (err) {
+    throw new Error(
+      `Cannot read repo skills dir ${REPO_SKILLS_DIR}: ${err instanceof Error ? err.message : String(err)}`
+    )
+  }
+  return entries
     .filter((d) => d.isDirectory())
     .map((d) => d.name)
     .filter((name) => existsSync(join(REPO_SKILLS_DIR, name, "SKILL.md")))
@@ -25,7 +35,14 @@ function repoSkills(): string[] {
 }
 
 function readFrontmatter(file: string): Record<string, unknown> {
-  const raw = readFileSync(file, "utf8")
+  // Guarded IO: re-throw with context so a broken SKILL.md names the file
+  // (clean-code.missing-error-handling).
+  let raw;
+  try {
+    raw = readFileSync(file, "utf8")
+  } catch (err) {
+    throw new Error(`${file}: cannot read file (${err instanceof Error ? err.message : String(err)})`)
+  }
   const m = raw.match(/^---\s*\r?\n([\s\S]*?)\r?\n---\s*\r?\n/)
   if (!m) throw new Error(`${file}: missing YAML frontmatter`)
   return parse(m[1]) as Record<string, unknown>
@@ -58,8 +75,4 @@ describe("bundled skills", () => {
       expect((fm.description as string).trim().length, `${name}: non-empty description`).toBeGreaterThan(0)
     }
   })
-
-  it("bundles clipboard-vision, which the old hardcoded list omitted (#448)", () => {
-    expect(existsSync(join(SKILLS_DIR, "clipboard-vision", "SKILL.md"))).toBe(true)
-  })
 })

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "type": "module",
   "main": "index.ts",
   "description": "XP-Gate quality gates + AI workflow skills + Sprint Flow TUI sidebar for OpenCode",

package/plugins/opencode/scripts/prepack.cjs

@@ -16,7 +16,6 @@ const REPO_ROOT = path.resolve(PLUGIN_ROOT, '..', '..');
 
 const CORE_SKILLS = [
   'admin-template-guidelines',
-  'clipboard-vision',
   'delphi-review',
   'improve-codebase-architecture',
   'ralph-loop',

package/plugins/opencode/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/opencode/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/qoder/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.25.0",
+  "version": "0.27.0",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 12 pre-commit gates (Gate 0-11), 8 pre-push gates, Sprint Flow (6 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, exactly 3 experts with distinct executable model IDs). Supports design, requirements, and code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 **6-phase** development pipeline (v2.0 compact redesign, Issue #290): PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE. Phase 3/6 BUILD default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE between DESIGN (2/6) and BUILD (3/6): design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
-**Generated:** 2026-10-14
-**Commit:** a757d625
+**Generated:** 2026-10-16
+**Commit:** 429f73c5
 **Branch:** main
-**Version:** 0.25.0.0
+**Version:** 0.27.0.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.

package/principles/rules/__tests__/clean-code/code-duplication.test.ts

@@ -56,15 +56,16 @@ describe('code-duplication.ts - Code Duplication Rule', () => {
     expect(violations).toEqual([]);
   });
 
-  it('should return empty array when adapter throws', () => {
-    const mockAdapterThrowing = {
-      ...mockAdapter,
-      duplicationPercentage: () => { throw new Error('jscpd error'); }
-    };
+  it('should propagate adapter errors (no empty catch, #392)', () => {
+    // #392-#4: the rule no longer swallows adapter errors in an empty catch, so
+    // a throwing adapter getter must bubble up to analyzer.runRuleOnFile().
+    const mockAdapterThrowing = Object.defineProperty(
+      { ...mockAdapter },
+      'duplicationPercentage',
+      { get: () => { throw new Error('jscpd error'); } }
+    );
     
-    const violations = codeDuplicationRule.check('test.ts', mockAdapterThrowing as never);
-    
-    expect(violations).toEqual([]);
+    expect(() => codeDuplicationRule.check('test.ts', mockAdapterThrowing as never)).toThrow('jscpd error');
   });
 
   it('should handle multiple files with duplication', () => {

package/principles/rules/__tests__/clean-code/deep-nesting.test.ts

@@ -61,14 +61,13 @@ function deeplyNested() {
     expect(deepNestingRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractFunctions: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = deepNestingRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    // Rule errors must bubble up to analyzer.runRuleOnFile(), not be swallowed.
+    expect(() => deepNestingRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/god-class.test.ts

@@ -98,14 +98,12 @@ class DataClass {
     expect(godClassRule.severity).toBe('warning');
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       extractClasses: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = godClassRule.check('test.ts', mockAdapterThatThrows as never);
-    
-    expect(violations.length).toBe(0);
+    expect(() => godClassRule.check('test.ts', mockAdapterThatThrows as never)).toThrow('Adapter failed');
   });
 });

package/principles/rules/__tests__/clean-code/large-file.test.ts

@@ -61,14 +61,12 @@ describe('largeFileRule', () => {
     expect(largeFileRule.threshold).toEqual(1150);
   });
 
-  it('should return empty violations when adapter throws error', () => {
+  it('should propagate adapter errors (no empty catch, #392)', () => {
     const mockAdapterThatThrows = {
       ...mockAdapter,
       countLines: () => { throw new Error('Adapter failed'); }
     };
     
-    const violations = largeFileRule.check('test.ts', mockAdapterThatThrows);
-    
-    expect(violations).toHaveLength(0);
+    expect(() => largeFileRule.check('test.ts', mockAdapterThatThrows)).toThrow('Adapter failed');
   });
 });