Remove unused config interface

Author: SerafimArts

src/Context/ChildContext.php

@@ -7,7 +7,7 @@
 use TypeLang\Mapper\Context\Path\Entry\EntryInterface;
 use TypeLang\Mapper\Context\Path\Path;
 use TypeLang\Mapper\Context\Path\PathInterface;
-use TypeLang\Mapper\Runtime\ConfigurationInterface;
+use TypeLang\Mapper\Runtime\Configuration;
 use TypeLang\Mapper\Runtime\Extractor\TypeExtractorInterface;
 use TypeLang\Mapper\Runtime\Parser\TypeParserInterface;
 use TypeLang\Mapper\Runtime\Repository\TypeRepositoryInterface;
@@ -19,7 +19,7 @@ protected function __construct(
         private readonly EntryInterface $entry,
         mixed $value,
         Direction $direction,
-        ConfigurationInterface $config,
+        Configuration $config,
         TypeExtractorInterface $extractor,
         TypeParserInterface $parser,
         TypeRepositoryInterface $types,

src/Context/Context.php

@@ -5,27 +5,24 @@
 namespace TypeLang\Mapper\Context;
 
 use JetBrains\PhpStorm\Language;
-use Psr\Log\LoggerInterface;
 use TypeLang\Mapper\Context\Path\Entry\EntryInterface;
 use TypeLang\Mapper\Context\Path\PathInterface;
-use TypeLang\Mapper\Runtime\ConfigurationInterface;
+use TypeLang\Mapper\Runtime\Configuration;
 use TypeLang\Mapper\Runtime\Extractor\TypeExtractorInterface;
 use TypeLang\Mapper\Runtime\Parser\TypeParserInterface;
 use TypeLang\Mapper\Runtime\Repository\TypeRepositoryInterface;
-use TypeLang\Mapper\Runtime\Tracing\TracerInterface;
 use TypeLang\Mapper\Type\TypeInterface;
 use TypeLang\Parser\Node\Stmt\TypeStatement;
 
 abstract class Context implements
-    ConfigurationInterface,
     TypeExtractorInterface,
     TypeParserInterface,
     TypeRepositoryInterface
 {
     protected function __construct(
         protected readonly mixed $value,
         protected readonly Direction $direction,
-        protected readonly ConfigurationInterface $config,
+        protected readonly Configuration $config,
         protected readonly TypeExtractorInterface $extractor,
         protected readonly TypeParserInterface $parser,
         protected readonly TypeRepositoryInterface $types,
@@ -69,16 +66,6 @@ public function isStrictTypesEnabled(): bool
         return $this->config->isStrictTypesEnabled();
     }
 
-    public function findLogger(): ?LoggerInterface
-    {
-        return $this->config->findLogger();
-    }
-
-    public function findTracer(): ?TracerInterface
-    {
-        return $this->config->findTracer();
-    }
-
     /**
      * @api
      */

src/Context/RootContext.php

@@ -7,7 +7,6 @@
 use TypeLang\Mapper\Context\Path\Path;
 use TypeLang\Mapper\Context\Path\PathInterface;
 use TypeLang\Mapper\Runtime\Configuration;
-use TypeLang\Mapper\Runtime\ConfigurationInterface;
 use TypeLang\Mapper\Runtime\Extractor\TypeExtractorInterface;
 use TypeLang\Mapper\Runtime\Parser\TypeParserInterface;
 use TypeLang\Mapper\Runtime\Repository\TypeRepositoryInterface;
@@ -22,20 +21,18 @@ final class RootContext extends Context
 
     public static function forNormalization(
         mixed $value,
-        ConfigurationInterface $config,
+        Configuration $config,
         TypeExtractorInterface $extractor,
         TypeParserInterface $parser,
         TypeRepositoryInterface $types,
     ): self {
-        if ($config instanceof Configuration) {
-            // Disable strict-types for normalization if option is not set
-            if (!$config->isStrictTypesOptionDefined()) {
-                $config = $config->withStrictTypes(false);
-            }
-
-            // ...
+        // Disable strict-types for normalization if option is not set
+        if (!$config->isStrictTypesOptionDefined()) {
+            $config = $config->withStrictTypes(false);
         }
 
+        // ...
+
         return new self(
             value: $value,
             direction: Direction::Normalize,
@@ -48,20 +45,18 @@ public static function forNormalization(
 
     public static function forDenormalization(
         mixed $value,
-        ConfigurationInterface $config,
+        Configuration $config,
         TypeExtractorInterface $extractor,
         TypeParserInterface $parser,
         TypeRepositoryInterface $types,
     ): self {
-        if ($config instanceof Configuration) {
-            // Enable strict-types for denormalization if option is not set
-            if (!$config->isStrictTypesOptionDefined()) {
-                $config = $config->withStrictTypes(true);
-            }
-
-            // ...
+        // Enable strict-types for denormalization if option is not set
+        if (!$config->isStrictTypesOptionDefined()) {
+            $config = $config->withStrictTypes(true);
         }
 
+        // ...
+
         return new self(
             value: $value,
             direction: Direction::Denormalize,

src/Runtime/Configuration.php

@@ -7,7 +7,7 @@
 use Psr\Log\LoggerInterface;
 use TypeLang\Mapper\Runtime\Tracing\TracerInterface;
 
-final class Configuration implements ConfigurationInterface
+final class Configuration
 {
     /**
      * Default value for {@see $isObjectsAsArrays} option.
@@ -65,9 +65,16 @@ public function withObjectsAsArrays(?bool $enabled = null): self
         );
     }
 
+    /**
+     * Specifies the default normalization settings for the object.
+     *
+     * In case of the method returns {@see true}, the object will be converted
+     * to an associative array (hash map) unless otherwise specified.
+     */
     public function isObjectAsArray(): bool
     {
-        return $this->isObjectsAsArrays ?? self::OBJECTS_AS_ARRAYS_DEFAULT_VALUE;
+        return $this->isObjectsAsArrays
+            ?? self::OBJECTS_AS_ARRAYS_DEFAULT_VALUE;
     }
 
     /**
@@ -97,9 +104,17 @@ public function withStrictTypes(?bool $enabled = null): self
         );
     }
 
+    /**
+     * In case of method returns {@see true}, all types will be checked
+     * for compliance.
+     *
+     * Otherwise, the value will attempt to be converted to the
+     * required type if possible.
+     */
     public function isStrictTypesEnabled(): bool
     {
-        return $this->isStrictTypes ?? self::STRICT_TYPES_DEFAULT_VALUE;
+        return $this->isStrictTypes
+            ?? self::STRICT_TYPES_DEFAULT_VALUE;
     }
 
     /**
@@ -128,6 +143,10 @@ public function withLogger(?LoggerInterface $logger = null): self
         );
     }
 
+    /**
+     * If this method returns {@see LoggerInterface}, then the given logger
+     * will be enabled. Otherwise logger should be disabled.
+     */
     public function findLogger(): ?LoggerInterface
     {
         return $this->logger;
@@ -150,6 +169,10 @@ public function withTracer(?TracerInterface $tracer = null): self
         );
     }
 
+    /**
+     * If this method returns {@see TracerInterface}, then the application
+     * tracing will be enabled. Otherwise tracing should be disabled.
+     */
     public function findTracer(): ?TracerInterface
     {
         return $this->tracer;

src/Runtime/ConfigurationInterface.php

@@ -4,18 +4,6 @@
 
 namespace TypeLang\Mapper\Tests\Mapping;
 
-use TypeLang\Mapper\Platform\PlatformInterface;
-use TypeLang\Mapper\Platform\StandardPlatform;
-use TypeLang\Mapper\Runtime\Configuration;
-use TypeLang\Mapper\Runtime\ConfigurationInterface;
-use TypeLang\Mapper\Context\Direction;
-use TypeLang\Mapper\Context\RootContext;
-use TypeLang\Mapper\Runtime\Extractor\NativeTypeExtractor;
-use TypeLang\Mapper\Runtime\Extractor\TypeExtractorInterface;
-use TypeLang\Mapper\Runtime\Parser\TypeLangParser;
-use TypeLang\Mapper\Runtime\Parser\TypeParserInterface;
-use TypeLang\Mapper\Runtime\Repository\TypeRepository;
-use TypeLang\Mapper\Runtime\Repository\TypeRepositoryInterface;
 use TypeLang\Mapper\Tests\TestCase;
 
 abstract class MappingTestCase extends TestCase {}

tests/Mapping/MappingTestCase.php

@@ -11,7 +11,6 @@
 use TypeLang\Mapper\Platform\PlatformInterface;
 use TypeLang\Mapper\Platform\StandardPlatform;
 use TypeLang\Mapper\Runtime\Configuration;
-use TypeLang\Mapper\Runtime\ConfigurationInterface;
 use TypeLang\Mapper\Runtime\Extractor\NativeTypeExtractor;
 use TypeLang\Mapper\Runtime\Extractor\TypeExtractorInterface;
 use TypeLang\Mapper\Runtime\Parser\TypeLangParser;
@@ -42,7 +41,7 @@ private static function dataProviderKeyOf(mixed $value): string
         ]);
     }
 
-    protected function createConfiguration(bool $strictTypes = true): ConfigurationInterface
+    protected function createConfiguration(bool $strictTypes = true): Configuration
     {
         return new Configuration(
             isStrictTypes: $strictTypes,