@routerhub/agent-rules 1.5.72 → 1.5.73

package/AGENTS.base.md

@@ -118,12 +118,16 @@ Closes #456
   1. 打开 PR 页面，逐一查看 Copilot review 提出的每一条问题。
   2. 逐条判断问题是否有道理：确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复；误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改，然后 Resolve。
   3. 对于有道理的问题，直接修改代码、提交并推送到该 PR 分支。
-  4. ⚠️ **强制步骤：修改代码并推送后，必须在 GitHub PR 页面上逐条点击 Resolve conversation。** 此步骤不可跳过——即使已回复评论、已推送修复代码，只要未点 Resolve，该条评论仍处于”未解决”状态，reviewer 无法判断是否已处理完毕。每修复一条，立即 Resolve 一条，不要等所有问题修完再批量操作。
-  5. 所有问题处理完毕后，重新请求 Copilot review。
+  4. ⚠️ **强制步骤：所有问题处理完毕后，重新请求 Copilot review。**
+  5. ⚠️ **Resolve 规则：必须等下一轮 Copilot review 确认改对了再点 Resolve，禁止 push 后立刻 Resolve。** 具体做法：
+     - 本轮修复了问题 A → push → 请求下一轮 review → 查看新一轮 review 结果：
+       - 若 Copilot **未再提及问题 A**（说明已改对）→ 回到上轮 review 的对应评论，点击 **Resolve conversation**。
+       - 若 Copilot **再次提出问题 A**（说明没改对或改法有问题）→ 不点 Resolve，继续修改并 push，再请求新一轮 review，直到该问题不再出现为止。
+     - 这样做的好处：每条 conversation 的 Resolve 状态真正代表「该问题已修复且通过 reviewer 验证」，而不是「我改了一下但不知道对不对」。
   6. 重复以上步骤，直到满足以下任一终止条件为止。
-- 每一条 Copilot review 评论处理完毕后，必须点击 **Resolve conversation**，不能只回复不 Resolve。不点 Resolve 会导致该评论一直处于未解决状态，无法判断是否已完成处理。
+- ⚠️ **Resolve 时机：必须等下一轮 Copilot review 确认该问题不再出现后，才能点击 Resolve conversation。** 禁止 push 修复代码后立刻 Resolve——此时尚不知道修复是否正确。只有新一轮 review 未再提及该问题时，才能确认改对了并 Resolve。
 - 每次修改后提交的 commit 信息必须中文，格式：`修复 Copilot review 问题：[简要描述]`。
-- 如果 Copilot review 连续两轮提出同一问题但实际已修复或无道理，需在 PR 评论中明确说明原因并直接 Resolve，不再反复修改。
+- 如果 Copilot review 连续两轮对同一问题提出类似建议，但代码实际上已正确修复（或该建议本身无道理/属于误报），需在 PR 评论中明确说明原因并直接 Resolve，不再反复修改。
 - ⚠️ **循环终止条件（满足任意一条即停止）**：
   1. **无新问题**：Copilot review 本轮未提出任何新问题 → 自然终止，循环结束。
   2. **仅剩可忽略级别建议**：本轮 Copilot review 提出的所有问题均标注了 `nit:`、`optional:`、`question:` 或建议将配置项写入 `.env` 等可忽略类型，无任何 `Request changes` 级别的问题 → 逐条回复说明原因并 Resolve，然后终止循环，无需再请求新一轮 review。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@routerhub/agent-rules",
-  "version": "1.5.72",
+  "version": "1.5.73",
   "description": "Shared Copilot agent rules and guidelines for RouterHub projects",
   "main": "AGENTS.base.md",
   "bin": {

package/rules/global.md

@@ -120,12 +120,16 @@ Closes #456
   1. 打开 PR 页面，逐一查看 Copilot review 提出的每一条问题。
   2. 逐条判断问题是否有道理：确实存在的代码缺陷、逻辑错误、安全风险、性能问题等 → 必须修复；误报、与需求不符、风格偏好无实质影响等 → 在评论中回复解释为何不改，然后 Resolve。
   3. 对于有道理的问题，直接修改代码、提交并推送到该 PR 分支。
-  4. ⚠️ **强制步骤：修改代码并推送后，必须在 GitHub PR 页面上逐条点击 Resolve conversation。** 此步骤不可跳过——即使已回复评论、已推送修复代码，只要未点 Resolve，该条评论仍处于”未解决”状态，reviewer 无法判断是否已处理完毕。每修复一条，立即 Resolve 一条，不要等所有问题修完再批量操作。
-  5. 所有问题处理完毕后，重新请求 Copilot review。
+  4. ⚠️ **强制步骤：所有问题处理完毕后，重新请求 Copilot review。**
+  5. ⚠️ **Resolve 规则：必须等下一轮 Copilot review 确认改对了再点 Resolve，禁止 push 后立刻 Resolve。** 具体做法：
+     - 本轮修复了问题 A → push → 请求下一轮 review → 查看新一轮 review 结果：
+       - 若 Copilot **未再提及问题 A**（说明已改对）→ 回到上轮 review 的对应评论，点击 **Resolve conversation**。
+       - 若 Copilot **再次提出问题 A**（说明没改对或改法有问题）→ 不点 Resolve，继续修改并 push，再请求新一轮 review，直到该问题不再出现为止。
+     - 这样做的好处：每条 conversation 的 Resolve 状态真正代表「该问题已修复且通过 reviewer 验证」，而不是「我改了一下但不知道对不对」。
   6. 重复以上步骤，直到满足以下任一终止条件为止。
-- 每一条 Copilot review 评论处理完毕后，必须点击 **Resolve conversation**，不能只回复不 Resolve。不点 Resolve 会导致该评论一直处于未解决状态，无法判断是否已完成处理。
+- ⚠️ **Resolve 时机：必须等下一轮 Copilot review 确认该问题不再出现后，才能点击 Resolve conversation。** 禁止 push 修复代码后立刻 Resolve——此时尚不知道修复是否正确。只有新一轮 review 未再提及该问题时，才能确认改对了并 Resolve。
 - 每次修改后提交的 commit 信息必须中文，格式：`修复 Copilot review 问题：[简要描述]`。
-- 如果 Copilot review 连续两轮提出同一问题但实际已修复或无道理，需在 PR 评论中明确说明原因并直接 Resolve，不再反复修改。
+- 如果 Copilot review 连续两轮对同一问题提出类似建议，但代码实际上已正确修复（或该建议本身无道理/属于误报），需在 PR 评论中明确说明原因并直接 Resolve，不再反复修改。
 - ⚠️ **循环终止条件（满足任意一条即停止）**：
   1. **无新问题**：Copilot review 本轮未提出任何新问题 → 自然终止，循环结束。
   2. **仅剩可忽略级别建议**：本轮 Copilot review 提出的所有问题均标注了 `nit:`、`optional:`、`question:` 或建议将配置项写入 `.env` 等可忽略类型，无任何 `Request changes` 级别的问题 → 逐条回复说明原因并 Resolve，然后终止循环，无需再请求新一轮 review。