Skip to content

chore(camNC) Add Claude PR review workflow and pre-push review rule - #614

Merged
mergify[bot] merged 4 commits into
mainfrom
claude/project-thread-32voz6
Nov 11, 2026
Merged

mergify[bot] merged 4 commits into
mainfrom
claude/project-thread-32voz6

Conversation

@Schibum

@Schibum Schibum commented Nov 11, 2026 •

Copy link
Copy Markdown
Collaborator

Requested by schibum

Claude review workflow

The new .github/workflows/claude-review.yml runs anthropics/claude-code-action (pinned to v1.0.271) with the code-review plugin, using a Claude subscription token stored in the CLAUDE_CODE_OAUTH_TOKEN secret. It posts inline review comments.

  • Triggers: a PR that opens, is reopened or leaves draft. Also an @claude review comment from an owner, member or collaborator. It does not run on every push, to save subscription usage.
  • Model: Sonnet.
  • Fork PRs are skipped on both paths. A step checks isCrossRepository before anything checks out PR code, because the job holds the token and a PR-write GITHUB_TOKEN.
  • Concurrency: set at job level, so unrelated PR comments (including Mergify's) don't cancel a running review.
  • Comments are posted with GITHUB_TOKEN, so no GitHub app install is needed.
  • Not a required check, so Mergify's queue conditions are unchanged.

Until the secret is added, the job fails on PRs. That doesn't block anything, because the check is not required.

AGENTS.md

  • Before opening a PR or pushing to one, an agent has a subagent review its diff against main. It fixes the confirmed findings and reruns the affected checks. Pushes to a branch with no open PR run no CI, so they are exempt.
  • New note: address the Claude review findings before queueing, and re-run the review with @claude review.

Checks

  • prettier --check passes on both files, and the workflow parses as YAML.
  • A subagent reviewed this diff. Its findings are fixed: the concurrency group was cancelling runs on unrelated comments, the comment path was checking out fork code, and one AGENTS.md sentence had the wrong clause.
  • actionlint was not run.
  • The workflow can't run until the secret exists. Its first real run will be on the next PR after that.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP

Every PR push runs CI, so catch bugs locally first.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@Schibum Schibum self-assigned this Nov 11, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown
Codex Review Summary: ❌ Failed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ❌ Failed 2026-11-11T07:09:50.367917Z
We couldn’t complete this review. Comment @codex review to try again.
09e2776 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@mergify

mergify Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Queue requested — the merge queue status continues in this comment ↓.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
camnc-preview 9719f82 Commit Preview URL

Branch Preview URL
Nov 11 2026, 07:38 AM

Reviews same-repo PRs when they open, reopen or leave draft, and on a
collaborator's "@claude review" comment. Not a required check.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP
@Schibum Schibum changed the title docs(camNC) Review the diff with a subagent before pushing to a PR chore(camNC) Add Claude PR review workflow and pre-push review rule Nov 11, 2026
The code-review plugin stops when Claude has already commented, so a re-run after fixes did nothing. Also point it at AGENTS.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01QRhGQtkiBiWR8ZSACmv6PP

Schibum commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator Author

·@·m·ergifyio q·ueue


Generated by Claude Code

@Schibum

Schibum commented Nov 11, 2026

Copy link
Copy Markdown
Collaborator Author

@Mergifyio queue

@mergify

mergify Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

✅ Merged into main as 91acf54 at 13:10 UTC

50 min in the queue, 15 min of it CI.

So far

10 merge conditions met
  • check-success = Behaviour specs (1/4)
  • check-success = Behaviour specs (2/4)
  • check-success = Behaviour specs (3/4)
  • check-success = Behaviour specs (4/4)
  • check-success = Run tests
  • check-success = Tests (app-browser-1)
  • check-success = Tests (app-browser-2)
  • check-success = Tests (app-browser-3)
  • github-review-approved · required by GitHub repository ruleset rule main
  • any of · required by GitHub repository ruleset rule main:
    • check-success = @github-actions/Run tests
    • check-neutral = @github-actions/Run tests
    • check-skipped = @github-actions/Run tests

Times in UTC · dashboard · how the merge queue works

@mergify
mergify Bot merged commit 91acf54 into main Nov 11, 2026
11 checks passed
@mergify
mergify Bot deleted the claude/project-thread-32voz6 branch November 11, 2026 13:10
@mergify mergify Bot removed the queued label Nov 11, 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.

3 participants