Skip to content

fix: @claude の返信を過去レビューと誤認して差分レビューになる問題を修正 - #143

Merged
fumikito merged 1 commit into
mainfrom
fix/claude-review-previous-detection
Oct 15, 2026
Merged

fumikito merged 1 commit into
mainfrom
fix/claude-review-previous-detection

Conversation

@fumikito

Copy link
Copy Markdown
Member

問題

@claude の返信しかない PR でも、自動レビューが差分レビュー(INCREMENTAL REVIEW)のモードで動いていました。

実例は hametuha/chiramise#16 です。

  • 経緯: Claude Assistant が Issue から作ったブランチの PR で、Assistant がテストを直したあと、CI が通って自動レビューが走りました。
  • 結果: 初回レビューなのに差分モードで動き、レビューのコメントが投稿されませんでした。ラベルは付いています。

原因

Fetch PR conversation and commits ステップは、claude[bot] のコメントをすべて過去のレビューとして数えています。Claude Assistant の返信も同じ claude[bot] から投稿されるため、レビュー済みだと誤って判定されていました。

修正

claude[bot] のコメントのうち、本文に「AI トリアージ」を含むものだけを数えます。

  • 通常のレビューの見出し(AI トリアージ: PR #n)と、差分レビューの見出し(AI トリアージ(差分レビュー): PR #n)の両方が該当します。
  • 会話の全文(pr_conversation)は、これまでどおり Assistant の返信も含めてプロンプトに渡します。

jq の条件は、Assistant の返信1件とトリアージ2件の入力で 2 になることを確認しました。

確認

マージ後、chiramise#16 に @claude auto-review とコメントし、通常レビューとしてコメントが投稿されることを確認します。

🤖 Generated with Claude Code

claude[bot] のコメントをすべて過去レビューとして数えていたため、
@claude への返信しかない PR でも INCREMENTAL REVIEW になっていた。
「AI トリアージ」を含むコメントだけを数える。

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito
fumikito enabled auto-merge October 15, 2026 16:47
@fumikito
fumikito merged commit 3c68efc into main Oct 15, 2026
9 checks passed
@fumikito
fumikito deleted the fix/claude-review-previous-detection branch October 15, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant