@diff-review-system/drs 3.3.1 → 4.0.1

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 (387) hide show
  1. package/.pi/agents/describe/pr-describer.md +14 -0
  2. package/.pi/agents/review/unified-reviewer.md +31 -1
  3. package/.pi/agents/task/agents-md-updater.md +26 -0
  4. package/.pi/agents/task/changelog-updater.md +29 -0
  5. package/.pi/agents/task/review-issue-fixer.md +42 -0
  6. package/.pi/agents/visual/pr-explainer.md +205 -0
  7. package/.pi/workflows/github-pr-describe.yaml +26 -0
  8. package/.pi/workflows/github-pr-fix-review-issues-stacked.yaml +148 -0
  9. package/.pi/workflows/github-pr-post-comment.yaml +19 -0
  10. package/.pi/workflows/github-pr-review-post.yaml +43 -0
  11. package/.pi/workflows/github-pr-review.yaml +364 -0
  12. package/.pi/workflows/github-pr-show-changes.yaml +25 -0
  13. package/.pi/workflows/github-pr-update-agents-md-stacked.yaml +103 -0
  14. package/.pi/workflows/github-pr-visual-explain.yaml +35 -0
  15. package/.pi/workflows/gitlab-mr-describe.yaml +24 -0
  16. package/.pi/workflows/gitlab-mr-fix-review-issues-stacked.yaml +144 -0
  17. package/.pi/workflows/gitlab-mr-post-comment.yaml +17 -0
  18. package/.pi/workflows/gitlab-mr-review.yaml +364 -0
  19. package/.pi/workflows/gitlab-mr-show-changes.yaml +23 -0
  20. package/.pi/workflows/gitlab-mr-update-agents-md-stacked.yaml +100 -0
  21. package/.pi/workflows/gitlab-mr-visual-explain.yaml +33 -0
  22. package/.pi/workflows/local-changelog-update.yaml +23 -0
  23. package/.pi/workflows/local-fix-review-issues.yaml +111 -0
  24. package/.pi/workflows/local-review.yaml +24 -0
  25. package/.pi/workflows/local-update-agents-md.yaml +24 -0
  26. package/.pi/workflows/local-visual-explain.yaml +31 -0
  27. package/.pi/workflows/release-changelog-finalize.yaml +47 -0
  28. package/.pi/workflows/tag-changelog-update.yaml +26 -0
  29. package/README.md +281 -104
  30. package/dist/ci/runner.d.ts.map +1 -1
  31. package/dist/ci/runner.js +9 -8
  32. package/dist/ci/runner.js.map +1 -1
  33. package/dist/cli/index.js +95 -325
  34. package/dist/cli/index.js.map +1 -1
  35. package/dist/cli/init.d.ts.map +1 -1
  36. package/dist/cli/init.js +25 -23
  37. package/dist/cli/init.js.map +1 -1
  38. package/dist/cli/run-agent.d.ts +26 -0
  39. package/dist/cli/run-agent.d.ts.map +1 -0
  40. package/dist/cli/run-agent.js +143 -0
  41. package/dist/cli/run-agent.js.map +1 -0
  42. package/dist/cli/workflow.d.ts +105 -0
  43. package/dist/cli/workflow.d.ts.map +1 -0
  44. package/dist/cli/workflow.js +3309 -0
  45. package/dist/cli/workflow.js.map +1 -0
  46. package/dist/github/client.d.ts +12 -0
  47. package/dist/github/client.d.ts.map +1 -1
  48. package/dist/github/client.js +27 -0
  49. package/dist/github/client.js.map +1 -1
  50. package/dist/github/platform-adapter.d.ts +6 -1
  51. package/dist/github/platform-adapter.d.ts.map +1 -1
  52. package/dist/github/platform-adapter.js +84 -8
  53. package/dist/github/platform-adapter.js.map +1 -1
  54. package/dist/gitlab/client.d.ts +11 -0
  55. package/dist/gitlab/client.d.ts.map +1 -1
  56. package/dist/gitlab/client.js +11 -0
  57. package/dist/gitlab/client.js.map +1 -1
  58. package/dist/gitlab/platform-adapter.d.ts +3 -1
  59. package/dist/gitlab/platform-adapter.d.ts.map +1 -1
  60. package/dist/gitlab/platform-adapter.js +32 -1
  61. package/dist/gitlab/platform-adapter.js.map +1 -1
  62. package/dist/lib/agent-id.d.ts +9 -0
  63. package/dist/lib/agent-id.d.ts.map +1 -0
  64. package/dist/lib/agent-id.js +32 -0
  65. package/dist/lib/agent-id.js.map +1 -0
  66. package/dist/lib/comment-formatter.d.ts +15 -1
  67. package/dist/lib/comment-formatter.d.ts.map +1 -1
  68. package/dist/lib/comment-formatter.js +53 -4
  69. package/dist/lib/comment-formatter.js.map +1 -1
  70. package/dist/lib/comment-manager.d.ts +4 -0
  71. package/dist/lib/comment-manager.d.ts.map +1 -1
  72. package/dist/lib/comment-manager.js +7 -1
  73. package/dist/lib/comment-manager.js.map +1 -1
  74. package/dist/lib/comment-poster.d.ts +2 -2
  75. package/dist/lib/comment-poster.d.ts.map +1 -1
  76. package/dist/lib/comment-poster.js +31 -4
  77. package/dist/lib/comment-poster.js.map +1 -1
  78. package/dist/lib/config.d.ts +160 -44
  79. package/dist/lib/config.d.ts.map +1 -1
  80. package/dist/lib/config.js +475 -101
  81. package/dist/lib/config.js.map +1 -1
  82. package/dist/lib/context-compression.d.ts +10 -0
  83. package/dist/lib/context-compression.d.ts.map +1 -1
  84. package/dist/lib/context-compression.js +101 -13
  85. package/dist/lib/context-compression.js.map +1 -1
  86. package/dist/lib/context-loader.d.ts +5 -4
  87. package/dist/lib/context-loader.d.ts.map +1 -1
  88. package/dist/lib/context-loader.js +79 -7
  89. package/dist/lib/context-loader.js.map +1 -1
  90. package/dist/lib/describe-core.d.ts.map +1 -1
  91. package/dist/lib/describe-core.js +3 -2
  92. package/dist/lib/describe-core.js.map +1 -1
  93. package/dist/lib/description-executor.js +1 -1
  94. package/dist/lib/description-executor.js.map +1 -1
  95. package/dist/lib/diff-lines.d.ts +18 -0
  96. package/dist/lib/diff-lines.d.ts.map +1 -0
  97. package/dist/lib/diff-lines.js +40 -0
  98. package/dist/lib/diff-lines.js.map +1 -0
  99. package/dist/lib/exit.js +4 -4
  100. package/dist/lib/exit.js.map +1 -1
  101. package/dist/lib/html-artifact.d.ts +14 -0
  102. package/dist/lib/html-artifact.d.ts.map +1 -0
  103. package/dist/lib/html-artifact.js +59 -0
  104. package/dist/lib/html-artifact.js.map +1 -0
  105. package/dist/lib/issue-parser.js +3 -3
  106. package/dist/lib/issue-parser.js.map +1 -1
  107. package/dist/lib/json-output-schema.d.ts +70 -0
  108. package/dist/lib/json-output-schema.d.ts.map +1 -1
  109. package/dist/lib/json-output-schema.js +40 -0
  110. package/dist/lib/json-output-schema.js.map +1 -1
  111. package/dist/lib/logger.d.ts +1 -1
  112. package/dist/lib/logger.d.ts.map +1 -1
  113. package/dist/lib/platform-client.d.ts +26 -0
  114. package/dist/lib/platform-client.d.ts.map +1 -1
  115. package/dist/lib/review-artifact.d.ts +69 -0
  116. package/dist/lib/review-artifact.d.ts.map +1 -0
  117. package/dist/lib/review-artifact.js +171 -0
  118. package/dist/lib/review-artifact.js.map +1 -0
  119. package/dist/lib/review-core.d.ts +6 -4
  120. package/dist/lib/review-core.d.ts.map +1 -1
  121. package/dist/lib/review-core.js +88 -173
  122. package/dist/lib/review-core.js.map +1 -1
  123. package/dist/lib/review-orchestrator.d.ts +23 -0
  124. package/dist/lib/review-orchestrator.d.ts.map +1 -1
  125. package/dist/lib/review-orchestrator.js +31 -21
  126. package/dist/lib/review-orchestrator.js.map +1 -1
  127. package/dist/lib/review-usage.d.ts +4 -0
  128. package/dist/lib/review-usage.d.ts.map +1 -1
  129. package/dist/lib/review-usage.js +25 -0
  130. package/dist/lib/review-usage.js.map +1 -1
  131. package/dist/lib/trace-collector.d.ts +105 -0
  132. package/dist/lib/trace-collector.d.ts.map +1 -0
  133. package/dist/lib/trace-collector.js +255 -0
  134. package/dist/lib/trace-collector.js.map +1 -0
  135. package/dist/lib/trace-html.d.ts +3 -0
  136. package/dist/lib/trace-html.d.ts.map +1 -0
  137. package/dist/lib/trace-html.js +349 -0
  138. package/dist/lib/trace-html.js.map +1 -0
  139. package/dist/lib/workflow-artifacts.d.ts +54 -0
  140. package/dist/lib/workflow-artifacts.d.ts.map +1 -0
  141. package/dist/lib/workflow-artifacts.js +150 -0
  142. package/dist/lib/workflow-artifacts.js.map +1 -0
  143. package/dist/pi/sdk.d.ts.map +1 -1
  144. package/dist/pi/sdk.js +605 -16
  145. package/dist/pi/sdk.js.map +1 -1
  146. package/dist/runtime/agent-loader.d.ts +10 -6
  147. package/dist/runtime/agent-loader.d.ts.map +1 -1
  148. package/dist/runtime/agent-loader.js +55 -29
  149. package/dist/runtime/agent-loader.js.map +1 -1
  150. package/dist/runtime/built-in-paths.d.ts +1 -0
  151. package/dist/runtime/built-in-paths.d.ts.map +1 -1
  152. package/dist/runtime/built-in-paths.js +7 -0
  153. package/dist/runtime/built-in-paths.js.map +1 -1
  154. package/dist/runtime/client.d.ts +14 -0
  155. package/dist/runtime/client.d.ts.map +1 -1
  156. package/dist/runtime/client.js +87 -56
  157. package/dist/runtime/client.js.map +1 -1
  158. package/dist/runtime/path-config.d.ts +2 -2
  159. package/dist/runtime/path-config.d.ts.map +1 -1
  160. package/dist/runtime/path-config.js +8 -8
  161. package/dist/runtime/path-config.js.map +1 -1
  162. package/package.json +22 -16
  163. package/.pi/agents/review/documentation.md +0 -56
  164. package/.pi/agents/review/performance.md +0 -53
  165. package/.pi/agents/review/quality.md +0 -59
  166. package/.pi/agents/review/security.md +0 -53
  167. package/.pi/agents/review/style.md +0 -132
  168. package/dist/cli/describe-mr.d.ts +0 -11
  169. package/dist/cli/describe-mr.d.ts.map +0 -1
  170. package/dist/cli/describe-mr.js +0 -134
  171. package/dist/cli/describe-mr.js.map +0 -1
  172. package/dist/cli/describe-pr.d.ts +0 -12
  173. package/dist/cli/describe-pr.d.ts.map +0 -1
  174. package/dist/cli/describe-pr.js +0 -135
  175. package/dist/cli/describe-pr.js.map +0 -1
  176. package/dist/cli/post-comments.d.ts +0 -20
  177. package/dist/cli/post-comments.d.ts.map +0 -1
  178. package/dist/cli/post-comments.js +0 -225
  179. package/dist/cli/post-comments.js.map +0 -1
  180. package/dist/cli/review-local.d.ts +0 -13
  181. package/dist/cli/review-local.d.ts.map +0 -1
  182. package/dist/cli/review-local.integration.test.d.ts +0 -2
  183. package/dist/cli/review-local.integration.test.d.ts.map +0 -1
  184. package/dist/cli/review-local.integration.test.js +0 -343
  185. package/dist/cli/review-local.integration.test.js.map +0 -1
  186. package/dist/cli/review-local.js +0 -90
  187. package/dist/cli/review-local.js.map +0 -1
  188. package/dist/cli/review-local.live.e2e.test.d.ts +0 -2
  189. package/dist/cli/review-local.live.e2e.test.d.ts.map +0 -1
  190. package/dist/cli/review-local.live.e2e.test.js +0 -153
  191. package/dist/cli/review-local.live.e2e.test.js.map +0 -1
  192. package/dist/cli/review-local.test.d.ts +0 -2
  193. package/dist/cli/review-local.test.d.ts.map +0 -1
  194. package/dist/cli/review-local.test.js +0 -164
  195. package/dist/cli/review-local.test.js.map +0 -1
  196. package/dist/cli/review-mr.d.ts +0 -22
  197. package/dist/cli/review-mr.d.ts.map +0 -1
  198. package/dist/cli/review-mr.js +0 -181
  199. package/dist/cli/review-mr.js.map +0 -1
  200. package/dist/cli/review-mr.test.d.ts +0 -2
  201. package/dist/cli/review-mr.test.d.ts.map +0 -1
  202. package/dist/cli/review-mr.test.js +0 -142
  203. package/dist/cli/review-mr.test.js.map +0 -1
  204. package/dist/cli/review-pr.d.ts +0 -22
  205. package/dist/cli/review-pr.d.ts.map +0 -1
  206. package/dist/cli/review-pr.js +0 -181
  207. package/dist/cli/review-pr.js.map +0 -1
  208. package/dist/cli/review-pr.test.d.ts +0 -2
  209. package/dist/cli/review-pr.test.d.ts.map +0 -1
  210. package/dist/cli/review-pr.test.js +0 -137
  211. package/dist/cli/review-pr.test.js.map +0 -1
  212. package/dist/cli/review-url.d.ts +0 -35
  213. package/dist/cli/review-url.d.ts.map +0 -1
  214. package/dist/cli/review-url.js +0 -110
  215. package/dist/cli/review-url.js.map +0 -1
  216. package/dist/cli/review-url.test.d.ts +0 -2
  217. package/dist/cli/review-url.test.d.ts.map +0 -1
  218. package/dist/cli/review-url.test.js +0 -132
  219. package/dist/cli/review-url.test.js.map +0 -1
  220. package/dist/cli/show-changes.d.ts +0 -15
  221. package/dist/cli/show-changes.d.ts.map +0 -1
  222. package/dist/cli/show-changes.js +0 -184
  223. package/dist/cli/show-changes.js.map +0 -1
  224. package/dist/github/client.test.d.ts +0 -2
  225. package/dist/github/client.test.d.ts.map +0 -1
  226. package/dist/github/client.test.js +0 -206
  227. package/dist/github/client.test.js.map +0 -1
  228. package/dist/github/platform-adapter.test.d.ts +0 -2
  229. package/dist/github/platform-adapter.test.d.ts.map +0 -1
  230. package/dist/github/platform-adapter.test.js +0 -40
  231. package/dist/github/platform-adapter.test.js.map +0 -1
  232. package/dist/gitlab/diff-parser.test.d.ts +0 -2
  233. package/dist/gitlab/diff-parser.test.d.ts.map +0 -1
  234. package/dist/gitlab/diff-parser.test.js +0 -315
  235. package/dist/gitlab/diff-parser.test.js.map +0 -1
  236. package/dist/gitlab/platform-adapter.test.d.ts +0 -2
  237. package/dist/gitlab/platform-adapter.test.d.ts.map +0 -1
  238. package/dist/gitlab/platform-adapter.test.js +0 -21
  239. package/dist/gitlab/platform-adapter.test.js.map +0 -1
  240. package/dist/index.test.d.ts +0 -2
  241. package/dist/index.test.d.ts.map +0 -1
  242. package/dist/index.test.js +0 -7
  243. package/dist/index.test.js.map +0 -1
  244. package/dist/lib/code-quality-report.test.d.ts +0 -2
  245. package/dist/lib/code-quality-report.test.d.ts.map +0 -1
  246. package/dist/lib/code-quality-report.test.js +0 -327
  247. package/dist/lib/code-quality-report.test.js.map +0 -1
  248. package/dist/lib/comment-formatter.test.d.ts +0 -2
  249. package/dist/lib/comment-formatter.test.d.ts.map +0 -1
  250. package/dist/lib/comment-formatter.test.js +0 -694
  251. package/dist/lib/comment-formatter.test.js.map +0 -1
  252. package/dist/lib/comment-manager.test.d.ts +0 -2
  253. package/dist/lib/comment-manager.test.d.ts.map +0 -1
  254. package/dist/lib/comment-manager.test.js +0 -680
  255. package/dist/lib/comment-manager.test.js.map +0 -1
  256. package/dist/lib/comment-poster.test.d.ts +0 -5
  257. package/dist/lib/comment-poster.test.d.ts.map +0 -1
  258. package/dist/lib/comment-poster.test.js +0 -245
  259. package/dist/lib/comment-poster.test.js.map +0 -1
  260. package/dist/lib/config-model-overrides.test.d.ts +0 -12
  261. package/dist/lib/config-model-overrides.test.d.ts.map +0 -1
  262. package/dist/lib/config-model-overrides.test.js +0 -254
  263. package/dist/lib/config-model-overrides.test.js.map +0 -1
  264. package/dist/lib/config.test.d.ts +0 -2
  265. package/dist/lib/config.test.d.ts.map +0 -1
  266. package/dist/lib/config.test.js +0 -73
  267. package/dist/lib/config.test.js.map +0 -1
  268. package/dist/lib/context-compression.test.d.ts +0 -2
  269. package/dist/lib/context-compression.test.d.ts.map +0 -1
  270. package/dist/lib/context-compression.test.js +0 -337
  271. package/dist/lib/context-compression.test.js.map +0 -1
  272. package/dist/lib/context-loader.test.d.ts +0 -2
  273. package/dist/lib/context-loader.test.d.ts.map +0 -1
  274. package/dist/lib/context-loader.test.js +0 -207
  275. package/dist/lib/context-loader.test.js.map +0 -1
  276. package/dist/lib/cursor-fix-link.test.d.ts +0 -2
  277. package/dist/lib/cursor-fix-link.test.d.ts.map +0 -1
  278. package/dist/lib/cursor-fix-link.test.js +0 -70
  279. package/dist/lib/cursor-fix-link.test.js.map +0 -1
  280. package/dist/lib/describe-core.test.d.ts +0 -2
  281. package/dist/lib/describe-core.test.d.ts.map +0 -1
  282. package/dist/lib/describe-core.test.js +0 -208
  283. package/dist/lib/describe-core.test.js.map +0 -1
  284. package/dist/lib/describe-output-path.test.d.ts +0 -2
  285. package/dist/lib/describe-output-path.test.d.ts.map +0 -1
  286. package/dist/lib/describe-output-path.test.js +0 -51
  287. package/dist/lib/describe-output-path.test.js.map +0 -1
  288. package/dist/lib/describe-parser.test.d.ts +0 -2
  289. package/dist/lib/describe-parser.test.d.ts.map +0 -1
  290. package/dist/lib/describe-parser.test.js +0 -282
  291. package/dist/lib/describe-parser.test.js.map +0 -1
  292. package/dist/lib/description-executor.test.d.ts +0 -2
  293. package/dist/lib/description-executor.test.d.ts.map +0 -1
  294. package/dist/lib/description-executor.test.js +0 -128
  295. package/dist/lib/description-executor.test.js.map +0 -1
  296. package/dist/lib/description-formatter.test.d.ts +0 -2
  297. package/dist/lib/description-formatter.test.d.ts.map +0 -1
  298. package/dist/lib/description-formatter.test.js +0 -57
  299. package/dist/lib/description-formatter.test.js.map +0 -1
  300. package/dist/lib/diff-parser.test.d.ts +0 -2
  301. package/dist/lib/diff-parser.test.d.ts.map +0 -1
  302. package/dist/lib/diff-parser.test.js +0 -335
  303. package/dist/lib/diff-parser.test.js.map +0 -1
  304. package/dist/lib/error-comment-poster.test.d.ts +0 -2
  305. package/dist/lib/error-comment-poster.test.d.ts.map +0 -1
  306. package/dist/lib/error-comment-poster.test.js +0 -128
  307. package/dist/lib/error-comment-poster.test.js.map +0 -1
  308. package/dist/lib/exit.test.d.ts +0 -2
  309. package/dist/lib/exit.test.d.ts.map +0 -1
  310. package/dist/lib/exit.test.js +0 -120
  311. package/dist/lib/exit.test.js.map +0 -1
  312. package/dist/lib/issue-parser.test.d.ts +0 -2
  313. package/dist/lib/issue-parser.test.d.ts.map +0 -1
  314. package/dist/lib/issue-parser.test.js +0 -281
  315. package/dist/lib/issue-parser.test.js.map +0 -1
  316. package/dist/lib/json-output-schema.test.d.ts +0 -2
  317. package/dist/lib/json-output-schema.test.d.ts.map +0 -1
  318. package/dist/lib/json-output-schema.test.js +0 -92
  319. package/dist/lib/json-output-schema.test.js.map +0 -1
  320. package/dist/lib/json-output.test.d.ts +0 -2
  321. package/dist/lib/json-output.test.d.ts.map +0 -1
  322. package/dist/lib/json-output.test.js +0 -141
  323. package/dist/lib/json-output.test.js.map +0 -1
  324. package/dist/lib/logger.test.d.ts +0 -2
  325. package/dist/lib/logger.test.d.ts.map +0 -1
  326. package/dist/lib/logger.test.js +0 -324
  327. package/dist/lib/logger.test.js.map +0 -1
  328. package/dist/lib/position-validator.test.d.ts +0 -2
  329. package/dist/lib/position-validator.test.d.ts.map +0 -1
  330. package/dist/lib/position-validator.test.js +0 -128
  331. package/dist/lib/position-validator.test.js.map +0 -1
  332. package/dist/lib/prompt-budget.test.d.ts +0 -2
  333. package/dist/lib/prompt-budget.test.d.ts.map +0 -1
  334. package/dist/lib/prompt-budget.test.js +0 -55
  335. package/dist/lib/prompt-budget.test.js.map +0 -1
  336. package/dist/lib/repository-validator.test.d.ts +0 -5
  337. package/dist/lib/repository-validator.test.d.ts.map +0 -1
  338. package/dist/lib/repository-validator.test.js +0 -341
  339. package/dist/lib/repository-validator.test.js.map +0 -1
  340. package/dist/lib/review-core.test.d.ts +0 -2
  341. package/dist/lib/review-core.test.d.ts.map +0 -1
  342. package/dist/lib/review-core.test.js +0 -600
  343. package/dist/lib/review-core.test.js.map +0 -1
  344. package/dist/lib/review-orchestrator.test.d.ts +0 -2
  345. package/dist/lib/review-orchestrator.test.d.ts.map +0 -1
  346. package/dist/lib/review-orchestrator.test.js +0 -531
  347. package/dist/lib/review-orchestrator.test.js.map +0 -1
  348. package/dist/lib/review-output-path.test.d.ts +0 -2
  349. package/dist/lib/review-output-path.test.d.ts.map +0 -1
  350. package/dist/lib/review-output-path.test.js +0 -83
  351. package/dist/lib/review-output-path.test.js.map +0 -1
  352. package/dist/lib/review-parser.test.d.ts +0 -2
  353. package/dist/lib/review-parser.test.d.ts.map +0 -1
  354. package/dist/lib/review-parser.test.js +0 -130
  355. package/dist/lib/review-parser.test.js.map +0 -1
  356. package/dist/lib/review-usage.test.d.ts +0 -2
  357. package/dist/lib/review-usage.test.d.ts.map +0 -1
  358. package/dist/lib/review-usage.test.js +0 -83
  359. package/dist/lib/review-usage.test.js.map +0 -1
  360. package/dist/lib/unified-review-executor.d.ts +0 -60
  361. package/dist/lib/unified-review-executor.d.ts.map +0 -1
  362. package/dist/lib/unified-review-executor.js +0 -207
  363. package/dist/lib/unified-review-executor.js.map +0 -1
  364. package/dist/lib/unified-review-executor.test.d.ts +0 -5
  365. package/dist/lib/unified-review-executor.test.d.ts.map +0 -1
  366. package/dist/lib/unified-review-executor.test.js +0 -472
  367. package/dist/lib/unified-review-executor.test.js.map +0 -1
  368. package/dist/lib/write-json-output.test.d.ts +0 -2
  369. package/dist/lib/write-json-output.test.d.ts.map +0 -1
  370. package/dist/lib/write-json-output.test.js +0 -259
  371. package/dist/lib/write-json-output.test.js.map +0 -1
  372. package/dist/pi/sdk.test.d.ts +0 -2
  373. package/dist/pi/sdk.test.d.ts.map +0 -1
  374. package/dist/pi/sdk.test.js +0 -449
  375. package/dist/pi/sdk.test.js.map +0 -1
  376. package/dist/runtime/agent-loader.test.d.ts +0 -2
  377. package/dist/runtime/agent-loader.test.d.ts.map +0 -1
  378. package/dist/runtime/agent-loader.test.js +0 -280
  379. package/dist/runtime/agent-loader.test.js.map +0 -1
  380. package/dist/runtime/client.test.d.ts +0 -2
  381. package/dist/runtime/client.test.d.ts.map +0 -1
  382. package/dist/runtime/client.test.js +0 -523
  383. package/dist/runtime/client.test.js.map +0 -1
  384. package/dist/runtime/path-config.test.d.ts +0 -2
  385. package/dist/runtime/path-config.test.d.ts.map +0 -1
  386. package/dist/runtime/path-config.test.js +0 -112
  387. package/dist/runtime/path-config.test.js.map +0 -1
@@ -1,137 +0,0 @@
1
- import { beforeEach, describe, expect, it, vi } from 'vitest';
2
- import { reviewPR } from './review-pr.js';
3
- const { mockGitHubClient, createGitHubClient, executeUnifiedReview } = vi.hoisted(() => ({
4
- mockGitHubClient: {
5
- getPullRequest: vi.fn(),
6
- getPRFiles: vi.fn(),
7
- },
8
- createGitHubClient: vi.fn(),
9
- executeUnifiedReview: vi.fn(),
10
- }));
11
- vi.mock('../github/client.js', () => ({
12
- createGitHubClient,
13
- }));
14
- vi.mock('../lib/unified-review-executor.js', () => ({
15
- executeUnifiedReview,
16
- }));
17
- const baseConfig = {
18
- pi: {},
19
- gitlab: { url: 'https://gitlab.com', token: 'token' },
20
- github: { token: 'token' },
21
- review: {
22
- agents: ['security', 'quality'],
23
- ignorePatterns: [],
24
- },
25
- };
26
- const baseOptions = {
27
- owner: 'octocat',
28
- repo: 'hello-world',
29
- prNumber: 17,
30
- postComments: true,
31
- postErrorComment: true,
32
- describe: false,
33
- postDescription: false,
34
- };
35
- describe('review-pr', () => {
36
- beforeEach(() => {
37
- vi.clearAllMocks();
38
- createGitHubClient.mockReturnValue(mockGitHubClient);
39
- executeUnifiedReview.mockResolvedValue(undefined);
40
- mockGitHubClient.getPullRequest.mockResolvedValue({
41
- number: 17,
42
- title: 'Test PR',
43
- body: 'Test description',
44
- user: { login: 'octocat' },
45
- head: {
46
- ref: 'feature/pi-migration',
47
- sha: 'head-sha',
48
- },
49
- base: {
50
- ref: 'main',
51
- },
52
- });
53
- mockGitHubClient.getPRFiles.mockResolvedValue([
54
- {
55
- filename: 'src/app.ts',
56
- status: 'modified',
57
- additions: 2,
58
- deletions: 1,
59
- changes: 3,
60
- patch: '@@ -8,2 +8,3 @@\n context line\n+new line\n-removed line',
61
- },
62
- {
63
- filename: 'src/old.ts',
64
- status: 'removed',
65
- additions: 0,
66
- deletions: 5,
67
- changes: 5,
68
- patch: '@@ -1 +0,0 @@\n-old line',
69
- },
70
- ]);
71
- });
72
- it('loads PR context once and forwards diff-aware validators to unified review', async () => {
73
- await reviewPR(baseConfig, baseOptions);
74
- expect(createGitHubClient).toHaveBeenCalledTimes(1);
75
- expect(mockGitHubClient.getPullRequest).toHaveBeenCalledWith('octocat', 'hello-world', 17);
76
- expect(mockGitHubClient.getPRFiles).toHaveBeenCalledWith('octocat', 'hello-world', 17);
77
- const unifiedOptions = vi.mocked(executeUnifiedReview).mock.calls[0][1];
78
- expect(unifiedOptions.pullRequest).toEqual(expect.objectContaining({
79
- number: 17,
80
- title: 'Test PR',
81
- headSha: 'head-sha',
82
- }));
83
- expect(unifiedOptions.changedFiles).toEqual(expect.arrayContaining([
84
- expect.objectContaining({ filename: 'src/app.ts', status: 'modified' }),
85
- expect.objectContaining({ filename: 'src/old.ts', status: 'removed' }),
86
- ]));
87
- expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 8)).toBe(true);
88
- expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 9)).toBe(true);
89
- expect(unifiedOptions.lineValidator?.isValidLine('src/app.ts', 10)).toBe(false);
90
- expect(unifiedOptions.lineValidator?.isValidLine('src/old.ts', 1)).toBe(false);
91
- const inlinePosition = unifiedOptions.createInlinePosition?.({
92
- category: 'QUALITY',
93
- severity: 'HIGH',
94
- title: 'Example issue',
95
- file: 'src/app.ts',
96
- line: 9,
97
- problem: 'Problem',
98
- solution: 'Solution',
99
- agent: 'quality',
100
- }, {
101
- head: {
102
- sha: 'ignored-sha',
103
- },
104
- });
105
- expect(inlinePosition).toEqual({
106
- path: 'src/app.ts',
107
- line: 9,
108
- commitSha: 'head-sha',
109
- });
110
- });
111
- it('rejects with actionable guidance when GITHUB_TOKEN is missing', async () => {
112
- createGitHubClient.mockImplementationOnce(() => {
113
- throw new Error('GITHUB_TOKEN environment variable is required');
114
- });
115
- await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('Set GITHUB_TOKEN');
116
- expect(executeUnifiedReview).not.toHaveBeenCalled();
117
- });
118
- it('maps GitHub authentication failures to actionable errors', async () => {
119
- const authError = Object.assign(new Error('Bad credentials'), { status: 401 });
120
- mockGitHubClient.getPullRequest.mockRejectedValueOnce(authError);
121
- await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub authentication failed for octocat/hello-world#17');
122
- expect(executeUnifiedReview).not.toHaveBeenCalled();
123
- });
124
- it('maps GitHub not found failures with repository/pr remediation', async () => {
125
- const notFoundError = Object.assign(new Error('Not Found'), { status: 404 });
126
- mockGitHubClient.getPullRequest.mockRejectedValueOnce(notFoundError);
127
- await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub pull request not found: octocat/hello-world#17');
128
- expect(executeUnifiedReview).not.toHaveBeenCalled();
129
- });
130
- it('maps GitHub rate limit failures with retry guidance', async () => {
131
- const rateLimitError = Object.assign(new Error('API rate limit exceeded'), { status: 403 });
132
- mockGitHubClient.getPRFiles.mockRejectedValueOnce(rateLimitError);
133
- await expect(reviewPR(baseConfig, baseOptions)).rejects.toThrow('GitHub API rate limit reached while loading octocat/hello-world#17');
134
- expect(executeUnifiedReview).not.toHaveBeenCalled();
135
- });
136
- });
137
- //# sourceMappingURL=review-pr.test.js.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"review-pr.test.js","sourceRoot":"","sources":["../../src/cli/review-pr.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAI9D,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAE1C,MAAM,EAAE,gBAAgB,EAAE,kBAAkB,EAAE,oBAAoB,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IACvF,gBAAgB,EAAE;QAChB,cAAc,EAAE,EAAE,CAAC,EAAE,EAAE;QACvB,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE;KACpB;IACD,kBAAkB,EAAE,EAAE,CAAC,EAAE,EAAE;IAC3B,oBAAoB,EAAE,EAAE,CAAC,EAAE,EAAE;CAC9B,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,qBAAqB,EAAE,GAAG,EAAE,CAAC,CAAC;IACpC,kBAAkB;CACnB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,mCAAmC,EAAE,GAAG,EAAE,CAAC,CAAC;IAClD,oBAAoB;CACrB,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,oBAAoB,EAAE,KAAK,EAAE,OAAO,EAAE;IACrD,MAAM,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;IAC1B,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,EAAE;KACnB;CACsB,CAAC;AAE1B,MAAM,WAAW,GAAG;IAClB,KAAK,EAAE,SAAS;IAChB,IAAI,EAAE,aAAa;IACnB,QAAQ,EAAE,EAAE;IACZ,YAAY,EAAE,IAAI;IAClB,gBAAgB,EAAE,IAAI;IACtB,QAAQ,EAAE,KAAK;IACf,eAAe,EAAE,KAAK;CACvB,CAAC;AAEF,QAAQ,CAAC,WAAW,EAAE,GAAG,EAAE;IACzB,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QAEnB,kBAAkB,CAAC,eAAe,CAAC,gBAAgB,CAAC,CAAC;QACrD,oBAAoB,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAElD,gBAAgB,CAAC,cAAc,CAAC,iBAAiB,CAAC;YAChD,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,kBAAkB;YACxB,IAAI,EAAE,EAAE,KAAK,EAAE,SAAS,EAAE;YAC1B,IAAI,EAAE;gBACJ,GAAG,EAAE,sBAAsB;gBAC3B,GAAG,EAAE,UAAU;aAChB;YACD,IAAI,EAAE;gBACJ,GAAG,EAAE,MAAM;aACZ;SACF,CAAC,CAAC;QAEH,gBAAgB,CAAC,UAAU,CAAC,iBAAiB,CAAC;YAC5C;gBACE,QAAQ,EAAE,YAAY;gBACtB,MAAM,EAAE,UAAU;gBAClB,SAAS,EAAE,CAAC;gBACZ,SAAS,EAAE,CAAC;gBACZ,OAAO,EAAE,CAAC;gBACV,KAAK,EAAE,0DAA0D;aAClE;YACD;gBACE,QAAQ,EAAE,YAAY;gBACtB,MAAM,EAAE,SAAS;gBACjB,SAAS,EAAE,CAAC;gBACZ,SAAS,EAAE,CAAC;gBACZ,OAAO,EAAE,CAAC;gBACV,KAAK,EAAE,0BAA0B;aAClC;SACF,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4EAA4E,EAAE,KAAK,IAAI,EAAE;QAC1F,MAAM,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QAExC,MAAM,CAAC,kBAAkB,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,gBAAgB,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,SAAS,EAAE,aAAa,EAAE,EAAE,CAAC,CAAC;QAC3F,MAAM,CAAC,gBAAgB,CAAC,UAAU,CAAC,CAAC,oBAAoB,CAAC,SAAS,EAAE,aAAa,EAAE,EAAE,CAAC,CAAC;QAEvF,MAAM,cAAc,GAAG,EAAE,CAAC,MAAM,CAAC,oBAAoB,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC;QAEhG,MAAM,CAAC,cAAc,CAAC,WAAW,CAAC,CAAC,OAAO,CACxC,MAAM,CAAC,gBAAgB,CAAC;YACtB,MAAM,EAAE,EAAE;YACV,KAAK,EAAE,SAAS;YAChB,OAAO,EAAE,UAAU;SACpB,CAAC,CACH,CAAC;QACF,MAAM,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC,OAAO,CACzC,MAAM,CAAC,eAAe,CAAC;YACrB,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;YACvE,MAAM,CAAC,gBAAgB,CAAC,EAAE,QAAQ,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,EAAE,CAAC;SACvE,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC9E,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAChF,MAAM,CAAC,cAAc,CAAC,aAAa,EAAE,WAAW,CAAC,YAAY,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAE/E,MAAM,cAAc,GAAG,cAAc,CAAC,oBAAoB,EAAE,CAC1D;YACE,QAAQ,EAAE,SAAS;YACnB,QAAQ,EAAE,MAAM;YAChB,KAAK,EAAE,eAAe;YACtB,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,OAAO,EAAE,SAAS;YAClB,QAAQ,EAAE,UAAU;YACpB,KAAK,EAAE,SAAS;SACF,EAChB;YACE,IAAI,EAAE;gBACJ,GAAG,EAAE,aAAa;aACnB;SACF,CACF,CAAC;QAEF,MAAM,CAAC,cAAc,CAAC,CAAC,OAAO,CAAC;YAC7B,IAAI,EAAE,YAAY;YAClB,IAAI,EAAE,CAAC;YACP,SAAS,EAAE,UAAU;SACtB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,kBAAkB,CAAC,sBAAsB,CAAC,GAAG,EAAE;YAC7C,MAAM,IAAI,KAAK,CAAC,+CAA+C,CAAC,CAAC;QACnE,CAAC,CAAC,CAAC;QAEH,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,kBAAkB,CAAC,CAAC;QACpF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,0DAA0D,EAAE,KAAK,IAAI,EAAE;QACxE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,iBAAiB,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC/E,gBAAgB,CAAC,cAAc,CAAC,qBAAqB,CAAC,SAAS,CAAC,CAAC;QAEjE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,yDAAyD,CAC1D,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,aAAa,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,WAAW,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC7E,gBAAgB,CAAC,cAAc,CAAC,qBAAqB,CAAC,aAAa,CAAC,CAAC;QAErE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,uDAAuD,CACxD,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;QACnE,MAAM,cAAc,GAAG,MAAM,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,yBAAyB,CAAC,EAAE,EAAE,MAAM,EAAE,GAAG,EAAE,CAAC,CAAC;QAC5F,gBAAgB,CAAC,UAAU,CAAC,qBAAqB,CAAC,cAAc,CAAC,CAAC;QAElE,MAAM,MAAM,CAAC,QAAQ,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC7D,oEAAoE,CACrE,CAAC;QACF,MAAM,CAAC,oBAAoB,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
@@ -1,35 +0,0 @@
1
- import type { DRSConfig } from '../lib/config.js';
2
- export interface ReviewUrlOptions {
3
- url: string;
4
- postComments: boolean;
5
- postErrorComment: boolean;
6
- fixInCursor?: boolean;
7
- skipFixInCursor?: boolean;
8
- describe: boolean;
9
- postDescription: boolean;
10
- codeQualityReport?: string;
11
- outputPath?: string;
12
- jsonOutput?: boolean;
13
- baseBranch?: string;
14
- debug?: boolean;
15
- thinkingLevel?: string;
16
- }
17
- export type ParsedReviewUrl = {
18
- platform: 'github';
19
- owner: string;
20
- repo: string;
21
- prNumber: number;
22
- } | {
23
- platform: 'gitlab';
24
- projectId: string;
25
- mrIid: number;
26
- };
27
- /**
28
- * Parse a GitHub or GitLab PR/MR URL into platform-specific identifiers.
29
- */
30
- export declare function parseReviewUrl(url: string): ParsedReviewUrl;
31
- /**
32
- * Route review execution based on PR/MR URL.
33
- */
34
- export declare function reviewByUrl(config: DRSConfig, options: ReviewUrlOptions): Promise<void>;
35
- //# sourceMappingURL=review-url.d.ts.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"review-url.d.ts","sourceRoot":"","sources":["../../src/cli/review-url.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAIlD,MAAM,WAAW,gBAAgB;IAC/B,GAAG,EAAE,MAAM,CAAC;IACZ,YAAY,EAAE,OAAO,CAAC;IACtB,gBAAgB,EAAE,OAAO,CAAC;IAC1B,WAAW,CAAC,EAAE,OAAO,CAAC;IACtB,eAAe,CAAC,EAAE,OAAO,CAAC;IAC1B,QAAQ,EAAE,OAAO,CAAC;IAClB,eAAe,EAAE,OAAO,CAAC;IACzB,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,KAAK,CAAC,EAAE,OAAO,CAAC;IAChB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,MAAM,MAAM,eAAe,GACvB;IACE,QAAQ,EAAE,QAAQ,CAAC;IACnB,KAAK,EAAE,MAAM,CAAC;IACd,IAAI,EAAE,MAAM,CAAC;IACb,QAAQ,EAAE,MAAM,CAAC;CAClB,GACD;IACE,QAAQ,EAAE,QAAQ,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,EAAE,MAAM,CAAC;CACf,CAAC;AAaN;;GAEG;AACH,wBAAgB,cAAc,CAAC,GAAG,EAAE,MAAM,GAAG,eAAe,CAkE3D;AAED;;GAEG;AACH,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CA+C7F"}
@@ -1,110 +0,0 @@
1
- import chalk from 'chalk';
2
- import { reviewMR } from './review-mr.js';
3
- import { reviewPR } from './review-pr.js';
4
- function sanitizeUrlForError(url) {
5
- try {
6
- const parsed = new URL(url);
7
- parsed.username = '';
8
- parsed.password = '';
9
- return `${parsed.origin}${parsed.pathname}`;
10
- }
11
- catch {
12
- return url.replace(/:\/\/[^@/\s]+@/g, '://').replace(/[?#].*$/, '');
13
- }
14
- }
15
- /**
16
- * Parse a GitHub or GitLab PR/MR URL into platform-specific identifiers.
17
- */
18
- export function parseReviewUrl(url) {
19
- const safeUrl = sanitizeUrlForError(url);
20
- let parsedUrl;
21
- try {
22
- parsedUrl = new URL(url);
23
- }
24
- catch {
25
- throw new Error(`Invalid URL: ${safeUrl}`);
26
- }
27
- const pathParts = parsedUrl.pathname.split('/').filter(Boolean);
28
- // GitHub: /owner/repo/pull/<number>
29
- if (pathParts.length >= 4 && pathParts[2] === 'pull') {
30
- const owner = pathParts[0];
31
- const repo = pathParts[1];
32
- const prNumber = parseInt(pathParts[3] ?? '', 10);
33
- if (!owner || !repo || !Number.isSafeInteger(prNumber) || prNumber <= 0) {
34
- throw new Error(`Invalid GitHub PR URL format: ${safeUrl}. Expected https://<host>/<owner>/<repo>/pull/<number>.`);
35
- }
36
- return {
37
- platform: 'github',
38
- owner,
39
- repo,
40
- prNumber,
41
- };
42
- }
43
- // GitLab: /group/subgroup/repo/-/merge_requests/<number>
44
- const mergeRequestIndex = pathParts.indexOf('merge_requests');
45
- if (mergeRequestIndex >= 0) {
46
- if (mergeRequestIndex < 2 || pathParts[mergeRequestIndex - 1] !== '-') {
47
- throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Expected .../<group>/<repo>/-/merge_requests/<number>.`);
48
- }
49
- const mrIid = parseInt(pathParts[mergeRequestIndex + 1] ?? '', 10);
50
- if (!Number.isSafeInteger(mrIid) || mrIid <= 0) {
51
- throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Merge request IID must be a positive integer.`);
52
- }
53
- const repo = pathParts[mergeRequestIndex - 2];
54
- const ownerParts = pathParts.slice(0, mergeRequestIndex - 2);
55
- if (!repo || ownerParts.length === 0) {
56
- throw new Error(`Invalid GitLab MR URL format: ${safeUrl}. Expected .../<group>/<repo>/-/merge_requests/<number>.`);
57
- }
58
- return {
59
- platform: 'gitlab',
60
- projectId: `${ownerParts.join('/')}/${repo}`,
61
- mrIid,
62
- };
63
- }
64
- throw new Error(`Unsupported review URL: ${safeUrl}. Expected a GitHub pull request URL (.../pull/<number>) or GitLab merge request URL (.../-/merge_requests/<number>).`);
65
- }
66
- /**
67
- * Route review execution based on PR/MR URL.
68
- */
69
- export async function reviewByUrl(config, options) {
70
- const parsed = parseReviewUrl(options.url);
71
- if (parsed.platform === 'github') {
72
- if (options.codeQualityReport) {
73
- console.warn(chalk.yellow('⚠ --code-quality-report is only supported for GitLab MRs. Ignoring this option for GitHub PR reviews.'));
74
- }
75
- await reviewPR(config, {
76
- owner: parsed.owner,
77
- repo: parsed.repo,
78
- prNumber: parsed.prNumber,
79
- postComments: options.postComments,
80
- postErrorComment: options.postErrorComment,
81
- fixInCursor: options.fixInCursor,
82
- skipFixInCursor: options.skipFixInCursor,
83
- describe: options.describe,
84
- postDescription: options.postDescription,
85
- outputPath: options.outputPath,
86
- jsonOutput: options.jsonOutput,
87
- baseBranch: options.baseBranch,
88
- debug: options.debug,
89
- thinkingLevel: options.thinkingLevel,
90
- });
91
- return;
92
- }
93
- await reviewMR(config, {
94
- projectId: parsed.projectId,
95
- mrIid: parsed.mrIid,
96
- postComments: options.postComments,
97
- postErrorComment: options.postErrorComment,
98
- fixInCursor: options.fixInCursor,
99
- skipFixInCursor: options.skipFixInCursor,
100
- describe: options.describe,
101
- postDescription: options.postDescription,
102
- codeQualityReport: options.codeQualityReport,
103
- outputPath: options.outputPath,
104
- jsonOutput: options.jsonOutput,
105
- baseBranch: options.baseBranch,
106
- debug: options.debug,
107
- thinkingLevel: options.thinkingLevel,
108
- });
109
- }
110
- //# sourceMappingURL=review-url.js.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"review-url.js","sourceRoot":"","sources":["../../src/cli/review-url.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AAC1C,OAAO,EAAE,QAAQ,EAAE,MAAM,gBAAgB,CAAC;AA+B1C,SAAS,mBAAmB,CAAC,GAAW;IACtC,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;QAC5B,MAAM,CAAC,QAAQ,GAAG,EAAE,CAAC;QACrB,MAAM,CAAC,QAAQ,GAAG,EAAE,CAAC;QACrB,OAAO,GAAG,MAAM,CAAC,MAAM,GAAG,MAAM,CAAC,QAAQ,EAAE,CAAC;IAC9C,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,GAAG,CAAC,OAAO,CAAC,iBAAiB,EAAE,KAAK,CAAC,CAAC,OAAO,CAAC,SAAS,EAAE,EAAE,CAAC,CAAC;IACtE,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,cAAc,CAAC,GAAW;IACxC,MAAM,OAAO,GAAG,mBAAmB,CAAC,GAAG,CAAC,CAAC;IAEzC,IAAI,SAAc,CAAC;IACnB,IAAI,CAAC;QACH,SAAS,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;IAC3B,CAAC;IAAC,MAAM,CAAC;QACP,MAAM,IAAI,KAAK,CAAC,gBAAgB,OAAO,EAAE,CAAC,CAAC;IAC7C,CAAC;IAED,MAAM,SAAS,GAAG,SAAS,CAAC,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;IAEhE,oCAAoC;IACpC,IAAI,SAAS,CAAC,MAAM,IAAI,CAAC,IAAI,SAAS,CAAC,CAAC,CAAC,KAAK,MAAM,EAAE,CAAC;QACrD,MAAM,KAAK,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;QAC3B,MAAM,IAAI,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;QAC1B,MAAM,QAAQ,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;QAElD,IAAI,CAAC,KAAK,IAAI,CAAC,IAAI,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,QAAQ,CAAC,IAAI,QAAQ,IAAI,CAAC,EAAE,CAAC;YACxE,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,yDAAyD,CAClG,CAAC;QACJ,CAAC;QAED,OAAO;YACL,QAAQ,EAAE,QAAQ;YAClB,KAAK;YACL,IAAI;YACJ,QAAQ;SACT,CAAC;IACJ,CAAC;IAED,yDAAyD;IACzD,MAAM,iBAAiB,GAAG,SAAS,CAAC,OAAO,CAAC,gBAAgB,CAAC,CAAC;IAC9D,IAAI,iBAAiB,IAAI,CAAC,EAAE,CAAC;QAC3B,IAAI,iBAAiB,GAAG,CAAC,IAAI,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;YACtE,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,0DAA0D,CACnG,CAAC;QACJ,CAAC;QAED,MAAM,KAAK,GAAG,QAAQ,CAAC,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;QACnE,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,KAAK,CAAC,IAAI,KAAK,IAAI,CAAC,EAAE,CAAC;YAC/C,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,iDAAiD,CAC1F,CAAC;QACJ,CAAC;QAED,MAAM,IAAI,GAAG,SAAS,CAAC,iBAAiB,GAAG,CAAC,CAAC,CAAC;QAC9C,MAAM,UAAU,GAAG,SAAS,CAAC,KAAK,CAAC,CAAC,EAAE,iBAAiB,GAAG,CAAC,CAAC,CAAC;QAC7D,IAAI,CAAC,IAAI,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACrC,MAAM,IAAI,KAAK,CACb,iCAAiC,OAAO,0DAA0D,CACnG,CAAC;QACJ,CAAC;QAED,OAAO;YACL,QAAQ,EAAE,QAAQ;YAClB,SAAS,EAAE,GAAG,UAAU,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,IAAI,EAAE;YAC5C,KAAK;SACN,CAAC;IACJ,CAAC;IAED,MAAM,IAAI,KAAK,CACb,2BAA2B,OAAO,uHAAuH,CAC1J,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,MAAiB,EAAE,OAAyB;IAC5E,MAAM,MAAM,GAAG,cAAc,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;IAE3C,IAAI,MAAM,CAAC,QAAQ,KAAK,QAAQ,EAAE,CAAC;QACjC,IAAI,OAAO,CAAC,iBAAiB,EAAE,CAAC;YAC9B,OAAO,CAAC,IAAI,CACV,KAAK,CAAC,MAAM,CACV,uGAAuG,CACxG,CACF,CAAC;QACJ,CAAC;QAED,MAAM,QAAQ,CAAC,MAAM,EAAE;YACrB,KAAK,EAAE,MAAM,CAAC,KAAK;YACnB,IAAI,EAAE,MAAM,CAAC,IAAI;YACjB,QAAQ,EAAE,MAAM,CAAC,QAAQ;YACzB,YAAY,EAAE,OAAO,CAAC,YAAY;YAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;YAC1C,WAAW,EAAE,OAAO,CAAC,WAAW;YAChC,eAAe,EAAE,OAAO,CAAC,eAAe;YACxC,QAAQ,EAAE,OAAO,CAAC,QAAQ;YAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;YACxC,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,KAAK,EAAE,OAAO,CAAC,KAAK;YACpB,aAAa,EAAE,OAAO,CAAC,aAAa;SACrC,CAAC,CAAC;QACH,OAAO;IACT,CAAC;IAED,MAAM,QAAQ,CAAC,MAAM,EAAE;QACrB,SAAS,EAAE,MAAM,CAAC,SAAS;QAC3B,KAAK,EAAE,MAAM,CAAC,KAAK;QACnB,YAAY,EAAE,OAAO,CAAC,YAAY;QAClC,gBAAgB,EAAE,OAAO,CAAC,gBAAgB;QAC1C,WAAW,EAAE,OAAO,CAAC,WAAW;QAChC,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,QAAQ,EAAE,OAAO,CAAC,QAAQ;QAC1B,eAAe,EAAE,OAAO,CAAC,eAAe;QACxC,iBAAiB,EAAE,OAAO,CAAC,iBAAiB;QAC5C,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,UAAU,EAAE,OAAO,CAAC,UAAU;QAC9B,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,aAAa,EAAE,OAAO,CAAC,aAAa;KACrC,CAAC,CAAC;AACL,CAAC"}
@@ -1,2 +0,0 @@
1
- export {};
2
- //# sourceMappingURL=review-url.test.d.ts.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"review-url.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-url.test.ts"],"names":[],"mappings":""}
@@ -1,132 +0,0 @@
1
- import { beforeEach, describe, expect, it, vi } from 'vitest';
2
- import { parseReviewUrl, reviewByUrl } from './review-url.js';
3
- const { reviewPR, reviewMR } = vi.hoisted(() => ({
4
- reviewPR: vi.fn(),
5
- reviewMR: vi.fn(),
6
- }));
7
- vi.mock('./review-pr.js', () => ({
8
- reviewPR,
9
- }));
10
- vi.mock('./review-mr.js', () => ({
11
- reviewMR,
12
- }));
13
- const baseConfig = {
14
- pi: {},
15
- gitlab: { url: 'https://gitlab.com', token: 'token' },
16
- github: { token: 'token' },
17
- review: {
18
- agents: ['security', 'quality'],
19
- ignorePatterns: [],
20
- },
21
- };
22
- describe('parseReviewUrl', () => {
23
- it('parses GitHub pull request URLs', () => {
24
- const parsed = parseReviewUrl('https://github.com/octocat/hello-world/pull/123');
25
- expect(parsed).toEqual({
26
- platform: 'github',
27
- owner: 'octocat',
28
- repo: 'hello-world',
29
- prNumber: 123,
30
- });
31
- });
32
- it('parses GitLab merge request URLs with subgroups', () => {
33
- const parsed = parseReviewUrl('https://gitlab.example.com/org/security/tools/drs/-/merge_requests/42');
34
- expect(parsed).toEqual({
35
- platform: 'gitlab',
36
- projectId: 'org/security/tools/drs',
37
- mrIid: 42,
38
- });
39
- });
40
- it('throws for unsupported URLs', () => {
41
- expect(() => parseReviewUrl('https://example.com/org/repo/issues/10')).toThrow('Unsupported review URL');
42
- });
43
- it('redacts URL credentials in error messages', () => {
44
- let message = '';
45
- try {
46
- parseReviewUrl('https://alice:super-secret-token@example.com/not-a-pr');
47
- }
48
- catch (error) {
49
- message = error instanceof Error ? error.message : String(error);
50
- }
51
- expect(message).toContain('Unsupported review URL: https://example.com/not-a-pr');
52
- expect(message).not.toContain('alice');
53
- expect(message).not.toContain('super-secret-token');
54
- });
55
- });
56
- describe('reviewByUrl', () => {
57
- beforeEach(() => {
58
- vi.clearAllMocks();
59
- reviewPR.mockResolvedValue(undefined);
60
- reviewMR.mockResolvedValue(undefined);
61
- });
62
- it('routes GitHub URLs to reviewPR', async () => {
63
- await reviewByUrl(baseConfig, {
64
- url: 'https://github.com/octocat/hello-world/pull/10',
65
- postComments: true,
66
- postErrorComment: true,
67
- describe: true,
68
- postDescription: false,
69
- outputPath: 'review.json',
70
- jsonOutput: true,
71
- baseBranch: 'release/2026-01',
72
- debug: true,
73
- });
74
- expect(reviewPR).toHaveBeenCalledWith(baseConfig, {
75
- owner: 'octocat',
76
- repo: 'hello-world',
77
- prNumber: 10,
78
- postComments: true,
79
- postErrorComment: true,
80
- describe: true,
81
- postDescription: false,
82
- outputPath: 'review.json',
83
- jsonOutput: true,
84
- baseBranch: 'release/2026-01',
85
- debug: true,
86
- });
87
- expect(reviewMR).not.toHaveBeenCalled();
88
- });
89
- it('routes GitLab URLs to reviewMR', async () => {
90
- await reviewByUrl(baseConfig, {
91
- url: 'https://gitlab.com/group/subgroup/repo/-/merge_requests/88',
92
- postComments: false,
93
- postErrorComment: false,
94
- describe: false,
95
- postDescription: false,
96
- codeQualityReport: 'gl-code-quality-report.json',
97
- outputPath: 'review.json',
98
- jsonOutput: false,
99
- baseBranch: 'main',
100
- debug: false,
101
- });
102
- expect(reviewMR).toHaveBeenCalledWith(baseConfig, {
103
- projectId: 'group/subgroup/repo',
104
- mrIid: 88,
105
- postComments: false,
106
- postErrorComment: false,
107
- describe: false,
108
- postDescription: false,
109
- codeQualityReport: 'gl-code-quality-report.json',
110
- outputPath: 'review.json',
111
- jsonOutput: false,
112
- baseBranch: 'main',
113
- debug: false,
114
- });
115
- expect(reviewPR).not.toHaveBeenCalled();
116
- });
117
- it('warns and ignores --code-quality-report for GitHub URLs', async () => {
118
- const warnSpy = vi.spyOn(console, 'warn').mockImplementation(() => undefined);
119
- await reviewByUrl(baseConfig, {
120
- url: 'https://github.com/octocat/hello-world/pull/9',
121
- postComments: false,
122
- postErrorComment: false,
123
- describe: false,
124
- postDescription: false,
125
- codeQualityReport: 'gl-code-quality-report.json',
126
- jsonOutput: false,
127
- });
128
- expect(warnSpy).toHaveBeenCalledWith(expect.stringContaining('--code-quality-report is only supported for GitLab MRs'));
129
- warnSpy.mockRestore();
130
- });
131
- });
132
- //# sourceMappingURL=review-url.test.js.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"review-url.test.js","sourceRoot":"","sources":["../../src/cli/review-url.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,OAAO,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,iBAAiB,CAAC;AAE9D,MAAM,EAAE,QAAQ,EAAE,QAAQ,EAAE,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAC/C,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;IACjB,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC/B,QAAQ;CACT,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE,CAAC,CAAC;IAC/B,QAAQ;CACT,CAAC,CAAC,CAAC;AAEJ,MAAM,UAAU,GAAG;IACjB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,oBAAoB,EAAE,KAAK,EAAE,OAAO,EAAE;IACrD,MAAM,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE;IAC1B,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;QAC/B,cAAc,EAAE,EAAE;KACnB;CACsB,CAAC;AAE1B,QAAQ,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,EAAE,CAAC,iCAAiC,EAAE,GAAG,EAAE;QACzC,MAAM,MAAM,GAAG,cAAc,CAAC,iDAAiD,CAAC,CAAC;QAEjF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,QAAQ,EAAE,QAAQ;YAClB,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,aAAa;YACnB,QAAQ,EAAE,GAAG;SACd,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,GAAG,EAAE;QACzD,MAAM,MAAM,GAAG,cAAc,CAC3B,uEAAuE,CACxE,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,CAAC,OAAO,CAAC;YACrB,QAAQ,EAAE,QAAQ;YAClB,SAAS,EAAE,wBAAwB;YACnC,KAAK,EAAE,EAAE;SACV,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6BAA6B,EAAE,GAAG,EAAE;QACrC,MAAM,CAAC,GAAG,EAAE,CAAC,cAAc,CAAC,wCAAwC,CAAC,CAAC,CAAC,OAAO,CAC5E,wBAAwB,CACzB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2CAA2C,EAAE,GAAG,EAAE;QACnD,IAAI,OAAO,GAAG,EAAE,CAAC;QAEjB,IAAI,CAAC;YACH,cAAc,CAAC,uDAAuD,CAAC,CAAC;QAC1E,CAAC;QAAC,OAAO,KAAK,EAAE,CAAC;YACf,OAAO,GAAG,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC;QACnE,CAAC;QAED,MAAM,CAAC,OAAO,CAAC,CAAC,SAAS,CAAC,sDAAsD,CAAC,CAAC;QAClF,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,OAAO,CAAC,CAAC;QACvC,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,oBAAoB,CAAC,CAAC;IACtD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;IAC3B,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,QAAQ,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QACtC,QAAQ,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,gDAAgD;YACrD,YAAY,EAAE,IAAI;YAClB,gBAAgB,EAAE,IAAI;YACtB,QAAQ,EAAE,IAAI;YACd,eAAe,EAAE,KAAK;YACtB,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,IAAI;YAChB,UAAU,EAAE,iBAAiB;YAC7B,KAAK,EAAE,IAAI;SACZ,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,oBAAoB,CAAC,UAAU,EAAE;YAChD,KAAK,EAAE,SAAS;YAChB,IAAI,EAAE,aAAa;YACnB,QAAQ,EAAE,EAAE;YACZ,YAAY,EAAE,IAAI;YAClB,gBAAgB,EAAE,IAAI;YACtB,QAAQ,EAAE,IAAI;YACd,eAAe,EAAE,KAAK;YACtB,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,IAAI;YAChB,UAAU,EAAE,iBAAiB;YAC7B,KAAK,EAAE,IAAI;SACZ,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,4DAA4D;YACjE,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,KAAK;YACjB,UAAU,EAAE,MAAM;YAClB,KAAK,EAAE,KAAK;SACb,CAAC,CAAC;QAEH,MAAM,CAAC,QAAQ,CAAC,CAAC,oBAAoB,CAAC,UAAU,EAAE;YAChD,SAAS,EAAE,qBAAqB;YAChC,KAAK,EAAE,EAAE;YACT,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,aAAa;YACzB,UAAU,EAAE,KAAK;YACjB,UAAU,EAAE,MAAM;YAClB,KAAK,EAAE,KAAK;SACb,CAAC,CAAC;QACH,MAAM,CAAC,QAAQ,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC,CAAC;QAE9E,MAAM,WAAW,CAAC,UAAU,EAAE;YAC5B,GAAG,EAAE,+CAA+C;YACpD,YAAY,EAAE,KAAK;YACnB,gBAAgB,EAAE,KAAK;YACvB,QAAQ,EAAE,KAAK;YACf,eAAe,EAAE,KAAK;YACtB,iBAAiB,EAAE,6BAA6B;YAChD,UAAU,EAAE,KAAK;SAClB,CAAC,CAAC;QAEH,MAAM,CAAC,OAAO,CAAC,CAAC,oBAAoB,CAClC,MAAM,CAAC,gBAAgB,CAAC,wDAAwD,CAAC,CAClF,CAAC;QAEF,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
@@ -1,15 +0,0 @@
1
- import type { DRSConfig } from '../lib/config.js';
2
- export interface ShowChangesOptions {
3
- owner?: string;
4
- repo?: string;
5
- prNumber?: number;
6
- projectId?: string;
7
- mrIid?: number;
8
- baseBranch?: string;
9
- file?: string;
10
- outputPath?: string;
11
- jsonOutput?: boolean;
12
- workingDir?: string;
13
- }
14
- export declare function showChanges(config: DRSConfig, options: ShowChangesOptions): Promise<void>;
15
- //# sourceMappingURL=show-changes.d.ts.map
@@ -1 +0,0 @@
1
- {"version":3,"file":"show-changes.d.ts","sourceRoot":"","sources":["../../src/cli/show-changes.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAQlD,MAAM,WAAW,kBAAkB;IACjC,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,OAAO,CAAC;IACrB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAyGD,wBAAsB,WAAW,CAAC,MAAM,EAAE,SAAS,EAAE,OAAO,EAAE,kBAAkB,GAAG,OAAO,CAAC,IAAI,CAAC,CAqH/F"}
@@ -1,184 +0,0 @@
1
- import chalk from 'chalk';
2
- import { writeFile } from 'fs/promises';
3
- import { resolve } from 'path';
4
- import { buildBaseInstructions } from '../lib/review-core.js';
5
- import { filterIgnoredFiles } from '../lib/review-orchestrator.js';
6
- import { createGitHubClient } from '../github/client.js';
7
- import { GitHubPlatformAdapter } from '../github/platform-adapter.js';
8
- import { createGitLabClient } from '../gitlab/client.js';
9
- import { GitLabPlatformAdapter } from '../gitlab/platform-adapter.js';
10
- function parseNumber(value, label) {
11
- if (typeof value === 'number' && Number.isFinite(value))
12
- return value;
13
- if (typeof value === 'string') {
14
- const parsed = parseInt(value, 10);
15
- if (Number.isFinite(parsed))
16
- return parsed;
17
- }
18
- throw new Error(`${label} is required`);
19
- }
20
- function ensureFileMatch(files, filename) {
21
- if (!filename)
22
- return files;
23
- const matches = files.filter((file) => file.filename === filename);
24
- if (matches.length === 0) {
25
- throw new Error(`No matching file "${filename}" found in PR/MR changes.`);
26
- }
27
- return matches;
28
- }
29
- function normalizeBaseBranch(baseBranch) {
30
- if (!baseBranch)
31
- return undefined;
32
- return baseBranch.startsWith('origin/') ? baseBranch : `origin/${baseBranch}`;
33
- }
34
- function resolveBaseBranch(cliBaseBranch, targetBranch) {
35
- if (cliBaseBranch) {
36
- const resolved = normalizeBaseBranch(cliBaseBranch);
37
- return {
38
- baseBranch: cliBaseBranch,
39
- resolvedBaseBranch: resolved,
40
- diffCommand: `git diff ${resolved}...HEAD -- <file>`,
41
- source: 'cli',
42
- };
43
- }
44
- if (process.env.DRS_BASE_BRANCH) {
45
- const resolved = normalizeBaseBranch(process.env.DRS_BASE_BRANCH);
46
- return {
47
- baseBranch: process.env.DRS_BASE_BRANCH,
48
- resolvedBaseBranch: resolved,
49
- diffCommand: `git diff ${resolved}...HEAD -- <file>`,
50
- source: 'env:DRS_BASE_BRANCH',
51
- };
52
- }
53
- if (process.env.GITHUB_BASE_REF) {
54
- const resolved = normalizeBaseBranch(process.env.GITHUB_BASE_REF);
55
- return {
56
- baseBranch: process.env.GITHUB_BASE_REF,
57
- resolvedBaseBranch: resolved,
58
- diffCommand: `git diff ${resolved}...HEAD -- <file>`,
59
- source: 'env:GITHUB_BASE_REF',
60
- };
61
- }
62
- if (process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME) {
63
- const resolved = normalizeBaseBranch(process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME);
64
- return {
65
- baseBranch: process.env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME,
66
- resolvedBaseBranch: resolved,
67
- diffCommand: `git diff ${resolved}...HEAD -- <file>`,
68
- source: 'env:CI_MERGE_REQUEST_TARGET_BRANCH_NAME',
69
- };
70
- }
71
- if (targetBranch) {
72
- const resolved = normalizeBaseBranch(targetBranch);
73
- return {
74
- baseBranch: targetBranch,
75
- resolvedBaseBranch: resolved,
76
- diffCommand: `git diff ${resolved}...HEAD -- <file>`,
77
- source: 'pr:targetBranch',
78
- };
79
- }
80
- return {
81
- diffCommand: 'git diff HEAD~1 -- <file>',
82
- };
83
- }
84
- export async function showChanges(config, options) {
85
- const workingDir = options.workingDir ?? process.cwd();
86
- const isGitHub = Boolean(options.owner ?? options.repo ?? options.prNumber);
87
- const isGitLab = Boolean(options.projectId ?? options.mrIid);
88
- if (isGitHub && isGitLab) {
89
- throw new Error('Specify either GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr), not both.');
90
- }
91
- if (!isGitHub && !isGitLab) {
92
- throw new Error('Specify GitHub options (--owner/--repo/--pr) or GitLab options (--project/--mr).');
93
- }
94
- if (isGitHub) {
95
- const owner = options.owner;
96
- const repo = options.repo;
97
- if (!owner || !repo) {
98
- throw new Error('GitHub requires --owner and --repo');
99
- }
100
- const prNumber = parseNumber(options.prNumber, 'PR number (--pr)');
101
- const projectId = `${owner}/${repo}`;
102
- const githubClient = createGitHubClient();
103
- const platformClient = new GitHubPlatformAdapter(githubClient);
104
- const pr = await platformClient.getPullRequest(projectId, prNumber);
105
- const allFiles = await platformClient.getChangedFiles(projectId, prNumber);
106
- const filesWithDiffs = allFiles
107
- .filter((file) => file.status !== 'removed')
108
- .map((file) => ({ filename: file.filename, patch: file.patch }));
109
- const filteredNames = filterIgnoredFiles(filesWithDiffs.map((file) => file.filename), config);
110
- const filteredFiles = filesWithDiffs.filter((file) => filteredNames.includes(file.filename));
111
- const scopedFiles = ensureFileMatch(filteredFiles, options.file);
112
- const label = `PR/MR #${pr.number}`;
113
- const baseBranchResolution = resolveBaseBranch(options.baseBranch, pr.targetBranch);
114
- const instructions = buildBaseInstructions(label, scopedFiles.map((file) => ({ filename: file.filename })), baseBranchResolution.diffCommand);
115
- const payload = {
116
- label,
117
- files: scopedFiles,
118
- instructions,
119
- resolvedBaseBranch: baseBranchResolution.resolvedBaseBranch,
120
- diffCommand: baseBranchResolution.diffCommand,
121
- diffCommandSource: baseBranchResolution.source,
122
- metadata: {
123
- projectId,
124
- prNumber: pr.number,
125
- title: pr.title,
126
- sourceBranch: pr.sourceBranch,
127
- targetBranch: pr.targetBranch,
128
- headSha: pr.headSha,
129
- },
130
- };
131
- await writeOutput(payload, options.outputPath, options.jsonOutput, workingDir);
132
- return;
133
- }
134
- const projectId = options.projectId;
135
- if (!projectId) {
136
- throw new Error('GitLab requires --project');
137
- }
138
- const mrIid = parseNumber(options.mrIid, 'MR IID (--mr)');
139
- const gitlabClient = createGitLabClient();
140
- const platformClient = new GitLabPlatformAdapter(gitlabClient);
141
- const pr = await platformClient.getPullRequest(projectId, mrIid);
142
- const allFiles = await platformClient.getChangedFiles(projectId, mrIid);
143
- const filesWithDiffs = allFiles
144
- .filter((file) => file.status !== 'removed')
145
- .map((file) => ({ filename: file.filename, patch: file.patch }));
146
- const filteredNames = filterIgnoredFiles(filesWithDiffs.map((file) => file.filename), config);
147
- const filteredFiles = filesWithDiffs.filter((file) => filteredNames.includes(file.filename));
148
- const scopedFiles = ensureFileMatch(filteredFiles, options.file);
149
- const label = `PR/MR #${pr.number}`;
150
- const baseBranchResolution = resolveBaseBranch(options.baseBranch, pr.targetBranch);
151
- const instructions = buildBaseInstructions(label, scopedFiles.map((file) => ({ filename: file.filename })), baseBranchResolution.diffCommand);
152
- const payload = {
153
- label,
154
- files: scopedFiles,
155
- instructions,
156
- resolvedBaseBranch: baseBranchResolution.resolvedBaseBranch,
157
- diffCommand: baseBranchResolution.diffCommand,
158
- diffCommandSource: baseBranchResolution.source,
159
- metadata: {
160
- projectId,
161
- prNumber: pr.number,
162
- title: pr.title,
163
- sourceBranch: pr.sourceBranch,
164
- targetBranch: pr.targetBranch,
165
- headSha: pr.headSha,
166
- },
167
- };
168
- await writeOutput(payload, options.outputPath, options.jsonOutput, workingDir);
169
- }
170
- async function writeOutput(payload, outputPath, jsonOutput, workingDir = process.cwd()) {
171
- let output = jsonOutput ? JSON.stringify(payload, null, 2) : payload.instructions;
172
- if (!jsonOutput && payload.resolvedBaseBranch) {
173
- output = `${output}\n\nBase branch resolved to: ${payload.resolvedBaseBranch} (${payload.diffCommandSource})`;
174
- }
175
- if (outputPath) {
176
- const fullPath = resolve(workingDir, outputPath);
177
- await writeFile(fullPath, output, 'utf-8');
178
- console.log(chalk.green(`✓ Output written to ${outputPath}\n`));
179
- }
180
- if (jsonOutput ?? !outputPath) {
181
- console.log(output);
182
- }
183
- }
184
- //# sourceMappingURL=show-changes.js.map