Skip the Carried Verbatim Trees in the Reviewer Configs by ptr727 · Pull Request #1239 · ptr727/PlexCleaner

ptr727 · 2026-11-10T00:27:36Z

Summary

Writes the reviewer configs the hub now renders for this repository, from ptr727/ProjectTemplate's scripts/agent_policy.py at hub 9f3e03c (ptr727/ProjectTemplate#4020).

.coderabbit.yaml gains reviews.path_filters excluding .github/skills/** and lib/fleet/sh/**.
.pr_agent.toml gains an [ignore] glob naming the same trees.

Both trees are hub text the repository carries byte for byte, so a reviewer's finding on them belongs at the hub rather than here. Every other setting in both files is unchanged.

Verification

The diff is six added lines, three per file. Both files parse, and the existing settings and line endings are unchanged.
A recorded local strict review pass found nothing, and the diff-scoped prose gate is clean.
Qodo reads .pr_agent.toml from the default branch, so its exclusion binds once this reaches main.

Summary by CodeRabbit

Chores
- Updated automated review settings so selected internal guidance and shell-script areas are excluded from review scope. Changes to those areas will no longer be included in automated reviews.

Rendered from ptr727/ProjectTemplate's scripts/agent_policy.py at hub 9f3e03c. CodeRabbit and Qodo now skip the trees the hub carries byte for byte, so a finding on hub text lands at the hub rather than on this repository.

coderabbitai · 2026-11-10T00:27:45Z

No actionable comments were generated in the recent review.

Walkthrough

The CodeRabbit and PR Agent configurations now exclude .github/skills/** and lib/fleet/sh/** from review.

Changes: Review path exclusions

Layer / File(s)	Summary
Configure ignored paths `.coderabbit.yaml`, `.pr_agent.toml`	Both configurations add exclusions for `.github/skills/` and `lib/fleet/sh/`.

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

Change: Other

Possibly related PRs

ptr727/PlexCleaner#1064: Resyncs .github/skills/** from the hub and routes findings on those files to the hub.

Suggested labels: chore

Merge Risk: Minimal · up to e0fe2

The change excludes the carried trees from review; no merge-blocking risk was identified.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly summarizes the exclusions for carried verbatim trees in the reviewer configurations.
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: 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. (2 skipped: 2 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

Recent review info

Run configuration

Source mode: Auto

Repository: ptr727/PlexCleaner

Branch: reviewer-skip-carried-trees

File: .coderabbit.yaml

Commit: e0fe260

Layers: Repository YAML

Review profile: CHILL

Plan: Advanced

Run ID: `6761d361-4b56-4686-8b3b-645c21afa460`

Commits

Reviewing files that changed from the base of the PR and between `632b59c` and `e0fe260`.

Files selected for processing: 2

`.coderabbit.yaml`

`.pr_agent.toml`

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

qodo-free-for-open-source-projects · 2026-11-10T00:28:07Z

PR Summary by Qodo

Exclude hub-carried trees from reviewer configs

⚙️ Configuration changes 🕐 Less than 10 minutes

AI Description

• Exclude hub-carried skill and shell trees from CodeRabbit and Qodo reviews.
• Keep findings on verbatim hub content directed to the hub, without changing other reviewer
 settings.

Diagram

graph TD
  CR["CodeRabbit"] --> CRC[".coderabbit.yaml"] -->|excludes| Trees["Hub-carried trees"]
  Q["Qodo"] --> QC[".pr_agent.toml"] -->|ignores| Trees

High-Level Assessment

Reviewer-specific path exclusions are the appropriate narrow change: they avoid findings on verbatim hub content without disabling reviews elsewhere. Qodo's exclusion takes effect when this configuration reaches the default branch.

Files changed (2) +6 / -0

Other (2) +6 / -0

.coderabbit.yamlExclude hub-carried trees from CodeRabbit +3/-0

Exclude hub-carried trees from CodeRabbit
• Adds negative review path filters for .github/skills/** and lib/fleet/sh/**. Existing auto-review settings remain unchanged.
.coderabbit.yaml

.pr_agent.tomlIgnore hub-carried trees in Qodo +3/-0

Ignore hub-carried trees in Qodo
• Adds an ignore glob for the same two trees. The existing auto-feedback setting remains unchanged.
.pr_agent.toml

ptr727 · 2026-11-10T00:29:05Z

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

qodo-free-for-open-source-projects · 2026-11-10T00:29:06Z

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (1) 📎 Requirement gaps (0) 🎨 UX issues (0) 🔗 Cross-repo conflicts (0) 📜 Skill insights (0)

1. Review tools silently skip findings on two directory trees 📘 Rule violation ✧ Quality

Description

The new reviews.path_filters in .coderabbit.yaml and [ignore] glob in .pr_agent.toml exclude
.github/skills/** and lib/fleet/sh/** from CodeRabbit and Qodo review through repository
configuration rather than analysis-tool suppression metadata. When a PR changes either tree,
findings from both reviewers are hidden; for changes to carried review skills, Copilot also loads
the changed skill from the PR head.

Code

.coderabbit.yaml[R7-9]

+  path_filters:
+    - "!.github/skills/**"
+    - "!lib/fleet/sh/**"

Evidence

The added filters in .coderabbit.yaml and .pr_agent.toml exclude the same two trees from
CodeRabbit and Qodo review, hiding otherwise supported findings by default. The repository
configures Qodo for automatic review and CodeRabbit for explicit review, while its Copilot
instructions say Copilot loads review skills from the PR head despite the rule that the carried
skill tree must not be edited locally. The stated intent to defer findings to the hub does not limit
the filters to canonical-content findings or prevent them from applying to future changes.

Rule 3008765: Do not hide or downgrade supported static analysis findings in review output
.coderabbit.yaml[7-9]
.pr_agent.toml[4-5]
.github/agent-policy.json[5-38]
.github/copilot-instructions.md[3-5]
GOVERNANCE.md[321-322]

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

## Issue description
The PR adds review-tool exclusions for `.github/skills/**` and `lib/fleet/sh/**` that hide otherwise supported findings by default. In particular, both independent reviewers skip changes to carried review skills even though Copilot consumes those skills from the PR head.
## Fix Focus Areas
- .coderabbit.yaml[7-9]
- .pr_agent.toml[4-5]
## Recommended Fix
Remove the `.github/skills/**` exclusions from both reviewer configurations so changes to carried skills remain independently reviewed for unauthorized edits or instruction tampering, while directing findings about canonical skill content to the hub. Remove the `lib/fleet/sh/**` exclusions as well, unless the hub-ownership rationale is accepted as a documented exception; if retained, record that exception explicitly in the PR description or team policy so the suppression is traceable.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Tip of the day

💡 Did you know, you can move or hide resolved findings to focus on what still needs action

More tips ↗ | Customize Qodo ↗ | Qodo docs ↗

Copilot

🟢 Approval recommended

The supplied review evidence reports no active issues, with changes limited to reviewer exclusion settings while preserving the existing configuration.

No open issues 🎉

What changed in this PR

Updates the repository’s automated reviewer configurations to exclude hub-owned, verbatim trees so findings are raised against the canonical source instead.

Changes:

Adds CodeRabbit exclusions for .github/skills/** and lib/fleet/sh/**.
Adds matching Qodo/PR-Agent ignore globs.
Preserves all existing reviewer settings.

File	Description
`.pr_agent.toml`	Excludes the carried verbatim trees from PR-Agent/Qodo reviews.
`.coderabbit.yaml`	Excludes the carried verbatim trees from CodeRabbit reviews.

_{Review effort: Lite}

ptr727 · 2026-11-10T00:33:16Z

@coderabbitai review

coderabbitai · 2026-11-10T00:33:22Z

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.

ptr727 · 2026-11-10T00:33:48Z

@qodo-free-for-open-source-projects please dismiss finding 1 (Review tools silently skip findings on two directory trees; .coderabbit.yaml:7-9, .pr_agent.toml:4-5). Declined: the exclusion of .github/skills/** and lib/fleet/sh/** is this PR's stated purpose, not a defect. GOVERNANCE.md on develop states of .github/skills/: 'the fleet Skills, carried whole from the hub and never edited here.' Ownership of that text sits at ptr727/ProjectTemplate, which authors it, so a finding on its content belongs to the hub, and findings on every other path stay enabled in both reviewers.

qodo-free-for-open-source-projects · 2026-11-10T00:34:26Z

free-for-open-source-projects please dismiss finding 1 (Review tools silently skip findings on two directory trees; .coderabbit.yaml:7-9, .pr_agent.toml:4-5). Declined: the exclusion of .github/skills/** and lib/fleet/sh/** is this PR's stated purpose, not a defect. GOVERNANCE.md on develop states of .github/skills/: 'the fleet Skills, carried whole from the hub and never edited here.' Ownership of that text sits at ptr727/ProjectTemplate, which authors it, so a finding on its content belongs to the hub, and findings on every other path stay enabled in both reviewers.

I can't dismiss finding 1 right now. The findings catalog for this PR is empty, so I have no stable ID to match to "Review tools silently skip findings on two directory trees."

Your reasoning doesn't change that. Dismissals go through the stable finding ID. If the finding shows up in the catalog on a later run, ask me again and I'll dismiss it, provided you're the PR author. You can also dismiss it directly in the Qodo UI.

If you want to narrow the exclusions rather than dismiss the finding, the rule's scope can be edited on the Rule 3008765 page in the Qodo portal (https://app.qodo.ai/rules/3008765?state=active). The exclusions themselves are in .coderabbit.yaml (lines 7-9) and .pr_agent.toml (lines 4-5).

Skip the Carried Verbatim Trees in the Reviewer Configs

e0fe260

Rendered from ptr727/ProjectTemplate's scripts/agent_policy.py at hub 9f3e03c. CodeRabbit and Qodo now skip the trees the hub carries byte for byte, so a finding on hub text lands at the hub rather than on this repository.

Copilot AI lite review requested due to automatic review settings November 10, 2026 00:27

Copilot started reviewing on behalf of ptr727 November 10, 2026 00:28

View session

Copilot AI reviewed Nov 10, 2026

View reviewed changes

ptr727 merged commit 5f32edd into develop Nov 10, 2026
16 checks passed

ptr727 deleted the reviewer-skip-carried-trees branch November 10, 2026 00:38

ptr727 mentioned this pull request Nov 10, 2026

Promote Develop: Resync the Instruction Set and Fleet Skills with the Hub #1240

Merged

Navigation Menu

Repository navigation

Skip the Carried Verbatim Trees in the Reviewer Configs - #1239

Skip the Carried Verbatim Trees in the Reviewer Configs#1239
ptr727 merged 1 commit into
developfrom
reviewer-skip-carried-trees

ptr727 commented Nov 10, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Nov 10, 2026 •

edited

Loading

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

coderabbitai Bot commented Nov 10, 2026 •

edited

Loading

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

ptr727 commented Nov 10, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Verification

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Nov 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026

PR Summary by Qodo

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Code Review by Qodo

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

coderabbitai Bot commented Nov 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ptr727 commented Nov 10, 2026

Uh oh!

qodo-free-for-open-source-projects Bot commented Nov 10, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

ptr727 commented Nov 10, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Nov 10, 2026 •

edited

Loading

qodo-free-for-open-source-projects Bot commented Nov 10, 2026 •

edited

Loading

coderabbitai Bot commented Nov 10, 2026 •

edited

Loading