fix: tools were being stripped from API requests

- resolveAsyncFunctions was skipping 'tools' key, removing API-formatted tools
- ModelResult was also stripping 'tools' when building baseRequest
- Tools are now preserved through the async resolution pipeline
- Both tests now pass: tools are sent to API and model calls them correctly

Author: mattapperson

src/lib/async-params.ts

@@ -79,8 +79,9 @@ export async function resolveAsyncFunctions(
 
   // Iterate over all keys in the input
   for (const [key, value] of Object.entries(input)) {
-    // Skip stopWhen and tools - they're handled separately
-    if (key === 'stopWhen' || key === 'tools') {
+    // Skip stopWhen - it's handled separately in ModelResult
+    // Note: tools are already in API format at this point (converted in callModel()), so we include them
+    if (key === 'stopWhen') {
       continue;
     }
 

src/lib/model-result.ts

@@ -175,9 +175,10 @@ export class ModelResult {
         );
       } else {
         // Already resolved, extract non-function fields
-        // Since request is CallModelInput, we need to filter out tools/stopWhen
+        // Since request is CallModelInput, we need to filter out stopWhen
+        // Note: tools are already in API format at this point (converted in callModel())
         // eslint-disable-next-line @typescript-eslint/no-unused-vars
-        const { tools, stopWhen, ...rest } = this.options.request;
+        const { stopWhen, ...rest } = this.options.request;
         // Cast to ResolvedCallModelInput - we know it's resolved if hasAsyncFunctions returned false
         baseRequest = rest as ResolvedCallModelInput;
       }