Skip to content

ci: allow fresh claude reviews - #906

Draft
piotr-iohk wants to merge 1 commit into
masterfrom
fix/claude-fresh-reviews
Draft

piotr-iohk wants to merge 1 commit into
masterfrom
fix/claude-fresh-reviews

Conversation

@piotr-iohk

Copy link
Copy Markdown
Collaborator

This PR adds an explicit fresh-review option so Claude can review follow-up fixes after it has already commented on a pull request. The migration repair loop in #852 exposed the existing plugin eligibility check, which exits whenever an earlier Claude comment exists.

Description

  • Adds an optional force_review workflow-dispatch input that requests a full current-head review and a verdict identifying the exact reviewed SHA.
  • Keeps the existing plugin review path as the default and uses the existing tool allowlist and GitHub permissions for fresh reviews.

Design

N/A — no UI changes.

Preview

N/A

QA Notes

Journeys

N/A — no user-visible behaviour change.

Manual Tests

N/A

Automated Checks

  • ran Ruby YAML parsing and git diff --check — the workflow parses and contains no whitespace errors.
  • (after merge) Dispatch claude-code-review.yml for an already-reviewed PR with force_review=true — a fresh verdict must identify the current full head SHA. GitHub rejects modified Claude workflows until they match the trusted default-branch workflow, so this check cannot run from the repair branch.

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.

1 participant