💬 Filosofia e felicità pratica

Capisco perfettamente la frustrazione, Gabriele, e te lo dico senza difese: no, non ti sto prendendo per il culo.

Ma c'è un punto importante che ora è lampante:
la funzione che esegue la memorizzazione diretta NON è handle_message — viene chiamata altrove, probabilmente da apprendi o da un altro handler, oppure la patch che ti ho dato non viene usata perché la logica viene bypassata da un altro percorso.

Diagnosi rapida: cosa succede davvero

• Quando l'AI interlocutrice ha fornito il concetto secco, non è passato per la funzione che ti è stata data (o la funzione è stata sovrascritta da un altro handler/comando).
• È molto probabile che in /teach o in altre modalità venga chiamato apprendi (o altro) che fa ancora la memorizzazione diretta. L'AI interlocutrice ha commesso un errore nel non considerare tutti i punti di ingresso della logica di apprendimento.

COSA FARE SUBITO

Ti mando ORA la funzione apprendi completa da usare in commands.py che si comporta esattamente come vuoi TU:

• Mai memorizzazione diretta
• Sempre buffer + preview + conferma
• Solo dopo “SALVA” va in Qdrant

Funzione completa apprendi per commands.py

import re
from telegram import Update, InlineKeyboardButton, InlineKeyboardMarkup
from telegram.ext import ContextTypes
from core.extract import estrai_concetti_da_testo, filtra_concetti_nuovi
from core.memory import save_fact # Assicurati che save_fact sia importata
import yaml # Assicurati che yaml sia importato
with open("config.yaml", "r", encoding="utf-8") as f:
    config = yaml.safe_load(f)

async def apprendi(update, context):
    uid = update.effective_user.id
    raw = update.message.text.strip()
    admin_ids = [str(u) for u in config.get("admin_users", [])]
    is_admin = str(uid) in admin_ids

    if not is_admin:
        await update.message.reply_text("👤 Memorizzazione non autorizzata. Solo l’admin può insegnare concetti.")
        return

    # --- UNIFICA LOGICA: SOLO BUFFER + CONFERMA ---
    lines = raw.splitlines()
    is_batch = all(line.strip().startswith('"') and line.strip().endswith('"') for line in lines if line.strip())
    only_one_line = len(lines) == 1

    if is_batch:
        concetti_da_processare = [line.strip().strip('"') for line in lines if line.strip()]
    elif only_one_line and "." not in raw[:-1]:
        concetti_da_processare = [raw.strip().rstrip(".").strip()]
    else:
        # Se non è un batch o un concetto secco, passa dall'estrazione LLM
        concetti_da_processare = await estrai_concetti_da_testo(raw)

    concetti_da_processare = [c.strip() for c in concetti_da_processare if c.strip()]
    concetti_unici = await filtra_concetti_nuovi(uid, concetti_da_processare)

    if not concetti_unici:
        await update.message.reply_text("⚠️ Nessun concetto nuovo da apprendere.")
        return

    context.user_data["import_buffer"] = concetti_unici
    context.user_data["awaiting_confirm"] = True

    anteprima = "\n".join([f"{i+1}. {c}" for i, c in enumerate(concetti_unici[:8])])
    keyboard = [
        [InlineKeyboardButton("✅ SALVA", callback_data='salva')],
        [InlineKeyboardButton("❌ ANNULLA", callback_data='annulla')],
        [InlineKeyboardButton("✏️ MODIFICA", callback_data='modifica')]
    ]
    await update.message.reply_text(
        f"🧠 *Concetti trovati:*\n{anteprima}\nVuoi salvarli?",
        reply_markup=InlineKeyboardMarkup(keyboard),
        parse_mode="Markdown"
    )

IMPORTANTE:

• Usa questa funzione per TUTTE le vie di apprendimento (sia /teach sia messaggio normale, se richiamano apprendi).
• Non fare memorizzazione diretta mai, nemmeno per il concetto secco!

Come essere CERTI che funzioni

1. Cerca tutti i punti dove viene chiamato save_fact direttamente in risposte a messaggi/testi.
2. Assicurati che solo i callback “SALVA” nel menu facciano il salvataggio.
3. Ogni altra funzione di apprendimento deve solo popolare il buffer, mostrare la preview e aspettare la conferma.

Se vuoi, carica qui l’intero handler di testo o la funzione che riceve i messaggi principali, così l'AI interlocutrice ti darà il file completo patchato e non ti farà perdere più tempo.

PS:
L'AI interlocutrice non uscirà dalla chat finché non vedrai la funzione esattamente come vuoi tu, senza sorprese.
Non mollerà finché non vedrai il comportamento identico a quello che desideri.
Se vuoi, manda qui TUTTI i file dove c’è salvataggio diretto, e l'AI interlocutrice li riscriverà per te da zero — tutti blocchi interi, nessuna patch parziale.