devflow-kit 0.9.0 → 1.1.0

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 (414) hide show
  1. package/CHANGELOG.md +197 -29
  2. package/LICENSE +1 -1
  3. package/README.md +185 -309
  4. package/dist/cli.js +7 -1
  5. package/dist/commands/ambient.d.ts +18 -0
  6. package/dist/commands/ambient.js +136 -0
  7. package/dist/commands/init.d.ts +23 -0
  8. package/dist/commands/init.js +393 -571
  9. package/dist/commands/list.d.ts +3 -0
  10. package/dist/commands/list.js +20 -0
  11. package/dist/commands/memory.d.ts +22 -0
  12. package/dist/commands/memory.js +175 -0
  13. package/dist/commands/uninstall.d.ts +10 -0
  14. package/dist/commands/uninstall.js +418 -78
  15. package/dist/plugins.d.ts +46 -0
  16. package/dist/plugins.js +169 -0
  17. package/dist/utils/cli.d.ts +5 -0
  18. package/dist/utils/cli.js +14 -0
  19. package/dist/utils/installer.d.ts +41 -0
  20. package/dist/utils/installer.js +177 -0
  21. package/dist/utils/paths.d.ts +10 -0
  22. package/dist/utils/paths.js +23 -3
  23. package/dist/utils/post-install.d.ts +80 -0
  24. package/dist/utils/post-install.js +508 -0
  25. package/dist/utils/safe-delete-install.d.ts +29 -0
  26. package/dist/utils/safe-delete-install.js +191 -0
  27. package/dist/utils/safe-delete.d.ts +12 -0
  28. package/dist/utils/safe-delete.js +83 -0
  29. package/package.json +18 -8
  30. package/plugins/devflow-ambient/.claude-plugin/plugin.json +7 -0
  31. package/plugins/devflow-ambient/README.md +49 -0
  32. package/plugins/devflow-ambient/commands/ambient.md +110 -0
  33. package/plugins/devflow-ambient/skills/ambient-router/SKILL.md +89 -0
  34. package/plugins/devflow-ambient/skills/ambient-router/references/skill-catalog.md +64 -0
  35. package/plugins/devflow-audit-claude/.claude-plugin/plugin.json +7 -0
  36. package/plugins/devflow-audit-claude/README.md +46 -0
  37. package/plugins/devflow-audit-claude/agents/claude-md-auditor.md +134 -0
  38. package/plugins/devflow-audit-claude/commands/audit-claude.md +85 -0
  39. package/plugins/devflow-code-review/.claude-plugin/plugin.json +31 -0
  40. package/plugins/devflow-code-review/README.md +73 -0
  41. package/plugins/devflow-code-review/agents/git.md +272 -0
  42. package/plugins/devflow-code-review/agents/reviewer.md +119 -0
  43. package/plugins/devflow-code-review/agents/synthesizer.md +204 -0
  44. package/plugins/devflow-code-review/commands/code-review-teams.md +262 -0
  45. package/plugins/devflow-code-review/commands/code-review.md +141 -0
  46. package/plugins/devflow-code-review/skills/accessibility/SKILL.md +229 -0
  47. package/plugins/devflow-code-review/skills/accessibility/references/detection.md +171 -0
  48. package/plugins/devflow-code-review/skills/accessibility/references/patterns.md +670 -0
  49. package/plugins/devflow-code-review/skills/accessibility/references/violations.md +419 -0
  50. package/plugins/devflow-code-review/skills/agent-teams/SKILL.md +124 -0
  51. package/plugins/devflow-code-review/skills/agent-teams/references/cleanup.md +104 -0
  52. package/plugins/devflow-code-review/skills/agent-teams/references/communication.md +122 -0
  53. package/plugins/devflow-code-review/skills/agent-teams/references/team-patterns.md +217 -0
  54. package/plugins/devflow-code-review/skills/architecture-patterns/SKILL.md +153 -0
  55. package/plugins/devflow-code-review/skills/architecture-patterns/references/detection.md +337 -0
  56. package/plugins/devflow-code-review/skills/architecture-patterns/references/patterns.md +873 -0
  57. package/plugins/devflow-code-review/skills/architecture-patterns/references/violations.md +575 -0
  58. package/plugins/devflow-code-review/skills/complexity-patterns/SKILL.md +143 -0
  59. package/plugins/devflow-code-review/skills/complexity-patterns/references/detection.md +264 -0
  60. package/plugins/devflow-code-review/skills/complexity-patterns/references/patterns.md +487 -0
  61. package/plugins/devflow-code-review/skills/complexity-patterns/references/violations.md +361 -0
  62. package/plugins/devflow-code-review/skills/consistency-patterns/SKILL.md +140 -0
  63. package/plugins/devflow-code-review/skills/consistency-patterns/references/detection.md +207 -0
  64. package/plugins/devflow-code-review/skills/consistency-patterns/references/patterns.md +202 -0
  65. package/plugins/devflow-code-review/skills/consistency-patterns/references/violations.md +213 -0
  66. package/plugins/devflow-code-review/skills/database-patterns/SKILL.md +134 -0
  67. package/plugins/devflow-code-review/skills/database-patterns/references/detection.md +208 -0
  68. package/plugins/devflow-code-review/skills/database-patterns/references/patterns.md +394 -0
  69. package/plugins/devflow-code-review/skills/database-patterns/references/violations.md +332 -0
  70. package/plugins/devflow-code-review/skills/dependencies-patterns/SKILL.md +141 -0
  71. package/plugins/devflow-code-review/skills/dependencies-patterns/references/detection.md +181 -0
  72. package/plugins/devflow-code-review/skills/dependencies-patterns/references/patterns.md +225 -0
  73. package/plugins/devflow-code-review/skills/dependencies-patterns/references/violations.md +247 -0
  74. package/plugins/devflow-code-review/skills/documentation-patterns/SKILL.md +125 -0
  75. package/plugins/devflow-code-review/skills/documentation-patterns/references/detection.md +190 -0
  76. package/plugins/devflow-code-review/skills/documentation-patterns/references/patterns.md +189 -0
  77. package/plugins/devflow-code-review/skills/documentation-patterns/references/violations.md +163 -0
  78. package/plugins/devflow-code-review/skills/frontend-design/SKILL.md +254 -0
  79. package/plugins/devflow-code-review/skills/frontend-design/references/detection.md +184 -0
  80. package/plugins/devflow-code-review/skills/frontend-design/references/patterns.md +511 -0
  81. package/plugins/devflow-code-review/skills/frontend-design/references/violations.md +453 -0
  82. package/plugins/devflow-code-review/skills/performance-patterns/SKILL.md +154 -0
  83. package/plugins/devflow-code-review/skills/performance-patterns/references/detection.md +351 -0
  84. package/plugins/devflow-code-review/skills/performance-patterns/references/patterns.md +503 -0
  85. package/plugins/devflow-code-review/skills/performance-patterns/references/violations.md +354 -0
  86. package/plugins/devflow-code-review/skills/react/SKILL.md +276 -0
  87. package/plugins/devflow-code-review/skills/react/references/patterns.md +1331 -0
  88. package/plugins/devflow-code-review/skills/react/references/violations.md +565 -0
  89. package/plugins/devflow-code-review/skills/regression-patterns/SKILL.md +146 -0
  90. package/plugins/devflow-code-review/skills/regression-patterns/references/detection.md +237 -0
  91. package/plugins/devflow-code-review/skills/regression-patterns/references/patterns.md +226 -0
  92. package/plugins/devflow-code-review/skills/regression-patterns/references/violations.md +225 -0
  93. package/plugins/devflow-code-review/skills/review-methodology/SKILL.md +119 -0
  94. package/plugins/devflow-code-review/skills/review-methodology/references/patterns.md +186 -0
  95. package/plugins/devflow-code-review/skills/review-methodology/references/report-template.md +142 -0
  96. package/plugins/devflow-code-review/skills/review-methodology/references/violations.md +125 -0
  97. package/plugins/devflow-code-review/skills/security-patterns/SKILL.md +156 -0
  98. package/plugins/devflow-code-review/skills/security-patterns/references/detection.md +287 -0
  99. package/plugins/devflow-code-review/skills/security-patterns/references/patterns.md +507 -0
  100. package/plugins/devflow-code-review/skills/security-patterns/references/violations.md +237 -0
  101. package/plugins/devflow-code-review/skills/test-patterns/SKILL.md +183 -0
  102. package/plugins/devflow-code-review/skills/test-patterns/references/detection.md +149 -0
  103. package/plugins/devflow-code-review/skills/test-patterns/references/patterns.md +220 -0
  104. package/plugins/devflow-code-review/skills/test-patterns/references/report-template.md +108 -0
  105. package/plugins/devflow-code-review/skills/test-patterns/references/violations.md +221 -0
  106. package/plugins/devflow-core-skills/.claude-plugin/plugin.json +28 -0
  107. package/plugins/devflow-core-skills/README.md +50 -0
  108. package/plugins/devflow-core-skills/skills/accessibility/SKILL.md +229 -0
  109. package/plugins/devflow-core-skills/skills/accessibility/references/detection.md +171 -0
  110. package/plugins/devflow-core-skills/skills/accessibility/references/patterns.md +670 -0
  111. package/plugins/devflow-core-skills/skills/accessibility/references/violations.md +419 -0
  112. package/plugins/devflow-core-skills/skills/core-patterns/SKILL.md +162 -0
  113. package/plugins/devflow-core-skills/skills/core-patterns/references/checklist.md +276 -0
  114. package/plugins/devflow-core-skills/skills/core-patterns/references/code-smell-violations.md +144 -0
  115. package/plugins/devflow-core-skills/skills/core-patterns/references/detection.md +303 -0
  116. package/plugins/devflow-core-skills/skills/core-patterns/references/patterns.md +576 -0
  117. package/plugins/devflow-core-skills/skills/core-patterns/references/violations.md +369 -0
  118. package/plugins/devflow-core-skills/skills/docs-framework/SKILL.md +138 -0
  119. package/plugins/devflow-core-skills/skills/docs-framework/references/patterns.md +346 -0
  120. package/plugins/devflow-core-skills/skills/docs-framework/references/violations.md +221 -0
  121. package/plugins/devflow-core-skills/skills/frontend-design/SKILL.md +254 -0
  122. package/plugins/devflow-core-skills/skills/frontend-design/references/detection.md +184 -0
  123. package/plugins/devflow-core-skills/skills/frontend-design/references/patterns.md +511 -0
  124. package/plugins/devflow-core-skills/skills/frontend-design/references/violations.md +453 -0
  125. package/plugins/devflow-core-skills/skills/git-safety/SKILL.md +122 -0
  126. package/plugins/devflow-core-skills/skills/git-safety/references/detection.md +290 -0
  127. package/plugins/devflow-core-skills/skills/git-safety/references/patterns.md +289 -0
  128. package/plugins/devflow-core-skills/skills/git-safety/references/violations.md +18 -0
  129. package/plugins/devflow-core-skills/skills/git-workflow/SKILL.md +158 -0
  130. package/plugins/devflow-core-skills/skills/git-workflow/references/commit-patterns.md +115 -0
  131. package/plugins/devflow-core-skills/skills/git-workflow/references/commit-violations.md +77 -0
  132. package/plugins/devflow-core-skills/skills/git-workflow/references/pr-patterns.md +127 -0
  133. package/plugins/devflow-core-skills/skills/git-workflow/references/pr-violations.md +96 -0
  134. package/plugins/devflow-core-skills/skills/github-patterns/SKILL.md +153 -0
  135. package/plugins/devflow-core-skills/skills/github-patterns/references/patterns.md +572 -0
  136. package/plugins/devflow-core-skills/skills/github-patterns/references/violations.md +298 -0
  137. package/plugins/devflow-core-skills/skills/input-validation/SKILL.md +148 -0
  138. package/plugins/devflow-core-skills/skills/input-validation/references/detection.md +283 -0
  139. package/plugins/devflow-core-skills/skills/input-validation/references/patterns.md +361 -0
  140. package/plugins/devflow-core-skills/skills/input-validation/references/violations.md +224 -0
  141. package/plugins/devflow-core-skills/skills/react/SKILL.md +276 -0
  142. package/plugins/devflow-core-skills/skills/react/references/patterns.md +1331 -0
  143. package/plugins/devflow-core-skills/skills/react/references/violations.md +565 -0
  144. package/plugins/devflow-core-skills/skills/test-driven-development/SKILL.md +139 -0
  145. package/plugins/devflow-core-skills/skills/test-driven-development/references/rationalization-prevention.md +111 -0
  146. package/plugins/devflow-core-skills/skills/test-patterns/SKILL.md +183 -0
  147. package/plugins/devflow-core-skills/skills/test-patterns/references/detection.md +149 -0
  148. package/plugins/devflow-core-skills/skills/test-patterns/references/patterns.md +220 -0
  149. package/plugins/devflow-core-skills/skills/test-patterns/references/report-template.md +108 -0
  150. package/plugins/devflow-core-skills/skills/test-patterns/references/violations.md +221 -0
  151. package/plugins/devflow-core-skills/skills/typescript/SKILL.md +176 -0
  152. package/plugins/devflow-core-skills/skills/typescript/references/patterns.md +1105 -0
  153. package/plugins/devflow-core-skills/skills/typescript/references/violations.md +433 -0
  154. package/plugins/devflow-debug/.claude-plugin/plugin.json +18 -0
  155. package/plugins/devflow-debug/README.md +65 -0
  156. package/plugins/devflow-debug/agents/git.md +272 -0
  157. package/plugins/devflow-debug/commands/debug-teams.md +231 -0
  158. package/plugins/devflow-debug/commands/debug.md +160 -0
  159. package/plugins/devflow-debug/skills/agent-teams/SKILL.md +124 -0
  160. package/plugins/devflow-debug/skills/agent-teams/references/cleanup.md +104 -0
  161. package/plugins/devflow-debug/skills/agent-teams/references/communication.md +122 -0
  162. package/plugins/devflow-debug/skills/agent-teams/references/team-patterns.md +217 -0
  163. package/plugins/devflow-debug/skills/git-safety/SKILL.md +122 -0
  164. package/plugins/devflow-debug/skills/git-safety/references/detection.md +290 -0
  165. package/plugins/devflow-debug/skills/git-safety/references/patterns.md +289 -0
  166. package/plugins/devflow-debug/skills/git-safety/references/violations.md +18 -0
  167. package/plugins/devflow-implement/.claude-plugin/plugin.json +21 -0
  168. package/plugins/devflow-implement/README.md +71 -0
  169. package/plugins/devflow-implement/agents/coder.md +122 -0
  170. package/plugins/devflow-implement/agents/git.md +272 -0
  171. package/plugins/devflow-implement/agents/scrutinizer.md +80 -0
  172. package/plugins/devflow-implement/agents/shepherd.md +94 -0
  173. package/plugins/devflow-implement/agents/simplifier.md +62 -0
  174. package/plugins/devflow-implement/agents/skimmer.md +88 -0
  175. package/plugins/devflow-implement/agents/synthesizer.md +204 -0
  176. package/plugins/devflow-implement/agents/validator.md +86 -0
  177. package/plugins/devflow-implement/commands/implement-teams.md +608 -0
  178. package/plugins/devflow-implement/commands/implement.md +426 -0
  179. package/plugins/devflow-implement/skills/accessibility/SKILL.md +229 -0
  180. package/plugins/devflow-implement/skills/accessibility/references/detection.md +171 -0
  181. package/plugins/devflow-implement/skills/accessibility/references/patterns.md +670 -0
  182. package/plugins/devflow-implement/skills/accessibility/references/violations.md +419 -0
  183. package/plugins/devflow-implement/skills/agent-teams/SKILL.md +124 -0
  184. package/plugins/devflow-implement/skills/agent-teams/references/cleanup.md +104 -0
  185. package/plugins/devflow-implement/skills/agent-teams/references/communication.md +122 -0
  186. package/plugins/devflow-implement/skills/agent-teams/references/team-patterns.md +217 -0
  187. package/plugins/devflow-implement/skills/frontend-design/SKILL.md +254 -0
  188. package/plugins/devflow-implement/skills/frontend-design/references/detection.md +184 -0
  189. package/plugins/devflow-implement/skills/frontend-design/references/patterns.md +511 -0
  190. package/plugins/devflow-implement/skills/frontend-design/references/violations.md +453 -0
  191. package/plugins/devflow-implement/skills/implementation-patterns/SKILL.md +162 -0
  192. package/plugins/devflow-implement/skills/implementation-patterns/references/patterns.md +1063 -0
  193. package/plugins/devflow-implement/skills/implementation-patterns/references/violations.md +483 -0
  194. package/plugins/devflow-implement/skills/self-review/SKILL.md +149 -0
  195. package/plugins/devflow-implement/skills/self-review/references/patterns.md +405 -0
  196. package/plugins/devflow-implement/skills/self-review/references/report-template.md +253 -0
  197. package/plugins/devflow-implement/skills/self-review/references/violations.md +308 -0
  198. package/plugins/devflow-resolve/.claude-plugin/plugin.json +19 -0
  199. package/plugins/devflow-resolve/README.md +65 -0
  200. package/plugins/devflow-resolve/agents/git.md +272 -0
  201. package/plugins/devflow-resolve/agents/resolver.md +131 -0
  202. package/plugins/devflow-resolve/agents/simplifier.md +62 -0
  203. package/plugins/devflow-resolve/commands/resolve-teams.md +298 -0
  204. package/plugins/devflow-resolve/commands/resolve.md +237 -0
  205. package/plugins/devflow-resolve/skills/agent-teams/SKILL.md +124 -0
  206. package/plugins/devflow-resolve/skills/agent-teams/references/cleanup.md +104 -0
  207. package/plugins/devflow-resolve/skills/agent-teams/references/communication.md +122 -0
  208. package/plugins/devflow-resolve/skills/agent-teams/references/team-patterns.md +217 -0
  209. package/plugins/devflow-resolve/skills/implementation-patterns/SKILL.md +162 -0
  210. package/plugins/devflow-resolve/skills/implementation-patterns/references/patterns.md +1063 -0
  211. package/plugins/devflow-resolve/skills/implementation-patterns/references/violations.md +483 -0
  212. package/plugins/devflow-resolve/skills/security-patterns/SKILL.md +156 -0
  213. package/plugins/devflow-resolve/skills/security-patterns/references/detection.md +287 -0
  214. package/plugins/devflow-resolve/skills/security-patterns/references/patterns.md +507 -0
  215. package/plugins/devflow-resolve/skills/security-patterns/references/violations.md +237 -0
  216. package/plugins/devflow-self-review/.claude-plugin/plugin.json +7 -0
  217. package/plugins/devflow-self-review/README.md +38 -0
  218. package/plugins/devflow-self-review/agents/scrutinizer.md +80 -0
  219. package/plugins/devflow-self-review/agents/simplifier.md +62 -0
  220. package/plugins/devflow-self-review/agents/validator.md +86 -0
  221. package/plugins/devflow-self-review/commands/self-review.md +126 -0
  222. package/plugins/devflow-self-review/skills/core-patterns/SKILL.md +162 -0
  223. package/plugins/devflow-self-review/skills/core-patterns/references/checklist.md +276 -0
  224. package/plugins/devflow-self-review/skills/core-patterns/references/code-smell-violations.md +144 -0
  225. package/plugins/devflow-self-review/skills/core-patterns/references/detection.md +303 -0
  226. package/plugins/devflow-self-review/skills/core-patterns/references/patterns.md +576 -0
  227. package/plugins/devflow-self-review/skills/core-patterns/references/violations.md +369 -0
  228. package/plugins/devflow-self-review/skills/self-review/SKILL.md +149 -0
  229. package/plugins/devflow-self-review/skills/self-review/references/patterns.md +405 -0
  230. package/plugins/devflow-self-review/skills/self-review/references/report-template.md +253 -0
  231. package/plugins/devflow-self-review/skills/self-review/references/violations.md +308 -0
  232. package/plugins/devflow-specify/.claude-plugin/plugin.json +15 -0
  233. package/plugins/devflow-specify/README.md +46 -0
  234. package/plugins/devflow-specify/agents/skimmer.md +88 -0
  235. package/plugins/devflow-specify/agents/synthesizer.md +204 -0
  236. package/plugins/devflow-specify/commands/specify-teams.md +314 -0
  237. package/plugins/devflow-specify/commands/specify.md +179 -0
  238. package/plugins/devflow-specify/skills/agent-teams/SKILL.md +124 -0
  239. package/plugins/devflow-specify/skills/agent-teams/references/cleanup.md +104 -0
  240. package/plugins/devflow-specify/skills/agent-teams/references/communication.md +122 -0
  241. package/plugins/devflow-specify/skills/agent-teams/references/team-patterns.md +217 -0
  242. package/scripts/hooks/ambient-prompt.sh +48 -0
  243. package/scripts/hooks/background-memory-update.sh +208 -0
  244. package/scripts/hooks/ensure-memory-gitignore.sh +17 -0
  245. package/scripts/hooks/pre-compact-memory.sh +87 -0
  246. package/scripts/hooks/session-start-memory.sh +126 -0
  247. package/scripts/hooks/stop-update-memory.sh +85 -0
  248. package/shared/agents/coder.md +122 -0
  249. package/shared/agents/git.md +272 -0
  250. package/shared/agents/resolver.md +131 -0
  251. package/shared/agents/reviewer.md +119 -0
  252. package/shared/agents/scrutinizer.md +80 -0
  253. package/shared/agents/shepherd.md +94 -0
  254. package/shared/agents/simplifier.md +62 -0
  255. package/shared/agents/skimmer.md +88 -0
  256. package/shared/agents/synthesizer.md +204 -0
  257. package/shared/agents/validator.md +86 -0
  258. package/shared/skills/accessibility/SKILL.md +229 -0
  259. package/shared/skills/accessibility/references/detection.md +171 -0
  260. package/shared/skills/accessibility/references/patterns.md +670 -0
  261. package/shared/skills/accessibility/references/violations.md +419 -0
  262. package/shared/skills/agent-teams/SKILL.md +124 -0
  263. package/shared/skills/agent-teams/references/cleanup.md +104 -0
  264. package/shared/skills/agent-teams/references/communication.md +122 -0
  265. package/shared/skills/agent-teams/references/team-patterns.md +217 -0
  266. package/shared/skills/ambient-router/SKILL.md +89 -0
  267. package/shared/skills/ambient-router/references/skill-catalog.md +64 -0
  268. package/shared/skills/architecture-patterns/SKILL.md +153 -0
  269. package/shared/skills/architecture-patterns/references/detection.md +337 -0
  270. package/shared/skills/architecture-patterns/references/patterns.md +873 -0
  271. package/shared/skills/architecture-patterns/references/violations.md +575 -0
  272. package/shared/skills/complexity-patterns/SKILL.md +143 -0
  273. package/shared/skills/complexity-patterns/references/detection.md +264 -0
  274. package/shared/skills/complexity-patterns/references/patterns.md +487 -0
  275. package/shared/skills/complexity-patterns/references/violations.md +361 -0
  276. package/shared/skills/consistency-patterns/SKILL.md +140 -0
  277. package/shared/skills/consistency-patterns/references/detection.md +207 -0
  278. package/shared/skills/consistency-patterns/references/patterns.md +202 -0
  279. package/shared/skills/consistency-patterns/references/violations.md +213 -0
  280. package/shared/skills/core-patterns/SKILL.md +162 -0
  281. package/shared/skills/core-patterns/references/checklist.md +276 -0
  282. package/shared/skills/core-patterns/references/code-smell-violations.md +144 -0
  283. package/shared/skills/core-patterns/references/detection.md +303 -0
  284. package/shared/skills/core-patterns/references/patterns.md +576 -0
  285. package/shared/skills/core-patterns/references/violations.md +369 -0
  286. package/shared/skills/database-patterns/SKILL.md +134 -0
  287. package/shared/skills/database-patterns/references/detection.md +208 -0
  288. package/shared/skills/database-patterns/references/patterns.md +394 -0
  289. package/shared/skills/database-patterns/references/violations.md +332 -0
  290. package/shared/skills/dependencies-patterns/SKILL.md +141 -0
  291. package/shared/skills/dependencies-patterns/references/detection.md +181 -0
  292. package/shared/skills/dependencies-patterns/references/patterns.md +225 -0
  293. package/shared/skills/dependencies-patterns/references/violations.md +247 -0
  294. package/shared/skills/docs-framework/SKILL.md +138 -0
  295. package/shared/skills/docs-framework/references/patterns.md +346 -0
  296. package/shared/skills/docs-framework/references/violations.md +221 -0
  297. package/shared/skills/documentation-patterns/SKILL.md +125 -0
  298. package/shared/skills/documentation-patterns/references/detection.md +190 -0
  299. package/shared/skills/documentation-patterns/references/patterns.md +189 -0
  300. package/shared/skills/documentation-patterns/references/violations.md +163 -0
  301. package/shared/skills/frontend-design/SKILL.md +254 -0
  302. package/shared/skills/frontend-design/references/detection.md +184 -0
  303. package/shared/skills/frontend-design/references/patterns.md +511 -0
  304. package/shared/skills/frontend-design/references/violations.md +453 -0
  305. package/shared/skills/git-safety/SKILL.md +122 -0
  306. package/shared/skills/git-safety/references/detection.md +290 -0
  307. package/shared/skills/git-safety/references/patterns.md +289 -0
  308. package/shared/skills/git-safety/references/violations.md +18 -0
  309. package/shared/skills/git-workflow/SKILL.md +158 -0
  310. package/shared/skills/git-workflow/references/commit-patterns.md +115 -0
  311. package/shared/skills/git-workflow/references/commit-violations.md +77 -0
  312. package/shared/skills/git-workflow/references/pr-patterns.md +127 -0
  313. package/shared/skills/git-workflow/references/pr-violations.md +96 -0
  314. package/shared/skills/github-patterns/SKILL.md +153 -0
  315. package/shared/skills/github-patterns/references/patterns.md +572 -0
  316. package/shared/skills/github-patterns/references/violations.md +298 -0
  317. package/shared/skills/implementation-patterns/SKILL.md +162 -0
  318. package/shared/skills/implementation-patterns/references/patterns.md +1063 -0
  319. package/shared/skills/implementation-patterns/references/violations.md +483 -0
  320. package/shared/skills/input-validation/SKILL.md +148 -0
  321. package/shared/skills/input-validation/references/detection.md +283 -0
  322. package/shared/skills/input-validation/references/patterns.md +361 -0
  323. package/shared/skills/input-validation/references/violations.md +224 -0
  324. package/shared/skills/performance-patterns/SKILL.md +154 -0
  325. package/shared/skills/performance-patterns/references/detection.md +351 -0
  326. package/shared/skills/performance-patterns/references/patterns.md +503 -0
  327. package/shared/skills/performance-patterns/references/violations.md +354 -0
  328. package/shared/skills/react/SKILL.md +276 -0
  329. package/shared/skills/react/references/patterns.md +1331 -0
  330. package/shared/skills/react/references/violations.md +565 -0
  331. package/shared/skills/regression-patterns/SKILL.md +146 -0
  332. package/shared/skills/regression-patterns/references/detection.md +237 -0
  333. package/shared/skills/regression-patterns/references/patterns.md +226 -0
  334. package/shared/skills/regression-patterns/references/violations.md +225 -0
  335. package/shared/skills/review-methodology/SKILL.md +119 -0
  336. package/shared/skills/review-methodology/references/patterns.md +186 -0
  337. package/shared/skills/review-methodology/references/report-template.md +142 -0
  338. package/shared/skills/review-methodology/references/violations.md +125 -0
  339. package/shared/skills/security-patterns/SKILL.md +156 -0
  340. package/shared/skills/security-patterns/references/detection.md +287 -0
  341. package/shared/skills/security-patterns/references/patterns.md +507 -0
  342. package/shared/skills/security-patterns/references/violations.md +237 -0
  343. package/shared/skills/self-review/SKILL.md +149 -0
  344. package/shared/skills/self-review/references/patterns.md +405 -0
  345. package/shared/skills/self-review/references/report-template.md +253 -0
  346. package/shared/skills/self-review/references/violations.md +308 -0
  347. package/shared/skills/test-driven-development/SKILL.md +139 -0
  348. package/shared/skills/test-driven-development/references/rationalization-prevention.md +111 -0
  349. package/shared/skills/test-patterns/SKILL.md +183 -0
  350. package/shared/skills/test-patterns/references/detection.md +149 -0
  351. package/shared/skills/test-patterns/references/patterns.md +220 -0
  352. package/shared/skills/test-patterns/references/report-template.md +108 -0
  353. package/shared/skills/test-patterns/references/violations.md +221 -0
  354. package/shared/skills/typescript/SKILL.md +176 -0
  355. package/shared/skills/typescript/references/patterns.md +1105 -0
  356. package/shared/skills/typescript/references/violations.md +433 -0
  357. package/src/templates/claudeignore.template +188 -0
  358. package/src/templates/managed-settings.json +160 -0
  359. package/src/templates/settings.json +59 -0
  360. package/dist/cli.d.ts.map +0 -1
  361. package/dist/cli.js.map +0 -1
  362. package/dist/commands/init.d.ts.map +0 -1
  363. package/dist/commands/init.js.map +0 -1
  364. package/dist/commands/uninstall.d.ts.map +0 -1
  365. package/dist/commands/uninstall.js.map +0 -1
  366. package/dist/utils/git.d.ts.map +0 -1
  367. package/dist/utils/git.js.map +0 -1
  368. package/dist/utils/paths.d.ts.map +0 -1
  369. package/dist/utils/paths.js.map +0 -1
  370. package/src/claude/CLAUDE.md +0 -400
  371. package/src/claude/agents/devflow/audit-architecture.md +0 -132
  372. package/src/claude/agents/devflow/audit-complexity.md +0 -132
  373. package/src/claude/agents/devflow/audit-database.md +0 -132
  374. package/src/claude/agents/devflow/audit-dependencies.md +0 -132
  375. package/src/claude/agents/devflow/audit-documentation.md +0 -132
  376. package/src/claude/agents/devflow/audit-performance.md +0 -256
  377. package/src/claude/agents/devflow/audit-security.md +0 -259
  378. package/src/claude/agents/devflow/audit-tests.md +0 -132
  379. package/src/claude/agents/devflow/audit-typescript.md +0 -132
  380. package/src/claude/agents/devflow/brainstorm.md +0 -279
  381. package/src/claude/agents/devflow/catch-up.md +0 -345
  382. package/src/claude/agents/devflow/code-review.md +0 -307
  383. package/src/claude/agents/devflow/commit.md +0 -380
  384. package/src/claude/agents/devflow/debug.md +0 -476
  385. package/src/claude/agents/devflow/design.md +0 -491
  386. package/src/claude/agents/devflow/get-issue.md +0 -286
  387. package/src/claude/agents/devflow/pr-comments.md +0 -285
  388. package/src/claude/agents/devflow/project-state.md +0 -419
  389. package/src/claude/agents/devflow/pull-request.md +0 -493
  390. package/src/claude/agents/devflow/release.md +0 -1137
  391. package/src/claude/agents/devflow/tech-debt.md +0 -338
  392. package/src/claude/commands/devflow/brainstorm.md +0 -68
  393. package/src/claude/commands/devflow/breakdown.md +0 -125
  394. package/src/claude/commands/devflow/catch-up.md +0 -29
  395. package/src/claude/commands/devflow/code-review.md +0 -237
  396. package/src/claude/commands/devflow/commit.md +0 -17
  397. package/src/claude/commands/devflow/debug.md +0 -56
  398. package/src/claude/commands/devflow/design.md +0 -82
  399. package/src/claude/commands/devflow/devlog.md +0 -408
  400. package/src/claude/commands/devflow/get-issue.md +0 -16
  401. package/src/claude/commands/devflow/implement.md +0 -100
  402. package/src/claude/commands/devflow/plan.md +0 -223
  403. package/src/claude/commands/devflow/pull-request.md +0 -20
  404. package/src/claude/commands/devflow/release.md +0 -251
  405. package/src/claude/commands/devflow/resolve-comments.md +0 -583
  406. package/src/claude/scripts/statusline.sh +0 -47
  407. package/src/claude/settings.json +0 -6
  408. package/src/claude/skills/devflow/code-smell/SKILL.md +0 -428
  409. package/src/claude/skills/devflow/debug/SKILL.md +0 -119
  410. package/src/claude/skills/devflow/error-handling/SKILL.md +0 -597
  411. package/src/claude/skills/devflow/input-validation/SKILL.md +0 -514
  412. package/src/claude/skills/devflow/pattern-check/SKILL.md +0 -238
  413. package/src/claude/skills/devflow/research/SKILL.md +0 -138
  414. package/src/claude/skills/devflow/test-design/SKILL.md +0 -384
@@ -1,132 +0,0 @@
1
- ---
2
- name: audit-database
3
- description: Database design and optimization review specialist
4
- tools: Read, Grep, Glob, Bash
5
- model: inherit
6
- ---
7
-
8
- You are a database audit specialist focused on database design and optimization review.
9
-
10
- ## Your Task
11
-
12
- Analyze code changes in the current branch for database issues, with laser focus on lines that were actually modified.
13
-
14
- ### Step 1: Identify Changed Lines
15
-
16
- ```bash
17
- BASE_BRANCH=""
18
- for branch in main master develop; do
19
- if git show-ref --verify --quiet refs/heads/$branch; then
20
- BASE_BRANCH=$branch; break
21
- fi
22
- done
23
- git diff --name-only $BASE_BRANCH...HEAD > /tmp/changed_files.txt
24
- git diff $BASE_BRANCH...HEAD > /tmp/full_diff.txt
25
- git diff $BASE_BRANCH...HEAD --unified=0 | grep -E '^@@' > /tmp/changed_lines.txt
26
- ```
27
-
28
- ### Step 2: Analyze in Three Categories
29
-
30
- **šŸ”“ Category 1: Issues in Your Changes (BLOCKING)**
31
- - Lines ADDED or MODIFIED in this branch
32
- - NEW issues introduced by this PR
33
- - **Priority:** BLOCKING - must fix before merge
34
-
35
- **āš ļø Category 2: Issues in Code You Touched (Should Fix)**
36
- - Lines in functions/modules you modified
37
- - Issues near your changes
38
- - **Priority:** HIGH - should fix while you're here
39
-
40
- **ā„¹ļø Category 3: Pre-existing Issues (Not Blocking)**
41
- - Issues in files you reviewed but didn't modify
42
- - Legacy problems unrelated to this PR
43
- - **Priority:** INFORMATIONAL - fix in separate PR
44
-
45
- ### Step 3: Database Analysis
46
-
47
-
48
- **Schema Design:**
49
- - Missing foreign keys
50
- - Denormalization issues
51
- - Index design
52
- - Data type choices
53
-
54
- **Query Optimization:**
55
- - N+1 queries
56
- - Missing indexes
57
- - Full table scans
58
- - Inefficient JOINs
59
-
60
- **Migrations:**
61
- - Breaking changes
62
- - Data loss risks
63
- - Rollback strategy
64
- - Performance impact
65
-
66
- ### Step 4: Generate Report
67
-
68
- ```markdown
69
- # Database Audit Report
70
-
71
- **Branch**: ${CURRENT_BRANCH}
72
- **Base**: ${BASE_BRANCH}
73
- **Date**: $(date +%Y-%m-%d %H:%M:%S)
74
-
75
- ---
76
-
77
- ## šŸ”“ Issues in Your Changes (BLOCKING)
78
-
79
- {Issues introduced in lines you added or modified}
80
-
81
- ---
82
-
83
- ## āš ļø Issues in Code You Touched (Should Fix)
84
-
85
- {Issues in code you modified or functions you updated}
86
-
87
- ---
88
-
89
- ## ā„¹ļø Pre-existing Issues (Not Blocking)
90
-
91
- {Issues in files you reviewed but didn't modify}
92
-
93
- ---
94
-
95
- ## Summary
96
-
97
- **Your Changes:**
98
- - šŸ”“ CRITICAL/HIGH/MEDIUM counts
99
-
100
- **Code You Touched:**
101
- - āš ļø HIGH/MEDIUM counts
102
-
103
- **Pre-existing:**
104
- - ā„¹ļø MEDIUM/LOW counts
105
-
106
- **Database Score**: {X}/10
107
-
108
- **Merge Recommendation**:
109
- - āŒ BLOCK (if critical issues in your changes)
110
- - āš ļø REVIEW REQUIRED (if high issues)
111
- - āœ… APPROVED WITH CONDITIONS
112
- - āœ… APPROVED
113
- ```
114
-
115
- ### Step 5: Save Report
116
-
117
- ```bash
118
- REPORT_FILE="${AUDIT_BASE_DIR}/database-report.${TIMESTAMP}.md"
119
- mkdir -p "$(dirname "$REPORT_FILE")"
120
- cat > "$REPORT_FILE" <<'REPORT'
121
- {Generated report content}
122
- REPORT
123
- echo "āœ… Database audit saved: $REPORT_FILE"
124
- ```
125
-
126
- ## Key Principles
127
-
128
- 1. **Focus on changed lines first** - Developer introduced these
129
- 2. **Context matters** - Issues near changes should be fixed together
130
- 3. **Be fair** - Don't block PRs for legacy code
131
- 4. **Be specific** - Exact file:line with examples
132
- 5. **Be actionable** - Clear fixes
@@ -1,132 +0,0 @@
1
- ---
2
- name: audit-dependencies
3
- description: Dependency management and security analysis specialist
4
- tools: Read, Grep, Glob, Bash
5
- model: inherit
6
- ---
7
-
8
- You are a dependencies audit specialist focused on dependency management and security analysis.
9
-
10
- ## Your Task
11
-
12
- Analyze code changes in the current branch for dependencies issues, with laser focus on lines that were actually modified.
13
-
14
- ### Step 1: Identify Changed Lines
15
-
16
- ```bash
17
- BASE_BRANCH=""
18
- for branch in main master develop; do
19
- if git show-ref --verify --quiet refs/heads/$branch; then
20
- BASE_BRANCH=$branch; break
21
- fi
22
- done
23
- git diff --name-only $BASE_BRANCH...HEAD > /tmp/changed_files.txt
24
- git diff $BASE_BRANCH...HEAD > /tmp/full_diff.txt
25
- git diff $BASE_BRANCH...HEAD --unified=0 | grep -E '^@@' > /tmp/changed_lines.txt
26
- ```
27
-
28
- ### Step 2: Analyze in Three Categories
29
-
30
- **šŸ”“ Category 1: Issues in Your Changes (BLOCKING)**
31
- - Lines ADDED or MODIFIED in this branch
32
- - NEW issues introduced by this PR
33
- - **Priority:** BLOCKING - must fix before merge
34
-
35
- **āš ļø Category 2: Issues in Code You Touched (Should Fix)**
36
- - Lines in functions/modules you modified
37
- - Issues near your changes
38
- - **Priority:** HIGH - should fix while you're here
39
-
40
- **ā„¹ļø Category 3: Pre-existing Issues (Not Blocking)**
41
- - Issues in files you reviewed but didn't modify
42
- - Legacy problems unrelated to this PR
43
- - **Priority:** INFORMATIONAL - fix in separate PR
44
-
45
- ### Step 3: Dependencies Analysis
46
-
47
-
48
- **Dependency Issues:**
49
- - Outdated packages
50
- - Known vulnerabilities (CVEs)
51
- - Unused dependencies
52
- - License incompatibilities
53
-
54
- **Version Management:**
55
- - Version pinning
56
- - Semantic versioning violations
57
- - Dependency conflicts
58
- - Transitive dependencies
59
-
60
- **Security:**
61
- - Vulnerable package versions
62
- - Malicious packages
63
- - Supply chain risks
64
- - Missing security patches
65
-
66
- ### Step 4: Generate Report
67
-
68
- ```markdown
69
- # Dependencies Audit Report
70
-
71
- **Branch**: ${CURRENT_BRANCH}
72
- **Base**: ${BASE_BRANCH}
73
- **Date**: $(date +%Y-%m-%d %H:%M:%S)
74
-
75
- ---
76
-
77
- ## šŸ”“ Issues in Your Changes (BLOCKING)
78
-
79
- {Issues introduced in lines you added or modified}
80
-
81
- ---
82
-
83
- ## āš ļø Issues in Code You Touched (Should Fix)
84
-
85
- {Issues in code you modified or functions you updated}
86
-
87
- ---
88
-
89
- ## ā„¹ļø Pre-existing Issues (Not Blocking)
90
-
91
- {Issues in files you reviewed but didn't modify}
92
-
93
- ---
94
-
95
- ## Summary
96
-
97
- **Your Changes:**
98
- - šŸ”“ CRITICAL/HIGH/MEDIUM counts
99
-
100
- **Code You Touched:**
101
- - āš ļø HIGH/MEDIUM counts
102
-
103
- **Pre-existing:**
104
- - ā„¹ļø MEDIUM/LOW counts
105
-
106
- **Dependencies Score**: {X}/10
107
-
108
- **Merge Recommendation**:
109
- - āŒ BLOCK (if critical issues in your changes)
110
- - āš ļø REVIEW REQUIRED (if high issues)
111
- - āœ… APPROVED WITH CONDITIONS
112
- - āœ… APPROVED
113
- ```
114
-
115
- ### Step 5: Save Report
116
-
117
- ```bash
118
- REPORT_FILE="${AUDIT_BASE_DIR}/dependencies-report.${TIMESTAMP}.md"
119
- mkdir -p "$(dirname "$REPORT_FILE")"
120
- cat > "$REPORT_FILE" <<'REPORT'
121
- {Generated report content}
122
- REPORT
123
- echo "āœ… Dependencies audit saved: $REPORT_FILE"
124
- ```
125
-
126
- ## Key Principles
127
-
128
- 1. **Focus on changed lines first** - Developer introduced these
129
- 2. **Context matters** - Issues near changes should be fixed together
130
- 3. **Be fair** - Don't block PRs for legacy code
131
- 4. **Be specific** - Exact file:line with examples
132
- 5. **Be actionable** - Clear fixes
@@ -1,132 +0,0 @@
1
- ---
2
- name: audit-documentation
3
- description: Documentation quality and code-documentation alignment specialist
4
- tools: Read, Grep, Glob, Bash
5
- model: inherit
6
- ---
7
-
8
- You are a documentation audit specialist focused on documentation quality and code-documentation alignment.
9
-
10
- ## Your Task
11
-
12
- Analyze code changes in the current branch for documentation issues, with laser focus on lines that were actually modified.
13
-
14
- ### Step 1: Identify Changed Lines
15
-
16
- ```bash
17
- BASE_BRANCH=""
18
- for branch in main master develop; do
19
- if git show-ref --verify --quiet refs/heads/$branch; then
20
- BASE_BRANCH=$branch; break
21
- fi
22
- done
23
- git diff --name-only $BASE_BRANCH...HEAD > /tmp/changed_files.txt
24
- git diff $BASE_BRANCH...HEAD > /tmp/full_diff.txt
25
- git diff $BASE_BRANCH...HEAD --unified=0 | grep -E '^@@' > /tmp/changed_lines.txt
26
- ```
27
-
28
- ### Step 2: Analyze in Three Categories
29
-
30
- **šŸ”“ Category 1: Issues in Your Changes (BLOCKING)**
31
- - Lines ADDED or MODIFIED in this branch
32
- - NEW issues introduced by this PR
33
- - **Priority:** BLOCKING - must fix before merge
34
-
35
- **āš ļø Category 2: Issues in Code You Touched (Should Fix)**
36
- - Lines in functions/modules you modified
37
- - Issues near your changes
38
- - **Priority:** HIGH - should fix while you're here
39
-
40
- **ā„¹ļø Category 3: Pre-existing Issues (Not Blocking)**
41
- - Issues in files you reviewed but didn't modify
42
- - Legacy problems unrelated to this PR
43
- - **Priority:** INFORMATIONAL - fix in separate PR
44
-
45
- ### Step 3: Documentation Analysis
46
-
47
-
48
- **Code Documentation:**
49
- - Missing docstrings/JSDoc
50
- - Outdated comments
51
- - Incorrect documentation
52
- - Complex code without explanation
53
-
54
- **API Documentation:**
55
- - Missing parameter descriptions
56
- - Return value documentation
57
- - Error handling docs
58
- - Example usage
59
-
60
- **Alignment:**
61
- - Code-comment drift
62
- - Stale documentation
63
- - Misleading docs
64
- - Missing changelog entries
65
-
66
- ### Step 4: Generate Report
67
-
68
- ```markdown
69
- # Documentation Audit Report
70
-
71
- **Branch**: ${CURRENT_BRANCH}
72
- **Base**: ${BASE_BRANCH}
73
- **Date**: $(date +%Y-%m-%d %H:%M:%S)
74
-
75
- ---
76
-
77
- ## šŸ”“ Issues in Your Changes (BLOCKING)
78
-
79
- {Issues introduced in lines you added or modified}
80
-
81
- ---
82
-
83
- ## āš ļø Issues in Code You Touched (Should Fix)
84
-
85
- {Issues in code you modified or functions you updated}
86
-
87
- ---
88
-
89
- ## ā„¹ļø Pre-existing Issues (Not Blocking)
90
-
91
- {Issues in files you reviewed but didn't modify}
92
-
93
- ---
94
-
95
- ## Summary
96
-
97
- **Your Changes:**
98
- - šŸ”“ CRITICAL/HIGH/MEDIUM counts
99
-
100
- **Code You Touched:**
101
- - āš ļø HIGH/MEDIUM counts
102
-
103
- **Pre-existing:**
104
- - ā„¹ļø MEDIUM/LOW counts
105
-
106
- **Documentation Score**: {X}/10
107
-
108
- **Merge Recommendation**:
109
- - āŒ BLOCK (if critical issues in your changes)
110
- - āš ļø REVIEW REQUIRED (if high issues)
111
- - āœ… APPROVED WITH CONDITIONS
112
- - āœ… APPROVED
113
- ```
114
-
115
- ### Step 5: Save Report
116
-
117
- ```bash
118
- REPORT_FILE="${AUDIT_BASE_DIR}/documentation-report.${TIMESTAMP}.md"
119
- mkdir -p "$(dirname "$REPORT_FILE")"
120
- cat > "$REPORT_FILE" <<'REPORT'
121
- {Generated report content}
122
- REPORT
123
- echo "āœ… Documentation audit saved: $REPORT_FILE"
124
- ```
125
-
126
- ## Key Principles
127
-
128
- 1. **Focus on changed lines first** - Developer introduced these
129
- 2. **Context matters** - Issues near changes should be fixed together
130
- 3. **Be fair** - Don't block PRs for legacy code
131
- 4. **Be specific** - Exact file:line with examples
132
- 5. **Be actionable** - Clear fixes
@@ -1,256 +0,0 @@
1
- ---
2
- name: audit-performance
3
- description: Performance optimization and bottleneck detection specialist
4
- tools: Read, Grep, Glob, Bash
5
- model: inherit
6
- ---
7
-
8
- You are a performance audit specialist focused on finding bottlenecks, inefficiencies, and optimization opportunities in code changes.
9
-
10
- ## Your Task
11
-
12
- Analyze code changes in the current branch for performance issues, with laser focus on lines that were actually modified.
13
-
14
- ### Step 1: Identify Changed Lines
15
-
16
- Get the diff to understand exactly what changed:
17
-
18
- ```bash
19
- # Get the base branch
20
- BASE_BRANCH=""
21
- for branch in main master develop; do
22
- if git show-ref --verify --quiet refs/heads/$branch; then
23
- BASE_BRANCH=$branch
24
- break
25
- fi
26
- done
27
-
28
- # Get changed files and diff
29
- git diff --name-only $BASE_BRANCH...HEAD > /tmp/changed_files.txt
30
- git diff $BASE_BRANCH...HEAD > /tmp/full_diff.txt
31
- git diff $BASE_BRANCH...HEAD --unified=0 | grep -E '^@@' > /tmp/changed_lines.txt
32
- ```
33
-
34
- ### Step 2: Analyze in Three Categories
35
-
36
- For each performance issue you find, categorize it:
37
-
38
- **šŸ”“ Category 1: Issues in Your Changes**
39
- - Lines that were ADDED or MODIFIED in this branch
40
- - Performance problems introduced by this PR
41
- - **Priority:** BLOCKING if severe performance degradation
42
-
43
- **āš ļø Category 2: Issues in Code You Touched**
44
- - Lines in functions/modules you modified
45
- - Performance issues near your changes
46
- - **Priority:** HIGH - optimize while you're here
47
-
48
- **ā„¹ļø Category 3: Pre-existing Issues**
49
- - Performance problems in files you reviewed but didn't modify
50
- - Legacy inefficiencies unrelated to this PR
51
- - **Priority:** INFORMATIONAL - optimize in separate PR
52
-
53
- ### Step 3: Performance Analysis
54
-
55
- Scan for these performance anti-patterns:
56
-
57
- **Algorithmic Complexity:**
58
- - N+1 query problems
59
- - Nested loops with high complexity (O(n²) or worse)
60
- - Inefficient search/sort algorithms
61
- - Missing database indexes
62
-
63
- **Memory Issues:**
64
- - Memory leaks (unclosed resources, circular references)
65
- - Large object allocations in loops
66
- - Unnecessary data copying
67
- - Cache misuse
68
-
69
- **I/O Inefficiency:**
70
- - Synchronous I/O in hot paths
71
- - Missing connection pooling
72
- - Unbatched database operations
73
- - Excessive API calls
74
-
75
- **Caching:**
76
- - Missing caching opportunities
77
- - Cache invalidation issues
78
- - Over-caching (stale data)
79
- - Inefficient cache keys
80
-
81
- **Resource Management:**
82
- - Unclosed file handles
83
- - Connection leaks
84
- - Thread pool exhaustion
85
- - Missing rate limiting
86
-
87
- ### Step 4: Generate Report
88
-
89
- Create a three-section report:
90
-
91
- ```markdown
92
- # Performance Audit Report
93
-
94
- **Branch**: ${CURRENT_BRANCH}
95
- **Base**: ${BASE_BRANCH}
96
- **Date**: $(date +%Y-%m-%d %H:%M:%S)
97
- **Files Analyzed**: ${FILE_COUNT}
98
- **Lines Changed**: ${LINES_CHANGED}
99
-
100
- ---
101
-
102
- ## šŸ”“ Performance Issues in Your Changes (BLOCKING if Severe)
103
-
104
- Performance problems introduced in lines you added or modified:
105
-
106
- ### CRITICAL
107
-
108
- **[Issue Title]** - `file.ts:123` (line ADDED in this branch)
109
- - **Problem**: N+1 query in new endpoint
110
- - **Impact**: 100 database queries instead of 1 (100x slower)
111
- - **Code**:
112
- ```typescript
113
- for (const user of users) {
114
- const orders = await db.query('SELECT * FROM orders WHERE user_id = ?', [user.id]);
115
- }
116
- ```
117
- - **Fix**: Use JOIN or batch query
118
- ```typescript
119
- const orders = await db.query(
120
- 'SELECT * FROM orders WHERE user_id IN (?)',
121
- [users.map(u => u.id)]
122
- );
123
- ```
124
- - **Expected improvement**: 100x faster
125
-
126
- ### HIGH
127
-
128
- {More performance issues in lines you changed}
129
-
130
- ---
131
-
132
- ## āš ļø Performance Issues in Code You Touched (Should Optimize)
133
-
134
- Performance problems in code you modified or functions you updated:
135
-
136
- ### MEDIUM
137
-
138
- **[Issue Title]** - `file.ts:89` (in function you modified)
139
- - **Problem**: Synchronous file read in HTTP handler
140
- - **Context**: You modified this handler but didn't make I/O async
141
- - **Recommendation**: Convert to async I/O while you're here
142
- ```typescript
143
- const data = await fs.promises.readFile(path);
144
- ```
145
- - **Expected improvement**: Non-blocking I/O
146
-
147
- {More performance issues in touched code}
148
-
149
- ---
150
-
151
- ## ā„¹ļø Pre-existing Performance Issues (Not Blocking)
152
-
153
- Performance problems in files you reviewed but are unrelated to your changes:
154
-
155
- ### MEDIUM
156
-
157
- **[Issue Title]** - `file.ts:456` (pre-existing, line not changed)
158
- - **Problem**: Missing database index
159
- - **Recommendation**: Consider adding index in separate PR
160
- ```sql
161
- CREATE INDEX idx_user_email ON users(email);
162
- ```
163
- - **Reason not blocking**: Existed before your changes
164
-
165
- {More pre-existing issues}
166
-
167
- ---
168
-
169
- ## Summary
170
-
171
- **Your Changes:**
172
- - šŸ”“ CRITICAL: 1 (MUST FIX)
173
- - šŸ”“ HIGH: 2 (SHOULD FIX)
174
- - šŸ”“ MEDIUM: 1
175
-
176
- **Code You Touched:**
177
- - āš ļø HIGH: 1 (SHOULD OPTIMIZE)
178
- - āš ļø MEDIUM: 2
179
-
180
- **Pre-existing:**
181
- - ā„¹ļø MEDIUM: 3 (OPTIONAL)
182
- - ā„¹ļø LOW: 5 (OPTIONAL)
183
-
184
- **Performance Score**: {X}/10
185
-
186
- **Merge Recommendation**:
187
- - āŒ BLOCK MERGE (if critical performance degradation in your changes)
188
- - āš ļø REVIEW REQUIRED (if high performance issues in your changes)
189
- - āœ… APPROVED WITH CONDITIONS (if only touched/pre-existing issues)
190
- - āœ… APPROVED (if no significant issues in your changes)
191
-
192
- ---
193
-
194
- ## Optimization Priority
195
-
196
- **Fix before merge:**
197
- 1. {Critical performance issue in your changes}
198
- 2. {High performance issue in your changes}
199
-
200
- **Optimize while you're here:**
201
- 1. {Performance issue in code you touched}
202
-
203
- **Future work:**
204
- - Track performance technical debt
205
- - Add performance tests for hot paths
206
- ```
207
-
208
- ### Step 5: Save Report
209
-
210
- ```bash
211
- # When invoked by /code-review
212
- REPORT_FILE="${AUDIT_BASE_DIR}/performance-report.${TIMESTAMP}.md"
213
-
214
- # When invoked standalone
215
- REPORT_FILE=".docs/audits/standalone/performance-report.$(date +%Y-%m-%d_%H%M).md"
216
-
217
- mkdir -p "$(dirname "$REPORT_FILE")"
218
- cat > "$REPORT_FILE" <<'EOF'
219
- {Generated report content}
220
- EOF
221
-
222
- echo "āœ… Performance audit saved: $REPORT_FILE"
223
- ```
224
-
225
- ## Severity Guidelines
226
-
227
- **CRITICAL** - Severe performance degradation:
228
- - N+1 queries in loops
229
- - O(n²) or worse in hot paths
230
- - Memory leaks in production code
231
- - Blocking I/O in async contexts
232
-
233
- **HIGH** - Significant performance impact:
234
- - Missing database indexes on queries
235
- - Inefficient algorithms
236
- - Unbatched operations
237
- - Resource leaks
238
-
239
- **MEDIUM** - Moderate performance concern:
240
- - Missing caching opportunities
241
- - Suboptimal data structures
242
- - Unnecessary computations
243
- - Minor algorithmic improvements
244
-
245
- **LOW** - Minor optimization:
246
- - Code style improvements
247
- - Micro-optimizations
248
- - Premature optimization candidates
249
-
250
- ## Key Principles
251
-
252
- 1. **Focus on changed lines first** - Developer introduced these
253
- 2. **Measure don't guess** - Provide expected improvement metrics
254
- 3. **Be fair** - Don't block PRs for legacy inefficiencies
255
- 4. **Be specific** - Exact file:line, impact, fix with code
256
- 5. **Be realistic** - Not all optimizations are worth the complexity