Fix/relatd mcp performance (#648)

* fix: related fields laoded from index

* Fix styling

* fix: no index in repository

* Fix styling

* fix: wip

* Fix styling

* Fix styling

* fix: wip

---------

Co-authored-by: binaryk <binaryk@users.noreply.github.com>

Author: binaryk

src/Eager/Related.php

@@ -64,6 +64,7 @@ public function resolveField(Repository $repository): EagerField
     {
         return $this
             ->field
+            ->forMcp($repository->isForMcp())
             ->columns($this->getColumns())
             ->resolve($repository);
     }

src/Eager/RelatedCollection.php

@@ -11,6 +11,7 @@
 use Binaryk\LaravelRestify\Fields\MorphToMany;
 use Binaryk\LaravelRestify\Filters\SortableFilter;
 use Binaryk\LaravelRestify\Http\Requests\RestifyRequest;
+use Binaryk\LaravelRestify\MCP\Concerns\HasMcpTools;
 use Binaryk\LaravelRestify\Repositories\Repository;
 use Illuminate\Support\Collection;
 
@@ -97,6 +98,21 @@ public function forIndex(RestifyRequest $request, Repository $repository): self
         });
     }
 
+    public function forMcp(RestifyRequest $request, Repository $repository): self
+    {
+        return $this->filter(function (Related $related) {
+            // If there's an EagerField, check its repository class
+            if ($related->field && $related->field->repositoryClass) {
+                return in_array(HasMcpTools::class, class_uses_recursive($related->field->repositoryClass), true);
+            }
+
+            // For string relationships (without EagerField), we need to find the repository
+            // This happens when relationships are defined as static::$related = ['user']
+            // We'll allow these through and let the serialization handle the filtering
+            return true;
+        });
+    }
+
     public function inRequest(RestifyRequest $request, Repository $repository): self
     {
         return $this->filter(function (mixed $repositoryRelatedField, $repositoryRelatedKey) use (
@@ -167,7 +183,8 @@ public function forRequest(RestifyRequest $request, Repository $repository): sel
             ->authorized($request)
             ->inRequest($request, $repository)
             ->when($request->isShowRequest(), fn (self $collection) => $collection->forShow($request, $repository))
-            ->when($request->isIndexRequest(), fn (self $collection) => $collection->forIndex($request, $repository));
+            ->when($request->isIndexRequest(), fn (self $collection) => $collection->forIndex($request, $repository))
+            ->when($repository->isForMcp(), fn (self $collection) => $collection->forIndex($request, $repository));
     }
 
     public function unserialized(RestifyRequest $request, Repository $repository)

src/Fields/BelongsToMany.php

@@ -5,11 +5,11 @@
 use Binaryk\LaravelRestify\Contracts\RestifySearchable;
 use Binaryk\LaravelRestify\Fields\Concerns\Attachable;
 use Binaryk\LaravelRestify\Http\Requests\RestifyRequest;
+use Binaryk\LaravelRestify\MCP\Requests\McpRequest;
 use Binaryk\LaravelRestify\Repositories\PivotsCollection;
 use Binaryk\LaravelRestify\Repositories\Repository;
 use Closure;
 use Illuminate\Auth\Access\AuthorizationException;
-use Illuminate\Http\Request;
 
 class BelongsToMany extends EagerField
 {
@@ -51,8 +51,15 @@ public function resolve($repository, $attribute = null)
 
         $this->value = $paginator->map(function ($item) {
             try {
-                return $this->repositoryClass::resolveWith($item)
-                    ->allowToShow(app(Request::class))
+                /**
+                 * @var Repository $repositoryFromClass
+                 */
+                $repositoryFromClass = $this->repositoryClass::resolveWith($item);
+
+                return $repositoryFromClass
+                    ->allowToShow(
+                        $this->isForMcp() ? app(McpRequest::class) : app(RestifyRequest::class)
+                    )
                     ->withPivots(
                         PivotsCollection::make($this->pivotFields)
                             ->map(fn (Field $field) => clone $field)

src/Fields/EagerField.php

@@ -4,6 +4,7 @@
 
 use Binaryk\LaravelRestify\Filters\RelatedQuery;
 use Binaryk\LaravelRestify\Http\Requests\RestifyRequest;
+use Binaryk\LaravelRestify\MCP\Requests\McpRequest;
 use Binaryk\LaravelRestify\Repositories\Repository;
 use Binaryk\LaravelRestify\Restify;
 use Binaryk\LaravelRestify\Traits\HasColumns;
@@ -30,6 +31,8 @@ class EagerField extends Field
 
     private RelatedQuery $relatedQuery;
 
+    public bool $forMcp = false;
+
     public function __construct($attribute, ?string $parentRepository = null)
     {
         parent::__construct(attribute: $attribute);
@@ -85,7 +88,7 @@ public function resolve($repository, $attribute = null)
             $serializableRepository = $this->repositoryClass::resolveWith($relatedModel);
 
             $this->value = $serializableRepository
-                ->allowToShow($repository->request ?? app(Request::class))
+                ->allowToShow($this->isForMcp() ? app(McpRequest::class) : app(RestifyRequest::class))
                 ->columns()
                 ->eager($this);
         } catch (AuthorizationException) {
@@ -160,4 +163,22 @@ public function qualifySortable(RestifyRequest $request): ?string
 
         return $table.'.attributes.'.$this->sortableColumn;
     }
+
+    public function forMcp(bool|callable $forMcp = false): self
+    {
+        if (is_callable($forMcp)) {
+            $this->forMcp = $forMcp();
+
+            return $this;
+        }
+
+        $this->forMcp = $forMcp;
+
+        return $this;
+    }
+
+    public function isForMcp(): bool
+    {
+        return $this->forMcp;
+    }
 }

src/MCP/Concerns/McpActionTool.php

@@ -63,7 +63,7 @@ public function actionTool(Action $action, array $arguments, McpActionRequest $a
 
     public static function actionToolSchema(Action $action, ToolInputSchema $schema, McpActionRequest $mcpRequest): void
     {
-        $modelName = class_basename(static::$model);
+        $modelName = class_basename(static::guessModelClassName());
 
         // Add action-specific validation rules
         $actionRules = $action->rules();

src/MCP/Concerns/McpDestroyTool.php

@@ -25,7 +25,7 @@ public function deleteTool(array $arguments, McpRequest $request): array
     public static function destroyToolSchema(ToolInputSchema $schema): void
     {
         $key = static::uriKey();
-        $modelName = class_basename(static::$model);
+        $modelName = class_basename(static::guessModelClassName());
 
         $schema->string('id')
             ->description("The ID of the $modelName to delete")

src/MCP/Concerns/McpGetterTool.php

@@ -50,7 +50,7 @@ public function getterTool(Getter $getter, array $arguments, McpGetterRequest $g
 
     public static function getterToolSchema(Getter $getter, ToolInputSchema $schema, McpGetterRequest $mcpRequest): void
     {
-        $modelName = class_basename(static::$model);
+        $modelName = class_basename(static::guessModelClassName());
 
         // Add getter-specific validation rules if the getter has a rules method
         if (method_exists($getter, 'rules')) {

src/MCP/Concerns/McpShowTool.php

@@ -36,7 +36,7 @@ public function showTool(array $arguments, McpRequest $request): array
 
     public static function showToolSchema(ToolInputSchema $schema): void
     {
-        $modelName = class_basename(static::$model);
+        $modelName = class_basename(static::guessModelClassName());
 
         $schema->string('id')
             ->description("The ID of the $modelName to retrieve")

src/MCP/Concerns/McpStoreTool.php

@@ -21,7 +21,7 @@ public function storeTool(array $arguments, McpRequest $request): array
 
     public static function storeToolSchema(ToolInputSchema $schema): void
     {
-        $repository = static::resolveWith(app(static::$model));
+        $repository = static::resolveWith(static::newModel());
 
         $repository->collectFields($request = app(McpRequest::class))
             ->forStore($request, $repository)

src/MCP/Concerns/McpUpdateTool.php

@@ -25,7 +25,7 @@ public function updateTool(array $arguments, McpRequest $request): array
     public static function updateToolSchema(ToolInputSchema $schema): void
     {
         $key = static::uriKey();
-        $modelName = class_basename(static::$model);
+        $modelName = class_basename(static::guessModelClassName());
 
         $schema->string('id')
             ->description("The ID of the $modelName to update")