@mrciphersmith/keryx 0.3.115 → 0.3.117
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/README.md +6 -2
- package/dist/cli.js +2967 -2377
- package/dist/core.js +59 -15
- package/package.json +1 -1
- package/src/gdskills/bundled/install-manifest.json +1 -0
- package/src/gdskills/bundled/skills/review/review-layout/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.detail.md +39 -0
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +5 -6
- package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-finding.schema.json +9 -0
- package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-input.schema.json +4 -0
- package/src/gdskills/bundled/skills/review/review-pr-contract/SKILL.md +191 -0
- package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +5 -5
- package/src/gdskills/contracts/subagent-dispatch.schema.json +4 -0
package/README.md
CHANGED
|
@@ -617,7 +617,11 @@ Grouped by what you are trying to do, not by internal module layout.
|
|
|
617
617
|
taken on the reviewer's word, and a quote that cannot be found is marked
|
|
618
618
|
unlocatable rather than carrying a number nobody checked. Each round also
|
|
619
619
|
states its price — estimated before dispatch, recorded after, and divided by
|
|
620
|
-
the findings that survived it.
|
|
620
|
+
the findings that survived it. A round also says what it left undone: mutation
|
|
621
|
+
candidates that never ran (`review mutate --cap`), severities held because only
|
|
622
|
+
a presentation mutant survived, and follow-ups handed back to the parent, in
|
|
623
|
+
`review status` and `review report-parity`. A pull-request round adds
|
|
624
|
+
`review-pr-contract`, which reads the diff against the PR's own stated contract.
|
|
621
625
|
|
|
622
626
|
**Operate agents**
|
|
623
627
|
|
|
@@ -1150,7 +1154,7 @@ Every managed review start, attach or ingest (and any `dispatch-check` or `blast
|
|
|
1150
1154
|
|
|
1151
1155
|
### Gates that keep a review report honest
|
|
1152
1156
|
|
|
1153
|
-
`keryx review complete` on a pull-request package refuses a merged PR or a moved head until the first line of `report.md` names the reviewed SHA and says so, and refuses a report with no `Evidence level:` line (`static only — did not run: <what>` or `ran — <what>`; a package ingested before the field existed only warns). `keryx review ingest` reads each finding's quote at its `path` at the reviewed SHA and refuses one that does not resolve there, adds a `citation not found` line to a rebuttal that cites a rule or document missing from the tree (the finding is never deleted), and a finding can close as `narrowed` only with a "what remains" text. See [review with a record](docs/docs/guides/review-with-a-record.md).
|
|
1157
|
+
`keryx review complete` on a pull-request package (one that targets a PR, or whose `--ref` is a PR URL under any `--target`) refuses a merged PR or a moved head until the first line of `report.md` names the reviewed SHA and says so, and refuses a report with no `Evidence level:` line (`static only — did not run: <what>` or `ran — <what>`; a package ingested before the field existed only warns). `keryx review ingest` reads each finding's quote at its `path` at the reviewed SHA and refuses one that does not resolve there, adds a `citation not found` line to a rebuttal that cites a rule or document missing from the tree (the finding is never deleted), and a finding can close as `narrowed` only with a "what remains" text. See [review with a record](docs/docs/guides/review-with-a-record.md).
|
|
1154
1158
|
|
|
1155
1159
|
### Review bot on pull requests
|
|
1156
1160
|
|