@fiado/type-kit 3.28.0 → 3.29.0

package/bin/messaging/dtos/MessageDeliveryRequest.js

@@ -14,6 +14,8 @@ const class_transformer_1 = require("class-transformer");
 const class_validator_1 = require("class-validator");
 const DeliveryChannelEnum_1 = require("../enums/DeliveryChannelEnum");
 const ContentTypeEnum_1 = require("../enums/ContentTypeEnum");
+const IsDestinationByChannel_1 = require("../validators/IsDestinationByChannel");
+const IsNotBlank_1 = require("../validators/IsNotBlank");
 class MessageDeliveryRequest {
 }
 exports.MessageDeliveryRequest = MessageDeliveryRequest;
@@ -24,21 +26,18 @@ __decorate([
 ], MessageDeliveryRequest.prototype, "deliveryChannel", void 0);
 __decorate([
     (0, class_transformer_1.Expose)(),
-    (0, class_validator_1.IsString)(),
-    (0, class_validator_1.IsNotEmpty)(),
+    (0, class_validator_1.Validate)(IsDestinationByChannel_1.IsDestinationByChannel),
     __metadata("design:type", String)
 ], MessageDeliveryRequest.prototype, "destination", void 0);
 __decorate([
     (0, class_transformer_1.Expose)(),
     (0, class_validator_1.ValidateIf)(o => o.deliveryChannel === DeliveryChannelEnum_1.DeliveryChannelEnum.MAIL),
-    (0, class_validator_1.IsString)(),
-    (0, class_validator_1.IsNotEmpty)(),
+    (0, class_validator_1.Validate)(IsNotBlank_1.IsNotBlank),
     __metadata("design:type", String)
 ], MessageDeliveryRequest.prototype, "subject", void 0);
 __decorate([
     (0, class_transformer_1.Expose)(),
-    (0, class_validator_1.IsString)(),
-    (0, class_validator_1.IsNotEmpty)(),
+    (0, class_validator_1.Validate)(IsNotBlank_1.IsNotBlank),
     __metadata("design:type", String)
 ], MessageDeliveryRequest.prototype, "message", void 0);
 __decorate([

package/bin/messaging/index.d.ts

@@ -1,4 +1,6 @@
 export * from './enums/DeliveryChannelEnum';
 export * from './enums/ContentTypeEnum';
+export * from './validators/IsDestinationByChannel';
+export * from './validators/IsNotBlank';
 export * from './dtos/MessageDeliveryRequest';
 export * from './dtos/MessageDeliveryResponse';

package/bin/messaging/index.js

@@ -16,5 +16,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
 Object.defineProperty(exports, "__esModule", { value: true });
 __exportStar(require("./enums/DeliveryChannelEnum"), exports);
 __exportStar(require("./enums/ContentTypeEnum"), exports);
+__exportStar(require("./validators/IsDestinationByChannel"), exports);
+__exportStar(require("./validators/IsNotBlank"), exports);
 __exportStar(require("./dtos/MessageDeliveryRequest"), exports);
 __exportStar(require("./dtos/MessageDeliveryResponse"), exports);

package/bin/messaging/validators/IsDestinationByChannel.d.ts

@@ -0,0 +1,5 @@
+import { ValidatorConstraintInterface, ValidationArguments } from 'class-validator';
+export declare class IsDestinationByChannel implements ValidatorConstraintInterface {
+    validate(value: unknown, args: ValidationArguments): boolean;
+    defaultMessage(args: ValidationArguments): string;
+}

package/bin/messaging/validators/IsDestinationByChannel.js

@@ -0,0 +1,69 @@
+"use strict";
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.IsDestinationByChannel = void 0;
+const class_validator_1 = require("class-validator");
+const DeliveryChannelEnum_1 = require("../enums/DeliveryChannelEnum");
+/**
+ * Valida que `destination` matchee el formato correcto segun el `deliveryChannel`
+ * declarado en el mismo DTO.
+ *
+ * Por que un custom validator en lugar de dos `@ValidateIf` + `@IsEmail`/`@Matches`:
+ * - `@ValidateIf` afecta a **toda la propiedad**, no solo a los decoradores que
+ *   vienen despues. Cuando se ponen dos `@ValidateIf` consecutivos sobre la
+ *   misma propiedad, el segundo sobrescribe al primero y class-validator
+ *   ignora silenciosamente los validators afectados.
+ * - Originalmente descubierto en `sns-ses-connector/tests/RUN_LOG.md` T7/T10/T11
+ *   (2026-05-21) — la version anterior del DTO dejaba pasar
+ *   `destination: "12345"` con canal SMS y `destination: "no-es-email"` con
+ *   canal MAIL. Validator migrado del connector al type-kit en 3.29.0 para
+ *   eliminar el DTO_LOCAL del connector y exponer la regla a todos los
+ *   callers de `MessageDeliveryRequest`.
+ *
+ * Reglas:
+ * - Canal `SMS` → `destination` debe matchear E.164: `^\+\d{10,15}$`
+ * - Canal `MAIL` → `destination` debe matchear formato email
+ * - Canal invalido o ausente → falla (el `@IsEnum` del DTO genera su propio mensaje aparte)
+ * - String vacio o whitespace → falla
+ *
+ * Uso tipico:
+ * ```ts
+ *   @Validate(IsDestinationByChannel)
+ *   destination: string;
+ * ```
+ */
+const E164_REGEX = /^\+\d{10,15}$/;
+const EMAIL_REGEX = /^[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,}$/;
+let IsDestinationByChannel = class IsDestinationByChannel {
+    validate(value, args) {
+        if (typeof value !== 'string' || value.trim().length === 0)
+            return false;
+        const obj = args.object;
+        const channel = obj.deliveryChannel;
+        if (channel === DeliveryChannelEnum_1.DeliveryChannelEnum.SMS)
+            return E164_REGEX.test(value);
+        if (channel === DeliveryChannelEnum_1.DeliveryChannelEnum.MAIL)
+            return EMAIL_REGEX.test(value);
+        return false;
+    }
+    defaultMessage(args) {
+        const obj = args.object;
+        const channel = obj.deliveryChannel;
+        if (channel === DeliveryChannelEnum_1.DeliveryChannelEnum.SMS) {
+            return 'destination debe ser un telefono en formato E.164 (ej. +5215512345678) para canal SMS';
+        }
+        if (channel === DeliveryChannelEnum_1.DeliveryChannelEnum.MAIL) {
+            return 'destination debe ser un email valido para canal MAIL';
+        }
+        return 'destination requerido y debe corresponder al deliveryChannel';
+    }
+};
+exports.IsDestinationByChannel = IsDestinationByChannel;
+exports.IsDestinationByChannel = IsDestinationByChannel = __decorate([
+    (0, class_validator_1.ValidatorConstraint)({ name: 'IsDestinationByChannel', async: false })
+], IsDestinationByChannel);

package/bin/messaging/validators/IsNotBlank.d.ts

@@ -0,0 +1,24 @@
+import { ValidatorConstraintInterface, ValidationArguments } from 'class-validator';
+/**
+ * Valida que el valor sea un string con al menos un caracter no-whitespace.
+ *
+ * Por que no usar `@IsNotEmpty()` de class-validator:
+ * - `@IsNotEmpty()` solo rechaza `""`, `null`, `undefined`, `false`, `0`.
+ * - **No** rechaza strings de solo whitespace (`"   "`).
+ *
+ * En el sns-ses-connector eso era un bug latente: un `message: "   "` pasaba la
+ * validacion, llegaba al SDK de SNS/SES, y se enviaba un SMS o email con
+ * contenido vacio al usuario final (y AWS cobraba). Originalmente descubierto
+ * en `sns-ses-connector/tests/RUN_LOG.md` T8b (2026-05-21). Validator migrado
+ * al type-kit en 3.29.0 para eliminar el DTO_LOCAL del connector.
+ *
+ * Uso tipico:
+ * ```ts
+ *   @Validate(IsNotBlank)
+ *   message: string;
+ * ```
+ */
+export declare class IsNotBlank implements ValidatorConstraintInterface {
+    validate(value: unknown, _args: ValidationArguments): boolean;
+    defaultMessage(args: ValidationArguments): string;
+}

package/bin/messaging/validators/IsNotBlank.js

@@ -0,0 +1,41 @@
+"use strict";
+var __decorate = (this && this.__decorate) || function (decorators, target, key, desc) {
+    var c = arguments.length, r = c < 3 ? target : desc === null ? desc = Object.getOwnPropertyDescriptor(target, key) : desc, d;
+    if (typeof Reflect === "object" && typeof Reflect.decorate === "function") r = Reflect.decorate(decorators, target, key, desc);
+    else for (var i = decorators.length - 1; i >= 0; i--) if (d = decorators[i]) r = (c < 3 ? d(r) : c > 3 ? d(target, key, r) : d(target, key)) || r;
+    return c > 3 && r && Object.defineProperty(target, key, r), r;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.IsNotBlank = void 0;
+const class_validator_1 = require("class-validator");
+/**
+ * Valida que el valor sea un string con al menos un caracter no-whitespace.
+ *
+ * Por que no usar `@IsNotEmpty()` de class-validator:
+ * - `@IsNotEmpty()` solo rechaza `""`, `null`, `undefined`, `false`, `0`.
+ * - **No** rechaza strings de solo whitespace (`"   "`).
+ *
+ * En el sns-ses-connector eso era un bug latente: un `message: "   "` pasaba la
+ * validacion, llegaba al SDK de SNS/SES, y se enviaba un SMS o email con
+ * contenido vacio al usuario final (y AWS cobraba). Originalmente descubierto
+ * en `sns-ses-connector/tests/RUN_LOG.md` T8b (2026-05-21). Validator migrado
+ * al type-kit en 3.29.0 para eliminar el DTO_LOCAL del connector.
+ *
+ * Uso tipico:
+ * ```ts
+ *   @Validate(IsNotBlank)
+ *   message: string;
+ * ```
+ */
+let IsNotBlank = class IsNotBlank {
+    validate(value, _args) {
+        return typeof value === 'string' && value.trim().length > 0;
+    }
+    defaultMessage(args) {
+        return `${args.property} no puede ser vacio ni solo espacios en blanco`;
+    }
+};
+exports.IsNotBlank = IsNotBlank;
+exports.IsNotBlank = IsNotBlank = __decorate([
+    (0, class_validator_1.ValidatorConstraint)({ name: 'IsNotBlank', async: false })
+], IsNotBlank);

package/package.json

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

package/src/messaging/dtos/MessageDeliveryRequest.ts

@@ -1,7 +1,9 @@
 import { Expose } from 'class-transformer';
-import { IsEnum, IsNotEmpty, IsString, ValidateIf } from 'class-validator';
+import { IsEnum, Validate, ValidateIf } from 'class-validator';
 import { DeliveryChannelEnum } from '../enums/DeliveryChannelEnum';
 import { ContentTypeEnum } from '../enums/ContentTypeEnum';
+import { IsDestinationByChannel } from '../validators/IsDestinationByChannel';
+import { IsNotBlank } from '../validators/IsNotBlank';
 
 export class MessageDeliveryRequest {
     @Expose()
@@ -9,19 +11,16 @@ export class MessageDeliveryRequest {
     deliveryChannel: DeliveryChannelEnum;
 
     @Expose()
-    @IsString()
-    @IsNotEmpty()
+    @Validate(IsDestinationByChannel)
     destination: string;
 
     @Expose()
     @ValidateIf(o => o.deliveryChannel === DeliveryChannelEnum.MAIL)
-    @IsString()
-    @IsNotEmpty()
+    @Validate(IsNotBlank)
     subject: string;
 
     @Expose()
-    @IsString()
-    @IsNotEmpty()
+    @Validate(IsNotBlank)
     message: string;
 
     @Expose()

package/src/messaging/index.ts

@@ -1,4 +1,6 @@
-export * from './enums/DeliveryChannelEnum';
-export * from './enums/ContentTypeEnum';
-export * from './dtos/MessageDeliveryRequest';
-export * from './dtos/MessageDeliveryResponse';
+export * from './enums/DeliveryChannelEnum';
+export * from './enums/ContentTypeEnum';
+export * from './validators/IsDestinationByChannel';
+export * from './validators/IsNotBlank';
+export * from './dtos/MessageDeliveryRequest';
+export * from './dtos/MessageDeliveryResponse';

package/src/messaging/validators/IsDestinationByChannel.ts

@@ -0,0 +1,61 @@
+import { ValidatorConstraint, ValidatorConstraintInterface, ValidationArguments } from 'class-validator';
+import { DeliveryChannelEnum } from '../enums/DeliveryChannelEnum';
+
+/**
+ * Valida que `destination` matchee el formato correcto segun el `deliveryChannel`
+ * declarado en el mismo DTO.
+ *
+ * Por que un custom validator en lugar de dos `@ValidateIf` + `@IsEmail`/`@Matches`:
+ * - `@ValidateIf` afecta a **toda la propiedad**, no solo a los decoradores que
+ *   vienen despues. Cuando se ponen dos `@ValidateIf` consecutivos sobre la
+ *   misma propiedad, el segundo sobrescribe al primero y class-validator
+ *   ignora silenciosamente los validators afectados.
+ * - Originalmente descubierto en `sns-ses-connector/tests/RUN_LOG.md` T7/T10/T11
+ *   (2026-05-21) — la version anterior del DTO dejaba pasar
+ *   `destination: "12345"` con canal SMS y `destination: "no-es-email"` con
+ *   canal MAIL. Validator migrado del connector al type-kit en 3.29.0 para
+ *   eliminar el DTO_LOCAL del connector y exponer la regla a todos los
+ *   callers de `MessageDeliveryRequest`.
+ *
+ * Reglas:
+ * - Canal `SMS` → `destination` debe matchear E.164: `^\+\d{10,15}$`
+ * - Canal `MAIL` → `destination` debe matchear formato email
+ * - Canal invalido o ausente → falla (el `@IsEnum` del DTO genera su propio mensaje aparte)
+ * - String vacio o whitespace → falla
+ *
+ * Uso tipico:
+ * ```ts
+ *   @Validate(IsDestinationByChannel)
+ *   destination: string;
+ * ```
+ */
+const E164_REGEX = /^\+\d{10,15}$/;
+const EMAIL_REGEX = /^[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Za-z]{2,}$/;
+
+@ValidatorConstraint({ name: 'IsDestinationByChannel', async: false })
+export class IsDestinationByChannel implements ValidatorConstraintInterface {
+    validate(value: unknown, args: ValidationArguments): boolean {
+        if (typeof value !== 'string' || value.trim().length === 0) return false;
+
+        const obj = args.object as { deliveryChannel?: unknown };
+        const channel = obj.deliveryChannel;
+
+        if (channel === DeliveryChannelEnum.SMS) return E164_REGEX.test(value);
+        if (channel === DeliveryChannelEnum.MAIL) return EMAIL_REGEX.test(value);
+
+        return false;
+    }
+
+    defaultMessage(args: ValidationArguments): string {
+        const obj = args.object as { deliveryChannel?: unknown };
+        const channel = obj.deliveryChannel;
+
+        if (channel === DeliveryChannelEnum.SMS) {
+            return 'destination debe ser un telefono en formato E.164 (ej. +5215512345678) para canal SMS';
+        }
+        if (channel === DeliveryChannelEnum.MAIL) {
+            return 'destination debe ser un email valido para canal MAIL';
+        }
+        return 'destination requerido y debe corresponder al deliveryChannel';
+    }
+}

package/src/messaging/validators/IsNotBlank.ts

@@ -0,0 +1,31 @@
+import { ValidatorConstraint, ValidatorConstraintInterface, ValidationArguments } from 'class-validator';
+
+/**
+ * Valida que el valor sea un string con al menos un caracter no-whitespace.
+ *
+ * Por que no usar `@IsNotEmpty()` de class-validator:
+ * - `@IsNotEmpty()` solo rechaza `""`, `null`, `undefined`, `false`, `0`.
+ * - **No** rechaza strings de solo whitespace (`"   "`).
+ *
+ * En el sns-ses-connector eso era un bug latente: un `message: "   "` pasaba la
+ * validacion, llegaba al SDK de SNS/SES, y se enviaba un SMS o email con
+ * contenido vacio al usuario final (y AWS cobraba). Originalmente descubierto
+ * en `sns-ses-connector/tests/RUN_LOG.md` T8b (2026-05-21). Validator migrado
+ * al type-kit en 3.29.0 para eliminar el DTO_LOCAL del connector.
+ *
+ * Uso tipico:
+ * ```ts
+ *   @Validate(IsNotBlank)
+ *   message: string;
+ * ```
+ */
+@ValidatorConstraint({ name: 'IsNotBlank', async: false })
+export class IsNotBlank implements ValidatorConstraintInterface {
+    validate(value: unknown, _args: ValidationArguments): boolean {
+        return typeof value === 'string' && value.trim().length > 0;
+    }
+
+    defaultMessage(args: ValidationArguments): string {
+        return `${args.property} no puede ser vacio ni solo espacios en blanco`;
+    }
+}