@devlas/dte-sii 2.14.2 → 2.15.0

package/cert/BoletaCert.js

@@ -447,7 +447,11 @@ class BoletaCert {
     // 4. Enviar EnvioBOLETA
     console.log('\nEnviando EnvioBOLETA al SII...');
     const enviador = new EnviadorSII(this.certificado, this.ambiente);
-    const resultadoBoleta = await enviador.enviarBoletaSoap(envioBoleta);
+    // `let`, no `const`: la rama de duplicado más abajo lo reasigna. Con `const` eso lanzaba
+    // `TypeError: Assignment to constant variable` justo en el path que pretende recuperarse.
+    // Sin disparador confirmado: el duplicado real de EnvioBOLETA vuelve STATUS 99 (no 7), y
+    // los STATUS 7 observados son cvc-* de Libros, que EnviadorSII rutea a duplicado:false.
+    let resultadoBoleta = await enviador.enviarBoletaSoap(envioBoleta);
     
     if (!resultadoBoleta.ok) {
       if (resultadoBoleta.duplicado) {

package/cert/CertRunner.js

@@ -58,6 +58,32 @@ const { STEPS, emitProgress } = require('../utils/progress');
  * @property {string} [sessionPath]
  */
 
+/**
+ * CAF que consume cada set de certificación. **Fuente única de verdad**: la usan tanto los
+ * `ejecutarSet*` (como fallback cuando el set no trae `cafRequired`) como
+ * `precargarPlanDeCorrida()`. Antes estos números vivían inline en cada `ejecutarSet*` y los
+ * consumidores los duplicaban por su cuenta, así que un cambio acá los dejaba desincronizados
+ * y timbrando mal.
+ *
+ * El valor de `guia` es una función porque depende de cuántos casos traiga el set.
+ * El default de 3 sale de tráfico real: en 17 corridas registradas el set de guía vino
+ * siempre con `cafRequired: { 52: 3 }` y 3 casos. Antes la lib usaba 1, que timbraba de menos.
+ */
+const CAF_POR_SET = {
+  basico: { 33: 4, 56: 1, 61: 3 },
+  guia:   (setData) => ({ 52: setData?.casos?.length || 3 }),
+  exenta: { 34: 3, 56: 1, 61: 4 },
+  compra: { 46: 1, 56: 1, 61: 1 },
+};
+
+/** Nombre de la estructura y su plan de CAF, en el orden en que se ejecutan los sets. */
+const SETS_DE_CORRIDA = [
+  ['basico', 'setBasico',        CAF_POR_SET.basico],
+  ['guia',   'setGuiaDespacho',  CAF_POR_SET.guia],
+  ['exenta', 'setFacturaExenta', CAF_POR_SET.exenta],
+  ['compra', 'setFacturaCompra', CAF_POR_SET.compra],
+];
+
 class CertRunner {
   /**
    * @param {CertConfig} config - Configuración del runner
@@ -302,6 +328,45 @@ class CertRunner {
     return this._cafsPrecargados;
   }
 
+  /**
+   * Timbra por adelantado el total de folios de la corrida, armando el plan desde los sets ya
+   * cargados en `_estructuras`. Azúcar sobre `precargarCafsDeSets()` para que el consumidor no
+   * tenga que duplicar los `cafRequired` de cada set (ver `CAF_POR_SET`).
+   *
+   * Llamar después de `obtenerSets()` y antes del primer `ejecutarSet*`.
+   *
+   * ⚠️ **Hay que pasar los sets que realmente se van a ejecutar.** `_estructuras` puede traer
+   * los cuatro aunque la corrida ejecute uno solo, porque se rehidrata desde disco. Precargar
+   * todo en ese caso timbraría folios de sets que nadie va a emitir, y quedarían autorizados
+   * sin usar: exactamente el gatillo del racionamiento del SII que este método viene a evitar.
+   *
+   * @param {Array<'basico'|'guia'|'exenta'|'compra'>} [sets] - Sets de esta corrida.
+   * @returns {Promise<Object>} { 33: cafPath, 56: cafPath, ... }
+   */
+  async precargarPlanDeCorrida(sets = ['basico', 'guia', 'exenta', 'compra']) {
+    if (!this._estructuras) {
+      throw new Error('Ejecutar obtenerSets() antes de precargarPlanDeCorrida().');
+    }
+
+    const pedidos = new Set(sets);
+    const planes = [];
+    for (const [nombre, claveEstructura, fallback] of SETS_DE_CORRIDA) {
+      if (!pedidos.has(nombre)) continue;
+      const setData = this._estructuras[claveEstructura];
+      if (!setData) continue; // set opcional ausente en esta corrida
+      planes.push(
+        setData.cafRequired || (typeof fallback === 'function' ? fallback(setData) : fallback)
+      );
+    }
+
+    if (!planes.length) {
+      console.log('\n Nada que precargar: ninguno de los sets pedidos está presente.');
+      return {};
+    }
+    return this.precargarCafsDeSets(planes);
+  }
+
+
   /**
    * Devuelve los CAF precargados que le tocan a un set.
    *
@@ -803,7 +868,7 @@ class CertRunner {
 
   async ejecutarSetBasico(casos) {
     emitProgress(STEPS.SET_START, { set: 'basico' });
-    const r = await this._ejecutarSet(SetBasico, 'setBasico', 'basico', { 33: 4, 56: 1, 61: 3 }, 'basico', casos);
+    const r = await this._ejecutarSet(SetBasico, 'setBasico', 'basico', CAF_POR_SET.basico, 'basico', casos);
     if (r.success) emitProgress(STEPS.SET_OK, { set: 'basico', trackId: r.trackId });
     else emitProgress(STEPS.SET_ERROR, { set: 'basico', error: r.error });
     return r;
@@ -812,7 +877,7 @@ class CertRunner {
   async ejecutarSetGuia(casos) {
     emitProgress(STEPS.SET_START, { set: 'guia' });
     const r = await this._ejecutarSet(SetGuia, 'setGuiaDespacho', 'guia',
-      (setData) => ({ 52: setData.casos?.length || 1 }), 'guia', casos);
+      CAF_POR_SET.guia, 'guia', casos);
     if (r.success) emitProgress(STEPS.SET_OK, { set: 'guia', trackId: r.trackId });
     else emitProgress(STEPS.SET_ERROR, { set: 'guia', error: r.error });
     return r;
@@ -820,7 +885,7 @@ class CertRunner {
 
   async ejecutarSetExenta(casos) {
     emitProgress(STEPS.SET_START, { set: 'exenta' });
-    const r = await this._ejecutarSet(SetExenta, 'setFacturaExenta', 'exenta', { 34: 3, 56: 1, 61: 4 }, 'exenta', casos);
+    const r = await this._ejecutarSet(SetExenta, 'setFacturaExenta', 'exenta', CAF_POR_SET.exenta, 'exenta', casos);
     if (r.success) emitProgress(STEPS.SET_OK, { set: 'exenta', trackId: r.trackId });
     else emitProgress(STEPS.SET_ERROR, { set: 'exenta', error: r.error });
     return r;
@@ -828,7 +893,7 @@ class CertRunner {
 
   async ejecutarSetCompra(casos) {
     emitProgress(STEPS.SET_START, { set: 'compra' });
-    const r = await this._ejecutarSet(SetCompra, 'setFacturaCompra', 'compra', { 46: 1, 56: 1, 61: 1 }, 'compra', casos);
+    const r = await this._ejecutarSet(SetCompra, 'setFacturaCompra', 'compra', CAF_POR_SET.compra, 'compra', casos);
     if (r.success) emitProgress(STEPS.SET_OK, { set: 'compra', trackId: r.trackId });
     else emitProgress(STEPS.SET_ERROR, { set: 'compra', error: r.error });
     return r;
@@ -3944,12 +4009,43 @@ class CertRunner {
     const listaParaDeclarar = estadoResp.includes('P90');
     const estadoMatch = estadoResp.match(/"(P\d+)"/);
     const listaVacia = /\/\/OK\[0,\[\]/.test(estadoResp);
-    const estado = listaParaDeclarar ? 'P90'
-                 : estadoMatch ? estadoMatch[1]
-                 : listaVacia ? 'sin_estado_sok_pendiente'
-                 : 'desconocido';
 
-    return { success: true, inscrita: true, listaParaDeclarar, estado };
+    // ⚠️ La consulta pregunta puntualmente por el estado 90, así que una lista vacía
+    // significa "no está en P90" — y eso pasa en DOS situaciones opuestas:
+    //   a) todavía no llegó (el SII no emitió el SOK del set), o
+    //   b) YA lo pasó: declaró cumplimiento y avanzó a P91.
+    //
+    // Darlas por (a) siempre hacía que, después de una declaración exitosa, se
+    // informara "esperando aprobación del SII (SOK)". La pantalla le pedía al usuario
+    // esperar una etapa que ya había superado (visto el 17/08/2026, RUT 78441936-3:
+    // declaró a las 14:01 con DECLARACION EFECTUADA y a las 14:5x seguía diciendo que
+    // faltaba el SOK).
+    //
+    // Se desempata preguntando por el 91, que es el estado al que mueve
+    // `autorizarEmpresaBolProd`.
+    let estado = listaParaDeclarar ? 'P90'
+               : estadoMatch ? estadoMatch[1]
+               : listaVacia ? 'sin_estado_sok_pendiente'
+               : 'desconocido';
+    let yaDeclarada = false;
+
+    if (estado === 'sin_estado_sok_pendiente') {
+      try {
+        const resp91 = await gwtPost(
+          `7|0|7|${CBE_BASE}|${CBE_POLICY}|${CBE_SVC}|obtenerEstadoAutorizaEmp|java.lang.Integer/3438268394|java.lang.String/2004016611|${dvUp}|1|2|3|4|4|5|6|5|6|5|${rutNum}|7|5|91|0|`
+        );
+        if (resp91.includes('P91')) {
+          estado = 'P91';
+          yaDeclarada = true;
+        }
+      } catch (e) {
+        // Si la segunda consulta falla se conserva la lectura original: es un dato de
+        // más, no una dependencia.
+        console.log(` [!] No se pudo desempatar el estado con P91: ${e.message}`);
+      }
+    }
+
+    return { success: true, inscrita: true, listaParaDeclarar, estado, yaDeclarada };
   }
 
   async completarDeclaracionBoletaPortal({
@@ -3973,6 +4069,13 @@ class CertRunner {
     if (!estadoActual.inscrita) {
       return { success: false, error: 'No se pudo obtener representante vigente (facade CBE)' };
     }
+    // Ya declaró y avanzó a P91: la etapa está cumplida, no pendiente. Sin este caso,
+    // reintentar la declaración sobre una empresa ya declarada devolvía `pendingSok` y
+    // la etapa quedaba "esperando" para siempre, esperando un SOK que ya había llegado
+    // y ya se había usado.
+    if (estadoActual.yaDeclarada) {
+      return { success: true, yaDeclarada: true, mensaje: 'La declaración de cumplimiento ya estaba efectuada (P91)' };
+    }
     if (!estadoActual.listaParaDeclarar) {
       const legible = estadoActual.estado === 'sin_estado_sok_pendiente'
         ? 'sin estado registrado (SOK aún no emitido)'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devlas/dte-sii",
-  "version": "2.14.2",
+  "version": "2.15.0",
   "description": "Facturación y boletas electrónicas para el SII de Chile. Genera, timbra, firma y envía DTEs, libros electrónicos y automatiza la certificación.",
   "main": "index.js",
   "types": "dte-sii.d.ts",
@@ -8,7 +8,7 @@
   "author": "Devlas SpA <ti@devlas.cl> (https://devlas.cl)",
   "homepage": "https://github.com/devlas-cl/dte-sii",
   "scripts": {
-    "test": "node test/pfx.test.js && node test/folios-consolidados.test.js && node test/c14n-apostrofe.test.js"
+    "test": "node test/pfx.test.js && node test/folios-consolidados.test.js && node test/c14n-apostrofe.test.js && node test/precargar-plan-corrida.test.js"
   },
   "repository": {
     "type": "git",