diff --git a/.claude/settings.json b/.claude/settings.json new file mode 100644 index 0000000..fe99239 --- /dev/null +++ b/.claude/settings.json @@ -0,0 +1,5 @@ +{ + "permissions": { + "additionalDirectories": ["/home/runner/.claude/plugins"] + } +} diff --git a/.github/workflows/claude-review.yml b/.github/workflows/claude-review.yml index 7eae945..9395fdf 100644 --- a/.github/workflows/claude-review.yml +++ b/.github/workflows/claude-review.yml @@ -25,15 +25,26 @@ jobs: - uses: anthropics/claude-code-action@v1 with: claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }} + allowed_bots: claude-agent-ahrzb # agents push feature branches as this App bot plugin_marketplaces: https://github.com/anthropics/claude-plugins-official.git plugins: superpowers@claude-plugins-official - claude_args: --model claude-sonnet-5 --max-turns 20 - show_full_output: true + # Grant the comment tools — without these the agent has no way to + # write anything back to the PR (this was the "nothing gets posted" bug). + claude_args: >- + --model claude-sonnet-5 --max-turns 30 + --allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)" + track_progress: true # tracking comment + PR context prompt: | - Review this pull request. + Review this pull request. Use the superpowers:requesting-code-review + skill for the review methodology, diffing the PR branch against + ${{ github.event.pull_request.base.ref }}. - Use the superpowers:requesting-code-review skill and follow it exactly. - Diff the PR branch against ${{ github.event.pull_request.base.ref }}. + Then POST the review to GitHub — a review that isn't posted is useless: + - inline comments (mcp__github_inline_comment__create_inline_comment) + for specific, line-level issues + - one top-level PR comment (`gh pr comment`) for the overall summary + and assessment Report only what you verified in the diff. No speculative findings, no - style nits. If nothing needs changing, say so in one line. + style nits. If nothing needs changing, post a one-line top-level + comment saying so. diff --git a/.gitignore b/.gitignore index 75e546b..1c09cc8 100644 --- a/.gitignore +++ b/.gitignore @@ -9,8 +9,9 @@ wheels/ # Virtual environments .venv -# Claude Code settings -.claude +# Claude Code settings (track shared project settings; ignore everything else) +.claude/* +!.claude/settings.json .worktrees/