Struct handling with readonly fields (wip)

src/MapTo/ClassMappingContext.cs

@@ -11,7 +11,7 @@ namespace MapTo
         internal ClassMappingContext(Compilation compilation, SourceGenerationOptions sourceGenerationOptions, TypeDeclarationSyntax typeSyntax)
             : base(compilation, sourceGenerationOptions, typeSyntax) { }
 
-        protected override ImmutableArray<MappedProperty> GetMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
+        protected override ImmutableArray<MappedProperty> GetSourceMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
         {
             var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
 
@@ -23,5 +23,18 @@ namespace MapTo
                 .Where(mappedProperty => mappedProperty is not null)
                 .ToImmutableArray()!;
         }
+
+        protected override ImmutableArray<MappedProperty> GetTypeMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
+        {
+            var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
+
+            return typeSymbol
+                .GetAllMembers()
+                .OfType<IPropertySymbol>()
+                .Where(p => !p.HasAttribute(IgnorePropertyAttributeTypeSymbol))
+                .Select(property => MapProperty(typeSymbol, sourceProperties, property))
+                .Where(mappedProperty => mappedProperty is not null)
+                .ToImmutableArray()!;
+        }
     }
 }

src/MapTo/Extensions/CommonExtensions.cs

@@ -0,0 +1,31 @@
+using MapTo.Sources;
+using System;
+using System.Collections.Generic;
+using System.Text;
+
+namespace MapTo.Extensions
+{
+    internal static class CommonExtensions
+    {
+        internal static SourceBuilder WriteComment(this SourceBuilder builder, string comment)
+        {
+            return builder.WriteLine($"// {comment}");
+        }
+
+        internal static SourceBuilder WriteMappedProperties(this SourceBuilder builder, System.Collections.Immutable.ImmutableArray<MappedProperty> mappedProperties)
+        {
+            foreach (var item in mappedProperties)
+            {
+                builder.WriteComment($"Name: {item.Name}");
+                builder.WriteComment($"Type: {item.Type}");
+                builder.WriteComment($"MappedSourcePropertyTypeName: {item.MappedSourcePropertyTypeName}");
+                builder.WriteComment($"IsEnumerable: {item.IsEnumerable}");
+                builder.WriteComment($"SourcePropertyName: {item.SourcePropertyName}");
+
+            }
+
+            return builder;
+        }
+
+    }
+}

src/MapTo/MappingContext.cs

@@ -9,6 +9,12 @@ using Microsoft.CodeAnalysis.CSharp.Syntax;
 
 namespace MapTo
 {
+    internal static class MappingContextExtensions
+    {
+        internal static ImmutableArray<MappedProperty> GetReadOnlyMappedProperties(this ImmutableArray<MappedProperty> mappedProperties) => mappedProperties.Where(p => p.isReadOnly).ToImmutableArray()!;
+        internal static ImmutableArray<MappedProperty> GetWritableMappedProperties(this ImmutableArray<MappedProperty> mappedProperties) => mappedProperties.Where(p => !p.isReadOnly).ToImmutableArray()!;
+    }
+
     internal abstract class MappingContext
     {
         private readonly List<SymbolDisplayPart> _ignoredNamespaces;
@@ -60,6 +66,7 @@ namespace MapTo
         {
             MappingContext context = typeSyntax switch
             {
+                StructDeclarationSyntax => new StructMappingContext(compilation, sourceGenerationOptions, typeSyntax),
                 ClassDeclarationSyntax => new ClassMappingContext(compilation, sourceGenerationOptions, typeSyntax),
                 RecordDeclarationSyntax => new RecordMappingContext(compilation, sourceGenerationOptions, typeSyntax),
                 _ => throw new ArgumentOutOfRangeException()
@@ -100,7 +107,8 @@ namespace MapTo
             return sourceProperties.SingleOrDefault(p => p.Name == propertyName);
         }
 
-        protected abstract ImmutableArray<MappedProperty> GetMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass);
+        protected abstract ImmutableArray<MappedProperty> GetSourceMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass);
+        protected abstract ImmutableArray<MappedProperty> GetTypeMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass);
 
         protected INamedTypeSymbol? GetSourceTypeSymbol(TypeDeclarationSyntax typeDeclarationSyntax, SemanticModel? semanticModel = null) =>
             GetSourceTypeSymbol(typeDeclarationSyntax.GetAttribute(MapFromAttributeSource.AttributeName), semanticModel);
@@ -136,6 +144,7 @@ namespace MapTo
                 return null;
             }
 
+           
             string? converterFullyQualifiedName = null;
             var converterParameters = ImmutableArray<string>.Empty;
             ITypeSymbol? mappedSourcePropertyType = null;
@@ -161,9 +170,39 @@ namespace MapTo
                 converterParameters.ToImmutableArray(),
                 sourceProperty.Name,
                 ToQualifiedDisplayName(mappedSourcePropertyType),
-                ToQualifiedDisplayName(enumerableTypeArgumentType));
+                ToQualifiedDisplayName(enumerableTypeArgumentType),
+               (property as IPropertySymbol).IsReadOnly);
+;
         }
+        protected virtual MappedProperty? MapPropertySimple(ISymbol sourceTypeSymbol, ISymbol property)
+        {
+            if (!property.TryGetTypeSymbol(out var propertyType))
+            {
+                return null;
+            }
 
+
+            string? converterFullyQualifiedName = null;
+            var converterParameters = ImmutableArray<string>.Empty;
+            ITypeSymbol? mappedSourcePropertyType = null;
+            ITypeSymbol? enumerableTypeArgumentType = null;
+
+
+            AddUsingIfRequired(propertyType);
+            AddUsingIfRequired(enumerableTypeArgumentType);
+            AddUsingIfRequired(mappedSourcePropertyType);
+
+            return new MappedProperty(
+                property.Name,
+                ToQualifiedDisplayName(propertyType) ?? propertyType.Name,
+                converterFullyQualifiedName,
+                converterParameters.ToImmutableArray(),
+                property.Name,
+                ToQualifiedDisplayName(mappedSourcePropertyType),
+                ToQualifiedDisplayName(enumerableTypeArgumentType),
+               (property as IPropertySymbol).IsReadOnly);
+            ;
+        }
         protected bool TryGetMapTypeConverter(ISymbol property, IPropertySymbol sourceProperty, out string? converterFullyQualifiedName,
             out ImmutableArray<string> converterParameters)
         {
@@ -260,7 +299,7 @@ namespace MapTo
             var isTypeInheritFromMappedBaseClass = IsTypeInheritFromMappedBaseClass(semanticModel);
             var shouldGenerateSecondaryConstructor = ShouldGenerateSecondaryConstructor(semanticModel, sourceTypeSymbol);
 
-            var mappedProperties = GetMappedProperties(typeSymbol, sourceTypeSymbol, isTypeInheritFromMappedBaseClass);
+            var mappedProperties = GetSourceMappedProperties(typeSymbol, sourceTypeSymbol, isTypeInheritFromMappedBaseClass);
             if (!mappedProperties.Any())
             {
                 AddDiagnostic(DiagnosticsFactory.NoMatchingPropertyFoundError(TypeSyntax.GetLocation(), typeSymbol, sourceTypeSymbol));
@@ -269,6 +308,8 @@ namespace MapTo
 
             AddUsingIfRequired(mappedProperties.Any(p => p.IsEnumerable), "System.Linq");
 
+            var allProperties = GetTypeMappedProperties(sourceTypeSymbol, typeSymbol , isTypeInheritFromMappedBaseClass);
+
             return new MappingModel(
                 SourceGenerationOptions,
                 TypeSyntax.GetNamespace(),
@@ -279,11 +320,14 @@ namespace MapTo
                 sourceTypeIdentifierName,
                 sourceTypeSymbol.ToDisplayString(),
                 mappedProperties,
+                allProperties,
                 isTypeInheritFromMappedBaseClass,
                 Usings,
                 shouldGenerateSecondaryConstructor);
         }
 
+       
+
         private INamedTypeSymbol? GetTypeConverterBaseInterface(ITypeSymbol converterTypeSymbol, ISymbol property, IPropertySymbol sourceProperty)
         {
             if (!property.TryGetTypeSymbol(out var propertyType))

src/MapTo/Models.cs

@@ -29,7 +29,8 @@ namespace MapTo
         ImmutableArray<string> TypeConverterParameters,
         string SourcePropertyName,
         string? MappedSourcePropertyTypeName,
-        string? EnumerableTypeArgument)
+        string? EnumerableTypeArgument,
+        bool isReadOnly)
     {
         public bool IsEnumerable => EnumerableTypeArgument is not null;
     }
@@ -43,7 +44,8 @@ namespace MapTo
         string SourceNamespace,
         string SourceTypeIdentifierName,
         string SourceTypeFullName,
-        ImmutableArray<MappedProperty> MappedProperties,
+        ImmutableArray<MappedProperty> SourceProperties,
+        ImmutableArray<MappedProperty> TypeProperties,
         bool HasMappedBaseClass,
         ImmutableArray<string> Usings,
         bool GenerateSecondaryConstructor

src/MapTo/RecordMappingContext.cs

@@ -11,7 +11,7 @@ namespace MapTo
         internal RecordMappingContext(Compilation compilation, SourceGenerationOptions sourceGenerationOptions, TypeDeclarationSyntax typeSyntax)
             : base(compilation, sourceGenerationOptions, typeSyntax) { }
 
-        protected override ImmutableArray<MappedProperty> GetMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
+        protected override ImmutableArray<MappedProperty> GetSourceMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
         {
             var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
             return typeSymbol.GetMembers()
@@ -24,5 +24,19 @@ namespace MapTo
                 .Where(mappedProperty => mappedProperty is not null)
                 .ToImmutableArray()!;
         }
+
+        protected override ImmutableArray<MappedProperty> GetTypeMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool isInheritFromMappedBaseClass)
+        {
+            var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
+            return typeSymbol.GetMembers()
+                .OfType<IMethodSymbol>()
+                .OrderByDescending(s => s.Parameters.Length)
+                .First(s => s.Name == ".ctor")
+                .Parameters
+                .Where(p => !p.HasAttribute(IgnorePropertyAttributeTypeSymbol))
+                .Select(property => MapProperty(typeSymbol, sourceProperties, property))
+                .Where(mappedProperty => mappedProperty is not null)
+                .ToImmutableArray()!;
+        }
     }
 }

src/MapTo/Sources/MapClassSource.cs

@@ -21,28 +21,19 @@ namespace MapTo.Sources
                 .WriteLine($"partial class {model.TypeIdentifierName}")
                 .WriteOpeningBracket();
 
-                // Class body
-                if (model.GenerateSecondaryConstructor)
-                {
-                    builder
-                        .GenerateSecondaryConstructor(model)
-                        .WriteLine();
-                }
+            builder
+                .GeneratePublicConstructor(model)
+                .WriteLine();
 
-                builder
-                    .GeneratePrivateConstructor(model)
-                    .WriteLine()
-                    .GenerateFactoryMethod(model)
-                    .GenerateUpdateMethod(model)
+            if(!AllPropertiesReadOnly(model))
+            {
+                builder.GenerateUpdateMethod(model);
+            }
 
-                    // End class declaration
+            builder
                     .WriteClosingBracket()
                     .WriteLine()
 
-                    // Extension class declaration
-                    .GenerateSourceTypeExtensionClass(model)
-
-                    // End namespace declaration
                     .WriteClosingBracket();
             
             return new(builder.ToString(), $"{model.Namespace}.{model.TypeIdentifierName}.g.cs");
@@ -66,34 +57,37 @@ namespace MapTo.Sources
                 .WriteLine($"{model.Options.ConstructorAccessModifier.ToLowercaseString()} {model.TypeIdentifierName}({model.SourceType} {sourceClassParameterName})")
                 .WriteLine($"    : this(new {MappingContextSource.ClassName}(), {sourceClassParameterName}) {{ }}");
         }
-        
-        private static SourceBuilder GeneratePrivateConstructor(this SourceBuilder builder, MappingModel model)
+
+
+        private static SourceBuilder GeneratePublicConstructor(this SourceBuilder builder, MappingModel model)
         {
             var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
             const string mappingContextParameterName = "context";
 
-            var baseConstructor = model.HasMappedBaseClass ? $" : base({mappingContextParameterName}, {sourceClassParameterName})" : string.Empty;
+            var baseConstructor = /*model.HasMappedBaseClass ? $" : base({mappingContextParameterName}, {sourceClassParameterName})" :*/ string.Empty;
 
             builder
-                .WriteLine($"private protected {model.TypeIdentifierName}({MappingContextSource.ClassName} {mappingContextParameterName}, {model.SourceType} {sourceClassParameterName}){baseConstructor}")
+                .WriteLine($"public {model.TypeIdentifierName}({model.SourceType} {sourceClassParameterName}){baseConstructor}")
                 .WriteOpeningBracket()
-                .WriteLine($"if ({mappingContextParameterName} == null) throw new ArgumentNullException(nameof({mappingContextParameterName}));")
-                .WriteLine($"if ({sourceClassParameterName} == null) throw new ArgumentNullException(nameof({sourceClassParameterName}));")
-                .WriteLine()
-                .WriteLine($"{mappingContextParameterName}.{MappingContextSource.RegisterMethodName}({sourceClassParameterName}, this);")
-                .WriteLine().
+                //.WriteLine($"if ({mappingContextParameterName} == null) throw new ArgumentNullException(nameof({mappingContextParameterName}));")
+                //.WriteLine($"if ({sourceClassParameterName} == null) throw new ArgumentNullException(nameof({sourceClassParameterName}));")
+                //.WriteLine()
+                //.WriteLine($"{mappingContextParameterName}.{MappingContextSource.RegisterMethodName}({sourceClassParameterName}, this);")
 
-            WriteProperties( model, sourceClassParameterName, mappingContextParameterName);
+                .WriteProperties( model.SourceProperties, sourceClassParameterName, mappingContextParameterName, false);
 
             // End constructor declaration
             return builder.WriteClosingBracket();
         }
 
-        private static SourceBuilder WriteProperties(this SourceBuilder builder, MappingModel model,
-            string? sourceClassParameterName, string mappingContextParameterName)
+        private static SourceBuilder WriteProperties(this SourceBuilder builder, System.Collections.Immutable.ImmutableArray<MappedProperty> properties,
+            string? sourceClassParameterName, string mappingContextParameterName, bool fromUpdate)
         {
-            foreach (var property in model.MappedProperties)
+
+            foreach (var property in properties)
             {
+                if (property.isReadOnly && fromUpdate) continue;
+
                 if (property.TypeConverter is null)
                 {
                     if (property.IsEnumerable)
@@ -105,7 +99,7 @@ namespace MapTo.Sources
                     {
                         builder.WriteLine(property.MappedSourcePropertyTypeName is null
                             ? $"{property.Name} = {sourceClassParameterName}.{property.SourcePropertyName};"
-                            : $"{property.Name} = {mappingContextParameterName}.{MappingContextSource.MapMethodName}<{property.MappedSourcePropertyTypeName}, {property.Type}>({sourceClassParameterName}.{property.SourcePropertyName});");
+                            : "");
                     }
                 }
                 else
@@ -123,28 +117,25 @@ namespace MapTo.Sources
 
         }
 
-        private static SourceBuilder GenerateFactoryMethod(this SourceBuilder builder, MappingModel model)
+        private static bool AllPropertiesReadOnly(MappingModel model)
         {
-            var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
+            foreach (var property in model.SourceProperties)
+            {
+                if (!property.isReadOnly) return false;
+            }
+            return true ;
 
-            return builder
-                .GenerateConvertorMethodsXmlDocs(model, sourceClassParameterName)
-                .WriteLineIf(model.Options.SupportNullableStaticAnalysis, $"[return: NotNullIfNotNull(\"{sourceClassParameterName}\")]")
-                .WriteLine($"{model.Options.GeneratedMethodsAccessModifier.ToLowercaseString()} static {model.TypeIdentifierName}{model.Options.NullableReferenceSyntax} From({model.SourceType}{model.Options.NullableReferenceSyntax} {sourceClassParameterName})")
-                .WriteOpeningBracket()
-                .WriteLine($"return {sourceClassParameterName} == null ? null : {MappingContextSource.ClassName}.{MappingContextSource.FactoryMethodName}<{model.SourceType}, {model.TypeIdentifierName}>({sourceClassParameterName});")
-                .WriteClosingBracket();
         }
-        
+
         private static SourceBuilder GenerateUpdateMethod(this SourceBuilder builder, MappingModel model)
         {
             var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
 
              builder
                 .GenerateUpdaterMethodsXmlDocs(model, sourceClassParameterName)
-                .WriteLine($"public void {model.Options.NullableReferenceSyntax}Update({model.SourceType}{model.Options.NullableReferenceSyntax} {sourceClassParameterName})")
+                .WriteLine($"public void Update({model.SourceType}{model.Options.NullableReferenceSyntax} {sourceClassParameterName})")
                 .WriteOpeningBracket()
-                .WriteProperties( model, sourceClassParameterName,"context" )
+                .WriteProperties( model.SourceProperties.GetWritableMappedProperties(), sourceClassParameterName,"context", true )
                 .WriteClosingBracket();
 
              return builder;
@@ -181,15 +172,6 @@ namespace MapTo.Sources
                 .WriteLine($"/// <param name=\"{sourceClassParameterName}\">The instance of <see cref=\"{model.SourceType}\"/> to use as source.</param>");
         }
 
-        private static SourceBuilder GenerateSourceTypeExtensionClass(this SourceBuilder builder, MappingModel model)
-        {
-            return builder
-                .WriteLine($"{model.Options.GeneratedMethodsAccessModifier.ToLowercaseString()} static partial class {model.SourceTypeIdentifierName}To{model.TypeIdentifierName}Extensions")
-                .WriteOpeningBracket()
-                .GenerateSourceTypeExtensionMethod(model)
-                .WriteClosingBracket();
-        }
-        
         private static SourceBuilder GenerateSourceTypeExtensionMethod(this SourceBuilder builder, MappingModel model)
         {
             var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();

src/MapTo/Sources/MapFromAttributeSource.cs

@@ -26,7 +26,7 @@ namespace MapTo.Sources
             }
 
             builder
-                .WriteLine("[AttributeUsage(AttributeTargets.Class, Inherited = false, AllowMultiple = false)]")
+                .WriteLine("[AttributeUsage(AttributeTargets.Class | AttributeTargets.Struct)]")
                 .WriteLine($"public sealed class {AttributeName}Attribute : Attribute")
                 .WriteOpeningBracket();
 

src/MapTo/Sources/MapRecordSource.cs

@@ -96,9 +96,9 @@ namespace MapTo.Sources
         private static SourceBuilder WriteProperties(this SourceBuilder builder, MappingModel model, string sourceClassParameterName,
             string mappingContextParameterName)
         {
-            for (var i = 0; i < model.MappedProperties.Length; i++)
+            for (var i = 0; i < model.SourceProperties.Length; i++)
             {
-                var property = model.MappedProperties[i];
+                var property = model.SourceProperties[i];
                 if (property.TypeConverter is null)
                 {
                     if (property.IsEnumerable)
@@ -123,7 +123,7 @@ namespace MapTo.Sources
                         $"{property.Name}: new {property.TypeConverter}().Convert({sourceClassParameterName}.{property.SourcePropertyName}, {parameters})");
                 }
 
-                if (i < model.MappedProperties.Length - 1)
+                if (i < model.SourceProperties.Length - 1)
                 {
                     builder.Write(", ");
                 }

src/MapTo/Sources/MapStructSource.cs

@@ -1,5 +1,6 @@
 using MapTo.Extensions;
 using static MapTo.Sources.Constants;
+using System.Collections.Generic;
 
 namespace MapTo.Sources
 {
@@ -18,19 +19,19 @@ namespace MapTo.Sources
                 .WriteOpeningBracket()
 
                 // Class declaration
-                .WriteLine($"partial class {model.TypeIdentifierName}")
-                .WriteOpeningBracket();
+                .WriteLine($"partial struct {model.TypeIdentifierName}")
+                .WriteOpeningBracket()
+                .WriteLine()
 
-                // Class body
-                if (model.GenerateSecondaryConstructor)
-                {
-                    builder
-                        .GenerateSecondaryConstructor(model)
-                        .WriteLine();
-                }
+            // Class body
+                .GeneratePublicConstructor(model);
 
-                builder
-                    .GeneratePrivateConstructor(model)
+            if (!AllPropertiesAreReadOnly(model))
+            {
+                builder.GenerateUpdateMethod(model);
+            }
+
+            builder
                     .WriteLine()
                     // End class declaration
                     .WriteClosingBracket()
@@ -38,9 +39,19 @@ namespace MapTo.Sources
 
                     // End namespace declaration
                     .WriteClosingBracket();
-            
+
             return new(builder.ToString(), $"{model.Namespace}.{model.TypeIdentifierName}.g.cs");
         }
+        private static bool AllPropertiesAreReadOnly(MappingModel model)
+        {
+            foreach (var property in model.SourceProperties)
+            {
+                if (!property.isReadOnly) return false;
+            }
+            return true;
+
+        }
+
 
         private static SourceBuilder GenerateSecondaryConstructor(this SourceBuilder builder, MappingModel model)
         {
@@ -50,7 +61,7 @@ namespace MapTo.Sources
             {
                 builder
                     .WriteLine("/// <summary>")
-                    .WriteLine($"/// Initializes a new instance of the <see cref=\"{model.TypeIdentifierName}\"/> class")
+                    .WriteLine($"/// Initializes a new instance of the <see cref=\"{model.TypeIdentifierName}\"/> struct")
                     .WriteLine($"/// using the property values from the specified <paramref name=\"{sourceClassParameterName}\"/>.")
                     .WriteLine("/// </summary>")
                     .WriteLine($"/// <exception cref=\"ArgumentNullException\">{sourceClassParameterName} is null</exception>");
@@ -60,32 +71,48 @@ namespace MapTo.Sources
                 .WriteLine($"{model.Options.ConstructorAccessModifier.ToLowercaseString()} {model.TypeIdentifierName}({model.SourceType} {sourceClassParameterName})")
                 .WriteLine($"    : this(new {MappingContextSource.ClassName}(), {sourceClassParameterName}) {{ }}");
         }
-        
-        private static SourceBuilder GeneratePrivateConstructor(this SourceBuilder builder, MappingModel model)
+
+
+        private static SourceBuilder GeneratePublicConstructor(this SourceBuilder builder, MappingModel model)
         {
             var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
             const string mappingContextParameterName = "context";
 
-            var baseConstructor = model.HasMappedBaseClass ? $" : base({mappingContextParameterName}, {sourceClassParameterName})" : string.Empty;
+            var baseConstructor = /*model.HasMappedBaseClass ? $" : base({mappingContextParameterName}, {sourceClassParameterName})" :*/ string.Empty;
+
+            var readOnlyProperties = model.TypeProperties.GetReadOnlyMappedProperties();
+
+            var readOnlyFields = "";
+
+            for (int i = 0; i < readOnlyProperties.Length; i++)
+            {
+                var property = readOnlyProperties[i];
+                readOnlyFields += $"{property.Type} {property.SourcePropertyName.ToCamelCase()}";
+                if (i != readOnlyProperties.Length - 1) readOnlyFields += " ,";
+            }
+
 
             builder
-                .WriteLine($"private protected {model.TypeIdentifierName}({MappingContextSource.ClassName} {mappingContextParameterName}, {model.SourceType} {sourceClassParameterName}){baseConstructor}")
+                .WriteLine($"public {model.TypeIdentifierName}({model.SourceType} {sourceClassParameterName}{(string.IsNullOrEmpty(readOnlyFields) ? "" : $", {readOnlyFields}")}){baseConstructor}")
                 .WriteOpeningBracket()
-                .WriteLine()
-                .WriteLine($"{mappingContextParameterName}.{MappingContextSource.RegisterMethodName}({sourceClassParameterName}, this);")
-                .WriteLine().
-
-            WriteProperties( model, sourceClassParameterName, mappingContextParameterName);
+                .TryWriteProperties(model.SourceProperties, readOnlyProperties, sourceClassParameterName, mappingContextParameterName, false);
 
             // End constructor declaration
             return builder.WriteClosingBracket();
         }
 
-        private static SourceBuilder WriteProperties(this SourceBuilder builder, MappingModel model,
-            string? sourceClassParameterName, string mappingContextParameterName)
+        private static SourceBuilder TryWriteProperties(this SourceBuilder builder, System.Collections.Immutable.ImmutableArray<MappedProperty> properties, System.Collections.Immutable.ImmutableArray<MappedProperty>? otherProperties,
+            string? sourceClassParameterName, string mappingContextParameterName, bool fromUpdate)
         {
-            foreach (var property in model.MappedProperties)
+            if (fromUpdate)
             {
+                properties = properties.GetWritableMappedProperties();
+            }
+
+            foreach (var property in properties)
+            {
+                if (property.isReadOnly && fromUpdate) continue;
+
                 if (property.TypeConverter is null)
                 {
                     if (property.IsEnumerable)
@@ -97,7 +124,7 @@ namespace MapTo.Sources
                     {
                         builder.WriteLine(property.MappedSourcePropertyTypeName is null
                             ? $"{property.Name} = {sourceClassParameterName}.{property.SourcePropertyName};"
-                            : $"{property.Name} = {mappingContextParameterName}.{MappingContextSource.MapMethodName}<{property.MappedSourcePropertyTypeName}, {property.Type}>({sourceClassParameterName}.{property.SourcePropertyName});");
+                            : "");
                     }
                 }
                 else
@@ -111,27 +138,39 @@ namespace MapTo.Sources
                 }
 
             }
+
+
+            if (otherProperties == null) return builder;
+
+            foreach (var property in otherProperties)
+            {
+    
+                builder.WriteLine(property.MappedSourcePropertyTypeName is null
+                ? $"{property.Name} = {property.SourcePropertyName.ToCamelCase()};"
+                : "");
+
+            }
+
             return builder;
 
         }
 
-        
-        private static SourceBuilder GenerateConvertorMethodsXmlDocs(this SourceBuilder builder, MappingModel model, string sourceClassParameterName)
-        {
-            if (!model.Options.GenerateXmlDocument)
-            {
-                return builder;
-            }
 
-            return builder
-                .WriteLine("/// <summary>")
-                .WriteLine($"/// Creates a new instance of <see cref=\"{model.TypeIdentifierName}\"/> and sets its participating properties")
-                .WriteLine($"/// using the property values from <paramref name=\"{sourceClassParameterName}\"/>.")
-                .WriteLine("/// </summary>")
-                .WriteLine($"/// <param name=\"{sourceClassParameterName}\">The instance of <see cref=\"{model.SourceType}\"/> to use as source.</param>")
-                .WriteLine($"/// <returns>A new instance of <see cred=\"{model.TypeIdentifierName}\"/> -or- <c>null</c> if <paramref name=\"{sourceClassParameterName}\"/> is <c>null</c>.</returns>");
+
+        private static SourceBuilder GenerateUpdateMethod(this SourceBuilder builder, MappingModel model)
+        {
+            var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
+
+            builder
+               .GenerateUpdaterMethodsXmlDocs(model, sourceClassParameterName)
+               .WriteLine($"public void Update({model.SourceType} {sourceClassParameterName})")
+               .WriteOpeningBracket()
+               .TryWriteProperties(model.SourceProperties, null, sourceClassParameterName, "context", true)
+               .WriteClosingBracket();
+
+            return builder;
         }
-        
+
         private static SourceBuilder GenerateUpdaterMethodsXmlDocs(this SourceBuilder builder, MappingModel model, string sourceClassParameterName)
         {
             if (!model.Options.GenerateXmlDocument)
@@ -147,26 +186,5 @@ namespace MapTo.Sources
                 .WriteLine($"/// <param name=\"{sourceClassParameterName}\">The instance of <see cref=\"{model.SourceType}\"/> to use as source.</param>");
         }
 
-        private static SourceBuilder GenerateSourceTypeExtensionClass(this SourceBuilder builder, MappingModel model)
-        {
-            return builder
-                .WriteLine($"{model.Options.GeneratedMethodsAccessModifier.ToLowercaseString()} static partial class {model.SourceTypeIdentifierName}To{model.TypeIdentifierName}Extensions")
-                .WriteOpeningBracket()
-                .GenerateSourceTypeExtensionMethod(model)
-                .WriteClosingBracket();
-        }
-        
-        private static SourceBuilder GenerateSourceTypeExtensionMethod(this SourceBuilder builder, MappingModel model)
-        {
-            var sourceClassParameterName = model.SourceTypeIdentifierName.ToCamelCase();
-
-            return builder
-                .GenerateConvertorMethodsXmlDocs(model, sourceClassParameterName)
-                .WriteLineIf(model.Options.SupportNullableStaticAnalysis, $"[return: NotNullIfNotNull(\"{sourceClassParameterName}\")]")
-                .WriteLine($"{model.Options.GeneratedMethodsAccessModifier.ToLowercaseString()} static {model.TypeIdentifierName}{model.Options.NullableReferenceSyntax} To{model.TypeIdentifierName}(this {model.SourceType}{model.Options.NullableReferenceSyntax} {sourceClassParameterName})")
-                .WriteOpeningBracket()
-                .WriteLine($"return {sourceClassParameterName} == null ? null : new {model.TypeIdentifierName}({sourceClassParameterName});")
-                .WriteClosingBracket();
-        }
     }
 }

src/MapTo/Sources/ReadOnlyPropertyAttributeSource.cs

@@ -0,0 +1,36 @@
+using static MapTo.Sources.Constants;
+
+namespace MapTo.Sources
+{
+    internal static class ReadOnlyPropertyAttributeSource
+    {
+        internal const string AttributeName = "ReadOnlyProperty";
+        internal const string AttributeClassName = AttributeName + "Attribute";
+        internal const string FullyQualifiedName = RootNamespace + "." + AttributeClassName;
+
+        internal static SourceCode Generate(SourceGenerationOptions options)
+        {
+            var builder = new SourceBuilder()
+                .WriteLine(GeneratedFilesHeader)
+                .WriteLine("using System;")
+                .WriteLine()
+                .WriteLine($"namespace {RootNamespace}")
+                .WriteOpeningBracket();
+
+            if (options.GenerateXmlDocument)
+            {
+                builder
+                    .WriteLine("/// <summary>")
+                    .WriteLine("/// Specifies that the annotated property should be excluded.")
+                    .WriteLine("/// </summary>");
+            }
+
+            builder
+                .WriteLine("[AttributeUsage(AttributeTargets.Property, Inherited = false, AllowMultiple = false)]")
+                .WriteLine($"public sealed class {AttributeClassName} : Attribute {{ }}")
+                .WriteClosingBracket();
+
+            return new(builder.ToString(), $"{AttributeClassName}.g.cs");
+        }
+    }
+}

src/MapTo/StructMappingContext.cs

@@ -0,0 +1,40 @@
+using System.Collections.Immutable;
+using System.Linq;
+using MapTo.Extensions;
+using Microsoft.CodeAnalysis;
+using Microsoft.CodeAnalysis.CSharp.Syntax;
+
+namespace MapTo
+{
+    internal class StructMappingContext : MappingContext
+    {
+        internal StructMappingContext(Compilation compilation, SourceGenerationOptions sourceGenerationOptions, TypeDeclarationSyntax typeSyntax)
+            : base(compilation, sourceGenerationOptions, typeSyntax) { }
+
+        protected override ImmutableArray<MappedProperty> GetSourceMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool hasInheritedClass)
+        {
+            var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
+
+            return typeSymbol
+                .GetAllMembers()
+                .OfType<IPropertySymbol>()
+                .Where(p => !p.HasAttribute(IgnorePropertyAttributeTypeSymbol))
+                .Select(property => MapProperty(sourceTypeSymbol, sourceProperties, property))
+                .Where(mappedProperty => mappedProperty is not null)
+                .ToImmutableArray()!;
+        }
+        protected override ImmutableArray<MappedProperty> GetTypeMappedProperties(ITypeSymbol typeSymbol, ITypeSymbol sourceTypeSymbol, bool hasInheritedClass)
+        {
+            var sourceProperties = sourceTypeSymbol.GetAllMembers().OfType<IPropertySymbol>().ToArray();
+
+            return sourceTypeSymbol
+                .GetAllMembers()
+                .OfType<IPropertySymbol>()
+                .Where(p => !p.HasAttribute(IgnorePropertyAttributeTypeSymbol))
+                .Select(property => MapPropertySimple(typeSymbol, property))
+                .Where(mappedProperty => mappedProperty is not null)
+                .ToImmutableArray()!;
+        }
+
+    }
+}

test/TestConsoleApp/Data/Models/Employee.cs

@@ -10,6 +10,5 @@ namespace TestConsoleApp.Data.Models
 
         public string EmployeeCode { get; set; }
 
-        public Manager Manager { get; set; }
     }
 }

test/TestConsoleApp/Data/Models/Manager.cs

@@ -1,13 +0,0 @@
-using System;
-using System.Collections.Generic;
-using System.Text;
-
-namespace TestConsoleApp.Data.Models
-{
-    public class Manager: Employee
-    {
-        public int Level { get; set; }
-
-        public IEnumerable<Employee> Employees { get; set; } = Array.Empty<Employee>();
-    }
-}

test/TestConsoleApp/Data/Models/MyStruct.cs

@@ -0,0 +1,25 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using TestConsoleApp.ViewModels;
+using MapTo;
+
+namespace TestConsoleApp.Data.Models
+{
+    [MapFrom(typeof(MyStructViewModel))]
+    public partial struct MyStruct
+    {
+        public int SomeInt { get; set; }
+
+        public string ReadOnlyString { get; }
+
+        public MyStruct(int someInt, string readOnlyString)
+        {
+            SomeInt = someInt;
+            ReadOnlyString = readOnlyString;
+        }
+
+    }
+}

test/TestConsoleApp/Data/Models/Profile.cs

@@ -1,11 +0,0 @@
-namespace TestConsoleApp.Data.Models
-{
-    public class Profile
-    {
-        public string FirstName { get; set; }
-
-        public string LastName { get; set; }
-
-        public string FullName => $"{FirstName} {LastName}";
-    }
-}

test/TestConsoleApp/Data/Models/User.cs

@@ -1,13 +1,15 @@
 using System;
+using TestConsoleApp.ViewModels;
+using MapTo;
 
 namespace TestConsoleApp.Data.Models
 {
-    public class User
+    [MapFrom(typeof(UserViewModel))]
+    public partial class User
     {
         public int Id { get; set; }
 
         public DateTimeOffset RegisteredAt { get; set; }
 
-        public Profile Profile { get; set; }
     }
 }

test/TestConsoleApp/Program.cs

@@ -2,7 +2,6 @@
 using MapTo;
 using TestConsoleApp.Data.Models;
 using TestConsoleApp.ViewModels;
-using TestConsoleApp.ViewModels2;
 
 namespace TestConsoleApp
 {
@@ -11,7 +10,6 @@ namespace TestConsoleApp
         private static void Main(string[] args)
         {
             //UserTest();
-            CyclicReferenceTest();
              
             // EmployeeManagerTest();
             Console.WriteLine("done");
@@ -19,74 +17,23 @@ namespace TestConsoleApp
 
         private static void EmployeeManagerTest()
         {
-            var manager1 = new Manager
-            {
-                Id = 1,
-                EmployeeCode = "M001",
-                Level = 100
-            };
-
-            var manager2 = new Manager
-            {
-                Id = 2,
-                EmployeeCode = "M002",
-                Level = 100,
-                Manager = manager1
-            };
-
+            
             var employee1 = new Employee
             {
                 Id = 101,
                 EmployeeCode = "E101",
-                Manager = manager1
             };
 
             var employee2 = new Employee
             {
                 Id = 102,
                 EmployeeCode = "E102",
-                Manager = manager2
             };
 
-            manager1.Employees = new[] { employee1, manager2 };
-            manager2.Employees = new[] { employee2 };
 
-            manager1.ToManagerViewModel();
-            employee1.ToEmployeeViewModel();
         }
 
-        private static ManagerViewModel CyclicReferenceTest()
-        {
-            var manager1 = new Manager
-            {
-                Id = 1,
-                EmployeeCode = "M001",
-                Level = 100
-            };
 
-            manager1.Manager = manager1;
-            return manager1.ToManagerViewModel();
-        }
-
-        private static void UserTest()
-        {
-            var user = new User
-            {
-                Id = 1234,
-                RegisteredAt = DateTimeOffset.Now,
-                Profile = new Profile
-                {
-                    FirstName = "John",
-                    LastName = "Doe"
-                }
-            };
-
-            var vm = user.ToUserViewModel();
-
-            Console.WriteLine("Key: {0}", vm.Key);
-            Console.WriteLine("RegisteredAt: {0}", vm.RegisteredAt);
-            Console.WriteLine("FirstName: {0}", vm.Profile.FirstName);
-            Console.WriteLine("LastName: {0}", vm.Profile.LastName);
-        }
+ 
     }
 }

test/TestConsoleApp/ViewModels/EmployeeViewModel.cs

@@ -1,16 +1,13 @@
 using MapTo;
 using TestConsoleApp.Data.Models;
-using TestConsoleApp.ViewModels2;
 
 namespace TestConsoleApp.ViewModels
 {
     [MapFrom(typeof(Employee))]
     public partial class EmployeeViewModel
     {
-        public int Id { get; set; }
+        public int Id { get; }
 
-        public string EmployeeCode { get; set; }
 
-        public ManagerViewModel Manager { get; set; }
     }
 }

test/TestConsoleApp/ViewModels/ManagerViewModel.cs

@@ -1,16 +0,0 @@
-using System;
-using System.Collections.Generic;
-using MapTo;
-using TestConsoleApp.Data.Models;
-using TestConsoleApp.ViewModels;
-
-namespace TestConsoleApp.ViewModels2
-{
-    [MapFrom(typeof(Manager))]
-    public partial class ManagerViewModel : EmployeeViewModel
-    {
-        public int Level { get; set; }
-
-        public IEnumerable<EmployeeViewModel> Employees { get; set; } = Array.Empty<EmployeeViewModel>();
-    }
-}

test/TestConsoleApp/ViewModels/MyStructViewModel.cs

@@ -0,0 +1,22 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using TestConsoleApp.Data.Models;
+using MapTo;
+
+namespace TestConsoleApp.ViewModels
+{
+    [MapFrom(typeof(MyStruct))]
+
+    public partial struct MyStructViewModel
+    {
+        public int SomeInt { get; set; }
+
+        public MyStructViewModel(int someInt)
+        {
+            SomeInt = someInt;
+        }
+    }
+}

test/TestConsoleApp/ViewModels/ProfileViewModel.cs

@@ -1,13 +0,0 @@
-using MapTo;
-using TestConsoleApp.Data.Models;
-
-namespace TestConsoleApp.ViewModels
-{
-    [MapFrom(typeof(Profile))]
-    public partial class ProfileViewModel
-    {
-        public string FirstName { get; }
-
-        public string LastName { get; }
-    }
-}

test/TestConsoleApp/ViewModels/UserViewModel.cs

@@ -7,18 +7,8 @@ namespace TestConsoleApp.ViewModels
     [MapFrom(typeof(User))]
     public partial class UserViewModel
     {
-        [MapProperty(SourcePropertyName = nameof(User.Id))]
-        [MapTypeConverter(typeof(IdConverter))]
-        public string Key { get; }
+        public int Id { get; set; }
 
-        public DateTimeOffset RegisteredAt { get; set; }
-        
-        // [IgnoreProperty]
-        public ProfileViewModel Profile { get; set; }
-
-        private class IdConverter : ITypeConverter<int, string>
-        {
-            public string Convert(int source, object[]? converterParameters) => $"{source:X}";
-        }
+        public DateTimeOffset RegisteredAt { get;  }
     }
 }