From 7820a9a48e22de996e6b335b643b30613cdffa34 Mon Sep 17 00:00:00 2001
From: DevMando <codefilez@gmail.com>
Date: Mon, 7 Sep 2026 18:45:11 -0700
Subject: [PATCH] Fix: Stop repeated context-window notices when restoring
 sessions

---
 .../MandoCode.Desktop.Tests.csproj            |  1 +
 .../ModelNoticeReplayTests.cs                 | 32 +++++++++++++++++++
 .../Services/ModelNoticeReplay.cs             | 19 +++++++++++
 .../Services/TranscriptHtmlBuilder.cs         |  3 +-
 .../ViewModels/ChatController.cs              |  6 ++--
 5 files changed, 58 insertions(+), 3 deletions(-)
 create mode 100644 src/MandoCode.Desktop.Tests/ModelNoticeReplayTests.cs
 create mode 100644 src/MandoCode.Desktop/Services/ModelNoticeReplay.cs

diff --git a/src/MandoCode.Desktop.Tests/MandoCode.Desktop.Tests.csproj b/src/MandoCode.Desktop.Tests/MandoCode.Desktop.Tests.csproj
index 19df441..54141f2 100644
--- a/src/MandoCode.Desktop.Tests/MandoCode.Desktop.Tests.csproj
+++ b/src/MandoCode.Desktop.Tests/MandoCode.Desktop.Tests.csproj
@@ -56,6 +56,7 @@
          the file-naming convention is a real contract worth pinning; pure System.IO, no WinUI. -->
     <Compile Include="..\MandoCode.Desktop\Services\BuiltInBackgrounds.cs" Link="src\BuiltInBackgrounds.cs" />
     <Compile Include="..\MandoCode.Desktop\Services\TranscriptJournal.cs" Link="src\TranscriptJournal.cs" />
+    <Compile Include="..\MandoCode.Desktop\Services\ModelNoticeReplay.cs" Link="src\ModelNoticeReplay.cs" />
     <Compile Include="..\MandoCode.Desktop\Services\ConversationLog.cs" Link="src\ConversationLog.cs" />
     <!-- History's full-text matching + snippet extraction. Pure (text in, match out); the file reads
          and caching live in ConversationTextCache, which is NOT compiled here because it touches
diff --git a/src/MandoCode.Desktop.Tests/ModelNoticeReplayTests.cs b/src/MandoCode.Desktop.Tests/ModelNoticeReplayTests.cs
new file mode 100644
index 0000000..c05e92e
--- /dev/null
+++ b/src/MandoCode.Desktop.Tests/ModelNoticeReplayTests.cs
@@ -0,0 +1,32 @@
+using MandoCode.Desktop.Services;
+using Xunit;
+
+namespace MandoCode.Desktop.Tests;
+
+public class ModelNoticeReplayTests
+{
+    private const string Sizing = "Context window sized to 16k tokens for this model tier (applies from your next message).";
+    private static string Notice(string text, string kind = "dim") =>
+        $"<div class=\"notice-card {kind}\"><span class=\"notice-emoji\">ℹ️</span><span class=\"notice-text\">{text}</span></div>";
+
+    [Fact]
+    public void RepeatedRestoresDropAllHistoricalSetupNoticesButKeepConversation()
+    {
+        var answer = "<div class=\"assistant\">How can I help?</div>";
+        var history = new[] { answer, Notice(Sizing), Notice(Sizing), Notice(Sizing),
+            Notice("Cloud models run on ollama.com and need an active cloud subscription.") };
+        var replay = history.Where(h => !ModelNoticeReplay.IsTransient(h)).ToArray();
+        Assert.Equal(new[] { answer }, replay);
+        Assert.Equal(replay, replay.Where(h => !ModelNoticeReplay.IsTransient(h)));
+    }
+
+    [Fact]
+    public void QuotedWordingAndActualWarningsArePreserved()
+    {
+        Assert.False(ModelNoticeReplay.IsTransient($"<div class=\"assistant\">{Notice(Sizing)}</div>"));
+        Assert.False(ModelNoticeReplay.IsTransient($"<div class=\"user-echo\">{Sizing}</div>"));
+        Assert.False(ModelNoticeReplay.IsTransient(Notice(Sizing, "warn")));
+        Assert.False(ModelNoticeReplay.IsTransient(Notice("The context window is too small for this request.")));
+        Assert.False(ModelNoticeReplay.IsTransient(Notice("Project root changed to: C:\\project")));
+    }
+}
diff --git a/src/MandoCode.Desktop/Services/ModelNoticeReplay.cs b/src/MandoCode.Desktop/Services/ModelNoticeReplay.cs
new file mode 100644
index 0000000..c55b7eb
--- /dev/null
+++ b/src/MandoCode.Desktop/Services/ModelNoticeReplay.cs
@@ -0,0 +1,19 @@
+using System.Text.RegularExpressions;
+
+namespace MandoCode.Desktop.Services;
+
+/// <summary>Old model setup notices describe a prior runtime, not the restored conversation.</summary>
+public static class ModelNoticeReplay
+{
+    public static bool IsTransient(string html)
+    {
+        // Match only standalone system notices. Keep user messages, assistant explanations,
+        // and errors even when they quote the same wording.
+        var match = Regex.Match(html,
+            "\\A<div class=\"notice-card dim\"><span class=\"notice-emoji\">[^<]*</span><span class=\"notice-text\">([^<]*)</span></div>\\z");
+        if (!match.Success) return false;
+        var text = match.Groups[1].Value;
+        return text == "Cloud models run on ollama.com and need an active cloud subscription."
+            || Regex.IsMatch(text, @"\AContext window sized to \d+k tokens for this model tier \(applies from your next message\)\.\z");
+    }
+}
diff --git a/src/MandoCode.Desktop/Services/TranscriptHtmlBuilder.cs b/src/MandoCode.Desktop/Services/TranscriptHtmlBuilder.cs
index 5e84ea9..4d82e8f 100644
--- a/src/MandoCode.Desktop/Services/TranscriptHtmlBuilder.cs
+++ b/src/MandoCode.Desktop/Services/TranscriptHtmlBuilder.cs
@@ -103,7 +103,8 @@ public static bool IsEphemeralStatus(string blockHtml) =>
         // Boot/progress narration — true only while it was happening. ("Project root
         // changed to: X" is deliberately NOT here: that's a real event, kept as history.)
         || blockHtml.Contains(">Rebuilding the AI session for the new project…<", StringComparison.Ordinal)
-        || blockHtml.Contains(">✓ Ready.<", StringComparison.Ordinal);
+        || blockHtml.Contains(">✓ Ready.<", StringComparison.Ordinal)
+        || ModelNoticeReplay.IsTransient(blockHtml);
 
     /// <summary>A compact status pill — a colored state dot, a bold primary value, and an
     /// optional dim qualifier. The dot replaces status emoji: crisp and theme-aware.
diff --git a/src/MandoCode.Desktop/ViewModels/ChatController.cs b/src/MandoCode.Desktop/ViewModels/ChatController.cs
index f7281cd..7e30364 100644
--- a/src/MandoCode.Desktop/ViewModels/ChatController.cs
+++ b/src/MandoCode.Desktop/ViewModels/ChatController.cs
@@ -1583,7 +1583,8 @@ private async Task ApplyModelSwitchAsync(string modelTag)
         if (recommendedCtx > 0 && recommendedCtx != _config.ContextLength)
         {
             _config.ContextLength = recommendedCtx;
-            _transcript.Append(_html.Dim($"Context window sized to {recommendedCtx / 1024}k tokens for this model tier (applies from your next message)."));
+            if (!DeferModelAnnouncement)
+                _transcript.Append(_html.Dim($"Context window sized to {recommendedCtx / 1024}k tokens for this model tier (applies from your next message)."));
         }
 
         _busy.Start("Switching model...");
@@ -1609,7 +1610,8 @@ private async Task ApplyModelSwitchAsync(string modelTag)
         if (MandoCodeConfig.IsCloudModel(modelTag) && !_cloudNoticeShown)
         {
             _cloudNoticeShown = true;
-            _transcript.Append(_html.Dim("Cloud models run on ollama.com and need an active cloud subscription."));
+            if (!DeferModelAnnouncement)
+                _transcript.Append(_html.Dim("Cloud models run on ollama.com and need an active cloud subscription."));
         }
 
         // Only mention the cleared context — and offer a snapshot — when there was actually a