Skip to content

ci: restore Claude reviews with a supported Bedrock model - #288

Draft
ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
WAL-2202-fix-claude-reviews
Draft

ralph-bitgo[bot] wants to merge 1 commit into
masterfrom
WAL-2202-fix-claude-reviews

Conversation

@ralph-bitgo

@ralph-bitgo ralph-bitgo Bot commented Oct 20, 2026

Copy link
Copy Markdown

What

  • Select the active Sonnet 4.6 US Bedrock inference profile with claude_args: --model in both Claude action paths.
  • Migrate bare @claude reviews to the supported prompt input with repository/PR context and GitHub CLI tools for reading and posting reviews.
  • Keep custom mentions in interactive mode, preserving comment/conversation context.
  • Replace ignored timeout_minutes with a ten-minute job timeout and document usage in .github/README.md.
  • Leave AWS roles, region, permissions, action pin, and application code unchanged.

Why

Claude reviews fail with a model end-of-life 404. The pinned action ignores the obsolete anthropic_model input, so a model-string replacement alone would still use the retired default. The prompt and timeout inputs also need their supported equivalents to restore the intended review workflow.

Test plan

  • actionlint -shellcheck= .github/workflows/claude-code.yaml passed using the BitGoJS locked Nix tool source.
  • Throwaway smoke harness: YAML parsing, all inputs checked against the pinned action manifest, explicit model argument/timeout checks, and executed prompt selection/loading for bare, whitespace-padded, and two custom mentions; all four cases passed.
  • Checked the pinned action's migration guide and mode detector, plus AWS's active Sonnet 4.6 US inference profile and us-west-2 availability.
  • After merge, invoke bare @claude and a custom mention; confirm successful inference and posted feedback. Live inference was not exercised: the sandbox has no AWS inference credentials, and comment-triggered workflows use the default branch definition. The inference role must allow the new profile and its destination models.

Application builds/unit tests are not applicable to this workflow-only change. Pre-PR ACI gate skipped: no .github/ai-checks/pr-risk on the base branch.

Ticket: WAL-2202

Select Sonnet 4.6 through claude_args and migrate review prompts to
the pinned action's supported inputs. Keep custom mentions interactive,
restore the ten-minute timeout, and document review operation.

Claude reviews fail because the default model has reached end of life
and the obsolete model input is ignored by the pinned action.

Ticket: WAL-2202
Session-Id: 146db0fb-b94c-4d9e-806e-4c150d26cfd3
Task-Id: e2770251-64bd-49f8-9e3f-2606f7e17c68
Requested-By: Zahin Mohammad <zahinmohammad@bitgo.com>
@linear-code

linear-code Bot commented Oct 20, 2026

Copy link
Copy Markdown

WAL-2202

This branch has not been deployed

No deployments
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