ManagerService.Tests/Controllers/AiControllerTests.cs

@@ -105,92 +105,6 @@ namespace ManagerService.Tests.Controllers
             Assert.IsType<ForbidResult>(result);
         }
 
-        // ── JOURNALISATION DES QUESTIONS ─────────────────────────────────────
-
-        /// <summary>
-        /// Un tour déclenché par le mode proactif ne doit pas apparaître dans
-        /// « Ce que demandent vos visiteurs » : sa « question » est une consigne que le
-        /// système s'est écrite à lui-même. Mais ses jetons sont bien consommés, donc
-        /// bien comptés — c'est la paire qui compte, pas chaque moitié isolément.
-        /// </summary>
-        [Fact]
-        public async Task Chat_AutoTriggered_CountsTokensButLogsNoVisitorQuestion()
-        {
-            using var db = DbContextFactory.Create();
-            SeedAssistantInstance(db);
-
-            var request = MakeRequest("i1");
-            request.IsAutoTriggered = true;
-            await BuildController(db).Chat(request);
-
-            Assert.Empty(db.VisitorQuestions);
-            Assert.Equal(42, db.Instances.First().AiTokensThisMonth);
-        }
-
-        [Fact]
-        public async Task Chat_AskedByVisitor_LogsVisitorQuestion()
-        {
-            using var db = DbContextFactory.Create();
-            SeedAssistantInstance(db);
-
-            await BuildController(db).Chat(MakeRequest("i1"));
-
-            var logged = Assert.Single(db.VisitorQuestions);
-            Assert.Equal("Bonjour", logged.Question);
-            Assert.Equal(42, db.Instances.First().AiTokensThisMonth);
-        }
-
-        /// <summary>
-        /// Deux tours d'une même conversation partagent leur ConversationId — c'est le seul
-        /// lien entre eux côté serveur, l'historique étant reconstruit par le client.
-        /// ⚠️ Aucun client ne l'envoyait jusqu'au 2026-08-13 : le repli `Guid.NewGuid()`
-        /// s'appliquait à chaque appel, donc chaque question formait sa propre conversation.
-        /// </summary>
-        [Fact]
-        public async Task Chat_SameConversationId_LinksBothTurns()
-        {
-            using var db = DbContextFactory.Create();
-            SeedAssistantInstance(db);
-
-            var controller = BuildController(db);
-            var first = MakeRequest("i1");
-            first.ConversationId = "conv-1";
-            var second = MakeRequest("i1");
-            second.ConversationId = "conv-1";
-
-            await controller.Chat(first);
-            await controller.Chat(second);
-
-            Assert.Equal(2, db.VisitorQuestions.Count());
-            Assert.All(db.VisitorQuestions, q => Assert.Equal("conv-1", q.ConversationId));
-        }
-
-        [Fact]
-        public async Task Chat_NoConversationId_FallsBackToAGeneratedOne()
-        {
-            using var db = DbContextFactory.Create();
-            SeedAssistantInstance(db);
-
-            await BuildController(db).Chat(MakeRequest("i1"));
-
-            Assert.False(string.IsNullOrWhiteSpace(db.VisitorQuestions.First().ConversationId));
-        }
-
-        private static void SeedAssistantInstance(MyInfoMateDbContext db)
-        {
-            db.Instances.Add(new Instance
-            {
-                Id = "i1", Name = "Musée", IsAssistant = true, DateCreation = DateTime.UtcNow,
-                AiTokensPerMonth = 1_000, AiUsageMonthKey = DateTime.UtcNow.ToString("yyyy-MM")
-            });
-            db.ApplicationInstances.Add(new ApplicationInstance
-            {
-                Id = "ai1", InstanceId = "i1", AppType = AppType.Tablet, IsAssistant = true,
-                Languages = new List<string>()
-            });
-            db.SaveChanges();
-        }
-
         // ── QUOTA COUNTER ────────────────────────────────────────────────────
 
         [Fact]

ManagerService/Controllers/AiController.cs

@@ -378,16 +378,8 @@ namespace ManagerService.Controllers
 
                 var result = await _assistantService.ChatAsync(request);
 
-                // Les jetons sont comptés dans tous les cas — un tour proactif coûte de
-                // l'argent réel au quota du client, l'exclure du compteur serait pire que
-                // le bruit qu'on retire juste en dessous.
                 RecordUsage(instance, result.TokensUsed);
-
-                // Mais un tour que le système s'est déclenché à lui-même n'est pas une
-                // question de visiteur : il ne va ni dans « Ce que demandent vos visiteurs »,
-                // ni dans les trous de contenu, ni dans les thèmes du lot J.
-                if (!request.IsAutoTriggered)
-                    RecordVisitorQuestion(request, result);
+                RecordVisitorQuestion(request, result);
 
                 return Ok(result);
             }

ManagerService/DTOs/AiChatDTO.cs

@@ -24,18 +24,6 @@ namespace ManagerService.DTOs
         /// dates en toutes lettres, réponses courtes audio-friendly.
         /// </summary>
         public bool IsVoice { get; set; } = false;
-        /// <summary>
-        /// true = le tour a été déclenché par le système, pas demandé par le visiteur —
-        /// c'est le cas du mode proactif, où l'approche d'un point d'intérêt fait parler
-        /// le guide tout seul.
-        ///
-        /// ⚠️ Ces tours ne sont **pas journalisés** dans <c>VisitorQuestion</c> : leur
-        /// « question » est une consigne que le système s'est écrite à lui-même
-        /// (« Tu es un guide audio de musée. Le visiteur vient d'entrer dans la zone X… »),
-        /// et l'onglet Guide IA existe pour montrer ce que les **humains** demandent.
-        /// Leurs jetons, eux, restent comptés : ils sont réellement consommés.
-        /// </summary>
-        public bool IsAutoTriggered { get; set; } = false;
     }
 
     public class AiChatMessage