`IServiceProviderIsService` not returning `true` for various registrations

I was just playing using Lamar with minimal APIs and I’ve found there are various cases where injected services aren’t being inferred because Lamar’s IServiceProviderIsService implementation isn’t return true for various cases:

• IEnumerable<T> where multiple T registered
• IGenericService<T> open generic
• ConcreteClass when not explicitly registered

Sample app demonstrating the issue below. Note that in all cases, add [FromServices] fixes the issue - it’s not the registration at issue, it’s the call to IServiceProviderIsService that’s wrong. This repros on .NET 6 and .NET 7, giving the following error (indicating none of the args were resolved from DI)

InvalidOperationException: Body was inferred but the method does not allow inferred body parameters.
Below is the list of parameters that we found:

Parameter | Source
---------------------------------------------------------------------------------
services  | Body (Inferred)
concrete  | UNKNOWN
generic   | UNKNOWN

I haven’t done an extensive search for all the cases that are/aren’t found, but these seem to be the main culprits

Thanks!

using Lamar;
using Lamar.Microsoft.DependencyInjection;

var builder = WebApplication.CreateBuilder(args);
builder.Host.UseLamar(services =>
{
    services.Scan(s =>
    {
        s.AssemblyContainingType(typeof(Program));
        s.WithDefaultConventions();
        s.AddAllTypesOf<IService>();
    });
});

var app = builder.Build();

app.MapGet("/", (IEnumerable<IService> services,
    ConcreteClass concrete,
    IGenericService<ConcreteClass> generic) => "Hello World!");

app.Run();

public interface IService { }
public class ServiceA : IService { }
public class ServiceB : IService { }

public class ConcreteClass {}

public interface IGenericService<T> {}
public class GenericService<T> {}

[jeremydmiller]

Think this will be in a new Lamar 10.0 soon.

[jeremydmiller]

Ugh, @andrewlock I didn’t realize that was part of their “spec” for how that was supposed to work. Okay. This is one of those times when I curse the conforming container behavior that’s rammed down our throats