fix(bindings): remove early return in isImplementedBy loop (#2085)

The isImplementedBy function was returning true after finding the first
matching tool, instead of checking all required tools in the binding.
This caused false positives when only some required tools were present.

Author: viktormarinho

.cursor/worktrees.json

@@ -1,7 +1,7 @@
 {
-    "setup-worktree": [
-        "bun install",
-        "cp $ROOT_WORKTREE_PATH/apps/mesh/auth-config.json apps/mesh/auth-config.json",
-        "cp $ROOT_WORKTREE_PATH/apps/mesh/config.json apps/mesh/config.json"
-    ]
-}
+  "setup-worktree": [
+    "bun install",
+    "cp $ROOT_WORKTREE_PATH/apps/mesh/auth-config.json apps/mesh/auth-config.json",
+    "cp $ROOT_WORKTREE_PATH/apps/mesh/config.json apps/mesh/config.json"
+  ]
+}

apps/docs/server/main.ts

@@ -20,7 +20,9 @@ const runtime = withRuntime<Env>({
         env: "development",
       });
 
-    return await assetsHandler(req) ?? new Response("Not found", { status: 404 });
+    return (
+      (await assetsHandler(req)) ?? new Response("Not found", { status: 404 })
+    );
   },
 });
 

packages/bindings/src/core/binder.ts

@@ -171,8 +171,6 @@ export function createBindingChecker<TDefinition extends readonly ToolBinder[]>(
         if (!matchedTool) {
           return false;
         }
-        return true;
-
         // FIXME @mcandeia Zod to JSONSchema converstion is creating inconsistent schemas
       }
       return true;