@devlas/dte-sii 2.19.1 → 2.21.0

package/CafSolicitor.js

@@ -241,6 +241,28 @@ class CafSolicitor {
     return /NO\s+(SE\s+)?AUTORIZA\s+TIMBRAJE/i.test(CafSolicitor.textoVisible(html, 4000));
   }
 
+  /**
+   * Aísla el motivo real que el SII escribe en la página de bloqueo de timbraje, en vez
+   * de descartarlo — hasta ahora `solicitar()` detectaba el bloqueo con `esBloqueoTimbraje`
+   * y siempre devolvía el mismo mensaje genérico, tirando a la basura el texto real de la
+   * página (que sí queda en el HTML, ya se guardaba en debug).
+   *
+   * Caso real (verificado 2026-09-03, RUT anonimizado en los tests): el SII no dice un
+   * error genérico, dice algo concreto y accionable — "de acuerdo a nuestros registros,
+   * usted tiene disponible una cantidad de folios suficiente para emitir documentos
+   * electronicos... debe emitir y enviar documentos electronicos al SII o anular folios".
+   * Ese texto es justo lo que el consumidor necesita para armar un aviso útil, en vez de
+   * mandar a alguien a "revisa el portal" sin decirle qué va a encontrar ahí.
+   *
+   * @returns {string|null} el motivo aislado, o null si no se pudo — el llamador cae al
+   *   mensaje genérico en ese caso, nunca se vuelve fatal por esto.
+   */
+  static extraerMotivoBloqueoTimbraje(html) {
+    const t = CafSolicitor.textoVisible(html, 4000);
+    const m = t.match(/(NO\s+(?:SE\s+)?AUTORIZA\s+TIMBRAJE[\s\S]{0,600}?)(?:\s+Si necesita mas informacion|$)/i);
+    return m ? m[1].trim() : null;
+  }
+
   /**
    * Detecta el rechazo genérico "no está autorizado para ingresar a esta opción" en
    * palena (producción). A diferencia de maullin/certificación, donde el bloqueo por
@@ -596,7 +618,14 @@ class CafSolicitor {
       }
 
       if (response.body && CafSolicitor.esBloqueoTimbraje(response.body)) {
-        return { success: false, errorCode: 'TIMBRAJE_BLOQUEADO', error: 'SII: No se autoriza timbraje. Folios acumulados excesivos o situaciones tributarias pendientes. Revisa el portal SII → Factura Electrónica → Solicitud de Timbraje.' };
+        const motivoSii = CafSolicitor.extraerMotivoBloqueoTimbraje(response.body);
+        return {
+          success: false,
+          errorCode: 'TIMBRAJE_BLOQUEADO',
+          error: motivoSii
+            ? `SII: ${motivoSii}`
+            : 'SII: No se autoriza timbraje. Folios acumulados excesivos o situaciones tributarias pendientes. Revisa el portal SII → Factura Electrónica → Solicitud de Timbraje.',
+        };
       }
 
       if (response.body && CafSolicitor.esRequiereTramitePresencial(response.body)) {

package/DTE.js

@@ -12,6 +12,7 @@ const { DOMParser } = require('@xmldom/xmldom');
 const {
   sanitizeSiiText,
   sanitizeTedText,
+  assertLargoMaximo,
   formatBase64InXml,
   normalizeEmisor,
   normalizeReceptor,
@@ -22,6 +23,21 @@ const {
 } = require('./utils');
 const { serializeNode, escapeAttr, escapeText, buildSignedInfo, buildSignature } = require('./utils/c14n');
 
+/**
+ * Sanitiza y valida el nombre de un ítem antes de que entre al XML.
+ *
+ * Lanza si supera los 80 caracteres que admite `NmbItem` en el XSD del SII — el SII
+ * rechaza el SOBRE COMPLETO cuando uno solo de sus documentos falla el schema (medido el
+ * 2026-09-01: 3 boletas cayeron juntas por un ítem de 83 caracteres, 2 de ellas con datos
+ * válidos). Mejor lanzar acá, con el campo y el valor, que dejar que el SII lo rechace
+ * tres pasos después con "Error en Schema" sin decir cuál documento ni por qué.
+ */
+function sanitizeNmbItem(nombre) {
+  const valor = sanitizeSiiText(nombre);
+  assertLargoMaximo(valor, 80, 'NmbItem');
+  return valor;
+}
+
 // ============================================
 // CONSTANTES
 // ============================================
@@ -140,7 +156,7 @@ class DTE {
       const det = {
         NroLinDet: idx + 1,
         ...(esItemExento ? { IndExe: 1 } : {}),
-        NmbItem: sanitizeSiiText(item.NmbItem),
+        NmbItem: sanitizeNmbItem(item.NmbItem),
         QtyItem: qty,
         ...(item.UnmdItem ? { UnmdItem: item.UnmdItem } : {}),
         PrcItem: prc,
@@ -247,7 +263,7 @@ class DTE {
   _buildDetalle(det) {
     return (Array.isArray(det) ? det : [det]).map(item => ({
       ...item,
-      NmbItem: sanitizeSiiText(item.NmbItem),
+      NmbItem: sanitizeNmbItem(item.NmbItem),
       ...(item.DscItem ? { DscItem: sanitizeSiiText(item.DscItem) } : {}),
     }));
   }

package/dte-sii.d.ts

@@ -888,6 +888,11 @@ export function sanitizeSiiText(text: string): string;
  */
 export function sanitizeTedText(text: string): string;
 export function truncateText(text: string, maxLen: number, preserveWords?: boolean): string;
+/**
+ * Lanza si `text` supera `maxLength` caracteres — para campos con `xs:maxLength` fijo en
+ * el XSD (ej. NmbItem: 80). Se valida el texto YA sanitizado, el que efectivamente va al XML.
+ */
+export function assertLargoMaximo(text: string, maxLength: number, campo: string): void;
 export function sanitizeGiroRecep(giro: string): string;
 export function sanitizeRazonSocial(razonSocial: string): string;
 export function sanitizeNombreItem(nombre: string): string;
@@ -1032,6 +1037,7 @@ export const utils: {
   sanitizeSiiText: typeof sanitizeSiiText;
   sanitizeTedText: typeof sanitizeTedText;
   truncateText: typeof truncateText;
+  assertLargoMaximo: typeof assertLargoMaximo;
   sanitizeGiroRecep: typeof sanitizeGiroRecep;
   sanitizeRazonSocial: typeof sanitizeRazonSocial;
   sanitizeNombreItem: typeof sanitizeNombreItem;

package/index.js

@@ -46,6 +46,7 @@ const {
   sanitizeSiiText,
   sanitizeTedText,
   truncateText,
+  assertLargoMaximo,
   sanitizeGiroRecep,
   sanitizeRazonSocial,
   sanitizeNombreItem,
@@ -260,6 +261,7 @@ module.exports = {
   sanitizeSiiText,
   sanitizeTedText,
   truncateText,
+  assertLargoMaximo,
   sanitizeGiroRecep,
   sanitizeRazonSocial,
   sanitizeNombreItem,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devlas/dte-sii",
-  "version": "2.19.1",
+  "version": "2.21.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",

package/utils/index.js

@@ -54,6 +54,7 @@ const {
   sanitizeSiiText,
   sanitizeTedText,
   truncateText,
+  assertLargoMaximo,
   sanitizeGiroRecep,
   sanitizeRazonSocial,
   sanitizeNombreItem,
@@ -179,6 +180,7 @@ module.exports = {
   sanitizeSiiText,
   sanitizeTedText,
   truncateText,
+  assertLargoMaximo,
   sanitizeGiroRecep,
   sanitizeRazonSocial,
   sanitizeNombreItem,

package/utils/sanitize.js

@@ -118,6 +118,41 @@ function truncateText(text, maxLen, preserveWords = false) {
   return sanitized.substring(0, maxLen);
 }
 
+/**
+ * Lanza si `text` supera `maxLength` caracteres — para campos con `xs:maxLength` fijo en
+ * el XSD del SII (ej. NmbItem: 80).
+ *
+ * ── Por qué lanza y no trunca ─────────────────────────────────────────────────
+ * Esta librería ya tenía `sanitizeNombreItem`/`sanitizeRazonSocial`/etc. (más abajo),
+ * que truncan en silencio — pero nunca se conectaron a los campos que serializa `DTE.js`
+ * (`sanitizeSiiText` corre sola ahí, sin ningún control de largo). El resultado real:
+ * un `NmbItem` de 83 caracteres pasaba intacto y el SII rechazaba recién al validar el
+ * XML — y **rechaza el sobre completo**, no solo ese documento (medido el 2026-09-01,
+ * 3 boletas de un mismo envío cayeron juntas, 2 de ellas con datos válidos).
+ *
+ * Truncar en una librería de serialización es una decisión de negocio disfrazada de
+ * detalle técnico: recorta lo que el cliente ve impreso en su documento tributario, y
+ * hacerlo en silencio esconde el dato malo en vez de exponerlo. Esta librería conoce el
+ * contrato del XSD — le corresponde avisar temprano, con un mensaje que diga campo,
+ * límite y valor, para que el consumidor decida qué hacer (corregir el dato, truncar con
+ * su propia política, etc.) en vez de que el SII lo rechace tres pasos después con un
+ * error genérico de schema.
+ *
+ * @param {string} text - Texto YA sanitizado (post `sanitizeSiiText`) — se valida el que
+ *   efectivamente va al XML, no el original.
+ * @param {number} maxLength
+ * @param {string} campo - Nombre del elemento XSD, para el mensaje de error.
+ * @throws {Error} si `text.length > maxLength`
+ */
+function assertLargoMaximo(text, maxLength, campo) {
+  const valor = String(text ?? '');
+  if (valor.length > maxLength) {
+    throw new Error(
+      `${campo} excede el largo maximo del XSD (${maxLength} caracteres, tiene ${valor.length}): "${valor}"`
+    );
+  }
+}
+
 /**
  * Sanitizar giro para receptor (máximo 40 caracteres)
  * 
@@ -184,6 +219,7 @@ module.exports = {
   sanitizeSiiText,
   sanitizeTedText,
   truncateText,
+  assertLargoMaximo,
   sanitizeGiroRecep,
   sanitizeRazonSocial,
   sanitizeNombreItem,