docrev 0.10.1 → 0.10.2

package/lib/pdf-import.ts

@@ -1,524 +1,524 @@
-/**
- * PDF comment extraction for docrev
- *
- * Extracts annotations (comments, highlights, sticky notes) from PDF files
- * and converts them to CriticMarkup format for insertion into markdown.
- * Also extracts the actual text content under highlights using pdfjs-dist.
- */
-
-import * as fs from 'fs';
-import { PDFDocument } from 'pdf-lib';
-
-/**
- * Annotation types we care about
- */
-const COMMENT_TYPES = [
-  'Text',      // Sticky notes
-  'FreeText',  // Text boxes
-  'Highlight', // Highlighted text with comment
-  'Underline', // Underlined text with comment
-  'StrikeOut', // Strikethrough (deletion suggestion)
-  'Squiggly',  // Squiggly underline
-  'Popup',     // Popup comments (attached to other annotations)
-];
-
-/**
- * Raw PDF annotation extracted from pdf-lib
- */
-export interface PdfAnnotation {
-  type: string;
-  page: number;
-  contents: string;
-  author: string;
-  date: string;
-  rect: number[];
-  quadPoints: number[];
-}
-
-/**
- * PDF comment converted to CriticMarkup format
- */
-export interface PdfComment {
-  author: string;
-  text: string;
-  page: number;
-  type: string;
-  date?: string;
-}
-
-/**
- * PDF annotation with extracted highlighted text
- */
-export interface PdfAnnotationWithText extends PdfAnnotation {
-  highlightedText: string;
-}
-
-/**
- * Options for PDF extraction
- */
-export interface ExtractOptions {
-  timeout?: number;
-}
-
-/**
- * Options for markdown insertion
- */
-export interface InsertOptions {
-  sectionPerPage?: boolean;
-}
-
-/**
- * Statistics about PDF comments
- */
-export interface PdfCommentStats {
-  total: number;
-  byType: Record<string, number>;
-  byAuthor: Record<string, number>;
-  byPage: Record<number, number>;
-}
-
-/**
- * Text item from pdfjs-dist
- */
-interface PdfTextItem {
-  str: string;
-  x: number;
-  y: number;
-  width: number;
-  height: number;
-}
-
-/**
- * Extract raw annotations from a PDF file
- * @param pdfPath - Path to PDF file
- * @param options - { timeout: number (ms) }
- * @returns Array of PDF annotations
- */
-export async function extractPdfAnnotations(
-  pdfPath: string,
-  options: ExtractOptions = {}
-): Promise<PdfAnnotation[]> {
-  const { timeout = 30000 } = options;
-
-  // Validate file exists
-  if (!fs.existsSync(pdfPath)) {
-    throw new Error(`File not found: ${pdfPath}`);
-  }
-
-  let pdfBytes: Buffer;
-  try {
-    pdfBytes = fs.readFileSync(pdfPath);
-  } catch (err) {
-    const error = err as Error;
-    throw new Error(`Cannot read PDF file: ${error.message}`);
-  }
-
-  // Create a promise that rejects after timeout
-  const timeoutPromise = new Promise<never>((_, reject) => {
-    setTimeout(() => reject(new Error(`PDF extraction timed out after ${timeout / 1000}s`)), timeout);
-  });
-
-  let pdfDoc: PDFDocument;
-  try {
-    pdfDoc = await Promise.race([
-      PDFDocument.load(pdfBytes, { ignoreEncryption: true }),
-      timeoutPromise,
-    ]);
-  } catch (err) {
-    const error = err as Error;
-    if (error.message.includes('timed out')) {
-      throw error;
-    }
-    throw new Error(`Invalid or corrupted PDF file: ${error.message}`);
-  }
-
-  const annotations: PdfAnnotation[] = [];
-  const pages = pdfDoc.getPages();
-
-  for (let pageNum = 0; pageNum < pages.length; pageNum++) {
-    const page = pages[pageNum];
-    const annots = page.node.Annots();
-
-    if (!annots) continue;
-
-    const annotRefs = annots.asArray();
-
-    for (const annotRef of annotRefs) {
-      try {
-        const annot = (annotRef as any).dict || annotRef;
-        if (!annot) continue;
-
-        // Get annotation type
-        const subtypeName = annot.get(pdfDoc.context.obj('Subtype'));
-        const subtype = subtypeName?.toString?.()?.replace('/', '') || '';
-
-        if (!COMMENT_TYPES.includes(subtype)) continue;
-
-        // Extract contents (the comment text)
-        const contentsObj = annot.get(pdfDoc.context.obj('Contents'));
-        const contents = contentsObj?.toString?.() || contentsObj?.decodeText?.() || '';
-
-        // Extract author (T field in PDF spec)
-        const authorObj = annot.get(pdfDoc.context.obj('T'));
-        const author = authorObj?.toString?.() || authorObj?.decodeText?.() || 'Unknown';
-
-        // Extract modification date
-        const dateObj = annot.get(pdfDoc.context.obj('M'));
-        const dateStr = dateObj?.toString?.() || '';
-        const date = parsePdfDate(dateStr);
-
-        // Extract rectangle (position on page)
-        const rectObj = annot.get(pdfDoc.context.obj('Rect'));
-        const rect = rectObj?.asArray?.()?.map((n: any) => n?.asNumber?.() || 0) || [0, 0, 0, 0];
-
-        // Extract QuadPoints for highlights (the actual text bounds)
-        const quadObj = annot.get(pdfDoc.context.obj('QuadPoints'));
-        const quadPoints = quadObj?.asArray?.()?.map((n: any) => n?.asNumber?.() || 0) || [];
-
-        // Skip empty annotations
-        if (!contents.trim() && subtype !== 'StrikeOut') continue;
-
-        annotations.push({
-          type: subtype,
-          page: pageNum + 1,
-          contents: cleanPdfString(contents),
-          author: cleanPdfString(author),
-          date,
-          rect,
-          quadPoints,
-        });
-      } catch (err) {
-        // Skip malformed annotations
-        continue;
-      }
-    }
-  }
-
-  // Sort by page, then by vertical position (top to bottom)
-  annotations.sort((a, b) => {
-    if (a.page !== b.page) return a.page - b.page;
-    // Higher Y = higher on page in PDF coords
-    return (b.rect[1] || 0) - (a.rect[1] || 0);
-  });
-
-  return annotations;
-}
-
-/**
- * Parse PDF date string (D:YYYYMMDDHHmmSS format)
- * @param dateStr - PDF date string
- * @returns ISO date string
- */
-function parsePdfDate(dateStr: string): string {
-  if (!dateStr) return '';
-
-  // Remove D: prefix and timezone info
-  const clean = dateStr.replace(/^D:/, '').replace(/[Z+-].*$/, '');
-
-  if (clean.length >= 8) {
-    const year = clean.slice(0, 4);
-    const month = clean.slice(4, 6);
-    const day = clean.slice(6, 8);
-    return `${year}-${month}-${day}`;
-  }
-
-  return '';
-}
-
-/**
- * Clean PDF string (remove parentheses, decode escape sequences)
- * @param str - Raw PDF string
- * @returns Cleaned string
- */
-function cleanPdfString(str: string): string {
-  if (!str) return '';
-
-  return str
-    .replace(/^\(/, '')  // Remove leading paren
-    .replace(/\)$/, '')  // Remove trailing paren
-    .replace(/\\n/g, '\n')  // Newlines
-    .replace(/\\r/g, '')    // Carriage returns
-    .replace(/\\t/g, ' ')   // Tabs
-    .replace(/\\\(/g, '(')  // Escaped parens
-    .replace(/\\\)/g, ')')
-    .replace(/\\\\/g, '\\') // Escaped backslash
-    .trim();
-}
-
-/**
- * Convert PDF annotations to CriticMarkup comments
- * @param annotations - From extractPdfAnnotations
- * @returns Array of PDF comments
- */
-export function annotationsToComments(annotations: PdfAnnotation[]): PdfComment[] {
-  return annotations
-    .filter(a => a.contents.trim())
-    .map(a => ({
-      author: a.author || 'Reviewer',
-      text: a.contents,
-      page: a.page,
-      type: a.type,
-      date: a.date,
-    }));
-}
-
-/**
- * Extract comments from PDF and format for display
- * @param pdfPath - Path to PDF file
- * @returns Array of PDF comments
- */
-export async function extractPdfComments(pdfPath: string): Promise<PdfComment[]> {
-  const annotations = await extractPdfAnnotations(pdfPath);
-  return annotationsToComments(annotations);
-}
-
-/**
- * Insert PDF comments into markdown based on page/position heuristics
- * Since PDFs don't have direct text anchors like Word, we use page numbers
- * and append comments to the end of corresponding sections
- *
- * @param markdown - The markdown content
- * @param comments - Comments from extractPdfComments
- * @param options - { sectionPerPage: boolean }
- * @returns Markdown with comments inserted
- */
-export function insertPdfCommentsIntoMarkdown(
-  markdown: string,
-  comments: PdfComment[],
-  options: InsertOptions = {}
-): string {
-  if (comments.length === 0) return markdown;
-
-  // Group comments by page
-  const commentsByPage = new Map<number, PdfComment[]>();
-  for (const c of comments) {
-    if (!commentsByPage.has(c.page)) {
-      commentsByPage.set(c.page, []);
-    }
-    commentsByPage.get(c.page)!.push(c);
-  }
-
-  // Strategy: Append all comments at the end with page references
-  // This is the safest approach since we can't reliably map PDF positions to markdown
-  const lines = markdown.split('\n');
-  const commentBlock: string[] = [];
-
-  commentBlock.push('');
-  commentBlock.push('<!-- PDF Comments -->');
-
-  for (const [page, pageComments] of Array.from(commentsByPage.entries())) {
-    for (const c of pageComments) {
-      const authorPrefix = c.author ? `${c.author}: ` : '';
-      const pageRef = `[p.${page}]`;
-      commentBlock.push(`{>>${authorPrefix}${pageRef} ${c.text}<<}`);
-    }
-  }
-
-  return lines.join('\n') + commentBlock.join('\n');
-}
-
-/**
- * Format PDF comments for CLI display
- * @param comments - Array of PDF comments
- * @returns Formatted string
- */
-export function formatPdfComments(comments: PdfComment[]): string {
-  if (comments.length === 0) {
-    return 'No comments found in PDF.';
-  }
-
-  const lines: string[] = [];
-  let currentPage = 0;
-
-  for (const c of comments) {
-    if (c.page !== currentPage) {
-      if (currentPage > 0) lines.push('');
-      lines.push(`Page ${c.page}:`);
-      currentPage = c.page;
-    }
-
-    const typeIcon = getTypeIcon(c.type);
-    const author = c.author || 'Unknown';
-    lines.push(`  ${typeIcon} [${author}] ${c.text}`);
-  }
-
-  return lines.join('\n');
-}
-
-/**
- * Get icon for annotation type
- * @param type - Annotation type
- * @returns Icon string
- */
-function getTypeIcon(type: string): string {
-  switch (type) {
-    case 'Text': return '📝';      // Sticky note
-    case 'FreeText': return '💬';  // Text box
-    case 'Highlight': return '🖍️'; // Highlight
-    case 'Underline': return '📍'; // Underline
-    case 'StrikeOut': return '❌'; // Strikethrough
-    case 'Squiggly': return '〰️';  // Squiggly
-    default: return '💬';
-  }
-}
-
-/**
- * Get statistics about PDF comments
- * @param comments - Array of PDF comments
- * @returns Statistics object
- */
-export function getPdfCommentStats(comments: PdfComment[]): PdfCommentStats {
-  const stats: PdfCommentStats = {
-    total: comments.length,
-    byType: {},
-    byAuthor: {},
-    byPage: {},
-  };
-
-  for (const c of comments) {
-    stats.byType[c.type] = (stats.byType[c.type] || 0) + 1;
-    stats.byAuthor[c.author] = (stats.byAuthor[c.author] || 0) + 1;
-    stats.byPage[c.page] = (stats.byPage[c.page] || 0) + 1;
-  }
-
-  return stats;
-}
-
-/**
- * Extract text content from a PDF page
- * @param page - pdfjs page object
- * @returns Array of text items with positions
- */
-async function getPageTextItems(page: any): Promise<PdfTextItem[]> {
-  const textContent = await page.getTextContent();
-  return textContent.items.map((item: any) => ({
-    str: item.str,
-    x: item.transform[4],
-    y: item.transform[5],
-    width: item.width,
-    height: item.height,
-  }));
-}
-
-/**
- * Check if a point is inside a quadrilateral defined by QuadPoints
- * QuadPoints format: [x1,y1, x2,y2, x3,y3, x4,y4] for each quad
- * @param x - X coordinate
- * @param y - Y coordinate
- * @param quad - 8 numbers defining corners
- * @returns True if point is inside quad
- */
-function isPointInQuad(x: number, y: number, quad: number[]): boolean {
-  if (quad.length < 8) return false;
-
-  // Get bounding box from quad points
-  const xs = [quad[0], quad[2], quad[4], quad[6]];
-  const ys = [quad[1], quad[3], quad[5], quad[7]];
-  const minX = Math.min(...xs);
-  const maxX = Math.max(...xs);
-  const minY = Math.min(...ys);
-  const maxY = Math.max(...ys);
-
-  return x >= minX && x <= maxX && y >= minY && y <= maxY;
-}
-
-/**
- * Extract highlighted text from a PDF using QuadPoints
- * @param pdfPath - Path to PDF file
- * @param annotations - Annotations with quadPoints from extractPdfAnnotations
- * @returns Annotations with highlighted text extracted
- */
-export async function extractHighlightedText(
-  pdfPath: string,
-  annotations: PdfAnnotation[]
-): Promise<PdfAnnotationWithText[]> {
-  const pdfBytes = fs.readFileSync(pdfPath);
-  const data = new Uint8Array(pdfBytes);
-
-  // Load pdfjs-dist dynamically (requires DOMMatrix, not available in Node 18)
-  const { getDocument } = await import('pdfjs-dist/legacy/build/pdf.mjs');
-  const loadingTask = getDocument({ data, useSystemFonts: true });
-  const pdfDoc = await loadingTask.promise;
-
-  const results: PdfAnnotationWithText[] = [];
-
-  for (const annot of annotations) {
-    // Only process text markup annotations (Highlight, Underline, StrikeOut, Squiggly)
-    if (!['Highlight', 'Underline', 'StrikeOut', 'Squiggly'].includes(annot.type)) {
-      results.push({ ...annot, highlightedText: '' });
-      continue;
-    }
-
-    if (!annot.quadPoints || annot.quadPoints.length < 8) {
-      results.push({ ...annot, highlightedText: '' });
-      continue;
-    }
-
-    try {
-      const page = await pdfDoc.getPage(annot.page);
-      const textItems = await getPageTextItems(page);
-
-      // Split quadPoints into individual quads (8 numbers each)
-      const quads: number[][] = [];
-      for (let i = 0; i < annot.quadPoints.length; i += 8) {
-        quads.push(annot.quadPoints.slice(i, i + 8));
-      }
-
-      // Find text items that fall within any of the quads
-      const matchedText: string[] = [];
-      for (const item of textItems) {
-        // Check if text item center is in any quad
-        const centerX = item.x + (item.width || 0) / 2;
-        const centerY = item.y + (item.height || 0) / 2;
-
-        for (const quad of quads) {
-          if (isPointInQuad(centerX, centerY, quad) || isPointInQuad(item.x, item.y, quad)) {
-            matchedText.push(item.str);
-            break;
-          }
-        }
-      }
-
-      results.push({
-        ...annot,
-        highlightedText: matchedText.join(' ').trim(),
-      });
-    } catch (err) {
-      // If text extraction fails, just return empty
-      results.push({ ...annot, highlightedText: '' });
-    }
-  }
-
-  return results;
-}
-
-/**
- * Extract annotations with highlighted text in one call
- * @param pdfPath - Path to PDF file
- * @returns Annotations with highlighted text
- */
-export async function extractPdfAnnotationsWithText(pdfPath: string): Promise<PdfAnnotationWithText[]> {
-  const annotations = await extractPdfAnnotations(pdfPath);
-  return extractHighlightedText(pdfPath, annotations);
-}
-
-/**
- * Format annotation with highlighted text for display
- * @param annot - Annotation with highlightedText
- * @returns Formatted string
- */
-export function formatAnnotationWithText(annot: PdfAnnotationWithText): string {
-  const typeIcon = getTypeIcon(annot.type);
-  const author = annot.author || 'Unknown';
-  const parts: string[] = [`${typeIcon} [${author}]`];
-
-  if (annot.highlightedText) {
-    parts.push(`"${annot.highlightedText}"`);
-  }
-
-  if (annot.contents) {
-    parts.push(`→ ${annot.contents}`);
-  }
-
-  return parts.join(' ');
-}
+/**
+ * PDF comment extraction for docrev
+ *
+ * Extracts annotations (comments, highlights, sticky notes) from PDF files
+ * and converts them to CriticMarkup format for insertion into markdown.
+ * Also extracts the actual text content under highlights using pdfjs-dist.
+ */
+
+import * as fs from 'fs';
+import { PDFDocument } from 'pdf-lib';
+
+/**
+ * Annotation types we care about
+ */
+const COMMENT_TYPES = [
+  'Text',      // Sticky notes
+  'FreeText',  // Text boxes
+  'Highlight', // Highlighted text with comment
+  'Underline', // Underlined text with comment
+  'StrikeOut', // Strikethrough (deletion suggestion)
+  'Squiggly',  // Squiggly underline
+  'Popup',     // Popup comments (attached to other annotations)
+];
+
+/**
+ * Raw PDF annotation extracted from pdf-lib
+ */
+export interface PdfAnnotation {
+  type: string;
+  page: number;
+  contents: string;
+  author: string;
+  date: string;
+  rect: number[];
+  quadPoints: number[];
+}
+
+/**
+ * PDF comment converted to CriticMarkup format
+ */
+export interface PdfComment {
+  author: string;
+  text: string;
+  page: number;
+  type: string;
+  date?: string;
+}
+
+/**
+ * PDF annotation with extracted highlighted text
+ */
+export interface PdfAnnotationWithText extends PdfAnnotation {
+  highlightedText: string;
+}
+
+/**
+ * Options for PDF extraction
+ */
+export interface ExtractOptions {
+  timeout?: number;
+}
+
+/**
+ * Options for markdown insertion
+ */
+export interface InsertOptions {
+  sectionPerPage?: boolean;
+}
+
+/**
+ * Statistics about PDF comments
+ */
+export interface PdfCommentStats {
+  total: number;
+  byType: Record<string, number>;
+  byAuthor: Record<string, number>;
+  byPage: Record<number, number>;
+}
+
+/**
+ * Text item from pdfjs-dist
+ */
+interface PdfTextItem {
+  str: string;
+  x: number;
+  y: number;
+  width: number;
+  height: number;
+}
+
+/**
+ * Extract raw annotations from a PDF file
+ * @param pdfPath - Path to PDF file
+ * @param options - { timeout: number (ms) }
+ * @returns Array of PDF annotations
+ */
+export async function extractPdfAnnotations(
+  pdfPath: string,
+  options: ExtractOptions = {}
+): Promise<PdfAnnotation[]> {
+  const { timeout = 30000 } = options;
+
+  // Validate file exists
+  if (!fs.existsSync(pdfPath)) {
+    throw new Error(`File not found: ${pdfPath}`);
+  }
+
+  let pdfBytes: Buffer;
+  try {
+    pdfBytes = fs.readFileSync(pdfPath);
+  } catch (err) {
+    const error = err as Error;
+    throw new Error(`Cannot read PDF file: ${error.message}`);
+  }
+
+  // Create a promise that rejects after timeout
+  const timeoutPromise = new Promise<never>((_, reject) => {
+    setTimeout(() => reject(new Error(`PDF extraction timed out after ${timeout / 1000}s`)), timeout);
+  });
+
+  let pdfDoc: PDFDocument;
+  try {
+    pdfDoc = await Promise.race([
+      PDFDocument.load(pdfBytes, { ignoreEncryption: true }),
+      timeoutPromise,
+    ]);
+  } catch (err) {
+    const error = err as Error;
+    if (error.message.includes('timed out')) {
+      throw error;
+    }
+    throw new Error(`Invalid or corrupted PDF file: ${error.message}`);
+  }
+
+  const annotations: PdfAnnotation[] = [];
+  const pages = pdfDoc.getPages();
+
+  for (let pageNum = 0; pageNum < pages.length; pageNum++) {
+    const page = pages[pageNum];
+    const annots = page.node.Annots();
+
+    if (!annots) continue;
+
+    const annotRefs = annots.asArray();
+
+    for (const annotRef of annotRefs) {
+      try {
+        const annot = (annotRef as any).dict || annotRef;
+        if (!annot) continue;
+
+        // Get annotation type
+        const subtypeName = annot.get(pdfDoc.context.obj('Subtype'));
+        const subtype = subtypeName?.toString?.()?.replace('/', '') || '';
+
+        if (!COMMENT_TYPES.includes(subtype)) continue;
+
+        // Extract contents (the comment text)
+        const contentsObj = annot.get(pdfDoc.context.obj('Contents'));
+        const contents = contentsObj?.toString?.() || contentsObj?.decodeText?.() || '';
+
+        // Extract author (T field in PDF spec)
+        const authorObj = annot.get(pdfDoc.context.obj('T'));
+        const author = authorObj?.toString?.() || authorObj?.decodeText?.() || 'Unknown';
+
+        // Extract modification date
+        const dateObj = annot.get(pdfDoc.context.obj('M'));
+        const dateStr = dateObj?.toString?.() || '';
+        const date = parsePdfDate(dateStr);
+
+        // Extract rectangle (position on page)
+        const rectObj = annot.get(pdfDoc.context.obj('Rect'));
+        const rect = rectObj?.asArray?.()?.map((n: any) => n?.asNumber?.() || 0) || [0, 0, 0, 0];
+
+        // Extract QuadPoints for highlights (the actual text bounds)
+        const quadObj = annot.get(pdfDoc.context.obj('QuadPoints'));
+        const quadPoints = quadObj?.asArray?.()?.map((n: any) => n?.asNumber?.() || 0) || [];
+
+        // Skip empty annotations
+        if (!contents.trim() && subtype !== 'StrikeOut') continue;
+
+        annotations.push({
+          type: subtype,
+          page: pageNum + 1,
+          contents: cleanPdfString(contents),
+          author: cleanPdfString(author),
+          date,
+          rect,
+          quadPoints,
+        });
+      } catch (err) {
+        // Skip malformed annotations
+        continue;
+      }
+    }
+  }
+
+  // Sort by page, then by vertical position (top to bottom)
+  annotations.sort((a, b) => {
+    if (a.page !== b.page) return a.page - b.page;
+    // Higher Y = higher on page in PDF coords
+    return (b.rect[1] || 0) - (a.rect[1] || 0);
+  });
+
+  return annotations;
+}
+
+/**
+ * Parse PDF date string (D:YYYYMMDDHHmmSS format)
+ * @param dateStr - PDF date string
+ * @returns ISO date string
+ */
+function parsePdfDate(dateStr: string): string {
+  if (!dateStr) return '';
+
+  // Remove D: prefix and timezone info
+  const clean = dateStr.replace(/^D:/, '').replace(/[Z+-].*$/, '');
+
+  if (clean.length >= 8) {
+    const year = clean.slice(0, 4);
+    const month = clean.slice(4, 6);
+    const day = clean.slice(6, 8);
+    return `${year}-${month}-${day}`;
+  }
+
+  return '';
+}
+
+/**
+ * Clean PDF string (remove parentheses, decode escape sequences)
+ * @param str - Raw PDF string
+ * @returns Cleaned string
+ */
+function cleanPdfString(str: string): string {
+  if (!str) return '';
+
+  return str
+    .replace(/^\(/, '')  // Remove leading paren
+    .replace(/\)$/, '')  // Remove trailing paren
+    .replace(/\\n/g, '\n')  // Newlines
+    .replace(/\\r/g, '')    // Carriage returns
+    .replace(/\\t/g, ' ')   // Tabs
+    .replace(/\\\(/g, '(')  // Escaped parens
+    .replace(/\\\)/g, ')')
+    .replace(/\\\\/g, '\\') // Escaped backslash
+    .trim();
+}
+
+/**
+ * Convert PDF annotations to CriticMarkup comments
+ * @param annotations - From extractPdfAnnotations
+ * @returns Array of PDF comments
+ */
+export function annotationsToComments(annotations: PdfAnnotation[]): PdfComment[] {
+  return annotations
+    .filter(a => a.contents.trim())
+    .map(a => ({
+      author: a.author || 'Reviewer',
+      text: a.contents,
+      page: a.page,
+      type: a.type,
+      date: a.date,
+    }));
+}
+
+/**
+ * Extract comments from PDF and format for display
+ * @param pdfPath - Path to PDF file
+ * @returns Array of PDF comments
+ */
+export async function extractPdfComments(pdfPath: string): Promise<PdfComment[]> {
+  const annotations = await extractPdfAnnotations(pdfPath);
+  return annotationsToComments(annotations);
+}
+
+/**
+ * Insert PDF comments into markdown based on page/position heuristics
+ * Since PDFs don't have direct text anchors like Word, we use page numbers
+ * and append comments to the end of corresponding sections
+ *
+ * @param markdown - The markdown content
+ * @param comments - Comments from extractPdfComments
+ * @param options - { sectionPerPage: boolean }
+ * @returns Markdown with comments inserted
+ */
+export function insertPdfCommentsIntoMarkdown(
+  markdown: string,
+  comments: PdfComment[],
+  options: InsertOptions = {}
+): string {
+  if (comments.length === 0) return markdown;
+
+  // Group comments by page
+  const commentsByPage = new Map<number, PdfComment[]>();
+  for (const c of comments) {
+    if (!commentsByPage.has(c.page)) {
+      commentsByPage.set(c.page, []);
+    }
+    commentsByPage.get(c.page)!.push(c);
+  }
+
+  // Strategy: Append all comments at the end with page references
+  // This is the safest approach since we can't reliably map PDF positions to markdown
+  const lines = markdown.split('\n');
+  const commentBlock: string[] = [];
+
+  commentBlock.push('');
+  commentBlock.push('<!-- PDF Comments -->');
+
+  for (const [page, pageComments] of Array.from(commentsByPage.entries())) {
+    for (const c of pageComments) {
+      const authorPrefix = c.author ? `${c.author}: ` : '';
+      const pageRef = `[p.${page}]`;
+      commentBlock.push(`{>>${authorPrefix}${pageRef} ${c.text}<<}`);
+    }
+  }
+
+  return lines.join('\n') + commentBlock.join('\n');
+}
+
+/**
+ * Format PDF comments for CLI display
+ * @param comments - Array of PDF comments
+ * @returns Formatted string
+ */
+export function formatPdfComments(comments: PdfComment[]): string {
+  if (comments.length === 0) {
+    return 'No comments found in PDF.';
+  }
+
+  const lines: string[] = [];
+  let currentPage = 0;
+
+  for (const c of comments) {
+    if (c.page !== currentPage) {
+      if (currentPage > 0) lines.push('');
+      lines.push(`Page ${c.page}:`);
+      currentPage = c.page;
+    }
+
+    const typeIcon = getTypeIcon(c.type);
+    const author = c.author || 'Unknown';
+    lines.push(`  ${typeIcon} [${author}] ${c.text}`);
+  }
+
+  return lines.join('\n');
+}
+
+/**
+ * Get icon for annotation type
+ * @param type - Annotation type
+ * @returns Icon string
+ */
+function getTypeIcon(type: string): string {
+  switch (type) {
+    case 'Text': return '📝';      // Sticky note
+    case 'FreeText': return '💬';  // Text box
+    case 'Highlight': return '🖍️'; // Highlight
+    case 'Underline': return '📍'; // Underline
+    case 'StrikeOut': return '❌'; // Strikethrough
+    case 'Squiggly': return '〰️';  // Squiggly
+    default: return '💬';
+  }
+}
+
+/**
+ * Get statistics about PDF comments
+ * @param comments - Array of PDF comments
+ * @returns Statistics object
+ */
+export function getPdfCommentStats(comments: PdfComment[]): PdfCommentStats {
+  const stats: PdfCommentStats = {
+    total: comments.length,
+    byType: {},
+    byAuthor: {},
+    byPage: {},
+  };
+
+  for (const c of comments) {
+    stats.byType[c.type] = (stats.byType[c.type] || 0) + 1;
+    stats.byAuthor[c.author] = (stats.byAuthor[c.author] || 0) + 1;
+    stats.byPage[c.page] = (stats.byPage[c.page] || 0) + 1;
+  }
+
+  return stats;
+}
+
+/**
+ * Extract text content from a PDF page
+ * @param page - pdfjs page object
+ * @returns Array of text items with positions
+ */
+async function getPageTextItems(page: any): Promise<PdfTextItem[]> {
+  const textContent = await page.getTextContent();
+  return textContent.items.map((item: any) => ({
+    str: item.str,
+    x: item.transform[4],
+    y: item.transform[5],
+    width: item.width,
+    height: item.height,
+  }));
+}
+
+/**
+ * Check if a point is inside a quadrilateral defined by QuadPoints
+ * QuadPoints format: [x1,y1, x2,y2, x3,y3, x4,y4] for each quad
+ * @param x - X coordinate
+ * @param y - Y coordinate
+ * @param quad - 8 numbers defining corners
+ * @returns True if point is inside quad
+ */
+function isPointInQuad(x: number, y: number, quad: number[]): boolean {
+  if (quad.length < 8) return false;
+
+  // Get bounding box from quad points
+  const xs = [quad[0], quad[2], quad[4], quad[6]];
+  const ys = [quad[1], quad[3], quad[5], quad[7]];
+  const minX = Math.min(...xs);
+  const maxX = Math.max(...xs);
+  const minY = Math.min(...ys);
+  const maxY = Math.max(...ys);
+
+  return x >= minX && x <= maxX && y >= minY && y <= maxY;
+}
+
+/**
+ * Extract highlighted text from a PDF using QuadPoints
+ * @param pdfPath - Path to PDF file
+ * @param annotations - Annotations with quadPoints from extractPdfAnnotations
+ * @returns Annotations with highlighted text extracted
+ */
+export async function extractHighlightedText(
+  pdfPath: string,
+  annotations: PdfAnnotation[]
+): Promise<PdfAnnotationWithText[]> {
+  const pdfBytes = fs.readFileSync(pdfPath);
+  const data = new Uint8Array(pdfBytes);
+
+  // Load pdfjs-dist dynamically (requires DOMMatrix, not available in Node 18)
+  const { getDocument } = await import('pdfjs-dist/legacy/build/pdf.mjs');
+  const loadingTask = getDocument({ data, useSystemFonts: true });
+  const pdfDoc = await loadingTask.promise;
+
+  const results: PdfAnnotationWithText[] = [];
+
+  for (const annot of annotations) {
+    // Only process text markup annotations (Highlight, Underline, StrikeOut, Squiggly)
+    if (!['Highlight', 'Underline', 'StrikeOut', 'Squiggly'].includes(annot.type)) {
+      results.push({ ...annot, highlightedText: '' });
+      continue;
+    }
+
+    if (!annot.quadPoints || annot.quadPoints.length < 8) {
+      results.push({ ...annot, highlightedText: '' });
+      continue;
+    }
+
+    try {
+      const page = await pdfDoc.getPage(annot.page);
+      const textItems = await getPageTextItems(page);
+
+      // Split quadPoints into individual quads (8 numbers each)
+      const quads: number[][] = [];
+      for (let i = 0; i < annot.quadPoints.length; i += 8) {
+        quads.push(annot.quadPoints.slice(i, i + 8));
+      }
+
+      // Find text items that fall within any of the quads
+      const matchedText: string[] = [];
+      for (const item of textItems) {
+        // Check if text item center is in any quad
+        const centerX = item.x + (item.width || 0) / 2;
+        const centerY = item.y + (item.height || 0) / 2;
+
+        for (const quad of quads) {
+          if (isPointInQuad(centerX, centerY, quad) || isPointInQuad(item.x, item.y, quad)) {
+            matchedText.push(item.str);
+            break;
+          }
+        }
+      }
+
+      results.push({
+        ...annot,
+        highlightedText: matchedText.join(' ').trim(),
+      });
+    } catch (err) {
+      // If text extraction fails, just return empty
+      results.push({ ...annot, highlightedText: '' });
+    }
+  }
+
+  return results;
+}
+
+/**
+ * Extract annotations with highlighted text in one call
+ * @param pdfPath - Path to PDF file
+ * @returns Annotations with highlighted text
+ */
+export async function extractPdfAnnotationsWithText(pdfPath: string): Promise<PdfAnnotationWithText[]> {
+  const annotations = await extractPdfAnnotations(pdfPath);
+  return extractHighlightedText(pdfPath, annotations);
+}
+
+/**
+ * Format annotation with highlighted text for display
+ * @param annot - Annotation with highlightedText
+ * @returns Formatted string
+ */
+export function formatAnnotationWithText(annot: PdfAnnotationWithText): string {
+  const typeIcon = getTypeIcon(annot.type);
+  const author = annot.author || 'Unknown';
+  const parts: string[] = [`${typeIcon} [${author}]`];
+
+  if (annot.highlightedText) {
+    parts.push(`"${annot.highlightedText}"`);
+  }
+
+  if (annot.contents) {
+    parts.push(`→ ${annot.contents}`);
+  }
+
+  return parts.join(' ');
+}