codex-review-mcp 1.1.0 → 1.2.0

package/dist/mcp-server.js

@@ -6,7 +6,7 @@ import { performCodeReview } from './tools/performCodeReview.js';
 const server = new McpServer({ name: 'codex-review-mcp', version: '0.1.0' });
 server.registerTool('perform_code_review', {
     title: 'Perform Code Review',
-    description: 'Review git diffs in the current repo and return actionable Markdown feedback.',
+    description: 'Review git diffs in the current repo and return actionable Markdown feedback. In auto mode (default), reviews uncommitted changes if present, otherwise reviews current branch vs default branch. No need to commit changes first - the tool reviews your working tree.',
     inputSchema: {
         target: z.enum(['auto', 'staged', 'head', 'range']).default('auto'),
         baseRef: z.string().optional(),

package/dist/review/collectDiff.js

@@ -54,8 +54,14 @@ export async function collectDiff(input, workspaceDir) {
                 // Branch doesn't exist, try next
             }
         }
-        // Last resort: use HEAD~1 as baseline
-        return 'HEAD~1';
+        // Last resort: use HEAD~1 as baseline if it exists
+        try {
+            await exec('git', ['rev-parse', '--verify', 'HEAD~1'], { cwd: repoRoot });
+            return 'HEAD~1';
+        }
+        catch {
+            return null;
+        }
     }
     async function hasUncommittedChanges(repoRoot) {
         try {
@@ -75,6 +81,15 @@ export async function collectDiff(input, workspaceDir) {
             return null;
         }
     }
+    async function hasHeadCommit(repoRoot) {
+        try {
+            await exec('git', ['rev-parse', '--verify', 'HEAD'], { cwd: repoRoot });
+            return true;
+        }
+        catch {
+            return false;
+        }
+    }
     // Priority order: explicit workspaceDir param > env vars > process.cwd()
     const preferredStart = workspaceDir || process.env.CODEX_REPO_ROOT || process.env.WORKSPACE_ROOT || process.env.INIT_CWD || process.cwd();
     const preferredRoot = await findRepoRoot(preferredStart);
@@ -88,13 +103,22 @@ export async function collectDiff(input, workspaceDir) {
         // Auto mode: detect what to review
         const hasChanges = await hasUncommittedChanges(repoRoot);
         if (hasChanges) {
-            // Review uncommitted changes vs HEAD
-            args.push('HEAD');
+            // Review uncommitted changes vs HEAD (or staged if no commits yet)
+            if (await hasHeadCommit(repoRoot)) {
+                args.push('HEAD');
+            }
+            else {
+                args.splice(1, 0, '--staged');
+            }
         }
         else {
             // No uncommitted changes, review branch vs default
             const currentBranch = await getCurrentBranch(repoRoot);
             const defaultBranch = await detectDefaultBranch(repoRoot);
+            if (!defaultBranch) {
+                // Can't determine default branch - nothing to review
+                return '';
+            }
             if (currentBranch === defaultBranch) {
                 // On default branch with no changes - nothing to review
                 return '';

package/dist/review/gatherContext.js

@@ -1,6 +1,7 @@
 import { promises as fs } from 'node:fs';
 import { join } from 'node:path';
 const CANDIDATE_FILES = [
+    '.cursor/rules/tsx.mdc',
     '.cursor/rules/project.mdc',
     '.cursor/rules/**/*.md',
     'CODE_REVIEW.md',

package/dist/tools/performCodeReview.js

@@ -8,7 +8,7 @@ export async function performCodeReview(input, onProgress) {
     await onProgress?.('Collecting diff…', 10, 100);
     const diffText = await collectDiff(input, input.workspaceDir);
     if (!diffText.trim()) {
-        throw new Error('No diff found. Ensure you have changes or a valid git range.');
+        throw new Error('No changes to review. You are on the default branch with a clean working tree.');
     }
     await onProgress?.('Gathering context…', 30, 100);
     const context = await gatherContext();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codex-review-mcp",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "main": "index.js",
   "scripts": {
     "build": "tsc",