Skip to content

ci: add Claude/Codex/Pi AI reviewers on PR ready + commit#1383

Merged
rubenfiszel merged 1 commit intomainfrom
ai-pr-reviewers
May 4, 2026
Merged

ci: add Claude/Codex/Pi AI reviewers on PR ready + commit#1383
rubenfiszel merged 1 commit intomainfrom
ai-pr-reviewers

Conversation

@rubenfiszel
Copy link
Copy Markdown
Contributor

Summary

Mirrors the AI-reviewer setup that runs on the windmill repo. Adds three independent reviewers that each fire on pull_request: [opened, ready_for_review, synchronize] (i.e. on mark-as-ready and on every new commit):

  • Claude (pr-ready-review.yml) — anthropics/claude-code-action, posts inline comments + a top-level summary.
  • Codex (codex-pr-review.yml) — OpenAI gpt-5.5, posts a single ## Codex Review comment.
  • Pi (pi-pr-review.yml) — DeepSeek-backed, posts a single ## Pi Review comment.

A fourth workflow (pr-review-commands.yml) lets reviewers re-trigger any of them by commenting /review, /claude, /codex, or /pi on the PR.

All four gate on windmill-labs org membership by reusing the existing windmill-labs/windmill/.github/workflows/check-org-membership.yml@main workflow (the same one already used by claude.yml here), so no new shared-workflow file is needed.

Docs-specific review prompt

.github/review-prompt-shared.md is rewritten for this repo: it focuses on AGENTS.md / writing_style_guide.md compliance, broken internal links/anchors, frontmatter description presence and length, sidebar registration, EE-tier mention//pricing linking, JSON-LD on non-doc pages, and the style rules (sentence case headings, no em dashes, no bold misuse, etc.) instead of the backend/Rust/EE concerns from the windmill repo.

Required secrets

The workflows skip cleanly if their respective secret is not set, so the PR is safe to merge before secrets land:

  • CLAUDE_CODE_OAUTH_TOKEN (already configured for claude.yml)
  • ORG_ACCESS_TOKEN (already configured)
  • CODEX_AUTH_JSON (optional — Codex review skips if absent)
  • DEEPSEEK_API_KEY (optional — Pi review skips if absent)

Test plan

  • Open this PR and confirm Claude review fires automatically.
  • Push a follow-up commit and confirm Claude/Codex/Pi each re-fire on synchronize.
  • Comment /review and confirm all three reviewers re-run.
  • Comment /claude (or /codex, /pi) and confirm only the targeted one runs.
  • Open a draft PR and confirm reviewers do not run until it is marked ready.
  • Confirm a non-org-member's PR is skipped by the membership gate.

🤖 Generated with Claude Code

Adds three AI reviewer workflows mirroring the setup in the windmill repo:
- Claude (anthropics/claude-code-action)
- Codex (OpenAI gpt-5.5)
- Pi (DeepSeek)

Each fires on PR opened, ready_for_review, and synchronize (every new
commit), and gates by windmill-labs org membership via the reusable
check-org-membership workflow already used by claude.yml.

Also adds a docs-tailored shared review prompt that focuses on
AGENTS.md / writing_style_guide.md compliance, broken links/anchors,
frontmatter, sidebar registration, and JSON-LD instead of the backend /
EE concerns from the windmill repo.

A pr-review-commands workflow lets reviewers re-trigger any of the
three via /review, /codex, /pi, or /claude comments.
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying windmilldocs with  Cloudflare Pages  Cloudflare Pages

Latest commit: ea79163
Status: ✅  Deploy successful!
Preview URL: https://a49f1e3f.windmilldocs.pages.dev
Branch Preview URL: https://ai-pr-reviewers.windmilldocs.pages.dev

View logs

@rubenfiszel rubenfiszel merged commit 4a4fb68 into main May 4, 2026
12 checks 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.

1 participant