korekt-cli 0.9.5 → 0.9.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.9.5",
+  "version": "0.9.7",
   "description": "AI-powered code review CLI - Keep your kode korekt",
   "main": "src/index.js",
   "bin": {

package/src/git-logic.js

@@ -316,9 +316,10 @@ export async function getContributors(diffRange, repoRootPath) {
  */
 export async function runLocalReview(targetBranch = null, ignorePatterns = null) {
   try {
-    // 1. Get Repo URL, current branch name, and repository root
+    // 1. Get Repo URL, current branch name, commit hash, and repository root
     const { stdout: repoUrl } = await execa('git', ['remote', 'get-url', 'origin']);
     const { stdout: sourceBranch } = await execa('git', ['rev-parse', '--abbrev-ref', 'HEAD']);
+    const { stdout: commitHash } = await execa('git', ['rev-parse', 'HEAD']);
     let branchName = sourceBranch.trim();
 
     // Handle detached HEAD state (common in CI pipelines)
@@ -533,6 +534,7 @@ export async function runLocalReview(targetBranch = null, ignorePatterns = null)
       changed_files: changedFiles,
       source_branch: branchName,
       destination_branch: targetBranch,
+      commit_hash: commitHash.trim(),
       author_email,
       author_name,
       contributors,

package/src/git-logic.test.js

@@ -259,6 +259,64 @@ describe('runLocalReview - branch fetching', () => {
   });
 });
 
+describe('runLocalReview - no files changed', () => {
+  beforeEach(() => {
+    vi.mock('execa');
+    vi.spyOn(console, 'error').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('should return payload with empty changed_files when diff has no files', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+
+      if (command.includes('remote get-url origin')) {
+        return { stdout: 'https://github.com/user/repo.git' };
+      }
+      if (command.includes('rev-parse --abbrev-ref HEAD')) {
+        return { stdout: 'feature-branch' };
+      }
+      if (command === 'git rev-parse HEAD') {
+        return { stdout: 'abc123def456789' };
+      }
+      if (command.includes('rev-parse --show-toplevel')) {
+        return { stdout: '/path/to/repo' };
+      }
+      if (command.includes('rev-parse --verify main')) {
+        return { stdout: 'commit-hash' };
+      }
+      if (command === 'git fetch origin main') {
+        return { stdout: '' };
+      }
+      if (command.includes('merge-base origin/main HEAD')) {
+        return { stdout: 'abc123' };
+      }
+      if (command.includes('log --no-merges --pretty=%B---EOC---')) {
+        return { stdout: 'feat: some commit---EOC---' };
+      }
+      if (command.includes('log --no-merges --format=%ae|%an')) {
+        return { stdout: 'user@example.com|User Name' };
+      }
+      // No files changed
+      if (command.includes('diff --name-status')) {
+        return { stdout: '' };
+      }
+
+      return { stdout: '' };
+    });
+
+    const result = await runLocalReview('main');
+
+    expect(result).not.toBeNull();
+    expect(result.changed_files).toEqual([]);
+    expect(result.source_branch).toBe('feature-branch');
+    expect(result.destination_branch).toBe('main');
+  });
+});
+
 describe('runLocalReview - fork point detection', () => {
   beforeEach(() => {
     vi.mock('execa');
@@ -279,6 +337,9 @@ describe('runLocalReview - fork point detection', () => {
       if (command.includes('rev-parse --abbrev-ref HEAD')) {
         return { stdout: 'feature-branch' };
       }
+      if (command === 'git rev-parse HEAD') {
+        return { stdout: 'abc123def456789' };
+      }
       if (command.includes('rev-parse --show-toplevel')) {
         return { stdout: '/path/to/repo' };
       }
@@ -329,6 +390,9 @@ describe('runLocalReview - fork point detection', () => {
       if (command.includes('rev-parse --abbrev-ref HEAD')) {
         return { stdout: 'feature-branch' };
       }
+      if (command === 'git rev-parse HEAD') {
+        return { stdout: 'abc123def456789' };
+      }
       if (command.includes('rev-parse --show-toplevel')) {
         return { stdout: '/path/to/repo' };
       }
@@ -1116,6 +1180,71 @@ describe('destination_branch in payload', () => {
   });
 });
 
+describe('commit_hash in payload', () => {
+  beforeEach(() => {
+    vi.mock('execa');
+    vi.mock('./utils.js', () => ({
+      detectCIProvider: vi.fn().mockReturnValue(null),
+      getPrUrl: vi.fn().mockReturnValue(null),
+      getSourceBranchFromCI: vi.fn().mockReturnValue(null),
+    }));
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('should include commit_hash in payload', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+
+      if (command.includes('remote get-url origin')) {
+        return { stdout: 'https://github.com/user/repo.git' };
+      }
+      if (command.includes('rev-parse --abbrev-ref HEAD')) {
+        return { stdout: 'feature-branch' };
+      }
+      if (command === 'git rev-parse HEAD') {
+        return { stdout: 'a1b2c3d4e5f6g7h8i9j0' };
+      }
+      if (command.includes('rev-parse --show-toplevel')) {
+        return { stdout: '/path/to/repo' };
+      }
+      if (command.includes('rev-parse --verify main')) {
+        return { stdout: 'commit-hash' };
+      }
+      if (command === 'git fetch origin main') {
+        return { stdout: '' };
+      }
+      if (command.includes('merge-base origin/main HEAD')) {
+        return { stdout: 'abc123' };
+      }
+      if (command.includes('log --no-merges --pretty=%B---EOC---')) {
+        return { stdout: 'feat: add feature---EOC---' };
+      }
+      if (command.includes('log --no-merges --format=%ae|%an')) {
+        return { stdout: 'user@example.com|User Name' };
+      }
+      if (command.includes('diff --name-status')) {
+        return { stdout: 'M\tfile.js' };
+      }
+      if (command.includes('diff -U15')) {
+        return { stdout: 'diff content' };
+      }
+      if (command.includes('show abc123:file.js')) {
+        return { stdout: 'original content' };
+      }
+
+      return { stdout: '' };
+    });
+
+    const result = await runLocalReview('main');
+
+    expect(result).toBeDefined();
+    expect(result.commit_hash).toBe('a1b2c3d4e5f6g7h8i9j0');
+  });
+});
+
 describe('getSourceBranchFromCI', () => {
   const originalEnv = process.env;
 

package/src/index.js

@@ -165,6 +165,12 @@ program
       process.exit(1);
     }
 
+    // Exit successfully if no files to review (common in CI when PR has no code changes)
+    if (payload.changed_files.length === 0) {
+      log(chalk.green('✓ No files changed to review. Exiting successfully.'));
+      return;
+    }
+
     // If dry-run, just show the payload and exit
     if (options.dryRun) {
       log(chalk.yellow('\n📋 Dry Run - Payload that would be sent:\n'));