Skip to content

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

Merged
ben-miru merged 1 commit into
mainfrom
ben/bump-ai-review-524b5a7
Oct 12, 2026
Merged

ben-miru merged 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 6472b6d 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 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>
Comment thread .github/workflows/ai-review.yml
@github-actions

Copy link
Copy Markdown

AI review: approve for cf40bc6

This PR changes one line: it moves the reusable workflow mirurobotics/.github/.github/workflows/ai-review.yml from 6472b6d to 524b5a7. It's still pinned to a full commit SHA, and permissions, triggers and secrets are unchanged. I couldn't verify the upstream diff because this job was denied access to the compare API, so I haven't checked what 524b5a7 actually contains. I left a non-blocking inline note asking a maintainer to confirm that 524b5a7 is on .github main and only contains mirurobotics/.github#6. That matters because the bump changes how the required review gate treats earlier threads. Nothing in this repo's diff is blocking.

Review run

@ben-miru
ben-miru merged commit 00a8442 into main Oct 12, 2026
2 checks passed
@ben-miru
ben-miru deleted the ben/bump-ai-review-524b5a7 branch October 12, 2026 16:50
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