Skip to content

ci: bump ai-review workflow to 524b5a7 - #15

Closed
ben-miru wants to merge 1 commit into
mainfrom
ben/bump-ai-review-524b5a7
Closed

ben-miru wants to merge 1 commit into
mainfrom
ben/bump-ai-review-524b5a7

Conversation

@ben-miru

@ben-miru ben-miru commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Bumps the pinned mirurobotics/.github AI review workflow from 78e2fba to 524b5a7.

What changes: mirurobotics/.github#6. Before reviewing, the workflow fetches the PR's earlier inline review threads, with their replies and resolved state, and the PR conversation. The prompt then tells Claude not to re-raise points that are already settled.

  • Only the PR author, an OWNER, MEMBER or COLLABORATOR, or a resolved mark can settle a thread. None of them can clear a blocking finding unless the code backs it up.
  • If fetching the earlier comments fails, the review fails closed with an error status. Re-run the job.
  • Claude gets no new tool permissions.

This also picks up mirurobotics/.github#3 from between the old pin and #6: Claude now runs at high effort by default.
This PR is reviewed by the currently pinned version, because pull_request_target runs the workflow from the base branch. The new behaviour starts on PRs opened after this merges.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Picks up mirurobotics/.github#6: the AI reviewer reads earlier review threads and the PR conversation, so it stops re-raising points that were already answered.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ben-miru ben-miru closed this Oct 12, 2026
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