Skip to content

fix: restore production PDF URL imports - #68

Merged
Talljack merged 3 commits into
mainfrom
fix/pdfjs-pdf-import-runtime-v2
Jun 6, 2026
Merged

Talljack merged 3 commits into
mainfrom
fix/pdfjs-pdf-import-runtime-v2

Conversation

@Talljack

@Talljack Talljack commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • switch server-side PDF extraction from pdf-parse to pdfjs-dist to avoid the production DOMMatrix crash
  • reuse the shared PDF extractor in the upload route
  • add a local module declaration for the pdfjs legacy entry and cover the extraction flow with a unit test

Testing

  • pnpm typecheck
  • pnpm test src/lib/extract-text.test.ts
  • pnpm build

@vercel

vercel Bot commented Jun 6, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
echo-type Ready Ready Preview, Comment Jun 6, 2026 4:25am

@Talljack
Talljack merged commit 781eda4 into main Jun 6, 2026
6 checks passed
@Talljack
Talljack deleted the fix/pdfjs-pdf-import-runtime-v2 branch June 6, 2026 04:26

This branch was successfully deployed

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant