kodevu 0.1.65 → 0.1.67

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 (2) hide show
  1. package/package.json +1 -1
  2. package/src/reviewers.js +15 -8
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "kodevu",
3
- "version": "0.1.65",
3
+ "version": "0.1.67",
4
4
  "license": "MIT",
5
5
  "type": "module",
6
6
  "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",
package/src/reviewers.js CHANGED
@@ -180,13 +180,8 @@ export const REVIEWERS = {
180
180
  "utf8"
181
181
  );
182
182
 
183
- const args = [
184
- "run",
185
- "Please perform the code review strictly following the instructions and unified diff provided in the attached file.",
186
- "-f",
187
- reviewInputFile,
188
- "--pure"
189
- ];
183
+ // Use a short message on the command line and pass full instructions via -f file
184
+ const args = ["run", "Review attached file", "-f", reviewInputFile, "--pure"];
190
185
 
191
186
  const execResult = await runCommand("opencode", args, {
192
187
  cwd: workingDir,
@@ -195,9 +190,21 @@ export const REVIEWERS = {
195
190
  debug: config.debug
196
191
  });
197
192
 
193
+ const message = execResult.stdout || execResult.stderr || "";
194
+
195
+ // Treat non-zero exit codes as failures and throw so callers can handle them.
196
+ if (typeof execResult.code === "number" && execResult.code !== 0) {
197
+ const err = new Error(
198
+ `OpenCode exited with code ${execResult.code}: ${execResult.stderr || execResult.stdout || ""}`
199
+ );
200
+ // attach execResult for callers that want more details
201
+ err.execResult = execResult;
202
+ throw err;
203
+ }
204
+
198
205
  return {
199
206
  ...execResult,
200
- message: execResult.stdout
207
+ message
201
208
  };
202
209
  } finally {
203
210
  await fs.rm(tempDir, { recursive: true, force: true });