[ptr727]

Adds the same agentPolicy override to three repositories, so explicit review gets tested on active work. This is the maintainer's choice from the review-cutover track (#3877). The repositories are PlexCleaner (public), and Financial-Modeling and HomeAutomation-Config (both private).

• Copilot: explicit, after-checks, per push with at most 3 rounds, gating. These are the hub's own values.
• CodeRabbit: explicit, after-checks, once. These are the hub's own values, and they spend CodeRabbit's rate-limited quota once per pull request. CodeRabbit does review both private repositories. Its plan's rate limit, not their visibility, is what holds it back.
• Qodo: keeps the fleet default, automatic. Turning a bot off is left to a test of its own.

facts derives the same settings for each repository: CodeRabbit enabled: false, Qodo disable_auto_feedback: false, and Copilot's ruleset rule off on both branches. A write --dry-run against each repository's develop changes .github/agent-policy.json and .coderabbit.yaml, and leaves .pr_agent.toml unchanged.

Next, on this session's "go", each repository's own session writes its files into its develop. Then configure.sh apply runs for each on the maintainer's go-ahead.

Verified locally: spec/validate.py OK, and the test_spec_validate, test_audit_helpers and test_agent_policy suites pass.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Chores
  • Review automation settings have been updated for three repositories. Copilot reviews run for up to three rounds after checks complete, while CodeRabbit reviews run once after checks complete. These settings standardize when automated reviews run and how many review rounds are allowed.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Three repository entries gain review policies for Copilot and CodeRabbit. Copilot triggers explicitly after checks, with a maximum of three rounds. CodeRabbit triggers explicitly after checks and runs once.

Changes: Repository review policies

Layer / File(s)	Summary
Add per-repository review settings registry/repos.json	Financial-Modeling, PlexCleaner, and HomeAutomation-Config receive the same review settings for Copilot and CodeRabbit.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related issues

• Session Handoff [review-cutover]: Step-2 Reports In, Step 3 Left to Peer Sessions, Check-In 1 Next #3877: This PR adds per-repository overrides to the review policy model described by the cutover track.
• Decide: How the Coordinated Cutover to Explicit Bot-Review Requests Is Triggered, Scheduled, and Gated #3826: This PR sets explicit, after-checks review policies for three repositories, but does not define the fleet-wide trigger.
• Reviewer Reference Says CodeRabbit Skips Private Repositories, but It Reviews Them #3925: This PR sets CodeRabbit to explicit, once per pull request, for the three repositories discussed in the issue.
• Stop pr_review.py Requesting Copilot on Feature Pull Requests into Develop #3758: This PR changes Copilot review timing for three repositories, but does not resolve the issue’s pr_review.py behavior.

Possibly related PRs

• ptr727/ProjectTemplate#3904: This PR also affects the CodeRabbit configuration rendered for the repositories changed here.
• ptr727/ProjectTemplate#3921: This PR’s CodeRabbit gate depends on repository configuration rendered by the same system.

Merge Risk: Minimal · up to 626d7

The registry changes move the three repositories to explicit reviews. No merge-blocking issue is identified; repository writes and the approved apply remain planned.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title states the main change: three repositories (PlexCleaner, Financial-Modeling, HomeAutomation-Config) move their Copilot and CodeRabbit reviews from automatic to explicit requests. The raw sum…
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Title check

Explanation

The title states the main change: three repositories (PlexCleaner, Financial-Modeling, HomeAutomation-Config) move their Copilot and CodeRabbit reviews from automatic to explicit requests. The raw summary confirms that each repository now declares an agentPolicy in registry/repos.json that triggers Copilot and CodeRabbit explicitly after checks. The title is specific, concise, and matches the changeset, so it passes.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

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

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

• X
• Mastodon
• Reddit
• LinkedIn

Recent review info

Run configuration

• Source mode: Auto
• Repository: ptr727/ProjectTemplate
• Branch: feature/plexcleaner-explicit-review
• File: .coderabbit.yaml
• Commit: 626d7cd
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: a07a6fc0-aad1-4263-8710-ea26f9e61d26

Commits

Reviewing files that changed from the base of the PR and between f96b8b7 and 626d7cd.

Files selected for processing: 1

• registry/repos.json

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

[ptr727]

A recorded local strict-review pass covers head cfdbc50d92c23feaf945d0b045e06ecabf010a10, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[Copilot]

🟢 Approval recommended

No active review issues remain, so the repository-specific review policy changes are ready for approval.

No open issues 🎉

What changed in this PR

This PR adds a PlexCleaner-specific agent policy that makes Copilot and CodeRabbit reviews explicit while retaining the fleet-default Qodo policy.

Changes:

• Configures Copilot for explicit, after-checks reviews with up to three rounds per push.
• Configures CodeRabbit for explicit, after-checks reviews with one round.
• Leaves Qodo inherited from the automatic fleet default.
• Review feedback requests a regression test pinning PlexCleaner's rendered policy.

File	Summary
registry/​repos.json	Adds PlexCleaner's per-repository review policy overrides.

_{Review effort: Lite}

[ptr727]

A recorded local strict-review pass covers head 1befc63166e9cfa66131119397682587a1c21132, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.26%. Comparing base (b38e084) to head (626d7cd).
⚠️ Report is 113 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3924      +/-   ##
===========================================
- Coverage    81.60%   78.26%   -3.35%     
===========================================
  Files           26       35       +9     
  Lines         9070    12328    +3258     
===========================================
+ Hits          7402     9648    +2246     
- Misses        1668     2680    +1012     

Flag	Coverage Δ
python-3.13	78.26% <ø> (-3.35%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ptr727]

A recorded local strict-review pass covers head f4a2949456c20addb7a154500977c6839b4d2969, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[ptr727]

A recorded local strict-review pass covers head 626d7cd09853e0ff1197ed8354491df303fa9b6a, the content this pull request carries at that commit against develop, and it recorded 0 findings.

[ptr727]

@coderabbitai review

[ptr727]

/review

[coderabbitai]

Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.