.

modules/core/src/common/helpers/number-helper.ts

@@ -1,5 +1,5 @@
-const toSafeNumber = (value: number | null | undefined): number => {
-  return value ?? 0;
+const toSafeNumber = (value: string | number | null | undefined): number => {
+  return Number(value ?? 0);
 };
 
 export const NumberHelper = {

modules/core/src/web/lib/helpers/form-utils.ts

@@ -1,28 +1,54 @@
+type PickFormDirtyValuesOptions<T extends Record<string, any>> = {
+  replaceTopLevelArrayKeys?: Array<keyof T>;
+};
+
 /**
  * Extrae solo los valores marcados como "dirty" por react-hook-form,
  * respetando la estructura anidada de dirtyFields.
+ *
+ * Regla especial opcional: Si una key de primer nivel
+ * está configurada en `replaceTopLevelArrayKeys` y tiene
+ * cualquier dirty anidado, se incluye el array completo
+ * en lugar de intentar hacer patch recursivo
  */
 export function pickFormDirtyValues<T extends Record<string, any>>(
   values: T,
-  dirtyFields: Partial<Record<keyof T, any>>
+  dirtyFields: Partial<Record<keyof T, any>>,
+  options?: PickFormDirtyValuesOptions<T>
 ): Partial<T> {
   const result: Partial<T> = {};
+  const replaceTopLevelArrayKeys = options?.replaceTopLevelArrayKeys ?? [];
 
   for (const key in dirtyFields) {
     if (!Object.hasOwn(dirtyFields, key)) continue;
 
-    const isDirty = dirtyFields[key];
-    const value = values[key];
+    const typedKey = key as keyof T;
+    const isDirty = dirtyFields[typedKey];
+    const value = values[typedKey];
 
     if (isDirty === true) {
-      // 🔹 Campo "leaf": se ha tocado → copiar valor
-      result[key] = value;
-    } else if (typeof isDirty === "object" && isDirty !== null) {
-      // 🔹 Campo anidado: recursión
-      const nested = pickFormDirtyValues(value, isDirty);
-      if (Object.keys(nested).length > 0) {
-        result[key] = nested as any;
-      }
+      result[typedKey] = value;
+      continue;
+    }
+
+    if (typeof isDirty !== "object" || isDirty === null) {
+      continue;
+    }
+
+    const shouldReplaceTopLevelArray =
+      replaceTopLevelArrayKeys.includes(typedKey) &&
+      Array.isArray(value) &&
+      formHasAnyDirty(isDirty);
+
+    if (shouldReplaceTopLevelArray) {
+      result[typedKey] = value;
+      continue;
+    }
+
+    const nested = pickFormDirtyValues(value, isDirty as Partial<Record<keyof typeof value, any>>);
+
+    if (Object.keys(nested).length > 0) {
+      result[typedKey] = nested as T[keyof T];
     }
   }
 
@@ -37,7 +63,7 @@ export function formHasAnyDirty(dirtyFields: Partial<Record<string, any>> | bool
   if (dirtyFields === false || dirtyFields == null) return false;
 
   if (typeof dirtyFields === "object") {
-    return Object.values(dirtyFields).some((v) => formHasAnyDirty(v));
+    return Object.values(dirtyFields).some((value) => formHasAnyDirty(value));
   }
 
   return false;

modules/customer-invoices/src/api/application/proformas/di/proforma-input-mappers.di.ts

@@ -3,13 +3,12 @@ import type { ICatalogs } from "@erp/core/api";
 import {
   CreateProformaInputMapper,
   type ICreateProformaInputMapper,
-  type IUpdateProformaInputMapper,
   UpdateProformaInputMapper,
 } from "../mappers";
 
 export interface IProformaInputMappers {
   createInputMapper: ICreateProformaInputMapper;
-  updateInputMapper: IUpdateProformaInputMapper;
+  updateInputMapper: UpdateProformaInputMapper;
 }
 
 export const buildProformaInputMappers = (catalogs: ICatalogs): IProformaInputMappers => {

modules/customer-invoices/src/api/application/proformas/di/proforma-use-cases.di.ts

@@ -1,7 +1,7 @@
 import type { ITransactionManager } from "@erp/core/api";
 
 import type { IIssuedInvoicePublicServices } from "../../issued-invoices";
-import type { ICreateProformaInputMapper, IUpdateProformaInputMapper } from "../mappers";
+import type { ICreateProformaInputMapper, UpdateProformaInputMapper } from "../mappers";
 import type {
   IProformaCreator,
   IProformaFinder,
@@ -97,7 +97,7 @@ export function buildIssueProformaUseCase(deps: {
 
 export function buildUpdateProformaUseCase(deps: {
   updater: IProformaUpdater;
-  dtoMapper: IUpdateProformaInputMapper;
+  dtoMapper: UpdateProformaInputMapper;
   fullSnapshotBuilder: IProformaFullSnapshotBuilder;
   transactionManager: ITransactionManager;
 }) {

modules/customer-invoices/src/api/application/proformas/mappers/update-proforma-input.mapper.ts

@@ -1,3 +1,5 @@
+import { NumberHelper } from "@erp/core";
+import { DiscountPercentage } from "@erp/core/api";
 import {
   CurrencyCode,
   DomainError,
@@ -13,7 +15,14 @@ import {
 import { Result, isNullishOrEmpty, toPatchField } from "@repo/rdx-utils";
 
 import type { UpdateProformaByIdRequestDTO } from "../../../../common/dto";
-import { InvoiceSerie, type ProformaPatchProps } from "../../../domain";
+import {
+  InvoiceSerie,
+  ItemAmount,
+  ItemDescription,
+  ItemQuantity,
+  type ProformaItemPatchProps,
+  type ProformaPatchProps,
+} from "../../../domain";
 
 /**
  * UpdateProformaPropsMapper
@@ -37,7 +46,10 @@ export interface IUpdateProformaInputMapper {
 }
 
 export class UpdateProformaInputMapper implements IUpdateProformaInputMapper {
-  public map(dto: UpdateProformaByIdRequestDTO, params: { companyId: UniqueID }) {
+  public map(
+    dto: UpdateProformaByIdRequestDTO,
+    params: { companyId: UniqueID }
+  ): Result<ProformaPatchProps> {
     try {
       const errors: ValidationErrorDetail[] = [];
       const props: ProformaPatchProps = {};
@@ -128,15 +140,149 @@ export class UpdateProformaInputMapper implements IUpdateProformaInputMapper {
         );
       });
 
-      if (errors.length > 0) {
-        return Result.fail(
-          new ValidationErrorCollection("Proforma invoice props mapping failed (update)", errors)
-        );
+      if (dto.items) {
+        const itemsProps = this.mapItemsProps(dto, { errors });
+        props.items = itemsProps;
       }
 
+      this.throwIfValidationErrors(errors);
+
       return Result.ok(props);
     } catch (err: unknown) {
-      return Result.fail(new DomainError("Proforma invoice props mapping failed", { cause: err }));
+      return Result.fail(new DomainError("Proforma proforma props mapping failed", { cause: err }));
     }
   }
+
+  private throwIfValidationErrors(errors: ValidationErrorDetail[]): void {
+    if (errors.length > 0) {
+      throw new ValidationErrorCollection("Customer proforma props mapping failed", errors);
+    }
+  }
+
+  private mapItemsProps(
+    dto: UpdateProformaByIdRequestDTO,
+    params: {
+      errors: ValidationErrorDetail[];
+    }
+  ): ProformaItemPatchProps[] {
+    const itemsProps: ProformaItemPatchProps[] = [];
+
+    dto.items?.forEach((item, index) => {
+      const description = extractOrPushError(
+        maybeFromNullableResult(item.description, (v) => ItemDescription.create(v)),
+        `items[${index}].description`,
+        params.errors
+      );
+
+      const quantity = extractOrPushError(
+        maybeFromNullableResult(item.quantity, (v) =>
+          ItemQuantity.create({ value: NumberHelper.toSafeNumber(v) })
+        ),
+        `items[${index}].quantity`,
+        params.errors
+      );
+
+      const unitAmount = extractOrPushError(
+        maybeFromNullableResult(item.unit_amount, (v) =>
+          ItemAmount.create({ value: NumberHelper.toSafeNumber(v) })
+        ),
+        `items[${index}].unit_amount`,
+        params.errors
+      );
+
+      const discountPercentage = extractOrPushError(
+        maybeFromNullableResult(item.item_discount_percentage, (v) =>
+          DiscountPercentage.create({ value: NumberHelper.toSafeNumber(v.value) })
+        ),
+        `items[${index}].discount_percentage`,
+        params.errors
+      );
+
+      /*const taxes = this.mapTaxesProps(item.taxes, {
+        itemIndex: index,
+        errors: params.errors,
+      });*/
+
+      this.throwIfValidationErrors(params.errors);
+
+      itemsProps.push({
+        description: description!,
+        quantity: quantity!,
+        unitAmount: unitAmount!,
+        itemDiscountPercentage: discountPercentage!,
+        //taxes,
+      });
+    });
+
+    return itemsProps;
+  }
+
+  /* Devuelve las propiedades de los impustos de una línea de detalle */
+
+  /*private mapTaxesProps(
+    taxesDTO: Pick<ProformaItemRequestDTO, "taxes">["taxes"],
+    params: { itemIndex: number; errors: ValidationErrorDetail[] }
+  ): ProformaItemTaxesProps {
+    const { itemIndex, errors } = params;
+
+    const taxesProps: ProformaItemTaxesProps = {
+      iva: Maybe.none(),
+      retention: Maybe.none(),
+      rec: Maybe.none(),
+    };
+
+    // Normaliza: "" -> []
+    const taxStrCodes = taxesDTO
+      .split(",")
+      .map((s) => s.trim())
+      .filter((s) => s.length > 0);
+
+    taxStrCodes.forEach((strCode, taxIndex) => {
+      const taxResult = Tax.createFromCode(strCode, this.taxCatalog);
+
+      if (!taxResult.isSuccess) {
+        errors.push({
+          path: `items[${itemIndex}].taxes[${taxIndex}]`,
+          message: taxResult.error.message,
+        });
+        return;
+      }
+
+      const tax = taxResult.data;
+
+      if (tax.isVATLike()) {
+        if (taxesProps.iva.isSome()) {
+          errors.push({
+            path: `items[${itemIndex}].taxes`,
+            message: "Multiple taxes for group VAT are not allowed",
+          });
+        }
+        taxesProps.iva = Maybe.some(tax);
+      }
+
+      if (tax.isRetention()) {
+        if (taxesProps.retention.isSome()) {
+          errors.push({
+            path: `items[${itemIndex}].taxes`,
+            message: "Multiple taxes for group retention are not allowed",
+          });
+        }
+        taxesProps.retention = Maybe.some(tax);
+      }
+
+      if (tax.isRec()) {
+        if (taxesProps.rec.isSome()) {
+          errors.push({
+            path: `items[${itemIndex}].taxes`,
+            message: "Multiple taxes for group rec are not allowed",
+          });
+        }
+        taxesProps.rec = Maybe.some(tax);
+      }
+    });
+
+    this.throwIfValidationErrors(errors);
+
+    return taxesProps;
+  }*/
 }

modules/customer-invoices/src/api/application/proformas/services/proforma-creator.ts

@@ -1,7 +1,7 @@
 import type { UniqueID } from "@repo/rdx-ddd";
 import { Result } from "@repo/rdx-utils";
 
-import { type IProformaCreateProps, Proforma } from "../../../domain";
+import { Proforma, type ProformaCreateProps } from "../../../domain";
 import type { IProformaRepository } from "../repositories";
 
 import type { IProformaNumberGenerator } from "./proforma-number-generator.interface";
@@ -9,7 +9,7 @@ import type { IProformaNumberGenerator } from "./proforma-number-generator.inter
 export interface IProformaCreatorParams {
   companyId: UniqueID;
   id: UniqueID;
-  props: Omit<IProformaCreateProps, "invoiceNumber">;
+  props: Omit<ProformaCreateProps, "invoiceNumber">;
   transaction: unknown;
 }
 

modules/customer-invoices/src/api/application/proformas/services/proforma-updater.ts

@@ -8,7 +8,7 @@ export interface IProformaUpdater {
   update(params: {
     companyId: UniqueID;
     id: UniqueID;
-    props: ProformaPatchProps;
+    patchProps: ProformaPatchProps;
     transaction: unknown;
   }): Promise<Result<Proforma, Error>>;
 }
@@ -27,12 +27,12 @@ export class ProformaUpdater implements IProformaUpdater {
   async update(params: {
     companyId: UniqueID;
     id: UniqueID;
-    props: ProformaPatchProps;
+    patchProps: ProformaPatchProps;
     transaction: unknown;
   }): Promise<Result<Proforma, Error>> {
-    const { companyId, id, props, transaction } = params;
+    const { companyId, id, patchProps, transaction } = params;
 
-    console.log("props => ", props);
+    console.log("patchProps => ", patchProps);
 
     // Recuperar agregado existente
     const existingResult = await this.repository.getByIdInCompany(companyId, id, transaction);
@@ -44,14 +44,12 @@ export class ProformaUpdater implements IProformaUpdater {
     const proforma = existingResult.data;
 
     // Aplicar cambios en el agregado
-    const updateResult = proforma.update(props);
+    const updateResult = proforma.update(patchProps);
 
     if (updateResult.isFailure) {
       return Result.fail(updateResult.error);
     }
 
-    console.log(proforma.operationDate);
-
     // Persistir cambios
     const saveResult = await this.repository.update(proforma, transaction);
 

modules/customer-invoices/src/api/application/proformas/use-cases/update-proforma.use-case.ts

@@ -4,7 +4,7 @@ import { Result } from "@repo/rdx-utils";
 
 import type { UpdateProformaByIdRequestDTO } from "../../../../common";
 import type { ProformaPatchProps } from "../../../domain";
-import type { IUpdateProformaInputMapper } from "../mappers";
+import type { UpdateProformaInputMapper } from "../mappers";
 import type { IProformaUpdater } from "../services";
 import type { IProformaFullSnapshotBuilder } from "../snapshot-builders";
 
@@ -15,14 +15,14 @@ type UpdateProformaUseCaseInput = {
 };
 
 type UpdateProformaUseCaseDeps = {
-  dtoMapper: IUpdateProformaInputMapper;
+  dtoMapper: UpdateProformaInputMapper;
   updater: IProformaUpdater;
   fullSnapshotBuilder: IProformaFullSnapshotBuilder;
   transactionManager: ITransactionManager;
 };
 
 export class UpdateProformaUseCase {
-  private readonly dtoMapper: IUpdateProformaInputMapper;
+  private readonly dtoMapper: UpdateProformaInputMapper;
   private readonly updater: IProformaUpdater;
   private readonly fullSnapshotBuilder: IProformaFullSnapshotBuilder;
   private readonly transactionManager: ITransactionManager;
@@ -60,7 +60,7 @@ export class UpdateProformaUseCase {
         const updateResult = await this.updater.update({
           companyId,
           id: proformaId,
-          props: patchProps,
+          patchProps,
           transaction,
         });
 

modules/customer-invoices/src/api/domain/proformas/aggregates/proforma.aggregate.ts

@@ -24,6 +24,7 @@ import {
   type IProformaItemCreateProps,
   type IProformaItems,
   ProformaItem,
+  type ProformaItemPatchProps,
   ProformaItems,
 } from "../entities";
 import { ProformaItemMismatch } from "../errors";
@@ -56,6 +57,10 @@ export interface IProformaCreateProps {
   globalDiscountPercentage: DiscountPercentage;
 }
 
+export type ProformaPatchProps = Partial<Omit<IProformaCreateProps, "companyId" | "items">> & {
+  items?: ProformaItemPatchProps[];
+};
+
 export interface IProformaTotals {
   subtotalAmount: InvoiceAmount;
 
@@ -100,10 +105,6 @@ export interface IProforma {
   totals(): IProformaTotals;
 }
 
-export type ProformaPatchProps = Partial<Omit<IProformaCreateProps, "companyId" | "items">> & {
-  //items?: ProformaItems;
-};
-
 export type InternalProformaProps = Omit<IProformaCreateProps, "items">;
 
 export class Proforma extends AggregateRoot<InternalProformaProps> implements IProforma {
@@ -156,9 +157,47 @@ export class Proforma extends AggregateRoot<InternalProformaProps> implements IP
     return new Proforma(props, items, id);
   }
 
-  private initializeItems(itemsProps: IProformaItemCreateProps[]): Result<void, Error> {
+  // Mutabilidad
+  public update(patchProps: ProformaPatchProps): Result<Proforma, Error> {
+    const { items, ...otherProps } = patchProps;
+
+    const candidateProps: InternalProformaProps = {
+      ...this.props,
+      ...otherProps,
+    };
+
+    // Validacciones
+
+    // Aplicar cambios
+    Object.assign(this.props, candidateProps);
+
+    // Reemplazo de items (si se proporciona)
+    if (items) {
+      this.initializeItems(items);
+    }
+
+    return Result.ok();
+  }
+
+  private initializeItems(
+    itemsProps: IProformaItemCreateProps[] | ProformaItemPatchProps[]
+  ): Result<void, Error> {
+    this._items.reset();
+
     for (const [index, itemProps] of itemsProps.entries()) {
-      const itemResult = ProformaItem.create(itemProps);
+      const { languageCode, currencyCode, globalDiscountPercentage, ...restProps } = {
+        languageCode: this.languageCode,
+        currencyCode: this.currencyCode,
+        globalDiscountPercentage: this.globalDiscountPercentage,
+        ...itemProps,
+      };
+
+      const itemResult = ProformaItem.create({
+        ...restProps,
+        languageCode,
+        currencyCode,
+        globalDiscountPercentage,
+      });
 
       if (itemResult.isFailure) {
         return Result.fail(itemResult.error);
@@ -246,20 +285,6 @@ export class Proforma extends AggregateRoot<InternalProformaProps> implements IP
     return this.paymentMethod.isSome();
   }
 
-  // Mutabilidad
-  public update(patch: ProformaPatchProps): Result<Proforma, Error> {
-    const candidateProps: InternalProformaProps = {
-      ...this.props,
-      ...patch,
-    };
-
-    // Validacciones
-
-    Object.assign(this.props, candidateProps);
-
-    return Result.ok();
-  }
-
   public issue(): Result<void, Error> {
     if (!this.props.status.canTransitionTo("issued")) {
       return Result.fail(
@@ -274,7 +299,6 @@ export class Proforma extends AggregateRoot<InternalProformaProps> implements IP
     this.props.status = InvoiceStatus.issued();
     return Result.ok();
   }
-
   // Cálculos
 
   /**
@@ -326,25 +350,6 @@ export class Proforma extends AggregateRoot<InternalProformaProps> implements IP
     return Result.ok();
   }
 
-  /*public updateItem(itemId: UniqueID, props: IProformaItemProps): Result<void, Error> {
-    const item = this._items.find((i) => i.id.equals(itemId));
-    if (!item) {
-      return Result.fail(new Error("Item not found"));
-    }
-
-    return item.update(props);
-  }*/
-
-  /*public removeItem(itemId: UniqueID): Result<void, Error> {
-    const removed = this._items.removeWhere(i => i.id.equals(itemId));
-
-    if (!removed) {
-      return Result.fail(new Error("Item not found"));
-    }
-
-    return Result.ok();
-  }*/
-
   // Helpers
 
   /**

modules/customer-invoices/src/api/domain/proformas/entities/proforma-items/proforma-item.entity.ts

@@ -42,6 +42,11 @@ export interface IProformaItemCreateProps {
   currencyCode: CurrencyCode; // Para cálculos y formateos de moneda
 }
 
+export type ProformaItemPatchProps = Omit<
+  IProformaItemCreateProps,
+  "globalDiscountPercentage" | "languageCode" | "currencyCode"
+>;
+
 export interface IProformaItemTotals {
   subtotalAmount: ItemAmount;
 

modules/customer-invoices/src/api/infrastructure/common/persistence/sequelize/mappers/domain/customer-invoice-item.mapper.ts

@@ -17,12 +17,12 @@ import {
 import { Result } from "@repo/rdx-utils";
 
 import {
-  type IProformaCreateProps,
   IssuedInvoiceItem,
   ItemAmount,
   ItemDescription,
   ItemQuantity,
   type Proforma,
+  type ProformaCreateProps,
 } from "../../../../../../domain";
 
 export interface ICustomerInvoiceItemDomainMapper
@@ -62,7 +62,7 @@ export class CustomerInvoiceItemDomainMapper
     const { errors, index, attributes } = params as {
       index: number;
       errors: ValidationErrorDetail[];
-      attributes: Partial<IProformaCreateProps>;
+      attributes: Partial<ProformaCreateProps>;
     };
 
     const itemId = extractOrPushError(
@@ -157,7 +157,7 @@ export class CustomerInvoiceItemDomainMapper
     const { errors, index } = params as {
       index: number;
       errors: ValidationErrorDetail[];
-      attributes: Partial<IProformaCreateProps>;
+      attributes: Partial<ProformaCreateProps>;
     };
 
     // 1) Valores escalares (atributos generales)

modules/customer-invoices/src/api/infrastructure/common/persistence/sequelize/mappers/domain/customer-invoice.mapper.ts

@@ -20,12 +20,12 @@ import { Maybe, Result, isNullishOrEmpty } from "@repo/rdx-utils";
 
 import {
   CustomerInvoiceItems,
-  type IProformaCreateProps,
   InvoiceNumber,
   InvoicePaymentMethod,
   InvoiceSerie,
   InvoiceStatus,
   Proforma,
+  type ProformaCreateProps,
 } from "../../../../../../domain";
 import type {
   CustomerInvoiceCreationAttributes,
@@ -249,7 +249,7 @@ export class CustomerInvoiceDomainMapper
         items: itemsResults.data.getAll(),
       });
 
-      const invoiceProps: IProformaCreateProps = {
+      const invoiceProps: ProformaCreateProps = {
         companyId: attributes.companyId!,
 
         isProforma: attributes.isProforma,

modules/customer-invoices/src/api/infrastructure/common/persistence/sequelize/mappers/domain/invoice-recipient.mapper.ts

@@ -16,9 +16,9 @@ import {
 import { Maybe, Result } from "@repo/rdx-utils";
 
 import {
-  type IProformaCreateProps,
   InvoiceRecipient,
   type Proforma,
+  type ProformaCreateProps,
 } from "../../../../../../domain";
 import type { CustomerInvoiceModel } from "../../../../sequelize";
 
@@ -34,7 +34,7 @@ export class InvoiceRecipientDomainMapper {
 
     const { errors, attributes } = params as {
       errors: ValidationErrorDetail[];
-      attributes: Partial<IProformaCreateProps>;
+      attributes: Partial<ProformaCreateProps>;
     };
 
     const { isProforma } = attributes;

modules/customer-invoices/src/api/infrastructure/common/persistence/sequelize/mappers/domain/invoice-verifactu.mapper.ts

@@ -12,8 +12,8 @@ import {
 import { Maybe, Result } from "@repo/rdx-utils";
 
 import {
-  type IProformaCreateProps,
   type Proforma,
+  type ProformaCreateProps,
   VerifactuRecord,
   VerifactuRecordEstado,
 } from "../../../../../../domain";
@@ -43,7 +43,7 @@ export class CustomerInvoiceVerifactuDomainMapper
   ): Result<Maybe<VerifactuRecord>, Error> {
     const { errors, attributes } = params as {
       errors: ValidationErrorDetail[];
-      attributes: Partial<IProformaCreateProps>;
+      attributes: Partial<ProformaCreateProps>;
     };
 
     if (!source) {

modules/customer-invoices/src/api/infrastructure/proformas/express/mappers/create-proforma-request-mapper.ts

@@ -17,7 +17,6 @@ import { Maybe, Result } from "@repo/rdx-utils";
 
 import type { CreateProformaItemRequestDTO, CreateProformaRequestDTO } from "../../../../../common";
 import {
-  type IProformaCreateProps,
   type IProformaItemCreateProps,
   InvoiceNumber,
   InvoicePaymentMethod,
@@ -29,6 +28,7 @@ import {
   ItemAmount,
   ItemDescription,
   ItemQuantity,
+  type ProformaCreateProps,
 } from "../../../../domain";
 
 /**
@@ -149,7 +149,7 @@ export class CreateProformaRequestMapper {
         );
       }
 
-      const proformaProps: Omit<IProformaCreateProps, "items"> & { items: IProformaItemCreateProps[] } = {
+      const proformaProps: Omit<ProformaCreateProps, "items"> & { items: IProformaItemCreateProps[] } = {
         companyId,
         status: defaultStatus!,
 

modules/customer-invoices/src/api/infrastructure/proformas/persistence/sequelize/mappers/domain/sequelize-proforma-item-domain.mapper.ts

@@ -16,12 +16,12 @@ import {
 import { Result } from "@repo/rdx-utils";
 
 import {
-  type IProformaCreateProps,
   type IProformaItemCreateProps,
   ItemAmount,
   ItemDescription,
   ItemQuantity,
   type Proforma,
+  type ProformaCreateProps,
   ProformaItem,
   ProformaItemTaxes,
   type ProformaItemTaxesProps,
@@ -58,7 +58,7 @@ export class SequelizeProformaItemDomainMapper extends SequelizeDomainMapper<
     const { errors, index, parent } = params as {
       index: number;
       errors: ValidationErrorDetail[];
-      parent: Partial<IProformaCreateProps>;
+      parent: Partial<ProformaCreateProps>;
     };
 
     const itemId = extractOrPushError(
@@ -139,7 +139,7 @@ export class SequelizeProformaItemDomainMapper extends SequelizeDomainMapper<
     const { errors, index } = params as {
       index: number;
       errors: ValidationErrorDetail[];
-      parent: Partial<IProformaCreateProps>;
+      parent: Partial<ProformaCreateProps>;
     };
 
     // 1) Valores escalares (atributos generales)

modules/customer-invoices/src/api/infrastructure/proformas/persistence/sequelize/mappers/domain/sequelize-proforma-recipient-domain.mapper.ts

@@ -14,7 +14,7 @@ import {
 } from "@repo/rdx-ddd";
 import { Maybe, Result } from "@repo/rdx-utils";
 
-import { type IProformaCreateProps, InvoiceRecipient } from "../../../../../../domain";
+import { InvoiceRecipient, type ProformaCreateProps } from "../../../../../../domain";
 import type { CustomerInvoiceModel } from "../../../../../common";
 
 export class SequelizeProformaRecipientDomainMapper {
@@ -28,7 +28,7 @@ export class SequelizeProformaRecipientDomainMapper {
 
     const { errors, parent } = params as {
       errors: ValidationErrorDetail[];
-      parent: Partial<IProformaCreateProps>;
+      parent: Partial<ProformaCreateProps>;
     };
 
     const _name = source.current_customer.name;

modules/customer-invoices/src/common/dto/request/proformas/update-proforma-by-id.request.dto.ts

@@ -4,11 +4,14 @@ import { z } from "zod/v4";
 export const UpdateProformaItemRequestSchema = z.object({
   id: z.uuid(),
   position: z.string(),
-  description: z.string().optional(),
-  quantity: NumericStringSchema.optional(),
-  unit_amount: NumericStringSchema.optional(),
-  item_discount_percentage: PercentageSchema.optional(),
-  taxes: z.string().optional(),
+  description: z.string().default(""),
+  quantity: NumericStringSchema.default(""),
+  unit_amount: NumericStringSchema.default(""),
+  item_discount_percentage: PercentageSchema.default({
+    value: "0",
+    scale: "2",
+  }),
+  taxes: z.string().default(""),
 });
 
 export const UpdateProformaByIdParamsRequestSchema = z.object({

modules/customer-invoices/src/web/proformas/update/controllers/use-update-proforma-controller.ts

@@ -1,3 +1,4 @@
+import { formHasAnyDirty } from "@erp/core/client";
 import { useHookForm } from "@erp/core/hooks";
 import type { CustomerSelectionOption } from "@erp/customers";
 import { showErrorToast, showSuccessToast, showWarningToast } from "@repo/rdx-ui/helpers";
@@ -114,12 +115,19 @@ export const useUpdateProformaController = (
         return;
       }
 
+      console.log(form.formState.dirtyFields);
+
+      if (!formHasAnyDirty(form.formState.dirtyFields)) {
+        showWarningToast(
+          t("proformas.update.no_changes.title"),
+          t("proformas.update.no_changes.message")
+        );
+        return;
+      }
+
       const previousData = proformaData;
 
       const patchData = buildProformaUpdatePatch(formData, form.formState.dirtyFields);
-
-      console.log(patchData);
-
       const params = buildUpdateProformaByIdParams(proformaId, patchData);
 
       try {
@@ -132,7 +140,7 @@ export const useUpdateProformaController = (
           keepDirty: false,
         });
 
-        setSelectedCustomer(mapProformaToSelectedCustomer(proformaData));
+        setSelectedCustomer(mapProformaToSelectedCustomer(updated));
 
         if (options?.successToasts !== false) {
           showSuccessToast(
@@ -153,6 +161,8 @@ export const useUpdateProformaController = (
           { keepDirty: false }
         );
 
+        setSelectedCustomer(previousData ? mapProformaToSelectedCustomer(previousData) : null);
+
         if (options?.errorToasts !== false) {
           showErrorToast(t("proformas.update.error.title"), normalizedError.message);
         }

modules/customer-invoices/src/web/proformas/update/utils/build-proforma-items-update-patch.ts

@@ -1,14 +0,0 @@
-import type { ProformaItemUpdateForm, ProformaItemUpdatePatch } from "../entities";
-
-export const buildProformaItemsUpdatePatch = (
-  items: ProformaItemUpdateForm[]
-): ProformaItemUpdatePatch[] => {
-  return items.map((item, index) => ({
-    id: item.id,
-    position: index,
-    description: item.description.trim(),
-    quantity: item.quantity,
-    unitAmount: item.unitAmount,
-    itemDiscountPercentage: item.itemDiscountPercentage,
-  }));
-};

modules/customer-invoices/src/web/proformas/update/utils/build-proforma-update-patch.ts

@@ -3,8 +3,6 @@ import type { FieldNamesMarkedBoolean } from "react-hook-form";
 
 import type { ProformaUpdateForm, ProformaUpdatePatch } from "../entities";
 
-import { buildProformaItemsUpdatePatch } from "./build-proforma-items-update-patch";
-
 export const buildProformaUpdatePatch = (
   formData: ProformaUpdateForm,
   dirtyFields: FieldNamesMarkedBoolean<ProformaUpdateForm>
@@ -13,10 +11,7 @@ export const buildProformaUpdatePatch = (
     return {};
   }
 
-  const itemsPatch = buildProformaItemsUpdatePatch(formData.items);
-
-  return {
-    ...pickFormDirtyValues(formData, dirtyFields),
-    items: itemsPatch,
-  } as ProformaUpdatePatch;
+  return pickFormDirtyValues(formData, dirtyFields, {
+    replaceTopLevelArrayKeys: ["items"],
+  }) satisfies ProformaUpdatePatch;
 };

modules/customer-invoices/src/web/proformas/update/utils/build-update-proforma-by-id-params.ts

@@ -23,12 +23,24 @@ export const buildUpdateProformaByIdParams = (
 
     language_code: patch.languageCode,
     currency_code: patch.currencyCode,
+
+    items: patch.items?.map((item) => ({
+      id: item.id,
+      position: String(item.position),
+      description: item.description,
+      quantity: item.quantity === null ? undefined : String(item.quantity),
+      unit_amount: item.unitAmount === null ? undefined : String(item.unitAmount),
+      item_discount_percentage:
+        item.itemDiscountPercentage === null
+          ? undefined
+          : { value: String(item.itemDiscountPercentage), scale: "2" },
+    })),
   };
 
   return {
     id,
-    data: {
-      ...Object.fromEntries(Object.entries(data).filter(([, value]) => value !== undefined)),
-    },
+    data: Object.fromEntries(
+      Object.entries(data).filter(([, value]) => value !== undefined)
+    ) satisfies UpdateProformaByIdParams["data"],
   };
 };