Commit 95d4942
committed
Update Claude Code GitHub Workflow (#235)
* "Update Claude PR Assistant workflow"
* "Update Claude Code Review workflow"
* fix: allow CodeFactor bot to trigger Claude Code Review workflow
The claude-code-action@v1 blocks bot actors by default for security.
Added 'codefactor-io[bot]' to allowed_bots parameter to permit
automated CodeFactor commits to trigger the code review workflow.
This resolves the "Workflow initiated by non-human actor" error
that occurred when CodeFactor made automated commits to PRs.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
---------
Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>
fix: Correct Claude Code Review workflow plugin configuration
The Claude Code Review workflow was crashing with SDK errors due to
incorrect plugin configuration syntax introduced in commit ebf913f.
- Plugin marketplace format was incorrect: used full git URL instead of GitHub short format
- Plugin reference format was wrong: `code-review@claude-code-plugins` instead of `code-review`
- Prompt used CLI command syntax instead of natural language instructions
- Changed `plugin_marketplaces` from `https://github.com/anthropics/claude-code.git` to `anthropics/claude-code`
- Changed `plugins` from `code-review@claude-code-plugins` to `code-review`
- Replaced CLI command prompt with descriptive natural language instructions
- Kept existing `allowed_bots: 'codefactor-io[bot]'` configuration
- GitHub Actions workflow should complete without exit code 1
- No SDK crash errors in logs
- Claude successfully posts review comments on PRs
- [Claude Code Plugins Documentation](https://code.claude.com/docs/en/plugins)
- [Discover and install prebuilt plugins](https://code.claude.com/docs/en/discover-plugins)
- [Code-Review Plugin README](https://github.com/anthropics/claude-code/blob/main/plugins/code-review/README.md)
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
fix: Use full Git URL format for Claude Code plugin marketplace
Changes plugin_marketplaces from short format 'anthropics/claude-code' to
full Git URL 'https://github.com/anthropics/claude-code.git' to resolve
"Invalid marketplace URL format" error in GitHub Actions workflow.
The claude-code-action requires full Git URLs ending in .git for the
plugin_marketplaces parameter as specified in its action.yml.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
fix: Add required permissions to Claude Code Review workflow
Added additional_permissions to allow Claude bot to:
- Run GitHub CLI commands (gh pr view, etc.)
- Execute Swift build and test commands
- Use the code-review skill
This resolves permission denials that prevented the code review
workflow from functioning properly.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
fix: Remove Swift build/test permissions from code review workflow
Removed Swift build and test permissions since the Ubuntu runner
doesn't have Swift installed. Code review focuses on static analysis
of code changes, while build/test verification is handled by the
main CI workflow with proper Swift containers.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
fix: Use claude_args with --allowedTools instead of additional_permissions
The additional_permissions parameter is only for GitHub API permissions
(like actions: read), not Claude Code tool permissions.
Tool permissions should be granted via claude_args with --allowedTools:
- Bash(gh *) for GitHub CLI commands
- Skill(code-review:*) for the code-review skill
This fixes the "Invalid permission value 'Skill' for '- tool'" error.
Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
Enable download-platform for iOS, watchOS, tvOS, and visionOS
Added 'download-platform' option to various build matrices.
Enable download-platform for iOS, watchOS, tvOS, and visionOS
Added 'download-platform' option to various build matrices.1 parent cb27cef commit 95d4942
2 files changed
Lines changed: 7 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
99 | 99 | | |
100 | 100 | | |
101 | 101 | | |
| 102 | + | |
102 | 103 | | |
103 | 104 | | |
104 | 105 | | |
105 | 106 | | |
106 | 107 | | |
107 | 108 | | |
| 109 | + | |
108 | 110 | | |
109 | 111 | | |
110 | 112 | | |
| |||
113 | 115 | | |
114 | 116 | | |
115 | 117 | | |
| 118 | + | |
116 | 119 | | |
117 | 120 | | |
118 | 121 | | |
119 | 122 | | |
120 | 123 | | |
121 | 124 | | |
122 | 125 | | |
| 126 | + | |
123 | 127 | | |
124 | 128 | | |
125 | 129 | | |
126 | 130 | | |
127 | 131 | | |
128 | 132 | | |
129 | 133 | | |
| 134 | + | |
130 | 135 | | |
131 | 136 | | |
132 | 137 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
| 5 | + | |
6 | 6 | | |
7 | 7 | | |
8 | 8 | | |
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
| 39 | + | |
39 | 40 | | |
40 | 41 | | |
41 | 42 | | |
| |||
0 commit comments