@mrciphersmith/keryx 0.2.98 → 0.2.99

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 (182) hide show
  1. package/dist/cli.js +4057 -2510
  2. package/dist/core.js +39 -1
  3. package/package.json +1 -1
  4. package/src/gdskills/bundled/rules/core/cli-interface-design.mdc +237 -0
  5. package/src/gdskills/bundled/rules/core/definition-of-done.mdc +116 -0
  6. package/src/gdskills/bundled/rules/core/skills-storage-workflow.mdc +101 -11
  7. package/src/gdskills/bundled/rules/core/subagent-status-protocol.md +9 -2
  8. package/src/gdskills/bundled/skills/core/reviewer-skill-creator/SKILL.md +42 -5
  9. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.md +19 -3
  10. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.md +20 -4
  11. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +32 -9
  12. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +18 -4
  13. package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md +21 -5
  14. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.md +4 -4
  15. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/orchestrator-prompt.md +1 -1
  16. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.md +42 -2
  17. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +23 -9
  18. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +33 -31
  19. package/src/gdskills/bundled/skills/orchestration/task-implementer/output-contract.schema.json +32 -1
  20. package/src/gdskills/bundled/skills/planning/autodoc-analyst/SKILL.md +16 -0
  21. package/src/gdskills/bundled/skills/planning/autodoc-architect/SKILL.md +16 -0
  22. package/src/gdskills/bundled/skills/planning/autodoc-assembler/SKILL.md +16 -0
  23. package/src/gdskills/bundled/skills/planning/autodoc-orchestrator/SKILL.md +17 -0
  24. package/src/gdskills/bundled/skills/planning/autodoc-scanner/SKILL.md +16 -0
  25. package/src/gdskills/bundled/skills/planning/autodoc-writer/SKILL.md +16 -0
  26. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.md +28 -3
  27. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.codex.md +17 -0
  28. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.cursor.md +17 -0
  29. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.md +17 -0
  30. package/src/gdskills/bundled/skills/planning/docpack-orchestrator/SKILL.md +32 -2
  31. package/src/gdskills/bundled/skills/planning/docpack-review/SKILL.md +14 -2
  32. package/src/gdskills/bundled/skills/planning/interview/SKILL.md +29 -7
  33. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +32 -6
  34. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.codex.md +16 -0
  35. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.cursor.md +16 -0
  36. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.md +16 -0
  37. package/src/gdskills/bundled/skills/planning/planner/SKILL.codex.md +17 -0
  38. package/src/gdskills/bundled/skills/planning/planner/SKILL.cursor.md +17 -0
  39. package/src/gdskills/bundled/skills/planning/planner/SKILL.md +17 -0
  40. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.md +20 -3
  41. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.codex.md +16 -0
  42. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.cursor.md +16 -0
  43. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.md +16 -0
  44. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.codex.md +16 -0
  45. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.cursor.md +16 -0
  46. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.md +16 -0
  47. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.codex.md +4 -0
  48. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.cursor.md +4 -0
  49. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.md +4 -0
  50. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.codex.md +4 -0
  51. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.cursor.md +4 -0
  52. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.md +4 -0
  53. package/src/gdskills/bundled/skills/platform/agent-entrypoint-distiller/SKILL.md +31 -4
  54. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.md +26 -2
  55. package/src/gdskills/bundled/skills/platform/hookify/SKILL.md +28 -3
  56. package/src/gdskills/bundled/skills/quality/api-truth/SKILL.md +226 -0
  57. package/src/gdskills/bundled/skills/quality/changelog/SKILL.md +24 -4
  58. package/src/gdskills/bundled/skills/quality/commit/SKILL.md +24 -3
  59. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.md +24 -3
  60. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.md +25 -4
  61. package/src/gdskills/bundled/skills/quality/deploy/SKILL.md +26 -3
  62. package/src/gdskills/bundled/skills/quality/deprecation-path/SKILL.md +268 -0
  63. package/src/gdskills/bundled/skills/quality/fresh-eyes/SKILL.md +190 -0
  64. package/src/gdskills/bundled/skills/quality/metaproject-security/SKILL.md +24 -3
  65. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.md +29 -8
  66. package/src/gdskills/bundled/skills/quality/pr/SKILL.md +24 -4
  67. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.md +25 -2
  68. package/src/gdskills/bundled/skills/quality/push/SKILL.md +24 -3
  69. package/src/gdskills/bundled/skills/quality/root-cause/SKILL.md +204 -0
  70. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.md +25 -4
  71. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.md +24 -3
  72. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.md +17 -2
  73. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.md +40 -5
  74. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.md +41 -1
  75. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.md +44 -2
  76. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.md +44 -4
  77. package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +3 -3
  78. package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +2 -3
  79. package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +4 -4
  80. package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +36 -2
  81. package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +37 -3
  82. package/src/gdskills/bundled/skills/review/review-frontend/SKILL.md +2 -4
  83. package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +36 -2
  84. package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +3 -5
  85. package/src/gdskills/bundled/skills/review/review-layout/SKILL.md +23 -2
  86. package/src/gdskills/bundled/skills/review/review-logic/SKILL.md +3 -3
  87. package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +9 -29
  88. package/src/gdskills/bundled/skills/review/review-performance/SKILL.md +9 -9
  89. package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +3 -2
  90. package/src/gdskills/bundled/skills/review/review-regression/SKILL.md +33 -2
  91. package/src/gdskills/bundled/skills/review/review-security-code/SKILL.md +4 -2
  92. package/src/gdskills/bundled/skills/review/review-style/SKILL.md +2 -2
  93. package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +40 -2
  94. package/src/gdskills/bundled/skills/review/review-verifier/SKILL.md +1 -1
  95. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.codex.md +0 -330
  96. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.cursor.md +0 -330
  97. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.opencode.md +0 -330
  98. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.zed.md +0 -330
  99. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.codex.md +0 -655
  100. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.cursor.md +0 -655
  101. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.opencode.md +0 -655
  102. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.zed.md +0 -655
  103. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +0 -424
  104. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +0 -424
  105. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +0 -424
  106. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +0 -424
  107. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +0 -163
  108. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +0 -163
  109. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.codex.md +0 -373
  110. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.cursor.md +0 -373
  111. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.opencode.md +0 -373
  112. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.zed.md +0 -373
  113. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.codex.md +0 -374
  114. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.cursor.md +0 -374
  115. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.opencode.md +0 -374
  116. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.zed.md +0 -374
  117. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +0 -2232
  118. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +0 -2232
  119. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +0 -2232
  120. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +0 -2232
  121. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +0 -668
  122. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +0 -668
  123. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +0 -668
  124. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +0 -668
  125. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.codex.md +0 -90
  126. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.cursor.md +0 -90
  127. package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +0 -187
  128. package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +0 -187
  129. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +0 -105
  130. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +0 -105
  131. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.codex.md +0 -193
  132. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.cursor.md +0 -193
  133. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.opencode.md +0 -193
  134. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.zed.md +0 -193
  135. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.codex.md +0 -87
  136. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.cursor.md +0 -87
  137. package/src/gdskills/bundled/skills/platform/hookify/SKILL.codex.md +0 -100
  138. package/src/gdskills/bundled/skills/platform/hookify/SKILL.cursor.md +0 -100
  139. package/src/gdskills/bundled/skills/quality/changelog/SKILL.codex.md +0 -84
  140. package/src/gdskills/bundled/skills/quality/changelog/SKILL.cursor.md +0 -84
  141. package/src/gdskills/bundled/skills/quality/commit/SKILL.codex.md +0 -66
  142. package/src/gdskills/bundled/skills/quality/commit/SKILL.cursor.md +0 -66
  143. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.codex.md +0 -66
  144. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.cursor.md +0 -66
  145. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.codex.md +0 -81
  146. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.cursor.md +0 -81
  147. package/src/gdskills/bundled/skills/quality/deploy/SKILL.codex.md +0 -70
  148. package/src/gdskills/bundled/skills/quality/deploy/SKILL.cursor.md +0 -70
  149. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.codex.md +0 -83
  150. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.cursor.md +0 -83
  151. package/src/gdskills/bundled/skills/quality/pr/SKILL.codex.md +0 -75
  152. package/src/gdskills/bundled/skills/quality/pr/SKILL.cursor.md +0 -75
  153. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.codex.md +0 -378
  154. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.cursor.md +0 -378
  155. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.opencode.md +0 -378
  156. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.zed.md +0 -378
  157. package/src/gdskills/bundled/skills/quality/push/SKILL.codex.md +0 -52
  158. package/src/gdskills/bundled/skills/quality/push/SKILL.cursor.md +0 -52
  159. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.codex.md +0 -108
  160. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.cursor.md +0 -108
  161. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.codex.md +0 -80
  162. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.cursor.md +0 -80
  163. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.codex.md +0 -345
  164. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.cursor.md +0 -345
  165. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.opencode.md +0 -345
  166. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.zed.md +0 -345
  167. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.codex.md +0 -203
  168. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.cursor.md +0 -203
  169. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.opencode.md +0 -203
  170. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.zed.md +0 -203
  171. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.codex.md +0 -243
  172. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.cursor.md +0 -243
  173. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.opencode.md +0 -243
  174. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.zed.md +0 -243
  175. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.codex.md +0 -259
  176. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.cursor.md +0 -259
  177. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.opencode.md +0 -259
  178. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.zed.md +0 -259
  179. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.codex.md +0 -168
  180. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.cursor.md +0 -168
  181. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.opencode.md +0 -168
  182. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.zed.md +0 -168
@@ -1,168 +0,0 @@
1
- ---
2
- name: code-style-review
3
- description: "Detailed code style and architecture review using code-style-patterns.mdc. Reviews current branch changes. Checks naming, organization, patterns, TypeScript usage. Use when: style validation needed, architecture review."
4
- triggers:
5
- - "Style review"
6
- - "Check code style"
7
- - "Architecture review"
8
- metadata:
9
- author: "MrCipherSmith"
10
- version: "1.0.0"
11
- category: "review"
12
- compatible_harnesses: "cursor,codex,zed,opencode,claude"
13
- license: "MIT"
14
- ---
15
-
16
-
17
- # Code Style Review
18
-
19
- Проводи ревью только изменений текущей ветки от момента отбранчевания от родительской ветки, строго по правилам из `.metaproject/rules/core/code-style-patterns.mdc`.
20
-
21
- ## Scope
22
-
23
- - Если пользователь **не передал commit hash/range**, ревьюируй полный срез от merge-base до текущего рабочего дерева:
24
- - закоммиченные изменения (`BASE_SHA..HEAD`)
25
- - локальные незакоммиченные (`staged/unstaged/untracked`)
26
- - Если пользователь **явно передал commit hash/range**, ревьюируй только запрошенный диапазон; незакоммиченные изменения не добавляй, если это отдельно не попросили.
27
- - Не обсуждай легаси вне изменённого скоупа.
28
- - Фидбек должен быть подробным: проблема -> почему это проблема -> где (файл/участок) -> что сделать -> пример исправления.
29
- - Исправления предлагай в виде unified diff-патчей, не вноси изменения в код напрямую.
30
-
31
- ## Scope Detection
32
-
33
- See shared script: `.metaproject/skills/gdskills/shared/git-merge-base.md`
34
-
35
- Run the script from that file to determine MERGE_BASE and SCOPE before proceeding with the review.
36
-
37
- Собери входные данные:
38
-
39
- ### A) Режим по умолчанию (без hash/range)
40
-
41
- ```bash
42
- git status
43
-
44
- # Закоммиченная часть ветки
45
- git log --oneline "${BASE_SHA}..HEAD"
46
- git diff --name-status --find-renames "${BASE_SHA}..HEAD"
47
- git diff --find-renames "${BASE_SHA}..HEAD"
48
-
49
- # Полный текущий срез от merge-base до рабочего дерева
50
- # включает commit'ы + staged + unstaged
51
- # (untracked смотри через git status / git ls-files)
52
- git diff --name-status --find-renames "${BASE_SHA}"
53
- git diff --find-renames "${BASE_SHA}"
54
- git ls-files --others --exclude-standard
55
- ```
56
-
57
- ### B) Режим с явным hash/range
58
-
59
- ```bash
60
- git show --stat --name-status --patch <COMMIT_SHA>
61
- git log --oneline <FROM_SHA>..<TO_SHA>
62
- git diff --name-status --find-renames <FROM_SHA>..<TO_SHA>
63
- git diff --find-renames <FROM_SHA>..<TO_SHA>
64
- ```
65
-
66
- Используй только эти изменения как вход для ревью.
67
-
68
- ## Review checklist (по `.metaproject/rules/core/code-style-patterns.mdc`)
69
-
70
- ### TypeScript Strictness
71
- - Запрещён `any` (предлагай `unknown` + type guards / корректные типы).
72
- - Props через `interface`, префикс `I*`.
73
- - Предпочитай `?.` и `??`, избегай `!` (non-null assertion).
74
-
75
- ### MobX
76
- - В сторах обязателен `makeObservable(this)` в конструкторе.
77
- - Для асинхронных мутаций после `await` - `runInAction`.
78
- - Методы экшенов: `@action.bound`.
79
- - Derived state: `@computed`.
80
- - Для коллекций/списков - `@observable.shallow` (когда применимо).
81
- - Inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный `@action.bound` только для методов вызываемых из React-компонентов.
82
-
83
- ### React Components
84
- - Компоненты, читающие observable, обязаны быть `observer(...)`.
85
- - MVVM: логика и side effects в Store/Service, а не во View.
86
- - Запрещай бизнес-логику в `useEffect` (предлагай store actions/reactions).
87
- - Следи за порядком структуры файла: imports -> interfaces -> component -> helpers.
88
-
89
- ### Architecture & Layers
90
- - API/IO не во View: в Service/Store.
91
- - Взаимодействие со Store - через actions (без прямых мутаций вне экшенов).
92
-
93
- ### Anti-patterns severity
94
- - Critical: API во View, missing `observer`, direct store mutation, грубые нарушения MobX async.
95
- - Warnings: `console.log` вместо `AppLogger`, inline object props без memo/const, тяжёлая логика в render, спорные типы, public inter-store callbacks/handlers.
96
-
97
- ## Output format (обязательный отчёт)
98
-
99
- Сформируй отчёт в Markdown строго по шаблону:
100
-
101
- ```markdown
102
- ## Summary
103
- - [1-3 bullets] Что изменилось и общий уровень соответствия стилю
104
-
105
- ## Scope
106
- - Branch: `<BRANCH>`
107
- - Parent ref: `<PARENT>`
108
- - Merge-base: `<BASE_SHA>`
109
- - Scope mode: `<default-with-uncommitted | explicit-hash-range>`
110
-
111
- ## Critical issues (must fix)
112
- ### [Короткий заголовок проблемы]
113
- - **Rule**: [ссылка/название секции из core/code-style-patterns.mdc]
114
- - **Why**: [объяснение]
115
- - **Where**: `path/to/file.tsx` (строки из diff)
116
- - **Fix**: [что сделать]
117
- - **Proposed patch**:
118
- ```diff
119
- [unified diff]
120
- ```
121
-
122
- ## Warnings
123
- ... (тот же формат, без "must fix")
124
-
125
- ## Suggestions
126
- ... (тот же формат, improvements без обязательности)
127
-
128
- ## File-by-file notes
129
- - `path/to/file`: [коротко, только по изменённым участкам]
130
- ```
131
-
132
- ## Rules of engagement
133
-
134
- - Привязывай замечания к конкретным изменённым строкам (по diff).
135
- - Если есть несколько вариантов, предлагай один default и короткий альтернативный.
136
- - Не предлагай новые библиотеки без явного запроса.
137
-
138
- ---
139
-
140
- ## Scope Boundaries
141
-
142
- This skill covers **code style and architecture patterns** — naming, organization, TypeScript usage, pattern compliance per `code-style-patterns.mdc`.
143
-
144
- | Concern | This skill | Use instead |
145
- |---------|-----------|-------------|
146
- | Naming conventions, file organization, import order | ✅ YES | — |
147
- | TypeScript patterns, component/hook structure | ✅ YES | — |
148
- | Architecture pattern compliance | ✅ YES | — |
149
- | Logic correctness, type safety deep dives | ❌ NO | `code-ai-review` or `code-learned-review` |
150
- | MobX store internals | ❌ NO | `code-mobx-store-review` |
151
-
152
- ---
153
-
154
- ## Job Context Awareness
155
-
156
- When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
157
-
158
- ```
159
- JOB_NAME: <job-name>
160
- CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
161
- ```
162
-
163
- If provided and the file exists, read the context document before starting the review. Use it to:
164
- - Understand which libraries and patterns were intentionally chosen for the implementation
165
- - Validate code style against documented project conventions from the context
166
- - Provide more accurate findings by understanding the project's architectural decisions
167
-
168
- If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.
@@ -1,168 +0,0 @@
1
- ---
2
- name: code-style-review
3
- description: "Detailed code style and architecture review using code-style-patterns.mdc. Reviews current branch changes. Checks naming, organization, patterns, TypeScript usage. Use when: style validation needed, architecture review."
4
- triggers:
5
- - "Style review"
6
- - "Check code style"
7
- - "Architecture review"
8
- metadata:
9
- author: "MrCipherSmith"
10
- version: "1.0.0"
11
- category: "review"
12
- compatible_harnesses: "cursor,codex,zed,opencode,claude"
13
- license: "MIT"
14
- ---
15
-
16
-
17
- # Code Style Review
18
-
19
- Проводи ревью только изменений текущей ветки от момента отбранчевания от родительской ветки, строго по правилам из `.metaproject/rules/core/code-style-patterns.mdc`.
20
-
21
- ## Scope
22
-
23
- - Если пользователь **не передал commit hash/range**, ревьюируй полный срез от merge-base до текущего рабочего дерева:
24
- - закоммиченные изменения (`BASE_SHA..HEAD`)
25
- - локальные незакоммиченные (`staged/unstaged/untracked`)
26
- - Если пользователь **явно передал commit hash/range**, ревьюируй только запрошенный диапазон; незакоммиченные изменения не добавляй, если это отдельно не попросили.
27
- - Не обсуждай легаси вне изменённого скоупа.
28
- - Фидбек должен быть подробным: проблема -> почему это проблема -> где (файл/участок) -> что сделать -> пример исправления.
29
- - Исправления предлагай в виде unified diff-патчей, не вноси изменения в код напрямую.
30
-
31
- ## Scope Detection
32
-
33
- See shared script: `.metaproject/skills/gdskills/shared/git-merge-base.md`
34
-
35
- Run the script from that file to determine MERGE_BASE and SCOPE before proceeding with the review.
36
-
37
- Собери входные данные:
38
-
39
- ### A) Режим по умолчанию (без hash/range)
40
-
41
- ```bash
42
- git status
43
-
44
- # Закоммиченная часть ветки
45
- git log --oneline "${BASE_SHA}..HEAD"
46
- git diff --name-status --find-renames "${BASE_SHA}..HEAD"
47
- git diff --find-renames "${BASE_SHA}..HEAD"
48
-
49
- # Полный текущий срез от merge-base до рабочего дерева
50
- # включает commit'ы + staged + unstaged
51
- # (untracked смотри через git status / git ls-files)
52
- git diff --name-status --find-renames "${BASE_SHA}"
53
- git diff --find-renames "${BASE_SHA}"
54
- git ls-files --others --exclude-standard
55
- ```
56
-
57
- ### B) Режим с явным hash/range
58
-
59
- ```bash
60
- git show --stat --name-status --patch <COMMIT_SHA>
61
- git log --oneline <FROM_SHA>..<TO_SHA>
62
- git diff --name-status --find-renames <FROM_SHA>..<TO_SHA>
63
- git diff --find-renames <FROM_SHA>..<TO_SHA>
64
- ```
65
-
66
- Используй только эти изменения как вход для ревью.
67
-
68
- ## Review checklist (по `.metaproject/rules/core/code-style-patterns.mdc`)
69
-
70
- ### TypeScript Strictness
71
- - Запрещён `any` (предлагай `unknown` + type guards / корректные типы).
72
- - Props через `interface`, префикс `I*`.
73
- - Предпочитай `?.` и `??`, избегай `!` (non-null assertion).
74
-
75
- ### MobX
76
- - В сторах обязателен `makeObservable(this)` в конструкторе.
77
- - Для асинхронных мутаций после `await` - `runInAction`.
78
- - Методы экшенов: `@action.bound`.
79
- - Derived state: `@computed`.
80
- - Для коллекций/списков - `@observable.shallow` (когда применимо).
81
- - Inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный `@action.bound` только для методов вызываемых из React-компонентов.
82
-
83
- ### React Components
84
- - Компоненты, читающие observable, обязаны быть `observer(...)`.
85
- - MVVM: логика и side effects в Store/Service, а не во View.
86
- - Запрещай бизнес-логику в `useEffect` (предлагай store actions/reactions).
87
- - Следи за порядком структуры файла: imports -> interfaces -> component -> helpers.
88
-
89
- ### Architecture & Layers
90
- - API/IO не во View: в Service/Store.
91
- - Взаимодействие со Store - через actions (без прямых мутаций вне экшенов).
92
-
93
- ### Anti-patterns severity
94
- - Critical: API во View, missing `observer`, direct store mutation, грубые нарушения MobX async.
95
- - Warnings: `console.log` вместо `AppLogger`, inline object props без memo/const, тяжёлая логика в render, спорные типы, public inter-store callbacks/handlers.
96
-
97
- ## Output format (обязательный отчёт)
98
-
99
- Сформируй отчёт в Markdown строго по шаблону:
100
-
101
- ```markdown
102
- ## Summary
103
- - [1-3 bullets] Что изменилось и общий уровень соответствия стилю
104
-
105
- ## Scope
106
- - Branch: `<BRANCH>`
107
- - Parent ref: `<PARENT>`
108
- - Merge-base: `<BASE_SHA>`
109
- - Scope mode: `<default-with-uncommitted | explicit-hash-range>`
110
-
111
- ## Critical issues (must fix)
112
- ### [Короткий заголовок проблемы]
113
- - **Rule**: [ссылка/название секции из core/code-style-patterns.mdc]
114
- - **Why**: [объяснение]
115
- - **Where**: `path/to/file.tsx` (строки из diff)
116
- - **Fix**: [что сделать]
117
- - **Proposed patch**:
118
- ```diff
119
- [unified diff]
120
- ```
121
-
122
- ## Warnings
123
- ... (тот же формат, без "must fix")
124
-
125
- ## Suggestions
126
- ... (тот же формат, improvements без обязательности)
127
-
128
- ## File-by-file notes
129
- - `path/to/file`: [коротко, только по изменённым участкам]
130
- ```
131
-
132
- ## Rules of engagement
133
-
134
- - Привязывай замечания к конкретным изменённым строкам (по diff).
135
- - Если есть несколько вариантов, предлагай один default и короткий альтернативный.
136
- - Не предлагай новые библиотеки без явного запроса.
137
-
138
- ---
139
-
140
- ## Scope Boundaries
141
-
142
- This skill covers **code style and architecture patterns** — naming, organization, TypeScript usage, pattern compliance per `code-style-patterns.mdc`.
143
-
144
- | Concern | This skill | Use instead |
145
- |---------|-----------|-------------|
146
- | Naming conventions, file organization, import order | ✅ YES | — |
147
- | TypeScript patterns, component/hook structure | ✅ YES | — |
148
- | Architecture pattern compliance | ✅ YES | — |
149
- | Logic correctness, type safety deep dives | ❌ NO | `code-ai-review` or `code-learned-review` |
150
- | MobX store internals | ❌ NO | `code-mobx-store-review` |
151
-
152
- ---
153
-
154
- ## Job Context Awareness
155
-
156
- When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
157
-
158
- ```
159
- JOB_NAME: <job-name>
160
- CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
161
- ```
162
-
163
- If provided and the file exists, read the context document before starting the review. Use it to:
164
- - Understand which libraries and patterns were intentionally chosen for the implementation
165
- - Validate code style against documented project conventions from the context
166
- - Provide more accurate findings by understanding the project's architectural decisions
167
-
168
- If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.
@@ -1,168 +0,0 @@
1
- ---
2
- name: code-style-review
3
- description: "Detailed code style and architecture review using code-style-patterns.mdc. Reviews current branch changes. Checks naming, organization, patterns, TypeScript usage. Use when: style validation needed, architecture review."
4
- triggers:
5
- - "Style review"
6
- - "Check code style"
7
- - "Architecture review"
8
- metadata:
9
- author: "MrCipherSmith"
10
- version: "1.0.0"
11
- category: "review"
12
- compatible_harnesses: "cursor,codex,zed,opencode,claude"
13
- license: "MIT"
14
- ---
15
-
16
-
17
- # Code Style Review
18
-
19
- Проводи ревью только изменений текущей ветки от момента отбранчевания от родительской ветки, строго по правилам из `.metaproject/rules/core/code-style-patterns.mdc`.
20
-
21
- ## Scope
22
-
23
- - Если пользователь **не передал commit hash/range**, ревьюируй полный срез от merge-base до текущего рабочего дерева:
24
- - закоммиченные изменения (`BASE_SHA..HEAD`)
25
- - локальные незакоммиченные (`staged/unstaged/untracked`)
26
- - Если пользователь **явно передал commit hash/range**, ревьюируй только запрошенный диапазон; незакоммиченные изменения не добавляй, если это отдельно не попросили.
27
- - Не обсуждай легаси вне изменённого скоупа.
28
- - Фидбек должен быть подробным: проблема -> почему это проблема -> где (файл/участок) -> что сделать -> пример исправления.
29
- - Исправления предлагай в виде unified diff-патчей, не вноси изменения в код напрямую.
30
-
31
- ## Scope Detection
32
-
33
- See shared script: `.metaproject/skills/gdskills/shared/git-merge-base.md`
34
-
35
- Run the script from that file to determine MERGE_BASE and SCOPE before proceeding with the review.
36
-
37
- Собери входные данные:
38
-
39
- ### A) Режим по умолчанию (без hash/range)
40
-
41
- ```bash
42
- git status
43
-
44
- # Закоммиченная часть ветки
45
- git log --oneline "${BASE_SHA}..HEAD"
46
- git diff --name-status --find-renames "${BASE_SHA}..HEAD"
47
- git diff --find-renames "${BASE_SHA}..HEAD"
48
-
49
- # Полный текущий срез от merge-base до рабочего дерева
50
- # включает commit'ы + staged + unstaged
51
- # (untracked смотри через git status / git ls-files)
52
- git diff --name-status --find-renames "${BASE_SHA}"
53
- git diff --find-renames "${BASE_SHA}"
54
- git ls-files --others --exclude-standard
55
- ```
56
-
57
- ### B) Режим с явным hash/range
58
-
59
- ```bash
60
- git show --stat --name-status --patch <COMMIT_SHA>
61
- git log --oneline <FROM_SHA>..<TO_SHA>
62
- git diff --name-status --find-renames <FROM_SHA>..<TO_SHA>
63
- git diff --find-renames <FROM_SHA>..<TO_SHA>
64
- ```
65
-
66
- Используй только эти изменения как вход для ревью.
67
-
68
- ## Review checklist (по `.metaproject/rules/core/code-style-patterns.mdc`)
69
-
70
- ### TypeScript Strictness
71
- - Запрещён `any` (предлагай `unknown` + type guards / корректные типы).
72
- - Props через `interface`, префикс `I*`.
73
- - Предпочитай `?.` и `??`, избегай `!` (non-null assertion).
74
-
75
- ### MobX
76
- - В сторах обязателен `makeObservable(this)` в конструкторе.
77
- - Для асинхронных мутаций после `await` - `runInAction`.
78
- - Методы экшенов: `@action.bound`.
79
- - Derived state: `@computed`.
80
- - Для коллекций/списков - `@observable.shallow` (когда применимо).
81
- - Inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный `@action.bound` только для методов вызываемых из React-компонентов.
82
-
83
- ### React Components
84
- - Компоненты, читающие observable, обязаны быть `observer(...)`.
85
- - MVVM: логика и side effects в Store/Service, а не во View.
86
- - Запрещай бизнес-логику в `useEffect` (предлагай store actions/reactions).
87
- - Следи за порядком структуры файла: imports -> interfaces -> component -> helpers.
88
-
89
- ### Architecture & Layers
90
- - API/IO не во View: в Service/Store.
91
- - Взаимодействие со Store - через actions (без прямых мутаций вне экшенов).
92
-
93
- ### Anti-patterns severity
94
- - Critical: API во View, missing `observer`, direct store mutation, грубые нарушения MobX async.
95
- - Warnings: `console.log` вместо `AppLogger`, inline object props без memo/const, тяжёлая логика в render, спорные типы, public inter-store callbacks/handlers.
96
-
97
- ## Output format (обязательный отчёт)
98
-
99
- Сформируй отчёт в Markdown строго по шаблону:
100
-
101
- ```markdown
102
- ## Summary
103
- - [1-3 bullets] Что изменилось и общий уровень соответствия стилю
104
-
105
- ## Scope
106
- - Branch: `<BRANCH>`
107
- - Parent ref: `<PARENT>`
108
- - Merge-base: `<BASE_SHA>`
109
- - Scope mode: `<default-with-uncommitted | explicit-hash-range>`
110
-
111
- ## Critical issues (must fix)
112
- ### [Короткий заголовок проблемы]
113
- - **Rule**: [ссылка/название секции из core/code-style-patterns.mdc]
114
- - **Why**: [объяснение]
115
- - **Where**: `path/to/file.tsx` (строки из diff)
116
- - **Fix**: [что сделать]
117
- - **Proposed patch**:
118
- ```diff
119
- [unified diff]
120
- ```
121
-
122
- ## Warnings
123
- ... (тот же формат, без "must fix")
124
-
125
- ## Suggestions
126
- ... (тот же формат, improvements без обязательности)
127
-
128
- ## File-by-file notes
129
- - `path/to/file`: [коротко, только по изменённым участкам]
130
- ```
131
-
132
- ## Rules of engagement
133
-
134
- - Привязывай замечания к конкретным изменённым строкам (по diff).
135
- - Если есть несколько вариантов, предлагай один default и короткий альтернативный.
136
- - Не предлагай новые библиотеки без явного запроса.
137
-
138
- ---
139
-
140
- ## Scope Boundaries
141
-
142
- This skill covers **code style and architecture patterns** — naming, organization, TypeScript usage, pattern compliance per `code-style-patterns.mdc`.
143
-
144
- | Concern | This skill | Use instead |
145
- |---------|-----------|-------------|
146
- | Naming conventions, file organization, import order | ✅ YES | — |
147
- | TypeScript patterns, component/hook structure | ✅ YES | — |
148
- | Architecture pattern compliance | ✅ YES | — |
149
- | Logic correctness, type safety deep dives | ❌ NO | `code-ai-review` or `code-learned-review` |
150
- | MobX store internals | ❌ NO | `code-mobx-store-review` |
151
-
152
- ---
153
-
154
- ## Job Context Awareness
155
-
156
- When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
157
-
158
- ```
159
- JOB_NAME: <job-name>
160
- CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
161
- ```
162
-
163
- If provided and the file exists, read the context document before starting the review. Use it to:
164
- - Understand which libraries and patterns were intentionally chosen for the implementation
165
- - Validate code style against documented project conventions from the context
166
- - Provide more accurate findings by understanding the project's architectural decisions
167
-
168
- If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.
@@ -1,168 +0,0 @@
1
- ---
2
- name: code-style-review
3
- description: "Detailed code style and architecture review using code-style-patterns.mdc. Reviews current branch changes. Checks naming, organization, patterns, TypeScript usage. Use when: style validation needed, architecture review."
4
- triggers:
5
- - "Style review"
6
- - "Check code style"
7
- - "Architecture review"
8
- metadata:
9
- author: "MrCipherSmith"
10
- version: "1.0.0"
11
- category: "review"
12
- compatible_harnesses: "cursor,codex,zed,opencode,claude"
13
- license: "MIT"
14
- ---
15
-
16
-
17
- # Code Style Review
18
-
19
- Проводи ревью только изменений текущей ветки от момента отбранчевания от родительской ветки, строго по правилам из `.metaproject/rules/core/code-style-patterns.mdc`.
20
-
21
- ## Scope
22
-
23
- - Если пользователь **не передал commit hash/range**, ревьюируй полный срез от merge-base до текущего рабочего дерева:
24
- - закоммиченные изменения (`BASE_SHA..HEAD`)
25
- - локальные незакоммиченные (`staged/unstaged/untracked`)
26
- - Если пользователь **явно передал commit hash/range**, ревьюируй только запрошенный диапазон; незакоммиченные изменения не добавляй, если это отдельно не попросили.
27
- - Не обсуждай легаси вне изменённого скоупа.
28
- - Фидбек должен быть подробным: проблема -> почему это проблема -> где (файл/участок) -> что сделать -> пример исправления.
29
- - Исправления предлагай в виде unified diff-патчей, не вноси изменения в код напрямую.
30
-
31
- ## Scope Detection
32
-
33
- See shared script: `.metaproject/skills/gdskills/shared/git-merge-base.md`
34
-
35
- Run the script from that file to determine MERGE_BASE and SCOPE before proceeding with the review.
36
-
37
- Собери входные данные:
38
-
39
- ### A) Режим по умолчанию (без hash/range)
40
-
41
- ```bash
42
- git status
43
-
44
- # Закоммиченная часть ветки
45
- git log --oneline "${BASE_SHA}..HEAD"
46
- git diff --name-status --find-renames "${BASE_SHA}..HEAD"
47
- git diff --find-renames "${BASE_SHA}..HEAD"
48
-
49
- # Полный текущий срез от merge-base до рабочего дерева
50
- # включает commit'ы + staged + unstaged
51
- # (untracked смотри через git status / git ls-files)
52
- git diff --name-status --find-renames "${BASE_SHA}"
53
- git diff --find-renames "${BASE_SHA}"
54
- git ls-files --others --exclude-standard
55
- ```
56
-
57
- ### B) Режим с явным hash/range
58
-
59
- ```bash
60
- git show --stat --name-status --patch <COMMIT_SHA>
61
- git log --oneline <FROM_SHA>..<TO_SHA>
62
- git diff --name-status --find-renames <FROM_SHA>..<TO_SHA>
63
- git diff --find-renames <FROM_SHA>..<TO_SHA>
64
- ```
65
-
66
- Используй только эти изменения как вход для ревью.
67
-
68
- ## Review checklist (по `.metaproject/rules/core/code-style-patterns.mdc`)
69
-
70
- ### TypeScript Strictness
71
- - Запрещён `any` (предлагай `unknown` + type guards / корректные типы).
72
- - Props через `interface`, префикс `I*`.
73
- - Предпочитай `?.` и `??`, избегай `!` (non-null assertion).
74
-
75
- ### MobX
76
- - В сторах обязателен `makeObservable(this)` в конструкторе.
77
- - Для асинхронных мутаций после `await` - `runInAction`.
78
- - Методы экшенов: `@action.bound`.
79
- - Derived state: `@computed`.
80
- - Для коллекций/списков - `@observable.shallow` (когда применимо).
81
- - Inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный `@action.bound` только для методов вызываемых из React-компонентов.
82
-
83
- ### React Components
84
- - Компоненты, читающие observable, обязаны быть `observer(...)`.
85
- - MVVM: логика и side effects в Store/Service, а не во View.
86
- - Запрещай бизнес-логику в `useEffect` (предлагай store actions/reactions).
87
- - Следи за порядком структуры файла: imports -> interfaces -> component -> helpers.
88
-
89
- ### Architecture & Layers
90
- - API/IO не во View: в Service/Store.
91
- - Взаимодействие со Store - через actions (без прямых мутаций вне экшенов).
92
-
93
- ### Anti-patterns severity
94
- - Critical: API во View, missing `observer`, direct store mutation, грубые нарушения MobX async.
95
- - Warnings: `console.log` вместо `AppLogger`, inline object props без memo/const, тяжёлая логика в render, спорные типы, public inter-store callbacks/handlers.
96
-
97
- ## Output format (обязательный отчёт)
98
-
99
- Сформируй отчёт в Markdown строго по шаблону:
100
-
101
- ```markdown
102
- ## Summary
103
- - [1-3 bullets] Что изменилось и общий уровень соответствия стилю
104
-
105
- ## Scope
106
- - Branch: `<BRANCH>`
107
- - Parent ref: `<PARENT>`
108
- - Merge-base: `<BASE_SHA>`
109
- - Scope mode: `<default-with-uncommitted | explicit-hash-range>`
110
-
111
- ## Critical issues (must fix)
112
- ### [Короткий заголовок проблемы]
113
- - **Rule**: [ссылка/название секции из core/code-style-patterns.mdc]
114
- - **Why**: [объяснение]
115
- - **Where**: `path/to/file.tsx` (строки из diff)
116
- - **Fix**: [что сделать]
117
- - **Proposed patch**:
118
- ```diff
119
- [unified diff]
120
- ```
121
-
122
- ## Warnings
123
- ... (тот же формат, без "must fix")
124
-
125
- ## Suggestions
126
- ... (тот же формат, improvements без обязательности)
127
-
128
- ## File-by-file notes
129
- - `path/to/file`: [коротко, только по изменённым участкам]
130
- ```
131
-
132
- ## Rules of engagement
133
-
134
- - Привязывай замечания к конкретным изменённым строкам (по diff).
135
- - Если есть несколько вариантов, предлагай один default и короткий альтернативный.
136
- - Не предлагай новые библиотеки без явного запроса.
137
-
138
- ---
139
-
140
- ## Scope Boundaries
141
-
142
- This skill covers **code style and architecture patterns** — naming, organization, TypeScript usage, pattern compliance per `code-style-patterns.mdc`.
143
-
144
- | Concern | This skill | Use instead |
145
- |---------|-----------|-------------|
146
- | Naming conventions, file organization, import order | ✅ YES | — |
147
- | TypeScript patterns, component/hook structure | ✅ YES | — |
148
- | Architecture pattern compliance | ✅ YES | — |
149
- | Logic correctness, type safety deep dives | ❌ NO | `code-ai-review` or `code-learned-review` |
150
- | MobX store internals | ❌ NO | `code-mobx-store-review` |
151
-
152
- ---
153
-
154
- ## Job Context Awareness
155
-
156
- When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
157
-
158
- ```
159
- JOB_NAME: <job-name>
160
- CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
161
- ```
162
-
163
- If provided and the file exists, read the context document before starting the review. Use it to:
164
- - Understand which libraries and patterns were intentionally chosen for the implementation
165
- - Validate code style against documented project conventions from the context
166
- - Provide more accurate findings by understanding the project's architectural decisions
167
-
168
- If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.