@flusys/nestjs-email 3.0.0 → 4.0.0-rc

package/fesm/providers/sendgrid-provider.js

@@ -11,7 +11,8 @@ function _define_property(obj, key, value) {
     }
     return obj;
 }
-import { Logger } from '@nestjs/common';
+import { InternalServerErrorException } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
 export class SendGridProvider {
     async initialize(config) {
         this.apiKey = config.apiKey;
@@ -20,10 +21,11 @@ export class SendGridProvider {
             const sgMailModule = await new Function('return import("@sendgrid/mail")')();
             this.sgMail = sgMailModule.default || sgMailModule;
             this.sgMail.setApiKey(config.apiKey);
-            this.logger.log('SendGrid Provider initialized');
-        } catch (error) {
-            this.logger.error('Failed to initialize SendGrid:', this.extractError(error));
-            throw new Error('SendGrid initialization failed. Install: npm install @sendgrid/mail');
+        } catch  {
+            throw new InternalServerErrorException({
+                message: 'SendGrid initialization failed. Install: npm install @sendgrid/mail',
+                messageKey: SYSTEM_MESSAGES.SDK_NOT_INSTALLED
+            });
         }
     }
     async sendEmail(options) {
@@ -35,13 +37,15 @@ export class SendGridProvider {
             const msg = this.buildMessage(options, true);
             const [response] = await this.sgMail.send(msg);
             const messageId = response.headers['x-message-id'] || response.headers['X-Message-Id'];
-            this.logger.log(`Email sent via SendGrid: ${messageId}`);
             return {
                 success: true,
                 messageId
             };
         } catch (error) {
-            return this.handleError('SendGrid send failed', error);
+            return {
+                success: false,
+                error: this.extractError(error)
+            };
         }
     }
     async sendBulkEmails(options) {
@@ -56,7 +60,6 @@ export class SendGridProvider {
                     success: true
                 }));
         } catch (error) {
-            this.logger.error('SendGrid bulk send failed:', error);
             const errorMessage = this.extractError(error);
             return options.map(()=>({
                     success: false,
@@ -106,16 +109,7 @@ export class SendGridProvider {
         const sgError = error;
         return sgError.response?.body?.errors?.[0]?.message || sgError.message || 'Unknown error';
     }
-    handleError(context, error) {
-        const message = this.extractError(error);
-        this.logger.error(`${context}: ${message}`, error);
-        return {
-            success: false,
-            error: message
-        };
-    }
     constructor(){
-        _define_property(this, "logger", new Logger(SendGridProvider.name));
         _define_property(this, "sgMail", null);
         _define_property(this, "apiKey", null);
     }

package/fesm/providers/smtp-provider.js

@@ -11,12 +11,10 @@ function _define_property(obj, key, value) {
     }
     return obj;
 }
-import { Logger } from '@nestjs/common';
 const VERIFY_TIMEOUT_MS = 10_000;
 const SEND_TIMEOUT_MS = 30_000;
 export class SmtpProvider {
     async initialize(config) {
-        this.logger.log(`Initializing SMTP: host=${config.host}, port=${config.port}`);
         const nodemailer = await import('nodemailer');
         this.transporter = nodemailer.createTransport({
             host: config.host,
@@ -31,7 +29,7 @@ export class SmtpProvider {
                 minVersion: config.tls?.minVersion ?? 'TLSv1.2'
             }
         });
-        await this.verifyConnection(config);
+        await this.verifyConnection();
     }
     async sendEmail(options) {
         if (!this.transporter) return {
@@ -56,13 +54,15 @@ export class SmtpProvider {
                     }))
             };
             const info = await this.withTimeout(this.transporter.sendMail(mailOptions), SEND_TIMEOUT_MS, 'SMTP send timeout');
-            this.logger.log(`Email sent via SMTP: ${info.messageId}`);
             return {
                 success: true,
                 messageId: info.messageId
             };
         } catch (error) {
-            return this.handleError('SMTP send failed', error);
+            return {
+                success: false,
+                error: this.extractError(error)
+            };
         }
     }
     async sendBulkEmails(options) {
@@ -84,13 +84,8 @@ export class SmtpProvider {
         }
     }
     // ─── Private Helpers ────────────────────────────────────────────────────────
-    async verifyConnection(config) {
-        try {
-            await this.withTimeout(this.transporter.verify(), VERIFY_TIMEOUT_MS, 'SMTP verification timeout');
-            this.logger.log(`SMTP Provider verified: ${config.host}:${config.port}`);
-        } catch (error) {
-            this.logger.warn(`SMTP verification failed: ${this.extractError(error)}`);
-        }
+    async verifyConnection() {
+        await this.withTimeout(this.transporter.verify(), VERIFY_TIMEOUT_MS, 'SMTP verification timeout');
     }
     formatFrom(from, fromName) {
         return fromName ? `"${fromName}" <${from}>` : from;
@@ -108,16 +103,7 @@ export class SmtpProvider {
     extractError(error) {
         return error instanceof Error ? error.message : 'Unknown error';
     }
-    handleError(context, error) {
-        const message = this.extractError(error);
-        this.logger.error(`${context}: ${message}`, error instanceof Error ? error.stack : undefined);
-        return {
-            success: false,
-            error: message
-        };
-    }
     constructor(){
-        _define_property(this, "logger", new Logger(SmtpProvider.name));
         _define_property(this, "transporter", null);
     }
 }

package/fesm/services/email-datasource.provider.js

@@ -26,7 +26,8 @@ function _ts_param(paramIndex, decorator) {
     };
 }
 import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
-import { Inject, Injectable, InternalServerErrorException, Logger, Optional, Scope } from '@nestjs/common';
+import { SYSTEM_MESSAGES } from '@flusys/nestjs-shared/constants';
+import { Inject, Injectable, InternalServerErrorException, Optional, Scope } from '@nestjs/common';
 import { REQUEST } from '@nestjs/core';
 import { Request } from 'express';
 import { getEmailEntitiesByConfig } from '../entities';
@@ -61,7 +62,10 @@ export class EmailDataSourceProvider extends MultiTenantDataSourceService {
         }
         const config = this.getDefaultDatabaseConfig();
         if (!config) {
-            throw new InternalServerErrorException('No database config available. Provide defaultDatabaseConfig or tenantDefaultDatabaseConfig.');
+            throw new InternalServerErrorException({
+                message: 'No database config available. Provide defaultDatabaseConfig or tenantDefaultDatabaseConfig.',
+                messageKey: SYSTEM_MESSAGES.DATABASE_CONFIG_NOT_AVAILABLE
+            });
         }
         const connectionPromise = this.createDataSourceFromConfig(config);
         EmailDataSourceProvider.singleConnectionLock = connectionPromise;
@@ -94,7 +98,7 @@ export class EmailDataSourceProvider extends MultiTenantDataSourceService {
         }
     }
     constructor(configService, request){
-        super(EmailDataSourceProvider.buildParentOptions(configService.getOptions()), request), _define_property(this, "configService", void 0), _define_property(this, "logger", void 0), this.configService = configService, this.logger = new Logger(EmailDataSourceProvider.name);
+        super(EmailDataSourceProvider.buildParentOptions(configService.getOptions()), request), _define_property(this, "configService", void 0), this.configService = configService;
     }
 }
 _define_property(EmailDataSourceProvider, "tenantConnections", new Map());

package/fesm/services/email-provider-config.service.js

@@ -119,7 +119,7 @@ export class EmailProviderConfigService extends RequestScopedApiService {
         });
     }
     constructor(cacheManager, utilsService, emailConfig, dataSourceProvider){
-        super('emailConfig', null, cacheManager, utilsService, EmailProviderConfigService.name, true), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "emailConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.emailConfig = emailConfig, this.dataSourceProvider = dataSourceProvider;
+        super('emailConfig', null, cacheManager, utilsService, EmailProviderConfigService.name, true, 'email'), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "emailConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.emailConfig = emailConfig, this.dataSourceProvider = dataSourceProvider;
     }
 }
 EmailProviderConfigService = _ts_decorate([

package/fesm/services/email-send.service.js

@@ -25,9 +25,13 @@ function _ts_param(paramIndex, decorator) {
         decorator(target, key, paramIndex);
     };
 }
+import { LogAction } from '@flusys/nestjs-shared';
+import { EMAIL_SEND_MESSAGES } from '../config';
+import { ILoggedUserInfo } from '@flusys/nestjs-shared/interfaces';
 import { escapeHtmlVariables, validateCompanyOwnership } from '@flusys/nestjs-shared/utils';
-import { BadRequestException, Inject, Injectable, Logger, NotFoundException, Scope } from '@nestjs/common';
+import { BadRequestException, Inject, Injectable, NotFoundException, Scope } from '@nestjs/common';
 import { EmailConfigService } from './email-config.service';
+import { SendEmailDto, SendTemplateEmailDto } from '../dtos';
 import { EmailFactoryService } from '../providers';
 import { EmailProviderConfigService } from './email-provider-config.service';
 import { EmailTemplateService } from './email-template.service';
@@ -37,7 +41,7 @@ export class EmailSendService {
     async sendEmail(dto, user) {
         const { provider, config } = await this.getEmailProviderWithConfig(dto.emailConfigId, user);
         const sender = this.resolveSender(dto, config);
-        const result = await provider.sendEmail({
+        return await provider.sendEmail({
             to: dto.to,
             cc: dto.cc,
             bcc: dto.bcc,
@@ -48,15 +52,13 @@ export class EmailSendService {
             replyTo: dto.replyTo,
             attachments: this.buildAttachments(dto.attachments)
         });
-        this.logResult(dto.to, result);
-        return result;
     }
     async sendTemplateEmail(dto, user) {
         const template = await this.resolveTemplate(dto, user);
         const { subject, html, text } = this.buildTemplateContent(template, dto.variables);
         const { provider, config } = await this.getEmailProviderWithConfig(dto.emailConfigId, user);
         const sender = this.resolveSender(dto, config);
-        const result = await provider.sendEmail({
+        return await provider.sendEmail({
             to: dto.to,
             cc: dto.cc,
             bcc: dto.bcc,
@@ -67,8 +69,6 @@ export class EmailSendService {
             replyTo: dto.replyTo,
             attachments: this.buildAttachments(dto.attachments)
         });
-        this.logResult(dto.to, result, template.slug);
-        return result;
     }
     async sendTestEmail(emailConfigId, recipient, user) {
         const { provider, config } = await this.getEmailProviderWithConfig(emailConfigId, user);
@@ -84,7 +84,6 @@ export class EmailSendService {
             from: config.fromEmail || undefined,
             fromName: config.fromName || this.emailConfigService.getDefaultFromName()
         });
-        this.logResult(recipient, result);
         return result;
     }
     // ─── Private: Provider & Config Resolution ──────────────────────────────────
@@ -101,14 +100,29 @@ export class EmailSendService {
     }
     async resolveConfigById(id, user) {
         const config = await this.emailProviderConfigService.findByIdDirect(id);
-        if (!config) throw new NotFoundException('Email configuration not found');
+        if (!config) {
+            throw new NotFoundException({
+                message: 'Email configuration not found',
+                messageKey: EMAIL_SEND_MESSAGES.CONFIG_NOT_FOUND
+            });
+        }
         validateCompanyOwnership(config, user, this.emailConfigService.isCompanyFeatureEnabled(), 'Email configuration');
-        if (!config.isActive) throw new BadRequestException('Email configuration is inactive');
+        if (!config.isActive) {
+            throw new BadRequestException({
+                message: 'Email configuration is inactive',
+                messageKey: EMAIL_SEND_MESSAGES.CONFIG_INACTIVE
+            });
+        }
         return config;
     }
     async resolveDefaultConfig(user) {
         const config = await this.emailProviderConfigService.getDefaultConfig(user);
-        if (!config) throw new NotFoundException('No default email configuration found');
+        if (!config) {
+            throw new NotFoundException({
+                message: 'No default email configuration found',
+                messageKey: EMAIL_SEND_MESSAGES.CONFIG_DEFAULT_NOT_FOUND
+            });
+        }
         return config;
     }
     // ─── Private: Template Resolution ───────────────────────────────────────────
@@ -119,10 +133,23 @@ export class EmailSendService {
         } else if (dto.templateSlug) {
             template = await this.emailTemplateService.findBySlug(dto.templateSlug, user);
         } else {
-            throw new BadRequestException('templateId or templateSlug is required');
+            throw new BadRequestException({
+                message: 'templateId or templateSlug is required',
+                messageKey: EMAIL_SEND_MESSAGES.TEMPLATE_ID_OR_SLUG_REQUIRED
+            });
+        }
+        if (!template) {
+            throw new NotFoundException({
+                message: 'Email template not found',
+                messageKey: EMAIL_SEND_MESSAGES.TEMPLATE_NOT_FOUND
+            });
+        }
+        if (!template.isActive) {
+            throw new BadRequestException({
+                message: 'Email template is inactive',
+                messageKey: EMAIL_SEND_MESSAGES.TEMPLATE_INACTIVE
+            });
         }
-        if (!template) throw new NotFoundException('Email template not found');
-        if (!template.isActive) throw new BadRequestException('Email template is inactive');
         validateCompanyOwnership(template, user, this.emailConfigService.isCompanyFeatureEnabled(), 'Email template');
         return template;
     }
@@ -163,12 +190,6 @@ export class EmailSendService {
             return varName in safeVars && safeVars[varName] !== undefined ? String(safeVars[varName]) : match;
         });
     }
-    logResult(to, result, templateSlug) {
-        const recipient = Array.isArray(to) ? to.join(', ') : to;
-        const status = result.success ? 'SUCCESS' : 'FAILED';
-        const message = templateSlug ? `Template email sent to ${recipient} using template "${templateSlug}": ${status}` : `Email sent to ${recipient}: ${status}`;
-        this.logger.log(message);
-    }
     buildTestEmailHtml(safeConfig) {
         return `
       <div style="font-family: Arial, sans-serif; max-width: 600px; margin: 0 auto;">
@@ -187,14 +208,49 @@ export class EmailSendService {
         _define_property(this, "emailConfigService", void 0);
         _define_property(this, "emailProviderConfigService", void 0);
         _define_property(this, "emailTemplateService", void 0);
-        _define_property(this, "logger", void 0);
         this.emailFactory = emailFactory;
         this.emailConfigService = emailConfigService;
         this.emailProviderConfigService = emailProviderConfigService;
         this.emailTemplateService = emailTemplateService;
-        this.logger = new Logger(EmailSendService.name);
     }
 }
+_ts_decorate([
+    LogAction({
+        action: 'email.send',
+        module: 'email'
+    }),
+    _ts_metadata("design:type", Function),
+    _ts_metadata("design:paramtypes", [
+        typeof SendEmailDto === "undefined" ? Object : SendEmailDto,
+        typeof ILoggedUserInfo === "undefined" ? Object : ILoggedUserInfo
+    ]),
+    _ts_metadata("design:returntype", Promise)
+], EmailSendService.prototype, "sendEmail", null);
+_ts_decorate([
+    LogAction({
+        action: 'email.sendTemplate',
+        module: 'email'
+    }),
+    _ts_metadata("design:type", Function),
+    _ts_metadata("design:paramtypes", [
+        typeof SendTemplateEmailDto === "undefined" ? Object : SendTemplateEmailDto,
+        typeof ILoggedUserInfo === "undefined" ? Object : ILoggedUserInfo
+    ]),
+    _ts_metadata("design:returntype", Promise)
+], EmailSendService.prototype, "sendTemplateEmail", null);
+_ts_decorate([
+    LogAction({
+        action: 'email.sendTest',
+        module: 'email'
+    }),
+    _ts_metadata("design:type", Function),
+    _ts_metadata("design:paramtypes", [
+        String,
+        String,
+        typeof ILoggedUserInfo === "undefined" ? Object : ILoggedUserInfo
+    ]),
+    _ts_metadata("design:returntype", Promise)
+], EmailSendService.prototype, "sendTestEmail", null);
 EmailSendService = _ts_decorate([
     Injectable({
         scope: Scope.REQUEST

package/fesm/services/email-template.service.js

@@ -128,11 +128,10 @@ export class EmailTemplateService extends RequestScopedApiService {
         });
         if (existing && JSON.stringify(dto.schema) !== JSON.stringify(existing.schema)) {
             entity.schemaVersion = existing.schemaVersion + 1;
-            this.logger.log(`Schema version incremented to ${entity.schemaVersion} for template ${updateDto.id}`);
         }
     }
     constructor(cacheManager, utilsService, emailConfig, dataSourceProvider){
-        super('emailTemplate', null, cacheManager, utilsService, EmailTemplateService.name, true), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "emailConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.emailConfig = emailConfig, this.dataSourceProvider = dataSourceProvider;
+        super('emailTemplate', null, cacheManager, utilsService, EmailTemplateService.name, true, 'email'), _define_property(this, "cacheManager", void 0), _define_property(this, "utilsService", void 0), _define_property(this, "emailConfig", void 0), _define_property(this, "dataSourceProvider", void 0), this.cacheManager = cacheManager, this.utilsService = utilsService, this.emailConfig = emailConfig, this.dataSourceProvider = dataSourceProvider;
     }
 }
 EmailTemplateService = _ts_decorate([

package/interfaces/email-config.interface.d.ts

@@ -3,7 +3,7 @@ import { EmailProviderTypeEnum } from '../enums';
 export interface IEmailConfig extends IIdentity {
     name: string;
     provider: EmailProviderTypeEnum;
-    config: Record<string, any>;
+    config: Record<string, unknown>;
     fromEmail: string | null;
     fromName: string | null;
     isActive: boolean;

package/interfaces/email-provider.interface.d.ts

@@ -1,6 +1,6 @@
 export interface IEmailProviderConfig {
     provider: string;
-    config: Record<string, any>;
+    config: Record<string, unknown>;
 }
 export interface IEmailAttachment {
     filename: string;
@@ -29,6 +29,6 @@ export interface IEmailProvider {
     sendEmail(options: IEmailSendOptions): Promise<IEmailSendResult>;
     sendBulkEmails(options: IEmailSendOptions[]): Promise<IEmailSendResult[]>;
     healthCheck(): Promise<boolean>;
-    initialize?(config: any): Promise<void>;
+    initialize?(config: unknown): Promise<void>;
     close?(): Promise<void>;
 }

package/interfaces/email-template.interface.d.ts

@@ -37,7 +37,7 @@ export interface IEmailBlock {
     id: string;
     type: 'text' | 'image' | 'button' | 'divider' | 'html';
     order: number;
-    content: Record<string, any>;
+    content: Record<string, unknown>;
     style?: IEmailBlockStyle;
 }
 export interface IEmailBlockStyle {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@flusys/nestjs-email",
-  "version": "3.0.0",
+  "version": "4.0.0-rc",
   "description": "Modular email package with SMTP, SendGrid, and Mailgun providers",
   "main": "cjs/index.js",
   "module": "fesm/index.js",
@@ -104,7 +104,7 @@
     }
   },
   "dependencies": {
-    "@flusys/nestjs-core": "3.0.0",
-    "@flusys/nestjs-shared": "3.0.0"
+    "@flusys/nestjs-core": "4.0.0-rc",
+    "@flusys/nestjs-shared": "4.0.0-rc"
   }
 }

package/providers/email-factory.service.d.ts

@@ -1,7 +1,6 @@
 import { OnModuleDestroy } from '@nestjs/common';
 import { IEmailProvider, IEmailProviderConfig } from '../interfaces';
 export declare class EmailFactoryService implements OnModuleDestroy {
-    private readonly logger;
     private readonly providerCache;
     createProvider(config: IEmailProviderConfig): Promise<IEmailProvider>;
     onModuleDestroy(): Promise<void>;

package/providers/mailgun-provider.d.ts

@@ -1,6 +1,5 @@
 import { IEmailProvider, IEmailSendOptions, IEmailSendResult, IMailgunConfig } from '../interfaces';
 export declare class MailgunProvider implements IEmailProvider {
-    private readonly logger;
     private client;
     private domain;
     initialize(config: IMailgunConfig): Promise<void>;
@@ -11,5 +10,4 @@ export declare class MailgunProvider implements IEmailProvider {
     private buildMessageData;
     private joinAddresses;
     private extractError;
-    private handleError;
 }

package/providers/sendgrid-provider.d.ts

@@ -1,6 +1,5 @@
 import { IEmailProvider, IEmailSendOptions, IEmailSendResult, ISendGridConfig } from '../interfaces';
 export declare class SendGridProvider implements IEmailProvider {
-    private readonly logger;
     private sgMail;
     private apiKey;
     initialize(config: ISendGridConfig): Promise<void>;
@@ -11,5 +10,4 @@ export declare class SendGridProvider implements IEmailProvider {
     private buildMessage;
     private toArray;
     private extractError;
-    private handleError;
 }

package/providers/smtp-provider.d.ts

@@ -1,6 +1,5 @@
 import { IEmailProvider, IEmailSendOptions, IEmailSendResult, ISmtpConfig } from '../interfaces';
 export declare class SmtpProvider implements IEmailProvider {
-    private readonly logger;
     private transporter;
     initialize(config: ISmtpConfig): Promise<void>;
     sendEmail(options: IEmailSendOptions): Promise<IEmailSendResult>;
@@ -12,5 +11,4 @@ export declare class SmtpProvider implements IEmailProvider {
     private joinAddresses;
     private withTimeout;
     private extractError;
-    private handleError;
 }

package/services/email-datasource.provider.d.ts

@@ -1,12 +1,10 @@
 import { MultiTenantDataSourceService } from '@flusys/nestjs-shared/modules';
 import { IDatabaseConfig, ITenantDatabaseConfig } from '@flusys/nestjs-core';
-import { Logger } from '@nestjs/common';
 import { Request } from 'express';
 import { DataSource } from 'typeorm';
 import { EmailConfigService } from './email-config.service';
 export declare class EmailDataSourceProvider extends MultiTenantDataSourceService {
     private readonly configService;
-    protected readonly logger: Logger;
     protected static readonly tenantConnections: Map<string, DataSource>;
     protected static singleDataSource: DataSource | null;
     protected static readonly tenantsRegistry: Map<string, ITenantDatabaseConfig>;

package/services/email-send.service.d.ts

@@ -10,7 +10,6 @@ export declare class EmailSendService {
     private readonly emailConfigService;
     private readonly emailProviderConfigService;
     private readonly emailTemplateService;
-    private readonly logger;
     constructor(emailFactory: EmailFactoryService, emailConfigService: EmailConfigService, emailProviderConfigService: EmailProviderConfigService, emailTemplateService: EmailTemplateService);
     sendEmail(dto: SendEmailDto, user?: ILoggedUserInfo): Promise<IEmailSendResult>;
     sendTemplateEmail(dto: SendTemplateEmailDto, user?: ILoggedUserInfo): Promise<IEmailSendResult>;
@@ -23,6 +22,5 @@ export declare class EmailSendService {
     private resolveSender;
     private buildAttachments;
     private interpolateVariables;
-    private logResult;
     private buildTestEmailHtml;
 }