Skip to content

Commit 6d15896

Browse files
alpha-86claude
alpha-86
and
claude
committed
governance: GOV-009 + GOV-010 skill 结构规范化 (Refs #16)
GOV-009: 职责归属矩阵 + Gate 2 签字规范 - Engineer + QA 联合签字 Tech Spec，Architect 不签自己的 - 6 个 SKILL 强制规则补充"产出者不评审自己"声明 - QA 三方签字定义纠错（CTO → Architect） GOV-010: Skill 结构冲突修复 - HR#1 命名统一为 Human Review #1 - Architect 补全"职责边界"章节 - Platform/SRE 补全"职责边界"和"输出格式"章节 - PMO 章节顺序修正 - Team Lead 禁止行为补充产出者不评审自己原则 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 0dfbdc7 commit 6d15896

15 files changed

Lines changed: 539 additions & 178 deletions

‎docs/README.md‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ AgentDevFlow 是一套自包含的通用产研 Agent 能力包，目标是把可
2525
- 文档规范、会议纪要、Todo 闭环、Issue 驱动交付等过程治理机制
2626
- Agent 创建规范、平台适配约束和跨平台接入方式
2727
- 面向自动化编排的共享 prompts、workflows、templates、playbooks
28+
- **gstack/superpower 增强层**：可选的能力插件层，增强各角色在特定阶段的分析、评审和协作能力。详见 [增强层文档](../platforms/enhancement-layer.md) 和[使用指南](../guides/enhancement-guide.md)。
2829

2930
## 阅读顺序
3031

‎docs/pmo/issues/009_2026-04-17_governance_role_attribution_and_gate2_signatory.md‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# GOV-009 职责归属矩阵缺失 + Gate 2 签字规范问题
2+
3+
## 基本信息
4+
5+
| 字段 | 内容 |
6+
|------|------|
7+
| 类别 | governance |
8+
| 优先级 | P1 |
9+
| 状态 | Open |
10+
| 发现日期 | 2026-04-17 |
11+
| 发现者 | PMO |
12+
| Resolution | `../resolutions/009_2026-04-17_governance_role_attribution_and_gate2_signatory_resolution.md` |
13+
14+
## 问题描述
15+
16+
### 问题 1: Gate 2 — Architect self-review
17+
18+
`prompts/002_develop_pipeline.md` Gate 2 规定"Architect（必签）"，但 Architect 也是 Tech Spec 的产出者，造成 self-review。
19+
20+
**影响**：Tech Spec 评审缺乏真正的独立制衡。
21+
22+
### 问题 2: 职责归属矩阵缺失
23+
24+
PRD / Tech Spec / QA Case 设计文档 / 代码 / 测试报告 各交付物的产出角色、评审角色、评审立场未明确。
25+
26+
### 问题 3: Gate 定义不一致
27+
28+
`prompts/004_delivery_gates.md`（V2.3）和 `prompts/002_develop_pipeline.md`（Gate 0~5）存在重复定义。
29+
30+
## 讨论结论
31+
32+
详见 `../resolutions/009_2026-04-17_governance_role_attribution_and_gate2_signatory_resolution.md`
33+
34+
**已达成的宏观结论**：
35+
1. Gate 2：Engineer + QA 签字，Architect 主持不签自己的
36+
2. 产出者不评审自己，三角制衡
37+
3. 统一 Gate 定义到 `002_develop_pipeline.md`
38+
4. 下一轮逐个细化 5 个产出物的评审逻辑
39+
40+
## 关联 Issue
41+
42+
- Refs #3（Tech Spec self-review 发生在此 Issue 的 Gate 2）
43+
- Refs #16（GitHub Issue 追踪）
44+
45+
## 里程碑
46+
47+
- [ ] 讨论 PRD 评审逻辑
48+
- [ ] 讨论 Tech Spec 评审逻辑
49+
- [ ] 讨论 QA Case 设计文档评审逻辑
50+
- [ ] 讨论代码评审逻辑
51+
- [ ] 讨论测试报告评审逻辑
52+
- [ ] 统一 Gate 定义文档
53+
- [ ] GitHub Issue 关闭
54+
55+
---
56+
57+
*由 PMO (adf-pmo) 生成 | 2026-04-17*

‎docs/pmo/issues/010_2026-04-17_governance_skill_structure_consistency.md‎

Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
# GOV-010 Skill 结构冲突与不一致
2+
3+
## 基本信息
4+
5+
| 字段 | 内容 |
6+
|------|------|
7+
| 类别 | governance |
8+
| 优先级 | P0 |
9+
| 状态 | Closed |
10+
| 发现日期 | 2026-04-17 |
11+
| 发现者 | PMO |
12+
| Resolution | `../resolutions/010_2026-04-17_governance_skill_structure_consistency_resolution.md` |
13+
| GitHub Issue | https://github.com/alpha-86/AgentDevFlow/issues/17 |
14+
15+
## 问题描述
16+
17+
### P0-1: Architect 强制规则未声明"不签自己的交付物"
18+
19+
`prompts/002_product_engineering_roles.md` 明确"产出者不评审自己"原则，但 `skills/architect/SKILL.md` 强制规则中未显式声明。
20+
21+
### P0-2: QA "三方签字"定义错误
22+
23+
`skills/qa-engineer/SKILL.md` L23 写"三方签字（PM + CTO + Engineer）"，但"CTO"角色在 pipeline 中不存在，应为"Architect"。
24+
25+
### P1-3: HR#1 命名混乱
26+
27+
7 个 SKILL 中出现三种写法：人工评审 #1、Human Review #1、PR#1，应统一为"Human Review #1"。
28+
29+
### P1-4: Architect 缺少"职责边界"章节
30+
31+
PM、QA、Engineer、PMO、Team Lead 都有"职责边界"章节，Architect 是唯一缺失的核心角色。
32+
33+
### P1-5: Platform/SRE 缺少"职责边界"和"输出格式"章节
34+
35+
唯一同时缺少这两个章节的角色。
36+
37+
### P2-6: "产出者不评审自己"原则未在各 SKILL 中显式声明
38+
39+
`002_product_engineering_roles.md` 已明确，但各 SKILL 强制规则中无显式声明。
40+
41+
### P2-7: PMO 章节顺序异常
42+
43+
"何时启用"在"必读文档"之后，与其他 SKILL 顺序不一致。
44+
45+
## 讨论结论
46+
47+
详见 `../resolutions/010_2026-04-17_governance_skill_structure_consistency_resolution.md`
48+
49+
## 关联 Issue
50+
51+
- Refs #16（GitHub Issue 追踪）
52+
53+
## 里程碑
54+
55+
- [x] 创建 PMO Issue
56+
- [x] 执行 P0-1/P0-2 修复
57+
- [x] 执行 P1-3 HR#1 命名统一
58+
- [x] 执行 P1-4 Architect 职责边界补全
59+
- [x] 执行 P1-5 Platform/SRE 章节补全
60+
- [x] 执行 P2-6 产出者不评审自己原则声明
61+
- [x] 执行 P2-7 PMO 章节顺序修正
62+
- [x] 创建 GitHub Issue
63+
- [ ] 修复验收（待 Team Lead 确认）
64+
65+
---
66+
67+
*由 PMO (adf-pmo) 生成 | 2026-04-17*

‎docs/pmo/resolutions/009_2026-04-17_governance_role_attribution_and_gate2_signatory_resolution.md‎

Lines changed: 121 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,121 @@
1+
# GOV-009 职责归属矩阵 + Gate 2 签字规范
2+
3+
## 基本信息
4+
5+
| 字段 | 内容 |
6+
|------|------|
7+
| PMO Issue | `docs/pmo/issues/009_2026-04-17_governance_role_attribution_and_gate2_signatory.md` |
8+
| GitHub Issue | https://github.com/alpha-86/AgentDevFlow/issues/16 |
9+
| 讨论日期 | 2026-04-17 |
10+
| 验收日期 | 2026-04-17 |
11+
| 状态 | ✅ 执行完成，待验收 |
12+
13+
---
14+
15+
## 一、问题回顾
16+
17+
### Issue 1: Gate 2 签字 — Architect self-review
18+
19+
**问题描述**：`prompts/002_develop_pipeline.md` Gate 2 章节规定"Architect（必签）"，但 Architect 也是 Tech Spec 的产出者，造成 self-review 违规。
20+
21+
**影响**：Tech Spec 的评审缺乏真正的独立制衡，降低了技术方案评审的质量保障。
22+
23+
### Issue 2: 职责归属矩阵缺失
24+
25+
**问题描述**：PRD / Tech Spec / QA Case 设计文档 / 代码 / 测试报告各交付物的产出角色和评审角色未明确，尤其是每个评审角色的**立场**未定义。
26+
27+
**影响**：Agent 在执行评审时缺乏明确立场，可能导致评审流于形式。
28+
29+
### Issue 3: Gate 定义不一致
30+
31+
**问题描述**：`prompts/004_delivery_gates.md`（Gate 编号 V2.3）和 `prompts/002_develop_pipeline.md`（Gate 0~5）各有一套 Gate 定义，存在不一致。
32+
33+
**影响**：Agent 和 Human 引用不同版本，导致流程执行歧义。
34+
35+
---
36+
37+
## 二、讨论对齐结论
38+
39+
### 结论 1: Gate 2 签字规范
40+
41+
**决定**：Architect 主持 Tech Spec 评审，**Engineer + QA 签字**。Architect 作为评审人参与，不签自己的 Tech Spec。
42+
43+
**理由**：产出者不评审自己的交付物，是评审的基本原则。Engineer 作为实现者，最有立场评估 Tech Spec 的可实现性；QA 作为测试路径设计者，评估 Tech Spec 的可测试性。
44+
45+
**执行动作**：更新 `prompts/002_develop_pipeline.md` Gate 2 签字要求，将 "Architect（必签）" 改为 "Engineer（必签）" + "QA（必签）"。
46+
47+
---
48+
49+
### 结论 2: 职责归属原则
50+
51+
**核心原则**：产出者不评审自己交付物，通过三角制衡确保评审质量。
52+
53+
**完整产出物清单**（含评审立场定义，已通过 `/plan-ceo-review` 讨论细化）：
54+
55+
| # | 产出物 | 产出角色 | 评审角色 | 评审立场 | 状态 |
56+
|---|--------|---------|---------|---------|------|
57+
| 1 | PRD | PM | Architect（必签）、PM（自审） | Architect：技术可行性 + 需求合理性；PM：需求完整性 + 范围控制 | ✅ 已讨论 |
58+
| 2 | Tech Spec | Architect | Engineer（必签）、QA（必签） | Engineer：可实现性；QA：可测试性；PRD 不含技术实现，Tech Spec 不含代码 | ✅ 已讨论 |
59+
| 3 | QA Case 设计文档 | QA | Architect（Architecture 维度）、QA（Test 维度，自审）、Engineer（可实现性维度）、QA+Engineer（Data 维度，联合） | Architecture：技术可测性；Test：测试覆盖完整性；可实现性：测试步骤是否可达；Data：测试数据设计合理性 | ✅ 已讨论 |
60+
| 4 | 代码 | Engineer | Architect（必签）、QA（必签） | Architect：架构一致性（是否符合 Tech Spec 设计）；QA：功能覆盖率（是否覆盖 QA Case 所有用例） | ✅ 已讨论 |
61+
| 5 | 测试报告 | QA | PM（必签）、Engineer（必签） | PM：验收口径（是否满足 PRD 验收标准，残留风险是否可接受）；Engineer：缺陷处理（所有缺陷是否已修复，修复方案是否合理） | ✅ 已讨论 |
62+
63+
**核心原则**：
64+
- 产出者不评审自己（三角制衡）
65+
- PRD 不含技术实现内容
66+
- Tech Spec 不含代码内容
67+
- QA Case 设计文档 ≠ case 实现代码
68+
- 测试报告 = 跑完所有 case + 驱动 Engineer 修复 bug 后的过程记录文档
69+
- 引入 gstack 多维度评审模式：评审分多个独立维度，每个维度有明确立场
70+
71+
**执行动作**：下一步将完整结论写入 `prompts/002_product_engineering_roles.md` 和 `prompts/002_develop_pipeline.md`。
72+
73+
---
74+
75+
### 结论 3: Gate 定义统一
76+
77+
**决定**：以 `prompts/002_develop_pipeline.md` 为唯一 Gate 定义源，`prompts/004_delivery_gates.md` 中的 Gate 编号定义（V2.3）删除或降级为 CI 检查逻辑参考附录。
78+
79+
**理由**：两份文档存在本质重复，以更完整的 pipeline doc 为准，消除二义性。
80+
81+
**执行动作**：
82+
1. 将 `004_delivery_gates.md` 中 CI 检查逻辑部分迁移到独立文件 `ci-gate-checks.md`
83+
2. 删除 `004_delivery_gates.md` 中的 Gate 定义章节
84+
3. 或直接删除 `004_delivery_gates.md`，将 CI 检查逻辑集成到 `021_platform_checks_and_gate_automation.md`
85+
86+
---
87+
88+
## 三、GitHub Issue 追踪
89+
90+
| 字段 | 内容 |
91+
|------|------|
92+
| GitHub Issue URL | 待创建 |
93+
| 责任人 | PMO |
94+
| 关联 PMO Issue | `../issues/009_2026-04-17_governance_role_attribution_and_gate2_signatory.md` |
95+
96+
---
97+
98+
## 四、待办清单
99+
100+
| # | 待办 | 负责人 | 状态 |
101+
|---|------|--------|------|
102+
| 1 | 更新 Gate 2 签字规范：Engineer + QA 必签，Architect 主持 | PMO | ✅ done |
103+
| 2 | 将 5 个产出物的评审角色+立场写入 `prompts/002_product_engineering_roles.md` | PMO | ✅ done |
104+
| 3 | 更新 `prompts/002_develop_pipeline.md` Gate 2 签字矩阵 | PMO | ✅ done |
105+
| 4 | 统一 Gate 定义：删除 `004_delivery_gates.md` 重复内容 | PMO | ✅ done |
106+
| 5 | GitHub Issue #16 关联 Issue #3（Gate 2 合规偏离） | PMO | pending |
107+
108+
---
109+
110+
## 五、验收结论
111+
112+
| 字段 | 内容 |
113+
|------|------|
114+
| 最终验收人 | Team Lead |
115+
| 验收日期 | 待填 |
116+
| GitHub Issue 状态 | 待创建 |
117+
| PMO Issue 状态 | Open |
118+
119+
---
120+
121+
*由 PMO (adf-pmo) 基于 `/plan-ceo-review` 讨论生成 | 2026-04-17*

‎docs/pmo/resolutions/010_2026-04-17_governance_skill_structure_consistency_resolution.md‎

Lines changed: 112 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,112 @@
1+
# GOV-010 Skill 结构冲突与不一致修复
2+
3+
## 基本信息
4+
5+
| 字段 | 内容 |
6+
|------|------|
7+
| PMO Issue | `docs/pmo/issues/010_2026-04-17_governance_skill_structure_consistency.md` |
8+
| GitHub Issue | https://github.com/alpha-86/AgentDevFlow/issues/17 |
9+
| 讨论日期 | 2026-04-17 |
10+
| 验收日期 | 待填 |
11+
| 状态 | ✅ 执行完成 |
12+
13+
---
14+
15+
## 一、P0 冲突修复
16+
17+
### P0-1: 产出者不评审自己的交付物
18+
19+
**决定**：在每个 SKILL 的"禁止行为"章节中显式声明"不得签署自己的交付物"。
20+
21+
**理由**：产出者不评审自己是三角制衡的核心原则，必须在每个角色的 SKILL 中显式声明，而非仅在 pipeline doc 中定义。
22+
23+
**执行动作**：
24+
- 在 `skills/architect/SKILL.md` 禁止行为章节补充
25+
- 在 `skills/qa-engineer/SKILL.md` 禁止行为章节补充
26+
- 在 `skills/product-manager/SKILL.md` 禁止行为章节补充
27+
- 在 `skills/engineer/SKILL.md` 禁止行为章节补充
28+
- 在 `skills/team-lead/SKILL.md` 禁止行为章节补充
29+
- 在 `skills/platform-sre/SKILL.md` 禁止行为章节补充
30+
31+
### P0-2: QA "三方签字"定义修正
32+
33+
**决定**：将 `skills/qa-engineer/SKILL.md` L23 中的"三方签字（PM + CTO + Engineer）"改为"三方签字（PM + Architect + Engineer）"。
34+
35+
**理由**："CTO"角色在 pipeline 中不存在，实际应为"Architect"。
36+
37+
---
38+
39+
## 二、P1 结构一致性修复
40+
41+
### P1-3: HR#1 命名统一
42+
43+
**决定**：统一为"Human Review #1"。
44+
45+
**理由**：消除命名歧义，与 `prompts/002_develop_pipeline.md` 保持一致。
46+
47+
**执行动作**：grep 全量替换所有 SKILL 文件中的"人工评审 #1"和"PR#1"为"Human Review #1"。
48+
49+
### P1-4: Architect 职责边界章节补全
50+
51+
**决定**：为 `skills/architect/SKILL.md` 补充完整的"职责边界"章节。
52+
53+
**理由**：Architect 是唯一缺少该章节的核心角色，导致与其他角色结构不一致。
54+
55+
**执行动作**：补充以下子章节：
56+
- "完成"定义
57+
- PR 合并权限
58+
- 每个 Gate 的 Architect 职责与限制表
59+
60+
### P1-5: Platform/SRE 章节补全
61+
62+
**决定**：为 `skills/platform-sre/SKILL.md` 补充"职责边界"和"输出格式"章节。
63+
64+
**理由**：Platform/SRE 是唯一同时缺少这两个章节的角色。
65+
66+
---
67+
68+
## 三、P2 原则声明
69+
70+
### P2-6: "产出者不评审自己"原则显式声明
71+
72+
**决定**：在每个 SKILL 的"禁止行为"章节补充"不得签署自己的交付物"。
73+
74+
（已在 P0-1 中覆盖，统一执行）
75+
76+
### P2-7: PMO 章节顺序修正
77+
78+
**决定**：将 `skills/pmo/SKILL.md` 中的"何时启用"移至"必读文档"之前。
79+
80+
**理由**：与其他 SKILL 的章节顺序保持一致。
81+
82+
---
83+
84+
## 四、执行清单
85+
86+
| # | 待办 | 文件 | 状态 |
87+
|---|------|------|------|
88+
| 1 | P0-1: Architect 禁止行为补充 | `skills/architect/SKILL.md` | ✅ done |
89+
| 2 | P0-1: 所有 SKILL 禁止行为补充 | `skills/*/SKILL.md` | ✅ done |
90+
| 3 | P0-2: QA 三方签字修正 | `skills/qa-engineer/SKILL.md` | ✅ done |
91+
| 4 | P1-3: HR#1 命名统一 | `skills/*/SKILL.md` | ✅ done |
92+
| 5 | P1-4: Architect 职责边界补全 | `skills/architect/SKILL.md` | ✅ done |
93+
| 6 | P1-5: Platform/SRE 章节补全 | `skills/platform-sre/SKILL.md` | ✅ done |
94+
| 7 | P2-7: PMO 章节顺序修正 | `skills/pmo/SKILL.md` | ✅ done |
95+
| 8 | GitHub Issue 创建 | — | ✅ done (#17) |
96+
| 9 | QA Case L7 重复修正 | `skills/qa-engineer/SKILL.md` | ✅ done |
97+
| 10 | Team Lead 产出者不评审自己补充 | `skills/team-lead/SKILL.md` | ✅ done |
98+
99+
---
100+
101+
## 五、验收结论
102+
103+
| 字段 | 内容 |
104+
|------|------|
105+
| 最终验收人 | Team Lead |
106+
| 验收日期 | 待填 |
107+
| GitHub Issue 状态 | 待创建 |
108+
| PMO Issue 状态 | Open |
109+
110+
---
111+
112+
*由 PMO (adf-pmo) 基于 `/plan-ceo-review` 讨论生成 | 2026-04-17*

0 commit comments

Comments
 (0)