@boyingliu01/xp-gate 0.32.2 → 0.33.0

package/plugins/claude-code/skills/sprint-flow/references/phase-3-build.md

@@ -22,6 +22,32 @@
 
 ---
 
+## 提交前自检清单（P1 红线前移 — 2026-10-30，直接解 #652）
+
+> 背景：47.6% 的提交是 fix/revert（323/679）。多数返工是"提交时才被门禁打回"——
+> 红线原本只活在 `githooks/` 里，AI 在 BUILD 期间看不到。这一页把会**阻断提交**的
+> 阈值提前到编码时自检，目标是"第一次提交就过"，而不是"提交→被打回→改→再提交"。
+
+**每次准备 `git commit` 前，逐条自检（按被拦频率排序）：**
+
+| # | 门禁 | 红线（会被 BLOCK/WARN 的硬阈值） | 自查方法 |
+|---|------|------------------------------|----------|
+| 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>` |
+| 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 check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
+一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
+再决定是否继续写——而不是写到一半才被 `git commit` 拦下。
+
+**根因提醒**：Gate 3/6 是"返工双雄"。写完一个函数先扫一眼长度和嵌套，比提交后返工快得多。
+
+---
+
 ## BUILD Entry Contract Check (MANDATORY — v0.18.0+, 程序化)
 
 **Purpose**: Validate `slices-manifest.json` schema and slice↔REQ cross-artifact consistency before entering BUILD.
@@ -236,6 +262,27 @@ Without these events, the Harness can only infer changes from file diffs and has
 
 ---
 
+## 工具调用效率自检（P2 硬性 — 2026-10-30 诊断）
+
+> BUILD 阶段是工具调用最密集的阶段，也是"等待时间长"的主要来源。依据
+> `docs/plans/2026-10-30-dev-speed-diagnosis.md`：工具延迟占总耗时 58%，其中 32.7% 的
+> shell 调用是纯读取/搜索、calls/step 仅 1.03、`job_output` 阻塞 66.5 h。以下四条在 BUILD
+> 期间是硬性自检项，不满足不得宣称 REQ 完成。
+
+| ❌ 低效做法 | ✅ 正确做法 |
+|------------|------------|
+| `cat` / `Get-Content` / `sed -n` / `head` 读文件 | 用 `read` 原生工具（毫秒级，可并发） |
+| `grep` / `rg` / `Select-String` / `findstr` 搜内容 | 用 `grep` 原生工具 |
+| `find` / `ls` / `Get-ChildItem` 找文件 | 用 `glob` 原生工具 |
+| 一个一个发独立调用（改完再想下一个） | 多个互不依赖的调用放进**同一步**并发发出 |
+| 用 `job_output wait:true` / `sleep` 反复阻塞等长任务 | `run_in_background` + 完成通知，需要时短超时取一次 |
+| 打包/发布/插件改动只靠 CI 发现契约破坏 | 先写契约测试（`plugins/dsh/src/contract.test.ts` 范式），本地先红 |
+
+**验收目标**：纯读取 shell 占比 <5%（现 32.7%）；calls/step ≥2.0（现 1.03）；`job_output`
+调用数下降 ≥50%。这三项在 `xp-gate retro` 的 activity 统计里可复查。
+
+---
+
 ## Blind Review — Read-Only Subagent Convention (v0.18.0+, 证据型标准步骤)
 
 **Purpose**: Code review by an isolated subagent with read-only tool access. Replaces the former `freeze/unfreeze` mechanism.

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:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.0.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.32.2",
+  "version": "0.33.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/copy-client.cjs

@@ -29,4 +29,9 @@ fs.mkdirSync(path.dirname(OUT), { recursive: true });
 fs.copyFileSync(SRC, OUT);
 
 const rel = (p) => path.relative(PLUGIN_ROOT, p).split(path.sep).join('/');
-console.log(`[client] ${rel(SRC)} -> ${rel(OUT)}`);
+// stderr, not stdout: this runs from `prepack`, and npm's `pack --json` contract
+// is that stdout carries the JSON blob and nothing else. A single console.log here
+// made `npm pack --dry-run --json | jq ...` fail with "Invalid numeric literal"
+// and took the whole publish job down. `prepack.cjs` already logs to stderr for
+// the same reason; match it.
+console.error(`[client] ${rel(SRC)} -> ${rel(OUT)}`);

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

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/sprint-flow/references/phase-3-build.md

@@ -22,6 +22,32 @@
 
 ---
 
+## 提交前自检清单（P1 红线前移 — 2026-10-30，直接解 #652）
+
+> 背景：47.6% 的提交是 fix/revert（323/679）。多数返工是"提交时才被门禁打回"——
+> 红线原本只活在 `githooks/` 里，AI 在 BUILD 期间看不到。这一页把会**阻断提交**的
+> 阈值提前到编码时自检，目标是"第一次提交就过"，而不是"提交→被打回→改→再提交"。
+
+**每次准备 `git commit` 前，逐条自检（按被拦频率排序）：**
+
+| # | 门禁 | 红线（会被 BLOCK/WARN 的硬阈值） | 自查方法 |
+|---|------|------------------------------|----------|
+| 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>` |
+| 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 check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
+一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
+再决定是否继续写——而不是写到一半才被 `git commit` 拦下。
+
+**根因提醒**：Gate 3/6 是"返工双雄"。写完一个函数先扫一眼长度和嵌套，比提交后返工快得多。
+
+---
+
 ## BUILD Entry Contract Check (MANDATORY — v0.18.0+, 程序化)
 
 **Purpose**: Validate `slices-manifest.json` schema and slice↔REQ cross-artifact consistency before entering BUILD.
@@ -236,6 +262,27 @@ Without these events, the Harness can only infer changes from file diffs and has
 
 ---
 
+## 工具调用效率自检（P2 硬性 — 2026-10-30 诊断）
+
+> BUILD 阶段是工具调用最密集的阶段，也是"等待时间长"的主要来源。依据
+> `docs/plans/2026-10-30-dev-speed-diagnosis.md`：工具延迟占总耗时 58%，其中 32.7% 的
+> shell 调用是纯读取/搜索、calls/step 仅 1.03、`job_output` 阻塞 66.5 h。以下四条在 BUILD
+> 期间是硬性自检项，不满足不得宣称 REQ 完成。
+
+| ❌ 低效做法 | ✅ 正确做法 |
+|------------|------------|
+| `cat` / `Get-Content` / `sed -n` / `head` 读文件 | 用 `read` 原生工具（毫秒级，可并发） |
+| `grep` / `rg` / `Select-String` / `findstr` 搜内容 | 用 `grep` 原生工具 |
+| `find` / `ls` / `Get-ChildItem` 找文件 | 用 `glob` 原生工具 |
+| 一个一个发独立调用（改完再想下一个） | 多个互不依赖的调用放进**同一步**并发发出 |
+| 用 `job_output wait:true` / `sleep` 反复阻塞等长任务 | `run_in_background` + 完成通知，需要时短超时取一次 |
+| 打包/发布/插件改动只靠 CI 发现契约破坏 | 先写契约测试（`plugins/dsh/src/contract.test.ts` 范式），本地先红 |
+
+**验收目标**：纯读取 shell 占比 <5%（现 32.7%）；calls/step ≥2.0（现 1.03）；`job_output`
+调用数下降 ≥50%。这三项在 `xp-gate retro` 的 activity 统计里可复查。
+
+---
+
 ## Blind Review — Read-Only Subagent Convention (v0.18.0+, 证据型标准步骤)
 
 **Purpose**: Code review by an isolated subagent with read-only tool access. Replaces the former `freeze/unfreeze` mechanism.

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

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

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest"
-import { readFileSync } from "node:fs"
+import { readFileSync, readdirSync } from "node:fs"
 import { join, resolve } from "node:path"
 import { fileURLToPath } from "node:url"
 
@@ -95,4 +95,28 @@ describe("packaging contract", () => {
     // A report with no explicit verdict must not be counted toward consensus.
     expect(core).toContain("INVALID_NO_VERDICT")
   })
+
+  /**
+   * @test REQ-DSH-019 prepack scripts keep stdout clean for `npm pack --json`
+   * @intent 发布流水线用 `npm pack --dry-run --json | jq ...` 量 tarball 大小。
+   *         `prepack` 里任何一句 console.log 都会混进那个 JSON 流，jq 报
+   *         "Invalid numeric literal" 并以 exit 5 结束——0.32.2 的 dsh 插件正是
+   *         因此没发出去，而 create-release 依赖三个 publish job，于是 tag 与
+   *         release 一起被跳过。日志一律走 stderr。
+   * @covers AC-DSH-019-01
+   */
+  it("prepack-time scripts log to stderr, never stdout", () => {
+    const scriptsDir = join(PKG_ROOT, "scripts")
+    const files = readdirSync(scriptsDir).filter((f) => f.endsWith(".cjs"))
+    expect(files.length, "no prepack scripts found").toBeGreaterThan(0)
+
+    for (const f of files) {
+      const source = readFileSync(join(scriptsDir, f), "utf8")
+      // Strip comments so an explanatory "console.log" in prose does not fail it.
+      const code = source.replace(/\/\*[\s\S]*?\*\//g, "").replace(/\/\/.*$/gm, "")
+      expect(code, `${f} writes to stdout and would corrupt npm pack --json`).not.toMatch(
+        /console\.log\s*\(/,
+      )
+    }
+  })
 })

package/plugins/opencode/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@boyingliu01/opencode-plugin",
-  "version": "0.32.2",
+  "version": "0.33.0",
   "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:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/sprint-flow/references/phase-3-build.md

@@ -22,6 +22,32 @@
 
 ---
 
+## 提交前自检清单（P1 红线前移 — 2026-10-30，直接解 #652）
+
+> 背景：47.6% 的提交是 fix/revert（323/679）。多数返工是"提交时才被门禁打回"——
+> 红线原本只活在 `githooks/` 里，AI 在 BUILD 期间看不到。这一页把会**阻断提交**的
+> 阈值提前到编码时自检，目标是"第一次提交就过"，而不是"提交→被打回→改→再提交"。
+
+**每次准备 `git commit` 前，逐条自检（按被拦频率排序）：**
+
+| # | 门禁 | 红线（会被 BLOCK/WARN 的硬阈值） | 自查方法 |
+|---|------|------------------------------|----------|
+| 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>` |
+| 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 check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
+一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
+再决定是否继续写——而不是写到一半才被 `git commit` 拦下。
+
+**根因提醒**：Gate 3/6 是"返工双雄"。写完一个函数先扫一眼长度和嵌套，比提交后返工快得多。
+
+---
+
 ## BUILD Entry Contract Check (MANDATORY — v0.18.0+, 程序化)
 
 **Purpose**: Validate `slices-manifest.json` schema and slice↔REQ cross-artifact consistency before entering BUILD.
@@ -236,6 +262,27 @@ Without these events, the Harness can only infer changes from file diffs and has
 
 ---
 
+## 工具调用效率自检（P2 硬性 — 2026-10-30 诊断）
+
+> BUILD 阶段是工具调用最密集的阶段，也是"等待时间长"的主要来源。依据
+> `docs/plans/2026-10-30-dev-speed-diagnosis.md`：工具延迟占总耗时 58%，其中 32.7% 的
+> shell 调用是纯读取/搜索、calls/step 仅 1.03、`job_output` 阻塞 66.5 h。以下四条在 BUILD
+> 期间是硬性自检项，不满足不得宣称 REQ 完成。
+
+| ❌ 低效做法 | ✅ 正确做法 |
+|------------|------------|
+| `cat` / `Get-Content` / `sed -n` / `head` 读文件 | 用 `read` 原生工具（毫秒级，可并发） |
+| `grep` / `rg` / `Select-String` / `findstr` 搜内容 | 用 `grep` 原生工具 |
+| `find` / `ls` / `Get-ChildItem` 找文件 | 用 `glob` 原生工具 |
+| 一个一个发独立调用（改完再想下一个） | 多个互不依赖的调用放进**同一步**并发发出 |
+| 用 `job_output wait:true` / `sleep` 反复阻塞等长任务 | `run_in_background` + 完成通知，需要时短超时取一次 |
+| 打包/发布/插件改动只靠 CI 发现契约破坏 | 先写契约测试（`plugins/dsh/src/contract.test.ts` 范式），本地先红 |
+
+**验收目标**：纯读取 shell 占比 <5%（现 32.7%）；calls/step ≥2.0（现 1.03）；`job_output`
+调用数下降 ≥50%。这三项在 `xp-gate retro` 的 activity 统计里可复查。
+
+---
+
 ## Blind Review — Read-Only Subagent Convention (v0.18.0+, 证据型标准步骤)
 
 **Purpose**: Code review by an isolated subagent with read-only tool access. Replaces the former `freeze/unfreeze` mechanism.

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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.32.2",
+  "version": "0.33.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-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/sprint-flow/references/phase-3-build.md

@@ -22,6 +22,32 @@
 
 ---
 
+## 提交前自检清单（P1 红线前移 — 2026-10-30，直接解 #652）
+
+> 背景：47.6% 的提交是 fix/revert（323/679）。多数返工是"提交时才被门禁打回"——
+> 红线原本只活在 `githooks/` 里，AI 在 BUILD 期间看不到。这一页把会**阻断提交**的
+> 阈值提前到编码时自检，目标是"第一次提交就过"，而不是"提交→被打回→改→再提交"。
+
+**每次准备 `git commit` 前，逐条自检（按被拦频率排序）：**
+
+| # | 门禁 | 红线（会被 BLOCK/WARN 的硬阈值） | 自查方法 |
+|---|------|------------------------------|----------|
+| 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>` |
+| 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 check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
+一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
+再决定是否继续写——而不是写到一半才被 `git commit` 拦下。
+
+**根因提醒**：Gate 3/6 是"返工双雄"。写完一个函数先扫一眼长度和嵌套，比提交后返工快得多。
+
+---
+
 ## BUILD Entry Contract Check (MANDATORY — v0.18.0+, 程序化)
 
 **Purpose**: Validate `slices-manifest.json` schema and slice↔REQ cross-artifact consistency before entering BUILD.
@@ -236,6 +262,27 @@ Without these events, the Harness can only infer changes from file diffs and has
 
 ---
 
+## 工具调用效率自检（P2 硬性 — 2026-10-30 诊断）
+
+> BUILD 阶段是工具调用最密集的阶段，也是"等待时间长"的主要来源。依据
+> `docs/plans/2026-10-30-dev-speed-diagnosis.md`：工具延迟占总耗时 58%，其中 32.7% 的
+> shell 调用是纯读取/搜索、calls/step 仅 1.03、`job_output` 阻塞 66.5 h。以下四条在 BUILD
+> 期间是硬性自检项，不满足不得宣称 REQ 完成。
+
+| ❌ 低效做法 | ✅ 正确做法 |
+|------------|------------|
+| `cat` / `Get-Content` / `sed -n` / `head` 读文件 | 用 `read` 原生工具（毫秒级，可并发） |
+| `grep` / `rg` / `Select-String` / `findstr` 搜内容 | 用 `grep` 原生工具 |
+| `find` / `ls` / `Get-ChildItem` 找文件 | 用 `glob` 原生工具 |
+| 一个一个发独立调用（改完再想下一个） | 多个互不依赖的调用放进**同一步**并发发出 |
+| 用 `job_output wait:true` / `sleep` 反复阻塞等长任务 | `run_in_background` + 完成通知，需要时短超时取一次 |
+| 打包/发布/插件改动只靠 CI 发现契约破坏 | 先写契约测试（`plugins/dsh/src/contract.test.ts` 范式），本地先红 |
+
+**验收目标**：纯读取 shell 占比 <5%（现 32.7%）；calls/step ≥2.0（现 1.03）；`job_output`
+调用数下降 ≥50%。这三项在 `xp-gate retro` 的 activity 统计里可复查。
+
+---
+
 ## Blind Review — Read-Only Subagent Convention (v0.18.0+, 证据型标准步骤)
 
 **Purpose**: Code review by an isolated subagent with read-only tool access. Replaces the former `freeze/unfreeze` mechanism.

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/scripts/sync-package-content.js

@@ -221,7 +221,7 @@ function syncHooks() {
 
   // #505: hook-module-resolution.sh is GENERATED, but it must still ship beside
   // pre-commit — the hook aborts with FATAL when it is missing, by design.
-  const HOOK_FILES = ['pre-commit', 'pre-push', 'post-merge', 'adapter-common.sh', 'hook-module-resolution.sh'];
+  const HOOK_FILES = ['pre-commit', 'pre-push', 'post-commit', 'post-merge', 'adapter-common.sh', 'hook-module-resolution.sh'];
   let copied = 0;
 
   for (const name of HOOK_FILES) {

package/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.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/skills/sprint-flow/references/phase-3-build.md

@@ -22,6 +22,32 @@
 
 ---
 
+## 提交前自检清单（P1 红线前移 — 2026-10-30，直接解 #652）
+
+> 背景：47.6% 的提交是 fix/revert（323/679）。多数返工是"提交时才被门禁打回"——
+> 红线原本只活在 `githooks/` 里，AI 在 BUILD 期间看不到。这一页把会**阻断提交**的
+> 阈值提前到编码时自检，目标是"第一次提交就过"，而不是"提交→被打回→改→再提交"。
+
+**每次准备 `git commit` 前，逐条自检（按被拦频率排序）：**
+
+| # | 门禁 | 红线（会被 BLOCK/WARN 的硬阈值） | 自查方法 |
+|---|------|------------------------------|----------|
+| 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>` |
+| 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 check . --changed --brief` 只跑"本次 diff 会触发的门禁"，
+一屏出 PASS/SKIP/BLOCK，目标 <5 s。把它放在 ralph-loop 每个 REQ 的 step 0，先看红不红，
+再决定是否继续写——而不是写到一半才被 `git commit` 拦下。
+
+**根因提醒**：Gate 3/6 是"返工双雄"。写完一个函数先扫一眼长度和嵌套，比提交后返工快得多。
+
+---
+
 ## BUILD Entry Contract Check (MANDATORY — v0.18.0+, 程序化)
 
 **Purpose**: Validate `slices-manifest.json` schema and slice↔REQ cross-artifact consistency before entering BUILD.
@@ -236,6 +262,27 @@ Without these events, the Harness can only infer changes from file diffs and has
 
 ---
 
+## 工具调用效率自检（P2 硬性 — 2026-10-30 诊断）
+
+> BUILD 阶段是工具调用最密集的阶段，也是"等待时间长"的主要来源。依据
+> `docs/plans/2026-10-30-dev-speed-diagnosis.md`：工具延迟占总耗时 58%，其中 32.7% 的
+> shell 调用是纯读取/搜索、calls/step 仅 1.03、`job_output` 阻塞 66.5 h。以下四条在 BUILD
+> 期间是硬性自检项，不满足不得宣称 REQ 完成。
+
+| ❌ 低效做法 | ✅ 正确做法 |
+|------------|------------|
+| `cat` / `Get-Content` / `sed -n` / `head` 读文件 | 用 `read` 原生工具（毫秒级，可并发） |
+| `grep` / `rg` / `Select-String` / `findstr` 搜内容 | 用 `grep` 原生工具 |
+| `find` / `ls` / `Get-ChildItem` 找文件 | 用 `glob` 原生工具 |
+| 一个一个发独立调用（改完再想下一个） | 多个互不依赖的调用放进**同一步**并发发出 |
+| 用 `job_output wait:true` / `sleep` 反复阻塞等长任务 | `run_in_background` + 完成通知，需要时短超时取一次 |
+| 打包/发布/插件改动只靠 CI 发现契约破坏 | 先写契约测试（`plugins/dsh/src/contract.test.ts` 范式），本地先红 |
+
+**验收目标**：纯读取 shell 占比 <5%（现 32.7%）；calls/step ≥2.0（现 1.03）；`job_output`
+调用数下降 ≥50%。这三项在 `xp-gate retro` 的 activity 统计里可复查。
+
+---
+
 ## Blind Review — Read-Only Subagent Convention (v0.18.0+, 证据型标准步骤)
 
 **Purpose**: Code review by an isolated subagent with read-only tool access. Replaces the former `freeze/unfreeze` mechanism.

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

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
 **Generated:** 2026-10-30
-**Commit:** 78786c3f
+**Commit:** 3c5e9aaf
 **Branch:** main
-**Version:** 0.32.2.0
+**Version:** 0.33.0.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.