Skip to content
Merged
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
6 changes: 2 additions & 4 deletions .github/workflows/claude.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,10 +37,7 @@ jobs:

- name: Run Claude Code
id: claude
# Temporary fork pin with fix for execution-log loss on max-turns.
# Revert to anthropics/claude-code-action@<v1-SHA> once upstream PR lands:
# https://github.com/anthropics/claude-code-action/pull/1253
uses: STRd6/claude-code-action@be3b21a974a263b4a592276c2d732fb6bbd92e7b
uses: anthropics/claude-code-action@8cf3482550831fb35a4fc3fbf7ca139cf8028b4c # v1.0.233

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Failed runs lose logs

The upstream action lacks the fork鈥檚 fix for writing an execution file when a run throws, including when it reaches the 100-turn limit. Because the upload step requires steps.claude.outputs.execution_file, it skips those runs even with always(). This makes failed Claude runs harder to diagnose.

with:
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}

Expand All @@ -56,6 +53,7 @@ jobs:
claude_args: |
--allowedTools Edit,MultiEdit,Write,Read,Glob,Grep,LS,Bash(git:*),Bash(pnpm build:*),Bash(pnpm test:*),Bash(gh pr:*),Bash(gh issue:*)
--max-turns 100
--model claude-opus-5-5

- name: Upload Claude log
if: always() && steps.claude.outputs.execution_file
Expand Down
Loading