Skip to content

[codex] 收敛 preflight 规则并优化 comment 结构#111

Merged
utopiafar merged 2 commits into
memex-lab:mainfrom
utopiafar:codex/preflight-separated-comment
May 17, 2026
Merged

[codex] 收敛 preflight 规则并优化 comment 结构#111
utopiafar merged 2 commits into
memex-lab:mainfrom
utopiafar:codex/preflight-separated-comment

Conversation

@utopiafar
Copy link
Copy Markdown
Collaborator

@utopiafar utopiafar commented May 17, 2026

背景

#108 已经让 preflight 支持中英双语输出,但 PR comment 仍然是中英文交替阅读,信息密度高时不太顺。同时,原 preflight 规则对业务目录和改动规模的高风险判定偏硬,容易把普通代码变更误伤成高风险。

改动

  • 将 preflight Markdown comment 调整为独立的中文区块和 English 区块。
  • 从人类可读 comment 中移除 risk score,避免把内部粗粒度数值当成有意义的风险评分。
  • 移除 workflow 追加的中英交替 shadow-mode footer,让 comment 全部由脚本生成的语言区块承载。
  • 收敛高风险规则范围:保留 .github/**analysis_options.yaml、preflight/review policy 控制文件、diff 截断、不可 review 的二进制内容。
  • 将普通业务路径、规模阈值、敏感关键词、l10n 单边改动、generated-with-source 等改为 warning,不再仅凭目录或规模把普通代码判为高风险。
  • 更新中英文文档,明确 preflight 与 Flutter CI 的分工:preflight 只做治理面硬规则,普通代码质量由 flutter analyze / flutter test / 必要 compile-build 承担。
  • 更新脚本测试,覆盖独立中英文区块、comment 不展示分数,以及普通 app code 不再因目录本身判高风险。

验证

  • python3 -m unittest tests.tools.test_pr_policy_check -v
  • python3 -m py_compile scripts/pr_policy_check.py tests/tools/test_pr_policy_check.py
  • 手动运行 scripts/pr_policy_check.py 生成 Markdown,确认输出为中文区块 + English 区块。

说明

这次仍保持 shadow mode,不把 preflight 判定作为阻塞依据。后续如果接入低风险快速通道,建议把 flutter analyze 作为必需质量门禁:优先要求全绿;如果仓库存在历史 analyzer 问题,再维护 baseline 并要求 PR 不新增问题。

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 17, 2026

PR Policy Preflight / PR 规则预检

  • Decision / 判定: HIGH RISK / 高风险
  • Risk score / 风险分: 160
  • Changed files / 变更文件数: 5
  • Changed lines / 变更行数: 338
  • Diff truncated / Diff 是否截断: false

Findings / 规则命中

  • high / 高风险 github-config .github/workflows/pr-policy-preflight.yml
    • EN: GitHub configuration or workflow changed.
    • 中文: GitHub 配置或 workflow 发生变化。
  • high / 高风险 review-policy docs/pr-policy-preflight.en.md
    • EN: Review policy, preflight script, or control file changed.
    • 中文: Review policy、preflight 脚本或控制文件发生变化。
  • high / 高风险 review-policy docs/pr-policy-preflight.zh.md
    • EN: Review policy, preflight script, or control file changed.
    • 中文: Review policy、preflight 脚本或控制文件发生变化。
  • high / 高风险 review-policy scripts/pr_policy_check.py
    • EN: Review policy, preflight script, or control file changed.
    • 中文: Review policy、preflight 脚本或控制文件发生变化。

Shadow mode: this result is informational and does not block merge.

Shadow mode:当前结果仅供参考,不会阻塞合并。

@utopiafar utopiafar changed the title [codex] 优化 preflight 中英文 comment 结构 [codex] 收敛 preflight 规则并优化 comment 结构 May 17, 2026
@utopiafar utopiafar merged commit 3cc9c06 into memex-lab:main May 17, 2026
1 check passed
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