@fiado/type-kit 3.148.0 → 3.149.0

package/_test_/unit/platformRbac/auth/PrepareChallengeRequest.test.ts

@@ -0,0 +1,64 @@
+import 'reflect-metadata';
+import { plainToInstance } from 'class-transformer';
+import { validate } from 'class-validator';
+import { PrepareChallengeRequest } from '../../../../src/platformRbac/auth/PrepareChallengeRequest';
+import { ChallengeNameEnum } from '../../../../src/platformRbac/enums/ChallengeNameEnum';
+
+/**
+ * DEC-RBAC-094: `email` es opcional y robusto a string vacío. Un usuario phone-primario
+ * (sin email) hace login MFA (SMS_OTP / TOTP) sin email — el trigger CreateAuthChallenge
+ * ya no manda `''`. `@ValidateIf` saltea undefined/null/'' (`@IsOptional` NO saltea `''`).
+ * La rama EMAIL_OTP del preparer sigue necesitando email en runtime — eso NO lo valida el DTO.
+ */
+describe('PrepareChallengeRequest', () => {
+    const base = {
+        cognitoSub: 'f4586478-20c1-700c-c40a-2396db1affe3',
+        challengeName: ChallengeNameEnum.SMS_OTP,
+        phone: '+524422194128',
+    };
+
+    it('valida SMS_OTP sin email (undefined)', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, { ...base });
+        const errors = await validate(dto);
+        expect(errors).toEqual([]);
+    });
+
+    it('valida SMS_OTP con email = "" (string vacío del trigger legacy)', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, { ...base, email: '' });
+        const errors = await validate(dto);
+        expect(errors).toEqual([]);
+    });
+
+    it('valida SMS_OTP con email = null', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, { ...base, email: null });
+        const errors = await validate(dto);
+        expect(errors).toEqual([]);
+    });
+
+    it('valida cuando email SÍ viene y es un email válido', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, {
+            ...base,
+            challengeName: ChallengeNameEnum.EMAIL_OTP,
+            email: 'user@dominio.com',
+        });
+        const errors = await validate(dto);
+        expect(errors).toEqual([]);
+    });
+
+    it('falla cuando email viene pero NO es un email válido', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, { ...base, email: 'no-es-email' });
+        const errors = await validate(dto);
+        expect(errors.length).toBeGreaterThan(0);
+        expect(errors[0].property).toBe('email');
+    });
+
+    it('falla si challengeName no es un valor del enum', async () => {
+        const dto = plainToInstance(PrepareChallengeRequest, {
+            ...base,
+            challengeName: 'BOGUS' as ChallengeNameEnum,
+        });
+        const errors = await validate(dto);
+        expect(errors.length).toBeGreaterThan(0);
+        expect(errors[0].property).toBe('challengeName');
+    });
+});

package/bin/platformRbac/auth/PrepareChallengeRequest.d.ts

@@ -2,6 +2,6 @@ import { ChallengeNameEnum } from '../enums/ChallengeNameEnum';
 export declare class PrepareChallengeRequest {
     cognitoSub: string;
     challengeName: ChallengeNameEnum;
-    email: string;
+    email?: string;
     phone?: string;
 }

package/bin/platformRbac/auth/PrepareChallengeRequest.js

@@ -24,6 +24,7 @@ __decorate([
     __metadata("design:type", String)
 ], PrepareChallengeRequest.prototype, "challengeName", void 0);
 __decorate([
+    (0, class_validator_1.ValidateIf)((o) => o.email !== undefined && o.email !== null && o.email !== ''),
     (0, class_validator_1.IsEmail)(),
     __metadata("design:type", String)
 ], PrepareChallengeRequest.prototype, "email", void 0);

package/package.json

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

package/src/platformRbac/auth/PrepareChallengeRequest.ts

@@ -1,4 +1,4 @@
-import { IsEmail, IsEnum, IsOptional, IsString } from 'class-validator';
+import { IsEmail, IsEnum, IsOptional, IsString, ValidateIf } from 'class-validator';
 import { ChallengeNameEnum } from '../enums/ChallengeNameEnum';
 
 export class PrepareChallengeRequest {
@@ -8,8 +8,14 @@ export class PrepareChallengeRequest {
     @IsEnum(ChallengeNameEnum)
     challengeName!: ChallengeNameEnum;
 
+    // DEC-RBAC-094: email OPCIONAL y robusto a string vacío. Un usuario phone-primario (sin email)
+    // hace login MFA (SMS_OTP/TOTP) sin email; el trigger CreateAuthChallenge ya no manda ''. Se usa
+    // @ValidateIf (no @IsOptional) porque @IsOptional NO saltea '' — acá necesitamos que undefined,
+    // null y '' NO rompan la validación. Cuando email SÍ trae valor no vacío, debe ser un email válido.
+    // La rama EMAIL_OTP del preparer sigue exigiendo email en runtime (no se fuerza en el DTO). Ver DECISIONES.md.
+    @ValidateIf((o) => o.email !== undefined && o.email !== null && o.email !== '')
     @IsEmail()
-    email!: string;
+    email?: string;
 
     // DEC-RBAC-083: teléfono destino del challenge SMS_OTP. Lo popula el trigger CreateAuthChallenge
     // (JWTAuth-cognito) desde el atributo phone_number de Cognito, igual que email. Opcional: ausente