Fix tool wrapper in skills load

Refined skill tools loading to ensure each tool has a proper type wrapper ('function'), preventing AI tool-definition errors during reasoning. Updated mapping to auto-add type if missing.

X-Lovable-Edit-ID: edt-c56420cd-5739-4a7e-918a-0a9b61d1a695
Co-authored-by: magnusfroste <38864257+magnusfroste@users.noreply.github.com>

Author: lovable-dev[bot]

supabase/functions/_shared/agent-reason.ts

@@ -1780,7 +1780,12 @@ export async function loadSkillTools(supabase: any, scope: 'internal' | 'externa
 
   return (skills || [])
     .filter((s: any) => s.tool_definition?.function)
-    .map((s: any) => s.tool_definition);
+    .map((s: any) => {
+      const td = s.tool_definition;
+      // Ensure every tool has the required 'type' field
+      if (!td.type) td.type = 'function';
+      return td;
+    });
 }
 
 // ─── Non-Streaming Reason Loop ────────────────────────────────────────────────