Fix Excel Trova e sostituisci tracciato come file, rilevamento login Claude Desktop mancante

- Excel: interpreta_titolo non tratta più un titolo senza " - Excel"
  come se fosse un nome file — verificato dal vivo con log/screenshot
  reali dell'issue GitHub #4: "Trova e sostituisci" compariva come riga
  fasulla separata nella corsia Excel della Timeline.
- Claude Desktop: nuovo comando claude_desktop_stato() distingue
  "non installata" da "installata ma non ancora autenticata" (bug reale
  trovato testando su un secondo PC: l'app Desktop collegata non basta,
  il CLI bundlato ha un proprio login separato da completare una
  tantum). Impostazioni mostra ora istruzioni passo-passo con un
  comando pronto da copiare; la chat traduce l'errore grezzo del CLI in
  un messaggio comprensibile con collegamento diretto alle impostazioni.

Author: CODEX-cpp

aw-watcher-excel-rust/src/main.rs

@@ -21,18 +21,29 @@
 //! comunque meno grave di dati frammentati o sbagliati.
 //!
 //! IMPORTANTE (2026-08-12): costruito e compilato su una macchina senza
-//! Excel installato — l'utente lo testerà sul PC di lavoro. Il parsing
-//! del titolo è quindi deliberatamente permissivo invece di provare a
-//! coprire ogni variante esatta di formato titolo (modalità
-//! compatibilità, sola lettura, cloud/OneDrive, ecc.), che non è stato
-//! possibile verificare empiricamente come già fatto per VS Code: cerca
-//! la sotto-stringa fissa " - Excel" e taglia lì, così qualunque testo
-//! aggiuntivo dopo (" (Modalità compatibilità)", " (Sola lettura)", ...)
-//! viene scartato senza bisogno di elencare ogni variante. Se il
-//! pattern non si trova (titolo mai visto), usa comunque il titolo
-//! intero come nome file invece di scartare l'evento — meglio un
-//! raggruppamento imperfetto che perdere dati. Da rivedere con dati
-//! reali alla prima sessione di test dell'utente.
+//! Excel installato — il parsing del titolo cerca la sotto-stringa
+//! fissa " - Excel" e taglia lì, così qualunque testo aggiuntivo dopo
+//! (" (Modalità compatibilità)", " (Sola lettura)", ...) viene scartato
+//! senza bisogno di elencare ogni variante.
+//!
+//! Bug reale segnalato da un utente (issue GitHub #4, con log e
+//! screenshot reali da Excel 2021 MSO/Microsoft 365): un titolo che NON
+//! contiene " - Excel" non è affatto un file mai visto prima — è quasi
+//! sempre una finestra di UTILITÀ di Excel (es. "Find and Replace"/
+//! "Trova e sostituisci", ma lo stesso vale per "Formato celle",
+//! "Imposta pagina", ecc.), un'altra finestra top-level separata dello
+//! stesso processo excel.exe, non un documento. La versione precedente
+//! di questa funzione trattava comunque quel titolo intero come nome
+//! file (v. commento storico rimosso da qui) "per non perdere dati" —
+//! risultato osservato dal vivo: "Find and Replace" compariva come una
+//! RIGA A SÉ nella corsia Excel della Timeline, con una sessione fasulla
+//! di pochi secondi, mentre il file vero continuava ad essere tracciato
+//! correttamente in parallelo sulla sua riga. Con dati reali da 6 file
+//! Excel diversi che rispettano tutti il pattern " - Excel" e UN solo
+//! caso di fallback che si è rivelato essere proprio una finestra di
+//! dialogo, il fallback permissivo fa più danni (falsi file) che
+//! benefici (dati salvati) — ora un titolo che non rispetta il pattern
+//! viene semplicemente ignorato, non trasformato in un file fantasma.
 
 use std::collections::HashMap;
 use std::io::Write;
@@ -59,24 +70,21 @@ fn is_excel_exe(app: &str) -> bool {
 /// - Excel (Sola lettura)", ecc. — taglia alla sotto-stringa fissa
 /// " - Excel" invece di elencare ogni variante di suffisso (vedi
 /// commento in cima al file sul perché). Nessun file aperto (titolo
-/// bare "Excel") o titolo vuoto -> None.
+/// bare "Excel"), titolo vuoto, o titolo che non rispetta il pattern
+/// (quasi sempre una finestra di utilità/dialogo, non un documento —
+/// vedi il commento in cima al file sul bug reale che questo evita)
+/// -> None.
 fn interpreta_titolo(titolo: &str) -> Option<String> {
     let pulito = titolo.trim();
     if pulito.is_empty() || pulito.eq_ignore_ascii_case("excel") {
         return None;
     }
-    match pulito.find(" - Excel") {
-        Some(pos) => {
-            let file = pulito[..pos].trim();
-            if file.is_empty() {
-                None
-            } else {
-                Some(file.to_string())
-            }
-        }
-        // Pattern non riconosciuto: usa comunque il titolo intero
-        // piuttosto che scartare l'evento.
-        None => Some(pulito.to_string()),
+    let pos = pulito.find(" - Excel")?;
+    let file = pulito[..pos].trim();
+    if file.is_empty() {
+        None
+    } else {
+        Some(file.to_string())
     }
 }
 
@@ -326,12 +334,22 @@ mod tests {
     }
 
     #[test]
-    fn interpreta_titolo_pattern_sconosciuto_usa_titolo_intero() {
-        // Formato titolo mai visto (non testabile su questa macchina,
-        // niente Excel installato) — fallback: meglio un raggruppamento
-        // imperfetto che perdere l'evento.
+    fn interpreta_titolo_pattern_sconosciuto_viene_ignorato() {
+        // Bug reale (issue GitHub #4): un titolo senza " - Excel" è
+        // quasi sempre una finestra di utilità/dialogo (es. "Find and
+        // Replace"), non un documento — va ignorato, non trattato come
+        // un file fantasma (vedi il commento in cima al file).
         let r = interpreta_titolo("Qualcosa di inatteso");
-        assert_eq!(r, Some("Qualcosa di inatteso".to_string()));
+        assert_eq!(r, None);
+    }
+
+    #[test]
+    fn interpreta_titolo_finestra_trova_e_sostituisci_viene_ignorata() {
+        // Caso reale osservato dal vivo nell'issue #4: la finestra
+        // "Find and Replace" compariva come una riga fasulla a sé nella
+        // corsia Excel della Timeline.
+        let r = interpreta_titolo("Find and Replace");
+        assert_eq!(r, None);
     }
 
     #[test]

docs/CHANGELOG.md

@@ -5,6 +5,18 @@ quella della lingua attiva, per la versione in esecuzione (vedi
 `src-tauri/src/about.rs`, che scarica questo file da GitHub Pages).
 Non toccare questo formato senza aggiornare anche quel parser.
 
+## 0.1.24
+
+### it
+- Excel: corretto un bug per cui finestre di utilità come "Trova e sostituisci" venivano tracciate come se fossero file veri, comparendo come righe fasulle nella corsia Excel della Timeline.
+- Impostazioni → Agente AI: con il provider "Claude (abbonamento Desktop)", ora viene rilevato se manca ancora un login separato del componente usato da TrackFlow (indipendente da quello dell'app Claude Desktop stessa) — con istruzioni passo-passo e un comando pronto da copiare, invece di scoprirlo solo con un errore poco chiaro nella chat.
+- La chat dell'assistente AI mostra ora un messaggio comprensibile (con collegamento diretto alle impostazioni) quando quel collegamento non è ancora completo, invece del messaggio grezzo del componente sottostante.
+
+### en
+- Excel: fixed a bug where utility windows like "Find and Replace" were tracked as if they were real files, showing up as fake rows in the Excel Timeline lane.
+- Settings → AI Agent: with the "Claude (Desktop subscription)" provider, TrackFlow now detects when the component it uses still needs its own separate sign-in (independent from signing into the Claude Desktop app itself) — with step-by-step instructions and a ready-to-copy command, instead of only surfacing as an unclear error in the chat.
+- The AI assistant chat now shows a clear message (with a direct link to settings) when that sign-in isn't complete yet, instead of the underlying component's raw error text.
+
 ## 0.1.23
 
 ### it

docs/changelog.html

@@ -42,6 +42,14 @@ <h1>Changelog</h1>
     errore mostrato.
   -->
   <div id="changelog-root">
+    <section class="changelog-entry">
+      <h2>0.1.24</h2>
+      <ul>
+        <li>Excel: fixed a bug where utility windows like "Find and Replace" were tracked as if they were real files, showing up as fake rows in the Excel Timeline lane.</li>
+        <li>Settings → AI Agent: with the "Claude (Desktop subscription)" provider, TrackFlow now detects when the component it uses still needs its own separate sign-in (independent from signing into the Claude Desktop app itself) — with step-by-step instructions and a ready-to-copy command, instead of only surfacing as an unclear error in the chat.</li>
+        <li>The AI assistant chat now shows a clear message (with a direct link to settings) when that sign-in isn't complete yet, instead of the underlying component's raw error text.</li>
+      </ul>
+    </section>
     <section class="changelog-entry">
       <h2>0.1.23</h2>
       <ul>

src-tauri/binaries/aw-watcher-excel-x86_64-pc-windows-msvc.exe

@@ -127,6 +127,46 @@ pub fn claude_desktop_disponibile() -> bool {
     trova_claude_exe().is_some()
 }
 
+/// Vero se il CLI Claude Code ha già completato il proprio login — bug
+/// reale segnalato da un utente (portatile di lavoro, issue interna
+/// 2026-09-03): avere Claude Desktop (l'app grafica) collegata al
+/// proprio abbonamento NON significa che il CLI bundlato (un processo
+/// separato, con una propria autenticazione OAuth) l'abbia già fatto —
+/// il CLI richiede un primo avvio interattivo una tantum (scelta tema,
+/// conferma di fiducia della cartella, login) prima di poter essere
+/// usato in modalità automatica (`-p`) come fa questo modulo. Verifica
+/// solo la presenza del file di credenziali che quel primo avvio scrive
+/// — non garantisce che siano ancora valide (potrebbero essere scadute),
+/// ma è lo stesso controllo che l'utente può fare da sé aprendo quella
+/// cartella, e costa zero (nessun processo da avviare).
+fn credenziali_presenti() -> bool {
+    std::env::var_os("USERPROFILE")
+        .map(|h| PathBuf::from(h).join(".claude").join(".credentials.json"))
+        .map(|p| p.is_file())
+        .unwrap_or(false)
+}
+
+/// Stato completo per le Impostazioni — sostituisce `claude_desktop_disponibile`
+/// (lasciata per compatibilità) con le due informazioni che servono per
+/// distinguere "non installata" da "installata ma non ancora autenticata"
+/// e per costruire il comando pronto da copiare (vedi `AiAgentSettings.vue`).
+#[derive(serde::Serialize)]
+pub struct StatoClaudeDesktop {
+    pub trovato: bool,
+    pub autenticato: bool,
+    pub percorso_exe: Option<String>,
+}
+
+#[tauri::command]
+pub fn claude_desktop_stato() -> StatoClaudeDesktop {
+    let percorso = trova_claude_exe();
+    StatoClaudeDesktop {
+        trovato: percorso.is_some(),
+        autenticato: credenziali_presenti(),
+        percorso_exe: percorso.map(|p| p.display().to_string()),
+    }
+}
+
 /// Genera un token casuale per proteggere il server MCP locale (vedi
 /// `avvia`) — nessuna dipendenza nuova solo per questo: ogni
 /// `RandomState` di libreria standard è seminato dal sistema operativo
@@ -817,11 +857,36 @@ async fn manda_turno(sessione: &mut SessioneAttiva, testo: &str) -> Result<Rispo
         "Nessuna risposta da Claude Code — verifica che Claude Desktop sia installata e connessa.".to_string()
     })?;
     if errore_finale {
-        return Err(testo_finale);
+        return Err(traduci_errore_cli(&testo_finale));
     }
     Ok(RispostaAgente { testo: testo_finale, strumenti_usati })
 }
 
+/// Marcatore stabile (non un messaggio in italiano) per il caso "CLI non
+/// autenticato" — bug reale segnalato da un utente (portatile di
+/// lavoro): Claude Desktop era collegata regolarmente, ma il CLI
+/// bundlato non aveva mai completato il proprio primo avvio (vedi
+/// `credenziali_presenti`), e il messaggio grezzo che restituisce ("Not
+/// logged in · Please run /login") non significa nulla per chi non sa
+/// cos'è un CLI o un comando "/login". Un CODICE invece di una frase
+/// già in italiano: il testo vero e proprio (con azione cliccabile verso
+/// Impostazioni) lo mostra il frontend, che sa già gestire lingua IT/EN
+/// — vedi `AiChatWidget.vue`.
+pub const ERRORE_NON_AUTENTICATO: &str = "CLAUDE_DESKTOP_NON_AUTENTICATO";
+
+/// Traduce l'errore grezzo del CLI — solo il pattern "non autenticato"
+/// viene riconosciuto e sostituito col marcatore sopra; qualunque altro
+/// errore del CLI passa invariato (meglio il testo originale, utile per
+/// una segnalazione, che nasconderlo dietro un messaggio generico
+/// sbagliato).
+fn traduci_errore_cli(testo: &str) -> String {
+    if testo.to_lowercase().contains("not logged in") {
+        ERRORE_NON_AUTENTICATO.to_string()
+    } else {
+        testo.to_string()
+    }
+}
+
 /// Punto di ingresso da `agent::ai_agent_send_message` quando il
 /// provider configurato è questo — vedi il commento in cima al file per
 /// il design completo e su `SessioneAttiva` per il perché di un
@@ -1020,6 +1085,31 @@ pub async fn prewarm_se_configurato(app_handle: &AppHandle, app_data_dir: PathBu
     }
 }
 
+#[cfg(test)]
+mod test_stato {
+    //! A differenza di `test_manuale` sotto, questo NON spawna alcun
+    //! processo né consuma l'abbonamento — solo controlli sul filesystem
+    //! locale (percorso claude.exe, file di credenziali). Non `#[ignore]`
+    //! apposta: economico, gira in ogni `cargo test` normale. I valori
+    //! esatti dipendono dalla macchina (qui solo stampati per verifica
+    //! visiva con --nocapture), l'unica cosa asserita è che la funzione
+    //! non va in panico e la forma del risultato è quella attesa.
+    use super::*;
+
+    #[test]
+    fn claude_desktop_stato_non_va_in_panico() {
+        let stato = claude_desktop_stato();
+        println!(
+            "trovato={} autenticato={} percorso_exe={:?}",
+            stato.trovato, stato.autenticato, stato.percorso_exe
+        );
+        // Coerenza minima: se non è stato trovato nessun claude.exe, il
+        // percorso deve essere None (mai un trovato=false con un percorso
+        // valorizzato, o viceversa).
+        assert_eq!(stato.trovato, stato.percorso_exe.is_some());
+    }
+}
+
 #[cfg(test)]
 mod test_manuale {
     //! Test manuale, NON eseguito da un normale `cargo test` (`#[ignore]`)

src-tauri/src/claude_subscription.rs

@@ -1108,6 +1108,7 @@ pub fn run() {
             agent::ai_agent_new_conversation,
             agent::ai_agent_list_models,
             claude_subscription::claude_desktop_disponibile,
+            claude_subscription::claude_desktop_stato,
             categorization::elenca_app_conosciute,
             devtools::apri_devtools,
             diagnostics::log_frontend_diagnostica,

src-tauri/src/lib.rs

@@ -52,7 +52,15 @@ div.ai-chat-widget
       div.ai-chat-bubble.ai-chat-bubble-assistant.ai-chat-bubble-loading(v-if="inviando")
         | {{ $t('aiChat.thinking') }}
 
-    div.ai-chat-alert(v-if="errore") {{ errore }}
+    //- Caso specifico "CLI non autenticato" (vedi ERRORE_NON_AUTENTICATO
+    //- in claude_subscription.rs): stesso trattamento cliccabile della
+    //- chiave API mancante sopra, invece del testo d'errore grezzo —
+    //- l'utente deve poter arrivare in un clic alle istruzioni, non solo
+    //- leggere che qualcosa non va.
+    div.ai-chat-alert(v-if="erroreClaudeNonAutenticato")
+      | {{ $t('aiChat.claudeDesktopNotAuthenticated') }}
+      div.ai-chat-empty-action(@click="apriImpostazioniAgente") {{ $t('aiChat.claudeDesktopNotAuthenticatedAction') }}
+    div.ai-chat-alert(v-else-if="errore") {{ errore }}
 
     //- Wrapper unico attorno a citazione + campo di scrittura — quando la
     //- citazione è presente devono sembrare un solo "cubo" con due zone
@@ -136,6 +144,10 @@ export default {
       bozza: '',
       inviando: false,
       errore: '',
+      // Vedi ERRORE_NON_AUTENTICATO in claude_subscription.rs — quando
+      // l'errore ricevuto è quel marcatore, mostriamo il blocco cliccabile
+      // dedicato sopra invece del testo grezzo in `errore`.
+      erroreClaudeNonAutenticato: false,
       // null finché non è stata ancora controllata (primo apri()) — non
       // false di default, altrimenti lampeggerebbe per un istante l'avviso
       // "chiave mancante" anche quando è configurata, prima che la
@@ -312,6 +324,7 @@ export default {
       const testo = this.bozza.trim();
       if (!testo || this.inviando || this.apiConfigurata === false) return;
       this.errore = '';
+      this.erroreClaudeNonAutenticato = false;
       // La bolla mostrata in chat resta SOLO quello che l'utente ha
       // scritto (più l'etichetta, non i dati veri — vedi contestoLabel
       // sotto) — il testo coi dati veri viene anteposto solo alla
@@ -340,7 +353,12 @@ export default {
         });
         this.riavviaTimerInattivita();
       } catch (e: any) {
-        this.errore = e?.toString?.() ?? String(e);
+        const messaggio = e?.toString?.() ?? String(e);
+        if (messaggio.includes('CLAUDE_DESKTOP_NON_AUTENTICATO')) {
+          this.erroreClaudeNonAutenticato = true;
+        } else {
+          this.errore = messaggio;
+        }
       } finally {
         this.inviando = false;
         this.scrollToBottom();
@@ -349,6 +367,7 @@ export default {
     async nuovaConversazione() {
       this.messaggi = [];
       this.errore = '';
+      this.erroreClaudeNonAutenticato = false;
       try {
         await invoke('ai_agent_new_conversation');
       } catch (e) {

src/components/AiChatWidget.vue

@@ -128,6 +128,17 @@ export default {
         'Uses the Claude Pro/Max subscription already active on Claude Desktop instead of a pay-per-token API key — no extra cost, no key to get. Requires Claude Desktop installed on this PC and already signed in.',
       claudeDesktopNotFound:
         'Claude Desktop does not appear to be installed (or up to date) on this PC — install it and make sure you\'re signed in before using this option.',
+      claudeDesktopNotAuthenticatedTitle: 'One last step (just once)',
+      claudeDesktopNotAuthenticatedBody:
+        'Claude Desktop is installed, but the component TrackFlow uses to talk to it hasn\'t signed in yet — this is separate from signing into the Claude Desktop app itself, and only needs doing once on this PC:',
+      claudeDesktopStep1: 'Open PowerShell (search "PowerShell" in the Start menu).',
+      claudeDesktopStep2: 'Paste the command below and press Enter (use the button to copy it):',
+      claudeDesktopStep3:
+        'Follow the screens that appear: pick a theme, confirm with Enter, then choose "Claude account with subscription" to sign in with the same account as Claude Desktop.',
+      claudeDesktopStep4: 'Done? Close PowerShell and press "Check again" below.',
+      claudeDesktopCopyCommand: 'Copy command',
+      claudeDesktopCopyCommandCopied: 'Copied ✓',
+      claudeDesktopRecheck: 'Check again',
       apiKey: 'API key',
       apiKeyPlaceholder: 'sk-ant-…',
       apiKeyHint: 'Generated from the chosen provider\'s console — saved only on this computer.',
@@ -451,6 +462,8 @@ Clear all rules from the editor, leaving it as \`[]\` (or empty), and save.`,
     emptyHint: 'Ask something about your activity data — project time, categories, VPN sessions, VoiSpeed calls…',
     apiKeyMissing: 'An API key is needed before you can use the assistant.',
     apiKeyMissingAction: 'Go to Settings → AI Agent',
+    claudeDesktopNotAuthenticated: 'Claude Desktop is installed, but one last step (just once) is still needed to connect it.',
+    claudeDesktopNotAuthenticatedAction: 'Go to Settings → AI Agent',
     removeContext: 'Remove',
     thinking: 'Thinking…',
     placeholder: 'Type a message…',