fix: implement dynamic create DTO for Block

api/src/chat/dto/block.dto.ts

@@ -21,6 +21,7 @@ import {
   IsString,
 } from 'class-validator';
 
+import { DtoConfig } from '@/utils/types/dto.types';
 import { IsObjectId } from '@/utils/validation-rules/is-object-id';
 
 import { CaptureVar } from '../schemas/types/capture-var';
@@ -146,3 +147,7 @@ export class BlockUpdateDto extends PartialType(
   @IsOptional()
   trigger_channels?: string[];
 }
+
+export type BlockDTOMap = DtoConfig<{
+  create: BlockCreateDto;
+}>;

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

@@ -22,7 +22,7 @@ import { LoggerService } from '@/logger/logger.service';
 import { BaseRepository, DeleteResult } from '@/utils/generics/base-repository';
 import { TFilterQuery } from '@/utils/types/filter.types';
 
-import { BlockCreateDto, BlockUpdateDto } from '../dto/block.dto';
+import { BlockCreateDto, BlockDTOMap, BlockUpdateDto } from '../dto/block.dto';
 import {
   Block,
   BLOCK_POPULATE,
@@ -34,7 +34,8 @@ import {
 export class BlockRepository extends BaseRepository<
   Block,
   BlockPopulate,
-  BlockFull
+  BlockFull,
+  BlockDTOMap
 > {
   constructor(
     readonly eventEmitter: EventEmitter2,

api/src/chat/schemas/block.schema.ts

@@ -43,7 +43,7 @@ export class BlockStub extends BaseSchema {
     validate: isPatternList,
     default: [],
   })
-  patterns?: Pattern[];
+  patterns: Pattern[];
 
   @Prop([
     {
@@ -52,7 +52,7 @@ export class BlockStub extends BaseSchema {
       default: [],
     },
   ])
-  trigger_labels?: unknown;
+  trigger_labels: unknown;
 
   @Prop([
     {
@@ -61,19 +61,19 @@ export class BlockStub extends BaseSchema {
       default: [],
     },
   ])
-  assign_labels?: unknown;
+  assign_labels: unknown;
 
   @Prop({
     type: Object,
     default: [],
   })
-  trigger_channels?: string[];
+  trigger_channels: string[];
 
   @Prop({
     type: Object,
     default: {},
   })
-  options?: BlockOptions;
+  options: BlockOptions;
 
   @Prop({
     type: Object,
@@ -88,13 +88,13 @@ export class BlockStub extends BaseSchema {
       default: [],
     },
   ])
-  nextBlocks?: unknown;
+  nextBlocks: unknown;
 
   @Prop({
     type: MongooseSchema.Types.ObjectId,
     ref: 'Block',
   })
-  attachedBlock?: unknown;
+  attachedBlock: unknown;
 
   @Prop({
     type: MongooseSchema.Types.ObjectId,
@@ -106,14 +106,14 @@ export class BlockStub extends BaseSchema {
     type: Boolean,
     default: false,
   })
-  starts_conversation?: boolean;
+  starts_conversation: boolean;
 
   @Prop({
     type: Object,
     validate: isValidVarCapture,
     default: [],
   })
-  capture_vars?: CaptureVar[];
+  capture_vars: CaptureVar[];
 
   @Prop({
     type: Object,
@@ -125,22 +125,22 @@ export class BlockStub extends BaseSchema {
     type: Boolean,
     default: false,
   })
-  builtin?: boolean;
+  builtin: boolean;
 }
 
 @Schema({ timestamps: true })
 export class Block extends BlockStub {
-  @Transform(({ obj }) => obj.trigger_labels?.map((elem) => elem.toString()))
-  trigger_labels?: string[];
+  @Transform(({ obj }) => obj.trigger_labels.map((elem) => elem.toString()))
+  trigger_labels: string[];
 
-  @Transform(({ obj }) => obj.assign_labels?.map((elem) => elem.toString()))
-  assign_labels?: string[];
+  @Transform(({ obj }) => obj.assign_labels.map((elem) => elem.toString()))
+  assign_labels: string[];
 
-  @Transform(({ obj }) => obj.nextBlocks?.map((elem) => elem.toString()))
-  nextBlocks?: string[];
+  @Transform(({ obj }) => obj.nextBlocks.map((elem) => elem.toString()))
+  nextBlocks: string[];
 
   @Transform(({ obj }) => obj.attachedBlock?.toString() || null)
-  attachedBlock?: string;
+  attachedBlock: string;
 
   @Transform(({ obj }) => obj.category.toString())
   category: string;
@@ -161,16 +161,16 @@ export class BlockFull extends BlockStub {
   assign_labels: Label[];
 
   @Type(() => Block)
-  nextBlocks?: Block[];
+  nextBlocks: Block[];
 
   @Type(() => Block)
-  attachedBlock?: Block;
+  attachedBlock: Block;
 
   @Type(() => Category)
   category: Category;
 
   @Type(() => Block)
-  previousBlocks: Block[];
+  previousBlocks?: Block[];
 
   @Type(() => Block)
   attachedToBlock?: Block;

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

@@ -22,6 +22,7 @@ import { SettingService } from '@/setting/services/setting.service';
 import { BaseService } from '@/utils/generics/base-service';
 import { getRandom } from '@/utils/helpers/safeRandom';
 
+import { BlockDTOMap } from '../dto/block.dto';
 import { BlockRepository } from '../repositories/block.repository';
 import { Block, BlockFull, BlockPopulate } from '../schemas/block.schema';
 import { Context } from '../schemas/types/context';
@@ -35,7 +36,12 @@ import { Payload, StdQuickReply } from '../schemas/types/quick-reply';
 import { SubscriberContext } from '../schemas/types/subscriberContext';
 
 @Injectable()
-export class BlockService extends BaseService<Block, BlockPopulate, BlockFull> {
+export class BlockService extends BaseService<
+  Block,
+  BlockPopulate,
+  BlockFull,
+  BlockDTOMap
+> {
   constructor(
     readonly repository: BlockRepository,
     private readonly contentService: ContentService,

api/src/i18n/services/translation.service.spec.ts

@@ -154,6 +154,7 @@ describe('TranslationService', () => {
         },
       },
       options: {},
+      attachedBlock: null,
     };
 
     const mockedPlugin: any = {

api/src/utils/test/fixtures/block.ts

@@ -8,17 +8,35 @@
 
 import mongoose from 'mongoose';
 
-import { BlockCreateDto } from '@/chat/dto/block.dto';
-import { BlockModel, Block } from '@/chat/schemas/block.schema';
+import { Block, BlockModel } from '@/chat/schemas/block.schema';
 import { CategoryModel } from '@/chat/schemas/category.schema';
 import { FileType } from '@/chat/schemas/types/attachment';
 import { ButtonType } from '@/chat/schemas/types/button';
 import { QuickReplyType } from '@/chat/schemas/types/quick-reply';
+import { BaseSchema } from '@/utils/generics/base-schema';
 
 import { getFixturesWithDefaultValues } from '../defaultValues';
-import { TFixturesDefaultValues } from '../types';
 
-export const blocks: BlockCreateDto[] = [
+export const fieldsWithDefaultValues = {
+  options: {},
+  nextBlocks: [],
+  capture_vars: [],
+  assign_labels: [],
+  trigger_labels: [],
+  builtin: false,
+  starts_conversation: false,
+  attachedBlock: null,
+  attachedToBlock: null,
+} satisfies Partial<Block>;
+
+type TFieldWithDefaultValues =
+  | keyof typeof fieldsWithDefaultValues
+  | keyof BaseSchema;
+type TTransformedField<T> = Omit<T, TFieldWithDefaultValues> &
+  Partial<Pick<Block, TFieldWithDefaultValues>>;
+type TBlock = TTransformedField<Block>;
+
+export const blocks: TBlock[] = [
   {
     name: 'hasNextBlocks',
     patterns: ['Hi'],
@@ -163,20 +181,9 @@ export const blocks: BlockCreateDto[] = [
   },
 ];
 
-export const blockDefaultValues: TFixturesDefaultValues<Block> = {
-  options: {},
-  builtin: false,
-  nextBlocks: [],
-  capture_vars: [],
-  assign_labels: [],
-  trigger_labels: [],
-  starts_conversation: false,
-  attachedToBlock: null,
-};
-
-export const blockFixtures = getFixturesWithDefaultValues<Block>({
+export const blockFixtures = getFixturesWithDefaultValues<TBlock>({
   fixtures: blocks,
-  defaultValues: blockDefaultValues,
+  defaultValues: fieldsWithDefaultValues,
 });
 
 export const installBlockFixtures = async () => {

api/src/utils/test/mocks/block.ts

@@ -95,6 +95,7 @@ export const baseBlockInstance = {
   category: undefined,
   previousBlocks: [],
   trigger_channels: [],
+  nextBlocks: [],
   ...modelInstance,
 };
 
@@ -103,6 +104,7 @@ export const blockEmpty: BlockFull = {
   name: 'Empty',
   patterns: [],
   message: [''],
+  nextBlocks: [],
 };
 
 // Translation Data