@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.
Files changed (51) hide show
  1. package/README.md +15 -10
  2. package/dist/cli.js +11276 -9120
  3. package/dist/proxy-worker.js +12 -12
  4. package/package.json +1 -1
  5. package/src/gdgraph/build.ts +12 -0
  6. package/src/gdgraph/query.ts +51 -1
  7. package/src/gdgraph/types.ts +77 -0
  8. package/src/gdgraph/wiki-layer-no-git.test.ts +73 -0
  9. package/src/gdgraph/wiki-layer.test.ts +211 -0
  10. package/src/gdgraph/wiki-layer.ts +213 -0
  11. package/src/gdskills/bundled/rules/core/api-contracts.mdc +6 -4
  12. package/src/gdskills/bundled/rules/core/model-selection.mdc +4 -5
  13. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +4 -38
  14. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +4 -38
  15. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +4 -38
  16. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +4 -38
  17. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +4 -38
  18. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +1 -3
  19. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +1 -3
  20. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +1 -3
  21. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +35 -34
  22. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +35 -34
  23. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +35 -34
  24. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +35 -34
  25. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +35 -34
  26. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/input-contract.schema.json +1 -1
  27. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +2 -4
  28. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +2 -4
  29. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +2 -4
  30. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +2 -4
  31. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +2 -4
  32. package/src/gdskills/bundled/skills/planning/autodoc-orchestrator/SKILL.md +10 -6
  33. package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +1 -1
  34. package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +1 -1
  35. package/src/gdskills/bundled/skills/planning/interview/SKILL.md +1 -1
  36. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +1 -1
  37. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +1 -1
  38. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +1 -1
  39. package/src/gdskills/bundled/skills/platform/agent-entrypoint-distiller/SKILL.md +4 -0
  40. package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +1 -2
  41. package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +2 -2
  42. package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +1 -2
  43. package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +4 -0
  44. package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +4 -0
  45. package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +4 -0
  46. package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +1 -2
  47. package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +8 -13
  48. package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +2 -3
  49. package/src/gdskills/bundled/skills/review/review-style/SKILL.md +2 -3
  50. package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +5 -0
  51. 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. (This rule used to say the opposite — "return the JSON result
538
- object as your final message" — which contradicted 6.2, `## Reporting
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. (This rule used to say the opposite — "return the JSON result
538
- object as your final message" — which contradicted 6.2, `## Reporting
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: "autodoc", "generate docs for my project", "document this codebase",
7
- "reverse engineer documentation", "create project documentation from code",
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 before implementation, design, or migration when requirements are unclear and targeted clarifying questions are needed to gather precise context."
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 before implementation, design, or migration when requirements are unclear and targeted clarifying questions are needed to gather precise context."
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 before implementation, design, or migration when requirements are unclear and targeted clarifying questions are needed to gather precise context."
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 requirements are ambiguous and precise clarification is needed before proceeding with a complex task."
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 requirements are ambiguous and precise clarification is needed before proceeding with a complex task."
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 requirements are ambiguous and precise clarification is needed before proceeding with a complex task."
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
- Triggered by: "review architecture", "check architecture", "architectural review",
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. Covers "review backend", "backend review",
7
- "review API", "review NestJS", or dispatched by review-orchestrator with --backend flag.
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
- Triggered by: "review clean code", "check clean code", "Uncle Bob review", "SOLID review",
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
- Triggered by: "review highload", "review scalability", "highload review",
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.** A misspelling used to be
208
- accepted with exit 0, so `review complete --disposition ...` printed
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, and this file used to say
683
- otherwise.** `reviewer-input.schema.json` states the rule and no production
684
- TypeScript loads that schema; reviewer dispatch is an action the host agent
685
- takes, not a `keryx` invocation, so there is no point at which a malformed
686
- dispatch could be rejected. `reviewer-input` is also absent from the `CONTRACTS`
687
- registry (`src/gdskills/contracts.ts`), so `keryx skills contracts validate`
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. Covers "analyze PR comments", "review PR feedback", "what did reviewers
8
- say", "parse PR #N", "explain PR comments", and — with `--fix` — validating every
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. Covers "review style",
7
- "style review", "check naming", "check readability", or dispatched by review-orchestrator
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. Covers "verify the findings", "check these
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