korekt-cli 0.9.6 → 0.10.0

package/package.json

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

package/scripts/azure.sh

@@ -232,6 +232,9 @@ post_review_thread() {
 TOTAL_ISSUES=$(jq -r '.data.summary.total_issues // 0' "$RESULTS_FILE")
 TOTAL_PRAISES=$(jq -r '.data.summary.total_praises // 0' "$RESULTS_FILE")
 CRITICAL_ISSUES=$(jq -r '.data.summary.critical // 0' "$RESULTS_FILE")
+CHANGE_SUMMARY=$(jq -r '.data.change_classification?.summary // ""' "$RESULTS_FILE")
+CHANGE_INTENT=$(jq -r '.data.change_classification?.intent // ""' "$RESULTS_FILE")
+CHANGE_ASPECTS=$(jq -r '.data.change_classification?.aspects // [] | join(", ")' "$RESULTS_FILE")
 
 # Post inline comments for issues (excluding low severity)
 if [ "$TOTAL_ISSUES" -gt 0 ] && [ "$POST_INLINE_COMMENTS" = "true" ]; then
@@ -318,6 +321,20 @@ else
   echo "🤖 **Automated Code Review Results**" >> "$COMMENT_FILE"
   echo "" >> "$COMMENT_FILE"
 
+  # Change Summary section
+  if [ -n "$CHANGE_SUMMARY" ]; then
+    echo "### 📝 Change Summary" >> "$COMMENT_FILE"
+    echo "$CHANGE_SUMMARY" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+
+    # Build metadata line
+    META=""
+    [ -n "$CHANGE_INTENT" ] && META="Intent: $CHANGE_INTENT"
+    [ -n "$CHANGE_ASPECTS" ] && { [ -n "$META" ] && META="$META | "; META="${META}Aspects: $CHANGE_ASPECTS"; }
+    [ -n "$META" ] && echo "_${META}_" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+  fi
+
   # Praises section
   if [ "$TOTAL_PRAISES" -gt 0 ]; then
     echo "### ✨ Praises ($TOTAL_PRAISES)" >> "$COMMENT_FILE"

package/scripts/bitbucket.sh

@@ -205,6 +205,9 @@ post_inline_comment() {
 TOTAL_ISSUES=$(jq -r '.data.summary.total_issues // 0' "$RESULTS_FILE")
 TOTAL_PRAISES=$(jq -r '.data.summary.total_praises // 0' "$RESULTS_FILE")
 CRITICAL_ISSUES=$(jq -r '.data.summary.critical // 0' "$RESULTS_FILE")
+CHANGE_SUMMARY=$(jq -r '.data.change_classification?.summary // ""' "$RESULTS_FILE")
+CHANGE_INTENT=$(jq -r '.data.change_classification?.intent // ""' "$RESULTS_FILE")
+CHANGE_ASPECTS=$(jq -r '.data.change_classification?.aspects // [] | join(", ")' "$RESULTS_FILE")
 
 # Post inline comments for issues (excluding low severity)
 if [ "$TOTAL_ISSUES" -gt 0 ] && [ "$POST_INLINE_COMMENTS" = "true" ]; then
@@ -288,6 +291,20 @@ else
   echo "🤖 **Automated Code Review Results**" >> "$COMMENT_FILE"
   echo "" >> "$COMMENT_FILE"
 
+  # Change Summary section
+  if [ -n "$CHANGE_SUMMARY" ]; then
+    echo "### 📝 Change Summary" >> "$COMMENT_FILE"
+    echo "$CHANGE_SUMMARY" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+
+    # Build metadata line
+    META=""
+    [ -n "$CHANGE_INTENT" ] && META="Intent: $CHANGE_INTENT"
+    [ -n "$CHANGE_ASPECTS" ] && { [ -n "$META" ] && META="$META | "; META="${META}Aspects: $CHANGE_ASPECTS"; }
+    [ -n "$META" ] && echo "_${META}_" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+  fi
+
   # Praises section
   if [ "$TOTAL_PRAISES" -gt 0 ]; then
     echo "### ✨ Praises ($TOTAL_PRAISES)" >> "$COMMENT_FILE"

package/scripts/github.sh

@@ -204,6 +204,9 @@ post_review_comment() {
 TOTAL_ISSUES=$(jq -r '.data.summary.total_issues // 0' "$RESULTS_FILE")
 TOTAL_PRAISES=$(jq -r '.data.summary.total_praises // 0' "$RESULTS_FILE")
 CRITICAL_ISSUES=$(jq -r '.data.summary.critical // 0' "$RESULTS_FILE")
+CHANGE_SUMMARY=$(jq -r '.data.change_classification?.summary // ""' "$RESULTS_FILE")
+CHANGE_INTENT=$(jq -r '.data.change_classification?.intent // ""' "$RESULTS_FILE")
+CHANGE_ASPECTS=$(jq -r '.data.change_classification?.aspects // [] | join(", ")' "$RESULTS_FILE")
 
 # Post inline comments for issues
 if [ "$TOTAL_ISSUES" -gt 0 ]; then
@@ -287,6 +290,20 @@ else
   echo "🤖 **Automated Code Review Results**" >> "$COMMENT_FILE"
   echo "" >> "$COMMENT_FILE"
 
+  # Change Summary section
+  if [ -n "$CHANGE_SUMMARY" ]; then
+    echo "### 📝 Change Summary" >> "$COMMENT_FILE"
+    echo "$CHANGE_SUMMARY" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+
+    # Build metadata line
+    META=""
+    [ -n "$CHANGE_INTENT" ] && META="Intent: $CHANGE_INTENT"
+    [ -n "$CHANGE_ASPECTS" ] && { [ -n "$META" ] && META="$META | "; META="${META}Aspects: $CHANGE_ASPECTS"; }
+    [ -n "$META" ] && echo "_${META}_" >> "$COMMENT_FILE"
+    echo "" >> "$COMMENT_FILE"
+  fi
+
   # Praises section
   if [ "$TOTAL_PRAISES" -gt 0 ]; then
     echo "### ✨ Praises ($TOTAL_PRAISES)" >> "$COMMENT_FILE"

package/src/formatter.js

@@ -76,10 +76,28 @@ function toAbsolutePath(filePath) {
  * @param {Object} data - The API response data
  */
 export function formatReviewOutput(data) {
-  const { review, summary } = data.data;
+  const { review, summary, change_classification: changeClassification } = data.data;
 
   console.log(chalk.bold.blue('🤖 Automated Code Review Results\n'));
 
+  // --- Change Summary Section ---
+  if (changeClassification && changeClassification.summary) {
+    console.log(chalk.bold.cyan('📝 Change Summary\n'));
+    console.log(`  ${changeClassification.summary}\n`);
+
+    const meta = [];
+    if (changeClassification.intent) {
+      meta.push(`Intent: ${changeClassification.intent}`);
+    }
+    if (changeClassification.aspects && changeClassification.aspects.length > 0) {
+      meta.push(`Aspects: ${changeClassification.aspects.join(', ')}`);
+    }
+    if (meta.length > 0) {
+      console.log(chalk.gray(`  ${meta.join(' | ')}`));
+    }
+    console.log(); // Spacing after change summary
+  }
+
   // --- Praises Section ---
   if (review && review.praises && review.praises.length > 0) {
     console.log(chalk.bold.magenta(`✨ Praises (${summary.total_praises})`));

package/src/formatter.test.js

@@ -0,0 +1,96 @@
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { formatReviewOutput } from './formatter.js';
+
+describe('formatReviewOutput', () => {
+  let consoleSpy;
+
+  beforeEach(() => {
+    consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  describe('change_classification output', () => {
+    it('should display change summary when change_classification.summary is present', () => {
+      const data = {
+        data: {
+          review: { issues: [], praises: [] },
+          summary: { total_issues: 0, total_praises: 0 },
+          change_classification: {
+            intent: 'fix',
+            aspects: ['security', 'tests'],
+            summary: 'Fix authentication bypass vulnerability',
+          },
+        },
+      };
+
+      formatReviewOutput(data);
+
+      const allCalls = consoleSpy.mock.calls.map((call) => call[0]);
+      expect(allCalls.some((call) => call.includes('Change Summary'))).toBe(true);
+      expect(
+        allCalls.some((call) => call.includes('Fix authentication bypass vulnerability'))
+      ).toBe(true);
+      expect(allCalls.some((call) => call.includes('Intent: fix'))).toBe(true);
+      expect(allCalls.some((call) => call.includes('Aspects: security, tests'))).toBe(true);
+    });
+
+    it('should not display change_classification section when summary is missing', () => {
+      const data = {
+        data: {
+          review: { issues: [], praises: [] },
+          summary: { total_issues: 0, total_praises: 0 },
+          change_classification: null,
+        },
+      };
+
+      formatReviewOutput(data);
+
+      const allCalls = consoleSpy.mock.calls.map((call) => call[0]);
+      expect(allCalls.some((call) => call && call.includes('Change Summary'))).toBe(false);
+    });
+
+    it('should handle change_classification with only summary (no intent/aspects)', () => {
+      const data = {
+        data: {
+          review: { issues: [], praises: [] },
+          summary: { total_issues: 0, total_praises: 0 },
+          change_classification: {
+            summary: 'Add new feature',
+          },
+        },
+      };
+
+      formatReviewOutput(data);
+
+      const allCalls = consoleSpy.mock.calls.map((call) => call[0]);
+      expect(allCalls.some((call) => call.includes('Change Summary'))).toBe(true);
+      expect(allCalls.some((call) => call.includes('Add new feature'))).toBe(true);
+      // Should not have metadata line when no intent/aspects
+      expect(allCalls.some((call) => call && call.includes('Intent:'))).toBe(false);
+    });
+
+    it('should handle change_classification with empty aspects array', () => {
+      const data = {
+        data: {
+          review: { issues: [], praises: [] },
+          summary: { total_issues: 0, total_praises: 0 },
+          change_classification: {
+            intent: 'feature',
+            aspects: [],
+            summary: 'Add user dashboard',
+          },
+        },
+      };
+
+      formatReviewOutput(data);
+
+      const allCalls = consoleSpy.mock.calls.map((call) => call[0]);
+      expect(allCalls.some((call) => call.includes('Add user dashboard'))).toBe(true);
+      expect(allCalls.some((call) => call && call.includes('Intent: feature'))).toBe(true);
+      expect(allCalls.some((call) => call && call.includes('Aspects:'))).toBe(false);
+    });
+  });
+});

package/src/git-logic.test.js

@@ -259,6 +259,64 @@ describe('runLocalReview - branch fetching', () => {
   });
 });
 
+describe('runLocalReview - no files changed', () => {
+  beforeEach(() => {
+    vi.mock('execa');
+    vi.spyOn(console, 'error').mockImplementation(() => {});
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('should return payload with empty changed_files when diff has no files', 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 === 'git rev-parse HEAD') {
+        return { stdout: 'abc123def456789' };
+      }
+      if (command.includes('rev-parse --show-toplevel')) {
+        return { stdout: '/path/to/repo' };
+      }
+      if (command.includes('rev-parse --verify main')) {
+        return { stdout: 'commit-hash' };
+      }
+      if (command === 'git fetch origin main') {
+        return { stdout: '' };
+      }
+      if (command.includes('merge-base origin/main HEAD')) {
+        return { stdout: 'abc123' };
+      }
+      if (command.includes('log --no-merges --pretty=%B---EOC---')) {
+        return { stdout: 'feat: some commit---EOC---' };
+      }
+      if (command.includes('log --no-merges --format=%ae|%an')) {
+        return { stdout: 'user@example.com|User Name' };
+      }
+      // No files changed
+      if (command.includes('diff --name-status')) {
+        return { stdout: '' };
+      }
+
+      return { stdout: '' };
+    });
+
+    const result = await runLocalReview('main');
+
+    expect(result).not.toBeNull();
+    expect(result.changed_files).toEqual([]);
+    expect(result.source_branch).toBe('feature-branch');
+    expect(result.destination_branch).toBe('main');
+  });
+});
+
 describe('runLocalReview - fork point detection', () => {
   beforeEach(() => {
     vi.mock('execa');

package/src/index.js

@@ -165,6 +165,12 @@ program
       process.exit(1);
     }
 
+    // Exit successfully if no files to review (common in CI when PR has no code changes)
+    if (payload.changed_files.length === 0) {
+      log(chalk.green('✓ No files changed to review. Exiting successfully.'));
+      return;
+    }
+
     // If dry-run, just show the payload and exit
     if (options.dryRun) {
       log(chalk.yellow('\n📋 Dry Run - Payload that would be sent:\n'));