Skip to content

feat(ci): add @claude PR reviews to AKM#232

Merged
pranavjain97 merged 2 commits into
masterfrom
wal-1545-add-claude-reviews
Jun 11, 2026
Merged

feat(ci): add @claude PR reviews to AKM#232
pranavjain97 merged 2 commits into
masterfrom
wal-1545-add-claude-reviews

Conversation

@bitgo-ai-agent-dev

Copy link
Copy Markdown

Summary

  • Adds code_review_prompt_path and always_apply_review_prompt: true to the existing claude-code.yaml workflow so that every @claude mention on a PR triggers an AKM-specific code review
  • Adds .github/prompts/code-review.md with a review checklist tailored to this repo's domain: cryptographic/key-management safety, MPC protocol sequencing, mTLS config, monetary-math correctness, and test coverage

What changed

.github/workflows/claude-code.yaml — two new with: parameters wired to the existing BitGo/github-ai-assistant reusable workflow:

  • code_review_prompt_path: .github/prompts/code-review.md
  • always_apply_review_prompt: true — the review prompt is applied on every @claude invocation; any words the user adds after @claude steer the review rather than replace it

.github/prompts/code-review.md — AKM-specific review prompt covering:

  • Correctness & logic across both advanced-wallet-manager and master-express modes
  • Cryptographic/key-management safety (critical: no key material in logs)
  • MPC/DKG/DSG round sequencing and signing path correctness
  • Resiliency of key-provider and AWM HTTP calls
  • Input validation, mTLS cert handling, and no-floats-for-amounts rules
  • Test coverage expectations

How it was tested

Matches the established pattern used in bitgo-microservices, kafka-client, and indexer. No runtime changes — CI-only.

Linear

WAL-1545

@linear-code

linear-code Bot commented Jun 10, 2026

Copy link
Copy Markdown

WAL-1545

@bitgo-ai-agent-dev bitgo-ai-agent-dev Bot force-pushed the wal-1545-add-claude-reviews branch from 6b11d82 to 81bceda Compare June 10, 2026 21:29
Wire up the github-ai-assistant reusable workflow to run an
AKM-specific code review prompt on every @claude mention in
a PR. Add code_review_prompt_path and always_apply_review_prompt
so the review fires for any @claude invocation, with the
user's words adding emphasis on top of the default checklist.

The review prompt covers the areas most critical to this repo:
cryptographic / key-management safety, MPC protocol sequencing,
mTLS config, monetary-math correctness, and test coverage.

Ticket: WAL-1545
Session-Id: 370d5696-0241-42b2-9607-273a4cd2ed99
Task-Id: d63c168d-4c13-4704-bf9e-61f2c30b9985
@bitgo-ai-agent-dev bitgo-ai-agent-dev Bot force-pushed the wal-1545-add-claude-reviews branch 2 times, most recently from 770f84b to b26ec47 Compare June 10, 2026 21:37
@pranavjain97 pranavjain97 marked this pull request as ready for review June 11, 2026 17:07
@pranavjain97 pranavjain97 requested review from a team as code owners June 11, 2026 17:07
Prevent unauthorized invocation by external contributors by gating
on OWNER/MEMBER author_association.

WAL-1545
@pranavjain97 pranavjain97 force-pushed the wal-1545-add-claude-reviews branch from dd2095f to 4ebc5d2 Compare June 11, 2026 17:31
@pranavjain97 pranavjain97 merged commit 06e0356 into master Jun 11, 2026
21 checks passed
@pranavjain97 pranavjain97 deleted the wal-1545-add-claude-reviews branch June 11, 2026 18:58
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