@deinossrl/dgp-agent 1.5.22 → 1.5.24

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog - DGP Agent
 
+## [1.5.24] - 2026-01-13
+
+### Changed
+- Improved progress calculation in `pg_restore`: "dropping" actions are now logged but do not increment the processed object count, preventing the progress from exceeding 100%. "From TOC entry" lines are ignored to avoid double counting.
+
+## [1.5.23] - 2026-01-13
+### Fixed
+- **Critical**: Lógica invertida en flags `include_owner` e `include_privileges`. Ahora se respeta correctamente la exclusión con `--no-owner` y `--no-privileges` cuando el usuario desmarca las opciones.
+
+## [1.5.23] - 2026-01-13
+
+### Changed
+- Improved progress calculation in `pg_restore`: "dropping" actions are now logged but do not increment the processed object count, preventing the progress from exceeding 100%. "From TOC entry" lines are ignored to avoid double counting.
+
 ## [1.5.22] - 2026-01-13
 
 ### Fixed

package/index.mjs

@@ -1746,10 +1746,12 @@ async function executePgDump(command) {
     }
 
     // Incluir opciones adicionales
-    if (params.include_privileges) {
+    // NOTA: pg_dump incluye owner y privilegios por defecto.
+    // Usamos --no-owner y --no-privileges para excluirlos.
+    if (!params.include_privileges) {
       pgDumpArgs.push('--no-privileges');
     }
-    if (params.include_owner) {
+    if (!params.include_owner) {
       pgDumpArgs.push('--no-owner');
     }
 
@@ -2381,26 +2383,34 @@ async function executePgRestore(command) {
           const isLoopMessage = trimmed.includes('processing item') || trimmed.includes('processing data');
           const now = Date.now();
 
-          // LOGICA: Filtrar, calcular y MOSTRAR PROGRESO (Consola y Web)
-          // "from TOC entry" es clave para objetos metadata
-          // "dropping" es para actividad de limpieza (si --clean)
-          if (trimmed.includes('processing') || trimmed.includes('creating') || trimmed.includes('restoring') || trimmed.includes('creando') || trimmed.includes('procesando') || trimmed.includes('from TOC entry') || trimmed.includes('dropping')) {
-            processedObjects++;
+          // LOGICA: Filtrar, calcular y MOSTRAR PROGRESO
+          const isDropping = trimmed.includes('dropping');
+          const isCreating = trimmed.includes('processing') || trimmed.includes('creating') || trimmed.includes('restoring') || trimmed.includes('creando') || trimmed.includes('procesando');
+
+          if (isDropping || isCreating) {
+            // Solo incrementar si estamos CREANDO/RESTAURANDO (el TOC cuenta los objetos a crear)
+            // Ignoramos 'dropping' para el contador (es fase de limpieza) y 'from TOC entry' (es redundante)
+            if (isCreating) {
+              processedObjects++;
+            }
 
-            // Si es un item procesado, mostrar progreso en consola
+            // Mostrar progreso en consola (throttled)
             if (now - lastLogTime > 500) { // Cada 500ms localmente
               let progressMsg = '';
+              const safeProcessed = Math.min(processedObjects, totalObjects || processedObjects); // Cap visualmente
+
               if (totalObjects > 0) {
-                const percentage = Math.min(Math.round((processedObjects / totalObjects) * 100), 100);
-                progressMsg = `Progreso: ${percentage}% (${processedObjects}/${totalObjects})`;
+                const percentage = Math.min(Math.round((safeProcessed / totalObjects) * 100), 100);
+                progressMsg = `Progreso: ${percentage}% (${safeProcessed}/${totalObjects})`;
               } else {
-                progressMsg = `Objetos procesados: ${processedObjects}`;
+                progressMsg = `Objetos procesados: ${safeProcessed}`;
               }
 
-              // Limpiar detalle
+              // Limpiar detalle para el log
               let detail = trimmed;
               if (detail.includes('processing item')) detail = detail.replace('processing item', '').trim();
               if (detail.includes('creating')) detail = detail.replace('creating', 'creando').trim();
+              if (detail.includes('dropping')) detail = detail.replace('dropping', 'limpiando').trim();
 
               progressMsg += ` - ${detail.substring(0, 50)}`;
               console.log(`${colors.cyan}[INFO] ${progressMsg}${colors.reset}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deinossrl/dgp-agent",
-  "version": "1.5.22",
+  "version": "1.5.24",
   "description": "Agente local para Despliegue-GPT - Reporta el estado del repositorio Git a la plataforma TenMinute IA",
   "main": "index.mjs",
   "bin": {