kodevu 0.1.42 → 0.1.44

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "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/config.js

@@ -161,7 +161,8 @@ export function parseCliArgs(argv) {
     }
 
     if (value === "--last" || value === "-n") {
-      if (!hasNextValue) throw new Error(`Missing value for ${value}`);
+      const hasLastValue = nextValue !== undefined && /^-?\d+$/.test(nextValue);
+      if (!hasLastValue) throw new Error(`Missing value for ${value}`);
       args.last = nextValue;
       index += 1;
       continue;
@@ -258,7 +259,7 @@ export async function resolveConfig(cliArgs = {}) {
   config.last = Number(config.last);
   config.outputFormats = normalizeOutputFormats(config.outputFormats);
 
-  if (!config.rev && (isNaN(config.last) || config.last <= 0)) {
+  if (!config.rev && (isNaN(config.last) || config.last === 0)) {
     config.last = 1;
   }
 
@@ -277,7 +278,7 @@ Options:
   --prompt, -p      Additional instructions or @file.txt to read from file
   --lang, -l        Output language (e.g. zh, en, auto)
   --rev, -v         Review specific revision(s), hashes, branches or ranges (comma-separated)
-  --last, -n        Review the latest N revisions (ignored if --rev is provided) (default: 1)
+  --last, -n        Review the latest N revisions; use negative (-N) to review only the Nth-from-last revision (default: 1)
   --output, -o      Output directory (default: ~/.kodevu)
   --format, -f      Output formats (markdown, json, comma-separated)
   --debug, -d       Print extra debug information

package/src/report-generator.js

@@ -208,6 +208,7 @@ export function buildPrompt(config, backend, targetInfo, details, reviewDiffPayl
     canReadRelatedFiles
       ? `${getPhrase("workspaceRoot", lang)} ${workspaceRoot}\n${getPhrase("besidesDiff", lang)}`
       : getPhrase("noWorkspace", lang),
+    getPhrase("reviewFromDiff", lang),
     getPhrase("fileRefs", lang),
     getPhrase("langRule", lang, { langName }),
     getPhrase("outputDirective", lang)

package/src/review-runner.js

@@ -159,6 +159,9 @@ export async function runReviewCycle(config) {
 
   if (config.rev) {
     changeIdsToReview = await backend.resolveChangeIds(config, targetInfo, config.rev);
+  } else if (config.last < 0) {
+    const candidates = await backend.getLatestChangeIds(config, targetInfo, Math.abs(config.last));
+    changeIdsToReview = candidates.length > 0 ? [candidates[0]] : [];
   } else {
     changeIdsToReview = await backend.getLatestChangeIds(config, targetInfo, config.last || 1);
   }

package/src/reviewers.js

@@ -76,28 +76,50 @@ export const REVIEWERS = {
     responseSectionTitle: "Copilot Response",
     emptyResponseText: "_No final response returned from copilot._",
     async run(config, workingDir, promptText, diffText) {
+      const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), "kodevu-copilot-"));
+      const reviewInputFile = path.join(tempDir, "review-input.md");
+      const copilotPrompt = [
+        `Use the file-reading tools to open this exact file path: ${reviewInputFile}`,
+        "That file contains the full review instructions and the unified diff to review.",
+        "Follow the instructions from that file exactly and output the final review directly.",
+        "Do not ask clarifying questions. Do not mention tool usage. Do not say you are ready.",
+        "Start immediately with the review content."
+      ].join("\n");
       const args = [
         "-p",
-        promptText,
+        copilotPrompt,
         "-s",
         "--no-color",
         "--no-ask-user",
+        "--no-custom-instructions",
         "--allow-all-tools",
         "--add-dir",
-        workingDir
+        workingDir,
+        "--add-dir",
+        tempDir
       ];
-      const execResult = await runCommand("copilot", args, {
-        cwd: workingDir,
-        input: ["Unified diff:", diffText].join("\n\n"),
-        allowFailure: true,
-        timeoutMs: config.commandTimeoutMs,
-        debug: config.debug
-      });
 
-      return {
-        ...execResult,
-        message: execResult.stdout
-      };
+      try {
+        await fs.writeFile(
+          reviewInputFile,
+          [promptText, "### Unified Diff", "```diff", diffText, "```"].join("\n\n"),
+          "utf8"
+        );
+
+        const execResult = await runCommand("copilot", args, {
+          cwd: workingDir,
+          allowFailure: true,
+          timeoutMs: config.commandTimeoutMs,
+          debug: config.debug
+        });
+
+        return {
+          ...execResult,
+          message: execResult.stdout
+        };
+      } finally {
+        await fs.rm(tempDir, { recursive: true, force: true });
+      }
     }
   }
 };