Conversation
Production PDF parsing failed with "ReferenceError: DOMMatrix is not defined" (pdfjs expects browser canvas globals that Node lacks). Add @napi-rs/canvas as a real dependency and install DOMMatrix/ImageData/ Path2D on globalThis before pdf-parse loads. Local parsing was unaffected, which masked the bug until the prod smoke test. Also pace inline admin email fan-out with a 300ms gap between sends: sequential-but-instant rejections still tripped Resend's requests-per-second cap (observed 429 in prod EmailLog). Co-authored-by: Claude <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThis PR adds a Canvas polyfill to enable server-side PDF text extraction and implements pacing for admin notification email dispatches. The canvas dependency polyfills missing DOM globals in Node environments, while the dispatch change throttles consecutive email sends with 300ms inter-send delays. ChangesPDF Text Extraction Polyfill
Admin Email Dispatch Pacing
🎯 3 (Moderate) | ⏱️ ~20 minutes
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint install timed out. The project may have too many dependencies for the sandbox. Comment |
Promotes main to master for production deploy.
Changes
ReferenceError: DOMMatrix is not definedthat failed every PDF parse on Vercel (PR fix(resume): polyfill canvas DOM globals for PDF parsing on Vercel #93)Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Bug Fixes
Performance