Skip to content

Conversation

@strickvl
Copy link
Contributor

@strickvl strickvl commented Nov 28, 2025

Summary

  • Fixes the Claude Code Action trigger pattern from /claude to @claude in both workflow files
  • The action uses tag mode by default and expects @claude mentions as the trigger
  • The previous /claude pattern was being matched by workflow if: conditions, but the action's internal trigger detection was failing silently with: No trigger was met for @claude

Background

This fix is based on testing done in the private zenml-io/gh-action-testing repo. The Claude Code Action has a two-layer trigger system:

  1. Workflow layer: Your if: condition (e.g., contains(github.event.comment.body, '@claude'))
  2. Action layer: Internal trigger detection that looks for @claude mentions

Both layers must align for the action to execute. Previously, only the workflow layer was passing.

Changes

File Change
.github/workflows/claude.yml Updated 8 trigger patterns from /claude to @claude
.github/workflows/claude-code-review.yml Updated trigger from /claude /full-review to @claude /full-review

Test plan

  • Tested trigger patterns in zenml-io/gh-action-testing (private repo)

Notes

  • The post-comment job fixes from the learnings doc are not applicable here since these workflows use the Claude Code Action's built-in commenting via track_progress: true
  • No Codex workflow exists in this repo, so those learnings don't apply

The Claude Code Action uses tag mode by default and expects @claude
mentions as the trigger pattern. The previous /claude pattern was
being matched by workflow conditions but the action's internal
trigger detection was failing silently.
@strickvl strickvl added bug Something isn't working internal To filter out internal PRs and issues labels Nov 28, 2025
@strickvl strickvl requested a review from safoinme November 28, 2025 12:23
@github-actions
Copy link
Contributor

Documentation Link Check Results

Absolute links check failed
There are broken absolute links in the documentation. See workflow logs for details
Relative links check passed
Last checked: 2025-11-28 12:25:49 UTC

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the Claude workflow trigger patterns to align with the Claude Code Action's default tag mode, changing from /claude to @claude. The previous mismatch caused the action's internal trigger detection to fail silently, even though the workflow conditions were passing.

Key Changes:

  • Updated trigger patterns from /claude to @claude across both Claude workflow files
  • Ensured the workflow layer and action layer trigger detection now align properly

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/claude.yml Updated 4 trigger conditions to use @claude instead of /claude, excluding @claude /full-review pattern
.github/workflows/claude-code-review.yml Updated the full review trigger from /claude /full-review to @claude /full-review

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@strickvl strickvl changed the title Fix Claude workflow triggers to use @claude instead of /claude Fix Claude workflow triggers to use @claude instead of /claude Dec 1, 2025
@strickvl strickvl merged commit bf11c13 into develop Dec 1, 2025
58 of 63 checks passed
@strickvl strickvl deleted the bug/fix-bot-trigger-commands-workflows branch December 1, 2025 07:40
@strickvl strickvl added the snack label Dec 1, 2025
@strickvl strickvl linked an issue Dec 1, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working internal To filter out internal PRs and issues snack

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix Claude workflow triggers to use @claude instead of /claude

3 participants