Skip to content

Conversation

@strickvl
Copy link
Contributor

Summary

  • Fixed duplicate Claude workflow triggers when using @claude /full-review
  • Added negative conditions to claude.yml to exclude /full-review comments

Problem

When someone commented @claude /full-review on a PR, both workflows would trigger:

  1. claude.yml (generic @claude handler)
  2. claude-code-review.yml (specific /full-review handler)

This resulted in two separate review comments being posted.

Solution

Added !contains(...) exclusion checks to the three comment-related trigger conditions in claude.yml:

  • issue_comment
  • pull_request_review_comment
  • pull_request_review

Now the workflows are mutually exclusive:

  • @claude /full-review → only triggers claude-code-review.yml
  • @claude (any other text) → only triggers claude.yml

Test plan

  • Comment @claude /full-review on a PR and verify only one review is posted
  • Comment @claude (without /full-review) and verify the generic workflow runs

Added exclusion conditions to claude.yml to prevent it from triggering when @claude /full-review is used. This ensures only claude-code-review.yml handles full review requests, eliminating duplicate review comments.
@strickvl strickvl merged commit c5d95b8 into main Nov 14, 2025
2 checks passed
@strickvl strickvl deleted the bug/duplicate-claude-bots branch November 14, 2025 16:46
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.

2 participants