codex-review-mcp 1.3.0 → 1.3.1

package/dist/review/collectDiff.js

@@ -153,8 +153,31 @@ export async function collectDiff(input, workspaceDir) {
             maxBuffer: 10 * 1024 * 1024,
             cwd: repoRoot,
         });
+        let diffText = stdout;
+        // In auto mode with uncommitted changes, also include untracked files
+        if (input.target === 'auto' && await hasUncommittedChanges(repoRoot)) {
+            try {
+                const { stdout: untrackedFiles } = await exec('git', ['ls-files', '--others', '--exclude-standard'], { encoding: 'utf8', cwd: repoRoot });
+                const files = untrackedFiles.trim().split('\n').filter(f => f);
+                for (const file of files) {
+                    try {
+                        const { stdout: fileDiff } = await exec('git', ['diff', '--no-index', '--unified=0', '/dev/null', file], { encoding: 'utf8', cwd: repoRoot });
+                        diffText += '\n' + fileDiff;
+                    }
+                    catch (diffErr) {
+                        // git diff --no-index returns exit code 1 when there are differences
+                        if (diffErr.stdout) {
+                            diffText += '\n' + diffErr.stdout;
+                        }
+                    }
+                }
+            }
+            catch {
+                // If we can't get untracked files, continue with just tracked changes
+            }
+        }
         // Drop obvious binary diffs
-        const text = stdout
+        const text = diffText
             .split('\n')
             .filter((line) => !line.startsWith('Binary files '))
             .join('\n');

package/dist/tools/performCodeReview.test.js

@@ -85,4 +85,19 @@ describe('performCodeReview', () => {
             expect(collectDiffModule.collectDiff).toHaveBeenCalled();
         }
     });
+    it('should handle diffs with untracked files', async () => {
+        const diffWithUntracked = `diff --git a/tracked.ts b/tracked.ts
++++ b/tracked.ts
++modified tracked file
+diff --git a/newfile.ts b/newfile.ts
+new file mode 100644
++++ b/newfile.ts
++new untracked file`;
+        vi.spyOn(collectDiffModule, 'collectDiff').mockResolvedValue(diffWithUntracked);
+        vi.spyOn(gatherContextModule, 'gatherContext').mockResolvedValue('');
+        vi.spyOn(invokeAgentModule, 'invokeAgent').mockResolvedValue('# Review');
+        const result = await performCodeReview({ target: 'auto' });
+        expect(result).toBeTruthy();
+        expect(collectDiffModule.collectDiff).toHaveBeenCalledWith(expect.objectContaining({ target: 'auto' }), undefined);
+    });
 });

package/package.json

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