@mrciphersmith/keryx 0.2.9

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 (297) hide show
  1. package/LICENSE +21 -0
  2. package/README.md +351 -0
  3. package/dist/cli.js +48427 -0
  4. package/dist/proxy-worker.js +381 -0
  5. package/package.json +63 -0
  6. package/src/gdgraph/affected.test.ts +100 -0
  7. package/src/gdgraph/affected.ts +135 -0
  8. package/src/gdgraph/build-integrity.test.ts +193 -0
  9. package/src/gdgraph/build-lang.test.ts +399 -0
  10. package/src/gdgraph/build.test.ts +120 -0
  11. package/src/gdgraph/build.ts +804 -0
  12. package/src/gdgraph/config.test.ts +47 -0
  13. package/src/gdgraph/config.ts +113 -0
  14. package/src/gdgraph/core-sources.test.ts +99 -0
  15. package/src/gdgraph/core-sources.ts +20 -0
  16. package/src/gdgraph/enrich.ts +79 -0
  17. package/src/gdgraph/fallback.test.ts +153 -0
  18. package/src/gdgraph/find.test.ts +78 -0
  19. package/src/gdgraph/find.ts +107 -0
  20. package/src/gdgraph/pagerank.ts +129 -0
  21. package/src/gdgraph/path.test.ts +56 -0
  22. package/src/gdgraph/path.ts +112 -0
  23. package/src/gdgraph/query.ts +133 -0
  24. package/src/gdgraph/repomap.test.ts +110 -0
  25. package/src/gdgraph/repomap.ts +193 -0
  26. package/src/gdgraph/service.test.ts +59 -0
  27. package/src/gdgraph/service.ts +51 -0
  28. package/src/gdgraph/staleness.ts +20 -0
  29. package/src/gdgraph/symbol.test.ts +89 -0
  30. package/src/gdgraph/symbol.ts +123 -0
  31. package/src/gdgraph/symbols-capability.test.ts +41 -0
  32. package/src/gdgraph/symbols-capability.ts +49 -0
  33. package/src/gdgraph/target.ts +56 -0
  34. package/src/gdgraph/treesitter/adapter.test.ts +249 -0
  35. package/src/gdgraph/treesitter/adapter.ts +213 -0
  36. package/src/gdgraph/treesitter/extract.test.ts +278 -0
  37. package/src/gdgraph/treesitter/extract.ts +389 -0
  38. package/src/gdgraph/treesitter/grammars.ts +110 -0
  39. package/src/gdgraph/treesitter/no-treesitter-import.test.ts +51 -0
  40. package/src/gdgraph/treesitter/resolve-calls.test.ts +38 -0
  41. package/src/gdgraph/types.ts +63 -0
  42. package/src/gdskills/bundled/rules/core/api-contracts.mdc +148 -0
  43. package/src/gdskills/bundled/rules/core/async-patterns.mdc +185 -0
  44. package/src/gdskills/bundled/rules/core/clean-architecture.mdc +163 -0
  45. package/src/gdskills/bundled/rules/core/code-review-ai-assistant.mdc +49 -0
  46. package/src/gdskills/bundled/rules/core/code-review-b091-profile.mdc +48 -0
  47. package/src/gdskills/bundled/rules/core/code-style-patterns.mdc +54 -0
  48. package/src/gdskills/bundled/rules/core/commit-message-formatting.mdc +25 -0
  49. package/src/gdskills/bundled/rules/core/database-patterns.mdc +162 -0
  50. package/src/gdskills/bundled/rules/core/documentation-management.mdc +106 -0
  51. package/src/gdskills/bundled/rules/core/error-handling.mdc +163 -0
  52. package/src/gdskills/bundled/rules/core/execution-metrics.md +99 -0
  53. package/src/gdskills/bundled/rules/core/frontend-assistant.mdc +40 -0
  54. package/src/gdskills/bundled/rules/core/git-rules.mdc +37 -0
  55. package/src/gdskills/bundled/rules/core/gproject-contracts.mdc +198 -0
  56. package/src/gdskills/bundled/rules/core/implementation-doc-mandate.mdc +149 -0
  57. package/src/gdskills/bundled/rules/core/implementation-plans.mdc +58 -0
  58. package/src/gdskills/bundled/rules/core/jobs-documentation.mdc +221 -0
  59. package/src/gdskills/bundled/rules/core/mobx-store-template.mdc +561 -0
  60. package/src/gdskills/bundled/rules/core/model-selection.mdc +53 -0
  61. package/src/gdskills/bundled/rules/core/nestjs-dto.mdc +50 -0
  62. package/src/gdskills/bundled/rules/core/playwright-testing.mdc +321 -0
  63. package/src/gdskills/bundled/rules/core/requirements-management.mdc +35 -0
  64. package/src/gdskills/bundled/rules/core/requirements-package-standard.mdc +103 -0
  65. package/src/gdskills/bundled/rules/core/review-agent-profile.mdc +49 -0
  66. package/src/gdskills/bundled/rules/core/review-strict-profile.mdc +44 -0
  67. package/src/gdskills/bundled/rules/core/rule-management-workflow.mdc +62 -0
  68. package/src/gdskills/bundled/rules/core/security-baseline.mdc +155 -0
  69. package/src/gdskills/bundled/rules/core/shared-definitions.mdc +79 -0
  70. package/src/gdskills/bundled/rules/core/skill-lifecycle.mdc +55 -0
  71. package/src/gdskills/bundled/rules/core/skills-storage-workflow.mdc +88 -0
  72. package/src/gdskills/bundled/rules/core/solid-principles.mdc +149 -0
  73. package/src/gdskills/bundled/rules/core/storybook-guidelines.mdc +36 -0
  74. package/src/gdskills/bundled/rules/core/subagent-context-construction.md +107 -0
  75. package/src/gdskills/bundled/rules/core/subagent-status-protocol.md +169 -0
  76. package/src/gdskills/bundled/rules/core/tdd-workflow.mdc +96 -0
  77. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.codex.md +47 -0
  78. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.cursor.md +55 -0
  79. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.md +352 -0
  80. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.opencode.md +39 -0
  81. package/src/gdskills/bundled/skills/orchestration/code-verifier/SKILL.zed.md +29 -0
  82. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.codex.md +567 -0
  83. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.cursor.md +567 -0
  84. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.md +655 -0
  85. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.opencode.md +567 -0
  86. package/src/gdskills/bundled/skills/orchestration/context-collector/SKILL.zed.md +567 -0
  87. package/src/gdskills/bundled/skills/orchestration/context-collector/input-contract.schema.json +134 -0
  88. package/src/gdskills/bundled/skills/orchestration/context-collector/orchestrator-prompt.md +183 -0
  89. package/src/gdskills/bundled/skills/orchestration/context-collector/output-contract.schema.json +208 -0
  90. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.codex.md +1412 -0
  91. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.cursor.md +1412 -0
  92. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.detail.md +735 -0
  93. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.md +468 -0
  94. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.opencode.md +1412 -0
  95. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/SKILL.zed.md +1412 -0
  96. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.md +62 -0
  97. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/analysis-request.template.md +79 -0
  98. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/input-contract.schema.json +247 -0
  99. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/orchestrator-prompt.md +362 -0
  100. package/src/gdskills/bundled/skills/orchestration/feature-analyzer/output-contract.schema.json +87 -0
  101. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.codex.md +105 -0
  102. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.cursor.md +105 -0
  103. package/src/gdskills/bundled/skills/orchestration/feature-dev/SKILL.md +165 -0
  104. package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/SKILL.md +379 -0
  105. package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/input-contract.schema.json +33 -0
  106. package/src/gdskills/bundled/skills/orchestration/flow-orchestrator/output-contract.schema.json +41 -0
  107. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.codex.md +307 -0
  108. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.cursor.md +307 -0
  109. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.md +373 -0
  110. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.opencode.md +307 -0
  111. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/SKILL.zed.md +307 -0
  112. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/input-contract.schema.json +130 -0
  113. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/issue-request.template.md +50 -0
  114. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/orchestrator-prompt.md +155 -0
  115. package/src/gdskills/bundled/skills/orchestration/issue-analyzer/output-contract.schema.json +61 -0
  116. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.codex.md +365 -0
  117. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.cursor.md +365 -0
  118. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.md +373 -0
  119. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.opencode.md +365 -0
  120. package/src/gdskills/bundled/skills/orchestration/job-documenter/SKILL.zed.md +365 -0
  121. package/src/gdskills/bundled/skills/orchestration/job-documenter/input-contract.schema.json +55 -0
  122. package/src/gdskills/bundled/skills/orchestration/job-documenter/output-contract.schema.json +58 -0
  123. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.codex.md +1701 -0
  124. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.cursor.md +1701 -0
  125. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.md +1734 -0
  126. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.opencode.md +1701 -0
  127. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/SKILL.zed.md +1701 -0
  128. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/input-contract.schema.json +188 -0
  129. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/orchestrator-prompt.md +179 -0
  130. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/output-contract.schema.json +103 -0
  131. package/src/gdskills/bundled/skills/orchestration/job-orchestrator/state.schema.json +164 -0
  132. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.codex.md +395 -0
  133. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.cursor.md +395 -0
  134. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.md +556 -0
  135. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.opencode.md +395 -0
  136. package/src/gdskills/bundled/skills/orchestration/task-implementer/SKILL.zed.md +395 -0
  137. package/src/gdskills/bundled/skills/orchestration/task-implementer/input-contract.schema.json +204 -0
  138. package/src/gdskills/bundled/skills/orchestration/task-implementer/orchestrator-prompt.md +206 -0
  139. package/src/gdskills/bundled/skills/orchestration/task-implementer/output-contract.schema.json +51 -0
  140. package/src/gdskills/bundled/skills/orchestration/task-implementer/task-request.template.md +111 -0
  141. package/src/gdskills/bundled/skills/planning/autodoc-analyst/SKILL.md +163 -0
  142. package/src/gdskills/bundled/skills/planning/autodoc-architect/SKILL.md +161 -0
  143. package/src/gdskills/bundled/skills/planning/autodoc-assembler/SKILL.md +128 -0
  144. package/src/gdskills/bundled/skills/planning/autodoc-orchestrator/SKILL.md +321 -0
  145. package/src/gdskills/bundled/skills/planning/autodoc-scanner/SKILL.md +166 -0
  146. package/src/gdskills/bundled/skills/planning/autodoc-writer/SKILL.md +253 -0
  147. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.claude.md +46 -0
  148. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.codex.md +90 -0
  149. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.cursor.md +90 -0
  150. package/src/gdskills/bundled/skills/planning/brainstorm/SKILL.md +90 -0
  151. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.codex.md +33 -0
  152. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.cursor.md +33 -0
  153. package/src/gdskills/bundled/skills/planning/consistency-checker/SKILL.md +190 -0
  154. package/src/gdskills/bundled/skills/planning/docpack-orchestrator/SKILL.md +142 -0
  155. package/src/gdskills/bundled/skills/planning/docpack-orchestrator/input-contract.schema.json +31 -0
  156. package/src/gdskills/bundled/skills/planning/docpack-orchestrator/output-contract.schema.json +31 -0
  157. package/src/gdskills/bundled/skills/planning/docpack-review/SKILL.md +80 -0
  158. package/src/gdskills/bundled/skills/planning/docpack-review/input-contract.schema.json +20 -0
  159. package/src/gdskills/bundled/skills/planning/docpack-review/output-contract.schema.json +36 -0
  160. package/src/gdskills/bundled/skills/planning/interview/SKILL.codex.md +181 -0
  161. package/src/gdskills/bundled/skills/planning/interview/SKILL.cursor.md +181 -0
  162. package/src/gdskills/bundled/skills/planning/interview/SKILL.md +187 -0
  163. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.claude.md +94 -0
  164. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.codex.md +99 -0
  165. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.cursor.md +99 -0
  166. package/src/gdskills/bundled/skills/planning/interviewer/SKILL.md +105 -0
  167. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.codex.md +33 -0
  168. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.cursor.md +33 -0
  169. package/src/gdskills/bundled/skills/planning/patterns-researcher/SKILL.md +237 -0
  170. package/src/gdskills/bundled/skills/planning/planner/SKILL.codex.md +33 -0
  171. package/src/gdskills/bundled/skills/planning/planner/SKILL.cursor.md +33 -0
  172. package/src/gdskills/bundled/skills/planning/planner/SKILL.md +190 -0
  173. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.codex.md +193 -0
  174. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.cursor.md +193 -0
  175. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.md +193 -0
  176. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.opencode.md +193 -0
  177. package/src/gdskills/bundled/skills/planning/prd-creator/SKILL.zed.md +193 -0
  178. package/src/gdskills/bundled/skills/planning/prd-creator/input-contract.schema.json +63 -0
  179. package/src/gdskills/bundled/skills/planning/prd-creator/output-contract.schema.json +175 -0
  180. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.codex.md +31 -0
  181. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.cursor.md +31 -0
  182. package/src/gdskills/bundled/skills/planning/problem-definer/SKILL.md +153 -0
  183. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.codex.md +32 -0
  184. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.cursor.md +32 -0
  185. package/src/gdskills/bundled/skills/planning/project-discovery/SKILL.md +166 -0
  186. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.codex.md +44 -0
  187. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.cursor.md +44 -0
  188. package/src/gdskills/bundled/skills/planning/spec-writer/SKILL.md +243 -0
  189. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.codex.md +43 -0
  190. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.cursor.md +43 -0
  191. package/src/gdskills/bundled/skills/planning/stack-advisor/SKILL.md +193 -0
  192. package/src/gdskills/bundled/skills/platform/agent-entrypoint-distiller/SKILL.md +47 -0
  193. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.codex.md +87 -0
  194. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.cursor.md +87 -0
  195. package/src/gdskills/bundled/skills/platform/claude-md-management/SKILL.md +87 -0
  196. package/src/gdskills/bundled/skills/platform/hookify/SKILL.codex.md +100 -0
  197. package/src/gdskills/bundled/skills/platform/hookify/SKILL.cursor.md +100 -0
  198. package/src/gdskills/bundled/skills/platform/hookify/SKILL.md +100 -0
  199. package/src/gdskills/bundled/skills/quality/changelog/SKILL.claude.md +45 -0
  200. package/src/gdskills/bundled/skills/quality/changelog/SKILL.codex.md +84 -0
  201. package/src/gdskills/bundled/skills/quality/changelog/SKILL.cursor.md +84 -0
  202. package/src/gdskills/bundled/skills/quality/changelog/SKILL.md +84 -0
  203. package/src/gdskills/bundled/skills/quality/commit/SKILL.codex.md +66 -0
  204. package/src/gdskills/bundled/skills/quality/commit/SKILL.cursor.md +66 -0
  205. package/src/gdskills/bundled/skills/quality/commit/SKILL.md +66 -0
  206. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.claude.md +40 -0
  207. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.codex.md +66 -0
  208. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.cursor.md +66 -0
  209. package/src/gdskills/bundled/skills/quality/db-migrate/SKILL.md +66 -0
  210. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.claude.md +45 -0
  211. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.codex.md +81 -0
  212. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.cursor.md +81 -0
  213. package/src/gdskills/bundled/skills/quality/dependency-update/SKILL.md +81 -0
  214. package/src/gdskills/bundled/skills/quality/deploy/SKILL.claude.md +42 -0
  215. package/src/gdskills/bundled/skills/quality/deploy/SKILL.codex.md +70 -0
  216. package/src/gdskills/bundled/skills/quality/deploy/SKILL.cursor.md +70 -0
  217. package/src/gdskills/bundled/skills/quality/deploy/SKILL.md +70 -0
  218. package/src/gdskills/bundled/skills/quality/metaproject-security/SKILL.md +103 -0
  219. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.claude.md +48 -0
  220. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.codex.md +83 -0
  221. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.cursor.md +83 -0
  222. package/src/gdskills/bundled/skills/quality/perf-check/SKILL.md +83 -0
  223. package/src/gdskills/bundled/skills/quality/pr/SKILL.codex.md +75 -0
  224. package/src/gdskills/bundled/skills/quality/pr/SKILL.cursor.md +75 -0
  225. package/src/gdskills/bundled/skills/quality/pr/SKILL.md +75 -0
  226. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.codex.md +364 -0
  227. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.cursor.md +295 -0
  228. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.md +378 -0
  229. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.opencode.md +364 -0
  230. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/SKILL.zed.md +38 -0
  231. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/input-contract.schema.json +53 -0
  232. package/src/gdskills/bundled/skills/quality/pr-issue-documenter/output-contract.schema.json +46 -0
  233. package/src/gdskills/bundled/skills/quality/push/SKILL.codex.md +52 -0
  234. package/src/gdskills/bundled/skills/quality/push/SKILL.cursor.md +52 -0
  235. package/src/gdskills/bundled/skills/quality/push/SKILL.md +52 -0
  236. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.claude.md +40 -0
  237. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.codex.md +61 -0
  238. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.cursor.md +61 -0
  239. package/src/gdskills/bundled/skills/quality/security-audit/SKILL.md +61 -0
  240. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.claude.md +30 -0
  241. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.codex.md +75 -0
  242. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.cursor.md +75 -0
  243. package/src/gdskills/bundled/skills/quality/test-gen/SKILL.md +75 -0
  244. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.codex.md +64 -0
  245. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.cursor.md +74 -0
  246. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.md +339 -0
  247. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.opencode.md +48 -0
  248. package/src/gdskills/bundled/skills/quality/tests-creator/SKILL.zed.md +44 -0
  249. package/src/gdskills/bundled/skills/quality/tests-creator/input-contract.schema.json +121 -0
  250. package/src/gdskills/bundled/skills/quality/tests-creator/output-contract.schema.json +127 -0
  251. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.codex.md +204 -0
  252. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.cursor.md +204 -0
  253. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.md +203 -0
  254. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.opencode.md +204 -0
  255. package/src/gdskills/bundled/skills/review/code-ai-review/SKILL.zed.md +204 -0
  256. package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.codex.md +209 -0
  257. package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.cursor.md +209 -0
  258. package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.md +208 -0
  259. package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.opencode.md +209 -0
  260. package/src/gdskills/bundled/skills/review/code-b091-review/SKILL.zed.md +209 -0
  261. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.codex.md +263 -0
  262. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.cursor.md +263 -0
  263. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.md +258 -0
  264. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.opencode.md +263 -0
  265. package/src/gdskills/bundled/skills/review/code-mobx-store-review/SKILL.zed.md +263 -0
  266. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.codex.md +175 -0
  267. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.cursor.md +175 -0
  268. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.md +168 -0
  269. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.opencode.md +175 -0
  270. package/src/gdskills/bundled/skills/review/code-style-review/SKILL.zed.md +175 -0
  271. package/src/gdskills/bundled/skills/review/review-architecture/SKILL.md +360 -0
  272. package/src/gdskills/bundled/skills/review/review-backend/SKILL.md +337 -0
  273. package/src/gdskills/bundled/skills/review/review-clean-code/SKILL.md +477 -0
  274. package/src/gdskills/bundled/skills/review/review-core-boundaries/SKILL.md +93 -0
  275. package/src/gdskills/bundled/skills/review/review-flow-graph/SKILL.md +119 -0
  276. package/src/gdskills/bundled/skills/review/review-frontend/SKILL.md +595 -0
  277. package/src/gdskills/bundled/skills/review/review-frontend-conventions/SKILL.md +144 -0
  278. package/src/gdskills/bundled/skills/review/review-highload/SKILL.md +519 -0
  279. package/src/gdskills/bundled/skills/review/review-logic/SKILL.md +323 -0
  280. package/src/gdskills/bundled/skills/review/review-orchestrator/SKILL.md +927 -0
  281. package/src/gdskills/bundled/skills/review/review-orchestrator/input-contract.schema.json +153 -0
  282. package/src/gdskills/bundled/skills/review/review-orchestrator/output-contract.schema.json +77 -0
  283. package/src/gdskills/bundled/skills/review/review-orchestrator/review-context.schema.json +389 -0
  284. package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-finding.schema.json +189 -0
  285. package/src/gdskills/bundled/skills/review/review-orchestrator/reviewer-input.schema.json +241 -0
  286. package/src/gdskills/bundled/skills/review/review-performance/SKILL.md +344 -0
  287. package/src/gdskills/bundled/skills/review/review-pr-feedback/SKILL.md +327 -0
  288. package/src/gdskills/bundled/skills/review/review-security-code/SKILL.md +348 -0
  289. package/src/gdskills/bundled/skills/review/review-strict/SKILL.md +328 -0
  290. package/src/gdskills/bundled/skills/review/review-style/SKILL.md +343 -0
  291. package/src/gdskills/bundled/skills/review/review-testing-practices/SKILL.md +134 -0
  292. package/src/gdskills/bundled/skills/shared/git-merge-base.md +33 -0
  293. package/src/gdskills/contracts/agent-event.schema.json +32 -0
  294. package/src/gdskills/contracts/orchestrator-state.schema.json +58 -0
  295. package/src/gdskills/contracts/review-finding.schema.json +59 -0
  296. package/src/gdskills/contracts/subagent-dispatch.schema.json +123 -0
  297. package/src/gdskills/contracts/subagent-result.schema.json +136 -0
@@ -0,0 +1,209 @@
1
+ ---
2
+ name: code-b091-review
3
+ description: "Performs strict b091-style code review following code-review-b091-profile.mdc. Reviews current branch changes from merge-base. Direct, no-fluff feedback focused on logic correctness. Use when: b091 review requested, strict validation needed."
4
+ triggers:
5
+ - "Review as b091"
6
+ - "b091 style review"
7
+ - "b091 review"
8
+ metadata:
9
+ author: "MrCipherSmith"
10
+ version: "1.0.0"
11
+ category: "review"
12
+ license: "MIT"
13
+ compatibility: "cursor,codex,zed,opencode"
14
+ ---
15
+
16
+
17
+ # Code Review as b091 (только текущая ветка)
18
+
19
+ ## Workflow
20
+
21
+ Copy this checklist and track progress:
22
+
23
+ ```
24
+ b091 Review Progress:
25
+ - [ ] Step 1: Determine parent branch and calculate merge-base
26
+ - [ ] Step 2: Collect git diff (committed + local changes)
27
+ - [ ] Step 3: Apply b091 principles (logic in correct layer, no ducttape)
28
+ - [ ] Step 4: Check types (no any, typed mocks)
29
+ - [ ] Step 5: Verify conventions (currentState, I prefix, etc.)
30
+ - [ ] Step 6: Challenge all assumptions
31
+ - [ ] Step 7: Generate direct report with patches
32
+ ```
33
+
34
+ ## Обязательные правила
35
+
36
+ 1) **Скоуп по умолчанию (без commit hash/range в запросе)**: ревью включает **все** изменения в текущей ветке от точки ответвления (merge-base) от родительской ветки:
37
+ - закоммиченные (`BASE_SHA..HEAD`)
38
+ - локальные незакоммиченные (staged/unstaged/untracked)
39
+ 2) **Скоуп при явном commit hash/range**: если пользователь явно передал hash или диапазон, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
40
+ 3) **Стиль/принципы b091**: ревью строго по `.metaproject/rules/core/code-review-b091-profile.mdc` (разделы правил и промпт).
41
+ 4) **Результат**: подробный отчёт по проблемам с объяснением и предложением исправлений (минимальные патчи там, где это просто).
42
+
43
+ ## Определение parent ref (детерминированно, именно родитель ветки)
44
+
45
+ Используй первый существующий вариант:
46
+
47
+ 1. `origin/main`
48
+ 2. `origin/master`
49
+ 3. `main`
50
+ 4. `master`
51
+ 5. `@{upstream}` **только если это не текущая feature-ветка**
52
+
53
+ ```bash
54
+ BRANCH="$(git rev-parse --abbrev-ref HEAD)"
55
+ UPSTREAM_REF="$(git rev-parse --abbrev-ref --symbolic-full-name @{upstream} 2>/dev/null || true)"
56
+
57
+ PARENT=""
58
+ if git rev-parse --verify -q "origin/main" >/dev/null; then
59
+ PARENT="origin/main"
60
+ elif git rev-parse --verify -q "origin/master" >/dev/null; then
61
+ PARENT="origin/master"
62
+ elif git rev-parse --verify -q "main" >/dev/null; then
63
+ PARENT="main"
64
+ elif git rev-parse --verify -q "master" >/dev/null; then
65
+ PARENT="master"
66
+ elif [ -n "$UPSTREAM_REF" ] && [ "$UPSTREAM_REF" != "$BRANCH" ] && [ "$UPSTREAM_REF" != "origin/$BRANCH" ]; then
67
+ PARENT="@{upstream}"
68
+ else
69
+ echo "Cannot determine parent ref" >&2
70
+ exit 1
71
+ fi
72
+
73
+ BASE_SHA="$(git merge-base HEAD "$PARENT")"
74
+ ```
75
+
76
+ ## Команды, чтобы собрать “срез” для ревью
77
+
78
+ ### A) Режим по умолчанию (пользователь НЕ дал hash/range)
79
+
80
+ ```bash
81
+ git status
82
+
83
+ # Закоммиченная часть ветки
84
+ git log --oneline "${BASE_SHA}..HEAD"
85
+ git diff --stat "${BASE_SHA}..HEAD"
86
+ git diff --name-status "${BASE_SHA}..HEAD"
87
+ git diff "${BASE_SHA}..HEAD"
88
+
89
+ # Полный текущий срез от merge-base до рабочего дерева:
90
+ # включает commit'ы + staged + unstaged
91
+ # (untracked смотри через git status / git ls-files)
92
+ git diff --stat "${BASE_SHA}"
93
+ git diff --name-status "${BASE_SHA}"
94
+ git diff "${BASE_SHA}"
95
+ git ls-files --others --exclude-standard
96
+ ```
97
+
98
+ ### B) Режим с явным hash/range
99
+
100
+ Если пользователь дал конкретный hash/range, используй его буквально:
101
+
102
+ ```bash
103
+ # Примеры:
104
+ git show --stat --name-status --patch <COMMIT_SHA>
105
+ git log --oneline <FROM_SHA>..<TO_SHA>
106
+ git diff --stat <FROM_SHA>..<TO_SHA>
107
+ git diff --name-status <FROM_SHA>..<TO_SHA>
108
+ git diff <FROM_SHA>..<TO_SHA>
109
+ ```
110
+
111
+ ## Как ревьюить (по b091)
112
+
113
+ Применяй чек-лист и промпт из `.metaproject/rules/core/code-review-b091-profile.mdc`. Ключевые акценты b091:
114
+
115
+ - Логика должна жить в правильном слое (часто — в store), чтобы её можно было нормально тестировать.
116
+ - Против “patch/glue/ducttape”: требуй чинить причину, а не маскировать симптомы.
117
+ - В новом коде никаких `any`, `as any`, небезопасных кастов; моки — типизированные и не наследуются от реальных реализаций.
118
+ - Не плодить дублирование правил/проверок: один источник правды, переиспользовать сеттеры/методы.
119
+ - Конвенции проекта обязательны (например, `currentState`, `I` для интерфейсов, консистентные типы).
120
+ - Магические числа / `setTimeout(0)` / "страшные" хаки — только с чётким "почему", либо перепроектировать.
121
+ - **Accessibility модификаторы в сторах**: inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный метод допустим только если он вызывается из React-компонента. Спросить: "Вызывается ли этот метод из JSX?" Если нет — `private`.
122
+ - С MobX по умолчанию **не добавлять** `useCallback`/`useMemo` без явной необходимости.
123
+ - Предложения должны проходить линт; не нитпикать автоформатирование.
124
+ - Не раздувать scope: большие рефакторы “not today”, если вне задачи.
125
+
126
+ ## Формат вывода (подробный отчёт + тон b091)
127
+
128
+ Пиши по-русски, но допускай короткие b091-маркеры (например, `not today ;P`, “ducttape”, “broken thinking”) только по делу и без перехода на личности.
129
+
130
+ Используй структуру:
131
+
132
+ ```markdown
133
+ ## Вердикт (b091)
134
+ <OK / needs work / fundamentally wrong> + 1–3 самых важных тезиса.
135
+
136
+ ## Скоуп ревью (только текущая ветка)
137
+ - Ветка: `<BRANCH>`
138
+ - Parent ref: `<PARENT>`
139
+ - Merge-base: `<BASE_SHA>`
140
+ - Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
141
+ - Коммиты (merge-base..HEAD): <N>
142
+ - Изменённые файлы: <список или количество>
143
+
144
+ ## Находки (подробно)
145
+ ### Архитектура и место логики
146
+ <находки>
147
+
148
+ ### Типы, касты, моки
149
+ <находки>
150
+
151
+ ### Корректность и семантика состояний
152
+ <находки>
153
+
154
+ ### Конвенции и консистентность
155
+ <находки>
156
+
157
+ ### Линт/формат/шум
158
+ <находки>
159
+
160
+ ### UX/edge cases
161
+ <находки>
162
+
163
+ ### Тестируемость
164
+ <находки>
165
+
166
+ ## Предложенные исправления (patches)
167
+ <минимальные unified diff-патчи для очевидных правок>
168
+ ```
169
+
170
+ ### Формат каждой находки (обязательно)
171
+
172
+ Для каждой находки укажи:
173
+
174
+ - **Severity**: `blocker` / `major` / `minor`
175
+ - **Location**: путь файла + релевантный hunk/фрагмент из diff
176
+ - **Problem**: что не так
177
+ - **Why it matters**: почему это важно (корректность/тестируемость/конвенции/поддерживаемость)
178
+ - **Suggested fix**: что конкретно сделать (точечно, без раздувания scope)
179
+ - **Optional patch**: если фикс простой — unified diff
180
+
181
+ Пример блока патча:
182
+
183
+ ```diff
184
+ diff --git a/path/file.ts b/path/file.ts
185
+ index 0000000..1111111 100644
186
+ --- a/path/file.ts
187
+ +++ b/path/file.ts
188
+ @@ -1,3 +1,3 @@
189
+ -const x = 1;
190
+ +const x = 2;
191
+ ```
192
+
193
+ ---
194
+
195
+ ## Job Context Awareness
196
+
197
+ When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
198
+
199
+ ```
200
+ JOB_NAME: <job-name>
201
+ CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
202
+ ```
203
+
204
+ If provided and the file exists, read the context document before starting the review. Use it to:
205
+ - Understand which libraries and patterns were intentionally chosen for the implementation
206
+ - Avoid flagging correct library usage as issues
207
+ - Provide more accurate findings by understanding the project's architectural decisions
208
+
209
+ If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.
@@ -0,0 +1,208 @@
1
+ ---
2
+ name: code-b091-review
3
+ description: "Performs strict b091-style code review following code-review-b091-profile.mdc. Reviews current branch changes from merge-base. Direct, no-fluff feedback focused on logic correctness. Use when: b091 review requested, strict validation needed."
4
+ triggers:
5
+ - "Review as b091"
6
+ - "b091 style review"
7
+ - "b091 review"
8
+ metadata:
9
+ author: "MrCipherSmith"
10
+ version: "1.0.0"
11
+ category: "review"
12
+ license: "MIT"
13
+ compatibility: "cursor,codex,zed,opencode"
14
+ ---
15
+
16
+ # Code Review as b091 (только текущая ветка)
17
+
18
+ ## Workflow
19
+
20
+ Copy this checklist and track progress:
21
+
22
+ ```
23
+ b091 Review Progress:
24
+ - [ ] Step 1: Determine parent branch and calculate merge-base
25
+ - [ ] Step 2: Collect git diff (committed + local changes)
26
+ - [ ] Step 3: Apply b091 principles (logic in correct layer, no ducttape)
27
+ - [ ] Step 4: Check types (no any, typed mocks)
28
+ - [ ] Step 5: Verify conventions (currentState, I prefix, etc.)
29
+ - [ ] Step 6: Challenge all assumptions
30
+ - [ ] Step 7: Generate direct report with patches
31
+ ```
32
+
33
+ ## Обязательные правила
34
+
35
+ 1. **Скоуп по умолчанию (без commit hash/range в запросе)**: ревью включает **все** изменения в текущей ветке от точки ответвления (merge-base) от родительской ветки:
36
+ - закоммиченные (`BASE_SHA..HEAD`)
37
+ - локальные незакоммиченные (staged/unstaged/untracked)
38
+ 2. **Скоуп при явном commit hash/range**: если пользователь явно передал hash или диапазон, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
39
+ 3. **Стиль/принципы b091**: ревью строго по `.metaproject/rules/core/code-review-b091-profile.mdc` (разделы правил и промпт).
40
+ 4. **Результат**: подробный отчёт по проблемам с объяснением и предложением исправлений (минимальные патчи там, где это просто).
41
+
42
+ ## Scope Detection
43
+
44
+ See shared script: `skills/shared/git-merge-base.md`
45
+
46
+ Run the script from that file to determine MERGE_BASE and SCOPE before proceeding with the review.
47
+
48
+ ## Команды, чтобы собрать “срез” для ревью
49
+
50
+ ### A) Режим по умолчанию (пользователь НЕ дал hash/range)
51
+
52
+ ```bash
53
+ git status
54
+
55
+ # Закоммиченная часть ветки
56
+ git log --oneline "${BASE_SHA}..HEAD"
57
+ git diff --stat "${BASE_SHA}..HEAD"
58
+ git diff --name-status "${BASE_SHA}..HEAD"
59
+ git diff "${BASE_SHA}..HEAD"
60
+
61
+ # Полный текущий срез от merge-base до рабочего дерева:
62
+ # включает commit'ы + staged + unstaged
63
+ # (untracked смотри через git status / git ls-files)
64
+ git diff --stat "${BASE_SHA}"
65
+ git diff --name-status "${BASE_SHA}"
66
+ git diff "${BASE_SHA}"
67
+ git ls-files --others --exclude-standard
68
+ ```
69
+
70
+ ### B) Режим с явным hash/range
71
+
72
+ Если пользователь дал конкретный hash/range, используй его буквально:
73
+
74
+ ```bash
75
+ # Примеры:
76
+ git show --stat --name-status --patch <COMMIT_SHA>
77
+ git log --oneline <FROM_SHA>..<TO_SHA>
78
+ git diff --stat <FROM_SHA>..<TO_SHA>
79
+ git diff --name-status <FROM_SHA>..<TO_SHA>
80
+ git diff <FROM_SHA>..<TO_SHA>
81
+ ```
82
+
83
+ ## Как ревьюить (по b091)
84
+
85
+ Применяй чек-лист и промпт из `.metaproject/rules/core/code-review-b091-profile.mdc`. Ключевые акценты b091:
86
+
87
+ - Логика должна жить в правильном слое (часто — в store), чтобы её можно было нормально тестировать. Никогда не мешать Domain Store и UI Component Store (это жёстко карается как нарушение SOLID/Layering).
88
+ - Против “patch/glue/ducttape”: требуй чинить причину, а не маскировать симптомы.
89
+ - Строго против раздутых, "водянистых" AI-сгенерированных описаний PR. Только суть и польза.
90
+ - Не допускай регрессий и возвращения старых архитектурных ошибок (например, добавления UI-моделей в core DTO).
91
+ - В новом коде никаких `any`, `as any`, небезопасных кастов; моки — типизированные и не наследуются от реальных реализаций.
92
+ - Не плодить дублирование правил/проверок: один источник правды, переиспользовать сеттеры/методы.
93
+ - Конвенции проекта обязательны (например, `currentState`, `I` для интерфейсов, консистентные типы).
94
+ - Магические числа / `setTimeout(0)` / "страшные" хаки — только с чётким "почему", либо перепроектировать.
95
+ - **Accessibility модификаторы в сторах**: inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный метод допустим только если он вызывается из React-компонента. Спросить: "Вызывается ли этот метод из JSX?" Если нет — `private`.
96
+ - С MobX по умолчанию **не добавлять** `useCallback`/`useMemo` без явной необходимости.
97
+ - Предложения должны проходить линт; не нитпикать автоформатирование.
98
+ - Не раздувать scope: большие рефакторы “not today”, если вне задачи.
99
+
100
+ ## Формат вывода (подробный отчёт + тон b091)
101
+
102
+ Пиши по-русски, но допускай короткие b091-маркеры (например, `not today ;P`, “ducttape”, “broken thinking”) только по делу и без перехода на личности.
103
+
104
+ Используй структуру:
105
+
106
+ ```markdown
107
+ ## Вердикт (b091)
108
+
109
+ <OK / needs work / fundamentally wrong> + 1–3 самых важных тезиса.
110
+
111
+ ## Скоуп ревью (только текущая ветка)
112
+
113
+ - Ветка: `<BRANCH>`
114
+ - Parent ref: `<PARENT>`
115
+ - Merge-base: `<BASE_SHA>`
116
+ - Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
117
+ - Коммиты (merge-base..HEAD): <N>
118
+ - Изменённые файлы: <список или количество>
119
+
120
+ ## Находки (подробно)
121
+
122
+ ### Архитектура и место логики
123
+
124
+ <находки>
125
+
126
+ ### Типы, касты, моки
127
+
128
+ <находки>
129
+
130
+ ### Корректность и семантика состояний
131
+
132
+ <находки>
133
+
134
+ ### Конвенции и консистентность
135
+
136
+ <находки>
137
+
138
+ ### Линт/формат/шум
139
+
140
+ <находки>
141
+
142
+ ### UX/edge cases
143
+
144
+ <находки>
145
+
146
+ ### Тестируемость
147
+
148
+ <находки>
149
+
150
+ ## Предложенные исправления (patches)
151
+
152
+ <минимальные unified diff-патчи для очевидных правок>
153
+ ```
154
+
155
+ ### Формат каждой находки (обязательно)
156
+
157
+ Для каждой находки укажи:
158
+
159
+ - **Severity**: `blocker` / `major` / `minor`
160
+ - **Location**: путь файла + релевантный hunk/фрагмент из diff
161
+ - **Problem**: что не так
162
+ - **Why it matters**: почему это важно (корректность/тестируемость/конвенции/поддерживаемость)
163
+ - **Suggested fix**: что конкретно сделать (точечно, без раздувания scope)
164
+ - **Optional patch**: если фикс простой — unified diff
165
+
166
+ Пример блока патча:
167
+
168
+ ```diff
169
+ diff --git a/path/file.ts b/path/file.ts
170
+ index 0000000..1111111 100644
171
+ --- a/path/file.ts
172
+ +++ b/path/file.ts
173
+ @@ -1,3 +1,3 @@
174
+ -const x = 1;
175
+ +const x = 2;
176
+ ```
177
+
178
+ ---
179
+
180
+ ## Scope Boundaries
181
+
182
+ This skill covers **b091-style code review** — direct, logic-first, no-fluff feedback following `code-review-b091-profile.mdc`.
183
+
184
+ | Concern | This skill | Use instead |
185
+ | ------------------------------------------------------------- | ---------- | ------------------------ |
186
+ | Logic correctness, layer violations, type safety, conventions | ✅ YES | — |
187
+ | General AI review (broader quality, performance, UX) | ❌ NO | `code-ai-review` |
188
+ | MobX store internals (actions, computed, reactions) | ❌ NO | `code-mobx-store-review` |
189
+ | Pure style/naming/architecture pattern audit | ❌ NO | `code-style-review` |
190
+
191
+ ---
192
+
193
+ ## Job Context Awareness
194
+
195
+ When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
196
+
197
+ ```
198
+ JOB_NAME: <job-name>
199
+ CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
200
+ ```
201
+
202
+ If provided and the file exists, read the context document before starting the review. Use it to:
203
+
204
+ - Understand which libraries and patterns were intentionally chosen for the implementation
205
+ - Avoid flagging correct library usage as issues
206
+ - Provide more accurate findings by understanding the project's architectural decisions
207
+
208
+ If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.
@@ -0,0 +1,209 @@
1
+ ---
2
+ name: code-b091-review
3
+ description: "Performs strict b091-style code review following code-review-b091-profile.mdc. Reviews current branch changes from merge-base. Direct, no-fluff feedback focused on logic correctness. Use when: b091 review requested, strict validation needed."
4
+ triggers:
5
+ - "Review as b091"
6
+ - "b091 style review"
7
+ - "b091 review"
8
+ metadata:
9
+ author: "MrCipherSmith"
10
+ version: "1.0.0"
11
+ category: "review"
12
+ license: "MIT"
13
+ compatibility: "cursor,codex,zed,opencode"
14
+ ---
15
+
16
+
17
+ # Code Review as b091 (только текущая ветка)
18
+
19
+ ## Workflow
20
+
21
+ Copy this checklist and track progress:
22
+
23
+ ```
24
+ b091 Review Progress:
25
+ - [ ] Step 1: Determine parent branch and calculate merge-base
26
+ - [ ] Step 2: Collect git diff (committed + local changes)
27
+ - [ ] Step 3: Apply b091 principles (logic in correct layer, no ducttape)
28
+ - [ ] Step 4: Check types (no any, typed mocks)
29
+ - [ ] Step 5: Verify conventions (currentState, I prefix, etc.)
30
+ - [ ] Step 6: Challenge all assumptions
31
+ - [ ] Step 7: Generate direct report with patches
32
+ ```
33
+
34
+ ## Обязательные правила
35
+
36
+ 1) **Скоуп по умолчанию (без commit hash/range в запросе)**: ревью включает **все** изменения в текущей ветке от точки ответвления (merge-base) от родительской ветки:
37
+ - закоммиченные (`BASE_SHA..HEAD`)
38
+ - локальные незакоммиченные (staged/unstaged/untracked)
39
+ 2) **Скоуп при явном commit hash/range**: если пользователь явно передал hash или диапазон, ревьюй только запрошенный диапазон; локальные незакоммиченные изменения не добавляй, если это отдельно не попросили.
40
+ 3) **Стиль/принципы b091**: ревью строго по `.metaproject/rules/core/code-review-b091-profile.mdc` (разделы правил и промпт).
41
+ 4) **Результат**: подробный отчёт по проблемам с объяснением и предложением исправлений (минимальные патчи там, где это просто).
42
+
43
+ ## Определение parent ref (детерминированно, именно родитель ветки)
44
+
45
+ Используй первый существующий вариант:
46
+
47
+ 1. `origin/main`
48
+ 2. `origin/master`
49
+ 3. `main`
50
+ 4. `master`
51
+ 5. `@{upstream}` **только если это не текущая feature-ветка**
52
+
53
+ ```bash
54
+ BRANCH="$(git rev-parse --abbrev-ref HEAD)"
55
+ UPSTREAM_REF="$(git rev-parse --abbrev-ref --symbolic-full-name @{upstream} 2>/dev/null || true)"
56
+
57
+ PARENT=""
58
+ if git rev-parse --verify -q "origin/main" >/dev/null; then
59
+ PARENT="origin/main"
60
+ elif git rev-parse --verify -q "origin/master" >/dev/null; then
61
+ PARENT="origin/master"
62
+ elif git rev-parse --verify -q "main" >/dev/null; then
63
+ PARENT="main"
64
+ elif git rev-parse --verify -q "master" >/dev/null; then
65
+ PARENT="master"
66
+ elif [ -n "$UPSTREAM_REF" ] && [ "$UPSTREAM_REF" != "$BRANCH" ] && [ "$UPSTREAM_REF" != "origin/$BRANCH" ]; then
67
+ PARENT="@{upstream}"
68
+ else
69
+ echo "Cannot determine parent ref" >&2
70
+ exit 1
71
+ fi
72
+
73
+ BASE_SHA="$(git merge-base HEAD "$PARENT")"
74
+ ```
75
+
76
+ ## Команды, чтобы собрать “срез” для ревью
77
+
78
+ ### A) Режим по умолчанию (пользователь НЕ дал hash/range)
79
+
80
+ ```bash
81
+ git status
82
+
83
+ # Закоммиченная часть ветки
84
+ git log --oneline "${BASE_SHA}..HEAD"
85
+ git diff --stat "${BASE_SHA}..HEAD"
86
+ git diff --name-status "${BASE_SHA}..HEAD"
87
+ git diff "${BASE_SHA}..HEAD"
88
+
89
+ # Полный текущий срез от merge-base до рабочего дерева:
90
+ # включает commit'ы + staged + unstaged
91
+ # (untracked смотри через git status / git ls-files)
92
+ git diff --stat "${BASE_SHA}"
93
+ git diff --name-status "${BASE_SHA}"
94
+ git diff "${BASE_SHA}"
95
+ git ls-files --others --exclude-standard
96
+ ```
97
+
98
+ ### B) Режим с явным hash/range
99
+
100
+ Если пользователь дал конкретный hash/range, используй его буквально:
101
+
102
+ ```bash
103
+ # Примеры:
104
+ git show --stat --name-status --patch <COMMIT_SHA>
105
+ git log --oneline <FROM_SHA>..<TO_SHA>
106
+ git diff --stat <FROM_SHA>..<TO_SHA>
107
+ git diff --name-status <FROM_SHA>..<TO_SHA>
108
+ git diff <FROM_SHA>..<TO_SHA>
109
+ ```
110
+
111
+ ## Как ревьюить (по b091)
112
+
113
+ Применяй чек-лист и промпт из `.metaproject/rules/core/code-review-b091-profile.mdc`. Ключевые акценты b091:
114
+
115
+ - Логика должна жить в правильном слое (часто — в store), чтобы её можно было нормально тестировать.
116
+ - Против “patch/glue/ducttape”: требуй чинить причину, а не маскировать симптомы.
117
+ - В новом коде никаких `any`, `as any`, небезопасных кастов; моки — типизированные и не наследуются от реальных реализаций.
118
+ - Не плодить дублирование правил/проверок: один источник правды, переиспользовать сеттеры/методы.
119
+ - Конвенции проекта обязательны (например, `currentState`, `I` для интерфейсов, консистентные типы).
120
+ - Магические числа / `setTimeout(0)` / "страшные" хаки — только с чётким "почему", либо перепроектировать.
121
+ - **Accessibility модификаторы в сторах**: inter-store callbacks (`onChangeX`, `onFireX`, `handleX`, `syncX`) обязаны быть `private`. Публичный метод допустим только если он вызывается из React-компонента. Спросить: "Вызывается ли этот метод из JSX?" Если нет — `private`.
122
+ - С MobX по умолчанию **не добавлять** `useCallback`/`useMemo` без явной необходимости.
123
+ - Предложения должны проходить линт; не нитпикать автоформатирование.
124
+ - Не раздувать scope: большие рефакторы “not today”, если вне задачи.
125
+
126
+ ## Формат вывода (подробный отчёт + тон b091)
127
+
128
+ Пиши по-русски, но допускай короткие b091-маркеры (например, `not today ;P`, “ducttape”, “broken thinking”) только по делу и без перехода на личности.
129
+
130
+ Используй структуру:
131
+
132
+ ```markdown
133
+ ## Вердикт (b091)
134
+ <OK / needs work / fundamentally wrong> + 1–3 самых важных тезиса.
135
+
136
+ ## Скоуп ревью (только текущая ветка)
137
+ - Ветка: `<BRANCH>`
138
+ - Parent ref: `<PARENT>`
139
+ - Merge-base: `<BASE_SHA>`
140
+ - Режим скоупа: `<default-with-uncommitted | explicit-hash-range>`
141
+ - Коммиты (merge-base..HEAD): <N>
142
+ - Изменённые файлы: <список или количество>
143
+
144
+ ## Находки (подробно)
145
+ ### Архитектура и место логики
146
+ <находки>
147
+
148
+ ### Типы, касты, моки
149
+ <находки>
150
+
151
+ ### Корректность и семантика состояний
152
+ <находки>
153
+
154
+ ### Конвенции и консистентность
155
+ <находки>
156
+
157
+ ### Линт/формат/шум
158
+ <находки>
159
+
160
+ ### UX/edge cases
161
+ <находки>
162
+
163
+ ### Тестируемость
164
+ <находки>
165
+
166
+ ## Предложенные исправления (patches)
167
+ <минимальные unified diff-патчи для очевидных правок>
168
+ ```
169
+
170
+ ### Формат каждой находки (обязательно)
171
+
172
+ Для каждой находки укажи:
173
+
174
+ - **Severity**: `blocker` / `major` / `minor`
175
+ - **Location**: путь файла + релевантный hunk/фрагмент из diff
176
+ - **Problem**: что не так
177
+ - **Why it matters**: почему это важно (корректность/тестируемость/конвенции/поддерживаемость)
178
+ - **Suggested fix**: что конкретно сделать (точечно, без раздувания scope)
179
+ - **Optional patch**: если фикс простой — unified diff
180
+
181
+ Пример блока патча:
182
+
183
+ ```diff
184
+ diff --git a/path/file.ts b/path/file.ts
185
+ index 0000000..1111111 100644
186
+ --- a/path/file.ts
187
+ +++ b/path/file.ts
188
+ @@ -1,3 +1,3 @@
189
+ -const x = 1;
190
+ +const x = 2;
191
+ ```
192
+
193
+ ---
194
+
195
+ ## Job Context Awareness
196
+
197
+ When dispatched by `job-orchestrator` as part of a job pipeline, the prompt MAY include:
198
+
199
+ ```
200
+ JOB_NAME: <job-name>
201
+ CONTEXT_PATH: .metaproject/jobs/<job-name>/ai/context.md
202
+ ```
203
+
204
+ If provided and the file exists, read the context document before starting the review. Use it to:
205
+ - Understand which libraries and patterns were intentionally chosen for the implementation
206
+ - Avoid flagging correct library usage as issues
207
+ - Provide more accurate findings by understanding the project's architectural decisions
208
+
209
+ If the file does not exist or is not provided, proceed normally — context is optional and non-blocking.