reviewing-changes

変更のレビュー - Android 向け追加事項

このスキルでは、基本の bitwarden-code-reviewer エージェント標準を補完する、Android 固有のワークフローの追加事項を説明します。

手順

重要：フィードバックを提供する前に、各手順を順に実施してください。各チェックリストファイルには、レビュー時の思考を体系化するためのガイダンスが含まれています。

手順 1：追加情報を取得する

利用可能なツール（JIRA MCP、GitHub API）を使用して、プルリクエストに関連する追加情報を取得します。

プルリクエストのタイトルと説明だけではコンテキストが不十分な場合は、レビュー担当者に追加情報を依頼します。

• JIRA チケットへのリンクを追加する
• GitHub issue を関連付ける
• 別のプルリクエストへのリンクを追加する
• PR のタイトルまたは本文に詳細を追加する

Android のメタデータチェック — 以下のいずれかが欠けている場合は ❓ を付けます。

• PR に *Screen.kt または Composable の変更が含まれているが、スクリーンショットがない
• PR で新しい ViewModel または Repository を追加しているが、テスト計画またはテストファイルの変更がない

手順 2：Android 向けの調整を加えて変更の種類を判定する

エージェントの基本的な変更種別の判定を使用し、Android 固有の調整を加えます。

Android 固有のパターン：

• 機能追加：新しい ViewModel、新しい Repository、新しい @Composable 関数、新しい *Screen.kt ファイル
• UI の改善：*Screen.kt、*Composable.kt、ui/ パッケージ内のファイルのみの変更
• インフラストラクチャ：.github/workflows/、gradle/、build.gradle.kts、libs.versions.toml の変更
• 依存関係の更新：バージョン更新を伴う libs.versions.toml または build.gradle.kts のみの変更

手順 3：適切なチェックリストを読み込む

判定した種類に応じて、該当するチェックリストファイルを読み込みます。

• 依存関係の更新 → checklists/dependency-update.md（簡易レビュー）
• バグ修正 → checklists/bug-fix.md（重点的なレビュー）
• 機能追加 → checklists/feature-addition.md（包括的なレビュー）
• UI の改善 → checklists/ui-refinement.md（デザイン重視のレビュー）
• リファクタリング → checklists/refactoring.md（パターン重視のレビュー）
• インフラストラクチャ → checklists/infrastructure.md（ツール重視のレビュー）

チェックリストには、以下が含まれています。

• 複数パスで行うレビュー戦略
• 種類ごとに注目する領域
• 確認すべき項目と省略すべき項目
• 思考を体系化するためのガイダンス

手順 4：チェックリストに従ってレビューを実施する

チェックリストの複数パスで行う戦略に従い、各パスを順に検討します。

手順 5：必要に応じて Android の参考資料を参照する

特定の質問に必要な場合にのみ、参考ファイルを読み込みます。

• 再レビュー → reviewing-incremental-changes エージェントスキルを呼び出します。変更された行だけを対象とし、変更されていないコードの新たな問題は指摘しないでください
• 問題の優先順位付け → reference/priority-framework.md（Critical と Suggested と Optional）
• フィードバックの表現 → reference/review-psychology.md（質問と命令、I メッセージ）
• アーキテクチャに関する質問 → reference/architectural-patterns.md（MVVM、Hilt DI、モジュール構成、エラー処理）
• セキュリティに関する質問（簡易リファレンス） → reference/security-patterns.md（一般的なパターンとアンチパターン）
• セキュリティに関する質問（包括的な資料） → docs/ARCHITECTURE.md#security（ゼロ知識アーキテクチャの全体像）
• テストに関する質問 → reference/testing-patterns.md（ユニットテスト、モック、null 安全性）
• UI に関する質問 → reference/ui-patterns.md（Compose のパターン、テーマ設定）
• スタイルに関する質問（プロジェクト固有） → reference/style-patterns.md（レビューで適用する Kotlin のルール）
• スタイルに関する質問（一般） → docs/STYLE_AND_BEST_PRACTICES.md

基本原則

• 優先順位：セキュリティ → 正確性 → 破壊的変更 → パフォーマンス → 保守性
• 適切な深さ：変更の複雑さとリスクに応じて、レビューの厳密さを調整します
• 具体的な参照：正確な位置を示すには、必ず file:line_number 形式を使用します
• 実行可能なフィードバック：問題点だけでなく、何をすべきか、その理由も伝えます
• 効率的なレビュー：複数パスの戦略を用い、関連しない内容は確認を省略します
• Android のパターン：MVVM、Hilt DI、Compose の規約、Kotlin の慣用表現を検証します