Skip to content

feat: add more event triggers to interactive caller#2

Merged
shuse2 merged 1 commit into
mainfrom
fix/update-caller-conditions
Mar 9, 2026
Merged

feat: add more event triggers to interactive caller#2
shuse2 merged 1 commit into
mainfrom
fix/update-caller-conditions

Conversation

@Balanced02
Copy link
Copy Markdown
Contributor

Summary

  • Adds pull_request_review_comment, pull_request_review, and issues event triggers to the interactive caller example
  • Updates the if condition to check for @claude mentions across all event types
  • Updates concurrency group to handle events that use pull_request.number instead of issue.number

This is the minimal fix for what PR #1 was trying to achieve — no need to rewrite the reusable workflow.

Replaces #1 — that PR had several issues:

  1. Converted the reusable workflow to a standalone one, which would break all downstream repos
  2. Downgraded pull-requests and issues permissions from write to read, which would prevent Claude from commenting and pushing changes
  3. Changed the secrets reference from secrets.claude_code_oauth_token (caller-passed) to secrets.CLAUDE_CODE_OAUTH_TOKEN (repo secret), which doesn't work in reusable workflows
  4. Removed track_progress, use_sticky_comment, and claude_args configuration

Test plan

  • Verify Claude responds to @claude in issue comments
  • Verify Claude responds to @claude in PR review comments
  • Verify Claude responds to @claude in PR reviews
  • Verify Claude responds to @claude in new issues
  • Verify bot actors are still excluded

🤖 Generated with Claude Code

Update the caller workflow example to handle pull_request_review_comment,
pull_request_review, and issues events in addition to issue_comment.
This is the minimal change needed rather than rewriting the reusable workflow.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@shuse2 shuse2 merged commit c6151d7 into main Mar 9, 2026
@shuse2 shuse2 deleted the fix/update-caller-conditions branch March 9, 2026 13:27
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