fix: improve Node.js stream type handling in form data

nylas · Dec 24, 2024 · a9568ac · a9568ac
1 parent 031b8ce
commit a9568ac
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/resources/messages.ts b/src/resources/messages.ts
@@ -328,8 +328,14 @@ export class Messages extends Resource {
         const contentId = attachment.contentId || `file${index}`;
         // Handle different types of content (Buffer, ReadableStream, string)
         let file;
-        if (attachment.content instanceof ReadableStream || attachment.content instanceof Readable) {
-          // For ReadableStream or NodeJS.ReadableStream, use it directly as the file
+        if (attachment.content instanceof ReadableStream) {
+          // For web ReadableStream
+          file = attachment.content;
+        } else if (
+          // @ts-ignore - Check for Node.js Readable stream
+          typeof attachment.content?.pipe === 'function'
+        ) {
+          // For Node.js streams (which have pipe method)
           file = attachment.content;
         } else if (
           attachment.content instanceof Buffer ||