kodevu 0.1.51 → 0.1.53

package/SKILL.md

@@ -0,0 +1,63 @@
+---
+name: kodevu
+description: A tool to fetch Git commits or SVN revisions, send the diff to a supported AI reviewer CLI, and write configurable review reports.
+---
+
+# Kodevu Skill
+
+Kodevu is a Node.js tool that fetches Git commits or SVN revisions, sends the diff to a supported AI reviewer CLI, and writes review results to report files.
+
+## Usage
+
+Use `npx kodevu` to review a codebase. It is designed to be stateless and requires no configuration files.
+
+### Reviewing the latest commit
+
+```bash
+npx kodevu .
+```
+
+### Reviewing a specific commit
+
+```bash
+npx kodevu . --rev <commit-hash>
+```
+
+### Reviewing the last N commits
+
+```bash
+npx kodevu . --last 3
+```
+
+### Supported Reviewers
+
+`kodevu` supports several AI reviewer CLIs: `auto`, `openai`, `gemini`, `codex`, `copilot`. The default is `auto`. Use the `--reviewer` option to override.
+
+Example using OpenAI:
+```bash
+npx kodevu . --reviewer openai --openai-api-key <YOUR_API_KEY> --openai-model gpt-4o
+```
+
+### Generating JSON Reports
+
+By default, review reports are generated as Markdown files in `~/.kodevu/`. You can specify `--format json` or change the output directory using `--output <dir>`.
+```bash
+npx kodevu . --format json --output ./reports
+```
+
+### Formatting the Prompt
+
+You can provide clear instructions to the reviewer using `--prompt`:
+```bash
+npx kodevu . --prompt "Focus on security issues and suggest optimizations."
+```
+Or from a file: `--prompt @my-rules.txt`
+
+## Working with Target Repositories
+
+- `target`: Repository path (Git) or SVN URL/Working copy (default: `.`).
+
+For example, to run on a specific path, you can use:
+```bash
+npx kodevu /path/to/project --last 1
+```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "kodevu",
-  "version": "0.1.51",
+  "version": "0.1.53",
   "license": "MIT",
   "type": "module",
   "description": "Poll SVN revisions or Git commits, send each change diff to a reviewer CLI, and write configurable review reports.",
@@ -9,7 +9,8 @@
   },
   "files": [
     "src",
-    "README.md"
+    "README.md",
+    "SKILL.md"
   ],
   "scripts": {
     "start": "node src/index.js",

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