docs: マージ後の振り返りステップをpr-review-flow skillに追加 - #39
Conversation
ユーザー依頼: Issueクローズのたびに作業内容を振り返り、次回の 作業効率・トークン効率の改善点をまとめる習慣を standing rule として 定着させる。skillに書いておくことで将来のセッションでも自動的に 思い出せるようにした。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 31 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughPRマージ後またはIssueクローズ後に、成功点と次回の改善点を記録する手順を追加します。再利用可能な知見は関連ドキュメントやメモリへ反映し、更新先をコメントに記録します。 Changesマージ後振り返り
Estimated code review effort: 1 (Trivial) | ~2 minutes Possibly related issues
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
レビュー結果このPRは 内容について追加された「マージ後の振り返り」節は、CLAUDE.mdの「壊れたらCIが赤くなることで品質を担保する」という前提とは異なる、CIで検証できない運用上の習慣づけです。ただしこのskillファイル自体が既に手順・運用ルールを記述する場所であり、他の節(Draftフェーズの反復、quota失敗時の見分け方など)も同種の非機械的な手順なので、位置づけとしては一貫しています。内容も「うまくいった点/次回改善したい点を数行で」と簡潔さを保つ指示になっており、CLAUDE.mdの簡潔さの精神とも合致しています。 気になった点(ブロッキングではなく確認事項)
総評docs/skillの追加としては簡潔で目的も明確、既存ドキュメントとの矛盾も見当たりません。上記のIssue化省略の扱いについて方針だけ確認できればマージして問題ないと思います。 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/pr-review-flow/SKILL.md:
- Around line 85-92: Update the post-merge retrospective requirements in the PR
workflow so the report is recorded in a fixed location, such as a PR or Issue
comment, and completion includes verifying that it was posted. Replace the
optional “consider updating” guidance with a requirement to update the relevant
skill, CLAUDE.md, or memory when findings are reusable or needed for recurrence
prevention, and record the update destination in the retrospective.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 8ce5db26-382a-4227-9405-8c25bf7da7f8
📒 Files selected for processing (1)
.claude/skills/pr-review-flow/SKILL.md
CodeRabbit/Claudeレビューの指摘を反映。Issue未作成時の発火タイミングと、 振り返りの記録先(PR/Issueコメント)、知見反映を「検討」から要件へ変更。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Round 1 レビュー指摘の分類と対応
commit 7b12d8d で対応。lint/typecheck/testはすべてグリーン。 |
レビュー結果このPRは
上記観点はコード変更PRではないため素通しですが、内容自体は「マージ後の振り返り」という新しい運用ルールの追加なので、指示文としての一貫性・自律実行時の詰まりにくさの観点で見ました。インラインで3点コメントしています。
いずれも運用ドキュメントの記述の詰めの甘さで、実害のあるバグではありません。内容の方向性(振り返りを機械的な習慣として定着させる)自体はCLAUDE.mdの「機械が止められるかを先に考える」という方針とも整合しており妥当だと思います。 |
claude-reviewの指摘を反映。PR/Issue両方存在する場合の記録先優先順位、 Issue未クローズ時に記録が止まらないようフォールバックを追加。CLAUDE.mdの Issue化必須ルールへの新規の例外は書かず、トリガーをPRマージ時点の記録に統一。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Round 2 レビュー指摘の分類と対応
commit 5328dd5 で対応。lint/typecheck/testはすべてグリーン。 |
レビュー結果このPRは
内容についてすでに2ラウンドの
気になった点(ブロッキングではない)インラインで1点コメントしました。「クローズは後追いで確認する」の後追いを実行する具体的なトリガーが未定義で、実運用では確認自体が忘れられるリスクがあります。次回以降で同種のフォールバックが増えるようなら明文化を検討してください。 総評docs/skillの追加として簡潔で、既存ドキュメントとの矛盾もありません。上記nit以外に指摘はなく、マージして問題ないと思います。 |
Round 3 レビュー指摘の分類と対応
3ラウンドの反復でCI(lint/typecheck/unit-test/db-test/types-check)は全てグリーン、claude-review/codex-review/CodeRabbitの実質的な指摘はここまでで解消済みと判断し、このラウンドで反復を終了する。Draftのまま維持する。 |
「後追いで確認する」に具体的なトリガーがなく形骸化しうる、という指摘。 「次にそのIssue/PRに触れる機会に確認する」と起点を明記した (能動的な確認タスクは増やさない、軽量さは維持)。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
レビュー指摘分類 (commit 3bb4e6d)
Draft中の指摘はこれで出尽くしました。CI確認後Ready化します。 |
レビュー結果(総評)このPRは
内容について既に
内容は「うまくいった点/次回改善したい点を数行で」という簡潔さの指示になっており、 気になった点(いずれも非ブロッキング)
総評docs/skillの追加として簡潔で目的も明確、既存ドキュメント( |
レビュー指摘分類 (commit 3bb4e6d、追加分)
CI全green、これ以上の指摘なし。Ready化します。 |
There was a problem hiding this comment.
Pull request overview
pr-review-flow skill（PR運用手順）に「マージ後の振り返り」ステップを追加し、PR/Issue完了ごとに短い振り返りを記録して次回の作業効率・トークン効率の改善につなげる運用を明文化するPRです。
Changes:
- PRマージ後に、PRまたは紐づくIssueへ短い振り返りコメントを残す手順を追加
- 「うまくいった点」「次回改善したい点」の記録観点を明記
- 再利用可能な知見があれば skill /
CLAUDE.md/ メモリへ反映し、反映先をコメントに記録する方針を追加
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
マージ後の振り返りうまくいった点:
次回改善したい点:
|
反映先の追記上記の振り返り2点をトリアージし、実際に反映しました。
|
ユーザー指摘: 振り返りコメントをIssueに残しているのに、対応(skill/CLAUDE.md更新) 自体はIssue化せず直接PRにしていた(PR #39, #42自体もその例)。次回以降は規模を 問わずIssueを立ててから対応する運用に統一する。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
* docs: マージ後の振り返りに改善点の対応方針を明文化 PR #41の振り返り(Issue #40)で「改善点をmemoryに残したが、その判断基準や memory自体の肥大化対策がskillに書かれていなかった」ことに気づいた。 対応方針(memory / このskillやCLAUDE.md / 別Issue / 対応不要)の判断基準と、 MEMORY.mdの200行切り捨てを踏まえた棚卸しの目安を明記する。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * docs: 改善点の対応をIssue化必須に変更 ユーザー指摘: 振り返りコメントをIssueに残しているのに、対応(skill/CLAUDE.md更新) 自体はIssue化せず直接PRにしていた(PR #39, #42自体もその例)。次回以降は規模を 問わずIssueを立ててから対応する運用に統一する。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * docs: memoryの参照先を明確化 Copilotの指摘(PR #42): 「MEMORY.md」がこのリポジトリ内のファイルだと 誤解される書き方だった。Claude Code自身のメモリシステムを指すことを明記する。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
概要
ユーザー依頼(Issue #36・#37対応中の会話より): PRマージ・Issueクローズのたびに作業を振り返り、次回の作業効率・トークン効率の改善点をまとめる習慣を定着させる。
変更内容
.claude/skills/pr-review-flow/SKILL.md— 「マージ後の振り返り」節を追加。うまくいった点/次回改善したい点を短く報告するステップを明記備考
Issue化はせず直接対応(スコープの小さいdocs追加のため)。
Summary by CodeRabbit