kodevu 0.1.51 → 0.1.52

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.51",
+  "version": "0.1.52",
   "license": "MIT",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",

package/src/git-client.js

@@ -47,13 +47,20 @@ export async function getTargetInfo(config) {
   const lookupCwd = targetStat.isDirectory() ? requestedTargetPath : path.dirname(requestedTargetPath);
   const topLevelResult = await runGit(config, ["rev-parse", "--show-toplevel"], {
     cwd: lookupCwd,
-    trim: true
+    trim: true,
+    allowFailure: true
   });
+
+  if (topLevelResult.code !== 0) {
+    throw new Error(`Git target path is not within a Git repository: ${requestedTargetPath}`);
+  }
+
   const repoRootPath = path.resolve(topLevelResult.stdout);
   const relativeTargetPath = toPosixPath(path.relative(repoRootPath, requestedTargetPath));
   const branchResult = await runGit(config, ["rev-parse", "--abbrev-ref", "HEAD"], {
     cwd: repoRootPath,
-    trim: true
+    trim: true,
+    allowFailure: true
   });
 
   return {

package/src/review-runner.js

@@ -98,6 +98,12 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
     }
   }
 
+  if (!reviewerResult || reviewerResult.code !== 0 || reviewerResult.timedOut) {
+    throw new Error(
+      `${reviewer?.displayName || config.reviewer} failed for ${details.displayId}`
+    );
+  }
+
   progress?.update(0.82, "writing report");
   logger.debug(`Token usage: input=${tokenUsage.inputTokens} output=${tokenUsage.outputTokens} total=${tokenUsage.totalTokens} source=${tokenUsage.source}`);
   const report = buildReport(currentReviewerConfig, backend, targetInfo, details, diffPayloads, reviewer, reviewerResult, tokenUsage);
@@ -115,14 +121,6 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
     );
   }
 
-  if (reviewerResult.code !== 0 || reviewerResult.timedOut) {
-    throw new Error(
-      `${reviewer.displayName} failed for ${details.displayId}; report written to ${outputFile}${
-        shouldWriteFormat(config, "json") ? ` and ${jsonOutputFile}` : ""
-      }`
-    );
-  }
-
   const outputLabels = [
     shouldWriteFormat(config, "markdown") ? `md: ${outputFile}` : null,
     shouldWriteFormat(config, "json") ? `json: ${jsonOutputFile}` : null