Skip to content

Add Claude Code GitHub Workflow#159

Merged
shdwmtr merged 2 commits intomainfrom
add-claude-github-actions-1775533879350
Apr 7, 2026
Merged

Add Claude Code GitHub Workflow#159
shdwmtr merged 2 commits intomainfrom
add-claude-github-actions-1775533879350

Conversation

@shdwmtr
Copy link
Copy Markdown
Member

@shdwmtr shdwmtr commented Apr 7, 2026

No description provided.

@shdwmtr shdwmtr merged commit 3da9e54 into main Apr 7, 2026
26 of 27 checks passed
@shdwmtr
Copy link
Copy Markdown
Member Author

shdwmtr commented Apr 7, 2026

@claude review this

@claude
Copy link
Copy Markdown

claude bot commented Apr 7, 2026

Claude finished @shdwmtr's task in 37s —— View job


Claude Code Workflow Review

  • Gather context
  • Read workflow files
  • Provide review feedback

This PR adds two workflow files to integrate Claude Code into the repository. The PR is already merged — this review is informational.

claude.yml — On-demand Claude interactions

Issues:

  1. Missing write permissions (claude.yml:22-26): Claude needs pull-requests: write and issues: write to post comments in response to @claude mentions. Without these, the action will fail to write its responses. This explains why this very workflow run needed special handling.

    permissions:
      contents: write       # needed if Claude pushes changes
      pull-requests: write  # needed to post PR comments
      issues: write         # needed to post issue comments
      id-token: write
      actions: read
  2. Redundant actions: read (claude.yml:25 and claude.yml:40-41): actions: read appears in both the permissions block and the additional_permissions input. The latter is redundant — the GitHub token already gets the permission from the permissions block.


claude-code-review.yml — Automatic PR review

Issues:

  1. Missing write permissions (claude-code-review.yml:22-27): Same problem as above — Claude needs pull-requests: write to post review comments. Currently only has read permissions.

  2. code-review plugin dependency (claude-code-review.yml:39-40): This workflow pulls a plugin from an external marketplace (https://github.com/anthropics/claude-code.git) via plugins: 'code-review@claude-code-plugins'. This is a supply chain risk — if that repository or branch is compromised or removed, the workflow breaks or worse, runs malicious code. Consider pinning to a specific commit SHA rather than a mutable branch reference, or verify this is an official/trusted Anthropic source.

  3. fetch-depth: 1 (claude-code-review.yml:32-33): Shallow clone may cause issues if Claude needs to compare the PR diff against a base that isn't the immediate parent. For most PRs this is fine, but it can cause git diff failures on PRs with many commits. Consider fetch-depth: 0 for review workflows.


Summary

The primary actionable issue is missing write permissions on both workflows — the Claude action cannot post comments without pull-requests: write and issues: write. These should be patched.

@ArjixWasTaken
Copy link
Copy Markdown

ArjixWasTaken commented Apr 7, 2026

@claude review this

Edit: looks like it only runs for those with write access, good

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