fix: do not overwrite new names or emails for signers

.env.example

@@ -50,9 +50,7 @@ NEXT_PRIVATE_SMTP_SECURE=
 NEXT_PRIVATE_SMTP_FROM_NAME="No Reply @ Documenso"
 # REQUIRED: Defines the email address to use as the from address.
 NEXT_PRIVATE_SMTP_FROM_ADDRESS="noreply@documenso.com"
-# OPTIONAL: The API key to use for Resend.com
+# OPTIONAL: The API key to use for the MailChannels proxy endpoint.
-NEXT_PRIVATE_RESEND_API_KEY=
-# OPTIONAL: The API key to use for MailChannels.
 NEXT_PRIVATE_MAILCHANNELS_API_KEY=
 # OPTIONAL: The endpoint to use for the MailChannels API if using a proxy.
 NEXT_PRIVATE_MAILCHANNELS_ENDPOINT=

package-lock.json

@@ -16958,29 +16958,6 @@
         "node": ">=0.10.5"
       }
     },
-    "node_modules/resend": {
-      "version": "1.1.0",
-      "resolved": "https://registry.npmjs.org/resend/-/resend-1.1.0.tgz",
-      "integrity": "sha512-it8TIDVT+/gAiJsUlv2tdHuvzwCCv4Zwu+udDqIm/dIuByQwe68TtFDcPccxqpSVVrNCBxxXLzsdT1tsV+P3GA==",
-      "dependencies": {
-        "@react-email/render": "0.0.7",
-        "type-fest": "3.13.0"
-      },
-      "engines": {
-        "node": ">=18"
-      }
-    },
-    "node_modules/resend/node_modules/type-fest": {
-      "version": "3.13.0",
-      "resolved": "https://registry.npmjs.org/type-fest/-/type-fest-3.13.0.tgz",
-      "integrity": "sha512-Gur3yQGM9qiLNs0KPP7LPgeRbio2QTt4xXouobMCarR0/wyW3F+F/+OWwshg3NG0Adon7uQfSZBpB46NfhoF1A==",
-      "engines": {
-        "node": ">=14.16"
-      },
-      "funding": {
-        "url": "https://github.com/sponsors/sindresorhus"
-      }
-    },
     "node_modules/resolve": {
       "version": "1.22.2",
       "resolved": "https://registry.npmjs.org/resolve/-/resolve-1.22.2.tgz",
@@ -19796,8 +19773,7 @@
       "dependencies": {
         "@react-email/components": "^0.0.7",
         "nodemailer": "^6.9.3",
-        "react-email": "^1.9.4",
+        "react-email": "^1.9.4"
-        "resend": "^1.1.0"
       },
       "devDependencies": {
         "@documenso/tailwind-config": "*",

packages/email/mailer.ts

@@ -1,7 +1,6 @@
 import { createTransport } from 'nodemailer';
 
 import { MailChannelsTransport } from './transports/mailchannels';
-import { ResendTransport } from './transports/resend';
 
 const getTransport = () => {
   const transport = process.env.NEXT_PRIVATE_SMTP_TRANSPORT ?? 'smtp-auth';
@@ -15,14 +14,6 @@ const getTransport = () => {
     );
   }
 
-  if (transport === 'resend') {
-    return createTransport(
-      ResendTransport.makeTransport({
-        apiKey: process.env.NEXT_PRIVATE_RESEND_API_KEY || '',
-      }),
-    );
-  }
-
   if (transport === 'smtp-api') {
     if (!process.env.NEXT_PRIVATE_SMTP_HOST || !process.env.NEXT_PRIVATE_SMTP_APIKEY) {
       throw new Error(

packages/email/package.json

@@ -19,8 +19,7 @@
   "dependencies": {
     "@react-email/components": "^0.0.7",
     "nodemailer": "^6.9.3",
-    "react-email": "^1.9.4",
+    "react-email": "^1.9.4"
-    "resend": "^1.1.0"
   },
   "devDependencies": {
     "@documenso/tailwind-config": "*",

packages/email/transports/resend.ts

@@ -1,145 +0,0 @@
-import { type SentMessageInfo, type Transport } from 'nodemailer';
-import type Mail from 'nodemailer/lib/mailer';
-import type MailMessage from 'nodemailer/lib/mailer/mail-message';
-import { Resend } from 'resend';
-
-const VERSION = '1.0.0';
-
-type ResendTransportOptions = {
-  apiKey: string;
-};
-
-type ResendResponseError = {
-  statusCode: number;
-  name: string;
-  message: string;
-};
-
-const isResendResponseError = (error: unknown): error is ResendResponseError => {
-  // We could use Zod here, but it's not worth the extra bundle size
-  return (
-    typeof error === 'object' &&
-    error !== null &&
-    'statusCode' in error &&
-    typeof error.statusCode === 'number' &&
-    'name' in error &&
-    typeof error.name === 'string' &&
-    'message' in error &&
-    typeof error.message === 'string'
-  );
-};
-
-/**
- * Transport for sending email via the Resend SDK.
- */
-export class ResendTransport implements Transport<SentMessageInfo> {
-  public name = 'ResendMailTransport';
-  public version = VERSION;
-
-  private _client: Resend;
-  private _options: ResendTransportOptions;
-
-  public static makeTransport(options: Partial<ResendTransportOptions>) {
-    return new ResendTransport(options);
-  }
-
-  constructor(options: Partial<ResendTransportOptions>) {
-    const { apiKey = '' } = options;
-
-    this._options = {
-      apiKey,
-    };
-
-    this._client = new Resend(apiKey);
-  }
-
-  public send(mail: MailMessage, callback: (_err: Error | null, _info: SentMessageInfo) => void) {
-    if (!mail.data.to || !mail.data.from) {
-      return callback(new Error('Missing required fields "to" or "from"'), null);
-    }
-
-    this._client
-      .sendEmail({
-        subject: mail.data.subject ?? '',
-        from: this.toResendFromAddress(mail.data.from),
-        to: this.toResendAddresses(mail.data.to),
-        cc: this.toResendAddresses(mail.data.cc),
-        bcc: this.toResendAddresses(mail.data.bcc),
-        html: mail.data.html?.toString() || '',
-        text: mail.data.text?.toString() || '',
-        attachments: this.toResendAttachments(mail.data.attachments),
-      })
-      .then((response) => {
-        if (isResendResponseError(response)) {
-          throw new Error(`[${response.statusCode}]: ${response.name} ${response.message}`);
-        }
-
-        callback(null, response);
-      })
-      .catch((error) => {
-        callback(error, null);
-      });
-  }
-
-  private toResendAddresses(addresses: Mail.Options['to']) {
-    if (!addresses) {
-      return [];
-    }
-
-    if (typeof addresses === 'string') {
-      return [addresses];
-    }
-
-    if (Array.isArray(addresses)) {
-      return addresses.map((address) => {
-        if (typeof address === 'string') {
-          return address;
-        }
-
-        return address.address;
-      });
-    }
-
-    return [addresses.address];
-  }
-
-  private toResendFromAddress(address: Mail.Options['from']) {
-    if (!address) {
-      return '';
-    }
-
-    if (typeof address === 'string') {
-      return address;
-    }
-
-    return `${address.name} <${address.address}>`;
-  }
-
-  private toResendAttachments(attachments: Mail.Options['attachments']) {
-    if (!attachments) {
-      return [];
-    }
-
-    return attachments.map((attachment) => {
-      if (!attachment.filename || !attachment.content) {
-        throw new Error('Attachment is missing filename or content');
-      }
-
-      if (typeof attachment.content === 'string') {
-        return {
-          filename: attachment.filename,
-          content: Buffer.from(attachment.content),
-        };
-      }
-
-      if (attachment.content instanceof Buffer) {
-        return {
-          filename: attachment.filename,
-          content: attachment.content,
-        };
-      }
-
-      throw new Error('Attachment content must be a string or a buffer');
-    });
-  }
-}

packages/email/tsconfig.json

@@ -1,10 +1,5 @@
 {
   "extends": "@documenso/tsconfig/react-library.json",
-  "compilerOptions": {
-    "types": [
-      "@documenso/tsconfig/process-env.d.ts",
-    ]
-  },
   "include": ["**/*.ts", "**/*.tsx", "**/*.d.ts", "**/*.json"],
   "exclude": ["dist", "build", "node_modules"]
 }

packages/lib/server-only/recipient/set-recipients-for-document.ts

@@ -57,12 +57,13 @@ export const setRecipientsForDocument = async ({
 
       return {
         ...recipient,
-        ...existing,
+        _persisted: existing,
       };
     })
     .filter((recipient) => {
       return (
-        recipient.sendStatus !== SendStatus.SENT && recipient.signingStatus !== SigningStatus.SIGNED
+        recipient._persisted?.sendStatus !== SendStatus.SENT &&
+        recipient._persisted?.signingStatus !== SigningStatus.SIGNED
       );
     });
 
@@ -72,7 +73,7 @@ export const setRecipientsForDocument = async ({
     linkedRecipients.map((recipient) =>
       prisma.recipient.upsert({
         where: {
-          id: recipient.id ?? -1,
+          id: recipient._persisted?.id ?? -1,
           documentId,
         },
         update: {

packages/tsconfig/process-env.d.ts

@@ -27,9 +27,7 @@ declare namespace NodeJS {
     NEXT_PRIVATE_SIGNING_LOCAL_FILE_CONTENTS?: string;
     NEXT_PRIVATE_SIGNING_LOCAL_FILE_ENCODING?: string;
 
-    NEXT_PRIVATE_SMTP_TRANSPORT?: 'mailchannels' | 'resend' | 'smtp-auth' | 'smtp-api';
+    NEXT_PRIVATE_SMTP_TRANSPORT?: 'mailchannels' | 'smtp-auth' | 'smtp-api';
-
-    NEXT_PRIVATE_RESEND_API_KEY?: string;
 
     NEXT_PRIVATE_MAILCHANNELS_API_KEY?: string;
     NEXT_PRIVATE_MAILCHANNELS_DKIM_DOMAIN?: string;

turbo.json

@@ -50,7 +50,6 @@
     "NEXT_PRIVATE_SIGNING_LOCAL_FILE_CONTENTS",
     "NEXT_PRIVATE_SIGNING_LOCAL_FILE_ENCODING",
     "NEXT_PRIVATE_SMTP_TRANSPORT",
-    "NEXT_PRIVATE_RESEND_API_KEY",
     "NEXT_PRIVATE_MAILCHANNELS_API_KEY",
     "NEXT_PRIVATE_MAILCHANNELS_ENDPOINT",
     "NEXT_PRIVATE_MAILCHANNELS_DKIM_DOMAIN",