Skip to content

Claude auto reviewer: set model and sticky comment - #85

Merged
shrunyan merged 1 commit into
masterfrom
feat/claude-auto-reviewer-model-sticky-comments
Jun 9, 2026
Merged

Claude auto reviewer: set model and sticky comment#85
shrunyan merged 1 commit into
masterfrom
feat/claude-auto-reviewer-model-sticky-comments

Conversation

@mattezra

@mattezra mattezra commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@mattezra mattezra self-assigned this Jun 8, 2026
@mattezra
mattezra requested a review from shrunyan June 8, 2026 23:50
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Code Review

Overview: This PR makes three improvements to the Claude auto-reviewer GitHub Actions workflow:

  1. Adds use_sticky_comment: true so the reviewer updates a single comment per PR rather than posting a new one on each synchronize event — cleaner UX and avoids comment spam.
  2. Pins the model to claude-sonnet-4-6 for consistency and reproducibility.
  3. Raises --max-turns to 25 to allow the reviewer to handle larger/more complex PRs.

Looks good overall. A few notes:

  • Empty PR body — the description is blank. Even a one-liner explaining why these values were chosen (e.g. the default turn limit was too low for multi-file PRs) would help future maintainers.

  • Model pinningclaude-sonnet-4-6 is the right ID format. Keep in mind this will need a manual update when you want to move to a newer model; it won't auto-upgrade. That's usually the desired behaviour for CI workflows, so this is fine.

  • --max-turns 25 — see inline comment. This is functionally fine but worth a note for cost awareness.

  • use_sticky_comment + cancel-in-progress: true — these two settings pair well together: rapid-fire pushes cancel the in-flight review and the sticky comment gets a single final update rather than a pile of partial comments.

No blocking issues. Changes are low-risk configuration adjustments.

Comment thread .github/workflows/claude-auto-reviewer.yml
@shrunyan
shrunyan merged commit 6cf8e8a into master Jun 9, 2026
1 check passed
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