refactor: remove unused JsonOptions and simplify Json adapter implementation

- Replaced custom logic with native JSON functions, utilizing `JSON_UNESCAPED_UNICODE` and `JSON_UNESCAPED_SLASHES`.
- Removed unsupported options such as `cycle_check` and `enable_json_expr_finder`.
- Add tests for serialization and deserialization

Signed-off-by: mmalac <mmalac@sygic.com>

Author: mmalac

psalm-baseline.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="6.14.3@d0b040a91f280f071c1abcb1b77ce3822058725a">
+<files psalm-version="6.15.1@28dc127af1b5aecd52314f6f645bafc10d0e11f9">
   <file src="src/Adapter/Json.php">
     <MoreSpecificImplementedParamType>
       <code><![CDATA[$options]]></code>
@@ -8,12 +8,6 @@
       <code><![CDATA[$options]]></code>
     </NonInvariantDocblockPropertyType>
   </file>
-  <file src="src/Adapter/JsonOptions.php">
-    <PossiblyUnusedMethod>
-      <code><![CDATA[setCycleCheck]]></code>
-      <code><![CDATA[setEnableJsonExprFinder]]></code>
-    </PossiblyUnusedMethod>
-  </file>
   <file src="src/Adapter/PhpSerialize.php">
     <MoreSpecificImplementedParamType>
       <code><![CDATA[$options]]></code>

src/Adapter/Json.php

@@ -8,14 +8,12 @@
 use JsonException;
 use Laminas\Serializer\Exception;
 
-use function in_array;
-use function is_array;
-use function is_int;
-use function is_object;
 use function json_decode;
 use function json_encode;
 
 use const JSON_THROW_ON_ERROR;
+use const JSON_UNESCAPED_SLASHES;
+use const JSON_UNESCAPED_UNICODE;
 
 final class Json extends AbstractAdapter
 {
@@ -56,15 +54,8 @@ public function getOptions(): JsonOptions
      */
     public function serialize(mixed $value): string
     {
-        $options    = $this->getOptions();
-        $cycleCheck = $options->getCycleCheck();
-        $opts       = [
-            'enableJsonExprFinder' => $options->getEnableJsonExprFinder(),
-            'objectDecodeType'     => $options->isAssocArray(),
-        ];
-
         try {
-            return $this->encode($value, $cycleCheck, $opts);
+            return $this->encode($value);
         } catch (InvalidArgumentException $e) {
             throw new Exception\InvalidArgumentException('Serialization failed: ' . $e->getMessage(), 0, $e);
         } catch (JsonException $e) {
@@ -89,45 +80,20 @@ public function unserialize(string $serialized): mixed
         return $ret;
     }
 
-    /**
-     * @param mixed[] $options
-     */
-    private function encode(mixed $value, bool $cycleCheck, array $options): string
+    private function encode(mixed $value): string
     {
-        if ($cycleCheck) {
-            $seen         = [];
-            $detectCycles = function (mixed &$val) use (&$seen, &$detectCycles): void {
-                if (is_array($val) || is_object($val)) {
-                    if (in_array($val, $seen, true)) {
-                        throw new InvalidArgumentException("Cycle detected in value to be JSON encoded");
-                    }
-                    $seen[] = $val;
-                    foreach ($val as &$item) {
-                        $detectCycles($item);
-                    }
-                }
-            };
-            $detectCycles($value);
-        }
-
-        $jsonOptions = isset($options['json_encode_options']) && is_int($options['json_encode_options'])
-            ? $options['json_encode_options']
-            : 0;
-
-        $encoded = json_encode($value, $jsonOptions | JSON_THROW_ON_ERROR);
-        if ($encoded === false) {
-            throw new JsonException('Syntax error');
-        }
-
-        return $encoded;
+        return json_encode(
+            $value,
+            JSON_UNESCAPED_UNICODE | JSON_UNESCAPED_SLASHES | JSON_THROW_ON_ERROR
+        );
     }
 
     private function decode(string $value, bool $assoc): mixed
     {
         return json_decode(
             $value,
             $assoc,
-            512, // depth
+            512,
             JSON_THROW_ON_ERROR
         );
     }

src/Adapter/JsonOptions.php

@@ -4,38 +4,10 @@
 
 namespace Laminas\Serializer\Adapter;
 
-use Laminas\Serializer\Exception;
-
 final class JsonOptions extends AdapterOptions
 {
-    protected bool $cycleCheck = false;
-
-    protected bool $enableJsonExprFinder = false;
-    private bool $assocArray             = true;
-
-    public function setCycleCheck(bool $flag): void
-    {
-        $this->cycleCheck = $flag;
-    }
-
-    public function getCycleCheck(): bool
-    {
-        return $this->cycleCheck;
-    }
-
-    public function setEnableJsonExprFinder(bool $flag): void
-    {
-        $this->enableJsonExprFinder = $flag;
-    }
-
-    public function getEnableJsonExprFinder(): bool
-    {
-        return $this->enableJsonExprFinder;
-    }
+    private bool $assocArray = true;
 
-    /**
-     * @throws Exception\InvalidArgumentException
-     */
     public function setAssocArray(bool $assocArray): void
     {
         $this->assocArray = $assocArray;