docs: aic-codeowners skill + contributor guidance for generated CODEOWNERS#1348
Merged
jasonqinzhou merged 3 commits intoJul 13, 2026
Merged
Conversation
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
dagil-nvidia
force-pushed
the
dagil-nvidia/codeowners-docs-skill
branch
2 times, most recently
from
July 12, 2026 22:07
a9440bb to
1a96748
Compare
dagil-nvidia
force-pushed
the
dagil-nvidia/codeowners-as-code
branch
from
July 12, 2026 22:23
7b83480 to
337eec5
Compare
…WNERS Companion to the CODEOWNERS-as-code PR: an .agents/skills/aic-codeowners skill covering the four flows (who reviews a change, fixing a failing codeowners check, changing routing, external contributor grants), plus CONTRIBUTING.md and AGENTS.md pointers so contributors and agents find the playbook when the gate fires. Signed-off-by: Dan Gil <dagil@nvidia.com>
Triage the codeowners workflow by its failing step instead of assuming every failure is an uncovered path. Document sampled coverage output, multi-claim fixes, generated advisory artifacts, and repository-rule enforcement so agents preserve routing and commit every changed source/output together. Signed-off-by: Daniel Gil <dagil@nvidia.com>
Signed-off-by: Daniel Gil <dagil@nvidia.com>
dagil-nvidia
force-pushed
the
dagil-nvidia/codeowners-docs-skill
branch
from
July 12, 2026 22:23
1a96748 to
8c7f516
Compare
jasonqinzhou
approved these changes
Jul 13, 2026
jasonqinzhou
merged commit Jul 13, 2026
66d932b
into
dagil-nvidia/codeowners-as-code
16 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Overview:
Companion to #1347 (stacked on its branch; merges after it). Adds the contributor-facing playbook for the generated CODEOWNERS system.
Details:
.agents/skills/aic-codeownersskill: who reviews my change, triaging a failingcodeownerscheck by step (unit tests, coverage, or drift), applying the narrowest ownership claim(s), and regenerating every affected artifact, removals (drift-step failures + dead-glob pruning), routing changes, and external contributor area grants.who_owns.py --changedpreview, and the gate-failure fix in the PR checklist section.Where Should the Reviewer Start?
.agents/skills/aic-codeowners/SKILL.md- the docs changes are one-paragraph pointers to it.Related Issues: (Use One of the Action Keywords Closes / Fixes / Resolves / Relates to)
🤖 Generated with Claude Code