@boyingliu01/opencode-plugin 0.12.8 → 0.12.10

package/package.json

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

package/skills/sprint-flow/SKILL.md

@@ -13,7 +13,7 @@ description: >
   - "start sprint"
   - "一键开发"
   - "/sprint-flow"
-   触发后第一行输出: `Sprint Flow: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → SHIP → LAND → USER ACCEPTANCE → FEEDBACK → CLEANUP`
+   触发后第一行输出: `Sprint Flow: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → FEEDBACK → SHIP → LAND → USER ACCEPTANCE → CLEANUP`
   用法: /sprint-flow "[需求描述]"
   示例: /sprint-flow "开发访谈机器人，支持多轮对话"
   可选参数:
@@ -37,6 +37,44 @@ triggers:
   - "开发新功能"
   - "实现 X"
   - "一键开发"
+triggers_negative_examples:
+  - "实现排序算法"          # algorithm implementation, not feature development
+  - "实现一下"              # casual "do it", not a sprint request
+  - "帮我实现这个函数"      # single function, not full feature
+  - "怎么实现登录功能"      # asking HOW, not requesting development
+  - "start spring boot"     # framework name, not sprint
+  - "开发环境配置"          # environment setup, not feature development
+  - "一键部署"              # deploy, not develop
+  - "新功能建议"            # feature suggestion/request, not execution
+  - "implement a sort function"  # algorithm, not full feature
+  - "how to implement auth"      # educational question
+triggers_negative_test_cases:
+  - input: "实现排序算法"
+    expect: "NOT triggered"
+  - input: "实现一下"
+    expect: "NOT triggered"
+  - input: "帮我实现这个函数"
+    expect: "NOT triggered"
+  - input: "怎么实现登录功能"
+    expect: "NOT triggered"
+  - input: "start spring boot"
+    expect: "NOT triggered"
+  - input: "开发环境配置"
+    expect: "NOT triggered"
+  - input: "一键部署"
+    expect: "NOT triggered"
+  - input: "新功能建议"
+    expect: "NOT triggered"
+  - input: "implement a sort function"
+    expect: "NOT triggered"
+  - input: "how to implement auth"
+    expect: "NOT triggered"
+  - input: "开发用户认证模块"
+    expect: "triggered"
+  - input: "/sprint-flow \"开发访谈机器人\""
+    expect: "triggered"
+  - input: "一键开发 REST API"
+    expect: "triggered"
 workflow_steps:
   - "Phase -1: ISOLATE"
   - "Phase -0.5: AUTO-ESTIMATE"
@@ -108,6 +146,35 @@ workflow_steps:
 
 ---
 
+## Unique Value Proposition
+
+Sprint Flow is NOT just a sequential launcher of existing skills. Here's what makes it different from manually running each skill:
+
+### Why Sprint Flow vs Manual Skill Execution
+
+| Dimension | Manual Execution | Sprint Flow |
+|-----------|-----------------|-------------|
+| **Context Continuity** | Each skill starts fresh; lost design decisions between phases | Phase summaries + sprint-state.json maintain full traceability across 11 phases |
+| **Gate Enforcement** | No enforcement — easy to skip Delphi, skip TDD, skip UAT | HARD-GATE: design must be APPROVED (≥90% Delphi consensus) before coding; UAT is mandatory (no bypass) |
+| **Token Efficiency** | Linear context accumulation across phases — ~150K+ tokens | Ralph-loop default: 40-67% token savings via clean REQ-level contexts |
+| **Emergent Requirements** | Discovered late, silently ignored or merged | Phase 7 USER ACCEPTANCE explicitly captures emergent issues via template; triggers Sprint 2 for critical issues |
+| **Quality Ecosystem** | No integration with quality gates | Integrated with xp-gate's full quality ecosystem: Gate 5 (coverage), Gate M2 (mock density), Delphi code-walkthrough |
+| **Progress Tracking** | Ad-hoc, memory-based | `.sprint-state/` persistence with phase history, timing, and metrics — `--status` renders progress dashboard |
+
+### Key Differentiators
+
+1. **40-67% Token Savings via Ralph Loop**: Phase 2 default (`ralph-loop`) processes one REQ at a time with clean context, avoiding the linear context accumulation that costs 150K+ tokens in parallel mode.
+
+2. **HARD-GATE Discipline**: Design must pass Delphi review (≥90% consensus, ≥2 model providers, domestic models only) before Phase 2 BUILD can start. This is enforced both in SKILL.md instructions and via the Claude Code plugin's PreToolUse hook.
+
+3. **Emergent Requirements Acknowledgment**: Based on research showing 78% of failures are invisible to AI (arXiv study), Phase 7 USER ACCEPTANCE is a mandatory manual verification step — cannot be automated, skipped, or bypassed.
+
+4. **XP-Gate Quality Ecosystem Integration**: Sprint Flow is part of the broader xp-gate ecosystem. Phase 2 BUILD integrates with Gate 5 (test coverage ≥80%), Gate M2 (mock density ≤30%), and Gate MW (code-walkthrough validation). Phase 3 REVIEW runs Delphi code-walkthrough that generates `.code-walkthrough-result.json` for pre-push enforcement.
+
+5. **Full Lifecycle Coverage**: From worktree isolation (Phase -1) to cleanup (Phase 8), sprint-flow covers the entire development lifecycle — not just the "write code" part. AUTO-ESTIMATE (Phase -0.5) prevents over-engineering by routing lightweight changes through simplified workflows.
+
+---
+
 ## 完整流程（默认无参数）
 
 调用 `/sprint-flow "[需求描述]"` 后，自动执行以下流程：
@@ -162,7 +229,7 @@ Phase 8: CLEANUP → worktree remove + branch delete + sprint summary
 
 **Phase Flow**:
 ```
-ISOLATE → AUTO-ESTIMATE → THINK → PLAN → [GITHOOKS-GATE] → BUILD → REVIEW → SHIP → LAND → USER ACCEPT → FEEDBACK → CLEANUP
+ISOLATE → AUTO-ESTIMATE → THINK → PLAN → [GITHOOKS-GATE] → BUILD → REVIEW → FEEDBACK → SHIP → LAND → USER ACCEPTANCE → CLEANUP
 ```
 
 **Hard Gates**:
@@ -202,7 +269,7 @@ ISOLATE → AUTO-ESTIMATE → THINK → PLAN → [GITHOOKS-GATE] → BUILD → R
 6. 触发 `/sprint-flow` 后，**第一行输出应包含工作流阶段概览**：
 
 ```
-Sprint Flow: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → SHIP → LAND → USER ACCEPTANCE → FEEDBACK → CLEANUP
+Sprint Flow: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → FEEDBACK → SHIP → LAND → USER ACCEPTANCE → CLEANUP
 ```
 
 ### Phase -1: ISOLATE（git worktree 隔离）
@@ -508,17 +575,22 @@ See [Output Contract](#output-contract) below for the canonical machine-readable
 ## References
 
 详细指令文件位于 `@references/`:
-- `@references/phase-minus-1-isolate.md` — Phase -1 详细指令
-- `@references/phase-minus-0-5-auto-estimate.md` — Phase -0.5 详细指令
-- `@references/phase-0-think.md` — Phase 0 详细指令
-- `@references/phase-1-plan.md` — Phase 1 详细指令
-- `@references/phase-2-build.md` — Phase 2 详细指令
-- `@references/phase-3-review.md` — Phase 3 详细指令
-- `@references/phase-4-uat.md` — Phase 7 USER ACCEPTANCE 详细指令（人工）
-- `@references/phase-5-feedback.md` — Phase 4 FEEDBACK 详细指令
-- `@references/phase-6-ship.md` — Phase 5 SHIP 详细指令
-- `@references/phase-7-land.md` — Phase 6 LAND 详细指令
-- `@references/phase-8-cleanup.md` — Phase 8 CLEANUP 详细指令
+
+> ⚠️ **Filename numbering note**: Reference filenames use legacy numbering that does NOT match the canonical phase numbers in this document. The table below provides the correct mapping.
+
+| File | Canonical Phase | Phase Name |
+|------|----------------|------------|
+| `@references/phase-minus-1-isolate.md` | Phase -1 | ISOLATE |
+| `@references/phase-minus-0-5-auto-estimate.md` | Phase -0.5 | AUTO-ESTIMATE |
+| `@references/phase-0-think.md` | Phase 0 | THINK |
+| `@references/phase-1-plan.md` | Phase 1 | PLAN |
+| `@references/phase-2-build.md` | Phase 2 | BUILD |
+| `@references/phase-3-review.md` | Phase 3 | REVIEW |
+| `@references/phase-5-feedback.md` | Phase 4 | FEEDBACK (⚠️ filename says phase-5) |
+| `@references/phase-6-ship.md` | Phase 5 | SHIP (⚠️ filename says phase-6) |
+| `@references/phase-7-land.md` | Phase 6 | LAND (⚠️ filename says phase-7) |
+| `@references/phase-4-uat.md` | Phase 7 | USER ACCEPTANCE (⚠️ filename says phase-4) |
+| `@references/phase-8-cleanup.md` | Phase 8 | CLEANUP |
 
 ---
 
@@ -543,3 +615,34 @@ See [Output Contract](#output-contract) below for the canonical machine-readable
 | 78% failures invisible | arXiv research | Phase 7 必要性证明 |
 | Think → Plan → Build → Ship | gstack ETHOS | 整体流程设计 |
 
+---
+
+## Phase Flow Consistency
+
+This section documents the canonical 11-phase order to ensure all future edits keep the phase sequence synchronized across all locations in this document and its reference files.
+
+**Canonical Phase Order** (Phase -1, -0.5, 0..8):
+
+| Phase # | Name | Phase # | Name |
+|---------|------|---------|------|
+| -1 | ISOLATE | 4 | FEEDBACK |
+| -0.5 | AUTO-ESTIMATE | 5 | SHIP |
+| 0 | THINK | 6 | LAND |
+| 1 | PLAN | 7 | USER ACCEPTANCE |
+| 2 | BUILD | 8 | CLEANUP |
+| 3 | REVIEW | | |
+
+**Locations that MUST keep this order synchronized**:
+1. **Frontmatter `workflow_steps`** (lines ~40-51): Phase list
+2. **Frontmatter `description` trigger text** (line ~16): Flow string
+3. **Body "完整流程" section** (lines ~115-127): Phase descriptions in order
+4. **Body "Phase Flow" diagram** (line ~164): ASCII flow diagram
+5. **Body "强制输出格式规范" section** (line ~204): Output format
+6. **Body "Workflow Steps" table** (lines ~149-162): Step/Phase table
+7. **Body "References" section** (lines ~516-521): Reference file list
+8. **`references/orchestration-rules.md` Phase Subagent Dispatch Matrix**: Phase/Name order
+9. **`references/phase-4-uat.md`**: Phase 7 USER ACCEPTANCE (⚠️ file named phase-4, phase number is 7)
+10. **`references/phase-5-feedback.md`**: Phase 4 FEEDBACK (⚠️ file named phase-5, phase number is 4)
+
+**⚠️ Known naming discrepancy**: Reference files use legacy numbering (`phase-4-uat.md` for Phase 7, `phase-5-feedback.md` for Phase 4, `phase-6-ship.md` for Phase 5, `phase-7-land.md` for Phase 6). The canonical phase NUMBER is determined by the `Phase #` column above — reference file names are mapped via the References section, not by their filename digits.
+

package/skills/sprint-flow/__tests__/sprint-flow.test.cjs

@@ -0,0 +1,444 @@
+/**
+ * @test REQ-SF-001 Sprint Flow Trigger Accuracy
+ * @intent Verify that sprint-flow SKILL.md trigger detection correctly distinguishes
+ *        between valid sprint-flow requests and unrelated queries
+ * @covers AC-SF-001-01, AC-SF-001-02, AC-SF-001-03
+ */
+const fs = require('fs');
+const path = require('path');
+
+const SKILL_MD_PATH = path.join(__dirname, '..', 'SKILL.md');
+const REFERENCES_DIR = path.join(__dirname, '..', 'references');
+const PHASE_2_BUILD_PATH = path.join(REFERENCES_DIR, 'phase-2-build.md');
+const ORCHESTRATION_RULES_PATH = path.join(REFERENCES_DIR, 'orchestration-rules.md');
+
+let skillContent = '';
+
+function loadSkillMd() {
+  skillContent = fs.readFileSync(SKILL_MD_PATH, 'utf-8');
+}
+
+function parseFrontmatter(content) {
+  const match = content.match(/^---\n([\s\S]*?)\n---/);
+  if (!match) return {};
+  const yaml = match[1];
+  const result = {};
+  // Parse simple YAML key-value pairs and lists
+  const lines = yaml.split('\n');
+  let currentKey = null;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (trimmed === '' || trimmed.startsWith('#')) continue;
+    const kvMatch = trimmed.match(/^(\w[\w_]*):\s*(.*)/);
+    if (kvMatch) {
+      currentKey = kvMatch[1];
+      const val = kvMatch[2].trim();
+      if (val === '') {
+        result[currentKey] = [];
+      } else if (val.startsWith('"') || val.startsWith("'")) {
+        result[currentKey] = val.slice(1, -1);
+      } else {
+        result[currentKey] = val;
+      }
+    } else if (trimmed.startsWith('- ') && currentKey) {
+      const item = trimmed.slice(2).trim().replace(/^["'](.*)["'].*$/, '$1');
+      if (!Array.isArray(result[currentKey])) result[currentKey] = [];
+      result[currentKey].push(item);
+    }
+  }
+  return result;
+}
+
+// === Trigger Accuracy Tests ===
+
+function testPositiveTriggersArePresent() {
+  const frontmatter = parseFrontmatter(skillContent);
+  const triggers = frontmatter.triggers || [];
+  const required = ['/sprint-flow', 'start sprint', '开发新功能', '一键开发'];
+  const missing = required.filter(r => !triggers.some(t => t.includes(r)));
+  if (missing.length > 0) {
+    throw new Error(`Missing positive triggers: ${missing.join(', ')}`);
+  }
+  console.log('  ✓ Positive triggers are present');
+}
+
+function testNegativeTriggersArePresent() {
+  const frontmatter = parseFrontmatter(skillContent);
+  const negTriggers = frontmatter.triggers_negative_examples || [];
+  if (negTriggers.length < 8) {
+    throw new Error(`triggers_negative_examples must have at least 8 entries, got ${negTriggers.length}`);
+  }
+  console.log(`  ✓ Negative triggers have ${negTriggers.length} entries (>= 8 required)`);
+}
+
+function testNegativeTriggerPhrases() {
+  // These phrases must NOT trigger sprint-flow
+  const shouldNotTrigger = [
+    '实现排序算法',
+    '实现一下',
+    '帮我实现这个函数',
+    '怎么实现登录功能',
+    'start spring boot',
+    '开发环境配置',
+    '一键部署',
+    '新功能建议',
+    'implement a sort function',
+    'how to implement auth',
+  ];
+  for (const phrase of shouldNotTrigger) {
+    // Verify phrase is in triggers_negative_examples
+    const frontmatter = parseFrontmatter(skillContent);
+    const negTriggers = frontmatter.triggers_negative_examples || [];
+    const found = negTriggers.some(n => n.includes(phrase) || phrase.includes(n));
+    if (!found) {
+      throw new Error(`Negative trigger phrase "${phrase}" not found in triggers_negative_examples`);
+    }
+  }
+  console.log(`  ✓ All ${shouldNotTrigger.length} negative trigger phrases verified`);
+}
+
+function testPositiveTriggerPhrases() {
+  const shouldTrigger = [
+    '开发用户认证模块',
+    '/sprint-flow',
+    '一键开发 REST API',
+  ];
+  for (const phrase of shouldTrigger) {
+    const frontmatter = parseFrontmatter(skillContent);
+    const triggers = frontmatter.triggers || [];
+    // Match against frontmatter triggers OR check body trigger table for broader patterns
+    const matched = triggers.some(t =>
+      phrase.includes(t) || t.includes(phrase) || phrase.includes('/sprint-flow')
+    ) || skillContent.includes(phrase);
+    if (!matched) {
+      throw new Error(`Positive trigger phrase "${phrase}" should match at least one trigger`);
+    }
+  }
+  console.log(`  ✓ All ${shouldTrigger.length} positive trigger phrases verified`);
+}
+
+function testNegativeTestCasesExist() {
+  // Count test case entries by matching "input:" lines in triggers_negative_test_cases YAML block
+  const fmMatch = skillContent.match(/^---\n([\s\S]*?)\n---/);
+  if (!fmMatch) throw new Error('Frontmatter not found');
+  const yaml = fmMatch[1];
+
+  // Find triggers_negative_test_cases block and count "- input:" lines within it
+  const testCasesSection = yaml.split('triggers_negative_test_cases:')[1];
+  if (!testCasesSection) throw new Error('triggers_negative_test_cases not found in frontmatter');
+
+  const count = (testCasesSection.match(/^\s+- input:/gm) || []).length;
+  if (count < 11) {
+    throw new Error(`triggers_negative_test_cases must have at least 11 entries, got ${count}`);
+  }
+  console.log(`  ✓ Negative test cases have ${count} entries (>= 11 required)`);
+}
+
+// === Phase Transition Tests ===
+
+function testWorkflowStepsOrder() {
+  const frontmatter = parseFrontmatter(skillContent);
+  const steps = frontmatter.workflow_steps || [];
+  if (steps.length !== 11) {
+    throw new Error(`workflow_steps must have exactly 11 entries, got ${steps.length}`);
+  }
+
+  const expectedOrder = [
+    'ISOLATE', 'AUTO-ESTIMATE', 'THINK', 'PLAN', 'BUILD',
+    'REVIEW', 'FEEDBACK', 'SHIP', 'LAND', 'USER ACCEPTANCE', 'CLEANUP',
+  ];
+
+  for (let i = 0; i < expectedOrder.length; i++) {
+    if (!steps[i].includes(expectedOrder[i])) {
+      throw new Error(
+        `workflow_steps[${i}] should contain "${expectedOrder[i]}", got "${steps[i]}"`
+      );
+    }
+  }
+  console.log('  ✓ workflow_steps order matches canonical 11-phase sequence');
+}
+
+function testPhaseFlowDiagramOrder() {
+  // The Phase Flow diagram in the body must show: ISOLATE → ... → FEEDBACK → SHIP → LAND → USER ACCEPTANCE → CLEANUP
+  const flowMatch = skillContent.match(/ISOLATE →.*CLEANUP/);
+  if (!flowMatch) {
+    throw new Error('Phase Flow diagram not found in SKILL.md body');
+  }
+  const flow = flowMatch[0];
+
+  // Verify FEEDBACK comes before SHIP (not after USER ACCEPTANCE)
+  const feedbackPos = flow.indexOf('FEEDBACK');
+  const shipPos = flow.indexOf('SHIP');
+  const userAcceptPos = flow.indexOf('USER ACCEPT');
+  if (feedbackPos === -1 || shipPos === -1 || userAcceptPos === -1) {
+    throw new Error('Phase Flow diagram missing required phases');
+  }
+  if (feedbackPos > shipPos) {
+    throw new Error('FEEDBACK must come before SHIP in phase flow diagram');
+  }
+  if (shipPos > userAcceptPos) {
+    throw new Error('SHIP must come before USER ACCEPTANCE in phase flow diagram');
+  }
+  console.log('  ✓ Phase Flow diagram has correct ordering');
+}
+
+function testPhaseFlowConsistencySectionExists() {
+  if (!skillContent.includes('Phase Flow Consistency')) {
+    throw new Error('SKILL.md must contain "Phase Flow Consistency" section');
+  }
+  console.log('  ✓ Phase Flow Consistency section exists');
+}
+
+function testCanonicalPhaseOrderTableExists() {
+  if (!skillContent.includes('Canonical Phase Order')) {
+    throw new Error('SKILL.md must contain "Canonical Phase Order" table');
+  }
+  console.log('  ✓ Canonical Phase Order table exists');
+}
+
+function testAll11PhasesInPhaseFlowConsistency() {
+  const section = skillContent.split('Phase Flow Consistency')[1];
+  const requiredPhases = [
+    'ISOLATE', 'AUTO-ESTIMATE', 'THINK', 'PLAN', 'BUILD',
+    'REVIEW', 'FEEDBACK', 'SHIP', 'LAND', 'USER ACCEPTANCE', 'CLEANUP',
+  ];
+  for (const phase of requiredPhases) {
+    if (!section.includes(phase)) {
+      throw new Error(`Phase Flow Consistency section missing phase: ${phase}`);
+    }
+  }
+  console.log('  ✓ All 11 phases referenced in Phase Flow Consistency section');
+}
+
+// === Force Levels Tests ===
+
+function testForceLevelsDocumentExists() {
+  const forceLevelsPath = path.join(REFERENCES_DIR, 'force-levels.md');
+  if (!fs.existsSync(forceLevelsPath)) {
+    throw new Error('references/force-levels.md must exist');
+  }
+  const content = fs.readFileSync(forceLevelsPath, 'utf-8');
+  if (!content.includes('轻量') || !content.includes('标准') || !content.includes('复杂')) {
+    throw new Error('force-levels.md must document lightweight/standard/complex levels');
+  }
+  console.log('  ✓ force-levels.md exists and documents three levels');
+}
+
+function testForceLevelsRequiresDelphi() {
+  const forceLevelsContent = fs.readFileSync(
+    path.join(REFERENCES_DIR, 'force-levels.md'), 'utf-8'
+  );
+  if (!forceLevelsContent.includes('Delphi') && !forceLevelsContent.includes('delphi')) {
+    throw new Error('force-levels.md must reference Delphi review requirement');
+  }
+  console.log('  ✓ force-levels.md requires Delphi review for all levels');
+}
+
+// === Value Proposition Tests ===
+
+function testUniqueValuePropositionExists() {
+  if (!skillContent.includes('Unique Value Proposition')) {
+    throw new Error('SKILL.md must contain "Unique Value Proposition" section');
+  }
+  console.log('  ✓ Unique Value Proposition section exists');
+}
+
+function testUvpMentionsTokenSavings() {
+  const section = skillContent.split('Unique Value Proposition')[1];
+  if (!section.includes('40') || !section.includes('67') || !section.includes('token')) {
+    throw new Error('Unique Value Proposition must mention 40-67% token savings');
+  }
+  console.log('  ✓ UVP mentions 40-67% token savings');
+}
+
+function testUvpMentionsHardGate() {
+  const section = skillContent.split('Unique Value Proposition')[1];
+  if (!section.includes('HARD-GATE')) {
+    throw new Error('Unique Value Proposition must mention HARD-GATE discipline');
+  }
+  console.log('  ✓ UVP mentions HARD-GATE discipline');
+}
+
+function testUvpMentionsEmergentRequirements() {
+  const section = skillContent.split('Unique Value Proposition')[1];
+  if (!section.toLowerCase().includes('emergent')) {
+    throw new Error('Unique Value Proposition must mention emergent requirements');
+  }
+  console.log('  ✓ UVP mentions emergent requirements');
+}
+
+// === Phase Timing Tests ===
+
+function testTimingSectionExists() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  if (!phase2BuildContent.includes('Timing & Stability')) {
+    throw new Error('phase-2-build.md must contain "Timing & Stability" section');
+  }
+  console.log('  ✓ Timing & Stability section exists in phase-2-build.md');
+}
+
+function testTimeoutRecommendationsExist() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  const timeoutSection = phase2BuildContent.split('Timing & Stability')[1];
+  if (!timeoutSection || !timeoutSection.includes('Timeout')) {
+    throw new Error('Timing & Stability section must include timeout recommendations');
+  }
+  console.log('  ✓ Timeout recommendations exist');
+}
+
+function testExecutionTimeEstimatesExist() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  if (!phase2BuildContent.includes('Expected Time')) {
+    throw new Error('phase-2-build.md must include expected execution time estimates');
+  }
+  console.log('  ✓ Expected execution time estimates exist');
+}
+
+// === Orchestration Rules Tests ===
+
+function testOrchestrationRulesExists() {
+  if (!fs.existsSync(ORCHESTRATION_RULES_PATH)) {
+    throw new Error('references/orchestration-rules.md must exist');
+  }
+  console.log('  ✓ orchestration-rules.md exists');
+}
+
+function testPhaseSubagentMatrixOrder() {
+  const orchContent = fs.readFileSync(ORCHESTRATION_RULES_PATH, 'utf-8');
+  const expectedOrder = [
+    'ISOLATE', 'AUTO-ESTIMATE', 'THINK', 'PLAN', 'BUILD',
+    'REVIEW', 'USER ACCEPT', 'FEEDBACK', 'SHIP', 'LAND', 'CLEANUP',
+  ];
+  // The matrix order reflects file names, but phases should all be present
+  for (const phase of ['ISOLATE', 'THINK', 'PLAN', 'BUILD', 'REVIEW', 'CLEANUP']) {
+    if (!orchContent.includes(phase)) {
+      throw new Error(`orchestration-rules.md missing phase: ${phase}`);
+    }
+  }
+  console.log('  ✓ Orchestration rules covers all key phases');
+}
+
+// === Run All Tests ===
+
+function runTests(opts = {}) {
+  const { exitOnFail = false } = opts;
+  let passed = 0;
+  let failed = 0;
+  const errors = [];
+
+  const tests = [
+    { name: 'Positive triggers are present', fn: testPositiveTriggersArePresent },
+    { name: 'Negative triggers have >= 8 entries', fn: testNegativeTriggersArePresent },
+    { name: 'Negative trigger phrases verified', fn: testNegativeTriggerPhrases },
+    { name: 'Positive trigger phrases verified', fn: testPositiveTriggerPhrases },
+    { name: 'Negative test cases exist (>= 11)', fn: testNegativeTestCasesExist },
+    { name: 'workflow_steps order matches canonical', fn: testWorkflowStepsOrder },
+    { name: 'Phase Flow diagram has correct ordering', fn: testPhaseFlowDiagramOrder },
+    { name: 'Phase Flow Consistency section exists', fn: testPhaseFlowConsistencySectionExists },
+    { name: 'Canonical Phase Order table exists', fn: testCanonicalPhaseOrderTableExists },
+    { name: 'All 11 phases in consistency section', fn: testAll11PhasesInPhaseFlowConsistency },
+    { name: 'force-levels.md exists with three levels', fn: testForceLevelsDocumentExists },
+    { name: 'force-levels.md requires Delphi review', fn: testForceLevelsRequiresDelphi },
+    { name: 'Unique Value Proposition exists', fn: testUniqueValuePropositionExists },
+    { name: 'UVP mentions 40-67% token savings', fn: testUvpMentionsTokenSavings },
+    { name: 'UVP mentions HARD-GATE', fn: testUvpMentionsHardGate },
+    { name: 'UVP mentions emergent requirements', fn: testUvpMentionsEmergentRequirements },
+    { name: 'Timing & Stability section exists', fn: testTimingSectionExists },
+    { name: 'Timeout recommendations exist', fn: testTimeoutRecommendationsExist },
+    { name: 'Execution time estimates exist', fn: testExecutionTimeEstimatesExist },
+    { name: 'orchestration-rules.md exists', fn: testOrchestrationRulesExists },
+    { name: 'Orchestration covers all key phases', fn: testPhaseSubagentMatrixOrder },
+    { name: 'Uncommitted Changes Gate section exists', fn: testUncommittedGateExists },
+    { name: 'Uncommitted gate has escape valve', fn: testUncommittedGateEscapeValve },
+    { name: 'Uncommitted gate has log path', fn: testUncommittedGateLogPath },
+  ];
+
+  loadSkillMd();
+
+  for (const { name, fn } of tests) {
+    try {
+      fn();
+      passed++;
+    } catch (e) {
+      failed++;
+      errors.push(`FAIL: ${name}: ${e.message}`);
+    }
+  }
+
+  console.log(`\n=== Results: ${passed} passed, ${failed} failed ===\n`);
+  errors.forEach(e => console.log(e));
+
+  if (exitOnFail && failed > 0) {
+    process.exit(1);
+  }
+  return failed > 0 ? 1 : 0;
+}
+
+// Uncommitted Changes Gate tests (must run after the section is added)
+function testUncommittedGateExists() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  if (!phase2BuildContent.includes('Uncommitted Changes Gate')) {
+    throw new Error('phase-2-build.md must contain "Uncommitted Changes Gate" section');
+  }
+  console.log('  ✓ Uncommitted Changes Gate section exists');
+}
+
+function testUncommittedGateEscapeValve() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  if (!phase2BuildContent.includes('SKIP_UNCOMMITTED_GATE')) {
+    throw new Error('Uncommitted Changes Gate must provide SKIP_UNCOMMITTED_GATE escape valve');
+  }
+  console.log('  ✓ Uncommitted gate has SKIP_UNCOMMITTED_GATE escape valve');
+}
+
+function testUncommittedGateLogPath() {
+  const phase2BuildContent = fs.readFileSync(PHASE_2_BUILD_PATH, 'utf-8');
+  if (!phase2BuildContent.includes('.sprint-state/uncommitted-gate-log.json')) {
+    throw new Error('Uncommitted Changes Gate must log to .sprint-state/uncommitted-gate-log.json');
+  }
+  console.log('  ✓ Uncommitted gate logs to .sprint-state/uncommitted-gate-log.json');
+}
+
+// When run directly (node script.js), call with exitOnFail for CLI behavior.
+if (require.main === module) {
+  const exitCode = runTests({ exitOnFail: true });
+  process.exit(exitCode);
+}
+
+// Vitest discovery — wrap each test function as a vitest it() block
+describe('sprint-flow SKILL.md', () => {
+  beforeAll(() => { loadSkillMd(); });
+
+  const tests = [
+    { name: 'Positive triggers are present', fn: testPositiveTriggersArePresent },
+    { name: 'Negative triggers have >= 8 entries', fn: testNegativeTriggersArePresent },
+    { name: 'Negative trigger phrases verified', fn: testNegativeTriggerPhrases },
+    { name: 'Positive trigger phrases verified', fn: testPositiveTriggerPhrases },
+    { name: 'Negative test cases exist (>= 11)', fn: testNegativeTestCasesExist },
+    { name: 'workflow_steps order matches canonical', fn: testWorkflowStepsOrder },
+    { name: 'Phase Flow diagram has correct ordering', fn: testPhaseFlowDiagramOrder },
+    { name: 'Phase Flow Consistency section exists', fn: testPhaseFlowConsistencySectionExists },
+    { name: 'Canonical Phase Order table exists', fn: testCanonicalPhaseOrderTableExists },
+    { name: 'All 11 phases in consistency section', fn: testAll11PhasesInPhaseFlowConsistency },
+    { name: 'force-levels.md exists with three levels', fn: testForceLevelsDocumentExists },
+    { name: 'force-levels.md requires Delphi review', fn: testForceLevelsRequiresDelphi },
+    { name: 'Unique Value Proposition exists', fn: testUniqueValuePropositionExists },
+    { name: 'UVP mentions 40-67% token savings', fn: testUvpMentionsTokenSavings },
+    { name: 'UVP mentions HARD-GATE', fn: testUvpMentionsHardGate },
+    { name: 'UVP mentions emergent requirements', fn: testUvpMentionsEmergentRequirements },
+    { name: 'Timing & Stability section exists', fn: testTimingSectionExists },
+    { name: 'Timeout recommendations exist', fn: testTimeoutRecommendationsExist },
+    { name: 'Execution time estimates exist', fn: testExecutionTimeEstimatesExist },
+    { name: 'orchestration-rules.md exists', fn: testOrchestrationRulesExists },
+    { name: 'Orchestration covers all key phases', fn: testPhaseSubagentMatrixOrder },
+    { name: 'Uncommitted Changes Gate section exists', fn: testUncommittedGateExists },
+    { name: 'Uncommitted gate has escape valve', fn: testUncommittedGateEscapeValve },
+    { name: 'Uncommitted gate has log path', fn: testUncommittedGateLogPath },
+  ];
+
+  for (const { name, fn } of tests) {
+    it(name, () => { fn(); });
+  }
+});
+
+module.exports = { runTests };

package/skills/sprint-flow/references/phase-2-build.md

@@ -20,6 +20,65 @@
 
 ---
 
+---
+
+## Uncommitted Changes Gate
+
+**Purpose**: Prevent entering BUILD with uncommitted changes that could mix with sprint work.
+
+**Execution**: Before entering Phase 2 BUILD, the orchestrator MUST check for uncommitted changes in the worktree.
+
+### Gate Logic
+
+```bash
+# Check for uncommitted changes
+UNCOMMITTED=$(git status --porcelain 2>/dev/null | wc -l | tr -d ' ')
+
+if [ "$SKIP_UNCOMMITTED_GATE" = "1" ]; then
+  echo "[UNCOMMITTED-GATE] Skipped (SKIP_UNCOMMITTED_GATE=1)"
+  echo "{\"skipped\":true,\"reason\":\"SKIP_UNCOMMITTED_GATE=1\",\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+elif [ "$UNCOMMITTED" -gt 0 ]; then
+  echo "⚠️ [UNCOMMITTED-GATE] Found ${UNCOMMITTED} uncommitted files in worktree:"
+  git status --short 2>/dev/null | head -20
+  echo ""
+  echo "Uncommitted changes may conflict with sprint work. Recommended actions:"
+  echo "  1. Commit changes: git add -A && git commit -m 'pre-sprint: save work before BUILD'"
+  echo "  2. Stash changes: git stash push -m 'pre-sprint stash'"
+  echo "  3. Skip gate: export SKIP_UNCOMMITTED_GATE=1 (not recommended)"
+  echo ""
+  echo "Logging to .sprint-state/uncommitted-gate-log.json"
+  echo "{\"blocked\":true,\"uncommitted_files\":${UNCOMMITTED},\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+  echo "[BLOCK] Uncommitted changes detected. Please commit, stash, or set SKIP_UNCOMMITTED_GATE=1 to continue."
+  exit 1
+else
+  echo "✅ [UNCOMMITTED-GATE] Worktree clean. Proceeding to BUILD."
+  echo "{\"clean\":true,\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+fi
+```
+
+### Escape Valve
+
+```bash
+# Skip the uncommitted changes gate (use with caution)
+SKIP_UNCOMMITTED_GATE=1
+```
+
+### Log Format (`.sprint-state/uncommitted-gate-log.json`)
+
+```json
+{
+  "clean": true,
+  "blocked": false,
+  "skipped": false,
+  "uncommitted_files": 0,
+  "timestamp": "2026-07-05T10:00:00Z"
+}
+```
+
+**Log location**: `.sprint-state/uncommitted-gate-log.json` — written on every gate execution for audit trail.
+
+---
+
 ## TDD 强制执行
 
 ### Gate 5a-BLOCK: 新增文件测试强制
@@ -41,3 +100,48 @@
 # 非 main/master 分支临时跳过 Gate 5a-BLOCK
 SKIP_GATE_5A_BLOCK=1 git commit -m "message"
 ```
+
+---
+
+## Timing & Stability
+
+This section documents expected execution times and timeout handling for each Phase 2 sub-step to reduce execution timing stddev and prevent pipeline stalls.
+
+### Expected Execution Times
+
+| Step | Description | Expected Time | Timeout | On Timeout |
+|------|-------------|--------------|---------|------------|
+| DELPHI-GATE | Verify delphi-reviewed.json exists | 1-2s | 10s | BLOCK (critical gate) |
+| ralph-loop (per REQ) | TDD + verification per requirement | 5-15 min | 30 min/REQ | Mark REQ as `timeout`, continue next REQ |
+| parallel dispatch | Multi-agent parallel build | 10-30 min | 45 min | Collect partial results, continue |
+| TDD cycle (per unit) | RED → GREEN → REFACTOR | 2-5 min | 10 min | Skip unit, log failure |
+| freeze + blind-review | Code review in isolation | 5-10 min | 20 min | WARNING, continue |
+| verification | Test suite + coverage check | 2-5 min | 15 min | Retry once, then BLOCK |
+| cost monitor | Token cost accounting | <1s | 5s | Skip, log warning |
+| Phase 2 total (lightweight) | ≤3 REQs | 15-30 min | 45 min | — |
+| Phase 2 total (standard) | 4-10 REQs | 30-120 min | 150 min | — |
+| Phase 2 total (complex) | >10 REQs | 60-240 min | 300 min | — |
+
+### Stability Guidelines
+
+1. **Timeout handling**: All Phase 2 sub-steps MUST have explicit timeouts. If a step times out, log the failure to `.sprint-state/phase-outputs/phase-2-errors.json` and continue to the next step (except DELPHI-GATE which is a hard BLOCK).
+
+2. **Retry strategy**: For recoverable failures (verification, TDD cycle):
+   - First failure: log warning, retry once
+   - Second failure: log error, BLOCK and prompt user decision
+   - Do NOT retry more than twice for any single sub-step
+
+3. **Parallel dispatch stability**: When using `--mode parallel`, if any agent fails:
+   - Collect partial results from successful agents
+   - Rerun failed agent(s) individually with `--mode ralph-loop`
+   - Do NOT rerun the entire parallel batch
+
+4. **Cost monitor thresholds**: 
+   - Token cost per REQ > 50,000 → WARNING (review REQ scope)
+   - Token cost per REQ > 100,000 → BLOCK (REQ too large, split into smaller units)
+
+5. **StdDev reduction**: To reduce execution timing variability:
+   - Cache dependency installation results between REQs
+   - Reuse TDD scaffolding across similar REQ types
+   - Pre-compute code structure analysis once at Phase start
+   - Batch lint/format operations at the end, not per REQ