korekt-cli 0.5.0 → 0.6.0

package/package.json

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

package/src/git-logic.js

@@ -280,6 +280,55 @@ export async function runUncommittedReview(
   }
 }
 
+/**
+ * Extract contributors from git commits in a range
+ * Returns the author (most commits) and full list of contributors
+ * @param {string} diffRange - The git range to analyze (e.g., "abc123..HEAD")
+ * @param {string} repoRootPath - The repository root directory
+ * @returns {Object} - { author_email, author_name, contributors[] }
+ */
+export async function getContributors(diffRange, repoRootPath) {
+  try {
+    // Get all commit authors with email and name
+    const { stdout: authorOutput } = await execa('git', ['log', '--format=%ae|%an', diffRange], {
+      cwd: repoRootPath,
+    });
+
+    if (!authorOutput.trim()) {
+      return { author_email: null, author_name: null, contributors: [] };
+    }
+
+    const lines = authorOutput.trim().split('\n').filter(Boolean);
+
+    // Count commits per email and track name
+    const contributorMap = new Map();
+    for (const line of lines) {
+      const [email, name] = line.split('|');
+      if (!email) continue;
+
+      if (!contributorMap.has(email)) {
+        contributorMap.set(email, { email, name: name || email, commits: 0 });
+      }
+      contributorMap.get(email).commits++;
+    }
+
+    // Convert to array and sort by commits (descending)
+    const contributors = Array.from(contributorMap.values()).sort((a, b) => b.commits - a.commits);
+
+    // Author = most commits
+    const author = contributors[0] || null;
+
+    return {
+      author_email: author?.email || null,
+      author_name: author?.name || null,
+      contributors,
+    };
+  } catch (error) {
+    console.warn(chalk.yellow('Could not extract contributors:'), error.message);
+    return { author_email: null, author_name: null, contributors: [] };
+  }
+}
+
 /**
  * Main function to analyze local git changes and prepare review payload
  * @param {string|null} targetBranch - The branch to compare against. If null, uses git reflog to find fork point.
@@ -486,12 +535,21 @@ export async function runLocalReview(
       });
     }
 
-    // 5. Assemble the final payload
+    // 5. Get contributors from commits
+    const { author_email, author_name, contributors } = await getContributors(
+      diffRange,
+      repoRootPath
+    );
+
+    // 6. Assemble the final payload
     return {
       repo_url: normalizeRepoUrl(repoUrl.trim()),
       commit_messages: commitMessages,
       changed_files: changedFiles,
       source_branch: branchName,
+      author_email,
+      author_name,
+      contributors,
     };
   } catch (error) {
     console.error(chalk.red('Failed to run local review analysis:'), error.message);

package/src/git-logic.test.js

@@ -6,6 +6,7 @@ import {
   truncateContent,
   normalizeRepoUrl,
   shouldIgnoreFile,
+  getContributors,
 } from './git-logic.js';
 import { execa } from 'execa';
 
@@ -571,3 +572,138 @@ describe('shouldIgnoreFile', () => {
     expect(shouldIgnoreFile('file.js', pattern)).toBe(false);
   });
 });
+
+describe('getContributors', () => {
+  beforeEach(() => {
+    vi.mock('execa');
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('should extract single contributor with commit count', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+      if (command.includes('log --format=%ae|%an')) {
+        return {
+          stdout: 'john@example.com|John Doe\njohn@example.com|John Doe\njohn@example.com|John Doe',
+        };
+      }
+      throw new Error(`Unmocked command: ${command}`);
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.author_email).toBe('john@example.com');
+    expect(result.author_name).toBe('John Doe');
+    expect(result.contributors).toHaveLength(1);
+    expect(result.contributors[0]).toEqual({
+      email: 'john@example.com',
+      name: 'John Doe',
+      commits: 3,
+    });
+  });
+
+  it('should identify author as contributor with most commits', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+      if (command.includes('log --format=%ae|%an')) {
+        return {
+          stdout: [
+            'alice@example.com|Alice Smith',
+            'alice@example.com|Alice Smith',
+            'alice@example.com|Alice Smith',
+            'alice@example.com|Alice Smith',
+            'alice@example.com|Alice Smith',
+            'bob@example.com|Bob Jones',
+            'bob@example.com|Bob Jones',
+            'charlie@example.com|Charlie Brown',
+          ].join('\n'),
+        };
+      }
+      throw new Error(`Unmocked command: ${command}`);
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.author_email).toBe('alice@example.com');
+    expect(result.author_name).toBe('Alice Smith');
+    expect(result.contributors).toHaveLength(3);
+    expect(result.contributors[0].commits).toBe(5); // Alice - most commits
+    expect(result.contributors[1].commits).toBe(2); // Bob
+    expect(result.contributors[2].commits).toBe(1); // Charlie
+  });
+
+  it('should handle empty commit range', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+      if (command.includes('log --format=%ae|%an')) {
+        return { stdout: '' };
+      }
+      throw new Error(`Unmocked command: ${command}`);
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.author_email).toBeNull();
+    expect(result.author_name).toBeNull();
+    expect(result.contributors).toEqual([]);
+  });
+
+  it('should handle git command errors gracefully', async () => {
+    vi.mocked(execa).mockImplementation(async () => {
+      throw new Error('Git error');
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.author_email).toBeNull();
+    expect(result.author_name).toBeNull();
+    expect(result.contributors).toEqual([]);
+  });
+
+  it('should handle missing name in git log', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+      if (command.includes('log --format=%ae|%an')) {
+        return { stdout: 'user@example.com|' };
+      }
+      throw new Error(`Unmocked command: ${command}`);
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.author_email).toBe('user@example.com');
+    expect(result.author_name).toBe('user@example.com'); // Falls back to email
+    expect(result.contributors[0].name).toBe('user@example.com');
+  });
+
+  it('should sort contributors by commit count descending', async () => {
+    vi.mocked(execa).mockImplementation(async (cmd, args) => {
+      const command = [cmd, ...args].join(' ');
+      if (command.includes('log --format=%ae|%an')) {
+        return {
+          stdout: [
+            'c@example.com|C User',
+            'a@example.com|A User',
+            'a@example.com|A User',
+            'a@example.com|A User',
+            'b@example.com|B User',
+            'b@example.com|B User',
+          ].join('\n'),
+        };
+      }
+      throw new Error(`Unmocked command: ${command}`);
+    });
+
+    const result = await getContributors('abc123..HEAD', '/path/to/repo');
+
+    expect(result.contributors[0].email).toBe('a@example.com');
+    expect(result.contributors[0].commits).toBe(3);
+    expect(result.contributors[1].email).toBe('b@example.com');
+    expect(result.contributors[1].commits).toBe(2);
+    expect(result.contributors[2].email).toBe('c@example.com');
+    expect(result.contributors[2].commits).toBe(1);
+  });
+});

package/src/index.js

@@ -200,7 +200,7 @@ program
       log(`  Commits: ${chalk.cyan(payload.commit_messages.length)}`);
       log(`  Files: ${chalk.cyan(payload.changed_files.length)}\n`);
 
-      log(chalk.bold('  Files to review:'));
+      log(chalk.bold(`  ${payload.changed_files.length} files to review:`));
       payload.changed_files.forEach((file) => {
         const statusColor =
           {
@@ -364,7 +364,7 @@ async function reviewUncommitted(mode, options) {
   if (!options.json) {
     log(chalk.yellow('\n📋 Ready to submit uncommitted changes for review:\n'));
     log(chalk.gray('  Comparing against HEAD (last commit)\n'));
-    log(chalk.bold('  Files to review:'));
+    log(chalk.bold(`  ${payload.changed_files.length} files to review:`));
     payload.changed_files.forEach((file) => {
       const statusColor =
         {