@devlas/dte-sii 2.5.12 → 2.5.14

package/SiiCertificacion.js

@@ -16,9 +16,10 @@
  */
 
 const SiiSession = require('./SiiSession.js');
+const { STEPS, emitProgress } = require('./utils/progress');
 
 /**
- * Etapas de certificación DTE
+ * Etapas de certificacion DTE
  */
 const ETAPAS_CERTIFICACION = {
   SET_BASICO: 'SET_BASICO',
@@ -71,6 +72,20 @@ class SiiCertificacion {
       pfxPassword: options.pfxPassword,
       ambiente: 'certificacion',
     });
+
+    // Reutilización de sesión: cargar desde archivo y guardar automáticamente tras cada login
+    if (options.sessionPath) {
+      this._sessionPath = options.sessionPath;
+      this.session.loadSession(options.sessionPath);
+      const _origEnsure = this.session.ensureSession.bind(this.session);
+      const _sess = this.session;
+      const _sp = options.sessionPath;
+      this.session.ensureSession = async function(targetPath) {
+        const result = await _origEnsure(targetPath);
+        _sess.saveSession(_sp);
+        return result;
+      };
+    }
   }
 
   /**
@@ -378,14 +393,22 @@ class SiiCertificacion {
       // Parsear estado de cada set/libro
       const estadoSets = {};
       
-      // Buscar filas de la tabla con formato:
-      // <td>SET/LIBRO nombre</td><td><b>ESTADO</b></td>
-      const rowRegex = /<tr[^>]*>[\s\S]*?<td[^>]*>[\s\S]*?(SET[^<]*|LIBRO[^<]*)<\/font><\/td>[\s\S]*?<td[^>]*>[\s\S]*?<b>([^<]+)<\/b>/gi;
-      let rowMatch;
-      while ((rowMatch = rowRegex.exec(html)) !== null) {
-        const nombre = rowMatch[1].trim();
-        const estado = rowMatch[2].trim().toUpperCase();
-        estadoSets[nombre] = estado;
+      // Parsear estado de cada set/libro fila por fila para evitar falsos positivos
+      // (la regex global cruzaba filas y asignaba REVISADO CONFORME de SET CASO GENERAL a LIBRO DE VENTAS)
+      const rows = html.split(/<\/tr>/i);
+      for (const row of rows) {
+        const nameMatch = /(SET[^<\n\r]*|LIBRO[^<\n\r]*)<\/font><\/td>/i.exec(row);
+        if (!nameMatch) continue;
+        const nombre = nameMatch[1].trim();
+        // Estado explícito en <b>...</b>
+        const stateMatch = /<b>([^<]+)<\/b>/i.exec(row);
+        if (stateMatch) {
+          estadoSets[nombre] = stateMatch[1].trim().toUpperCase();
+        } else {
+          // Fila con campos input: leer valor EST oculto (S01 = sin declarar, S21 = declarado)
+          const estMatch = /name="EST\d+"[^>]*value="([^"]+)"/i.exec(row);
+          estadoSets[nombre] = estMatch ? estMatch[1] : 'S01';
+        }
       }
       
       // Verificar si todos los sets requeridos están REVISADO CONFORME
@@ -510,7 +533,7 @@ class SiiCertificacion {
       }
       
       if (process.env.DEBUG_SII) {
-        console.log('   [DEBUG] Campos hidden encontrados:', Object.keys(hiddenFields).join(', '));
+        console.log(' [DEBUG] Campos hidden encontrados:', Object.keys(hiddenFields).join(', '));
       }
 
       // Si no hay formulario para declarar (página de estado/reparos)
@@ -541,16 +564,17 @@ class SiiCertificacion {
       // El SII cambia el orden según qué sets están aprobados
       const fieldMapping = {};
       
-      if (process.env.DEBUG_SII) {
+      // Guardar pe_avance2 para debug (siempre)
+      {
         const fs = require('fs');
         const path = require('path');
-        const debugDir = process.env.SII_DEBUG_DIR || path.join(__dirname, '../../debug');
+        const debugDir = process.env.SII_DEBUG_DIR || path.join(__dirname, '../../debug/cert-v2');
         if (!fs.existsSync(debugDir)) {
           fs.mkdirSync(debugDir, { recursive: true });
         }
-        const debugPath = path.join(debugDir, 'pe_avance2.html');
+        const debugPath = path.join(debugDir, 'pe_avance2_form.html');
         fs.writeFileSync(debugPath, formHtml, 'utf8');
-        console.log('   [DEBUG] HTML guardado en:', debugPath);
+
       }
       
       // Extraer todas las filas <tr>...</tr> del formulario
@@ -587,7 +611,7 @@ class SiiCertificacion {
               // Solo agregar si tiene NUM_ENV (no si tiene REVISADO CONFORME o EN REVISION)
               fieldMapping[pattern.name] = parseInt(numEnvMatch[1]);
               if (process.env.DEBUG_SII) {
-                console.log(`   [DEBUG] ${pattern.name} → NUM_ENV${numEnvMatch[1]}`);
+                console.log(` [DEBUG] ${pattern.name} → NUM_ENV${numEnvMatch[1]}`);
               }
             }
             break; // Ya encontramos la fila para este pattern
@@ -596,7 +620,7 @@ class SiiCertificacion {
       }
       
       if (process.env.DEBUG_SII) {
-        console.log('   [DEBUG] Mapeo dinámico de índices:', fieldMapping);
+        console.log(' [DEBUG] Mapeo dinámico de índices:', fieldMapping);
       }
 
       if (sets.setSimulacion && !fieldMapping.setSimulacion) {
@@ -637,7 +661,7 @@ class SiiCertificacion {
             numEnvValues[index] = cleanTrackId;
             fecEnvValues[index] = formatDate(fecha);
             if (String(trackId) !== cleanTrackId) {
-              console.log(`   [TrackID] Normalizado: ${trackId} → ${cleanTrackId}`);
+
             }
           }
         }
@@ -668,16 +692,16 @@ class SiiCertificacion {
 
       // Debug: mostrar datos del formulario
       if (process.env.DEBUG_SII) {
-        console.log('   [DEBUG] Datos del formulario a enviar:');
+        console.log(' [DEBUG] Datos del formulario a enviar:');
         for (const [k, v] of Object.entries(formData)) {
-          console.log(`      ${k}: ${v || '(vacío)'}`);
+          console.log(` ${k}: ${v || '(vacío)'}`);
         }
         
         // Mostrar el body codificado
         const SiiSession = require('./SiiSession');
         const encodedBody = SiiSession.formEncode(formData);
-        console.log('   [DEBUG] Body codificado (primeros 500 chars):');
-        console.log(`      ${encodedBody.substring(0, 500)}`);
+        console.log(' [DEBUG] Body codificado (primeros 500 chars):');
+        console.log(` ${encodedBody.substring(0, 500)}`);
       }
 
       // 5. Enviar formulario de declaración
@@ -697,7 +721,7 @@ class SiiCertificacion {
       
       // Debug - guardar respuesta
       if (process.env.DEBUG_SII) {
-        console.log(`   [DEBUG] Respuesta: ${body.length} bytes, status: ${declareResponse.status}`);
+        console.log(` [DEBUG] Respuesta: ${body.length} bytes, status: ${declareResponse.status}`);
         // Guardar respuesta de pe_avance3 para debug
         const fs = require('fs');
         const path = require('path');
@@ -707,7 +731,7 @@ class SiiCertificacion {
         }
         const debugPath = path.join(debugDir, 'pe_avance3_response.html');
         fs.writeFileSync(debugPath, body);
-        console.log(`   [DEBUG] Respuesta guardada en: ${debugPath}`);
+        console.log(` [DEBUG] Respuesta guardada en: ${debugPath}`);
       }
       
       const bodyLower = body.toLowerCase();
@@ -737,13 +761,117 @@ class SiiCertificacion {
       // Éxito si el form se envió sin errores de sesión/contenido.
       // El estado del envío (ERRORES O REPAROS / EN REVISION / REVISADO CONFORME)
       // NO determina el éxito de la declaración — eso se resuelve vía polling.
-      const success = (!hasError || hasSuccess) && !sesionExpirada && !contenidoNoCorresponde;
+      const postOk = (!hasError || hasSuccess) && !sesionExpirada && !contenidoNoCorresponde;
+
+      if (!postOk) {
+        return {
+          success: false,
+          error: errorMsg || undefined,
+          status: declareResponse.status,
+          rawHtml: body,
+          formHtml,
+          setsDeclarados: Object.keys(sets),
+          formDataSent: formData,
+        };
+      }
+
+      // Guardar HTML de pe_avance3 (respuesta de declaración) siempre para debug
+      {
+        const fs = require('fs');
+        const path = require('path');
+        const debugDir = process.env.SII_DEBUG_DIR || path.join(__dirname, '../../debug/cert-v2');
+        if (!fs.existsSync(debugDir)) fs.mkdirSync(debugDir, { recursive: true });
+        fs.writeFileSync(path.join(debugDir, 'pe_avance3_response.html'), body, 'utf8');
+
+      }
+
+      // 7. VERIFICACIÓN POST-DECLARACIÓN: Re-leer pe_avance2 y confirmar que los TrackIDs se guardaron
+      // Si aparece "EN REVISION" → la declaración fue aceptada y el SII está procesando
+      // Si reaparecen inputs vacíos → la declaración no se guardó
+      let verificado = true;
+      let verificacionError = '';
+      let enRevision = false;
+      let camposVacios = [];
+      try {
+        const verifyResponse = await this.session.submitForm(
+          '/cvc_cgi/dte/pe_avance2',
+          { RUT_EMP: this.rutEmpresa, DV_EMP: this.dvEmpresa, ACEPTAR: 'Continuar' },
+          'https://maullin.sii.cl/cvc_cgi/dte/pe_avance1'
+        );
+        const verifyHtml = verifyResponse.body || '';
+
+        // Guardar HTML de verificación pe_avance2 siempre
+        {
+          const fs = require('fs');
+          const path = require('path');
+          const debugDir = process.env.SII_DEBUG_DIR || path.join(__dirname, '../../debug/cert-v2');
+          if (!fs.existsSync(debugDir)) fs.mkdirSync(debugDir, { recursive: true });
+          fs.writeFileSync(path.join(debugDir, 'pe_avance2_verify.html'), verifyHtml, 'utf8');
+
+        }
+
+        // Para cada set que declaramos, verificar estado
+        const verifyRows = verifyHtml.split(/<\/tr>/i);
+        const camposEnRevision = [];
+        for (const [setName, index] of Object.entries(fieldMapping)) {
+          if (!sets[setName]) continue;
+          for (const row of verifyRows) {
+            const numEnvMatch = row.match(new RegExp(`NAME="NUM_ENV${index}"`, 'i'));
+            if (!numEnvMatch) {
+              // Si no hay input NUM_ENV, verificar si aparece REVISADO CONFORME o EN REVISION
+              const labelPattern = patterns.find(p => p.name === setName);
+              if (labelPattern && labelPattern.label.test(row)) {
+                if (/<b>[^<]*REVISADO CONFORME[^<]*<\/b>/i.test(row)) break;
+                if (/EN REVISION/i.test(row)) {
+                  camposEnRevision.push(setName);
+                  break;
+                }
+              }
+              continue;
+            }
+            // Tiene input — verificar si tiene valor o está REVISADO CONFORME
+            const tieneConforme = /<b>[^<]*REVISADO CONFORME[^<]*<\/b>/i.test(row);
+            if (tieneConforme) break;
+            if (/EN REVISION/i.test(row)) {
+              camposEnRevision.push(setName);
+              break;
+            }
+            const valueMatch = row.match(new RegExp(`NAME="NUM_ENV${index}"[^>]*value="([^"]*)"`, 'i'));
+            const tieneValor = valueMatch && valueMatch[1] && valueMatch[1].trim() !== '';
+            if (!tieneValor) {
+              camposVacios.push(setName);
+            }
+            break;
+          }
+        }
+
+        if (camposEnRevision.length > 0) {
+          enRevision = true;
+          console.log(` [...] EN REVISION: ${camposEnRevision.join(', ')} — declaración aceptada, SII procesando`);
+        }
+
+        if (camposVacios.length > 0 && !enRevision) {
+          verificado = false;
+          verificacionError = `Declaración NO se guardó en el portal SII. Campos vacíos para: ${camposVacios.join(', ')}. Posible error de sesión o TrackID no reconocido.`;
+          console.log(` [!] ${verificacionError}`);
+        }
+      } catch (verifyErr) {
+        console.log(` [!] No se pudo verificar la declaración: ${verifyErr.message}`);
+      }
+
+      // allRejected: SII rechazó todos los sets declarados — período incorrecto, no tiene sentido reintentar
+      const _declaredCount = Object.keys(fieldMapping).filter(k => sets[k]).length;
+      const allRejected = !enRevision && camposVacios.length > 0 && camposVacios.length >= _declaredCount && _declaredCount > 0;
 
       return {
-        success,
-        error: errorMsg || undefined,
+        success: verificado || enRevision,
+        error: verificacionError || errorMsg || undefined,
+        verificado,
+        enRevision,
+        allRejected,
         status: declareResponse.status,
         rawHtml: body,
+        formHtml,
         setsDeclarados: Object.keys(sets),
         formDataSent: formData,
       };
@@ -798,7 +926,7 @@ class SiiCertificacion {
         }
         const debugPath = path.join(debugDir, 'pe_avance2_avanzar.html');
         fs.writeFileSync(debugPath, formHtml, 'utf8');
-        console.log('   [DEBUG] HTML avanzar guardado en:', debugPath);
+        console.log(' [DEBUG] HTML avanzar guardado en:', debugPath);
       }
 
       let avanceResponse = await this.session.submitForm(
@@ -824,7 +952,7 @@ class SiiCertificacion {
         }
         const debugPath = path.join(debugDir, 'pe_avance3_avanzar_response.html');
         fs.writeFileSync(debugPath, body);
-        console.log('   [DEBUG] Respuesta avanzar guardada en:', debugPath);
+        console.log(' [DEBUG] Respuesta avanzar guardada en:', debugPath);
       }
 
       const hasError = body.toLowerCase().includes('error') && !body.includes('Error de Sesión');
@@ -1161,6 +1289,7 @@ class SiiCertificacion {
       if (onProgress) {
         onProgress({ intento, maxIntentos, estado: 'polling' });
       }
+      emitProgress(STEPS.POLLING, { intento, max: maxIntentos, label: 'sets' });
 
       const result = await this.verAvanceParsed();
       if (!result.success) continue;
@@ -1180,10 +1309,12 @@ class SiiCertificacion {
       }
 
       if (todosConformes) {
+        emitProgress(STEPS.SETS_APPROVED);
         return { success: true, estados: estadosRelevantes };
       }
 
       if (algunoRechazado) {
+        emitProgress(STEPS.SETS_REJECTED);
         return { success: false, error: 'Sets rechazados', estados: estadosRelevantes };
       }
     }

package/SiiPortalAuth.js

@@ -173,10 +173,10 @@ class SiiPortalAuth {
     if (cached) {
       const valida = await this._validarSesion(cached);
       if (valida) {
-        console.log('[SiiPortalAuth] ♻️  Reutilizando sesión SII cacheada');
+        console.log('[SiiPortalAuth] Reutilizando sesión SII cacheada');
         return cached;
       }
-      console.log('[SiiPortalAuth] ⚠️  Sesión cacheada expirada, re-autenticando...');
+      console.warn('[SiiPortalAuth] Sesión cacheada expirada, re-autenticando...');
     }
 
     // ── 2. Nueva autenticación ────────────────────────────────────────────────
@@ -187,6 +187,7 @@ class SiiPortalAuth {
       { cookieJar }
     );
 
+    console.log('[SiiPortalAuth] Autenticando con certificado en herculesr.sii.cl...');
     const r2 = await this._request(
       `https://herculesr.sii.cl/cgi_AUT2000/CAutInicio.cgi?${TARGET}`,
       {
@@ -196,6 +197,7 @@ class SiiPortalAuth {
         usarCert: true,
       }
     );
+    console.log(`[SiiPortalAuth] Respuesta herculesr: status=${r2.status}`);
 
     // Verificar mensaje de límite de sesiones
     if (r2.body.includes('m\u00e1ximo de sesiones') || r2.body.includes('maximo de sesiones') ||
@@ -222,19 +224,25 @@ class SiiPortalAuth {
    */
   async _validarSesion(cookieJar) {
     try {
+      console.log('[SiiPortalAuth] Validando sesión cacheada en SII...');
       const res = await this._request(
         'https://maullin.sii.cl/cvc_cgi/dte/ad_empresa1',
         { cookieJar: { ...cookieJar } } // copia para no contaminar
       );
       // Si redirige al login SII → expirada
       const loc = res.headers['location'] || '';
-      if (loc.includes('InicioAutenticacion') || loc.includes('IngresoRutClave')) return false;
+      if (loc.includes('InicioAutenticacion') || loc.includes('IngresoRutClave')) {
+        console.warn('[SiiPortalAuth] Validación: SII redirigió al login → sesión expirada');
+        return false;
+      }
       // Si el body contiene formulario de empresa → válida
       const valida = res.status === 200 && (res.body.includes('RUT_EMP') || res.body.includes('ad_empresa'));
+      console.log(`[SiiPortalAuth] Validación: status=${res.status} → sesión ${valida ? 'VÁLIDA ✓' : 'INVÁLIDA ✗'}`);
       // Refrescar timestamp del caché para extender TTL mientras la sesión se usa activamente
       if (valida) SiiPortalAuth._guardarSesionCache(this._certHash, cookieJar);
       return valida;
-    } catch {
+    } catch (err) {
+      console.warn('[SiiPortalAuth] Validación: error de red →', err.message);
       return false;
     }
   }
@@ -242,13 +250,25 @@ class SiiPortalAuth {
   /** Lee sesión cacheada del disco para el cert dado. @private */
   static _cargarSesionCache(certHash) {
     try {
-      if (!fs.existsSync(SESSION_CACHE_PATH)) return null;
+if (!fs.existsSync(SESSION_CACHE_PATH)) {
+        console.log('[SiiPortalAuth] Cache: archivo no existe →', SESSION_CACHE_PATH);
+        return null;
+      }
       const data = JSON.parse(fs.readFileSync(SESSION_CACHE_PATH, 'utf8'));
-      if (data.certHash !== certHash) return null;
+      if (data.certHash !== certHash) {
+        console.log(`[SiiPortalAuth] Cache: cert no coincide (guardado=${data.certHash} actual=${certHash})`);
+        return null;
+      }
+      const edadMin = Math.round((Date.now() - data.ts) / 60000);
       // TTL: 90 minutos (SII permite ~2h de inactividad; se refresca en cada validación)
-      if (Date.now() - data.ts > 90 * 60 * 1000) return null;
+      if (Date.now() - data.ts > 90 * 60 * 1000) {
+        console.log(`[SiiPortalAuth] Cache: sesión expirada (edad=${edadMin} min, TTL=90 min)`);
+        return null;
+      }
+      console.log(`[SiiPortalAuth] Cache: sesión encontrada`);
       return data.cookies;
-    } catch {
+    } catch (err) {
+      console.warn('[SiiPortalAuth] Cache: error leyendo caché →', err.message);
       return null;
     }
   }
@@ -262,7 +282,11 @@ class SiiPortalAuth {
         ts: Date.now(),
         cookies: cookieJar,
       }), 'utf8');
-    } catch { /* no crítico */ }
+      const cookieKeys = Object.keys(cookieJar);
+      console.log(`[SiiPortalAuth] Cache: sesión guardada`);
+    } catch (err) {
+      console.warn('[SiiPortalAuth] Cache: error guardando caché →', err.message);
+    }
   }
 
   /** Borra la sesión cacheada (útil para forzar re-login). */

package/SiiSession.js

@@ -279,7 +279,7 @@ class SiiSession {
     // Detectar bloqueo por demasiadas sesiones
     if (response.body && response.body.includes('superado el m')) {
       const errorMsg = 'SII: Demasiadas sesiones abiertas. Espera ~30 min o cierra sesión manualmente en el portal SII.';
-      console.error(`\n❌ ${errorMsg}\n`);
+      console.error(`\n[ERR] ${errorMsg}\n`);
       throw new Error(errorMsg);
     }
 
@@ -292,7 +292,7 @@ class SiiSession {
         // Verificar si el login resultó en bloqueo por sesiones
         if (response.body && response.body.includes('superado el m')) {
           const errorMsg = 'SII: Demasiadas sesiones abiertas. Espera ~30 min o cierra sesión manualmente en el portal SII.';
-          console.error(`\n❌ ${errorMsg}\n`);
+          console.error(`\n[ERR] ${errorMsg}\n`);
           throw new Error(errorMsg);
         }
       }
@@ -386,10 +386,10 @@ class SiiSession {
       }
       
       this.cookieJar = data.cookieJar || '';
-      console.log('Sesión SII cargada desde archivo');
+      console.log('[SessionSii] Sesión SII cargada desde archivo');
       return true;
     } catch (err) {
-      console.log('Error cargando sesión SII:', err.message);
+      console.log('[SessionSii] Error cargando sesión SII:', err.message);
       return false;
     }
   }