@fiado/type-kit 3.418.0 → 3.419.0

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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fiado/type-kit",
-  "version": "3.418.0",
+  "version": "3.419.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';