fix(trigger): add react-dom and react-email to additionalPackages#4052
fix(trigger): add react-dom and react-email to additionalPackages#4052waleedlatif1 merged 1 commit intostagingfrom
Conversation
PR SummaryLow Risk Overview Reviewed by Cursor Bugbot for commit 9e8e442. Configure here. |
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Greptile SummaryThis PR adds Confidence Score: 5/5Safe to merge — minimal, targeted fix that explicitly bundles required runtime dependencies for the Trigger.dev worker. Single-file change with 7 lines added, directly addressing a known runtime error. The added packages ( No files require special attention.
|
| Filename | Overview |
|---|---|
| apps/sim/trigger.config.ts | Adds react-dom and @react-email/render to additionalPackages so they are explicitly bundled in the Trigger.dev worker; change is minimal and directly addresses the reported runtime error. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[Trigger.dev Worker Build] --> B[additionalPackages extension]
B --> C[unpdf]
B --> D[pdf-lib]
B --> E[isolated-vm]
B --> F[pptxgenjs]
B --> G["react-dom (NEW)"]
B --> H["@react-email/render (NEW)"]
I[lifecycleEmailTask] --> J[renderOnboardingFollowupEmail]
J --> K["@react-email/components render()"]
K --> L["react-dom/server renderToPipeableStream"]
L --> G
K --> H
Reviews (1): Last reviewed commit: "fix(trigger): add react-dom and react-em..." | Re-trigger Greptile
Summary
react-domand@react-email/renderto Trigger.devadditionalPackagesreactDOMServer.renderToPipeableStream is not a functionerror in lifecycle email taskType of Change
Testing
Tested manually
Checklist