korekt-cli 0.12.0 → 0.13.2

package/README.md

@@ -4,17 +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
-
-`kk` integrates seamlessly with your local Git workflow to provide intelligent code reviews powered by AI.
-
-## Features
-
-*   **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 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
+AI-powered code review from your terminal.
 
 ## Installation
 
@@ -22,112 +12,81 @@ AI-powered code review CLI - Keep your kode korekt
 npm install -g korekt-cli
 ```
 
-## Quick Start
-
-Configure the CLI with your API credentials:
+## Setup
 
 ```bash
 kk config --key YOUR_API_KEY
-kk config --endpoint https://api.korekt.ai/api/review
 ```
 
-Run your first review:
-
-```bash
-# Review committed changes against a target branch
-kk review main
-
-# Review only staged changes
-kk stg
-
-# Review only unstaged changes
-kk diff
+## Local Workflow
 
+### Review Your Changes
 
+```bash
+kk review main              # Review commits against main
+kk stg                      # Review staged changes
+kk diff                     # Review unstaged changes
 ```
 
-## Usage
-
-### Configuration
+### Choose AI Model
 
 ```bash
-# Set API key
-kk config --key YOUR_API_KEY
-
-# Set API endpoint
-kk config --endpoint https://api.korekt.ai/api/review
-
-# Show current configuration
-kk config --show
+kk review -m                        # Interactive model picker
+kk review -m gemini-3-flash-preview # Direct selection
 ```
 
-### Review Commands
+Available models (ranked by recommendation):
 
-```bash
-# Review committed changes (auto-detect base branch)
-kk review
+1. **gemini-3-flash-preview** - Most efficient, recommended for daily use
+2. **gemini-3-pro-preview** - Best quality for complex reviews
+3. **gemini-2.5-pro** - High quality alternative
+4. **gemini-2.5-flash** - Legacy, avoid
 
-# Review against specific branch
-kk review main
+### Ignore Files
 
-# Review with ignored files
+```bash
 kk review main --ignore "*.lock" "dist/*"
+```
 
-# Dry run (preview payload without sending)
-kk review main --dry-run
+## CI/CD Integration
 
-# Output JSON for CI/CD integration
-kk review main --json
+### Post to Pull Request
 
-# Review staged changes only
-kk stg
-# Aliases: kk staged, kk cached
+```bash
+kk review --comment         # Auto-posts findings to PR
+```
 
-# Review unstaged changes only
-kk diff
+Works with GitHub Actions, Azure Pipelines, and Bitbucket Pipelines.
 
-# JSON output works with all review commands
-kk stg --json
-kk diff --json
+### Post to Ticket
+
+```bash
+kk review --post-ticket     # Posts findings to linked Jira/Azure ticket
 ```
 
-### Alternative Command
+Ticket IDs are automatically extracted from branch names and commit messages.
 
-Both `kk` and `korekt` commands are available:
+### JSON Output
 
 ```bash
-korekt review main  # Same as: kk review main
+kk review main --json       # Machine-readable output
 ```
 
 ## Environment Variables
 
-You can also configure using environment variables:
-
 ```bash
 export KOREKT_API_KEY="your-api-key"
-export KOREKT_API_ENDPOINT="https://api.korekt.ai/api/review"
 ```
 
-Note: Config file takes precedence over environment variables.
+Alternative to `kk config --key`. Config file takes precedence.
 
 ## Help
 
-For more options and detailed help:
-
 ```bash
 kk --help
 kk review --help
 ```
 
-## Development
-
-To run tests:
-```bash
-npm test
-```
-
 ## License
 
-MIT © [Vladan Djokic](https://korekt.ai)
-
-See [LICENSE](./LICENSE) for details.
+MIT - See [LICENSE](./LICENSE) for details.

package/package.json

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

package/src/config.js

@@ -44,13 +44,3 @@ export function getApiEndpoint() {
 export function setApiEndpoint(endpoint) {
   config.set('apiEndpoint', endpoint);
 }
-
-/**
- * Get all configuration
- */
-export function getConfig() {
-  return {
-    apiKey: getApiKey(),
-    apiEndpoint: getApiEndpoint(),
-  };
-}

package/src/index.js

@@ -72,6 +72,24 @@ async function confirmAction(message) {
   });
 }
 
+/**
+ * Handle skipped response from API (when integration mode prevents CLI review)
+ * @param {Object} response - Axios response object
+ * @param {Object} options - Command options (json flag, etc.)
+ * @param {Object} spinner - Ora spinner instance
+ * @returns {boolean} - True if response was skipped, false otherwise
+ */
+export function handleSkippedResponse(response, options, spinner) {
+  if (response.data.skipped) {
+    spinner.info(response.data.message || 'Review skipped.');
+    if (options.json) {
+      output(JSON.stringify(response.data, null, 2));
+    }
+    return true;
+  }
+  return false;
+}
+
 /**
  * Available Gemini models for code review
  */
@@ -339,6 +357,12 @@ program
 
       clearInterval(timer);
       const elapsed = Math.floor((Date.now() - startTime) / 1000);
+
+      // Handle skipped response (integration mode prevents CLI review)
+      if (handleSkippedResponse(response, options, spinner)) {
+        return;
+      }
+
       spinner.succeed(`Review completed in ${elapsed}s!`);
 
       // Handle --comment flag: post results to PR
@@ -523,6 +547,12 @@ async function reviewUncommitted(mode, options) {
 
     clearInterval(timer);
     const elapsed = Math.floor((Date.now() - startTime) / 1000);
+
+    // Handle skipped response (integration mode prevents CLI review)
+    if (handleSkippedResponse(response, options, spinner)) {
+      return;
+    }
+
     spinner.succeed(`Review completed in ${elapsed}s!`);
 
     // Output results to stdout

package/src/index.test.js

@@ -1,5 +1,11 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { truncateFileData, formatErrorOutput, detectCIProvider, getPrUrl } from './index.js';
+import {
+  truncateFileData,
+  formatErrorOutput,
+  detectCIProvider,
+  getPrUrl,
+  handleSkippedResponse,
+} from './index.js';
 
 describe('CLI JSON output mode', () => {
   let stdoutSpy;
@@ -549,6 +555,114 @@ describe('--model flag behavior', () => {
   });
 });
 
+describe('skipped response handling', () => {
+  let stdoutSpy;
+
+  beforeEach(() => {
+    stdoutSpy = vi.spyOn(process.stdout, 'write').mockImplementation(() => true);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('should return true and call spinner.info when response is skipped', () => {
+    const response = {
+      data: {
+        skipped: true,
+        reason: 'webhook_mode_active',
+        message: 'Review skipped due to webhook mode.',
+      },
+    };
+    const options = { json: false };
+    const spinner = { info: vi.fn() };
+
+    const result = handleSkippedResponse(response, options, spinner);
+
+    expect(result).toBe(true);
+    expect(spinner.info).toHaveBeenCalledWith('Review skipped due to webhook mode.');
+  });
+
+  it('should return false when response is not skipped', () => {
+    const response = {
+      data: {
+        review: { issues: [], praises: [] },
+        summary: { total_issues: 0 },
+      },
+    };
+    const options = { json: false };
+    const spinner = { info: vi.fn() };
+
+    const result = handleSkippedResponse(response, options, spinner);
+
+    expect(result).toBe(false);
+    expect(spinner.info).not.toHaveBeenCalled();
+  });
+
+  it('should output JSON to stdout when json option is true and response is skipped', () => {
+    const response = {
+      data: {
+        skipped: true,
+        reason: 'local_reviews_disabled',
+        message: 'Local reviews disabled.',
+      },
+    };
+    const options = { json: true };
+    const spinner = { info: vi.fn() };
+
+    handleSkippedResponse(response, options, spinner);
+
+    expect(stdoutSpy).toHaveBeenCalledWith(JSON.stringify(response.data, null, 2) + '\n');
+  });
+
+  it('should not output JSON when json option is false', () => {
+    const response = {
+      data: {
+        skipped: true,
+        reason: 'reviews_disabled',
+        message: 'Reviews disabled.',
+      },
+    };
+    const options = { json: false };
+    const spinner = { info: vi.fn() };
+
+    handleSkippedResponse(response, options, spinner);
+
+    expect(stdoutSpy).not.toHaveBeenCalled();
+  });
+
+  it('should use default message when response message is empty', () => {
+    const response = {
+      data: {
+        skipped: true,
+        reason: 'webhook_mode_active',
+      },
+    };
+    const options = { json: false };
+    const spinner = { info: vi.fn() };
+
+    handleSkippedResponse(response, options, spinner);
+
+    expect(spinner.info).toHaveBeenCalledWith('Review skipped.');
+  });
+
+  it('should handle all skip reason types', () => {
+    const reasons = ['webhook_mode_active', 'reviews_disabled', 'local_reviews_disabled'];
+    const spinner = { info: vi.fn() };
+
+    reasons.forEach((reason) => {
+      const response = {
+        data: { skipped: true, reason, message: `Skipped: ${reason}` },
+      };
+
+      const result = handleSkippedResponse(response, { json: false }, spinner);
+      expect(result).toBe(true);
+    });
+
+    expect(spinner.info).toHaveBeenCalledTimes(3);
+  });
+});
+
 describe('getPrUrl', () => {
   const originalEnv = process.env;