Feat/get user roles

[MayueCif]

增加获取用户角色的接口

[Copilot]

Pull request overview

This pull request adds a new API endpoint to retrieve user roles for a specific user. The main feature adds GET /api/user/{id}/roles endpoint that returns a list of enabled roles associated with a user, including role code, name, and the binding time. However, the PR has critical compilation issues due to a missing type definition.

Changes:

• Added new GET endpoint /api/user/{id}/roles to retrieve user roles by user ID
• Added query handler GetUserRolesAsync and query class UserRolesQuery to support the new endpoint
• Enhanced RoleModel mapping in User.cs to include additional properties (CreationTime, ModificationTime, Enabled)
• Added enabled role filtering to GetRoleSelectAsync method
• Removed parameterless constructor from RoleSelectDto
• Minor code formatting improvements

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
src/Services/Masa.Auth.Service.Admin/Services/UserService.cs	Adds new MapGet route for getting user roles endpoint
src/Services/Masa.Auth.Service.Admin/Application/Subjects/QueryHandler.cs	Implements GetUserRolesAsync query handler to fetch enabled roles for a user
src/Services/Masa.Auth.Service.Admin/Application/Subjects/Queries/UserRolesQuery.cs	Defines new query class for user roles retrieval
src/Services/Masa.Auth.Service.Admin/Application/Permissions/QueryHandler.cs	Adds filtering for enabled roles in GetRoleSelectAsync helper method
src/Domain/Masa.Auth.Domain/Subjects/Aggregates/User.cs	Enriches RoleModel with CreationTime, ModificationTime, and Enabled properties
src/Contracts/Masa.Auth.Contracts.Admin/Permissions/RoleSelectDto.cs	Removes parameterless constructor

Comments suppressed due to low confidence (1)

src/Contracts/Masa.Auth.Contracts.Admin/Permissions/RoleSelectDto.cs:26

• Removing the parameterless constructor may cause issues with serialization/deserialization in certain scenarios and breaks consistency with similar DTOs in the codebase (e.g., UserSelectDto in src/Contracts/Masa.Auth.Contracts.Admin/Subjects/UserSelectDto.cs:22). The Name and Code properties are non-nullable strings but are no longer initialized to empty strings when a parameterless constructor would be used. Consider keeping the parameterless constructor for compatibility, or verify that all serialization scenarios work correctly without it.

    public RoleSelectDto(Guid id, string name, string code, int limit, int availableQuantity)
    {
        Id = id;
        Name = name;
        Code = code;
        Limit = limit;
        AvailableQuantity = availableQuantity;
        CreationTime = creationTime;
    }

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The UserRoleModel type does not exist in the codebase. This query definition will cause compilation errors. You need to either:

1. Create a UserRoleModel class/record in the appropriate location (e.g., in src/Contracts/Masa.Auth.Contracts.Admin/Subjects/), or
2. Use an existing type like RoleModel or RoleSelectDto

Based on the query handler implementation that uses Code, Name, and BindTime properties, you should create a new UserRoleModel with these properties.

Suggested change

	public record UserRoleModel(string Code, string Name, DateTime BindTime);

[Copilot]

The UserRoleModel type does not exist in the codebase, which will cause compilation errors. The query handler attempts to create instances of UserRoleModel with Code, Name, and BindTime properties, but this type is not defined anywhere in the repository or imported from external packages.

[Copilot]

The UserRoleModel return type does not exist in the codebase, which will cause compilation errors. This endpoint cannot function until the UserRoleModel class/record is defined with the appropriate properties (Code, Name, BindTime based on the query handler implementation).

[Copilot]

Potential null reference exception when accessing ur.Role properties. The Where clause filters for ur.Role.Enabled, but if ur.Role is null at the time of Select, this will throw a NullReferenceException. Consider adding an additional null check in the Where clause or using null-conditional operators in the Select clause to make the code more defensive.

[Copilot]

Call to obsolete method MapGet.

Suggested change

	MapGet(GetListByRoleAsync, "getListByRole");
	MapGet(GetUserRolesAsync, "{id}/roles");
	MapGet(GetClaimValuesAsync, "claim-values/{id}");
	MapGet(GetClaimValuesAsync, "claim-values");
	MapGet("getListByRole", GetListByRoleAsync);
	MapGet("{id}/roles", GetUserRolesAsync);
	MapGet("claim-values/{id}", GetClaimValuesAsync);
	MapGet("claim-values", GetClaimValuesAsync);