Merge pull request #2022 from bmad-code-org/chore/coderabbit-default-profile

chore(review): replace adversarial CodeRabbit with skill-validator refs

.coderabbit.yaml

@@ -60,23 +60,33 @@ reviews:
     - "!**/validation-report-*.md"
     - "!CHANGELOG.md"
   path_instructions:
-    - path: "**/*"
+    - path: "src/**/skills/**"
       instructions: |
-        You are a cynical, jaded reviewer with zero patience for sloppy work.
-        This PR was submitted by a clueless weasel and you expect to find problems.
-        Be skeptical of everything.
-        Look for what's missing, not just what's wrong.
-        Use a precise, professional tone — no profanity or personal attacks.
-
-        Review with extreme skepticism — assume problems exist.
-        Find at least 10 issues to fix or improve.
-
-        Do NOT:
-        - Comment on formatting, linting, or style
-        - Give "looks good" passes
-        - Anchor on any specific ruleset — reason freely
-
-        If you find zero issues, re-analyze — this is suspicious.
+        Skill file. Apply the full rule catalog defined in tools/skill-validator.md.
+        That document is the single source of truth for all skill validation rules
+        covering SKILL.md metadata, workflow.md constraints, step file structure,
+        path references, variable resolution, sequential execution, and skill
+        invocation syntax.
+    - path: "src/**/workflows/**"
+      instructions: |
+        Legacy workflow file (pre-skill conversion). Apply the full rule catalog
+        defined in tools/skill-validator.md — the same rules apply to workflows
+        that are being converted to skills.
+    - path: "src/**/tasks/**"
+      instructions: |
+        Task file. Apply the full rule catalog defined in tools/skill-validator.md.
+    - path: "src/**/*.agent.yaml"
+      instructions: |
+        Agent definition file. Check:
+        - Has metadata section with id, name, title, icon, and module
+        - Defines persona with role, identity, communication_style, and principles
+        - Menu triggers reference valid skill names that exist
+    - path: "docs/**/*.md"
+      instructions: |
+        Documentation file. Check internal markdown links point to existing files.
+    - path: "tools/**"
+      instructions: |
+        Build script/tooling. Check error handling and proper exit codes.
 chat:
   auto_reply: true # Response to mentions in comments, a la @coderabbit review
 issue_enrichment:

AGENTS.md

@@ -7,3 +7,5 @@ Open source framework for structured, agent-assisted software delivery.
 - Use Conventional Commits for every commit.
 - Before pushing, run `npm ci && npm run quality` on `HEAD` in the exact checkout you are about to push.
   `quality` mirrors the checks in `.github/workflows/quality.yaml`.
+
+- Skill validation rules are in `tools/skill-validator.md`.