@diff-review-system/drs 2.2.1 → 3.0.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 (240) hide show
  1. package/README.md +228 -92
  2. package/dist/ci/runner.d.ts.map +1 -1
  3. package/dist/ci/runner.js +19 -22
  4. package/dist/ci/runner.js.map +1 -1
  5. package/dist/cli/describe-mr.d.ts.map +1 -1
  6. package/dist/cli/describe-mr.js +39 -20
  7. package/dist/cli/describe-mr.js.map +1 -1
  8. package/dist/cli/describe-pr.d.ts.map +1 -1
  9. package/dist/cli/describe-pr.js +39 -20
  10. package/dist/cli/describe-pr.js.map +1 -1
  11. package/dist/cli/index.js +11 -7
  12. package/dist/cli/index.js.map +1 -1
  13. package/dist/cli/init.d.ts.map +1 -1
  14. package/dist/cli/init.js +30 -2
  15. package/dist/cli/init.js.map +1 -1
  16. package/dist/cli/post-comments.d.ts.map +1 -1
  17. package/dist/cli/post-comments.js +5 -5
  18. package/dist/cli/post-comments.js.map +1 -1
  19. package/dist/cli/review-local.d.ts.map +1 -1
  20. package/dist/cli/review-local.integration.test.d.ts +2 -0
  21. package/dist/cli/review-local.integration.test.d.ts.map +1 -0
  22. package/dist/cli/review-local.integration.test.js +343 -0
  23. package/dist/cli/review-local.integration.test.js.map +1 -0
  24. package/dist/cli/review-local.js +5 -4
  25. package/dist/cli/review-local.js.map +1 -1
  26. package/dist/cli/review-local.live.e2e.test.d.ts +2 -0
  27. package/dist/cli/review-local.live.e2e.test.d.ts.map +1 -0
  28. package/dist/cli/review-local.live.e2e.test.js +154 -0
  29. package/dist/cli/review-local.live.e2e.test.js.map +1 -0
  30. package/dist/cli/review-local.test.d.ts +2 -0
  31. package/dist/cli/review-local.test.d.ts.map +1 -0
  32. package/dist/cli/review-local.test.js +164 -0
  33. package/dist/cli/review-local.test.js.map +1 -0
  34. package/dist/cli/review-mr.d.ts +1 -1
  35. package/dist/cli/review-mr.d.ts.map +1 -1
  36. package/dist/cli/review-mr.js +92 -17
  37. package/dist/cli/review-mr.js.map +1 -1
  38. package/dist/cli/review-mr.test.d.ts +2 -0
  39. package/dist/cli/review-mr.test.d.ts.map +1 -0
  40. package/dist/cli/review-mr.test.js +142 -0
  41. package/dist/cli/review-mr.test.js.map +1 -0
  42. package/dist/cli/review-pr.d.ts +1 -1
  43. package/dist/cli/review-pr.d.ts.map +1 -1
  44. package/dist/cli/review-pr.js +96 -13
  45. package/dist/cli/review-pr.js.map +1 -1
  46. package/dist/cli/review-pr.test.d.ts +2 -0
  47. package/dist/cli/review-pr.test.d.ts.map +1 -0
  48. package/dist/cli/review-pr.test.js +137 -0
  49. package/dist/cli/review-pr.test.js.map +1 -0
  50. package/dist/cli/show-changes.js +4 -4
  51. package/dist/github/platform-adapter.js +2 -2
  52. package/dist/gitlab/client.js +1 -1
  53. package/dist/lib/code-quality-report.js +1 -1
  54. package/dist/lib/comment-formatter.d.ts +2 -1
  55. package/dist/lib/comment-formatter.d.ts.map +1 -1
  56. package/dist/lib/comment-formatter.js +33 -1
  57. package/dist/lib/comment-formatter.js.map +1 -1
  58. package/dist/lib/comment-formatter.test.js +43 -0
  59. package/dist/lib/comment-formatter.test.js.map +1 -1
  60. package/dist/lib/comment-manager.d.ts.map +1 -1
  61. package/dist/lib/comment-manager.js +4 -3
  62. package/dist/lib/comment-manager.js.map +1 -1
  63. package/dist/lib/comment-poster.d.ts +2 -1
  64. package/dist/lib/comment-poster.d.ts.map +1 -1
  65. package/dist/lib/comment-poster.js +2 -2
  66. package/dist/lib/comment-poster.js.map +1 -1
  67. package/dist/lib/comment-poster.test.js +27 -11
  68. package/dist/lib/comment-poster.test.js.map +1 -1
  69. package/dist/lib/config-model-overrides.test.d.ts +1 -1
  70. package/dist/lib/config-model-overrides.test.js +2 -2
  71. package/dist/lib/config-model-overrides.test.js.map +1 -1
  72. package/dist/lib/config.d.ts +34 -7
  73. package/dist/lib/config.d.ts.map +1 -1
  74. package/dist/lib/config.js +35 -13
  75. package/dist/lib/config.js.map +1 -1
  76. package/dist/lib/config.test.js +16 -0
  77. package/dist/lib/config.test.js.map +1 -1
  78. package/dist/lib/context-compression.d.ts +27 -1
  79. package/dist/lib/context-compression.d.ts.map +1 -1
  80. package/dist/lib/context-compression.js +106 -4
  81. package/dist/lib/context-compression.js.map +1 -1
  82. package/dist/lib/context-compression.test.js +305 -1
  83. package/dist/lib/context-compression.test.js.map +1 -1
  84. package/dist/lib/context-loader.d.ts +3 -2
  85. package/dist/lib/context-loader.d.ts.map +1 -1
  86. package/dist/lib/context-loader.js +11 -11
  87. package/dist/lib/context-loader.js.map +1 -1
  88. package/dist/lib/description-executor.d.ts +19 -2
  89. package/dist/lib/description-executor.d.ts.map +1 -1
  90. package/dist/lib/description-executor.js +52 -21
  91. package/dist/lib/description-executor.js.map +1 -1
  92. package/dist/lib/description-executor.test.d.ts +2 -0
  93. package/dist/lib/description-executor.test.d.ts.map +1 -0
  94. package/dist/lib/description-executor.test.js +120 -0
  95. package/dist/lib/description-executor.test.js.map +1 -0
  96. package/dist/lib/description-formatter.d.ts +8 -3
  97. package/dist/lib/description-formatter.d.ts.map +1 -1
  98. package/dist/lib/description-formatter.js +88 -13
  99. package/dist/lib/description-formatter.js.map +1 -1
  100. package/dist/lib/description-formatter.test.d.ts +2 -0
  101. package/dist/lib/description-formatter.test.d.ts.map +1 -0
  102. package/dist/lib/description-formatter.test.js +57 -0
  103. package/dist/lib/description-formatter.test.js.map +1 -0
  104. package/dist/lib/diff-parser.test.d.ts +2 -0
  105. package/dist/lib/diff-parser.test.d.ts.map +1 -0
  106. package/dist/lib/diff-parser.test.js +335 -0
  107. package/dist/lib/diff-parser.test.js.map +1 -0
  108. package/dist/lib/exit.d.ts +35 -0
  109. package/dist/lib/exit.d.ts.map +1 -0
  110. package/dist/lib/exit.js +53 -0
  111. package/dist/lib/exit.js.map +1 -0
  112. package/dist/lib/exit.test.d.ts +2 -0
  113. package/dist/lib/exit.test.d.ts.map +1 -0
  114. package/dist/lib/exit.test.js +120 -0
  115. package/dist/lib/exit.test.js.map +1 -0
  116. package/dist/lib/format-utils.d.ts +3 -0
  117. package/dist/lib/format-utils.d.ts.map +1 -0
  118. package/dist/lib/format-utils.js +7 -0
  119. package/dist/lib/format-utils.js.map +1 -0
  120. package/dist/lib/json-output.d.ts +4 -1
  121. package/dist/lib/json-output.d.ts.map +1 -1
  122. package/dist/lib/json-output.js +2 -1
  123. package/dist/lib/json-output.js.map +1 -1
  124. package/dist/lib/json-output.test.d.ts +2 -0
  125. package/dist/lib/json-output.test.d.ts.map +1 -0
  126. package/dist/lib/json-output.test.js +135 -0
  127. package/dist/lib/json-output.test.js.map +1 -0
  128. package/dist/lib/logger.d.ts +10 -2
  129. package/dist/lib/logger.d.ts.map +1 -1
  130. package/dist/lib/logger.js +22 -4
  131. package/dist/lib/logger.js.map +1 -1
  132. package/dist/lib/logger.test.d.ts +2 -0
  133. package/dist/lib/logger.test.d.ts.map +1 -0
  134. package/dist/lib/logger.test.js +324 -0
  135. package/dist/lib/logger.test.js.map +1 -0
  136. package/dist/lib/position-validator.test.d.ts +2 -0
  137. package/dist/lib/position-validator.test.d.ts.map +1 -0
  138. package/dist/lib/position-validator.test.js +128 -0
  139. package/dist/lib/position-validator.test.js.map +1 -0
  140. package/dist/lib/repository-validator.js +1 -1
  141. package/dist/lib/review-core.d.ts +9 -4
  142. package/dist/lib/review-core.d.ts.map +1 -1
  143. package/dist/lib/review-core.js +207 -112
  144. package/dist/lib/review-core.js.map +1 -1
  145. package/dist/lib/review-core.test.js +76 -30
  146. package/dist/lib/review-core.test.js.map +1 -1
  147. package/dist/lib/review-orchestrator.d.ts +12 -7
  148. package/dist/lib/review-orchestrator.d.ts.map +1 -1
  149. package/dist/lib/review-orchestrator.js +78 -22
  150. package/dist/lib/review-orchestrator.js.map +1 -1
  151. package/dist/lib/review-orchestrator.test.js +160 -42
  152. package/dist/lib/review-orchestrator.test.js.map +1 -1
  153. package/dist/lib/review-parser.test.d.ts +2 -0
  154. package/dist/lib/review-parser.test.d.ts.map +1 -0
  155. package/dist/lib/review-parser.test.js +130 -0
  156. package/dist/lib/review-parser.test.js.map +1 -0
  157. package/dist/lib/review-usage.d.ts +32 -0
  158. package/dist/lib/review-usage.d.ts.map +1 -0
  159. package/dist/lib/review-usage.js +72 -0
  160. package/dist/lib/review-usage.js.map +1 -0
  161. package/dist/lib/review-usage.test.d.ts +2 -0
  162. package/dist/lib/review-usage.test.d.ts.map +1 -0
  163. package/dist/lib/review-usage.test.js +83 -0
  164. package/dist/lib/review-usage.test.js.map +1 -0
  165. package/dist/lib/unified-review-executor.d.ts +6 -2
  166. package/dist/lib/unified-review-executor.d.ts.map +1 -1
  167. package/dist/lib/unified-review-executor.js +54 -28
  168. package/dist/lib/unified-review-executor.js.map +1 -1
  169. package/dist/lib/unified-review-executor.test.js +138 -16
  170. package/dist/lib/unified-review-executor.test.js.map +1 -1
  171. package/dist/lib/write-json-output.test.d.ts +2 -0
  172. package/dist/lib/write-json-output.test.d.ts.map +1 -0
  173. package/dist/lib/write-json-output.test.js +259 -0
  174. package/dist/lib/write-json-output.test.js.map +1 -0
  175. package/dist/pi/sdk.d.ts +94 -0
  176. package/dist/pi/sdk.d.ts.map +1 -0
  177. package/dist/pi/sdk.js +486 -0
  178. package/dist/pi/sdk.js.map +1 -0
  179. package/dist/pi/sdk.test.d.ts +2 -0
  180. package/dist/pi/sdk.test.d.ts.map +1 -0
  181. package/dist/pi/sdk.test.js +331 -0
  182. package/dist/pi/sdk.test.js.map +1 -0
  183. package/dist/{opencode → runtime}/agent-loader.d.ts +7 -5
  184. package/dist/runtime/agent-loader.d.ts.map +1 -0
  185. package/dist/{opencode → runtime}/agent-loader.js +24 -19
  186. package/dist/runtime/agent-loader.js.map +1 -0
  187. package/dist/runtime/agent-loader.test.d.ts +2 -0
  188. package/dist/runtime/agent-loader.test.d.ts.map +1 -0
  189. package/dist/runtime/agent-loader.test.js +280 -0
  190. package/dist/runtime/agent-loader.test.js.map +1 -0
  191. package/dist/runtime/built-in-paths.d.ts +2 -0
  192. package/dist/runtime/built-in-paths.d.ts.map +1 -0
  193. package/dist/runtime/built-in-paths.js +14 -0
  194. package/dist/runtime/built-in-paths.js.map +1 -0
  195. package/dist/{opencode → runtime}/client.d.ts +35 -18
  196. package/dist/runtime/client.d.ts.map +1 -0
  197. package/dist/runtime/client.js +486 -0
  198. package/dist/runtime/client.js.map +1 -0
  199. package/dist/{opencode → runtime}/client.test.d.ts.map +1 -1
  200. package/dist/runtime/client.test.js +392 -0
  201. package/dist/runtime/client.test.js.map +1 -0
  202. package/dist/runtime/path-config.d.ts +8 -0
  203. package/dist/runtime/path-config.d.ts.map +1 -0
  204. package/dist/runtime/path-config.js +68 -0
  205. package/dist/runtime/path-config.js.map +1 -0
  206. package/dist/runtime/path-config.test.d.ts +2 -0
  207. package/dist/runtime/path-config.test.d.ts.map +1 -0
  208. package/dist/runtime/path-config.test.js +103 -0
  209. package/dist/runtime/path-config.test.js.map +1 -0
  210. package/package.json +5 -5
  211. package/.opencode/opencode.jsonc +0 -15
  212. package/.opencode/tool/write_json_output.ts +0 -24
  213. package/.opencode/tools/drs_skill.ts +0 -67
  214. package/dist/lib/skills-prompt.d.ts +0 -3
  215. package/dist/lib/skills-prompt.d.ts.map +0 -1
  216. package/dist/lib/skills-prompt.js +0 -70
  217. package/dist/lib/skills-prompt.js.map +0 -1
  218. package/dist/opencode/agent-loader.d.ts.map +0 -1
  219. package/dist/opencode/agent-loader.js.map +0 -1
  220. package/dist/opencode/client.d.ts.map +0 -1
  221. package/dist/opencode/client.js +0 -456
  222. package/dist/opencode/client.js.map +0 -1
  223. package/dist/opencode/client.test.js +0 -317
  224. package/dist/opencode/client.test.js.map +0 -1
  225. package/dist/opencode/opencode-paths.d.ts +0 -2
  226. package/dist/opencode/opencode-paths.d.ts.map +0 -1
  227. package/dist/opencode/opencode-paths.js +0 -7
  228. package/dist/opencode/opencode-paths.js.map +0 -1
  229. package/dist/opencode/skill-loader.d.ts +0 -15
  230. package/dist/opencode/skill-loader.d.ts.map +0 -1
  231. package/dist/opencode/skill-loader.js +0 -88
  232. package/dist/opencode/skill-loader.js.map +0 -1
  233. /package/{.opencode/agent → .pi/agents}/describe/pr-describer.md +0 -0
  234. /package/{.opencode/agent → .pi/agents}/review/documentation.md +0 -0
  235. /package/{.opencode/agent → .pi/agents}/review/performance.md +0 -0
  236. /package/{.opencode/agent → .pi/agents}/review/quality.md +0 -0
  237. /package/{.opencode/agent → .pi/agents}/review/security.md +0 -0
  238. /package/{.opencode/agent → .pi/agents}/review/style.md +0 -0
  239. /package/{.opencode/agent → .pi/agents}/review/unified-reviewer.md +0 -0
  240. /package/dist/{opencode → runtime}/client.test.d.ts +0 -0
@@ -0,0 +1,2 @@
1
+ export {};
2
+ //# sourceMappingURL=review-local.integration.test.d.ts.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"review-local.integration.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-local.integration.test.ts"],"names":[],"mappings":""}
@@ -0,0 +1,343 @@
1
+ import { beforeEach, describe, expect, it, vi } from 'vitest';
2
+ import { reviewLocal } from './review-local.js';
3
+ const mocks = vi.hoisted(() => ({
4
+ git: {
5
+ checkIsRepo: vi.fn(),
6
+ diff: vi.fn(),
7
+ },
8
+ createRuntimeClientInstance: vi.fn(),
9
+ createSession: vi.fn(),
10
+ streamMessages: vi.fn(),
11
+ closeSession: vi.fn(),
12
+ shutdown: vi.fn(),
13
+ }));
14
+ vi.mock('simple-git', () => ({
15
+ default: vi.fn(() => mocks.git),
16
+ }));
17
+ vi.mock('../runtime/client.js', () => ({
18
+ createRuntimeClientInstance: mocks.createRuntimeClientInstance,
19
+ }));
20
+ const simulatedDiff = [
21
+ 'diff --git a/src/app.ts b/src/app.ts',
22
+ 'index 1111111..2222222 100644',
23
+ '--- a/src/app.ts',
24
+ '+++ b/src/app.ts',
25
+ '@@ -1,3 +1,5 @@',
26
+ ' export function run(input: string) {',
27
+ '+ const query = "SELECT * FROM users WHERE id = " + input;',
28
+ ' return input;',
29
+ ' }',
30
+ 'diff --git a/src/app.test.ts b/src/app.test.ts',
31
+ 'index 3333333..4444444 100644',
32
+ '--- a/src/app.test.ts',
33
+ '+++ b/src/app.test.ts',
34
+ '@@ -1,2 +1,3 @@',
35
+ ' test("run", () => {',
36
+ '+ expect(true).toBe(true);',
37
+ ' });',
38
+ ].join('\n');
39
+ const simulatedCliDiff = [
40
+ 'diff --git a/src/cli/index.ts b/src/cli/index.ts',
41
+ 'index 1234567..89abcde 100644',
42
+ '--- a/src/cli/index.ts',
43
+ '+++ b/src/cli/index.ts',
44
+ '@@ -60,6 +60,7 @@ program',
45
+ " .command('review-local')",
46
+ " .description('Review local changes before pushing')",
47
+ "+ .option('--skip-repo-check', 'Skip git repository validation for advanced usage')",
48
+ " .option('--staged', 'Review staged changes only')",
49
+ " .option('--output <path>', 'Write review results to JSON file')",
50
+ " .option('--json', 'Output raw JSON to stdout')",
51
+ ].join('\n');
52
+ const integrationConfig = {
53
+ pi: {},
54
+ gitlab: { url: '', token: '' },
55
+ github: { token: '' },
56
+ review: {
57
+ agents: ['security'],
58
+ default: {
59
+ model: 'anthropic/claude-sonnet-4-5-20250929',
60
+ skills: [],
61
+ },
62
+ ignorePatterns: ['*.test.ts'],
63
+ mode: 'multi-agent',
64
+ describe: {
65
+ enabled: false,
66
+ postDescription: false,
67
+ },
68
+ },
69
+ describe: {
70
+ includeProjectContext: false,
71
+ },
72
+ contextCompression: {
73
+ enabled: false,
74
+ },
75
+ };
76
+ describe('review-local integration (simulated diffs)', () => {
77
+ beforeEach(() => {
78
+ vi.clearAllMocks();
79
+ vi.spyOn(console, 'log').mockImplementation(() => { });
80
+ mocks.git.checkIsRepo.mockResolvedValue(true);
81
+ mocks.git.diff.mockResolvedValue(simulatedDiff);
82
+ mocks.createSession.mockResolvedValue({
83
+ id: 'session-1',
84
+ agent: 'review/security',
85
+ createdAt: new Date('2026-02-22T00:00:00Z'),
86
+ });
87
+ const reviewPayload = {
88
+ timestamp: '2026-02-22T00:00:00Z',
89
+ summary: {
90
+ filesReviewed: 1,
91
+ issuesFound: 1,
92
+ bySeverity: {
93
+ CRITICAL: 0,
94
+ HIGH: 0,
95
+ MEDIUM: 0,
96
+ LOW: 1,
97
+ },
98
+ byCategory: {
99
+ SECURITY: 1,
100
+ QUALITY: 0,
101
+ STYLE: 0,
102
+ PERFORMANCE: 0,
103
+ DOCUMENTATION: 0,
104
+ },
105
+ },
106
+ issues: [
107
+ {
108
+ category: 'SECURITY',
109
+ severity: 'LOW',
110
+ title: 'Avoid SQL string concatenation',
111
+ file: 'src/app.ts',
112
+ line: 2,
113
+ problem: 'Query strings are built using concatenated user input.',
114
+ solution: 'Use parameterized queries to avoid injection risks.',
115
+ references: [],
116
+ agent: 'security',
117
+ },
118
+ ],
119
+ };
120
+ mocks.streamMessages.mockImplementation(async function* () {
121
+ yield {
122
+ id: 'assistant-1',
123
+ role: 'assistant',
124
+ content: JSON.stringify(reviewPayload),
125
+ timestamp: new Date('2026-02-22T00:00:01Z'),
126
+ };
127
+ });
128
+ mocks.closeSession.mockResolvedValue(undefined);
129
+ mocks.shutdown.mockResolvedValue(undefined);
130
+ mocks.createRuntimeClientInstance.mockResolvedValue({
131
+ createSession: mocks.createSession,
132
+ streamMessages: mocks.streamMessages,
133
+ closeSession: mocks.closeSession,
134
+ shutdown: mocks.shutdown,
135
+ getMinContextWindow: vi.fn(() => undefined),
136
+ });
137
+ });
138
+ it('runs review-local end-to-end using parsed git diff input and filtered files', async () => {
139
+ const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
140
+ await reviewLocal(integrationConfig, { staged: false, jsonOutput: false, debug: false });
141
+ expect(mocks.git.diff).toHaveBeenCalledWith();
142
+ expect(mocks.createRuntimeClientInstance).toHaveBeenCalledTimes(1);
143
+ expect(mocks.createSession).toHaveBeenCalledTimes(1);
144
+ expect(mocks.createSession).toHaveBeenCalledWith(expect.objectContaining({
145
+ agent: 'review/security',
146
+ }));
147
+ const prompt = mocks.createSession.mock.calls[0][0].message;
148
+ expect(prompt).toContain('src/app.ts');
149
+ expect(prompt).not.toContain('src/app.test.ts');
150
+ expect(prompt).toContain('+ const query = "SELECT * FROM users WHERE id = " + input;');
151
+ expect(mocks.streamMessages).toHaveBeenCalledWith('session-1');
152
+ expect(mocks.closeSession).toHaveBeenCalledWith('session-1');
153
+ expect(mocks.shutdown).toHaveBeenCalledTimes(1);
154
+ expect(exitSpy).not.toHaveBeenCalledWith(1);
155
+ exitSpy.mockRestore();
156
+ });
157
+ it('logs configured skill usage when reviewing CLI flag changes', async () => {
158
+ const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
159
+ mocks.git.diff.mockResolvedValue(simulatedCliDiff);
160
+ const reviewPayload = {
161
+ timestamp: '2026-02-22T00:00:00Z',
162
+ summary: {
163
+ filesReviewed: 1,
164
+ issuesFound: 1,
165
+ bySeverity: {
166
+ CRITICAL: 0,
167
+ HIGH: 0,
168
+ MEDIUM: 1,
169
+ LOW: 0,
170
+ },
171
+ byCategory: {
172
+ SECURITY: 0,
173
+ QUALITY: 1,
174
+ STYLE: 0,
175
+ PERFORMANCE: 0,
176
+ DOCUMENTATION: 0,
177
+ },
178
+ },
179
+ issues: [
180
+ {
181
+ category: 'QUALITY',
182
+ severity: 'MEDIUM',
183
+ title: 'Missing integration test for new review-local CLI flag',
184
+ file: 'src/cli/index.ts',
185
+ line: 63,
186
+ problem: 'A new CLI flag was introduced without integration-level coverage.',
187
+ solution: 'Add an integration test that verifies the new flag is parsed and propagated to command execution.',
188
+ references: [],
189
+ agent: 'quality',
190
+ },
191
+ ],
192
+ };
193
+ mocks.streamMessages.mockImplementation(async function* () {
194
+ yield {
195
+ id: 'tool-1',
196
+ role: 'tool',
197
+ toolName: 'skill',
198
+ content: JSON.stringify({ name: 'cli-testing', usage: 'applied' }),
199
+ timestamp: new Date('2026-02-22T00:00:00Z'),
200
+ };
201
+ yield {
202
+ id: 'assistant-2',
203
+ role: 'assistant',
204
+ content: JSON.stringify(reviewPayload),
205
+ timestamp: new Date('2026-02-22T00:00:01Z'),
206
+ };
207
+ });
208
+ const configWithCliSkill = {
209
+ ...integrationConfig,
210
+ review: {
211
+ ...integrationConfig.review,
212
+ agents: ['quality'],
213
+ default: {
214
+ ...integrationConfig.review.default,
215
+ skills: ['cli-testing'],
216
+ },
217
+ },
218
+ };
219
+ await reviewLocal(configWithCliSkill, { staged: false, jsonOutput: false, debug: false });
220
+ expect(mocks.createSession).toHaveBeenCalledWith(expect.objectContaining({
221
+ agent: 'review/quality',
222
+ }));
223
+ const prompt = mocks.createSession.mock.calls[0][0].message;
224
+ expect(prompt).toContain('src/cli/index.ts');
225
+ expect(prompt).toContain("+ .option('--skip-repo-check'");
226
+ expect(console.log).toHaveBeenCalledWith(expect.stringContaining('Loaded skill: cli-testing'));
227
+ expect(exitSpy).not.toHaveBeenCalledWith(1);
228
+ exitSpy.mockRestore();
229
+ });
230
+ it('custom agent override prompt flows through to session message', async () => {
231
+ const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
232
+ // Dynamically mock buildReviewPrompt to simulate agent override
233
+ const contextLoader = await import('../lib/context-loader.js');
234
+ const buildPromptSpy = vi
235
+ .spyOn(contextLoader, 'buildReviewPrompt')
236
+ .mockImplementation((_agentName, _basePrompt, reviewLabel, changedFiles) => {
237
+ return (`# Custom Rails Security Agent\n\n` +
238
+ `Focus on mass assignment and CSRF.\n\n` +
239
+ `Review the following files from ${reviewLabel}:\n` +
240
+ changedFiles.map((f) => `- ${f}`).join('\n'));
241
+ });
242
+ const reviewPayload = {
243
+ timestamp: '2026-02-22T00:00:00Z',
244
+ summary: {
245
+ filesReviewed: 1,
246
+ issuesFound: 0,
247
+ bySeverity: { CRITICAL: 0, HIGH: 0, MEDIUM: 0, LOW: 0 },
248
+ byCategory: {
249
+ SECURITY: 0,
250
+ QUALITY: 0,
251
+ STYLE: 0,
252
+ PERFORMANCE: 0,
253
+ DOCUMENTATION: 0,
254
+ },
255
+ },
256
+ issues: [],
257
+ };
258
+ mocks.streamMessages.mockImplementation(async function* () {
259
+ yield {
260
+ id: 'assistant-1',
261
+ role: 'assistant',
262
+ content: JSON.stringify(reviewPayload),
263
+ timestamp: new Date('2026-02-22T00:00:01Z'),
264
+ };
265
+ });
266
+ await reviewLocal(integrationConfig, { staged: false, jsonOutput: false, debug: false });
267
+ expect(buildPromptSpy).toHaveBeenCalledWith('security', expect.any(String), expect.any(String), expect.any(Array), expect.any(String), expect.anything(), undefined);
268
+ const prompt = mocks.createSession.mock.calls[0][0].message;
269
+ expect(prompt).toContain('Custom Rails Security Agent');
270
+ expect(prompt).toContain('mass assignment and CSRF');
271
+ expect(prompt).toContain('src/app.ts');
272
+ buildPromptSpy.mockRestore();
273
+ exitSpy.mockRestore();
274
+ });
275
+ it('new custom agent runs when configured in review.agents', async () => {
276
+ const exitSpy = vi.spyOn(process, 'exit').mockImplementation((() => undefined));
277
+ // Mock agent-loader to include a brand new custom agent
278
+ const agentLoader = await import('../runtime/agent-loader.js');
279
+ const loadAgentsSpy = vi.spyOn(agentLoader, 'loadReviewAgents').mockReturnValue([
280
+ {
281
+ name: 'review/api-reviewer',
282
+ path: '/project/.drs/agents/api-reviewer/agent.md',
283
+ description: 'API contract reviewer',
284
+ prompt: 'Review REST API contracts and OpenAPI compliance.',
285
+ model: 'anthropic/claude-sonnet-4-5-20250929',
286
+ },
287
+ ]);
288
+ const reviewPayload = {
289
+ timestamp: '2026-02-22T00:00:00Z',
290
+ summary: {
291
+ filesReviewed: 1,
292
+ issuesFound: 1,
293
+ bySeverity: { CRITICAL: 0, HIGH: 0, MEDIUM: 1, LOW: 0 },
294
+ byCategory: {
295
+ SECURITY: 0,
296
+ QUALITY: 0,
297
+ STYLE: 0,
298
+ PERFORMANCE: 0,
299
+ DOCUMENTATION: 1,
300
+ },
301
+ },
302
+ issues: [
303
+ {
304
+ category: 'DOCUMENTATION',
305
+ severity: 'MEDIUM',
306
+ title: 'Missing API endpoint docs',
307
+ file: 'src/app.ts',
308
+ line: 2,
309
+ problem: 'New endpoint lacks OpenAPI annotations.',
310
+ solution: 'Add @swagger JSDoc annotations.',
311
+ references: [],
312
+ agent: 'api-reviewer',
313
+ },
314
+ ],
315
+ };
316
+ mocks.streamMessages.mockImplementation(async function* () {
317
+ yield {
318
+ id: 'assistant-1',
319
+ role: 'assistant',
320
+ content: JSON.stringify(reviewPayload),
321
+ timestamp: new Date('2026-02-22T00:00:01Z'),
322
+ };
323
+ });
324
+ const configWithNewAgent = {
325
+ ...integrationConfig,
326
+ review: {
327
+ ...integrationConfig.review,
328
+ agents: ['api-reviewer'],
329
+ },
330
+ };
331
+ await reviewLocal(configWithNewAgent, { staged: false, jsonOutput: false, debug: false });
332
+ expect(mocks.createSession).toHaveBeenCalledTimes(1);
333
+ expect(mocks.createSession).toHaveBeenCalledWith(expect.objectContaining({
334
+ agent: 'review/api-reviewer',
335
+ }));
336
+ const prompt = mocks.createSession.mock.calls[0][0].message;
337
+ expect(prompt).toContain('src/app.ts');
338
+ expect(exitSpy).not.toHaveBeenCalledWith(1);
339
+ loadAgentsSpy.mockRestore();
340
+ exitSpy.mockRestore();
341
+ });
342
+ });
343
+ //# sourceMappingURL=review-local.integration.test.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"review-local.integration.test.js","sourceRoot":"","sources":["../../src/cli/review-local.integration.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9D,OAAO,EAAE,WAAW,EAAE,MAAM,mBAAmB,CAAC;AAEhD,MAAM,KAAK,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAC9B,GAAG,EAAE;QACH,WAAW,EAAE,EAAE,CAAC,EAAE,EAAE;QACpB,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE;KACd;IACD,2BAA2B,EAAE,EAAE,CAAC,EAAE,EAAE;IACpC,aAAa,EAAE,EAAE,CAAC,EAAE,EAAE;IACtB,cAAc,EAAE,EAAE,CAAC,EAAE,EAAE;IACvB,YAAY,EAAE,EAAE,CAAC,EAAE,EAAE;IACrB,QAAQ,EAAE,EAAE,CAAC,EAAE,EAAE;CAClB,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,YAAY,EAAE,GAAG,EAAE,CAAC,CAAC;IAC3B,OAAO,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,KAAK,CAAC,GAAG,CAAC;CAChC,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,sBAAsB,EAAE,GAAG,EAAE,CAAC,CAAC;IACrC,2BAA2B,EAAE,KAAK,CAAC,2BAA2B;CAC/D,CAAC,CAAC,CAAC;AAEJ,MAAM,aAAa,GAAG;IACpB,sCAAsC;IACtC,+BAA+B;IAC/B,kBAAkB;IAClB,kBAAkB;IAClB,iBAAiB;IACjB,uCAAuC;IACvC,6DAA6D;IAC7D,kBAAkB;IAClB,IAAI;IACJ,gDAAgD;IAChD,+BAA+B;IAC/B,uBAAuB;IACvB,uBAAuB;IACvB,iBAAiB;IACjB,sBAAsB;IACtB,6BAA6B;IAC7B,MAAM;CACP,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,gBAAgB,GAAG;IACvB,kDAAkD;IAClD,+BAA+B;IAC/B,wBAAwB;IACxB,wBAAwB;IACxB,2BAA2B;IAC3B,6BAA6B;IAC7B,wDAAwD;IACxD,sFAAsF;IACtF,sDAAsD;IACtD,oEAAoE;IACpE,mDAAmD;CACpD,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAEb,MAAM,iBAAiB,GAAG;IACxB,EAAE,EAAE,EAAE;IACN,MAAM,EAAE,EAAE,GAAG,EAAE,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE;IAC9B,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,EAAE;IACrB,MAAM,EAAE;QACN,MAAM,EAAE,CAAC,UAAU,CAAC;QACpB,OAAO,EAAE;YACP,KAAK,EAAE,sCAAsC;YAC7C,MAAM,EAAE,EAAE;SACX;QACD,cAAc,EAAE,CAAC,WAAW,CAAC;QAC7B,IAAI,EAAE,aAAa;QACnB,QAAQ,EAAE;YACR,OAAO,EAAE,KAAK;YACd,eAAe,EAAE,KAAK;SACvB;KACF;IACD,QAAQ,EAAE;QACR,qBAAqB,EAAE,KAAK;KAC7B;IACD,kBAAkB,EAAE;QAClB,OAAO,EAAE,KAAK;KACf;CACsB,CAAC;AAE1B,QAAQ,CAAC,4CAA4C,EAAE,GAAG,EAAE;IAC1D,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAEtD,KAAK,CAAC,GAAG,CAAC,WAAW,CAAC,iBAAiB,CAAC,IAAI,CAAC,CAAC;QAC9C,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,iBAAiB,CAAC,aAAa,CAAC,CAAC;QAEhD,KAAK,CAAC,aAAa,CAAC,iBAAiB,CAAC;YACpC,EAAE,EAAE,WAAW;YACf,KAAK,EAAE,iBAAiB;YACxB,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;SAC5C,CAAC,CAAC;QAEH,MAAM,aAAa,GAAG;YACpB,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE;gBACP,aAAa,EAAE,CAAC;gBAChB,WAAW,EAAE,CAAC;gBACd,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,GAAG,EAAE,CAAC;iBACP;gBACD,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,OAAO,EAAE,CAAC;oBACV,KAAK,EAAE,CAAC;oBACR,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;iBACjB;aACF;YACD,MAAM,EAAE;gBACN;oBACE,QAAQ,EAAE,UAAU;oBACpB,QAAQ,EAAE,KAAK;oBACf,KAAK,EAAE,gCAAgC;oBACvC,IAAI,EAAE,YAAY;oBAClB,IAAI,EAAE,CAAC;oBACP,OAAO,EAAE,wDAAwD;oBACjE,QAAQ,EAAE,qDAAqD;oBAC/D,UAAU,EAAE,EAAE;oBACd,KAAK,EAAE,UAAU;iBAClB;aACF;SACF,CAAC;QAEF,KAAK,CAAC,cAAc,CAAC,kBAAkB,CAAC,KAAK,SAAS,CAAC;YACrD,MAAM;gBACJ,EAAE,EAAE,aAAa;gBACjB,IAAI,EAAE,WAAW;gBACjB,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC;gBACtC,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;aAC5C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,KAAK,CAAC,YAAY,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAChD,KAAK,CAAC,QAAQ,CAAC,iBAAiB,CAAC,SAAS,CAAC,CAAC;QAE5C,KAAK,CAAC,2BAA2B,CAAC,iBAAiB,CAAC;YAClD,aAAa,EAAE,KAAK,CAAC,aAAa;YAClC,cAAc,EAAE,KAAK,CAAC,cAAc;YACpC,YAAY,EAAE,KAAK,CAAC,YAAY;YAChC,QAAQ,EAAE,KAAK,CAAC,QAAQ;YACxB,mBAAmB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,SAAS,CAAC;SAC5C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;QAC3F,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,CAAC,SAAS,CAAQ,CAAC,CAAC;QAEvF,MAAM,WAAW,CAAC,iBAAiB,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QAEzF,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,oBAAoB,EAAE,CAAC;QAC9C,MAAM,CAAC,KAAK,CAAC,2BAA2B,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACnE,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC9C,MAAM,CAAC,gBAAgB,CAAC;YACtB,KAAK,EAAE,iBAAiB;SACzB,CAAC,CACH,CAAC;QAEF,MAAM,MAAM,GAAI,KAAK,CAAC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC,OAAO,CAAC;QACrF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QACvC,MAAM,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,SAAS,CAAC,iBAAiB,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,6DAA6D,CAAC,CAAC;QAExF,MAAM,CAAC,KAAK,CAAC,cAAc,CAAC,CAAC,oBAAoB,CAAC,WAAW,CAAC,CAAC;QAC/D,MAAM,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC,oBAAoB,CAAC,WAAW,CAAC,CAAC;QAC7D,MAAM,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QAChD,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC;QAE5C,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;QAC3E,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,CAAC,SAAS,CAAQ,CAAC,CAAC;QAEvF,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,iBAAiB,CAAC,gBAAgB,CAAC,CAAC;QAEnD,MAAM,aAAa,GAAG;YACpB,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE;gBACP,aAAa,EAAE,CAAC;gBAChB,WAAW,EAAE,CAAC;gBACd,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,IAAI,EAAE,CAAC;oBACP,MAAM,EAAE,CAAC;oBACT,GAAG,EAAE,CAAC;iBACP;gBACD,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,OAAO,EAAE,CAAC;oBACV,KAAK,EAAE,CAAC;oBACR,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;iBACjB;aACF;YACD,MAAM,EAAE;gBACN;oBACE,QAAQ,EAAE,SAAS;oBACnB,QAAQ,EAAE,QAAQ;oBAClB,KAAK,EAAE,wDAAwD;oBAC/D,IAAI,EAAE,kBAAkB;oBACxB,IAAI,EAAE,EAAE;oBACR,OAAO,EAAE,mEAAmE;oBAC5E,QAAQ,EACN,mGAAmG;oBACrG,UAAU,EAAE,EAAE;oBACd,KAAK,EAAE,SAAS;iBACjB;aACF;SACF,CAAC;QAEF,KAAK,CAAC,cAAc,CAAC,kBAAkB,CAAC,KAAK,SAAS,CAAC;YACrD,MAAM;gBACJ,EAAE,EAAE,QAAQ;gBACZ,IAAI,EAAE,MAAM;gBACZ,QAAQ,EAAE,OAAO;gBACjB,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,EAAE,IAAI,EAAE,aAAa,EAAE,KAAK,EAAE,SAAS,EAAE,CAAC;gBAClE,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;aAC5C,CAAC;YAEF,MAAM;gBACJ,EAAE,EAAE,aAAa;gBACjB,IAAI,EAAE,WAAW;gBACjB,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC;gBACtC,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;aAC5C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,MAAM,kBAAkB,GAAG;YACzB,GAAG,iBAAiB;YACpB,MAAM,EAAE;gBACN,GAAG,iBAAiB,CAAC,MAAM;gBAC3B,MAAM,EAAE,CAAC,SAAS,CAAC;gBACnB,OAAO,EAAE;oBACP,GAAG,iBAAiB,CAAC,MAAM,CAAC,OAAO;oBACnC,MAAM,EAAE,CAAC,aAAa,CAAC;iBACxB;aACF;SACsB,CAAC;QAE1B,MAAM,WAAW,CAAC,kBAAkB,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QAE1F,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC9C,MAAM,CAAC,gBAAgB,CAAC;YACtB,KAAK,EAAE,gBAAgB;SACxB,CAAC,CACH,CAAC;QAEF,MAAM,MAAM,GAAI,KAAK,CAAC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC,OAAO,CAAC;QACrF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,kBAAkB,CAAC,CAAC;QAC7C,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,gCAAgC,CAAC,CAAC;QAE3D,MAAM,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,oBAAoB,CAAC,MAAM,CAAC,gBAAgB,CAAC,2BAA2B,CAAC,CAAC,CAAC;QAC/F,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC;QAE5C,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,+DAA+D,EAAE,KAAK,IAAI,EAAE;QAC7E,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,CAAC,SAAS,CAAQ,CAAC,CAAC;QAEvF,gEAAgE;QAChE,MAAM,aAAa,GAAG,MAAM,MAAM,CAAC,0BAA0B,CAAC,CAAC;QAC/D,MAAM,cAAc,GAAG,EAAE;aACtB,KAAK,CAAC,aAAa,EAAE,mBAAmB,CAAC;aACzC,kBAAkB,CACjB,CAAC,UAAkB,EAAE,WAAmB,EAAE,WAAmB,EAAE,YAAsB,EAAE,EAAE;YACvF,OAAO,CACL,mCAAmC;gBACnC,wCAAwC;gBACxC,mCAAmC,WAAW,KAAK;gBACnD,YAAY,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAC7C,CAAC;QACJ,CAAC,CACF,CAAC;QAEJ,MAAM,aAAa,GAAG;YACpB,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE;gBACP,aAAa,EAAE,CAAC;gBAChB,WAAW,EAAE,CAAC;gBACd,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE;gBACvD,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,OAAO,EAAE,CAAC;oBACV,KAAK,EAAE,CAAC;oBACR,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;iBACjB;aACF;YACD,MAAM,EAAE,EAAE;SACX,CAAC;QAEF,KAAK,CAAC,cAAc,CAAC,kBAAkB,CAAC,KAAK,SAAS,CAAC;YACrD,MAAM;gBACJ,EAAE,EAAE,aAAa;gBACjB,IAAI,EAAE,WAAW;gBACjB,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC;gBACtC,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;aAC5C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,MAAM,WAAW,CAAC,iBAAiB,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QAEzF,MAAM,CAAC,cAAc,CAAC,CAAC,oBAAoB,CACzC,UAAU,EACV,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAClB,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAClB,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,EACjB,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,EAClB,MAAM,CAAC,QAAQ,EAAE,EACjB,SAAS,CACV,CAAC;QAEF,MAAM,MAAM,GAAI,KAAK,CAAC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC,OAAO,CAAC;QACrF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,6BAA6B,CAAC,CAAC;QACxD,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAC;QACrD,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QAEvC,cAAc,CAAC,WAAW,EAAE,CAAC;QAC7B,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,OAAO,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC,GAAG,EAAE,CAAC,SAAS,CAAQ,CAAC,CAAC;QAEvF,wDAAwD;QACxD,MAAM,WAAW,GAAG,MAAM,MAAM,CAAC,4BAA4B,CAAC,CAAC;QAC/D,MAAM,aAAa,GAAG,EAAE,CAAC,KAAK,CAAC,WAAW,EAAE,kBAAkB,CAAC,CAAC,eAAe,CAAC;YAC9E;gBACE,IAAI,EAAE,qBAAqB;gBAC3B,IAAI,EAAE,4CAA4C;gBAClD,WAAW,EAAE,uBAAuB;gBACpC,MAAM,EAAE,mDAAmD;gBAC3D,KAAK,EAAE,sCAAsC;aAC9C;SACF,CAAC,CAAC;QAEH,MAAM,aAAa,GAAG;YACpB,SAAS,EAAE,sBAAsB;YACjC,OAAO,EAAE;gBACP,aAAa,EAAE,CAAC;gBAChB,WAAW,EAAE,CAAC;gBACd,UAAU,EAAE,EAAE,QAAQ,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,GAAG,EAAE,CAAC,EAAE;gBACvD,UAAU,EAAE;oBACV,QAAQ,EAAE,CAAC;oBACX,OAAO,EAAE,CAAC;oBACV,KAAK,EAAE,CAAC;oBACR,WAAW,EAAE,CAAC;oBACd,aAAa,EAAE,CAAC;iBACjB;aACF;YACD,MAAM,EAAE;gBACN;oBACE,QAAQ,EAAE,eAAe;oBACzB,QAAQ,EAAE,QAAQ;oBAClB,KAAK,EAAE,2BAA2B;oBAClC,IAAI,EAAE,YAAY;oBAClB,IAAI,EAAE,CAAC;oBACP,OAAO,EAAE,yCAAyC;oBAClD,QAAQ,EAAE,iCAAiC;oBAC3C,UAAU,EAAE,EAAE;oBACd,KAAK,EAAE,cAAc;iBACtB;aACF;SACF,CAAC;QAEF,KAAK,CAAC,cAAc,CAAC,kBAAkB,CAAC,KAAK,SAAS,CAAC;YACrD,MAAM;gBACJ,EAAE,EAAE,aAAa;gBACjB,IAAI,EAAE,WAAW;gBACjB,OAAO,EAAE,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC;gBACtC,SAAS,EAAE,IAAI,IAAI,CAAC,sBAAsB,CAAC;aAC5C,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,MAAM,kBAAkB,GAAG;YACzB,GAAG,iBAAiB;YACpB,MAAM,EAAE;gBACN,GAAG,iBAAiB,CAAC,MAAM;gBAC3B,MAAM,EAAE,CAAC,cAAc,CAAC;aACzB;SACsB,CAAC;QAE1B,MAAM,WAAW,CAAC,kBAAkB,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,UAAU,EAAE,KAAK,EAAE,KAAK,EAAE,KAAK,EAAE,CAAC,CAAC;QAE1F,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;QACrD,MAAM,CAAC,KAAK,CAAC,aAAa,CAAC,CAAC,oBAAoB,CAC9C,MAAM,CAAC,gBAAgB,CAAC;YACtB,KAAK,EAAE,qBAAqB;SAC7B,CAAC,CACH,CAAC;QAEF,MAAM,MAAM,GAAI,KAAK,CAAC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAyB,CAAC,OAAO,CAAC;QACrF,MAAM,CAAC,MAAM,CAAC,CAAC,SAAS,CAAC,YAAY,CAAC,CAAC;QAEvC,MAAM,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,oBAAoB,CAAC,CAAC,CAAC,CAAC;QAC5C,aAAa,CAAC,WAAW,EAAE,CAAC;QAC5B,OAAO,CAAC,WAAW,EAAE,CAAC;IACxB,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
@@ -1,5 +1,6 @@
1
1
  import simpleGit from 'simple-git';
2
2
  import chalk from 'chalk';
3
+ import { exitProcess } from '../lib/exit.js';
3
4
  import { parseDiff, getChangedFiles, getFilesWithDiffs } from '../lib/diff-parser.js';
4
5
  import { formatTerminalIssue } from '../lib/comment-formatter.js';
5
6
  import { executeReview, displayReviewSummary, hasBlockingIssues, } from '../lib/review-orchestrator.js';
@@ -43,11 +44,11 @@ export async function reviewLocal(config, options) {
43
44
  };
44
45
  const result = await executeReview(config, source);
45
46
  // Handle JSON output
46
- const wantsJsonOutput = options.jsonOutput || options.outputPath;
47
+ const wantsJsonOutput = options.jsonOutput === true ? true : Boolean(options.outputPath);
47
48
  if (wantsJsonOutput) {
48
49
  const jsonOutput = formatReviewJson(result.summary, result.issues, {
49
50
  source: `local-${options.staged ? 'staged' : 'unstaged'}`,
50
- });
51
+ }, result.usage);
51
52
  if (options.outputPath) {
52
53
  await writeReviewJson(jsonOutput, options.outputPath, cwd);
53
54
  console.log(chalk.green(`\n✓ Review results written to ${options.outputPath}\n`));
@@ -68,7 +69,7 @@ export async function reviewLocal(config, options) {
68
69
  // Recommendation
69
70
  if (hasBlockingIssues(result)) {
70
71
  console.log(chalk.red.bold('\n⚠️ Recommendation: Fix critical/high issues before pushing\n'));
71
- process.exit(1);
72
+ exitProcess(1);
72
73
  }
73
74
  else {
74
75
  console.log(chalk.green('\n✓ No critical issues found. Safe to push.\n'));
@@ -81,7 +82,7 @@ export async function reviewLocal(config, options) {
81
82
  else {
82
83
  // Still exit with error code for blocking issues even in JSON mode
83
84
  if (hasBlockingIssues(result)) {
84
- process.exit(1);
85
+ exitProcess(1);
85
86
  }
86
87
  }
87
88
  }
@@ -1 +1 @@
1
- {"version":3,"file":"review-local.js","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,YAAY,CAAC;AACnC,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AACtF,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EACL,aAAa,EACb,oBAAoB,EACpB,iBAAiB,GAElB,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAS3F;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,MAAiB,EAAE,OAA2B;IAC9E,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC,CAAC;IAEjE,MAAM,GAAG,GAAG,SAAS,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAE1B,qCAAqC;IACrC,MAAM,MAAM,GAAG,MAAM,GAAG,CAAC,WAAW,EAAE,CAAC;IACvC,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CAAC,sEAAsE,CAAC,CAAC;IAC1F,CAAC;IAED,0CAA0C;IAC1C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,eAAe,CAAC,CAAC,CAAC;IAE1F,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;IAElF,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,0BAA0B,CAAC,CAAC,CAAC;QACtD,OAAO;IACT,CAAC;IAED,mDAAmD;IACnD,MAAM,KAAK,GAAG,SAAS,CAAC,QAAQ,CAAC,CAAC;IAClC,MAAM,YAAY,GAAG,eAAe,CAAC,KAAK,CAAC,CAAC;IAC5C,MAAM,cAAc,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;IAEhD,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wBAAwB,CAAC,CAAC,CAAC;QACpD,OAAO;IACT,CAAC;IAED,wEAAwE;IACxE,MAAM,MAAM,GAAiB;QAC3B,IAAI,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,OAAO;QAC5D,KAAK,EAAE,YAAY;QACnB,cAAc,EAAE,2DAA2D;QAC3E,OAAO,EAAE,EAAE;QACX,UAAU,EAAE,GAAG;QACf,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,MAAM,EAAE,OAAO,CAAC,MAAM;KACvB,CAAC;IAEF,MAAM,MAAM,GAAG,MAAM,aAAa,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEnD,qBAAqB;IACrB,MAAM,eAAe,GAAG,OAAO,CAAC,UAAU,IAAI,OAAO,CAAC,UAAU,CAAC;IAEjE,IAAI,eAAe,EAAE,CAAC;QACpB,MAAM,UAAU,GAAG,gBAAgB,CAAC,MAAM,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE;YACjE,MAAM,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,EAAE;SAC1D,CAAC,CAAC;QAEH,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,MAAM,eAAe,CAAC,UAAU,EAAE,OAAO,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC;YAC3D,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,iCAAiC,OAAO,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC;QACpF,CAAC;QAED,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,eAAe,CAAC,UAAU,CAAC,CAAC;QAC9B,CAAC;IACH,CAAC;IAED,uDAAuD;IACvD,0DAA0D;IAC1D,IAAI,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC;QACxB,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,oBAAoB,CAAC,MAAM,CAAC,CAAC;YAE7B,6BAA6B;YAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;gBAClC,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC,CAAC;YAC1C,CAAC;YAED,iBAAiB;YACjB,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;gBAC9B,OAAO,CAAC,GAAG,CACT,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAClF,CAAC;gBACF,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;YAClB,CAAC;iBAAM,CAAC;gBACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC,CAAC;YAC5E,CAAC;QACH,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC,CAAC;QACtE,CAAC;IACH,CAAC;SAAM,CAAC;QACN,mEAAmE;QACnE,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;YAC9B,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAClB,CAAC;IACH,CAAC;AACH,CAAC"}
1
+ {"version":3,"file":"review-local.js","sourceRoot":"","sources":["../../src/cli/review-local.ts"],"names":[],"mappings":"AAAA,OAAO,SAAS,MAAM,YAAY,CAAC;AACnC,OAAO,KAAK,MAAM,OAAO,CAAC;AAE1B,OAAO,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAC;AAC7C,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AACtF,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EACL,aAAa,EACb,oBAAoB,EACpB,iBAAiB,GAElB,MAAM,+BAA+B,CAAC;AACvC,OAAO,EAAE,gBAAgB,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAS3F;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,MAAiB,EAAE,OAA2B;IAC9E,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,kCAAkC,CAAC,CAAC,CAAC;IAEjE,MAAM,GAAG,GAAG,SAAS,EAAE,CAAC;IACxB,MAAM,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;IAE1B,qCAAqC;IACrC,MAAM,MAAM,GAAG,MAAM,GAAG,CAAC,WAAW,EAAE,CAAC;IACvC,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,IAAI,KAAK,CAAC,sEAAsE,CAAC,CAAC;IAC1F,CAAC;IAED,0CAA0C;IAC1C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,WAAW,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,eAAe,CAAC,CAAC,CAAC;IAE1F,MAAM,QAAQ,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;IAElF,IAAI,CAAC,QAAQ,IAAI,QAAQ,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9C,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,0BAA0B,CAAC,CAAC,CAAC;QACtD,OAAO;IACT,CAAC;IAED,mDAAmD;IACnD,MAAM,KAAK,GAAG,SAAS,CAAC,QAAQ,CAAC,CAAC;IAClC,MAAM,YAAY,GAAG,eAAe,CAAC,KAAK,CAAC,CAAC;IAC5C,MAAM,cAAc,GAAG,iBAAiB,CAAC,KAAK,CAAC,CAAC;IAEhD,IAAI,YAAY,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,wBAAwB,CAAC,CAAC,CAAC;QACpD,OAAO;IACT,CAAC;IAED,wEAAwE;IACxE,MAAM,MAAM,GAAiB;QAC3B,IAAI,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,OAAO;QAC5D,KAAK,EAAE,YAAY;QACnB,cAAc,EAAE,2DAA2D;QAC3E,OAAO,EAAE,EAAE;QACX,UAAU,EAAE,GAAG;QACf,KAAK,EAAE,OAAO,CAAC,KAAK;QACpB,MAAM,EAAE,OAAO,CAAC,MAAM;KACvB,CAAC;IAEF,MAAM,MAAM,GAAG,MAAM,aAAa,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAEnD,qBAAqB;IACrB,MAAM,eAAe,GAAG,OAAO,CAAC,UAAU,KAAK,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC;IAEzF,IAAI,eAAe,EAAE,CAAC;QACpB,MAAM,UAAU,GAAG,gBAAgB,CACjC,MAAM,CAAC,OAAO,EACd,MAAM,CAAC,MAAM,EACb;YACE,MAAM,EAAE,SAAS,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,UAAU,EAAE;SAC1D,EACD,MAAM,CAAC,KAAK,CACb,CAAC;QAEF,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,MAAM,eAAe,CAAC,UAAU,EAAE,OAAO,CAAC,UAAU,EAAE,GAAG,CAAC,CAAC;YAC3D,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,iCAAiC,OAAO,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC;QACpF,CAAC;QAED,IAAI,OAAO,CAAC,UAAU,EAAE,CAAC;YACvB,eAAe,CAAC,UAAU,CAAC,CAAC;QAC9B,CAAC;IACH,CAAC;IAED,uDAAuD;IACvD,0DAA0D;IAC1D,IAAI,CAAC,OAAO,CAAC,UAAU,EAAE,CAAC;QACxB,IAAI,MAAM,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,oBAAoB,CAAC,MAAM,CAAC,CAAC;YAE7B,6BAA6B;YAC7B,KAAK,MAAM,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,CAAC;gBAClC,OAAO,CAAC,GAAG,CAAC,mBAAmB,CAAC,KAAK,CAAC,CAAC,CAAC;YAC1C,CAAC;YAED,iBAAiB;YACjB,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;gBAC9B,OAAO,CAAC,GAAG,CACT,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,iEAAiE,CAAC,CAClF,CAAC;gBACF,WAAW,CAAC,CAAC,CAAC,CAAC;YACjB,CAAC;iBAAM,CAAC;gBACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,+CAA+C,CAAC,CAAC,CAAC;YAC5E,CAAC;QACH,CAAC;aAAM,CAAC;YACN,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,KAAK,CAAC,yCAAyC,CAAC,CAAC,CAAC;QACtE,CAAC;IACH,CAAC;SAAM,CAAC;QACN,mEAAmE;QACnE,IAAI,iBAAiB,CAAC,MAAM,CAAC,EAAE,CAAC;YAC9B,WAAW,CAAC,CAAC,CAAC,CAAC;QACjB,CAAC;IACH,CAAC;AACH,CAAC"}
@@ -0,0 +1,2 @@
1
+ export {};
2
+ //# sourceMappingURL=review-local.live.e2e.test.d.ts.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"review-local.live.e2e.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-local.live.e2e.test.ts"],"names":[],"mappings":""}
@@ -0,0 +1,154 @@
1
+ /**
2
+ * Live E2E test for review-local.
3
+ *
4
+ * Run manually (never in CI):
5
+ * DRS_E2E_LIVE=1 npm test -- src/cli/review-local.live.e2e.test.ts
6
+ *
7
+ * Optional:
8
+ * DRS_E2E_MODEL=provider/model-id
9
+ */
10
+ import { spawn } from 'child_process';
11
+ import { existsSync, mkdtempSync, mkdirSync, readFileSync, rmSync, writeFileSync } from 'fs';
12
+ import { tmpdir } from 'os';
13
+ import { join, resolve } from 'path';
14
+ import simpleGit from 'simple-git';
15
+ import { config as loadDotenv } from 'dotenv';
16
+ import { describe, expect, it } from 'vitest';
17
+ loadDotenv();
18
+ const shouldRunLiveE2E = !process.env.CI && process.env.DRS_E2E_LIVE === '1';
19
+ const liveModel = process.env.DRS_E2E_MODEL ?? process.env.REVIEW_DEFAULT_MODEL ?? 'opencode/minimax-m2.5-free';
20
+ function runReviewLocalCli(cwd, outputPath) {
21
+ return new Promise((resolvePromise, reject) => {
22
+ const repoRoot = process.cwd();
23
+ const tsxBin = resolve(repoRoot, 'node_modules', '.bin', process.platform === 'win32' ? 'tsx.cmd' : 'tsx');
24
+ const child = spawn(tsxBin, [resolve(repoRoot, 'src/cli/index.ts'), 'review-local', '--output', outputPath], {
25
+ cwd,
26
+ env: {
27
+ ...process.env,
28
+ REVIEW_DEFAULT_MODEL: liveModel,
29
+ },
30
+ stdio: ['ignore', 'pipe', 'pipe'],
31
+ });
32
+ let logs = '';
33
+ child.stdout.on('data', (chunk) => {
34
+ logs += chunk.toString();
35
+ });
36
+ child.stderr.on('data', (chunk) => {
37
+ logs += chunk.toString();
38
+ });
39
+ child.on('error', reject);
40
+ child.on('close', (code) => {
41
+ resolvePromise({ code: code ?? -1, logs });
42
+ });
43
+ });
44
+ }
45
+ const describeLive = shouldRunLiveE2E ? describe : describe.skip;
46
+ describeLive('review-local live e2e (real LLM)', () => {
47
+ it('runs full review-local pipeline with skill usage and writes JSON output', async () => {
48
+ const tempRoot = mkdtempSync(join(tmpdir(), 'drs-live-e2e-'));
49
+ const repoDir = join(tempRoot, 'repo');
50
+ const outputPath = 'review-local-live-output.json';
51
+ const hasAnyProviderCredential = Boolean(process.env.ANTHROPIC_API_KEY ??
52
+ process.env.OPENAI_API_KEY ??
53
+ process.env.ZHIPU_API_KEY ??
54
+ process.env.OPENCODE_API_KEY);
55
+ if (!hasAnyProviderCredential) {
56
+ throw new Error('Live E2E requires provider credentials. Set ANTHROPIC_API_KEY/OPENAI_API_KEY/ZHIPU_API_KEY or another supported provider key.');
57
+ }
58
+ try {
59
+ mkdirSync(repoDir, { recursive: true });
60
+ const git = simpleGit(repoDir);
61
+ await git.init();
62
+ await git.addConfig('user.name', 'DRS E2E');
63
+ await git.addConfig('user.email', 'drs-e2e@example.com');
64
+ mkdirSync(join(repoDir, 'src'), { recursive: true });
65
+ mkdirSync(join(repoDir, '.drs'), { recursive: true });
66
+ mkdirSync(join(repoDir, '.drs', 'skills', 'cli-testing'), { recursive: true });
67
+ mkdirSync(join(repoDir, '.drs', 'agents', 'security'), { recursive: true });
68
+ mkdirSync(join(repoDir, '.drs', 'agents', 'sql-reviewer'), { recursive: true });
69
+ writeFileSync(join(repoDir, '.drs', 'skills', 'cli-testing', 'SKILL.md'), [
70
+ '---',
71
+ 'name: cli-testing',
72
+ 'description: Validate CLI flag behavior and coverage',
73
+ '---',
74
+ '',
75
+ '# CLI Testing Skill',
76
+ '',
77
+ '- Check new or changed CLI flags for behavior and test coverage.',
78
+ '- Flag missing integration tests for CLI option changes.',
79
+ '',
80
+ ].join('\n'), 'utf-8');
81
+ writeFileSync(join(repoDir, '.drs', 'agents', 'security', 'agent.md'), [
82
+ '---',
83
+ 'description: Security agent override for live E2E skill assertion',
84
+ '---',
85
+ '',
86
+ 'You are a security review agent.',
87
+ '',
88
+ 'Before your analysis, call the skill tool to load `cli-testing` exactly once.',
89
+ 'Then continue your security review and follow all output instructions from the user prompt.',
90
+ '',
91
+ ].join('\n'), 'utf-8');
92
+ writeFileSync(join(repoDir, '.drs', 'agents', 'sql-reviewer', 'agent.md'), [
93
+ '---',
94
+ 'description: SQL injection and database query reviewer',
95
+ '---',
96
+ '',
97
+ 'You are a database query reviewer specializing in SQL injection detection.',
98
+ 'Review code changes for unsafe SQL patterns and follow all output instructions from the user prompt.',
99
+ '',
100
+ ].join('\n'), 'utf-8');
101
+ writeFileSync(join(repoDir, '.drs/drs.config.yaml'), [
102
+ 'review:',
103
+ ' default:',
104
+ ` model: ${liveModel}`,
105
+ ' skills:',
106
+ ' - cli-testing',
107
+ ' mode: multi-agent',
108
+ ' agents:',
109
+ ' - security',
110
+ ' - sql-reviewer',
111
+ ' ignorePatterns: []',
112
+ ' describe:',
113
+ ' enabled: false',
114
+ ' postDescription: false',
115
+ 'contextCompression:',
116
+ ' enabled: false',
117
+ '',
118
+ ].join('\n'), 'utf-8');
119
+ writeFileSync(join(repoDir, 'src', 'query.ts'), ['export function findUser(userId: string) {', ' return userId.trim();', '}', ''].join('\n'), 'utf-8');
120
+ await git.add('.');
121
+ await git.commit('baseline for live e2e');
122
+ writeFileSync(join(repoDir, 'src', 'query.ts'), [
123
+ 'export function findUser(userId: string) {',
124
+ ' const sql = "SELECT * FROM users WHERE id = " + userId;',
125
+ ' return sql;',
126
+ '}',
127
+ '',
128
+ ].join('\n'), 'utf-8');
129
+ const result = await runReviewLocalCli(repoDir, outputPath);
130
+ if (!new Set([0, 1]).has(result.code)) {
131
+ throw new Error(`review-local exited with code ${result.code}.\nLogs:\n${result.logs}`);
132
+ }
133
+ const outputFile = join(repoDir, outputPath);
134
+ if (!existsSync(outputFile)) {
135
+ throw new Error(`Expected output file was not created.\nLogs:\n${result.logs}`);
136
+ }
137
+ const outputRaw = readFileSync(outputFile, 'utf-8');
138
+ const output = JSON.parse(outputRaw);
139
+ expect(output.summary.filesReviewed).toBeGreaterThanOrEqual(1);
140
+ expect(Array.isArray(output.issues)).toBe(true);
141
+ expect(output.metadata?.source).toBe('local-unstaged');
142
+ expect(result.logs).toContain('Loaded skill: cli-testing');
143
+ // Verify custom agent override from .drs/agents/security/agent.md was loaded
144
+ expect(result.logs).toContain('agent definitions for Pi runtime');
145
+ // Verify both agents ran: built-in override + brand new custom agent
146
+ expect(result.logs).toContain('Selected Agents: security, sql-reviewer');
147
+ expect(result.logs).toContain('Running sql-reviewer review');
148
+ }
149
+ finally {
150
+ rmSync(tempRoot, { recursive: true, force: true });
151
+ }
152
+ }, 300000);
153
+ });
154
+ //# sourceMappingURL=review-local.live.e2e.test.js.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"review-local.live.e2e.test.js","sourceRoot":"","sources":["../../src/cli/review-local.live.e2e.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,EAAE,KAAK,EAAE,MAAM,eAAe,CAAC;AACtC,OAAO,EAAE,UAAU,EAAE,WAAW,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,IAAI,CAAC;AAC7F,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAC5B,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AACrC,OAAO,SAAS,MAAM,YAAY,CAAC;AACnC,OAAO,EAAE,MAAM,IAAI,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC9C,OAAO,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAE9C,UAAU,EAAE,CAAC;AAEb,MAAM,gBAAgB,GAAG,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,IAAI,OAAO,CAAC,GAAG,CAAC,YAAY,KAAK,GAAG,CAAC;AAC7E,MAAM,SAAS,GACb,OAAO,CAAC,GAAG,CAAC,aAAa,IAAI,OAAO,CAAC,GAAG,CAAC,oBAAoB,IAAI,4BAA4B,CAAC;AAEhG,SAAS,iBAAiB,CACxB,GAAW,EACX,UAAkB;IAElB,OAAO,IAAI,OAAO,CAAC,CAAC,cAAc,EAAE,MAAM,EAAE,EAAE;QAC5C,MAAM,QAAQ,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;QAC/B,MAAM,MAAM,GAAG,OAAO,CACpB,QAAQ,EACR,cAAc,EACd,MAAM,EACN,OAAO,CAAC,QAAQ,KAAK,OAAO,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,KAAK,CACjD,CAAC;QAEF,MAAM,KAAK,GAAG,KAAK,CACjB,MAAM,EACN,CAAC,OAAO,CAAC,QAAQ,EAAE,kBAAkB,CAAC,EAAE,cAAc,EAAE,UAAU,EAAE,UAAU,CAAC,EAC/E;YACE,GAAG;YACH,GAAG,EAAE;gBACH,GAAG,OAAO,CAAC,GAAG;gBACd,oBAAoB,EAAE,SAAS;aAChC;YACD,KAAK,EAAE,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,CAAC;SAClC,CACF,CAAC;QAEF,IAAI,IAAI,GAAG,EAAE,CAAC;QACd,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE;YACxC,IAAI,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;QAC3B,CAAC,CAAC,CAAC;QACH,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,KAAa,EAAE,EAAE;YACxC,IAAI,IAAI,KAAK,CAAC,QAAQ,EAAE,CAAC;QAC3B,CAAC,CAAC,CAAC;QAEH,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC;QAC1B,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE;YACzB,cAAc,CAAC,EAAE,IAAI,EAAE,IAAI,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC;QAC7C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC;AAED,MAAM,YAAY,GAAG,gBAAgB,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,QAAQ,CAAC,IAAI,CAAC;AAEjE,YAAY,CAAC,kCAAkC,EAAE,GAAG,EAAE;IACpD,EAAE,CAAC,yEAAyE,EAAE,KAAK,IAAI,EAAE;QACvF,MAAM,QAAQ,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,eAAe,CAAC,CAAC,CAAC;QAC9D,MAAM,OAAO,GAAG,IAAI,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QACvC,MAAM,UAAU,GAAG,+BAA+B,CAAC;QAEnD,MAAM,wBAAwB,GAAG,OAAO,CACtC,OAAO,CAAC,GAAG,CAAC,iBAAiB;YAC7B,OAAO,CAAC,GAAG,CAAC,cAAc;YAC1B,OAAO,CAAC,GAAG,CAAC,aAAa;YACzB,OAAO,CAAC,GAAG,CAAC,gBAAgB,CAC7B,CAAC;QAEF,IAAI,CAAC,wBAAwB,EAAE,CAAC;YAC9B,MAAM,IAAI,KAAK,CACb,+HAA+H,CAChI,CAAC;QACJ,CAAC;QAED,IAAI,CAAC;YACH,SAAS,CAAC,OAAO,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YAExC,MAAM,GAAG,GAAG,SAAS,CAAC,OAAO,CAAC,CAAC;YAC/B,MAAM,GAAG,CAAC,IAAI,EAAE,CAAC;YACjB,MAAM,GAAG,CAAC,SAAS,CAAC,WAAW,EAAE,SAAS,CAAC,CAAC;YAC5C,MAAM,GAAG,CAAC,SAAS,CAAC,YAAY,EAAE,qBAAqB,CAAC,CAAC;YAEzD,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,KAAK,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YACrD,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YACtD,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,aAAa,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YAC/E,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,UAAU,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YAC5E,SAAS,CAAC,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,cAAc,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;YAEhF,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,aAAa,EAAE,UAAU,CAAC,EAC1D;gBACE,KAAK;gBACL,mBAAmB;gBACnB,sDAAsD;gBACtD,KAAK;gBACL,EAAE;gBACF,qBAAqB;gBACrB,EAAE;gBACF,kEAAkE;gBAClE,0DAA0D;gBAC1D,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,EACZ,OAAO,CACR,CAAC;YAEF,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,UAAU,EAAE,UAAU,CAAC,EACvD;gBACE,KAAK;gBACL,mEAAmE;gBACnE,KAAK;gBACL,EAAE;gBACF,kCAAkC;gBAClC,EAAE;gBACF,+EAA+E;gBAC/E,6FAA6F;gBAC7F,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,EACZ,OAAO,CACR,CAAC;YAEF,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,cAAc,EAAE,UAAU,CAAC,EAC3D;gBACE,KAAK;gBACL,wDAAwD;gBACxD,KAAK;gBACL,EAAE;gBACF,4EAA4E;gBAC5E,sGAAsG;gBACtG,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,EACZ,OAAO,CACR,CAAC;YAEF,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,sBAAsB,CAAC,EACrC;gBACE,SAAS;gBACT,YAAY;gBACZ,cAAc,SAAS,EAAE;gBACzB,aAAa;gBACb,qBAAqB;gBACrB,qBAAqB;gBACrB,WAAW;gBACX,gBAAgB;gBAChB,oBAAoB;gBACpB,sBAAsB;gBACtB,aAAa;gBACb,oBAAoB;gBACpB,4BAA4B;gBAC5B,qBAAqB;gBACrB,kBAAkB;gBAClB,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,EACZ,OAAO,CACR,CAAC;YAEF,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,KAAK,EAAE,UAAU,CAAC,EAChC,CAAC,4CAA4C,EAAE,yBAAyB,EAAE,GAAG,EAAE,EAAE,CAAC,CAAC,IAAI,CACrF,IAAI,CACL,EACD,OAAO,CACR,CAAC;YAEF,MAAM,GAAG,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;YACnB,MAAM,GAAG,CAAC,MAAM,CAAC,uBAAuB,CAAC,CAAC;YAE1C,aAAa,CACX,IAAI,CAAC,OAAO,EAAE,KAAK,EAAE,UAAU,CAAC,EAChC;gBACE,4CAA4C;gBAC5C,2DAA2D;gBAC3D,eAAe;gBACf,GAAG;gBACH,EAAE;aACH,CAAC,IAAI,CAAC,IAAI,CAAC,EACZ,OAAO,CACR,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,iBAAiB,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;YAE5D,IAAI,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;gBACtC,MAAM,IAAI,KAAK,CAAC,iCAAiC,MAAM,CAAC,IAAI,aAAa,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;YAC1F,CAAC;YAED,MAAM,UAAU,GAAG,IAAI,CAAC,OAAO,EAAE,UAAU,CAAC,CAAC;YAC7C,IAAI,CAAC,UAAU,CAAC,UAAU,CAAC,EAAE,CAAC;gBAC5B,MAAM,IAAI,KAAK,CAAC,iDAAiD,MAAM,CAAC,IAAI,EAAE,CAAC,CAAC;YAClF,CAAC;YAED,MAAM,SAAS,GAAG,YAAY,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;YACpD,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,SAAS,CAIlC,CAAC;YAEF,MAAM,CAAC,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAC;YAC/D,MAAM,CAAC,KAAK,CAAC,OAAO,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;YAChD,MAAM,CAAC,MAAM,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;YACvD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,2BAA2B,CAAC,CAAC;YAE3D,6EAA6E;YAC7E,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,kCAAkC,CAAC,CAAC;YAElE,qEAAqE;YACrE,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,yCAAyC,CAAC,CAAC;YACzE,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,SAAS,CAAC,6BAA6B,CAAC,CAAC;QAC/D,CAAC;gBAAS,CAAC;YACT,MAAM,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QACrD,CAAC;IACH,CAAC,EAAE,MAAM,CAAC,CAAC;AACb,CAAC,CAAC,CAAC"}
@@ -0,0 +1,2 @@
1
+ export {};
2
+ //# sourceMappingURL=review-local.test.d.ts.map
@@ -0,0 +1 @@
1
+ {"version":3,"file":"review-local.test.d.ts","sourceRoot":"","sources":["../../src/cli/review-local.test.ts"],"names":[],"mappings":""}