Skill.md検索

テストが通うためだけに書かれた「偽装コード」（テスト環境フラグ、テスト対象をバイパスするモック、空の例外処理など）を検出し、本当の品質リスクを指摘できます。 黙って飲み込まれている例外（空のcatchブロック、エラーを無視する処理）を発見し、本番環境での隠れたバグリスクを減らせます。 テストコード内の条件分岐を検出してシンプルなテストに分割するよう提案し、テスト自体のバグ混入リスクを低減できます。 期待値をハードコード化するよう指導し、テストを読むだけで仕様が理解できるテストに改善できます。 テストのない新規コードパス（ゴーストコード）を特定し、見落とされた機能の品質穴を埋めるテスト追加を促せます。 コード品質とテスト信頼性を重視するソフトウェア開発チーム PRレビューやテスト改善の依頼を受けるエンジニア テストカバレッジは高いのに本番バグが多い、という課題を抱えるチーム QAエンジニアが実装品質とテスト信頼性を守るためのコードレビュー指針。基本マインドセット：「コードは本当に正しく動作しているか、テストをパスさせるふりをしているだけか」と自問すること。レビュー観点は5項目。(1)テスト偽装コード検出：テスト環境フラグ、テスト対象バイパスモック、例外の黙った飲み込み。(2)握りつぶされた例外：空catchブロック、広過ぎるcatch、エラーの無視。(3)テストコード内の条件分岐：ifやswitch、三項演算子があると検証意図が不明確化。(4)ハードコードされたアサーション：期待値をリテラルにし、テストから仕様が一目瞭然に。(5)ゴーストコード：テストのない新規コードパスを検出。スコープ制限：原則ブランチスコープ内に限定だが、広範囲影響時はスコープ外テスト追加許可。

モノレポ環境で複数のパッケージを扱う際に、依存関係や exports 設定の不整合を防ぎながら開発できます。 Yarn を用いた統一されたコマンド操作により、全パッケージの構造と整合性を保ったまま機能追加・保守が可能です。 package.json の exports フィールドを適切に管理し、パッケージごとの公開 API を明確に定義できます。 修正前に必ずリポジトリスキャンを行い、変更がアーキテクチャルールに違反していないかセルフチェックできます。 Node.js モノレポを運用・保守しており、パッケージ間の依存関係が複雑になってきた開発チーム yarn workspace を活用して複数パッケージを統一管理したい人 Yarn 以外のパッケージマネージャー混在による問題を防ぎたい人 新機能追加時に既存の API やモジュール構成を壊さないようにしたい人 コマンド制約として Yarn のみの使用を厳守し、npm の使用は厳禁です。cd コマンドでの作業ディレクトリ移動は禁止し、常にルートディレクトリを起点として yarn workspace または --cwd フラグで操作を完結させます。ビルドや検証は影響を受けるパッケージに限定し、テスト実行時は npx vitest run を使用します。package.json の exports フィールド管理では Strict Exports により外部公開 API を厳選定義し、一括公開は避けます。更新時は既存定義を削除・破壊せず、差分のみを正確に追記します。AI 操作プロトコルとして、修正前に必ずパッケージ構造・依存関係・exports 現状をスキャンし、変更前後のセルフチェックを常に実施します。

テストをパスさせるためだけに書かれた"テスト偽装コード"を検出し、実装の正しさを検証します try-catchで例外を握りつぶしているコードを見つけ出し、エラーハンドリングの改善を提案できます テストコード内の条件分岐を指摘し、シンプルで信頼性の高いテストへのリファクタリングを促します テストされていない新規コードパス（"ゴーストコード"）を検出し、カバレッジを確保できます QA・テスト担当者（テスト品質とコードカバレッジを高めたい方） エンジニア・チームリード（本当に正しいテストが書かれているか確認したい方） コードレビュア（PR審査時にテスト品質を厳密に判定したい方） QAエンジニアとしてコード品質・テスト信頼性を守る専門スキルです。基本マインドセット：「テストが通っているだけか、本当に正しいか」「実装の振る舞いを保証しているか」「テスト削除で落ちるか」「パイプライン全体の動作を証明するテストはあるか」を常に自問します。スコープ制限：トピックブランチの指摘はそのブランチスコープ内に限定、ただし末端修正が広範囲に影響する場合はスコープ外テスト追加を許可（カバレッジ優先）。レビュー観点：（1）テスト偽装検出=test環境専用フラグ・テスト対象をモック化・エラーを黙って飲み込むコード、（2）握りつぶし例外=空のcatchブロック・広いキャッチ・失敗を検出不可にする戻り値変換、（3）テスト内の条件分岐=テスト自体にバグ潜在、パラメータ化テスト推奨、（4）ハードコード推奨=期待値はリテラルで、スナップショットは例外、（5）ゴーストコード=テストのない新規コードパス検出。言語・フレームワーク非依存で全リポジトリに対応、PRレビュー・テスト改善リクエスト時は必ず起動します。

リポジトリ全体を言語非依存で分析 — プログラミング言語やフレームワークに関わらず、構造・アーキテクチャ・技術スタック・テスト体制を一望できます。 メンテナンス性と運用課題を可視化 — ドキュメントの充実度、コード品質、エラーハンドリング、ログ体制を評価し、実際の運用で起きやすい問題を早期発見します。 破壊的変更のマイグレーションガイドを検査 — バージョンアップ時の影響範囲と対応手順が適切に整理されているか確認し、ユーザーが困らないようにします。 README やドキュメントの品質をレビュー — セットアップ手順、API仕様、参考資料へのリンクが実装と一致しているか、誰でも情報を見つけられるかを評価します。 プロダクトマネージャー・テックリード — 新機能追加時の長期的なメンテナンスコストを見積もりたい人 プロジェクト開始時や大型リファクタ前 — コード品質を把握し、改善優先度を決めたい人 エンジニア採用・オンボーディング支援 — 新しいメンバーが効率的に理解・貢献できるドキュメント体制を整備したい人