diffity 0.1.0 → 0.1.2

package/packages/cli/src/agent.ts

@@ -1,187 +0,0 @@
-import type { Command } from 'commander';
-import pc from 'picocolors';
-import { isGitRepo } from '@diffity/git';
-import { getCurrentSession } from './session.js';
-import {
-  createThread,
-  getThreadsForSession,
-  getThread,
-  addReply,
-  updateThreadStatus,
-  type ThreadStatus,
-  type Thread,
-} from './threads.js';
-
-function requireSession() {
-  if (!isGitRepo()) {
-    console.error(pc.red('Error: Not a git repository'));
-    process.exit(1);
-  }
-
-  const session = getCurrentSession();
-  if (!session) {
-    console.error(pc.red('Error: No active review session.'));
-    console.error(pc.dim('Start diffity first to create a session.'));
-    process.exit(1);
-  }
-  return session;
-}
-
-function resolveThreadId(shortId: string, sessionId: string): Thread {
-  const thread = getThread(shortId);
-  if (!thread) {
-    console.error(pc.red(`Error: Thread not found: ${shortId}`));
-    process.exit(1);
-  }
-  if (thread.sessionId !== sessionId) {
-    console.error(pc.red(`Error: Thread ${shortId} does not belong to current session`));
-    process.exit(1);
-  }
-  return thread;
-}
-
-function formatThreadLine(thread: Thread): string {
-  const shortId = thread.id.slice(0, 8);
-  const isGeneral = thread.filePath === '__general__';
-  const statusColor = thread.status === 'open' ? pc.yellow : thread.status === 'resolved' ? pc.green : thread.status === 'dismissed' ? pc.dim : pc.cyan;
-  const statusLabel = statusColor(`[${thread.status}]`);
-  const firstComment = thread.comments[0]?.body || '';
-  const truncated = firstComment.length > 80 ? firstComment.slice(0, 77) + '...' : firstComment;
-
-  if (isGeneral) {
-    return `${statusLabel.padEnd(22)} ${pc.dim(shortId)}  ${pc.bold('General comment')}\n${''.padEnd(15)}${pc.dim('"')}${truncated}${pc.dim('"')}`;
-  }
-
-  const lineRange = thread.startLine === thread.endLine
-    ? `${thread.startLine}`
-    : `${thread.startLine}-${thread.endLine}`;
-  const location = `${thread.filePath}:${lineRange}`;
-  const sideLabel = thread.side === 'old' ? '(old)' : '(new)';
-
-  return `${statusLabel.padEnd(22)} ${pc.dim(shortId)}  ${location} ${pc.dim(sideLabel)}\n${''.padEnd(15)}${pc.dim('"')}${truncated}${pc.dim('"')}`;
-}
-
-export function registerAgentCommands(program: Command): void {
-  const agent = program
-    .command('agent')
-    .description('Agent commands for interacting with review comments')
-    .addHelpText('after', `
-Examples:
-  $ diffity agent list --status open --json
-  $ diffity agent comment --file src/app.ts --line 42 --body "Missing null check"
-  $ diffity agent resolve abc123 --summary "Added null check"
-  $ diffity agent reply abc123 --body "Good catch, fixed"
-  $ diffity agent general-comment --body "Overall this looks good, just a few nits"`);
-
-  agent
-    .command('list')
-    .description('List comment threads in the current session (use --json for full details)')
-    .option('--status <status>', 'Filter by status (open, resolved, dismissed)')
-    .option('--json', 'Output as JSON')
-    .action((opts) => {
-      const validStatuses = ['open', 'resolved', 'dismissed'];
-      if (opts.status && !validStatuses.includes(opts.status)) {
-        console.error(pc.red(`Error: Invalid status "${opts.status}". Must be one of: ${validStatuses.join(', ')}`));
-        process.exit(1);
-      }
-      const session = requireSession();
-      const threads = getThreadsForSession(session.id, opts.status as ThreadStatus | undefined);
-
-      if (opts.json) {
-        console.log(JSON.stringify(threads, null, 2));
-        return;
-      }
-
-      if (threads.length === 0) {
-        console.log(pc.dim('No threads found.'));
-        return;
-      }
-
-      for (const thread of threads) {
-        console.log(formatThreadLine(thread));
-      }
-    });
-
-  agent
-    .command('comment')
-    .description('Create a new comment thread')
-    .requiredOption('--file <path>', 'File path (relative to repo root)')
-    .requiredOption('--line <n>', 'Line number (1-indexed)', parseInt)
-    .option('--end-line <n>', 'End line for multi-line comments (1-indexed)', parseInt)
-    .option('--side <side>', 'Which side of the diff (new or old)', 'new')
-    .requiredOption('--body <text>', 'Comment body')
-    .action((opts) => {
-      if (opts.side !== 'new' && opts.side !== 'old') {
-        console.error(pc.red(`Error: Invalid side "${opts.side}". Must be "new" or "old"`));
-        process.exit(1);
-      }
-      const session = requireSession();
-      const endLine = opts.endLine ?? opts.line;
-      const thread = createThread(
-        session.id,
-        opts.file,
-        opts.side,
-        opts.line,
-        endLine,
-        opts.body,
-        { name: 'Agent', type: 'agent' },
-      );
-      console.log(pc.green(`Created thread ${thread.id.slice(0, 8)}`));
-    });
-
-  agent
-    .command('resolve')
-    .description('Resolve a thread (marks as fixed)')
-    .argument('<thread-id>', 'Thread ID (or 8-char prefix)')
-    .option('--summary <text>', 'What was done to resolve it')
-    .action((id: string, opts) => {
-      const session = requireSession();
-      const thread = resolveThreadId(id, session.id);
-      const author = opts.summary ? { name: 'Agent', type: 'agent' as const } : undefined;
-      updateThreadStatus(thread.id, 'resolved', opts.summary, author);
-      console.log(pc.green(`Resolved thread ${thread.id.slice(0, 8)}`));
-    });
-
-  agent
-    .command('dismiss')
-    .description('Dismiss a thread (marks as won\'t fix)')
-    .argument('<thread-id>', 'Thread ID (or 8-char prefix)')
-    .option('--reason <text>', 'Why the thread is being dismissed')
-    .action((id: string, opts) => {
-      const session = requireSession();
-      const thread = resolveThreadId(id, session.id);
-      const author = opts.reason ? { name: 'Agent', type: 'agent' as const } : undefined;
-      updateThreadStatus(thread.id, 'dismissed', opts.reason, author);
-      console.log(pc.green(`Dismissed thread ${thread.id.slice(0, 8)}`));
-    });
-
-  agent
-    .command('reply')
-    .description('Reply to a comment thread')
-    .argument('<thread-id>', 'Thread ID (or 8-char prefix)')
-    .requiredOption('--body <text>', 'Reply body')
-    .action((id: string, opts) => {
-      const session = requireSession();
-      const thread = resolveThreadId(id, session.id);
-      addReply(thread.id, opts.body, { name: 'Agent', type: 'agent' });
-      console.log(pc.green(`Replied to thread ${thread.id.slice(0, 8)}`));
-    });
-
-  agent
-    .command('general-comment')
-    .description('Create a general comment on the entire diff (not tied to a specific file or line)')
-    .requiredOption('--body <text>', 'Comment body')
-    .action((opts) => {
-      const session = requireSession();
-      const thread = createThread(
-        session.id,
-        '__general__',
-        'new',
-        0,
-        0,
-        opts.body,
-        { name: 'Agent', type: 'agent' },
-      );
-      console.log(pc.green(`Created general comment ${thread.id.slice(0, 8)}`));
-    });
-}

package/packages/cli/src/db.ts

@@ -1,58 +0,0 @@
-import Database from 'better-sqlite3';
-import { join } from 'node:path';
-import { getDiffityDir } from '@diffity/git';
-
-let db: Database.Database | null = null;
-
-export function getDb(): Database.Database {
-  if (db) {
-    return db;
-  }
-
-  const dbPath = join(getDiffityDir(), 'reviews.db');
-  db = new Database(dbPath);
-  db.pragma('journal_mode = WAL');
-  db.pragma('foreign_keys = ON');
-  migrateDb(db);
-  return db;
-}
-
-function migrateDb(db: Database.Database): void {
-  db.exec(`
-    CREATE TABLE IF NOT EXISTS review_sessions (
-      id TEXT PRIMARY KEY,
-      ref TEXT NOT NULL,
-      head_hash TEXT NOT NULL,
-      created_at TEXT NOT NULL DEFAULT (datetime('now'))
-    );
-
-    CREATE TABLE IF NOT EXISTS comment_threads (
-      id TEXT PRIMARY KEY,
-      session_id TEXT NOT NULL REFERENCES review_sessions(id),
-      file_path TEXT NOT NULL,
-      side TEXT NOT NULL,
-      start_line INTEGER NOT NULL,
-      end_line INTEGER NOT NULL,
-      status TEXT NOT NULL DEFAULT 'open',
-      anchor_content TEXT,
-      created_at TEXT NOT NULL DEFAULT (datetime('now')),
-      updated_at TEXT NOT NULL DEFAULT (datetime('now'))
-    );
-
-    CREATE TABLE IF NOT EXISTS comments (
-      id TEXT PRIMARY KEY,
-      thread_id TEXT NOT NULL REFERENCES comment_threads(id) ON DELETE CASCADE,
-      author_name TEXT NOT NULL,
-      author_type TEXT NOT NULL,
-      body TEXT NOT NULL,
-      created_at TEXT NOT NULL DEFAULT (datetime('now'))
-    );
-  `);
-}
-
-export function closeDb(): void {
-  if (db) {
-    db.close();
-    db = null;
-  }
-}

package/packages/cli/src/index.ts

@@ -1,196 +0,0 @@
-import { Command } from 'commander';
-import { execSync } from 'node:child_process';
-import { rmSync, existsSync } from 'node:fs';
-import { join } from 'node:path';
-import { homedir } from 'node:os';
-import { createRequire } from 'node:module';
-import open from 'open';
-import pc from 'picocolors';
-import { isGitRepo } from '@diffity/git';
-import { startServer } from './server.js';
-import { registerAgentCommands } from './agent.js';
-
-const require = createRequire(import.meta.url);
-const pkg = require('../package.json');
-
-const program = new Command();
-
-program
-  .name('diffity')
-  .description('GitHub-style git diff viewer in the browser')
-  .version(pkg.version)
-  .argument('[refs...]', 'Git refs to diff (e.g. HEAD~3, main, main..feature)')
-  .option('--port <port>', 'Port to use', '5391')
-  .option('--no-open', 'Do not open browser automatically')
-  .option('--quiet', 'Minimal terminal output')
-  .option('--dark', 'Open in dark mode (default: light)')
-  .option('--unified', 'Open in unified view (default: split)')
-  .addHelpText('after', `
-Examples:
-  $ diffity                    Working tree changes
-  $ diffity HEAD~1             Last commit vs working tree
-  $ diffity HEAD~3             Last 3 commits vs working tree
-  $ diffity abc1234            Changes since a specific commit
-  $ diffity main..feature      Compare branches
-  $ diffity main feature       Same as main..feature
-  $ diffity v1.0.0..v2.0.0    Compare tags`)
-  .action(async (refs: string[], opts) => {
-    if (!isGitRepo()) {
-      console.error(pc.red('Error: Not a git repository'));
-      process.exit(1);
-    }
-
-    const diffArgs: string[] = [];
-    let description = '';
-
-    if (refs.length === 1) {
-      const ref = refs[0];
-      if (ref.includes('..')) {
-        diffArgs.push(ref);
-        description = ref;
-      } else {
-        diffArgs.push(ref);
-        description = `Changes from ${ref}`;
-      }
-    } else if (refs.length === 2) {
-      diffArgs.push(`${refs[0]}..${refs[1]}`);
-      description = `${refs[0]}..${refs[1]}`;
-    } else {
-      description = 'Unstaged changes';
-    }
-
-    const port = parseInt(opts.port, 10);
-
-    let effectiveRef: string;
-    if (refs.length > 0) {
-      effectiveRef = refs.length === 2 ? `${refs[0]}..${refs[1]}` : refs[0];
-    } else {
-      effectiveRef = 'work';
-    }
-
-    try {
-      const { port: actualPort, close } = await startServer({ port, diffArgs, description, effectiveRef });
-      const urlParams = new URLSearchParams({ ref: effectiveRef });
-      if (opts.dark) {
-        urlParams.set('theme', 'dark');
-      }
-      if (opts.unified) {
-        urlParams.set('view', 'unified');
-      }
-      const url = `http://localhost:${actualPort}/?${urlParams.toString()}`;
-
-      if (!opts.quiet) {
-        console.log('');
-        console.log(pc.bold('  diffity'));
-        console.log(`  ${pc.dim(description)}`);
-        console.log('');
-        console.log(`  ${pc.green('→')} ${pc.cyan(url)}`);
-        console.log(`  ${pc.dim('Press Ctrl+C to stop')}`);
-        console.log('');
-      }
-
-      process.on('SIGINT', () => {
-        if (!opts.quiet) {
-          console.log(pc.dim('\n  Shutting down...'));
-        }
-        close();
-        process.exit(0);
-      });
-
-      process.on('SIGTERM', () => {
-        close();
-        process.exit(0);
-      });
-
-      if (opts.open !== false) {
-        await open(url);
-      }
-    } catch (err) {
-      console.error(pc.red(`Failed to start server: ${err}`));
-      process.exit(1);
-    }
-  });
-
-program
-  .command('prune')
-  .description('Remove all diffity data (database, sessions) for all repos')
-  .action(() => {
-    const dir = join(homedir(), '.diffity');
-    if (!existsSync(dir)) {
-      console.log(pc.dim('Nothing to prune.'));
-      return;
-    }
-
-    rmSync(dir, { recursive: true, force: true });
-    console.log(pc.green('Pruned all diffity data (~/.diffity).'));
-  });
-
-program
-  .command('update')
-  .description('Update diffity to the latest version')
-  .action(() => {
-    try {
-      const registry = execSync('npm view diffity version', { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] }).trim();
-      if (registry === pkg.version) {
-        console.log(pc.green(`Already on the latest version (${pkg.version}).`));
-        return;
-      }
-      console.log(`${pc.dim(`Current: ${pkg.version}`)} → ${pc.bold(registry)}`);
-      console.log(pc.dim('Updating...'));
-      execSync('npm install -g diffity@latest', { stdio: 'inherit' });
-      console.log(pc.green(`Updated to ${registry}.`));
-    } catch {
-      console.error(pc.red('Failed to update. Try running: npm install -g diffity@latest'));
-      process.exit(1);
-    }
-  });
-
-program
-  .command('doctor')
-  .description('Check that diffity can run correctly')
-  .action(() => {
-    let ok = true;
-
-    process.stdout.write('  git          ');
-    try {
-      const gitVersion = execSync('git --version', { encoding: 'utf-8', stdio: ['pipe', 'pipe', 'pipe'] }).trim();
-      console.log(pc.green(`✓ ${gitVersion}`));
-    } catch {
-      console.log(pc.red('✗ git not found'));
-      ok = false;
-    }
-
-    process.stdout.write('  git repo     ');
-    if (isGitRepo()) {
-      console.log(pc.green('✓ inside a git repository'));
-    } else {
-      console.log(pc.yellow('- not inside a git repository'));
-    }
-
-    process.stdout.write('  node         ');
-    console.log(pc.green(`✓ ${process.version}`));
-
-    process.stdout.write('  sqlite       ');
-    try {
-      require('better-sqlite3');
-      console.log(pc.green('✓ better-sqlite3 loaded'));
-    } catch {
-      console.log(pc.red('✗ better-sqlite3 failed to load (native module issue)'));
-      ok = false;
-    }
-
-    process.stdout.write('  version      ');
-    console.log(pc.green(`✓ diffity ${pkg.version}`));
-
-    console.log('');
-    if (ok) {
-      console.log(pc.green('  All checks passed.'));
-    } else {
-      console.log(pc.red('  Some checks failed. Fix the issues above and try again.'));
-      process.exit(1);
-    }
-  });
-
-registerAgentCommands(program);
-
-program.parse();

package/packages/cli/src/review-routes.ts

@@ -1,150 +0,0 @@
-import type { IncomingMessage, ServerResponse } from 'node:http';
-import {
-  createThread,
-  getThreadsForSession,
-  addReply,
-  updateThreadStatus,
-  deleteThread,
-  deleteAllThreadsForSession,
-  deleteComment,
-  type ThreadStatus,
-} from './threads.js';
-import { getCurrentSession } from './session.js';
-
-function sendJson(res: ServerResponse, data: unknown) {
-  res.writeHead(200, { 'Content-Type': 'application/json' });
-  res.end(JSON.stringify(data));
-}
-
-function sendError(res: ServerResponse, status: number, message: string) {
-  res.writeHead(status, { 'Content-Type': 'application/json' });
-  res.end(JSON.stringify({ error: message }));
-}
-
-function readBody(req: IncomingMessage): Promise<string> {
-  return new Promise((resolve, reject) => {
-    const chunks: Buffer[] = [];
-    req.on('data', (chunk: Buffer) => chunks.push(chunk));
-    req.on('end', () => resolve(Buffer.concat(chunks).toString()));
-    req.on('error', reject);
-  });
-}
-
-export function handleReviewRoute(req: IncomingMessage, res: ServerResponse, pathname: string, url: URL): boolean {
-  if (pathname === '/api/sessions/current' && req.method === 'GET') {
-    const session = getCurrentSession();
-    sendJson(res, session);
-    return true;
-  }
-
-  if (pathname === '/api/threads' && req.method === 'GET') {
-    const sid = url.searchParams.get('session');
-    if (!sid) {
-      sendError(res, 400, 'Missing session parameter');
-      return true;
-    }
-    const status = url.searchParams.get('status') as ThreadStatus | null;
-    const threads = getThreadsForSession(sid, status || undefined);
-    sendJson(res, threads);
-    return true;
-  }
-
-  if (pathname === '/api/threads' && req.method === 'DELETE') {
-    readBody(req).then((raw) => {
-      try {
-        const body = JSON.parse(raw);
-        const { sessionId: sid } = body;
-        if (!sid) {
-          sendError(res, 400, 'Missing sessionId');
-          return;
-        }
-        deleteAllThreadsForSession(sid);
-        sendJson(res, { ok: true });
-      } catch (err) {
-        sendError(res, 500, `Failed to delete all threads: ${err}`);
-      }
-    });
-    return true;
-  }
-
-  if (pathname === '/api/threads' && req.method === 'POST') {
-    readBody(req).then((raw) => {
-      try {
-        const body = JSON.parse(raw);
-        const { sessionId: sid, filePath, side, startLine, endLine, body: commentBody, author, anchorContent } = body;
-        if (!sid || !filePath || !side || typeof startLine !== 'number' || typeof endLine !== 'number' || !commentBody || !author) {
-          sendError(res, 400, 'Missing required fields');
-          return;
-        }
-        const thread = createThread(sid, filePath, side, startLine, endLine, commentBody, author, anchorContent);
-        sendJson(res, thread);
-      } catch (err) {
-        sendError(res, 500, `Failed to create thread: ${err}`);
-      }
-    });
-    return true;
-  }
-
-  const threadReplyMatch = pathname.match(/^\/api\/threads\/([^/]+)\/reply$/);
-  if (threadReplyMatch && req.method === 'POST') {
-    readBody(req).then((raw) => {
-      try {
-        const body = JSON.parse(raw);
-        const { body: commentBody, author } = body;
-        if (!commentBody || !author) {
-          sendError(res, 400, 'Missing body or author');
-          return;
-        }
-        const comment = addReply(threadReplyMatch[1], commentBody, author);
-        sendJson(res, comment);
-      } catch (err) {
-        sendError(res, 500, `Failed to add reply: ${err}`);
-      }
-    });
-    return true;
-  }
-
-  const threadStatusMatch = pathname.match(/^\/api\/threads\/([^/]+)\/status$/);
-  if (threadStatusMatch && req.method === 'PATCH') {
-    readBody(req).then((raw) => {
-      try {
-        const body = JSON.parse(raw);
-        const { status, summary } = body;
-        if (!status) {
-          sendError(res, 400, 'Missing status');
-          return;
-        }
-        const summaryAuthor = summary ? { name: 'System', type: 'user' as const } : undefined;
-        updateThreadStatus(threadStatusMatch[1], status, summary, summaryAuthor);
-        sendJson(res, { ok: true });
-      } catch (err) {
-        sendError(res, 500, `Failed to update thread status: ${err}`);
-      }
-    });
-    return true;
-  }
-
-  const threadDeleteMatch = pathname.match(/^\/api\/threads\/([^/]+)$/);
-  if (threadDeleteMatch && req.method === 'DELETE') {
-    try {
-      deleteThread(threadDeleteMatch[1]);
-      sendJson(res, { ok: true });
-    } catch (err) {
-      sendError(res, 500, `Failed to delete thread: ${err}`);
-    }
-    return true;
-  }
-
-  const commentDeleteMatch = pathname.match(/^\/api\/comments\/([^/]+)$/);
-  if (commentDeleteMatch && req.method === 'DELETE') {
-    try {
-      deleteComment(commentDeleteMatch[1]);
-      sendJson(res, { ok: true });
-    } catch (err) {
-      sendError(res, 500, `Failed to delete comment: ${err}`);
-    }
-    return true;
-  }
-
-  return false;
-}