docrev 0.11.3 → 0.12.0

package/lib/import.ts

@@ -1,932 +0,0 @@
-/**
- * Import functionality - convert Word docs to annotated Markdown
- *
- * Orchestration workflows + re-exports from extraction/diff/restore modules
- */
-
-import * as fs from 'fs';
-import * as path from 'path';
-import { stripAnnotations } from './annotations.js';
-import { readImageRegistry } from './image-registry.js';
-import { exec } from 'child_process';
-import { promisify } from 'util';
-
-// Import from split modules
-import {
-  extractFromWord,
-  extractWordComments,
-  extractCommentAnchors,
-  extractWordTables,
-} from './word-extraction.js';
-import type {
-  WordComment,
-  CommentAnchorData,
-  WordTable,
-  ExtractFromWordResult,
-} from './word-extraction.js';
-import {
-  generateSmartDiff,
-  generateAnnotatedDiff,
-  cleanupAnnotations,
-  fixCitationAnnotations,
-} from './diff-engine.js';
-import {
-  restoreCrossrefFromWord,
-  restoreImagesFromRegistry,
-  parseVisibleComments,
-  convertVisibleComments,
-} from './restore-references.js';
-import { findAnchorInText, findAllOccurrences } from './anchor-match.js';
-
-/**
- * Pick the best position from candidate `occurrences` given the
- * surrounding `before` / `after` context from the docx, while
- * respecting `usedPositions` to avoid stacking distinct comments at
- * the same anchor instance.
- *
- * Returns the chosen position, or -1 if every candidate is already used.
- */
-function pickBestOccurrence(
-  occurrences: number[],
-  result: string,
-  before: string,
-  after: string,
-  anchorLen: number,
-  usedPositions: Set<number>,
-  nearPos?: number,
-): number {
-  if (occurrences.length === 0) return -1;
-  if (occurrences.length === 1) {
-    return usedPositions.has(occurrences[0]) ? -1 : occurrences[0];
-  }
-
-  let bestIdx = occurrences.find(p => !usedPositions.has(p)) ?? -1;
-  if (bestIdx < 0) return -1;
-  let bestScore = -1;
-
-  // Tie-break by context score first; when scores tie, prefer the occurrence
-  // nearest `nearPos` (a section-position estimate) if given, else the
-  // leftmost. The estimate only orders equally-scored candidates — it is never
-  // the returned position.
-  const closer = (pos: number, incumbent: number) =>
-    nearPos === undefined ? pos < incumbent : Math.abs(pos - nearPos) < Math.abs(incumbent - nearPos);
-
-  for (const pos of occurrences) {
-    if (usedPositions.has(pos)) continue;
-    let score = 0;
-
-    if (before) {
-      const contextBefore = result.slice(Math.max(0, pos - before.length - 20), pos).toLowerCase();
-      const beforeLower = before.toLowerCase();
-      const beforeWords = beforeLower.split(/\s+/).filter(w => w.length > 3);
-      for (const word of beforeWords) {
-        if (contextBefore.includes(word)) score += 2;
-      }
-      if (contextBefore.includes(beforeLower.slice(-30))) score += 5;
-    }
-
-    if (after) {
-      const contextAfter = result.slice(pos + anchorLen, pos + anchorLen + after.length + 20).toLowerCase();
-      const afterLower = after.toLowerCase();
-      const afterWords = afterLower.split(/\s+/).filter(w => w.length > 3);
-      for (const word of afterWords) {
-        if (contextAfter.includes(word)) score += 2;
-      }
-      if (contextAfter.includes(afterLower.slice(0, 30))) score += 5;
-    }
-
-    if (score > bestScore || (score === bestScore && closer(pos, bestIdx))) {
-      bestScore = score;
-      bestIdx = pos;
-    }
-  }
-
-  return bestIdx;
-}
-
-// Re-export everything so existing imports from './import.js' still work
-export {
-  extractFromWord,
-  extractWordComments,
-  extractCommentAnchors,
-  extractHeadings,
-  extractWordTables,
-} from './word-extraction.js';
-export type {
-  WordComment,
-  TextNode,
-  CommentAnchorData,
-  CommentAnchorsResult,
-  DocxHeading,
-  WordTable,
-  ParsedRow,
-  ExtractFromWordOptions,
-  ExtractMessage,
-  ExtractFromWordResult,
-} from './word-extraction.js';
-
-export {
-  generateSmartDiff,
-  generateAnnotatedDiff,
-  cleanupAnnotations,
-  fixCitationAnnotations,
-} from './diff-engine.js';
-export type {
-  GenerateSmartDiffOptions,
-} from './diff-engine.js';
-
-export {
-  restoreCrossrefFromWord,
-  restoreImagesFromRegistry,
-  parseVisibleComments,
-  convertVisibleComments,
-} from './restore-references.js';
-export type {
-  RestoreCrossrefResult,
-  RestoreImagesResult,
-} from './restore-references.js';
-
-const execAsync = promisify(exec);
-
-// ============================================
-// Type Definitions (orchestration-specific)
-// ============================================
-
-export interface InsertCommentsOptions {
-  quiet?: boolean;
-  sectionBoundary?: { start: number; end: number } | null;
-  /**
-   * When true (default), comments wrap their anchor text in `[anchor]{.mark}`
-   * so the rebuilt docx restores the original Word comment range. When false,
-   * comments are inserted as standalone `{>>...<<}` blocks adjacent to the
-   * anchor — the prose stays byte-identical except for the inserted blocks.
-   *
-   * Set to false from `sync --comments-only` so a draft revised after the
-   * docx was sent for review keeps its prose intact, and so multiple
-   * comments sharing one anchor don't produce nested broken markup.
-   */
-  wrapAnchor?: boolean;
-  /**
-   * Mutable output: when provided, the function fills in counters so callers
-   * can distinguish placement outcomes in their summary. `placed` counts
-   * insertions made at a matched anchor or context, `lowConfidence` counts
-   * insertions made at an approximate (proportional / context-only) position
-   * that should be reviewed with `rev verify-anchors`, `deduped` counts
-   * comments already present at their anchor (skipped on re-sync), and
-   * `unmatched` counts comments that could not be placed at all.
-   */
-  outStats?: { placed: number; deduped: number; unmatched: number; lowConfidence?: number };
-}
-
-/**
- * Whether a resolved placement landed on matched anchor/context text (high) or
- * on an approximate position the user should review (low). Low-confidence
- * strategies place the comment without matching the anchor word itself.
- */
-function placementConfidence(strategy: string | undefined): 'high' | 'low' {
-  if (!strategy) return 'high';
-  return /proportional|position-only|context/.test(strategy) ? 'low' : 'high';
-}
-
-export interface CommentWithPos {
-  id: string;
-  author: string;
-  text: string;
-  date: string;
-  pos: number;
-  anchorText: string | null;
-  anchorEnd?: number;
-  isEmpty?: boolean;
-  strategy?: string;
-}
-
-export type { AnchorSearchResult } from './anchor-match.js';
-
-export interface MarkdownPrefixResult {
-  prefix: string;
-  content: string;
-}
-
-export interface ImportWordWithTrackChangesOptions {
-  mediaDir?: string;
-  projectDir?: string;
-}
-
-export interface ImportWordWithTrackChangesResult {
-  text: string;
-  stats: {
-    insertions: number;
-    deletions: number;
-    substitutions: number;
-    comments: number;
-    total: number;
-    hasTrackChanges: boolean;
-    trackChangeStats: { insertions: number; deletions: number };
-  };
-  extractedMedia: string[];
-  comments: WordComment[];
-}
-
-export interface ImportFromWordOptions {
-  author?: string;
-  sectionContent?: string;
-  figuresDir?: string;
-  wordTables?: WordTable[];
-}
-
-export interface ImportFromWordResult {
-  annotated: string;
-  stats: {
-    insertions: number;
-    deletions: number;
-    substitutions: number;
-    comments: number;
-    total: number;
-  };
-  extractedMedia: string[];
-}
-
-export interface MovedFile {
-  from: string;
-  to: string;
-  name: string;
-}
-
-export interface MoveExtractedMediaResult {
-  moved: MovedFile[];
-  errors: string[];
-}
-
-// ============================================
-// Functions
-// ============================================
-
-/**
- * If `pos` lands inside a section file's leading `# Heading` line (or the
- * blank line right after it), advance past the first paragraph break so
- * the comment stays inside the section. A comment authored at the very
- * start of a Word section maps to `pos === 0`, but inserting at column 0
- * of a markdown file that begins with `# Heading` puts the `{>>...<<}`
- * before the heading marker — Pandoc then treats the line as ordinary
- * paragraph text and the comment renders in the previous section.
- */
-function pushPastSectionHeading(text: string, pos: number): number {
-  if (pos > 0) {
-    const headingMatch = text.match(/^#{1,6}\s.+$/m);
-    if (!headingMatch || headingMatch.index === undefined) return pos;
-    const headingEnd = headingMatch.index + headingMatch[0].length;
-    if (pos >= headingEnd) return pos;
-  }
-  // pos is at-or-before the first heading line. Advance to the first
-  // non-blank position after the heading paragraph.
-  const headingLine = text.match(/^#{1,6}\s.+(?:\n|$)/m);
-  if (!headingLine || headingLine.index === undefined) return pos;
-  let after = headingLine.index + headingLine[0].length;
-  // Skip blank lines so we land at the start of the first body paragraph.
-  while (after < text.length && (text[after] === '\n' || text[after] === '\r')) {
-    after++;
-  }
-  return after;
-}
-
-/**
- * Snap a position to the nearest whitespace boundary within ±50 chars so a
- * proportional fallback insertion never lands mid-word.
- */
-function snapToWordBoundary(text: string, pos: number): number {
-  if (pos <= 0) return 0;
-  if (pos >= text.length) return text.length;
-  if (/\s/.test(text[pos] ?? '')) return pos;
-  for (let d = 1; d <= 50; d++) {
-    if (pos + d < text.length && /\s/.test(text[pos + d] ?? '')) return pos + d;
-    if (pos - d >= 0 && /\s/.test(text[pos - d] ?? '')) return pos - d;
-  }
-  return pos;
-}
-
-/**
- * Final-resort placement when every text-matching strategy failed. The docx
- * carries a real `<w:commentRangeStart w:id="N">` marker at a known offset
- * inside its body text — that's a structural anchor, even if the anchored
- * span itself is empty and the surrounding context drifted in the target.
- *
- * Map docPosition into the target markdown proportionally and snap to a word
- * boundary. This is approximate when the document was heavily restructured,
- * but it's strictly better than silently dropping a reviewer's comment: the
- * comment lands in roughly the right neighborhood and the reviewer can
- * relocate it during their next pass.
- */
-function proportionalFallback(
-  anchorData: CommentAnchorData,
-  target: string,
-): number | null {
-  if (anchorData.docLength <= 0) return null;
-  const proportion = Math.min(anchorData.docPosition / anchorData.docLength, 1.0);
-  const rawPos = Math.floor(proportion * target.length);
-  return pushPastSectionHeading(target, snapToWordBoundary(target, rawPos));
-}
-
-/**
- * Insert comments into markdown text based on anchor texts with context
- */
-export function insertCommentsIntoMarkdown(
-  markdown: string,
-  comments: WordComment[],
-  anchors: Map<string, CommentAnchorData | string>,
-  options: InsertCommentsOptions = {}
-): string {
-  const { quiet = false, sectionBoundary = null, wrapAnchor = true, outStats } = options;
-  let result = markdown;
-  let unmatchedCount = 0;
-  let placedCount = 0;
-  let lowConfidenceCount = 0;
-  const duplicateWarnings: string[] = [];
-  const usedPositions = new Set<number>(); // For tie-breaking: track used positions
-
-  // Resolve threading: replies share their parent's anchor in Word, so they
-  // must inherit the parent's position and ride alongside it as one cluster.
-  // Letting each reply run through anchor scoring scatters the cluster (the
-  // same docPosition forces `usedPositions` to push later replies onto a
-  // different occurrence), which on re-build looks like independent comments
-  // and loses the paraIdParent threading. See gcol33/docrev issue #2.
-  const inputById = new Map<string, WordComment>();
-  for (const c of comments) inputById.set(c.id, c);
-  function rootIdOf(c: WordComment): string {
-    let cur: WordComment = c;
-    const seen = new Set<string>();
-    while (cur.parentId && !seen.has(cur.id)) {
-      seen.add(cur.id);
-      const parent = inputById.get(cur.parentId);
-      if (!parent || parent === cur) break;
-      cur = parent;
-    }
-    return cur.id;
-  }
-  const replyRootId = new Map<string, string>();
-  for (const c of comments) {
-    const root = rootIdOf(c);
-    if (root !== c.id) replyRootId.set(c.id, root);
-  }
-
-  // Anchor matching primitives live in lib/anchor-match.ts so that
-  // `rev verify-anchors` can use the same strategies for drift reporting.
-
-  // Get all positions in order (for sequential tie-breaking).
-  // Replies skip scoring entirely — they piggyback on their root's position
-  // in the emit pass below.
-  const commentsWithPositions = comments.map((c): CommentWithPos => {
-    if (replyRootId.has(c.id)) {
-      return { ...c, pos: -1, anchorText: null, strategy: 'reply' };
-    }
-    const anchorData = anchors.get(c.id);
-    if (!anchorData) {
-      unmatchedCount++;
-      return { ...c, pos: -1, anchorText: null };
-    }
-
-    // Support both old format (string) and new format ({anchor, before, after})
-    const anchor = typeof anchorData === 'string' ? anchorData : anchorData.anchor;
-    const before = typeof anchorData === 'object' ? anchorData.before : '';
-    const after = typeof anchorData === 'object' ? anchorData.after : '';
-    const isEmpty = typeof anchorData === 'object' && anchorData.isEmpty;
-    const docPosition = typeof anchorData === 'object' ? anchorData.docPosition : undefined;
-
-    // Section-scoped insertion. The comment is known to belong to this
-    // section (the caller already filtered by docPosition); place it by
-    // matching its anchor text within the section, never by mapping a length
-    // ratio across the docx and markdown coordinate systems. The section
-    // estimate only seeds duplicate tie-breaking and the final fallback.
-    if (sectionBoundary && docPosition !== undefined) {
-      const sectionLength = sectionBoundary.end - sectionBoundary.start;
-      if (sectionLength > 0) {
-        const relativePos = docPosition < sectionBoundary.start ? 0 : docPosition - sectionBoundary.start;
-        const proportion = Math.min(relativePos / sectionLength, 1.0);
-        const estimatedPos = Math.floor(proportion * result.length);
-
-        // Empty anchor: before/after context is the only text signal.
-        if ((!anchor || isEmpty) && (before || after)) {
-          const ctx = findAnchorInText('', result, before, after);
-          if (ctx.occurrences.length > 0) {
-            const pos = pushPastSectionHeading(result, ctx.occurrences[0]);
-            return { ...c, pos, anchorText: null, isEmpty: true, strategy: `ctx:${ctx.strategy}` };
-          }
-        }
-
-        // Non-empty anchor: locate it anywhere in the section text. Duplicates
-        // are disambiguated by before/after context and, only as a tie-break,
-        // by proximity to the section estimate.
-        if (anchor && !isEmpty) {
-          const { occurrences, matchedAnchor, strategy } = findAnchorInText(anchor, result, before, after);
-          if (occurrences.length > 0) {
-            const anchorLen = matchedAnchor ? matchedAnchor.length : 0;
-            const chosen = pickBestOccurrence(occurrences, result, before, after, anchorLen, usedPositions, estimatedPos);
-            const finalIdx = chosen >= 0 ? chosen : occurrences[0];
-            if (occurrences.length > 1 && matchedAnchor) {
-              duplicateWarnings.push(`"${matchedAnchor.slice(0, 40)}${matchedAnchor.length > 40 ? '...' : ''}" appears ${occurrences.length} times in section`);
-            }
-            usedPositions.add(finalIdx);
-            if (matchedAnchor) {
-              return { ...c, pos: finalIdx, anchorText: matchedAnchor, anchorEnd: finalIdx + anchorLen, strategy: `section:${strategy}` };
-            }
-            return { ...c, pos: finalIdx, anchorText: null, strategy: `section:${strategy}` };
-          }
-        }
-
-        // Anchor text and context are both gone from the section. The docx
-        // marker's offset is the only remaining signal: map it proportionally
-        // and snap to a word boundary. This is an approximate placement, not a
-        // match — flagged low-confidence so the summary surfaces it.
-        const insertPos = pushPastSectionHeading(result, snapToWordBoundary(result, estimatedPos));
-        return { ...c, pos: insertPos, anchorText: null, strategy: 'proportional-fallback' };
-      }
-    }
-
-    // Handle empty anchors
-    if (!anchor || isEmpty) {
-      if (before || after) {
-        const { occurrences } = findAnchorInText('', result, before, after);
-        if (occurrences.length > 0) {
-          return { ...c, pos: occurrences[0], anchorText: null, isEmpty: true };
-        }
-      }
-      // Last resort: docx carried a structural marker at docPosition; map
-      // it proportionally into the target so the comment isn't dropped.
-      if (typeof anchorData === 'object') {
-        const fallback = proportionalFallback(anchorData, result);
-        if (fallback !== null) {
-          return { ...c, pos: fallback, anchorText: null, isEmpty: true, strategy: 'proportional-fallback' };
-        }
-      }
-      unmatchedCount++;
-      return { ...c, pos: -1, anchorText: null, isEmpty: true };
-    }
-
-    // Text-based matching strategies
-    const { occurrences, matchedAnchor, strategy, stripped } = findAnchorInText(anchor, result, before, after);
-
-    if (occurrences.length === 0) {
-      // Same last-resort as the empty-anchor path: anchor text is gone from
-      // the target, but the marker's text-offset survived extraction.
-      if (typeof anchorData === 'object') {
-        const fallback = proportionalFallback(anchorData, result);
-        if (fallback !== null) {
-          return { ...c, pos: fallback, anchorText: null, strategy: 'proportional-fallback' };
-        }
-      }
-      unmatchedCount++;
-      return { ...c, pos: -1, anchorText: null };
-    }
-
-    const anchorLen = matchedAnchor ? matchedAnchor.length : 0;
-
-    if (occurrences.length === 1) {
-      if (matchedAnchor) {
-        return { ...c, pos: occurrences[0], anchorText: matchedAnchor, anchorEnd: occurrences[0] + anchorLen };
-      } else {
-        return { ...c, pos: occurrences[0], anchorText: null };
-      }
-    }
-
-    // Multiple occurrences - use context for disambiguation
-    if (matchedAnchor) {
-      duplicateWarnings.push(`"${matchedAnchor.slice(0, 40)}${matchedAnchor.length > 40 ? '...' : ''}" appears ${occurrences.length} times`);
-    }
-
-    const bestIdx = pickBestOccurrence(occurrences, result, before, after, anchorLen, usedPositions);
-    const finalIdx = bestIdx >= 0 ? bestIdx : occurrences[0];
-    usedPositions.add(finalIdx);
-
-    if (matchedAnchor) {
-      return { ...c, pos: finalIdx, anchorText: matchedAnchor, anchorEnd: finalIdx + anchorLen };
-    } else {
-      return { ...c, pos: finalIdx, anchorText: null };
-    }
-  });
-
-  // Group comments into clusters (root + ordered replies). The root carries
-  // the resolved position; replies inherit it and ride along in input order
-  // so the rebuilt CriticMarkup looks like `{>>p<<}{>>r1<<}{>>r2<<}[anchor]`
-  // and adjacency-based reply detection picks the cluster up again.
-  const byId = new Map<string, CommentWithPos>();
-  for (const cwp of commentsWithPositions) byId.set(cwp.id, cwp);
-  const repliesByRoot = new Map<string, CommentWithPos[]>();
-  for (const c of comments) {
-    const rootId = replyRootId.get(c.id);
-    if (!rootId) continue;
-    const cwp = byId.get(c.id);
-    if (!cwp) continue;
-    const list = repliesByRoot.get(rootId);
-    if (list) list.push(cwp);
-    else repliesByRoot.set(rootId, [cwp]);
-  }
-
-  // Replies whose root never resolved (parent missing from the input slice or
-  // parent unmatched) count as unmatched too — there's no position to attach
-  // them to.
-  for (const [rootId, replies] of repliesByRoot) {
-    const root = byId.get(rootId);
-    if (!root || root.pos < 0) {
-      unmatchedCount += replies.length;
-    }
-  }
-
-  // Roots only — replies attach during emission.
-  const rootsWithPos = commentsWithPositions.filter(
-    c => !replyRootId.has(c.id)
-  );
-
-  // Log any unmatched roots for debugging
-  const unmatched = rootsWithPos.filter((c) => c.pos < 0);
-  if (process.env.DEBUG) {
-    console.log(`[DEBUG] insertComments: ${comments.length} input, ${rootsWithPos.length} roots, ${unmatched.length} unmatched roots, ${replyRootId.size} replies`);
-    if (unmatched.length > 0) {
-      unmatched.forEach(c => console.log(`[DEBUG]   Unmatched ID=${c.id}: anchor="${(c.anchorText || 'none').slice(0,30)}"`));
-    }
-  }
-
-  const matchedRoots = rootsWithPos.filter((c) => c.pos >= 0);
-
-  // Sort by position descending (insert from end to avoid offset issues)
-  matchedRoots.sort((a, b) => b.pos - a.pos);
-
-  // Insert each cluster. With `wrapAnchor` (the default), the anchor text
-  // gets wrapped in `[anchor]{.mark}` so the rebuilt docx restores the
-  // original Word comment range. Without it, the comment block is inserted
-  // adjacent to the anchor and prose stays untouched — required for
-  // comments-only sync where multiple comments may share one anchor.
-  // Skip insertion when the parent's CriticMarkup already lives near the
-  // target — re-running sync against the same docx would otherwise stack
-  // duplicates. A 200-char window catches both wrapped
-  // (`{>>...<<}[anchor]{.mark}`) and bare (`{>>...<<}anchor`) forms while
-  // ignoring incidental matches farther away.
-  let dedupedCount = 0;
-  for (const c of matchedRoots) {
-    const parentBlock = `{>>${c.author}: ${c.text}<<}`;
-    const replies = repliesByRoot.get(c.id) ?? [];
-    const windowStart = Math.max(0, c.pos - 200);
-    const windowEnd = Math.min(result.length, c.pos + 200);
-    if (result.slice(windowStart, windowEnd).includes(parentBlock)) {
-      // Cluster already synced; treat all members as deduped.
-      dedupedCount += 1 + replies.length;
-      continue;
-    }
-    // Replies carry an explicit `↪ ` author prefix so the round-trip does not
-    // depend on positional adjacency in the markdown. On dense reviewer docs
-    // distinct clusters frequently land at the same anchor position; without
-    // the prefix the re-parse would misthread them. The injection side strips
-    // `↪ ` back off the author so Word renders the original name.
-    const replyBlocks = replies.map(r => `{>>↪ ${r.author}: ${r.text}<<}`);
-    const combined = parentBlock + replyBlocks.join('');
-    if (wrapAnchor && c.anchorText && c.anchorEnd) {
-      const before = result.slice(0, c.pos);
-      const anchor = result.slice(c.pos, c.anchorEnd);
-      const after = result.slice(c.anchorEnd);
-      result = before + combined + `[${anchor}]{.mark}` + after;
-    } else {
-      result = result.slice(0, c.pos) + combined + result.slice(c.pos);
-    }
-    // Replies inherit the root's confidence — they ride the same position.
-    if (placementConfidence(c.strategy) === 'low') {
-      lowConfidenceCount += 1 + replies.length;
-    } else {
-      placedCount += 1 + replies.length;
-    }
-  }
-
-  if (outStats) {
-    outStats.placed = placedCount;
-    outStats.lowConfidence = lowConfidenceCount;
-    outStats.deduped = dedupedCount;
-    outStats.unmatched = unmatchedCount;
-  }
-
-  // Log warnings unless quiet mode
-  if (!quiet) {
-    if (unmatchedCount > 0) {
-      console.warn(`Warning: ${unmatchedCount} comment(s) could not be matched to anchor text`);
-    }
-    if (lowConfidenceCount > 0) {
-      console.warn(`Note: ${lowConfidenceCount} comment(s) placed approximately — run \`rev verify-anchors\` to review`);
-    }
-    if (dedupedCount > 0) {
-      console.warn(`Note: ${dedupedCount} comment(s) already present at anchor — skipped to avoid duplication`);
-    }
-    if (duplicateWarnings.length > 0) {
-      console.warn(`Warning: Duplicate anchor text found (using context & tie-breaks for placement):`);
-      for (const w of duplicateWarnings) {
-        console.warn(`  - ${w}`);
-      }
-    }
-  }
-
-  return result;
-}
-
-/**
- * Import Word document with track changes directly as CriticMarkup
- */
-export async function importWordWithTrackChanges(
-  docxPath: string,
-  options: ImportWordWithTrackChangesOptions = {}
-): Promise<ImportWordWithTrackChangesResult> {
-  const { mediaDir, projectDir } = options;
-  const docxDir = path.dirname(docxPath);
-  const targetMediaDir = mediaDir || path.join(docxDir, 'media');
-  const targetProjectDir = projectDir || docxDir;
-
-  const registry = readImageRegistry(targetProjectDir);
-  const hasRegistry = registry && registry.figures && registry.figures.length > 0;
-
-  // First pass: count images
-  const { stdout: rawText } = await execAsync(
-    `pandoc "${docxPath}" -t markdown --wrap=none --track-changes=all`,
-    { maxBuffer: 50 * 1024 * 1024 }
-  );
-
-  const wordImageCount = (rawText.match(/!\[[^\]]*\]\(media\/[^)]+\)/g) || []).length;
-  const registryCount = hasRegistry ? registry.figures.length : 0;
-
-  const needsMediaExtraction = wordImageCount > registryCount;
-
-  if (hasRegistry) {
-    console.log(`Registry has ${registryCount} figures, Word doc has ${wordImageCount} images`);
-    if (needsMediaExtraction) {
-      console.log(`Extracting media (${wordImageCount - registryCount} new image(s) detected)`);
-    } else {
-      console.log(`Using existing figures from registry`);
-    }
-  }
-
-  // Extract from Word
-  const extracted = await extractFromWord(docxPath, {
-    mediaDir: targetMediaDir,
-    skipMediaExtraction: !needsMediaExtraction,
-  });
-
-  let text = extracted.text;
-  const extractedMedia = extracted.extractedMedia || [];
-  const comments = extracted.comments || [];
-  const anchors = extracted.anchors || new Map();
-
-  // Log messages
-  for (const msg of extracted.messages || []) {
-    if (msg.type === 'info') {
-      console.log(msg.message);
-    } else if (msg.type === 'warning') {
-      console.warn(`Warning: ${msg.message}`);
-    }
-  }
-
-  // Restore crossref
-  const crossrefResult = restoreCrossrefFromWord(text, targetProjectDir);
-  text = crossrefResult.text;
-  if (crossrefResult.restored > 0) {
-    console.log(`Restored ${crossrefResult.restored} crossref reference(s)`);
-  }
-
-  // Restore images
-  const imageRestoreResult = restoreImagesFromRegistry(text, targetProjectDir, crossrefResult.restoredLabels);
-  text = imageRestoreResult.text;
-  if (imageRestoreResult.restored > 0) {
-    console.log(`Restored ${imageRestoreResult.restored} image(s) from registry`);
-  }
-
-  // Insert comments
-  if (comments.length > 0) {
-    text = insertCommentsIntoMarkdown(text, comments, anchors);
-    console.log(`Inserted ${comments.length} comment(s)`);
-  }
-
-  // Clean up
-  text = cleanupAnnotations(text);
-
-  // Count final changes
-  const insertions = (text.match(/\{\+\+/g) || []).length;
-  const deletions = (text.match(/\{--/g) || []).length;
-  const substitutions = (text.match(/\{~~/g) || []).length;
-  const commentCount = (text.match(/\{>>/g) || []).length;
-
-  return {
-    text,
-    stats: {
-      insertions,
-      deletions,
-      substitutions,
-      comments: commentCount,
-      total: insertions + deletions + substitutions + commentCount,
-      hasTrackChanges: extracted.hasTrackChanges,
-      trackChangeStats: extracted.trackChangeStats,
-    },
-    extractedMedia,
-    comments,
-  };
-}
-
-/**
- * Read a `.docx` into the same CriticMarkup-annotated Markdown that
- * `rev import` produces, for read-only inspection (`rev status`, `rev
- * comments`, ...). Track changes become `{++..++}` / `{--..--}` / `{~~..~~}`
- * and comments are placed at their anchors, so `countAnnotations` /
- * `getComments` and the rest operate on it exactly as they do on imported
- * Markdown — the numbers match a subsequent `rev import`.
- *
- * Side-effect free: no media is extracted, nothing is written to disk, and
- * extraction/placement messages are discarded (the caller only wants the
- * annotated text). Works with or without pandoc — the XML fallback emits the
- * same CriticMarkup — so it never reads the binary ZIP as text.
- */
-export async function readDocxAsAnnotatedMarkdown(docxPath: string): Promise<string> {
-  const extracted = await extractFromWord(docxPath, { skipMediaExtraction: true });
-  let text = extracted.text;
-  const comments = extracted.comments || [];
-  const anchors = extracted.anchors || new Map();
-
-  if (comments.length > 0) {
-    text = insertCommentsIntoMarkdown(text, comments, anchors, { quiet: true });
-  }
-
-  return cleanupAnnotations(text);
-}
-
-/**
- * Legacy import function: Word doc → annotated MD via diff
- */
-export async function importFromWord(
-  docxPath: string,
-  originalMdPath: string,
-  options: ImportFromWordOptions = {}
-): Promise<ImportFromWordResult> {
-  const { author = 'Reviewer', sectionContent, figuresDir } = options;
-  const projectDir = path.dirname(originalMdPath);
-
-  let wordText: string;
-  let extractedMedia: string[] = [];
-  let wordTables: WordTable[] = options.wordTables || [];
-  let hasTrackChanges = false;
-
-  if (sectionContent !== undefined) {
-    let annotated = cleanupAnnotations(sectionContent);
-
-    const insertions = (annotated.match(/\{\+\+/g) || []).length;
-    const deletions = (annotated.match(/\{--/g) || []).length;
-    const substitutions = (annotated.match(/\{~~/g) || []).length;
-    const commentCount = (annotated.match(/\{>>/g) || []).length;
-
-    return {
-      annotated,
-      stats: {
-        insertions,
-        deletions,
-        substitutions,
-        comments: commentCount,
-        total: insertions + deletions + substitutions + commentCount,
-      },
-      extractedMedia: [],
-    };
-  } else {
-    const docxDir = path.dirname(docxPath);
-    const mediaDir = figuresDir || docxDir;
-
-    const extracted = await extractFromWord(docxPath, { mediaDir });
-    wordText = extracted.text;
-    extractedMedia = extracted.extractedMedia || [];
-    wordTables = extracted.tables || [];
-    hasTrackChanges = extracted.hasTrackChanges || false;
-
-    for (const msg of extracted.messages || []) {
-      if (msg.type === 'info') {
-        console.log(msg.message);
-      } else if (msg.type === 'warning') {
-        console.warn(`Warning: ${msg.message}`);
-      }
-    }
-
-    if (hasTrackChanges) {
-      const crossrefResult = restoreCrossrefFromWord(wordText, projectDir);
-      wordText = crossrefResult.text;
-      if (crossrefResult.restored > 0) {
-        console.log(`Restored ${crossrefResult.restored} crossref reference(s)`);
-      }
-
-      const imageRestoreResult = restoreImagesFromRegistry(wordText, projectDir, crossrefResult.restoredLabels);
-      wordText = imageRestoreResult.text;
-      if (imageRestoreResult.restored > 0) {
-        console.log(`Restored ${imageRestoreResult.restored} image(s) from registry`);
-      }
-
-      const comments = extracted.comments || [];
-      const anchors = extracted.anchors || new Map();
-      if (comments.length > 0) {
-        wordText = insertCommentsIntoMarkdown(wordText, comments, anchors);
-        console.log(`Inserted ${comments.length} comment(s)`);
-      }
-
-      wordText = cleanupAnnotations(wordText);
-
-      const insertions = (wordText.match(/\{\+\+/g) || []).length;
-      const deletions = (wordText.match(/\{--/g) || []).length;
-      const substitutions = (wordText.match(/\{~~/g) || []).length;
-      const commentCount = (wordText.match(/\{>>/g) || []).length;
-
-      return {
-        annotated: wordText,
-        stats: {
-          insertions,
-          deletions,
-          substitutions,
-          comments: commentCount,
-          total: insertions + deletions + substitutions + commentCount,
-        },
-        extractedMedia,
-      };
-    }
-
-    console.warn('Warning: No track changes detected in Word document.');
-    console.warn('  For best results, reviewers should use Track Changes in Word.');
-    console.warn('  Falling back to diff-based import (comparing against original MD).');
-    console.warn('  This approach may produce less accurate change annotations.');
-
-    const crossrefResult = restoreCrossrefFromWord(wordText, projectDir);
-    wordText = crossrefResult.text;
-    if (crossrefResult.restored > 0) {
-      console.log(`Restored ${crossrefResult.restored} crossref reference(s)`);
-    }
-
-    const imageRestoreResult = restoreImagesFromRegistry(wordText, projectDir, crossrefResult.restoredLabels);
-    wordText = imageRestoreResult.text;
-    if (imageRestoreResult.restored > 0) {
-      console.log(`Restored ${imageRestoreResult.restored} image(s) from registry`);
-    }
-  }
-
-  // Read original markdown
-  let originalMd = fs.readFileSync(originalMdPath, 'utf-8');
-
-  // Strip existing annotations
-  originalMd = stripAnnotations(originalMd, { keepComments: false });
-
-  // Load image registry
-  const imageRegistry = readImageRegistry(projectDir);
-
-  // Generate diff
-  let annotated = generateSmartDiff(originalMd, wordText, author, { wordTables, imageRegistry });
-
-  // Clean up
-  annotated = cleanupAnnotations(annotated);
-
-  // Fix citation annotations
-  annotated = fixCitationAnnotations(annotated, originalMd);
-
-  // Convert visible comments
-  annotated = convertVisibleComments(annotated);
-
-  // Count changes
-  const insertions = (annotated.match(/\{\+\+/g) || []).length;
-  const deletions = (annotated.match(/\{--/g) || []).length;
-  const substitutions = (annotated.match(/\{~~/g) || []).length;
-  const comments = (annotated.match(/\{>>/g) || []).length;
-
-  return {
-    annotated,
-    stats: {
-      insertions,
-      deletions,
-      substitutions,
-      comments,
-      total: insertions + deletions + substitutions + comments,
-    },
-    extractedMedia,
-  };
-}
-
-/**
- * Move extracted media files to a figures directory with better names
- */
-export function moveExtractedMedia(
-  mediaFiles: string[],
-  figuresDir: string,
-  prefix: string = 'figure'
-): MoveExtractedMediaResult {
-  const moved: MovedFile[] = [];
-  const errors: string[] = [];
-
-  if (!fs.existsSync(figuresDir)) {
-    fs.mkdirSync(figuresDir, { recursive: true });
-  }
-
-  for (let i = 0; i < mediaFiles.length; i++) {
-    const src = mediaFiles[i];
-    const ext = path.extname(src).toLowerCase();
-    const newName = `${prefix}${i + 1}${ext}`;
-    const dest = path.join(figuresDir, newName);
-
-    try {
-      fs.copyFileSync(src, dest);
-      moved.push({ from: src, to: dest, name: newName });
-    } catch (err: any) {
-      errors.push(`Failed to copy ${src}: ${err.message}`);
-    }
-  }
-
-  return { moved, errors };
-}