[ptr727]

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 to exclude selected repository areas from review. This affects which files are included in automated review; it does not change application behavior.

[coderabbitai]

No actionable comments were generated in the recent review.

Walkthrough

Both automatic-review configurations now exclude .github/skills/** and lib/fleet/sh/**.

Changes: Automatic review path filters

Layer / File(s)	Summary
Add path exclusions .coderabbit.yaml, .pr_agent.toml	Both configurations exclude .github/skills/** and lib/fleet/sh/** from automatic review.

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

Change: Bug fix

Possibly related PRs

• ptr727/NxWitness#659: Adds the reviewer configurations that this change extends.
• ptr727/NxWitness#660: Promotes the reviewer configurations that this change extends.
• ptr727/NxWitness#670: Re-vendors files under .github/skills/**, which this change excludes from automatic review.

Merge Risk: Minimal · up to aa8f6

No merge-blocking issue was established. Qodo’s exclusion takes effect after the default-branch configuration is updated.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly describes the main change: excluding the carried trees from 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

• X
• Mastodon
• Reddit
• LinkedIn

Recent review info

Run configuration

• Source mode: Auto
• Repository: ptr727/NxWitness
• Branch: reviewer-skip-carried-trees
• File: .coderabbit.yaml
• Commit: aa8f6b8
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Advanced
• Run ID: e10e6393-612b-4b67-ac49-77c545dcee88

Commits

Reviewing files that changed from the base of the PR and between eb81cac and aa8f6b8.

Files selected for processing: 2

• .coderabbit.yaml
• .pr_agent.toml

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 aa8f6b838600dff87945c78a4e34c19122b8aa87, 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 57.01%. Comparing base (eb81cac) to head (aa8f6b8).

Additional details and impacted files

@@           Coverage Diff            @@
##           develop     #671   +/-   ##
========================================
  Coverage    57.01%   57.01%           
========================================
  Files           15       15           
  Lines         1375     1375           
  Branches       108      108           
========================================
  Hits           784      784           
  Misses         573      573           
  Partials        18       18           

Flag	Coverage Δ
dotnet	57.01% <ø> (ø)

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.

[Copilot]

🟢 Approval recommended

No active review issues are identified, and the changes add the intended ignore patterns while preserving existing reviewer settings.

No open issues 🎉

What changed in this PR

Updates repository reviewer configurations to exclude carried verbatim trees whose findings belong in the ProjectTemplate hub.

Changes:

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

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

_{Review effort: Lite}