[Agent] Update Tool usage to use getters

Author: OskarStark

src/agent/src/InputProcessor/SystemPromptInputProcessor.php

@@ -68,8 +68,8 @@ public function processInput(Input $input): void
 
             $tools = implode(\PHP_EOL.\PHP_EOL, array_map(
                 fn (Tool $tool) => <<<TOOL
-                    ## {$tool->name}
-                    {$tool->description}
+                    ## {$tool->getName()}
+                    {$tool->getDescription()}
                     TOOL,
                 $this->toolbox->getTools()
             ));

src/agent/src/Toolbox/AgentProcessor.php

@@ -52,7 +52,7 @@ public function processInput(Input $input): void
         $options = $input->getOptions();
         // only filter tool map if list of strings is provided as option
         if (isset($options['tools']) && $this->isFlatStringArray($options['tools'])) {
-            $toolMap = array_values(array_filter($toolMap, fn (Tool $tool) => \in_array($tool->name, $options['tools'], true)));
+            $toolMap = array_values(array_filter($toolMap, fn (Tool $tool) => \in_array($tool->getName(), $options['tools'], true)));
         }
 
         $options['tools'] = $toolMap;

src/agent/src/Toolbox/FaultTolerantToolbox.php

@@ -40,7 +40,7 @@ public function execute(ToolCall $toolCall): mixed
         } catch (ToolExecutionExceptionInterface $e) {
             return $e->getToolCallResult();
         } catch (ToolNotFoundException) {
-            $names = array_map(fn (Tool $metadata) => $metadata->name, $this->getTools());
+            $names = array_map(fn (Tool $metadata) => $metadata->getName(), $this->getTools());
 
             return \sprintf('Tool "%s" was not found, please use one of these: %s', $toolCall->getName(), implode(', ', $names));
         }

src/agent/src/Toolbox/ToolCallArgumentResolver.php

@@ -43,7 +43,7 @@ public function __construct(
      */
     public function resolveArguments(Tool $metadata, ToolCall $toolCall): array
     {
-        $method = new \ReflectionMethod($metadata->reference->getClass(), $metadata->reference->getMethod());
+        $method = new \ReflectionMethod($metadata->getReference()->getClass(), $metadata->getReference()->getMethod());
 
         /** @var array<string, \ReflectionParameter> $parameters */
         $parameters = array_column($method->getParameters(), null, 'name');

src/agent/src/Toolbox/Toolbox.php

@@ -82,7 +82,7 @@ public function execute(ToolCall $toolCall): mixed
 
             $arguments = $this->argumentResolver->resolveArguments($metadata, $toolCall);
             $this->eventDispatcher?->dispatch(new ToolCallArgumentsResolved($tool, $metadata, $arguments));
-            $result = $tool->{$metadata->reference->getMethod()}(...$arguments);
+            $result = $tool->{$metadata->getReference()->getMethod()}(...$arguments);
             $this->eventDispatcher?->dispatch(new ToolCallSucceeded($tool, $metadata, $arguments, $result));
         } catch (ToolExecutionExceptionInterface $e) {
             $this->eventDispatcher?->dispatch(new ToolCallFailed($tool, $metadata, $arguments ?? [], $e));
@@ -99,7 +99,7 @@ public function execute(ToolCall $toolCall): mixed
     private function getMetadata(ToolCall $toolCall): Tool
     {
         foreach ($this->getTools() as $metadata) {
-            if ($metadata->name === $toolCall->getName()) {
+            if ($metadata->getName() === $toolCall->getName()) {
                 return $metadata;
             }
         }
@@ -109,13 +109,13 @@ private function getMetadata(ToolCall $toolCall): Tool
 
     private function getExecutable(Tool $metadata): object
     {
-        $className = $metadata->reference->getClass();
+        $className = $metadata->getReference()->getClass();
         foreach ($this->tools as $tool) {
             if ($tool instanceof $className) {
                 return $tool;
             }
         }
 
-        throw ToolNotFoundException::notFoundForReference($metadata->reference);
+        throw ToolNotFoundException::notFoundForReference($metadata->getReference());
     }
 }

src/agent/tests/Toolbox/MetadataFactory/ReflectionFactoryTest.php

@@ -127,10 +127,10 @@ className: ToolMultiple::class,
 
     private function assertToolConfiguration(Tool $metadata, string $className, string $name, string $description, string $method, array $parameters): void
     {
-        $this->assertSame($className, $metadata->reference->getClass());
-        $this->assertSame($method, $metadata->reference->getMethod());
-        $this->assertSame($name, $metadata->name);
-        $this->assertSame($description, $metadata->description);
-        $this->assertSame($parameters, $metadata->parameters);
+        $this->assertSame($className, $metadata->getReference()->getClass());
+        $this->assertSame($method, $metadata->getReference()->getMethod());
+        $this->assertSame($name, $metadata->getName());
+        $this->assertSame($description, $metadata->getDescription());
+        $this->assertSame($parameters, $metadata->getParameters());
     }
 }