@arela/uploader 1.1.2 → 1.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@arela/uploader",
-  "version": "1.1.2",
+  "version": "1.1.4",
   "description": "CLI to upload files/directories to Arela",
   "bin": {
     "arela": "./src/index.js"

package/src/config/config.js

@@ -37,10 +37,10 @@ class Config {
       const __dirname = path.dirname(__filename);
       const packageJsonPath = path.resolve(__dirname, '../../package.json');
       const packageJson = JSON.parse(fs.readFileSync(packageJsonPath, 'utf-8'));
-      return packageJson.version || '1.1.2';
+      return packageJson.version || '1.1.4';
     } catch (error) {
       console.warn('⚠️ Could not read package.json version, using fallback');
-      return '1.1.2';
+      return '1.1.4';
     }
   }
 

package/src/document-type-shared.js

@@ -100,7 +100,7 @@ export function extractDocumentFields(source, fileExtension, filePath) {
 
       // Resolve final type if the definition supports it (e.g., pedimento_simplificado vs proforma)
       const resolvedType = docType.resolveType
-        ? docType.resolveType(fields)
+        ? docType.resolveType(fields, filePath)
         : docType.type;
 
       console.log(`   → Resolved type: ${resolvedType}`);

package/src/document-types/pedimento-completo.js

@@ -74,8 +74,15 @@ export const pedimentoCompletoDefinition = {
    *   - R1 rectifications require fechaPagoRectificacion
    *   - Everything else requires paymentDate
    * No payment evidence ⇒ proforma_completo.
+   *
+   * Exception: the "CoveFact" annex (…-CoveFact.pdf) doesn't always print the
+   * payment section, so the payment-based demotion misfires and the file gets
+   * stuck as proforma_completo (NON_PUSHABLE) forever — the CLI never
+   * re-identifies rows that already have a detected_type. The filename is the
+   * payment-independent signal for this variant.
    */
-  resolveType: (fields) => {
+  resolveType: (fields, filePath) => {
+    if (/covefact/i.test(filePath ?? '')) return 'pedimento_completo';
     const clavePedimento =
       fields?.find((f) => f.name === 'clavePedimento')?.value ?? null;
     const paymentDate =

package/src/file-detection.js

@@ -5,6 +5,15 @@ import { PDFParse } from 'pdf-parse';
 import { extractDocumentFields } from './document-type-shared.js';
 import { classifyDocument } from './scoring/scoring-engine.js';
 
+// pdf.js can reject internal worker promises on corrupt PDFs (e.g.
+// XRefEntryException) that no try/catch around getText() can reach. Track the
+// last PDF handed to the parser so the global unhandledRejection handler can
+// name it; the event fires after finally blocks run, so it is never cleared.
+let currentPdfFile = null;
+function getCurrentPdfFile() {
+  return currentPdfFile;
+}
+
 // Document types that participate in arela_path composition.
 const ARELA_PATH_TYPES = new Set([
   'pedimento_simplificado',
@@ -47,6 +56,15 @@ function composeArelaPath(
     return null;
   }
 
+  // The CoveFact annex resolves to pedimento_completo without payment
+  // evidence (see pedimento-completo.js resolveType), so it must never anchor
+  // an operation on its own: with no self-composed path it only becomes
+  // pushable when a PAID sibling pedimento propagates the folder's
+  // arela_path. Unpaid operations therefore never reach Arela through it.
+  if (/covefact/i.test(filePath ?? '')) {
+    return null;
+  }
+
   const rfc = fields?.find((f) => f.name === 'rfc')?.value;
   let patente = fields?.find((f) => f.name === 'patente')?.value;
   const aduana = fields?.find((f) => f.name === 'aduanaEntradaSalida')?.value;
@@ -255,6 +273,7 @@ export class FileDetectionService {
    */
   async extractTextFromPDF(filePath) {
     let parser;
+    currentPdfFile = filePath;
     try {
       const dataBuffer = fs.readFileSync(filePath);
       const uint8Array = new Uint8Array(dataBuffer);
@@ -316,4 +335,4 @@ export class FileDetectionService {
 }
 
 export default FileDetectionService;
-export { composeArelaPath };
+export { composeArelaPath, getCurrentPdfFile };

package/src/index.js

@@ -13,8 +13,21 @@ import watchCommand from './commands/WatchCommand.js';
 import workerCommand from './commands/WorkerCommand.js';
 import appConfig from './config/config.js';
 import ErrorHandler from './errors/ErrorHandler.js';
+import { getCurrentPdfFile } from './file-detection.js';
 import logger from './services/LoggingService.js';
 
+// Exception names thrown by pdf.js (used via pdf-parse) when a PDF is
+// malformed, encrypted, or truncated. Matched by error.name because the
+// classes live inside pdfjs-dist and are not exported by pdf-parse.
+const PDFJS_PARSE_ERRORS = new Set([
+  'XRefEntryException',
+  'XRefParseException',
+  'InvalidPDFException',
+  'MissingPDFException',
+  'PasswordException',
+  'FormatError',
+]);
+
 /**
  * Arela Uploader CLI
  * Professional file uploader with document detection and organization
@@ -660,6 +673,22 @@ class ArelaUploaderCLI {
     process.on('unhandledRejection', (reason, promise) => {
       const error =
         reason instanceof Error ? reason : new Error(String(reason));
+
+      // pdf.js rejects internal worker promises when a PDF is corrupt; those
+      // rejections cannot be caught around the parser call, and one bad file
+      // must not abort the rest of the batch.
+      if (PDFJS_PARSE_ERRORS.has(error.name)) {
+        const file = getCurrentPdfFile();
+        console.warn(
+          `⚠️ Corrupt or unreadable PDF${file ? ` (${file})` : ''}: ${error.message} — skipping`,
+        );
+        this.errorHandler.handleError(error, {
+          context: 'unhandledRejection',
+          file,
+        });
+        return;
+      }
+
       this.errorHandler.handleFatalError(error, {
         context: 'unhandledRejection',
         promise: promise.toString(),

package/src/scoring/scoring-engine.js

@@ -225,7 +225,7 @@ export function classifyDocument(matchers, { source, extension, filePath }) {
   }
 
   const resolvedType = def.resolveType
-    ? def.resolveType(fields)
+    ? def.resolveType(fields, filePath)
     : def.documentType;
   const pedimento = def.extractNumPedimento
     ? def.extractNumPedimento(source, fields, filePath)