anyone-can-architect 0.2.0 → 0.4.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/CONTEXT.md +2 -2
- package/README.md +6 -4
- package/README.zh-CN.md +4 -2
- package/SSOT.md +5 -0
- package/VERSION +1 -1
- package/axioms/README.md +1 -1
- package/axioms/writing.md +3 -1
- package/dist/antigravity/aca/SKILL.md +3 -2
- package/dist/antigravity/aca-design/SKILL.md +12 -6
- package/dist/antigravity/aca-implement/SKILL.md +102 -114
- package/dist/antigravity/aca-release/SKILL.md +13 -20
- package/dist/antigravity/aca-repo-audit/SKILL.md +13 -9
- package/dist/antigravity/aca-review/SKILL.md +146 -182
- package/dist/antigravity/aca-test/SKILL.md +112 -0
- package/dist/antigravity/aca-walkthrough/SKILL.md +18 -14
- package/dist/antigravity-cli/aca/SKILL.md +3 -2
- package/dist/antigravity-cli/aca-design/SKILL.md +12 -6
- package/dist/antigravity-cli/aca-implement/SKILL.md +102 -114
- package/dist/antigravity-cli/aca-release/SKILL.md +13 -20
- package/dist/antigravity-cli/aca-repo-audit/SKILL.md +13 -9
- package/dist/antigravity-cli/aca-review/SKILL.md +146 -182
- package/dist/antigravity-cli/aca-test/SKILL.md +112 -0
- package/dist/antigravity-cli/aca-walkthrough/SKILL.md +18 -14
- package/dist/codex/aca/SKILL.md +3 -2
- package/dist/codex/aca-design/SKILL.md +12 -6
- package/dist/codex/aca-implement/SKILL.md +102 -114
- package/dist/codex/aca-release/SKILL.md +13 -20
- package/dist/codex/aca-repo-audit/SKILL.md +13 -9
- package/dist/codex/aca-review/SKILL.md +146 -182
- package/dist/codex/aca-test/SKILL.md +112 -0
- package/dist/codex/aca-walkthrough/SKILL.md +18 -14
- package/dist/cursor/aca/SKILL.md +3 -2
- package/dist/cursor/aca-design/SKILL.md +12 -6
- package/dist/cursor/aca-implement/SKILL.md +102 -114
- package/dist/cursor/aca-release/SKILL.md +13 -20
- package/dist/cursor/aca-repo-audit/SKILL.md +13 -9
- package/dist/cursor/aca-review/SKILL.md +146 -182
- package/dist/cursor/aca-test/SKILL.md +113 -0
- package/dist/cursor/aca-walkthrough/SKILL.md +18 -14
- package/dist/grok/aca/SKILL.md +3 -2
- package/dist/grok/aca-design/SKILL.md +12 -6
- package/dist/grok/aca-implement/SKILL.md +102 -114
- package/dist/grok/aca-release/SKILL.md +13 -20
- package/dist/grok/aca-repo-audit/SKILL.md +13 -9
- package/dist/grok/aca-review/SKILL.md +146 -182
- package/dist/grok/aca-test/SKILL.md +112 -0
- package/dist/grok/aca-walkthrough/SKILL.md +18 -14
- package/doc-design/document-sets/module.md +6 -2
- package/doc-design/document-sets/ops-guide.md +25 -27
- package/doc-design/documents/class-design.md +7 -4
- package/doc-design/documents/instrumentation.md +3 -2
- package/doc-design/documents/integration-guide.md +7 -5
- package/doc-design/documents/interaction.md +516 -143
- package/doc-design/documents/interface.md +12 -7
- package/doc-design/documents/release.md +137 -105
- package/doc-design/patterns/interface.md +135 -14
- package/doc-design/repo-types/client-sdk.md +3 -3
- package/doc-design/repo-types/web-app.md +11 -5
- package/meta-skills/aca/SKILL.md +3 -2
- package/meta-skills/aca/SKILL.md.tmpl +3 -2
- package/package.json +1 -1
- package/skills/aca-design/SKILL.md +12 -6
- package/skills/aca-design/SKILL.md.tmpl +9 -5
- package/skills/aca-implement/SKILL.md +102 -114
- package/skills/aca-implement/SKILL.md.tmpl +103 -115
- package/skills/aca-release/SKILL.md +13 -20
- package/skills/aca-release/SKILL.md.tmpl +13 -20
- package/skills/aca-repo-audit/SKILL.md +13 -9
- package/skills/aca-repo-audit/SKILL.md.tmpl +10 -8
- package/skills/aca-review/SKILL.md +146 -182
- package/skills/aca-review/SKILL.md.tmpl +143 -181
- package/skills/aca-test/SKILL.md +112 -0
- package/skills/aca-test/SKILL.md.tmpl +110 -0
- package/skills/aca-walkthrough/SKILL.md +18 -14
- package/skills/aca-walkthrough/SKILL.md.tmpl +15 -13
- package/zh-CN/CONTEXT.md +2 -2
- package/zh-CN/SSOT.md +5 -0
- package/zh-CN/axioms/README.md +1 -1
- package/zh-CN/axioms/writing.md +3 -1
- package/zh-CN/dist/antigravity/aca/SKILL.md +3 -2
- package/zh-CN/dist/antigravity/aca-design/SKILL.md +12 -6
- package/zh-CN/dist/antigravity/aca-implement/SKILL.md +102 -114
- package/zh-CN/dist/antigravity/aca-release/SKILL.md +71 -69
- package/zh-CN/dist/antigravity/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/dist/antigravity/aca-review/SKILL.md +143 -179
- package/zh-CN/dist/antigravity/aca-test/SKILL.md +112 -0
- package/zh-CN/dist/antigravity/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/dist/antigravity-cli/aca/SKILL.md +3 -2
- package/zh-CN/dist/antigravity-cli/aca-design/SKILL.md +12 -6
- package/zh-CN/dist/antigravity-cli/aca-implement/SKILL.md +102 -114
- package/zh-CN/dist/antigravity-cli/aca-release/SKILL.md +71 -69
- package/zh-CN/dist/antigravity-cli/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/dist/antigravity-cli/aca-review/SKILL.md +143 -179
- package/zh-CN/dist/antigravity-cli/aca-test/SKILL.md +112 -0
- package/zh-CN/dist/antigravity-cli/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/dist/codex/aca/SKILL.md +3 -2
- package/zh-CN/dist/codex/aca-design/SKILL.md +12 -6
- package/zh-CN/dist/codex/aca-implement/SKILL.md +102 -114
- package/zh-CN/dist/codex/aca-release/SKILL.md +71 -69
- package/zh-CN/dist/codex/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/dist/codex/aca-review/SKILL.md +143 -179
- package/zh-CN/dist/codex/aca-test/SKILL.md +112 -0
- package/zh-CN/dist/codex/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/dist/cursor/aca/SKILL.md +3 -2
- package/zh-CN/dist/cursor/aca-design/SKILL.md +12 -6
- package/zh-CN/dist/cursor/aca-implement/SKILL.md +102 -114
- package/zh-CN/dist/cursor/aca-release/SKILL.md +71 -69
- package/zh-CN/dist/cursor/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/dist/cursor/aca-review/SKILL.md +143 -179
- package/zh-CN/dist/cursor/aca-test/SKILL.md +113 -0
- package/zh-CN/dist/cursor/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/dist/grok/aca/SKILL.md +3 -2
- package/zh-CN/dist/grok/aca-design/SKILL.md +12 -6
- package/zh-CN/dist/grok/aca-implement/SKILL.md +102 -114
- package/zh-CN/dist/grok/aca-release/SKILL.md +71 -69
- package/zh-CN/dist/grok/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/dist/grok/aca-review/SKILL.md +143 -179
- package/zh-CN/dist/grok/aca-test/SKILL.md +112 -0
- package/zh-CN/dist/grok/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/doc-design/document-sets/module.md +6 -2
- package/zh-CN/doc-design/document-sets/ops-guide.md +25 -27
- package/zh-CN/doc-design/documents/class-design.md +7 -4
- package/zh-CN/doc-design/documents/instrumentation.md +3 -2
- package/zh-CN/doc-design/documents/integration-guide.md +7 -5
- package/zh-CN/doc-design/documents/interaction.md +516 -143
- package/zh-CN/doc-design/documents/interface.md +12 -7
- package/zh-CN/doc-design/documents/release.md +138 -106
- package/zh-CN/doc-design/patterns/interface.md +135 -14
- package/zh-CN/doc-design/repo-types/client-sdk.md +3 -3
- package/zh-CN/doc-design/repo-types/web-app.md +11 -5
- package/zh-CN/meta-skills/aca/SKILL.md +3 -2
- package/zh-CN/meta-skills/aca/SKILL.md.tmpl +3 -2
- package/zh-CN/skills/aca-design/SKILL.md +12 -6
- package/zh-CN/skills/aca-design/SKILL.md.tmpl +9 -5
- package/zh-CN/skills/aca-implement/SKILL.md +102 -114
- package/zh-CN/skills/aca-implement/SKILL.md.tmpl +102 -114
- package/zh-CN/skills/aca-release/SKILL.md +71 -69
- package/zh-CN/skills/aca-release/SKILL.md.tmpl +71 -69
- package/zh-CN/skills/aca-repo-audit/SKILL.md +39 -29
- package/zh-CN/skills/aca-repo-audit/SKILL.md.tmpl +36 -28
- package/zh-CN/skills/aca-review/SKILL.md +143 -179
- package/zh-CN/skills/aca-review/SKILL.md.tmpl +140 -178
- package/zh-CN/skills/aca-test/SKILL.md +112 -0
- package/zh-CN/skills/aca-test/SKILL.md.tmpl +110 -0
- package/zh-CN/skills/aca-walkthrough/SKILL.md +18 -14
- package/zh-CN/skills/aca-walkthrough/SKILL.md.tmpl +15 -13
|
@@ -41,18 +41,20 @@ flowchart TD
|
|
|
41
41
|
|
|
42
42
|
## 3 审计准备
|
|
43
43
|
|
|
44
|
-
1.
|
|
44
|
+
1. **确定本次审计范围**
|
|
45
45
|
优先采用开发者指定的范围,其次沿用调用方明确的范围
|
|
46
46
|
未指定时,默认审计整个仓库
|
|
47
47
|
记录本次检查对应的提交及纳入范围的未提交改动
|
|
48
|
-
2.
|
|
48
|
+
2. **检查工程类型**
|
|
49
|
+
运行 `bash ~/.claude/skills/aca/bin/aca-repo-type-check` 检查工程类型
|
|
49
50
|
类型缺失、无效或尚未支持时,将类型规则核对列为未完成
|
|
50
51
|
类型缺失或无效时,建议通过 `/aca-init` 修正
|
|
51
52
|
继续其余不依赖类型规则的检查
|
|
52
|
-
3.
|
|
53
|
+
3. **确定是否包含实现审计**
|
|
53
54
|
开发者要求或 `/aca-release` 调用时,包含实现审计
|
|
54
55
|
其他情况执行文档审计
|
|
55
|
-
4.
|
|
56
|
+
4. **读取规则并定位入口**
|
|
57
|
+
读取有效类型的规则与 `CONTEXT.md`,核对项目术语
|
|
56
58
|
从根 README 定位本次审计范围的文档入口
|
|
57
59
|
入口无法定位时,请开发者提供位置
|
|
58
60
|
位置明确前,将依赖这些内容的检查列为未完成
|
|
@@ -61,21 +63,24 @@ flowchart TD
|
|
|
61
63
|
|
|
62
64
|
### 4.1 读取文档
|
|
63
65
|
|
|
64
|
-
1.
|
|
66
|
+
1. **建立本次范围的文档清单**
|
|
65
67
|
根据类型规则的必要条件与已选用的文档组,列出应有的文档
|
|
66
68
|
对照 README 入口与实际目录,记录缺失文档及未登记的文档
|
|
67
|
-
2.
|
|
69
|
+
2. **读取清单文档与关联引用**
|
|
70
|
+
读取清单中的文档,沿引用补齐理解与核验所需的内容
|
|
68
71
|
范围外的引用作为关联依据,不自动扩大审计范围
|
|
69
|
-
3.
|
|
72
|
+
3. **提取适用检查条件**
|
|
73
|
+
读取对应的文档组、Doc Design 与实际采用的 Pattern
|
|
70
74
|
提取适用检查条件,作为后续逐项核对的依据
|
|
71
75
|
|
|
72
76
|
### 4.2 机械检查
|
|
73
77
|
|
|
74
|
-
1.
|
|
78
|
+
1. **确定实际检查路径**
|
|
79
|
+
根据[文档清单](#41-读取文档)确定实际检查路径
|
|
75
80
|
文档图检查覆盖本次审计的文档目录
|
|
76
81
|
行文检查覆盖本次审计的全部文档,包括涉及的项目级文件
|
|
77
82
|
审计范围包含 Context 时,检查其结构与格式
|
|
78
|
-
2.
|
|
83
|
+
2. **执行或复用机械检查**
|
|
79
84
|
本会话已有相同范围的结果,且相关文件、检查规则与工具均未变化时,可复用
|
|
80
85
|
在报告中注明原检查结果与复用依据
|
|
81
86
|
不满足复用条件时,将以下占位符替换为实际路径并执行对应检查
|
|
@@ -86,7 +91,7 @@ flowchart TD
|
|
|
86
91
|
bash ~/.claude/skills/aca/bin/aca-context-check <本次审计的 Context 文档>
|
|
87
92
|
```
|
|
88
93
|
|
|
89
|
-
3.
|
|
94
|
+
3. **根据实际诊断处理检查结果**
|
|
90
95
|
检查通过时,记录结果与依据
|
|
91
96
|
检出问题时,核实诊断,将成立的问题列为必须处理
|
|
92
97
|
工具缺失或运行失败时,将对应检查列为未完成并说明原因
|
|
@@ -95,17 +100,18 @@ flowchart TD
|
|
|
95
100
|
|
|
96
101
|
### 4.3 内容检查
|
|
97
102
|
|
|
98
|
-
1.
|
|
103
|
+
1. **逐项核对适用检查条件**
|
|
104
|
+
逐项核对[读取文档](#41-读取文档)中提取的适用检查条件
|
|
99
105
|
结合[行文公理](#12-行文公理)核对表达
|
|
100
106
|
每项记录核验结果、对应位置与依据
|
|
101
107
|
不适用的条件说明原因
|
|
102
|
-
2.
|
|
108
|
+
2. **顺着文档主线核对跨文档关系**
|
|
103
109
|
- 入口和阅读顺序是否明确
|
|
104
110
|
- 同一功能、术语、版本或流程是否互相矛盾
|
|
105
111
|
- 同一信息是否由多篇文档重复维护
|
|
106
112
|
- 是否存在过时内容、残稿、空文档或无法指导行动的空话
|
|
107
113
|
- 设计正文是否只描述当前目标设计,未混入旧版本差异、改动过程或改动原因
|
|
108
|
-
3.
|
|
114
|
+
3. **记录问题与未完成项**
|
|
109
115
|
问题记录位置、依据、影响和处理建议
|
|
110
116
|
跨文档重复时,指出信息应保留的位置
|
|
111
117
|
其他位置建议删除重复内容或改为引用
|
|
@@ -119,15 +125,15 @@ flowchart TD
|
|
|
119
125
|
|
|
120
126
|
本次包含实现审计时,双向核对本次范围内的全部设计项,不抽样
|
|
121
127
|
|
|
122
|
-
1.
|
|
128
|
+
1. **列出应实现的设计项**
|
|
123
129
|
以本次已确认的交付范围为依据,列出模块、行为、接口、状态、错误处理与验收条件
|
|
124
|
-
2.
|
|
130
|
+
2. **从设计核对实现**
|
|
125
131
|
为每个设计项定位对应代码、测试和配置
|
|
126
132
|
核对是否完整实现、是否符合设计约定
|
|
127
|
-
3.
|
|
133
|
+
3. **从实现核对设计**
|
|
128
134
|
检查实际业务行为能否对应到设计说明
|
|
129
135
|
辅助函数、工具代码与工程样板,核对其对设计行为的影响
|
|
130
|
-
4.
|
|
136
|
+
4. **记录对应关系与偏移**
|
|
131
137
|
每个设计项记录核对结果与依据
|
|
132
138
|
每个偏移给出设计文档与实现位置
|
|
133
139
|
缺少一侧时,说明缺失内容
|
|
@@ -135,16 +141,17 @@ flowchart TD
|
|
|
135
141
|
|
|
136
142
|
### 5.2 运行验证
|
|
137
143
|
|
|
138
|
-
1.
|
|
139
|
-
|
|
144
|
+
1. **确定验证要求**
|
|
145
|
+
按 `/aca-test` 的边界规则与仓库 `release` 文档,确定本次范围所需的测试与构建命令
|
|
146
|
+
仓库未规定分级时,默认执行完整测试与构建
|
|
140
147
|
核对所需环境、配置与夹具
|
|
141
|
-
2.
|
|
142
|
-
|
|
148
|
+
2. **执行或复用验证结果**
|
|
149
|
+
已有 `/aca-test` 或构建结果覆盖所需范围,且受审提交、测试、夹具、配置、依赖环境及命令均未变化时,可复用
|
|
143
150
|
在报告中注明原验证结果与复用依据
|
|
144
|
-
|
|
151
|
+
不满足复用条件时,在隔离环境中按 `/aca-test` 的执行规则运行所需测试与构建
|
|
145
152
|
静态阅读不能代替执行验证
|
|
146
153
|
验证过程中不修改项目源码、设计文档或提交记录
|
|
147
|
-
3.
|
|
154
|
+
3. **记录结果并处理异常**
|
|
148
155
|
验证通过时,记录结果与依据
|
|
149
156
|
验证失败时,核实诊断并记录成立的问题
|
|
150
157
|
环境缺失或无法执行时,将对应验证列为未完成并说明原因
|
|
@@ -155,15 +162,15 @@ flowchart TD
|
|
|
155
162
|
|
|
156
163
|
### 6.1 复核结果
|
|
157
164
|
|
|
158
|
-
1.
|
|
165
|
+
1. **核对证据对应关系**
|
|
159
166
|
确认检查结果与本次受审内容一致
|
|
160
167
|
核对复用结果是否满足对应条件
|
|
161
|
-
2.
|
|
168
|
+
2. **复核问题清单**
|
|
162
169
|
核对问题涉及的文件与证据,确认成立后纳入报告
|
|
163
170
|
已证伪的问题排除
|
|
164
171
|
证据不足的问题列为未完成核验
|
|
165
172
|
合并重复问题,保留相关位置、依据与影响
|
|
166
|
-
3.
|
|
173
|
+
3. **核对检查覆盖**
|
|
167
174
|
对照文档清单、适用检查条件及本次实现审计要求,核对是否遗漏
|
|
168
175
|
能补查的项目完成补查
|
|
169
176
|
仍无法完成的项目,记录范围、原因与恢复条件
|
|
@@ -177,7 +184,7 @@ flowchart TD
|
|
|
177
184
|
| 未完成 | 未满足通过条件,且无已确认的必须处理项 |
|
|
178
185
|
|
|
179
186
|
必需检查包括本次范围的文档读取、机械检查与内容检查
|
|
180
|
-
|
|
187
|
+
包含实现审计时,还包括全部一致性核对、本次范围所需的测试与构建
|
|
181
188
|
同时存在必须处理项与未完成检查时,结论为需要处理,并列出未完成项
|
|
182
189
|
可选精简不影响通过
|
|
183
190
|
|
|
@@ -227,7 +234,8 @@ flowchart TD
|
|
|
227
234
|
| 缺少核验条件 | 补齐后通过 `/aca-repo-audit` 继续本次审计 |
|
|
228
235
|
| 方案仍需论证 | `/aca-specify` 论证方案 |
|
|
229
236
|
| 修订方案已明确,仅需改文档 | `/aca-design` 修订文档 |
|
|
230
|
-
| 实现偏移 | `/aca-implement`
|
|
237
|
+
| 实现偏移 | `/aca-implement` 修复代码或测试 |
|
|
238
|
+
| 测试或构建未完成 | `/aca-test` 执行验证 |
|
|
231
239
|
| 审计通过 | 返回调用方,按已确认的安排继续 |
|
|
232
240
|
|
|
233
241
|
问题修正后,重新审计受影响范围
|
|
@@ -1,14 +1,14 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: aca-review
|
|
3
|
-
description:
|
|
3
|
+
description: 审查当前范围的设计文档或代码结构,判定通过、需要调整或未完成,不修改受审文档与代码
|
|
4
4
|
---
|
|
5
5
|
<!-- AUTO-GENERATED from SKILL.md.tmpl — do not edit directly -->
|
|
6
6
|
<!-- Regenerate: bash scripts/build-skills.sh --host claude -->
|
|
7
7
|
|
|
8
8
|
# aca-review
|
|
9
9
|
|
|
10
|
-
|
|
11
|
-
|
|
10
|
+
审查当前范围的设计文档或代码结构,判定通过、需要调整或未完成
|
|
11
|
+
不修改设计文档、项目源码与测试代码,不运行受审代码与测试
|
|
12
12
|
发布前的仓库整体审计由 `/aca-repo-audit` 执行
|
|
13
13
|
|
|
14
14
|
## 1 原则
|
|
@@ -17,11 +17,11 @@ description: 审查当前范围的设计文档或代码实现,判定通过、
|
|
|
17
17
|
|
|
18
18
|
1. **依据驱动**
|
|
19
19
|
以已确认的需求与适用规则审查设计
|
|
20
|
-
|
|
20
|
+
以已确认的设计审查代码的模块化与整体结构
|
|
21
21
|
2. **独立判断**
|
|
22
|
-
|
|
22
|
+
按审查维度独立形成结论
|
|
23
23
|
3. **证据判定**
|
|
24
|
-
|
|
24
|
+
以代码结构与文档事实判定问题及审查结论
|
|
25
25
|
4. **范围约束**
|
|
26
26
|
审查结论仅覆盖本次范围
|
|
27
27
|
|
|
@@ -36,7 +36,7 @@ description: 审查当前范围的设计文档或代码实现,判定通过、
|
|
|
36
36
|
- 先交代整体,再展开细节,参见[金字塔原理](~/.claude/skills/aca/SSOT.md#2-文档结构)
|
|
37
37
|
- 表达判断时,先给结论,再给依据
|
|
38
38
|
- 不增加信息的标题层级应当合并
|
|
39
|
-
-
|
|
39
|
+
- 有时序或优先级的条目用编号列表,包含子项时每步以加粗短语开头表达目的,具体操作与条件分支缩进在该步下
|
|
40
40
|
- 表格只用于多行同构、每行有两个以上属性、行之间没有先后关系的内容
|
|
41
41
|
- 列表项直接写完整的句子
|
|
42
42
|
- 一个条目有分支时,将分支缩进列在该条目下
|
|
@@ -55,6 +55,8 @@ description: 审查当前范围的设计文档或代码实现,判定通过、
|
|
|
55
55
|
- 删除不增加信息、也不帮助理解的内容
|
|
56
56
|
- 一行一句,每句只表达一个事实
|
|
57
57
|
- 操作步骤直接写动作
|
|
58
|
+
- 能用具体做法完整表达的约束,直接写做法
|
|
59
|
+
- 无法用具体做法完整表达的禁止事项仍须保留
|
|
58
60
|
- 有先后关系的动作分项写
|
|
59
61
|
- 执行者、操作对象与指代必须明确
|
|
60
62
|
- 上下文已经明确的执行者、操作对象与指代不重复说明
|
|
@@ -81,12 +83,12 @@ description: 审查当前范围的设计文档或代码实现,判定通过、
|
|
|
81
83
|
```mermaid
|
|
82
84
|
flowchart TD
|
|
83
85
|
Start([接收审查请求]) --> Scope["确认审查范围与执行模型 · §3"]
|
|
84
|
-
Scope --> Check["
|
|
85
|
-
Check --> Audit["
|
|
86
|
-
Audit --> Verify["
|
|
87
|
-
Verify --> Issues{"存在已确认的必须调整项 · §
|
|
88
|
-
Issues -- 是,需要调整 --> Report(["输出结果与下一步 · §
|
|
89
|
-
Issues -- 否 --> Complete{"通过条件全部满足 · §
|
|
86
|
+
Scope --> Check["判断审查类型并准备依据 · §4"]
|
|
87
|
+
Check --> Audit["按审查类型执行独立审查 · §5"]
|
|
88
|
+
Audit --> Verify["复核与判定 · §6"]
|
|
89
|
+
Verify --> Issues{"存在已确认的必须调整项 · §6.2"}
|
|
90
|
+
Issues -- 是,需要调整 --> Report(["输出结果与下一步 · §7"])
|
|
91
|
+
Issues -- 否 --> Complete{"通过条件全部满足 · §6.2"}
|
|
90
92
|
Complete -- 是,通过 --> Report
|
|
91
93
|
Complete -- 否,未完成 --> Report
|
|
92
94
|
```
|
|
@@ -95,81 +97,72 @@ flowchart TD
|
|
|
95
97
|
|
|
96
98
|
### 3.1 对象、范围与前提
|
|
97
99
|
|
|
98
|
-
1.
|
|
99
|
-
|
|
100
|
-
|
|
101
|
-
|
|
102
|
-
|
|
103
|
-
|
|
104
|
-
|
|
105
|
-
|
|
106
|
-
|
|
107
|
-
|
|
108
|
-
|
|
109
|
-
|
|
110
|
-
|
|
111
|
-
|
|
112
|
-
|
|
113
|
-
|
|
114
|
-
|
|
115
|
-
|
|
116
|
-
读取本次范围的设计文档,并沿引用补齐理解与核验所需的文档
|
|
100
|
+
1. **确认审查对象与范围**
|
|
101
|
+
- 开发者已指定审查对象与范围:优先按其指定执行
|
|
102
|
+
- 开发者未指定:沿用调用方已明确的审查对象与范围
|
|
103
|
+
- 对象或范围仍未明确:结合当前工作区改动与近期相关提交确定,读取近期相关提交的说明与 diff
|
|
104
|
+
- 仍无法确定审查对象或范围:请开发者明确
|
|
105
|
+
2. **记录比较基线**
|
|
106
|
+
- 记录审查对象、范围与比较基线
|
|
107
|
+
- 注明本次受审提交及纳入审查的未提交改动
|
|
108
|
+
3. **检查工程类型**
|
|
109
|
+
- 运行 `bash ~/.claude/skills/aca/bin/aca-repo-type-check` 检查工程类型
|
|
110
|
+
- 工程类型缺失或无效:将类型规则核对列为未完成,并建议运行 `/aca-init`
|
|
111
|
+
- 工程类型尚未支持:将类型规则核对列为未完成,并注明缺少对应规则
|
|
112
|
+
4. **定位并读取通用审查依据**
|
|
113
|
+
- 输入未包含设计文档位置时,从根 README 的文档入口定位;无法定位时请开发者提供位置,位置明确前将依赖这些内容的检查列为未完成
|
|
114
|
+
- 读取本次已确认的问题、目标与改动范围
|
|
115
|
+
- 工程类型有效且已支持时,读取对应的 Repo Type 规则
|
|
116
|
+
- 读取 `CONTEXT.md`,核对本次审查涉及的业务概念与定义
|
|
117
|
+
- 读取本次范围的设计文档,并沿引用补齐理解与核验所需的文档
|
|
117
118
|
|
|
118
119
|
### 3.2 Agent 与模型选择
|
|
119
120
|
|
|
120
|
-
1.
|
|
121
|
-
|
|
122
|
-
|
|
123
|
-
|
|
124
|
-
|
|
125
|
-
|
|
126
|
-
|
|
127
|
-
|
|
128
|
-
|
|
129
|
-
|
|
130
|
-
|
|
131
|
-
说明推荐依据与可选的备选组合,请开发者确认
|
|
132
|
-
开发者确认后,再执行审查
|
|
133
|
-
已确认的选择与可用条件未变化时,沿用已有确认
|
|
134
|
-
4. 将确认的执行组合用于[任务编排](#53-模型与任务编排)
|
|
135
|
-
所选组合无法调用时,使用已确认的备选
|
|
136
|
-
无已确认的备选时,报告变化并请开发者确认新的组合
|
|
137
|
-
在报告中注明实际选择、回落及原因
|
|
121
|
+
1. **优先复用已确认模型**
|
|
122
|
+
- 开发者已指定 Agent 环境或模型:按指定执行
|
|
123
|
+
- 已有确认过的审查模型,且可用条件未变化:默认直接复用上次确认的 Agent 环境与模型,跳过环境扫描与确认
|
|
124
|
+
2. **首次审查扫描并确认模型**
|
|
125
|
+
- 首次执行审查且未指定模型时,扫描当前及本机已有的其他 AI Agent 环境与可调用模型
|
|
126
|
+
- 优先推荐不同于主 agent 的可调用旗舰模型;没有其他适合且可调用的候选时,提出使用本模型的方案
|
|
127
|
+
- 展示候选的 Agent 环境、模型、可用状态与推荐依据,请开发者确认后再执行审查,后续审查默认复用该确认结果
|
|
128
|
+
3. **处理执行回落**
|
|
129
|
+
- 将确认或复用的执行组合用于[任务编排与执行](#53-任务编排与执行)
|
|
130
|
+
- 所选组合无法调用时,使用已确认的备选;无已确认备选时,报告变化并请开发者确认新的组合
|
|
131
|
+
- 在[审查报告](#71-审查报告)中注明实际执行的 Agent 环境、模型、复用情况及回落原因
|
|
138
132
|
|
|
139
133
|
## 4 读取依据与执行检查
|
|
140
134
|
|
|
141
135
|
```mermaid
|
|
142
136
|
flowchart TD
|
|
143
|
-
Kind["判断审查类型 · §4.1"] -- 设计审查 --> Design["
|
|
144
|
-
Kind -- 实现审查 --> Implementation["
|
|
145
|
-
Design -->
|
|
146
|
-
Implementation -->
|
|
137
|
+
Kind["判断审查类型 · §4.1"] -- 设计审查 --> Design["设计审查准备与机械检查 · §4.2"]
|
|
138
|
+
Kind -- 实现审查 --> Implementation["实现审查依据准备 · §4.3"]
|
|
139
|
+
Design --> Next(["进入独立审查 · §5"])
|
|
140
|
+
Implementation --> Next
|
|
147
141
|
```
|
|
148
142
|
|
|
149
143
|
### 4.1 判断审查类型
|
|
150
144
|
|
|
151
|
-
1.
|
|
152
|
-
|
|
153
|
-
|
|
154
|
-
-
|
|
155
|
-
-
|
|
156
|
-
- 同时涉及设计与实现:分别执行两类审查
|
|
157
|
-
3. 仍无法确定审查类型时,请开发者明确
|
|
145
|
+
1. **确定本次审查类型**
|
|
146
|
+
- 开发者已指定审查类型:优先按其指定执行
|
|
147
|
+
- 开发者未指定:沿用调用方已明确的审查类型
|
|
148
|
+
- 审查类型未明确:根据受审对象与实际改动判断(设计文档执行设计审查,代码执行实现审查,同时涉及两者时分别执行两类审查)
|
|
149
|
+
- 仍无法确定审查类型:请开发者明确
|
|
158
150
|
|
|
159
|
-
|
|
151
|
+
选择两类审查时,两条分支均执行
|
|
160
152
|
|
|
161
153
|
### 4.2 设计审查
|
|
162
154
|
|
|
163
|
-
|
|
164
|
-
|
|
165
|
-
|
|
166
|
-
|
|
167
|
-
|
|
168
|
-
|
|
169
|
-
|
|
170
|
-
|
|
171
|
-
|
|
172
|
-
|
|
155
|
+
设计审查默认用于跨模块范围的设计变更(涉及多个模块、项目级架构、模块划分或跨模块接口与协作契约),或开发者显式指定的设计审查
|
|
156
|
+
|
|
157
|
+
1. **提取适用检查条件**
|
|
158
|
+
- 读取本次范围对应的文档组、Doc Design 与实际采用的 Pattern
|
|
159
|
+
- 提取其中适用的检查条件,作为设计审查的核验依据
|
|
160
|
+
2. **复用或执行机械检查**
|
|
161
|
+
- 本会话已有相同范围的机械检查结果,且相关文件、检查规则与工具均未变化时,直接复用并在报告中注明原结果及复用依据
|
|
162
|
+
- 不满足复用条件时,将以下命令中的占位路径替换为本次受审文档的实际路径并执行
|
|
163
|
+
- 行文检查覆盖本次受审的全部文档,包括纳入审查范围的项目级记录
|
|
164
|
+
- 文档图检查选择包含本次受审文档、且具有 README 入口的文档树
|
|
165
|
+
- Context 的机械检查用于核对结构与格式
|
|
173
166
|
|
|
174
167
|
```bash
|
|
175
168
|
bash ~/.claude/skills/aca/bin/aca-doc-graph-check <文档目录>
|
|
@@ -177,111 +170,94 @@ flowchart TD
|
|
|
177
170
|
bash ~/.claude/skills/aca/bin/aca-prose-check <本次受审的全部文档路径>
|
|
178
171
|
```
|
|
179
172
|
|
|
180
|
-
|
|
181
|
-
|
|
182
|
-
1. 读取本次范围对应的已确认设计、测试设计与仓库实现约定
|
|
183
|
-
2. 按已确定的比较基线读取本次 diff 与改动文件
|
|
184
|
-
读取受影响的调用方与依赖代码
|
|
185
|
-
读取改动所在函数、类及相关调用链,理解其执行上下文
|
|
186
|
-
3. 将实现与测试分别对应到设计约定和测试场景
|
|
187
|
-
4. 根据本次改动的影响范围,执行或复用设计与仓库约定要求的测试、构建及静态检查
|
|
188
|
-
设计要求通过实际运行核验的行为,按对应测试用例执行验证
|
|
189
|
-
已有验证结果对应相同受审内容,且测试用例、夹具、配置与依赖环境均未变化时,可复用
|
|
190
|
-
复用时,在报告中注明原验证结果及本次满足的复用条件
|
|
191
|
-
验证证据不足或不满足复用条件时,执行对应检查
|
|
192
|
-
按测试设计使用隔离的验证环境
|
|
193
|
-
验证过程中不修改提交记录
|
|
194
|
-
|
|
195
|
-
### 4.4 处理检查结果
|
|
196
|
-
|
|
197
|
-
两类审查的工具检查均按本节处理
|
|
173
|
+
3. **处理机械检查结果**
|
|
198
174
|
|
|
199
175
|
| 检查情况 | 处理 |
|
|
200
176
|
| --- | --- |
|
|
201
|
-
|
|
|
177
|
+
| 通过或满足复用条件 | 记录结果与验证或复用依据 |
|
|
202
178
|
| 返回失败 | 核对诊断,将已确认的问题列为必须调整 |
|
|
203
179
|
| 无法运行或诊断无法核实 | 对应检查或问题核验列为未完成,注明原因 |
|
|
204
180
|
|
|
181
|
+
### 4.3 实现审查
|
|
182
|
+
|
|
183
|
+
1. **读取结构设计与组织约定**
|
|
184
|
+
- 读取本次范围对应的架构设计、模块边界、接口定义、内部组成与仓库代码组织约定
|
|
185
|
+
2. **读取受审代码的整体结构**
|
|
186
|
+
- 按已确定的比较基线读取本次改动涉及的目录与文件分布、模块依赖、函数与类定义及跨模块组装接线
|
|
187
|
+
3. **明确审查边界**
|
|
188
|
+
- 实现审查默认用于跨模块范围或改变了模块整体代码结构的变更(仅模块内部且未改变整体代码结构的改动不用执行实现审查,直接进入 `/aca-test`)
|
|
189
|
+
- 实现审查只检查整体结构(模块结构、整体代码分布、函数构造),不检查具体的代码细节与实现逻辑
|
|
190
|
+
- 实现审查不直接运行代码、测试或构建,运行验证在审查通过后由 `/aca-test` 执行
|
|
191
|
+
|
|
205
192
|
## 5 审查编排与执行
|
|
206
193
|
|
|
207
194
|
### 5.1 设计审查维度
|
|
208
195
|
|
|
209
|
-
|
|
196
|
+
设计审查作为一个整体审查任务,围绕以下四项核对受审设计文档
|
|
197
|
+
|
|
198
|
+
1. **需求与范围承接**
|
|
210
199
|
核对设计是否完整承接本次已确认的问题、目标与范围
|
|
211
|
-
2.
|
|
212
|
-
|
|
213
|
-
|
|
214
|
-
|
|
215
|
-
|
|
216
|
-
3.
|
|
217
|
-
核对模块依赖与接口约定是否足以指导模块协作
|
|
218
|
-
核对数据、状态变化、错误处理与边界行为是否足以指导具体实现
|
|
200
|
+
2. **职责边界与协作机制**
|
|
201
|
+
核对职责与边界划分、模块依赖与接口约定是否符合业务分工并足以指导模块协作
|
|
202
|
+
核对协作与处理机制、数据与状态变化、错误处理与边界行为是否足以指导具体实现
|
|
203
|
+
涉及用户交互时,核对交互流程、各交互状态及成功与失败路径是否提供明确的反馈与操作指引
|
|
204
|
+
核对设计是否存在会导致业务行为不同的歧义,以及实施前是否仍需补充设计决定
|
|
205
|
+
3. **验证设计完备性**
|
|
219
206
|
核对测试设计是否明确夹具、场景、执行方式与预期结果
|
|
220
207
|
核对测试结果能否判定对应验收条件是否满足
|
|
221
|
-
|
|
222
|
-
核对实施前是否仍需补充设计决定
|
|
223
|
-
4. **整体视角**
|
|
208
|
+
4. **关联一致性与表达规范**
|
|
224
209
|
核对文档入口与引用能否准确定位本次设计及其依赖
|
|
225
210
|
核对相关文档之间是否存在设计遗漏、约定冲突、重复定义或术语不一致
|
|
211
|
+
核对适用文档组、Doc Design 与 Pattern 的检查条件是否满足
|
|
226
212
|
|
|
227
213
|
### 5.2 实现审查维度
|
|
228
214
|
|
|
229
|
-
|
|
230
|
-
|
|
231
|
-
1.
|
|
232
|
-
|
|
233
|
-
2.
|
|
234
|
-
|
|
235
|
-
|
|
236
|
-
|
|
237
|
-
|
|
238
|
-
3
|
|
239
|
-
|
|
240
|
-
|
|
241
|
-
|
|
242
|
-
|
|
243
|
-
|
|
244
|
-
|
|
245
|
-
|
|
246
|
-
实现审查为结构设定、具体逻辑实现与整体结构实现分别分配独立子 agent
|
|
247
|
-
各子 agent 独立形成结论,不预先读取其他子 agent 的审查结果
|
|
248
|
-
2. 按[Agent 与模型选择](#32-agent-与模型选择)中已确认的组合安排执行
|
|
249
|
-
使用本模型时,各审查任务仍独立执行
|
|
250
|
-
3. 为各子 agent 提供任务上下文
|
|
251
|
-
- 负责的审查项及分配的适用检查条件
|
|
215
|
+
实现审查作为一个整体审查任务,以已确认的设计文档为依据,只检查代码的模块化与整体结构,不检查具体的代码细节与实现逻辑
|
|
216
|
+
|
|
217
|
+
1. **模块结构**
|
|
218
|
+
核对模块与子模块划分、职责归属、接口边界、依赖关系及跨模块组装接线是否符合设计
|
|
219
|
+
2. **整体代码分布**
|
|
220
|
+
核对目录与文件组织是否符合设计,各文件职责是否单一、归属是否清晰,是否存在职责混杂或跨层耦合
|
|
221
|
+
3. **函数构造**
|
|
222
|
+
核对函数与类的职责切分、签名设计、粒度及抽象层次是否清晰合理
|
|
223
|
+
|
|
224
|
+
### 5.3 任务编排与执行
|
|
225
|
+
|
|
226
|
+
1. **按审查类型派发独立审查任务**
|
|
227
|
+
- 仅执行设计审查或仅执行实现审查时,创建 1 个独立子 agent 任务完成该维度的全部核对项
|
|
228
|
+
- 同时执行设计审查与实现审查时,为设计审查与实现审查分别创建 1 个独立子 agent 任务
|
|
229
|
+
- 各审查任务按[Agent 与模型选择](#32-agent-与模型选择)安排执行,独立形成结论
|
|
230
|
+
2. **提供任务上下文**
|
|
231
|
+
- 负责的审查类型、对应核对项与适用检查条件
|
|
252
232
|
- 审查对象、范围与比较基线,以及受审提交和纳入审查的未提交改动
|
|
253
233
|
- 受审内容的位置及对应设计依据
|
|
254
|
-
-
|
|
255
|
-
-
|
|
256
|
-
|
|
257
|
-
|
|
258
|
-
|
|
259
|
-
|
|
260
|
-
|
|
261
|
-
|
|
262
|
-
|
|
263
|
-
|
|
264
|
-
|
|
265
|
-
|
|
266
|
-
|
|
267
|
-
|
|
268
|
-
|
|
269
|
-
|
|
270
|
-
|
|
271
|
-
|
|
272
|
-
|
|
273
|
-
|
|
274
|
-
|
|
275
|
-
|
|
276
|
-
|
|
277
|
-
仍无法完成的审查项或检查,列为未完成并注明原因
|
|
278
|
-
4. 汇总已确认的问题与检查完成情况,作为结论判定的依据
|
|
279
|
-
|
|
280
|
-
## 7 判定结论
|
|
234
|
+
- 设计审查所需的第四章机械检查结果与证据
|
|
235
|
+
- 实现审查所需的目录与文件结构、改动文件、模块依赖与仓库代码组织约定
|
|
236
|
+
3. **执行问题导向审查**
|
|
237
|
+
- 子 agent 逐项核验所负责维度的核对项与适用检查条件
|
|
238
|
+
- 发现问题时,按[审查报告](#71-审查报告)中的问题格式记录文件位置、问题、影响与调整建议
|
|
239
|
+
- 存在无法完成的检查项时,记录未完成项及原因
|
|
240
|
+
- 通过项与不适用项不逐条输出留痕明细,将发现的问题清单、未完成项与审查结论返回主 agent
|
|
241
|
+
|
|
242
|
+
## 6 复核与判定
|
|
243
|
+
|
|
244
|
+
### 6.1 复核与归并
|
|
245
|
+
|
|
246
|
+
1. **复核问题证据**
|
|
247
|
+
- 主 agent 逐条核对子 agent 返回的问题涉及的文件与证据,确认成立后纳入报告
|
|
248
|
+
- 已证伪的问题排除,证据不足的问题列为未完成核验
|
|
249
|
+
- 主 agent 可补充自行发现的问题,经同样的证据核验后纳入报告
|
|
250
|
+
- 合并重复问题,保留发生位置、证据与影响
|
|
251
|
+
2. **核对审查覆盖**
|
|
252
|
+
- 确认所选审查维度及其核对项均已完成核验
|
|
253
|
+
- 子 agent 未覆盖分配的范围或遗漏核对项时,补查对应内容并复核结果
|
|
254
|
+
- 仍无法完成的审查维度或检查,列为未完成并注明原因
|
|
255
|
+
|
|
256
|
+
### 6.2 判定结论
|
|
281
257
|
|
|
282
258
|
| 结论 | 条件 |
|
|
283
259
|
| --- | --- |
|
|
284
|
-
| 通过 |
|
|
260
|
+
| 通过 | 所选审查任务全部完成、适用检查条件全部核验通过、适用机械检查通过、无必须调整项 |
|
|
285
261
|
| 需要调整 | 存在已确认的必须调整项 |
|
|
286
262
|
| 未完成 | 未满足通过条件,且无已确认的必须调整项 |
|
|
287
263
|
|
|
@@ -289,9 +265,9 @@ flowchart TD
|
|
|
289
265
|
可选优化不影响通过
|
|
290
266
|
受审内容发生变化后,重新审查受影响范围,并更新对应结论
|
|
291
267
|
|
|
292
|
-
##
|
|
268
|
+
## 7 输出结果与下一步
|
|
293
269
|
|
|
294
|
-
###
|
|
270
|
+
### 7.1 审查报告
|
|
295
271
|
|
|
296
272
|
在当前对话中按以下模板输出审查结果,不写入项目文件
|
|
297
273
|
|
|
@@ -300,12 +276,13 @@ flowchart TD
|
|
|
300
276
|
|
|
301
277
|
| 项目 | 结果 |
|
|
302
278
|
| --- | --- |
|
|
303
|
-
| 对象 | <设计文档 /
|
|
279
|
+
| 对象 | <设计文档 / 代码结构 / 两者> |
|
|
304
280
|
| 范围 | <受审模块、文件或改动范围> |
|
|
305
281
|
| 基线 | <比较基线、受审提交及未提交改动> |
|
|
282
|
+
| 执行模型 | <实际执行的 Agent 环境、模型、复用情况及回落原因> |
|
|
306
283
|
| 结论依据 | <满足通过条件的情况,或阻止通过的问题与未完成检查> |
|
|
307
284
|
| 必须调整 | <问题数量与主要影响 / 无> |
|
|
308
|
-
| 工具检查 |
|
|
285
|
+
| 工具检查 | <设计审查机械检查的通过、复用、失败及未完成情况 / 实现审查不适用> |
|
|
309
286
|
| 未完成 | <本次范围内未完成的审查任务、检查及原因 / 无> |
|
|
310
287
|
| 下一步 | <一个立即可执行的动作> |
|
|
311
288
|
|
|
@@ -313,39 +290,26 @@ flowchart TD
|
|
|
313
290
|
|
|
314
291
|
- <文档章节链接 / 代码文件链接与行号>
|
|
315
292
|
- 问题:<实际内容、违反的设计约定或检查条件及具体偏差>
|
|
316
|
-
-
|
|
293
|
+
- 影响:<对设计理解、实施或代码结构维护的影响,以及适用的发生条件>
|
|
317
294
|
- 调整:<需要修正或确认的内容,以及修正后应满足的约定或检查条件>
|
|
318
295
|
|
|
319
296
|
## 可选优化
|
|
320
297
|
|
|
321
298
|
- <不影响审查通过的改善建议、对应位置与预期收益>
|
|
322
|
-
|
|
323
|
-
## 检查范围
|
|
324
|
-
|
|
325
|
-
- 阅读:<本次实际读取的文件>
|
|
326
|
-
- 检查条件:<条件及来源、逐项核验结果、不适用依据与未完成原因>
|
|
327
|
-
- 工具:<实际执行的命令与结果,或复用的原结果及复用依据>
|
|
328
|
-
|
|
329
|
-
| 审查项 | Agent 环境 | 模型 | 完成状态 | 说明 |
|
|
330
|
-
| --- | --- | --- | --- | --- |
|
|
331
|
-
| <设计审查视角 / 实现审查项> | <实际 Agent 环境> | <实际模型及可核实的版本> | <已完成 / 未完成> | <回落或未完成原因 / 无> |
|
|
332
299
|
```
|
|
333
300
|
|
|
334
|
-
|
|
335
|
-
实现审查列出结构设定、具体逻辑实现与整体结构实现三项
|
|
336
|
-
|
|
337
|
-
### 8.2 后续动作
|
|
301
|
+
### 7.2 后续动作
|
|
338
302
|
|
|
339
303
|
按审查结论与本轮进度选择下一步
|
|
340
304
|
|
|
341
305
|
| 结论 | 情况 | 下一步 |
|
|
342
306
|
| --- | --- | --- |
|
|
343
|
-
| 未完成 |
|
|
344
|
-
| 需要调整 | 实现问题 | `/aca-implement` 修复并验证 |
|
|
307
|
+
| 未完成 | 输入或核验条件缺失 | 报告恢复条件,补齐后通过 `/aca-review` 继续本次审查 |
|
|
345
308
|
| 需要调整 | 设计方案仍需论证或决定 | `/aca-specify` 补充或修订方案 |
|
|
346
309
|
| 需要调整 | 方案已明确,仅需修订设计文档 | `/aca-design` 修订文档 |
|
|
347
|
-
|
|
|
348
|
-
| 通过 |
|
|
349
|
-
| 通过 |
|
|
310
|
+
| 需要调整 | 代码结构问题 | `/aca-implement` 调整代码结构 |
|
|
311
|
+
| 通过 | 设计审查通过,本轮需要实施代码 | `/aca-implement` |
|
|
312
|
+
| 通过 | 设计审查通过,本轮仅改设计文档 | 按已确认的版本推进安排衔接 `/aca-release` |
|
|
313
|
+
| 通过 | 实现审查通过 | `/aca-test` 执行当前范围的测试验证 |
|
|
350
314
|
|
|
351
|
-
|
|
315
|
+
设计或代码结构修正后,通过 `/aca-review` 复审受影响范围
|