From 13d312fe0d2cd5092917dca458d8fb7b7b7f61b1 Mon Sep 17 00:00:00 2001
From: hanishkvc <hanishkvc@gmail.com>
Date: Tue, 28 Oct 2025 02:17:15 +0530
Subject: [PATCH] SimpleChatTC:ToolTemp: Ensure add removes non promoted
 ToolTemp

---
 tools/server/public_simplechat/simplechat.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/tools/server/public_simplechat/simplechat.js b/tools/server/public_simplechat/simplechat.js
index ccab26c393..b7e8af1603 100644
--- a/tools/server/public_simplechat/simplechat.js
+++ b/tools/server/public_simplechat/simplechat.js
@@ -407,11 +407,21 @@ class SimpleChat {
 
     /**
      * Add an entry into xchat.
+     * If the last message in chat history is a ToolTemp message, discard it
+     * as the runtime logic is asking for adding new message instead of promoting the tooltemp message.
+     *
      * NOTE: A new copy is created and added into xchat.
      * Also update iLastSys system prompt index tracker
      * @param {ChatMessageEx} chatMsg
      */
     add(chatMsg) {
+        if (this.xchat.length > 0) {
+            let lastIndex = this.xchat.length - 1;
+            if (this.xchat[lastIndex].ns.role == Roles.ToolTemp) {
+                console.debug("DBUG:SimpleChat:Add:Discarding prev ToolTemp message...")
+                this.xchat.pop()
+            }
+        }
         this.xchat.push(ChatMessageEx.newFrom(chatMsg));
         if (chatMsg.ns.role == Roles.System) {
             this.iLastSys = this.xchat.length - 1;