korekt-cli 0.6.0 → 0.7.0

package/README.md

@@ -12,7 +12,7 @@ AI-powered code review CLI - Keep your kode korekt
 
 *   **AI-Powered Analysis**: Get instant, intelligent code reviews with severity levels, categories, and actionable suggestions
 *   **Local Git Integration**: Works with committed changes, staged changes, and unstaged modifications
-*   **Ticket System Integration**: Automatically extracts ticket IDs from branch names and commit messages (Jira & Azure DevOps)
+*   **Ticket Context Enrichment**: Server-side ticket extraction from branch names and commit messages (Jira & Azure DevOps)
 *   **Beautiful Output**: Color-coded issues with severity indicators, file locations, and suggested fixes
 *   **Ultra-Fast**: Short command syntax (`kk`) for maximum developer efficiency
 
@@ -28,7 +28,7 @@ Configure the CLI with your API credentials:
 
 ```bash
 kk config --key YOUR_API_KEY
-kk config --endpoint https://api.korekt.ai/review/local
+kk config --endpoint https://api.korekt.ai/api/review
 ```
 
 Run your first review:
@@ -43,8 +43,7 @@ kk stg
 # Review only unstaged changes
 kk diff
 
-# Review all uncommitted changes (staged + unstaged)
-kk all
+
 ```
 
 ## Usage
@@ -56,10 +55,7 @@ kk all
 kk config --key YOUR_API_KEY
 
 # Set API endpoint
-kk config --endpoint https://api.korekt.ai/review/local
-
-# Set default ticket system (jira or ado)
-kk config --ticket-system jira
+kk config --endpoint https://api.korekt.ai/api/review
 
 # Show current configuration
 kk config --show
@@ -74,9 +70,6 @@ kk review
 # Review against specific branch
 kk review main
 
-# Review with ticket system override
-kk review main --ticket-system ado
-
 # Review with ignored files
 kk review main --ignore "*.lock" "dist/*"
 
@@ -93,16 +86,9 @@ kk stg
 # Review unstaged changes only
 kk diff
 
-# Review all uncommitted changes
-kk all
-
-# Include untracked files
-kk all --untracked
-
 # JSON output works with all review commands
 kk stg --json
 kk diff --json
-kk all --json
 ```
 
 ### Alternative Command
@@ -119,8 +105,7 @@ You can also configure using environment variables:
 
 ```bash
 export KOREKT_API_KEY="your-api-key"
-export KOREKT_API_ENDPOINT="https://api.korekt.ai/review/local"
-export KOREKT_TICKET_SYSTEM="jira"
+export KOREKT_API_ENDPOINT="https://api.korekt.ai/api/review"
 ```
 
 Note: Config file takes precedence over environment variables.

package/package.json

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

package/src/config.js

@@ -35,7 +35,7 @@ export function getApiEndpoint() {
   const configEndpoint = config.get('apiEndpoint');
   if (configEndpoint) return configEndpoint;
 
-  return process.env.KOREKT_API_ENDPOINT || 'https://api.korekt.ai/api/review/local';
+  return process.env.KOREKT_API_ENDPOINT || 'https://api.korekt.ai/api/review';
 }
 
 /**
@@ -45,24 +45,6 @@ export function setApiEndpoint(endpoint) {
   config.set('apiEndpoint', endpoint);
 }
 
-/**
- * Get the ticket system from config or environment
- * Priority: 1) config store, 2) .env file
- */
-export function getTicketSystem() {
-  const configTicketSystem = config.get('ticketSystem');
-  if (configTicketSystem) return configTicketSystem;
-
-  return process.env.KOREKT_TICKET_SYSTEM || null;
-}
-
-/**
- * Set the ticket system in config store
- */
-export function setTicketSystem(system) {
-  config.set('ticketSystem', system);
-}
-
 /**
  * Get all configuration
  */
@@ -70,6 +52,5 @@ export function getConfig() {
   return {
     apiKey: getApiKey(),
     apiEndpoint: getApiEndpoint(),
-    ticketSystem: getTicketSystem(),
   };
 }

package/src/git-logic.js

@@ -1,7 +1,5 @@
 import { execa } from 'execa';
 import chalk from 'chalk';
-import fs from 'fs';
-import path from 'path';
 
 /**
  * Truncate content to a maximum number of lines using "head and tail".
@@ -136,16 +134,11 @@ export function parseNameStatus(output) {
 }
 
 /**
- * Analyze uncommitted changes (staged, unstaged, or all)
- * @param {string} mode - 'staged', 'unstaged', or 'all'
- * @param {string|null} ticketSystem - The ticket system to use (jira or ado), or null to skip ticket extraction
+ * Analyze uncommitted changes (staged or unstaged)
+ * @param {string} mode - 'staged' or 'unstaged'
  * @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 = 'unstaged') {
   try {
     // 1. Get Repo URL, current branch name, and repository root
     const { stdout: repoUrl } = await execa('git', ['remote', 'get-url', 'origin']);
@@ -167,67 +160,23 @@ export async function runUncommittedReview(
     if (mode === 'staged') {
       nameStatusOutput = await git('diff', '--cached', '--name-status');
       console.error(chalk.gray('Analyzing staged changes...'));
-    } else if (mode === 'unstaged') {
+    } else {
       nameStatusOutput = await git('diff', '--name-status');
       console.error(chalk.gray('Analyzing unstaged changes...'));
-    } else {
-      // mode === 'all': combine staged and unstaged
-      const staged = await git('diff', '--cached', '--name-status');
-      const unstaged = await git('diff', '--name-status');
-      nameStatusOutput = [staged, unstaged].filter(Boolean).join('\n');
-      console.error(chalk.gray('Analyzing all uncommitted changes...'));
     }
 
     const fileList = parseNameStatus(nameStatusOutput);
     const changedFiles = [];
 
-    // Handle untracked files if requested
-    if (includeUntracked) {
-      console.error(chalk.gray('Analyzing untracked files...'));
-      const untrackedFilesOutput = await git('ls-files', '--others', '--exclude-standard');
-      const untrackedFiles = untrackedFilesOutput.split('\n').filter(Boolean);
-
-      for (const file of untrackedFiles) {
-        const fullPath = path.join(repoRootPath, file);
-        const content = fs.readFileSync(fullPath, 'utf-8');
-        const diff = content
-          .split('\n')
-          .map((line) => `+${line}`)
-          .join('\n');
-        changedFiles.push({
-          path: file,
-          status: 'A', // Untracked files are always additions
-          diff: diff,
-          content: '', // No old content
-        });
-        // Add to fileList to prevent duplication if it's also in nameStatusOutput (edge case)
-        fileList.push({ status: 'A', path: file, oldPath: file });
-      }
-    }
-
-    // Deduplicate file list before processing diffs
-    const processedPaths = new Set(changedFiles.map((f) => f.path));
-    const uniqueFileList = fileList.filter((file) => !processedPaths.has(file.path));
-
-    for (const file of uniqueFileList) {
+    for (const file of fileList) {
       const { status, path, oldPath } = file;
 
       // Get diff for this file
       let diff;
       if (mode === 'staged') {
         diff = await git('diff', '--cached', '-U15', '--', path);
-      } else if (mode === 'unstaged') {
-        diff = await git('diff', '-U15', '--', path);
       } else {
-        // For 'all', try staged first, then unstaged
-        try {
-          diff = await git('diff', '--cached', '-U15', '--', path);
-          if (!diff) {
-            diff = await git('diff', '-U15', '--', path);
-          }
-        } catch {
-          diff = await git('diff', '-U15', '--', path);
-        }
+        diff = await git('diff', '-U15', '--', path);
       }
 
       // Get current content from HEAD (before changes)
@@ -332,15 +281,10 @@ export async function getContributors(diffRange, repoRootPath) {
 /**
  * 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.
- * @param {string|null} ticketSystem - The ticket system to use (jira or ado), or null to skip ticket extraction
  * @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, ignorePatterns = null) {
   try {
     // 1. Get Repo URL, current branch name, and repository root
     const { stdout: repoUrl } = await execa('git', ['remote', 'get-url', 'origin']);

package/src/git-logic.test.js

@@ -83,7 +83,7 @@ describe('runUncommittedReview', () => {
       throw new Error(`Unmocked command: ${command}`);
     });
 
-    const result = await runUncommittedReview('staged', null);
+    const result = await runUncommittedReview('staged');
 
     expect(result).toBeDefined();
     expect(result.repo_url).toBe('https://github.com/user/repo'); // Normalized (no .git)
@@ -120,56 +120,13 @@ describe('runUncommittedReview', () => {
       throw new Error(`Unmocked command: ${command}`);
     });
 
-    const result = await runUncommittedReview('unstaged', null);
+    const result = await runUncommittedReview('unstaged');
 
     expect(result).toBeDefined();
     expect(result.source_branch).toBe('feature-branch');
     expect(result.changed_files).toHaveLength(1);
   });
 
-  it('should analyze all uncommitted changes', 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: '/fake/repo/path' };
-      }
-      if (command.includes('diff --cached --name-status')) {
-        return { stdout: 'M\tstaged.js' };
-      }
-      if (command === 'git diff --name-status') {
-        return { stdout: 'M\tunstaged.js' };
-      }
-      if (command.includes('diff --cached -U15 -- staged.js')) {
-        return { stdout: 'diff staged' };
-      }
-      if (command.includes('diff -U15 -- unstaged.js')) {
-        return { stdout: 'diff unstaged' };
-      }
-      if (command.includes('show HEAD:staged.js')) {
-        return { stdout: 'staged old content' };
-      }
-      if (command.includes('show HEAD:unstaged.js')) {
-        return { stdout: 'unstaged old content' };
-      }
-
-      throw new Error(`Unmocked command: ${command}`);
-    });
-
-    const result = await runUncommittedReview('all', null);
-
-    expect(result).toBeDefined();
-    expect(result.changed_files).toHaveLength(2);
-    expect(result.changed_files[0].path).toBe('staged.js');
-    expect(result.changed_files[1].path).toBe('unstaged.js');
-  });
-
   it('should return null when no changes found', async () => {
     vi.mocked(execa).mockImplementation(async (cmd, args) => {
       const command = [cmd, ...args].join(' ');
@@ -193,7 +150,7 @@ describe('runUncommittedReview', () => {
       throw new Error(`Unmocked command: ${command}`);
     });
 
-    const result = await runUncommittedReview('all', null);
+    const result = await runUncommittedReview('staged');
 
     expect(result).toBeNull();
   });
@@ -347,7 +304,7 @@ describe('runLocalReview - fork point detection', () => {
       throw new Error(`Unmocked command: ${command}`);
     });
 
-    const result = await runLocalReview(null, 'jira');
+    const result = await runLocalReview(null);
 
     expect(result).toBeDefined();
     expect(result.source_branch).toBe('feature-branch');
@@ -397,7 +354,7 @@ describe('runLocalReview - fork point detection', () => {
       throw new Error(`Unmocked command: ${command}`);
     });
 
-    const result = await runLocalReview('main', 'jira');
+    const result = await runLocalReview('main');
 
     expect(result).toBeDefined();
 

package/src/index.js

@@ -10,14 +10,7 @@ import { readFileSync } from 'fs';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { runLocalReview } from './git-logic.js';
-import {
-  getApiKey,
-  setApiKey,
-  getApiEndpoint,
-  setApiEndpoint,
-  getTicketSystem,
-  setTicketSystem,
-} from './config.js';
+import { getApiKey, setApiKey, getApiEndpoint, setApiEndpoint } from './config.js';
 import { formatReviewOutput } from './formatter.js';
 
 const require = createRequire(import.meta.url);
@@ -99,18 +92,15 @@ Examples:
   $ kk review main                 Review changes against main branch
   $ kk stg --dry-run               Preview staged changes review
   $ kk diff                        Review unstaged changes
-  $ kk all                         Review all uncommitted changes
   $ kk review main --json          Output raw JSON (for CI/CD integration)
 
 Common Options:
   --dry-run                        Show payload without sending to API
   --json                           Output raw API response as JSON
-  --ticket-system <system>         Use specific ticket system (jira or ado)
 
 Configuration:
   $ kk config --key YOUR_KEY
-  $ kk config --endpoint https://api.korekt.ai/review/local
-  $ kk config --ticket-system ado
+  $ kk config --endpoint https://api.korekt.ai/api/review
 
 CI/CD Integration:
   $ kk get-script github           Output GitHub Actions integration script
@@ -126,7 +116,6 @@ program
     '[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...>',
@@ -153,30 +142,14 @@ program
       process.exit(1);
     }
 
-    // Determine ticket system to use (or null if not configured)
-    const ticketSystem = options.ticketSystem || getTicketSystem() || null;
-
-    // Validate ticket system
-    if (ticketSystem && !['jira', 'ado'].includes(ticketSystem.toLowerCase())) {
-      log(chalk.red(`Invalid ticket system: ${ticketSystem}`));
-      log(chalk.gray('Valid options: jira, ado'));
-      process.exit(1);
-    }
-
     // Gather all data using our git logic module
-    const payload = await runLocalReview(targetBranch, ticketSystem, options.ignore);
+    const payload = await runLocalReview(targetBranch, options.ignore);
 
     if (!payload) {
       log(chalk.red('Could not proceed with review due to errors during analysis.'));
       process.exit(1);
     }
 
-    // Add ticket system to payload if specified
-    if (ticketSystem) {
-      payload.ticket_system = ticketSystem;
-      log(chalk.gray(`Using ticket system: ${ticketSystem}`));
-    }
-
     // If dry-run, just show the payload and exit
     if (options.dryRun) {
       log(chalk.yellow('\n📋 Dry Run - Payload that would be sent:\n'));
@@ -276,7 +249,6 @@ program
   .command('review-staged')
   .aliases(['stg', 'staged', 'cached'])
   .description('Review staged changes (git diff --cached)')
-  .option('--ticket-system <system>', 'Ticket system to use (jira or ado)')
   .option('--dry-run', 'Show payload without sending to API')
   .option('--json', 'Output raw API response as JSON')
   .action(async (options) => {
@@ -288,28 +260,13 @@ program
   .command('review-unstaged')
   .alias('diff')
   .description('Review unstaged changes (git diff)')
-  .option('--ticket-system <system>', 'Ticket system to use (jira or ado)')
   .option('--dry-run', 'Show payload without sending to API')
-  .option('--untracked', 'Include untracked files in the review')
   .option('--json', 'Output raw API response as JSON')
   .action(async (options) => {
     log(chalk.blue.bold('🚀 Reviewing unstaged changes...'));
     await reviewUncommitted('unstaged', options);
   });
 
-program
-  .command('review-all-uncommitted')
-  .alias('all')
-  .description('Review all uncommitted changes (staged + unstaged)')
-  .option('--ticket-system <system>', 'Ticket system to use (jira or ado)')
-  .option('--dry-run', 'Show payload without sending to API')
-  .option('--untracked', 'Include untracked files in the review')
-  .option('--json', 'Output raw API response as JSON')
-  .action(async (options) => {
-    log(chalk.blue.bold('🚀 Reviewing all uncommitted changes...'));
-    await reviewUncommitted('all', options);
-  });
-
 async function reviewUncommitted(mode, options) {
   const apiKey = getApiKey();
   if (!apiKey) {
@@ -325,27 +282,14 @@ async function reviewUncommitted(mode, options) {
     process.exit(1);
   }
 
-  const ticketSystem = options.ticketSystem || getTicketSystem() || null;
-
-  if (ticketSystem && !['jira', 'ado'].includes(ticketSystem.toLowerCase())) {
-    log(chalk.red(`Invalid ticket system: ${ticketSystem}`));
-    log(chalk.gray('Valid options: jira, ado'));
-    process.exit(1);
-  }
-
   const { runUncommittedReview } = await import('./git-logic.js');
-  const payload = await runUncommittedReview(mode, ticketSystem, options.untracked);
+  const payload = await runUncommittedReview(mode);
 
   if (!payload) {
     log(chalk.red('No changes found or error occurred during analysis.'));
     process.exit(1);
   }
 
-  if (ticketSystem) {
-    payload.ticket_system = ticketSystem;
-    log(chalk.gray(`Using ticket system: ${ticketSystem}`));
-  }
-
   if (options.dryRun) {
     log(chalk.yellow('\n📋 Dry Run - Payload that would be sent:\n'));
 
@@ -440,22 +384,17 @@ program
   .description('Configure API settings')
   .option('--key <key>', 'Your API key')
   .option('--endpoint <endpoint>', 'Your API endpoint URL')
-  .option('--ticket-system <system>', 'Ticket system (jira, ado)')
   .option('--show', 'Show current configuration')
   .action((options) => {
     // Show current config if --show flag is used
     if (options.show) {
       const apiKey = getApiKey();
       const apiEndpoint = getApiEndpoint();
-      const ticketSystem = getTicketSystem();
 
       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`
+        `  API Endpoint: ${apiEndpoint ? chalk.cyan(apiEndpoint) : chalk.red('✗ Not set')}\n`
       );
       return;
     }
@@ -476,29 +415,11 @@ program
       setApiEndpoint(options.endpoint);
       console.log(chalk.green('✓ API Endpoint saved successfully!'));
     }
-    if (options.ticketSystem !== undefined) {
-      if (options.ticketSystem === '') {
-        // Clear ticket system
-        setTicketSystem(null);
-        console.log(chalk.green('✓ Ticket System cleared!'));
-      } else {
-        // Validate ticket system
-        const validSystems = ['jira', 'ado'];
-        if (!validSystems.includes(options.ticketSystem.toLowerCase())) {
-          console.error(chalk.red(`Invalid ticket system: ${options.ticketSystem}`));
-          console.error(chalk.gray(`Valid options: ${validSystems.join(', ')}`));
-          return;
-        }
-        setTicketSystem(options.ticketSystem);
-        console.log(chalk.green('✓ Ticket System saved successfully!'));
-      }
-    }
-    if (!options.key && !options.endpoint && options.ticketSystem === undefined && !options.show) {
+    if (!options.key && !options.endpoint && !options.show) {
       console.log(chalk.yellow('Please provide at least one configuration option.'));
       console.log('\nUsage:');
       console.log('  kk config --key YOUR_API_KEY');
-      console.log('  kk config --endpoint https://api.korekt.ai/review/local');
-      console.log('  kk config --ticket-system jira');
+      console.log('  kk config --endpoint https://api.korekt.ai/api/review');
       console.log('  kk config --show              (view current configuration)');
     }
   });