From e6fd0ed05aacaae823cd5bf17dd2a6327d589658 Mon Sep 17 00:00:00 2001 From: hanishkvc Date: Mon, 3 Nov 2025 13:02:56 +0530 Subject: [PATCH] SimpleChatTC: ToolCalling enabled, Sliding window adjust Chances are for ai models which dont support tool calling, things will be such that the tool calls meta data shared will be silently ignored without much issue. So enabling tool calling feature by default, so that in case one is using a ai model with tool calling the feature is readily available for use. Revert SlidingWindow ChatHistory in Context from last 10 to last 5 (2 more then origianl, given more context support in todays models) by default, given that now tool handshakes go through the tools related side channel in the http handshake and arent morphed into normal user-assistant channel of the handshake. --- tools/server/public_simplechat/readme.md | 10 ++++++++++ tools/server/public_simplechat/simplechat.js | 4 ++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/tools/server/public_simplechat/readme.md b/tools/server/public_simplechat/readme.md index e3a835df0d..6a237918ee 100644 --- a/tools/server/public_simplechat/readme.md +++ b/tools/server/public_simplechat/readme.md @@ -589,6 +589,16 @@ SimpleProxy the whitelist inturn picked from config json * Helpers to fetch file from local file system or the web, transparently +Chances are for ai models which dont support tool calling, things will be such that the tool calls +meta data shared will be silently ignored without much issue. So enabling tool calling feature by default, +so that in case one is using a ai model with tool calling the feature is readily available for use. + +Revert SlidingWindow ChatHistory in Context from last 10 to last 5 (rather 2 more then origianl, +given more context support in todays models) by default, given that now tool handshakes go through +the tools related side channel in the http handshake and arent morphed into normal user-assistant +channel of the handshake. + + #### ToDo Is the tool call promise land trap deep enough, need to think through and explore around this once later. diff --git a/tools/server/public_simplechat/simplechat.js b/tools/server/public_simplechat/simplechat.js index 297f76366b..b145938af6 100644 --- a/tools/server/public_simplechat/simplechat.js +++ b/tools/server/public_simplechat/simplechat.js @@ -1304,7 +1304,7 @@ class Me { this.defaultChatIds = [ "Default", "Other" ]; this.multiChat = new MultiChatUI(); this.tools = { - enabled: false, + enabled: true, proxyUrl: "http://127.0.0.1:3128", proxyAuthInsecure: "NeverSecure", searchUrl: SearchURLS.duckduckgo, @@ -1328,7 +1328,7 @@ class Me { this.chatProps = { apiEP: ApiEP.Type.Chat, stream: true, - iRecentUserMsgCnt: 10, + iRecentUserMsgCnt: 5, bCompletionFreshChatAlways: true, bCompletionInsertStandardRolePrefix: false, bTrimGarbage: true,