Skip to content

fix(ci): review draft pull requests without backlog - #281

Merged
777genius merged 1 commit into
devfrom
fix/reviewrouter-draft-trigger
Jul 16, 2026
Merged

fix(ci): review draft pull requests without backlog#281
777genius merged 1 commit into
devfrom
fix/reviewrouter-draft-trigger

Conversation

@777genius

@777genius 777genius commented Jul 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • trigger ReviewRouter when an existing pull request is converted to draft
  • preserve REVIEW_ROUTER_REVIEW_DRAFTS as the explicit draft-review switch
  • keep the active review and only the latest pending run instead of a 100-run FIFO backlog

Validation

Summary by CodeRabbit

  • Chores
    • Updated pull request review automation to consistently respond when pull requests are converted to draft, across relevant pull request event types.
    • Added manual and scheduled triggers for periodic review refreshes.
    • Improved review execution controls with shared concurrency handling and a configurable review timeout.
    • Enhanced review configuration to better support draft-review behavior and review limits, including passing through updated review parameters.

Summary

  • updates a GitHub Actions workflow
📒 Files selected for processing (1)
  • .github/workflows/reviewrouter-codex.yml
📝 Walkthrough

Walkthrough

This PR updates 1 file across 1 CI workflow. The generated summary is based on the GitHub pull request file list and diff metadata, and the author's description above is preserved.

Changes

Cohort / File(s) Summary
CI workflow
.github/workflows/reviewrouter-codex.yml
Updates a GitHub Actions workflow.
Line stats: 1 modified; +2/-4.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The ReviewRouter Codex workflow now supports converted-to-draft, manual, and scheduled triggers, configurable review execution, and a separate scheduled OAuth refresh job.

Changes

ReviewRouter Workflow Expansion

Layer / File(s) Summary
Expand workflow entrypoints
.github/workflows/reviewrouter-codex.yml
Adds converted_to_draft, workflow_dispatch, and cron triggers, and sets workflow-level permissions to {}.
Configure Codex review execution
.github/workflows/reviewrouter-codex.yml
Adds configurable timeout and shared concurrency, draft-aware conditions, and additional ReviewRouter inputs while invoking @main.
Add Codex OAuth refresh
.github/workflows/reviewrouter-codex.yml
Adds a scheduled/manual refresh job using shared concurrency and codex-oauth-refresh mode parameters.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title matches the PR’s main change: enabling draft pull request review handling in the CI workflow.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/reviewrouter-draft-trigger

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@777genius
777genius changed the base branch from main to dev July 16, 2026 19:43
@777genius
777genius force-pushed the fix/reviewrouter-draft-trigger branch from bc7d60b to 2de59bd Compare July 16, 2026 19:47
@777genius 777genius changed the title fix(ci): trigger ReviewRouter when PR becomes draft fix(ci): review draft pull requests without backlog Jul 16, 2026

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

Copy link
Copy Markdown

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: 2de59bd6b0

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

Comment on lines 19 to 21
concurrency:
group: reviewrouter-codex-oauth-${{ github.repository_id }}-codex-rotating-1163183284
queue: max
cancel-in-progress: false

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore ReviewRouter queueing for pending runs

When a ReviewRouter job is already running and two or more PR events or refreshes arrive, removing queue: max makes this concurrency group fall back to GitHub Actions' default single pending run, where a newer pending run cancels/replaces the existing pending run (the GitHub concurrency docs describe queue: max as the option that allows up to 100 pending runs). Because codex-review and codex-refresh share this same group, bursts of ReviewRouter events can now silently drop reviews or refreshes; the same restoration is needed in the refresh block below.

Useful? React with 👍 / 👎.

@777genius
777genius merged commit 45e20e3 into dev Jul 16, 2026
16 checks passed
@777genius
777genius deleted the fix/reviewrouter-draft-trigger branch July 16, 2026 19:55
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