@fiado/type-kit 3.418.0 → 3.420.0

package/_test_/unit/messagesConnector/IsTemplatePlaceholders.test.ts

@@ -0,0 +1,43 @@
+import { IsTemplatePlaceholders } from '../../../src/messagesConnector/validators/IsTemplatePlaceholders';
+
+const validador = new IsTemplatePlaceholders();
+const acepta = (texto: unknown): boolean => validador.validate(texto);
+
+describe('IsTemplatePlaceholders', () => {
+    describe('acepta las dos formas que el renderizador sustituye', () => {
+        it.each([
+            ['sin placeholders', 'Hola, tu credito esta al corriente'],
+            ['forma simple', 'Hola {cliente_nombre}, tu cuota es {monto}'],
+            ['forma doble', 'Hola {{cliente_nombre}}, tu cuota es {{monto}}'],
+            ['las dos mezcladas', 'Hola {{cliente_nombre}}, vence el {fecha_venc}'],
+            ['nombre con digitos y guion bajo', 'Codigo {var1} y {otra_var2}'],
+            ['cadena vacia', ''],
+        ])('%s', (_caso, texto) => {
+            expect(acepta(texto)).toBe(true);
+        });
+    });
+
+    describe('rechaza lo que el renderizador no podria sustituir', () => {
+        it.each([
+            ['placeholder vacio simple', '{}'],
+            ['placeholder vacio doble', '{{}}'],
+            ['nombre con espacio', '{var name}'],
+            ['nombre con guion', '{var-1}'],
+            ['nombre que arranca con digito', '{1var}'],
+            ['apertura sin cierre', 'abc {def'],
+            ['cierre sin apertura', 'abc } def'],
+            ['llave suelta antes de uno valido', '{ {anidado}'],
+            ['doble sin cerrar del todo', '{{a}'],
+            ['no es string', 42],
+            ['nulo', null],
+        ])('%s', (_caso, texto) => {
+            expect(acepta(texto)).toBe(false);
+        });
+    });
+
+    it('el mensaje nombra las dos formas, para que el front sepa cual usar', () => {
+        const mensaje = validador.defaultMessage({ property: 'template' } as never);
+        expect(mensaje).toContain('{nombre}');
+        expect(mensaje).toContain('{{nombre}}');
+    });
+});

package/_test_/unit/messagesConnector/NotificationQueueMessageRequestV2.test.ts

@@ -4,6 +4,7 @@ import { validate } from 'class-validator';
 import { NotificationQueueMessageRequestV2 } from '../../../src/messagesConnector/dtos/NotificationQueueMessageRequestV2';
 import { NotificationOriginEnum } from '../../../src/messagesConnector/enums/NotificationOriginEnum';
 import { DeliveryChannelExtendedEnum } from '../../../src/messagesConnector/enums/DeliveryChannelExtendedEnum';
+import { MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH } from '../../../src/messagesConnector/constants/MessageIdempotencyLimits';
 
 const base = {
     templateId: 'OTP_CODE',
@@ -48,7 +49,7 @@ describe('NotificationQueueMessageRequestV2.origin', () => {
     });
 });
 
-describe('referenceMessageId — la llave de idempotencia de la cola', () => {
+describe('idempotencyKey — la llave de deduplicacion, aparte de la correlacion', () => {
     const validQueueMessage = {
         templateId: 'COLLECTION_PRE_DUE_2_DAYS',
         channelType: DeliveryChannelExtendedEnum.SMS,
@@ -59,19 +60,35 @@ describe('referenceMessageId — la llave de idempotencia de la cola', () => {
         destination: '+525512345678',
         variables: { name: 'Ana' },
     };
+    const build = (over: Record<string, unknown> = {}) =>
+        plainToInstance(NotificationQueueMessageRequestV2, { ...validQueueMessage, ...over });
 
-    // A diferencia de los DTOs HTTP, aca NO es @IsUUID: el productor manda su referencia de
-    // negocio, y messages-business la usa como llave junto con el tenant.
-    it('acepta una referencia que no es UUID', async () => {
-        const dto = plainToInstance(NotificationQueueMessageRequestV2,
-            { ...validQueueMessage, referenceMessageId: 'collections-2026-09-09-MORNING-0001' });
-        expect(await validate(dto)).toEqual([]);
+    it('acepta una llave de negocio que no es UUID', async () => {
+        expect(await validate(build({ idempotencyKey: 'run-42:credit-7:PRE_DUE_2_DAYS' }))).toEqual([]);
     });
 
-    it('sigue valido sin referencia: sin ella no hay deduplicacion, que es el comportamiento de siempre', async () => {
-        const dto = plainToInstance(NotificationQueueMessageRequestV2, validQueueMessage);
+    // Sin llave no hay deduplicacion: es el comportamiento de todos los productores de hoy.
+    it('sigue valido sin llave', async () => {
+        const dto = build();
         expect(await validate(dto)).toEqual([]);
-        expect(dto.referenceMessageId).toBeUndefined();
+        expect(dto.idempotencyKey).toBeUndefined();
+    });
+
+    // El `#` separa los tramos de la llave de MessageLogs_GT: adentro la haria ambigua.
+    it('rechaza una llave con "#"', async () => {
+        const errors = await validate(build({ idempotencyKey: 'run-42#credit-7' }));
+        expect(errors.some((e) => e.property === 'idempotencyKey' && e.constraints?.matches)).toBe(true);
+    });
+
+    it('rechaza una llave sin techo', async () => {
+        const errors = await validate(build({ idempotencyKey: 'x'.repeat(MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH + 1) }));
+        expect(errors.some((e) => e.property === 'idempotencyKey' && e.constraints?.maxLength)).toBe(true);
+    });
+
+    // referenceMessageId trae el id de la CORRIDA, no del mensaje: deduplicar con el aplastaria
+    // en uno todos los avisos de un mismo barrido.
+    it('referenceMessageId es correlacion y NO tiene las restricciones de la llave', async () => {
+        expect(await validate(build({ referenceMessageId: 'run-42#lo-que-sea' }))).toEqual([]);
     });
 
     it('ya no acepta messageLogId: era plomeria del lote retirado', () => {

package/bin/messagesConnector/constants/MessageIdempotencyLimits.d.ts

@@ -0,0 +1,5 @@
+/**
+ * Largo maximo de la llave de idempotencia de un mensaje de la cola. Entra en la llave de
+ * `MessageLogs_GT`, asi que acotarla es lo que evita una llave sin techo.
+ */
+export declare const MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH = 128;

package/bin/messagesConnector/constants/MessageIdempotencyLimits.js

@@ -0,0 +1,8 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH = void 0;
+/**
+ * Largo maximo de la llave de idempotencia de un mensaje de la cola. Entra en la llave de
+ * `MessageLogs_GT`, asi que acotarla es lo que evita una llave sin techo.
+ */
+exports.MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH = 128;

package/bin/messagesConnector/dtos/NotificationQueueMessageRequestV2.d.ts

@@ -11,11 +11,19 @@ export declare class NotificationQueueMessageRequestV2 {
     directoryId: string;
     destination: string;
     variables: Record<string, string>;
+    /** Referencia de correlacion del productor. NO deduplica: para eso esta `idempotencyKey`. */
+    referenceMessageId?: string;
     /**
-     * Referencia del productor. Cuando viaja, messages-business la usa como llave de
-     * idempotencia: el par (tenantId, referenceMessageId) se entrega UNA sola vez.
+     * Llave de idempotencia, opcional y explicita. Cuando viaja, messages-business entrega UNA
+     * sola vez el trio (tenantId, idempotencyKey, channelType).
+     *
+     * Va aparte de `referenceMessageId` a proposito: esa referencia identifica la CORRIDA que
+     * produjo el mensaje, no al mensaje, asi que usarla para deduplicar aplastaria en uno todos
+     * los avisos de un mismo barrido. La llave tiene que distinguir al destinatario.
+     *
+     * Un `#` en el valor hace ambigua la llave: no se admite.
      */
-    referenceMessageId?: string;
+    idempotencyKey?: string;
     data?: unknown;
     /** Quien disparo la notificacion. Sin default: su ausencia ya significa EVENT. */
     origin?: NotificationOriginEnum;

package/bin/messagesConnector/dtos/NotificationQueueMessageRequestV2.js

@@ -15,6 +15,7 @@ const class_transformer_1 = require("class-transformer");
 const DeliveryChannelExtendedEnum_1 = require("../enums/DeliveryChannelExtendedEnum");
 const NotificationOriginEnum_1 = require("../enums/NotificationOriginEnum");
 const IsStringValueRecord_1 = require("../validators/IsStringValueRecord");
+const MessageIdempotencyLimits_1 = require("../constants/MessageIdempotencyLimits");
 class NotificationQueueMessageRequestV2 {
 }
 exports.NotificationQueueMessageRequestV2 = NotificationQueueMessageRequestV2;
@@ -77,6 +78,15 @@ __decorate([
     (0, class_validator_1.IsString)(),
     __metadata("design:type", String)
 ], NotificationQueueMessageRequestV2.prototype, "referenceMessageId", void 0);
+__decorate([
+    (0, class_transformer_1.Expose)(),
+    (0, class_validator_1.IsOptional)(),
+    (0, class_validator_1.IsString)(),
+    (0, class_validator_1.IsNotEmpty)(),
+    (0, class_validator_1.MaxLength)(MessageIdempotencyLimits_1.MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH),
+    (0, class_validator_1.Matches)(/^[^#]+$/, { message: 'idempotencyKey no puede contener "#"' }),
+    __metadata("design:type", String)
+], NotificationQueueMessageRequestV2.prototype, "idempotencyKey", void 0);
 __decorate([
     (0, class_transformer_1.Expose)(),
     (0, class_validator_1.IsOptional)(),

package/bin/messagesConnector/index.d.ts

@@ -25,3 +25,4 @@ export * from './dtos/MdmNotifyItem';
 export * from './dtos/MdmNotifyRequest';
 export * from './dtos/MdmNotifyResponse';
 export * from './dtos/MdmNotifyBatchResponse';
+export * from './constants/MessageIdempotencyLimits';

package/bin/messagesConnector/index.js

@@ -41,3 +41,4 @@ __exportStar(require("./dtos/MdmNotifyItem"), exports);
 __exportStar(require("./dtos/MdmNotifyRequest"), exports);
 __exportStar(require("./dtos/MdmNotifyResponse"), exports);
 __exportStar(require("./dtos/MdmNotifyBatchResponse"), exports);
+__exportStar(require("./constants/MessageIdempotencyLimits"), exports);

package/bin/messagesConnector/validators/IsTemplatePlaceholders.d.ts

@@ -1,10 +1,11 @@
 import { ValidatorConstraintInterface, ValidationArguments } from 'class-validator';
 /**
- * Valida que un string solo contenga placeholders bien formados `{varName}` donde
- * `varName` matchea `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
+ * Valida que un string solo contenga placeholders bien formados, en cualquiera de las dos
+ * formas del catalogo: `{varName}` o `{{varName}}`, con `varName` matcheando
+ * `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
  *  - Placeholders con espacios o caracteres invalidos (`{var name}`, `{var-1}`).
- *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}` ).
- *  - Placeholders vacios (`{}`).
+ *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}`).
+ *  - Placeholders vacios (`{}`, `{{}}`).
  * Acepta texto sin placeholders en absoluto.
  *
  * Uso en DTOs: `@Validate(IsTemplatePlaceholders)` sobre un campo string.

package/bin/messagesConnector/validators/IsTemplatePlaceholders.js

@@ -10,11 +10,17 @@ exports.IsTemplatePlaceholders = void 0;
 const class_validator_1 = require("class-validator");
 const VALID_PLACEHOLDER = /^[a-zA-Z][a-zA-Z0-9_]*$/;
 /**
- * Valida que un string solo contenga placeholders bien formados `{varName}` donde
- * `varName` matchea `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
+ * Las DOS formas que el renderizador sustituye: doble `{{monto}}` y simple `{monto}`.
+ * La doble va primero para que no gane el `{monto}` interno, igual que alla.
+ */
+const PLACEHOLDER_PATTERN = /\{\{([^{}]*)\}\}|\{([^{}]*)\}/g;
+/**
+ * Valida que un string solo contenga placeholders bien formados, en cualquiera de las dos
+ * formas del catalogo: `{varName}` o `{{varName}}`, con `varName` matcheando
+ * `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
  *  - Placeholders con espacios o caracteres invalidos (`{var name}`, `{var-1}`).
- *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}` ).
- *  - Placeholders vacios (`{}`).
+ *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}`).
+ *  - Placeholders vacios (`{}`, `{{}}`).
  * Acepta texto sin placeholders en absoluto.
  *
  * Uso en DTOs: `@Validate(IsTemplatePlaceholders)` sobre un campo string.
@@ -23,33 +29,20 @@ let IsTemplatePlaceholders = class IsTemplatePlaceholders {
     validate(value) {
         if (typeof value !== 'string')
             return false;
-        // Detectar llaves desbalanceadas con un single-pass.
-        let depth = 0;
-        for (const ch of value) {
-            if (ch === '{') {
-                if (depth > 0)
-                    return false; // anidado no permitido
-                depth = 1;
-            }
-            else if (ch === '}') {
-                if (depth === 0)
-                    return false; // cierre sin apertura
-                depth = 0;
-            }
-        }
-        if (depth !== 0)
-            return false; // apertura sin cierre
-        // Validar que cada `{...}` matchea el regex.
-        const matches = value.match(/\{([^}]*)\}/g) ?? [];
-        for (const raw of matches) {
-            const inner = raw.slice(1, -1);
-            if (!VALID_PLACEHOLDER.test(inner))
-                return false;
-        }
-        return true;
+        // Se consumen los placeholders bien formados; lo que sobra no puede tener llaves.
+        let wellFormed = true;
+        const leftover = value.replace(PLACEHOLDER_PATTERN, (_match, doubleName, singleName) => {
+            const name = doubleName ?? singleName ?? '';
+            if (!VALID_PLACEHOLDER.test(name))
+                wellFormed = false;
+            return '';
+        });
+        if (!wellFormed)
+            return false;
+        return !leftover.includes('{') && !leftover.includes('}');
     }
     defaultMessage(args) {
-        return `${args.property} contiene placeholders mal formados; debe seguir el patron {nombre} con nombre = [a-zA-Z][a-zA-Z0-9_]*`;
+        return `${args.property} contiene placeholders mal formados; debe seguir el patron {nombre} o {{nombre}} con nombre = [a-zA-Z][a-zA-Z0-9_]*`;
     }
 };
 exports.IsTemplatePlaceholders = IsTemplatePlaceholders;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fiado/type-kit",
-  "version": "3.418.0",
+  "version": "3.420.0",
   "description": "",
   "main": "bin/index.js",
   "types": "bin/index.d.ts",

package/src/messagesConnector/constants/MessageIdempotencyLimits.ts

@@ -0,0 +1,5 @@
+/**
+ * Largo maximo de la llave de idempotencia de un mensaje de la cola. Entra en la llave de
+ * `MessageLogs_GT`, asi que acotarla es lo que evita una llave sin techo.
+ */
+export const MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH = 128;

package/src/messagesConnector/dtos/NotificationQueueMessageRequestV2.ts

@@ -1,8 +1,9 @@
-import { IsEnum, IsNotEmpty, IsObject, IsOptional, IsString, Validate } from 'class-validator';
+import { IsEnum, IsNotEmpty, IsObject, IsOptional, IsString, Matches, MaxLength, Validate } from 'class-validator';
 import { Expose } from 'class-transformer';
 import { DeliveryChannelExtendedEnum } from '../enums/DeliveryChannelExtendedEnum';
 import { NotificationOriginEnum } from '../enums/NotificationOriginEnum';
 import { IsStringValueRecord } from '../validators/IsStringValueRecord';
+import { MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH } from '../constants/MessageIdempotencyLimits';
 
 export class NotificationQueueMessageRequestV2 {
     /** Tenant emisor del mensaje. Si se omite, messages-business asume FIADO. */
@@ -50,14 +51,29 @@ export class NotificationQueueMessageRequestV2 {
     @Validate(IsStringValueRecord)
     variables!: Record<string, string>;
 
+    /** Referencia de correlacion del productor. NO deduplica: para eso esta `idempotencyKey`. */
+    @Expose()
+    @IsOptional()
+    @IsString()
+    referenceMessageId?: string;
+
     /**
-     * Referencia del productor. Cuando viaja, messages-business la usa como llave de
-     * idempotencia: el par (tenantId, referenceMessageId) se entrega UNA sola vez.
+     * Llave de idempotencia, opcional y explicita. Cuando viaja, messages-business entrega UNA
+     * sola vez el trio (tenantId, idempotencyKey, channelType).
+     *
+     * Va aparte de `referenceMessageId` a proposito: esa referencia identifica la CORRIDA que
+     * produjo el mensaje, no al mensaje, asi que usarla para deduplicar aplastaria en uno todos
+     * los avisos de un mismo barrido. La llave tiene que distinguir al destinatario.
+     *
+     * Un `#` en el valor hace ambigua la llave: no se admite.
      */
     @Expose()
     @IsOptional()
     @IsString()
-    referenceMessageId?: string;
+    @IsNotEmpty()
+    @MaxLength(MESSAGE_IDEMPOTENCY_KEY_MAX_LENGTH)
+    @Matches(/^[^#]+$/, { message: 'idempotencyKey no puede contener "#"' })
+    idempotencyKey?: string;
 
     @Expose()
     @IsOptional()

package/src/messagesConnector/index.ts

@@ -25,3 +25,4 @@ export * from './dtos/MdmNotifyItem';
 export * from './dtos/MdmNotifyRequest';
 export * from './dtos/MdmNotifyResponse';
 export * from './dtos/MdmNotifyBatchResponse';
+export * from './constants/MessageIdempotencyLimits';

package/src/messagesConnector/validators/IsTemplatePlaceholders.ts

@@ -3,11 +3,18 @@ import { ValidatorConstraint, ValidatorConstraintInterface, ValidationArguments
 const VALID_PLACEHOLDER = /^[a-zA-Z][a-zA-Z0-9_]*$/;
 
 /**
- * Valida que un string solo contenga placeholders bien formados `{varName}` donde
- * `varName` matchea `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
+ * Las DOS formas que el renderizador sustituye: doble `{{monto}}` y simple `{monto}`.
+ * La doble va primero para que no gane el `{monto}` interno, igual que alla.
+ */
+const PLACEHOLDER_PATTERN = /\{\{([^{}]*)\}\}|\{([^{}]*)\}/g;
+
+/**
+ * Valida que un string solo contenga placeholders bien formados, en cualquiera de las dos
+ * formas del catalogo: `{varName}` o `{{varName}}`, con `varName` matcheando
+ * `^[a-zA-Z][a-zA-Z0-9_]*$`. Rechaza:
  *  - Placeholders con espacios o caracteres invalidos (`{var name}`, `{var-1}`).
- *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}` ).
- *  - Placeholders vacios (`{}`).
+ *  - Llaves sin balancear (`abc {def`, `abc } def`, `{ {nested}`).
+ *  - Placeholders vacios (`{}`, `{{}}`).
  * Acepta texto sin placeholders en absoluto.
  *
  * Uso en DTOs: `@Validate(IsTemplatePlaceholders)` sobre un campo string.
@@ -17,29 +24,19 @@ export class IsTemplatePlaceholders implements ValidatorConstraintInterface {
     validate(value: unknown): boolean {
         if (typeof value !== 'string') return false;
 
-        // Detectar llaves desbalanceadas con un single-pass.
-        let depth = 0;
-        for (const ch of value) {
-            if (ch === '{') {
-                if (depth > 0) return false; // anidado no permitido
-                depth = 1;
-            } else if (ch === '}') {
-                if (depth === 0) return false; // cierre sin apertura
-                depth = 0;
-            }
-        }
-        if (depth !== 0) return false; // apertura sin cierre
+        // Se consumen los placeholders bien formados; lo que sobra no puede tener llaves.
+        let wellFormed = true;
+        const leftover = value.replace(PLACEHOLDER_PATTERN, (_match, doubleName?: string, singleName?: string) => {
+            const name = doubleName ?? singleName ?? '';
+            if (!VALID_PLACEHOLDER.test(name)) wellFormed = false;
+            return '';
+        });
+        if (!wellFormed) return false;
 
-        // Validar que cada `{...}` matchea el regex.
-        const matches = value.match(/\{([^}]*)\}/g) ?? [];
-        for (const raw of matches) {
-            const inner = raw.slice(1, -1);
-            if (!VALID_PLACEHOLDER.test(inner)) return false;
-        }
-        return true;
+        return !leftover.includes('{') && !leftover.includes('}');
     }
 
     defaultMessage(args: ValidationArguments): string {
-        return `${args.property} contiene placeholders mal formados; debe seguir el patron {nombre} con nombre = [a-zA-Z][a-zA-Z0-9_]*`;
+        return `${args.property} contiene placeholders mal formados; debe seguir el patron {nombre} o {{nombre}} con nombre = [a-zA-Z][a-zA-Z0-9_]*`;
     }
 }