feat: refactor, rename to MaxRecursiveDepth

src/Riok.Mapperly.Abstractions/MapperAttribute.cs

@@ -101,5 +101,5 @@ public class MapperAttribute : Attribute
     /// <summary>
     /// Defines the maximum recursive depth that an IQueryable mapping will use.
     /// </summary>
-    public int RecursiveDepth { get; set; } = 8;
+    public int MaxRecursiveDepth { get; set; } = 8;
 }

src/Riok.Mapperly.Abstractions/MapperMaxRecursiveDepthAttribute.cs

@@ -0,0 +1,22 @@
+namespace Riok.Mapperly.Abstractions;
+
+/// <summary>
+/// Defines the maximum recursive depth that an IQueryable mapping will use.
+/// </summary>
+[AttributeUsage(AttributeTargets.Method)]
+public sealed class MapperMaxRecursiveDepthAttribute : Attribute
+{
+    /// <summary>
+    /// Defines the maximum recursive depth that an IQueryable mapping will use.
+    /// </summary>
+    /// <param name="maxRecursiveDepth">The maximum recursive depth used when mapping IQueryable members.</param>
+    public MapperMaxRecursiveDepthAttribute(int maxRecursiveDepth)
+    {
+        MaxRecursiveDepth = maxRecursiveDepth;
+    }
+
+    /// <summary>
+    /// The maximum recursive depth used when mapping IQueryable members.
+    /// </summary>
+    public int MaxRecursiveDepth { get; }
+}

src/Riok.Mapperly/Configuration/MapperConfiguration.cs

@@ -107,5 +107,5 @@ public record MapperConfiguration
     /// <summary>
     /// Defines the maximum recursive depth that an IQueryable mapping will use.
     /// </summary>
-    public int? RecursiveDepth { get; init; }
+    public int? MaxRecursiveDepth { get; init; }
 }

src/Riok.Mapperly/Configuration/MapperConfigurationMerger.cs

@@ -54,7 +54,8 @@ public static MapperAttribute Merge(MapperConfiguration mapperConfiguration, Map
         mapper.IncludedMembers =
             mapperConfiguration.IncludedMembers ?? defaultMapperConfiguration.IncludedMembers ?? mapper.IncludedMembers;
 
-        mapper.RecursiveDepth = mapperConfiguration.RecursiveDepth ?? defaultMapperConfiguration.RecursiveDepth ?? mapper.RecursiveDepth;
+        mapper.MaxRecursiveDepth =
+            mapperConfiguration.MaxRecursiveDepth ?? defaultMapperConfiguration.MaxRecursiveDepth ?? mapper.MaxRecursiveDepth;
 
         return mapper;
     }

src/Riok.Mapperly/Configuration/MapperConfigurationReader.cs

@@ -34,7 +34,7 @@ MapperConfiguration defaultMapperConfiguration
                 Array.Empty<PropertyMappingConfiguration>(),
                 Mapper.IgnoreObsoleteMembersStrategy,
                 Mapper.RequiredMappingStrategy,
-                Mapper.RecursiveDepth
+                Mapper.MaxRecursiveDepth
             ),
             Array.Empty<DerivedTypeMappingConfiguration>()
         );
@@ -80,17 +80,18 @@ private PropertiesMappingConfiguration BuildPropertiesConfig(IMethodSymbol metho
         var requiredMapping = _dataAccessor.Access<MapperRequiredMappingAttribute>(method).FirstOrDefault() is not { } methodWarnUnmapped
             ? _defaultConfiguration.Properties.RequiredMappingStrategy
             : methodWarnUnmapped.RequiredMappingStrategy;
-        var recursiveDepth = _dataAccessor.Access<MapperRecursiveDepthAttribute>(method).FirstOrDefault() is not { } methodRecursiveDepth
-            ? _defaultConfiguration.Properties.RecursiveDepth
-            : methodRecursiveDepth.RecursiveDepth;
+        var maxRecursiveDepth = _dataAccessor.Access<MapperMaxRecursiveDepthAttribute>(method).FirstOrDefault()
+            is not { } methodMaxRecursiveDepth
+            ? _defaultConfiguration.Properties.MaxRecursiveDepth
+            : methodMaxRecursiveDepth.MaxRecursiveDepth;
 
         return new PropertiesMappingConfiguration(
             ignoredSourceProperties,
             ignoredTargetProperties,
             explicitMappings,
             ignoreObsolete,
             requiredMapping,
-            recursiveDepth
+            maxRecursiveDepth
         );
     }
 

src/Riok.Mapperly/Configuration/PropertiesMappingConfiguration.cs

@@ -8,5 +8,5 @@ public record PropertiesMappingConfiguration(
     IReadOnlyCollection<PropertyMappingConfiguration> ExplicitMappings,
     IgnoreObsoleteMembersStrategy IgnoreObsoleteMembersStrategy,
     RequiredMappingStrategy RequiredMappingStrategy,
-    int RecursiveDepth
+    int MaxRecursiveDepth
 );

src/Riok.Mapperly/Descriptors/InlineExpressionMappingBuilderContext.cs

@@ -16,7 +16,7 @@ public class InlineExpressionMappingBuilderContext : MappingBuilderContext
 {
     private readonly MappingCollection _inlineExpressionMappings;
     private readonly MappingBuilderContext _parentContext;
-    private readonly ImmutableDictionary<(ISymbol, ISymbol), int> _parentTypes;
+    private readonly ImmutableDictionary<TypeMappingKey, int> _parentTypes;
 
     public InlineExpressionMappingBuilderContext(MappingBuilderContext ctx, ITypeSymbol sourceType, ITypeSymbol targetType)
         : this(ctx, (ctx.FindMapping(sourceType, targetType) as IUserMapping)?.Method, sourceType, targetType) { }
@@ -34,13 +34,13 @@ ITypeSymbol target
 
         _parentTypes = ctx is InlineExpressionMappingBuilderContext inlineCtx
             ? inlineCtx._parentTypes
-            : ImmutableDictionary.Create<(ISymbol, ISymbol), int>(new SymbolTupleEqualityComparer());
+            : ImmutableDictionary.Create<TypeMappingKey, int>();
     }
 
     private InlineExpressionMappingBuilderContext(
         InlineExpressionMappingBuilderContext ctx,
         IMethodSymbol? userSymbol,
-        ImmutableDictionary<(ISymbol, ISymbol), int> parentTypes,
+        ImmutableDictionary<TypeMappingKey, int> parentTypes,
         ITypeSymbol source,
         ITypeSymbol target,
         bool clearDerivedTypes
@@ -73,9 +73,9 @@ conversionType is not MappingConversionType.EnumToString and not MappingConversi
     public override INewInstanceMapping? FindMapping(ITypeSymbol sourceType, ITypeSymbol targetType)
     {
         // check for recursive loop returning null to prevent a loop or default when recursive limit is reached.
-        if (_parentTypes.TryGetValue((sourceType, targetType), out var count))
+        if (_parentTypes.TryGetValue(new(sourceType, targetType), out var count))
         {
-            return count >= Configuration.Properties.RecursiveDepth ? new DefaultMemberMapping(sourceType, targetType) : null;
+            return count >= Configuration.Properties.MaxRecursiveDepth ? new DefaultMemberMapping(sourceType, targetType) : null;
         }
 
         if (_inlineExpressionMappings.Find(sourceType, targetType) is { } mapping)
@@ -170,9 +170,8 @@ protected override MappingBuilderContext ContextForMapping(
         MappingBuildingOptions options
     )
     {
-        var newParentTypes = _parentTypes.TryGetValue((sourceType, targetType), out var count)
-            ? _parentTypes.SetItem((sourceType, targetType), count + 1)
-            : _parentTypes.SetItem((sourceType, targetType), 1);
+        var mappingRecursionCount = _parentTypes.GetValueOrDefault(new(sourceType, targetType));
+        var newParentTypes = _parentTypes.SetItem(new(sourceType, targetType), mappingRecursionCount + 1);
         return new InlineExpressionMappingBuilderContext(
             this,
             userSymbol,