Skip to content

fix: replace claude-code-action with its secure drop-in replacement - #115

Merged
varunsh-coder merged 1 commit into
mainfrom
fix/404-error
Jun 18, 2026
Merged

varunsh-coder merged 1 commit into
mainfrom
fix/404-error

Conversation

@amanstep

@amanstep amanstep commented Jun 17, 2026 •

Copy link
Copy Markdown
Contributor

replace claude-code-action with its secure drop-in replacement and provide model arg to resolve 404 error about deprecated model

@amanstep amanstep changed the title fix: replace claude-code-action with its secure drop-in replacement a… fix: replace claude-code-action with its secure drop-in replacement Jun 17, 2026
Replaces the deprecated anthropics/claude-code-action@beta (which was
returning 404 errors due to an expired default Sonnet model) with the
step-security/claude-code-action drop-in fork at v1.0.146. The new
version uses an agentic execution model rather than the old direct_prompt
auto-post mode, so several adjustments were needed:

- Pin model explicitly to claude-sonnet-4-6 via claude_args
- Pass github_token explicitly so the action can post comments without
  relying on a separately installed GitHub App
- Raise job-level pull-requests / issues permissions from read to write
  so the GITHUB_TOKEN can post review comments
- Add REPO and PR NUMBER context to the prompt so Claude can target the
  right PR
- Allowlist the specific tools required for automated review via
  --allowedTools: gh pr view/diff/comment, curl, and the GitHub MCP
  inline-comment tool for line-level findings
- Instruct Claude to deliver the review via gh pr comment and the
  inline-comment MCP tool rather than as a chat response
- Add a standard review-comment format (Action Type, Passed Checks,
  Failed Checks, Warnings, Security Findings, Summary) so the posted
  comment is consistent and readable
@varunsh-coder
varunsh-coder merged commit e559213 into main Jun 18, 2026
5 checks passed
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