[j-ctang]

Closes #42
Risk: risk:feature

Summary

• CodeRabbit skips PRs whose title contains "Promote staging to main".
• PRs into staging keep their review; nothing else changes.

Before/After

• Before: promote PR Promote staging to main #38 got a CodeRabbit summary.
• After: config validates against CodeRabbit's schema; the next promote PR confirms the skip.

All tests pass.

Commit History

• Skip CodeRabbit on promote PRs

Sensitive Changes

• Reviewer config: skip promote PRs

Rollback

git revert <merge-sha>

Summary by CodeRabbit

• Chores
  • Automated reviews are now skipped for pull requests titled “Promote staging to main.”

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: j-ctang/claude-code-jev-prune/.coderabbit.yaml
• Review profile: CHILL
• Plan: Advanced
• Run ID: 7ad7fdf5-2148-49c4-91c7-c786fb96d3fe

📥 Commits

Reviewing files that changed from the base of the PR and between d7cc96c and 0f659e9.

📒 Files selected for processing: 1

• .coderabbit.yaml

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

📝 Walkthrough

The automatic review configuration now ignores PR titles containing “Promote staging to main”.

Changes: Promote PR review filter

Layer / File(s)	Summary
Configure title keyword filter .coderabbit.yaml	The automatic review configuration adds Promote staging to main to ignore_title_keywords.

Priority: ⬇️ Low

Severity of issue fixed: Low

Merge Risk: 🔵 Low · up to 0f659

A narrowly matching staging pull request could miss its CodeRabbit review; this is bounded but should be addressed through branch-aware configuration or follow-up.

Architecture risk: 🔵 Low · up to 0f659

The changed surface does not map to a changed system, dependency edge, entrypoint, or external dependency.

Changed systems: None identified.

Architecture concerns
No architecture-level concerns identified.

Review details

Before / after behavior

• observed — Modified behavior in .coderabbit.yaml: Configured automatic reviews to ignore PR titles containing Promote staging to main.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: skipping CodeRabbit reviews for promote PRs.
Description check		The description includes all required template sections and provides the change summary, expected behavior, validation status, sensitive-change details, and rollback procedure.
Linked Issues check		Issue #42 requires CodeRabbit to ignore promote PR titles and keep reviews for PRs targeting staging. The PR adds reviews.auto_review.ignore_title_keywords: ["Promote staging to main"] in `.codera…
Out of Scope Changes check		The diff changes only .coderabbit.yaml. The new setting directly implements issue #42. The added comments document the same configuration purpose. No unrelated code or behavior changes are present.

Full details: Linked Issues check

Explanation

Issue #42 requires CodeRabbit to ignore promote PR titles and keep reviews for PRs targeting staging. The PR adds reviews.auto_review.ignore_title_keywords: ["Promote staging to main"] in .coderabbit.yaml. The existing base_branches: ["^staging$"] remains unchanged, so the configured staging review behavior remains. The requested runtime confirmation for the next promote PR is an operational check, not a coding requirement.

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[j-ctang]

Verdict: Ready to merge

Review tier: S

• should-fix .coderabbit.yaml:27: I find that this title filter weakens automated review for every target branch, including staging. I would preserve automatic review until the exclusion can be restricted to verified staging-to-main promotions.