@boyingliu01/xp-gate 0.15.5 → 0.18.1

package/lib/sprint-state-migrator.js

@@ -100,6 +100,7 @@ function migrateState(state, stateDir) {
 function createInitialState(projectRoot) {
   return {
     _schema_version: 1,
+    evidence_schema_version: 2,
     id: `sprint-${Date.now()}`,
     task_description: '-',
     phase: 1,

package/lib/sprint-status.js

@@ -9,6 +9,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { execSync } = require('child_process');
 const { discoverActiveSprints } = require('./sprint-discovery');
 const { SprintStateManager } = require('./sprint-state-manager');
 
@@ -216,6 +217,109 @@ function jsonMode(state) {
   return JSON.stringify(state, null, 2);
 }
 
+// #369: fix commit matching — conventional commits + keyword boundary
+const FIX_RE = /^(fix(\(.+\))?:)|\b(fix|bugfix|hotfix|patch|修复)\b/i;
+
+/**
+ * Count fix commits in a git repo since a given date (#369).
+ * Scans all branches (--all) per Round 1 repo-wide decision.
+ * @param {string} repoDir - Repository root
+ * @param {string} sinceDate - ISO 8601 date string
+ * @returns {number} Count of fix commits
+ */
+function getFixCommitCount(repoDir, sinceDate) {
+  try {
+    const out = execSync(
+      `git log --all --since="${sinceDate}" --pretty=format:"%s"`,
+      { cwd: repoDir, encoding: 'utf8', stdio: ['pipe', 'pipe', 'pipe'] }
+    );
+    return out.split('\n').filter(l => l.trim() && FIX_RE.test(l.trim())).length;
+  } catch {
+    return 0;
+  }
+}
+
+/**
+ * Handle --rework-check mode: compute rework rate for recently closed sprints (#369).
+ * @param {string} searchDir - Project root directory
+ * @param {number} windowDays - Window in days after completed_at
+ * @returns {Promise<number>} Exit code
+ */
+async function handleReworkCheck(searchDir, windowDays) {
+  const stateDir = path.join(searchDir, '.sprint-state');
+  const historyDir = path.join(stateDir, 'sprint-history');
+  const sprintStateFile = path.join(stateDir, 'sprint-state.json');
+
+  const sprints = [];
+
+  // Current sprint-state.json
+  if (fs.existsSync(sprintStateFile)) {
+    try {
+      const state = JSON.parse(fs.readFileSync(sprintStateFile, 'utf8'));
+      if (state.metrics && state.metrics.completed_at) {
+        sprints.push({ id: state.id || 'unknown', state, file: sprintStateFile });
+      }
+    } catch { /* skip malformed */ }
+  }
+
+  // History files
+  if (fs.existsSync(historyDir)) {
+    for (const f of fs.readdirSync(historyDir).filter(f => f.endsWith('.json'))) {
+      try {
+        const fp = path.join(historyDir, f);
+        const state = JSON.parse(fs.readFileSync(fp, 'utf8'));
+        if (state.metrics && state.metrics.completed_at) {
+          sprints.push({ id: state.id || f.replace('.json', ''), state, file: fp });
+        }
+      } catch { /* skip malformed */ }
+    }
+  }
+
+  const now = Date.now();
+  const windowMs = windowDays * 86400000;
+  const inWindow = sprints.filter(s => {
+    const t = new Date(s.state.metrics.completed_at).getTime();
+    return !isNaN(t) && (now - t) <= windowMs;
+  });
+
+  if (inWindow.length === 0) {
+    console.log(`No closed sprints within ${windowDays} day${windowDays === 1 ? '' : 's'}`);
+    return 0;
+  }
+
+  console.log(`Rework Rate Check (window: ${windowDays} days)`);
+  console.log('─'.repeat(60));
+
+  let hasAlert = false;
+  for (const { id, state, file } of inWindow) {
+    const totalCommits = state.metrics.total_sprint_commits || 0;
+    const fixCommits = getFixCommitCount(searchDir, state.metrics.completed_at);
+    const rate = fixCommits / Math.max(totalCommits, 1);
+
+    // Write rework_rate back to the source file
+    state.metrics.rework_rate = rate;
+    try {
+      fs.writeFileSync(file, JSON.stringify(state, null, 2));
+    } catch { /* non-fatal */ }
+
+    const pct = (rate * 100).toFixed(1);
+    console.log(`  ${id}: ${pct}% (${fixCommits} fix / ${totalCommits} total commits)`);
+    if (rate > 0.30) {
+      console.log(`  ⚠️  ${id}: ${pct}% rework — exceeds 30% threshold`);
+      hasAlert = true;
+    }
+  }
+
+  console.log('─'.repeat(60));
+  if (hasAlert) {
+    console.log('⚠️  Rework rate alert: one or more sprints exceed 30% threshold');
+  } else {
+    console.log('All sprints within acceptable rework rate (≤30%)');
+  }
+
+  return 0;
+}
+
 /**
  * CLI entry point. Parses args and executes the appropriate mode.
  * @param {string[]} args - CLI subargs (without 'sprint-status')
@@ -237,6 +341,13 @@ async function handleSprintStatus(args = []) {
     }
   }
 
+  // #369: --rework-check is mutually exclusive with regular status view
+  if (args.includes('--rework-check')) {
+    const wdIdx = args.indexOf('--window-days');
+    const windowDays = (wdIdx >= 0 && args[wdIdx + 1]) ? parseInt(args[wdIdx + 1], 10) : 7;
+    return handleReworkCheck(searchDir, isNaN(windowDays) ? 7 : windowDays);
+  }
+
   if (allFlag) {
     return handleAllSprints(searchDir, jsonFlag, watchFlag);
   }

package/mock-policy/AGENTS.md

@@ -1,9 +1,9 @@
 # SRC/MOCK-POLICY KNOWLEDGE BASE
 
-**Generated:** 2026-07-23
-**Commit:** 766602e
+**Generated:** 2026-07-25
+**Commit:** 3150b22
 **Branch:** main
-**Version:** 0.15.5.0
+**Version:** 0.18.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-07-23
-**Commit:** 766602e
+**Generated:** 2026-07-25
+**Commit:** 3150b22
 **Branch:** main
-**Version:** 0.15.5.0
+**Version:** 0.18.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.15.5",
+  "version": "0.18.1",
   "description": "AI-driven development workflow: 6 quality gates + Delphi review + Sprint Flow",
   "bin": {
     "xp-gate": "./bin/xp-gate.js"
@@ -40,7 +40,8 @@
   },
   "scripts": {
     "version": "0.14.31",
-    "prepack": "node scripts/sync-package-content.js",
+    "license-check": "node scripts/check-mattpocock-license.cjs",
+    "prepack": "node scripts/check-mattpocock-license.cjs && node scripts/sync-package-content.js",
     "test": "echo \"Run tests\" && exit 0"
   },
   "keywords": [

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

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

package/plugins/claude-code/skills/batch-grill-me/SKILL.md

@@ -0,0 +1,15 @@
+---
+name: batch-grill-me
+description: A relentless interview that asks every frontier question at once, round by round.
+disable-model-invocation: true
+---
+
+Interview the user relentlessly until you reach a shared understanding. Map this as a **design tree**: every decision branches into the decisions that hang off it.
+
+Work the tree in **rounds**. The **frontier** is every decision whose prerequisites are already settled — the questions you can ask *now* without guessing at answers you haven't heard yet. Ask the whole frontier in one round: number each question and give your recommended answer. Then wait for the user's answers before the next round.
+
+Each round the user answers reshapes the tree — settled decisions push the frontier outward and unblock questions that depended on them. Recompute the frontier and ask the next round. A question whose answer depends on another question still open in this round belongs to a *later* round, not this one.
+
+Finding *facts* is your job, never the user's. When a frontier question needs a fact from the environment (filesystem, tools, etc.), dispatch a sub-agent to find it — don't ask the user for anything you could look up yourself. Don't block on it: a running exploration is an unsettled prerequisite, so only the questions downstream of it wait for the sub-agent to report — ask the rest of the frontier now. The *decisions* are the user's — put each to them and wait.
+
+The session is done when the frontier is empty: every branch of the design tree visited, nothing left silently assumed. Do not act on it until the user confirms you have reached a shared understanding.

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

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-07-23
-**Commit:** 766602e
+**Generated:** 2026-07-25
+**Commit:** 3150b22
 **Branch:** main
-**Version:** 0.15.5.0
+**Version:** 0.18.0.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

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

@@ -3,8 +3,9 @@ name: delphi-review
 version: 1.1.0
 description: >
   Performs multi-round anonymous expert consensus review using the Delphi method. Supports
-  design review (default) and code-walkthrough (--mode code-walkthrough) modes. Uses 2-3 experts
-  from at least 2 different domestic model providers with a >=90% statistical consensus threshold.
+  design review (default), code-walkthrough (--mode code-walkthrough), and requirements
+  (lightweight, --mode requirements) modes. Uses 2-3 experts from at least 2 different domestic
+  model providers with a >=90% statistical consensus threshold.
   Outputs structured verdict (APPROVED/PASS_WITH_CAVEATS/REQUEST_CHANGES/BLOCKED).
 
   WHAT: Anonymous multi-expert review with iterative consensus building for designs, plans,
@@ -20,12 +21,13 @@ description: >
   "多专家评审", "consensus review", "code walkthrough", "push review", "architecture review",
   "delphi review", "run delphi", "start delphi", "评审这个架构", "review this architecture",
   "评审PR", "review this PR with delphi", "delphi评审", "delphi 评审", "run delphi review",
-  "执行delphi", "启动delphi评审".
+  "执行delphi", "启动delphi评审", "review the requirements", "评审需求", "requirements review",
+  "/requirements-review".
   NEGATIVE TRIGGERS: "how does delphi work", "what is delphi review", "code review checklist",
   "review my code quickly", "can you review this", "peer review", "I need a review",
   "explain the review process", "review guidelines", "how to review a design",
   "code review template", "review format", "PR review", "帮我review一下",
-  "just review this", "quick review", "review steps".
+  "just review this", "quick review", "review steps", "review my requirements document".
 maturity: beta
 auto_continue: true
 triggers:
@@ -51,6 +53,10 @@ triggers:
   - "run delphi review"
   - "执行delphi"
   - "启动delphi评审"
+  - "review the requirements"
+  - "评审需求"
+  - "requirements review"
+  - "/requirements-review"
 triggers_negative_examples:
   - "how does delphi work"       # educational question about the process
   - "what is delphi review"      # asking for explanation, not execution
@@ -72,6 +78,7 @@ triggers_negative_examples:
   - "review format"              # asking about format, not executing
   - "show me how to review"      # educational request about reviewing
   - "review checklist example"   # asking for examples, not running review
+  - "review my requirements document"  # educational discussion about requirements, not executing review
 triggers_negative_test_cases:
   - input: "how does delphi work"
     expect: "NOT triggered"
@@ -129,6 +136,16 @@ triggers_negative_test_cases:
     expect: "NOT triggered"
   - input: "run delphi"
     expect: "triggered"
+  - input: "review the requirements"
+    expect: "triggered"
+  - input: "评审需求"
+    expect: "triggered"
+  - input: "requirements review"
+    expect: "triggered"
+  - input: "/requirements-review"
+    expect: "triggered"
+  - input: "review my requirements document"
+    expect: "NOT triggered"
 workflow_steps:
   - "Step 0: Input Validation → Output: [DelphiReview] or [DelphiReview:BLOCKED]"
   - "Round 1: Anonymous Independent Review → Output: [DelphiReview Round 1] expert JSON verdicts"
@@ -203,6 +220,10 @@ tools_denied:
 - run delphi review
 - 执行delphi
 - 启动delphi评审
+- review the requirements
+- 评审需求
+- requirements review
+- /requirements-review
 
 **NOT triggered by:**
 - "how does delphi work" (educational)
@@ -222,6 +243,7 @@ tools_denied:
 - "code review template" (asking for template)
 - "review format" (asking about format)
 - "show me how to review" (educational)
+- "review my requirements document" (educational discussion, not execution)
 
 ## Workflow
 
@@ -280,9 +302,12 @@ Each round MUST output a structured round marker:
 |------|------|------|------|
 | `design`（默认） | `/delphi-review` | 需求/设计/架构/PR 评审 | 共识报告 + specification.yaml |
 | `code-walkthrough` | `--mode code-walkthrough` | git push 前代码走查 | `.code-walkthrough-result.json` |
+| `requirements` | `--mode requirements` | Phase 2 THINK 阶段需求评审 | `.sprint-state/phase-outputs/requirements-reviewed.json` |
 
 **Code Walkthrough 模式**的完整规范 → 详见 `references/code-walkthrough.md`
 
+**Requirements 模式**的完整规范 → 详见 `references/requirements.md`
+
 ---
 
 ## 参数配置
@@ -419,6 +444,28 @@ Phase 0: 准备 → Round 1: 匿名独立评审 → 共识检查
 
 **For code-walkthrough mode**, output follows `.code-walkthrough-result.json` schema (see `references/code-walkthrough.md`).
 
+**For requirements mode**, output follows `requirements-reviewed.json` schema (see `references/requirements.md`):
+
+```json
+{
+  "mode": "requirements",
+  "verdict": "APPROVED|GAPS_FOUND",
+  "timestamp": "2026-07-25T10:30:00Z",
+  "requirements_hash": "<SHA-256 hex digest>",
+  "head_commit": "<git rev-parse HEAD>",
+  "context_file_used": "CONTEXT.md",
+  "round": 1,
+  "expert_verdicts": [
+    { "role": "architecture", "verdict": "APPROVED", "confidence": 9 },
+    { "role": "feasibility", "verdict": "APPROVED", "confidence": 8 }
+  ],
+  "requirements_statement": "<short summary of what was reviewed>",
+  "gaps_found": [],
+  "rounds_used": 1,
+  "escalation_needed": false
+}
+```
+
 **Anti-patterns mapping:**
 - `Round 1 → "评审完成"` → MUST NOT have `verdict: APPROVED` if `critical_issues` exist
 - `只处理 Critical，忽略 Major` → MUST include `major_concerns` array
@@ -452,6 +499,11 @@ Phase 0: 准备 → Round 1: 匿名独立评审 → 共识检查
 {"mode":"code-walkthrough","commit":"abc123...","timestamp":"...","verdict":"APPROVED","consensus_ratio":1.0}
 ```
 
+**Requirements mode APPROVED** → `.sprint-state/delphi-reviewed.json`:
+```json
+{"mode":"requirements","timestamp":"...","verdict":"APPROVED","consensus_ratio":1.0}
+```
+
 ---
 
 ## Anti-Patterns
@@ -468,6 +520,8 @@ Phase 0: 准备 → Round 1: 匿名独立评审 → 共识检查
 
 **Code-walkthrough 专属 Anti-Patterns** → 详见 `references/code-walkthrough.md`
 
+**Requirements 专属 Anti-Patterns** → 详见 `references/requirements.md`
+
 ---
 
 ## Red Flags
@@ -499,5 +553,6 @@ Phase 0: 准备 → Round 1: 匿名独立评审 → 共识检查
 | File | Content |
 |------|---------|
 | `@references/code-walkthrough.md` | Code-walkthrough mode specification |
+| `@references/requirements.md` | Requirements mode specification |
 | `@references/orchestrator-dispatch.md` | Orchestrator auto-dispatch rules (#218 subagent multi-round loop) |
 | `@references/round-templates.md` | Round templates (anonymous/exchange/final/fix report formats) |