apacuana-sdk-core 0.12.0 → 0.13.0

package/dist/types/signatures.d.ts

@@ -41,6 +41,10 @@ export type GetDigestData = {
      * - Identificador único de la firma.
      */
     signatureId: string;
+    /**
+     * - Documento a firmar en formato de archivo. Este campo es opcional.
+     */
+    document?: File | undefined;
 };
 export type SignaturePlacement = {
     /**
@@ -64,10 +68,6 @@ export type SignerData = {
      * - Nombre del firmante.
      */
     name: string;
-    /**
-     * - Referencia o identificador del firmante.
-     */
-    reference: string;
     /**
      * - Tipo de documento de identidad del firmante.
      */
@@ -80,7 +80,18 @@ export type SignerData = {
      * - Array con las coordenadas donde se estampará la firma.
      */
     signature: Array<SignaturePlacement>;
+    /**
+     * - Referencia a un documento pre-cargado. Se debe proporcionar 'reference' o 'file', pero no ambos.
+     */
+    reference?: string | undefined;
+    /**
+     * - Archivo del documento a firmar. Se debe proporcionar 'reference' o 'file', pero no ambos.
+     */
+    file?: File | undefined;
 };
+/**
+ * Define la estructura de datos para la respuesta de la creación de una firma.
+ */
 export type GetDigestResponseData = {
     /**
      * - El digest del documento que se va a firmar.
@@ -144,4 +155,8 @@ export type SignDocumentData = {
      * - Digest firmado.
      */
     signedDigest: string;
+    /**
+     * - Documento a firmar en formato de archivo. Este campo es opcional.
+     */
+    document?: File | undefined;
 };

package/dist/utils/helpers.d.ts

@@ -8,6 +8,7 @@ declare namespace _default {
     export { validateGetDocsData };
     export { validateGetDigestData };
     export { validateOnBoardingSignDocumentData };
+    export { createPayloadGetDigest };
 }
 export default _default;
 declare function getCertificateStatus(userData: any, certificateInDevice: any): string;
@@ -16,8 +17,9 @@ declare function arrayBufferToBase64(buffer: any): string;
 declare function encryptedCsr(csr: any, encryptKey?: string): {
     csr: any;
 };
-declare function validateOnBoardingSignerData(signerData: any): void;
+declare function validateOnBoardingSignerData(signerData: any): FormData;
 declare function validateCsr(encryptedCSR: any): void;
 declare function validateGetDocsData(data: any): void;
 declare function validateGetDigestData(signData: any): void;
-declare function validateOnBoardingSignDocumentData(signData: any): void;
+declare function validateOnBoardingSignDocumentData(signData: any): FormData;
+declare function createPayloadGetDigest(signData: any): FormData;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apacuana-sdk-core",
-  "version": "0.12.0",
+  "version": "0.13.0",
   "description": "Core SDK para interacciones con las APIs de Apacuana.",
   "main": "dist/index.js",
   "module": "dist/index.mjs",

package/src/api/signatures.js

@@ -22,25 +22,13 @@ import { INTEGRATION_TYPE } from "../utils/constant";
 // =================================================================
 
 const signDocumentOnBoarding = async (signData) => {
-  helpers.validateOnBoardingSignDocumentData(signData);
-  const { signature, cert, signedDigest } = signData;
-  try {
-    const signBody = {
-      positions: JSON.stringify(
-        signature.positions.map((p) => ({
-          x: p.x,
-          y: p.y,
-          page: p.page,
-          status: 1,
-        }))
-      ),
-      publickey: cert,
-      signeddigest: signedDigest,
-    };
+  const payload = helpers.validateOnBoardingSignDocumentData(signData);
 
+  const { signature } = signData;
+  try {
     const response = await httpRequest(
       `services/api/documents/sign/${signature.id}`,
-      signBody,
+      payload,
       "PUT"
     );
     return new ApacuanaSuccess(response);
@@ -64,11 +52,10 @@ const signDocumentOnPremise = async () => {
 
 const getDigestToSignOnBoarding = async (signData) => {
   try {
+    const formData = helpers.createPayloadGetDigest(signData);
     const response = await httpRequest(
       `services/api/documents/getdigest/${signData.signatureId}`,
-      {
-        publickey: signData.cert,
-      },
+      formData,
       "POST"
     );
     const digest = response.data?.digest || response.digest;
@@ -100,9 +87,9 @@ const getDigestToSignOnPremise = async () => {
 };
 
 const addSignerOnBoarding = async (signerData) => {
-  helpers.validateOnBoardingSignerData(signerData);
+  const payload = helpers.validateOnBoardingSignerData(signerData);
   try {
-    await httpRequest("services/api/documents/signing", signerData, "POST");
+    await httpRequest("services/api/documents/signingsdk", payload, "POST");
     return new ApacuanaSuccess({
       signer: signerData.typedoc + signerData.doc,
     });

package/src/index.js

@@ -7,7 +7,6 @@ import {
   getCertStatus,
   getCertTypes,
   getRequerimentsByTypeUser,
-  requestCertificate,
 } from "./api/certs";
 import {
   addSigner,
@@ -73,10 +72,6 @@ const apacuana = {
       );
     }
   },
-
-  /**
-   * Permite obtener la configuración actual del SDK.
-   */
   close: () => close(),
   getConfig,
   requestRevocation,
@@ -95,7 +90,7 @@ const apacuana = {
   getRequerimentsByTypeUser,
   createFaceLivenessSession,
   validateFaceLiveness,
-  requestCertificate,
+  // requestCertificate,
 };
 
 export default apacuana;

package/src/types/signatures.js

@@ -27,6 +27,7 @@
  * @typedef {object} GetDigestData
  * @property {string} cert - Certificado del firmante en formato base64.
  * @property {string} signatureId - Identificador único de la firma.
+ * @property {File} [document] - Documento a firmar en formato de archivo. Este campo es opcional.
  */
 
 /**
@@ -40,13 +41,15 @@
  * Define la estructura de datos para añadir un firmante.
  * @typedef {object} SignerData
  * @property {string} name - Nombre del firmante.
- * @property {string} reference - Referencia o identificador del firmante.
  * @property {string} typedoc - Tipo de documento de identidad del firmante.
  * @property {string} doc - Número de documento de identidad del firmante.
  * @property {Array<SignaturePlacement>} signature - Array con las coordenadas donde se estampará la firma.
+ * @property {string} [reference] - Referencia a un documento pre-cargado. Se debe proporcionar 'reference' o 'file', pero no ambos.
+ * @property {File} [file] - Archivo del documento a firmar. Se debe proporcionar 'reference' o 'file', pero no ambos.
  */
 
 /**
+ * Define la estructura de datos para la respuesta de la creación de una firma.
  * @typedef {object} GetDigestResponseData
  * @property {string} digest - El digest del documento que se va a firmar.
  */
@@ -86,6 +89,8 @@
  * @property {Array<object>} signature.positions - Posiciones de la firma.
  * @property {string} cert - Certificado en base64.
  * @property {string} signedDigest - Digest firmado.
+ * @property {File} [document] - Documento a firmar en formato de archivo. Este campo es opcional.
+ 
  */
 
 export {};

package/src/utils/helpers.js

@@ -65,8 +65,31 @@ const encryptedCsr = (csr, encryptKey = KEY_HEX) => {
   return body;
 };
 
+const createPayloadGetDigest = (signData) => {
+  const formData = new FormData();
+
+  // Iterate over the object's own properties
+  Object.keys(signData).forEach((key) => {
+    // Conditionally handle the 'document' field
+
+    if (key === "document" && signData[key] instanceof File) {
+      // Append the file directly
+      formData.append(key, signData[key]);
+    } else if (key === "cert") {
+      formData.append("publickey", signData[key]);
+    }
+  });
+  return formData;
+};
+
 const validateOnBoardingSignerData = (signerData) => {
-  if (!signerData.name || typeof signerData.name !== "string") {
+  // Create a copy of the original object to avoid modifying the function parameter
+  const validatedSignerData = { ...signerData };
+
+  if (
+    !validatedSignerData.name ||
+    typeof validatedSignerData.name !== "string"
+  ) {
     throw new ApacuanaAPIError(
       'El campo "name" es requerido y debe ser una cadena de texto.',
       400,
@@ -74,16 +97,46 @@ const validateOnBoardingSignerData = (signerData) => {
     );
   }
 
-  if (!signerData.reference || typeof signerData.reference !== "string") {
+  // Limpia 'reference' si es inválido o está vacío.
+  if (
+    !validatedSignerData.reference ||
+    typeof validatedSignerData.reference !== "string" ||
+    validatedSignerData.reference.trim() === ""
+  ) {
+    delete validatedSignerData.reference;
+  }
+
+  // Limpia 'file' si no es una instancia de File.
+  if (
+    typeof File === "undefined" ||
+    !validatedSignerData.file ||
+    !(validatedSignerData.file instanceof File)
+  ) {
+    delete validatedSignerData.file;
+  }
+
+  // Valida que solo uno de los dos campos exista después de la limpieza.
+  if (validatedSignerData.reference && validatedSignerData.file) {
+    throw new ApacuanaAPIError(
+      "No se pueden proporcionar 'reference' y 'file' simultáneamente.",
+      400,
+      "INVALID_PARAMETER_FORMAT"
+    );
+  }
+
+  if (!validatedSignerData.reference && !validatedSignerData.file) {
     throw new ApacuanaAPIError(
-      'El campo "reference" es requerido y debe ser una cadena de texto.',
+      "Se debe proporcionar un campo 'reference' o 'file' válido.",
       400,
       "INVALID_PARAMETER_FORMAT"
     );
   }
 
   const validDocTypes = ["V", "P", "E"];
-  if (!signerData.typedoc || !validDocTypes.includes(signerData.typedoc)) {
+  if (
+    !validatedSignerData.typedoc ||
+    !validDocTypes.includes(validatedSignerData.typedoc)
+  ) {
     throw new ApacuanaAPIError(
       'El campo "typedoc" es requerido y debe ser uno de los siguientes valores: V, P, E.',
       400,
@@ -91,7 +144,7 @@ const validateOnBoardingSignerData = (signerData) => {
     );
   }
 
-  if (!signerData.doc || typeof signerData.doc !== "string") {
+  if (!validatedSignerData.doc || typeof validatedSignerData.doc !== "string") {
     throw new ApacuanaAPIError(
       'El campo "doc" es requerido y debe ser una cadena de texto.',
       400,
@@ -100,8 +153,8 @@ const validateOnBoardingSignerData = (signerData) => {
   }
 
   if (
-    !Array.isArray(signerData.signature) ||
-    signerData.signature.length === 0
+    !Array.isArray(validatedSignerData.signature) ||
+    validatedSignerData.signature.length === 0
   ) {
     throw new ApacuanaAPIError(
       'El campo "signature" es requerido y debe ser un array no vacío.',
@@ -110,7 +163,7 @@ const validateOnBoardingSignerData = (signerData) => {
     );
   }
 
-  signerData.signature.forEach((sig, index) => {
+  validatedSignerData.signature.forEach((sig, index) => {
     if (!Number.isInteger(sig.page) || sig.page < 1) {
       throw new ApacuanaAPIError(
         `La página de la firma ${
@@ -141,6 +194,19 @@ const validateOnBoardingSignerData = (signerData) => {
       );
     }
   });
+
+  const formData = new FormData();
+  Object.keys(validatedSignerData).forEach((key) => {
+    if (key === "signature") {
+      formData.append(key, JSON.stringify(validatedSignerData[key]));
+    } else if (key === "file") {
+      formData.append("document", validatedSignerData[key]);
+    } else if (validatedSignerData[key] !== undefined) {
+      formData.append(key, validatedSignerData[key]);
+    }
+  });
+
+  return formData;
 };
 
 const validateCsr = (encryptedCSR) => {
@@ -243,6 +309,26 @@ const validateOnBoardingSignDocumentData = (signData) => {
       "INVALID_PARAMETER"
     );
   }
+
+  const formData = new FormData();
+  formData.append(
+    "positions",
+    JSON.stringify(
+      signature.positions.map((p) => ({
+        x: p.x,
+        y: p.y,
+        page: p.page,
+        status: 1,
+      }))
+    )
+  );
+  formData.append("publickey", cert);
+  formData.append("signeddigest", signedDigest);
+  if (signData.document && signData.document instanceof File) {
+    formData.append("document", signData.document);
+  }
+
+  return formData;
 };
 
 export default {
@@ -255,4 +341,5 @@ export default {
   validateGetDocsData,
   validateGetDigestData,
   validateOnBoardingSignDocumentData,
+  createPayloadGetDigest,
 };

package/src/utils/httpClient.js

@@ -129,18 +129,23 @@ export const httpRequest = async (path, data = {}, method = "POST") => {
   let dataToSend;
   const requestConfig = {};
 
-  const hasFile = Object.values(data).some(
-    (value) => typeof File !== "undefined" && value instanceof File
-  );
-  if (hasFile) {
-    const formData = new FormData();
-    Object.keys(data).forEach((key) => {
-      formData.append(key, data[key]);
-    });
-    dataToSend = formData;
+  if (typeof FormData !== "undefined" && data instanceof FormData) {
+    dataToSend = data;
   } else {
-    dataToSend = { ...data };
+    const hasFile = Object.values(data).some(
+      (value) => typeof File !== "undefined" && value instanceof File
+    );
+    if (hasFile) {
+      const formData = new FormData();
+      Object.keys(data).forEach((key) => {
+        formData.append(key, data[key]);
+      });
+      dataToSend = formData;
+    } else {
+      dataToSend = { ...data };
+    }
   }
+
   try {
     let response;
     switch (method.toUpperCase()) {

package/tests/api/signatures.test.js

@@ -24,9 +24,32 @@ jest.mock("../../src/utils/helpers", () => ({
   validateGetDocsData: jest.fn(),
   validateGetDigestData: jest.fn(),
   validateOnBoardingSignDocumentData: jest.fn(),
+  createPayloadGetDigest: jest.fn(),
 }));
 
 describe("API - Signatures", () => {
+  beforeAll(() => {
+    // FormData no existe en el entorno de Node.js de Jest.
+    // Creamos una simulación (mock) que incluye los métodos 'append' y 'forEach'.
+    global.FormData = class FormData {
+      constructor() {
+        this.data = new Map();
+      }
+      append(key, value) {
+        this.data.set(key, value);
+      }
+      forEach(callback) {
+        // Delegamos la llamada forEach al Map interno.
+        this.data.forEach((value, key) => callback(value, key));
+      }
+    };
+  });
+
+  afterAll(() => {
+    // Limpiamos la simulación global después de las pruebas.
+    delete global.FormData;
+  });
+
   afterEach(() => {
     jest.resetAllMocks();
   });
@@ -37,6 +60,12 @@ describe("API - Signatures", () => {
         integrationType: INTEGRATION_TYPE.ONBOARDING,
       });
       const signerData = { typedoc: "V", doc: "12345678" };
+
+      const formData = new FormData();
+      formData.append("typedoc", "V");
+      formData.append("doc", "12345678");
+
+      helpers.validateOnBoardingSignerData.mockReturnValue(formData);
       httpRequest.mockResolvedValue({ success: true });
 
       await addSigner(signerData);
@@ -45,8 +74,8 @@ describe("API - Signatures", () => {
         signerData
       );
       expect(httpRequest).toHaveBeenCalledWith(
-        "services/api/documents/signing",
-        signerData,
+        "services/api/documents/signingsdk",
+        formData,
         "POST"
       );
     });
@@ -177,12 +206,17 @@ describe("API - Signatures", () => {
       const mockResponse = { data: { digest: "test-digest" } };
       httpRequest.mockResolvedValue(mockResponse);
 
+      const expectedFormData = new FormData();
+      expectedFormData.append("publickey", "test-cert");
+      helpers.createPayloadGetDigest.mockReturnValue(expectedFormData);
+
       const result = await getDigest(signData);
 
       expect(helpers.validateGetDigestData).toHaveBeenCalledWith(signData);
+      expect(helpers.createPayloadGetDigest).toHaveBeenCalledWith(signData);
       expect(httpRequest).toHaveBeenCalledWith(
         `services/api/documents/getdigest/${signData.signatureId}`,
-        { publickey: signData.cert },
+        expectedFormData,
         "POST"
       );
       expect(result).toBeInstanceOf(ApacuanaSuccess);
@@ -243,20 +277,33 @@ describe("API - Signatures", () => {
       const mockResponse = { success: true };
       httpRequest.mockResolvedValue(mockResponse);
 
+      const expectedFormData = new FormData();
+      expectedFormData.append(
+        "positions",
+        JSON.stringify(
+          signData.signature.positions.map((p) => ({
+            x: p.x,
+            y: p.y,
+            page: p.page,
+            status: 1,
+          }))
+        )
+      );
+      expectedFormData.append("publickey", signData.cert);
+      expectedFormData.append("signeddigest", signData.signedDigest);
+
+      helpers.validateOnBoardingSignDocumentData.mockReturnValue(
+        expectedFormData
+      );
+
       const result = await signDocument(signData);
 
-      expect(
-        helpers.validateOnBoardingSignDocumentData
-      ).toHaveBeenCalledWith(signData);
+      expect(helpers.validateOnBoardingSignDocumentData).toHaveBeenCalledWith(
+        signData
+      );
       expect(httpRequest).toHaveBeenCalledWith(
         `services/api/documents/sign/${signData.signature.id}`,
-        {
-          positions: JSON.stringify([
-            { x: 0.1, y: 0.2, page: 1, status: 1 },
-          ]),
-          publickey: signData.cert,
-          signeddigest: signData.signedDigest,
-        },
+        expectedFormData,
         "PUT"
       );
       expect(result).toBeInstanceOf(ApacuanaSuccess);
@@ -306,10 +353,18 @@ describe("API - Signatures", () => {
 
     it("should throw an error if data or data.file is not provided", async () => {
       await expect(uploadSignatureVariant(null)).rejects.toThrow(
-        new ApacuanaAPIError("El parámetro 'file' es requerido.", 400, "INVALID_PARAMETER")
+        new ApacuanaAPIError(
+          "El parámetro 'file' es requerido.",
+          400,
+          "INVALID_PARAMETER"
+        )
       );
       await expect(uploadSignatureVariant({})).rejects.toThrow(
-        new ApacuanaAPIError("El parámetro 'file' es requerido.", 400, "INVALID_PARAMETER")
+        new ApacuanaAPIError(
+          "El parámetro 'file' es requerido.",
+          400,
+          "INVALID_PARAMETER"
+        )
       );
     });
 
@@ -325,7 +380,9 @@ describe("API - Signatures", () => {
 
     it("should throw an error if file is not a PNG", async () => {
       const invalidFile = new File([], "test.txt", { type: "text/plain" });
-      await expect(uploadSignatureVariant({ file: invalidFile })).rejects.toThrow(
+      await expect(
+        uploadSignatureVariant({ file: invalidFile })
+      ).rejects.toThrow(
         new ApacuanaAPIError(
           "El archivo debe ser de tipo PNG (image/png).",
           400,