Adding empty constructors

src/ClientGenerator/RoslynTypeResolver.cs

@@ -238,7 +238,7 @@ public ClassDeclarationSyntax GenerateClient(TypeName clientName, ServiceSourceC
 				properties: properties,
 				methods: methods,
 				attributes: attributes,
-				emptyReflectionContructor: true,
+				emptyConstructorAccess: AccessType.Protected,
 				subTypes: resolvedOptions
 					.SelectMany(o => o.Type?.GeneratedSyntax ?? Array.Empty<MemberDeclarationSyntax>())
 					.ToList()
@@ -526,7 +526,8 @@ internal ClassDeclarationSyntax GenerateRecord(TypeName recordTypeName, RecordSo
 			return GenerateClass(
 				name: recordTypeName,
 				properties: properties,
-				subTypes: subItems
+				subTypes: subItems,
+				emptyConstructorAccess: AccessType.Public
 			);
 
 		}
@@ -1193,28 +1194,29 @@ private static ClassDeclarationSyntax GenerateClass(
 			List<MethodDeclarationSyntax>? methods = null,
 			List<TypeName>? implementTypes = null,
 			List<AttributeListSyntax>? attributes = null,
-			bool emptyReflectionContructor = false,
+			AccessType? emptyConstructorAccess = null,
 			List<MemberDeclarationSyntax>? subTypes = null)
 		{
 			if (properties.Any(p => p.Type == null))
 			{
 				// TODO
 				throw new NotImplementedException("Null, empty or reserved properties are not yet supported");
 			}
-			IEnumerable<PropertyDeclarationSyntax> properySyntxList = properties
+			List<ConstructorDeclarationSyntax> constructors = new();
+			IEnumerable<PropertyDeclarationSyntax> properySyntaxList = properties
 				.Select(GenerateProperty)
 				.Where(p => p != null)!;
-
-			List<ConstructorDeclarationSyntax> constructors = new()
+			if (properties.Any())
 			{
-				GenerateConstructor(name, AccessType.Public, properties)
-			};
-			if (emptyReflectionContructor)
+				// Only create constructor if there are properties
+				constructors.Add(GenerateConstructor(name, AccessType.Public, properties));
+			}
+			if (emptyConstructorAccess != null)
 			{
 				// Empty Constrcutor for reflection
 				constructors.Add(GenerateConstructor(
 					name: name,
-					access: AccessType.Protected,
+					access: emptyConstructorAccess.Value,
 					properties: new List<ClassProperty>()
 				));
 			}
@@ -1226,7 +1228,7 @@ private static ClassDeclarationSyntax GenerateClass(
 				))
 				.WithMembers(SyntaxFactory.List(
 					// Properties
-					properySyntxList.Cast<MemberDeclarationSyntax>()
+					properySyntaxList.Cast<MemberDeclarationSyntax>()
 					// Constructors
 					.Concat(constructors.Cast<MemberDeclarationSyntax>())
 					// Methods

src/ClientGenerator/SyntaxRewriters/NamespacePrefixRemover.cs

@@ -22,17 +22,6 @@ public NamespacePrefixRemover(string modelNamespace)
 			this.ModelNamespace = modelNamespace;
 		}
 
-		[return: NotNullIfNotNull("node")]
-		public override SyntaxNode? Visit(SyntaxNode? node)
-		{
-			string? a = node?.ToString();
-			if (a != null && a.StartsWith("EdjCase.ICP.Agent.Agents"))
-			{
-				int b = 2;
-			}
-			return base.Visit(node);
-		}
-
 		public override SyntaxNode? VisitIdentifierName(IdentifierNameSyntax node)
 		{
 			return this.VisitInternal(node, base.VisitIdentifierName);

test/Common.Tests/Generators/__snapshots__/Dex.snap

@@ -404,6 +404,10 @@ namespace Test.Models
 			this.To = to;
 			this.ToAmount = toAmount;
 		}
+
+		public Order()
+		{
+		}
 	}
 }
 
@@ -674,5 +678,9 @@ namespace Test.Models
 			this.Owner = owner;
 			this.Token = token;
 		}
+
+		public Balance()
+		{
+		}
 	}
 }