jstar-reviewer 2.2.0 → 2.4.0

package/bin/jstar.js

@@ -29,7 +29,7 @@ function log(msg) {
 
 function printHelp() {
     log(`
-${COLORS.bold}🌟 J-Star Reviewer v2.2.0${COLORS.reset}
+${COLORS.bold}🌟 J-Star Reviewer v2.4.0${COLORS.reset}
 
 ${COLORS.dim}AI-powered code review with local embeddings${COLORS.reset}
 

package/dist/scripts/reviewer.js

@@ -91,6 +91,32 @@ function chunkDiffByFile(diff) {
 function sleep(ms) {
     return new Promise(resolve => setTimeout(resolve, ms));
 }
+async function getDefaultBranch() {
+    try {
+        // Method 1: Check remote HEAD (most reliable)
+        try {
+            const remote = await git.remote(['show', 'origin']);
+            if (typeof remote === 'string') {
+                const match = remote.match(/HEAD branch: (\S+)/);
+                if (match)
+                    return match[1];
+            }
+        }
+        catch (e) {
+            // No remote or network issue, proceed to local check
+        }
+        // Method 2: Check which common branch exists locally
+        const branches = await git.branchLocal();
+        if (branches.all.includes('main'))
+            return 'main';
+        if (branches.all.includes('master'))
+            return 'master';
+    }
+    catch (e) {
+        // Fallback
+    }
+    return 'main';
+}
 /**
  * Filter issues by confidence threshold and log what was removed
  */
@@ -172,12 +198,89 @@ async function main() {
     const detective = new detective_1.Detective(SOURCE_DIR);
     await detective.scan();
     detective.report();
-    // 1. Get the Diff
-    const diff = await git.diff(["--staged"]);
+    // 1. Determine Diff Target
+    const args = process.argv.slice(2);
+    let diff;
+    let reviewTarget = "Staged Changes";
+    if (args.includes('--last')) {
+        reviewTarget = "Last Commit";
+        diff = await git.diff(["HEAD~1", "HEAD"]);
+    }
+    else if (args.includes('--commit')) {
+        const hashIndex = args.indexOf('--commit') + 1;
+        if (hashIndex < args.length) {
+            const hash = args[hashIndex];
+            reviewTarget = `Commit ${hash}`;
+            diff = await git.diff([`${hash}~1`, `${hash}`]);
+        }
+        else {
+            logger_1.Logger.error(chalk_1.default.red("❌ Missing commit hash for --commit"));
+            return;
+        }
+    }
+    else if (args.includes('--range')) {
+        const rangeIndex = args.indexOf('--range') + 1;
+        if (rangeIndex + 1 < args.length) {
+            const start = args[rangeIndex];
+            const end = args[rangeIndex + 1];
+            reviewTarget = `Range ${start}..${end}`;
+            diff = await git.diff([start, end]);
+        }
+        else {
+            logger_1.Logger.error(chalk_1.default.red("❌ Missing arguments for --range (usage: --range <start> <end>)"));
+            return;
+        }
+    }
+    else if (args.includes('--pr')) {
+        const prIndex = args.indexOf('--pr');
+        // Check for --base flag first
+        let baseBranch = 'main';
+        if (args.includes('--base')) {
+            const baseIndex = args.indexOf('--base') + 1;
+            if (baseIndex < args.length) {
+                baseBranch = args[baseIndex];
+            }
+            else {
+                logger_1.Logger.error(chalk_1.default.red("❌ Missing branch name for --base"));
+                return;
+            }
+        }
+        else {
+            // Check positional argument (backward compatibility)
+            const potentialBase = args[prIndex + 1];
+            if (potentialBase && !potentialBase.startsWith('--')) {
+                baseBranch = potentialBase;
+            }
+            else {
+                // Auto-detect
+                baseBranch = await getDefaultBranch();
+            }
+        }
+        reviewTarget = `PR (HEAD vs ${baseBranch})`;
+        // Use triple-dot for merge-base difference (what a PR shows)
+        try {
+            diff = await git.diff([`${baseBranch}...HEAD`]);
+        }
+        catch (e) {
+            logger_1.Logger.error(chalk_1.default.red(`❌ Failed to diff against ${baseBranch}. Does the branch exist?`));
+            return;
+        }
+    }
+    else {
+        // Default: Staged changes
+        diff = await git.diff(["--staged"]);
+    }
     if (!diff) {
-        logger_1.Logger.info(chalk_1.default.green("\n✅ No staged changes to review. (Did you 'git add'?)"));
+        if (reviewTarget === "Staged Changes") {
+            logger_1.Logger.info(chalk_1.default.green("\n✅ No staged changes to review. (Did you 'git add'?)"));
+            logger_1.Logger.info(chalk_1.default.dim("   Tip: Use '--last' to review the previous commit."));
+        }
+        else {
+            logger_1.Logger.info(chalk_1.default.green(`\n✅ No changes found in ${reviewTarget}.`));
+        }
         return;
     }
+    logger_1.Logger.info(chalk_1.default.blue(`\n📝 Reviewing: ${reviewTarget}`));
     // 2. Load the Brain
     if (!fs.existsSync(STORAGE_DIR)) {
         logger_1.Logger.error(chalk_1.default.red("❌ Local Brain not found. Run 'pnpm run index:init' first."));

package/package.json

@@ -1,19 +1,10 @@
 {
   "name": "jstar-reviewer",
-  "version": "2.2.0",
+  "version": "2.4.0",
   "description": "Local-First, Context-Aware AI Code Reviewer - Works with any language",
   "bin": {
     "jstar": "bin/jstar.js"
   },
-  "scripts": {
-    "build": "tsc",
-    "index:init": "ts-node scripts/indexer.ts --init",
-    "index:watch": "ts-node scripts/indexer.ts --watch",
-    "review": "ts-node scripts/reviewer.ts",
-    "chat": "ts-node scripts/chat.ts",
-    "detect": "ts-node scripts/detective.ts",
-    "prepare": "husky install"
-  },
   "keywords": [
     "code-review",
     "ai",
@@ -65,5 +56,13 @@
   "type": "commonjs",
   "bugs": {
     "url": "https://github.com/JStaRFilms/jstar-code-review/issues"
+  },
+  "scripts": {
+    "build": "tsc",
+    "index:init": "ts-node scripts/indexer.ts --init",
+    "index:watch": "ts-node scripts/indexer.ts --watch",
+    "review": "ts-node scripts/reviewer.ts",
+    "chat": "ts-node scripts/chat.ts",
+    "detect": "ts-node scripts/detective.ts"
   }
 }

package/scripts/reviewer.ts

@@ -69,6 +69,30 @@ function sleep(ms: number): Promise<void> {
     return new Promise(resolve => setTimeout(resolve, ms));
 }
 
+async function getDefaultBranch(): Promise<string> {
+    try {
+        // Method 1: Check remote HEAD (most reliable)
+        try {
+            const remote = await git.remote(['show', 'origin']);
+            if (typeof remote === 'string') {
+                const match = remote.match(/HEAD branch: (\S+)/);
+                if (match) return match[1];
+            }
+        } catch (e) {
+            // No remote or network issue, proceed to local check
+        }
+
+        // Method 2: Check which common branch exists locally
+        const branches = await git.branchLocal();
+        if (branches.all.includes('main')) return 'main';
+        if (branches.all.includes('master')) return 'master';
+
+    } catch (e) {
+        // Fallback
+    }
+    return 'main';
+}
+
 /**
  * Filter issues by confidence threshold and log what was removed
  */
@@ -165,13 +189,84 @@ async function main() {
     await detective.scan();
     detective.report();
 
-    // 1. Get the Diff
-    const diff = await git.diff(["--staged"]);
+    // 1. Determine Diff Target
+    const args = process.argv.slice(2);
+    let diff: string;
+    let reviewTarget = "Staged Changes";
+
+    if (args.includes('--last')) {
+        reviewTarget = "Last Commit";
+        diff = await git.diff(["HEAD~1", "HEAD"]);
+    } else if (args.includes('--commit')) {
+        const hashIndex = args.indexOf('--commit') + 1;
+        if (hashIndex < args.length) {
+            const hash = args[hashIndex];
+            reviewTarget = `Commit ${hash}`;
+            diff = await git.diff([`${hash}~1`, `${hash}`]);
+        } else {
+            Logger.error(chalk.red("❌ Missing commit hash for --commit"));
+            return;
+        }
+    } else if (args.includes('--range')) {
+        const rangeIndex = args.indexOf('--range') + 1;
+        if (rangeIndex + 1 < args.length) {
+            const start = args[rangeIndex];
+            const end = args[rangeIndex + 1];
+            reviewTarget = `Range ${start}..${end}`;
+            diff = await git.diff([start, end]);
+        } else {
+            Logger.error(chalk.red("❌ Missing arguments for --range (usage: --range <start> <end>)"));
+            return;
+        }
+    } else if (args.includes('--pr')) {
+        const prIndex = args.indexOf('--pr');
+        // Check for --base flag first
+        let baseBranch = 'main';
+
+        if (args.includes('--base')) {
+            const baseIndex = args.indexOf('--base') + 1;
+            if (baseIndex < args.length) {
+                baseBranch = args[baseIndex];
+            } else {
+                Logger.error(chalk.red("❌ Missing branch name for --base"));
+                return;
+            }
+        } else {
+            // Check positional argument (backward compatibility)
+            const potentialBase = args[prIndex + 1];
+            if (potentialBase && !potentialBase.startsWith('--')) {
+                baseBranch = potentialBase;
+            } else {
+                // Auto-detect
+                baseBranch = await getDefaultBranch();
+            }
+        }
+
+        reviewTarget = `PR (HEAD vs ${baseBranch})`;
+        // Use triple-dot for merge-base difference (what a PR shows)
+        try {
+            diff = await git.diff([`${baseBranch}...HEAD`]);
+        } catch (e) {
+            Logger.error(chalk.red(`❌ Failed to diff against ${baseBranch}. Does the branch exist?`));
+            return;
+        }
+    } else {
+        // Default: Staged changes
+        diff = await git.diff(["--staged"]);
+    }
+
     if (!diff) {
-        Logger.info(chalk.green("\n✅ No staged changes to review. (Did you 'git add'?)"));
+        if (reviewTarget === "Staged Changes") {
+            Logger.info(chalk.green("\n✅ No staged changes to review. (Did you 'git add'?)"));
+            Logger.info(chalk.dim("   Tip: Use '--last' to review the previous commit."));
+        } else {
+            Logger.info(chalk.green(`\n✅ No changes found in ${reviewTarget}.`));
+        }
         return;
     }
 
+    Logger.info(chalk.blue(`\n📝 Reviewing: ${reviewTarget}`));
+
     // 2. Load the Brain
     if (!fs.existsSync(STORAGE_DIR)) {
         Logger.error(chalk.red("❌ Local Brain not found. Run 'pnpm run index:init' first."));