review
差分レビューをするときの既定フロー。
目的
- correctness、回帰リスク、検証不足を優先して洗い出す
- 保守性や重複は、致命的な問題の後に扱う
手順
git diff origin/main...か対象差分を確認する- 変更された領域の source of truth を特定する
- 既存の test macro、workflow、生成手順から外れていないかを確認する
- correctness と安全性を確認した後、既存 helper / 標準ライブラリ / platform 機能で置き換えられる実装や、現在の要件に不要な abstraction を確認する
- findings を重要度順に整理する
- 問題がなければ、未確認の検証範囲だけを残リスクとして記録する
観点
- Rust core と editor / docs の整合
- generated artifact の更新漏れ
- 既存 test macro を無視したテスト実装
- packaging / workflow への影響漏れ
- 共有経路ではなく症状ごとに重複した bug fix
- 未使用の拡張性、新規 dependency、将来用 scaffold による過剰実装