Skip to content

fix: enable CodeRabbit auto-review on alpha branch#1235

Merged
mergify[bot] merged 1 commit intomainfrom
fix/coderabbit-alpha-branch
Apr 7, 2026
Merged

fix: enable CodeRabbit auto-review on alpha branch#1235
mergify[bot] merged 1 commit intomainfrom
fix/coderabbit-alpha-branch

Conversation

@ambient-code
Copy link
Copy Markdown
Contributor

@ambient-code ambient-code bot commented Apr 7, 2026

Summary

  • Replace the catch-all .* regex in base_branches with explicit main and alpha entries so CodeRabbit reliably triggers auto-reviews on PRs targeting either branch.

Closes #1134

Test plan

  • Create a test PR targeting alpha and verify CodeRabbit triggers a review
  • Confirm PRs targeting main continue to receive CodeRabbit reviews

🤖 Generated with Claude Code

Replace the catch-all `.*` regex in base_branches with explicit `main`
and `alpha` entries so CodeRabbit reliably triggers on PRs targeting
either branch.

Closes #1134

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ambient-code ambient-code bot added the ambient-code:managed PR managed by AI automation label Apr 7, 2026
@mergify mergify bot added the queued label Apr 7, 2026
@mergify
Copy link
Copy Markdown

mergify bot commented Apr 7, 2026

Merge Queue Status

  • Entered queue2026-04-07 13:51 UTC · Rule: default
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-07 13:51 UTC · at caacbdeff4b17e34650c776b9ea0e4ecef0edc18

This pull request spent 15 seconds in the queue, including 1 second running CI.

Required conditions to merge

@mergify mergify bot merged commit 0c95b38 into main Apr 7, 2026
19 checks passed
@mergify mergify bot deleted the fix/coderabbit-alpha-branch branch April 7, 2026 13:51
@mergify mergify bot removed the queued label Apr 7, 2026
Gkrumbach07 added a commit that referenced this pull request Apr 7, 2026
## Summary

Rewrite the review queue from an LLM-powered session to a deterministic
shell script. Now acts as a strict readiness gate — only PRs that are
actually mergeable get added to the milestone and notified in Slack.

## Changes

### Review Queue (`pr-merge-review.yml`)
- **Hourly** (was daily at 1pm)
- **Shell-driven** (was LLM session — cheaper, faster, deterministic)
- **Strict readiness check**: CI must pass (not pending), no changes
requested, not draft
- **Milestone management**: adds ready PRs, removes ones that fell out
of readiness
- **Slack notification only for NEW additions** — not every run, not
every PR
- Milestone description auto-updated with ready/not-ready lists

### Amber Handler
- **Removed "PR created" Slack notifications** — these were noisy
- The review queue now handles the notification when PRs are actually
ready

## Slack message format
```
📋 2 PRs ready for review

• #1234 — Fix session timeout handling
• #1235 — Add model selector to header

Total in Review Queue: 5
```

## Test plan
- [ ] Manual dispatch review queue — verify milestone updated
- [ ] Open a PR with CI passing — verify it gets added and Slack
notified
- [ ] PR with failing CI — verify NOT added to milestone
- [ ] Draft PR — verify skipped

🤖 Generated with [Claude Code](https://claude.com/claude-code)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Chores**
  * Removed Slack prompts from PR-creation session flows
  * Review-queue schedule changed to run hourly on weekdays
  * Added concurrency controls to review-queue runs

* **New Features**
* Review queue now evaluates PR readiness (drafts, CI, reviews) and
updates a milestone
  * Slack notifications for PRs newly marked as ready
* Batch PR fixer now skips healthy PRs by checking CI, mergeability, and
review states
* PR comments and bodies now include a session footer/link when relevant
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Ambient Code Bot <bot@ambient-code.local>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ambient-code:managed PR managed by AI automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

enable coderabbit on alpha branch

1 participant