@mrciphersmith/keryx 0.2.75 → 0.2.77

package/src/gdskills/bundled/skills/review/review-style/SKILL.md

@@ -3,9 +3,8 @@ name: review-style
 model_tier: light
 description: |
   Use when: reviewing code for style, naming conventions, readability, and DRY violations —
-  without touching logic, architecture, security, or performance. Covers "review style",
-  "style review", "check naming", "check readability", or dispatched by review-orchestrator
-  with --style flag.
+  without touching logic, architecture, security, or performance. Dispatched by
+  review-orchestrator with the --style flag.
   NOT for: logic bugs, architectural violations, security vulnerabilities, performance
   anti-patterns, or any finding that could cause a functional regression.
 triggers:

package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md

@@ -10,6 +10,11 @@ description: |
   cannot fail is found by executing it rather than by reading it.
   Dispatched by review-orchestrator for --testing-practices,
   --project-conventions, --all, or changed test/e2e/story files.
+triggers:
+  - "review tests"
+  - "review testing practices"
+  - "review --testing-practices"
+  - dispatched by review-orchestrator
 metadata:
   author: "MrCipherSmith"
   version: "1.1.0"

package/src/gdskills/bundled/skills/review/review-verifier/SKILL.md

@@ -4,8 +4,7 @@ model_tier: light
 description: |
   Use when: consolidated findings from other reviewers need to be checked before they are
   reported — by RUNNING something that fails if the finding is real, or by confirming the
-  sites the finding named actually exist. Covers "verify the findings", "check these
-  findings", "review --verify", or dispatched by review-orchestrator as Wave C.
+  sites the finding named actually exist. Dispatched by review-orchestrator as Wave C.
   This reviewer can only DELETE. It cannot raise a severity, add a finding, or change a
   finding's text.
   NOT for: first-pass review (run domain reviewers first); re-scoring findings by re-reading

package/src/gdskills/contracts/review-finding.schema.json

@@ -95,6 +95,10 @@
       }
     },
     "severity": { "type": "string", "enum": ["blocker", "major", "minor", "info"] },
+    "repo": {
+      "type": ["string", "null"],
+      "description": "Which repository `file` and `line` are in. Null, the default, means the repository under review. A cross-repo claim names the producer here and pins the ref it was read at in `evidence`; without this field a finding about another service's behaviour could not say where its evidence lived, so the rule that such a claim is `info` unless it has a `file:line` at a pinned ref could not be checked by anything but the reviewer's word. Declared in BOTH this schema and skills/review/review-orchestrator/reviewer-finding.schema.json: this one is additionalProperties:false, so a finding carrying `repo` is rejected here if only the reviewer-side schema knows about it."
+    },
     "file": { "type": ["string", "null"] },
     "line": { "type": ["integer", "null"], "minimum": 1 },
     "symbol": { "type": ["string", "null"] },