Allow setting a group name for endpoints and have it be used to populate ApiDescription.GroupName in ApiExplorer for minimal APIs

[DamianEdwards]

APIs described by Microsoft.AspNetCore.Mvc.ApiExplorer.ApiDescription can optionally have a group name defined by the GroupName property. Today in the framework there isn't a way provided to set the group name for an endpoint via its metadata and the EndpointMetadataApiDescriptionProvider does not set ApiDescription.GroupName when it populates ApiExplorer with details of the registered endpoints.

ApiDescription.GroupName is used by frameworks like Swashbuckle to associate APIs with different OpenAPI documents, i.e. if the document name and API group name match the API is included in the document (see default logic here).

We should allow for setting the group name for endpoints via their metadata and have that used to populate the ApiDescription.GroupName property when the endpoints are added to ApiExplorer.

Proposed types for addition:

namespace Microsoft.AspNetCore.Routing;

/// <summary>
/// Declares the group name for this endpoint method or delegate.<br />
/// </summary>
[AttributeUsage(AttributeTargets.Method | AttributeTargets.Delegate, Inherited = false, AllowMultiple = false)]
public sealed class EndpointGroupNameAttribute: Attribute, IEndpointGroupNameMetadata
{
    public EndpointGroupNameAttribute(string endpointGroupName)
    {
        if (endpointGroupName == null)
        {
            throw new ArgumentNullException(nameof(endpointGroupName));
        }

        EndpointGroupName = endpointGroupName;
    }

    /// <summary>
    /// The endpoint group name.
    /// </summary>
    public string EndpointGroupName { get; }
}

/// <summary>
/// Specifies an endpoint group name in Microsoft.AspNetCore.Http.Endpoint.Metadata.
/// </summary>
public class EndpointGroupNameMetadata : IEndpointGroupNameMetadata
{
    public EndpointGroupNameMetadata(string endpointGroupName)
    {
        if (endpointGroupName == null)
        {
            throw new ArgumentNullException(nameof(endpointGroupName));
        }

        EndpointGroupName = endpointGroupName;
    }

    /// <summary>
    /// The endpoint group name.
    /// </summary>
    public string EndpointGroupName { get; }
}

/// <summary>
/// Defines a contract use to specify an endpoint group name in Microsoft.AspNetCore.Http.Endpoint.Metadata.
/// </summary>
public interface IEndpointGroupNameMetadata
{
    /// <summary>
    /// The endpoint group name.
    /// </summary>
    string EndpointGroupName { get; }
}

Proposed extension methods for addition:

namespace Microsoft.AspNetCore.Builder;

public static class RoutingEndpointConventionBuilderExtensions
{
+    /// <summary>
+    /// Adds an EndpointGroupNameMetadata item to the Metadata for all endpoints produced by the builder.
+    /// </summary>
+    /// <param name="builder"></param>
+    /// <param name="groupName"></param>
+    /// <returns>The Microsoft.AspNetCore.Builder.IEndpointConventionBuilder.</returns>
+    public static TBuilder WithGroupName<TBuilder>(this TBuilder builder, string groupName) where TBuilder : IEndpointConventionBuilder
+    {
+        builder.WithMetadata(new EndpointGroupNameMetadata(groupName));
+
+        return builder;
+    }
}

Example of these new types and methods being used with minimal APIs:

app.MapGet("/hello/{name}", (string name) => $"Hello {name}")
   .WithName("SayHelloToName")
   .WithGroupName("NameApi")

app.MapGet("/todos/get-all", async (TodoDb db) => await db.Todos.ToListAsync())
   .WithName("GetAllTodos")
   .WithGroupName("TodoApi");

[ghost]

Thanks for contacting us.

We're moving this issue to the Next sprint planning milestone for future evaluation / consideration. We would like to keep this around to collect more feedback, which can help us with prioritizing this work. We will re-evaluate this issue, during our next planning meeting(s).
If we later determine, that the issue has no community involvement, or it's very rare and low-impact issue, we will close it - so that the team can focus on more important and high impact issues.
To learn more about what to expect next and how this issue will be handled you can read more about our triage process here.