Skip to content

ci(triage): fire on PR comments + add is_pr/pr context (port of adcp#3316)#1028

Merged
bokelley merged 1 commit into
mainfrom
bokelley/triage-pr-comments
Apr 26, 2026
Merged

ci(triage): fire on PR comments + add is_pr/pr context (port of adcp#3316)#1028
bokelley merged 1 commit into
mainfrom
bokelley/triage-pr-comments

Conversation

@bokelley

Copy link
Copy Markdown
Contributor

Port of adcontextprotocol/adcp#3316. Same change applied here so PR review feedback in this repo also fires the triage routine.

Drops the github.event.issue.pull_request == null filter, adds is_pr flag and pr block to the routine payload (head_ref, base_ref, draft, state), and widens the self-loop guard to skip Fixed by Claude Code in addition to Triaged by Claude Code.

🤖 Generated with Claude Code

Port from adcontextprotocol/adcp#3316. The previous workflow filtered
PR comments out via `github.event.issue.pull_request == null` with a
comment claiming an "auto-fix" workflow handled them — but no auto-fix
workflow exists. PR review feedback sat unactioned until someone
manually invoked /triage.

Drop the filter. Route PR comments to the same triage routine, with
two adaptations on the payload so the routine can branch:

- is_pr: true|false flag at the top of the prompt
- pr block with head_ref, base_ref, draft, state when is_pr=true
- MODE directive when is_pr=true: apply requested fix as a follow-up
  commit on the PR's head branch, or post a reply if it's a question

Self-loop guard widened to skip "Fixed by Claude Code" in addition to
"Triaged by Claude Code" so PR-fix replies don't re-trigger.

Concurrency group already keys on issue.number, which GitHub assigns
sequentially across both issues and PRs, so PR runs won't collide
with issue runs.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bokelley bokelley merged commit 15ca89a into main Apr 26, 2026
9 checks passed
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