Skip to content

Commit 9652f08

Browse files
authored
Fix claude maybe (#1266)
1 parent 0ce8382 commit 9652f08

2 files changed

Lines changed: 5 additions & 7 deletions

File tree

.github/workflows/claude-code-review.yml

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,19 +28,18 @@ jobs:
2828

2929
- name: Run Claude Code Review
3030
uses: anthropics/claude-code-action@v1
31-
env:
32-
PATH: /home/runner/.local/bin:${{ env.PATH }}
3331
with:
3432
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
3533
github_token: ${{ github.token }}
3634

3735
plugin_marketplaces: "https://github.com/anthropics/claude-code.git"
3836
plugins: "code-review@claude-code-plugins"
3937

38+
# NOTE: do NOT use --dangerouslyDisableSandbox (it can crash the CLI).
39+
# This flag is for non-interactive CI runs (bypasses approval prompts).
4040
claude_args: >
4141
--dangerously-skip-permissions
42-
--dangerouslyDisableSandbox
43-
--max-turns 80
42+
--max-turns 10
4443
--allowedTools
4544
"Bash"
4645
@@ -58,7 +57,7 @@ jobs:
5857
3) find .claude/rules -maxdepth 1 -name "*.md" -print | head -n 10 | xargs -I{} cat "{}" 2>/dev/null || true
5958
4) gh pr view ${{ github.event.pull_request.number }} --repo ${{ github.repository }} --json title,body,files,changedFiles,additions,deletions,headRefOid
6059
5) gh pr diff ${{ github.event.pull_request.number }} --repo ${{ github.repository }}
61-
6) Post ONE top-level PR comment, then STOP.
60+
6) Post ONE top-level PR comment titled "Claude Code Review", then STOP.
6261
6362
Output format:
6463
- Head SHA
@@ -67,7 +66,6 @@ jobs:
6766
- Findings with file + line numbers when possible
6867
- If no issues: 0–3 improvement opportunities (only if confident)
6968
70-
Post ONE top-level PR comment titled "Claude Code Review".
7169
If posting is blocked, write the full review to the GitHub Actions job summary instead, then STOP.
7270
7371
additional_permissions: |

CLAUDE.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ source ./mfc.sh load -c p -m c # Load Phoenix CPU modules
5151

5252
# Other
5353
./mfc.sh validate case.py # Validate case file without running
54-
./mfc.sh params <query> # Search ~3,400 case parameters
54+
./mfc.sh params <query> # Search 3,400 case parameters
5555
./mfc.sh clean # Remove build artifacts
5656
./mfc.sh new <name> # Create new case from template
5757
```

0 commit comments

Comments
 (0)