From 2a8bd1c9e7e9303c75b65fdbea11955e7f72dd19 Mon Sep 17 00:00:00 2001
From: hanishkvc <hanishkvc@gmail.com>
Date: Tue, 14 Oct 2025 00:58:23 +0530
Subject: [PATCH] SimpleChatTC: Actual tool call implementations simplified

These no longer need to worry about

* setting up the console.log related redirection to capture
  the generated outputs, nor about
* setting up a dynamic function for executing the needed
  tool call related code

The web worker setup to help run tool calls in a relatively
isolated environment independent of the main browser env,
takes care of these.

One needs to only worry about getting the handle to the
web worker to use and inturn pass the need code wrt the
tool call to it.
---
 tools/server/public_simplechat/tooljs.mjs | 26 +++++++++++------------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/tools/server/public_simplechat/tooljs.mjs b/tools/server/public_simplechat/tooljs.mjs
index 23b340e514..5ee8e83004 100644
--- a/tools/server/public_simplechat/tooljs.mjs
+++ b/tools/server/public_simplechat/tooljs.mjs
@@ -7,7 +7,7 @@
 //
 
 
-import * as tconsole from "./toolsconsole.mjs"
+let gToolsWorker = /** @type{Worker} */(/** @type {unknown} */(null));
 
 
 let js_meta = {
@@ -31,16 +31,12 @@ let js_meta = {
 
 /**
  * Implementation of the javascript interpretor logic. Minimal skeleton for now.
- * ALERT: Has access to the javascript environment and can mess with it and beyond
+ * ALERT: Has access to the javascript web worker environment and can mess with it and beyond
  * @param {string} toolname
  * @param {any} obj
  */
 function js_run(toolname, obj) {
-    tconsole.console_redir()
-    let func = new Function(obj["code"])
-    func()
-    tconsole.console_revert()
-    tc_switch[toolname]["result"] = tconsole.gConsoleStr
+    gToolsWorker.postMessage({ name: toolname, code: obj["code"]})
 }
 
 
@@ -65,16 +61,12 @@ let calc_meta = {
 
 /**
  * Implementation of the simple calculator logic. Minimal skeleton for now.
- * ALERT: Has access to the javascript environment and can mess with it and beyond
+ * ALERT: Has access to the javascript web worker environment and can mess with it and beyond
  * @param {string} toolname
  * @param {any} obj
  */
 function calc_run(toolname, obj) {
-    tconsole.console_redir()
-    let func = new Function(`console.log(${obj["arithexpr"]})`)
-    func()
-    tconsole.console_revert()
-    tc_switch[toolname]["result"] = tconsole.gConsoleStr
+    gToolsWorker.postMessage({ name: toolname, code: `console.log(${obj["arithexpr"]})`})
 }
 
 
@@ -94,3 +86,11 @@ export let tc_switch = {
     }
 }
 
+
+/**
+ * Used to get hold of the web worker to use for running tool/function call related code
+ * @param {Worker} toolsWorker
+ */
+export function init(toolsWorker) {
+    gToolsWorker = toolsWorker
+}