@fiado/type-kit 3.627.0 → 3.629.0

package/src/loanConfig/dtos/requests/BulkGetParametersRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { ArrayNotEmpty, IsArray, IsEnum, IsString, ValidateNested } from 'class-validator';
+import { ArrayMaxSize, ArrayNotEmpty, IsArray, IsEnum, IsString, ValidateNested } from 'class-validator';
+import { BULK_GET_PARAMETERS_MAX_ITEMS } from '../../constants/ParameterRequestLimits';
 import { ParameterTypeEnum } from '../../enums/ParameterTypeEnum';
 
 export class ParameterKeyRef {
@@ -12,11 +13,12 @@ export class ParameterKeyRef {
     key!: string;
 }
 
-/** Body de POST /private/parameters/bulk-get — hot path para consumers (cachean 5 min). */
+/** Body de POST /private/parameters/bulk-get — hot path para consumers (cachean 5 min). Hasta 100 claves. */
 export class BulkGetParametersRequest {
     @Expose()
     @IsArray()
     @ArrayNotEmpty()
+    @ArrayMaxSize(BULK_GET_PARAMETERS_MAX_ITEMS)
     @ValidateNested({ each: true })
     @Type(() => ParameterKeyRef)
     items!: ParameterKeyRef[];

package/src/loanConfig/dtos/requests/BulkUpdateParametersRequest.ts

@@ -1,7 +1,23 @@
 import { Expose, Type } from 'class-transformer';
-import { ArrayNotEmpty, IsArray, IsEnum, IsOptional, IsString, ValidateNested } from 'class-validator';
+import {
+    ArrayMaxSize,
+    ArrayNotEmpty,
+    IsArray,
+    IsEnum,
+    IsString,
+    Matches,
+    MaxLength,
+    ValidateIf,
+    ValidateNested,
+} from 'class-validator';
+import {
+    BULK_UPDATE_PARAMETERS_MAX_ITEMS,
+    PARAMETER_REASON_NON_BLANK_PATTERN,
+    PARAMETER_REASON_MAX_LENGTH,
+} from '../../constants/ParameterRequestLimits';
 import { ParameterTypeEnum } from '../../enums/ParameterTypeEnum';
 
+/** Un parámetro del lote. Mismas reglas que `UpdateParameterRequest` para `value` y `reason`. */
 export class BulkUpdateParameterItem {
     @Expose()
     @IsEnum(ParameterTypeEnum)
@@ -11,22 +27,26 @@ export class BulkUpdateParameterItem {
     @IsString()
     key!: string;
 
+    /** Obligatorio; `null` deja el parámetro pendiente. */
     @Expose()
-    @IsOptional()
-    @IsString()
+    @ValidateIf((item: BulkUpdateParameterItem) => item.value !== null)
+    @IsString({ message: 'value es obligatorio: un string, o null para dejar el parámetro pendiente' })
     value!: string | null;
 
+    /** Obligatorio, no en blanco y de hasta 500 caracteres. */
     @Expose()
-    @IsOptional()
     @IsString()
-    reason?: string;
+    @Matches(PARAMETER_REASON_NON_BLANK_PATTERN, { message: 'reason no puede ir vacío ni en blanco' })
+    @MaxLength(PARAMETER_REASON_MAX_LENGTH)
+    reason!: string;
 }
 
-/** Body de POST /parameters/bulk-update — carga inicial / ajuste masivo. */
+/** Body de POST /parameters/bulk-update — carga inicial / ajuste masivo. Hasta 50 parámetros. */
 export class BulkUpdateParametersRequest {
     @Expose()
     @IsArray()
     @ArrayNotEmpty()
+    @ArrayMaxSize(BULK_UPDATE_PARAMETERS_MAX_ITEMS)
     @ValidateNested({ each: true })
     @Type(() => BulkUpdateParameterItem)
     parameters!: BulkUpdateParameterItem[];

package/src/loanConfig/dtos/requests/UpdateParameterRequest.ts

@@ -1,18 +1,26 @@
 import { Expose } from 'class-transformer';
-import { IsOptional, IsString } from 'class-validator';
+import { IsString, Matches, MaxLength, ValidateIf } from 'class-validator';
+import {
+    PARAMETER_REASON_NON_BLANK_PATTERN,
+    PARAMETER_REASON_MAX_LENGTH,
+} from '../../constants/ParameterRequestLimits';
 
 /**
- * Body de PUT /parameters/:paramType/:key. `value` puede ser null (parámetro pendiente).
- * `reason` opcional en F1; el manager lo exigirá para cambios críticos (modelo-datos §10).
+ * Body de PUT /parameters/:paramType/:key.
+ *
+ * - `value` es obligatorio pero puede ser `null` (parámetro pendiente). Omitirlo da 400: antes borraba
+ *   el valor y respondía 200 (L-140).
+ * - `reason` es obligatorio, no en blanco y de hasta 500 caracteres (decisión D-7, L-139).
  */
 export class UpdateParameterRequest {
     @Expose()
-    @IsOptional()
-    @IsString()
+    @ValidateIf((request: UpdateParameterRequest) => request.value !== null)
+    @IsString({ message: 'value es obligatorio: un string, o null para dejar el parámetro pendiente' })
     value!: string | null;
 
     @Expose()
-    @IsOptional()
     @IsString()
-    reason?: string;
+    @Matches(PARAMETER_REASON_NON_BLANK_PATTERN, { message: 'reason no puede ir vacío ni en blanco' })
+    @MaxLength(PARAMETER_REASON_MAX_LENGTH)
+    reason!: string;
 }

package/src/loanConfig/index.ts

@@ -3,6 +3,7 @@ export * from './enums/ParameterValueTypeEnum';
 
 // Constants
 export * from './constants/RecentParameterChangesLimits';
+export * from './constants/ParameterRequestLimits';
 
 // Entity DTOs
 export * from './dtos/Parameter';

package/src/loanCredit/constants/LoanInputLimits.ts

@@ -0,0 +1,29 @@
+/** Plazo máximo de un crédito en semanas, en el alta y en la reestructura. */
+export const LOAN_MAX_TERM_WEEKS = 104;
+
+/**
+ * Techo técnico de un monto en centavos: 100 000 millones de centavos (1 000 millones de pesos).
+ *
+ * No es política de negocio: un equipo cuesta decenas de miles de pesos. Deja cuatro órdenes de
+ * margen bajo `Number.MAX_SAFE_INTEGER` para que intereses, IVA y sumas de cuotas no pierdan el centavo.
+ */
+export const LOAN_AMOUNT_MAX_CENTS = 100_000_000_000;
+
+/** Largo máximo de un texto libre del backoffice (motivo, justificación); igual que la cancelación. */
+export const LOAN_FREE_TEXT_MAX_LENGTH = 500;
+
+/**
+ * Forma aceptada de un `retailCustomerId`. Laxa a propósito: el formato lo define retail-customer
+ * (hoy un ULID), así que solo se rechaza lo que no puede ser un id (espacios, comas, símbolos).
+ */
+export const RETAIL_CUSTOMER_ID_PATTERN = /^[0-9A-Za-z_-]{1,64}$/;
+export const RETAIL_CUSTOMER_ID_MESSAGE = 'retailCustomerId admite letras, números, guion y guion bajo (máximo 64)';
+
+/** CLABE interbancaria: exactamente 18 dígitos. La aplica el dominio (D9); el DTO del contrato solo acota el largo. */
+export const CLABE_PATTERN = /^\d{18}$/;
+export const CLABE_MESSAGE = 'La CLABE debe tener 18 dígitos';
+
+/** Teléfono en E.164: `+`, código de país y número; 16 caracteres como máximo. */
+export const E164_PHONE_PATTERN = /^\+[1-9]\d{7,14}$/;
+export const E164_PHONE_MAX_LENGTH = 16;
+export const E164_PHONE_MESSAGE = 'El teléfono debe ir en formato E.164 (por ejemplo +525512345678)';

package/src/loanCredit/dtos/LoanBeneficiary.ts

@@ -1,22 +1,32 @@
 import { Expose } from 'class-transformer';
-import { IsNotEmpty, IsOptional, IsString } from 'class-validator';
+import { IsNotEmpty, IsOptional, IsString, MaxLength } from 'class-validator';
+import { IsMxPhone } from '../../common/validators/IsMxPhone';
+
+/** Largo máximo del nombre completo, igual que en el contrato. */
+const FULL_NAME_MAX_LENGTH = 300;
+/** Largo máximo del parentesco. */
+const RELATIONSHIP_MAX_LENGTH = 120;
 
 /**
  * Beneficiario declarado por el acreditado (modelo-datos §5b `LoanBorrower.beneficiary`).
+ * Textos con tope y teléfono MX de 10 dígitos, con `+52` opcional (L-031).
  */
 export class LoanBeneficiary {
     @Expose()
     @IsString()
     @IsNotEmpty()
+    @MaxLength(FULL_NAME_MAX_LENGTH)
     fullName!: string;
 
     @Expose()
     @IsOptional()
     @IsString()
+    @MaxLength(RELATIONSHIP_MAX_LENGTH)
     relationship?: string;
 
     @Expose()
     @IsOptional()
     @IsString()
+    @IsMxPhone()
     phone?: string;
 }

package/src/loanCredit/dtos/LoanContractDetails.ts

@@ -6,7 +6,12 @@ const SHORT_TEXT = 120;
 /** Largo máximo de un texto largo del contrato (nombre, domicilio, razón social). */
 const LONG_TEXT = 300;
 /** IMEI: exactamente 15 dígitos. */
-import { IMEI_PATTERN } from '../../common/constants/InputPatterns';
+import {
+    EMAIL_MAX_LENGTH,
+    EMAIL_MESSAGE,
+    EMAIL_PATTERN,
+    IMEI_PATTERN,
+} from '../../common/constants/InputPatterns';
 
 /** Datos del acreditado tal como se imprimen en el contrato (Sección II.1). */
 export class LoanContractBorrowerDetails {
@@ -16,6 +21,10 @@ export class LoanContractBorrowerDetails {
     @MaxLength(LONG_TEXT)
     fullName: string;
 
+    /**
+     * Solo con tope de largo. retail-customer guarda el CURP de Metamap sin validar su formato, así que
+     * exigir `regexCurp` aquí tumbaría la firma. El formato lo revisa el dominio (loan-credit).
+     */
     @Expose()
     @IsString()
     @IsNotEmpty()
@@ -29,12 +38,21 @@ export class LoanContractBorrowerDetails {
     @MaxLength(LONG_TEXT)
     address: string;
 
+    /**
+     * Solo con tope de largo. El RFC de `people` solo se limpia de espacios y puede llegar en minúsculas
+     * o con 10 caracteres; si se rechaza, el wizard descarta todo `contractDetails` y la firma no sale.
+     * El formato (`RFC_PATTERN`) lo revisa el dominio.
+     */
     @Expose()
     @IsOptional()
     @IsString()
     @MaxLength(SHORT_TEXT)
     rfc?: string;
 
+    /**
+     * Fecha de nacimiento ya formateada para imprimir (el wizard manda `DD/MM/AAAA`). Solo con tope de
+     * largo: el dominio puede revisar la fecha con `isContractBirthDate` y dejarla en blanco si no cumple.
+     */
     @Expose()
     @IsOptional()
     @IsString()
@@ -57,7 +75,8 @@ export class LoanContractBorrowerDetails {
     @Expose()
     @IsOptional()
     @IsString()
-    @MaxLength(LONG_TEXT)
+    @MaxLength(EMAIL_MAX_LENGTH)
+    @Matches(EMAIL_PATTERN, { message: EMAIL_MESSAGE })
     email?: string;
 
     @Expose()
@@ -131,7 +150,11 @@ export class LoanContractProviderDetails {
 
 /** Cuenta Pago Confiado del acreditado, de la que se cobra el crédito (Secciones VI y VIII). */
 export class LoanContractPcfAccountDetails {
-    /** Tal como la tiene la cuenta; en dev puede no ser una CLABE de 18 dígitos. */
+    /**
+     * Tal como la tiene la cuenta, con tope de largo. La CLABE estricta de 18 dígitos (decisión D9) se
+     * aplica en el dominio: si no cumple `CLABE_PATTERN`, loan-credit deja la cuenta en blanco en el
+     * contrato. No se exige aquí para no rechazar la firma cuando la cuenta de dev no es una CLABE.
+     */
     @Expose()
     @IsString()
     @IsNotEmpty()

package/src/loanCredit/dtos/LoanDeviceNotifyItem.ts

@@ -1,12 +1,26 @@
 import { Expose } from 'class-transformer';
-import { IsEnum, IsNotEmpty, IsOptional, IsString, Matches, MaxLength } from 'class-validator';
+import { IsEnum, IsNotEmpty, IsOptional, IsString, Matches, MaxLength, ValidateIf } from 'class-validator';
 import { DeviceNotifyIntensityEnum } from '../../mdm/enums/DeviceNotifyIntensityEnum';
 import { MdmNoticeTypeEnum } from '../../mdm/enums/MdmNoticeTypeEnum';
 import { CREDIT_ID_MESSAGE, CREDIT_ID_PATTERN } from '../constants/CreditIdPattern';
 
+/** `true` si el valor viaja (ni ausente ni `null`). */
+const present = (value: unknown): boolean => value !== undefined && value !== null;
+
+/** El ítem trae un aviso por el camino legacy: `notificationCode`, o `title` junto con `content`. */
+const hasLegacyNotice = (item: LoanDeviceNotifyItem): boolean =>
+    present(item.notificationCode) || (present(item.title) && present(item.content));
+
+/** Mensaje cuando el ítem no trae ningún aviso. */
+export const LOAN_DEVICE_NOTICE_MISSING_MESSAGE =
+    'El aviso debe traer noticeType, o title y content, o notificationCode';
+
 /**
  * Ítem de notificación: a qué crédito se le avisa y con qué aviso.
  * El aviso viaja como `noticeType`; los tres campos de texto son el camino legacy.
+ *
+ * Cada ítem trae un aviso: `noticeType`, o `title` + `content`, o `notificationCode`. Si no trae
+ * ninguno, el error sale en `noticeType` (L-023).
  */
 export class LoanDeviceNotifyItem {
     @Expose()
@@ -22,8 +36,8 @@ export class LoanDeviceNotifyItem {
      * Cuando viaja, manda: los connectors ignoran `notificationCode`, `title` y `content`.
      */
     @Expose()
-    @IsOptional()
-    @IsEnum(MdmNoticeTypeEnum)
+    @ValidateIf((item: LoanDeviceNotifyItem) => present(item.noticeType) || !hasLegacyNotice(item))
+    @IsEnum(MdmNoticeTypeEnum, { message: LOAN_DEVICE_NOTICE_MISSING_MESSAGE })
     noticeType?: MdmNoticeTypeEnum;
 
     /** Código del catálogo de Datacultr. Camino legacy: solo se lee si no viaja `noticeType`. */

package/src/loanCredit/dtos/RestructureTerms.ts

@@ -1,12 +1,14 @@
 import { Expose } from 'class-transformer';
-import { IsInt, IsNumber, IsOptional, Min } from 'class-validator';
+import { IsInt, IsNumber, IsOptional, Max, Min } from 'class-validator';
+import { LOAN_MAX_TERM_WEEKS } from '../constants/LoanInputLimits';
 
 /** Términos de una reestructura: los propone el cobrador y el admin los aprueba o ajusta. */
 export class RestructureTerms {
-    /** Plazo nuevo en semanas; no puede ser menor a las cuotas restantes. */
+    /** Plazo nuevo en semanas; no puede ser menor a las cuotas restantes ni pasar el techo del alta (104). */
     @Expose()
     @IsInt()
     @Min(1)
+    @Max(LOAN_MAX_TERM_WEEKS)
     newTermWeeks!: number;
 
     /** TNA nueva en decimal (`2.30` = 230%). */

package/src/loanCredit/dtos/SignatureContext.ts

@@ -16,7 +16,11 @@ const USER_AGENT_TEXT = 512;
  * ceremonia: la IP y el navegador son del equipo de la tienda. Nunca se loguean.
  */
 export class SignatureContext {
-    /** IP de origen de la llamada, como la ve el API Gateway del wizard. */
+    /**
+     * IP de origen de la llamada, como la ve el API Gateway del wizard. Solo con tope de largo: sin
+     * `sourceIp` el wizard manda `"desconocida"`, y `@IsIP()` tumbaría la firma. El dominio decide qué
+     * hacer con un valor que no es IP.
+     */
     @Expose()
     @IsString()
     @IsNotEmpty()

package/src/loanCredit/dtos/requests/ActivationCheckRequest.ts

@@ -1,5 +1,12 @@
 import { Expose } from 'class-transformer';
-import { IsBoolean, IsOptional, IsString } from 'class-validator';
+import { IsBoolean, IsOptional, IsString, Matches, MaxLength } from 'class-validator';
+import {
+    IMEI_MESSAGE,
+    IMEI_PATTERN,
+    SKU_MAX_LENGTH,
+    SKU_MESSAGE,
+    SKU_PATTERN,
+} from '../../../common/constants/InputPatterns';
 
 /**
  * Body de `POST /private/credits/:creditId/activate-check` (loan-credit-business — M2 §5).
@@ -35,11 +42,14 @@ export class ActivationCheckRequest {
     @Expose()
     @IsOptional()
     @IsString()
+    @Matches(IMEI_PATTERN, { message: IMEI_MESSAGE })
     imei?: string;
 
-    /** SKU del producto vendido (paso 09). Lo congela el expediente MDM al enrolar. */
+    /** SKU del producto vendido (paso 09). Lo congela el expediente MDM al enrolar; mismas reglas que el enroll. */
     @Expose()
     @IsOptional()
     @IsString()
+    @MaxLength(SKU_MAX_LENGTH)
+    @Matches(SKU_PATTERN, { message: SKU_MESSAGE })
     productSku?: string;
 }

package/src/loanCredit/dtos/requests/BackofficeApproveRestructureRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { IsOptional, IsString, MinLength, ValidateNested } from 'class-validator';
+import { IsOptional, IsString, MaxLength, MinLength, ValidateNested } from 'class-validator';
+import { LOAN_FREE_TEXT_MAX_LENGTH } from '../../constants/LoanInputLimits';
 import { RestructureTerms } from '../RestructureTerms';
 
 /** Body de `POST /backoffice/credits/:creditId/restructures/:restructureId/approve` (loan-credit-business). */
@@ -8,6 +9,7 @@ export class BackofficeApproveRestructureRequest {
     @Expose()
     @IsString()
     @MinLength(1)
+    @MaxLength(LOAN_FREE_TEXT_MAX_LENGTH)
     reason!: string;
 
     /** Términos ajustados; ausente se aprueba como se propuso. */

package/src/loanCredit/dtos/requests/BackofficeProposeRestructureRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { IsDefined, IsString, MinLength, ValidateNested } from 'class-validator';
+import { IsDefined, IsString, MaxLength, MinLength, ValidateNested } from 'class-validator';
+import { LOAN_FREE_TEXT_MAX_LENGTH } from '../../constants/LoanInputLimits';
 import { RestructureTerms } from '../RestructureTerms';
 
 /** Body de `POST /backoffice/credits/:creditId/restructures` (loan-credit-business). */
@@ -8,6 +9,7 @@ export class BackofficeProposeRestructureRequest {
     @Expose()
     @IsString()
     @MinLength(1)
+    @MaxLength(LOAN_FREE_TEXT_MAX_LENGTH)
     justification!: string;
 
     @Expose()

package/src/loanCredit/dtos/requests/BackofficeRejectRestructureRequest.ts

@@ -1,5 +1,6 @@
 import { Expose } from 'class-transformer';
-import { IsString, MinLength } from 'class-validator';
+import { IsString, MaxLength, MinLength } from 'class-validator';
+import { LOAN_FREE_TEXT_MAX_LENGTH } from '../../constants/LoanInputLimits';
 
 /** Body de `POST /backoffice/credits/:creditId/restructures/:restructureId/reject` (loan-credit-business). */
 export class BackofficeRejectRestructureRequest {
@@ -7,5 +8,6 @@ export class BackofficeRejectRestructureRequest {
     @Expose()
     @IsString()
     @MinLength(1)
+    @MaxLength(LOAN_FREE_TEXT_MAX_LENGTH)
     reason!: string;
 }

package/src/loanCredit/dtos/requests/BackofficeWriteOffCreditRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { IsOptional, IsString, ValidateNested } from 'class-validator';
+import { IsOptional, IsString, MaxLength, ValidateNested } from 'class-validator';
+import { LOAN_FREE_TEXT_MAX_LENGTH } from '../../constants/LoanInputLimits';
 import { WriteOffConfirmations } from '../WriteOffConfirmations';
 
 /** Body de `POST /backoffice/credits/:creditId/write-off`. Los faltantes los responde el lambda con su code. */
@@ -8,6 +9,7 @@ export class BackofficeWriteOffCreditRequest {
     @Expose()
     @IsOptional()
     @IsString()
+    @MaxLength(LOAN_FREE_TEXT_MAX_LENGTH)
     reason?: string;
 
     @Expose()

package/src/loanCredit/dtos/requests/CreateCustomerPaymentRequest.ts

@@ -1,5 +1,5 @@
 import { Expose } from 'class-transformer';
-import { IsEnum, IsInt, IsOptional, IsUUID, Min } from 'class-validator';
+import { IsBoolean, IsEnum, IsInt, IsOptional, IsUUID, Min } from 'class-validator';
 import { LoanCustomerPaymentTypeEnum } from '../../enums/LoanCustomerPaymentTypeEnum';
 
 /** Body de `POST /private/customers/:directoryId/credits/:creditId/payments` (loan-credit-business). */
@@ -25,4 +25,13 @@ export class CreateCustomerPaymentRequest {
     @IsInt()
     @Min(1)
     installmentCount?: number;
+
+    /**
+     * Confirma un pago igual a otro reciente (mismo crédito, monto y actor en los últimos 10 min) que
+     * llegó con otra llave. Sin él, el servidor responde que pida confirmación al cliente (L-057, L-092).
+     */
+    @Expose()
+    @IsOptional()
+    @IsBoolean()
+    confirmDuplicate?: boolean;
 }

package/src/loanCredit/dtos/requests/CreateSellerPaymentRequest.ts

@@ -1,5 +1,5 @@
 import { Expose, Type } from 'class-transformer';
-import { IsEnum, IsInt, IsOptional, IsUUID, Min, ValidateNested } from 'class-validator';
+import { IsBoolean, IsEnum, IsInt, IsOptional, IsUUID, Min, ValidateNested } from 'class-validator';
 import { InStorePaymentMethodEnum } from '../../enums/InStorePaymentMethodEnum';
 import { LoanCustomerPaymentTypeEnum } from '../../enums/LoanCustomerPaymentTypeEnum';
 import { InStorePaymentDetail } from './InStorePaymentDetail';
@@ -37,4 +37,13 @@ export class CreateSellerPaymentRequest {
     @Expose()
     @IsUUID()
     idempotencyKey!: string;
+
+    /**
+     * Confirma un pago igual a otro reciente (mismo crédito, monto y actor en los últimos 10 min) que
+     * llegó con otra llave. Sin él, el servidor responde que pida confirmación al vendedor (L-057, L-092).
+     */
+    @Expose()
+    @IsOptional()
+    @IsBoolean()
+    confirmDuplicate?: boolean;
 }

package/src/loanCredit/dtos/requests/OriginateLoanCreditRequest.ts

@@ -5,9 +5,13 @@ import {
     IsNotEmpty,
     IsOptional,
     IsString,
+    Matches,
     Max,
+    MaxLength,
     Min,
 } from 'class-validator';
+import { SKU_MAX_LENGTH, SKU_MESSAGE, SKU_PATTERN } from '../../../common/constants/InputPatterns';
+import { LOAN_AMOUNT_MAX_CENTS, LOAN_MAX_TERM_WEEKS } from '../../constants/LoanInputLimits';
 import { ProductCategoryEnum } from '../../../retailCatalog/enums/ProductCategoryEnum';
 import { ProductTierEnum } from '../../../retailCatalog/enums/ProductTierEnum';
 import { ClientLevelEnum } from '../../../loanScoring/enums/ClientLevelEnum';
@@ -42,17 +46,20 @@ export class OriginateLoanCreditRequest {
     @Expose()
     @IsInt()
     @Min(1)
-    @Max(104)
+    @Max(LOAN_MAX_TERM_WEEKS)
     termWeeks!: number;
 
+    /** Precio del equipo en centavos; el techo es técnico, no de negocio (L-027). */
     @Expose()
     @IsInt()
     @Min(1)
+    @Max(LOAN_AMOUNT_MAX_CENTS)
     equipmentPriceCents!: number;
 
     @Expose()
     @IsInt()
     @Min(0)
+    @Max(LOAN_AMOUNT_MAX_CENTS)
     downPaymentCents!: number;
 
     /**
@@ -107,6 +114,8 @@ export class OriginateLoanCreditRequest {
     @Expose()
     @IsOptional()
     @IsString()
+    @MaxLength(SKU_MAX_LENGTH)
+    @Matches(SKU_PATTERN, { message: SKU_MESSAGE })
     productSku?: string;
 
     /** Tipo del equipo: resuelve promociones acotadas por tipo. */

package/src/loanCredit/dtos/requests/SegmentFactsBatchRequest.ts

@@ -1,7 +1,8 @@
 import { Expose } from 'class-transformer';
-import { ArrayMaxSize, ArrayMinSize, IsArray, IsNotEmpty, IsString } from 'class-validator';
+import { ArrayMaxSize, ArrayMinSize, IsArray, IsNotEmpty, IsString, Matches } from 'class-validator';
+import { RETAIL_CUSTOMER_ID_MESSAGE, RETAIL_CUSTOMER_ID_PATTERN } from '../../constants/LoanInputLimits';
 
-/** Body de `POST /private/borrowers/segment-facts`: datos para segmentar de hasta 100 clientes retail. */
+/** Body de `POST /private/borrowers/segment-facts`: datos para segmentar de hasta 100 clientes retail; cada id con forma de id (L-067). */
 export class SegmentFactsBatchRequest {
     @Expose()
     @IsArray()
@@ -9,5 +10,6 @@ export class SegmentFactsBatchRequest {
     @ArrayMaxSize(100)
     @IsString({ each: true })
     @IsNotEmpty({ each: true })
+    @Matches(RETAIL_CUSTOMER_ID_PATTERN, { each: true, message: RETAIL_CUSTOMER_ID_MESSAGE })
     retailCustomerIds!: string[];
 }

package/src/loanCredit/dtos/requests/SignLoanCreditRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { IsObject, IsOptional, IsString, Length, ValidateNested } from 'class-validator';
+import { IsObject, IsOptional, IsString, Length, Matches, MaxLength, ValidateNested } from 'class-validator';
+import { E164_PHONE_MAX_LENGTH, E164_PHONE_MESSAGE, E164_PHONE_PATTERN } from '../../constants/LoanInputLimits';
 import { LoanContractDetails } from '../LoanContractDetails';
 import { SignatureContext } from '../SignatureContext';
 
@@ -39,6 +40,8 @@ export class SignLoanCreditRequest {
      * posee (PII TRANSITORIA del cliente — nombreAcreditado, curp, domicilio, lugar de firma...).
      * Las aporta el WIZARD (dueño del dato); el motor las mezcla con las financieras (monto,
      * pagos, tasa) y las pasa a document-generator. NO se persisten ni loggean (silo PII-zero).
+     *
+     * @deprecated loan-credit ya no lo lee: los datos del contrato viajan en `contractDetails`.
      */
     @Expose()
     @IsOptional()
@@ -64,6 +67,8 @@ export class SignLoanCreditRequest {
     @Expose()
     @IsOptional()
     @IsString()
+    @MaxLength(E164_PHONE_MAX_LENGTH)
+    @Matches(E164_PHONE_PATTERN, { message: E164_PHONE_MESSAGE })
     otpDestination?: string;
 
     /**

package/src/loanCredit/dtos/requests/UpsertLoanBorrowerRequest.ts

@@ -1,5 +1,6 @@
 import { Expose, Type } from 'class-transformer';
-import { IsNotEmpty, IsOptional, IsString, ValidateNested } from 'class-validator';
+import { IsNotEmpty, IsOptional, IsString, Matches, ValidateNested } from 'class-validator';
+import { RETAIL_CUSTOMER_ID_MESSAGE, RETAIL_CUSTOMER_ID_PATTERN } from '../../constants/LoanInputLimits';
 import { LoanBeneficiary } from '../LoanBeneficiary';
 
 /**
@@ -15,9 +16,11 @@ import { LoanBeneficiary } from '../LoanBeneficiary';
  * `RETAIL_PULL_FAILED` (422/502 — ver openapi del motor).
  */
 export class UpsertLoanBorrowerRequest {
+    /** Id del cliente en retail-customer (hoy un ULID); largo y forma acotados (L-031). */
     @Expose()
     @IsString()
     @IsNotEmpty()
+    @Matches(RETAIL_CUSTOMER_ID_PATTERN, { message: RETAIL_CUSTOMER_ID_MESSAGE })
     retailCustomerId!: string;
 
     /** Beneficiario / a quién avisar (opcional — el wizard lo captura en su paso propio). */

package/src/loanCredit/index.ts

@@ -117,6 +117,8 @@ export * from './dtos/responses/LoanDeviceReleaseResult';
 export * from './dtos/responses/LoanDeviceReleaseResponse';
 
 export * from './validators/IsFutureIso8601DateTime';
+export * from './validators/IsContractBirthDate';
+export * from './constants/LoanInputLimits';
 
 // Candidatos de mensajería — contrato de `POST /private/messaging/candidates`
 // Re-export del enum canónico del momento (owner: messagesConnector — un solo origen de verdad).

package/src/loanCredit/validators/IsContractBirthDate.ts

@@ -0,0 +1,41 @@
+import { ValidateBy, ValidationOptions } from 'class-validator';
+
+/** `DD/MM/AAAA`: como la imprime el contrato y como la manda hoy el wizard. */
+const DAY_FIRST = /^(\d{2})\/(\d{2})\/(\d{4})$/;
+/** `AAAA-MM-DD`: como la guarda retail-customer. */
+const ISO_DATE = /^(\d{4})-(\d{2})-(\d{2})$/;
+
+/** `true` si año, mes y día forman una fecha de calendario real (rechaza el 31/02). */
+const isCalendarDate = (year: number, month: number, day: number): boolean => {
+    const date = new Date(Date.UTC(year, month - 1, day));
+    return date.getUTCFullYear() === year && date.getUTCMonth() === month - 1 && date.getUTCDate() === day;
+};
+
+/** `true` si el valor es una fecha `DD/MM/AAAA` o `AAAA-MM-DD` que existe en el calendario. */
+export const isContractBirthDate = (value: unknown): boolean => {
+    if (typeof value !== 'string') return false;
+    const dayFirst = DAY_FIRST.exec(value);
+    if (dayFirst) return isCalendarDate(Number(dayFirst[3]), Number(dayFirst[2]), Number(dayFirst[1]));
+    const iso = ISO_DATE.exec(value);
+    if (iso) return isCalendarDate(Number(iso[1]), Number(iso[2]), Number(iso[3]));
+    return false;
+};
+
+/**
+ * Fecha de nacimiento del contrato: `DD/MM/AAAA` (la que imprime la plantilla y manda el wizard) o
+ * `AAAA-MM-DD`. Sin hora: es una fecha de calendario, no un instante.
+ *
+ * `LoanContractDetails.birthDate` no lo usa: ahí solo hay tope de largo para no tumbar la firma. Es
+ * para el dominio, que con `isContractBirthDate` decide si imprime la fecha o la deja en blanco.
+ */
+export const IsContractBirthDate = (validationOptions?: ValidationOptions): PropertyDecorator =>
+    ValidateBy(
+        {
+            name: 'isContractBirthDate',
+            validator: {
+                validate: (value: unknown): boolean => isContractBirthDate(value),
+                defaultMessage: (): string => '$property debe ser una fecha DD/MM/AAAA o AAAA-MM-DD válida',
+            },
+        },
+        validationOptions,
+    );