@routerhub/agent-rules 1.5.143 → 1.5.144

package/package.json

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

package/skills/deploy-test/SKILL.md

@@ -40,6 +40,17 @@ git commit -m "..."  # 使用有意义的提交信息
 git push origin "$ORIGINAL_BRANCH"
 ```
 
+⚠️ **若本次改动经过循环 review（`loop-review` 修复过代码），必须确认所有 review 修复已同步到当前功能分支（PR 对应分支）并推送远程，再进行 test 合并。** 核对方式：
+
+```bash
+# 确认本地功能分支与远程已同步（本地无未推送提交）
+git status
+git log origin/"$ORIGINAL_BRANCH".."$ORIGINAL_BRANCH" --oneline   # 应无输出 = 无未推送提交
+```
+
+- ⚠️ 若 review 修复还没合进当前功能分支 → 先把修复 commit 到功能分支再推送，禁止带着"修复只在 test 分支"的状态去部署。
+- ⚠️ feature 分支与 test 分支两条线必须保持同步：**所有 review 修复代码必须先落 feature 分支（PR 分支），再合并到 test**，缺一不可。
+
 ### 3. test 分支检查
 
 如果 `test` 分支不存在，从主分支创建：

package/skills/loop-review/SKILL.md

@@ -121,6 +121,7 @@ gh api repos/$REPO/pulls/$PR/reviews?per_page=100 --jq \
 - 判定「值得修」的（无论严重度）→ 每个修复单独 commit，commit 信息用中文，正文引用对应 review 来源
 - 纯建议级且判定值得改的 → 合并为一个 commit
 - 判定「不值得修 / 已处理 / 误报」的 → 按步骤 4 回复，切断 bot 下轮重提
+- ⚠️ **所有修复必须直接 commit 在当前 PR 对应的 feature 分支上**（`git push origin <feature 分支>`），并推送到远程。禁止把 review 修复只放在其他地方（例如直接堆到 test 分支、或修改后不 commit）。**宗旨：feature 分支（PR）与 test 分支两条线保持同步，修复代码同时存在于 feature 分支和 test 分支，缺一不可。**
 
 ### 4. 对「不改」的建议回复（防重复循环，关键）
 
@@ -204,6 +205,7 @@ done
 - ⚠️ **禁止使用 `[skip ci]` / `[skip review]`**——循环依赖「每次 push 触发 review」，跳过就断了。
 - ⚠️ **必须设最大轮数（默认 5；用户明确要求深度循环时最多 8）和等待超时（默认 15 分钟）**，禁止无限循环 / 无限等待。
 - ⚠️ **已修复的功能性问题单独 commit**，禁止与大量建议级改动混在一起，保证每轮 push 的 diff 可读。
+- ⚠️ **每条 review 修复都必须同 commit 回 PR 对应的 feature 分支并推送远程**，不得只推到 test 分支或只放在工作区不 commit。feature 分支与 test 分支两条线必须同步，缺一不可（详见步骤 3）。
 - ⚠️ **等待 bot 审查期间遵守心跳约定**：超过 1 分钟无输出主动说明在等什么。
 - ⚠️ **commit 用中文、禁止 `git push --force`、commit 信息引用对应 review 来源**。
 - ⚠️ **两模型报同一问题合并为一条处理**，不要重复改两遍。