fix: prevent duplicate workflow runs for PRs

Add conditional logic to ensure only one workflow runs per PR:
- pull_request_target: Only for fork PRs (external contributors)
- pull_request: Only for same-repo PRs (internal branches)

This prevents both triggers from firing simultaneously and causing
conflicts or duplicate reviews.
This commit is contained in:
LUIS NOVO 2026-01-13 19:05:16 -03:00
parent dfc7235064
commit 03eb9b3555

View file

@ -14,11 +14,10 @@ on:
jobs:
claude-review:
# Optional: Filter by PR author
# if: |
# github.event.pull_request.user.login == 'external-contributor' ||
# github.event.pull_request.user.login == 'new-developer' ||
# github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'
# Run for fork PRs (via pull_request_target) OR same-repo PRs (via pull_request), but not both
if: |
(github.event_name == 'pull_request_target' && github.event.pull_request.head.repo.full_name != github.repository) ||
(github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository)
runs-on: ubuntu-latest
permissions: