@heihei0299/matt-skills 3.0.8 → 3.0.9
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.
|
@@ -65,7 +65,7 @@ Verify 通过后读取 [review.md](references/review.md)。
|
|
|
65
65
|
|
|
66
66
|
Review 通过后读取 [finalize.md](references/finalize.md)，只做 tracker/progress/status 收尾并记录 `issue_head`。
|
|
67
67
|
|
|
68
|
-
Finalize 不新增产品 Behavior，也不修改已经 Review
|
|
68
|
+
Finalize 不新增产品 Behavior，也不修改已经 Review 的实现内容。若收尾时发现实现、测试、文档/配置或验证遗漏，停止当前 issue，不标记 `resolved`，并按 `finalize.md` 报告遗漏请求决策；不得在 Finalize 中补改或重新进入 Red-Green、Verify 或 Review。
|
|
69
69
|
|
|
70
70
|
## 运行纪律
|
|
71
71
|
|
|
@@ -5,23 +5,19 @@
|
|
|
5
5
|
## 步骤
|
|
6
6
|
|
|
7
7
|
1. 确认 `review_head` 已记录，且 `issue_base...review_head` 对应的实现范围已经完成 Review。
|
|
8
|
-
2. 更新 Acceptance Criteria 与 progress/tracker，记录已 Review 的实现范围、Review
|
|
9
|
-
3.
|
|
10
|
-
4.
|
|
8
|
+
2. 更新 Acceptance Criteria 与 progress/tracker，记录已 Review 的实现范围、Review、验证与运行结果；此时不得提前标记 `resolved` 或解除 blockers。
|
|
9
|
+
3. 若 Finalize 过程中发现任何实现、测试、交付文档、配置或验证遗漏，立即停止当前 issue：保持未完成，不标记 `resolved`，不解除 blockers，不设置成功的 `issue_head`，并报告遗漏请求决策。不得在 Finalize 中补改，也不得自动重新进入 Red-Green、Verify 或 Review。
|
|
10
|
+
4. 仅在未发现上述遗漏且所有必要状态同步准备完成后，才同步 `resolved` / blockers 状态。
|
|
11
|
+
5. 若状态同步修改了仓库内的 tracker/progress/status 文件，将这些状态修改提交；不得在该提交中混入产品实现或其它未 Review 的交付修改。
|
|
12
|
+
6. 确认所有必要状态同步成功后，设置 `issue_head = HEAD`。
|
|
11
13
|
|
|
12
14
|
无需为了固定 commit 数量而 amend、squash 或重写当前 issue 的历史。
|
|
13
15
|
|
|
14
|
-
若 Finalize 中发现遗漏：
|
|
15
|
-
|
|
16
|
-
- 仅缺验证证据：返回 Verify 补足证据，再回 Finalize；
|
|
17
|
-
- 需要修改代码、测试、交付文档或配置：将该遗漏加入 `open_findings`，返回对应 Red-Green / Verify 完成修改与验证，提交修复后按 `review.md` 做增量 Review；`full_review_done` 保持为 `true`。
|
|
18
|
-
|
|
19
|
-
不得在 Finalize 中直接补实现后继续标记完成。
|
|
20
|
-
|
|
21
16
|
## 出口
|
|
22
17
|
|
|
23
18
|
- Acceptance Criteria 全部通过；
|
|
24
19
|
- `issue_base...review_head` 的实现范围已经完成 Review；
|
|
20
|
+
- Finalize 未发现实现、测试、交付文档、配置或验证遗漏；
|
|
25
21
|
- tracker/progress/status 与实际 Review、验证和完成状态一致；
|
|
26
22
|
- `review_head` 之后若存在 commits，只包含当前 issue 的状态收尾修改；
|
|
27
23
|
- `issue_head = HEAD` 已记录；
|
|
@@ -4,13 +4,14 @@
|
|
|
4
4
|
|
|
5
5
|
## 状态
|
|
6
6
|
|
|
7
|
-
|
|
7
|
+
当前 issue 首次进入 Review 时初始化以下状态，且仅初始化一次；后续因 finding 修复返回 Red-Green / Verify 后继续 Review 时保留现有状态，不得重新初始化：
|
|
8
8
|
|
|
9
9
|
- `issue_base`：当前 issue 开始时的 `HEAD`；
|
|
10
10
|
- `full_review_done = false`；
|
|
11
11
|
- `open_findings = []`；
|
|
12
12
|
- `last_reviewed_head = null`；
|
|
13
|
-
- `review_head = null
|
|
13
|
+
- `review_head = null`；
|
|
14
|
+
- `incremental_review_rounds = 0`。
|
|
14
15
|
|
|
15
16
|
## 1. 形成 committed Review Point
|
|
16
17
|
|
|
@@ -52,6 +53,10 @@
|
|
|
52
53
|
|
|
53
54
|
存在 `open_findings` 时，只处理已有 finding，不扩大当前 issue 范围。可将由同一改动共同解决的相关 findings 一起处理，不要求“一 finding 一 commit”。
|
|
54
55
|
|
|
56
|
+
每个 issue 最多执行 2 个逻辑增量 Review 轮次。只有正常形成增量 Review 结论的轮次才计数；工具错误、stream interruption、sub-agent failure 或其它未形成完整结论的技术失败不消耗轮次，只重试当前逻辑轮次。
|
|
57
|
+
|
|
58
|
+
每轮修复前，若 `incremental_review_rounds >= 2` 且 `open_findings` 仍非空，则停止当前 issue：不得再次启动增量 Review，不设置 `review_head`，不得进入 Finalize，并报告剩余 findings 请求决策。
|
|
59
|
+
|
|
55
60
|
每轮修复：
|
|
56
61
|
|
|
57
62
|
1. 修复选定的 open findings；
|
|
@@ -60,23 +65,35 @@
|
|
|
60
65
|
4. 将本轮修复提交到当前 issue 的 commit range，并确认不存在属于本轮修复的未提交修改；
|
|
61
66
|
5. 只针对 `last_reviewed_head...HEAD` 与本轮目标 findings 做增量 Review；增量 Review 不调用完整 `code-review`。
|
|
62
67
|
|
|
68
|
+
若增量 Review 正常形成结论，先设置 `incremental_review_rounds += 1`。
|
|
69
|
+
|
|
63
70
|
增量 Review 通过时：
|
|
64
71
|
|
|
65
72
|
- 仅关闭本轮已由证据确认解决的 findings；
|
|
66
73
|
- 设置 `last_reviewed_head = HEAD`。
|
|
67
74
|
|
|
68
|
-
增量 Review
|
|
75
|
+
增量 Review 正常完成但未通过时：
|
|
69
76
|
|
|
77
|
+
- 保留未关闭 findings；
|
|
78
|
+
- 不推进 `last_reviewed_head`；
|
|
79
|
+
- 若尚未达到 2 轮上限，下一轮继续从上一次成功的 `last_reviewed_head` 审查累计的未 Review 修复。
|
|
80
|
+
|
|
81
|
+
增量 Review 发生技术失败时：
|
|
82
|
+
|
|
83
|
+
- 不增加 `incremental_review_rounds`；
|
|
70
84
|
- 不推进 `last_reviewed_head`；
|
|
71
85
|
- 不移除未确认关闭的 findings；
|
|
72
|
-
-
|
|
86
|
+
- 恢复后重试当前逻辑轮次。
|
|
73
87
|
|
|
74
88
|
当 `open_findings` 为空时，设置 `review_head = last_reviewed_head`，Review 通过。
|
|
75
89
|
|
|
90
|
+
当 `incremental_review_rounds = 2` 且 `open_findings` 仍非空时，Review 不通过：保持 issue 未完成，不设置 `review_head`，不进入 Finalize，并报告剩余 findings 请求决策。
|
|
91
|
+
|
|
76
92
|
## 出口
|
|
77
93
|
|
|
78
94
|
- `full_review_done = true`；
|
|
79
95
|
- `open_findings` 为空；
|
|
80
96
|
- `review_head` 非空；
|
|
97
|
+
- `incremental_review_rounds <= 2`；
|
|
81
98
|
- `issue_base...review_head` 是已经完成 Review 的当前 issue 实现范围；
|
|
82
99
|
- 不存在属于该实现范围的未提交交付修改。
|