@ranger1/dx 0.1.48 → 0.1.49

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

@@ -14,7 +14,7 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`（必须透传，禁止自行生成）
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
@@ -40,7 +40,11 @@ tools:
    - 若你的发现 priority 比原问题高 ≥2 级（如 P3→P1, P2→P0），可以升级质疑
    - 否则不再提出
 
-判断"问题本质相同"时，比对 decision-log 中的 `essence` 字段与你发现的问题描述。
+4. **文件一致性**：
+   - 匹配 Decision Log 时，**必须检查 `file` 字段是否与当前 finding 的文件一致**。
+   - 若 decision-log 中的 `file` 与当前文件不一致（包括重命名、移动、删除），则**视为不同问题**，不进行 essence 匹配（即作为新问题处理）。
+   - 若 decision-log 条目缺少 `file` 字段，也视为不匹配。
+
 
 ### 禁止事项
 - ⛔ 不质疑已修复问题的实现方式（除非发现修复引入了新 bug）

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

@@ -15,7 +15,7 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`（必须透传，禁止自行生成）
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
@@ -41,7 +41,11 @@ tools:
    - 若你的发现 priority 比原问题高 ≥2 级（如 P3→P1, P2→P0），可以升级质疑
    - 否则不再提出
 
-判断"问题本质相同"时，比对 decision-log 中的 `essence` 字段与你发现的问题描述。
+4. **文件一致性**：
+   - 匹配 Decision Log 时，**必须检查 `file` 字段是否与当前 finding 的文件一致**。
+   - 若 decision-log 中的 `file` 与当前文件不一致（包括重命名、移动、删除），则**视为不同问题**，不进行 essence 匹配（即作为新问题处理）。
+   - 若 decision-log 条目缺少 `file` 字段，也视为不匹配。
+
 
 ### 禁止事项
 - ⛔ 不质疑已修复问题的实现方式（除非发现修复引入了新 bug）

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

@@ -14,7 +14,7 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`（必须透传，禁止自行生成）
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `contextFile: <filename>`
 
 ## 输出（强制）
@@ -40,7 +40,11 @@ tools:
    - 若你的发现 priority 比原问题高 ≥2 级（如 P3→P1, P2→P0），可以升级质疑
    - 否则不再提出
 
-判断"问题本质相同"时，比对 decision-log 中的 `essence` 字段与你发现的问题描述。
+4. **文件一致性**：
+   - 匹配 Decision Log 时，**必须检查 `file` 字段是否与当前 finding 的文件一致**。
+   - 若 decision-log 中的 `file` 与当前文件不一致（包括重命名、移动、删除），则**视为不同问题**，不进行 essence 匹配（即作为新问题处理）。
+   - 若 decision-log 条目缺少 `file` 字段，也视为不匹配。
+
 
 ### 禁止事项
 - ⛔ 不质疑已修复问题的实现方式（除非发现修复引入了新 bug）

package/@opencode/agents/gh-thread-reviewer.md

@@ -17,7 +17,7 @@ Harvest all GitHub PR review feedback (humans + bots, including Copilot) and nor
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`（必须透传，禁止自行生成）
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `contextFile: <filename>`
 
 ## Cache 约定（强制）
@@ -110,6 +110,11 @@ python3 ~/.opencode/agents/gh_review_harvest.py \
    - 若你的发现 priority 比原问题高 ≥2 级（如 P3→P1, P2→P0），可以升级质疑
    - 否则不再提出
 
+3. **文件一致性**：
+   - 匹配 Decision Log 时，**必须检查 `file` 字段是否与当前 finding 的文件一致**。
+   - 若 decision-log 中的 `file` 与当前文件不一致（包括重命名、移动、删除），则**视为不同问题**，不进行 essence 匹配（即作为新问题处理）。
+   - 若 decision-log 条目缺少 `file` 字段，也视为不匹配。
+
 判断"问题本质相同"时，比对 decision-log 中的 `essence` 字段与你发现的问题描述。
 
 ### 禁止事项

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

@@ -18,6 +18,16 @@ tools:
 - PR 编号（如：`PR #123` 或 `prNumber: 123`）
 - round（如：`round: 1`；无则默认 1）
 
+## 唯一标识 runId（强制）
+
+- 脚本必须生成全局唯一标识 `runId`：`<PR>-<ROUND>-<HEAD_SHORT>`
+- 其中：
+  - `<PR>`：PR 编号
+  - `<ROUND>`：当前轮次
+  - `<HEAD_SHORT>`：`headOid` 的前 7 位（git rev-parse --short HEAD）
+- `runId` 必须包含在返回的 JSON 中，供后续步骤使用。
+
+
 ## 输出（强制）
 
 脚本会写入项目内 `./.cache/`，stdout 只输出单一 JSON（可 `JSON.parse()`）。
@@ -25,6 +35,8 @@ tools:
 ## Cache 约定（强制）
 
 - 缓存目录固定为 `./.cache/`；交接一律传 `./.cache/<file>`（repo 相对路径），禁止 basename-only（如 `foo.md`）。
+- 文件命名：`./.cache/pr-context-pr<PR>-r<ROUND>-<RUN_ID>.md`
+- `RUN_ID` 格式必须为 `<PR>-<ROUND>-<HEAD_SHORT>`
 
 ## 调用脚本（强制）
 

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

@@ -32,7 +32,7 @@ tools:
 ### 必需输入
 
 - **PR 编号**：调用者必须在 prompt 中明确提供（如：`请修复 PR #123`）
-- **runId**：调用者必须在 prompt 中提供（必须透传，禁止自行生成）
+- **runId**：调用者必须在 prompt 中提供（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - **fixFile**：调用者必须在 prompt 中提供问题清单文件路径（repo 相对路径，例：`./.cache/fix-...md`）（Structured Handoff）
 
 ### 失败快速退出
@@ -154,12 +154,14 @@ PR: <PR_NUMBER>
 ### Fixed
 
 - id: <FINDING_ID>
+  file: <FILE_PATH>
   commit: <SHA>
   essence: <问题本质的一句话描述>
 
 ### Rejected
 
 - id: <FINDING_ID>
+  file: <FILE_PATH>
   priority: <P0|P1|P2|P3>
   reason: <拒绝原因>
   essence: <问题本质的一句话描述>
@@ -170,6 +172,8 @@ PR: <PR_NUMBER>
 - 如果文件不存在：创建新文件，包含 `# Decision Log` 头、`PR: <PR_NUMBER>` 字段，以及第一个 `## Round <ROUND>` 段落
 - 如果文件存在：追加新的 `## Round <ROUND>` 段落到文件末尾
 - **禁止删除或覆盖历史轮次的记录**
+- **file 字段**：必须记录问题所在的文件路径（repo 相对路径）。
+  - 对于 `pr-precheck` 产生的修复，`file` 字段可填 `__precheck__`。
 
 ### essence 字段要求
 
@@ -178,6 +182,7 @@ essence 是问题本质的一句话描述，用于后续轮次的智能匹配和
 - 简洁性：≤ 50 字
 - 问题导向：描述问题核心（而非具体代码位置、文件行号）
 - 可匹配性：后续轮次的 reviewer 能通过关键词匹配识别该问题
+- **文件强绑定**：必须假设问题与当前文件强绑定（若文件重命名，视为不同问题）
 
 **示例对比：**
 

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

@@ -16,13 +16,14 @@ tools:
 ## 输入（prompt 必须包含）
 
 - `PR #<number>`
+- `round: <number>`（默认 1）
 
 ## 一键脚本
 
 脚本位置：`~/.opencode/agents/pr_precheck.py`
 
 ```bash
-python3 ~/.opencode/agents/pr_precheck.py <PR_NUMBER>
+python3 ~/.opencode/agents/pr_precheck.py --pr <PR_NUMBER> --round <ROUND>
 ```
 
 ## 脚本输出处理（强制）
@@ -30,6 +31,7 @@ python3 ~/.opencode/agents/pr_precheck.py <PR_NUMBER>
 - 脚本 stdout 只会输出**单一一行 JSON**（可 `JSON.parse()`）。
 - **成功时**：你的最终输出必须是**脚本 stdout 的那一行 JSON 原样内容**。
   - 典型返回：`{"ok":true}` 或 `{"ok":false,"fixFile":"..."}`
+  - **重要**：如果返回 `fixFile`，请使用基于 `headOid` 的标准 runId（`<PR>-<ROUND>-<HEAD_SHORT>`）来命名文件。
   - 禁止：解释/分析/补充文字
   - 禁止：代码块（```）
   - 禁止：前后空行
@@ -83,5 +85,5 @@ git commit -m "chore(pr #<PR_NUMBER>): resolve merge conflicts"
 git push
 
 # 5) 重新运行预检脚本
-python3 ~/.opencode/agents/pr_precheck.py <PR_NUMBER>
+python3 ~/.opencode/agents/pr_precheck.py --pr <PR_NUMBER> --round <ROUND>
 ```

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

@@ -19,7 +19,7 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `contextFile: <path>`（例如：`./.cache/pr-context-...md`）
 - `reviewFile: <path>`（多行，1+ 条；例如：`./.cache/review-...md`）
 
@@ -27,7 +27,7 @@ tools:
 
 - `PR #<number>`
 - `round: <number>`
-- `runId: <string>`
+- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
 - `fixReportFile: <path>`（例如：`./.cache/fix-report-...md`）
 
 示例：
@@ -35,11 +35,11 @@ tools:
 ```text
 PR #123
 round: 1
-runId: abcdef123456
-  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
+runId: 123-1-a1b2c3d
+  contextFile: ./.cache/pr-context-pr123-r1-123-1-a1b2c3d.md
+  reviewFile: ./.cache/review-CDX-pr123-r1-123-1-a1b2c3d.md
+  reviewFile: ./.cache/review-CLD-pr123-r1-123-1-a1b2c3d.md
+  reviewFile: ./.cache/review-GMN-pr123-r1-123-1-a1b2c3d.md
 ```
 
 ## 执行方式（强制）
@@ -65,11 +65,19 @@ runId: abcdef123456
 
 如果 decision-log（`./.cache/decision-log-pr<PR_NUMBER>.md`）存在，你需要基于 LLM 判断每个新 finding 与已决策问题的本质是否相同，从而生成 **escalation_groups** 参数。
 
+**匹配原则**：
+- **Essence 匹配**：对比 `essence` 字段与新 finding 的问题本质。
+- **文件强绑定**：仅当 decision-log 条目的 `file` 与新 finding 的 `file` **完全一致**时才进行匹配。
+  - 若文件被重命名/删除/拆分，视为不同问题（为了稳定性，不处理复杂的 rename 映射）。
+  - 若 decision-log 条目缺少 `file` 字段（旧数据），则跳过匹配（视为不相关）。
+
 **流程**：
 
-1. 读取 decision-log，提取已 rejected 问题的 `essence` 字段
-2. 逐个新 finding，与所有已 rejected 问题的 essence 做语义比对（使用 LLM）
-3. 判断是否"问题本质相同"（即便表述不同）
+1. 读取 decision-log，提取已 rejected 问题的 `essence` 和 `file` 字段
+2. 逐个新 finding，**先检查 file 是否匹配**
+   - 若 file 不匹配 → 视为 New Issue
+   - 若 file 匹配 → 继续对比 essence
+3. 若 essence 也匹配（"问题本质相同"）：
 4. 收集可升级的问题（重新质疑阈值）：
    - **升级阈值**：优先级差距 ≥ 2 级
    - 例如：已 rejected P3 but finding 为 P1 → 可升级质疑

package/@opencode/agents/pr_context.py

@@ -6,7 +6,6 @@
 # - Prints exactly one JSON object to stdout
 
 import argparse
-import hashlib
 import json
 import os
 import re
@@ -183,20 +182,28 @@ def main(argv):
     pr_number = int(args.pr)
     round_num = int(args.round)
 
+    def _json_err(error_code, extra=None):
+        obj = {"error": error_code, "prNumber": pr_number, "round": round_num}
+        if isinstance(extra, dict) and extra:
+            obj.update(extra)
+        _json_out(obj)
+
     # Preconditions: be in a git repo and gh is authenticated.
     rc, out, _ = _run_capture(["git", "rev-parse", "--is-inside-work-tree"])
     if rc != 0 or out.strip() != "true":
-        _json_out({"error": "NOT_A_GIT_REPO"})
+        _json_err("NOT_A_GIT_REPO")
         return 1
 
     host = _detect_git_remote_host() or "github.com"
     rc, gh_out, gh_err = _run_capture(["gh", "auth", "status", "--hostname", host])
     if rc == 127:
-        _json_out({
-            "error": "GH_CLI_NOT_FOUND",
-            "detail": "gh not found in PATH",
-            "suggestion": "Install GitHub CLI: https://cli.github.com/",
-        })
+        _json_err(
+            "GH_CLI_NOT_FOUND",
+            {
+                "detail": "gh not found in PATH",
+                "suggestion": "Install GitHub CLI: https://cli.github.com/",
+            },
+        )
         return 1
     if rc != 0:
         # If host detection is wrong, a global check might still succeed.
@@ -207,44 +214,51 @@ def main(argv):
             detail = (gh_err or gh_out or "").strip()
             if len(detail) > 4000:
                 detail = detail[-4000:]
-            _json_out({
-                "error": "GH_NOT_AUTHENTICATED",
-                "host": host,
-                "detail": detail,
-                "suggestion": f"Run: gh auth login --hostname {host}",
-            })
+            _json_err(
+                "GH_NOT_AUTHENTICATED",
+                {
+                    "host": host,
+                    "detail": detail,
+                    "suggestion": f"Run: gh auth login --hostname {host}",
+                },
+            )
             return 1
 
     rc, owner_repo, _ = _run_capture(["gh", "repo", "view", "--json", "nameWithOwner", "--jq", ".nameWithOwner"])
     owner_repo = owner_repo.strip() if rc == 0 else ""
     if not owner_repo:
-        _json_out({"error": "REPO_NOT_FOUND"})
+        _json_err("REPO_NOT_FOUND")
         return 1
 
     fields = "number,url,title,body,isDraft,labels,baseRefName,headRefName,baseRefOid,headRefOid,comments"
     rc, pr_json, _ = _run_capture(["gh", "pr", "view", str(pr_number), "--repo", owner_repo, "--json", fields])
     if rc != 0:
-        _json_out({"error": "PR_NOT_FOUND_OR_NO_ACCESS"})
+        _json_err("PR_NOT_FOUND_OR_NO_ACCESS")
         return 1
     try:
         pr = json.loads(pr_json)
     except Exception:
-        _json_out({"error": "PR_NOT_FOUND_OR_NO_ACCESS"})
+        _json_err("PR_NOT_FOUND_OR_NO_ACCESS")
         return 1
 
     head_oid = (pr.get("headRefOid") or "").strip()
     base_oid = (pr.get("baseRefOid") or "").strip()
     base_ref = (pr.get("baseRefName") or "").strip()
+
+    if not head_oid:
+        _json_err("PR_HEAD_OID_NOT_FOUND")
+        return 1
+
+    head_short = head_oid[:7]
     if not base_ref:
         base_ref = _gh_default_branch(owner_repo)
     if not base_ref and not base_oid:
-        _json_out({"error": "PR_BASE_REF_NOT_FOUND"})
+        _json_err("PR_BASE_REF_NOT_FOUND")
         return 1
     head_ref = (pr.get("headRefName") or "").strip()
     url = (pr.get("url") or "").strip()
 
-    seed = f"{pr_number}:{round_num}:{head_oid}".encode("utf-8")
-    run_id = hashlib.sha1(seed).hexdigest()[:12]
+    run_id = f"{pr_number}-{round_num}-{head_short}"
 
     if base_ref:
         _git_fetch_origin(base_ref)
@@ -277,6 +291,7 @@ def main(argv):
         fp.write(f"- RunId: {run_id}\n")
         fp.write(f"- Base: {base_ref}\n")
         fp.write(f"- Head: {head_ref}\n")
+        fp.write(f"- HeadShort: {head_short}\n")
         fp.write(f"- HeadOid: {head_oid}\n")
         fp.write(f"- Draft: {pr.get('isDraft')}\n")
         fp.write(f"- Labels: {', '.join(labels) if labels else '(none)'}\n")
@@ -317,6 +332,7 @@ def main(argv):
             "runId": run_id,
             "repo": {"nameWithOwner": owner_repo},
             "headOid": head_oid,
+            "headShort": head_short,
             "existingMarkerCount": marker_count,
             # Handoff should be repo-relative path so downstream agents can read it directly.
             "contextFile": _repo_relpath(REPO_ROOT, context_path),