@gnpdev/rpa-tools 1.1.2 → 1.1.4

package/README.md

@@ -5,14 +5,14 @@ Herramientas de automatización para bots RPA: logging estructurado con **Pino**
 ## Requisitos
 
 - **Node.js** v18+
-- **pnpm** (recomendado)
+- **npm** o **pnpm**
 - Instancia de **MinIO** activa.
 - Base de datos **PostgreSQL**.
 
 ## Instalación
 
 ```bash
-pnpm add @gnpdev/rpa-tools
+npm install @gnpdev/rpa-tools
 ```
 
 ## Configuración Centralizada (Best Practice)
@@ -91,6 +91,10 @@ if (credentials) {
   // usar en el login de la web
 }
 
+// Si pueden existir múltiples registros con el mismo nombre:
+const allCredentials = await rpa.getCredentialsAll('Portal CRM');
+// allCredentials → Array de { nombre, usuario, password, idUsuario, status }
+
 // En caso de fallo de login o error en la aplicación
 await rpa.updateAppStatus('Portal CRM', false, 'Credenciales inválidas o bloqueo de cuenta');
 ```

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@gnpdev/rpa-tools",
-  "version": "1.1.2",
+  "version": "1.1.4",
   "description": "Libreria para logs y screenshot de bots",
   "author": "Sergio Antonio Trujillo del Valle",
   "main": "src/index.js",
@@ -21,10 +21,10 @@
     "pino": "^10.3.1",
     "pino-pretty": "^13.1.3",
     "sharp": "^0.34.5",
-    "pg": "^8.11.0"
+    "pg": "^8.11.0",
+    "@types/pg": "^8.20.0"
   },
   "devDependencies": {
-    "@types/pg": "^8.20.0",
     "@commitlint/cli": "^20.5.0",
     "@commitlint/config-conventional": "^20.5.0",
     "@release-it/conventional-changelog": "^10.0.6",

package/src/credentials.js

@@ -31,6 +31,22 @@ class CredentialManager {
     return rows.length > 0 ? rows[0] : null;
   }
 
+  /**
+   * Obtiene todas las credenciales que coincidan con el nombre y bot ID.
+   * @param {string} nombre - Nombre de la aplicación
+   * @returns {Promise<Array<{nombre: string, usuario: string, password: string, idUsuario: string, status: boolean}>>}
+   */
+  async getAppCredentialsAll(nombre) {
+    const { rows } = await this.pool.query(
+      `SELECT nombre, usuario, password, "idUsuario", status
+         FROM bots.tb_aplicaciones_bots
+        WHERE nombre = $1 AND bot_id = $2`,
+      [nombre, this.botId]
+    );
+
+    return rows;
+  }
+
   /**
    * Actualiza el estado y las observaciones de una aplicación.
    * @param {string} nombre - Nombre de la aplicación
@@ -48,6 +64,7 @@ class CredentialManager {
       );
       return true;
     } catch (err) {
+      console.error(`[Credentials] Error actualizando estado de "${nombre}":`, err.message);
       return false;
     }
   }

package/src/errorCapture.js

@@ -117,6 +117,9 @@ async function captureError({ page, context, err, botId, pool, minioClient, buck
   }
 
   // ── 3. Registro en bots.tb_error_bots ───────────────────────────────────
+  let pageUrl = null;
+  try { pageUrl = page.url(); } catch { /* page already closed */ }
+
   try {
     const { rows } = await pool.query(
       `INSERT INTO bots.tb_error_bots
@@ -127,7 +130,7 @@ async function captureError({ page, context, err, botId, pool, minioClient, buck
         botId,
         step                 ?? null,
         err.message,
-        page.url()           ?? null,
+        pageUrl,
         result.screenshotKey ?? null,
         result.traceKey      ?? null,
       ]
@@ -143,7 +146,7 @@ async function captureError({ page, context, err, botId, pool, minioClient, buck
   logger.error({
     ...stepInfo,
     err,
-    url:            page.url(),
+    url:            pageUrl,
     screenshotKey:  result.screenshotKey,
     traceKey:       result.traceKey,
     errorId:        result.errorId,

package/src/index.d.ts

@@ -2,14 +2,16 @@ import type { Pool }   from 'pg';
 import type { Logger }  from 'pino';
 import type { Page }    from 'playwright';
 
+export type { Logger };
+
 // ── Opciones de MinIO ────────────────────────────────────────────────────────
 export interface MinioConfig {
   endPoint:   string;
   port?:      number;
   useSSL?:    boolean;
-  accessKey:  string | any;
-  secretKey:  string | any;
-  bucket?:    string | any;
+  accessKey:  string;
+  secretKey:  string;
+  bucket?:    string;
 }
 
 // ── Opciones de screenshot ───────────────────────────────────────────────────
@@ -24,7 +26,7 @@ export interface ScreenshotConfig {
 
 // ── Opciones de Pino ─────────────────────────────────────────────────────────
 export interface LogConfig {
-  level?:       'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal' | 'silent' | any;
+  level?:       'trace' | 'debug' | 'info' | 'warn' | 'error' | 'fatal' | 'silent';
   /** Activar pino-pretty (solo dev). Default: false */
   pretty?:      boolean;
   pinoOptions?: Record<string, unknown>;
@@ -69,11 +71,17 @@ export interface RpaTools {
   watchDebugFlag: (page: any, pollMs?: number) => void;
 
   /**
-   * Recupera credenciales de una aplicación desde la base de datos.
+   * Recupera las credenciales de una aplicación desde la base de datos.
    * @param nombre - Nombre de la aplicación (ej. 'Portal CRM')
    */
   getCredentials: (nombre: string) => Promise<{ nombre: string; usuario: string; password: string; idUsuario: string; status: boolean } | null>;
 
+  /**
+   * Recupera TODAS las credenciales que coincidan con el nombre.
+   * @param nombre - Nombre de la aplicación (ej. 'Portal CRM')
+   */
+  getCredentialsAll: (nombre: string) => Promise<Array<{ nombre: string; usuario: string; password: string; idUsuario: string; status: boolean }>>;
+
   /**
    * Actualiza el estado y las observaciones de una aplicación.
    * @param nombre - Nombre de la aplicación (ej. 'Portal CRM')

package/src/index.js

@@ -15,7 +15,9 @@ const { PageCapturer }                      = require('./capture');
  * @param {object}  opts.minio          - config de MinIO
  * @param {object}  [opts.log]          - opciones de Pino
  *
- * @returns {{ logger, state, step, watchDebugFlag, getCredentials, updateAppStatus, isActive, capturePage, captureError, destroy }}
+ * @param {object}  [opts.screenshot]   - opciones de captura (formato, calidad)
+ *
+ * @returns {{ logger, state, step, watchDebugFlag, getCredentials, getCredentialsAll, updateAppStatus, isActive, capturePage, captureError, destroy }}
  */
 async function createRpaTools(opts = {}) {
   const {
@@ -23,6 +25,7 @@ async function createRpaTools(opts = {}) {
     log:  logCfg = {},
     db,
     minio: minioCfg,
+    screenshot: screenshotCfg,
   } = opts;
 
   const minio = minioCfg ?? {
@@ -44,7 +47,7 @@ async function createRpaTools(opts = {}) {
   await ensureBucket(minioClient, bucket);
   logger.info({ bucket }, 'MinIO listo');
 
-  const watcher     = new ScreenshotWatcher({ botId, pool: db, minioClient, bucket, logger });
+  const watcher     = new ScreenshotWatcher({ botId, pool: db, minioClient, bucket, logger, screenshot: screenshotCfg });
   const credentials = new CredentialManager({ botId, pool: db });
   const capturer    = new PageCapturer({ botId, minioClient, bucket, logger });
 
@@ -55,7 +58,7 @@ async function createRpaTools(opts = {}) {
     state,
     step: (name) => {
       state.currentStep = name;
-      logger.info(name + ' - Nuevo paso');
+      logger.info({ step: name }, 'Nuevo paso');
     },
     /**
      * Inicia el polling que activa/desactiva screenshots según bot_debug_config.
@@ -64,6 +67,7 @@ async function createRpaTools(opts = {}) {
      */
     watchDebugFlag: (page, pollMs) => watcher.watch(page, pollMs),
     getCredentials: (nombre) => credentials.getAppCredentials(nombre),
+    getCredentialsAll: (nombre) => credentials.getAppCredentialsAll(nombre),
     updateAppStatus: (nombre, status, observations) => credentials.updateAppStatus(nombre, status, observations),
     /**
      * Consulta si el bot está activo en la base de datos (columna 'estado').

package/src/storage.js

@@ -1,6 +1,15 @@
 'use strict';
 const Minio = require('minio');
 const sharp = require('sharp');
+const os = require('os');
+
+/**
+ * Retorna el identificador de la instancia actual (pod en K8s).
+ * En Kubernetes se usa process.env.HOSTNAME (nombre del pod).
+ */
+function getInstanceId() {
+  return process.env.HOSTNAME || os.hostname();
+}
 
 /**
  * Crea y retorna el cliente de MinIO.
@@ -49,6 +58,10 @@ async function toWebp(buffer, quality = 70) {
  * Si format='webp' convierte el buffer antes de subir.
  * Ruta generada: {botId}/{YYYY-MM-DD}/{ISO-timestamp}.{ext}
  *
+ * En BD guarda un arreglo JSONB en url_screenshot con la estructura:
+ *   [{ instancia: string, url: string, timestamp: string }]
+ * Cada instancia (pod) actualiza su propia entrada.
+ *
  * @param {import('minio').Client} client
  * @param {string} bucket
  * @param {string} botId
@@ -58,10 +71,11 @@ async function toWebp(buffer, quality = 70) {
  * @param {number}         [opts.quality=70]      - calidad 1-100
  * @param {object}         [opts.pool]            - Pool de base de datos
  * @param {object}         [opts.logger]          - Instancia de logger
+ * @param {string}         [opts.instancia]       - ID de instancia (default: HOSTNAME)
  * @returns {Promise<string>}   - object key en MinIO
  */
 async function uploadScreenshot(client, bucket, botId, buffer, opts = {}) {
-  const { format = 'webp', quality = 70, pool, logger } = opts;
+  const { format = 'webp', quality = 70, pool, logger, instancia = getInstanceId() } = opts;
 
   const finalBuffer = format === 'webp'
     ? await toWebp(buffer, quality)
@@ -69,29 +83,55 @@ async function uploadScreenshot(client, bucket, botId, buffer, opts = {}) {
 
   const contentType = format === 'webp' ? 'image/webp' : 'image/jpeg';
   const ext         = format === 'webp' ? 'webp' : 'jpg';
-  const key  = `${botId}.${ext}`;
+  const now  = new Date();
+  const date = now.toISOString().slice(0, 10);
+  const ts   = now.toISOString().replace(/[:.]/g, '-');
+  const key  = `capturas/${botId}/${date}/${ts}.${ext}`;
 
   await client.putObject(bucket, key, finalBuffer, finalBuffer.length, {
     'Content-Type': contentType,
   });
 
-  // ── 3. Registro en base de datos ─────────────────────────────────────────
+  // ── 3. Registro en base de datos (arreglo JSONB por instancia) ──────────
   if (pool) {
+    const pgClient = await pool.connect();
     try {
-      await pool.query(
+      await pgClient.query('BEGIN');
+
+      const { rows } = await pgClient.query(
+        `SELECT url_screenshot FROM bots.tb_bots WHERE id = $1::uuid FOR UPDATE`,
+        [botId]
+      );
+
+      const actual = (rows[0]?.url_screenshot || []);
+      const idx = actual.findIndex(e => e.instancia === instancia);
+
+      const entry = { instancia, url: key, timestamp: now.toISOString() };
+
+      if (idx >= 0) {
+        actual[idx] = entry;
+      } else {
+        actual.push(entry);
+      }
+
+      await pgClient.query(
         `UPDATE bots.tb_bots
-         SET url_screenshot = $1,
-             updated_at = NOW()
-         WHERE id = $2::uuid`,
-        [key, botId]
+            SET url_screenshot = $1::jsonb,
+                updated_at = NOW()
+          WHERE id = $2::uuid`,
+        [JSON.stringify(actual), botId]
       );
-      
+
+      await pgClient.query('COMMIT');
     } catch (err) {
+      await pgClient.query('ROLLBACK').catch(() => {});
       if (logger) {
-        logger.error({ err, botId, key }, 'No se pudo actualizar url_screenshot en BD');
+        logger.error({ err, botId, key, instancia }, 'No se pudo actualizar url_screenshot en BD');
       } else {
         console.error(`[Storage] Error BD (${botId}):`, err.message);
       }
+    } finally {
+      pgClient.release();
     }
   }