Skip to content

ci: switch to plugin-based code review with sticky comment#89

Open
exiao wants to merge 2 commits intomainfrom
ci/plugin-code-review
Open

ci: switch to plugin-based code review with sticky comment#89
exiao wants to merge 2 commits intomainfrom
ci/plugin-code-review

Conversation

@exiao
Copy link
Copy Markdown
Owner

@exiao exiao commented Apr 10, 2026

Upgrades claude-code-review.yml to use the \ plugin for multi-agent review, plus sticky comment infra and concurrency. Fixes wrong \ param (should be ), adds concurrency, full permissions, and \ flag.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fd11d1b9a0

ℹ️ 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".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

on:
pull_request:
types: [opened, synchronize, ready_for_review, reopened]
types: [opened, synchronize]
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Add ready_for_review back to pull_request triggers

Limiting this workflow to opened and synchronize causes draft PRs to miss review entirely when they are later marked ready without a new commit: the configured /code-review plugin is documented to skip draft PRs, so the opened run does nothing, and without a ready_for_review trigger there is no follow-up run. The Claude Code Action docs also call out ready_for_review (and reopened) for tracked PR reviews; dropping it is a functional regression in review coverage.

Useful? React with 👍 / 👎.

…eview workflow

Codex review flagged that dropping ready_for_review causes draft PRs
to miss review entirely when marked ready without a new commit.
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