@alvarobr/n8n-nodes-glpi-connect 1.0.1 → 1.1.0

package/README.md

@@ -28,7 +28,7 @@ Creado por **Álvaro Blanca Rodríguez**.
 - **Detalle**: técnicos, grupos, seguimientos y tareas públicos, soluciones, aprobaciones y activos asociados.
 - **Crear**: incidencia o solicitud con el usuario como solicitante y, opcionalmente, su equipo afectado. Antes de crearla avisa de posibles duplicados.
 - **Añadir comentario**: seguimiento firmado con el nombre del usuario. En un ticket resuelto solicita su reapertura.
-- **Cerrar o cancelar**: registra la solución («solucionado» o «cancelado») y cierra el ticket.
+- **Cerrar o cancelar**: registra un seguimiento con la confirmación del usuario, asigna el ticket a la cuenta técnica de la credencial (sin quitar a los técnicos anteriores), añade la solución, lo cierra y vuelve a consultarlo para confirmar el estado final. El motivo puede ser «solucionado», «cancelado» o «acción automática completada».
 - **Asociar activo**: relación nativa ticket-elemento de GLPI, solo con activos del usuario y sin duplicados.
 
 ### Activos

package/dist/nodes/GlpiConnect/GlpiClient.js

@@ -23,6 +23,7 @@ class GlpiClient {
         this.config = config;
         this.http = http;
         this.sessionToken = null;
+        this.sessionUser = null;
         const base = config.url.trim().replace(/\/+$/, '').replace(/\/apirest\.php$/, '');
         this.api = `${base}/apirest.php`;
         this.front = `${base}/front`;
@@ -59,6 +60,18 @@ class GlpiClient {
             // La sesión caducará sola en GLPI
         }
         this.sessionToken = null;
+        this.sessionUser = null;
+    }
+    /** ID del usuario de GLPI con el que se ha iniciado sesión (la cuenta técnica del token). */
+    async sessionUserId() {
+        if (this.sessionUser)
+            return this.sessionUser;
+        const body = await this.get('getFullSession');
+        const id = Number(body?.session?.glpiID);
+        if (!Number.isSafeInteger(id) || id <= 0)
+            throw new GlpiError('GLPI no ha indicado el usuario de la sesión.', 200, 'NO_SESSION_USER');
+        this.sessionUser = id;
+        return id;
     }
     /** Petición que devuelve el estado y el cuerpo sin lanzar error (para decidir según la respuesta). */
     async raw(method, path, options = {}) {

package/dist/nodes/GlpiConnect/GlpiConnect.node.js

@@ -80,7 +80,7 @@ const properties = [
             { name: 'Detalle', value: 'detalle', action: 'Ver el detalle de un ticket del usuario', description: 'Técnicos, seguimientos, tareas, soluciones, aprobaciones y activos asociados' },
             { name: 'Crear', value: 'crear', action: 'Crear un ticket para el usuario', description: 'Crea una incidencia o solicitud, opcionalmente asociada a un activo del usuario' },
             { name: 'Añadir comentario', value: 'comentar', action: 'Añadir un comentario a un ticket del usuario', description: 'Añade un seguimiento; en un ticket resuelto solicita su reapertura' },
-            { name: 'Cerrar o cancelar', value: 'cerrar', action: 'Cerrar o cancelar un ticket del usuario', description: 'Registra la solución y cierra el ticket' },
+            { name: 'Cerrar o cancelar', value: 'cerrar', action: 'Cerrar o cancelar un ticket del usuario', description: 'Registra un seguimiento, asigna el ticket a la cuenta de la integración, lo cierra y comprueba el estado final' },
             { name: 'Asociar activo', value: 'asociarActivo', action: 'Asociar un activo del usuario a su ticket', description: 'Crea la relación nativa ticket-elemento de GLPI' },
         ],
         default: 'listar',
@@ -262,6 +262,7 @@ const properties = [
         options: [
             { name: 'Solucionado', value: 'solucionado' },
             { name: 'Cancelado (ya no se necesita)', value: 'cancelado' },
+            { name: 'Acción automática completada', value: 'automatizado' },
         ],
         default: 'solucionado',
         displayOptions: show(['ticket'], ['cerrar']),

package/dist/nodes/GlpiConnect/operations.js

@@ -341,6 +341,28 @@ async function addFollowup(client, user, ticketId, text, control) {
         return { ok: false, ejecutado: false, error: 'GLPI no ha aceptado el comentario. No se ha modificado el ticket.', detalle: glpiDetail(error), ticket };
     }
 }
+const CLOSE_TEXTS = {
+    solucionado: {
+        label: 'Cerrar como solucionado',
+        followup: 'El usuario ha confirmado la resolución de la incidencia.',
+        solution: 'El usuario indica que el problema se ha solucionado.',
+    },
+    cancelado: {
+        label: 'Cancelar',
+        followup: 'El usuario ha confirmado que ya no necesita esta solicitud y pide cancelarla.',
+        solution: 'Solicitud cancelada por el usuario: ya no la necesita.',
+    },
+    automatizado: {
+        label: 'Cerrar tras la acción automática',
+        followup: 'Acción ejecutada automáticamente por el asistente a petición del usuario.',
+        solution: 'Resuelto mediante una acción automática del asistente de soporte.',
+    },
+};
+/**
+ * Cierre completo: seguimiento, asignación a la cuenta técnica de la integración,
+ * solución, estado Cerrado y comprobación final en GLPI.
+ * Los técnicos, seguimientos y soluciones anteriores se conservan.
+ */
 async function closeTicket(client, user, ticketId, reason, text, control) {
     const access = await loadEditableTicket(client, user, ticketId, true);
     if (!isAccess(access))
@@ -348,34 +370,81 @@ async function closeTicket(client, user, ticketId, reason, text, control) {
     const ticket = access.summary;
     if (ticket.estado_id === 6)
         return { ok: true, ejecutado: false, mensaje: `El ticket #${ticket.id} ya estaba cerrado.`, ticket };
+    const texts = CLOSE_TEXTS[reason] ?? CLOSE_TEXTS.solucionado;
     const explanation = String(text ?? '').trim();
-    const label = reason === 'cancelado' ? 'Cancelar' : 'Cerrar como solucionado';
+    const extra = explanation ? `<p>${(0, utils_1.escapeHtml)(explanation)}</p>` : '';
     const { execute, warning } = decide(control);
     if (!execute) {
-        return preview(`${label} el ticket #${ticket.id} «${ticket.titulo}»${explanation ? ` indicando: "${explanation}"` : ''}`, warning, { ticket });
+        return preview(`${texts.label} el ticket #${ticket.id} «${ticket.titulo}»${explanation ? ` indicando: "${explanation}"` : ''}`, warning, { ticket });
+    }
+    // 1. Seguimiento con la confirmación del usuario
+    try {
+        await client.call('POST', 'ITILFollowup', {
+            body: { input: { itemtype: 'Ticket', items_id: ticket.id, is_private: 0, content: `<p>${texts.followup}</p>${extra}${signature(user)}` } },
+        });
+    }
+    catch (error) {
+        return { ok: false, ejecutado: false, error: `GLPI no ha permitido registrar el seguimiento del ticket #${ticket.id}. No se ha modificado.`, detalle: glpiDetail(error), ticket };
+    }
+    // 2. Asignar a la cuenta técnica (se añade; los técnicos anteriores se mantienen)
+    const avisos = [];
+    try {
+        const agentId = await client.sessionUserId();
+        const actors = await client.get(`Ticket/${ticket.id}/Ticket_User`).catch(() => []);
+        const assigned = (Array.isArray(actors) ? actors : []).some((a) => a && Number(a.users_id) === agentId && Number(a.type) === 2);
+        if (!assigned)
+            await client.call('POST', 'Ticket_User', { body: { input: { tickets_id: ticket.id, users_id: agentId, type: 2 } } });
+    }
+    catch (error) {
+        avisos.push(`No se ha podido asignar el ticket a la cuenta técnica: ${glpiDetail(error)}`);
     }
-    // Primero la solución (si no la tiene), después el cierre
+    // 3. Solución (GLPI la necesita para pasar a Resuelto) y 4. Cerrado
     if (ticket.estado_id !== 5) {
-        const heading = reason === 'cancelado' ? 'Solicitud cancelada por el usuario: ya no la necesita.' : 'El usuario indica que el problema se ha solucionado.';
         try {
             await client.call('POST', 'ITILSolution', {
-                body: { input: { itemtype: 'Ticket', items_id: ticket.id, content: `<p>${heading}</p>${explanation ? `<p>${(0, utils_1.escapeHtml)(explanation)}</p>` : ''}${signature(user)}` } },
+                body: { input: { itemtype: 'Ticket', items_id: ticket.id, content: `<p>${texts.solution}</p>${extra}${signature(user)}` } },
             });
         }
         catch (error) {
-            return { ok: false, ejecutado: false, error: `GLPI no ha permitido cerrar el ticket #${ticket.id}. No se ha modificado.`, detalle: glpiDetail(error), ticket };
+            return (0, utils_1.compact)({
+                ok: false,
+                ejecutado: 'parcial',
+                error: `Se ha registrado el seguimiento, pero GLPI no ha permitido resolver el ticket #${ticket.id}, así que sigue abierto.`,
+                detalle: glpiDetail(error),
+                avisos: avisos.length ? avisos : undefined,
+                ticket,
+            });
         }
     }
-    const closed = await client.raw('PUT', `Ticket/${ticket.id}`, { body: { input: { id: ticket.id, status: 6 } } });
-    const closedOk = closed.statusCode < 300 && Array.isArray(closed.body) && closed.body.some((x) => x && x[String(ticket.id)] === true);
-    if (closedOk)
-        return { ok: true, ejecutado: true, mensaje: `El ticket #${ticket.id} ha quedado cerrado.`, ticket };
-    return {
-        ok: true,
+    await client.raw('PUT', `Ticket/${ticket.id}`, { body: { input: { id: ticket.id, status: 6 } } });
+    // 5. Comprobar en GLPI el estado final antes de dar el cierre por hecho
+    const check = await client.raw('GET', `Ticket/${ticket.id}`);
+    const finalStatus = check.statusCode < 300 && check.body && !Array.isArray(check.body) ? Number(check.body.status) : NaN;
+    const final = { ...ticket, estado: utils_1.TICKET_STATUS[finalStatus] ?? 'Desconocido', estado_id: finalStatus };
+    const notes = avisos.length ? avisos : undefined;
+    if (finalStatus === 6) {
+        return (0, utils_1.compact)({ ok: true, ejecutado: true, verificado: true, mensaje: `GLPI confirma que el ticket #${ticket.id} está cerrado.`, avisos: notes, ticket: final });
+    }
+    if (finalStatus === 5) {
+        return (0, utils_1.compact)({
+            ok: true,
+            ejecutado: 'parcial',
+            verificado: true,
+            mensaje: `El ticket #${ticket.id} ha quedado como Resuelto, pero GLPI no ha permitido cerrarlo directamente. Se cerrará automáticamente según la configuración de GLPI.`,
+            avisos: notes,
+            ticket: final,
+        });
+    }
+    return (0, utils_1.compact)({
+        ok: false,
         ejecutado: 'parcial',
-        mensaje: `Se ha registrado la solución y el ticket #${ticket.id} ha quedado como Resuelto, pero GLPI no ha permitido cerrarlo directamente. Se cerrará automáticamente según la configuración de GLPI.`,
-        ticket,
-    };
+        verificado: Number.isFinite(finalStatus),
+        error: Number.isFinite(finalStatus)
+            ? `GLPI indica que el ticket #${ticket.id} sigue en estado «${final.estado}». No se ha cerrado.`
+            : `No se ha podido comprobar en GLPI el estado final del ticket #${ticket.id}.`,
+        avisos: notes,
+        ticket: final,
+    });
 }
 /** Comprueba que un activo esté asignado al usuario. */
 async function loadOwnAsset(client, user, itemtype, itemsId) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alvarobr/n8n-nodes-glpi-connect",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "description": "Nodo de n8n para GLPI: autoservicio IT seguro para flujos y agentes de IA. Tickets, inventario, software y base de conocimiento con sesión automática, permisos por usuario y confirmación de cambios.",
   "keywords": [
     "n8n-community-node-package",