Skip to content

Fix: deduplicate @claude triggers#82

Merged
fumikito merged 1 commit intomainfrom
fix/claude-review-dedup
Mar 23, 2026
Merged

Fix: deduplicate @claude triggers#82
fumikito merged 1 commit intomainfrom
fix/claude-review-dedup

Conversation

@fumikito
Copy link
Contributor

Summary

  • claude-assistant.ymlclaude-review.ymlissue_comment トリガー重複を解消
  • @claude → アシスタントが応答(汎用)
  • @claude auto-review → PRで完全なコードレビューを手動起動(共有ワークフロー経由)
  • claude-review.yml から pull_request_review_comment / issues トリガーを削除(アシスタントに委譲)

変更点

claude-review.yml

  • call-claude ジョブ(汎用応答)→ manual-setup + manual-review ジョブ(完全レビュー)に置換
  • @claude auto-review + PRコメント + メンバーのみで発火
  • 共有ワークフロー claude-review.yml@main を呼び出すため、auto-reviewと同品質のレビュー

claude-assistant.yml

  • @claude auto-review を含むコメントを除外条件に追加

Test plan

  • Issueで @claude 質問 → アシスタントのみ応答(1回だけ)
  • PRで @claude auto-review → 完全レビューのみ実行(アシスタントは不発火)
  • CI通過後の自動レビューは変更なし

🤖 Generated with Claude Code

Co-authored-by: Claude <noreply@anthropic.com>
@fumikito fumikito enabled auto-merge March 23, 2026 10:21
@fumikito fumikito merged commit 48f6b36 into main Mar 23, 2026
9 checks passed
@fumikito fumikito deleted the fix/claude-review-dedup branch March 23, 2026 10:23
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