抽查近两个月合并的 40 个 PR，同一模块里同时存在 2 空格、4 空格和 Tab 三种缩进，分号有的带有的不带。评审记录里有 17 条在讨论格式，没一条提到逻辑。日志里能对上时间点，这种反而好排。
仓库没有统一的格式化配置，IDE 各用各的默认值。评审把风格检查当成人工职责，讨论成本高还容易漏。这一点先确认清楚，后面就不用反复试。
npx prettier --check 'src/**/*.{ts,tsx}'
npx eslint 'src/**/*.ts' --max-warnings 0
「代码风格人肉检查」确认了原因之后，改动尽量只碰那一处，顺手改的东西后面最容易出问题。偶发问题先统计频率，一小时几次和一天一次，查法完全不同。
改完顺手把文档补一句，省得下次自己重新推一遍。
处理上按这个顺序：仓库根目录放统一的 .editorconfig 和格式化配置；接入 pre-commit 自动格式化 提交前就改好；CI 加一步格式校验 不一致直接失败；把格式类评论从评审里去掉 交给工具。
结果对不对很好判断，全仓格式化检查一次通过，评审里不再出现缩进和分号相关的评论。