main_task_user.md中已经包含本组的diff文本，为什么还需要file_read_diff工具？

[BistuDu]

main_task_user.md中已经包含本组的diff文本，为什么还需要file_read_diff工具？
是不是有点多次一举了呢？是为了获取到组外的diff文本？那read_file工具就可以替代调用file_read_diff获取其他文件的代码呢。为什么没有去除file_read_diff工具？关于工具的设计，我不没有参透设计思路，还望您通俗的解释下。

[basil-k-aji-dev]

我去代码里对了一遍：main_task_user.md 里其实只有本组的 diff 全文，组外文件是没有 diff 正文的，所以 file_read_diff 并不是多此一举。

1. 模板里是两个不同的槽位

internal/config/template/prompts/main_task_user.md：

• {{diffs}} → <review_files>：本组文件的 diff 全文
• {{change_files}} → <other_changed_files>：只有文件名

{{change_files}} 由 buildChangeFilesExceptGroup 生成（internal/agent/agent.go:1626），每一项都走 formatDiffEntry（同文件 1612 行），输出形如：

MODIFIED   src/api/handler.go (+12/-3)

只有状态、路径、增删行数，一行 diff 正文都没有。而且函数名里的 ExceptGroup 就是在刻意把本组文件从这个清单里剔除，正是为了不跟 <review_files> 重复。

所以模型实际拿到的是：本组 = 完整 diff，组外 = 一份光秃秃的清单。file_read_diff 补的就是后面这一半 —— 它的 DiffMap 由 injectDiffMap（agent.go:608）按整个变更集的路径建好，正好覆盖组外文件。

2. file_read 替代不了，因为它读的是「改完之后」

file_read 最终走 FileReader.ReadLines，而 FileReader 按 ReviewMode 取内容（internal/tool/filereader.go:24-40）：workspace 模式读工作区，range 模式读 --to 指向的 ref，commit 模式读该 commit。三条路径取到的都是变更后的那一侧，没有任何一条能拿到变更前的内容。

而 diff 的信息量 = 新 ∖ 旧，光有「新」是还原不出「改了什么」的。举个最常见的场景：你在审 A 文件里的调用方，想确认被调的 B 函数签名是不是在这次变更里一起改了。file_read 给你的是当前签名 —— 改没改都长这样，看不出来；只有 file_read_diff 能告诉你这次到底动没动它。

3. 这个用法在注释里是写明了的

internal/diff/resolver.go:77-78：

The reviewing Agent reads related files through file_read_diff, so it can describe code from a file other than the one under review and still file the comment against the file under review — typically a declaration/implementation split...

典型例子就是声明/实现分离（头文件 + 源文件）：评论挂在本组的文件上，但证据来自组外文件的 diff。

4. 另外，全量扫描模式复用了这个工具

scan 模式没有 diff 的概念，injectScanContentMap（internal/scan/agent.go:413）把整份文件内容塞进同一个 DiffMap，而 {{change_files}} 直接填了固定字面量 (not applicable in full-scan mode)（scan/agent.go:33）。同一个工具在两种模式下语义不同，这也是它没法被 file_read 合并掉的原因之一。

---

一句话总结：{{diffs}} 管组内，file_read_diff 管组外，file_read 管「文件现在长什么样」—— 三者的信息面并不重叠。

以上基于 3bce7ad。@lizhengfeng101 麻烦确认一下有没有理解偏差，如果没问题这个应该就可以关掉了。

[wu21-web]

我感觉这个去掉没有必要，可以关掉了