@ranger1/dx 0.1.48 → 0.1.50

package/@opencode/commands/pr-review-loop.md

@@ -15,6 +15,19 @@ agent: sisyphus
 ## 输入
 
 - `{{PR_NUMBER}}`
+- `round`（默认 1，由调用者/循环控制）
+
+## 唯一标识 runId（强制）
+
+- 全局唯一标识 `runId` 格式：`<PR>-<ROUND>-<HEAD_SHORT>`
+- 其中：
+  - `<PR>`：PR 编号
+  - `<ROUND>`：当前轮次
+  - `<HEAD_SHORT>`：`headOid` 的前 7 位（git rev-parse --short HEAD）
+- 生成者：
+  - 第 1 步 `pr-context` 负责计算并返回 `runId`（基于当前 checkout 的 headOid）
+  - 后续所有步骤（reviewers, aggregate, fix）必须透传并使用该 `runId`
+  - 禁止任何下游步骤自行生成或篡改 `runId`
 
 ## Cache 约定（强制）
 
@@ -26,6 +39,7 @@ agent: sisyphus
 - `pr-precheck`
 - `pr-context`
 - `codex-reviewer`
+
 - `claude-reviewer`
 - `gemini-reviewer`
 - `gh-thread-reviewer`
@@ -37,14 +51,15 @@ agent: sisyphus
 
 0. Task: `pr-precheck`（强制 gate：编译/预检必须先通过）
 
-- prompt 必须包含：`PR #{{PR_NUMBER}}`
-- 若返回 `{"error":"..."}`：立即终止本轮并回传错误（不再调用 reviewers）
+- prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`（precheck 需计算并返回 runId，格式同 context）
+- 若返回 `{"error":"..."}`：立即终止本轮并回传错误
 - 若返回 `{"ok":false,"fixFile":"..."}`：
+  - 预检阶段 runId 同样基于 `headOid` 生成（`<PR>-<ROUND>-<HEAD_SHORT>`），可直接传给 fix。
   - 最多修复 2 次（防止无限循环）：
-    - 第 1 次：Task `pr-fix`（使用该 fixFile）→ 再 Task `pr-precheck`
+    - 第 1 次：Task `pr-fix`（传入 `fixFile`, `runId`, `round`）→ 再 Task `pr-precheck`
     - 若仍返回 `{"ok":false,"fixFile":"..."}`：第 2 次 Task `pr-fix` → 再 Task `pr-precheck`
   - 若仍不是 `{"ok":true}`：终止并回传错误（建议：`{"error":"PRECHECK_NOT_CLEAN_AFTER_FIX"}`）
-
+- 注意：预检失败产生的修复也应记录在 Decision Log 中（essence: `__precheck__` 或具体错误信息，file: `__precheck__`），以便后续追踪。
 
 ## 循环（最多 3 轮）
 
@@ -61,7 +76,8 @@ agent: sisyphus
 
 - prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`
 - 若返回 `{"error":"..."}`：立即终止本轮并回传错误（不再调用 reviewers）
-- 取出：`contextFile`、`runId`、`headOid`（如有）
+- 取出：`contextFile`、`runId`、`headOid`
+- **runId 校验**：确认返回的 `runId` 符合 `<PR>-<ROUND>-<HEAD_SHORT>` 格式
 - **CRITICAL**: 必须等待此 Task 成功完成并获取到 `contextFile` 后，才能进入 Step 2
 
 **检查 Decision Log**：
@@ -77,14 +93,14 @@ agent: sisyphus
   - `round: <ROUND>`
   - `runId: <RUN_ID>`（来自 Step 1 的输出，必须透传，禁止自行生成）
   - `contextFile: ./.cache/<file>.md`（来自 Step 1 的输出）
-  - `decisionLogFile: ./.cache/decision-log-pr{{PR_NUMBER}}.md`（如存在，来自检查后得出）
+  - `decisionLogFile: ./.cache/decision-log-pr{{PR_NUMBER}}.md`（如存在）
 - reviewer 默认读 `contextFile`；如果 `decisionLogFile` 存在，reviewer 应在 prompt 中提供该文件路径以参考前轮决策；必要时允许用 `git/gh` 只读命令拿 diff
 - 忽略问题：1.格式化代码引起的噪音 2.已经lint检查以外的格式问题 3.忽略单元测试不足的问题
 - 特别关注: 逻辑、安全、性能、可维护性
-- 同时要注意 pr 前面轮次的 修复和讨论，对于已经拒绝、已修复的问题不要反复的提出
-- 同时也要注意fix的过程中有没有引入新的问题。
-
-备注：fixFile 分为 `IssuesToFix`（P0/P1，必须修）与 `OptionalIssues`（P2/P3，pr-fix 自主裁决）。
+- 遵守 Decision Log：
+  - 已修复（Fixed）：不再提
+  - 已拒绝（Rejected）：除非优先级升级（P_new - P_old >= 2），否则不再提
+  - 任何新发现必须基于当前 `runId` 对应的代码状态
 - 每个 reviewer 输出：`reviewFile: ./.cache/<file>.md`（Markdown）
 
 3. Task: `pr-review-aggregate`
@@ -93,6 +109,10 @@ agent: sisyphus
 - 输出：`{"stop":true}` 或 `{"stop":false,"fixFile":"..."}`
 - 若 `stop=true`：本轮结束并退出循环
 - **唯一性约束**: 每轮只能发布一次 Review Summary；脚本内置幂等检查，重复调用不会重复发布
+- 智能聚合：
+  - 使用 LLM 对比 decision-log 中的 `essence` 与新 finding
+  - 仅当问题本质相同且优先级 delta < 2 时，自动归为 Repeated/Ignored
+  - 否则视为 New Issue 或 Escalation
 
 4. Task: `pr-fix`
 
@@ -102,9 +122,13 @@ agent: sisyphus
   - `runId: <RUN_ID>`（来自 Step 1 的输出，必须透传，禁止自行生成）
   - `fixFile: ./.cache/<file>.md`
 - 约定：`pr-fix` 对每个 findingId 单独 commit + push（一个 findingId 一个 commit），结束后再 `git push` 兜底
-
+- 决策记录：
+  - 修复成功：追加 Fixed 记录（含 `essence`）到 Decision Log
+  - 拒绝/无法修复：追加 Rejected 记录（含 `reason`, `essence`）到 Decision Log
+  - 范围限制：essence 匹配必须在 **同一个文件** 内（不支持跨文件/重命名追踪）
 - pr-fix 输出：`fixReportFile: ./.cache/<file>.md`（Markdown）
 
+
 5. Task: `pr-review-aggregate`（发布修复评论）
 
 - prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`、`runId: <RUN_ID>`、`fixReportFile: ./.cache/<file>.md`
@@ -119,6 +143,32 @@ agent: sisyphus
 
 - 回到 1（进入下一轮 reviewers）
 
+## 本地验证（脚本直跑）
+
+```bash
+# 0) 先确保 gh 已认证（host 从 git remote origin 推断；必要时用 --hostname）
+gh auth status
+
+# 1) precheck（round 1）
+python3 "./@opencode/agents/pr_precheck.py" --pr <PR_NUMBER> --round 1
+
+# 2) context（round 1）
+python3 "./@opencode/agents/pr_context.py" --pr <PR_NUMBER> --round 1
+
+# 3) 校验：两者都必须输出单行 JSON，且 runId 必须一致
+python3 "./@opencode/agents/pr_precheck.py" --pr <PR_NUMBER> --round 1 > ./.cache/_precheck.json
+python3 "./@opencode/agents/pr_context.py" --pr <PR_NUMBER> --round 1 > ./.cache/_context.json
+python3 - <<'PY'
+import json
+p=json.load(open('./.cache/_precheck.json'))
+c=json.load(open('./.cache/_context.json'))
+assert p.get('runId') == c.get('runId'), (p.get('runId'), c.get('runId'))
+print('OK', p.get('runId'))
+PY
+
+# 4) 运行脚本相关测试（注意：pytest 把 @ 当作 argfile；必须加 ./ 并加引号）
+python3 -m pytest -q "./@opencode/agents/test_pr_review_aggregate.py"
+```
 
 ## 终止与收尾（强制）
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ranger1/dx",
-  "version": "0.1.48",
+  "version": "0.1.50",
   "type": "module",
   "license": "MIT",
   "repository": {