Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
116 changes: 0 additions & 116 deletions .github/workflows/claude-code-review.yml

This file was deleted.

76 changes: 65 additions & 11 deletions .github/workflows/claude.yml
Original file line number Diff line number Diff line change
Expand Up @@ -36,9 +36,9 @@ jobs:
needs.check-team-member.outputs.is-team-member == 'true' &&
github.actor != 'github-actions[bot]' &&
(
(github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude') && !contains(github.event.comment.body, '@claude /full-review')) ||
(github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude') && !contains(github.event.comment.body, '@claude /full-review')) ||
(github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude') && !contains(github.event.review.body, '@claude /full-review')) ||
(github.event_name == 'issue_comment' && contains(github.event.comment.body, '@claude')) ||
(github.event_name == 'pull_request_review_comment' && contains(github.event.comment.body, '@claude')) ||
(github.event_name == 'pull_request_review' && contains(github.event.review.body, '@claude')) ||
(github.event_name == 'issues' && (contains(github.event.issue.body, '@claude') || contains(github.event.issue.title, '@claude')))
)
runs-on: ubuntu-latest
Expand All @@ -51,7 +51,7 @@ jobs:
pull-requests: read
issues: read
id-token: write
actions: read # Required for Claude to read CI results on PRs
actions: read
steps:
- name: Checkout repository
uses: actions/checkout@v5
Expand All @@ -65,16 +65,70 @@ jobs:
with:
anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
track_progress: true

# This is an optional setting that allows Claude to read CI results on PRs
additional_permissions: |
actions: read
# Optional: Give a custom prompt to Claude. If this is not specified, Claude will perform the instructions specified in the comment that tagged it.
# prompt: 'Update the pull request description to include a summary of changes.'
prompt: |
You are a helpful assistant for the ZenML repository. Help the user with whatever they asked.

If the user is asking for a code review (e.g., "@claude /full-review", "@claude review this", or similar), follow the detailed review guidelines below. Otherwise, just help them with their question or request.

---
CODE REVIEW GUIDELINES (use when reviewing PRs):

# Optional: Add claude_args to customize behavior and configuration
# See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md
# or https://docs.claude.com/en/docs/claude-code/cli-reference for available options
Focus on ensuring the changes are sound, clean, intentional, and void of regressions.
REPO: ${{ github.repository }}
PR NUMBER: ${{ github.event.pull_request.number }}
Context:
- You can fetch additional PR context via:
- gh pr view ${{ github.event.pull_request.number }} --json title,body,author,headRefName,baseRefName,commits
- gh pr diff ${{ github.event.pull_request.number }} --name-only
- Use the repository's CLAUDE.md (if present) and existing code patterns for conventions.
Primary Review Goals:
1) Verify Change Correctness
- Confirm the changes achieve their intended purpose
- Check for unintended side effects or regressions
- Validate edge cases are handled properly
- Ensure error paths are covered
2) Code Quality & Cleanliness
- Is the code readable and self-documenting?
- Are the changes minimal and focused?
- Do they follow existing patterns in the codebase?
- Are there any code smells or anti-patterns?
3) Intentionality Check
- Does every change have a clear purpose?
- Are there any accidental modifications?
- Is there dead code being introduced?
- Are the commit boundaries logical?
4) Potential Issues to Flag
- Performance degradations
- Security vulnerabilities
- Race conditions or concurrency issues
- Resource leaks (memory, file handles, etc.)
- Breaking changes to internal or public APIs
5) Constructive Suggestions
- Alternative approaches that might be cleaner
- Opportunities to reduce complexity
- Missing test coverage for critical paths
- Documentation gaps for complex logic
Documentation & Release Notes Review:
- Determine if user-facing behavior, APIs, CLI commands/flags, configuration options, or defaults have changed.
- Heuristics (run): gh pr diff ${{ github.event.pull_request.number }} --name-only
- If code changes are present under src/ or cli/ or other user-facing modules but docs/ are not updated, flag as "docs update needed".
- If docs/ changes exist, check if they match the code changes and are placed in the correct sections (e.g., docs/book/, docs/how-to/, docs/reference/).
- If CLI or config flags changed, ensure corresponding docs and examples are updated (including any README or quickstart sections if applicable).
- If migration or breaking changes are introduced, recommend adding an explicit migration note or changelog item.
- If documentation is needed, explicitly call it out in the review with concrete file/section suggestions.
Review Format:
- Start with a summary of what the changes accomplish.
- List any critical issues that must be addressed.
- Note minor improvements that would enhance quality.
- Acknowledge what's done particularly well.
- End with specific, actionable next steps if needed.
Output Instructions:
- Provide detailed feedback using inline-style code references in your comment for specific issues (quote file paths and line ranges when possible).
- Use top-level comments for general observations or praise.
- Use `gh pr comment` to post your review comment to the PR.
Be constructive and helpful in your feedback.
claude_args: --allowed-tools "mcp__github_inline_comment__create_inline_comment,Bash(gh
issue view:*),Bash(gh search:*),Bash(gh issue list:*),Bash(gh pr comment:*),Bash(gh
pr diff:*),Bash(gh pr view:*),Bash(gh pr list:*),Bash(bash scripts/format.sh:*)"
Loading