@routerhub/agent-rules 1.5.72 → 1.5.74
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/AGENTS.base.md +4 -4
- package/package.json +1 -1
- package/rules/global.md +4 -4
package/AGENTS.base.md
CHANGED
|
@@ -116,14 +116,14 @@ Closes #456
|
|
|
116
116
|
- 用户说”Copilot review 出了一些问题”时,同样触发以下循环流程。
|
|
117
117
|
- Copilot review 循环流程:
|
|
118
118
|
1. 打开 PR 页面,逐一查看 Copilot review 提出的每一条问题。
|
|
119
|
-
2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 →
|
|
119
|
+
2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
|
|
120
120
|
3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
|
|
121
|
-
4. ⚠️
|
|
121
|
+
4. ⚠️ **强制步骤:修改代码并 push 后,必须立刻在 GitHub PR 页面上点击 Resolve conversation。每 push 一条修复,立即 Resolve 对应的那条评论。不可批量延后、不可等到下一轮 review 后再 Resolve。** Copilot 每次 review 只看当前代码 diff,不依赖 Resolve 状态——代码改了就不会再提同一问题。因此 push = 问题已处理,应立即 Resolve 让 reviewer 知道该条已处理完毕。
|
|
122
122
|
5. 所有问题处理完毕后,重新请求 Copilot review。
|
|
123
123
|
6. 重复以上步骤,直到满足以下任一终止条件为止。
|
|
124
|
-
-
|
|
124
|
+
- ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve。
|
|
125
125
|
- 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
|
|
126
|
-
- 如果 Copilot review
|
|
126
|
+
- 如果 Copilot review 连续两轮对同一问题提出类似建议,但代码实际上已正确修复(或该建议本身无道理/属于误报),需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
|
|
127
127
|
- ⚠️ **循环终止条件(满足任意一条即停止)**:
|
|
128
128
|
1. **无新问题**:Copilot review 本轮未提出任何新问题 → 自然终止,循环结束。
|
|
129
129
|
2. **仅剩可忽略级别建议**:本轮 Copilot review 提出的所有问题均标注了 `nit:`、`optional:`、`question:` 或建议将配置项写入 `.env` 等可忽略类型,无任何 `Request changes` 级别的问题 → 逐条回复说明原因并 Resolve,然后终止循环,无需再请求新一轮 review。
|
package/package.json
CHANGED
package/rules/global.md
CHANGED
|
@@ -118,14 +118,14 @@ Closes #456
|
|
|
118
118
|
- 用户说”Copilot review 出了一些问题”时,同样触发以下循环流程。
|
|
119
119
|
- Copilot review 循环流程:
|
|
120
120
|
1. 打开 PR 页面,逐一查看 Copilot review 提出的每一条问题。
|
|
121
|
-
2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 →
|
|
121
|
+
2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
|
|
122
122
|
3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
|
|
123
|
-
4. ⚠️
|
|
123
|
+
4. ⚠️ **强制步骤:修改代码并 push 后,必须立刻在 GitHub PR 页面上点击 Resolve conversation。每 push 一条修复,立即 Resolve 对应的那条评论。不可批量延后、不可等到下一轮 review 后再 Resolve。** Copilot 每次 review 只看当前代码 diff,不依赖 Resolve 状态——代码改了就不会再提同一问题。因此 push = 问题已处理,应立即 Resolve 让 reviewer 知道该条已处理完毕。
|
|
124
124
|
5. 所有问题处理完毕后,重新请求 Copilot review。
|
|
125
125
|
6. 重复以上步骤,直到满足以下任一终止条件为止。
|
|
126
|
-
-
|
|
126
|
+
- ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve。
|
|
127
127
|
- 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
|
|
128
|
-
- 如果 Copilot review
|
|
128
|
+
- 如果 Copilot review 连续两轮对同一问题提出类似建议,但代码实际上已正确修复(或该建议本身无道理/属于误报),需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
|
|
129
129
|
- ⚠️ **循环终止条件(满足任意一条即停止)**:
|
|
130
130
|
1. **无新问题**:Copilot review 本轮未提出任何新问题 → 自然终止,循环结束。
|
|
131
131
|
2. **仅剩可忽略级别建议**:本轮 Copilot review 提出的所有问题均标注了 `nit:`、`optional:`、`question:` 或建议将配置项写入 `.env` 等可忽略类型,无任何 `Request changes` 级别的问题 → 逐条回复说明原因并 Resolve,然后终止循环,无需再请求新一轮 review。
|