trim '-' for match default "help", "list" command

Author: neuecc

sandbox/SingleContainedApp/Program.cs

@@ -102,15 +102,15 @@ public class OverrideCheck : ConsoleAppBase
         [Command(new[] { "unescape", "-h" }, "unescape base64url to base64")]
         public void Unescape([Option(0)]string input) => Console.WriteLine((input));
 
-        [Command(new[] { "help", "-h", "-help", "--help" }, "show help")]
-        public void Help()
-        {
-            Console.WriteLine("Usage: base64urls [-version] [-help] [decode|encode|escape|unescape] [args]");
-            Console.WriteLine("E.g., run this: base64urls decode QyMgaXMgYXdlc29tZQ==");
-            Console.WriteLine("E.g., run this: base64urls encode \"C# is awesome.\"");
-            Console.WriteLine("E.g., run this: base64urls escape \"This+is/goingto+escape==\"");
-            Console.WriteLine("E.g., run this: base64urls unescape \"This-is_goingto-escape\"");
-        }
+        //[Command(new[] { "help", "-h", "-help", "--help" }, "show help")]
+        //public void Help()
+        //{
+        //    Console.WriteLine("Usage: base64urls [-version] [-help] [decode|encode|escape|unescape] [args]");
+        //    Console.WriteLine("E.g., run this: base64urls decode QyMgaXMgYXdlc29tZQ==");
+        //    Console.WriteLine("E.g., run this: base64urls encode \"C# is awesome.\"");
+        //    Console.WriteLine("E.g., run this: base64urls escape \"This+is/goingto+escape==\"");
+        //    Console.WriteLine("E.g., run this: base64urls unescape \"This-is_goingto-escape\"");
+        //}
     }
 
     public class ComplexArgTest : ConsoleAppBase
@@ -132,15 +132,15 @@ class Program
     {
         static async Task Main(string[] args)
         {
-            args = new[] { "-array", "10,20,30", "-person", @"{""Age"":10,""Name"":""foo""}" };
+            // args = new[] { "-array", "10,20,30", "-person", @"{""Age"":10,""Name"":""foo""}" };
 
             await ConsoleAppHost.CreateDefaultBuilder()
                 .ConfigureServices((hostContext, services) =>
                 {
                     // mapping config json to IOption<MyConfig>
                     services.Configure<MyConfig>(hostContext.Configuration);
                 })
-                .RunConsoleAppEngineAsync<ComplexArgTest>(args);
+                .RunConsoleAppEngineAsync<OverrideCheck>(args);
         }
     }
 }

src/ConsoleAppFramework/ConsoleAppEngineHostBuilderExtensions.cs

@@ -15,7 +15,7 @@ public static class ConsoleAppEngineHostBuilderExtensions
 
         public static IHostBuilder UseConsoleAppEngine(this IHostBuilder hostBuilder, string[] args, IConsoleAppInterceptor? interceptor = null)
         {
-            if (args.Length == 0 || (args.Length == 1 && args[0].Equals(ListCommand, StringComparison.OrdinalIgnoreCase)))
+            if (args.Length == 0 || (args.Length == 1 && args[0].Trim('-').Equals(ListCommand, StringComparison.OrdinalIgnoreCase)))
             {
                 ShowMethodList();
                 hostBuilder.ConfigureServices(services =>
@@ -25,7 +25,7 @@ public static IHostBuilder UseConsoleAppEngine(this IHostBuilder hostBuilder, st
                 });
                 return hostBuilder;
             }
-            if (args.Length == 2 && args[0].Equals(HelpCommand, StringComparison.OrdinalIgnoreCase))
+            if (args.Length == 2 && args[0].Trim('-').Equals(HelpCommand, StringComparison.OrdinalIgnoreCase))
             {
                 var (t, mi) = GetTypeFromAssemblies(args[1]);
                 if (mi != null)
@@ -87,7 +87,6 @@ public static IHostBuilder UseConsoleAppEngine<T>(this IHostBuilder hostBuilder,
         {
             var method = typeof(T).GetMethods(BindingFlags.Public | BindingFlags.Instance | BindingFlags.DeclaredOnly);
             var defaultMethod = method.FirstOrDefault(x => x.GetCustomAttribute<CommandAttribute>() == null);
-            var hasList = method.Any(x => x.GetCustomAttribute<CommandAttribute>()?.EqualsAny(ListCommand) ?? false);
             var hasHelp = method.Any(x => x.GetCustomAttribute<CommandAttribute>()?.EqualsAny(HelpCommand) ?? false);
 
             if (args.Length == 0)
@@ -113,18 +112,7 @@ public static IHostBuilder UseConsoleAppEngine<T>(this IHostBuilder hostBuilder,
                 }
             }
 
-            if (!hasList && args.Length == 1 && args[0].Equals(ListCommand, StringComparison.OrdinalIgnoreCase))
-            {
-                ShowMethodList();
-                hostBuilder.ConfigureServices(services =>
-                {
-                    services.AddOptions<ConsoleLifetimeOptions>().Configure(x => x.SuppressStatusMessages = true);
-                    services.AddSingleton<IHostedService, EmptyHostedService>();
-                });
-                return hostBuilder;
-            }
-
-            if (!hasHelp && args.Length == 1 && args[0].Equals(HelpCommand, StringComparison.OrdinalIgnoreCase))
+            if (!hasHelp && args.Length == 1 && args[0].Trim('-').Equals(HelpCommand, StringComparison.OrdinalIgnoreCase))
             {
                 Console.Write(new CommandHelpBuilder().BuildHelpMessage(method, defaultMethod));
 
@@ -209,7 +197,7 @@ static List<Type> GetConsoleAppTypes()
                 {
                     if (cmdattr.CommandNames.Any(x => arg0.Equals(x, StringComparison.OrdinalIgnoreCase)))
                     {
-                        if(foundType != null && foundMethod != null)
+                        if (foundType != null && foundMethod != null)
                         {
                             throw new InvalidOperationException($"Duplicate ConsoleApp Command name is not allowed, {foundType.FullName}.{foundMethod.Name} and {baseType.FullName}.{method.Name}");
                         }
@@ -231,7 +219,7 @@ static List<Type> GetConsoleAppTypes()
                     }
                 }
             }
-            if(foundType != null && foundMethod != null)
+            if (foundType != null && foundMethod != null)
             {
                 return (foundType, foundMethod);
             }