为 review filter 增加禁用/配置选项

[sunflower-zyb]

背景

REVIEW_FILTER_TASK（review filter）会在每次 review 的 main_task 之后自动运行，用一次单独的 LLM 调用，按「仅凭当前文件 diff 能否直接证伪」的标准，剔除它认为不准确的评论（见 internal/agent/agent.go 的 executeReviewFilter）。

当前该流程无任何开关：

• executeReviewFilter 的无条件调用点：internal/agent/agent.go:1204（main 分支）/ internal/agent/agent.go:1210（v1.8.6）。
• 唯一提前返回的守卫是 ft == nil（模板未配置 REVIEW_FILTER_TASK），但该任务来自 go:embed 内建的 task_template.json，用户无法覆盖；LoadDefault() 只读嵌入资源，cmd 层也没有自定义模板 path 的入口。
• CLI 没有 --no-filter 之类的 flag；配置 / 环境变量中也没有 REVIEW_FILTER_ENABLED 这类开关（已在 v1.8.6 / v1.8.7 / main 三处源码全量 grep 确认）。

问题

由于 filter 遵循「falsify, not verify」且有「宁删勿留」倾向，它倾向于剔除需要跨文件 / 业务语义 / 运行时上下文才能证实的评论。但同时，这类评论恰恰是很多项目级自定义规则（rule.json）要求检查的核心点。

实测中（结合 --rule 项目规则）出现多条真实、正确的评论被 filter 误删。例如：

1. super 调用规范：某个 Scene 子类重写了空的 onCreate() 且未调用 super.onCreate()（基类负责生命周期日志）。正确且规则明确要求检查，但因「需跳看基类 BaseScene」而被 filter 剔除。
2. 状态机边界：代码把显示条件从 state == 2 放宽为 state != 0，但状态枚举还定义了 STATE_INVALID = -1。-1 != 0 恒真导致无效态也会显示。真实 bug，但需结合另一文件的枚举定义才能证实，被 filter 剔除。

这些被删的评论本身是对规则的正确实例化，可是用户没有途径关闭这道「宁缺毋滥」的闸门。

期望行为

希望提供一个显式开关，例如（任一或组合）：

• CLI：ocr review --no-review-filter（或 --disable-review-filter）；
• 或 env：OCR_REVIEW_FILTER=off；
• 或允许 REVIEW_FILTER_TASK 被自定义 / 用户模板覆盖（甚至在用户级 rule.json 里可配置是否启用）。

保留当前默认行为不变（默认开启），只是给需要完整保留 Agent 评论的用户一个退出路径。

可选补充

如果开关不便加，退一步也可以考虑：给 filter 增加一个「提示用户可关闭」的反馈，或当被删评论数较多时打印一条 --no-review-filter 提示，便于用户发现问题并跳转。

关联信息

• 相关代码：internal/agent/agent.go（executeReviewFilter / ReviewFilterTask / task_template.json）。
• 定位命令：ocr rules check <path>（可确认规则确实命中并进入 prompt；问题出在 filter 环节，而非规则解析）。

[sunflower-zyb]

自定义规则如下：

{
  "rules": [
    {
      "path": "**/ui/scene/**/*Scene.{java,kt}",
      "rule": "Scene 代码规范（适用工程 scene 包内的所有 Scene 子类）：1) 必须使用无参构造函数，禁止有参构造（导致反射创建失败），传参通过 Bundle（sceneManager.add(containerId, MyScene::class.java, tag, args)）；2) 禁止外部持有 Scene 实例（内存泄漏），判断是否已展示用 sceneManager.findSceneByTag(TAG)，增删通过 sceneManager 事务；3) 嵌套 View 的 bindScene() 必须在 onViewCreated() 中调用，禁止在 onCreateView() 中；4) 重写生命周期方法时规范调用 super：onViewCreated() 开头（bindScene 前）、onResume() 开头、onStop() 结尾、onDestroy() 结尾（先反注册自身资源）。另注意：dp 转 px 必须用工程提供的 ResUtil#dpToPx 工具方法，sp 转 px 用对应工具方法，禁止手写 (int)(dp*density)。"
    },
    {
      "path": "**/*.{java,kt}",
      "rule": "通用规则规范（适用范围：未被上述 Scene/View/ViewModel/信号规则覆盖的其他 Java/Kotlin 文件）：单位换算必须使用统一工具类，禁止手写计算：dp 转 px 应调用工程提供的 ResUtil#dpToPx 工具方法，sp 转 px 用对应工具方法；不要自行用 (int)(dp * density) 或 (int)(dp * getResources().getDisplayMetrics().density) 之类的方式计算。"
    }
  ]
}

测试代码如下：

class SpaceScene : BaseScene() {


    override fun onCreate() {

    }


    override fun onCreateView(
        inflater: LayoutInflater,
        parent: ViewGroup
    ): View {
        return SpaceExoView(inflater.context)
    }


    override fun onViewCreated(view: View) {
        super.onViewCreated(view)
        view.postDelayed({ XILog.i("SpaceScene", "onViewCreated") }, 1000)
    }

}

但结果如下：

[ocr] Review filter removed 1 comment(s) for ****/SpaceScene.kt
[ocr]   ▶ file_read start_line=60 end_line=177
[ocr]   ✔ file_read (0s)

或者有其他方案可以解决这个问题？

[lizhengfeng101]

我认为可以加一个 cli 参数来跳过filter 任务

[lizhengfeng101]

自定义规则如下：

{

  "rules": [

    {

      "path": "**/ui/scene/**/*Scene.{java,kt}",

      "rule": "Scene 代码规范（适用工程 scene 包内的所有 Scene 子类）：1) 必须使用无参构造函数，禁止有参构造（导致反射创建失败），传参通过 Bundle（sceneManager.add(containerId, MyScene::class.java, tag, args)）；2) 禁止外部持有 Scene 实例（内存泄漏），判断是否已展示用 sceneManager.findSceneByTag(TAG)，增删通过 sceneManager 事务；3) 嵌套 View 的 bindScene() 必须在 onViewCreated() 中调用，禁止在 onCreateView() 中；4) 重写生命周期方法时规范调用 super：onViewCreated() 开头（bindScene 前）、onResume() 开头、onStop() 结尾、onDestroy() 结尾（先反注册自身资源）。另注意：dp 转 px 必须用工程提供的 ResUtil#dpToPx 工具方法，sp 转 px 用对应工具方法，禁止手写 (int)(dp*density)。"

    },

    {

      "path": "**/*.{java,kt}",

      "rule": "通用规则规范（适用范围：未被上述 Scene/View/ViewModel/信号规则覆盖的其他 Java/Kotlin 文件）：单位换算必须使用统一工具类，禁止手写计算：dp 转 px 应调用工程提供的 ResUtil#dpToPx 工具方法，sp 转 px 用对应工具方法；不要自行用 (int)(dp * density) 或 (int)(dp * getResources().getDisplayMetrics().density) 之类的方式计算。"

    }

  ]

}

测试代码如下：

class SpaceScene : BaseScene() {





    override fun onCreate() {



    }





    override fun onCreateView(

        inflater: LayoutInflater,

        parent: ViewGroup

    ): View {

        return SpaceExoView(inflater.context)

    }





    override fun onViewCreated(view: View) {

        super.onViewCreated(view)

        view.postDelayed({ XILog.i("SpaceScene", "onViewCreated") }, 1000)

    }



}

但结果如下：

[ocr] Review filter removed 1 comment(s) for ****/SpaceScene.kt

[ocr]   ▶ file_read start_line=60 end_line=177

[ocr]   ✔ file_read (0s)

或者有其他方案可以解决这个问题？

你是否愿意参与增加一个cli 参数

[lizhengfeng101]

另外我们内部正在执行 filter 任务的 agent 化改造，会评估一下效果提升和 token 消耗的提升，可能这个特性上线后，你就不需要这个参数了。

[sunflower-zyb]

我们项目级的自定义规则（rule.json）需要跨文件 / 业务语义 / 运行时上下文，不要过滤这些规则产生的评论，只要达到这个效果，手段都OK。