korekt-cli 0.8.5 → 0.9.0

package/package.json

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

package/src/git-logic.js

@@ -1,6 +1,6 @@
 import { execa } from 'execa';
 import chalk from 'chalk';
-import { detectCIProvider } from './utils.js';
+import { detectCIProvider, getPrUrl } from './utils.js';
 
 /**
  * Truncate content to a maximum number of lines using "head and tail".
@@ -246,6 +246,7 @@ export async function runUncommittedReview(mode = 'unstaged') {
       source_branch: branchName,
       changed_lines: calculateChangedLines(changedFiles),
       is_ci: detectCIProvider() !== null,
+      pr_url: null, // Uncommitted changes are never part of a PR
     };
   } catch (error) {
     console.error(chalk.red('Failed to analyze uncommitted changes:'), error.message);
@@ -527,6 +528,7 @@ export async function runLocalReview(targetBranch = null, ignorePatterns = null)
       contributors,
       changed_lines: calculateChangedLines(changedFiles),
       is_ci: detectCIProvider() !== null,
+      pr_url: getPrUrl(),
     };
   } catch (error) {
     console.error(chalk.red('Failed to run local review analysis:'), error.message);

package/src/git-logic.test.js

@@ -837,6 +837,7 @@ describe('is_ci flag in payload', () => {
     vi.mock('execa');
     vi.mock('./utils.js', () => ({
       detectCIProvider: vi.fn(),
+      getPrUrl: vi.fn().mockReturnValue(null),
     }));
   });
 

package/src/index.js

@@ -17,7 +17,7 @@ import { formatReviewOutput } from './formatter.js';
 import { detectCIProvider, truncateFileData, formatErrorOutput } from './utils.js';
 
 // Re-export utilities for backward compatibility
-export { detectCIProvider, truncateFileData, formatErrorOutput } from './utils.js';
+export { detectCIProvider, truncateFileData, formatErrorOutput, getPrUrl } from './utils.js';
 
 const require = createRequire(import.meta.url);
 const { version } = require('../package.json');

package/src/index.test.js

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
-import { truncateFileData, formatErrorOutput, detectCIProvider } from './index.js';
+import { truncateFileData, formatErrorOutput, detectCIProvider, getPrUrl } from './index.js';
 
 describe('CLI JSON output mode', () => {
   let stdoutSpy;
@@ -438,3 +438,111 @@ describe('--comment flag behavior', () => {
     expect(shouldShowConfirmation).toBe(true);
   });
 });
+
+describe('getPrUrl', () => {
+  const originalEnv = process.env;
+
+  beforeEach(() => {
+    // Reset environment before each test
+    process.env = { ...originalEnv };
+    // Clear all PR-related env vars
+    delete process.env.GITHUB_REPOSITORY;
+    delete process.env.PR_NUMBER;
+    delete process.env.BITBUCKET_WORKSPACE;
+    delete process.env.BITBUCKET_REPO_SLUG;
+    delete process.env.BITBUCKET_PR_ID;
+    delete process.env.SYSTEM_COLLECTIONURI;
+    delete process.env.SYSTEM_TEAMPROJECT;
+    delete process.env.BUILD_REPOSITORY_NAME;
+    delete process.env.SYSTEM_PULLREQUEST_PULLREQUESTID;
+  });
+
+  afterEach(() => {
+    process.env = originalEnv;
+  });
+
+  it('should return GitHub PR URL when GitHub env vars are set', () => {
+    process.env.GITHUB_REPOSITORY = 'owner/repo';
+    process.env.PR_NUMBER = '123';
+
+    expect(getPrUrl()).toBe('https://github.com/owner/repo/pull/123');
+  });
+
+  it('should return Bitbucket PR URL when Bitbucket env vars are set', () => {
+    process.env.BITBUCKET_WORKSPACE = 'myworkspace';
+    process.env.BITBUCKET_REPO_SLUG = 'myrepo';
+    process.env.BITBUCKET_PR_ID = '456';
+
+    expect(getPrUrl()).toBe('https://bitbucket.org/myworkspace/myrepo/pull-requests/456');
+  });
+
+  it('should return Azure DevOps PR URL when Azure env vars are set', () => {
+    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMPROJECT = 'myproject';
+    process.env.BUILD_REPOSITORY_NAME = 'myrepo';
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
+
+    expect(getPrUrl()).toBe('https://dev.azure.com/myorg/myproject/_git/myrepo/pullrequest/789');
+  });
+
+  it('should strip trailing slash from Azure DevOps collection URI', () => {
+    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMPROJECT = 'myproject';
+    process.env.BUILD_REPOSITORY_NAME = 'myrepo';
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
+
+    const url = getPrUrl();
+    expect(url).not.toContain('myorg//myproject');
+    expect(url).toContain('myorg/myproject');
+  });
+
+  it('should URL-encode spaces in Azure DevOps project and repo names', () => {
+    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMPROJECT = 'My Project';
+    process.env.BUILD_REPOSITORY_NAME = 'My Repo';
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID = '789';
+
+    expect(getPrUrl()).toBe(
+      'https://dev.azure.com/myorg/My%20Project/_git/My%20Repo/pullrequest/789'
+    );
+  });
+
+  it('should return null when no PR env vars are set', () => {
+    expect(getPrUrl()).toBe(null);
+  });
+
+  it('should return null when only partial GitHub env vars are set', () => {
+    process.env.GITHUB_REPOSITORY = 'owner/repo';
+    // PR_NUMBER not set
+
+    expect(getPrUrl()).toBe(null);
+  });
+
+  it('should return null when only partial Bitbucket env vars are set', () => {
+    process.env.BITBUCKET_WORKSPACE = 'myworkspace';
+    process.env.BITBUCKET_REPO_SLUG = 'myrepo';
+    // BITBUCKET_PR_ID not set
+
+    expect(getPrUrl()).toBe(null);
+  });
+
+  it('should return null when only partial Azure env vars are set', () => {
+    process.env.SYSTEM_COLLECTIONURI = 'https://dev.azure.com/myorg/';
+    process.env.SYSTEM_TEAMPROJECT = 'myproject';
+    // BUILD_REPOSITORY_NAME and SYSTEM_PULLREQUEST_PULLREQUESTID not set
+
+    expect(getPrUrl()).toBe(null);
+  });
+
+  it('should prioritize GitHub over other providers when multiple are set', () => {
+    // Set all providers
+    process.env.GITHUB_REPOSITORY = 'owner/repo';
+    process.env.PR_NUMBER = '123';
+    process.env.BITBUCKET_WORKSPACE = 'myworkspace';
+    process.env.BITBUCKET_REPO_SLUG = 'myrepo';
+    process.env.BITBUCKET_PR_ID = '456';
+
+    // GitHub should be detected first due to check order
+    expect(getPrUrl()).toBe('https://github.com/owner/repo/pull/123');
+  });
+});

package/src/utils.js

@@ -15,6 +15,36 @@ export function detectCIProvider() {
   return null;
 }
 
+/**
+ * Build PR URL from CI environment variables
+ * @returns {string|null} Full PR URL or null if not in CI PR context
+ */
+export function getPrUrl() {
+  // GitHub Actions
+  if (process.env.GITHUB_REPOSITORY && process.env.PR_NUMBER) {
+    return `https://github.com/${process.env.GITHUB_REPOSITORY}/pull/${process.env.PR_NUMBER}`;
+  }
+  // Bitbucket Pipelines
+  if (
+    process.env.BITBUCKET_WORKSPACE &&
+    process.env.BITBUCKET_REPO_SLUG &&
+    process.env.BITBUCKET_PR_ID
+  ) {
+    return `https://bitbucket.org/${process.env.BITBUCKET_WORKSPACE}/${process.env.BITBUCKET_REPO_SLUG}/pull-requests/${process.env.BITBUCKET_PR_ID}`;
+  }
+  // Azure DevOps Pipelines
+  if (
+    process.env.SYSTEM_COLLECTIONURI &&
+    process.env.SYSTEM_TEAMPROJECT &&
+    process.env.BUILD_REPOSITORY_NAME &&
+    process.env.SYSTEM_PULLREQUEST_PULLREQUESTID
+  ) {
+    const collectionUri = process.env.SYSTEM_COLLECTIONURI.replace(/\/$/, '');
+    return `${collectionUri}/${encodeURIComponent(process.env.SYSTEM_TEAMPROJECT)}/_git/${encodeURIComponent(process.env.BUILD_REPOSITORY_NAME)}/pullrequest/${process.env.SYSTEM_PULLREQUEST_PULLREQUESTID}`;
+  }
+  return null;
+}
+
 /**
  * Truncates file data (diff and content) for display purposes
  * @param {Object} file - File object with path, status, diff, content, etc.