Fix review
Skill bright-room/br-claude-plugins/plugins/dev-workflow/skills/fix-review
Shared Claude Code plugins — reusable skills and commands across bright-room repositories.
npx -y skills add bright-room/br-claude-plugins --skill fix-reviewAssembled from the repository path, not quoted from the project. Check it against their README if it does not work.
2 things to look at
- no licenseNo license file was found in the repository. Code published without one is not open source by default, so using it at work is a question for whoever answers licensing questions where you are.
- 0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.
What its author says it does
Copied from the file, not written here
PR のレビュー指摘(未解決のインラインコメント)に対してコード修正を行う。PR 番号を指定する。
SKILL.md
5.0 KB, ~1.8k tokens by cl100k_base, as published. Nobody here has run it
Fix Review Skill
PR のレビュー指摘(未解決のインラインコメント)を読み取り、コード修正を行う。
このスキルはレビュー指摘への対応のみを行う。 プランに基づく新規実装は /implement-plan スキルを使用すること。
前提条件
ghCLI が認証済みであること- 対象の PR に未解決のレビュースレッドが存在すること
引数
$ARGUMENTS = <pr-number>
引数なしの場合はエラーとする。
手順
1. 未解決のレビュー指摘を取得
1-1. 未解決のレビュースレッドを取得
gh api graphql -f query='
{
repository(owner: "{owner}", name: "{repo}") {
pullRequest(number: <pr-number>) {
reviewThreads(first: 100) {
nodes {
isResolved
comments(first: 10) {
nodes {
body
path
line
diffHunk
author { login }
}
}
}
}
}
}
}'
isResolved: falseのスレッドのみを対象とする- bot コメント(CI 等)は除外する
- 各指摘の
path,line,diffHunkから修正箇所を特定する - 解釈不能な指摘はスキップし、対応できなかった旨をユーザーに報告する
未解決の指摘が存在しない場合は、その旨をユーザーに報告して終了する。
1-2. PR のブランチ情報を取得
gh pr view <pr-number> --json headRefName,baseRefName
2. プロジェクト固有情報の確認
.claude/skills/references/project-context.md の「実装ガイド」セクションを読み込み、以下を把握する:
- ビルド・フォーマットコマンドと注意事項
- 言語固有の実装規約
- テストの配置ルール
- CI に委ねてよい項目
3. PR のブランチにチェックアウト
PR の headRefName にチェックアウトし、最新を pull する。
git checkout <headRefName>
git pull origin <headRefName>
4. 指摘に基づくコード修正
各レビュー指摘に対してコード修正を行う。
修正時の注意事項
- 各指摘の
pathとlineから対象箇所を正確に特定すること diffHunkを参考に、指摘の文脈を正しく理解すること- 指摘の意図を汲み取り、表面的な修正ではなく本質的な改善を行うこと
- 既存のコードベースのパターン・命名規則に従うこと
- project-context.md に記載された言語固有の実装規約に従うこと
- 指摘に関連するテストの追加・修正が必要であれば対応すること
対応できない指摘の扱い
以下の場合は指摘をスキップし、後でユーザーに報告する:
- 指摘の意図が不明瞭で解釈できない場合
- 指摘の対象ファイル・行が現在のコードと一致しない場合
- 指摘の対応に設計判断が必要で、自動的に判断できない場合
5. ビルドとフォーマットの確認
project-context.md の「ビルド・フォーマットコマンド」セクションに記載されたコマンドを実行する。
- フォーマッターは必ずコミット前に実行すること
- ビルドやテストが失敗した場合は原因を特定し修正すること。修正後に再度実行し、成功するまで繰り返す
- CI に委ねてよい項目として記載されたテストはスキップしてよい
6. コミット
変更内容をコミットする。
- コミットメッセージは修正内容を簡潔に記載すること
- 例:
fix: improve error handling and add missing tests
- 例:
- 複数の論理的なまとまりがある場合は、適切にコミットを分割すること
- Co-Authored-By には実行時のモデル情報を使用すること
git add <files>
git commit -m "$(cat <<'EOF'
<commit message>
Co-Authored-By: <実行中のモデル名> <[email protected]>
EOF
)"
7. Push と報告
- リモートに Push する
git push origin <headRefName>
- 対応結果をユーザーに報告する:
- 対応した指摘: 各指摘に対してどのような修正を行ったかの要約
- 対応できなかった指摘: スキップした指摘とその理由
注意事項
- レビュー指摘の意図を正確に理解し、過不足のない修正を行うこと
- 推測ではなく、実際のコードを読んで確認した事実に基づいて修正すること
- 指摘されていない箇所への変更は最小限に留めること(指摘対応に必要な範囲のみ)
- ビルドが通らない状態でコミット・Push しないこと
- project-context.md に記載されたフォーマッターを忘れずに実行すること
What ships with it
Read from the repository
Just SKILL.md. No reference files, no scripts.