@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 CHANGED
@@ -116,14 +116,14 @@ Closes #456
116
116
  - 用户说”Copilot review 出了一些问题”时,同样触发以下循环流程。
117
117
  - Copilot review 循环流程:
118
118
  1. 打开 PR 页面,逐一查看 Copilot review 提出的每一条问题。
119
- 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,然后 Resolve。
119
+ 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
120
120
  3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
121
- 4. ⚠️ **强制步骤:修改代码并推送后,必须在 GitHub PR 页面上逐条点击 Resolve conversation。** 此步骤不可跳过——即使已回复评论、已推送修复代码,只要未点 Resolve,该条评论仍处于”未解决”状态,reviewer 无法判断是否已处理完毕。每修复一条,立即 Resolve 一条,不要等所有问题修完再批量操作。
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
- - 每一条 Copilot review 评论处理完毕后,必须点击 **Resolve conversation**,不能只回复不 Resolve。不点 Resolve 会导致该评论一直处于未解决状态,无法判断是否已完成处理。
124
+ - ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve
125
125
  - 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
126
- - 如果 Copilot review 连续两轮提出同一问题但实际已修复或无道理,需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
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
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "@routerhub/agent-rules",
3
- "version": "1.5.72",
3
+ "version": "1.5.74",
4
4
  "description": "Shared Copilot agent rules and guidelines for RouterHub projects",
5
5
  "main": "AGENTS.base.md",
6
6
  "bin": {
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. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,然后 Resolve。
121
+ 2. 逐条判断问题是否有道理:确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复;误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改,直接 Resolve。
122
122
  3. 对于有道理的问题,直接修改代码、提交并推送到该 PR 分支。
123
- 4. ⚠️ **强制步骤:修改代码并推送后,必须在 GitHub PR 页面上逐条点击 Resolve conversation。** 此步骤不可跳过——即使已回复评论、已推送修复代码,只要未点 Resolve,该条评论仍处于”未解决”状态,reviewer 无法判断是否已处理完毕。每修复一条,立即 Resolve 一条,不要等所有问题修完再批量操作。
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
- - 每一条 Copilot review 评论处理完毕后,必须点击 **Resolve conversation**,不能只回复不 Resolve。不点 Resolve 会导致该评论一直处于未解决状态,无法判断是否已完成处理。
126
+ - ⚠️ **强制步骤:每 push 一条修复代码后,必须立刻点击该条评论的 Resolve conversation。不可漏点、不可批量延后。** Copilot 每次 review 只看当前代码 diff——代码改了就不会再提同一问题,因此 push 即意味着问题已处理,应立即 Resolve
127
127
  - 每次修改后提交的 commit 信息必须中文,格式:`修复 Copilot review 问题:[简要描述]`。
128
- - 如果 Copilot review 连续两轮提出同一问题但实际已修复或无道理,需在 PR 评论中明确说明原因并直接 Resolve,不再反复修改。
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。