code-review

コードレビュー

1. 各コード変更に、そのコードの動作と防止すべき事象を検証するテストケースが含まれていることを確認します。
2. バグ修正のプルリクエストに、変更前にバグが再現し、変更後に修正されていることを示す回帰テストが含まれていることを確認します。レビュアーが比較を実行できない場合は、入手可能な根拠を使い、その制約を明記します。
3. 文言の変更に、リポジトリ内で一貫して使用されている用語が使われていることを確認します。この確認では、コミュニティが作成したカタログコンテンツと、その生成ドキュメントは対象外とします。
4. 影響を受ける領域を特定し、アーキテクチャとの一貫性を評価する前に、design/ 内の該当ドキュメントを読みます。CLI コマンドの変更では design/cli.md を、エージェント連携では design/integration.md を、ワークフローのステップタイプでは design/workflow-step.md を使用します。関連する場合は他の設計ドキュメントも読みますが、すべてのプルリクエストで全ドキュメントを読む必要はありません。
5. 実装の境界、所有権、ライフサイクル、提供要件など、該当する設計要件に照らして変更を確認します。このスキル内でルールを重複して記述するのではなく、ドキュメントを参照します。報告する不整合ごとに、該当するドキュメントとセクションを示します。
6. 説明のない設計違反と、意図的な設計変更を区別します。意図的な変更については、その根拠と証拠を評価します。設計ドキュメントを更新しただけでは、実装の妥当性を証明できません。