Skip to content

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

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

Description

Restore Claude reviews by selecting the active Sonnet 4.6 US Bedrock inference profile via claude_args: --model. The retired model fails with an end-of-life 404, and the pinned action ignores the obsolete anthropic_model input, so changing that input alone would not fix reviews.

Migrate the bare @claude review path to the supported prompt input, include repository/PR context, and allow the GitHub CLI commands needed to read and publish the review. Keep custom mentions in interactive mode so the action supplies comment/conversation context. Replace ignored timeout_minutes with a ten-minute job timeout. Document usage in .github/README.md.

No application code or dependencies change. Existing AWS roles, region, permissions, and action pin are unchanged.

Issue Number

WAL-2202

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

  • actionlint -shellcheck= .github/workflows/claude-pr.yml passed using the repository's locked Nix tool source.
  • A throwaway smoke harness parsed YAML, checked all supplied inputs against the pinned action's manifest, checked the explicit model argument and timeout, and executed the prompt-selection/loading shell steps for bare, whitespace-padded, and two custom mentions. All four cases passed.
  • Reviewed the pinned action's migration guide and mode-selection code: explicit prompts select automation; omitted prompts preserve interactive context.
  • AWS documents us.anthropic.claude-sonnet-4-6 as active and available from us-west-2.

Live inference was not exercised: the sandbox has no AWS inference credentials, and comment-triggered workflows use the default branch definition. After merge, request a bare @claude review and a custom mention; confirm successful inference and posted feedback. The inference role must allow the new profile and its destination models. Application builds/unit tests are not applicable to this workflow-only change.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My code compiles correctly for both Node and Browser environments — not applicable; no application changes
  • I have commented my code, particularly in hard-to-understand areas — workflow operation documented
  • My commits follow Conventional Commits and I have properly described any BREAKING CHANGES — none
  • The ticket or github issue was included in the commit message as a reference
  • I have made corresponding changes to the documentation and on any new/updated functions and/or methods — no functions changed
  • I have added tests that prove my fix is effective or that my feature works — throwaway workflow smoke checks; no permanent implementation-detail tests
  • New and existing unit tests pass locally with my changes — not applicable; workflow-only change

Pre-PR ACI gate skipped: no .github/ai-checks/pr-risk on the base branch.

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
Contributor

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