feat: Make all final patch operation classes readonly

Will break on 8.1, to be rebased once the project drops 8.1 support.

I have intentionally left the base `PatchOperation` class as *not*
readonly in case end-users want to extend it with mutable operation
DTOs for any reason. Instead, I have just left `op` as a readonly
property in the base class.

Author: acoulton

src/operations/Add.php

@@ -9,11 +9,11 @@
  *     value: mixed,
  * }
  */
-final class Add extends PatchOperation
+final readonly class Add extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
-        public readonly mixed $value,
+        public string $path,
+        public mixed $value,
     ) {
         parent::__construct('add');
     }

src/operations/Copy.php

@@ -9,11 +9,11 @@
  *     from: string,
  * }
  */
-final class Copy extends PatchOperation
+final readonly class Copy extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
-        public readonly string $from,
+        public string $path,
+        public string $from,
     ) {
         parent::__construct('copy');
     }

src/operations/Move.php

@@ -9,11 +9,11 @@
  *     from: string,
  * }
  */
-final class Move extends PatchOperation
+final readonly class Move extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
-        public readonly string $from,
+        public string $path,
+        public string $from,
     ) {
         parent::__construct('move');
     }

src/operations/PatchOperation.php

@@ -2,9 +2,9 @@
 
 namespace blancks\JsonPatch\operations;
 
-abstract class PatchOperation
+abstract readonly class PatchOperation
 {
     public function __construct(
-        public readonly string $op,
+        public string $op,
     ) {}
 }

src/operations/PatchOperationList.php

@@ -8,12 +8,12 @@
 use blancks\JsonPatch\json\handlers\JsonHandlerInterface;
 use stdClass;
 
-final class PatchOperationList implements \JsonSerializable
+final readonly class PatchOperationList implements \JsonSerializable
 {
     /**
      * @phpstan-var list<PatchOperation>
      */
-    public readonly array $operations;
+    public array $operations;
 
     /**
      * @param string $jsonOperations

src/operations/Remove.php

@@ -8,10 +8,10 @@
  *     path: string,
  * }
  */
-final class Remove extends PatchOperation
+final readonly class Remove extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
+        public string $path,
     ) {
         parent::__construct('remove');
     }

src/operations/Replace.php

@@ -9,11 +9,11 @@
  *     value: mixed,
  * }
  */
-final class Replace extends PatchOperation
+final readonly class Replace extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
-        public readonly mixed $value,
+        public string $path,
+        public mixed $value,
     ) {
         parent::__construct('replace');
     }

src/operations/Test.php

@@ -9,11 +9,11 @@
  *     value: mixed,
  * }
  */
-final class Test extends PatchOperation
+final readonly class Test extends PatchOperation
 {
     public function __construct(
-        public readonly string $path,
-        public readonly mixed $value,
+        public string $path,
+        public mixed $value,
     ) {
         parent::__construct('test');
     }

tests/operations/PatchOperationListTest.php

@@ -23,8 +23,8 @@
 use blancks\JsonPatchTest\JsonPatchCompliance;
 use PHPUnit\Framework\Attributes\CoversClass;
 use PHPUnit\Framework\Attributes\DataProvider;
-use PHPUnit\Framework\Assert;
 use PHPUnit\Framework\Attributes\UsesClass;
+use PHPUnit\Framework\Assert;
 use stdClass;
 use Throwable;
 
@@ -256,3 +256,23 @@ public function testItThrowsOnAttemptToCreateFromInvalidJson(string $json, strin
         PatchOperationList::fromJson($json);
     }
 }
+
+readonly class Append extends PatchOperation
+{
+    public function __construct(
+        public string $path,
+        public string $suffix,
+    ) {
+        parent::__construct('append');
+    }
+}
+
+readonly class CustomAdd extends PatchOperation
+{
+    public function __construct(
+        public string $path,
+        public mixed $value,
+    ) {
+        parent::__construct('add');
+    }
+}