nexusflex-mcp 3.70.0 → 3.71.0

package/api.mjs

@@ -47,8 +47,19 @@ if (EMAIL && PASSWORD && !/^https:\/\//i.test(API_URL)) {
   log("ADVERTENCIA: NEXUSFLEX_API_URL no es HTTPS y hay email/contraseña — viajarían sin cifrar.");
 }
 
-function guard(path) {
+// Operaciones habilitadas dentro de rutas financieras (decisión del dueño, 29/09), con su
+// MÉTODO exacto — mismas que RUTAS_MCP_OPERACION_OK en backend/src/middleware/auth.ts.
+// Ninguna paga, cobra ni transfiere: excluir/reincluir un envío de una liquidación (con
+// motivo) y ver/preparar los datos de facturación de un cliente (no emite comprobantes).
+const OPERACIONES_OK = [
+  { metodo: "POST", ruta: /^\/liquidaciones\/anteriores\/\d+\/(excluir|reincluir)\/?$/i },
+  { metodo: "GET", ruta: /^\/afip\/estado\/?$/i },
+  { metodo: "POST", ruta: /^\/afip\/preparar-cliente\/?$/i },
+];
+
+function guard(method, path) {
   const clean = path.split("?")[0];
+  if (OPERACIONES_OK.some((o) => o.metodo === String(method).toUpperCase() && o.ruta.test(clean))) return;
   if (RUTAS_PROHIBIDAS.some((re) => re.test(clean))) {
     throw new Error(`Ruta bloqueada por política del MCP (endpoints de dinero deshabilitados): ${clean}`);
   }
@@ -96,7 +107,7 @@ async function ensureToken() {
 
 /** Llamada autenticada a la API. Reintenta UNA vez si el token venció/se revocó. */
 export async function api(method, path, body) {
-  guard(path);
+  guard(method, path);
   let token = await ensureToken();
   const doFetch = (t) =>
     fetch(`${API_URL}${path}`, {

package/docs.mjs

@@ -158,7 +158,7 @@ const GROUPS = [
   { titulo: "WMS del nodo", tools: ["stock_nodo", "productos_nodo", "producto_crear"] },
   { titulo: "Gesti\xF3n del nodo", tools: ["clientes_del_nodo", "cliente_crear", "cliente_editar", "sucursales_cliente", "sucursal_perfil", "precios_ver", "precio_actualizar", "cuentas_vinculadas", "generar_enlace_vinculacion", "facturacion_estado", "facturacion_preparar_cliente"] },
   { titulo: "Marketing / atribuci\xF3n", tools: ["marketing_config_ver", "marketing_config_guardar"] },
-  { titulo: "Liquidaciones (consulta)", tools: ["liquidacion_buscar", "liquidacion_detalle", "liquidaciones_sin_recibir", "liquidacion_excluir_envio", "liquidacion_reincluir_envio", "cuenta_semanal_nodos"] },
+  { titulo: "Liquidaciones", tools: ["liquidacion_buscar", "liquidacion_detalle", "liquidaciones_sin_recibir", "liquidacion_excluir_envio", "liquidacion_reincluir_envio", "cuenta_semanal_nodos"] },
   { titulo: "Usuarios / choferes", tools: ["chofer_listar", "chofer_crear", "chofer_editar", "usuario_editar", "operador_crear", "cliente_generar_usuario", "cliente_resetear_clave", "usuario_habilitar_reparto"] },
   { titulo: "Administraci\xF3n (red)", tools: ["nodos_listar", "nodo_crear", "usuario_habilitar_rol", "grupo_crear", "logo_subir", "sugerencias_listar", "cierre_semanal_links", "nodo_renombrar", "grupo_miembros", "grupo_sumar_nodo", "grupo_expulsar_nodo", "grupo_admin_permiso"] }
 ];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nexusflex-mcp",
-  "version": "3.70.0",
+  "version": "3.71.0",
   "description": "MCP de Nexus Flex: operá tu nodo/cuenta desde un asistente de IA (altas de clientes, stock, KPIs). Login por autorización web (device-flow). NO toca dinero.",
   "type": "module",
   "license": "MIT",

package/server.mjs

@@ -56,8 +56,9 @@ const ALLOW_PRECIOS = truthy(process.env.NEXUSFLEX_MCP_ALLOW_PRECIOS);
 
 // ATADO CON: backend/src/services/mcp-remote.ts (MCP_VERSION + NOVEDADES + mismos tools, salvo SOLO_REMOTO)
 // y mcp/package.json. Lo verifica backend/src/coherencia.test.ts. Ver CLAUDE.md → "Cosas que van juntas".
-const MCP_VERSION = "3.70.0"; // bumpear junto con package.json + backend/services/mcp-remote.ts
+const MCP_VERSION = "3.71.0"; // bumpear junto con package.json + backend/services/mcp-remote.ts
 const NOVEDADES = [
+  "3.71.0 — Habilitadas por el dueño: `liquidacion_excluir_envio` / `liquidacion_reincluir_envio` (sacar o devolver UN envío del cobro de una liquidación emitida, con motivo obligatorio) y `facturacion_estado` / `facturacion_preparar_cliente` (qué falta para facturar y cargar los datos fiscales de un cliente) ya funcionan por MCP: el filtro de dinero las deja pasar con su método exacto. Siguen sin pagar, cobrar ni emitir comprobantes, y piden permiso finanzas. Además `mis_envios` ahora sí filtra por `estado`.",
   "3.70.0 — Descripciones al día: textos de las herramientas corregidos para que digan lo que hacen hoy (`envio_entregar`, `envio_asignar_mensajero`, `generar_enlace_vinculacion` vence en 48 hs, `grupo_miembros` y `provisorio_conciliar` apuntan a `grupos_tarifas`), `flujo` lista todos los flujos disponibles, `sugerencias_listar` solo para el superadmin, `envio_cargar` exige `telefono`, parámetros con formato (fechas YYYY-MM-DD, montos en ARS, nombre o id) y las instrucciones del conector más cortas (la guía sigue en `guia`). `liquidacion_excluir_envio`/`reincluir` avisan que hoy el filtro de dinero los bloquea por MCP (se hace desde la PWA).",
   "3.69.1 — AYUDA completa: `ayuda` ahora tiene la ficha (qué hace, cómo usarlo, ejemplo, qué NO hace) de los 20 tools que no la tenían (cierre semanal, zonas de grupo, tarifas de grupo, sin vendedor, corregir estado, pago al cadete, retiros…) y los 26 que no aparecían en el índice. La ayuda del paquete npm se genera desde la del conector, así que ya no se desincronizan. No mueve dinero.",
   "3.69.0 — Control por foto de la TANDA DE UN CADETE (propios + de otros nodos mezclados): en `envio_desde_etiqueta_ml` alcanza con el `mlQr` crudo — de ahí sale el número de ML y el vendedor, y el vendedor se reconoce igual que en el escáner (cuenta vinculada O cuenta aprendida en juntadas/escaneos; antes solo la vinculada). `nodoEntrega` ya no necesita `grupo`: si el cliente es del mismo nodo que entrega es un paquete propio (sin traspaso) y si es de otro nodo se usa el grupo que comparten. Si no conoce al vendedor, contesta 404 con el senderId para que preguntes de qué nodo/cliente es. Si el envío ya existía, no lo duplica: lo completa y lo avanza. No mueve dinero.",
@@ -254,7 +255,7 @@ if (isCliente) {
   tool("mis_envios", {
     title: "Mis envíos",
     description: "Tus envíos/paquetes (solo los tuyos), los 300 más recientes, cada uno con su `estado`. No incluye datos de otros clientes ni del nodo.",
-    inputSchema: { estado: z.string().optional().describe("Estado: 'A retirar', 'Colectado', 'En centro de distribución', 'En camino', 'Entregado' (o una novedad, ej. 'Comprador ausente'). Ojo: hoy el backend no aplica este filtro y devuelve todos; filtrá vos por el campo `estado` de cada envío.") },
+    inputSchema: { estado: z.string().optional().describe("Estado: 'A retirar', 'Colectado', 'En centro de distribución', 'En camino', 'Entregado' (o una novedad, ej. 'Comprador ausente'). Filtra por los que EMPIEZAN con ese texto ('Entregado' incluye 'Entregado (Flex)'); los 300 más recientes de ese estado.") },
   }, async ({ estado }) => run(() => api("GET", `/portal/envios${q({ estado })}`)));
 
   tool("mis_kpis", {
@@ -1192,7 +1193,7 @@ if (ALLOW_WRITE) {
     tool("liquidacion_excluir_envio", {
       title: "No cobrar un envío de una liquidación",
       description:
-        "Saca UN envío (por tracking) del cobro de una liquidación YA emitida y re-suma el total, ajustando el cargo en la cuenta corriente del cliente. El `motivo` es OBLIGATORIO: queda en el historial del envío y el vendedor lo ve en su portal. Si esa liquidación ya está en una factura de ARCA emitida, lo rechaza (haría falta una nota de crédito, que el sistema no emite). Scopeada a tu nodo. Hoy el filtro de dinero del backend bloquea esta ruta vía MCP (403): se hace desde la PWA.",
+        "Saca UN envío (por tracking) del cobro de una liquidación YA emitida y re-suma el total, ajustando el cargo en la cuenta corriente del cliente. El `motivo` es OBLIGATORIO: queda en el historial del envío y el vendedor lo ve en su portal. Si esa liquidación ya está en una factura de ARCA emitida, lo rechaza (haría falta una nota de crédito, que el sistema no emite). Scopeada a tu nodo.",
       inputSchema: {
         id: z.number().int().positive().describe("id de la liquidación"),
         tracking: z.string().min(1),
@@ -1202,7 +1203,7 @@ if (ALLOW_WRITE) {
     tool("liquidacion_reincluir_envio", {
       title: "Volver a cobrar un envío de una liquidación",
       description:
-        "Lo inverso de `liquidacion_excluir_envio`: devuelve al cobro un envío que se había excluido, re-suma y ajusta la cuenta corriente. El `motivo` es OBLIGATORIO. Mismo guard de factura emitida. Hoy el filtro de dinero del backend bloquea esta ruta vía MCP (403): se hace desde la PWA.",
+        "Lo inverso de `liquidacion_excluir_envio`: devuelve al cobro un envío que se había excluido, re-suma y ajusta la cuenta corriente. El `motivo` es OBLIGATORIO. Mismo guard de factura emitida.",
       inputSchema: {
         id: z.number().int().positive().describe("id de la liquidación"),
         tracking: z.string().min(1),