@mrciphersmith/keryx 0.2.97 → 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 (210) hide show
  1. package/dist/cli.js +4583 -2702
  2. package/dist/core.js +40 -2
  3. package/package.json +1 -1
  4. package/src/gdskills/bundled/rules/core/api-contracts.mdc +1 -0
  5. package/src/gdskills/bundled/rules/core/cli-interface-design.mdc +237 -0
  6. package/src/gdskills/bundled/rules/core/code-style-patterns.mdc +1 -0
  7. package/src/gdskills/bundled/rules/core/database-patterns.mdc +1 -0
  8. package/src/gdskills/bundled/rules/core/definition-of-done.mdc +116 -0
  9. package/src/gdskills/bundled/rules/core/documentation-management.mdc +33 -38
  10. package/src/gdskills/bundled/rules/core/error-handling.mdc +1 -11
  11. package/src/gdskills/bundled/rules/core/execution-metrics.md +1 -2
  12. package/src/gdskills/bundled/rules/core/frontend-assistant.mdc +1 -0
  13. package/src/gdskills/bundled/rules/core/git-concurrency.mdc +101 -0
  14. package/src/gdskills/bundled/rules/core/implementation-plans.mdc +23 -11
  15. package/src/gdskills/bundled/rules/core/mobx-store-template.mdc +1 -0
  16. package/src/gdskills/bundled/rules/core/nestjs-dto.mdc +1 -0
  17. package/src/gdskills/bundled/rules/core/playwright-testing.mdc +1 -0
  18. package/src/gdskills/bundled/rules/core/requirements-management.mdc +15 -11
  19. package/src/gdskills/bundled/rules/core/rule-management-workflow.mdc +29 -14
  20. package/src/gdskills/bundled/rules/core/shared-definitions.mdc +1 -1
  21. package/src/gdskills/bundled/rules/core/skill-lifecycle.mdc +9 -5
  22. package/src/gdskills/bundled/rules/core/skills-storage-workflow.mdc +156 -23
  23. package/src/gdskills/bundled/rules/core/storybook-guidelines.mdc +1 -0
  24. package/src/gdskills/bundled/rules/core/subagent-status-protocol.md +9 -2
  25. package/src/gdskills/bundled/skills/core/reviewer-skill-creator/SKILL.md +42 -5
  26. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.md +67 -74
  27. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.md +24 -8
  28. package/src/gdskills/bundled/skills/orchestration/context-collector/orchestrator-prompt.md +2 -2
  29. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.detail.md +12 -22
  30. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +44 -31
  31. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.md +2 -2
  32. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.template.md +1 -1
  33. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/input-contract.schema.json +4 -4
  34. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/orchestrator-prompt.md +2 -2
  35. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +20 -6
  36. package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md +67 -9
  37. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.md +6 -6
  38. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/orchestrator-prompt.md +1 -1
  39. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.md +45 -5
  40. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +88 -32
  41. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +52 -41
  42. package/src/gdskills/bundled/skills/orchestration/task-implementer/output-contract.schema.json +32 -1
  43. package/src/gdskills/bundled/skills/planning/autodoc-analyst/SKILL.md +16 -0
  44. package/src/gdskills/bundled/skills/planning/autodoc-architect/SKILL.md +16 -0
  45. package/src/gdskills/bundled/skills/planning/autodoc-assembler/SKILL.md +16 -0
  46. package/src/gdskills/bundled/skills/planning/autodoc-orchestrator/SKILL.md +17 -0
  47. package/src/gdskills/bundled/skills/planning/autodoc-scanner/SKILL.md +16 -0
  48. package/src/gdskills/bundled/skills/planning/autodoc-writer/SKILL.md +16 -0
  49. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.md +29 -4
  50. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.codex.md +17 -0
  51. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.cursor.md +17 -0
  52. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.md +17 -0
  53. package/src/gdskills/bundled/skills/planning/docpack-orchestrator/SKILL.md +32 -2
  54. package/src/gdskills/bundled/skills/planning/docpack-review/SKILL.md +14 -2
  55. package/src/gdskills/bundled/skills/planning/interview/SKILL.md +30 -8
  56. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +33 -7
  57. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.codex.md +16 -0
  58. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.cursor.md +16 -0
  59. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.md +16 -0
  60. package/src/gdskills/bundled/skills/planning/planner/SKILL.codex.md +17 -0
  61. package/src/gdskills/bundled/skills/planning/planner/SKILL.cursor.md +17 -0
  62. package/src/gdskills/bundled/skills/planning/planner/SKILL.md +17 -0
  63. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.md +27 -10
  64. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.codex.md +16 -0
  65. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.cursor.md +16 -0
  66. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.md +16 -0
  67. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.codex.md +16 -0
  68. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.cursor.md +16 -0
  69. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.md +16 -0
  70. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.codex.md +4 -0
  71. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.cursor.md +4 -0
  72. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.md +4 -0
  73. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.codex.md +4 -0
  74. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.cursor.md +4 -0
  75. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.md +4 -0
  76. package/src/gdskills/bundled/skills/platform/agent-entrypoint-distiller/SKILL.md +31 -4
  77. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.md +27 -3
  78. package/src/gdskills/bundled/skills/platform/hookify/SKILL.md +29 -4
  79. package/src/gdskills/bundled/skills/quality/api-truth/SKILL.md +226 -0
  80. package/src/gdskills/bundled/skills/quality/changelog/SKILL.md +25 -5
  81. package/src/gdskills/bundled/skills/quality/commit/SKILL.md +26 -5
  82. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.md +25 -4
  83. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.md +26 -5
  84. package/src/gdskills/bundled/skills/quality/deploy/SKILL.md +27 -4
  85. package/src/gdskills/bundled/skills/quality/deprecation-path/SKILL.md +268 -0
  86. package/src/gdskills/bundled/skills/quality/fresh-eyes/SKILL.md +190 -0
  87. package/src/gdskills/bundled/skills/quality/metaproject-security/SKILL.md +24 -3
  88. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.md +30 -9
  89. package/src/gdskills/bundled/skills/quality/pr/SKILL.md +25 -5
  90. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.md +27 -4
  91. package/src/gdskills/bundled/skills/quality/push/SKILL.md +25 -4
  92. package/src/gdskills/bundled/skills/quality/root-cause/SKILL.md +204 -0
  93. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.md +25 -4
  94. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.md +31 -5
  95. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.md +32 -11
  96. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.md +42 -7
  97. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.md +43 -3
  98. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.md +46 -4
  99. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.md +46 -6
  100. package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +5 -5
  101. package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +5 -6
  102. package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +6 -6
  103. package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +37 -3
  104. package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +38 -4
  105. package/src/gdskills/bundled/skills/review/review-frontend/SKILL.md +4 -6
  106. package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +37 -3
  107. package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +5 -7
  108. package/src/gdskills/bundled/skills/review/review-layout/SKILL.md +24 -3
  109. package/src/gdskills/bundled/skills/review/review-logic/SKILL.md +5 -5
  110. package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +49 -64
  111. package/src/gdskills/bundled/skills/review/review-orchestrator/input-contract.schema.json +1 -2
  112. package/src/gdskills/bundled/skills/review/review-orchestrator/review-context.schema.json +1 -5
  113. package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-input.schema.json +53 -9
  114. package/src/gdskills/bundled/skills/review/review-performance/SKILL.md +11 -11
  115. package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +9 -8
  116. package/src/gdskills/bundled/skills/review/review-regression/SKILL.md +33 -2
  117. package/src/gdskills/bundled/skills/review/review-security-code/SKILL.md +6 -4
  118. package/src/gdskills/bundled/skills/review/review-style/SKILL.md +5 -5
  119. package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +41 -3
  120. package/src/gdskills/bundled/skills/review/review-verifier/SKILL.md +2 -2
  121. package/src/gdskills/bundled/rules/core/review-agent-profile.mdc +0 -49
  122. package/src/gdskills/bundled/rules/core/review-strict-profile.mdc +0 -48
  123. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.codex.md +0 -353
  124. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.cursor.md +0 -353
  125. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.opencode.md +0 -353
  126. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.zed.md +0 -353
  127. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.codex.md +0 -655
  128. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.cursor.md +0 -655
  129. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.opencode.md +0 -655
  130. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.zed.md +0 -655
  131. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +0 -434
  132. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +0 -434
  133. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +0 -434
  134. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +0 -434
  135. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +0 -163
  136. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +0 -163
  137. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.codex.md +0 -373
  138. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.cursor.md +0 -373
  139. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.opencode.md +0 -373
  140. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.zed.md +0 -373
  141. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.codex.md +0 -374
  142. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.cursor.md +0 -374
  143. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.opencode.md +0 -374
  144. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.zed.md +0 -374
  145. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +0 -2190
  146. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +0 -2190
  147. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +0 -2190
  148. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +0 -2190
  149. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +0 -659
  150. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +0 -659
  151. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +0 -659
  152. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +0 -659
  153. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.codex.md +0 -90
  154. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.cursor.md +0 -90
  155. package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +0 -187
  156. package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +0 -187
  157. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +0 -105
  158. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +0 -105
  159. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.codex.md +0 -193
  160. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.cursor.md +0 -193
  161. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.opencode.md +0 -193
  162. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.zed.md +0 -193
  163. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.codex.md +0 -87
  164. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.cursor.md +0 -87
  165. package/src/gdskills/bundled/skills/platform/hookify/SKILL.codex.md +0 -100
  166. package/src/gdskills/bundled/skills/platform/hookify/SKILL.cursor.md +0 -100
  167. package/src/gdskills/bundled/skills/quality/changelog/SKILL.codex.md +0 -84
  168. package/src/gdskills/bundled/skills/quality/changelog/SKILL.cursor.md +0 -84
  169. package/src/gdskills/bundled/skills/quality/commit/SKILL.codex.md +0 -66
  170. package/src/gdskills/bundled/skills/quality/commit/SKILL.cursor.md +0 -66
  171. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.codex.md +0 -66
  172. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.cursor.md +0 -66
  173. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.codex.md +0 -81
  174. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.cursor.md +0 -81
  175. package/src/gdskills/bundled/skills/quality/deploy/SKILL.codex.md +0 -70
  176. package/src/gdskills/bundled/skills/quality/deploy/SKILL.cursor.md +0 -70
  177. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.codex.md +0 -83
  178. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.cursor.md +0 -83
  179. package/src/gdskills/bundled/skills/quality/pr/SKILL.codex.md +0 -75
  180. package/src/gdskills/bundled/skills/quality/pr/SKILL.cursor.md +0 -75
  181. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.codex.md +0 -378
  182. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.cursor.md +0 -378
  183. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.opencode.md +0 -378
  184. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.zed.md +0 -378
  185. package/src/gdskills/bundled/skills/quality/push/SKILL.codex.md +0 -52
  186. package/src/gdskills/bundled/skills/quality/push/SKILL.cursor.md +0 -52
  187. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.codex.md +0 -108
  188. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.cursor.md +0 -108
  189. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.codex.md +0 -75
  190. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.cursor.md +0 -75
  191. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.codex.md +0 -339
  192. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.cursor.md +0 -339
  193. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.opencode.md +0 -339
  194. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.zed.md +0 -339
  195. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.codex.md +0 -203
  196. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.cursor.md +0 -203
  197. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.opencode.md +0 -203
  198. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.zed.md +0 -203
  199. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.codex.md +0 -243
  200. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.cursor.md +0 -243
  201. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.opencode.md +0 -243
  202. package/src/gdskills/bundled/skills/review/code-learned-review/SKILL.zed.md +0 -243
  203. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.codex.md +0 -259
  204. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.cursor.md +0 -259
  205. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.opencode.md +0 -259
  206. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.zed.md +0 -259
  207. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.codex.md +0 -168
  208. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.cursor.md +0 -168
  209. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.opencode.md +0 -168
  210. 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"
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: `skills/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"
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: `skills/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"
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: `skills/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"
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: `skills/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.