code-quality-review

コードレビュー結果、code quality report、git diff、PR 差分、指定ファイルをもとに、挙動を変えずにコードを簡略化・リファクタリングするためのスキル。可読性、保守性、テスト容易性、複雑度、効率性、技術的負債の指摘を実装に反映し、変更範囲と検証結果を簡潔に報告するときに使う。ユーザーが…

実装済みの差分、PR、コミット、指定ファイルを「次に開発する人が困らないか」という観点でレビューするときに使う。変更箇所の見つけやすさ、責務境界、命名、型・契約、テストの読みやすさ、前提知識の残し方、追加ケースの足しやすさ、ローカル検証のしやすさを確認したい場合に使う。欠陥、仕様違反、セキュリティ、データ破壊の検出を…

Git リポジトリで変更内容を確認し、適切な粒度とメッセージで安全に commit を作成するときに使う。ユーザーが「commitして」「コミット作って」「差分をコミット」「変更を分けてコミット」「この変更をコミット」などを依頼した場合に使う。git status / diff / log を確認し、未確認の変更や…

実装済みの差分、PR、コミット、指定ファイルをレビューし、仕様違反、回帰、公開契約破壊、セキュリティ、データ整合性、並行性、検証不足などのリスクを重要度順に指摘するときに使う。実装方針の相談は code-refactor、コード変更の実装は code-general を優先する。

Git リポジトリで、ベースブランチまたは残したい履歴の終点 commit との差分に含まれる commit や巨大な未整理差分を、意味のある変更単位ごとに整理して commit し直すときに使う。ユーザーが「commit…

React / JSX / TSX / .jsx / .tsx 実装を書く前に、コンポーネント設計、Atomic Design を補助観点にした UI 分割、hooks、props、state、UI 実装の責務分離を確認するときに使う。

言語固有の実装スキルが適用できない、または言語をまたぐ既存コードベースの機能追加、修正、リファクタリングで守る共通原則と進め方を定義する。仕様や公開契約を壊さず、既存構造、命名、責務境界に合わせて小さく安全な差分を作るときに使う。Go 実装では code-go など該当する言語別スキルを併用し、レビューのみは…

関数名、メソッド名、クラス名、型名、変数名、定数名、ファイル名、ディレクトリ名などコード上の命名をレビューし、責務、抽象度、既存語彙、検索性、呼び出し側の読みやすさ、ユーザーの命名の好みに基づいて候補を出すときに使う。名前候補を比較したい、既存名を改善したい、命名の一貫性を確認したい、曖昧な名前、過度に汎用的な名前、…

GitHub Pull Request を安全な手順で作成または更新するときに使う。ユーザーが「PR 作って」「PR 出して」「pull request 作成して」「この変更を PR にして」「pr を出す手順を進めて」などを依頼した場合は必ず使う。差分確認、検証、commit、最新 base branch…

指定された agent を起動し、結果を待たずに投げっぱなしで作業を開始させるときに使う。ユーザーが「agent を立ち上げて」「worker に投げて」「scouter を走らせて」「結果を待たずに」「投げっぱなし」など、明示的にサブエージェント起動を依頼した場合は必ず使う。明示依頼がなくても、タスクが単純で不明瞭…

Git リポジトリで新しい作業を始める前に、最新のデフォルトブランチへ追従し、タスク内容に合う作業ブランチを作成して、不要なローカルブランチ整理を非同期に依頼するときに使う。ユーザーが「ブランチ切って」「作業を始める準備して」「新しいタスク用ブランチを作って」「最新 main から始めたい」などを依頼した場合はこの…

GitHub Actions や CI の失敗を調査し、原因を切り分けて修正するときに使う。ユーザーが「CI 落ちてる」「GitHub Actions 直して」「workflow が失敗」「checks が red」「test failure を直して」「lint/build が CI…