@mrciphersmith/keryx 0.2.76 → 0.2.77
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 +15 -10
- package/dist/cli.js +11276 -9120
- package/dist/proxy-worker.js +12 -12
- package/package.json +1 -1
- package/src/gdgraph/build.ts +12 -0
- package/src/gdgraph/query.ts +51 -1
- package/src/gdgraph/types.ts +77 -0
- package/src/gdgraph/wiki-layer-no-git.test.ts +73 -0
- package/src/gdgraph/wiki-layer.test.ts +211 -0
- package/src/gdgraph/wiki-layer.ts +213 -0
- package/src/gdskills/bundled/rules/core/api-contracts.mdc +6 -4
- package/src/gdskills/bundled/rules/core/model-selection.mdc +4 -5
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +4 -38
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +4 -38
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +4 -38
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +4 -38
- package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +4 -38
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +1 -3
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +1 -3
- package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +1 -3
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +35 -34
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +35 -34
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +35 -34
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +35 -34
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +35 -34
- package/src/gdskills/bundled/skills/orchestration/job-orchestrator/input-contract.schema.json +1 -1
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +2 -4
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +2 -4
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +2 -4
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +2 -4
- package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +2 -4
- package/src/gdskills/bundled/skills/planning/autodoc-orchestrator/SKILL.md +10 -6
- package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/planning/interview/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +1 -1
- package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +1 -1
- package/src/gdskills/bundled/skills/platform/agent-entrypoint-distiller/SKILL.md +4 -0
- package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +1 -2
- package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +2 -2
- package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +1 -2
- package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +4 -0
- package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +4 -0
- package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +4 -0
- package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +1 -2
- package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +8 -13
- package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +2 -3
- package/src/gdskills/bundled/skills/review/review-style/SKILL.md +2 -3
- package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +5 -0
- package/src/gdskills/bundled/skills/review/review-verifier/SKILL.md +1 -2
|
@@ -534,10 +534,8 @@ second copy of a schema is how that happens.
|
|
|
534
534
|
9. **DO** verify your work before reporting.
|
|
535
535
|
10. **DO** make `STATUS: <TOKEN>` the first line of your final message, and put no
|
|
536
536
|
JSON in the response body. The full JSON result is the file Phase 6.1 writes
|
|
537
|
-
and records.
|
|
538
|
-
|
|
539
|
-
Results`, and `parseChildResult`, the production function that throws on any
|
|
540
|
-
first line that is not a canonical STATUS token.)
|
|
537
|
+
and records. `parseChildResult` throws on any first line that is not a
|
|
538
|
+
canonical STATUS token.
|
|
541
539
|
|
|
542
540
|
---
|
|
543
541
|
|
|
@@ -534,10 +534,8 @@ second copy of a schema is how that happens.
|
|
|
534
534
|
9. **DO** verify your work before reporting.
|
|
535
535
|
10. **DO** make `STATUS: <TOKEN>` the first line of your final message, and put no
|
|
536
536
|
JSON in the response body. The full JSON result is the file Phase 6.1 writes
|
|
537
|
-
and records.
|
|
538
|
-
|
|
539
|
-
Results`, and `parseChildResult`, the production function that throws on any
|
|
540
|
-
first line that is not a canonical STATUS token.)
|
|
537
|
+
and records. `parseChildResult` throws on any first line that is not a
|
|
538
|
+
canonical STATUS token.
|
|
541
539
|
|
|
542
540
|
---
|
|
543
541
|
|
|
@@ -3,13 +3,17 @@ name: autodoc-orchestrator
|
|
|
3
3
|
description: >
|
|
4
4
|
Autonomous reverse-engineering documentation pipeline — scans an existing codebase
|
|
5
5
|
and produces comprehensive developer documentation without user involvement after initial setup.
|
|
6
|
-
Use when
|
|
7
|
-
|
|
8
|
-
"сгенерируй документацию проекта", "задокументируй кодовую базу",
|
|
9
|
-
"автодок", "autodoc".
|
|
10
|
-
Trigger on: user provides a repo path, asks to document existing code, wants
|
|
11
|
-
onboarding docs, API reference, or architecture overview generated automatically.
|
|
6
|
+
Use when the user provides a repo path, asks to document existing code, or wants
|
|
7
|
+
onboarding docs, an API reference, or an architecture overview generated automatically.
|
|
12
8
|
NOT for: writing new PRDs or planning new features (use gproject-orchestrator).
|
|
9
|
+
triggers:
|
|
10
|
+
- "autodoc"
|
|
11
|
+
- "автодок"
|
|
12
|
+
- "document this codebase"
|
|
13
|
+
- "generate docs for my project"
|
|
14
|
+
- "reverse engineer documentation"
|
|
15
|
+
- "задокументируй кодовую базу"
|
|
16
|
+
- "сгенерируй документацию проекта"
|
|
13
17
|
metadata:
|
|
14
18
|
version: 1.0.0
|
|
15
19
|
---
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interview
|
|
3
|
-
description: "Use
|
|
3
|
+
description: "Use to clarify implementation-specific ambiguities AFTER context has already been collected and the goal is known — the questions that sharpen a plan, not the ones that scope the request. This is the `implement`-intent interview job-orchestrator runs at 0.3. To pin down a vague request before any context is gathered, use `interviewer` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "/interview"
|
|
6
6
|
- "Interview"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interview
|
|
3
|
-
description: "Use
|
|
3
|
+
description: "Use to clarify implementation-specific ambiguities AFTER context has already been collected and the goal is known — the questions that sharpen a plan, not the ones that scope the request. This is the `implement`-intent interview job-orchestrator runs at 0.3. To pin down a vague request before any context is gathered, use `interviewer` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "/interview"
|
|
6
6
|
- "Interview"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interview
|
|
3
|
-
description: "Use
|
|
3
|
+
description: "Use to clarify implementation-specific ambiguities AFTER context has already been collected and the goal is known — the questions that sharpen a plan, not the ones that scope the request. This is the `implement`-intent interview job-orchestrator runs at 0.3. To pin down a vague request before any context is gathered, use `interviewer` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "/interview"
|
|
6
6
|
- "Interview"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interviewer
|
|
3
|
-
description: "Use when
|
|
3
|
+
description: "Use when a request is ambiguous and must be pinned down BEFORE any context is collected — the entry-point interview that turns a vague or expensive ask into a scoped brief. This is the `custom`-intent gate job-orchestrator runs at 0.1.5. For clarifying implementation specifics AFTER context is already collected, use `interview` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "Interview me"
|
|
6
6
|
- "Ask me questions"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interviewer
|
|
3
|
-
description: "Use when
|
|
3
|
+
description: "Use when a request is ambiguous and must be pinned down BEFORE any context is collected — the entry-point interview that turns a vague or expensive ask into a scoped brief. This is the `custom`-intent gate job-orchestrator runs at 0.1.5. For clarifying implementation specifics AFTER context is already collected, use `interview` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "Interview me"
|
|
6
6
|
- "Ask me questions"
|
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: interviewer
|
|
3
|
-
description: "Use when
|
|
3
|
+
description: "Use when a request is ambiguous and must be pinned down BEFORE any context is collected — the entry-point interview that turns a vague or expensive ask into a scoped brief. This is the `custom`-intent gate job-orchestrator runs at 0.1.5. For clarifying implementation specifics AFTER context is already collected, use `interview` instead."
|
|
4
4
|
triggers:
|
|
5
5
|
- "Interview me"
|
|
6
6
|
- "Ask me questions"
|
|
@@ -1,6 +1,10 @@
|
|
|
1
1
|
---
|
|
2
2
|
name: agent-entrypoint-distiller
|
|
3
3
|
description: Use when the user asks to split, decompose, distill, or refactor a large AGENTS.md or CLAUDE.md into Metaproject rules and project-specific skills while keeping root entrypoints compact.
|
|
4
|
+
triggers:
|
|
5
|
+
- "distill AGENTS.md"
|
|
6
|
+
- "split CLAUDE.md"
|
|
7
|
+
- "разнеси CLAUDE.md по правилам"
|
|
4
8
|
metadata:
|
|
5
9
|
version: "1.0.0"
|
|
6
10
|
category: platform
|
|
@@ -5,8 +5,7 @@ description: |
|
|
|
5
5
|
Use when: reviewing code for architectural violations — layer violations, dependency direction
|
|
6
6
|
mistakes, module boundary coupling, SOLID principle breaches, NestJS module/provider structure,
|
|
7
7
|
React MVVM boundary violations, or MobX store layer misplacement.
|
|
8
|
-
|
|
9
|
-
or dispatched by review-orchestrator with --architecture or --backend.
|
|
8
|
+
Dispatched by review-orchestrator with --architecture or --backend.
|
|
10
9
|
NOT for: style/naming preferences, logic correctness bugs, or security vulnerabilities.
|
|
11
10
|
triggers:
|
|
12
11
|
- "review architecture"
|
|
@@ -3,8 +3,8 @@ name: review-backend
|
|
|
3
3
|
model_tier: standard
|
|
4
4
|
description: |
|
|
5
5
|
Use when: reviewing NestJS backend changes — API design, service layer, DTO validation,
|
|
6
|
-
database patterns, and TypeScript correctness.
|
|
7
|
-
|
|
6
|
+
database patterns, and TypeScript correctness. Dispatched by review-orchestrator
|
|
7
|
+
with the --backend flag.
|
|
8
8
|
NOT for: frontend patterns, MobX, React components, general security vulnerabilities
|
|
9
9
|
(use review-security-code for XSS/injection/auth-bypass), or performance profiling
|
|
10
10
|
(use review-performance).
|
|
@@ -6,8 +6,7 @@ description: |
|
|
|
6
6
|
function/class level — meaningful names, small functions, single level of abstraction,
|
|
7
7
|
argument count, error handling, DRY, comment quality, and SOLID (SRP, OCP, LSP, ISP, DIP)
|
|
8
8
|
as applied to individual classes and functions.
|
|
9
|
-
|
|
10
|
-
"review --clean-code", or dispatched by review-orchestrator.
|
|
9
|
+
Dispatched by review-orchestrator.
|
|
11
10
|
NOT for: architectural layer violations (review-architecture), naming convention formatting
|
|
12
11
|
(review-style), logic correctness bugs (review-logic), or security (review-security-code).
|
|
13
12
|
triggers:
|
|
@@ -6,6 +6,10 @@ description: |
|
|
|
6
6
|
direction, feature-boundary leakage, abstraction stability, composition,
|
|
7
7
|
and blast-radius risks. Dispatched by review-orchestrator for
|
|
8
8
|
--core-boundaries, --project-conventions, --all, or src/core/** changes.
|
|
9
|
+
triggers:
|
|
10
|
+
- "review core boundaries"
|
|
11
|
+
- "review --core-boundaries"
|
|
12
|
+
- dispatched by review-orchestrator
|
|
9
13
|
metadata:
|
|
10
14
|
author: "MrCipherSmith"
|
|
11
15
|
version: "1.0.0"
|
|
@@ -7,6 +7,10 @@ description: |
|
|
|
7
7
|
boundaries, selection lifecycle, and large-graph performance. Dispatched by
|
|
8
8
|
review-orchestrator for --flow-graph, --project-conventions, --all, or
|
|
9
9
|
src/core/flow/** / graph abstraction changes.
|
|
10
|
+
triggers:
|
|
11
|
+
- "review flow graph"
|
|
12
|
+
- "review --flow-graph"
|
|
13
|
+
- dispatched by review-orchestrator
|
|
10
14
|
metadata:
|
|
11
15
|
author: "MrCipherSmith"
|
|
12
16
|
version: "1.0.0"
|
|
@@ -8,6 +8,10 @@ description: |
|
|
|
8
8
|
styling tokens, Storybook expectations, and local tooling rules. Dispatched
|
|
9
9
|
by review-orchestrator for --frontend-conventions, --project-conventions,
|
|
10
10
|
--all, or frontend src/**/*.ts(x) changes when local convention docs exist.
|
|
11
|
+
triggers:
|
|
12
|
+
- "review frontend conventions"
|
|
13
|
+
- "review --frontend-conventions"
|
|
14
|
+
- dispatched by review-orchestrator
|
|
11
15
|
metadata:
|
|
12
16
|
author: "MrCipherSmith"
|
|
13
17
|
version: "1.0.0"
|
|
@@ -6,8 +6,7 @@ description: |
|
|
|
6
6
|
race conditions, connection pool exhaustion, cache invalidation, missing indexes,
|
|
7
7
|
unbounded queues, missing backpressure, retry storms, idempotency gaps,
|
|
8
8
|
hot-path blocking I/O, and distributed systems anti-patterns.
|
|
9
|
-
|
|
10
|
-
"review concurrency", "review --highload", or dispatched by review-orchestrator.
|
|
9
|
+
Dispatched by review-orchestrator.
|
|
11
10
|
NOT for: frontend re-render performance (review-performance), general N+1 queries
|
|
12
11
|
(review-performance), clean code style (review-clean-code), or NestJS module structure
|
|
13
12
|
(review-architecture).
|
|
@@ -204,9 +204,8 @@ keryx review complete <review-id-or-path>
|
|
|
204
204
|
[--finding <id> --disposition <state> --evidence <ref>]...
|
|
205
205
|
```
|
|
206
206
|
|
|
207
|
-
**An unrecognised option is refused, not ignored
|
|
208
|
-
|
|
209
|
-
`status: closed` and wrote nothing at all.
|
|
207
|
+
**An unrecognised option is refused, not ignored** — a misspelled flag exits
|
|
208
|
+
non-zero rather than printing `status: closed` and writing nothing at all.
|
|
210
209
|
|
|
211
210
|
`--verifications` takes what `review-verifier` returned. `--scope` takes the
|
|
212
211
|
whole `--json` output of `keryx review scope`, so the package records what the
|
|
@@ -679,16 +678,12 @@ A **fix round** is any review of work produced to answer earlier findings. Set
|
|
|
679
678
|
`is_fix_round: true` on every reviewer input, and populate `prior_findings` with
|
|
680
679
|
the earlier findings and the disposition the fix claimed for each.
|
|
681
680
|
|
|
682
|
-
**Nothing refuses a dispatch that omits them
|
|
683
|
-
|
|
684
|
-
|
|
685
|
-
|
|
686
|
-
|
|
687
|
-
|
|
688
|
-
cannot be pointed at it either. The sentence that stood here until flow 209 told
|
|
689
|
-
you the schema would reject a dispatch without `prior_findings`; it asserted an
|
|
690
|
-
enforcement that has never existed, which is the same class of claim this whole
|
|
691
|
-
package was written to delete.
|
|
681
|
+
**Nothing refuses a dispatch that omits them.** `reviewer-input.schema.json`
|
|
682
|
+
states the rule and no production TypeScript loads that schema; reviewer
|
|
683
|
+
dispatch is an action the host agent takes, not a `keryx` invocation, so there
|
|
684
|
+
is no point at which a malformed dispatch could be rejected. `reviewer-input` is
|
|
685
|
+
also absent from the `CONTRACTS` registry (`src/gdskills/contracts.ts`), so
|
|
686
|
+
`keryx skills contracts validate` cannot be pointed at it either.
|
|
692
687
|
|
|
693
688
|
So this is a requirement on you, unenforced, and the only evidence it was met is
|
|
694
689
|
the `prior_findings` array the reviewer actually receives. A reviewer that cannot
|
|
@@ -4,9 +4,8 @@ model_tier: standard
|
|
|
4
4
|
description: |
|
|
5
5
|
Use when: a developer has received PR review comments and wants to understand them,
|
|
6
6
|
check whether they are still true of the code, act on them, or extract patterns
|
|
7
|
-
from them.
|
|
8
|
-
|
|
9
|
-
comment, planning the fix, driving it to a merged state and answering each reviewer.
|
|
7
|
+
from them. With `--fix`, also validates every comment, plans the fix, drives it to
|
|
8
|
+
a merged state and answers each reviewer.
|
|
10
9
|
NOT for: reviewing code directly — this skill reads human or bot PR feedback and
|
|
11
10
|
makes it actionable. To review code, use the domain review skills.
|
|
12
11
|
triggers:
|
|
@@ -3,9 +3,8 @@ name: review-style
|
|
|
3
3
|
model_tier: light
|
|
4
4
|
description: |
|
|
5
5
|
Use when: reviewing code for style, naming conventions, readability, and DRY violations —
|
|
6
|
-
without touching logic, architecture, security, or performance.
|
|
7
|
-
|
|
8
|
-
with --style flag.
|
|
6
|
+
without touching logic, architecture, security, or performance. Dispatched by
|
|
7
|
+
review-orchestrator with the --style flag.
|
|
9
8
|
NOT for: logic bugs, architectural violations, security vulnerabilities, performance
|
|
10
9
|
anti-patterns, or any finding that could cause a functional regression.
|
|
11
10
|
triggers:
|
|
@@ -10,6 +10,11 @@ description: |
|
|
|
10
10
|
cannot fail is found by executing it rather than by reading it.
|
|
11
11
|
Dispatched by review-orchestrator for --testing-practices,
|
|
12
12
|
--project-conventions, --all, or changed test/e2e/story files.
|
|
13
|
+
triggers:
|
|
14
|
+
- "review tests"
|
|
15
|
+
- "review testing practices"
|
|
16
|
+
- "review --testing-practices"
|
|
17
|
+
- dispatched by review-orchestrator
|
|
13
18
|
metadata:
|
|
14
19
|
author: "MrCipherSmith"
|
|
15
20
|
version: "1.1.0"
|
|
@@ -4,8 +4,7 @@ model_tier: light
|
|
|
4
4
|
description: |
|
|
5
5
|
Use when: consolidated findings from other reviewers need to be checked before they are
|
|
6
6
|
reported — by RUNNING something that fails if the finding is real, or by confirming the
|
|
7
|
-
sites the finding named actually exist.
|
|
8
|
-
findings", "review --verify", or dispatched by review-orchestrator as Wave C.
|
|
7
|
+
sites the finding named actually exist. Dispatched by review-orchestrator as Wave C.
|
|
9
8
|
This reviewer can only DELETE. It cannot raise a severity, add a finding, or change a
|
|
10
9
|
finding's text.
|
|
11
10
|
NOT for: first-pass review (run domain reviewers first); re-scoring findings by re-reading
|