add option '--generate-records' to improve generating of record types

Author: smdn

src/Smdn.Reflection.ReverseGenerating.ListApi/Smdn.Reflection.ReverseGenerating.ListApi.csproj

@@ -38,7 +38,7 @@ SPDX-License-Identifier: MIT
     <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="6.0.0" Condition="$(TargetFramework.StartsWith('net6.')) Or $(TargetFramework.StartsWith('netcoreapp'))" />
     <PackageReference Include="Microsoft.Extensions.Logging.Console" Version="8.0.0" Condition="$(TargetFramework.StartsWith('net8.'))" />
 
-    <ProjectOrPackageReference ReferencePackageVersion="[1.3.0,2.0.0)" Include="..\Smdn.Reflection.ReverseGenerating.ListApi.Core\Smdn.Reflection.ReverseGenerating.ListApi.Core.csproj" />
+    <ProjectOrPackageReference ReferencePackageVersion="[1.4.0,2.0.0)" Include="..\Smdn.Reflection.ReverseGenerating.ListApi.Core\Smdn.Reflection.ReverseGenerating.ListApi.Core.csproj" />
   </ItemGroup>
 
   <ItemGroup Condition=" '$(EnableFeature_BuildProjectFile)' == 'True' ">

src/Smdn.Reflection.ReverseGenerating.ListApi/Smdn.Reflection.ReverseGenerating.ListApi/RootCommandImplementation.cs

@@ -104,6 +104,11 @@ public sealed class RootCommandImplementation : ICommandHandler {
     description: "Generates declarations with nullable annotations.",
     getDefaultValue: static () => true
   );
+  private static readonly Option<bool> OptionGenerateRecords = new(
+    aliases: new[] { "--generate-records" },
+    description: "Generates record type declarations and hides compiler-generated mebers.",
+    getDefaultValue: static () => true
+  );
   // cSpell:enable
 
   private readonly Microsoft.Extensions.Logging.ILogger? logger;
@@ -130,6 +135,7 @@ internal Command CreateCommand()
       OptionGenerateMethodBody,
       OptionGenerateStaticMembersFirst,
       OptionGenerateNullableAnnotations,
+      OptionGenerateRecords,
     };
 
     rootCommand.Handler = this;
@@ -161,6 +167,10 @@ private static ApiListWriterOptions GetApiListWriterOptions(ParseResult parseRes
     options.Writer.OrderStaticMembersFirst          = parseResult.GetValueForOption(OptionGenerateStaticMembersFirst);
     options.Writer.WriteNullableAnnotationDirective = parseResult.GetValueForOption(OptionGenerateNullableAnnotations);
 
+    options.TypeDeclaration.EnableRecordTypes =
+    options.TypeDeclaration.OmitRecordImplicitInterface =
+    options.Writer.OmitCompilerGeneratedRecordEqualityMethods = parseResult.GetValueForOption(OptionGenerateRecords);
+
     options.AttributeDeclaration.TypeFilter     = AttributeFilter.Default;
 
     options.ValueDeclaration.UseDefaultLiteral  = true;

tests/Smdn.Reflection.ReverseGenerating.ListApi/Smdn.Reflection.ReverseGenerating.ListApi/RootCommandImplementation.GetApiListWriterOptions.cs

@@ -1,8 +1,8 @@
 // SPDX-FileCopyrightText: 2021 smdn <smdn@smdn.jp>
 // SPDX-License-Identifier: MIT
 using System;
+
 using NUnit.Framework;
-using Smdn.Reflection.ReverseGenerating;
 
 namespace Smdn.Reflection.ReverseGenerating.ListApi;
 
@@ -77,4 +77,19 @@ public void GetApiListWriterOptions_GenerateNullableAnnotations(string args, boo
 
     Assert.That(options.Writer.WriteNullableAnnotationDirective, Is.EqualTo(expected), $"args='{args}'");
   }
+
+  // cSpell:disable
+  [TestCase("--generate-records", true)]
+  [TestCase("--generate-records=true", true)]
+  [TestCase("--generate-records=false", false)]
+  [TestCase("", true)]
+  // cSpell:enable
+  public void GetApiListWriterOptions_GenerateRecords(string args, bool expected)
+  {
+    var options = GetApiListWriterOptions(args);
+
+    Assert.That(options.TypeDeclaration.EnableRecordTypes, Is.EqualTo(expected), $"args='{args}'");
+    Assert.That(options.TypeDeclaration.OmitRecordImplicitInterface, Is.EqualTo(expected), $"args='{args}'");
+    Assert.That(options.Writer.OmitCompilerGeneratedRecordEqualityMethods, Is.EqualTo(expected), $"args='{args}'");
+  }
 }