codeant-cli 0.3.1 → 0.3.3

package/dist/review/package.json

@@ -0,0 +1,3 @@
+{
+  "type": "module"
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeant-cli",
-  "version": "0.3.1",
+  "version": "0.3.3",
   "description": "Code review CLI tool",
   "type": "module",
   "bin": {
@@ -23,14 +23,13 @@
     "type": "git",
     "url": ""
   },
-  "main": "./src/reviewHeadless.js",
+  "main": "./dist/review/index.js",
   "exports": {
-    ".": "./src/reviewHeadless.js",
-    "./review": "./src/reviewHeadless.js"
+    ".": "./dist/review/index.js",
+    "./review": "./dist/review/index.js"
   },
   "files": [
-    "dist",
-    "src"
+    "dist"
   ],
   "dependencies": {
     "@gitbeaker/rest": "^43.8.0",

package/src/commands/getBaseUrl.js

@@ -1,21 +0,0 @@
-import React, { useEffect } from 'react';
-import { Text, Box, useApp } from 'ink';
-import { getBaseUrl } from '../utils/baseUrl.js';
-
-export default function GetBaseUrl() {
-  const { exit } = useApp();
-
-  const activeUrl = getBaseUrl();
-  const source = process.env.CODEANT_API_URL ? 'env' : 'config';
-
-  useEffect(() => {
-    exit();
-  }, []);
-
-  return React.createElement(
-    Box,
-    { flexDirection: 'column', padding: 1 },
-    React.createElement(Text, { bold: true }, 'Base URL: ', activeUrl),
-    React.createElement(Text, { color: 'gray' }, 'Source: ', source)
-  );
-}

package/src/commands/login.js

@@ -1,125 +0,0 @@
-import React, { useState, useEffect } from 'react';
-import { Text, Box, useApp } from 'ink';
-import { getConfigValue, setConfigValue } from '../utils/config.js';
-import { getBaseUrl } from '../utils/baseUrl.js';
-import { randomUUID } from 'crypto';
-
-const POLL_INTERVAL = 10000; // 10 seconds
-const TIMEOUT = 10 * 60 * 1000; // 10 minutes
-
-export default function Login() {
-  const { exit } = useApp();
-  const [status, setStatus] = useState('opening');
-  const [error, setError] = useState(null);
-
-  useEffect(() => {
-    // Check if already logged in
-    const existingToken = getConfigValue('apiKey');
-    if (existingToken) {
-      setStatus('already_logged_in');
-      setTimeout(() => exit(), 100);
-      return;
-    }
-
-    const token = randomUUID();
-    const baseUrl = getBaseUrl();
-    const loginUrl = `https://app.codeant.ai?ideLoginToken=${token}`;
-    const pollUrl = `${baseUrl}/extension/login/status?apiKey=${token}`;
-
-    // Open browser
-    import('open').then(({ default: open }) => {
-      open(loginUrl);
-      setStatus('waiting');
-    }).catch(() => {
-      // Fallback: just show the URL
-      console.log(`\nOpen this URL in your browser:\n${loginUrl}\n`);
-      setStatus('waiting');
-    });
-
-    // Poll for login status
-    let timeoutId;
-    const intervalId = setInterval(async () => {
-      try {
-        const response = await fetch(pollUrl);
-        const data = await response.json();
-
-        if (data.status === 'yes') {
-          clearInterval(intervalId);
-          clearTimeout(timeoutId);
-
-          // Save the API key
-          setConfigValue('apiKey', token);
-
-          // Check for custom API URL based on email domain
-          if (data.item?.email?.includes('commvault.com')) {
-            setConfigValue('baseUrl', 'https://codeant-api.commvault.com');
-          }
-
-          setStatus('success');
-          setTimeout(() => exit(), 100);
-        }
-      } catch (err) {
-        // Silently continue polling
-      }
-    }, POLL_INTERVAL);
-
-    // Timeout after 10 minutes
-    timeoutId = setTimeout(() => {
-      clearInterval(intervalId);
-      setError('Login timed out. Please try again.');
-      setStatus('error');
-      setTimeout(() => exit(new Error('Login timed out')), 100);
-    }, TIMEOUT);
-
-    // Cleanup
-    return () => {
-      clearInterval(intervalId);
-      clearTimeout(timeoutId);
-    };
-  }, []);
-
-  if (status === 'already_logged_in') {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'yellow' }, 'Already logged in.'),
-      React.createElement(Text, { color: 'gray' }, 'Run "codeant logout" first to re-authenticate.')
-    );
-  }
-
-  if (status === 'opening') {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, null, 'Opening browser...')
-    );
-  }
-
-  if (status === 'waiting') {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'cyan' }, 'Waiting for login...'),
-      React.createElement(Text, { color: 'gray' }, 'Complete the login in your browser.'),
-      React.createElement(Text, { color: 'gray' }, 'Checking every 10 seconds. Timeout in 10 minutes.')
-    );
-  }
-
-  if (status === 'success') {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'green' }, '✓ Login successful!')
-    );
-  }
-
-  if (status === 'error') {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'red' }, '✗ ', error)
-    );
-  }
-
-  return null;
-}

package/src/commands/logout.js

@@ -1,30 +0,0 @@
-import React, { useEffect } from 'react';
-import { Text, Box, useApp } from 'ink';
-import { getConfigValue, setConfigValue } from '../utils/config.js';
-
-export default function Logout() {
-  const { exit } = useApp();
-
-  const wasLoggedIn = !!getConfigValue('apiKey');
-
-  useEffect(() => {
-    if (wasLoggedIn) {
-      setConfigValue('apiKey', null);
-    }
-    exit();
-  }, []);
-
-  if (!wasLoggedIn) {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'yellow' }, 'Not logged in.')
-    );
-  }
-
-  return React.createElement(
-    Box,
-    { flexDirection: 'column', padding: 1 },
-    React.createElement(Text, { color: 'green' }, '✓ Logged out successfully.')
-  );
-}

package/src/commands/review.js

@@ -1,294 +0,0 @@
-import React, { useEffect, useState } from 'react';
-import { useApp } from 'ink';
-import { getConfigValue } from '../utils/config.js';
-import ReviewApiHelper from '../utils/reviewApiHelper.js';
-import { fetchApi } from '../utils/fetchApi.js';
-import fs from 'fs';
-import path from 'path';
-import { executeTool } from '../tools/executeTool.js';
-import { track, shutdown as analyticsShutdown } from '../utils/analytics.js';
-import {
-  renderInitializing,
-  renderFetchingDiff,
-  renderReviewing,
-  renderNoFiles,
-  renderError,
-  renderNotLoggedIn,
-  renderDone,
-  mapLabelToSeverity,
-} from '../components/ReviewUI.js';
-
-const MAX_TURNS = 5;
-
-export default function Review({ scanType = 'all', lastNCommits = 1, failOn = 'CRITICAL', include = [], exclude = [], baseBranch = null, baseCommit = null }) {
-  const { exit } = useApp();
-  const [status, setStatus] = useState('initializing');
-  const [error, setError] = useState(null);
-  const [currentMessage, setCurrentMessage] = useState('');
-  const [issues, setIssues] = useState([]);
-  const [summary, setSummary] = useState(null);
-  const [startTime] = useState(() => Date.now());
-  const [fileCount, setFileCount] = useState(0);
-  const [reviewMeta, setReviewMeta] = useState(null);
-
-  const apiKey = getConfigValue('apiKey');
-
-  useEffect(() => {
-    if (!apiKey) {
-      if (process.stdin.isTTY) {
-        // Interactive session: don't exit immediately, let component render the error
-        return;
-      }
-      const id = setTimeout(() => exit(new Error('Not logged in')), 0);
-      return () => clearTimeout(id);
-    }
-  }, [apiKey, exit]);
-
-  if (!apiKey) {
-    return renderNotLoggedIn();
-  }
-
-  useEffect(() => {
-    let cancelled = false;
-
-    async function runReview() {
-      const reviewStartTime = Date.now();
-      track('review_triggered', { scan_type: scanType, source: 'cli_interactive' });
-
-      try {
-        // ── Fetch diff ─────────────────────────────────────────────────────
-        setStatus('fetching_diff');
-
-        const helper = new ReviewApiHelper(process.cwd());
-        await helper.init();
-        const gitRoot = helper.getGitRoot() || process.cwd();
-        const requestBody = await helper.buildReviewApiRequest(scanType, include, exclude, { lastNCommits, baseBranch, baseCommit });
-
-        const meta = requestBody?._meta || null;
-        setFileCount(meta?.reviewed_files?.length || Object.keys(requestBody?.file_contents || {}).length);
-        setReviewMeta(meta);
-
-        // Strip _meta before sending to API
-        delete requestBody._meta;
-
-        if (!requestBody?.diff_content?.length) {
-          if (!cancelled) setStatus('no_files');
-          return;
-        }
-
-        if (cancelled) return;
-        setStatus('reviewing');
-
-        // ── Split into per-file requests ───────────────────────────────────
-        const perFileRequests = ReviewApiHelper.splitIntoPerFileRequests(requestBody);
-
-        // ── Per-file agent turn loops (parallel) ──────────────────────────
-        setCurrentMessage(`Analyzing ${perFileRequests.length} file${perFileRequests.length !== 1 ? 's' : ''} in parallel...`);
-
-        const perFileResults = await Promise.all(
-          perFileRequests.map((fileReq) => {
-            const filename = fileReq._filename;
-            delete fileReq._filename;
-
-            // Convert file_contents dict → file_content/file_path for API
-            if (fileReq.file_contents?.[filename]) {
-              fileReq.file_content = fileReq.file_contents[filename];
-              fileReq.file_path = filename;
-            }
-            delete fileReq.file_contents;
-
-            return runTurnLoop(
-              fileReq,
-              gitRoot,
-              () => cancelled,
-              () => {},
-              false
-            );
-          })
-        );
-
-        // Pair each file's suggestions with its own diff — skip files with no real suggestions
-        const perFileWithSuggestions = perFileRequests.map((fileReq, i) => ({
-          diff_content: fileReq.diff_content,
-          suggestions: perFileResults[i].finalMessage?.content,
-          output: perFileResults[i].finalOutput,
-        })).filter(r => r.output?.code_suggestions?.length > 0);
-
-        if (cancelled) return;
-
-        // ── Per-file reflector loops (parallel) ─────────────────────────────
-        setCurrentMessage('Running reflector...');
-        const reflectorResults = await Promise.all(
-          perFileWithSuggestions.map(({ diff_content, suggestions }) =>
-            runTurnLoop(
-              {
-                diff_content,
-                prompt_template_name: 'reflector',
-                extra_variables: { suggestion_str: suggestions },
-              },
-              gitRoot,
-              () => cancelled,
-              () => {},
-              true
-            )
-          )
-        );
-
-        if (cancelled) return;
-
-        const reviewIssues = reflectorResults.flatMap(r =>
-          (r.finalOutput?.code_suggestions || []).map(issue => ({
-            issue_content: issue.issue_content || '',
-            relevant_file: issue.relevant_file || 'Unknown',
-            start_line: issue.start_line || 0,
-            label: issue.label || 'Code Quality',
-          }))
-        );
-        setIssues(reviewIssues);
-        setSummary(null);
-        setStatus('done');
-
-        const labelCounts = {};
-        for (const i of reviewIssues) { labelCounts[i.label] = (labelCounts[i.label] || 0) + 1; }
-        track('review_completed', {
-          scan_type: scanType,
-          source: 'cli_interactive',
-          issue_count: reviewIssues.length,
-          file_count: perFileRequests.length,
-          duration_ms: Date.now() - reviewStartTime,
-          label_counts: labelCounts,
-        });
-        await analyticsShutdown();
-
-      } catch (err) {
-        if (!cancelled) {
-          const errMsg = err instanceof Error ? err.message : String(err);
-          track('review_error', { scan_type: scanType, source: 'cli_interactive', error: errMsg });
-          await analyticsShutdown();
-          setError(errMsg);
-          setStatus('error');
-        }
-      }
-    }
-
-    runReview();
-    return () => { cancelled = true; };
-  }, [scanType, lastNCommits, apiKey]);
-
-  // Handle exit on terminal states
-  useEffect(() => {
-    if (status === 'done') {
-      const severityOrder = ['INFO', 'MINOR', 'MAJOR', 'CRITICAL', 'BLOCKER'];
-      const failOnIndex = severityOrder.indexOf(failOn);
-      if (failOnIndex === -1) {
-        setTimeout(() => { process.exitCode = 1; exit(new Error(`Invalid failOn value: ${failOn}`)); }, 100);
-        return;
-      }
-      const hasBlocking = issues.some(issue =>
-        severityOrder.indexOf(mapLabelToSeverity(issue.label)) >= failOnIndex
-      );
-      if (hasBlocking) {
-        setTimeout(() => { process.exitCode = 1; exit(new Error('Issues detected')); }, 100);
-      } else {
-        setTimeout(() => exit(), 100);
-      }
-    } else if (status === 'no_files') {
-      setTimeout(() => exit(), 100);
-    } else if (status === 'error') {
-      setTimeout(() => exit(new Error(error)), 100);
-    }
-  }, [status, issues, failOn, exit, error]);
-
-  // ── Renders ──────────────────────────────────────────────────────────────
-
-  if (status === 'initializing') return renderInitializing(startTime);
-  if (status === 'fetching_diff') return renderFetchingDiff(startTime);
-  if (status === 'reviewing') return renderReviewing(currentMessage, startTime, reviewMeta);
-  if (status === 'no_files') return renderNoFiles(scanType, lastNCommits);
-  if (status === 'error') return renderError(error);
-  if (status === 'done') return renderDone(issues, summary, failOn, startTime, fileCount, reviewMeta);
-
-  return null;
-}
-
-// ── Turn loop ──────────────────────────────────────────────────────────────
-
-async function runTurnLoop(initialPayload, gitRoot, isCancelled, onMessage, isReflectorLoop) {
-  let nextPayload = initialPayload;
-  let finalMessage = null;
-  let finalOutput = null;
-
-  for (let turn = 0; turn < MAX_TURNS; turn+=1) {
-    if (isCancelled()) break;
-
-    onMessage('Analyzing code...');
-
-    const resp = await fetchApi('/extension/pr-review/agent/turn', 'POST', nextPayload);
-
-    if (!resp || typeof resp !== 'object') {
-      throw new Error('Invalid response from agent API');
-    }
-
-    const sessionId = resp.session_id;
-    const assistantMsg = resp.response;
-    const done = resp.done;
-
-    if (!sessionId) {
-      throw new Error('Missing session_id in agent response');
-    }
-
-    finalMessage = assistantMsg;
-    if (resp.output) finalOutput = resp.output;
-
-    if (done) {
-      // Handle parsing errors in final message by dumping response content (only on reflector loop)
-      if (isReflectorLoop && resp.parsing_error) {
-        const dumpDir = path.join(gitRoot, '.codeant', 'codeSuggestions');
-        const dumpFile = path.join(dumpDir, `code_suggestions_${Date.now()}.txt`);
-        let dumpContent = resp.response?.content || JSON.stringify(resp, null, 2);
-
-        // Ensure dumpContent is a string
-        if (typeof dumpContent !== 'string') {
-          dumpContent = JSON.stringify(dumpContent, null, 2);
-        }
-
-        // Unescape all serialization escape sequences
-        dumpContent = dumpContent
-          .replace(/\\x([0-9a-fA-F]{2})/g, (_, hex) => String.fromCharCode(parseInt(hex, 16)))
-          .replace(/\\u([0-9a-fA-F]{4})/g, (_, hex) => String.fromCharCode(parseInt(hex, 16)))
-          .replace(/\\n/g, '\n')
-          .replace(/\\r/g, '\r')
-          .replace(/\\t/g, '\t')
-          .replace(/\\"/g, '"')
-          .replace(/\\'/g, "'")
-          .replace(/\\\\/g, '\\');
-
-        try {
-          // Create directories if they don't exist
-          await fs.promises.mkdir(dumpDir, { recursive: true });
-          await fs.promises.writeFile(dumpFile, dumpContent, 'utf8');
-          console.error(`⚠ Parsing error encountered. Full response dumped to: ${dumpFile}`);
-        } catch (writeErr) {
-          console.error(`⚠ Parsing error encountered (could not dump to file: ${writeErr.message})`);
-        }
-      }
-      break;
-    }
-
-    const toolCalls = assistantMsg?.tool_calls || [];
-    if (toolCalls.length === 0) break;
-
-    // Run tool calls in parallel to avoid serial latency when agent requests multiple reads
-    const toolResults = await Promise.all(
-      toolCalls.map(async (tc) => ({
-        tool_call_id: tc.id,
-        content: await executeTool(tc, gitRoot),
-      }))
-    );
-
-    nextPayload = { session_id: sessionId, tool_results: toolResults };
-  }
-
-  return { finalMessage, finalOutput };
-}
-

package/src/commands/secrets.js

@@ -1,148 +0,0 @@
-import { useState, useEffect } from 'react';
-import { useApp } from 'ink';
-import { getConfigValue } from '../utils/config.js';
-import { fetchApi } from '../utils/fetchApi.js';
-import SecretsApiHelper from '../utils/secretsApiHelper.js';
-import {
-  renderInitializing,
-  renderFetchingDiff,
-  renderScanning,
-  renderNoFiles,
-  renderError,
-  renderNotLoggedIn,
-  renderDone,
-} from '../components/SecretsUI.js';
-
-export default function Secrets({ scanType = 'all', failOn = 'CRITICAL', include = [], exclude = [], lastNCommits = 1, baseBranch = null, baseCommit = null }) {
-  const { exit } = useApp();
-  const [status, setStatus] = useState('initializing');
-  const [secrets, setSecrets] = useState([]);
-  const [error, setError] = useState(null);
-  const [fileCount, setFileCount] = useState(0);
-  const [scanMeta, setScanMeta] = useState(null);
-  const [startTime] = useState(() => Date.now());
-
-  const apiKey = getConfigValue('apiKey');
-
-  // Handle not logged in state
-  useEffect(() => {
-    if (!apiKey) {
-      if (process.stdin.isTTY) {
-        return;
-      }
-      const id = setTimeout(() => exit(new Error('Not logged in')), 0);
-      return () => clearTimeout(id);
-    }
-  }, [apiKey, exit]);
-
-  if (!apiKey) {
-    return renderNotLoggedIn();
-  }
-
-  // Helper to check if a secret should cause failure based on failOn level
-  const shouldFailOn = (confidenceScore) => {
-    const score = confidenceScore?.toUpperCase();
-    if (score === 'FALSE_POSITIVE') return false;
-    if (failOn === 'HIGH') return score === 'HIGH';
-    if (failOn === 'MEDIUM') return score === 'HIGH' || score === 'MEDIUM';
-    return true; // 'all' - fail on any non-false-positive
-  };
-
-  useEffect(() => {
-    let cancelled = false;
-
-    async function scanSecrets() {
-      try {
-        if (cancelled) return;
-        setStatus('fetching_diff');
-
-        // Initialize git helper and get files
-        const helper = new SecretsApiHelper(process.cwd());
-        await helper.init();
-
-        if (cancelled) return;
-        const requestBody = await helper.buildSecretsApiRequest(scanType, include, exclude, { lastNCommits, baseBranch, baseCommit });
-        if (cancelled) return;
-
-        const meta = requestBody._meta || null;
-        setScanMeta(meta);
-
-        // Strip _meta before sending to API
-        delete requestBody._meta;
-
-        if (requestBody.files.length === 0) {
-          if (!cancelled) setStatus('no_files');
-          return;
-        }
-
-        setFileCount(requestBody.files.length);
-        setStatus('scanning');
-
-        // Call the secrets detection API
-        const response = await fetchApi(
-          '/extension/pr-review/secrets-detection',
-          'POST',
-          requestBody
-        );
-
-        if (cancelled) return;
-        const detectedSecrets = response.secretsDetected || [];
-
-        // Filter to only include files with actual secrets
-        const filesWithSecrets = detectedSecrets.filter(
-          file => file.secrets && file.secrets.length > 0
-        );
-
-        if (!cancelled) {
-          setSecrets(filesWithSecrets);
-          setStatus('done');
-        }
-      } catch (err) {
-        if (!cancelled) {
-          setError(err.message);
-          setStatus('error');
-        }
-      }
-    }
-
-    scanSecrets();
-
-    return () => {
-      cancelled = true;
-    };
-  }, [scanType, include, exclude, lastNCommits, baseBranch, baseCommit]);
-
-  // Handle exit after status changes
-  useEffect(() => {
-    if (status === 'done') {
-      // Check if any secrets should cause failure based on failOn level
-      const hasBlockingSecrets = secrets.some(file =>
-        file.secrets.some(secret => shouldFailOn(secret.confidence_score))
-      );
-
-      if (hasBlockingSecrets) {
-        setTimeout(() => {
-          process.exitCode = 1;
-          exit(new Error('Secrets detected'));
-        }, 100);
-      } else {
-        setTimeout(() => exit(), 100);
-      }
-    } else if (status === 'no_files') {
-      setTimeout(() => exit(), 100);
-    } else if (status === 'error') {
-      setTimeout(() => exit(new Error(error)), 100);
-    }
-  }, [status, secrets]);
-
-  // ── Renders ──────────────────────────────────────────────────────────────
-
-  if (status === 'initializing') return renderInitializing(startTime);
-  if (status === 'fetching_diff') return renderFetchingDiff(startTime);
-  if (status === 'scanning') return renderScanning(startTime, fileCount, scanMeta);
-  if (status === 'no_files') return renderNoFiles(scanType, lastNCommits, baseBranch, baseCommit);
-  if (status === 'error') return renderError(error);
-  if (status === 'done') return renderDone(secrets, failOn, shouldFailOn, startTime, fileCount, scanMeta);
-
-  return null;
-}

package/src/commands/setBaseUrl.js

@@ -1,37 +0,0 @@
-import React, { useEffect } from 'react';
-import { Text, Box, useApp } from 'ink';
-import { setConfigValue, CONFIG_FILE } from '../utils/config.js';
-
-export default function SetBaseUrl({ url }) {
-  const { exit } = useApp();
-
-  useEffect(() => {
-    if (!url) {
-      exit(new Error('URL is required'));
-      return;
-    }
-
-    try {
-      setConfigValue('baseUrl', url);
-      exit();
-    } catch (err) {
-      exit(err);
-    }
-  }, []);
-
-  if (!url) {
-    return React.createElement(
-      Box,
-      { flexDirection: 'column', padding: 1 },
-      React.createElement(Text, { color: 'red' }, '✗ Error: URL is required'),
-      React.createElement(Text, { color: 'gray' }, 'Usage: codeant set-base-url <url>')
-    );
-  }
-
-  return React.createElement(
-    Box,
-    { flexDirection: 'column', padding: 1 },
-    React.createElement(Text, { color: 'green' }, '✓ Base URL set to: ', url),
-    React.createElement(Text, { color: 'gray' }, 'Saved to: ', CONFIG_FILE)
-  );
-}