@fiado/type-kit 3.518.0 → 3.519.0

package/_test_/unit/retailWizard/fixtures/saleWithoutPhone.strict.ts

@@ -0,0 +1,57 @@
+import { SaleTypeEnum } from '../../../../src/retailWizard/enums/SaleTypeEnum';
+import { SaleFinancierEnum } from '../../../../src/retailWizard/enums/SaleFinancierEnum';
+import type { Sale } from '../../../../src/retailWizard/dtos/Sale';
+import type { SaleCompletedV1 } from '../../../../src/retailWizard/events/SaleCompletedV1';
+import type { SaleCompletedV2 } from '../../../../src/retailWizard/events/SaleCompletedV2';
+import type { CommissionSaleInput } from '../../../../src/retailCommission/dtos/CommissionAccrual';
+
+// Venta de contado sin teléfono; el test lo compila con strict, como los consumidores.
+export const saleSku: Sale['sku'] = null;
+
+export const completedV1: SaleCompletedV1 = {
+  saleId: 's-1',
+  folio: 'LG-T001-VTA-abcde-2026W40',
+  retailerId: 'r-1',
+  storeId: 'st-1',
+  retailCustomerId: 'c-1',
+  sku: null,
+  imei: null,
+  amountCents: 35000,
+  type: SaleTypeEnum.CASH,
+  soldAt: '2026-10-08T12:00:00.000Z',
+  occurredAt: '2026-10-08T12:00:00.000Z',
+};
+
+export const completedV2: SaleCompletedV2 = {
+  saleId: 's-1',
+  folio: 'LG-T001-VTA-abcde-2026W40',
+  retailerId: 'r-1',
+  storeId: 'st-1',
+  sellerRetailUserId: 'u-1',
+  retailCustomerId: 'c-1',
+  sku: null,
+  brand: null,
+  imei: null,
+  amountCents: 35000,
+  equipmentPriceCents: null,
+  downPaymentCents: null,
+  creditId: null,
+  type: SaleTypeEnum.CASH,
+  externalProvider: null,
+  soldAt: '2026-10-08T12:00:00.000Z',
+  occurredAt: '2026-10-08T12:00:00.000Z',
+};
+
+export const commissionInput: CommissionSaleInput = {
+  saleId: 's-1',
+  sellerId: 'u-1',
+  retailerId: 'r-1',
+  storeId: 'st-1',
+  sku: null,
+  brand: null,
+  equipmentPriceCents: null,
+  amountCents: 35000,
+  financier: Object.values(SaleFinancierEnum)[0],
+  paymentType: 'CASH',
+  soldAt: '2026-10-08T12:00:00.000Z',
+};

package/_test_/unit/retailWizard/saleWithoutPhone.test.ts

@@ -0,0 +1,46 @@
+import 'reflect-metadata';
+import * as path from 'path';
+import * as ts from 'typescript';
+import {
+  commissionInput,
+  completedV1,
+  completedV2,
+  saleSku,
+} from './fixtures/saleWithoutPhone.strict';
+
+const FIXTURE = path.join(__dirname, 'fixtures', 'saleWithoutPhone.strict.ts');
+
+// Los consumidores compilan con strict; sin strictNullChecks null cabe en string y no prueba nada.
+function strictDiagnostics(): string[] {
+  const program = ts.createProgram([FIXTURE], {
+    strict: true,
+    noEmit: true,
+    skipLibCheck: true,
+    experimentalDecorators: true,
+    emitDecoratorMetadata: true,
+    target: ts.ScriptTarget.ES2020,
+    module: ts.ModuleKind.CommonJS,
+    moduleResolution: ts.ModuleResolutionKind.Node10,
+  });
+  return ts
+    .getPreEmitDiagnostics(program)
+    .filter((d) => d.file?.fileName === FIXTURE)
+    .map((d) => ts.flattenDiagnosticMessageText(d.messageText, '\n'));
+}
+
+describe('venta sin teléfono — sku nullable', () => {
+  it('Sale, SaleCompletedV1/V2 y CommissionSaleInput aceptan sku null en strict', () => {
+    expect(strictDiagnostics()).toEqual([]);
+  });
+
+  it('el payload viaja con sku null', () => {
+    expect(saleSku).toBeNull();
+    expect(completedV2.sku).toBeNull();
+    expect(commissionInput.sku).toBeNull();
+  });
+
+  it('SaleCompletedV1 omite equipmentPriceCents sin equipo', () => {
+    expect(completedV1.sku).toBeNull();
+    expect(completedV1).not.toHaveProperty('equipmentPriceCents');
+  });
+});

package/_test_/unit/retailWizard/saleWithoutPhoneLines.test.ts

@@ -0,0 +1,82 @@
+import 'reflect-metadata';
+import { plainToInstance } from 'class-transformer';
+import { validateSync, type ValidationError } from 'class-validator';
+import {
+  LinePaymentTypeEnum,
+  RetailCommissionInboundMessage,
+  RetailCommissionInboundMessageTypeEnum,
+  SaleTypeEnum,
+  type WizardLineProduct,
+} from '../../../src/retailWizard';
+import { ItemTypeEnum, ProductCategoryEnum, TrackingModeEnum } from '../../../src/retailCatalog';
+
+function paths(errors: ValidationError[], prefix = ''): string[] {
+  return errors.flatMap((e) => {
+    const p = prefix ? `${prefix}.${e.property}` : e.property;
+    return e.children && e.children.length > 0 ? paths(e.children, p) : [p];
+  });
+}
+
+function errorsOf(payload: object): string[] {
+  return paths(validateSync(plainToInstance(RetailCommissionInboundMessage, payload) as object));
+}
+
+describe('venta sin teléfono — phoneCount en el mensaje a comisiones', () => {
+  const kit = {
+    lineId: 'L1',
+    sku: 'KIT-1',
+    qty: 1,
+    netCents: 349900,
+    itemType: ItemTypeEnum.PRODUCT,
+    category: ProductCategoryEnum.ACCESSORY,
+    tier: null,
+    brand: null,
+    paymentType: LinePaymentTypeEnum.CASH,
+  };
+  const message = (line: object) => ({
+    type: RetailCommissionInboundMessageTypeEnum.SALE_COMPLETED,
+    tenantId: 't',
+    retailerId: 'ret',
+    storeId: 'st',
+    saleId: 'sale-1',
+    sessionId: 'ses-1',
+    sellerRetailUserId: 'seller-1',
+    saleType: SaleTypeEnum.CASH,
+    externalProvider: null,
+    creditId: null,
+    soldAt: '2026-10-08T20:00:00.000Z',
+    occurredAt: '2026-10-08T20:00:00.000Z',
+    lines: [line],
+  });
+
+  it('un mensaje viejo sin phoneCount sigue validando', () => expect(errorsOf(message(kit))).toEqual([]));
+
+  it('acepta phoneCount en 0 y positivo', () => {
+    expect(errorsOf(message({ ...kit, phoneCount: 0 }))).toEqual([]);
+    expect(errorsOf(message({ ...kit, phoneCount: 2 }))).toEqual([]);
+  });
+
+  it('rechaza phoneCount negativo o no entero', () => {
+    expect(errorsOf(message({ ...kit, phoneCount: -1 }))).toEqual(['lines.0.phoneCount']);
+    expect(errorsOf(message({ ...kit, phoneCount: 1.5 }))).toEqual(['lines.0.phoneCount']);
+  });
+});
+
+describe('venta sin teléfono — itemType en la ficha de la línea de sesión', () => {
+  const base: WizardLineProduct = {
+    brand: 'Genérico',
+    model: 'Funda',
+    capacity: null,
+    color: null,
+    imageUrl: null,
+    tier: null,
+    category: ProductCategoryEnum.ACCESSORY,
+    trackingMode: TrackingModeEnum.QUANTITY,
+  };
+
+  it('acepta itemType y también su ausencia', () => {
+    const service: WizardLineProduct = { ...base, itemType: ItemTypeEnum.SERVICE };
+    expect(service.itemType).toBe(ItemTypeEnum.SERVICE);
+    expect(base.itemType).toBeUndefined();
+  });
+});

package/bin/retailCommission/dtos/CommissionAccrual.d.ts

@@ -69,7 +69,8 @@ export interface CommissionSaleInput {
     sellerId: string;
     retailerId: string;
     storeId: string;
-    sku: string;
+    /** `null` en la venta sin teléfono: se comisiona solo por accesorios. */
+    sku: string | null;
     brand: string | null;
     /**
      * 🔴 **La base de comisión es el PRECIO DEL EQUIPO, no lo cobrado en caja.**

package/bin/retailWizard/dtos/Sale.d.ts

@@ -20,8 +20,9 @@ export interface Sale {
     wizardSessionId: string;
     type: SaleTypeEnum;
     externalProvider: ExternalProviderEnum | null;
-    sku: string;
-    /** Líneas vendidas; `sku` e `imei` siguen siendo los del teléfono financiado o de la línea principal. */
+    /** SKU del teléfono de la venta; `null` en la venta sin teléfono (en DDB se omite). */
+    sku: string | null;
+    /** Líneas vendidas; `sku` e `imei` son los del teléfono, o `null` si la venta no lleva teléfono. */
     lines?: SaleLine[];
     /** SKUs de líneas y componentes, para filtrar ventas por SKU. */
     skus?: string[];

package/bin/retailWizard/dtos/WizardLineProduct.d.ts

@@ -1,3 +1,4 @@
+import type { ItemTypeEnum } from '../../retailCatalog/enums/ItemTypeEnum';
 import type { ProductCategoryEnum } from '../../retailCatalog/enums/ProductCategoryEnum';
 import type { ProductTierEnum } from '../../retailCatalog/enums/ProductTierEnum';
 import type { TrackingModeEnum } from '../../retailCatalog/enums/TrackingModeEnum';
@@ -12,4 +13,6 @@ export interface WizardLineProduct {
     tier: ProductTierEnum | null;
     category: ProductCategoryEnum;
     trackingMode: TrackingModeEnum;
+    /** Producto físico o servicio; ausente en líneas viejas (el consumidor lo toma como PRODUCT). */
+    itemType?: ItemTypeEnum;
 }

package/bin/retailWizard/events/RetailCommissionInboundMessage.d.ts

@@ -21,6 +21,8 @@ export declare class RetailCommissionInboundLine {
     brand: string | null;
     paymentType: LinePaymentTypeEnum;
     role?: WizardLineRoleEnum;
+    /** Teléfonos que trae la línea (kit), congelados al cerrar; ausente en mensajes viejos. */
+    phoneCount?: number;
 }
 /** Venta cerrada o cancelada que el wizard publica en RetailCommissionInboundQueue. */
 export declare class RetailCommissionInboundMessage {

package/bin/retailWizard/events/RetailCommissionInboundMessage.js

@@ -72,6 +72,12 @@ __decorate([
     (0, class_validator_1.IsEnum)(WizardLineRoleEnum_1.WizardLineRoleEnum),
     __metadata("design:type", String)
 ], RetailCommissionInboundLine.prototype, "role", void 0);
+__decorate([
+    (0, class_validator_1.IsOptional)(),
+    (0, class_validator_1.IsInt)(),
+    (0, class_validator_1.Min)(0),
+    __metadata("design:type", Number)
+], RetailCommissionInboundLine.prototype, "phoneCount", void 0);
 /** Venta cerrada o cancelada que el wizard publica en RetailCommissionInboundQueue. */
 class RetailCommissionInboundMessage {
 }

package/bin/retailWizard/events/SaleCompletedV1.d.ts

@@ -10,7 +10,8 @@ export interface SaleCompletedV1 {
     retailerId: string;
     storeId: string;
     retailCustomerId: string;
-    sku: string;
+    /** `null` en la venta sin teléfono. */
+    sku: string | null;
     imei: string | null;
     /**
      * Lo cobrado en caja al cerrar: enganche + lo no financiado (en contado, el total).
@@ -22,7 +23,7 @@ export interface SaleCompletedV1 {
      * sin `type` un crédito con $1,300 de enganche era indistinguible de un contado de $1,300.
      */
     type?: SaleTypeEnum;
-    /** Precio de lista del equipo; en contado coincide con `amountCents`. */
+    /** Precio de lista del equipo; en contado coincide con `amountCents`. Sin equipo se omite. */
     equipmentPriceCents?: number;
     /** Enganche cobrado (solo `type=CREDIT`). */
     downPaymentCents?: number;

package/bin/retailWizard/events/SaleCompletedV2.d.ts

@@ -23,7 +23,8 @@ export interface SaleCompletedV2 {
     /** Quién vendió — el eje del cálculo de comisión. Es lo que más falta en la V1. */
     sellerRetailUserId: string;
     retailCustomerId: string;
-    sku: string;
+    /** `null` en la venta sin teléfono. */
+    sku: string | null;
     /** Marca del equipo, del snapshot congelado al cerrar. `null` si no se pudo resolver. */
     brand: string | null;
     imei: string | null;

package/package.json

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

package/src/retailCommission/dtos/CommissionAccrual.ts

@@ -72,7 +72,8 @@ export interface CommissionSaleInput {
     sellerId: string;
     retailerId: string;
     storeId: string;
-    sku: string;
+    /** `null` en la venta sin teléfono: se comisiona solo por accesorios. */
+    sku: string | null;
     brand: string | null;
     /**
      * 🔴 **La base de comisión es el PRECIO DEL EQUIPO, no lo cobrado en caja.**

package/src/retailWizard/dtos/Sale.ts

@@ -21,8 +21,9 @@ export interface Sale {
   wizardSessionId: string;
   type: SaleTypeEnum;
   externalProvider: ExternalProviderEnum | null;
-  sku: string;
-  /** Líneas vendidas; `sku` e `imei` siguen siendo los del teléfono financiado o de la línea principal. */
+  /** SKU del teléfono de la venta; `null` en la venta sin teléfono (en DDB se omite). */
+  sku: string | null;
+  /** Líneas vendidas; `sku` e `imei` son los del teléfono, o `null` si la venta no lleva teléfono. */
   lines?: SaleLine[];
   /** SKUs de líneas y componentes, para filtrar ventas por SKU. */
   skus?: string[];

package/src/retailWizard/dtos/WizardLineProduct.ts

@@ -1,3 +1,4 @@
+import type { ItemTypeEnum } from '../../retailCatalog/enums/ItemTypeEnum';
 import type { ProductCategoryEnum } from '../../retailCatalog/enums/ProductCategoryEnum';
 import type { ProductTierEnum } from '../../retailCatalog/enums/ProductTierEnum';
 import type { TrackingModeEnum } from '../../retailCatalog/enums/TrackingModeEnum';
@@ -13,4 +14,6 @@ export interface WizardLineProduct {
     tier: ProductTierEnum | null;
     category: ProductCategoryEnum;
     trackingMode: TrackingModeEnum;
+    /** Producto físico o servicio; ausente en líneas viejas (el consumidor lo toma como PRODUCT). */
+    itemType?: ItemTypeEnum;
 }

package/src/retailWizard/events/RetailCommissionInboundMessage.ts

@@ -62,6 +62,12 @@ export class RetailCommissionInboundLine {
     @IsOptional()
     @IsEnum(WizardLineRoleEnum)
     role?: WizardLineRoleEnum;
+
+    /** Teléfonos que trae la línea (kit), congelados al cerrar; ausente en mensajes viejos. */
+    @IsOptional()
+    @IsInt()
+    @Min(0)
+    phoneCount?: number;
 }
 
 /** Venta cerrada o cancelada que el wizard publica en RetailCommissionInboundQueue. */

package/src/retailWizard/events/SaleCompletedV1.ts

@@ -11,7 +11,8 @@ export interface SaleCompletedV1 {
   retailerId: string;
   storeId: string;
   retailCustomerId: string;
-  sku: string;
+  /** `null` en la venta sin teléfono. */
+  sku: string | null;
   imei: string | null;
   /**
    * Lo cobrado en caja al cerrar: enganche + lo no financiado (en contado, el total).
@@ -23,7 +24,7 @@ export interface SaleCompletedV1 {
    * sin `type` un crédito con $1,300 de enganche era indistinguible de un contado de $1,300.
    */
   type?: SaleTypeEnum;
-  /** Precio de lista del equipo; en contado coincide con `amountCents`. */
+  /** Precio de lista del equipo; en contado coincide con `amountCents`. Sin equipo se omite. */
   equipmentPriceCents?: number;
   /** Enganche cobrado (solo `type=CREDIT`). */
   downPaymentCents?: number;

package/src/retailWizard/events/SaleCompletedV2.ts

@@ -24,7 +24,8 @@ export interface SaleCompletedV2 {
     /** Quién vendió — el eje del cálculo de comisión. Es lo que más falta en la V1. */
     sellerRetailUserId: string;
     retailCustomerId: string;
-    sku: string;
+    /** `null` en la venta sin teléfono. */
+    sku: string | null;
     /** Marca del equipo, del snapshot congelado al cerrar. `null` si no se pudo resolver. */
     brand: string | null;
     imei: string | null;