kodevu 0.1.33 → 0.1.36

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.33",
+  "version": "0.1.36",
   "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

@@ -214,6 +214,7 @@ export async function resolveConfig(cliArgs = {}) {
     config.target = process.cwd();
   }
 
+  config.baseDir = process.cwd();
   config.debug = Boolean(cliArgs.debug);
   config.reviewer = String(config.reviewer || "auto").toLowerCase();
   config.lang = String(config.lang || "auto").toLowerCase();
@@ -265,8 +266,8 @@ Options:
   --reviewer, -r    Reviewer (codex | gemini | copilot | auto, default: auto)
   --prompt, -p      Additional instructions or @file.txt to read from file
   --lang, -l        Output language (e.g. zh, en, auto)
-  --rev, -v         Review a specific revision or commit hash
-  --last, -n        Review the latest N revisions (default: 1)
+  --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)
   --output, -o      Output directory (default: ~/.kodevu)
   --format, -f      Output formats (markdown, json, comma-separated)
   --debug, -d       Print extra debug information

package/src/git-client.js

@@ -105,6 +105,25 @@ export async function isValidCheckpoint(config, targetInfo, checkpointCommit, la
   return ancestorResult.code === 0;
 }
 
+export async function resolveCommits(config, targetInfo, revSpec) {
+  const result = await runGit(
+    config,
+    ["rev-list", revSpec],
+    { cwd: targetInfo.repoRootPath, trim: true, allowFailure: true }
+  );
+
+  if (result.code !== 0) {
+    // Attempt fallback to a single hash resolution if rev-list fails (e.g. for non-standard specs)
+    const single = await runGit(config, ["rev-parse", revSpec], {
+      cwd: targetInfo.repoRootPath, trim: true, allowFailure: true
+    });
+    if (single.code === 0) return [single.stdout.trim()];
+    throw new Error(`Failed to resolve Git revision: ${revSpec}`);
+  }
+
+  return splitLines(result.stdout);
+}
+
 export async function getPendingCommits(config, targetInfo, startExclusive, endInclusive, limit) {
   const args = ["rev-list", "--reverse"];
 

package/src/review-runner.js

@@ -20,6 +20,7 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
   logger.info(`Starting review for ${backend.changeName} ${displayId}`);
   progress?.update(0.05, "loading change details");
   const details = await backend.getChangeDetails(config, targetInfo, changeId);
+  const resolvedChangeId = details.id;
 
   if (details.changedPaths.length === 0) {
     progress?.update(0.7, "writing skipped report");
@@ -29,7 +30,7 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
       "No file changes were captured for this change under the configured target."
     ].join("\n");
 
-    const markdownReportFile = path.join(config.outputDir, backend.getReportFileName(changeId));
+    const markdownReportFile = path.join(config.outputDir, backend.getReportFileName(resolvedChangeId));
     const jsonReportFile = markdownReportFile.replace(/\.md$/i, ".json");
 
     if (shouldWriteFormat(config, "markdown")) {
@@ -55,7 +56,7 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
   }
 
   progress?.update(0.2, "loading diff");
-  const diffText = await backend.getChangeDiff(config, targetInfo, changeId);
+  const diffText = await backend.getChangeDiff(config, targetInfo, resolvedChangeId);
   const reviewersToTry = [config.reviewer, ...(config.fallbackReviewers || [])];
 
   let reviewer;
@@ -99,7 +100,7 @@ async function reviewChange(config, backend, targetInfo, changeId, progress) {
   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);
-  const outputFile = path.join(config.outputDir, backend.getReportFileName(changeId));
+  const outputFile = path.join(config.outputDir, backend.getReportFileName(resolvedChangeId));
   const jsonOutputFile = outputFile.replace(/\.md$/i, ".json");
 
   if (shouldWriteFormat(config, "markdown")) {
@@ -156,7 +157,7 @@ export async function runReviewCycle(config) {
   let changeIdsToReview = [];
 
   if (config.rev) {
-    changeIdsToReview = [config.rev];
+    changeIdsToReview = await backend.resolveChangeIds(config, targetInfo, config.rev);
   } else {
     changeIdsToReview = await backend.getLatestChangeIds(config, targetInfo, config.last || 1);
   }

package/src/vcs-client.js

@@ -36,6 +36,10 @@ function createSvnBackend() {
       return `${datePrefix}-svn-r${revision}.md`;
     },
 
+    async resolveChangeIds(config, targetInfo, revString) {
+      if (!revString) return [];
+      return String(revString).split(',').map(s => s.trim()).filter(Boolean);
+    },
     async getTargetInfo(config) {
       return await svnClient.getTargetInfo(config);
     },
@@ -83,6 +87,16 @@ function createGitBackend() {
       return `${datePrefix}-git-${commitHash.slice(0, 12)}.md`;
     },
 
+    async resolveChangeIds(config, targetInfo, revString) {
+      if (!revString) return [];
+      const specs = String(revString).split(',').map(s => s.trim()).filter(Boolean);
+      const allHashes = [];
+      for (const spec of specs) {
+        const hashes = await gitClient.resolveCommits(config, targetInfo, spec);
+        allHashes.push(...hashes);
+      }
+      return [...new Set(allHashes)];
+    },
     async getTargetInfo(config) {
       return await gitClient.getTargetInfo(config);
     },
@@ -99,8 +113,8 @@ function createGitBackend() {
       const details = await gitClient.getCommitDetails(config, targetInfo, commitHash);
 
       return {
-        id: commitHash,
-        displayId: commitHash.slice(0, 12),
+        id: details.commitHash,
+        displayId: details.commitHash.slice(0, 12),
         author: details.author,
         date: details.date,
         message: details.message,