From a0f6762fda379be100c029d6e3051a69f52585e2 Mon Sep 17 00:00:00 2001
From: hanishkvc <hanishkvc@gmail.com>
Date: Tue, 14 Oct 2025 01:34:22 +0530
Subject: [PATCH] SimpleChatTC: Web worker flow initial go cleanup

Had forgotten to specify type as module wrt web worker, in order
to allow it to import the toolsconsole module.

Had forgotten to maintain the id of the timeout handler, which is
needed to clear/stop the timeout handler from triggering, if tool
call response is got well in time.

As I am currently reverting the console redirection at end of
handling a tool call code in the web worker message handler, I
need to setup the redirection each time. Also I had forgotten
to clear the console.log capture data space, before a new tool
call code is executed, this is also fixed by this change.

TODO: Need to abort the tool call code execution in the web worker
if possible in future, if the client / browser side times out
waiting for tool call response, ie if the tool call code is taking
up too much time.
---
 tools/server/public_simplechat/simplechat.js   | 3 ++-
 tools/server/public_simplechat/tools.mjs       | 2 +-
 tools/server/public_simplechat/toolsworker.mjs | 3 +--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/tools/server/public_simplechat/simplechat.js b/tools/server/public_simplechat/simplechat.js
index 925a181dbf..6b897448c2 100644
--- a/tools/server/public_simplechat/simplechat.js
+++ b/tools/server/public_simplechat/simplechat.js
@@ -678,6 +678,7 @@ class MultiChatUI {
         })
 
         tools.setup((name, data)=>{
+            clearTimeout(this.idTimeOut)
             this.elInUser.value = `<tool_response>${data}</tool_response>`
             this.ui_reset_userinput(false)
         })
@@ -791,7 +792,7 @@ class MultiChatUI {
             this.elInUser.value = `<tool_response>${toolResult}</tool_response>`
             this.ui_reset_userinput(false)
         } else {
-            setTimeout(() => {
+            this.idTimeOut = setTimeout(() => {
                 this.elInUser.value = `<tool_response>Tool/Function call ${toolname} taking too much time, aborting...</tool_response>`
                 this.ui_reset_userinput(false)
             }, 10000)
diff --git a/tools/server/public_simplechat/tools.mjs b/tools/server/public_simplechat/tools.mjs
index 4ece70ae56..75fe56e4f4 100644
--- a/tools/server/public_simplechat/tools.mjs
+++ b/tools/server/public_simplechat/tools.mjs
@@ -8,7 +8,7 @@
 import * as tjs from './tooljs.mjs'
 
 
-let gToolsWorker = new Worker('./toolsworker.mjs');
+let gToolsWorker = new Worker('./toolsworker.mjs', { type: 'module' });
 /**
  * @type {Object<string,Object<string,any>>}
  */
diff --git a/tools/server/public_simplechat/toolsworker.mjs b/tools/server/public_simplechat/toolsworker.mjs
index d17b772b4f..e370fd0a9d 100644
--- a/tools/server/public_simplechat/toolsworker.mjs
+++ b/tools/server/public_simplechat/toolsworker.mjs
@@ -13,9 +13,8 @@
 import * as tconsole from "./toolsconsole.mjs"
 
 
-tconsole.console_redir()
-
 self.onmessage = function (ev) {
+    tconsole.console_redir()
     try {
         eval(ev.data.code)
     } catch (/** @type {any} */error) {