refactor: migrate object mapping from AutoMapper to Mapperly (#10)

gdlcf88 · claude · web-flow · commit 0f8b6ed40c22 · 2026-07-24T20:16:47.000+08:00
Replace Volo.Abp.AutoMapper with Volo.Abp.Mapperly and convert the
AutoMapper profiles into Mapperly mappers.

Co-authored-by: Claude Opus 4.8 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Application/PhoneNumberLoginSampleApplicationAutoMapperProfile.cs b/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Application/PhoneNumberLoginSampleApplicationAutoMapperProfile.cs
diff --git a/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Application/PhoneNumberLoginSampleApplicationModule.cs b/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Application/PhoneNumberLoginSampleApplicationModule.cs
@@ -1,6 +1,5 @@
 ﻿using EasyAbp.Abp.PhoneNumberLogin;
 using Volo.Abp.Account;
-using Volo.Abp.AutoMapper;
 using Volo.Abp.FeatureManagement;
 using Volo.Abp.Identity;
 using Volo.Abp.Modularity;
@@ -23,12 +22,5 @@ namespace PhoneNumberLoginSample
     )]
     public class PhoneNumberLoginSampleApplicationModule : AbpModule
     {
-        public override void ConfigureServices(ServiceConfigurationContext context)
-        {
-            Configure<AbpAutoMapperOptions>(options =>
-            {
-                options.AddMaps<PhoneNumberLoginSampleApplicationModule>();
-            });
-        }
     }
 }
diff --git a/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Web/PhoneNumberLoginSampleWebAutoMapperProfile.cs b/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Web/PhoneNumberLoginSampleWebAutoMapperProfile.cs
diff --git a/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Web/PhoneNumberLoginSampleWebModule.cs b/samples/PhoneNumberLoginSample/src/PhoneNumberLoginSample.Web/PhoneNumberLoginSampleWebModule.cs
@@ -21,7 +21,6 @@
 using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared;
 using Volo.Abp.AspNetCore.Serilog;
 using Volo.Abp.Autofac;
-using Volo.Abp.AutoMapper;
 using Volo.Abp.FeatureManagement;
 using Volo.Abp.Identity.Web;
 using Volo.Abp.Localization;
@@ -84,7 +83,6 @@ public override void ConfigureServices(ServiceConfigurationContext context)
 
             ConfigureUrls(configuration);
             ConfigureAuthentication(context);
-            ConfigureAutoMapper();
             ConfigureVirtualFileSystem(hostingEnvironment);
             ConfigureLocalizationServices();
             ConfigureNavigationServices();
@@ -106,11 +104,6 @@ private void ConfigureAuthentication(ServiceConfigurationContext context)
                 .AuthenticationScheme);
         }
 
-        private void ConfigureAutoMapper()
-        {
-            Configure<AbpAutoMapperOptions>(options => { options.AddMaps<PhoneNumberLoginSampleWebModule>(); });
-        }
-
         private void ConfigureVirtualFileSystem(IWebHostEnvironment hostingEnvironment)
         {
             if (hostingEnvironment.IsDevelopment())
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp.Abp.PhoneNumberLogin.Application.csproj b/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp.Abp.PhoneNumberLogin.Application.csproj
@@ -8,7 +8,7 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <PackageReference Include="Volo.Abp.AutoMapper" Version="$(AbpVersion)" />
+    <PackageReference Include="Volo.Abp.Mapperly" Version="$(AbpVersion)" />
     <PackageReference Include="Volo.Abp.Ddd.Application" Version="$(AbpVersion)" />
     <PackageReference Include="Volo.Abp.Http.Client.IdentityModel" Version="$(AbpVersion)" />
     <PackageReference Include="EasyAbp.Abp.VerificationCode" Version="$(EasyAbpAbpVerificationCodeModuleVersion)" />
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/AbpPhoneNumberLoginApplicationModule.cs b/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/AbpPhoneNumberLoginApplicationModule.cs
@@ -1,6 +1,6 @@
 ﻿using EasyAbp.Abp.VerificationCode;
 using Microsoft.Extensions.DependencyInjection;
-using Volo.Abp.AutoMapper;
+using Volo.Abp.Mapperly;
 using Volo.Abp.Modularity;
 using Volo.Abp.Application;
 using Volo.Abp.Http.Client.IdentityModel;
@@ -11,19 +11,15 @@ namespace EasyAbp.Abp.PhoneNumberLogin
         typeof(AbpPhoneNumberLoginDomainModule),
         typeof(AbpPhoneNumberLoginApplicationContractsModule),
         typeof(AbpDddApplicationModule),
-        typeof(AbpAutoMapperModule),
+        typeof(AbpMapperlyModule),
         typeof(AbpVerificationCodeModule),
         typeof(AbpHttpClientIdentityModelModule)
     )]
     public class AbpPhoneNumberLoginApplicationModule : AbpModule
     {
         public override void ConfigureServices(ServiceConfigurationContext context)
         {
-            context.Services.AddAutoMapperObjectMapper<AbpPhoneNumberLoginApplicationModule>();
-            Configure<AbpAutoMapperOptions>(options =>
-            {
-                options.AddMaps<AbpPhoneNumberLoginApplicationModule>(validate: true);
-            });
+            context.Services.AddMapperlyObjectMapper<AbpPhoneNumberLoginApplicationModule>();
         }
     }
 }
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/PhoneNumberLoginApplicationAutoMapperProfile.cs b/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/PhoneNumberLoginApplicationAutoMapperProfile.cs
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/PhoneNumberLoginApplicationMapperlyMappers.cs b/src/EasyAbp.Abp.PhoneNumberLogin.Application/EasyAbp/Abp/PhoneNumberLogin/PhoneNumberLoginApplicationMapperlyMappers.cs
@@ -0,0 +1,15 @@
+using Riok.Mapperly.Abstractions;
+using Volo.Abp.Identity;
+using Volo.Abp.Mapperly;
+using IdentityUser = Volo.Abp.Identity.IdentityUser;
+
+namespace EasyAbp.Abp.PhoneNumberLogin
+{
+    [Mapper(RequiredMappingStrategy = RequiredMappingStrategy.Target)]
+    [MapExtraProperties]
+    public partial class IdentityUserToIdentityUserDtoMapper : MapperBase<IdentityUser, IdentityUserDto>
+    {
+        public override partial IdentityUserDto Map(IdentityUser source);
+        public override partial void Map(IdentityUser source, IdentityUserDto destination);
+    }
+}
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Web/AbpPhoneNumberLoginWebModule.cs b/src/EasyAbp.Abp.PhoneNumberLogin.Web/AbpPhoneNumberLoginWebModule.cs
@@ -5,7 +5,7 @@
 using Volo.Abp.Account.Web;
 using Volo.Abp.AspNetCore.Mvc.Localization;
 using Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared;
-using Volo.Abp.AutoMapper;
+using Volo.Abp.Mapperly;
 using Volo.Abp.Modularity;
 using Volo.Abp.UI.Navigation;
 using Volo.Abp.VirtualFileSystem;
@@ -19,7 +19,7 @@ namespace EasyAbp.Abp.PhoneNumberLogin.Web
         typeof(AbpAspNetCoreMvcUiThemeSharedModule),
         typeof(AbpAccountWebModule),
         typeof(AbpIdentityDomainSharedModule),
-        typeof(AbpAutoMapperModule)
+        typeof(AbpMapperlyModule)
     )]
     public class AbpPhoneNumberLoginWebModule : AbpModule
     {
@@ -48,11 +48,7 @@ public override void ConfigureServices(ServiceConfigurationContext context)
                 options.FileSets.AddEmbedded<AbpPhoneNumberLoginWebModule>();
             });
 
-            context.Services.AddAutoMapperObjectMapper<AbpPhoneNumberLoginWebModule>();
-            Configure<AbpAutoMapperOptions>(options =>
-            {
-                options.AddMaps<AbpPhoneNumberLoginWebModule>(validate: true);
-            });
+            context.Services.AddMapperlyObjectMapper<AbpPhoneNumberLoginWebModule>();
 
             Configure<RazorPagesOptions>(options =>
             {
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Web/EasyAbp.Abp.PhoneNumberLogin.Web.csproj b/src/EasyAbp.Abp.PhoneNumberLogin.Web/EasyAbp.Abp.PhoneNumberLogin.Web.csproj
@@ -13,7 +13,7 @@
 
   <ItemGroup>
     <PackageReference Include="Volo.Abp.Account.Web" Version="$(AbpVersion)" />
-    <PackageReference Include="Volo.Abp.AutoMapper" Version="$(AbpVersion)" />
+    <PackageReference Include="Volo.Abp.Mapperly" Version="$(AbpVersion)" />
     <PackageReference Include="Volo.Abp.AspNetCore.Mvc.UI.Theme.Shared" Version="$(AbpVersion)" />
     <PackageReference Include="Volo.Abp.Identity.Domain.Shared" Version="$(AbpVersion)" />
   </ItemGroup>
diff --git a/src/EasyAbp.Abp.PhoneNumberLogin.Web/PhoneNumberLoginWebAutoMapperProfile.cs b/src/EasyAbp.Abp.PhoneNumberLogin.Web/PhoneNumberLoginWebAutoMapperProfile.cs
diff --git a/test/EasyAbp.Abp.PhoneNumberLogin.Application.Tests/PhoneNumberLoginObjectMapping_Tests.cs b/test/EasyAbp.Abp.PhoneNumberLogin.Application.Tests/PhoneNumberLoginObjectMapping_Tests.cs
@@ -0,0 +1,43 @@
+using System;
+using Shouldly;
+using Volo.Abp.Data;
+using Volo.Abp.Identity;
+using Volo.Abp.ObjectMapping;
+using Xunit;
+
+namespace EasyAbp.Abp.PhoneNumberLogin
+{
+    public class PhoneNumberLoginObjectMapping_Tests : PhoneNumberLoginApplicationTestBase
+    {
+        private readonly IObjectMapper<AbpPhoneNumberLoginApplicationModule> _objectMapper;
+
+        public PhoneNumberLoginObjectMapping_Tests()
+        {
+            _objectMapper = GetRequiredService<IObjectMapper<AbpPhoneNumberLoginApplicationModule>>();
+        }
+
+        [Fact]
+        public void Should_Map_IdentityUser_To_IdentityUserDto()
+        {
+            // Arrange
+            var userId = Guid.NewGuid();
+            var user = new IdentityUser(userId, "john", "john@example.com")
+            {
+                Name = "John",
+                Surname = "Doe"
+            };
+            user.SetProperty("CustomProp", "custom-value");
+
+            // Act
+            var dto = _objectMapper.Map<IdentityUser, IdentityUserDto>(user);
+
+            // Assert
+            dto.Id.ShouldBe(userId);
+            dto.UserName.ShouldBe("john");
+            dto.Email.ShouldBe("john@example.com");
+            dto.Name.ShouldBe("John");
+            dto.Surname.ShouldBe("Doe");
+            dto.GetProperty<string>("CustomProp").ShouldBe("custom-value");
+        }
+    }
+}

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,5 @@`
`1`	`1`	`using EasyAbp.Abp.PhoneNumberLogin;`
`2`	`2`	`using Volo.Abp.Account;`
`3`		`-using Volo.Abp.AutoMapper;`
`4`	`3`	`using Volo.Abp.FeatureManagement;`
`5`	`4`	`using Volo.Abp.Identity;`
`6`	`5`	`using Volo.Abp.Modularity;`
`@@ -23,12 +22,5 @@ namespace PhoneNumberLoginSample`
`23`	`22`	`)]`
`24`	`23`	`public class PhoneNumberLoginSampleApplicationModule : AbpModule`
`25`	`24`	`{`
`26`		`- public override void ConfigureServices(ServiceConfigurationContext context)`
`27`		`- {`
`28`		`- Configure<AbpAutoMapperOptions>(options =>`
`29`		`- {`
`30`		`- options.AddMaps<PhoneNumberLoginSampleApplicationModule>();`
`31`		`- });`
`32`		`- }`
`33`	`25`	`}`
`34`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`using EasyAbp.Abp.VerificationCode;`
`2`	`2`	`using Microsoft.Extensions.DependencyInjection;`
`3`		`-using Volo.Abp.AutoMapper;`
	`3`	`+using Volo.Abp.Mapperly;`
`4`	`4`	`using Volo.Abp.Modularity;`
`5`	`5`	`using Volo.Abp.Application;`
`6`	`6`	`using Volo.Abp.Http.Client.IdentityModel;`
`@@ -11,19 +11,15 @@ namespace EasyAbp.Abp.PhoneNumberLogin`
`11`	`11`	`typeof(AbpPhoneNumberLoginDomainModule),`
`12`	`12`	`typeof(AbpPhoneNumberLoginApplicationContractsModule),`
`13`	`13`	`typeof(AbpDddApplicationModule),`
`14`		`- typeof(AbpAutoMapperModule),`
	`14`	`+ typeof(AbpMapperlyModule),`
`15`	`15`	`typeof(AbpVerificationCodeModule),`
`16`	`16`	`typeof(AbpHttpClientIdentityModelModule)`
`17`	`17`	`)]`
`18`	`18`	`public class AbpPhoneNumberLoginApplicationModule : AbpModule`
`19`	`19`	`{`
`20`	`20`	`public override void ConfigureServices(ServiceConfigurationContext context)`
`21`	`21`	`{`
`22`		`- context.Services.AddAutoMapperObjectMapper<AbpPhoneNumberLoginApplicationModule>();`
`23`		`- Configure<AbpAutoMapperOptions>(options =>`
`24`		`- {`
`25`		`- options.AddMaps<AbpPhoneNumberLoginApplicationModule>(validate: true);`
`26`		`- });`
	`22`	`+ context.Services.AddMapperlyObjectMapper<AbpPhoneNumberLoginApplicationModule>();`
`27`	`23`	`}`
`28`	`24`	`}`
`29`	`25`	`}`