Skip to content

fix: use pdfjs for server PDF imports - #67

Merged
Talljack merged 1 commit into
mainfrom
fix/pdfjs-pdf-import-runtime
Jun 6, 2026
Merged

fix: use pdfjs for server PDF imports#67
Talljack merged 1 commit into
mainfrom
fix/pdfjs-pdf-import-runtime

Conversation

@Talljack

@Talljack Talljack commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace server-side PDF parsing with pdfjs-dist text extraction to avoid the production DOMMatrix crash from pdf-parse
  • reuse the shared extractPdf helper from the PDF upload route
  • add a focused unit test for the PDF extraction flow

Testing

  • pnpm lint
  • pnpm typecheck
  • pnpm build
  • pnpm test

@vercel

vercel Bot commented Jun 6, 2026 •
edited
Loading

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
echo-type Error Error Jun 6, 2026 4:05am

@Talljack
Talljack merged commit 5374a47 into main Jun 6, 2026
3 of 5 checks passed
@Talljack
Talljack deleted the fix/pdfjs-pdf-import-runtime branch June 6, 2026 04:04

This branch had an error being deployed

1 failed deployment
Preview — f9e121e6 Deployed Jun 6, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Talljack