From c68316bf07e04723d2645fc5866e7e7901dab819 Mon Sep 17 00:00:00 2001
From: hanishkvc <hanishkvc@gmail.com>
Date: Fri, 28 Nov 2025 15:05:49 +0530
Subject: [PATCH] SimpleChatTCRV:CMTextFormat: PopOver & Message UI adjust

Add format selection box to the popover.

Update show_message logic to allow refreshing a existing message
ui element, rather than creating a new one.

Trigger refresh of the message ui element, when format selection
changes.
---
 tools/server/public_simplechat/index.html    |  4 ++
 tools/server/public_simplechat/simplechat.js | 43 ++++++++++++++++++--
 2 files changed, 43 insertions(+), 4 deletions(-)

diff --git a/tools/server/public_simplechat/index.html b/tools/server/public_simplechat/index.html
index 217b025039..d99489a24a 100644
--- a/tools/server/public_simplechat/index.html
+++ b/tools/server/public_simplechat/index.html
@@ -47,6 +47,10 @@
             </div>
 
             <div id="popover-chatmsg" popover="auto">
+                <select id="popover-chatmsg-format">
+                    <option>plain</option>
+                    <option>markdown</option>
+                </select>
                 <button id="popover-chatmsg-del" title="delete message"> &#x274C; </button>
                 <button id="popover-chatmsg-copy" title="copy message text"> &#x1F4CB; </button>
             </div>
diff --git a/tools/server/public_simplechat/simplechat.js b/tools/server/public_simplechat/simplechat.js
index 6511bd71f6..f61fa5780a 100644
--- a/tools/server/public_simplechat/simplechat.js
+++ b/tools/server/public_simplechat/simplechat.js
@@ -1232,6 +1232,7 @@ class MultiChatUI {
         this.elPopoverChatMsg = /** @type{HTMLElement} */(document.getElementById("popover-chatmsg"));
         this.elPopoverChatMsgCopyBtn = /** @type{HTMLElement} */(document.getElementById("popover-chatmsg-copy"));
         this.elPopoverChatMsgDelBtn = /** @type{HTMLElement} */(document.getElementById("popover-chatmsg-del"));
+        this.elPopoverChatMsgFormatSelect = /** @type{HTMLSelectElement} */(document.getElementById("popover-chatmsg-format"));
         this.uniqIdChatMsgPO = -1;
 
         // image popover menu
@@ -1439,13 +1440,24 @@ class MultiChatUI {
      * * Assistant which contains a tool req, shows tool call ui if needed. ie
      *   * if it is the last message OR
      *   * if it is the last but one message and there is a ToolTemp message next
+     *
+     * If secMainRefresh is
+     * * undefined : create and append a new message related ui element
+     * * provided : refresh its contents as needed
+     *
+     * To refresh a existing message related ui element without bothering about
+     * or messing with tool call edit/trigger or response related handling,
+     * remember to pass a large number wrt iFromLast and undefined wrt nextMsg
+     * * rather wrt current logic any number greater than 1 will do.
+     *
      * @param {HTMLElement | undefined} elParent
      * @param {string} chatId
      * @param {ChatMessageEx} msg
      * @param {number} iFromLast
      * @param {ChatMessageEx | undefined} nextMsg
+     * @param {HTMLElement | undefined} secMainRefresh - allow one to update a existing instance of the message ui element.
      */
-    show_message(elParent, chatId, msg, iFromLast, nextMsg) {
+    show_message(elParent, chatId, msg, iFromLast, nextMsg, secMainRefresh=undefined) {
         // Handle ToolTemp
         if (iFromLast == 0) {
             if (msg.ns.role === Roles.ToolTemp) {
@@ -1456,7 +1468,12 @@ class MultiChatUI {
             this.elInUser.dataset.role = (msg.ns.role == Roles.ToolTemp) ? Roles.ToolTemp : Roles.User
         }
         // Create main section
-        let secMain = document.createElement('section')
+        let secMain = secMainRefresh
+        if (!secMain) {
+            secMain = document.createElement('section')
+        } else {
+            secMain.replaceChildren()
+        }
         secMain.id = `cmuid${msg.uniqId}`
         secMain.dataset.cmuid = String(msg.uniqId)
         secMain.classList.add(`role-${msg.ns.role}`)
@@ -1482,7 +1499,9 @@ class MultiChatUI {
         secMain.addEventListener('mouseleave', (ev)=>{
             console.debug(`DBUG:MCUI:ChatMessageMLeave:${msg.uniqId}`)
         })
-        elParent?.append(secMain)
+        if (!secMainRefresh) {
+            elParent?.append(secMain)
+        }
         secMain.setAttribute("CMUniqId", String(msg.uniqId))
         this.elLastChatMessage = secMain;
         // Create role para
@@ -1558,7 +1577,7 @@ class MultiChatUI {
                 this.ui_toolcallvalidated_as_needed(msg, chatId, bAuto);
             }
         }
-        // Handle tool call message show
+        // Handle showing of tool calls in the message
         if (msg.ns.tool_calls) {
             for (const tc of msg.ns.tool_calls) {
                 this.show_message_toolcall(secContents, tc)
@@ -1846,6 +1865,22 @@ class MultiChatUI {
             navigator.clipboard.write([item])
         })
 
+        this.elPopoverChatMsgFormatSelect.addEventListener('change', (ev) =>{
+            console.log(`DBUG:SimpleChat:MCUI:ChatMsgPO:Format:${this.curChatId}:${this.uniqIdChatMsgPO}:${this.elPopoverChatMsgFormatSelect.value}`)
+            let chatSession = this.simpleChats[this.curChatId]
+            let index = chatSession.get_chatmessage_index(this.uniqIdChatMsgPO)
+            let chat = chatSession.xchat[index]
+            if (!chat.ns.has_content()) {
+                return
+            }
+            chat.textFormat = this.elPopoverChatMsgFormatSelect.value
+            /** @type {HTMLElement | null} */
+            let elMsg = this.elDivChat.querySelector(`[id="cmuid${this.uniqIdChatMsgPO}"]`)
+            if (elMsg) {
+                this.show_message(this.elDivChat, chatSession.chatId, chat, 123, undefined, elMsg)
+            }
+        })
+
     }
 
     /**