nayan-ai 1.0.0-beta.1 → 1.0.0-beta.3

package/src/codex.ts

@@ -1,158 +0,0 @@
-import { spawn } from 'child_process';
-import type { CodeIssue } from './types.js';
-import { logProcessor, type CodexEvent } from './logs.js';
-import { getReviewPrompt } from './prompt.js';
-
-export interface CodexOptions {
-  verbose?: boolean;
-}
-
-export async function analyzeWithCodex(
-  repoPath: string,
-  baseBranch: string = 'origin/main',
-  options: CodexOptions
-): Promise<CodeIssue[]> {
-  const response = await runCodexExec(repoPath, baseBranch, options);
-  return parseCodexResponse(response);
-}
-
-async function runCodexExec(
-  repoPath: string,
-  baseBranch: string,
-  options: CodexOptions
-): Promise<string> {
-  const prompt = getReviewPrompt(baseBranch);
-
-  return new Promise((resolve, reject) => {
-    // Use codex exec with --json for structured output
-    const args = ['@openai/codex', 'exec', '--json', '--full-auto', prompt];
-
-    if (options.verbose) {
-      console.log(`\n[Codex] Running: npx ${args.join(' ')}`);
-      console.log(`[Codex] Working directory: ${repoPath}`);
-    }
-
-    const startTime = Date.now();
-
-    const child = spawn('npx', args, {
-      cwd: repoPath,
-      stdio: ['pipe', 'pipe', 'pipe'],
-    });
-
-    let stdout = '';
-    let stderr = '';
-
-    // Reset log processor for new analysis
-    logProcessor.reset();
-
-    child.stdout.on('data', (data) => {
-      const chunk = data.toString();
-      stdout += chunk;
-
-      // Parse JSONL and show progress
-      for (const line of chunk.split('\n')) {
-        if (!line.trim()) continue;
-        try {
-          const event: CodexEvent = JSON.parse(line);
-
-          if (options.verbose) {
-            console.log(JSON.stringify(event, null, 2));
-          } else {
-            // Use log processor for nice interactive output
-            logProcessor.processEvent(event);
-          }
-        } catch {
-          // Not JSON
-        }
-      }
-    });
-
-    child.stderr.on('data', (data) => {
-      const chunk = data.toString();
-      stderr += chunk;
-      if (options.verbose) {
-        process.stderr.write(chunk);
-      }
-    });
-
-    child.on('close', (code) => {
-      const elapsed = Date.now() - startTime;
-
-      console.log(`\n[Codex] Completed in ${(elapsed / 1000).toFixed(1)}s`);
-
-      if (code !== 0) {
-        const errorMsg = stderr || stdout || 'Unknown error';
-        reject(new Error(`Codex review failed (exit ${code}): ${errorMsg.slice(0, 500)}`));
-        return;
-      }
-
-      resolve(stdout);
-    });
-
-    child.on('error', (err) => {
-      if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
-        reject(new Error('npx not found. Install Node.js/npm (Node 18+) to run nayan-ai.'));
-        return;
-      }
-      reject(err);
-    });
-  });
-}
-
-function parseCodexResponse(response: string): CodeIssue[] {
-  // codex exec --json outputs JSONL - parse each line looking for agent_message with issues
-  const lines = response.split('\n');
-
-  for (const line of lines) {
-    if (!line.trim()) continue;
-
-    try {
-      const event = JSON.parse(line);
-
-      // Look for agent_message in item.completed events
-      if (event.type === 'item.completed' && event.item?.type === 'agent_message') {
-        const text = event.item.text;
-        if (text) {
-          // Parse the JSON from the agent's message
-          const issuesJson = JSON.parse(text);
-          if (issuesJson.issues && Array.isArray(issuesJson.issues)) {
-            return issuesJson.issues
-              .filter((item: any) => item.message)
-              .map((item: any) => ({
-                filename: item.filename || 'unknown',
-                line: item.line || 0,
-                category: item.category || 'functionality',
-                severity: item.severity || 'info',
-                message: item.message,
-                suggestion: item.suggestion,
-              }));
-          }
-        }
-      }
-    } catch {
-      // Not valid JSON, skip
-    }
-  }
-
-  // Fallback: try to find raw JSON in the response
-  const jsonMatch = response.match(/\{\s*"issues"\s*:\s*\[[\s\S]*?\]\s*\}/);
-  if (jsonMatch) {
-    try {
-      const parsed = JSON.parse(jsonMatch[0]);
-      return (parsed.issues || [])
-        .filter((item: any) => item.message)
-        .map((item: any) => ({
-          filename: item.filename || 'unknown',
-          line: item.line || 0,
-          category: item.category || 'functionality',
-          severity: item.severity || 'info',
-          message: item.message,
-          suggestion: item.suggestion,
-        }));
-    } catch {
-      // ignore
-    }
-  }
-  
-  return [];
-}

package/src/github.ts

@@ -1,108 +0,0 @@
-import type { PRInfo, PullRequest, PullRequestFile, FileChange, ReviewComment } from './types.js';
-
-export class GitHubClient {
-  private token: string;
-  private apiBase: string;
-
-  constructor(token: string, githubUrl?: string) {
-    this.token = token;
-    this.apiBase = githubUrl 
-      ? `${githubUrl.replace(/\/$/, '')}/api/v3`
-      : 'https://api.github.com';
-  }
-
-  private async fetch<T>(endpoint: string, options: RequestInit = {}): Promise<T> {
-    const url = `${this.apiBase}${endpoint}`;
-    const response = await fetch(url, {
-      ...options,
-      headers: {
-        'Accept': 'application/vnd.github.v3+json',
-        'Authorization': `Bearer ${this.token}`,
-        'User-Agent': 'nayan-ai',
-        ...options.headers,
-      },
-    });
-
-    if (!response.ok) {
-      const body = await response.text();
-      throw new Error(`GitHub API error (${response.status}): ${body}`);
-    }
-
-    return response.json() as Promise<T>;
-  }
-
-  async getPullRequest(pr: PRInfo): Promise<PullRequest> {
-    return this.fetch<PullRequest>(`/repos/${pr.owner}/${pr.repo}/pulls/${pr.number}`);
-  }
-
-  async getPullRequestFiles(pr: PRInfo): Promise<PullRequestFile[]> {
-    return this.fetch<PullRequestFile[]>(`/repos/${pr.owner}/${pr.repo}/pulls/${pr.number}/files?per_page=100`);
-  }
-
-  async postReview(
-    pr: PRInfo,
-    commitId: string,
-    body: string,
-    comments: ReviewComment[]
-  ): Promise<void> {
-    await this.fetch(`/repos/${pr.owner}/${pr.repo}/pulls/${pr.number}/reviews`, {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({
-        commit_id: commitId,
-        body,
-        event: 'COMMENT',
-        comments,
-      }),
-    });
-  }
-
-  async postComment(pr: PRInfo, body: string): Promise<void> {
-    await this.fetch(`/repos/${pr.owner}/${pr.repo}/issues/${pr.number}/comments`, {
-      method: 'POST',
-      headers: { 'Content-Type': 'application/json' },
-      body: JSON.stringify({ body }),
-    });
-  }
-}
-
-export function parseFiles(files: PullRequestFile[]): FileChange[] {
-  return files
-    .filter((f): f is PullRequestFile & { patch: string } => !!f.patch)
-    .map((f) => ({
-      filename: f.filename,
-      patch: f.patch,
-    }));
-}
-
-export function parsePRReference(input: string): PRInfo & { githubUrl?: string } {
-  // Full URL: https://github.com/owner/repo/pull/123 or https://enterprise.example.com/owner/repo/pull/123
-  const urlMatch = input.match(/^(https?:\/\/[^/]+)\/([^/]+)\/([^/]+)\/pull\/(\d+)/);
-  if (urlMatch) {
-    const baseUrl = urlMatch[1];
-    // Only github.com (exactly) is public GitHub, everything else is enterprise
-    const isEnterprise = !baseUrl.match(/^https?:\/\/(www\.)?github\.com$/i);
-    return {
-      owner: urlMatch[2],
-      repo: urlMatch[3],
-      number: parseInt(urlMatch[4], 10),
-      githubUrl: isEnterprise ? baseUrl : undefined,
-    };
-  }
-
-  // Short reference: owner/repo#123
-  const shortMatch = input.match(/^([^/]+)\/([^#]+)#(\d+)$/);
-  if (shortMatch) {
-    return {
-      owner: shortMatch[1],
-      repo: shortMatch[2],
-      number: parseInt(shortMatch[3], 10),
-    };
-  }
-
-  throw new Error(
-    'Invalid PR reference. Use:\n' +
-    '  - https://github.com/owner/repo/pull/123\n' +
-    '  - owner/repo#123'
-  );
-}

package/src/index.ts

@@ -1,6 +0,0 @@
-export { GitHubClient, parseFiles, parsePRReference } from './github.js';
-export { analyzeWithCodex } from './codex.js';
-export { analyzeWithClaude } from './claude.js';
-export { issuesToReviewComments, generateSummary } from './analyzer.js';
-export { cloneRepo, getGitDiff, getChangedFiles } from './repo.js';
-export type * from './types.js';

package/src/logs.ts

@@ -1,253 +0,0 @@
-import chalk from 'chalk';
-import ora, { type Ora } from 'ora';
-
-export interface CodexEvent {
-  type: string;
-  item?: {
-    id?: string;
-    type?: string;
-    text?: string;
-    command?: string;
-    aggregated_output?: string;
-    exit_code?: number | null;
-    status?: string;
-  };
-  thread_id?: string;
-}
-
-export class LogProcessor {
-  private seenFiles = new Set<string>();
-  private currentPhase = '';
-  private currentSpinner: Ora | null = null;
-  private fileCount = 0;
-  private analyzedCount = 0;
-
-  processEvent(event: CodexEvent): void {
-    switch (event.type) {
-      case 'thread.started':
-        this.startSpinner('Starting analysis...');
-        break;
-
-      case 'turn.started':
-        // Silent - just marks the start of a turn
-        break;
-
-      case 'item.completed':
-        this.handleItemCompleted(event);
-        break;
-
-      case 'item.started':
-        this.handleItemStarted(event);
-        break;
-
-      case 'turn.completed':
-        // Analysis complete - handled elsewhere
-        break;
-    }
-  }
-
-  private startSpinner(text: string): void {
-    this.stopSpinner();
-    this.currentSpinner = ora({
-      text: chalk.cyan(text),
-      prefixText: ' ',
-      spinner: 'dots'
-    }).start();
-  }
-
-  private succeedSpinner(text?: string): void {
-    if (this.currentSpinner) {
-      this.currentSpinner.succeed(text ? chalk.green(text) : undefined);
-      this.currentSpinner = null;
-    } else if (text) {
-      console.log(chalk.green(`  ✔ ${text}`));
-    }
-  }
-
-  private stopSpinner(): void {
-    if (this.currentSpinner) {
-      this.currentSpinner.stop();
-      this.currentSpinner = null;
-    }
-  }
-
-  private logAndSpin(text: string): void {
-    // First, succeed the current spinner to print it
-    if (this.currentSpinner) {
-      this.currentSpinner.succeed();
-    }
-    // Then start a new spinner with the new text
-    this.currentSpinner = ora({
-      text: chalk.cyan(text),
-      prefixText: ' ',
-      spinner: 'dots'
-    }).start();
-  }
-
-  private handleItemStarted(event: CodexEvent): void {
-    const item = event.item;
-    if (!item) return;
-
-    if (item.type === 'command_execution' && item.command) {
-      const cmd = item.command;
-      
-      // Git diff command
-      if (cmd.includes('git diff')) {
-        this.setPhase('diff');
-        this.logAndSpin('Checking differences...');
-      }
-    }
-  }
-
-  private handleItemCompleted(event: CodexEvent): void {
-    const item = event.item;
-    if (!item) return;
-
-    if (item.type === 'reasoning' && item.text) {
-      this.handleReasoning(item.text);
-    } else if (item.type === 'command_execution') {
-      this.handleCommandCompleted(item);
-    } else if (item.type === 'agent_message') {
-      this.succeedSpinner('Analysis complete');
-    }
-  }
-
-  private handleReasoning(text: string): void {
-    // Clean up the reasoning text - remove ** markers
-    const cleanText = text.replace(/\*\*/g, '').trim();
-    
-    if (!cleanText) return;
-
-    // Split into title (first line/sentence) and details
-    const lines = cleanText.split('\n').filter(Boolean);
-    
-    // Check if it's a short title or has additional content
-    if (lines.length === 1 && cleanText.length < 80) {
-      // Short reasoning - just show as spinner
-      this.logAndSpin(`💭 ${cleanText}`);
-    } else {
-      // Long reasoning - extract title and show details as bullet points
-      let title = lines[0];
-      let details = '';
-      
-      // If single line but long, split at first semicolon or period after 40 chars
-      if (lines.length === 1) {
-        const splitMatch = cleanText.match(/^(.{30,80}?[;.])\s*(.+)$/);
-        if (splitMatch) {
-          title = splitMatch[1];
-          details = splitMatch[2];
-        }
-      } else {
-        details = lines.slice(1).join(' ');
-      }
-
-      // Show title with spinner
-      this.logAndSpin(`💭 ${title}`);
-      
-      // If there are details, print them as indented bullet points
-      if (details) {
-        // Succeed current spinner first
-        if (this.currentSpinner) {
-          this.currentSpinner.succeed();
-          this.currentSpinner = null;
-        }
-        // Print details indented
-        const detailLines = this.wrapText(details, 70);
-        detailLines.forEach(line => {
-          console.log(chalk.dim(`       ${line}`));
-        });
-      }
-    }
-  }
-
-  private wrapText(text: string, maxWidth: number): string[] {
-    const words = text.split(' ');
-    const lines: string[] = [];
-    let currentLine = '';
-
-    for (const word of words) {
-      if (currentLine.length + word.length + 1 <= maxWidth) {
-        currentLine += (currentLine ? ' ' : '') + word;
-      } else {
-        if (currentLine) lines.push(currentLine);
-        currentLine = word;
-      }
-    }
-    if (currentLine) lines.push(currentLine);
-    return lines;
-  }
-
-  private handleCommandCompleted(item: CodexEvent['item']): void {
-    if (!item) return;
-
-    const cmd = item.command || '';
-    const output = item.aggregated_output || '';
-
-    // Git diff completed - show changed files count
-    if (cmd.includes('git diff') && output) {
-      const diffFiles = output.match(/diff --git a\/([^\s]+)/g);
-      if (diffFiles && diffFiles.length > 0) {
-        this.fileCount = diffFiles.length;
-        this.succeedSpinner(`Found ${diffFiles.length} changed files`);
-      }
-    }
-
-    // File listing with rg --files
-    if (cmd.includes('rg --files') && output) {
-      const files = output.trim().split('\n').filter(Boolean);
-      if (files.length > 0) {
-        this.succeedSpinner(`Identified ${files.length} files to analyze`);
-      }
-    }
-
-    // File reading commands - extract and show file path
-    const fileMatch = this.extractFilePath(cmd);
-    if (fileMatch && !this.seenFiles.has(fileMatch)) {
-      this.seenFiles.add(fileMatch);
-      this.analyzedCount++;
-      this.logAndSpin(`📄 Analyzing ${fileMatch}`);
-    }
-  }
-
-  private extractFilePath(cmd: string): string | null {
-    // Skip git commands - they're not file reads
-    if (cmd.includes('git ')) return null;
-
-    // Match patterns like:
-    // sed -n '1,260p' path/to/file.tsx
-    // nl -ba path/to/file.ts
-    // cat path/to/file.ts
-    const patterns = [
-      /sed\s+-n\s+'[^']+'\s+([^\s|]+\.[a-z]{1,4})/i,
-      /nl\s+-ba\s+([^\s|]+\.[a-z]{1,4})/i,
-      /cat\s+([^\s|]+\.[a-z]{1,4})/i,
-    ];
-
-    for (const pattern of patterns) {
-      const match = cmd.match(pattern);
-      if (match && match[1]) {
-        // Skip if it's a glob pattern or doesn't look like a real file
-        if (match[1].includes('*')) continue;
-        // Must have a proper file extension (1-4 chars)
-        if (!/\.[a-z]{1,4}$/i.test(match[1])) continue;
-        return match[1];
-      }
-    }
-
-    return null;
-  }
-
-  private setPhase(phase: string): void {
-    this.currentPhase = phase;
-  }
-
-  reset(): void {
-    this.stopSpinner();
-    this.seenFiles.clear();
-    this.currentPhase = '';
-    this.fileCount = 0;
-    this.analyzedCount = 0;
-  }
-}
-
-export const logProcessor = new LogProcessor();

package/src/prompt.ts

@@ -1,75 +0,0 @@
-export function getReviewPrompt(baseBranch: string): string {
-  return `You are an expert code reviewer. Perform a comprehensive review of ALL code changes in this PR.
-
-STEP 1: Run "git diff -U10 ${baseBranch}...HEAD" to get all changed files with 10 lines of context.
-
-STEP 2: Analyze the diff directly (skip lock files like yarn.lock, package-lock.json, bun.lock):
-  - The diff provides sufficient context for most issues
-  - Only read the full file if you need more context to understand the change
-  - Focus on the actual changes (+ and - lines) and their immediate context
-
-STEP 3: Check each change for these categories:
-
-**BUGS & LOGIC ERRORS:**
-  - Off-by-one errors in loops/arrays
-  - Null/undefined access without checks
-  - Race conditions in async code
-  - Incorrect boolean logic or conditions
-  - Wrong variable usage or typos
-  - Infinite loops or recursion without exit
-  - Array mutation while iterating
-
-**ERROR HANDLING:**
-  - Missing try/catch for async operations
-  - Unhandled promise rejections
-  - Missing error states in UI
-  - Silent failures that should be logged
-
-**SECURITY:**
-  - SQL injection, XSS vulnerabilities
-  - Hardcoded secrets or credentials
-  - Unsafe user input handling
-  - Missing authentication/authorization checks
-
-**PERFORMANCE:**
-  - Unnecessary re-renders or computations
-  - Missing memoization where needed
-  - N+1 query patterns
-  - Memory leaks (event listeners, subscriptions not cleaned up)
-  - Large objects in state that should be normalized
-
-**TYPE SAFETY:**
-  - Any type usage that should be specific
-  - Missing null checks for optional values
-  - Type assertions that could fail at runtime
-
-**EDGE CASES:**
-  - Empty arrays/objects not handled
-  - Boundary conditions (0, negative, max values)
-  - Network failures not handled
-  - Loading/error states missing
-
-**CODE QUALITY:**
-  - Dead code or unused variables
-  - Duplicated logic that should be extracted
-  - Overly complex functions that should be split
-  - Missing cleanup in useEffect/subscriptions
-
-**TEST COVERAGE:**
-  - Check if tests are added for new functionality
-  - Check if existing tests need to be updated for changed code
-  - Identify critical logic that should have unit tests
-  - Flag functions with complex branching that need test coverage
-  - Check if edge cases in the code have corresponding test cases
-
-STEP 4: Output ALL issues as JSON (be thorough - report every issue you find):
-{"issues":[{"filename":"<path>","line":<num>,"category":"functionality"|"performance"|"readability","severity":"error"|"warning"|"info","message":"<description>","suggestion":"<fix>"}]}
-
-**SEVERITY GUIDELINES (be consistent):**
-- **error**: Will cause bugs, crashes, security vulnerabilities, or data loss. Examples: null pointer access, unhandled exceptions, SQL injection, race conditions, infinite loops.
-- **warning**: Potential problems that may cause issues in edge cases or under certain conditions. Examples: missing error handling, possible memory leaks, missing null checks for optional values, performance issues.
-- **info**: Code quality improvements, best practices, style suggestions. Examples: dead code, code duplication, missing tests, readability improvements.
-
-CRITICAL: Review EVERY changed file in the diff. Do not skip any. Be efficient - analyze the diff context first, only read full files when truly needed.
-If truly no issues after thorough analysis, return {"issues":[]}`;
-}

package/src/repo.ts

@@ -1,81 +0,0 @@
-import { spawn } from 'child_process';
-import { mkdtemp, rm } from 'fs/promises';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import type { PRInfo } from './types.js';
-
-export interface ClonedRepo {
-  path: string;
-  cleanup: () => Promise<void>;
-}
-
-export async function cloneRepo(
-  prInfo: PRInfo,
-  token: string,
-  githubUrl?: string
-): Promise<ClonedRepo> {
-  const tempDir = await mkdtemp(join(tmpdir(), 'nayan-ai-'));
-  
-  const baseUrl = githubUrl?.replace(/\/$/, '') || 'https://github.com';
-  const cloneUrl = `https://${token}@${baseUrl.replace(/^https?:\/\//, '')}/${prInfo.owner}/${prInfo.repo}.git`;
-
-  // Clone with enough depth to have main branch history
-  await runGit(['clone', '--depth', '100', cloneUrl, tempDir]);
-  // Fetch the main branch explicitly for comparison
-  await runGit(['fetch', 'origin', 'main:refs/remotes/origin/main', '--depth', '100'], tempDir);
-  // Fetch the PR branch
-  await runGit(['fetch', 'origin', `pull/${prInfo.number}/head:pr-branch`], tempDir);
-  await runGit(['checkout', 'pr-branch'], tempDir);
-
-  return {
-    path: tempDir,
-    cleanup: async () => {
-      try {
-        await rm(tempDir, { recursive: true, force: true });
-      } catch {
-        // ignore cleanup errors
-      }
-    },
-  };
-}
-
-export async function getGitDiff(repoPath: string): Promise<string> {
-  return runGit(['diff', 'origin/main...HEAD', '--unified=3'], repoPath);
-}
-
-export async function getChangedFiles(repoPath: string): Promise<string[]> {
-  const output = await runGit(['diff', 'origin/main...HEAD', '--name-only'], repoPath);
-  return output.split('\n').filter(Boolean);
-}
-
-function runGit(args: string[], cwd?: string): Promise<string> {
-  return new Promise((resolve, reject) => {
-    const child = spawn('git', args, {
-      cwd,
-      stdio: ['pipe', 'pipe', 'pipe'],
-    });
-
-    let stdout = '';
-    let stderr = '';
-
-    child.stdout.on('data', (data) => {
-      stdout += data.toString();
-    });
-
-    child.stderr.on('data', (data) => {
-      stderr += data.toString();
-    });
-
-    child.on('close', (code) => {
-      if (code !== 0) {
-        reject(new Error(`git ${args[0]} failed: ${stderr || stdout || 'Unknown error'}`));
-        return;
-      }
-      resolve(stdout);
-    });
-
-    child.on('error', (err) => {
-      reject(err);
-    });
-  });
-}