Skip to content

CI: Opus 5.5 + upstream claude-code-action v1.0.233 - #106

Merged
STRd6 merged 1 commit into
mainfrom
ci-opus-5-5-upstream-action
Sep 24, 2026
Merged

STRd6 merged 1 commit into
mainfrom
ci-opus-5-5-upstream-action

Conversation

@STRd6

@STRd6 STRd6 commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator
  • Pin anthropics/claude-code-action to v1.0.233 (8cf3482, same as the current v1 tag) in place of the STRd6 fork pin, which was 239 commits behind upstream.
  • Pass --model claude-opus-5-5 in claude_args (no model was set before).

Tradeoff: the fork carried the fix from anthropics/claude-code-action#1253, which is still open. Without it, when a run throws (for example, on max-turns), the execution file isn't written, so the "Upload Claude log" step is skipped for that run. Successful runs still upload their log.

🤖 Generated with Claude Code

https://claude.ai/code/session_014uvPVthmR8aWjxr2EM2CAX

Pin anthropics/claude-code-action to v1.0.233 in place of the STRd6
fork, and pass --model claude-opus-5-5.

Co-Authored-By: Claude Opus 5.5 (1M context) <[email protected]>
Claude-Session: https://claude.ai/code/session_014uvPVthmR8aWjxr2EM2CAX
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (ea2aa3b) to head (9a84321).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #106   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines         1886      1886           
  Branches       307       307           
=========================================
  Hits          1886      1886           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@greptile-apps

greptile-apps Bot commented Sep 24, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

The PR appears safe to merge if the acknowledged loss of failure logs is acceptable, though restoring those logs would improve diagnosis of failed runs.

Findings

  1. P2 Failed runs lose logs ▶

Summary

The workflow switches from a forked Claude Code action to upstream v1.0.233 and explicitly selects Opus 5.5. The switch retains successful-run log uploads but loses logs for runs that throw.

Reviews (1) · Last reviewed commit: "CI: run Claude on Opus 5.5 with upstream..."

# 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’s 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.

@STRd6
STRd6 merged commit 79a49a0 into main Sep 24, 2026
8 checks passed
@STRd6
STRd6 deleted the ci-opus-5-5-upstream-action branch September 24, 2026 20:35
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