Skip to content
cccskills
FreePublished on GitHub

self-review

自分の作業ブランチの PR 提出前セルフレビュー(diff の包括チェック)。「セルフレビューして」「消し忘れがないか確認して」「diff を徹底チェックして」「変更漏れ・削除漏れを見て」等の依頼時、PR 作成前の最終確認時に使用。境界: 他者 PR のレビューは pr-review、コードベース全体は codebase-review。

The English summary is being prepared. Showing the original description.

See how to install

Files included (1)

  • SKILL.md4.6 KB

SKILL.md (original)

Review the instructions the agent will receive before you install.

self-review - PR提出前のセルフレビュー

自分の作業ブランチのdiff全量を包括チェックし、消し忘れ・変更漏れ・意図しない変更を検出する。 コード品質のレビュー(バグ・設計)ではなく、変更の完全性と純度の検証に特化する。

既存設定との関係

  • Phase 4品質確認(context/workflow-rules.md): lint/format/typecheck/testを補完する。外部レビューの前に行うと指摘のノイズが減る
  • pr-reviewスキル: 他者PRのレビュー用。本スキルは自分のブランチのPR提出前チェック用

ワークフロー

1. ベースの確定とdiff全量取得

# BASE_BRANCHはPJ CLAUDE.md参照(未定義時: develop→main→master)
git log --oneline <BASE_BRANCH>..HEAD
git diff <BASE_BRANCH>...HEAD --stat
git diff <BASE_BRANCH>...HEAD
git status   # 未コミット変更の確認

diffは省略せず全ファイル確認する(一部だけでは消し忘れを拾えない)。

2. 消し忘れ・残存の検出

diffで削除・リネームされた識別子を抽出し、リポジトリ全体で残存検索する:

  • 対象識別子: メソッド名・クラス名・定数・env key・docker service名・キュー名・設定キー・ルーティングパス等
  • 検索範囲: コードに加えてconfig / docker-compose / .env* / CI定義(.github/workflows等)/ docs / テストfixture
  • コマンド: git grep -n "<identifier>" または rg -n "<identifier>"
# 例: 削除した識別子ごとに実行
git grep -n "<削除した識別子>"
  • 残存あり → 消し忘れとして報告(削除対象か、意図的に残すのかを判定)
  • 逆方向も確認: 削除した機能を参照していた呼び出し元が全て更新されているか

3. 意図しない変更の検出

  • format副作用: 自分のタスクと無関係なファイルの整形のみの変更(git diff --stat で変更行数が多い割に意図が不明なファイル)
  • 混入: .idea/ .vscode/ .env* *.log lockファイルの意図しない変更、一時ファイル
  • デバッグ残骸: console.log / print / debugger / コメントアウトされた試行コード / TODOの置き忘れ
  • 無関係コミット: git log に他タスクのコミットが混入していないか(rebase漏れ)

4. タスク要件との突合

双方向で確認する:

  1. 要件 → diff: 今回のタスク要件(ユーザー指示・計画書・チケット)の各項目が、diffのどの変更で満たされているか対応付ける。満たされていない要件があれば「漏れ」
  2. diff → 要件: diffの各変更が要件のどれかに紐づくか。紐づかない変更は「過剰(スコープ外変更)」として報告
  3. 足しすぎ: 要件に紐づく変更でも、頼まれていない機能・オプション・「後で使う」仕組み、既存の helper や標準ライブラリで済む新しい実装、数行のための新しい依存、実装が1つしかない抽象は「過剰(足しすぎ)」として報告する(探す順序は AGENTS.md「新しく作る前に」)

5. レポート

# セルフレビュー: <ブランチ名>

## 対象
- ブランチ: <branch> → <base>(コミットN件、ファイルM件)

## 消し忘れ・残存
- [なし / 検出内容(ファイル:行)]

## 意図しない変更
- [なし / 検出内容と推奨対応(戻す・別コミット化等)]

## 要件との突合
| 要件 | 対応する変更 | 判定 |
|------|------------|------|
| ... | ... | OK / 漏れ / 過剰 |

## 推奨アクション
1. [修正が必要な項目]

注意事項

  • 検出した問題の修正は報告後にユーザー判断を仰ぐ(勝手にファイルを戻さない。特に git checkout -- はformat副作用と断定できるファイルのみ・判断不能時はユーザー確認)
  • 「漏れなく」は要件に対する漏れの確認であり、無関係な要素の網羅列挙ではない(スコープ膨張禁止)

Reviews

No reviews yet. Tell others what you thought.

Comments

No login needed. Share how it worked for you, ask a question, or note where it did not run.

No comments yet. Be the first to write one.

Comments are published right away. Do not include personal information such as email addresses or phone numbers. Up to 2 URLs.

0 / 1,000 characters

This form is protected by reCAPTCHA, and the Google Privacy Policy and Terms of Service apply.

More skills from this repository

コードベース全体のレビュー・監査。perf/sec/test/arch/cq/docs の6観点を並列委譲し、優先度付き issue ファイルと観点サマリをメモリディレクトリに生成する。コードベース全体の監査・定期レビュー・リリース前品質確認の依頼時、/codebase-review 実行時に使用。境界: PR 単位は pr-review、自ブランチの提出前確認は self-review。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

commit

Free

変更を git-cz 形式でコミットする。リポジトリの直コミット可否の判定手順と、push 後の CI・conflict の確認を含む。/commit 実行時、「コミットして」「pushして」等の依頼時に使用。--push 引数または「pushして」の依頼で push も行う。境界: PR 作成は create-draft-pr、PR コメントへの対応コミットは pr-comment から呼ばれる。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

PR を Draft で作成する。変更をローカルで動かして確かめ、画面が変わるならスクリーンショットを gh の --attach で貼り、PR テンプレートを全セクション埋めて対象 repo の既存 PR の分量に揃え、出した後は CI・レビュー・マージを Monitor で見張る。PR 作成の依頼時、実装が一段落して PR 化する時、/create-draft-pr 実行時に使用(gh pr create を直接実行しない)。引数でベースブランチを指定できる。境界: 個別レビューコメントへの対応は pr-comment、PR 全体のレビューは pr-review、複数 PR 規模の計画は plan-feature-prs。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

スキルを新規作成する。「スキルを作って」「この手順をスキル化して」等の依頼時、/create-skill 実行時に使用。AGENTS.md・context・既存スキルと整合させ、重複・競合を避ける。境界: 既存スキル・指示ファイルの修正は update-inst、指示ファイル全体の監査は instructions-audit。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

抽象的な要件・事業側の要求を深掘りし、既存実装との整合を確認して実装マスタとシステム要件書を作成する。「こういう機能を作りたい」「この要求を満たす機能を設計して」等の抽象要件の提示時、既存機能の拡張や Phase 分割の要件定義開始時、/design-feature 実行時に使用。境界: 要件確定後の実装計画書・PR 分割と実装進行は plan-feature-prs。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

画面の設計判断(ナビゲーション・重ね方・通知と空状態と読み込み・外枠とトークンと状態表現)の型を決定表で選び、アプリ内の全画面で揃える。画面・ページ・レイアウト・ナビゲーション・ダイアログ・コンポーネントを新しく作る・作り直す時、モックやダッシュボードを作る時、サイドバー・タブ・モーダル・シート・toast・バナー・空状態のどれを使うか決める時に使用。PJ に components.json があれば shadcn の部品とトークンへの対応も扱う。主要なデザインシステム(Material・HIG・Carbon・Primer・Atlassian・Fluent・GOV.UK・NN/g・WAI-ARIA APG・WCAG)の一致点を規則にし、食い違いの採否を定めてある。境界: 画面の文言は writing-ui-text、画面に何を出すかと提示前の完了基準は context/ui-artifact-standards.md、コードの実装原則は writing-code、shadcn 部品の API と組み立ては shadcn。

The English summary is being prepared. Showing the original description.

ukwhatn/.claude4Updated Oct 21, 2026

See all skills by ukwhatn

Report a problem with this skill