korekt-cli 0.9.4 → 0.9.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.9.4",
+  "version": "0.9.6",
   "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)
@@ -532,6 +533,8 @@ export async function runLocalReview(targetBranch = null, ignorePatterns = null)
       commit_messages: commitMessages,
       changed_files: changedFiles,
       source_branch: branchName,
+      destination_branch: targetBranch,
+      commit_hash: commitHash.trim(),
       author_email,
       author_name,
       contributors,

package/src/git-logic.test.js

@@ -279,6 +279,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 +332,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' };
       }
@@ -1013,6 +1019,174 @@ describe('is_ci flag in payload', () => {
   });
 });
 
+describe('destination_branch 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 destination_branch when target branch is specified', 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.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.destination_branch).toBe('main');
+  });
+
+  it('should have null destination_branch when no target branch specified', 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.includes('rev-parse --show-toplevel')) {
+        return { stdout: '/path/to/repo' };
+      }
+      if (command.includes('reflog show --no-abbrev-commit')) {
+        return {
+          stdout:
+            '510572bc5197788770004d0d0585822adab0128f checkout: moving from main to feature-branch',
+        };
+      }
+      if (command.includes('log --no-merges --pretty=%B---EOC---')) {
+        return { stdout: 'feat: add feature---EOC---' };
+      }
+      if (command.includes('diff --name-status')) {
+        return { stdout: 'M\tfile.js' };
+      }
+      if (command.includes('diff -U15')) {
+        return { stdout: 'diff content' };
+      }
+      if (command.includes('show 510572bc5197788770004d0d0585822adab0128f:file.js')) {
+        return { stdout: 'original content' };
+      }
+
+      return { stdout: '' };
+    });
+
+    const result = await runLocalReview(); // No target branch
+
+    expect(result).toBeDefined();
+    expect(result.destination_branch).toBeNull();
+  });
+});
+
+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;