Skip to content

docs: wire code reviewer skill into routing - #9

Merged
pertrai1 merged 1 commit into
mainfrom
feat/wire-code-reviewer-skill
Apr 30, 2026
Merged

pertrai1 merged 1 commit into
mainfrom
feat/wire-code-reviewer-skill

Conversation

@pertrai1-bot

@pertrai1-bot pertrai1-bot commented Apr 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add routing frontmatter to the new code-reviewer skill
  • Register code-reviewer as the baseline Review Path skill in adaptive routing
  • Update root instructions, templates, README, and operator cheat sheet references

Verification

  • python3 frontmatter/path-reference validation: all 7 skills have name, description, version, and routing; referenced skills/*/SKILL.md paths resolve
  • git diff --check: pass

Architecture Boundaries

  • Modified zones: Markdown directive/skill/template/docs references only
  • Changed dependency edges: no code imports/exports/package dependencies changed
  • Reference integrity: skills/code-reviewer/SKILL.md references resolve from README, root instructions, templates, and adaptive routing

Summary by CodeRabbit

  • New Features

    • Introduced a mandatory Code Reviewer skill for pull requests, branches, diffs, and local changes.
    • Code review workflow now includes merge-risk assessment alongside correctness, security, and maintainability checks.
  • Documentation

    • Updated all agent documentation to reflect the new Code Reviewer skill integration.
    • Expanded guidance to cover comprehensive review scenarios beyond pull requests.

@pertrai1
pertrai1 merged commit da3e5a5 into main Apr 30, 2026
1 check was pending
@pertrai1
pertrai1 deleted the feat/wire-code-reviewer-skill branch April 30, 2026 23:35
@coderabbitai

coderabbitai Bot commented Apr 30, 2026 •

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 4d6ef850-c41b-43ee-bc21-f9f8bb984922

📥 Commits

Reviewing files that changed from the base of the PR and between 0e27280 and cb5068b.

📒 Files selected for processing (8)
  • AGENTS.md
  • README.md
  • directives/adaptive-routing.md
  • docs/agent-operator-cheat-sheet.md
  • skills/code-reviewer/SKILL.md
  • templates/AGENTS.md
  • templates/CLAUDE.md
  • templates/copilot-instructions.md

📝 Walkthrough

Walkthrough

This change establishes the Code Reviewer skill as a mandatory routed skill across the agent directive system. Documentation files, templates, and skill metadata are updated to integrate the Code Reviewer skill into review workflows and extend its scope from pull requests to include branches, diffs, and local changes.

Changes

Cohort / File(s) Summary
Core Documentation
AGENTS.md, README.md
Updated to establish Code Reviewer as a mandatory skill and expand its scope from pull-request-only review to include branches, diffs, and local changes. README's "What's Included" section incremented skill count (6 → 7) and added dedicated Code Reviewer section with review guidelines.
Directive Configuration
directives/adaptive-routing.md, docs/agent-operator-cheat-sheet.md
Adjusted review path specifications to include explicit code-reviewer skill baseline for standard PR/branch/diff/local-change review prior to conditional skill selections based on content type.
Skill Definition
skills/code-reviewer/SKILL.md
Expanded frontmatter metadata with version field (1.0.0), broadened description scope, and added routing configuration specifying multiple trigger types (pull-request, pr-review, code-review, branch-review, diff-review, merge-risk) and paths constraint.
Instruction Templates
templates/AGENTS.md, templates/CLAUDE.md, templates/copilot-instructions.md
Added Code Reviewer as a routed skill in the Skills sections, instructing agents/Copilot to invoke the skill for code review activities prior to other operations.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Possibly related PRs

  • feat: add root AGENTS instructions #7: Directly modifies the same AGENTS.md file to introduce the mandatory Code Reviewer routed skill as the primary change.
  • feat: add adaptive workflow routing #5: Introduced the adaptive routing and frontmatter-driven workflow infrastructure that this PR extends by adding the Code Reviewer skill with routing configuration.
  • feat: add self-audit skill #2: Established the pattern of adding new routed skills and updating documentation/templates to reference skills/*/SKILL.md files, which this PR follows.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/wire-code-reviewer-skill

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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants