chore(ci): add codegenie review workflow - #42
Open
pratikspatil024 wants to merge 1 commit into
Open
Conversation
On-demand AI review, triggered by commenting "codegenie review" on a PR. Mirrors the setup already running in bor. Upstream-merge PRs are never triggered, so the config is tuned for PoS-authored changes. Consensus, state, types, blocksync, privval, mempool, and abci are marked critical; store, evidence, statesync, and p2p follow. The light client and the upstream e2e harness are marked low. Generated protobuf and mockery output is skipped. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - uses: 0xPolygon/codegenie@v0.5.1 |
pratikspatil024
marked this pull request as ready for review
July 30, 2026 05:08
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds on-demand codegenie AI review to this fork, matching what already runs in
bor and heimdall-v2. Reviews are requested explicitly by commenting
codegenie reviewon a PR — no automatic run on open or on push, so this addsnothing to the normal CI path and blocks nothing.
The workflow file is byte-identical to bor's:
codegenie@v0.5.1,anthropic/claude-opus-5:high, a preflight that fails fast ifCLAUDE_API_KEYis unset, and a checkout pinned to the default branch ratherthan the PR head so a PR cannot modify the reviewer that reviews it.
CLAUDE_API_KEYis an org secret already available to this repo.Because this fork tracks upstream,
codegenie.tomlis tuned for PoS-authoredchanges rather than upstream merges — and since the trigger is comment-only,
upstream-merge PRs simply never invoke it.
consensus/**(state machine, voting, timeouts, WAL replay),state/**(block execution, validator-set updates, app-hash determinism),types/**(vote/block/validator-set types and voting-power validation),blocksync/**(catch-up and peer-height reporting),privval/**(double-sign protection),
mempool/**, andabci/**.store/**(block store, pruning and compaction),evidence/**,statesync/**,p2p/**.light/**(light client is not used in the PoS stack) andtest/**(upstream e2e harness).Generated output is skipped: protobuf bindings and the mockery destinations
under
**/mocks/**.Executed tests
No runtime behavior changes, so there is nothing for CI to exercise. What was
verified instead:
(
src/config/schema.ts), which is.strict()and requiresreasonon everyrule. All enum values valid.
matches zero files. This caught an
api/**skip rule copied from anotherrepo; this fork has no such directory, so it was dropped.
src/git/file-classifier.ts:258-266(last matching rule wins; skips only compete with rules that set a
processingMode), then spot-checked:consensus/state.go→ critical[consensus],blocksync/reactor.go→ critical[sync],privval/signer_client.go→ critical[keys],p2p/switch.go→ high,light/client.go→ low,abci/types/types.pb.go→ skip,evidence/mocks/block_store.go→ skip.yamllintclean on the workflow against this repo's own.github/linters/yaml-lint.yml, sincemarkdown-linter.ymlrunssuper-linter with
VALIDATE_YAML: trueover the whole codebase.End-to-end behavior of this exact workflow file is already proven in bor —
20 successful runs since 2026-07-26.
Rollout notes
Not consensus-affecting. No coordinated upgrade, no operator-facing change,
nothing in the built binary. CI-only, and non-blocking by construction: the
workflow only ever fires from an explicit PR comment, and codegenie posts a
COMMENT-type review that never approves or requests changes.No effect on upstream syncs. Findings are advisory and do not replace human
review.