@boyingliu01/xp-gate 0.35.6 → 0.35.8

package/lib/assertion-debt.js

@@ -0,0 +1,173 @@
+/**
+ * `xp-gate audit assertions` — make pre-existing assertion debt visible (#668).
+ *
+ * Gate 5d deliberately checks only the files in the diff: a full-repo scan on
+ * every commit is too slow. That has an expensive side effect. Pre-existing debt
+ * blocks unrelated changes — the author who fixed `writeReport()`'s error handling
+ * was stopped by four long-standing `expect(true).toBe(true)` lines in a file they
+ * had merely touched — and the block reads as "this change introduced a problem".
+ *
+ * The worst consequence is not that one false attribution. It is what it does to
+ * behaviour: noise in the attribution makes people reach for
+ * `@assertion-justified` instead of fixing the test. A gate that teaches people to
+ * annotate their way past it is worse than no gate. Hence a batch work queue.
+ *
+ * Scope: only the two shapes Gate 5d BLOCKS (tautological, empty catch). Weak-only
+ * assertions and mock echoes WARN there because each can be legitimate; putting them
+ * in a "debt to clear" list would invent work.
+ */
+'use strict';
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+/** Directories that never contain hand-written assertions worth auditing. */
+const SKIP_DIRS = new Set([
+  'node_modules',
+  '.git',
+  'dist',
+  'build',
+  'coverage',
+  '.stryker-tmp',
+  '.agent-tmp',
+  // Build output mirrored from src/ — listing it again would double-count debt and
+  // send a batch fixer to a file that regenerates.
+  '.xp-gate',
+]);
+
+/** Test-file patterns across the ecosystems XP-Gate supports. */
+function isTestFile(name) {
+  return (
+    /\.(test|spec)\.[cm]?[jt]sx?$/.test(name) ||
+    /(^|\/)(__tests__)\//.test(name) ||
+    /\.test\.bats$/.test(name) ||
+    /^test_.*\.py$/.test(name) ||
+    /_test\.(go|rs|py)$/.test(name)
+  );
+}
+
+/**
+ * Tautological assertions: the assertion cannot fail.
+ *
+ * Mirrors `classify_tautological_assertions` in githooks/lib/assertion-quality.sh.
+ * The two implementations must agree, because a debt list that disagrees with the
+ * gate it exists to serve sends people to fix lines that are not blocking and miss
+ * the ones that are.
+ */
+const TAUTOLOGY_PATTERNS = [
+  // (a) identical literal on both sides.
+  {
+    re: /expect\s*\(\s*(true|false|null|undefined|-?\d+|"[^"]*"|'[^']*')\s*\)\s*\.\s*to(?:Be|Equal|StrictEqual)\s*\(\s*\1\s*\)/,
+  },
+  // (b) a literal tested against the truthiness it already has.
+  { re: /expect\s*\(\s*(?:true|1|-1|"[^"]+"|'[^']+')\s*\)\s*\.\s*toBeTruthy\s*\(/ },
+  { re: /expect\s*\(\s*(?:false|0|null|undefined|""|'')\s*\)\s*\.\s*toBeFalsy\s*\(/ },
+];
+
+/** An empty catch body: the failure it guards is discarded. */
+const EMPTY_CATCH_RE = /catch\s*(?:\([^)]*\))?\s*\{\s*\}/;
+
+/**
+ * Is this line a fixture rather than a real assertion?
+ *
+ * A `.bats` test that exercises the Gate 5d classifier necessarily contains
+ * `'expect(true).toBe(true);'` as a quoted INPUT. Reporting it as debt would send a
+ * batch fixer to delete the very strings that pin the classifier — turning the
+ * "fix debt" instruction into an instruction to remove test coverage.
+ *
+ * The distinguishing feature is quoting: in a fixture the shape is wrapped in
+ * quotes and usually sits inside a list/assignment, where a real assertion is bare
+ * executable code.
+ */
+function isFixtureLine(trimmed) {
+  // A leading quote before the assertion shape: `'expect(...)...'` or "expect(...)".
+  if (/^['"`]/.test(trimmed)) return true;
+  // Assertion shape appearing inside a quoted string on the line.
+  if (/['"][^'"`]*expect\s*\(/.test(trimmed)) return true;
+  return false;
+}
+
+/** Classify one source line, or null when it is fine. */
+function classifyLine(text) {
+  const trimmed = text.trim();
+  if (trimmed === '') return null;
+  // A commented-out assertion is not an assertion.
+  if (/^(\/\/|\*|\/\*|#)/.test(trimmed)) return null;
+  if (isFixtureLine(trimmed)) return null;
+  for (const { re } of TAUTOLOGY_PATTERNS) {
+    if (re.test(trimmed)) return 'tautological';
+  }
+  if (EMPTY_CATCH_RE.test(trimmed)) return 'swallowed-error';
+  return null;
+}
+
+/** Find debt in one file's text. */
+function scanSource(text, file) {
+  const findings = [];
+  const lines = text.split(/\r?\n/);
+  for (let i = 0; i < lines.length; i += 1) {
+    const kind = classifyLine(lines[i]);
+    if (kind) findings.push({ line: i + 1, kind, text: lines[i].trim() });
+  }
+  return findings.length > 0 ? [{ file, findings }] : [];
+}
+
+/** Walk the tree collecting test files, bounded to avoid node_modules. */
+function listTestFiles(root, out = []) {
+  let entries;
+  try {
+    entries = fs.readdirSync(root, { withFileTypes: true });
+  } catch {
+    return out;
+  }
+  if (!Array.isArray(entries)) return out;
+  for (const entry of entries) {
+    if (entry.isDirectory()) {
+      if (SKIP_DIRS.has(entry.name)) continue;
+      listTestFiles(path.join(root, entry.name), out);
+      continue;
+    }
+    if (isTestFile(entry.name)) out.push(path.join(root, entry.name));
+  }
+  return out;
+}
+
+/**
+ * Scan for assertion debt.
+ *
+ * @param {{cwd?: string, source?: string, file?: string}} opts
+ * @returns {Array<{file: string, findings: Array<{line: number, kind: string, text: string}>}>}
+ */
+function scanForAssertionDebt(opts = {}) {
+  if (typeof opts.source === 'string') {
+    return scanSource(opts.source, opts.file || '<inline>');
+  }
+  const root = opts.cwd || process.cwd();
+  const results = [];
+  for (const abs of listTestFiles(root)) {
+    let text;
+    try {
+      text = fs.readFileSync(abs, 'utf8');
+    } catch {
+      continue;
+    }
+    const rel = path.relative(root, abs).split(path.sep).join('/');
+    results.push(...scanSource(text, rel));
+  }
+  return results.sort((a, b) => b.findings.length - a.findings.length || a.file.localeCompare(b.file));
+}
+
+/** Totals for a debt run: per-kind counts plus the file count to batch over. */
+function summarizeDebt(results) {
+  const byKind = { tautological: 0, 'swallowed-error': 0 };
+  let total = 0;
+  for (const entry of results) {
+    for (const finding of entry.findings) {
+      byKind[finding.kind] = (byKind[finding.kind] || 0) + 1;
+      total += 1;
+    }
+  }
+  return { files: results.length, total, byKind };
+}
+
+module.exports = { classifyLine, scanForAssertionDebt, summarizeDebt, isTestFile };

package/lib/gate-rules.js

@@ -0,0 +1,149 @@
+/**
+ * Collect the gate thresholds that actually block a commit, from the files that
+ * define them (#652).
+ *
+ * Why generated rather than hand-written: BUILD-phase docs used to say "there are
+ * gates 0-11" without saying what the numbers are. The numbers live in
+ * `.principlesrc`, `jscpd.conf.json`, `.ccn-baseline.json` and the hook itself. A
+ * hand-copied table is correct on the day it is written and wrong on the day
+ * someone tunes a threshold — and a self-check list that lies is worse than none,
+ * because it teaches the wrong rule and then contradicts it at commit time.
+ *
+ * Every rule therefore carries an `evidence` pointer, so "the limit is 50" can be
+ * checked or argued with instead of being indistinguishable from a guess.
+ */
+'use strict';
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+/** Read and parse a JSON file, or null when absent/unreadable. */
+function readJson(file) {
+  try {
+    return JSON.parse(fs.readFileSync(file, 'utf8'));
+  } catch {
+    return null;
+  }
+}
+
+/**
+ * Normalise severity spellings.
+ *
+ * The config files and the hook do not agree: `.principlesrc` says `warning`,
+ * Gate 3's boundary is `warn`, and both mean the same thing. Carrying two spellings
+ * into one list would make it look like two different severities exist.
+ */
+function normaliseSeverity(value) {
+  if (value === 'warning') return 'warn';
+  if (value === 'block' || value === 'warn' || value === 'info') return value;
+  return 'warn';
+}
+
+/** Flatten `.principlesrc` rules into entries with source + evidence. */
+function principlesRules(cwd) {
+  const file = path.join(cwd, '.principlesrc');
+  const cfg = readJson(file);
+  if (!cfg || !cfg.rules) return [];
+  const out = [];
+  for (const group of Object.keys(cfg.rules)) {
+    for (const [name, rule] of Object.entries(cfg.rules[group] || {})) {
+      out.push({
+        id: `${group}.${name}`,
+        source: 'principles',
+        severity: normaliseSeverity(rule.severity),
+        // Not every rule is threshold-shaped (e.g. unused-imports); report the
+        // one that exists rather than inventing a 0.
+        threshold: typeof rule.threshold === 'number' ? rule.threshold : null,
+        evidence: '.principlesrc',
+      });
+    }
+  }
+  return out;
+}
+
+/**
+ * Gate 2's duplicate-code rule.
+ *
+ * `githooks/pre-commit` prints "jscpd found duplicated code (warning, not blocking
+ * by default)" -- this gate reports, it does not block. The BUILD-phase doc used to
+ * claim "jscpd similarity >5% BLOCK", which sent people to refactor for a verdict
+ * that never fires. A generated list must record what the hook does, not what the
+ * old prose said.
+ */
+function jscpdRules(cwd) {
+  const cfg = readJson(path.join(cwd, 'jscpd.conf.json'));
+  // jscpd itself has no similarity threshold in this project's config and the hook
+  // never compares a percentage. Reporting null says "no limit declared" instead of
+  // inventing a number the hook does not enforce.
+  const threshold = cfg && typeof cfg.threshold === 'number' ? cfg.threshold : null;
+  return [
+    {
+      id: 'duplication.jscpd',
+      source: 'jscpd',
+      severity: 'info',
+      threshold,
+      evidence: 'githooks/pre-commit (Gate 2, advisory)',
+    },
+  ];
+}
+
+/** Gate 3 warn/block boundaries, which the hook itself owns. */
+function ccnRules(cwd) {
+  return [
+    { id: 'ccn.warn', source: 'ccn', severity: 'warn', threshold: 5, evidence: 'githooks/pre-commit (Gate 3)' },
+    { id: 'ccn.block', source: 'ccn', severity: 'block', threshold: 10, evidence: 'githooks/pre-commit (Gate 3)' },
+  ];
+}
+
+/** Boy Scout's structural limits, read from wherever they are configured. */
+function boyScoutRules(cwd) {
+  const out = [
+    {
+      id: 'boy-scout.new-file',
+      source: 'boy-scout',
+      severity: 'block',
+      threshold: 0,
+      evidence: 'src/principles/boy-scout.ts',
+    },
+  ];
+  // The per-function line and nesting limits are principles rules; when
+  // .principlesrc exists they are already listed by principlesRules, so only add
+  // here when it does not (so the BUILD list never silently loses them).
+  const cfg = readJson(path.join(cwd, '.principlesrc'));
+  if (!cfg || !cfg.rules || !cfg.rules['clean-code']) {
+    out.push(
+      { id: 'clean-code.long-function', source: 'boy-scout', severity: 'warn', threshold: 50, evidence: 'defaults' },
+      { id: 'clean-code.deep-nesting', source: 'boy-scout', severity: 'warn', threshold: 4, evidence: 'defaults' },
+    );
+  }
+  return out;
+}
+
+/**
+ * Collect every rule that can block or warn a commit.
+ *
+ * @param {{cwd?: string}} opts
+ * @returns {Array<{id: string, source: string, severity: string, threshold: number|null, evidence: string}>}
+ */
+function collectGateRules(opts = {}) {
+  const cwd = opts.cwd || process.cwd();
+  const rules = [...principlesRules(cwd), ...jscpdRules(cwd), ...ccnRules(cwd), ...boyScoutRules(cwd)];
+  // Stable order: blocking first, then warn/info, so a reader stops at what hurts.
+  const rank = { block: 0, warn: 1, info: 2 };
+  return rules.sort(
+    (a, b) => (rank[a.severity] ?? 3) - (rank[b.severity] ?? 3) || a.id.localeCompare(b.id),
+  );
+}
+
+/** Render the checklist as markdown with an evidence column. */
+function renderGateRules(rules) {
+  const lines = ['| Rule | Severity | Limit | Evidence |', '|---|---|---|---|'];
+  for (const rule of rules) {
+    lines.push(
+      `| \`${rule.id}\` | ${rule.severity} | ${rule.threshold === null ? '—' : rule.threshold} | \`${rule.evidence}\` |`,
+    );
+  }
+  return lines.join('\n');
+}
+
+module.exports = { collectGateRules, renderGateRules };

package/lib/mattpocock-license-verdict.json

@@ -2,6 +2,6 @@
   "verdict": "PASS",
   "license_type": "MIT",
   "sha256": "0e7ac423bf2c6e223b7c5b156f8cf72da49d748e56a1641402c31f22ad07dbb5",
-  "checked_at": "2026-10-30T18:23:41.600Z",
+  "checked_at": "2026-10-30T19:20:46.439Z",
   "source": "network"
 }

package/lib/mirror-manifest.js

@@ -0,0 +1,112 @@
+/**
+ * Mirror manifest: which canonical sources must be byte-identical in the npm package
+ * (#654).
+ *
+ * `githooks/**` and `plugins/*​/src/**` are canonical; `src/npm-package/**` holds
+ * byte-identical mirrors that get published. Editing a mirror only means the next
+ * prepack silently regenerates it — v0.27.3 shipped exactly that bug.
+ *
+ * `AGENTS.md` states the rule clearly and it still gets violated ~58 times, because
+ * the constraint is *structural* (which file is source, which is mirror, who depends
+ * on whom), not stylistic. Knowing the rule does not remove the need to track the
+ * mapping across a multi-file edit; that is tooling's job.
+ *
+ * Hence this module: the mapping is data, and a canonical file missing from the
+ * mapping is an error rather than a silent no-op.
+ */
+'use strict';
+
+const fs = require('node:fs');
+const path = require('node:path');
+
+/** Directories (repo-relative) whose files mirror into the package. */
+const MIRRORED_ROOTS = [
+  { from: 'githooks', to: 'src/npm-package/hooks' },
+];
+
+/** Individual canonical files that mirror into a different location. */
+const INDIVIDUAL_PAIRS = [
+  { source: 'githooks/pre-commit', mirror: 'src/npm-package/hooks/pre-commit' },
+  { source: 'githooks/pre-push', mirror: 'src/npm-package/hooks/pre-push' },
+];
+
+/** Paths inside a mirrored root that are NOT mirrored (tests, docs). */
+const EXCLUDED_PREFIXES = ['__tests__/', 'node_modules/', '.git/'];
+
+function toPosix(p) {
+  return p.split(path.sep).join('/');
+}
+
+/** Recursively list files under `dir`, repo-relative and forward-slashed. */
+function listFiles(root, dir, out = []) {
+  let entries;
+  try {
+    entries = fs.readdirSync(path.join(root, dir), { withFileTypes: true });
+  } catch {
+    return out;
+  }
+  if (!Array.isArray(entries)) return out;
+  for (const entry of entries) {
+    const rel = toPosix(path.join(dir, entry.name));
+    if (entry.isDirectory()) {
+      if (EXCLUDED_PREFIXES.some((p) => `${entry.name}/` === p)) continue;
+      listFiles(root, rel, out);
+    } else {
+      out.push(rel);
+    }
+  }
+  return out;
+}
+
+/**
+ * Expand the declared mapping into concrete source→mirror pairs.
+ *
+ * @param {{cwd?: string}} opts
+ * @returns {Array<{source: string, mirror: string}>}
+ */
+function declareSources(opts = {}) {
+  const cwd = opts.cwd || process.cwd();
+  const pairs = [...INDIVIDUAL_PAIRS];
+  const seen = new Set(pairs.map((p) => p.source));
+  for (const root of MIRRORED_ROOTS) {
+    for (const file of listFiles(cwd, root.from)) {
+      if (seen.has(file)) continue;
+      const rest = file.slice(root.from.length + 1);
+      if (EXCLUDED_PREFIXES.some((prefix) => rest.startsWith(prefix))) continue;
+      pairs.push({ source: file, mirror: `${root.to}/${rest}` });
+      seen.add(file);
+    }
+  }
+  return pairs.sort((a, b) => a.source.localeCompare(b.source));
+}
+
+/**
+ * Canonical files that exist but appear in no mapping entry.
+ *
+ * This is the check that turns "someone forgot to register java.sh" from a
+ * publish-time rollback into a red test at commit time. Silence here is the failure
+ * mode worth preventing: a file that is never mirrored looks fine locally and
+ * disappears from the published package.
+ *
+ * @param {{cwd?: string, files?: string[], declared?: Array<{source: string}>}} opts
+ * @returns {string[]}
+ */
+function findUnlistedSources(opts = {}) {
+  const cwd = opts.cwd || process.cwd();
+  const declared = opts.declared || declareSources({ cwd });
+  const declaredSet = new Set(declared.map((entry) => entry.source));
+  const present = opts.files || listFiles(cwd, 'githooks');
+  return present
+    .filter((file) => !EXCLUDED_PREFIXES.some((prefix) => file.slice('githooks/'.length).startsWith(prefix)))
+    .filter((file) => !declaredSet.has(file))
+    .sort();
+}
+
+/** Human-readable drift report; empty array means clean. */
+function describeUnlisted(unlisted) {
+  return unlisted.map(
+    (file) => `${file} is a canonical source with no mirror entry — the published package will not carry it`,
+  );
+}
+
+module.exports = { declareSources, findUnlistedSources, describeUnlisted, MIRRORED_ROOTS };

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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.35.6",
+  "version": "0.35.8",
   "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.35.6",
+  "version": "0.35.8",
   "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-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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/sprint-flow/references/phase-3-build.md

@@ -34,12 +34,25 @@
 |---|------|------------------------------|----------|
 | 1 | **Gate 3 CCN** | 单函数圈复杂度 **>10 BLOCK**、**>5 WARN** | 拆函数 / 早返回 / 抽策略表；`xp-gate check 3 <file>` |
 | 2 | **Gate 6 Boy Scout** | **新文件 0 warning**；修改文件 warning 数不得高于 `.warnings-baseline.json`；**单函数 ≤50 行**；**嵌套 ≤4 层** | 新文件当"零容忍"写；改动前 `xp-gate check 6` 看基线 |
-| 3 | **Gate 2 重复代码** | jscpd 相似度 **>5% BLOCK** | 抽公共函数，别复制粘贴；`xp-gate check 2 <dir>` |
+| 3 | **Gate 2 重复代码** | jscpd 报告重复**仅提示，不阻断** | 抽公共函数，别复制粘贴；`xp-gate check 2 <dir>` |
 | 4 | **Gate 4 Principles** | 14 条 Clean Code/SOLID 规则，任一 error/warning 即拦（按 `.principlesrc`） | `xp-gate check 4 <file>`；避免超长参数表、空 catch、`as any` |
 | 5 | **Gate 1 Lint** | 适配器 lint 任一 error | 提交前跑一次项目 lint；不要留未用变量/导入 |
 | 6 | **Gate 5 测试** | 测试全绿 + 覆盖率 ≥80%（新 .ts/.tsx 无测试直接 BLOCK） | 新代码先写测试（TDD）；`npx vitest run <file>` |
 | 7 | **mirror/sync 纪律** | 改 `githooks/` 或 `skills/` 后必须 `node src/npm-package/scripts/sync-package-content.js --fix` 且 `bash scripts/check-hook-mirror.sh` 报 PASS | 忘了同步会在 pre-push 的 mirror 校验被打回 |
 
+**权威清单用 `xp-gate gate-rules` 现取，不要背上面这张表。** 上表是手抄的,阈值真正定义在
+`.principlesrc`、`jscpd.conf.json` 和 `githooks/pre-commit` 里。`xp-gate gate-rules` 从那些
+文件生成一份带**出处**的清单:
+
+```bash
+xp-gate gate-rules          # markdown 表:规则 / 严重度 / 阈值 / 出处
+xp-gate gate-rules --json   # 机器可读
+```
+
+为什么必须生成而不是手写:这张表原本写"jscpd 相似度 >5% BLOCK",而 Gate 2 实际只是提示、
+从不阻断——有人会去为一个永不触发的裁定重构代码。**一个说谎的自检清单比没有清单更糟**,
+它会让人按错误规则写,然后在提交时被告知另一套。改阈值时只需改配置,清单自动跟上。
+
 **一条命令抢先看**：`xp-gate check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
 一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
 再决定是否继续写——而不是写到一半才被 `git commit` 拦下。

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/dsh/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/dsh-plugin-xp-gate",
-  "version": "0.35.6",
+  "version": "0.35.8",
   "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/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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/sprint-flow/references/phase-3-build.md

@@ -34,12 +34,25 @@
 |---|------|------------------------------|----------|
 | 1 | **Gate 3 CCN** | 单函数圈复杂度 **>10 BLOCK**、**>5 WARN** | 拆函数 / 早返回 / 抽策略表；`xp-gate check 3 <file>` |
 | 2 | **Gate 6 Boy Scout** | **新文件 0 warning**；修改文件 warning 数不得高于 `.warnings-baseline.json`；**单函数 ≤50 行**；**嵌套 ≤4 层** | 新文件当"零容忍"写；改动前 `xp-gate check 6` 看基线 |
-| 3 | **Gate 2 重复代码** | jscpd 相似度 **>5% BLOCK** | 抽公共函数，别复制粘贴；`xp-gate check 2 <dir>` |
+| 3 | **Gate 2 重复代码** | jscpd 报告重复**仅提示，不阻断** | 抽公共函数，别复制粘贴；`xp-gate check 2 <dir>` |
 | 4 | **Gate 4 Principles** | 14 条 Clean Code/SOLID 规则，任一 error/warning 即拦（按 `.principlesrc`） | `xp-gate check 4 <file>`；避免超长参数表、空 catch、`as any` |
 | 5 | **Gate 1 Lint** | 适配器 lint 任一 error | 提交前跑一次项目 lint；不要留未用变量/导入 |
 | 6 | **Gate 5 测试** | 测试全绿 + 覆盖率 ≥80%（新 .ts/.tsx 无测试直接 BLOCK） | 新代码先写测试（TDD）；`npx vitest run <file>` |
 | 7 | **mirror/sync 纪律** | 改 `githooks/` 或 `skills/` 后必须 `node src/npm-package/scripts/sync-package-content.js --fix` 且 `bash scripts/check-hook-mirror.sh` 报 PASS | 忘了同步会在 pre-push 的 mirror 校验被打回 |
 
+**权威清单用 `xp-gate gate-rules` 现取，不要背上面这张表。** 上表是手抄的,阈值真正定义在
+`.principlesrc`、`jscpd.conf.json` 和 `githooks/pre-commit` 里。`xp-gate gate-rules` 从那些
+文件生成一份带**出处**的清单:
+
+```bash
+xp-gate gate-rules          # markdown 表:规则 / 严重度 / 阈值 / 出处
+xp-gate gate-rules --json   # 机器可读
+```
+
+为什么必须生成而不是手写:这张表原本写"jscpd 相似度 >5% BLOCK",而 Gate 2 实际只是提示、
+从不阻断——有人会去为一个永不触发的裁定重构代码。**一个说谎的自检清单比没有清单更糟**,
+它会让人按错误规则写,然后在提交时被告知另一套。改阈值时只需改配置,清单自动跟上。
+
 **一条命令抢先看**：`xp-gate check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
 一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
 再决定是否继续写——而不是写到一半才被 `git commit` 拦下。

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/opencode/package.json

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

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

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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-30
-**Commit:** 35df3831
+**Commit:** b3549c25
 **Branch:** main
-**Version:** 0.35.6.0
+**Version:** 0.35.8.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.