@boyingliu01/xp-gate 0.15.5 → 0.18.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/adapter-common.sh +0 -0
- package/adapters/cpp.sh +0 -0
- package/adapters/dart.sh +0 -0
- package/adapters/flutter.sh +0 -0
- package/adapters/go.sh +0 -0
- package/adapters/iac.sh +0 -0
- package/adapters/java.sh +0 -0
- package/adapters/kotlin.sh +0 -0
- package/adapters/objectivec.sh +0 -0
- package/adapters/plugins/book299-20132-python/scripts/install-gradle.sh +0 -0
- package/adapters/plugins/book299-20132-python/scripts/install-maven.sh +0 -0
- package/adapters/plugins/book299-4081-c/scripts/install-gradle.sh +0 -0
- package/adapters/plugins/book299-4081-c/scripts/install-maven.sh +0 -0
- package/adapters/plugins/book299-4083-javascriptes5/scripts/install-gradle.sh +0 -0
- package/adapters/plugins/book299-4083-javascriptes5/scripts/install-maven.sh +0 -0
- package/adapters/powershell.sh +0 -0
- package/adapters/python.sh +0 -0
- package/adapters/shell.sh +0 -0
- package/adapters/swift.sh +0 -0
- package/adapters/typescript.sh +0 -0
- package/bin/xp-gate.js +29 -1
- package/gate-3.sh +0 -0
- package/gate-4.sh +0 -0
- package/gate-7.sh +0 -0
- package/gate-8.sh +0 -0
- package/gate-9.sh +0 -0
- package/hooks/adapter-common.sh +0 -0
- package/hooks/gate-3.sh +0 -0
- package/hooks/gate-4.sh +0 -0
- package/hooks/gate-7.sh +0 -0
- package/hooks/gate-8.sh +0 -0
- package/hooks/gate-9.sh +0 -0
- package/hooks/lib/now-ms.sh +33 -0
- package/hooks/pre-commit +63 -47
- package/hooks/pre-push +0 -0
- package/hooks/sprint-gate.sh +291 -0
- package/lib/__tests__/check-version.test.js +122 -0
- package/lib/__tests__/doctor.test.js +13 -15
- package/lib/__tests__/gate-audit.test.ts +38 -0
- package/lib/__tests__/phase-transition.test.js +512 -272
- package/lib/__tests__/retro.test.js +131 -0
- package/lib/__tests__/sprint-audit.test.js +207 -207
- package/lib/__tests__/sprint-status-rework.test.js +184 -0
- package/lib/check-version.js +9 -0
- package/lib/doctor-tui.js +268 -198
- package/lib/doctor.js +15 -3
- package/lib/gate-audit.ts +18 -7
- package/lib/gate-runner.js +77 -30
- package/lib/init.js +10 -0
- package/lib/language-tools.js +833 -833
- package/lib/mattpocock-license-verdict.json +7 -0
- package/lib/phase-transition.js +499 -249
- package/lib/retro.js +222 -0
- package/lib/shared-utils.js +7 -0
- package/lib/sprint-audit.js +235 -235
- package/lib/sprint-init.js +207 -0
- package/lib/sprint-state-migrator.js +1 -0
- package/lib/sprint-status.js +111 -0
- package/mock-policy/AGENTS.md +3 -3
- package/mutation/AGENTS.md +3 -3
- package/package.json +3 -2
- package/plugins/claude-code/.claude-plugin/plugin.json +1 -1
- package/plugins/claude-code/bin/delphi-review-guard.sh +0 -0
- package/plugins/claude-code/bin/xp-gate-check +0 -0
- package/plugins/claude-code/skills/batch-grill-me/SKILL.md +15 -0
- package/plugins/claude-code/skills/delphi-review/AGENTS.md +3 -3
- package/plugins/claude-code/skills/delphi-review/SKILL.md +59 -4
- package/plugins/claude-code/skills/delphi-review/references/requirements.md +390 -0
- package/plugins/claude-code/skills/delphi-review/references/round-templates.md +67 -0
- package/plugins/claude-code/skills/domain-modeling/ADR-FORMAT.md +64 -0
- package/plugins/claude-code/skills/domain-modeling/CONTEXT-FORMAT.md +60 -0
- package/plugins/claude-code/skills/domain-modeling/SKILL.md +74 -0
- package/plugins/claude-code/skills/grill-with-docs/SKILL.md +7 -0
- package/plugins/claude-code/skills/grilling/SKILL.md +12 -0
- package/plugins/claude-code/skills/sprint-flow/AGENTS.md +3 -3
- package/plugins/claude-code/skills/sprint-flow/SKILL.md +168 -43
- package/plugins/claude-code/skills/sprint-flow/references/components/memory.md +1 -1
- package/plugins/claude-code/skills/sprint-flow/references/components/middleware.md +1 -1
- package/plugins/claude-code/skills/sprint-flow/references/components/skill-invocations.md +21 -20
- package/plugins/claude-code/skills/sprint-flow/references/components/tool-descriptions.md +22 -23
- package/plugins/claude-code/skills/sprint-flow/references/orchestration-rules.md +14 -12
- package/plugins/claude-code/skills/sprint-flow/references/phase-2-design.md +145 -133
- package/plugins/claude-code/skills/sprint-flow/references/phase-3-build.md +69 -2
- package/plugins/claude-code/skills/sprint-flow/references/phase-4-verify.md +93 -31
- package/plugins/claude-code/skills/sprint-flow/references/phase-5-ship.md +99 -14
- package/plugins/claude-code/skills/sprint-flow/references/phase-6-close.md +37 -5
- package/plugins/claude-code/skills/sprint-flow/references/phase-overview.md +58 -53
- package/plugins/claude-code/skills/test-specification-alignment/AGENTS.md +3 -3
- package/plugins/claude-code/skills/test-specification-alignment/SKILL.md +8 -3
- package/plugins/claude-code/skills/to-issues/SKILL.md +1 -1
- package/plugins/opencode/index.ts +1 -0
- package/plugins/opencode/package.json +1 -1
- package/plugins/opencode/skills/batch-grill-me/SKILL.md +15 -0
- package/plugins/opencode/skills/delphi-review/AGENTS.md +3 -3
- package/plugins/opencode/skills/delphi-review/SKILL.md +59 -4
- package/plugins/opencode/skills/delphi-review/references/requirements.md +390 -0
- package/plugins/opencode/skills/delphi-review/references/round-templates.md +67 -0
- package/plugins/opencode/skills/domain-modeling/ADR-FORMAT.md +64 -0
- package/plugins/opencode/skills/domain-modeling/CONTEXT-FORMAT.md +60 -0
- package/plugins/opencode/skills/domain-modeling/SKILL.md +74 -0
- package/plugins/opencode/skills/grill-with-docs/SKILL.md +7 -0
- package/plugins/opencode/skills/grilling/SKILL.md +12 -0
- package/plugins/opencode/skills/sprint-flow/AGENTS.md +3 -3
- package/plugins/opencode/skills/sprint-flow/SKILL.md +168 -43
- package/plugins/opencode/skills/sprint-flow/references/components/memory.md +1 -1
- package/plugins/opencode/skills/sprint-flow/references/components/middleware.md +1 -1
- package/plugins/opencode/skills/sprint-flow/references/components/skill-invocations.md +21 -20
- package/plugins/opencode/skills/sprint-flow/references/components/tool-descriptions.md +22 -23
- package/plugins/opencode/skills/sprint-flow/references/orchestration-rules.md +14 -12
- package/plugins/opencode/skills/sprint-flow/references/phase-2-design.md +145 -133
- package/plugins/opencode/skills/sprint-flow/references/phase-3-build.md +69 -2
- package/plugins/opencode/skills/sprint-flow/references/phase-4-verify.md +93 -31
- package/plugins/opencode/skills/sprint-flow/references/phase-5-ship.md +99 -14
- package/plugins/opencode/skills/sprint-flow/references/phase-6-close.md +37 -5
- package/plugins/opencode/skills/sprint-flow/references/phase-overview.md +58 -53
- package/plugins/opencode/skills/test-specification-alignment/AGENTS.md +3 -3
- package/plugins/opencode/skills/test-specification-alignment/SKILL.md +8 -3
- package/plugins/opencode/skills/to-issues/SKILL.md +1 -1
- package/plugins/qoder/agents/delphi-architecture.md +48 -48
- package/plugins/qoder/agents/delphi-feasibility.md +48 -48
- package/plugins/qoder/agents/delphi-technical.md +48 -48
- package/plugins/qoder/bin/sprint-flow-guard.sh +0 -0
- package/plugins/qoder/bin/xp-gate-check +0 -0
- package/plugins/qoder/plugin.json +1 -1
- package/plugins/qoder/skills/delphi-review/AGENTS.md +3 -3
- package/plugins/qoder/skills/sprint-flow/AGENTS.md +3 -3
- package/plugins/qoder/skills/test-specification-alignment/AGENTS.md +3 -3
- package/principles/AGENTS.md +3 -3
- package/scripts/__tests__/check-mattpocock-license.test.js +229 -0
- package/scripts/__tests__/sync-package-content.test.js +50 -0
- package/scripts/check-mattpocock-license.cjs +302 -0
- package/scripts/delphi-external-review.cjs +432 -432
- package/scripts/migrate-audit-anomalies.mjs +78 -0
- package/scripts/sync-package-content.js +52 -5
- package/skills/batch-grill-me/SKILL.md +15 -0
- package/skills/delphi-review/AGENTS.md +3 -3
- package/skills/delphi-review/SKILL.md +59 -4
- package/skills/delphi-review/references/requirements.md +390 -0
- package/skills/delphi-review/references/round-templates.md +67 -0
- package/skills/domain-modeling/ADR-FORMAT.md +64 -0
- package/skills/domain-modeling/CONTEXT-FORMAT.md +60 -0
- package/skills/domain-modeling/SKILL.md +74 -0
- package/skills/grill-with-docs/SKILL.md +7 -0
- package/skills/grilling/SKILL.md +12 -0
- package/skills/sprint-flow/AGENTS.md +3 -3
- package/skills/sprint-flow/SKILL.md +168 -43
- package/skills/sprint-flow/references/components/memory.md +1 -1
- package/skills/sprint-flow/references/components/middleware.md +1 -1
- package/skills/sprint-flow/references/components/skill-invocations.md +21 -20
- package/skills/sprint-flow/references/components/tool-descriptions.md +22 -23
- package/skills/sprint-flow/references/orchestration-rules.md +14 -12
- package/skills/sprint-flow/references/phase-2-design.md +145 -133
- package/skills/sprint-flow/references/phase-3-build.md +69 -2
- package/skills/sprint-flow/references/phase-4-verify.md +93 -31
- package/skills/sprint-flow/references/phase-5-ship.md +99 -14
- package/skills/sprint-flow/references/phase-6-close.md +37 -5
- package/skills/sprint-flow/references/phase-overview.md +58 -53
- package/skills/test-specification-alignment/AGENTS.md +3 -3
- package/skills/test-specification-alignment/SKILL.md +8 -3
- package/skills/to-issues/SKILL.md +1 -1
- package/adapters/gate-3.sh +0 -68
- package/adapters/gate-4.sh +0 -80
- package/adapters/gate-7.sh +0 -45
- package/adapters/gate-8.sh +0 -56
- package/adapters/gate-9.sh +0 -104
|
@@ -10,7 +10,7 @@
|
|
|
10
10
|
| 原则 | 说明 |
|
|
11
11
|
|------|------|
|
|
12
12
|
| **单一入口** | 用户只需调用 `/sprint-flow`,自动串联全流程 |
|
|
13
|
-
| **自动流水线** |
|
|
13
|
+
| **自动流水线** | 自动执行多个阶段,原生步骤 + 内置 Matt Pocock skill |
|
|
14
14
|
| **关键节点暂停** | APPROVED 确认、DELPHI-GATE 通过、Ship 确认、⚠️ Phase 6 CLOSE 必须人工 |
|
|
15
15
|
| **承认 Emergent** | 用户验收环节必须人工,无法自动化(78% 失败不可见) |
|
|
16
16
|
| **复用现有 Skills** | 不重新发明,整合调用现有体系 |
|
|
@@ -52,11 +52,11 @@ Sprint Flow is NOT just a sequential launcher of existing skills. Here's what ma
|
|
|
52
52
|
|
|
53
53
|
```
|
|
54
54
|
Phase 1/6: PREP → worktree隔离 + 规模评估 + 流程路由
|
|
55
|
-
Phase 2/6: DESIGN →
|
|
56
|
-
Phase 3/6: BUILD → GITHOOKS-GATE → DELPHI-GATE → ralph-loop/TDD →
|
|
57
|
-
Phase 4/6: VERIFY → delphi-review --mode code-walkthrough → test-alignment →
|
|
58
|
-
Phase 5/6: SHIP →
|
|
59
|
-
Phase 6/6: CLOSE → USER ACCEPTANCE (⚠️ 人工) → emergent issues → cleanup
|
|
55
|
+
Phase 2/6: DESIGN → grill-with-docs → R1 需求评审 → 设计文档+APPROVAL → batch-grill-me → R2 delphi-review → to-issues → specification.yaml
|
|
56
|
+
Phase 3/6: BUILD → BUILD-ENTRY-CONTRACT → GITHOOKS-GATE → DELPHI-GATE → ralph-loop/TDD → blind-review (read-only subagent) → verification
|
|
57
|
+
Phase 4/6: VERIFY → delphi-review --mode code-walkthrough → test-alignment (#367 HARD-GATE) → xp-gate check --all → browser (Layer 4) → learnings + retro
|
|
58
|
+
Phase 5/6: SHIP → VERSION-GATE → 分支完成决策(4选项) → native ship (PR) → native land (merge+CI+canary)
|
|
59
|
+
Phase 6/6: CLOSE → USER ACCEPTANCE (⚠️ 人工) → #369 返工指标 → emergent issues → cleanup
|
|
60
60
|
```
|
|
61
61
|
|
|
62
62
|
**v2.0 Compact Redesign** (Issue #290): Merged from 11 phases to 6. PREP = old ISOLATE + AUTO-ESTIMATE. DESIGN = old THINK + PLAN. BUILD = old BUILD (unchanged). VERIFY = old REVIEW + FEEDBACK. SHIP = old SHIP + LAND. CLOSE = old USER ACCEPTANCE + CLEANUP.
|
|
@@ -66,11 +66,11 @@ Phase 6/6: CLOSE → USER ACCEPTANCE (⚠️ 人工) → emergent issues → cle
|
|
|
66
66
|
## 暂停点设计(不是随时停,而是设计明确的暂停点)
|
|
67
67
|
|
|
68
68
|
- **Phase 1/6 PREP**: 保护分支强制隔离 → 输出警告或自动创建 worktree → 自动恢复;AUTO-ESTIMATE 结果展示 → 用户确认 → 按路由继续
|
|
69
|
-
- **Phase 2/6 DESIGN**:
|
|
70
|
-
- **Phase 3/6 BUILD**: DELPHI-GATE 未通过 → BLOCK;验证失败 > max3 → 用户决策修复/放弃;成本超阈值 → 用户确认
|
|
71
|
-
- **Phase 4/6 VERIFY**: delphi code-walkthrough REQUEST_CHANGES → 用户处理 → 重新评审;
|
|
72
|
-
- **Phase 5/6 SHIP**:
|
|
73
|
-
- **Phase 6/6 CLOSE**: SHIP→CLOSE GATE (校验 merge 已完成 + git status clean + 当前在 main) → Backup sprint-state → USER ACCEPTANCE ⚠️ 必须人工验收 → 用户确认后继续;cleanup 完成/失败 → 用户确认 → 结束流程
|
|
69
|
+
- **Phase 2/6 DESIGN**: grill-with-docs 访谈 → R1 需求评审 GAPS_FOUND → 补充访谈(最多 2 轮);设计文档未 APPROVED (HARD-GATE) → 用户修改 → APPROVED 后继续;batch-grill-me 批量决策 → 用户确认;R2 delphi-review 未 APPROVED → 修复 → APPROVED → specification.yaml
|
|
70
|
+
- **Phase 3/6 BUILD**: BUILD-ENTRY-CONTRACT 未通过 → BLOCK;DELPHI-GATE 未通过 → BLOCK;验证失败 > max3 → 用户决策修复/放弃;成本超阈值 → 用户确认
|
|
71
|
+
- **Phase 4/6 VERIFY**: delphi code-walkthrough REQUEST_CHANGES → 用户处理 → 重新评审;test-alignment 失败 (#367) → 回退 Phase 3(不暂停);浏览器验证发现问题 → 回退 Phase 3(不暂停)
|
|
72
|
+
- **Phase 5/6 SHIP**: 分支完成决策 (4选项) → 确认;native ship PR → 用户确认合并;native land 完成/失败 → 用户确认;⚠️ SHIP must complete merge to main + release before Phase 6
|
|
73
|
+
- **Phase 6/6 CLOSE**: SHIP→CLOSE GATE (校验 merge 已完成 + git status clean + 当前在 main) → Backup sprint-state → #369 返工指标写入 → USER ACCEPTANCE ⚠️ 必须人工验收 → 用户确认后继续;cleanup 完成/失败 → 用户确认 → 结束流程
|
|
74
74
|
|
|
75
75
|
---
|
|
76
76
|
|
|
@@ -79,11 +79,11 @@ Phase 6/6: CLOSE → USER ACCEPTANCE (⚠️ 人工) → emergent issues → cle
|
|
|
79
79
|
| Step | Phase | Name | Key Actions | Output |
|
|
80
80
|
|------|-------|------|-------------|--------|
|
|
81
81
|
| 1 | **1/6** | **PREP** | Detect protected branch → Create git worktree → AUTO-ESTIMATE sizing → Classify (lightweight/standard/complex) | Worktree path + impact assessment |
|
|
82
|
-
| 2 | **2/6** | **DESIGN** |
|
|
83
|
-
| 3 | **3/6** | **BUILD** | GITHOOKS-GATE → DELPHI-GATE → ralph-loop (default) or parallel → TDD →
|
|
84
|
-
| 4 | **4/6** | **VERIFY** | delphi-review --mode code-walkthrough → test-specification-alignment →
|
|
85
|
-
| 5 | **5/6** | **SHIP** | VERSION-GATE →
|
|
86
|
-
| 6 | **6/6** | **CLOSE** | SHIP→CLOSE GATE (merge + release verified) → Backup sprint-state → USER ACCEPTANCE (⚠️ mandatory manual) → Capture emergent issues → Cleanup worktree + branch → Sprint summary | Emergent issues list + cleanup report |
|
|
82
|
+
| 2 | **2/6** | **DESIGN** | grill-with-docs → R1 需求评审 → 设计文档+APPROVAL (HARD-GATE) → batch-grill-me → R2 delphi-review (≥90% consensus) → to-issues → specification.yaml | specification.yaml + slices-manifest.json + requirements-reviewed.json |
|
|
83
|
+
| 3 | **3/6** | **BUILD** | BUILD-ENTRY-CONTRACT → GITHOOKS-GATE → DELPHI-GATE → ralph-loop (default) or parallel → TDD → blind-review (read-only subagent) → verification | MVP code |
|
|
84
|
+
| 4 | **4/6** | **VERIFY** | delphi-review --mode code-walkthrough → test-specification-alignment (#367 HARD-GATE) → xp-gate check --all → browser (Layer 4 optional) → learnings + xp-gate retro | Review report + feedback-log.md + test-alignment-report.json |
|
|
85
|
+
| 5 | **5/6** | **SHIP** | VERSION-GATE → 分支完成决策 (4选项) → native ship (create PR) → native land (merge + CI + canary) → release | PR URL + deploy status + merge confirmation |
|
|
86
|
+
| 6 | **6/6** | **CLOSE** | SHIP→CLOSE GATE (merge + release verified) → Backup sprint-state → #369 返工指标 → USER ACCEPTANCE (⚠️ mandatory manual) → Capture emergent issues → Cleanup worktree + branch → Sprint summary | Emergent issues list + cleanup report + metrics |
|
|
87
87
|
|
|
88
88
|
**Phase Flow**:
|
|
89
89
|
```
|
|
@@ -94,8 +94,8 @@ PREP → DESIGN → BUILD → VERIFY → SHIP (merge to main + release) → CLOS
|
|
|
94
94
|
```
|
|
95
95
|
|
|
96
96
|
**Hard Gates**:
|
|
97
|
-
- **Phase 2/6 → 3/6 (DESIGN → BUILD)**:
|
|
98
|
-
- **Phase 4/6 → 5/6 (VERIFY → SHIP)**: feedback-log.md must exist (HARD-GATE)
|
|
97
|
+
- **Phase 2/6 → 3/6 (DESIGN → BUILD)**: R1 需求评审 APPROVED + 设计文档用户 APPROVED + R2 delphi-review APPROVED (≥90% consensus) + GITHOOKS-GATE (hooks installed) + DELPHI-GATE (spec APPROVED) + BUILD-ENTRY-CONTRACT (manifest schema + slice↔REQ 一致性)
|
|
98
|
+
- **Phase 4/6 → 5/6 (VERIFY → SHIP)**: feedback-log.md must exist + test-alignment-report.json PASS (#367 程序化 HARD-GATE)
|
|
99
99
|
- **Phase 5/6 → 6/6 (SHIP → CLOSE)**: PR must be merged to main + release completed (HARD-GATE). See `references/phase-6-close.md#ship--close-gate`.
|
|
100
100
|
|
|
101
101
|
---
|
|
@@ -144,7 +144,7 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
144
144
|
6. 输出评估结果 → 用户确认后按路由继续
|
|
145
145
|
7. **参数**: `--no-isolate` (跳过), `--branch-name <name>`, `--force` (绕过保护分支)
|
|
146
146
|
|
|
147
|
-
### Phase 2/6: DESIGN (设计 — 需求探索 +
|
|
147
|
+
### Phase 2/6: DESIGN (设计 — 需求探索 + 双点评审)
|
|
148
148
|
|
|
149
149
|
**对应旧模型**: Phase 0 THINK + Phase 1 PLAN
|
|
150
150
|
|
|
@@ -153,14 +153,17 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
153
153
|
**详细指令**: 参见 `references/phase-2-design.md` — 完整流程、条件分支、HARD-GATE。
|
|
154
154
|
|
|
155
155
|
**快速参考**:
|
|
156
|
-
- **Step 0
|
|
157
|
-
- **Step 1:
|
|
158
|
-
- **Step 2:
|
|
159
|
-
- **Step 3:
|
|
160
|
-
- **Step 4:
|
|
161
|
-
- **Step 5:
|
|
162
|
-
- **
|
|
163
|
-
-
|
|
156
|
+
- **Step 0: CONTEXT.md 预检 (#322)**: 存在则 SKIP grill 访谈,但 R1 需求评审仍执行(CONTEXT.md 可能陈旧)
|
|
157
|
+
- **Step 1: grill-with-docs** — `skill(name="grill-with-docs")` — 逐个追问 + CONTEXT.md/ADR 沉淀。**HARD-GATE**: 设计未批准 → 不可进入实现
|
|
158
|
+
- **Step 2: R1 需求评审 (#368)** — `npx xp-gate delphi-review --mode requirements` — 轻量 2 专家 1 轮,输出 requirements-reviewed.json(含 requirements_hash 防陈旧绑定)。lightweight sprint 跳过 R1 合并入 R2
|
|
159
|
+
- **Step 3: 原生设计文档生成** — `docs/plans/YYYY-MM-DD-<topic>-design.md`(需求摘要、2-3 方案、推荐、成功标准)
|
|
160
|
+
- **Step 4: HARD-GATE APPROVAL** — 用户审批设计文档
|
|
161
|
+
- **Step 5: 路由分叉** — `change_type == "修改已存在代码"` → SKIP batch-grill-me, lightweight R2
|
|
162
|
+
- **Step 6: batch-grill-me** — `skill(name="batch-grill-me")` — 批量前置决策(替代 autoplan taste_decisions)
|
|
163
|
+
- **Step 7: R2 delphi-review** — `skill(name="delphi-review")` — 等待 APPROVED(≥90% 共识)
|
|
164
|
+
- **Step 8: to-issues** — `skill(name="to-issues")` — 垂直切片 Issue 拆分 → slices-manifest.json
|
|
165
|
+
- **Step 9: specification.yaml** — 从 APPROVED 设计文档自动提取(每 REQ 含验收标准 — #368)
|
|
166
|
+
- **Web 前端额外注入**: OPTIONAL design-shotgun(如已安装)
|
|
164
167
|
|
|
165
168
|
### Phase 2/6→3/6: GITHOOKS-GATE(质量门禁安装检查)
|
|
166
169
|
|
|
@@ -183,12 +186,13 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
183
186
|
**详细指令**: 参见 `references/phase-3-build.md`。
|
|
184
187
|
|
|
185
188
|
**快速参考**:
|
|
186
|
-
1. **
|
|
187
|
-
2. **
|
|
188
|
-
3.
|
|
189
|
-
4.
|
|
190
|
-
5.
|
|
191
|
-
6. **
|
|
189
|
+
1. **BUILD-ENTRY-CONTRACT (MANDATORY — v0.18.0+)**: `phase-transition 3 in_progress` 校验 slices-manifest.json schema + slice↔REQ 一致性 → 不合法 BLOCK
|
|
190
|
+
2. **DELPHI-GATE**: 验证 `.sprint-state/delphi-reviewed.json` 存在且 `verdict = "APPROVED"` → 否则 BLOCK
|
|
191
|
+
3. **TDD-GATE (MANDATORY — v0.14.0+)**: 在 delegation 前验证每个 REQ 存在 failing test。无 test → mark `[TDD-RED]`(ralph-loop 创建)。有 test 且 GREEN 且无实现 → BLOCK(TDD bypass)。详见 `references/phase-3-build.md#tdd-gate-pre-implementation-tdd-check-mandatory`。
|
|
192
|
+
4. **输入**: `slices-manifest.json`(Phase 2/6 生成),按 `execution_order` 逐个执行
|
|
193
|
+
5. **模式**: 默认 `ralph-loop`(逐 REQ 迭代,token 节约 40-67%),可选 `--mode parallel`
|
|
194
|
+
6. **Skill 步骤**: BUILD-ENTRY-CONTRACT → hooks-install → TDD-GATE → TDD (RED/GREEN/REFACTOR) → blind-review (read-only subagent, tools: [Read, Grep, Glob]) → verification-before-completion → 成本监控 → learnings.md 写入
|
|
195
|
+
7. **Mock Minimization**: integration-first, mock 仅限 external services, 密度 > 30% 需 `@mock-justified`
|
|
192
196
|
|
|
193
197
|
### Phase 4/6: VERIFY (验证 — 代码走查 + QA + 反馈获取)
|
|
194
198
|
|
|
@@ -198,11 +202,11 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
198
202
|
|
|
199
203
|
**快速参考**:
|
|
200
204
|
- **Orchestrator 直接执行**: delphi-review code-walkthrough 需要用户确认 verdict(Issue #249),**必须由 orchestrator 直接调用** `skill(name="delphi-review")`
|
|
201
|
-
- **执行顺序**: `delphi-review --mode code-walkthrough` → 等待 APPROVED → `test-specification-alignment` → `
|
|
202
|
-
- **
|
|
203
|
-
- **
|
|
204
|
-
- 输出: 评审报告 + feedback-log.md
|
|
205
|
-
- **HARD-GATE**: feedback-log.md must exist before Phase 5/6
|
|
205
|
+
- **执行顺序**: `delphi-review --mode code-walkthrough` → 等待 APPROVED → `test-specification-alignment` (#367 程序化 HARD-GATE: test-alignment-report.json + head_commit + spec_hash) → `xp-gate check --all` → 浏览器验证 (Layer 4: gstack browse > browser-use MCP > SKIP)
|
|
206
|
+
- **Web 前端**: `xp-gate ui-review`(原生)+ OPTIONAL gstack qa/design-review/benchmark(如已安装)
|
|
207
|
+
- **Feedback 子阶段**: learnings.md 写入(原生)+ `xp-gate retro`(原生,含 #369 返工率区块)+ OPTIONAL systematic-debugging(Layer 4,保留"无根因不修复"纪律)
|
|
208
|
+
- 输出: 评审报告 + feedback-log.md + test-alignment-report.json
|
|
209
|
+
- **HARD-GATE**: feedback-log.md must exist + test-alignment-report.json PASS before Phase 5/6
|
|
206
210
|
|
|
207
211
|
### 负载/压力测试(可选)
|
|
208
212
|
|
|
@@ -220,17 +224,17 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
220
224
|
**详细指令**: 参见 `references/phase-5-ship.md` — VERSION-GATE (MANDATORY) / finishing-a-development-branch / VERSION CHANGESET (Issue #142) / changeset schema。
|
|
221
225
|
|
|
222
226
|
**快速参考**:
|
|
223
|
-
- **Step 0: VERSION-GATE (MANDATORY —
|
|
224
|
-
- **Step 1:
|
|
225
|
-
- **
|
|
227
|
+
- **Step 0: VERSION-GATE (MANDATORY — 必须在分支完成决策之前)**: bump VERSION → update CHANGELOG.md → run sync-version.sh → commit + push → verify PR updated。详见 `references/phase-5-ship.md#step-0-version-gate`
|
|
228
|
+
- **Step 1: 分支完成决策(原生 4 选项)**: AskUserQuestion 菜单 (merge/PR/keep/discard),默认推荐 Option 2 (Push and create a Pull Request)
|
|
229
|
+
- **Step 2: 原生 Ship 步骤**: test → VERSION-GATE → commit → push → `gh pr create`
|
|
226
230
|
- 输入: phase-4-summary.md + feedback-log.md → 输出: PR URL (含版本变更 commit)
|
|
227
|
-
- **HARD-GATE**: Phase 4/6 未完成 → BLOCK。验证 `feedback-log.md`
|
|
231
|
+
- **HARD-GATE**: Phase 4/6 未完成 → BLOCK。验证 `feedback-log.md` 存在 + test-alignment-report.json PASS。
|
|
228
232
|
- **GITHOOKS-GATE**: 验证 hooks 完整性,缺失则 `githooks/install.sh`
|
|
229
|
-
- **LAND
|
|
233
|
+
- **LAND(原生步骤)**: merge 确认 → `gh run list` 等 CI (10min) → 等 Deploy (10min) → Canary Health Check (5min)
|
|
230
234
|
- **回滚**: `git revert` 最后一次 merge commit
|
|
231
235
|
- ⚠️ **SHIP COMPLETION**: Phase 5/6 结束前必须确认 PR 已 merge 到 main + release 已创建。未完成 merge 不得进入 Phase 6/6。详见 `references/phase-5-ship.md#ship-completion-gate`。
|
|
232
236
|
|
|
233
|
-
**⚠️ VERSION-GATE
|
|
237
|
+
**⚠️ VERSION-GATE 必须在分支完成决策之前执行。顺序反了会导致 PR 不含版本变更 → CI release workflow 不触发 → 无新版本发布。**
|
|
234
238
|
|
|
235
239
|
### Phase 6/6: CLOSE (收尾 — ⚠️ 人工验收 + 清理)
|
|
236
240
|
|
|
@@ -241,8 +245,9 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
241
245
|
**快速参考**:
|
|
242
246
|
- **SHIP→CLOSE GATE (MANDATORY — v0.14.3+)**: 校验 PR 已 merge、当前在 main 分支、`git status --porcelain` 为空。详见 `references/phase-6-close.md#ship--close-gate`。
|
|
243
247
|
- **Backup sprint-state**(MANDATORY): Phase 6 开始前将 `.sprint-state/` 备份到 repo 追踪路径,防止 worktree 清理后状态丢失。详见 `references/phase-6-close.md#backup-sprint-state`。
|
|
248
|
+
- **#369 返工追踪 (v0.18.0+)**: CLOSE 写入 `metrics.completed_at` + `metrics.total_sprint_commits`(merge-base 到合并点提交数)+ `metrics.merge_base_sha`。`xp-gate sprint-status --rework-check [--window-days N]` 仓库范围扫描 fix 提交,`rework_rate > 30%` 告警。"重新打开的 issue 数"指标已删除。
|
|
244
249
|
- **USER ACCEPTANCE**: ⚠️ **MUST NOT be automated, skipped, or bypassed.** 即使用户说"赶时间"、"跳过验收"、"直接发布",也必须暂停等待用户确认。使用 `@templates/emergent-issues-template.md` 检查清单
|
|
245
|
-
- **CLEANUP**: 保存分支信息 → `git worktree remove <worktree_path>`(精确路径,禁止通配符)→ 删除本地分支 + 远程分支 → 关闭遗留 OPEN PR → 更新 sprint-state.json → 释放 Sprint Lock
|
|
250
|
+
- **CLEANUP**: 保存分支信息 → `git worktree remove <worktree_path>`(精确路径,禁止通配符)→ 删除本地分支 + 远程分支 → 关闭遗留 OPEN PR → 更新 sprint-state.json(含 #369 metrics)→ learnings.md 写入 → 释放 Sprint Lock
|
|
246
251
|
- 输出 Cleanup Report + Sprint Summary
|
|
247
252
|
- **IF emergent issues → Sprint 2**
|
|
248
253
|
|
|
@@ -312,13 +317,13 @@ Sprint Flow: PREP → DESIGN → BUILD → VERIFY → SHIP → CLOSE
|
|
|
312
317
|
|
|
313
318
|
| Phase | Backend (default) | Web Frontend | Mobile | Load Test |
|
|
314
319
|
|-------|------------------|-------------|--------|-----------|
|
|
315
|
-
| 2/6 DESIGN | `
|
|
316
|
-
| 3/6 BUILD | TDD + blind-review | (同 backend) | + `vercel-react-native-skills` / `flutter-review` | (同) |
|
|
317
|
-
| 4/6 VERIFY | `delphi-review --mode code-walkthrough` + `test-specification-alignment` + k6/locust/gatling | + `
|
|
318
|
-
| 4/6 VERIFY (Feedback) |
|
|
319
|
-
| 5/6 SHIP |
|
|
320
|
-
| 6/6 CLOSE | — 人工验收 → cleanup | (同) | (同) | (同) |
|
|
321
|
-
| Browse |
|
|
320
|
+
| 2/6 DESIGN | `grill-with-docs` + R1 `delphi-review --mode requirements` + `batch-grill-me` + R2 `delphi-review` | + OPTIONAL `design-shotgun` | (同 web) | (通用) |
|
|
321
|
+
| 3/6 BUILD | TDD + blind-review (read-only subagent) | (同 backend) | + `vercel-react-native-skills` / `flutter-review` | (同) |
|
|
322
|
+
| 4/6 VERIFY | `delphi-review --mode code-walkthrough` + `test-specification-alignment` + `xp-gate check --all` + k6/locust/gatling | + `xp-gate ui-review` + OPTIONAL qa/design-review/benchmark | Flutter: `flutter-test` / RN: `detox E2E` | k6/locust/gatling |
|
|
323
|
+
| 4/6 VERIFY (Feedback) | learnings.md + `xp-gate retro` | (同) | (同) | (同) |
|
|
324
|
+
| 5/6 SHIP | VERSION-GATE + 原生 4 选项 + native ship + native land | (同) | + platform deploy | (同) |
|
|
325
|
+
| 6/6 CLOSE | — 人工验收 → #369 返工指标 → cleanup | (同) | (同) | (同) |
|
|
326
|
+
| Browse | Layer 4 可选链 (gstack browse > browser-use MCP > SKIP) | 部署 URL + 表单/交互 | Flutter Web / RN Web | (专用) |
|
|
322
327
|
|
|
323
328
|
**Mobile 工具链**: Flutter — `flutter analyze/test/build/pub publish`; RN — `metro/detox/jest/react-native run-ios/android`
|
|
324
329
|
|
|
@@ -451,7 +456,7 @@ Additional reference files:
|
|
|
451
456
|
| Verification improves 2-3x | Boris #1 tip | Phase 4/6 VERIFY 设计 |
|
|
452
457
|
| Emergent requirements 无法消除 | Mike Cohn, Rafael Santos | Phase 6/6 CLOSE 人工设计 |
|
|
453
458
|
| 78% failures invisible | arXiv research | Phase 6/6 必要性证明 |
|
|
454
|
-
| Think → Plan → Build → Ship |
|
|
459
|
+
| Think → Plan → Build → Ship | 行业通用流程 | 整体流程设计 |
|
|
455
460
|
|
|
456
461
|
---
|
|
457
462
|
|
|
@@ -1,9 +1,9 @@
|
|
|
1
1
|
# SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
|
|
2
2
|
|
|
3
|
-
**Generated:** 2026-07-
|
|
4
|
-
**Commit:**
|
|
3
|
+
**Generated:** 2026-07-25
|
|
4
|
+
**Commit:** 3150b22
|
|
5
5
|
**Branch:** main
|
|
6
|
-
**Version:** 0.
|
|
6
|
+
**Version:** 0.18.0.0
|
|
7
7
|
|
|
8
8
|
## OVERVIEW
|
|
9
9
|
Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.
|
|
@@ -10,7 +10,7 @@ description: "Use when asked to run tests, verify tests, align tests with specif
|
|
|
10
10
|
1. **Load Phase 0** — Verify `specification.yaml` and `tests/` directory exist; BLOCK if missing with guidance to complete brainstorming → delphi-review → specification-generator flow first
|
|
11
11
|
2. **Phase 1 Alignment (Modify Allowed)** — Parse specification.yaml + test files (AST); validate alignment rules; generate alignment report; optionally fix tests to align with requirements; require score ≥80% to proceed
|
|
12
12
|
3. **Coverage Mapping** — Map each REQ-* to at least one test with `@test` annotation; map each AC-* to at least one assertion; verify `@test`, `@intent`, `@covers` tags
|
|
13
|
-
4. **Pre-Phase 2 Freeze** —
|
|
13
|
+
4. **Pre-Phase 2 Freeze** — Lock test directories via read-only subagent convention (reviewer tools restricted to `[Read, Grep, Glob]`, no Write/Edit/Bash); return confirmation before proceeding
|
|
14
14
|
5. **Phase 2 Execution (Frozen)** — Run all tests; Agent BLOCKED from modifying/deleting/skipping tests; analyze failures into 4 categories
|
|
15
15
|
6. **Failure Classification** — For each failing test, output with Type, Test, Root Cause, Action fields; SPECIFICATION_ERROR requires ESCALATE_TO_HUMAN with user options (A: fix spec → re-Phase 1, B: confirm spec → modify code, C: clarify ambiguity)
|
|
16
16
|
7. **JSON Report Output** — Generate alignment report as valid JSON
|
|
@@ -31,11 +31,16 @@ description: "Use when asked to run tests, verify tests, align tests with specif
|
|
|
31
31
|
|
|
32
32
|
## Output Format (MANDATORY)
|
|
33
33
|
|
|
34
|
+
> **MANDATORY FILE OUTPUT**: When invoked, you MUST write this report as valid JSON to the file `.sprint-state/phase-outputs/test-alignment-report.json` in the project root. Stdout output alone is not sufficient — `xp-gate phase-transition 4 completed` validates the file, not the stdout.
|
|
35
|
+
|
|
34
36
|
```json
|
|
35
37
|
{
|
|
36
38
|
"alignment_status": "PASS|FAIL|BLOCKED",
|
|
37
39
|
"phase": "1|2",
|
|
38
40
|
"score": 85.5,
|
|
41
|
+
"head_commit": "<git rev-parse HEAD output>",
|
|
42
|
+
"spec_hash": "<SHA-256 of specification.yaml if exists, else null>",
|
|
43
|
+
"timestamp": "<ISO 8601>",
|
|
39
44
|
"misaligned_tests": [
|
|
40
45
|
{"test_name": "test_checkout", "spec_requirement": "REQ-003", "gap": "Missing @test annotation"}
|
|
41
46
|
],
|
|
@@ -44,7 +49,7 @@ description: "Use when asked to run tests, verify tests, align tests with specif
|
|
|
44
49
|
}
|
|
45
50
|
```
|
|
46
51
|
|
|
47
|
-
**Eval assertions check for:** `alignment_status`, `phase`, `score`, `anti_pattern_detected`.
|
|
52
|
+
**Eval assertions check for:** `alignment_status`, `phase`, `score`, `head_commit`, `spec_hash`, `anti_pattern_detected`.
|
|
48
53
|
|
|
49
54
|
---
|
|
50
55
|
|
|
@@ -53,7 +58,7 @@ description: "Use when asked to run tests, verify tests, align tests with specif
|
|
|
53
58
|
### Automatic
|
|
54
59
|
- BUILD (TDD + review) Round 1 after Driver outputs tests
|
|
55
60
|
- Gate 1 verification before proceeding
|
|
56
|
-
-
|
|
61
|
+
- Phase 5 SHIP release before deployment (native phase-5 steps)
|
|
57
62
|
|
|
58
63
|
### Manual
|
|
59
64
|
- `/test-specification-alignment`, `/verify-tests`, "run tests", "verify tests"
|
|
@@ -242,7 +242,7 @@ Phase 1: PLAN → autoplan → delphi-review → specification.yaml
|
|
|
242
242
|
→ **/to-issues** → slices-manifest.json + Issue 列表
|
|
243
243
|
Phase 2: BUILD → 按 execution_order 逐个切片执行
|
|
244
244
|
→ 每个切片 = 一个 REQ(ralph-loop 模式)
|
|
245
|
-
→
|
|
245
|
+
→ 或通过原生 Task 并行派发(无依赖的 AFK 切片)
|
|
246
246
|
Phase 3: REVIEW → **整体 review**(全部切片完成后统一 code-walkthrough)
|
|
247
247
|
→ browse / QA 验证端到端行为
|
|
248
248
|
Phase 4: USER ACC → 人工验收(全部切片集成后的完整功能)
|
|
@@ -1,48 +1,48 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: delphi-architecture
|
|
3
|
-
description: Delphi Method 架构评审专家。独立评审需求对齐度、系统一致性、模块边界清晰度和架构演进性。输出结构化评审意见和量化裁决。
|
|
4
|
-
model: Qwen3.7-Max
|
|
5
|
-
tools: Read, Grep, Glob
|
|
6
|
-
skills: []
|
|
7
|
-
---
|
|
8
|
-
|
|
9
|
-
你是 Delphi Method 架构评审专家(Architecture Expert)。
|
|
10
|
-
|
|
11
|
-
## 评审维度
|
|
12
|
-
|
|
13
|
-
1. **需求对齐度** — 设计是否完整覆盖所有需求,是否有遗漏的验收标准
|
|
14
|
-
2. **系统一致性** — 与现有架构风格、模式是否一致,是否引入不协调的技术选型
|
|
15
|
-
3. **模块边界清晰度** — 职责划分是否明确,耦合度是否合理,接口设计是否干净
|
|
16
|
-
4. **架构演进性** — 是否为未来扩展留有空间,是否过度设计或设计不足
|
|
17
|
-
5. **技术选型合理性** — 技术栈选择是否有充分依据,是否考虑了团队能力和项目约束
|
|
18
|
-
|
|
19
|
-
## 输出要求
|
|
20
|
-
|
|
21
|
-
你的评审必须返回以下结构化格式:
|
|
22
|
-
|
|
23
|
-
```
|
|
24
|
-
## 评审意见
|
|
25
|
-
|
|
26
|
-
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
-
### 置信度: 1-10
|
|
28
|
-
|
|
29
|
-
### 关键问题 (Critical Issues)
|
|
30
|
-
- 必须修复才能通过的问题
|
|
31
|
-
|
|
32
|
-
### 主要关注点 (Major Concerns)
|
|
33
|
-
- 需要认真考虑但不阻塞通过的问题
|
|
34
|
-
|
|
35
|
-
### 次要关注点 (Minor Concerns)
|
|
36
|
-
- 建议性改进
|
|
37
|
-
|
|
38
|
-
### 总结
|
|
39
|
-
- 一段话概括评审结论
|
|
40
|
-
```
|
|
41
|
-
|
|
42
|
-
## 评审原则
|
|
43
|
-
|
|
44
|
-
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
-
- 聚焦架构层面,不要纠结于代码细节
|
|
46
|
-
- 给出具体的改进建议,而不是泛泛而谈
|
|
47
|
-
- 置信度反映你对自己判断的确信程度
|
|
48
|
-
- 如果信息不足以做出判断,明确说明并给出 REQUEST_CHANGES
|
|
1
|
+
---
|
|
2
|
+
name: delphi-architecture
|
|
3
|
+
description: Delphi Method 架构评审专家。独立评审需求对齐度、系统一致性、模块边界清晰度和架构演进性。输出结构化评审意见和量化裁决。
|
|
4
|
+
model: Qwen3.7-Max
|
|
5
|
+
tools: Read, Grep, Glob
|
|
6
|
+
skills: []
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
你是 Delphi Method 架构评审专家(Architecture Expert)。
|
|
10
|
+
|
|
11
|
+
## 评审维度
|
|
12
|
+
|
|
13
|
+
1. **需求对齐度** — 设计是否完整覆盖所有需求,是否有遗漏的验收标准
|
|
14
|
+
2. **系统一致性** — 与现有架构风格、模式是否一致,是否引入不协调的技术选型
|
|
15
|
+
3. **模块边界清晰度** — 职责划分是否明确,耦合度是否合理,接口设计是否干净
|
|
16
|
+
4. **架构演进性** — 是否为未来扩展留有空间,是否过度设计或设计不足
|
|
17
|
+
5. **技术选型合理性** — 技术栈选择是否有充分依据,是否考虑了团队能力和项目约束
|
|
18
|
+
|
|
19
|
+
## 输出要求
|
|
20
|
+
|
|
21
|
+
你的评审必须返回以下结构化格式:
|
|
22
|
+
|
|
23
|
+
```
|
|
24
|
+
## 评审意见
|
|
25
|
+
|
|
26
|
+
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
+
### 置信度: 1-10
|
|
28
|
+
|
|
29
|
+
### 关键问题 (Critical Issues)
|
|
30
|
+
- 必须修复才能通过的问题
|
|
31
|
+
|
|
32
|
+
### 主要关注点 (Major Concerns)
|
|
33
|
+
- 需要认真考虑但不阻塞通过的问题
|
|
34
|
+
|
|
35
|
+
### 次要关注点 (Minor Concerns)
|
|
36
|
+
- 建议性改进
|
|
37
|
+
|
|
38
|
+
### 总结
|
|
39
|
+
- 一段话概括评审结论
|
|
40
|
+
```
|
|
41
|
+
|
|
42
|
+
## 评审原则
|
|
43
|
+
|
|
44
|
+
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
+
- 聚焦架构层面,不要纠结于代码细节
|
|
46
|
+
- 给出具体的改进建议,而不是泛泛而谈
|
|
47
|
+
- 置信度反映你对自己判断的确信程度
|
|
48
|
+
- 如果信息不足以做出判断,明确说明并给出 REQUEST_CHANGES
|
|
@@ -1,48 +1,48 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: delphi-feasibility
|
|
3
|
-
description: Delphi Method 可行性分析专家。独立评审实际约束、风险识别、执行复杂度、替代方案和回滚策略。输出结构化评审意见和量化裁决。
|
|
4
|
-
model: DeepSeek-V4-Pro
|
|
5
|
-
tools: Read, Grep, Glob
|
|
6
|
-
skills: []
|
|
7
|
-
---
|
|
8
|
-
|
|
9
|
-
你是 Delphi Method 可行性分析专家(Feasibility Expert)。
|
|
10
|
-
|
|
11
|
-
## 评审维度
|
|
12
|
-
|
|
13
|
-
1. **实际约束** — 时间/资源/依赖是否在现有约束下可行,是否有不切实际的假设
|
|
14
|
-
2. **风险识别和缓解** — 主要风险是否已识别,缓解措施是否充分,是否有单点故障
|
|
15
|
-
3. **执行复杂度** — 实现难度是否被低估,依赖链是否清晰,是否需要特定领域知识
|
|
16
|
-
4. **替代方案** — 是否有更简单或更可靠的替代方案,当前方案是否是最佳性价比
|
|
17
|
-
5. **回滚策略** — 如果实施失败,是否有退路,是否可渐进式 rollout
|
|
18
|
-
|
|
19
|
-
## 输出要求
|
|
20
|
-
|
|
21
|
-
你的评审必须返回以下结构化格式:
|
|
22
|
-
|
|
23
|
-
```
|
|
24
|
-
## 评审意见
|
|
25
|
-
|
|
26
|
-
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
-
### 置信度: 1-10
|
|
28
|
-
|
|
29
|
-
### 关键问题 (Critical Issues)
|
|
30
|
-
- 必须解决才能继续的问题(不可行假设、关键风险未缓解)
|
|
31
|
-
|
|
32
|
-
### 主要关注点 (Major Concerns)
|
|
33
|
-
- 需要认真考虑但不阻塞通过的问题(执行风险、复杂度估计)
|
|
34
|
-
|
|
35
|
-
### 次要关注点 (Minor Concerns)
|
|
36
|
-
- 建议性改进(更好的替代方案、优化建议)
|
|
37
|
-
|
|
38
|
-
### 总结
|
|
39
|
-
- 一段话概括评审结论
|
|
40
|
-
```
|
|
41
|
-
|
|
42
|
-
## 评审原则
|
|
43
|
-
|
|
44
|
-
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
-
- 聚焦可行性和风险,不要纠结于技术细节
|
|
46
|
-
- 从实际工程经验出发评估执行难度
|
|
47
|
-
- 给出具体的风险缓解建议
|
|
48
|
-
- 置信度反映你对自己判断的确信程度
|
|
1
|
+
---
|
|
2
|
+
name: delphi-feasibility
|
|
3
|
+
description: Delphi Method 可行性分析专家。独立评审实际约束、风险识别、执行复杂度、替代方案和回滚策略。输出结构化评审意见和量化裁决。
|
|
4
|
+
model: DeepSeek-V4-Pro
|
|
5
|
+
tools: Read, Grep, Glob
|
|
6
|
+
skills: []
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
你是 Delphi Method 可行性分析专家(Feasibility Expert)。
|
|
10
|
+
|
|
11
|
+
## 评审维度
|
|
12
|
+
|
|
13
|
+
1. **实际约束** — 时间/资源/依赖是否在现有约束下可行,是否有不切实际的假设
|
|
14
|
+
2. **风险识别和缓解** — 主要风险是否已识别,缓解措施是否充分,是否有单点故障
|
|
15
|
+
3. **执行复杂度** — 实现难度是否被低估,依赖链是否清晰,是否需要特定领域知识
|
|
16
|
+
4. **替代方案** — 是否有更简单或更可靠的替代方案,当前方案是否是最佳性价比
|
|
17
|
+
5. **回滚策略** — 如果实施失败,是否有退路,是否可渐进式 rollout
|
|
18
|
+
|
|
19
|
+
## 输出要求
|
|
20
|
+
|
|
21
|
+
你的评审必须返回以下结构化格式:
|
|
22
|
+
|
|
23
|
+
```
|
|
24
|
+
## 评审意见
|
|
25
|
+
|
|
26
|
+
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
+
### 置信度: 1-10
|
|
28
|
+
|
|
29
|
+
### 关键问题 (Critical Issues)
|
|
30
|
+
- 必须解决才能继续的问题(不可行假设、关键风险未缓解)
|
|
31
|
+
|
|
32
|
+
### 主要关注点 (Major Concerns)
|
|
33
|
+
- 需要认真考虑但不阻塞通过的问题(执行风险、复杂度估计)
|
|
34
|
+
|
|
35
|
+
### 次要关注点 (Minor Concerns)
|
|
36
|
+
- 建议性改进(更好的替代方案、优化建议)
|
|
37
|
+
|
|
38
|
+
### 总结
|
|
39
|
+
- 一段话概括评审结论
|
|
40
|
+
```
|
|
41
|
+
|
|
42
|
+
## 评审原则
|
|
43
|
+
|
|
44
|
+
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
+
- 聚焦可行性和风险,不要纠结于技术细节
|
|
46
|
+
- 从实际工程经验出发评估执行难度
|
|
47
|
+
- 给出具体的风险缓解建议
|
|
48
|
+
- 置信度反映你对自己判断的确信程度
|
|
@@ -1,48 +1,48 @@
|
|
|
1
|
-
---
|
|
2
|
-
name: delphi-technical
|
|
3
|
-
description: Delphi Method 技术实现评审专家。独立评审实现正确性、代码质量、边界处理、性能和可测试性。输出结构化评审意见和量化裁决。
|
|
4
|
-
model: GLM-5.2
|
|
5
|
-
tools: Read, Grep, Glob
|
|
6
|
-
skills: []
|
|
7
|
-
---
|
|
8
|
-
|
|
9
|
-
你是 Delphi Method 技术实现评审专家(Technical Expert)。
|
|
10
|
-
|
|
11
|
-
## 评审维度
|
|
12
|
-
|
|
13
|
-
1. **实现正确性** — 逻辑是否正确,边界条件是否处理,是否有 off-by-one 或空指针风险
|
|
14
|
-
2. **代码质量和设计模式** — 是否遵循 SOLID 原则,命名是否清晰,是否有不必要的复杂性
|
|
15
|
-
3. **边界情况和错误处理** — 异常路径是否覆盖,错误处理是否健壮,是否有静默失败
|
|
16
|
-
4. **性能影响** — 是否有性能瓶颈、N+1 查询、内存泄漏或资源未释放风险
|
|
17
|
-
5. **可测试性** — 代码是否易于编写单元测试,依赖是否可注入,是否有隐藏的全局状态
|
|
18
|
-
|
|
19
|
-
## 输出要求
|
|
20
|
-
|
|
21
|
-
你的评审必须返回以下结构化格式:
|
|
22
|
-
|
|
23
|
-
```
|
|
24
|
-
## 评审意见
|
|
25
|
-
|
|
26
|
-
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
-
### 置信度: 1-10
|
|
28
|
-
|
|
29
|
-
### 关键问题 (Critical Issues)
|
|
30
|
-
- 必须修复才能通过的问题(逻辑错误、安全漏洞、数据丢失风险)
|
|
31
|
-
|
|
32
|
-
### 主要关注点 (Major Concerns)
|
|
33
|
-
- 需要认真考虑但不阻塞通过的问题(性能隐患、设计不够优雅)
|
|
34
|
-
|
|
35
|
-
### 次要关注点 (Minor Concerns)
|
|
36
|
-
- 建议性改进(命名优化、代码组织)
|
|
37
|
-
|
|
38
|
-
### 总结
|
|
39
|
-
- 一段话概括评审结论
|
|
40
|
-
```
|
|
41
|
-
|
|
42
|
-
## 评审原则
|
|
43
|
-
|
|
44
|
-
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
-
- 聚焦实现细节和技术正确性
|
|
46
|
-
- 引用具体代码行号和文件路径
|
|
47
|
-
- 区分"必须修复"和"建议改进"
|
|
48
|
-
- 置信度反映你对自己判断的确信程度
|
|
1
|
+
---
|
|
2
|
+
name: delphi-technical
|
|
3
|
+
description: Delphi Method 技术实现评审专家。独立评审实现正确性、代码质量、边界处理、性能和可测试性。输出结构化评审意见和量化裁决。
|
|
4
|
+
model: GLM-5.2
|
|
5
|
+
tools: Read, Grep, Glob
|
|
6
|
+
skills: []
|
|
7
|
+
---
|
|
8
|
+
|
|
9
|
+
你是 Delphi Method 技术实现评审专家(Technical Expert)。
|
|
10
|
+
|
|
11
|
+
## 评审维度
|
|
12
|
+
|
|
13
|
+
1. **实现正确性** — 逻辑是否正确,边界条件是否处理,是否有 off-by-one 或空指针风险
|
|
14
|
+
2. **代码质量和设计模式** — 是否遵循 SOLID 原则,命名是否清晰,是否有不必要的复杂性
|
|
15
|
+
3. **边界情况和错误处理** — 异常路径是否覆盖,错误处理是否健壮,是否有静默失败
|
|
16
|
+
4. **性能影响** — 是否有性能瓶颈、N+1 查询、内存泄漏或资源未释放风险
|
|
17
|
+
5. **可测试性** — 代码是否易于编写单元测试,依赖是否可注入,是否有隐藏的全局状态
|
|
18
|
+
|
|
19
|
+
## 输出要求
|
|
20
|
+
|
|
21
|
+
你的评审必须返回以下结构化格式:
|
|
22
|
+
|
|
23
|
+
```
|
|
24
|
+
## 评审意见
|
|
25
|
+
|
|
26
|
+
### 裁决: APPROVED | REQUEST_CHANGES | REJECTED
|
|
27
|
+
### 置信度: 1-10
|
|
28
|
+
|
|
29
|
+
### 关键问题 (Critical Issues)
|
|
30
|
+
- 必须修复才能通过的问题(逻辑错误、安全漏洞、数据丢失风险)
|
|
31
|
+
|
|
32
|
+
### 主要关注点 (Major Concerns)
|
|
33
|
+
- 需要认真考虑但不阻塞通过的问题(性能隐患、设计不够优雅)
|
|
34
|
+
|
|
35
|
+
### 次要关注点 (Minor Concerns)
|
|
36
|
+
- 建议性改进(命名优化、代码组织)
|
|
37
|
+
|
|
38
|
+
### 总结
|
|
39
|
+
- 一段话概括评审结论
|
|
40
|
+
```
|
|
41
|
+
|
|
42
|
+
## 评审原则
|
|
43
|
+
|
|
44
|
+
- 独立评审,不要参考其他专家的意见(Round 1)
|
|
45
|
+
- 聚焦实现细节和技术正确性
|
|
46
|
+
- 引用具体代码行号和文件路径
|
|
47
|
+
- 区分"必须修复"和"建议改进"
|
|
48
|
+
- 置信度反映你对自己判断的确信程度
|
|
File without changes
|
|
File without changes
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "xp-gate",
|
|
3
|
-
"version": "0.
|
|
3
|
+
"version": "0.18.0",
|
|
4
4
|
"displayName": "XP-Gate",
|
|
5
5
|
"description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 10 quality gates (Gate 0-9), Sprint Flow (11 phases), and Delphi multi-expert review (>=90% consensus).",
|
|
6
6
|
"author": {
|
|
@@ -1,9 +1,9 @@
|
|
|
1
1
|
# SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
|
|
2
2
|
|
|
3
|
-
**Generated:** 2026-07-
|
|
4
|
-
**Commit:**
|
|
3
|
+
**Generated:** 2026-07-25
|
|
4
|
+
**Commit:** 3150b22
|
|
5
5
|
**Branch:** main
|
|
6
|
-
**Version:** 0.
|
|
6
|
+
**Version:** 0.18.0.0
|
|
7
7
|
|
|
8
8
|
## OVERVIEW
|
|
9
9
|
Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.
|
|
@@ -1,9 +1,9 @@
|
|
|
1
1
|
# SKILLS/SPRINT-FLOW KNOWLEDGE BASE
|
|
2
2
|
|
|
3
|
-
**Generated:** 2026-07-
|
|
4
|
-
**Commit:**
|
|
3
|
+
**Generated:** 2026-07-25
|
|
4
|
+
**Commit:** 3150b22
|
|
5
5
|
**Branch:** main
|
|
6
|
-
**Version:** 0.
|
|
6
|
+
**Version:** 0.18.0.0
|
|
7
7
|
|
|
8
8
|
## OVERVIEW
|
|
9
9
|
**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.
|