[feedback] reviewing seems too slow, one PR review costs 30mins+

[zengchuang-hw]

[hsliuustc0106]

I think we need to decouple the timeline to check whether we can reduce the reduandant process

[zuiho-kai]

我用用户举的 vllm-project/vllm-omni#4048 复测了一下。

当前结果：

• 命令：/imreview https://github.com/vllm-project/vllm-omni/pull/4048
• 总耗时：约 5分16秒
• 走的是 Direct mode
• validate_direct_review 正常完成
• 没复现 status poll 503
• 没复现 deepseek-v4-flash metadata warning
• 没有发 GitHub review/comment

这个 PR 本身确实会触发偏重的 review 路由：它有 9 个文件、约 667 行 +/- 变更，而且包含两个 PNG binary patch。GitHub 页面看是 +569/-97，但 raw patch 实际约 67k 行、4.4MB，所以它比表面看起来重很多。

不过这次没有复现 33 分钟。截图里的 33 分钟同时出现了：

• validate_direct_review 服务端 503
• status poll 服务端 503
• deepseek-v4-flash metadata warning

所以目前判断是：PR 4048 本身会让 review 成本变高，但 33 分钟主要不是正常预期耗时，更像是当时服务端/模型网关异常路径把耗时放大了。

建议后续做三件事：

1. review evidence 里过滤或摘要 binary patch，避免 PNG 这种内容撑爆上下文。
2. 调整 large_diff_files > 8 这类阈值，避免资产/测试较多的 PR 自动走最重路径。
3. 给 direct review validation / status poll 加耗时拆分日志，后面再遇到可以直接看到时间花在哪一步。

[zuiho-kai]

@zengchuang-hw 可能网不好，模型也比较慢