@ranger1/dx 0.1.90 → 0.1.92

package/codex/skills/pr-review-loop/references/agents/pr-review-aggregate.md

@@ -1,188 +0,0 @@
----
-description: aggregate PR reviews + create fix file
-mode: subagent
-model: openai/gpt-5.3-codex
-temperature: 0.1
-tools:
-  bash: true
----
-
-# PR Review Aggregator
-
-## Cache 约定（强制）
-
-- 缓存目录固定为 `./.cache/`；交接一律传 `./.cache/<file>`（repo 相对路径），禁止 basename-only（如 `foo.md`）。
-
-## 输入（两种模式）
-
-### 模式 A：评审聚合 + 生成 fixFile + 发布评审评论
-
-- `PR #<number>`
-- `round: <number>`
-- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
-- `contextFile: <path>`（例如：`./.cache/pr-context-...md`）
-- `reviewFile: <path>`（多行，1+ 条；例如：`./.cache/review-...md`）
-
-### 模式 B：发布修复评论（基于 fixReportFile）
-
-- `PR #<number>`
-- `round: <number>`
-- `runId: <string>`（必须透传，格式 `<PR>-<ROUND>-<HEAD_SHORT>`，禁止自行生成）
-- `fixReportFile: <path>`（例如：`./.cache/fix-report-...md`）
-
-示例：
-
-```text
-PR #123
-round: 1
-runId: 123-1-a1b2c3d
-  contextFile: ./.cache/pr-context-pr123-r1-123-1-a1b2c3d.md
-  reviewFile: ./.cache/review-SEC-pr123-r1-123-1-a1b2c3d.md
-  reviewFile: ./.cache/review-PERF-pr123-r1-123-1-a1b2c3d.md
-  reviewFile: ./.cache/review-MAINT-pr123-r1-123-1-a1b2c3d.md
-  reviewFile: ./.cache/review-BIZ-pr123-r1-123-1-a1b2c3d.md
-```
-
-## 执行方式（强制）
-
-所有确定性工作（发评论/生成 fixFile/输出 JSON）都由 `${CODEX_HOME:-$HOME/.codex}/skills/pr-review-loop/scripts/pr_review_aggregate.py` 完成。
-
-你只做三件事：
-
-1) 在模式 A 里读取 `contextFile`、所有 `reviewFile`，由大模型做最终语义裁决。
-2) 产出一份**结构化聚合结果 JSON**，再把它作为参数传给脚本（不落盘）。
-3) 调用脚本后，把脚本 stdout 的 JSON **原样返回**给调用者（不做解释/分析）。
-
-## 最终裁决边界（强制）
-
-- “是否存在问题”“哪些问题必须修”“是否可以 stop” 都由你基于 `reviewFile` 语义判断。
-- 脚本**不再**根据 reviewer 文本自行推断 `P0/P1`、`stop` 或“有没有问题”。
-- 如果 reviewer 文本里缺少关键信息，导致你无法可靠裁决，必须返回错误；禁止把不确定性伪装成 `stop=true`。
-
-## 聚合结果 JSON（模式 A 必须生成）
-
-你需要基于 `contextFile`、所有 `reviewFile`，先完成这些判断：
-
-- 去重：本质相同的问题只保留一条
-- decision-log 过滤：已 fixed 的问题过滤；已 rejected 的问题仅在满足升级质疑条件时保留
-- 分级：把保留的问题分成 `mustFixFindings` 和 `optionalFindings`
-- 终止判断：仅当你确认没有 `mustFixFindings` 时，才可令 `stop=true`
-
-然后输出一行 JSON，结构固定如下：
-
-```json
-{
-  "stop": false,
-  "mustFixFindings": [
-    {
-      "id": "SEC-001",
-      "priority": "P1",
-      "category": "bug",
-      "file": "apps/api/src/service.ts",
-      "line": "10",
-      "title": "标题",
-      "description": "描述",
-      "suggestion": "建议"
-    }
-  ],
-  "optionalFindings": [
-    {
-      "id": "STY-002",
-      "priority": "P3",
-      "category": "quality",
-      "file": "apps/web/src/page.tsx",
-      "line": "22",
-      "title": "标题",
-      "description": "描述",
-      "suggestion": "建议"
-    }
-  ]
-}
-```
-
-强约束：
-
-- 必须是一行 JSON，不要代码块，不要解释。
-- `stop=true` 时，`mustFixFindings` 必须为空数组。
-- `stop=false` 时，`mustFixFindings` 必须至少有一条。
-- 每个 finding 必须包含这些字段且非空：`id`、`priority`、`category`、`file`、`line`、`title`、`description`、`suggestion`
-- `priority` 只能是 `P0` / `P1` / `P2` / `P3`
-- `mustFixFindings` 只允许 `P0` / `P1`
-- `optionalFindings` 只允许 `P2` / `P3`
-
-## 重复分组与 decision-log（仅作为你的思考步骤）
-
-你仍然需要基于所有 `reviewFile` 内容判断重复问题和 decision-log 匹配，但这些中间结果**不再**单独传给脚本；它们只体现在最终的聚合结果 JSON 里。
-
-## 智能匹配（仅在模式 A + decision-log 存在时）
-
-如果 decision-log（`./.cache/decision-log-pr<PR_NUMBER>.md`）存在，你需要基于 LLM 判断每个新 finding 与已决策问题的本质是否相同，并把判断结果体现在最终聚合结果里。
-
-**匹配原则**：
-- **Essence 匹配**：对比 `essence` 字段与新 finding 的问题本质。
-- **文件强绑定**：仅当 decision-log 条目的 `file` 与新 finding 的 `file` **完全一致**时才进行匹配。
-  - 若文件被重命名/删除/拆分，视为不同问题（为了稳定性，不处理复杂的 rename 映射）。
-  - 若 decision-log 条目缺少 `file` 字段（旧数据），则跳过匹配（视为不相关）。
-
-**流程**：
-
-1. 读取 decision-log，提取已 rejected 问题的 `essence` 和 `file` 字段
-2. 逐个新 finding，**先检查 file 是否匹配**
-   - 若 file 不匹配 → 视为 New Issue
-   - 若 file 匹配 → 继续对比 essence
-3. 若 essence 也匹配（"问题本质相同"）：
-4. 收集可升级的问题（重新质疑阈值）：
-   - **升级阈值**：优先级差距 ≥ 2 级
-   - 例如：已 rejected P3 but finding 为 P1 → 可升级质疑
-   - 例如：已 rejected P2 but finding 为 P0 → 可升级质疑
-   - 例如：已 rejected P2 but finding 为 P1 → 不升级（仅差 1 级）
-5. 只把满足条件的升级问题保留到最终 `mustFixFindings` 或 `optionalFindings`；其余继续按 rejected 过滤。
-
-## 调用脚本（强制）
-
-模式 A（带 reviewFile + 聚合结果）：
-
-```bash
-python3 "${CODEX_HOME:-$HOME/.codex}/skills/pr-review-loop/scripts/pr_review_aggregate.py" \
-  --pr <PR_NUMBER> \
-  --round <ROUND> \
-  --run-id <RUN_ID> \
-  --context-file <CONTEXT_FILE> \
-  --review-file <REVIEW_FILE_1> \
-  --review-file <REVIEW_FILE_2> \
-  --review-file <REVIEW_FILE_3> \
-  --aggregate-result-b64 <BASE64_JSON>
-```
-
-**参数说明**：
-
-- `--aggregate-result-b64`：base64 编码的聚合结果 JSON
-
-模式 B（带 fixReportFile）：
-
-```bash
-python3 "${CODEX_HOME:-$HOME/.codex}/skills/pr-review-loop/scripts/pr_review_aggregate.py" \
-  --pr <PR_NUMBER> \
-  --round <ROUND> \
-  --run-id <RUN_ID> \
-  --fix-report-file <FIX_REPORT_FILE>
-```
-
-## 脚本输出处理（强制）
-
-- 脚本 stdout 只会输出**单一一行 JSON**（可 `JSON.parse()`）。
-- **成功时**：你的最终输出必须是**脚本 stdout 的那一行 JSON 原样内容**。
-  - 典型返回：`{"stop":true}` 或 `{"stop":false,"fixFile":"..."}` 或 `{"ok":true}`
-  - 禁止：解释/分析/补充文字
-  - 禁止：代码块（```）
-  - 禁止：前后空行
-- **失败/异常时**：
-  - 若脚本 stdout 已输出合法 JSON（包含 `error` 或其他字段）→ 仍然**原样返回该 JSON**。
-  - 若脚本未输出合法 JSON / 退出异常 → 仅返回一行 JSON：`{"error":"PR_REVIEW_AGGREGATE_AGENT_FAILED"}`（必要时可加 `detail` 字段）。
-
-## fixFile 结构（补充说明）
-
-脚本在模式 A 下根据你提供的聚合结果生成 fixFile，分为两段：
-
-- `## IssuesToFix`：只包含 P0/P1（必须修）
-- `## OptionalIssues`：包含 P2/P3（由 fixer 自主决定是否修复，或拒绝并说明原因）

package/codex/skills/pr-review-loop/references/skill-layout.md

@@ -1,25 +0,0 @@
-# pr-review-loop 技能目录说明
-
-本目录用于在当前仓库内运行 PR 审核闭环，不依赖历史外置目录。
-
-## 目录结构
-
-- `SKILL.md`：技能入口 + 主编排（唯一真值源）
-- `agents/openai.yaml`：技能 UI 元数据
-- `references/agents/*.md`：子代理输入输出契约
-- `scripts/*.py`：确定性脚本（context/harvest/aggregate）
-
-## 快速验证
-
-```bash
-python3 -m pytest -q "${CODEX_HOME:-$HOME/.codex}/skills/pr-review-loop/scripts/test_pr_review_aggregate.py"
-```
-
-## 核心约束
-
-- 缓存统一使用 `./.cache/`
-- `runId` 必须透传，禁止下游重算
-- reviewer 可并行，其它步骤严格串行
-- 修复阶段必须调用 `fixer`，编排器不得直接修代码
-- 错误处理采用“分级重试优先”，不是“见 error 立刻终止”
-- 需要联网步骤的角色显式配置 `network_access = true`

package/codex/skills/pr-review-loop/scripts/gh_review_harvest.py

@@ -1,292 +0,0 @@
-#!/usr/bin/env python3
-
-# Deterministic GitHub PR review harvester.
-#
-# - Fetches inline review threads via GraphQL (reviewThreads) with pagination.
-# - Fetches PR reviews and PR issue comments via REST (gh api) with pagination.
-# - Writes a raw JSON file into project cache: ./.cache/
-# - Prints exactly one JSON object to stdout: {"rawFile":"./.cache/...json"}
-
-import argparse
-import json
-import os
-import subprocess
-import sys
-from datetime import datetime, timezone
-from pathlib import Path
-
-
-MARKER_SUBSTR = "<!-- pr-review-loop-marker"
-
-def _repo_root():
-    try:
-        p = subprocess.run(
-            ["git", "rev-parse", "--show-toplevel"],
-            stdout=subprocess.PIPE,
-            stderr=subprocess.DEVNULL,
-            text=True,
-        )
-        out = (p.stdout or "").strip()
-        if p.returncode == 0 and out:
-            return Path(out)
-    except Exception:
-        pass
-    return Path.cwd()
-
-
-def _cache_dir(repo_root):
-    return (repo_root / ".cache").resolve()
-
-
-def _repo_relpath(repo_root, p):
-    try:
-        rel = p.resolve().relative_to(repo_root.resolve())
-        return "./" + rel.as_posix()
-    except Exception:
-        return os.path.basename(str(p))
-
-
-REPO_ROOT = _repo_root()
-CACHE_DIR = _cache_dir(REPO_ROOT)
-
-
-def _json_out(obj):
-    sys.stdout.write(json.dumps(obj, ensure_ascii=True))
-    sys.stdout.write("\n")
-
-
-def _run_capture(cmd):
-    try:
-        p = subprocess.run(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE, text=True)
-        return p.returncode, p.stdout, p.stderr
-    except FileNotFoundError as e:
-        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:
-        return False, "GH_CLI_NOT_FOUND", "gh not found in PATH"
-    if rc != 0:
-        detail = (err or out or "").strip()
-        if len(detail) > 4000:
-            detail = detail[-4000:]
-        return False, "GH_NOT_AUTHENTICATED", detail
-    return True, None, None
-
-
-def _resolve_owner_repo(explicit_repo):
-    if explicit_repo:
-        s = str(explicit_repo).strip()
-        if s and "/" in s:
-            return s
-    rc, out, _ = _run_capture(["gh", "repo", "view", "--json", "nameWithOwner", "--jq", ".nameWithOwner"])
-    owner_repo = out.strip() if rc == 0 else ""
-    return owner_repo or None
-
-
-def _gh_api_json(args):
-    rc, out, err = _run_capture(["gh", "api"] + args)
-    if rc != 0:
-        raise RuntimeError(f"GH_API_FAILED: {(err or out or '').strip()}")
-    try:
-        return json.loads(out or "null")
-    except Exception:
-        raise RuntimeError("GH_API_JSON_PARSE_FAILED")
-
-
-def _gh_api_graphql(query, variables):
-    cmd = ["gh", "api", "graphql", "-f", f"query={query}"]
-    for k, v in (variables or {}).items():
-        if isinstance(v, int):
-            cmd.extend(["-F", f"{k}={v}"])
-        elif v is None:
-            cmd.extend(["-f", f"{k}="])
-        else:
-            cmd.extend(["-f", f"{k}={v}"])
-
-    rc, out, err = _run_capture(cmd)
-    if rc != 0:
-        raise RuntimeError(f"GH_GRAPHQL_FAILED: {(err or out or '').strip()}")
-    try:
-        return json.loads(out or "null")
-    except Exception:
-        raise RuntimeError("GH_GRAPHQL_JSON_PARSE_FAILED")
-
-
-def _flatten_threads(gql_data):
-    threads = []
-    pr = (((gql_data or {}).get("data") or {}).get("repository") or {}).get("pullRequest") or {}
-    conn = pr.get("reviewThreads") or {}
-    nodes = conn.get("nodes") or []
-    for t in nodes:
-        is_resolved = bool((t or {}).get("isResolved"))
-        is_outdated = bool((t or {}).get("isOutdated"))
-        if is_resolved or is_outdated:
-            continue
-        comments_conn = (t or {}).get("comments") or {}
-        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(
-                {
-                    "id": (c or {}).get("id"),
-                    "databaseId": (c or {}).get("databaseId"),
-                    "url": (c or {}).get("url"),
-                    "author": {
-                        "login": author.get("login"),
-                        "type": author.get("__typename"),
-                    },
-                    "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"),
-                "isResolved": False,
-                "isOutdated": False,
-                "path": (t or {}).get("path"),
-                "line": (t or {}).get("line"),
-                "originalLine": (t or {}).get("originalLine"),
-                "startLine": (t or {}).get("startLine"),
-                "originalStartLine": (t or {}).get("originalStartLine"),
-                "comments": comments,
-            }
-        )
-
-    page_info = conn.get("pageInfo") or {}
-    return threads, {
-        "hasNextPage": bool(page_info.get("hasNextPage")),
-        "endCursor": page_info.get("endCursor"),
-    }
-
-
-def _fetch_all_review_threads(owner, repo, pr_number):
-    query = (
-        "query($owner:String!,$repo:String!,$prNumber:Int!,$after:String){"
-        "repository(owner:$owner,name:$repo){"
-        "pullRequest(number:$prNumber){"
-        "reviewThreads(first:100,after:$after){"
-        "pageInfo{hasNextPage endCursor}"
-        "nodes{"
-        "id isResolved isOutdated path line originalLine startLine originalStartLine "
-        "comments(first:100){nodes{"
-        "id databaseId url body bodyText createdAt updatedAt author{login __typename}"
-        "}}"
-        "}"
-        "}"
-        "}"
-        "}"
-        "}"
-    )
-
-    after = None
-    all_threads = []
-    while True:
-        data = _gh_api_graphql(query, {"owner": owner, "repo": repo, "prNumber": pr_number, "after": after})
-        threads, page = _flatten_threads(data)
-        all_threads.extend(threads)
-        if not page.get("hasNextPage"):
-            break
-        after = page.get("endCursor")
-        if not after:
-            break
-    return all_threads
-
-
-def main(argv):
-    class _ArgParser(argparse.ArgumentParser):
-        def error(self, message):
-            raise ValueError(message)
-
-    parser = _ArgParser(add_help=False)
-    parser.add_argument("--pr", type=int, required=True)
-    parser.add_argument("--round", type=int, default=1)
-    parser.add_argument("--run-id", required=True)
-    parser.add_argument("--repo")
-
-    try:
-        args = parser.parse_args(argv)
-    except ValueError:
-        _json_out({"error": "INVALID_ARGS"})
-        return 2
-
-    ok, code, detail = _require_gh_auth()
-    if not ok:
-        _json_out({"error": code, "detail": detail})
-        return 1
-
-    owner_repo = _resolve_owner_repo(args.repo)
-    if not owner_repo:
-        _json_out({"error": "REPO_NOT_FOUND"})
-        return 1
-    if "/" not in owner_repo:
-        _json_out({"error": "INVALID_REPO"})
-        return 1
-
-    owner, repo = owner_repo.split("/", 1)
-    pr_number = int(args.pr)
-    round_num = int(args.round)
-    run_id = str(args.run_id).strip()
-    if not run_id:
-        _json_out({"error": "MISSING_RUN_ID"})
-        return 1
-
-    CACHE_DIR.mkdir(parents=True, exist_ok=True)
-    raw_basename = f"gh-review-raw-pr{pr_number}-r{round_num}-{run_id}.json"
-    raw_path = CACHE_DIR / raw_basename
-
-    try:
-        threads = _fetch_all_review_threads(owner, repo, pr_number)
-
-        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,
-            "pr": pr_number,
-            "round": round_num,
-            "runId": run_id,
-            "generatedAt": now,
-            "reviewThreads": threads,
-            "reviews": reviews if isinstance(reviews, list) else [],
-            "issueComments": issue_comments if isinstance(issue_comments, list) else [],
-        }
-
-        raw_path.write_text(json.dumps(payload, ensure_ascii=True), encoding="utf-8", newline="\n")
-    except Exception as e:
-        _json_out({"error": "HARVEST_FAILED", "detail": str(e)[:800]})
-        return 1
-
-    _json_out({"rawFile": _repo_relpath(REPO_ROOT, raw_path)})
-    return 0
-
-
-if __name__ == "__main__":
-    raise SystemExit(main(sys.argv[1:]))