korekt-cli 0.12.0 → 0.13.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "korekt-cli",
-  "version": "0.12.0",
+  "version": "0.13.1",
   "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;