fix(api): make optional DI required

api/src/attachment/services/attachment.service.ts

@@ -8,7 +8,7 @@
 
 import { Readable, Stream } from 'stream';
 
-import { Injectable, Optional, StreamableFile } from '@nestjs/common';
+import { Injectable, StreamableFile } from '@nestjs/common';
 
 import { HelperService } from '@/helper/helper.service';
 import { HelperType } from '@/helper/types';
@@ -22,7 +22,7 @@ import { Attachment } from '../schemas/attachment.schema';
 export class AttachmentService extends BaseService<Attachment> {
   constructor(
     readonly repository: AttachmentRepository,
-    @Optional() private readonly helperService: HelperService,
+    private readonly helperService: HelperService,
   ) {
     super(repository);
   }

api/src/chat/repositories/category.repository.ts

@@ -6,7 +6,7 @@
  * 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
  */
 
-import { ForbiddenException, Injectable, Optional } from '@nestjs/common';
+import { ForbiddenException, Injectable } from '@nestjs/common';
 import { InjectModel } from '@nestjs/mongoose';
 import { Document, Model, Query } from 'mongoose';
 
@@ -26,7 +26,7 @@ export class CategoryRepository extends BaseRepository<
 > {
   constructor(
     @InjectModel(Category.name) readonly model: Model<Category>,
-    @Optional() private readonly blockService?: BlockService,
+    private readonly blockService: BlockService,
   ) {
     super(model, Category);
   }
@@ -49,7 +49,7 @@ export class CategoryRepository extends BaseRepository<
     criteria: TFilterQuery<Category>,
   ) {
     if (criteria._id) {
-      const block = await this.blockService?.findOneAndPopulate({
+      const block = await this.blockService.findOneAndPopulate({
         category: criteria._id,
       });
 

api/src/chat/repositories/context-var.repository.ts

@@ -10,7 +10,6 @@ import {
   ForbiddenException,
   Injectable,
   NotFoundException,
-  Optional,
 } from '@nestjs/common';
 import { InjectModel } from '@nestjs/mongoose';
 import { Document, Model, Query } from 'mongoose';
@@ -29,14 +28,11 @@ export class ContextVarRepository extends BaseRepository<
   never,
   ContextVarDto
 > {
-  private readonly blockService: BlockService;
-
   constructor(
     @InjectModel(ContextVar.name) readonly model: Model<ContextVar>,
-    @Optional() blockService?: BlockService,
+    private readonly blockService: BlockService,
   ) {
     super(model, ContextVar);
-    if (blockService) this.blockService = blockService;
   }
 
   /**
@@ -65,7 +61,7 @@ export class ContextVarRepository extends BaseRepository<
         throw new NotFoundException(`Context var with ID ${id} not found.`);
       }
 
-      const associatedBlocks = await this.blockService?.find({
+      const associatedBlocks = await this.blockService.find({
         capture_vars: { $elemMatch: { context_var: contextVar.name } },
       });
 

api/src/chat/services/message.service.ts

@@ -6,11 +6,7 @@
  * 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
  */
 
-import {
+import { Injectable, InternalServerErrorException } from '@nestjs/common';
-  Injectable,
-  InternalServerErrorException,
-  Optional,
-} from '@nestjs/common';
 
 import { BaseService } from '@/utils/generics/base-service';
 import {
@@ -36,14 +32,11 @@ export class MessageService extends BaseService<
   MessagePopulate,
   MessageFull
 > {
-  private readonly gateway: WebsocketGateway;
-
   constructor(
     private readonly messageRepository: MessageRepository,
-    @Optional() gateway?: WebsocketGateway,
+    private readonly gateway: WebsocketGateway,
   ) {
     super(messageRepository);
-    if (gateway) this.gateway = gateway;
   }
 
   /**

api/src/chat/services/subscriber.service.ts

@@ -6,11 +6,7 @@
  * 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
  */
 
-import {
+import { Injectable, InternalServerErrorException } from '@nestjs/common';
-  Injectable,
-  InternalServerErrorException,
-  Optional,
-} from '@nestjs/common';
 import { OnEvent } from '@nestjs/event-emitter';
 import mime from 'mime';
 import { v4 as uuidv4 } from 'uuid';
@@ -53,15 +49,12 @@ export class SubscriberService extends BaseService<
   SubscriberFull,
   SubscriberDto
 > {
-  private readonly gateway: WebsocketGateway;
-
   constructor(
     readonly repository: SubscriberRepository,
     protected readonly attachmentService: AttachmentService,
-    @Optional() gateway?: WebsocketGateway,
+    protected readonly gateway: WebsocketGateway,
   ) {
     super(repository);
-    if (gateway) this.gateway = gateway;
   }
 
   /**

api/src/cms/repositories/content-type.repository.ts

@@ -6,7 +6,7 @@
  * 2. All derivative works must include clear attribution to the original creator and software, Hexastack and Hexabot, in a prominent location (e.g., in the software's "About" section, documentation, and README file).
  */
 
-import { ForbiddenException, Injectable, Optional } from '@nestjs/common';
+import { ForbiddenException, Injectable } from '@nestjs/common';
 import { InjectModel } from '@nestjs/mongoose';
 import { Document, Model, Query } from 'mongoose';
 
@@ -28,7 +28,7 @@ export class ContentTypeRepository extends BaseRepository<
   constructor(
     @InjectModel(ContentType.name) readonly model: Model<ContentType>,
     @InjectModel(Content.name) private readonly contentModel: Model<Content>,
-    @Optional() private readonly blockService?: BlockService,
+    private readonly blockService: BlockService,
   ) {
     super(model, ContentType);
   }
@@ -53,7 +53,7 @@ export class ContentTypeRepository extends BaseRepository<
     criteria: TFilterQuery<ContentType>,
   ) {
     if (criteria._id) {
-      const associatedBlock = await this.blockService?.findOne({
+      const associatedBlock = await this.blockService.findOne({
         'options.content.entity': criteria._id,
       });
       if (associatedBlock) {

api/src/user/services/invitation.service.ts

@@ -12,7 +12,6 @@ import {
   Inject,
   Injectable,
   InternalServerErrorException,
-  Optional,
 } from '@nestjs/common';
 import { JwtService, JwtSignOptions } from '@nestjs/jwt';
 
@@ -41,7 +40,7 @@ export class InvitationService extends BaseService<
     @Inject(JwtService) private readonly jwtService: JwtService,
     protected readonly i18n: I18nService,
     public readonly languageService: LanguageService,
-    @Optional() private readonly mailerService?: MailerService,
+    private readonly mailerService: MailerService,
   ) {
     super(repository);
   }
@@ -61,7 +60,6 @@ export class InvitationService extends BaseService<
    */
   async create(dto: InvitationCreateDto): Promise<Invitation> {
     const jwt = await this.sign({ ...dto });
-    if (this.mailerService) {
     try {
       const defaultLanguage = await this.languageService.getDefaultLanguage();
       await this.mailerService.sendMail({
@@ -72,8 +70,7 @@ export class InvitationService extends BaseService<
           appUrl: config.uiBaseUrl,
           token: jwt,
           // TODO: Which language should we use?
-            t: (key: string) =>
+          t: (key: string) => this.i18n.t(key, { lang: defaultLanguage.code }),
-              this.i18n.t(key, { lang: defaultLanguage.code }),
         },
         subject: this.i18n.t('invitation_subject'),
       });
@@ -86,7 +83,6 @@ export class InvitationService extends BaseService<
       );
       throw new InternalServerErrorException('Could not send email');
     }
-    }
     const newInvitation = await super.create({ ...dto, token: jwt });
     return { ...newInvitation, token: jwt };
   }

api/src/user/services/passwordReset.service.ts

@@ -14,7 +14,6 @@ import {
   Injectable,
   InternalServerErrorException,
   NotFoundException,
-  Optional,
   UnauthorizedException,
 } from '@nestjs/common';
 import { JwtService, JwtSignOptions } from '@nestjs/jwt';
@@ -37,7 +36,7 @@ export class PasswordResetService {
     private readonly userService: UserService,
     public readonly i18n: I18nService,
     public readonly languageService: LanguageService,
-    @Optional() private readonly mailerService?: MailerService,
+    private readonly mailerService: MailerService,
   ) {}
 
   public readonly jwtSignOptions: JwtSignOptions = {
@@ -60,7 +59,6 @@ export class PasswordResetService {
     }
     const jwt = await this.sign({ ...dto });
 
-    if (this.mailerService) {
     try {
       const defaultLanguage = await this.languageService.getDefaultLanguage();
       await this.mailerService.sendMail({
@@ -71,8 +69,7 @@ export class PasswordResetService {
           appUrl: config.uiBaseUrl,
           token: jwt,
           first_name: user.first_name,
-            t: (key: string) =>
+          t: (key: string) => this.i18n.t(key, { lang: defaultLanguage.code }),
-              this.i18n.t(key, { lang: defaultLanguage.code }),
         },
         subject: this.i18n.t('password_reset_subject'),
       });
@@ -85,7 +82,6 @@ export class PasswordResetService {
       );
       throw new InternalServerErrorException('Could not send email');
     }
-    }
 
     // TODO: hash the token before saving it
     await this.userService.updateOne({ email: dto.email }, { resetToken: jwt });

api/src/user/services/validate-account.service.ts

@@ -12,7 +12,6 @@ import {
   Inject,
   Injectable,
   InternalServerErrorException,
-  Optional,
   UnauthorizedException,
 } from '@nestjs/common';
 import { JwtService, JwtSignOptions } from '@nestjs/jwt';
@@ -40,7 +39,7 @@ export class ValidateAccountService {
     private logger: LoggerService,
     private readonly i18n: I18nService,
     private readonly languageService: LanguageService,
-    @Optional() private readonly mailerService?: MailerService,
+    private readonly mailerService: MailerService,
   ) {}
 
   /**
@@ -77,7 +76,6 @@ export class ValidateAccountService {
   ) {
     const confirmationToken = await this.sign({ email: dto.email });
 
-    if (this.mailerService) {
     try {
       const defaultLanguage = await this.languageService.getDefaultLanguage();
       await this.mailerService.sendMail({
@@ -88,8 +86,7 @@ export class ValidateAccountService {
           appUrl: config.uiBaseUrl,
           token: confirmationToken,
           first_name: dto.first_name,
-            t: (key: string) =>
+          t: (key: string) => this.i18n.t(key, { lang: defaultLanguage.code }),
-              this.i18n.t(key, { lang: defaultLanguage.code }),
         },
         subject: this.i18n.t('account_confirmation_subject'),
       });
@@ -103,7 +100,6 @@ export class ValidateAccountService {
       throw new InternalServerErrorException('Could not send email');
     }
   }
-  }
 
   /**
    * Confirms a user's account by validating the provided confirmation token.