korekt-cli 0.3.0 → 0.4.1

package/README.md

@@ -4,7 +4,7 @@
 [![npm downloads](https://img.shields.io/npm/dm/korekt-cli.svg)](https://www.npmjs.com/package/korekt-cli)
 [![license](https://img.shields.io/npm/l/korekt-cli.svg)](https://www.npmjs.com/package/korekt-cli)
 
-AI-powered code review CLI - Keep your kode korekt ✨
+AI-powered code review CLI - Keep your kode korekt
 
 `kk` integrates seamlessly with your local Git workflow to provide intelligent code reviews powered by AI.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.3.0",
+  "version": "0.4.1",
   "description": "AI-powered code review CLI - Keep your kode korekt",
   "main": "src/index.js",
   "bin": {
@@ -11,7 +11,18 @@
   "scripts": {
     "test": "vitest",
     "test:watch": "vitest --watch",
-    "test:coverage": "vitest --coverage"
+    "test:coverage": "vitest --coverage",
+    "lint": "eslint src/**/*.js",
+    "lint:fix": "eslint src/**/*.js --fix",
+    "format": "prettier --write \"src/**/*.js\"",
+    "format:check": "prettier --check \"src/**/*.js\"",
+    "prepare": "husky"
+  },
+  "lint-staged": {
+    "src/**/*.js": [
+      "eslint --fix",
+      "prettier --write"
+    ]
   },
   "keywords": [
     "code-review",
@@ -45,6 +56,11 @@
     "ora": "^9.0.0"
   },
   "devDependencies": {
+    "@eslint/js": "^9.38.0",
+    "eslint": "^9.38.0",
+    "husky": "^9.1.7",
+    "lint-staged": "^16.2.6",
+    "prettier": "^3.6.2",
     "vitest": "^3.2.4"
   }
 }

package/src/config.js

@@ -72,4 +72,4 @@ export function getConfig() {
     apiEndpoint: getApiEndpoint(),
     ticketSystem: getTicketSystem(),
   };
-}
+}

package/src/formatter.js

@@ -39,9 +39,7 @@ const CATEGORY_ICONS = {
  */
 function formatCategory(category) {
   if (!category) return '';
-  return category
-    .replace(/_/g, ' ')
-    .replace(/\b\w/g, char => char.toUpperCase());
+  return category.replace(/_/g, ' ').replace(/\b\w/g, (char) => char.toUpperCase());
 }
 
 /**
@@ -52,7 +50,7 @@ function getGitRoot() {
   try {
     const { stdout } = execaSync('git', ['rev-parse', '--show-toplevel']);
     return stdout.trim();
-  } catch (error) {
+  } catch {
     // Fallback to current working directory if not in a git repo
     return process.cwd();
   }
@@ -83,11 +81,12 @@ export function formatReviewOutput(data) {
   // --- Praises Section ---
   if (review && review.praises && review.praises.length > 0) {
     console.log(chalk.bold.magenta(`✨ Praises (${summary.total_praises})`));
-    review.praises.forEach(praise => {
-      const categoryIcon = CATEGORY_ICONS[praise.category] || CATEGORY_ICONS.default;
+    review.praises.forEach((praise) => {
       const formattedCategory = formatCategory(praise.category);
       const absolutePath = toAbsolutePath(praise.file_path);
-      console.log(`  ✅ ${chalk.green.bold(formattedCategory)} in ${absolutePath}:${praise.line_number}`);
+      console.log(
+        `  ✅ ${chalk.green.bold(formattedCategory)} in ${absolutePath}:${praise.line_number}`
+      );
       console.log(`     ${praise.message}\n`);
     });
   }
@@ -99,7 +98,7 @@ export function formatReviewOutput(data) {
     // Severity Summary Table
     console.log(chalk.underline('Severity Count:'));
     const severities = ['critical', 'high', 'medium', 'low'];
-    severities.forEach(severity => {
+    severities.forEach((severity) => {
       const count = summary[severity] || 0;
       if (count > 0) {
         const icon = SEVERITY_ICONS[severity];
@@ -128,7 +127,10 @@ export function formatReviewOutput(data) {
       if (issue.suggested_fix) {
         console.log(chalk.bold('\n💡 Suggested Fix:'));
         // Indent the suggested fix for readability
-        const indentedFix = issue.suggested_fix.split('\n').map(line => `   ${line}`).join('\n');
+        const indentedFix = issue.suggested_fix
+          .split('\n')
+          .map((line) => `   ${line}`)
+          .join('\n');
         console.log(chalk.green(indentedFix));
       }
 

package/src/git-logic.js

@@ -28,26 +28,33 @@ export function truncateContent(content, maxLines = 2000) {
  */
 export function normalizeRepoUrl(url) {
   // Handle Azure DevOps SSH format: git@ssh.dev.azure.com:v3/org/project/repo
-  const azureDevOpsSshMatch = url.match(/git@ssh\.dev\.azure\.com:v3\/([^\/]+)\/([^\/]+)\/(.+)/);
+  const azureDevOpsSshMatch = url.match(/git@ssh\.dev\.azure\.com:v3\/([^/]+)\/([^/]+)\/(.+)/);
   if (azureDevOpsSshMatch) {
     const [, org, project, repo] = azureDevOpsSshMatch;
     return `https://dev.azure.com/${org}/${project}/_git/${repo}`;
   }
 
   // Handle GitHub SSH format: git@github.com:user/repo.git
-  const githubSshMatch = url.match(/git@github\.com:([^\/]+)\/(.+?)(?:\.git)?$/);
+  const githubSshMatch = url.match(/git@github\.com:([^/]+)\/(.+?)(?:\.git)?$/);
   if (githubSshMatch) {
     const [, user, repo] = githubSshMatch;
     return `https://github.com/${user}/${repo}`;
   }
 
   // Handle GitLab SSH format: git@gitlab.com:user/repo.git
-  const gitlabSshMatch = url.match(/git@gitlab\.com:([^\/]+)\/(.+?)(?:\.git)?$/);
+  const gitlabSshMatch = url.match(/git@gitlab\.com:([^/]+)\/(.+?)(?:\.git)?$/);
   if (gitlabSshMatch) {
     const [, user, repo] = gitlabSshMatch;
     return `https://gitlab.com/${user}/${repo}`;
   }
 
+  // Handle Bitbucket SSH format: git@bitbucket.org:user/repo.git
+  const bitbucketSshMatch = url.match(/git@bitbucket\.org:([^/]+)\/(.+?)(?:\.git)?$/);
+  if (bitbucketSshMatch) {
+    const [, user, repo] = bitbucketSshMatch;
+    return `https://bitbucket.org/${user}/${repo}`;
+  }
+
   // If already HTTPS or other format, return as-is (possibly removing .git suffix)
   return url.replace(/\.git$/, '');
 }
@@ -69,11 +76,11 @@ export function shouldIgnoreFile(filePath, patterns) {
     // Replace * with [^/]* (matches anything except /)
     // Replace ** with .* (matches anything including /)
     let regexPattern = pattern
-      .replace(/\./g, '\\.')  // Escape dots
-      .replace(/\*\*/g, '___DOUBLESTAR___')  // Temporarily replace **
-      .replace(/\*/g, '[^/]*')  // Replace single * with [^/]*
-      .replace(/___DOUBLESTAR___/g, '.*')  // Replace ** with .*
-      .replace(/\?/g, '.');  // Replace ? with .
+      .replace(/\./g, '\\.') // Escape dots
+      .replace(/\*\*/g, '___DOUBLESTAR___') // Temporarily replace **
+      .replace(/\*/g, '[^/]*') // Replace single * with [^/]*
+      .replace(/___DOUBLESTAR___/g, '.*') // Replace ** with .*
+      .replace(/\?/g, '.'); // Replace ? with .
 
     // Handle leading **/ pattern - make it optional so it matches both with and without directory prefix
     // For example, **/*.sql should match both "file.sql" and "dir/file.sql"
@@ -133,7 +140,11 @@ export function parseNameStatus(output) {
  * @param {string|null} ticketSystem - The ticket system to use (jira or ado), or null to skip ticket extraction
  * @returns {Object|null} - The payload object ready for API submission, or null on error
  */
-export async function runUncommittedReview(mode = 'all', ticketSystem = null, includeUntracked = false) {
+export async function runUncommittedReview(
+  mode = 'all',
+  _ticketSystem = null,
+  includeUntracked = false
+) {
   try {
     // 1. Get Repo URL and current branch name
     const { stdout: repoUrl } = await execa('git', ['remote', 'get-url', 'origin']);
@@ -164,12 +175,19 @@ export async function runUncommittedReview(mode = 'all', ticketSystem = null, in
     // Handle untracked files if requested
     if (includeUntracked) {
       console.log(chalk.gray('Analyzing untracked files...'));
-      const { stdout: untrackedFilesOutput } = await execa('git', ['ls-files', '--others', '--exclude-standard']);
+      const { stdout: untrackedFilesOutput } = await execa('git', [
+        'ls-files',
+        '--others',
+        '--exclude-standard',
+      ]);
       const untrackedFiles = untrackedFilesOutput.split('\n').filter(Boolean);
 
       for (const file of untrackedFiles) {
         const content = fs.readFileSync(file, 'utf-8');
-        const diff = content.split('\n').map(line => `+${line}`).join('\n');
+        const diff = content
+          .split('\n')
+          .map((line) => `+${line}`)
+          .join('\n');
         changedFiles.push({
           path: file,
           status: 'A', // Untracked files are always additions
@@ -182,8 +200,8 @@ export async function runUncommittedReview(mode = 'all', ticketSystem = null, in
     }
 
     // Deduplicate file list before processing diffs
-    const processedPaths = new Set(changedFiles.map(f => f.path));
-    const uniqueFileList = fileList.filter(file => !processedPaths.has(file.path));
+    const processedPaths = new Set(changedFiles.map((f) => f.path));
+    const uniqueFileList = fileList.filter((file) => !processedPaths.has(file.path));
 
     for (const file of uniqueFileList) {
       const { status, path, oldPath } = file;
@@ -199,7 +217,13 @@ export async function runUncommittedReview(mode = 'all', ticketSystem = null, in
       } else {
         // For 'all', try staged first, then unstaged
         try {
-          const { stdout: stagedDiff } = await execa('git', ['diff', '--cached', '-U15', '--', path]);
+          const { stdout: stagedDiff } = await execa('git', [
+            'diff',
+            '--cached',
+            '-U15',
+            '--',
+            path,
+          ]);
           if (stagedDiff) {
             diff = stagedDiff;
           } else {
@@ -218,8 +242,10 @@ export async function runUncommittedReview(mode = 'all', ticketSystem = null, in
         try {
           const { stdout: headContent } = await execa('git', ['show', `HEAD:${oldPath}`]);
           content = headContent;
-        } catch (e) {
-          console.warn(chalk.yellow(`Could not get HEAD content for ${oldPath}. Assuming it's new.`));
+        } catch {
+          console.warn(
+            chalk.yellow(`Could not get HEAD content for ${oldPath}. Assuming it's new.`)
+          );
         }
       }
 
@@ -268,7 +294,11 @@ export async function runUncommittedReview(mode = 'all', ticketSystem = null, in
  * @param {string[]|null} ignorePatterns - Array of glob patterns to ignore files
  * @returns {Object|null} - The payload object ready for API submission, or null on error
  */
-export async function runLocalReview(targetBranch = null, ticketSystem = null, ignorePatterns = null) {
+export async function runLocalReview(
+  targetBranch = null,
+  _ticketSystem = null,
+  ignorePatterns = null
+) {
   try {
     // 1. Get Repo URL, current branch name, and repository root
     const { stdout: repoUrl } = await execa('git', ['remote', 'get-url', 'origin']);
@@ -285,7 +315,7 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
       // Check if the branch exists locally
       try {
         await execa('git', ['rev-parse', '--verify', targetBranch]);
-      } catch (error) {
+      } catch {
         console.error(chalk.red(`Branch '${targetBranch}' does not exist locally.`));
         console.error(chalk.gray(`Please check out the branch first or specify a different one.`));
         return null;
@@ -299,8 +329,10 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
         // If fetch succeeded, use the remote-tracking branch for comparison
         // This is safer as it doesn't modify the user's local branch
         targetBranchRef = `origin/${targetBranch}`;
-        console.log(chalk.gray(`Using remote-tracking branch 'origin/${targetBranch}' for comparison.`));
-      } catch (fetchError) {
+        console.log(
+          chalk.gray(`Using remote-tracking branch 'origin/${targetBranch}' for comparison.`)
+        );
+      } catch {
         console.warn(chalk.yellow(`Could not fetch remote branch 'origin/${targetBranch}'.`));
         console.warn(chalk.gray(`Proceeding with local branch '${targetBranch}' for comparison.`));
         // targetBranchRef stays as targetBranch (local branch)
@@ -313,7 +345,12 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
     if (!targetBranch) {
       try {
         // Use git reflog to find where the branch was created
-        const { stdout: reflog } = await execa('git', ['reflog', 'show', '--no-abbrev-commit', branchName]);
+        const { stdout: reflog } = await execa('git', [
+          'reflog',
+          'show',
+          '--no-abbrev-commit',
+          branchName,
+        ]);
         const lines = reflog.split('\n');
 
         // Look for the branch creation point (last line in reflog)
@@ -322,15 +359,19 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
           const match = creationLine.match(/^([a-f0-9]{40})/);
           if (match) {
             mergeBase = match[1];
-            console.log(chalk.gray(`Auto-detected fork point from reflog: ${mergeBase.substring(0, 7)}`));
+            console.log(
+              chalk.gray(`Auto-detected fork point from reflog: ${mergeBase.substring(0, 7)}`)
+            );
           }
         }
 
         if (!mergeBase) {
           throw new Error('Could not find fork point in reflog');
         }
-      } catch (error) {
-        console.error(chalk.red('Could not auto-detect fork point. Please specify a target branch.'));
+      } catch {
+        console.error(
+          chalk.red('Could not auto-detect fork point. Please specify a target branch.')
+        );
         console.error(chalk.gray('Usage: kk review <target-branch>'));
         return null;
       }
@@ -349,21 +390,25 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
     console.log(chalk.gray(`Analyzing commits from ${mergeBase.substring(0, 7)} to HEAD...`));
 
     // 3. Get Commit Messages with proper delimiter
-    const { stdout: logOutput } = await execa('git', ['log', '--pretty=%B---EOC---', diffRange], { cwd: repoRootPath });
+    const { stdout: logOutput } = await execa('git', ['log', '--pretty=%B---EOC---', diffRange], {
+      cwd: repoRootPath,
+    });
     const commitMessages = logOutput
       .split('---EOC---')
       .map((msg) => msg.trim())
       .filter(Boolean);
 
     // 4. Get changed files and their status
-    const { stdout: nameStatusOutput } = await execa('git', ['diff', '--name-status', diffRange], { cwd: repoRootPath });
+    const { stdout: nameStatusOutput } = await execa('git', ['diff', '--name-status', diffRange], {
+      cwd: repoRootPath,
+    });
     const fileList = parseNameStatus(nameStatusOutput);
 
     // Filter out ignored files
     let filteredFileList = fileList;
     let ignoredCount = 0;
     if (ignorePatterns && ignorePatterns.length > 0) {
-      filteredFileList = fileList.filter(file => {
+      filteredFileList = fileList.filter((file) => {
         const ignored = shouldIgnoreFile(file.path, ignorePatterns);
         if (ignored) {
           ignoredCount++;
@@ -385,19 +430,22 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
 
       // Run git commands from the repository root to handle all file paths correctly
       // This works regardless of whether we're in a subdirectory or at the repo root
-      const { stdout: diff } = await execa('git', ['diff', '-U15', diffRange, '--', path], { cwd: repoRootPath });
+      const { stdout: diff } = await execa('git', ['diff', '-U15', diffRange, '--', path], {
+        cwd: repoRootPath,
+      });
 
       // Get the original content from the base commit
       let content = '';
       if (status !== 'A') {
         // Added files have no original content
         try {
-          const { stdout: originalContent } = await execa('git', [
-            'show',
-            `${mergeBase.trim()}:${oldPath}`,
-          ], { cwd: repoRootPath });
+          const { stdout: originalContent } = await execa(
+            'git',
+            ['show', `${mergeBase.trim()}:${oldPath}`],
+            { cwd: repoRootPath }
+          );
           content = originalContent;
-        } catch (e) {
+        } catch {
           // This can happen if a file was added and modified in the same branch
           console.warn(
             chalk.yellow(`Could not get original content for ${oldPath}. Assuming it was added.`)
@@ -437,4 +485,4 @@ export async function runLocalReview(targetBranch = null, ticketSystem = null, i
     }
     return null; // Return null to indicate failure
   }
-}
+}

package/src/git-logic.test.js

@@ -1,9 +1,6 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 import {
   parseNameStatus,
-  findJiraTicketIds,
-  findAdoTicketIds,
-  extractTicketIds,
   runLocalReview,
   runUncommittedReview,
   truncateContent,
@@ -221,14 +218,17 @@ describe('runLocalReview - branch fetching', () => {
 
     const result = await runLocalReview('non-existent-branch');
     expect(result).toBeNull();
-    expect(console.error).toHaveBeenCalledWith(expect.stringContaining("Branch 'non-existent-branch' does not exist locally."));
+    expect(console.error).toHaveBeenCalledWith(
+      expect.stringContaining("Branch 'non-existent-branch' does not exist locally.")
+    );
   });
 
   it('should fetch latest changes if target branch exists locally', async () => {
     vi.mocked(execa).mockImplementation(async (cmd, args) => {
       const command = [cmd, ...args].join(' ');
       // Common setup
-      if (command.includes('remote get-url origin')) return { stdout: 'https://github.com/user/repo.git' };
+      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: 'current-branch' };
       if (command.includes('rev-parse --show-toplevel')) return { stdout: '/path/to/repo' };
 
@@ -249,14 +249,15 @@ describe('runLocalReview - branch fetching', () => {
     const result = await runLocalReview('main');
     expect(result).not.toBeNull();
     const execaCalls = vi.mocked(execa).mock.calls;
-    const fetchCall = execaCalls.find(call => call[0] === 'git' && call[1].includes('fetch'));
+    const fetchCall = execaCalls.find((call) => call[0] === 'git' && call[1].includes('fetch'));
     expect(fetchCall).toBeDefined();
   });
 
   it('should warn and continue if fetch fails', 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('remote get-url origin'))
+        return { stdout: 'https://github.com/user/repo.git' };
       if (command.includes('rev-parse --abbrev-ref HEAD')) return { stdout: 'current-branch' };
       if (command.includes('rev-parse --show-toplevel')) return { stdout: '/path/to/repo' };
       if (command.includes('rev-parse --verify main')) return { stdout: 'commit-hash' };
@@ -278,8 +279,12 @@ describe('runLocalReview - branch fetching', () => {
 
     const result = await runLocalReview('main');
     expect(result).not.toBeNull(); // Should still proceed
-    expect(console.warn).toHaveBeenCalledWith(expect.stringContaining("Could not fetch remote branch 'origin/main'."));
-    expect(console.warn).toHaveBeenCalledWith(expect.stringContaining("Proceeding with local branch 'main' for comparison."));
+    expect(console.warn).toHaveBeenCalledWith(
+      expect.stringContaining("Could not fetch remote branch 'origin/main'.")
+    );
+    expect(console.warn).toHaveBeenCalledWith(
+      expect.stringContaining("Proceeding with local branch 'main' for comparison.")
+    );
   });
 });
 
@@ -308,7 +313,10 @@ describe('runLocalReview - fork point detection', () => {
       }
       if (command.includes('reflog show --no-abbrev-commit feature-branch')) {
         // Simulate reflog output - last line is where branch was created
-        return { stdout: 'abc123def456 feature-branch@{0}: commit: latest\nfedcba654321 feature-branch@{1}: commit: middle\n510572bc5197788770004d0d0585822adab0128f feature-branch@{2}: branch: Created from master' };
+        return {
+          stdout:
+            'abc123def456 feature-branch@{0}: commit: latest\nfedcba654321 feature-branch@{1}: commit: middle\n510572bc5197788770004d0d0585822adab0128f feature-branch@{2}: branch: Created from master',
+        };
       }
       if (command.includes('log --pretty=%B---EOC---') && command.includes('510572bc')) {
         return { stdout: 'feat: add feature---EOC---' };
@@ -333,9 +341,7 @@ describe('runLocalReview - fork point detection', () => {
 
     // Should have used reflog to find fork point
     const execaCalls = vi.mocked(execa).mock.calls;
-    const reflogCall = execaCalls.find(call =>
-      call[0] === 'git' && call[1].includes('reflog')
-    );
+    const reflogCall = execaCalls.find((call) => call[0] === 'git' && call[1].includes('reflog'));
     expect(reflogCall).toBeDefined();
   });
 
@@ -384,8 +390,9 @@ describe('runLocalReview - fork point detection', () => {
 
     // Should have used merge-base with origin/main (since fetch succeeded)
     const execaCalls = vi.mocked(execa).mock.calls;
-    const mergeBaseCall = execaCalls.find(call =>
-      call[0] === 'git' && call[1].includes('merge-base') && call[1].includes('origin/main')
+    const mergeBaseCall = execaCalls.find(
+      (call) =>
+        call[0] === 'git' && call[1].includes('merge-base') && call[1].includes('origin/main')
     );
     expect(mergeBaseCall).toBeDefined();
   });
@@ -450,6 +457,18 @@ describe('normalizeRepoUrl', () => {
     expect(normalizeRepoUrl(sshUrl)).toBe(expected);
   });
 
+  it('should normalize Bitbucket SSH URL to HTTPS', () => {
+    const sshUrl = 'git@bitbucket.org:user/repo.git';
+    const expected = 'https://bitbucket.org/user/repo';
+    expect(normalizeRepoUrl(sshUrl)).toBe(expected);
+  });
+
+  it('should normalize Bitbucket SSH URL without .git suffix', () => {
+    const sshUrl = 'git@bitbucket.org:user/repo';
+    const expected = 'https://bitbucket.org/user/repo';
+    expect(normalizeRepoUrl(sshUrl)).toBe(expected);
+  });
+
   it('should remove .git suffix from HTTPS URLs', () => {
     const httpsUrl = 'https://github.com/user/repo.git';
     const expected = 'https://github.com/user/repo';
@@ -539,4 +558,4 @@ describe('shouldIgnoreFile', () => {
     expect(shouldIgnoreFile('file.sql', pattern)).toBe(true);
     expect(shouldIgnoreFile('file.js', pattern)).toBe(false);
   });
-});
+});

package/src/index.js

@@ -6,7 +6,14 @@ import chalk from 'chalk';
 import readline from 'readline';
 import ora from 'ora';
 import { runLocalReview } from './git-logic.js';
-import { getApiKey, setApiKey, getApiEndpoint, setApiEndpoint, getTicketSystem, setTicketSystem } from './config.js';
+import {
+  getApiKey,
+  setApiKey,
+  getApiEndpoint,
+  setApiEndpoint,
+  getTicketSystem,
+  setTicketSystem,
+} from './config.js';
 import { formatReviewOutput } from './formatter.js';
 
 /**
@@ -31,7 +38,9 @@ program
   .name('kk')
   .description('AI-powered code review CLI - Keep your kode korekt')
   .version('0.2.0')
-  .addHelpText('after', `
+  .addHelpText(
+    'after',
+    `
 Examples:
   $ kk review                      Review committed changes (auto-detect base)
   $ kk review main                 Review changes against main branch
@@ -47,24 +56,29 @@ Configuration:
   $ kk config --key YOUR_KEY
   $ kk config --endpoint https://api.korekt.ai/review/local
   $ kk config --ticket-system ado
-`);
+`
+  );
 
 program
   .command('review')
   .description('Review the changes in the current branch.')
-  .argument('[target-branch]', 'The branch to compare against (e.g., main, develop). If not specified, auto-detects fork point.')
+  .argument(
+    '[target-branch]',
+    'The branch to compare against (e.g., main, develop). If not specified, auto-detects fork point.'
+  )
   .option('--ticket-system <system>', 'Ticket system to use (jira or ado)')
   .option('--dry-run', 'Show payload without sending to API')
-  .option('--ignore <patterns...>', 'Ignore files matching these patterns (e.g., "*.lock" "dist/*")')
+  .option(
+    '--ignore <patterns...>',
+    'Ignore files matching these patterns (e.g., "*.lock" "dist/*")'
+  )
   .action(async (targetBranch, options) => {
     const reviewTarget = targetBranch ? `against '${targetBranch}'` : '(auto-detecting fork point)';
     console.log(chalk.blue.bold(`🚀 Starting AI Code Review ${reviewTarget}...`));
 
     const apiKey = getApiKey();
     if (!apiKey) {
-      console.error(
-        chalk.red('API Key not found! Please run `kk config --key YOUR_KEY` first.')
-      );
+      console.error(chalk.red('API Key not found! Please run `kk config --key YOUR_KEY` first.'));
       return;
     }
 
@@ -107,12 +121,18 @@ program
       // Create a shortened version for display
       const displayPayload = {
         ...payload,
-        changed_files: payload.changed_files.map(file => ({
+        changed_files: payload.changed_files.map((file) => ({
           path: file.path,
           status: file.status,
           ...(file.old_path && { old_path: file.old_path }),
-          diff: file.diff.length > 500 ? `${file.diff.substring(0, 500)}... [truncated ${file.diff.length - 500} chars]` : file.diff,
-          content: file.content.length > 500 ? `${file.content.substring(0, 500)}... [truncated ${file.content.length - 500} chars]` : file.content,
+          diff:
+            file.diff.length > 500
+              ? `${file.diff.substring(0, 500)}... [truncated ${file.diff.length - 500} chars]`
+              : file.diff,
+          content:
+            file.content.length > 500
+              ? `${file.content.substring(0, 500)}... [truncated ${file.content.length - 500} chars]`
+              : file.content,
         })),
       };
 
@@ -129,14 +149,15 @@ program
     console.log(`  Files: ${chalk.cyan(payload.changed_files.length)}\n`);
 
     console.log(chalk.bold('  Files to review:'));
-    payload.changed_files.forEach(file => {
-      const statusColor = {
-        'M': chalk.yellow,
-        'A': chalk.green,
-        'D': chalk.red,
-        'R': chalk.blue,
-        'C': chalk.cyan,
-      }[file.status] || (text => text);
+    payload.changed_files.forEach((file) => {
+      const statusColor =
+        {
+          M: chalk.yellow,
+          A: chalk.green,
+          D: chalk.red,
+          R: chalk.blue,
+          C: chalk.cyan,
+        }[file.status] || ((text) => text);
       console.log(`    ${statusColor(file.status + ' ' + file.path)}`);
     });
     console.log();
@@ -161,7 +182,7 @@ program
     try {
       const response = await axios.post(apiEndpoint, payload, {
         headers: {
-          'Authorization': `Bearer ${apiKey}`,
+          Authorization: `Bearer ${apiKey}`,
           'Content-Type': 'application/json',
         },
       });
@@ -224,9 +245,7 @@ program
 async function reviewUncommitted(mode, options) {
   const apiKey = getApiKey();
   if (!apiKey) {
-    console.error(
-      chalk.red('API Key not found! Please run `kk config --key YOUR_KEY` first.')
-    );
+    console.error(chalk.red('API Key not found! Please run `kk config --key YOUR_KEY` first.'));
     return;
   }
 
@@ -265,12 +284,18 @@ async function reviewUncommitted(mode, options) {
 
     const displayPayload = {
       ...payload,
-      changed_files: payload.changed_files.map(file => ({
+      changed_files: payload.changed_files.map((file) => ({
         path: file.path,
         status: file.status,
         ...(file.old_path && { old_path: file.old_path }),
-        diff: file.diff.length > 500 ? `${file.diff.substring(0, 500)}... [truncated ${file.diff.length - 500} chars]` : file.diff,
-        content: file.content.length > 500 ? `${file.content.substring(0, 500)}... [truncated ${file.content.length - 500} chars]` : file.content,
+        diff:
+          file.diff.length > 500
+            ? `${file.diff.substring(0, 500)}... [truncated ${file.diff.length - 500} chars]`
+            : file.diff,
+        content:
+          file.content.length > 500
+            ? `${file.content.substring(0, 500)}... [truncated ${file.content.length - 500} chars]`
+            : file.content,
       })),
     };
 
@@ -284,14 +309,15 @@ async function reviewUncommitted(mode, options) {
   console.log(chalk.yellow('\n📋 Ready to submit uncommitted changes for review:\n'));
   console.log(chalk.gray('  Comparing against HEAD (last commit)\n'));
   console.log(chalk.bold('  Files to review:'));
-  payload.changed_files.forEach(file => {
-    const statusColor = {
-      'M': chalk.yellow,
-      'A': chalk.green,
-      'D': chalk.red,
-      'R': chalk.blue,
-      'C': chalk.cyan,
-    }[file.status] || (text => text);
+  payload.changed_files.forEach((file) => {
+    const statusColor =
+      {
+        M: chalk.yellow,
+        A: chalk.green,
+        D: chalk.red,
+        R: chalk.blue,
+        C: chalk.cyan,
+      }[file.status] || ((text) => text);
     console.log(`    ${statusColor(file.status + ' ' + file.path)}`);
   });
   console.log();
@@ -315,7 +341,7 @@ async function reviewUncommitted(mode, options) {
   try {
     const response = await axios.post(apiEndpoint, payload, {
       headers: {
-        'Authorization': `Bearer ${apiKey}`,
+        Authorization: `Bearer ${apiKey}`,
         'Content-Type': 'application/json',
       },
     });
@@ -355,8 +381,12 @@ program
 
       console.log(chalk.bold('\nCurrent Configuration:\n'));
       console.log(`  API Key: ${apiKey ? chalk.green('✓ Set') : chalk.red('✗ Not set')}`);
-      console.log(`  API Endpoint: ${apiEndpoint ? chalk.cyan(apiEndpoint) : chalk.red('✗ Not set')}`);
-      console.log(`  Ticket System: ${ticketSystem ? chalk.cyan(ticketSystem) : chalk.gray('Not configured')}\n`);
+      console.log(
+        `  API Endpoint: ${apiEndpoint ? chalk.cyan(apiEndpoint) : chalk.red('✗ Not set')}`
+      );
+      console.log(
+        `  Ticket System: ${ticketSystem ? chalk.cyan(ticketSystem) : chalk.gray('Not configured')}\n`
+      );
       return;
     }