@ranger1/dx 0.1.37 → 0.1.38

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

@@ -82,10 +82,7 @@ python3 ~/.opencode/agents/gh_review_harvest.py \
 
 - 读取 `rawFile`（JSON）后，提取“建议/问题”并生成 findings：
   - 覆盖 humans + bots（不做作者白名单）。
-  - 去噪：丢弃任何 body 中包含 `<!-- pr-review-loop-marker` 的内容。
   - 忽略纯审批/无内容：如 `LGTM`、`Looks good`、`Approved` 等。
-  - 默认策略：
-    - `isResolved=true` 或 `isOutdated=true` 的 thread 在 harvest 阶段直接丢弃（不进入 rawFile，不消耗 LLM token）。
   - 分类规则（大致）：
     - P0: 明确安全漏洞/数据泄漏/资金损失/远程执行
     - P1: 逻辑 bug/权限绕过/会导致线上错误

package/@opencode/agents/gh_review_harvest.py

@@ -16,6 +16,8 @@ from datetime import datetime, timezone
 from pathlib import Path
 
 
+MARKER_SUBSTR = "<!-- pr-review-loop-marker"
+
 def _repo_root():
     try:
         p = subprocess.run(
@@ -61,6 +63,15 @@ def _run_capture(cmd):
         return 127, "", str(e)
 
 
+def _has_loop_marker(text):
+    if not text:
+        return False
+    try:
+        return MARKER_SUBSTR in str(text)
+    except Exception:
+        return False
+
+
 def _require_gh_auth():
     rc, out, err = _run_capture(["gh", "auth", "status"])
     if rc == 127:
@@ -126,6 +137,10 @@ def _flatten_threads(gql_data):
         comments_nodes = comments_conn.get("nodes") or []
         comments = []
         for c in comments_nodes:
+            body = (c or {}).get("body") or ""
+            body_text = (c or {}).get("bodyText") or ""
+            if _has_loop_marker(body) or _has_loop_marker(body_text):
+                continue
             author = (c or {}).get("author") or {}
             comments.append(
                 {
@@ -136,12 +151,15 @@ def _flatten_threads(gql_data):
                         "login": author.get("login"),
                         "type": author.get("__typename"),
                     },
-                    "body": (c or {}).get("body") or "",
-                    "bodyText": (c or {}).get("bodyText") or "",
+                    "body": body,
+                    "bodyText": body_text,
                     "createdAt": (c or {}).get("createdAt"),
                     "updatedAt": (c or {}).get("updatedAt"),
                 }
             )
+
+        if not comments:
+            continue
         threads.append(
             {
                 "id": (t or {}).get("id"),
@@ -244,6 +262,11 @@ def main(argv):
         reviews = _gh_api_json([f"repos/{owner_repo}/pulls/{pr_number}/reviews", "--paginate"])
         issue_comments = _gh_api_json([f"repos/{owner_repo}/issues/{pr_number}/comments", "--paginate"])
 
+        if isinstance(reviews, list):
+            reviews = [r for r in reviews if not _has_loop_marker((r or {}).get("body") or "")]
+        if isinstance(issue_comments, list):
+            issue_comments = [c for c in issue_comments if not _has_loop_marker((c or {}).get("body") or "")]
+
         now = datetime.now(timezone.utc).isoformat()
         payload = {
             "repo": owner_repo,

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

@@ -67,7 +67,7 @@ agent: sisyphus
   - `runId: <RUN_ID>`（来自 Step 1 的输出，必须透传，禁止自行生成）
   - `contextFile: ./.cache/<file>.md`（来自 Step 1 的输出）
 - reviewer 默认读 `contextFile`；必要时允许用 `git/gh` 只读命令拿 diff
-- 忽略问题：1.格式化代码引起的噪音 2.已经lint检查以外的格式问题
+- 忽略问题：1.格式化代码引起的噪音 2.已经lint检查以外的格式问题 3.忽略单元测试不足的问题
 - 特别关注: 逻辑、安全、性能、可维护性
 - 同时要注意 pr 前面轮次的 修复和讨论，对于已经拒绝、已修复的问题不要反复的提出
 - 同时也要注意fix的过程中有没有引入新的问题。

package/package.json

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