@alvarobr/n8n-nodes-glpi-connect 1.0.1 → 1.2.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
@@ -38,6 +38,8 @@ Creado por **Álvaro Blanca Rodríguez**.
 ### Otros
 - **Usuario → Identificar por correo**: usuario de GLPI asociado a un correo.
 - **Base de conocimiento → Buscar preguntas frecuentes**: artículos FAQ públicos.
+- **Base de conocimiento → Índice de preguntas frecuentes**: títulos e ID de todas las FAQ públicas. Páselo al prompt del agente para que sepa qué artículos existen y no busque a ciegas.
+- **Base de conocimiento → Leer artículo**: contenido completo de una FAQ pública por su ID.
 - **Petición API → Petición personalizada**: cualquier ruta de `apirest.php` con la sesión ya gestionada. No comprueba permisos por usuario, así que **no debe conectarse a un agente de IA**.
 
 ## Seguridad

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',
@@ -112,7 +112,11 @@ const properties = [
         type: 'options',
         noDataExpression: true,
         displayOptions: show(['conocimiento']),
-        options: [{ name: 'Buscar preguntas frecuentes', value: 'buscar', action: 'Buscar en la base de conocimiento' }],
+        options: [
+            { name: 'Buscar preguntas frecuentes', value: 'buscar', action: 'Buscar en la base de conocimiento' },
+            { name: 'Índice de preguntas frecuentes', value: 'indice', action: 'Listar los títulos de la base de conocimiento', description: 'Títulos e ID de las preguntas frecuentes públicas, para que la IA sepa qué hay antes de consultar' },
+            { name: 'Leer artículo', value: 'leer', action: 'Leer un artículo de la base de conocimiento', description: 'Contenido completo de una pregunta frecuente por su ID' },
+        ],
         default: 'buscar',
     },
     {
@@ -262,6 +266,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']),
@@ -349,7 +354,15 @@ const properties = [
         required: true,
         default: '',
         placeholder: 'vpn',
-        displayOptions: show(['conocimiento']),
+        displayOptions: show(['conocimiento'], ['buscar']),
+    },
+    {
+        displayName: 'ID del artículo',
+        name: 'articuloId',
+        type: 'number',
+        required: true,
+        default: 0,
+        displayOptions: show(['conocimiento'], ['leer']),
     },
     // --- Petición API ---------------------------------------------------
     {
@@ -461,6 +474,10 @@ async function runOperation(ctx, client, users, i) {
     const resource = ctx.getNodeParameter('resource', i);
     const operation = ctx.getNodeParameter('operation', i);
     if (resource === 'conocimiento') {
+        if (operation === 'indice')
+            return ops.faqIndex(client);
+        if (operation === 'leer')
+            return ops.readFaq(client, Number(ctx.getNodeParameter('articuloId', i)));
         return ops.searchFaq(client, ctx.getNodeParameter('busqueda', i));
     }
     if (resource === 'api') {

package/dist/nodes/GlpiConnect/operations.js

@@ -21,6 +21,8 @@ exports.linkAsset = linkAsset;
 exports.createTicket = createTicket;
 exports.myAssets = myAssets;
 exports.computerSheet = computerSheet;
+exports.faqIndex = faqIndex;
+exports.readFaq = readFaq;
 exports.searchFaq = searchFaq;
 exports.apiRequest = apiRequest;
 const GlpiClient_1 = require("./GlpiClient");
@@ -341,6 +343,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 +372,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) {
@@ -638,6 +709,37 @@ async function computerSheet(client, user, computerId, search) {
 // ----------------------------------------------------------------------
 // Base de conocimiento
 // ----------------------------------------------------------------------
+const FAQ_INDEX_LIMIT = 200;
+/** Solo las preguntas frecuentes se consideran públicas para el usuario. */
+function isPublicFaq(article) {
+    return !!article && typeof article === 'object' && (article.is_faq === undefined || Number(article.is_faq) === 1);
+}
+/** Títulos de todas las preguntas frecuentes, para que la IA sepa qué existe antes de consultar. */
+async function faqIndex(client) {
+    const response = await client.raw('GET', 'KnowbaseItem', { qs: { range: `0-${FAQ_INDEX_LIMIT - 1}`, sort: 'name', order: 'ASC' } });
+    if (response.statusCode >= 300 || !Array.isArray(response.body)) {
+        return { ok: false, error: 'No se ha podido consultar la base de conocimiento.' };
+    }
+    const articles = response.body.filter(isPublicFaq).map((a) => ({ id: Number(a.id), titulo: (0, utils_1.plainText)(a.name, 200) }));
+    return (0, utils_1.compact)({
+        ok: true,
+        total: articles.length,
+        articulos: articles,
+        aviso: response.body.length >= FAQ_INDEX_LIMIT ? `Solo se muestran los ${FAQ_INDEX_LIMIT} primeros artículos.` : undefined,
+    });
+}
+/** Contenido completo de una pregunta frecuente. */
+async function readFaq(client, articleId) {
+    const id = Number(articleId);
+    if (!Number.isSafeInteger(id) || id <= 0)
+        return { ok: false, error: 'Falta el ID del artículo. Use el índice de la base de conocimiento.' };
+    const response = await client.raw('GET', `KnowbaseItem/${id}`);
+    if (response.statusCode >= 300 || !response.body || Array.isArray(response.body) || !isPublicFaq(response.body)) {
+        return { ok: false, error: `No existe una pregunta frecuente pública con el ID ${id}.` };
+    }
+    const a = response.body;
+    return { ok: true, articulo: { id, titulo: (0, utils_1.plainText)(a.name, 200), contenido: (0, utils_1.plainText)(a.answer, 8000), enlace: `${client.front}/helpdesk.faq.php?id=${id}` } };
+}
 async function searchFaq(client, text) {
     const term = String(text ?? '').trim();
     if (term.length < 2)
@@ -651,10 +753,9 @@ async function searchFaq(client, text) {
         if (r.statusCode >= 300 || !Array.isArray(r.body))
             continue;
         for (const a of r.body) {
-            if (!a || typeof a !== 'object' || seen.has(a.id))
+            if (!a || seen.has(a.id))
                 continue;
-            // Solo preguntas frecuentes (públicas)
-            if (a.is_faq !== undefined && Number(a.is_faq) !== 1)
+            if (!isPublicFaq(a))
                 continue;
             seen.add(a.id);
             articles.push({ id: a.id, titulo: (0, utils_1.plainText)(a.name, 200), contenido: (0, utils_1.plainText)(a.answer, 1500), enlace: `${client.front}/helpdesk.faq.php?id=${a.id}` });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alvarobr/n8n-nodes-glpi-connect",
-  "version": "1.0.1",
+  "version": "1.2.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",