chore: Add final, readonly, and internal if applicable to all classes.

Author: priyadi

CHANGELOG.md

@@ -5,6 +5,7 @@
 * perf(`TypeResolver`): Optimize `getSimpleTypes`.
 * fix(`MainTransformer`): Reduce GC interval to 500.
 * build: Require PHP 8.2.
+* chore: Add `final`, `readonly`, and `internal` if applicable to all classes.
 
 ## 0.9.0
 

src/Command/MappingCommand.php

@@ -22,8 +22,11 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
+/**
+ * @internal
+ */
 #[AsCommand(name: 'rekalogika:mapper:mapping', description: 'Dump mapping table')]
-class MappingCommand extends Command
+final class MappingCommand extends Command
 {
     public function __construct(
         private MappingFactoryInterface $mappingFactory

src/Command/MarkdownLikeTableStyle.php

@@ -18,7 +18,7 @@
 /**
  * Markdown-like table style, for the ease of copy-paste to documentation.
  */
-class MarkdownLikeTableStyle extends TableStyle
+final class MarkdownLikeTableStyle extends TableStyle
 {
     public function __construct()
     {

src/Command/TryCommand.php

@@ -25,8 +25,11 @@
 use Symfony\Component\Console\Output\OutputInterface;
 use Symfony\Component\Console\Style\SymfonyStyle;
 
+/**
+ * @internal
+ */
 #[AsCommand(name: 'rekalogika:mapper:try', description: 'Gets the mapping result from a source and target type pair.')]
-class TryCommand extends Command
+final class TryCommand extends Command
 {
     public function __construct(
         private TransformerRegistryInterface $transformerRegistry,

src/Command/TryPropertyCommand.php

@@ -25,8 +25,11 @@
 use Symfony\Component\Console\Style\SymfonyStyle;
 use Symfony\Component\PropertyInfo\PropertyInfoExtractorInterface;
 
+/**
+ * @internal
+ */
 #[AsCommand(name: 'rekalogika:mapper:tryproperty', description: 'Gets the mapping result by providing the class and property name of the source and target.')]
-class TryPropertyCommand extends Command
+final class TryPropertyCommand extends Command
 {
     public function __construct(
         private TransformerRegistryInterface $transformerRegistry,

src/CustomMapper/Exception/ObjectMapperNotFoundException.php

@@ -15,7 +15,10 @@
 
 use Rekalogika\Mapper\Exception\UnexpectedValueException;
 
-class ObjectMapperNotFoundException extends UnexpectedValueException
+/**
+ * @internal
+ */
+final class ObjectMapperNotFoundException extends UnexpectedValueException
 {
     public function __construct(
         string $sourceClass,

src/CustomMapper/Implementation/CachingObjectMapperResolver.php

@@ -17,7 +17,10 @@
 use Rekalogika\Mapper\CustomMapper\ObjectMapperResolverInterface;
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
-class CachingObjectMapperResolver implements ObjectMapperResolverInterface
+/**
+ * @internal
+ */
+final class CachingObjectMapperResolver implements ObjectMapperResolverInterface
 {
     /**
      * @var array<class-string,array<class-string,ServiceMethodSpecification>>

src/CustomMapper/Implementation/ObjectMapperResolver.php

@@ -19,7 +19,10 @@
 use Rekalogika\Mapper\CustomMapper\ObjectMapperTableFactoryInterface;
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
-class ObjectMapperResolver implements ObjectMapperResolverInterface
+/**
+ * @internal
+ */
+final class ObjectMapperResolver implements ObjectMapperResolverInterface
 {
     private ?ObjectMapperTable $objectMapperTable = null;
 

src/CustomMapper/Implementation/ObjectMapperTableFactory.php

@@ -17,7 +17,10 @@
 use Rekalogika\Mapper\CustomMapper\ObjectMapperTableFactoryInterface;
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
-class ObjectMapperTableFactory implements ObjectMapperTableFactoryInterface
+/**
+ * @internal
+ */
+final class ObjectMapperTableFactory implements ObjectMapperTableFactoryInterface
 {
     private ObjectMapperTable $objectMapperTable;
 

src/CustomMapper/Implementation/ObjectMapperTableWarmer.php

@@ -18,7 +18,10 @@
 use Symfony\Component\HttpKernel\KernelInterface;
 use Symfony\Component\VarExporter\VarExporter;
 
-class ObjectMapperTableWarmer implements CacheWarmerInterface
+/**
+ * @internal
+ */
+final readonly class ObjectMapperTableWarmer implements CacheWarmerInterface
 {
     public function __construct(
         private ObjectMapperTableFactoryInterface $objectMapperTableFactory,

src/CustomMapper/Implementation/PropertyMapperResolver.php

@@ -16,7 +16,10 @@
 use Rekalogika\Mapper\CustomMapper\PropertyMapperResolverInterface;
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
-class PropertyMapperResolver implements PropertyMapperResolverInterface
+/**
+ * @internal
+ */
+final class PropertyMapperResolver implements PropertyMapperResolverInterface
 {
     /**
      * @var array<class-string,array<string,array<class-string,ServiceMethodSpecification>>>

src/CustomMapper/Implementation/WarmableObjectMapperTableFactory.php

@@ -20,7 +20,10 @@
 use Symfony\Component\HttpKernel\KernelInterface;
 use Symfony\Component\VarExporter\VarExporter;
 
-class WarmableObjectMapperTableFactory implements
+/**
+ * @internal
+ */
+final class WarmableObjectMapperTableFactory implements
     ObjectMapperTableFactoryInterface,
     CacheWarmerInterface
 {

src/CustomMapper/ObjectMapperResolverInterface.php

@@ -16,6 +16,9 @@
 use Rekalogika\Mapper\CustomMapper\Exception\ObjectMapperNotFoundException;
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
+/**
+ * @internal
+ */
 interface ObjectMapperResolverInterface
 {
     /**

src/CustomMapper/ObjectMapperTable.php

@@ -17,8 +17,9 @@
 
 /**
  * @implements \IteratorAggregate<int,ObjectMapperTableEntry>
+ * @internal
  */
-class ObjectMapperTable implements \IteratorAggregate
+final class ObjectMapperTable implements \IteratorAggregate
 {
     /**
      * @var array<class-string,array<class-string,ServiceMethodSpecification>>

src/CustomMapper/ObjectMapperTableEntry.php

@@ -15,7 +15,10 @@
 
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
-class ObjectMapperTableEntry
+/**
+ * @internal
+ */
+final readonly class ObjectMapperTableEntry
 {
     /**
      * @param class-string $sourceClass

src/CustomMapper/ObjectMapperTableFactoryInterface.php

@@ -13,6 +13,9 @@
 
 namespace Rekalogika\Mapper\CustomMapper;
 
+/**
+ * @internal
+ */
 interface ObjectMapperTableFactoryInterface
 {
     public function createObjectMapperTable(): ObjectMapperTable;

src/CustomMapper/PropertyMapperResolverInterface.php

@@ -15,6 +15,9 @@
 
 use Rekalogika\Mapper\ServiceMethod\ServiceMethodSpecification;
 
+/**
+ * @internal
+ */
 interface PropertyMapperResolverInterface
 {
     /**

src/Debug/Helper.php

@@ -17,7 +17,7 @@
 use Rekalogika\Mapper\Util\TypeUtil;
 use Symfony\Component\PropertyInfo\Type;
 
-final class Helper
+final readonly class Helper
 {
     /**
      * @param Type|array<int,Type|MixedType> $type

src/Debug/MapperDataCollector.php

@@ -19,6 +19,9 @@
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
 
+/**
+ * @internal
+ */
 final class MapperDataCollector extends AbstractDataCollector
 {
     public function getName(): string

src/Debug/TraceData.php

@@ -19,6 +19,9 @@
 use Symfony\Component\PropertyInfo\Type;
 use Symfony\Component\VarExporter\LazyObjectInterface;
 
+/**
+ * @internal
+ */
 final class TraceData
 {
     private string $sourceType;

src/Debug/TraceableMappingFactory.php

@@ -16,6 +16,9 @@
 use Rekalogika\Mapper\Mapping\Mapping;
 use Rekalogika\Mapper\Mapping\MappingFactoryInterface;
 
+/**
+ * @internal
+ */
 final class TraceableMappingFactory implements MappingFactoryInterface
 {
     private bool $mappingCollected = false;

src/Debug/TraceableObjectToObjectMetadataFactory.php

@@ -16,7 +16,10 @@
 use Rekalogika\Mapper\Transformer\ObjectToObjectMetadata\ObjectToObjectMetadata;
 use Rekalogika\Mapper\Transformer\ObjectToObjectMetadata\ObjectToObjectMetadataFactoryInterface;
 
-final class TraceableObjectToObjectMetadataFactory implements ObjectToObjectMetadataFactoryInterface
+/**
+ * @internal
+ */
+final readonly class TraceableObjectToObjectMetadataFactory implements ObjectToObjectMetadataFactoryInterface
 {
     public function __construct(
         private ObjectToObjectMetadataFactoryInterface $decorated,

src/Debug/TraceableTransformer.php

@@ -23,6 +23,9 @@
 use Rekalogika\Mapper\Transformer\Contracts\TransformerInterface;
 use Symfony\Component\PropertyInfo\Type;
 
+/**
+ * @internal
+ */
 final class TraceableTransformer extends AbstractTransformerDecorator implements
     TransformerInterface,
     MainTransformerAwareInterface

src/DependencyInjection/CompilerPass/DebugPass.php

@@ -20,7 +20,10 @@
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Reference;
 
-final class DebugPass implements CompilerPassInterface
+/**
+ * @internal
+ */
+final readonly class DebugPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {

src/DependencyInjection/CompilerPass/ObjectMapperPass.php

@@ -18,7 +18,10 @@
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 
-final class ObjectMapperPass implements CompilerPassInterface
+/**
+ * @internal
+ */
+final readonly class ObjectMapperPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {

src/DependencyInjection/CompilerPass/PropertyMapperPass.php

@@ -18,7 +18,10 @@
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 
-final class PropertyMapperPass implements CompilerPassInterface
+/**
+ * @internal
+ */
+final readonly class PropertyMapperPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {

src/DependencyInjection/CompilerPass/RemoveOptionalDefinitionPass.php

@@ -18,7 +18,10 @@
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\Uid\Factory\UuidFactory;
 
-final class RemoveOptionalDefinitionPass implements CompilerPassInterface
+/**
+ * @internal
+ */
+final readonly class RemoveOptionalDefinitionPass implements CompilerPassInterface
 {
     public function process(ContainerBuilder $container): void
     {

src/DependencyInjection/RekalogikaMapperExtension.php

@@ -25,7 +25,10 @@
 use Symfony\Component\DependencyInjection\Extension\Extension;
 use Symfony\Component\DependencyInjection\Loader\PhpFileLoader;
 
-class RekalogikaMapperExtension extends Extension
+/**
+ * @internal
+ */
+final class RekalogikaMapperExtension extends Extension
 {
     public function load(array $configs, ContainerBuilder $container): void
     {

src/MainTransformer/MainTransformer.php

@@ -34,7 +34,10 @@
 use Rekalogika\Mapper\Util\TypeGuesser;
 use Symfony\Component\PropertyInfo\Type;
 
-class MainTransformer implements MainTransformerInterface
+/**
+ * @internal
+ */
+final class MainTransformer implements MainTransformerInterface
 {
     public static int $manualGcInterval = 500;
     private static int $runCounter = 1;

src/MainTransformer/Model/DebugContext.php

@@ -19,6 +19,7 @@
 /**
  * Debug context for main transformer. Used for tracing.
  *
+ * @internal
  * @immutable
  */
 final readonly class DebugContext

src/MainTransformer/Model/Path.php

@@ -18,7 +18,7 @@
  *
  * @immutable
  */
-readonly class Path implements \Stringable
+final readonly class Path implements \Stringable
 {
     /**
      * @param list<string> $path

src/Mapper.php

@@ -18,7 +18,10 @@
 use Rekalogika\Mapper\MainTransformer\MainTransformerInterface;
 use Rekalogika\Mapper\Util\TypeFactory;
 
-final class Mapper implements MapperInterface
+/**
+ * @internal
+ */
+final readonly class Mapper implements MapperInterface
 {
     public function __construct(
         private MainTransformerInterface $transformer,

src/MapperFactory/ServiceLocator.php

@@ -19,7 +19,7 @@
 /**
  * Simple container for non-framework use
  */
-final class ServiceLocator implements ContainerInterface
+final readonly class ServiceLocator implements ContainerInterface
 {
     /**
      * @param array<array-key,mixed> $services

src/Mapping/Implementation/MappingCacheWarmer.php

@@ -15,7 +15,10 @@
 
 use Symfony\Component\HttpKernel\CacheWarmer\CacheWarmerInterface;
 
-final class MappingCacheWarmer implements CacheWarmerInterface
+/**
+ * @internal
+ */
+final readonly class MappingCacheWarmer implements CacheWarmerInterface
 {
     public function __construct(
         private WarmableMappingFactory $warmableMappingFactory,

src/Mapping/Implementation/MappingFactory.php

@@ -22,7 +22,7 @@
 use Symfony\Component\PropertyInfo\Type;
 
 /**
- * Initialize transformer mappings
+ * @internal
  */
 final class MappingFactory implements MappingFactoryInterface
 {

src/Mapping/Implementation/WarmableMappingFactory.php

@@ -19,6 +19,9 @@
 use Symfony\Component\HttpKernel\KernelInterface;
 use Symfony\Component\VarExporter\VarExporter;
 
+/**
+ * @internal
+ */
 final class WarmableMappingFactory implements MappingFactoryInterface
 {
     public const CACHE_FILE = 'rekalogika_mapper_mapping.php';