korekt-cli 0.6.0 → 0.8.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.8.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

@@ -6,18 +6,13 @@ import chalk from 'chalk';
 import readline from 'readline';
 import ora from 'ora';
 import { createRequire } from 'module';
-import { readFileSync } from 'fs';
-import { join, dirname } from 'path';
+import { writeFileSync, mkdtempSync, rmSync } from 'fs';
+import { join, dirname, resolve } from 'path';
 import { fileURLToPath } from 'url';
+import { spawn } from 'child_process';
+import { tmpdir } from 'os';
 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);
@@ -78,11 +73,76 @@ async function confirmAction(message) {
     output: process.stdout,
   });
 
-  return new Promise((resolve) => {
+  return new Promise((resolvePromise) => {
     rl.question(message, (answer) => {
       rl.close();
       // Default to 'yes' if the user just presses Enter
-      resolve(answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes' || answer === '');
+      resolvePromise(
+        answer.toLowerCase() === 'y' || answer.toLowerCase() === 'yes' || answer === ''
+      );
+    });
+  });
+}
+
+/**
+ * Detect CI provider from environment variables
+ * @returns {string|null} Provider name or null if not detected
+ */
+export function detectCIProvider() {
+  if (process.env.GITHUB_TOKEN && process.env.GITHUB_REPOSITORY) {
+    return 'github';
+  }
+  if (process.env.SYSTEM_ACCESSTOKEN && process.env.SYSTEM_PULLREQUEST_PULLREQUESTID) {
+    return 'azure';
+  }
+  if (process.env.BITBUCKET_REPO_SLUG && process.env.BITBUCKET_PR_ID) {
+    return 'bitbucket';
+  }
+  return null;
+}
+
+/**
+ * Run the CI integration script to post comments
+ * @param {string} provider - CI provider (github, azure, bitbucket)
+ * @param {Object} results - Review results from API
+ * @returns {Promise<void>}
+ */
+async function runCIScript(provider, results) {
+  const __filename = fileURLToPath(import.meta.url);
+  const __dirname = dirname(__filename);
+  const scriptPath = resolve(__dirname, '..', 'scripts', `${provider}.sh`);
+
+  // Create secure temp directory and write results
+  const tempDir = mkdtempSync(join(tmpdir(), 'korekt-'));
+  const tempFile = join(tempDir, 'results.json');
+  writeFileSync(tempFile, JSON.stringify(results, null, 2));
+
+  return new Promise((resolvePromise, reject) => {
+    const cleanup = () => {
+      try {
+        rmSync(tempDir, { recursive: true, force: true });
+      } catch (err) {
+        log(chalk.yellow(`Warning: Failed to clean up temp directory: ${err.message}`));
+      }
+    };
+
+    const child = spawn('bash', [scriptPath, tempFile], {
+      stdio: 'inherit',
+      env: process.env,
+    });
+
+    child.on('close', (code) => {
+      cleanup();
+      if (code === 0) {
+        resolvePromise();
+      } else {
+        reject(new Error(`CI script exited with code ${code}`));
+      }
+    });
+
+    child.on('error', (err) => {
+      cleanup();
+      reject(err);
     });
   });
 }
@@ -99,23 +159,17 @@ 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)
+  $ kk review main --comment       Review and post comments to PR (CI/CD)
 
 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)
+  --comment                        Post review results as PR comments
 
 Configuration:
   $ kk config --key YOUR_KEY
-  $ kk config --endpoint https://api.korekt.ai/review/local
-  $ kk config --ticket-system ado
-
-CI/CD Integration:
-  $ kk get-script github           Output GitHub Actions integration script
-  $ kk get-script bitbucket        Output Bitbucket Pipelines integration script
-  $ kk get-script azure            Output Azure DevOps integration script
+  $ kk config --endpoint https://api.korekt.ai/api/review
 `
   );
 
@@ -126,13 +180,13 @@ 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...>',
     'Ignore files matching these patterns (e.g., "*.lock" "dist/*")'
   )
   .option('--json', 'Output raw API response as JSON')
+  .option('--comment', 'Post review results as PR comments (auto-detects CI provider)')
   .action(async (targetBranch, options) => {
     const reviewTarget = targetBranch ? `against '${targetBranch}'` : '(auto-detecting fork point)';
 
@@ -153,30 +207,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'));
@@ -193,8 +231,8 @@ program
       return;
     }
 
-    // Show summary and ask for confirmation (auto-confirm in JSON mode)
-    if (!options.json) {
+    // Show summary and ask for confirmation (auto-confirm in JSON/comment mode)
+    if (!options.json && !options.comment) {
       log(chalk.yellow('\n📋 Ready to submit for review:\n'));
       log(`  Branch: ${chalk.cyan(payload.source_branch)}`);
       log(`  Commits: ${chalk.cyan(payload.commit_messages.length)}`);
@@ -243,6 +281,32 @@ program
       const elapsed = Math.floor((Date.now() - startTime) / 1000);
       spinner.succeed(`Review completed in ${elapsed}s!`);
 
+      // Handle --comment flag: post results to PR
+      if (options.comment) {
+        const provider = detectCIProvider();
+        if (!provider) {
+          log(
+            chalk.red(
+              'Could not detect CI provider. Make sure required environment variables are set:'
+            )
+          );
+          log(chalk.gray('  GitHub: GITHUB_TOKEN, GITHUB_REPOSITORY, PR_NUMBER, COMMIT_HASH'));
+          log(chalk.gray('  Azure: SYSTEM_ACCESSTOKEN, SYSTEM_PULLREQUEST_PULLREQUESTID'));
+          log(chalk.gray('  Bitbucket: BITBUCKET_REPO_SLUG, BITBUCKET_PR_ID'));
+          process.exit(1);
+        }
+
+        log(chalk.blue(`Posting review comments to ${provider}...`));
+        try {
+          await runCIScript(provider, response.data);
+          log(chalk.green('Successfully posted review comments!'));
+        } catch (err) {
+          log(chalk.red(`Failed to post comments: ${err.message}`));
+          process.exit(1);
+        }
+        return;
+      }
+
       // Output results to stdout
       if (options.json) {
         output(JSON.stringify(response.data, null, 2));
@@ -276,7 +340,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 +351,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 +373,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 +475,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,76 +506,15 @@ 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)');
     }
   });
 
-program
-  .command('get-script <provider>')
-  .description('Output a CI/CD integration script for a specific provider')
-  .addHelpText(
-    'after',
-    `
-Providers:
-  github      GitHub Actions integration script
-  bitbucket   Bitbucket Pipelines integration script
-  azure       Azure DevOps integration script
-
-Usage:
-  kk get-script github | bash -s results.json
-  kk get-script bitbucket > bitbucket.sh && chmod +x bitbucket.sh
-  kk get-script azure > azure.sh
-`
-  )
-  .action((provider) => {
-    const validProviders = ['github', 'bitbucket', 'azure'];
-
-    if (!validProviders.includes(provider.toLowerCase())) {
-      console.error(chalk.red(`Invalid provider: ${provider}`));
-      console.error(chalk.gray(`Valid providers: ${validProviders.join(', ')}`));
-      process.exit(1);
-    }
-
-    try {
-      // Get the directory where this script is located
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
-
-      // Build path to the script file
-      const scriptPath = join(__dirname, '..', 'scripts', `${provider.toLowerCase()}.sh`);
-
-      // Read and output the script
-      const scriptContent = readFileSync(scriptPath, 'utf8');
-      output(scriptContent);
-    } catch (error) {
-      console.error(chalk.red(`Failed to read script: ${error.message}`));
-      process.exit(1);
-    }
-  });
-
 // Only parse arguments if this file is being run directly (not imported)
 // In tests, we set NODE_ENV to 'test' via vitest
 if (process.env.NODE_ENV !== 'test') {

package/src/index.test.js

@@ -1,8 +1,5 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { truncateFileData, formatErrorOutput } from './index.js';
-import { readFileSync } from 'fs';
-import { join, dirname } from 'path';
-import { fileURLToPath } from 'url';
+import { truncateFileData, formatErrorOutput, detectCIProvider } from './index.js';
 
 describe('CLI JSON output mode', () => {
   let stdoutSpy;
@@ -336,137 +333,108 @@ describe('CLI JSON output mode', () => {
   });
 });
 
-describe('get-script command', () => {
-  let stdoutSpy;
+describe('detectCIProvider', () => {
+  const originalEnv = process.env;
 
   beforeEach(() => {
-    stdoutSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+    // Reset environment before each test
+    process.env = { ...originalEnv };
+    // Clear all CI-related env vars
+    delete process.env.GITHUB_TOKEN;
+    delete process.env.GITHUB_REPOSITORY;
+    delete process.env.SYSTEM_ACCESSTOKEN;
+    delete process.env.SYSTEM_PULLREQUEST_PULLREQUESTID;
+    delete process.env.BITBUCKET_REPO_SLUG;
+    delete process.env.BITBUCKET_PR_ID;
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    process.env = originalEnv;
   });
 
-  describe('valid providers', () => {
-    it('should output github script to stdout', () => {
-      const output = (msg) => process.stdout.write(msg + '\n');
-
-      // Read actual github.sh script
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
-      const scriptPath = join(__dirname, '..', 'scripts', 'github.sh');
-      const scriptContent = readFileSync(scriptPath, 'utf8');
-
-      // Simulate get-script command output
-      output(scriptContent);
-
-      // Verify script was output to stdout
-      expect(stdoutSpy).toHaveBeenCalledWith(scriptContent + '\n');
-      expect(stdoutSpy).toHaveBeenCalledTimes(1);
-    });
+  it('should detect GitHub when GITHUB_TOKEN and GITHUB_REPOSITORY are set', () => {
+    process.env.GITHUB_TOKEN = 'ghp_test123';
+    process.env.GITHUB_REPOSITORY = 'owner/repo';
 
-    it('should output bitbucket script to stdout', () => {
-      const output = (msg) => process.stdout.write(msg + '\n');
+    expect(detectCIProvider()).toBe('github');
+  });
 
-      // Read actual bitbucket.sh script
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
-      const scriptPath = join(__dirname, '..', 'scripts', 'bitbucket.sh');
-      const scriptContent = readFileSync(scriptPath, 'utf8');
+  it('should detect Azure when SYSTEM_ACCESSTOKEN and SYSTEM_PULLREQUEST_PULLREQUESTID are set', () => {
+    process.env.SYSTEM_ACCESSTOKEN = 'azure_token';
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '123';
 
-      // Simulate get-script command output
-      output(scriptContent);
+    expect(detectCIProvider()).toBe('azure');
+  });
 
-      // Verify script was output to stdout
-      expect(stdoutSpy).toHaveBeenCalledWith(scriptContent + '\n');
-      expect(stdoutSpy).toHaveBeenCalledTimes(1);
-    });
+  it('should detect Bitbucket when BITBUCKET_REPO_SLUG and BITBUCKET_PR_ID are set', () => {
+    process.env.BITBUCKET_REPO_SLUG = 'my-repo';
+    process.env.BITBUCKET_PR_ID = '456';
 
-    it('should output azure script to stdout', () => {
-      const output = (msg) => process.stdout.write(msg + '\n');
+    expect(detectCIProvider()).toBe('bitbucket');
+  });
 
-      // Read actual azure.sh script
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
-      const scriptPath = join(__dirname, '..', 'scripts', 'azure.sh');
-      const scriptContent = readFileSync(scriptPath, 'utf8');
+  it('should return null when no CI provider env vars are set', () => {
+    expect(detectCIProvider()).toBe(null);
+  });
 
-      // Simulate get-script command output
-      output(scriptContent);
+  it('should return null when only partial GitHub env vars are set', () => {
+    process.env.GITHUB_TOKEN = 'ghp_test123';
+    // GITHUB_REPOSITORY not set
 
-      // Verify script was output to stdout
-      expect(stdoutSpy).toHaveBeenCalledWith(scriptContent + '\n');
-      expect(stdoutSpy).toHaveBeenCalledTimes(1);
-    });
+    expect(detectCIProvider()).toBe(null);
   });
 
-  describe('script completeness', () => {
-    it('should output complete bash scripts with proper structure', () => {
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
+  it('should return null when only partial Azure env vars are set', () => {
+    process.env.SYSTEM_ACCESSTOKEN = 'azure_token';
+    // SYSTEM_PULLREQUEST_PULLREQUESTID not set
 
-      // Test all three scripts
-      const providers = ['github', 'bitbucket', 'azure'];
-
-      providers.forEach((provider) => {
-        const scriptPath = join(__dirname, '..', 'scripts', `${provider}.sh`);
-        const scriptContent = readFileSync(scriptPath, 'utf8');
+    expect(detectCIProvider()).toBe(null);
+  });
 
-        // Verify script starts with shebang
-        expect(scriptContent).toMatch(/^#!/);
-        expect(scriptContent).toContain('#!/usr/bin/env bash');
+  it('should return null when only partial Bitbucket env vars are set', () => {
+    process.env.BITBUCKET_REPO_SLUG = 'my-repo';
+    // BITBUCKET_PR_ID not set
 
-        // Verify script has substantial content (not truncated)
-        const lineCount = scriptContent.split('\n').length;
-        expect(lineCount).toBeGreaterThan(400);
+    expect(detectCIProvider()).toBe(null);
+  });
 
-        // Verify script ends properly (has exit statement)
-        expect(scriptContent).toContain('exit');
-      });
-    });
+  it('should prioritize GitHub over other providers when multiple are set', () => {
+    // Set all providers
+    process.env.GITHUB_TOKEN = 'ghp_test123';
+    process.env.GITHUB_REPOSITORY = 'owner/repo';
+    process.env.SYSTEM_ACCESSTOKEN = 'azure_token';
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '123';
+    process.env.BITBUCKET_REPO_SLUG = 'my-repo';
+    process.env.BITBUCKET_PR_ID = '456';
+
+    // GitHub should be detected first due to check order
+    expect(detectCIProvider()).toBe('github');
   });
+});
 
-  describe('error handling', () => {
-    it('should reject invalid provider', () => {
-      const invalidProvider = 'gitlab';
-      const validProviders = ['github', 'bitbucket', 'azure'];
+describe('--comment flag behavior', () => {
+  it('should skip confirmation when --comment is set', () => {
+    const options = { comment: true };
 
-      // Test validation logic from index.js line 513
-      const isValid = validProviders.includes(invalidProvider.toLowerCase());
+    // Logic from index.js: if (!options.json && !options.comment) { confirmAction... }
+    const shouldShowConfirmation = !options.json && !options.comment;
 
-      expect(isValid).toBe(false);
+    expect(shouldShowConfirmation).toBe(false);
+  });
 
-      // In the actual command, this would trigger process.exit(1)
-      // We verify the validation logic correctly identifies invalid providers
-      expect(['github', 'bitbucket', 'azure']).not.toContain(invalidProvider);
-    });
+  it('should skip confirmation when both --json and --comment are set', () => {
+    const options = { json: true, comment: true };
 
-    it('should accept valid providers case-insensitively', () => {
-      const validProviders = ['github', 'bitbucket', 'azure'];
+    const shouldShowConfirmation = !options.json && !options.comment;
 
-      // Test that providers work case-insensitively
-      expect(validProviders.includes('GitHub'.toLowerCase())).toBe(true);
-      expect(validProviders.includes('BITBUCKET'.toLowerCase())).toBe(true);
-      expect(validProviders.includes('Azure'.toLowerCase())).toBe(true);
-    });
+    expect(shouldShowConfirmation).toBe(false);
   });
 
-  describe('bundled script files', () => {
-    it('should have all provider scripts bundled and readable', () => {
-      const __filename = fileURLToPath(import.meta.url);
-      const __dirname = dirname(__filename);
+  it('should show confirmation when neither --json nor --comment is set', () => {
+    const options = {};
 
-      const providers = ['github', 'bitbucket', 'azure'];
+    const shouldShowConfirmation = !options.json && !options.comment;
 
-      providers.forEach((provider) => {
-        const scriptPath = join(__dirname, '..', 'scripts', `${provider}.sh`);
-
-        // Should not throw - file exists and is readable
-        expect(() => {
-          const content = readFileSync(scriptPath, 'utf8');
-          expect(content.length).toBeGreaterThan(0);
-        }).not.toThrow();
-      });
-    });
+    expect(shouldShowConfirmation).toBe(true);
   });
 });