Refresh active tools on every iteration, since it can change

breadboard-ai · Jan 15, 2025 · 892eaf4 · 892eaf4
1 parent 87a05ff
commit 892eaf4
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/packages/bbrt/src/llm/conversation.ts b/packages/bbrt/src/llm/conversation.ts
@@ -100,10 +100,12 @@ export class Conversation {
     );
 
     const done = (async (): Promise<void> => {
-      const activeTools = await this.#getActiveTools();
       let remainingModelCalls = 1 + Math.max(0, MAX_TOOL_ITERATIONS);
       let functionCalls: ReactiveFunctionCallState[];
       do {
+        // Note that tools can change between iterations, since "activate_tool"
+        // could be called.
+        const activeTools = await this.#getActiveTools();
         const allowFunctionCalls = remainingModelCalls > 1;
         functionCalls = (
           await this.#callModel(