@ranger1/dx 0.1.34 → 0.1.36

package/@opencode/agents/claude-reviewer.md

@@ -14,20 +14,21 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
+- `runId: <string>`（必须透传，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
 
 只输出一行：
 
-`reviewFile: <filename>`
+`reviewFile: ./.cache/<file>.md`
 
 
 ## 规则
 
 - 默认已在 PR head 分支（可直接读工作区代码）
 - 可用 `git`/`gh` 只读命令获取 diff/上下文
-- 写入 reviewFile：`review-CLD-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
+- 写入 reviewFile：`./.cache/review-CLD-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
 - findings id 必须以 `CLD-` 开头
 
 ## Cache 约定（强制）

package/@opencode/agents/codex-reviewer.md

@@ -15,20 +15,21 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
+- `runId: <string>`（必须透传，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
 
 只输出一行：
 
-`reviewFile: <filename>`
+`reviewFile: ./.cache/<file>.md`
 
 
 ## 规则
 
 - 默认已在 PR head 分支（可直接读工作区代码）
 - 可用 `git`/`gh` 只读命令获取 diff/上下文
-- 写入 reviewFile：`review-CDX-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
+- 写入 reviewFile：`./.cache/review-CDX-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
 - findings id 必须以 `CDX-` 开头
 
 ## Cache 约定（强制）

package/@opencode/agents/gemini-reviewer.md

@@ -14,20 +14,21 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
+- `runId: <string>`（必须透传，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
 
 只输出一行：
 
-`reviewFile: <filename>`
+`reviewFile: ./.cache/<file>.md`
 
 
 ## 规则
 
 - 默认已在 PR head 分支（可直接读工作区代码）
 - 可用 `git`/`gh` 只读命令获取 diff/上下文
-- 写入 reviewFile：`review-GMN-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
+- 写入 reviewFile：`./.cache/review-GMN-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
 - findings id 必须以 `GMN-` 开头
 
 ## Cache 约定（强制）

package/@opencode/agents/pr-fix.md

@@ -32,6 +32,7 @@ tools:
 ### 必需输入
 
 - **PR 编号**：调用者必须在 prompt 中明确提供（如：`请修复 PR #123`）
+- **runId**：调用者必须在 prompt 中提供（必须透传，禁止自行生成）
 - **fixFile**：调用者必须在 prompt 中提供问题清单文件路径（repo 相对路径，例：`./.cache/fix-...md`）（Structured Handoff）
 
 ### 失败快速退出
@@ -126,11 +127,11 @@ Round: 2
 
 ## 输出（强制）
 
-写入：`fix-report-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
+写入：`./.cache/fix-report-pr<PR_NUMBER>-r<ROUND>-<RUN_ID>.md`
 
 最终只输出一行：
 
-`fixReportFile: <filename>`
+`fixReportFile: ./.cache/<file>.md`
 
 
 ## fixReportFile 内容格式（强制）
@@ -172,4 +173,4 @@ Rejected: <n>
 ## 输出有效性保证
 
 - fixReportFile 必须成功写入
-- stdout 只能输出一行 `fixReportFile: <filename>`
+- stdout 只能输出一行 `fixReportFile: ./.cache/<file>.md`

package/@opencode/agents/pr-review-aggregate.md

@@ -20,15 +20,15 @@ tools:
 - `PR #<number>`
 - `round: <number>`
 - `runId: <string>`
-- `contextFile: <filename>`
-- `reviewFile: <filename>`（三行，分别对应 CDX/CLD/GMN）
+- `contextFile: <path>`（例如：`./.cache/pr-context-...md`）
+- `reviewFile: <path>`（三行，分别对应 CDX/CLD/GMN，例如：`./.cache/review-...md`）
 
 ### 模式 B：发布修复评论（基于 fixReportFile）
 
 - `PR #<number>`
 - `round: <number>`
 - `runId: <string>`
-- `fixReportFile: <filename>`
+- `fixReportFile: <path>`（例如：`./.cache/fix-report-...md`）
 
 示例：
 
@@ -36,10 +36,10 @@ tools:
 PR #123
 round: 1
 runId: abcdef123456
-contextFile: pr-context-pr123-r1-abcdef123456.md
-reviewFile: review-CDX-pr123-r1-abcdef123456.md
-reviewFile: review-CLD-pr123-r1-abcdef123456.md
-reviewFile: review-GMN-pr123-r1-abcdef123456.md
+  contextFile: ./.cache/pr-context-pr123-r1-abcdef123456.md
+  reviewFile: ./.cache/review-CDX-pr123-r1-abcdef123456.md
+  reviewFile: ./.cache/review-CLD-pr123-r1-abcdef123456.md
+  reviewFile: ./.cache/review-GMN-pr123-r1-abcdef123456.md
 ```
 
 ## 执行方式（强制）

package/@opencode/agents/pr_review_aggregate.py

@@ -318,13 +318,66 @@ def _counts(findings):
     return c
 
 
-def _post_pr_comment(pr_number, body_ref):
+def _check_existing_comment(pr_number, run_id, round_num, comment_type):
+    """
+    Check if a comment with same runId/round/type already exists.
+    Returns True if duplicate exists (should skip posting).
+    
+    comment_type: "review-summary" or "fix-report" or "final-report"
+    """
+    try:
+        result = subprocess.run(
+            ["gh", "api", f"repos/:owner/:repo/issues/{pr_number}/comments", "--paginate"],
+            stdout=subprocess.PIPE,
+            stderr=subprocess.DEVNULL,
+            text=True,
+        )
+        if result.returncode != 0:
+            return False
+        
+        comments = json.loads(result.stdout or "[]")
+        
+        if comment_type == "review-summary":
+            type_header = f"## Review Summary (Round {round_num})"
+        elif comment_type == "fix-report":
+            type_header = f"## Fix Report (Round {round_num})"
+        elif comment_type == "final-report":
+            type_header = "## Final Report"
+        else:
+            return False
+        
+        run_id_pattern = f"RunId: {run_id}"
+        
+        for comment in comments:
+            body = comment.get("body", "")
+            if MARKER in body and type_header in body and run_id_pattern in body:
+                return True
+        
+        return False
+    except Exception:
+        return False
+
+
+def _post_pr_comment(pr_number, body_ref, run_id=None, round_num=None, comment_type=None):
+    """
+    Post a PR comment with idempotency check.
+    
+    If run_id, round_num, and comment_type are provided, checks for existing
+    duplicate before posting and skips if already posted.
+    
+    Returns: True if posted successfully or skipped (idempotent), False on error
+    """
     if isinstance(body_ref, Path):
         p = body_ref
     else:
         p = _resolve_ref(REPO_ROOT, CACHE_DIR, body_ref)
     if not p:
         return False
+    
+    if run_id and round_num and comment_type:
+        if _check_existing_comment(pr_number, run_id, round_num, comment_type):
+            return True
+    
     body_path = str(p)
     rc = subprocess.run(
         ["gh", "pr", "comment", str(pr_number), "--body-file", body_path],
@@ -397,6 +450,32 @@ def _render_mode_b_comment(pr_number, round_num, run_id, fix_report_md):
     return "\n".join(body)
 
 
+def _render_final_comment(pr_number, round_num, run_id, status):
+    lines = []
+    lines.append(MARKER)
+    lines.append("")
+    lines.append("## Final Report")
+    lines.append("")
+    lines.append(f"- PR: #{pr_number}")
+    lines.append(f"- Total Rounds: {round_num}")
+    lines.append(f"- RunId: {run_id}")
+    lines.append("")
+    
+    if status == "RESOLVED":
+        lines.append("### Status: ✅ All issues resolved")
+        lines.append("")
+        lines.append("All P0/P1/P2 issues from the automated review have been addressed.")
+        lines.append("The PR is ready for human review and merge.")
+    else:
+        lines.append("### Status: ⚠️ Max rounds reached")
+        lines.append("")
+        lines.append("The automated review loop has completed the maximum number of rounds (3).")
+        lines.append("Some issues may still remain. Please review the PR comments above for details.")
+    
+    lines.append("")
+    return "\n".join(lines)
+
+
 def main(argv):
     class _ArgParser(argparse.ArgumentParser):
         def error(self, message):
@@ -409,6 +488,7 @@ def main(argv):
     parser.add_argument("--context-file")
     parser.add_argument("--review-file", action="append", default=[])
     parser.add_argument("--fix-report-file")
+    parser.add_argument("--final-report")
     parser.add_argument("--duplicate-groups-json")
     parser.add_argument("--duplicate-groups-b64")
 
@@ -422,6 +502,7 @@ def main(argv):
     round_num = args.round
     run_id = str(args.run_id)
 
+    final_report = (args.final_report or "").strip() or None
     fix_report_file = (args.fix_report_file or "").strip() or None
     context_file = (args.context_file or "").strip() or None
     review_files = []
@@ -430,6 +511,17 @@ def main(argv):
         if s:
             review_files.append(s)
 
+    if final_report:
+        body = _render_final_comment(pr_number, round_num, run_id, final_report)
+        body_basename = f"review-aggregate-final-pr{pr_number}-{run_id}.md"
+        body_ref = _repo_relpath(REPO_ROOT, CACHE_DIR / body_basename)
+        _write_cache_text(body_ref, body)
+        if not _post_pr_comment(pr_number, body_ref, run_id=run_id, round_num=round_num, comment_type="final-report"):
+            _json_out({"error": "GH_PR_COMMENT_FAILED"})
+            return 1
+        _json_out({"ok": True, "final": True})
+        return 0
+
     if fix_report_file:
         fix_p = _resolve_ref(REPO_ROOT, CACHE_DIR, fix_report_file)
         if not fix_p or not fix_p.exists():
@@ -440,7 +532,7 @@ def main(argv):
         body_basename = f"review-aggregate-fix-comment-pr{pr_number}-r{round_num}-{run_id}.md"
         body_ref = _repo_relpath(REPO_ROOT, CACHE_DIR / body_basename)
         _write_cache_text(body_ref, body)
-        if not _post_pr_comment(pr_number, body_ref):
+        if not _post_pr_comment(pr_number, body_ref, run_id=run_id, round_num=round_num, comment_type="fix-report"):
             _json_out({"error": "GH_PR_COMMENT_FAILED"})
             return 1
         _json_out({"ok": True})
@@ -488,7 +580,7 @@ def main(argv):
     body_basename = f"review-aggregate-comment-pr{pr_number}-r{round_num}-{run_id}.md"
     body_ref = _repo_relpath(REPO_ROOT, CACHE_DIR / body_basename)
     _write_cache_text(body_ref, body)
-    if not _post_pr_comment(pr_number, body_ref):
+    if not _post_pr_comment(pr_number, body_ref, run_id=run_id, round_num=round_num, comment_type="review-summary"):
         _json_out({"error": "GH_PR_COMMENT_FAILED"})
         return 1
 

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

@@ -41,6 +41,13 @@ agent: sisyphus
 
 ## 循环（最多 3 轮）
 
+**⚠️ 严格串行执行要求（Critical）**:
+
+- 每个 Step 必须完成（收到返回值）后才能开始下一个 Step
+- **禁止任何步骤并行执行**（除了 Step 2 的三个 reviewer 可并行）
+- 如果任何步骤失败或超时，必须立即终止当前轮次，不能跳过或重试
+- 每个步骤的 Task 调用必须 await 返回结果，不能 fire-and-forget
+
 每轮按顺序执行：
 
 1. Task: `pr-context` **（必须先完成，不可与 Step 2 并行）**
@@ -56,35 +63,80 @@ agent: sisyphus
 - 每个 reviewer prompt 必须包含：
   - `PR #{{PR_NUMBER}}`
   - `round: <ROUND>`
-  - `contextFile: <filename>`（来自 Step 1 的输出）
+  - `runId: <RUN_ID>`（来自 Step 1 的输出，必须透传，禁止自行生成）
+  - `contextFile: ./.cache/<file>.md`（来自 Step 1 的输出）
 - reviewer 默认读 `contextFile`；必要时允许用 `git/gh` 只读命令拿 diff
 - 忽略问题：1.格式化代码引起的噪音 2.已经lint检查以外的格式问题
 - 特别关注: 逻辑、安全、性能、可维护性
 - 同时要注意 pr 前面轮次的 修复和讨论，对于已经拒绝、已修复的问题不要反复的提出
 - 同时也要注意fix的过程中有没有引入新的问题。
-- 每个 reviewer 输出：`reviewFile: <filename>`（Markdown）
+- 每个 reviewer 输出：`reviewFile: ./.cache/<file>.md`（Markdown）
 
 3. Task: `pr-review-aggregate`
 
-- prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`、`runId: <RUN_ID>`、`contextFile: <filename>`、三条 `reviewFile: <filename>`
+- prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`、`runId: <RUN_ID>`、`contextFile: ./.cache/<file>.md`、三条 `reviewFile: ./.cache/<file>.md`
 - 输出：`{"stop":true}` 或 `{"stop":false,"fixFile":"..."}`
 - 若 `stop=true`：本轮结束并退出循环
+- **唯一性约束**: 每轮只能发布一次 Review Summary；脚本内置幂等检查，重复调用不会重复发布
 
 4. Task: `pr-fix`
 
 - prompt 必须包含：
   - `PR #{{PR_NUMBER}}`
   - `round: <ROUND>`
-  - `fixFile: <filename>`
+  - `runId: <RUN_ID>`（来自 Step 1 的输出，必须透传，禁止自行生成）
+  - `fixFile: ./.cache/<file>.md`
 - 约定：`pr-fix` 对每个 findingId 单独 commit + push（一个 findingId 一个 commit），结束后再 `git push` 兜底
 
-- pr-fix 输出：`fixReportFile: <filename>`（Markdown）
+- pr-fix 输出：`fixReportFile: ./.cache/<file>.md`（Markdown）
 
 5. Task: `pr-review-aggregate`（发布修复评论）
 
-- prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`、`runId: <RUN_ID>`、`fixReportFile: <filename>`
+- prompt 必须包含：`PR #{{PR_NUMBER}}`、`round: <ROUND>`、`runId: <RUN_ID>`、`fixReportFile: ./.cache/<file>.md`
 - 输出：`{"ok":true}`
+- **唯一性约束**: 每轮只能发布一次 Fix Report；脚本内置幂等检查，重复调用不会重复发布
 
 6. 下一轮
 
 - 回到 1（进入下一轮 reviewers）
+
+
+## 终止与收尾（强制）
+
+循环结束时，必须发布一个最终评论到 PR，格式如下：
+
+### 情况 A: 所有问题已解决（stop=true）
+
+当 Step 3 返回 `{"stop":true}` 时，调用 `pr-review-aggregate` 发布收尾评论：
+
+- prompt 必须包含：
+  - `PR #{{PR_NUMBER}}`
+  - `round: <ROUND>`
+  - `runId: <RUN_ID>`
+  - `--final-report "RESOLVED"`（新增参数，表示所有问题已解决）
+
+### 情况 B: 达到最大轮次（3 轮后仍有问题）
+
+当循环完成 3 轮后仍未 stop，调用 `pr-review-aggregate` 发布收尾评论：
+
+- prompt 必须包含：
+  - `PR #{{PR_NUMBER}}`
+  - `round: 3`
+  - `runId: <RUN_ID>`
+  - `--final-report "MAX_ROUNDS_REACHED"`（新增参数，表示达到最大轮次）
+
+### 最终评论格式（由脚本生成）
+
+```markdown
+<!-- pr-review-loop-marker -->
+
+## Final Report
+
+- PR: #<PR_NUMBER>
+- Total Rounds: <N>
+- Status: ✅ All issues resolved / ⚠️ Max rounds reached (some issues may remain)
+
+### Summary
+
+[自动生成的总结]
+```

package/package.json

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