feat: bienvenida diaria; fix: el NLU filtraba JSON crudo al usuario
Bienvenida una vez por dia y por empresa, distinta por categoria (cat 1 no ve
ejemplos de descarga). Se manda aparte del retorno de process(), asi el usuario
recibe el saludo y acto seguido lo que pidio. __saludo sobrevive a
preservingReset para no repetirse tras cada informe.
Fix del leak: cuando la respuesta del modelo no parseaba como JSON se enviaba
cruda, y el usuario veia {"action":"chat","text":"... Ahora se rescata solo el
texto, incluso de JSON truncado, y si no hay nada legible se devuelve vacio
para caer al menu de fallback.
Ademas el prompt le prohibe preguntar por datos que una opcion ya pide: ante
"descargar informe de mantenimiento" debe rutear al selector de grupo en vez de
preguntar por chat.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Lizandro Guarnizo
co-authored by
Claude Sonnet 4.6
Claude Sonnet 4.6
parent
8825f4e821
commit
0bb1d3447e
+31
-3
@@ -360,7 +360,10 @@ PROMPT;
|
||||
$json = json_decode($clean, true);
|
||||
|
||||
if (!is_array($json) || !isset($json['action'])) {
|
||||
return ['action' => 'chat', 'text' => $clean];
|
||||
// JSON inválido o truncado. Mandar $clean tal cual le escupía el
|
||||
// {"action":"chat",...} al usuario, así que se rescata solo el texto.
|
||||
self::log("NLU: respuesta no parseable → " . mb_substr($clean, 0, 200));
|
||||
return ['action' => 'chat', 'text' => self::rescatarTexto($clean)];
|
||||
}
|
||||
|
||||
if ($json['action'] === 'route' && isset($json['key'])) {
|
||||
@@ -377,7 +380,29 @@ PROMPT;
|
||||
}
|
||||
}
|
||||
|
||||
return ['action' => 'chat', 'text' => $json['text'] ?? $clean];
|
||||
return ['action' => 'chat', 'text' => self::rescatarTexto((string)($json['text'] ?? $clean))];
|
||||
}
|
||||
|
||||
/**
|
||||
* Extrae el texto legible de una respuesta que no parseó (típicamente JSON
|
||||
* truncado por límite de tokens). Si no hay nada rescatable devuelve '',
|
||||
* y NormalBot cae al menú de fallback — mejor eso que mostrarle JSON.
|
||||
*/
|
||||
private static function rescatarTexto(string $crudo): string
|
||||
{
|
||||
$crudo = trim($crudo);
|
||||
if ($crudo === '') return '';
|
||||
|
||||
// "text":"lo que sirve → rescatar aunque falte el cierre
|
||||
if (preg_match('/"text"\s*:\s*"(.*?)(?:"\s*[,}]|$)/s', $crudo, $m)) {
|
||||
$texto = stripcslashes($m[1]);
|
||||
return str_contains($texto, '{"action"') ? '' : trim($texto);
|
||||
}
|
||||
|
||||
// Cualquier resto con pinta de JSON o de key interna no se muestra
|
||||
if (str_contains($crudo, '{"') || str_contains($crudo, '"action"')) return '';
|
||||
|
||||
return $crudo;
|
||||
}
|
||||
|
||||
private static function buildFlowCatalog(array $config, int $permType): array
|
||||
@@ -460,7 +485,10 @@ PROMPT;
|
||||
. " \"qué lotes llevan más tiempo sin corona\" → {\"action\":\"route\",\"key\":\"ciclo_mantenimiento_top\",\"entities\":{\"grupo\":\"corona\"}}\n"
|
||||
. " \"mantenimiento\" (sin grupo) → {\"action\":\"route\",\"key\":\"ciclo_mantenimiento\"}\n"
|
||||
. " \"quiero cambiar de finca\" → {\"action\":\"route\",\"key\":\"reset_finca\"}\n\n"
|
||||
. "Si no está claro o no hay opción correspondiente → responde SOLO:\n"
|
||||
. "NUNCA preguntes por un dato que una opción ya pide (grupo, finca, período):\n"
|
||||
. "ruteá a esa opción y el bot lo pregunta con su propia lista.\n"
|
||||
. " \"descargar informe de mantenimiento\" → route a ciclo_mantenimiento, NO preguntar el grupo.\n\n"
|
||||
. "Usá chat solo si el mensaje no corresponde a ninguna opción → responde SOLO:\n"
|
||||
. "{\"action\":\"chat\",\"text\":\"<respuesta corta en español, máx 2 oraciones>\"}\n\n"
|
||||
. "No inventes keys. Usa exactamente los keys de la lista.";
|
||||
}
|
||||
|
||||
+53
-1
@@ -19,6 +19,10 @@ class NormalBot
|
||||
$currentNode = $botCtx['current_node'];
|
||||
$normalized = self::normalize($input);
|
||||
|
||||
// 0. Saludo del día — va aparte y el flujo sigue igual, así el usuario
|
||||
// recibe la bienvenida y acto seguido lo que haya pedido.
|
||||
self::saludarUnaVezAlDia($config, $context, $company, $ctxId, $permType);
|
||||
|
||||
// 1. Commands always win — escape from any state (salir, menu, etc.)
|
||||
foreach ($commands as $keyword => $action) {
|
||||
if ($normalized === self::normalize((string)$keyword)) {
|
||||
@@ -383,6 +387,53 @@ class NormalBot
|
||||
return self::buildDynamicListResponse($items, $flow, $context['from'], $company);
|
||||
}
|
||||
|
||||
/**
|
||||
* Bienvenida una vez por día y por empresa. Se envía por fuera del retorno
|
||||
* de process() —igual que hace el selector multi-empresa— para no reemplazar
|
||||
* la respuesta que el usuario vino a buscar.
|
||||
*/
|
||||
private static function saludarUnaVezAlDia(array $config, array $context, array $company, int $ctxId, string $permType): void
|
||||
{
|
||||
$welcome = $config['welcome'] ?? [];
|
||||
if (empty($welcome['enabled'])) return;
|
||||
|
||||
$texto = (string)($welcome['text'][$permType] ?? '');
|
||||
if (trim($texto) === '') return;
|
||||
|
||||
$meta = ConversationContext::getMetadata($ctxId);
|
||||
$hoy = date('Y-m-d'); // el punto de entrada fija America/Bogota
|
||||
if (($meta['__saludo'] ?? '') === $hoy) return;
|
||||
|
||||
$meta['__saludo'] = $hoy;
|
||||
ConversationContext::updateMetadata($ctxId, $meta);
|
||||
|
||||
$nombre = trim((string)($context['name'] ?? ''));
|
||||
$texto = strtr($texto, [
|
||||
'{saludo}' => $nombre !== '' ? "Hola *{$nombre}*" : 'Hola',
|
||||
'{empresa}' => (string)($company['display_name'] ?: $company['name'] ?? ''),
|
||||
'{finca}' => (string)($meta['finca']['finca_label'] ?? ''),
|
||||
'{empresas}' => self::esMultiEmpresa($context['from'] ?? '')
|
||||
? "\n *cambiar empresa* para consultar otra empresa"
|
||||
: '',
|
||||
]);
|
||||
|
||||
WhatsAppSender::sendText($context['from'], $texto, (string)($context['phone_number_id'] ?? ''));
|
||||
}
|
||||
|
||||
/** Solo tiene sentido ofrecer "cambiar empresa" a quien pertenece a varias. */
|
||||
private static function esMultiEmpresa(string $waNumber): bool
|
||||
{
|
||||
if ($waNumber === '') return false;
|
||||
try {
|
||||
$stmt = db()->prepare("SELECT companies_json FROM multi_company_sessions WHERE wa_number = ? LIMIT 1");
|
||||
$stmt->execute([$waNumber]);
|
||||
$json = $stmt->fetchColumn();
|
||||
return $json !== false && count(json_decode((string)$json, true) ?: []) > 1;
|
||||
} catch (\PDOException $e) {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Guarda la opción elegida con su etiqueta, para poder mostrarle al usuario
|
||||
* en qué finca/grupo está parado sin tener que volver a consultar la API.
|
||||
@@ -1276,7 +1327,8 @@ class NormalBot
|
||||
$keep = [];
|
||||
// finca y grupo_mant se conservan: los submenús ofrecen varios informes sobre
|
||||
// la misma selección, y perderla dejaba {finca_id}/{grupo_id} sin resolver.
|
||||
foreach (['finca', 'grupo_mant'] as $group) {
|
||||
// __saludo también, si no se re-saluda después de cada informe.
|
||||
foreach (['finca', 'grupo_mant', '__saludo'] as $group) {
|
||||
if (isset($meta[$group])) $keep[$group] = $meta[$group];
|
||||
}
|
||||
ConversationContext::reset($ctxId);
|
||||
|
||||
Reference in New Issue
Block a user