Usage of `dotnet tools` on assemblies with CommandLineParser results in unexpected output
I was trying to add CommandLineParser to a asp.net core project of mine to conditionally apply seed and test data on runs.
If I run any sort of tool on the assembly I get output from the Parser.
F.e. dotnet ef migrations list
ERROR(S):
Option 'applicationName' is unknown.
-s, --seed Seed required data.
-t, --test Seed data for dev and testing purposes.
-a, --auth Seed auth data.
--help Display this help screen.
--version Display version information.
I'm assuming that the tools themselves are passing these options or something along the line. Everything seems to work properly, but is this something other people have dealt with in some way?
I ran into this as well - a couple potential solutions:
- Don't parse CLI flags when
EF.IsDsignTimeistrue. This only works if you don't need the CLI flags to properly build theIHostobject or don't need theIHostobject for configuring theDbContext(e.g., in a case whereDbContextconfigures itself independently of ASP.NET server build).
using Microsoft.EntityFrameworkCore;
static void Main(string[] args)
{
if (EF.IsDesignTime)
{
Log.Logger.Here().Information("Design-time detected. Skipping server startup.");
return;
}
// Parse flags as normal, build ASP.NET application
}
- Add the
applicationNameflag toCommandLineParserto avoid the error and build the web app as normal.
internal class Flags
{
#region ASP.NET Core Flags
[Option("applicationName", Default = "MyApp", HelpText = "The name of the application.")]
public string ApplicationName { get; set; }
// Other ASP.NET flags as-needed, e.g.:
[Option("environment", Default = "Production", HelpText = "The environment the application is running in.")]
public string Environment { get; set; }
#endregion
// Rest of flags...
}
Can't you set IgnoreUnknownArguments = true in parser settings?
@olstakh sounds like this would work! I think the only potential downside is if you always set this, you might miss some invalid flags in other cases (e.g., when a user is using the CLI vs ASP.NET). Doing something like this could avoid such cases while still not causing errors for ASP.NET flags at design time:
Parser parser = new(settings => {
if (EF.IsDesignTime)
{
settings.IgnoreUnknownArguments = true;
}
// Other options
settings.HelpWriter = Console.Error;
});