woocommerce-code-review

WooCommerce コードレビュー

WooCommerce のコーディング標準と規約に照らして、コードの変更をレビューします。

指摘すべき重大な違反

バックエンド PHP コード

詳細な標準については、woocommerce-backend-dev スキルを参照してください。これらの標準を指針として、次の違反や類似する違反を指摘してください。

アーキテクチャと構造：

• 独立した関数 - クラスメソッドを使用する必要があります（file-entities.md）
• DI 管理クラスに new を使用する - src/ 内のクラスには $container->get() を使用する必要があります（dependency-injection.md）
• src/Internal/ 外のクラス - 明示的に公開されている場合を除き、既定の配置場所は src/Internal/ です（file-entities.md）

命名と規約：

• camelCase 命名 - メソッド、変数、フックには snake_case を使用する必要があります（code-entities.md）
• Yoda 条件の違反 - WordPress Coding Standards に従う必要があります（coding-conventions.md）

ドキュメント：

• @since アノテーションがない - public/protected メソッドとフックには必須です（code-entities.md）
• Docblock がない - すべてのフックとメソッドに必須です（code-entities.md）
• 冗長な Docblock - 簡潔にし、1 行を理想とします（code-entities.md）

データの整合性：

• 検証がない - 削除や変更の前に、状態を確認する必要があります（data-integrity.md）

テスト：

• テストで $instance を使用する - 変数名には $sut を使用する必要があります（unit-tests.md）
• @testdox がない - テストメソッドの Docblock に必須です（unit-tests.md）
• テストファイルの命名 - includes/ と src/ の規約に従う必要があります（unit-tests.md）

UI テキストとコピー

woocommerce-copy-guidelines スキルを参照してください。次の点を指摘してください。

• UI でのタイトルケース - センテンスケースを使用する必要があります（sentence-case.md）
  • 誤り：「Save Changes」、「Order Details」、「Payment Options」
  • 正しい例：「Save changes」、「Order details」、「Payment options」
  • 例外：固有名詞（WooPayments）、頭字語（API）、ブランド名

レビューの進め方

1. 上記に挙げた重大な違反を確認する
2. 問題を指摘する際は、該当するスキルファイルを明記する
3. スキルのドキュメントにある正しい例を示す
4. 分かりやすいよう、関連する問題をまとめる
5. 建設的に伝える - 該当する場合は、その標準が設けられている理由を説明する

出力形式

見つかった違反ごとに：

[Issue Type]: [Specific problem]
Location: [File path and line number]
Standard: [Link to relevant skill file]
Fix: [Brief explanation or example]

注記

• 詳細な標準はすべて、woocommerce-backend-dev、woocommerce-dev-cycle、woocommerce-copy-guidelines の各スキルに記載されています
• 詳細な背景や例については、これらのスキルを参照してください
• 判断に迷う場合は、上記にリンクされている該当のスキルドキュメントを参照してください