Replace readonly keyword on classes with readonly properties for BC promise

Author: chr-hertel

src/agent/src/Agent.php

@@ -21,29 +21,29 @@
 /**
  * @author Christopher Hertel <[email protected]>
  */
-final readonly class Agent implements AgentInterface
+final class Agent implements AgentInterface
 {
     /**
      * @var InputProcessorInterface[]
      */
-    private array $inputProcessors;
+    private readonly array $inputProcessors;
 
     /**
      * @var OutputProcessorInterface[]
      */
-    private array $outputProcessors;
+    private readonly array $outputProcessors;
 
     /**
      * @param InputProcessorInterface[]  $inputProcessors
      * @param OutputProcessorInterface[] $outputProcessors
      * @param non-empty-string           $model
      */
     public function __construct(
-        private PlatformInterface $platform,
-        private string $model,
+        private readonly PlatformInterface $platform,
+        private readonly string $model,
         iterable $inputProcessors = [],
         iterable $outputProcessors = [],
-        private string $name = 'agent',
+        private readonly string $name = 'agent',
     ) {
         $this->inputProcessors = $this->initializeProcessors($inputProcessors, InputProcessorInterface::class);
         $this->outputProcessors = $this->initializeProcessors($outputProcessors, OutputProcessorInterface::class);

src/agent/src/Attribute/AsInputProcessor.php

@@ -15,15 +15,15 @@
  * @author Vincent Langlet <[email protected]>
  */
 #[\Attribute(\Attribute::TARGET_CLASS | \Attribute::IS_REPEATABLE)]
-final readonly class AsInputProcessor
+final class AsInputProcessor
 {
     /**
      * @param string|null $agent the service id of the agent which will use this processor,
      *                           null to register this processor for all existing agents
      */
     public function __construct(
-        public ?string $agent = null,
-        public int $priority = 0,
+        public readonly ?string $agent = null,
+        public readonly int $priority = 0,
     ) {
     }
 }

src/agent/src/Attribute/AsOutputProcessor.php

@@ -15,15 +15,15 @@
  * @author Vincent Langlet <[email protected]>
  */
 #[\Attribute(\Attribute::TARGET_CLASS | \Attribute::IS_REPEATABLE)]
-final readonly class AsOutputProcessor
+final class AsOutputProcessor
 {
     /**
      * @param string|null $agent the service id of the agent which will use this processor,
      *                           null to register this processor for all existing agents
      */
     public function __construct(
-        public ?string $agent = null,
-        public int $priority = 0,
+        public readonly ?string $agent = null,
+        public readonly int $priority = 0,
     ) {
     }
 }

src/agent/src/InputProcessor/SystemPromptInputProcessor.php

@@ -26,17 +26,17 @@
 /**
  * @author Christopher Hertel <[email protected]>
  */
-final readonly class SystemPromptInputProcessor implements InputProcessorInterface
+final class SystemPromptInputProcessor implements InputProcessorInterface
 {
     /**
      * @param \Stringable|TranslatableInterface|string|File $systemPrompt the system prompt to prepend to the input messages, or a File object to read from
      * @param ToolboxInterface|null                         $toolbox      the tool box to be used to append the tool definitions to the system prompt
      */
     public function __construct(
-        private \Stringable|TranslatableInterface|string|File $systemPrompt,
-        private ?ToolboxInterface $toolbox = null,
-        private ?TranslatorInterface $translator = null,
-        private LoggerInterface $logger = new NullLogger(),
+        private readonly \Stringable|TranslatableInterface|string|File $systemPrompt,
+        private readonly ?ToolboxInterface $toolbox = null,
+        private readonly ?TranslatorInterface $translator = null,
+        private readonly LoggerInterface $logger = new NullLogger(),
     ) {
         if ($this->systemPrompt instanceof TranslatableInterface && !$this->translator) {
             throw new RuntimeException('Translatable system prompt is not supported when no translator is provided.');

src/agent/src/Memory/EmbeddingProvider.php

@@ -23,12 +23,12 @@
 /**
  * @author Denis Zunke <[email protected]>
  */
-final readonly class EmbeddingProvider implements MemoryProviderInterface
+final class EmbeddingProvider implements MemoryProviderInterface
 {
     public function __construct(
-        private PlatformInterface $platform,
-        private Model $model,
-        private StoreInterface $vectorStore,
+        private readonly PlatformInterface $platform,
+        private readonly Model $model,
+        private readonly StoreInterface $vectorStore,
     ) {
     }
 

src/agent/src/Memory/Memory.php

@@ -14,10 +14,10 @@
 /**
  * @author Denis Zunke <[email protected]>
  */
-final readonly class Memory
+final class Memory
 {
     public function __construct(
-        private string $content,
+        private readonly string $content,
     ) {
     }
 

src/agent/src/Memory/MemoryInputProcessor.php

@@ -18,7 +18,7 @@
 /**
  * @author Denis Zunke <[email protected]>
  */
-final readonly class MemoryInputProcessor implements InputProcessorInterface
+final class MemoryInputProcessor implements InputProcessorInterface
 {
     private const MEMORY_PROMPT_MESSAGE = <<<MARKDOWN
         # Conversation Memory
@@ -31,7 +31,7 @@
     /**
      * @var MemoryProviderInterface[]
      */
-    private array $memoryProviders;
+    private readonly array $memoryProviders;
 
     public function __construct(
         MemoryProviderInterface ...$memoryProviders,

src/agent/src/Memory/StaticMemoryProvider.php

@@ -16,12 +16,12 @@
 /**
  * @author Denis Zunke <[email protected]>
  */
-final readonly class StaticMemoryProvider implements MemoryProviderInterface
+final class StaticMemoryProvider implements MemoryProviderInterface
 {
     /**
      * @var array<string>
      */
-    private array $memory;
+    private readonly array $memory;
 
     public function __construct(string ...$memory)
     {

src/agent/src/MultiAgent/Handoff.php

@@ -19,14 +19,14 @@
  *
  * @author Oskar Stark <[email protected]>
  */
-final readonly class Handoff
+final class Handoff
 {
     /**
      * @param string[] $when Keywords or phrases that indicate this handoff
      */
     public function __construct(
-        private AgentInterface $to,
-        private array $when,
+        private readonly AgentInterface $to,
+        private readonly array $when,
     ) {
         if ([] === $when) {
             throw new InvalidArgumentException('Handoff must have at least one "when" condition.');

src/agent/src/MultiAgent/Handoff/Decision.php

@@ -16,15 +16,15 @@
  *
  * @author Oskar Stark <[email protected]>
  */
-final readonly class Decision
+final class Decision
 {
     /**
      * @param string $agentName The name of the selected agent, or empty string if no specific agent is selected
      * @param string $reasoning The reasoning behind the selection
      */
     public function __construct(
-        private string $agentName,
-        private string $reasoning = 'No reasoning provided',
+        private readonly string $agentName,
+        private readonly string $reasoning = 'No reasoning provided',
     ) {
     }