docrev 0.11.2 → 0.12.0

package/lib/commands/merge-resolve.ts

@@ -1,378 +0,0 @@
-/**
- * MERGE, CONFLICTS, and MERGE-RESOLVE commands
- *
- * Commands for three-way merge of reviewer feedback and conflict resolution.
- * Split from sections.ts for maintainability.
- */
-
-import {
-  chalk,
-  fs,
-  path,
-  fmt,
-} from './context.js';
-import type { Command } from 'commander';
-import * as readline from 'readline';
-
-interface MergeOptions {
-  base?: string;
-  output?: string;
-  names?: string;
-  strategy: string;
-  diffLevel: 'sentence' | 'word';
-  dryRun?: boolean;
-  sections?: boolean;
-}
-
-/**
- * Register merge, conflicts, and merge-resolve commands with the program
- */
-export function register(program: Command): void {
-  // ==========================================================================
-  // MERGE command - Combine feedback from multiple reviewers (three-way merge)
-  // ==========================================================================
-
-  program
-    .command('merge')
-    .description('Merge feedback from multiple Word documents using three-way merge')
-    .argument('<docx...>', 'Word documents from reviewers')
-    .option('-b, --base <file>', 'Base document (original sent to reviewers). Auto-detected if not specified.')
-    .option('-o, --output <file>', 'Output file (default: writes to section files)')
-    .option('--names <names>', 'Reviewer names (comma-separated, in order of docx files)')
-    .option('--strategy <strategy>', 'Conflict resolution: first, latest, or interactive (default)', 'interactive')
-    .option('--diff-level <level>', 'Diff granularity: sentence or word (default: sentence)', 'sentence')
-    .option('--dry-run', 'Show conflicts without writing')
-    .option('--sections', 'Split merged output back to section files')
-    .action(async (docxFiles: string[], options: MergeOptions) => {
-      const {
-        mergeThreeWay,
-        formatConflict,
-        resolveConflict,
-        getBaseDocument,
-        checkBaseMatch,
-        saveConflicts,
-      } = await import('../merge.js');
-
-      // Validate reviewer files exist
-      for (const docx of docxFiles) {
-        if (!fs.existsSync(docx)) {
-          console.error(fmt.status('error', `Reviewer file not found: ${docx}`));
-          process.exit(1);
-        }
-      }
-
-      // Determine base document
-      let basePath = options.base;
-      let baseSource = 'specified';
-
-      if (!basePath) {
-        // Try to use .rev/base.docx
-        const projectDir = process.cwd();
-        basePath = getBaseDocument(projectDir) ?? undefined;
-
-        if (basePath) {
-          baseSource = 'auto (.rev/base.docx)';
-        } else {
-          console.log(chalk.yellow('\n  No base document found in .rev/base.docx'));
-          console.log(chalk.dim('  Tip: Run "rev build docx" to automatically save the base document.\n'));
-          console.error(fmt.status('error', 'Base document required. Use --base <file.docx>'));
-          process.exit(1);
-        }
-      }
-
-      if (!fs.existsSync(basePath)) {
-        console.error(fmt.status('error', `Base document not found: ${basePath}`));
-        process.exit(1);
-      }
-
-      // Check similarity between base and reviewer docs
-      const { matches, similarity } = await checkBaseMatch(basePath, docxFiles[0]);
-      if (!matches) {
-        console.log(chalk.yellow(`\n  Warning: Base document may not match reviewer file (${Math.round(similarity * 100)}% similar)`));
-        console.log(chalk.dim('  If this is wrong, use --base to specify the correct original document.\n'));
-      }
-
-      // Parse reviewer names
-      const names = options.names
-        ? options.names.split(',').map(n => n.trim())
-        : docxFiles.map((f, i) => {
-          // Try to extract name from filename (e.g., paper_reviewer_A.docx)
-          const basename = path.basename(f, '.docx');
-          const match = basename.match(/_([A-Za-z]+)$/);
-          return match ? match[1] : `Reviewer ${i + 1}`;
-        });
-
-      // Pad names if needed
-      while (names.length < docxFiles.length) {
-        names.push(`Reviewer ${names.length + 1}`);
-      }
-
-      const reviewerDocs = docxFiles.map((p, i) => ({
-        path: p,
-        name: names[i],
-      }));
-
-      console.log(fmt.header('Three-Way Merge'));
-      console.log();
-      console.log(chalk.dim(`  Base: ${path.basename(basePath)} (${baseSource})`));
-      console.log(chalk.dim(`  Reviewers: ${names.join(', ')}`));
-      console.log(chalk.dim(`  Diff level: ${options.diffLevel}`));
-      console.log();
-
-      const spin = fmt.spinner('Analyzing changes...').start();
-
-      try {
-        const { merged, conflicts, stats, baseText } = await mergeThreeWay(basePath, reviewerDocs, {
-          diffLevel: options.diffLevel,
-        });
-
-        spin.stop();
-
-        // Display stats
-        console.log(fmt.table(['Metric', 'Count'], [
-          ['Total changes', stats.totalChanges.toString()],
-          ['Non-conflicting', stats.nonConflicting.toString()],
-          ['Conflicts', stats.conflicts.toString()],
-          ['Comments', stats.comments.toString()],
-        ]));
-        console.log();
-
-        let finalMerged = merged;
-
-        // Handle conflicts
-        if (conflicts.length > 0) {
-          console.log(chalk.yellow(`Found ${conflicts.length} conflict(s):\n`));
-
-          if (options.strategy === 'first') {
-            // Auto-resolve: take first reviewer's change
-            for (const conflict of conflicts) {
-              console.log(chalk.dim(`  Conflict ${conflict.id}: using ${conflict.changes[0].reviewer}'s change`));
-              resolveConflict(conflict, 0);
-            }
-          } else if (options.strategy === 'latest') {
-            // Auto-resolve: take last reviewer's change
-            for (const conflict of conflicts) {
-              const lastIdx = conflict.changes.length - 1;
-              console.log(chalk.dim(`  Conflict ${conflict.id}: using ${conflict.changes[lastIdx].reviewer}'s change`));
-              resolveConflict(conflict, lastIdx);
-            }
-          } else if (!options.dryRun) {
-            // Interactive resolution
-            for (let i = 0; i < conflicts.length; i++) {
-              const conflict = conflicts[i];
-              console.log(chalk.bold(`\nConflict ${i + 1}/${conflicts.length} (${conflict.id}):`));
-              console.log(formatConflict(conflict, baseText));
-              console.log();
-
-              const rl = readline.createInterface({
-                input: process.stdin,
-                output: process.stdout,
-              });
-
-              const answer = await new Promise<string>((resolve) =>
-                rl.question(chalk.cyan(`  Choose (1-${conflict.changes.length}, s=skip): `), resolve)
-              );
-              rl.close();
-
-              if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
-                const choice = parseInt(answer) - 1;
-                if (choice >= 0 && choice < conflict.changes.length) {
-                  resolveConflict(conflict, choice);
-                  console.log(chalk.green(`  ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
-                }
-              } else {
-                console.log(chalk.dim('  Skipped (will need manual resolution)'));
-              }
-            }
-          }
-
-          // Save unresolved conflicts for later
-          const unresolved = conflicts.filter(c => c.resolved === null);
-          if (unresolved.length > 0) {
-            saveConflicts(process.cwd(), conflicts, basePath);
-            console.log(chalk.yellow(`\n  ${unresolved.length} unresolved conflict(s) saved to .rev/conflicts.json`));
-            console.log(chalk.dim('  Run "rev conflicts" to view, "rev merge-resolve" to resolve'));
-          }
-        }
-
-        // Write output
-        if (!options.dryRun) {
-          if (options.output) {
-            // Write to single file
-            fs.writeFileSync(options.output, finalMerged, 'utf-8');
-            console.log(fmt.status('success', `Merged output written to ${options.output}`));
-          } else if (options.sections) {
-            // Split to section files (TODO: implement section splitting)
-            console.log(chalk.yellow('  Section splitting not yet implemented'));
-            console.log(chalk.dim('  Use -o to specify output file'));
-          } else {
-            // Default: write to merged.md
-            const outPath = 'merged.md';
-            fs.writeFileSync(outPath, finalMerged, 'utf-8');
-            console.log(fmt.status('success', `Merged output written to ${outPath}`));
-          }
-
-          console.log();
-          console.log(chalk.dim('Next steps:'));
-          console.log(chalk.dim('  1. rev review merged.md     - Accept/reject changes'));
-          console.log(chalk.dim('  2. rev comments merged.md   - Address comments'));
-          if (conflicts.some(c => c.resolved === null)) {
-            console.log(chalk.dim('  3. rev merge-resolve        - Resolve remaining conflicts'));
-          }
-        } else {
-          console.log(fmt.status('info', 'Dry run - no output written'));
-        }
-      } catch (err) {
-        spin.stop();
-        const error = err as Error;
-        console.error(fmt.status('error', error.message));
-        if (process.env.DEBUG) console.error(error.stack);
-        process.exit(1);
-      }
-    });
-
-  // ==========================================================================
-  // CONFLICTS command - List unresolved conflicts
-  // ==========================================================================
-
-  program
-    .command('conflicts')
-    .description('List unresolved merge conflicts')
-    .action(async () => {
-      const { loadConflicts, formatConflict } = await import('../merge.js');
-      const projectDir = process.cwd();
-      const data = loadConflicts(projectDir);
-
-      if (!data) {
-        console.log(fmt.status('info', 'No conflicts file found'));
-        return;
-      }
-
-      const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
-
-      if (unresolved.length === 0) {
-        console.log(fmt.status('success', 'All conflicts resolved!'));
-        return;
-      }
-
-      console.log(fmt.header(`Unresolved Conflicts (${unresolved.length})`));
-      console.log();
-      console.log(chalk.dim(`  Base: ${data.base}`));
-      console.log(chalk.dim(`  Merged: ${data.merged}`));
-      console.log();
-
-      for (const conflict of unresolved) {
-        console.log(chalk.bold(`Conflict ${conflict.id}:`));
-        // Show abbreviated info
-        console.log(chalk.dim(`  Original: "${conflict.original.slice(0, 50)}${conflict.original.length > 50 ? '...' : ''}"`));
-        console.log(chalk.dim(`  Options: ${conflict.changes.map((c: any) => c.reviewer).join(', ')}`));
-        console.log();
-      }
-
-      console.log(chalk.dim('Run "rev merge-resolve" to resolve conflicts interactively'));
-    });
-
-  // ==========================================================================
-  // MERGE-RESOLVE command - Interactively resolve merge conflicts
-  // ==========================================================================
-
-  program
-    .command('merge-resolve')
-    .alias('mresolve')
-    .description('Resolve merge conflicts interactively')
-    .option('--theirs', 'Accept all changes from last reviewer')
-    .option('--ours', 'Accept all changes from first reviewer')
-    .action(async (options: { theirs?: boolean; ours?: boolean }) => {
-      const { loadConflicts, saveConflicts, clearConflicts, resolveConflict, formatConflict } = await import('../merge.js');
-      const projectDir = process.cwd();
-      const data = loadConflicts(projectDir);
-
-      if (!data) {
-        console.log(fmt.status('info', 'No conflicts to resolve'));
-        return;
-      }
-
-      const unresolved = data.conflicts.filter((c: any) => c.resolved === null);
-
-      if (unresolved.length === 0) {
-        console.log(fmt.status('success', 'All conflicts already resolved!'));
-        clearConflicts(projectDir);
-        return;
-      }
-
-      console.log(fmt.header(`Resolving ${unresolved.length} Conflict(s)`));
-      console.log();
-
-      if (options.theirs) {
-        // Accept all from last reviewer
-        for (const conflict of unresolved) {
-          const lastIdx = conflict.changes.length - 1;
-          resolveConflict(conflict, lastIdx);
-          console.log(chalk.dim(`  ${conflict.id}: accepted ${conflict.changes[lastIdx].reviewer}'s change`));
-        }
-        saveConflicts(projectDir, data.conflicts, data.base);
-        console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--theirs)`));
-      } else if (options.ours) {
-        // Accept all from first reviewer
-        for (const conflict of unresolved) {
-          resolveConflict(conflict, 0);
-          console.log(chalk.dim(`  ${conflict.id}: accepted ${conflict.changes[0].reviewer}'s change`));
-        }
-        saveConflicts(projectDir, data.conflicts, data.base);
-        console.log(fmt.status('success', `Resolved ${unresolved.length} conflicts (--ours)`));
-      } else {
-        // Interactive resolution
-        // Read base text for context display
-        let baseText = '';
-        try {
-          const { extractFromWord } = await import('../import.js');
-          const { text } = await extractFromWord(data.base);
-          baseText = text;
-        } catch {
-          // Can't read base, show without context
-        }
-
-        for (let i = 0; i < unresolved.length; i++) {
-          const conflict = unresolved[i];
-          console.log(chalk.bold(`\nConflict ${i + 1}/${unresolved.length} (${conflict.id}):`));
-          console.log(formatConflict(conflict, baseText));
-          console.log();
-
-          const rl = readline.createInterface({
-            input: process.stdin,
-            output: process.stdout,
-          });
-
-          const answer = await new Promise<string>((resolve) =>
-            rl.question(chalk.cyan(`  Choose (1-${conflict.changes.length}, s=skip, q=quit): `), resolve)
-          );
-          rl.close();
-
-          if (answer.toLowerCase() === 'q') {
-            console.log(chalk.dim('\n  Saving progress...'));
-            break;
-          }
-
-          if (answer.toLowerCase() !== 's' && !isNaN(parseInt(answer))) {
-            const choice = parseInt(answer) - 1;
-            if (choice >= 0 && choice < conflict.changes.length) {
-              resolveConflict(conflict, choice);
-              console.log(chalk.green(`  ✓ Applied: ${conflict.changes[choice].reviewer}'s change`));
-            }
-          } else {
-            console.log(chalk.dim('  Skipped'));
-          }
-        }
-
-        saveConflicts(projectDir, data.conflicts, data.base);
-
-        const remaining = data.conflicts.filter((c: any) => c.resolved === null).length;
-        if (remaining === 0) {
-          console.log(fmt.status('success', '\nAll conflicts resolved!'));
-          clearConflicts(projectDir);
-        } else {
-          console.log(chalk.yellow(`\n  ${remaining} conflict(s) remaining`));
-        }
-      }
-    });
-}

package/lib/commands/preview.ts

@@ -1,178 +0,0 @@
-/**
- * Preview commands: preview, watch
- *
- * Commands for building and viewing documents with live reload.
- */
-
-import type { Command } from 'commander';
-import {
-  chalk,
-  fs,
-  path,
-  fmt,
-  loadBuildConfig,
-} from './context.js';
-
-// Use the actual BuildConfig from build.ts which allows string|Author[]
-type BuildConfig = ReturnType<typeof loadBuildConfig>;
-
-// Options interfaces
-interface PreviewOptions {
-  // No options currently
-}
-
-interface WatchOptions {
-  open?: boolean;
-}
-
-/**
- * Register preview commands with the program
- */
-export function register(program: Command): void {
-  // ==========================================================================
-  // PREVIEW command - Build and open document
-  // ==========================================================================
-
-  program
-    .command('preview')
-    .description('Build and open document in default app')
-    .argument('[format]', 'Format to preview: pdf, docx', 'pdf')
-    .action(async (format: string, _options: PreviewOptions) => {
-      const { exec } = await import('child_process');
-      const { build } = await import('../build.js');
-
-      let config: Partial<BuildConfig> = {};
-      try {
-        config = loadBuildConfig('.') || {};
-      } catch (err) {
-        console.error(chalk.red('Not in a rev project directory (no rev.yaml found)'));
-        process.exit(1);
-      }
-
-      console.log(chalk.dim(`Building ${format}...`));
-      const result = await build('.', [format]);
-
-      const buildResult = result.results.find(r => r.format === format);
-      if (!buildResult?.success) {
-        const errorMsg = buildResult?.error || 'Unknown error';
-        console.error(chalk.red(`Build failed: ${errorMsg}`));
-        process.exit(1);
-      }
-
-      const outputFile = buildResult.outputPath;
-      if (!outputFile || !fs.existsSync(outputFile)) {
-        console.error(chalk.red(`Output file not found: ${outputFile}`));
-        process.exit(1);
-      }
-
-      // Open with system default
-      const openCmd = process.platform === 'darwin' ? 'open' :
-                      process.platform === 'win32' ? 'start' : 'xdg-open';
-
-      exec(`${openCmd} "${outputFile}"`, (err) => {
-        if (err) {
-          console.error(chalk.red(`Could not open file: ${err.message}`));
-        } else {
-          console.log(fmt.status('success', `Opened ${outputFile}`));
-        }
-      });
-    });
-
-  // ==========================================================================
-  // WATCH command - Auto-rebuild on changes
-  // ==========================================================================
-
-  program
-    .command('watch')
-    .description('Watch files and auto-rebuild on changes')
-    .argument('[format]', 'Format to build: pdf, docx, all', 'pdf')
-    .option('--no-open', 'Do not open after first build')
-    .action(async (format: string, options: WatchOptions) => {
-      const { exec } = await import('child_process');
-      const { build } = await import('../build.js');
-
-      let config: Partial<BuildConfig> = {};
-      try {
-        config = loadBuildConfig('.') || {};
-      } catch (err) {
-        console.error(chalk.red('Not in a rev project directory (no rev.yaml found)'));
-        process.exit(1);
-      }
-      let sections = config.sections || [];
-
-      if (sections.length === 0) {
-        sections = fs.readdirSync('.').filter(f =>
-          f.endsWith('.md') && !['README.md', 'CLAUDE.md', 'paper.md'].includes(f)
-        );
-      }
-
-      const filesToWatch = [
-        ...sections,
-        'rev.yaml',
-        config.bibliography || 'references.bib'
-      ].filter(f => fs.existsSync(f));
-
-      console.log(fmt.header('Watch Mode'));
-      console.log(chalk.dim(`Watching: ${filesToWatch.join(', ')}`));
-      console.log(chalk.dim('Press Ctrl+C to stop\n'));
-
-      let building = false;
-      let pendingBuild = false;
-
-      const doBuild = async (): Promise<void> => {
-        if (building) {
-          pendingBuild = true;
-          return;
-        }
-
-        building = true;
-        console.log(chalk.dim(`\n[${new Date().toLocaleTimeString()}] Rebuilding...`));
-
-        try {
-          const formats = format === 'all' ? ['pdf', 'docx'] : [format];
-          const result = await build('.', formats);
-
-          for (const r of result.results) {
-            if (r.success) {
-              console.log(chalk.green(`  ✓ ${r.format}: ${r.outputPath}`));
-            } else {
-              console.log(chalk.red(`  ✗ ${r.format}: ${r.error}`));
-            }
-          }
-        } catch (err) {
-          console.error(chalk.red(`  Build error: ${(err as Error).message}`));
-        }
-
-        building = false;
-        if (pendingBuild) {
-          pendingBuild = false;
-          doBuild();
-        }
-      };
-
-      // Initial build
-      await doBuild();
-
-      // Open after first build
-      if (options.open) {
-        const outputFile = format === 'docx' ?
-          (config.title?.replace(/[^a-zA-Z0-9]/g, '-').toLowerCase() || 'paper') + '.docx' :
-          (config.title?.replace(/[^a-zA-Z0-9]/g, '-').toLowerCase() || 'paper') + '.pdf';
-
-        if (fs.existsSync(outputFile)) {
-          const openCmd = process.platform === 'darwin' ? 'open' :
-                          process.platform === 'win32' ? 'start' : 'xdg-open';
-          exec(`${openCmd} "${outputFile}"`);
-        }
-      }
-
-      // Watch files
-      for (const file of filesToWatch) {
-        fs.watch(file, { persistent: true }, (eventType) => {
-          if (eventType === 'change') {
-            doBuild();
-          }
-        });
-      }
-    });
-}