fix: remove a dynamically added tool from an assistant

The tool is getting added to the block's tool list. So, it must also be removed from this list. Signed-off-by: Donnie Adams <[email protected]>
gptscript-ai · Aug 22, 2024 · ef69d55 · ef69d55
1 parent d7b8724
commit ef69d55
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/server/app.mjs b/server/app.mjs
@@ -319,12 +319,12 @@ const mount = async (
       runningScript = null;
     }
 
+    const stateTools = (state.tools || []).filter((t) => t !== tool);
     // find the root tool and then remove the tool
     for (let block of script) {
       if (block.type === 'tool') {
         if (!block.tools) break;
-        const stateTools = (state.tools || []).filter((t) => t !== tool);
-        block.tools = [...new Set(block.tools, ...stateTools)];
+        block.tools = [...new Set(block.tools.filter((t) => t !== tool))];
         break;
       }
     }
@@ -338,7 +338,7 @@ const mount = async (
 
     opts.chatState = JSON.stringify(currentState);
     state.chatState = JSON.stringify(currentState);
-    state.tools = state.tools.filter((t) => t !== tool);
+    state.tools = stateTools;
 
     if (threadID) {
       fs.writeFile(statePath, JSON.stringify(state), (err) => {