Skip to content

ci: Claude PR review — skip drafts, Opus 4.8 max thinking#145

Merged
vendz merged 3 commits into
devfrom
chore/claude-pr-review
Jun 22, 2026
Merged

ci: Claude PR review — skip drafts, Opus 4.8 max thinking#145
vendz merged 3 commits into
devfrom
chore/claude-pr-review

Conversation

@vendz

@vendz vendz commented Jun 22, 2026

Copy link
Copy Markdown
Member

Updates the automated Claude Code review workflow:

  • Skips draft PRs via a job-level guard (if: github.event.pull_request.draft == false); runs on ready_for_review so it triggers the moment a draft is marked ready.
  • Maximum thinking effort added to the existing Opus 4.8 model pin (--model claude-opus-4-8 --effort max).

Everything else (auth via CLAUDE_CODE_OAUTH_TOKEN, inline comments, track_progress) is unchanged.

🤖 Generated with Claude Code

- Skip draft PRs via job-level draft guard; review runs on ready_for_review.
- Add max thinking effort to the existing Opus 4.8 pin (--effort max).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Add a prompt instruction so concrete small fixes are posted as GitHub
```suggestion``` blocks (apply-able from the PR), like the Gemini reviewer.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Switch from the code-review plugin (which only posts one summary comment) to a
custom review prompt that uses mcp__github_inline_comment__create_inline_comment,
so findings land as line-anchored inline comments with apply-able suggestion
blocks. Keeps OAuth auth, draft skip, Opus 4.8 + max effort.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@VitraagVigyaan VitraagVigyaan temporarily deployed to chore/claude-pr-review - aashray-admin PR #145 June 22, 2026 00:54 — with Render Destroyed
@vendz vendz merged commit 453f57b into dev Jun 22, 2026
1 check passed
@vendz vendz deleted the chore/claude-pr-review branch June 22, 2026 01:07
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