fix(pdfs): use unpdf instead of pdf-parse

[waleedlatif1]

Summary

• use unpdf instead of pdf-parse

Type of Change

• New feature

Testing

Tested manually

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
docs	Ready	Preview	Comment	Nov 15, 2025 6:33am

[greptile-apps]

Greptile Overview

Greptile Summary

Replaced pdf-parse library with unpdf for PDF text extraction, simplifying the API and reducing complexity.

• Switched from class-based PDFParse instantiation to functional getDocumentProxy and extractText calls
• Removed info and version metadata fields from return value (no downstream dependencies found)
• Updated configuration files (next.config.ts, trigger.config.ts) to reference new library
• Maintained backward compatibility with FileParseResult interface structure

Confidence Score: 4/5

• This PR is safe to merge with minimal risk - it's a straightforward dependency swap
• Score reflects clean library migration with proper configuration updates. Minor concern about removed metadata fields (info, version) - while no current code uses them, this could affect future debugging or analytics needs. Manual testing was performed but automated tests weren't updated to verify the new implementation.
• Check apps/sim/lib/file-parsers/pdf-parser.ts - verify that removing info and version metadata won't impact any analytics or debugging workflows

Important Files Changed

File Analysis

Filename	Score	Overview
apps/sim/lib/file-parsers/pdf-parser.ts	4/5	Switched from pdf-parse to unpdf library, simplified API calls, removed info and version metadata fields
apps/sim/package.json	5/5	Removed pdf-parse and @types/pdf-parse dependencies, added unpdf 1.4.0
apps/sim/next.config.ts	5/5	Updated serverExternalPackages from pdf-parse to unpdf
apps/sim/trigger.config.ts	5/5	Updated build configuration to include unpdf instead of pdf-parse in additional packages

Sequence Diagram

sequenceDiagram
    participant Client
    participant PdfParser
    participant unpdf
    participant Buffer

    Client->>PdfParser: parseFile(filePath)
    PdfParser->>Buffer: readFile(filePath)
    Buffer-->>PdfParser: dataBuffer
    PdfParser->>PdfParser: parseBuffer(dataBuffer)
    PdfParser->>Buffer: new Uint8Array(dataBuffer)
    Buffer-->>PdfParser: uint8Array
    PdfParser->>unpdf: getDocumentProxy(uint8Array)
    unpdf-->>PdfParser: pdf
    PdfParser->>unpdf: extractText(pdf, {mergePages: true})
    unpdf-->>PdfParser: {totalPages, text}
    PdfParser->>PdfParser: cleanContent = text.replace(/\u0000/g, '')
    PdfParser-->>Client: {content, metadata: {pageCount, source}}

Loading

[greptile-apps]

_{4 files reviewed, no comments}

_{Edit Code Review Agent Settings | Greptile}