Skip to content

docs(architecture): 审查能力对标 PR-Agent / Qodo 的缺失能力清单 - #120

Merged
Kucell merged 1 commit into
mainfrom
docs/review-capability-gap-vs-pr-agent
Oct 23, 2026
Merged

Kucell merged 1 commit into
mainfrom
docs/review-capability-gap-vs-pr-agent

Conversation

@Kucell

@Kucell Kucell commented Oct 23, 2026

Copy link
Copy Markdown
Owner

背景

仓库既有 docs/architecture/harness-review-and-runtime-evidence-design.md §4 评估过「不把 PR-Agent 作为 core 依赖」,但当时的对标对象是 PR-Agent v1(Qodo Merge)。PR-Agent 现已并入 Qodo,v2 能力面大出一个量级,该评估的第 4 条论据需要重新裁决。

内容

以 Qodo v2 官方文档为基准、实读本仓代码,给出分级缺失清单:

  • A 类应自建 9 项(PR review 回写、finding 生命周期、行级锚定、blast radius、confidence/relevance、PR 摘要、多 reviewer 并行 + judge、规则挖掘、采纳率统计)
  • B 类部分自建 4 项(跨仓审查、PR Triage、多 Git provider、PR 历史索引)
  • C 类不追 4 项(Portal/企业治理、托管多租户、本地化部署、设为 required check)

核心结论

  1. cortex 强项在判定(schema 契约、finding↔changeset 归属、幻觉拦截、14 种机械断言、双候选区分度硬约束)——Qodo 无对等物。
  2. cortex 硬缺口在闭环:finding 产出了但没有出口(无 PR 回写、无生命周期、无采纳率)。
  3. 最小切口是单点的:给 vcs-pr 补 PR review 回写与 diff 读取。它同时卡住 Q10/Q12/Q15/Q16 四条能力链的共同上游。

既有 §4 的「不集成进 core」结论维持,但第 4 条「能力上无增量」替换为「Qodo v2 中与 cortex 定位相容的部分应自建」。

验证

  • 三道门禁全绿:check-l1-distribution / check-l3-boundary / check-template-parity
  • 文档内相对链接逐条核对存在
  • 承重数字实读核对:断言类型 14(agent-review-benchmark-runner.cjs:234-305)、package.json 依赖为空、L1 code-review 仅在 _shared 层

边界

  • cortex 侧结论均为代码实读,带文件路径。
  • Qodo 侧结论来自官方文档 12 个 .md 端点原文;Q4/Q6/Q7 标注 Beta、Q14 标注 Research Preview,对标时不应把 Beta 能力当作既成差距。
  • 未实测 knowledge-lint / architecture-guard 真实运行输出。

纯文档变更,不触碰 .agent gitlink,不改任何实现文件。

以 Qodo v2(PR-Agent 继任者)为基准,实读本仓代码后给出分级清单:
A 类应自建 9 项、B 类部分自建 4 项、C 类不追 4 项。

核心结论:既有 §4「不对接 PR-Agent」结论成立,但其第 4 条
「能力上无增量」已失效——Qodo v2 能力面比评估时的 v1 大一个量级。

最小切口是给 vcs-pr 补 PR review 回写与 diff 读取,这是 finding
生命周期、PR 摘要、采纳率统计、agent toolbox 四条链的共同上游。
@Kucell
Kucell merged commit b7c87ea into main Oct 23, 2026
2 checks passed
@Kucell
Kucell deleted the docs/review-capability-gap-vs-pr-agent branch October 23, 2026 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant