fix #2640

Author: jonsequitur

src/System.CommandLine.Tests/HelpOptionTests.cs

@@ -6,6 +6,7 @@
 using System.CommandLine.Invocation;
 using System.CommandLine.Tests.Utility;
 using System.IO;
+using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
 using Xunit;
@@ -255,6 +256,41 @@ public void The_users_can_set_max_width()
         output.ToString().Should().Contain($"  {secondPart}{Environment.NewLine}");
     }
 
+    [Fact] // https://github.com/dotnet/command-line-api/issues/2640
+    public void DefaultValueFactory_does_not_throw_when_help_is_invoked()
+    {
+        var invocationConfiguration = new InvocationConfiguration
+        {
+            Output = new StringWriter(),
+            Error = new StringWriter()
+        };
+
+        Command subcommand = new("do")
+        {
+            new Option<DirectoryInfo>("-x")
+            {
+                DefaultValueFactory = result =>
+                {
+                    result.AddError("Oops!");
+                    return null;
+                    
+                }
+            }
+        };
+        subcommand.SetAction(_ =>
+        {
+        });
+        RootCommand rootCommand = new()
+        {
+            subcommand
+        };
+
+        rootCommand.Parse("do --help").Invoke(invocationConfiguration);
+
+        invocationConfiguration.Error.ToString().Should().Be("");
+    }
+
+
     private sealed class CustomizedHelpAction : SynchronousCommandLineAction
     {
         internal const string CustomUsageText = "This is custom command usage example.";

src/System.CommandLine/Argument.cs

@@ -112,7 +112,11 @@ public List<Func<CompletionContext, IEnumerable<CompletionItem>>> CompletionSour
         /// <returns>Returns the default value for the argument, if defined. Null otherwise.</returns>
         public object? GetDefaultValue()
         {
-            return GetDefaultValue(new ArgumentResult(this, null!, null));
+            var command = Parents.FlattenBreadthFirst(x => x.Parents)
+                                 .OfType<Command>()
+                                 .FirstOrDefault();
+
+            return GetDefaultValue(new ArgumentResult(this, new SymbolResultTree(command!), null));
         }
 
         internal abstract object? GetDefaultValue(ArgumentResult argumentResult);

src/System.CommandLine/Parsing/SymbolResult.cs

@@ -15,7 +15,7 @@ public abstract class SymbolResult
 
         private protected SymbolResult(SymbolResultTree symbolResultTree, SymbolResult? parent)
         {
-            SymbolResultTree = symbolResultTree;
+            SymbolResultTree = symbolResultTree ?? throw new ArgumentNullException(nameof(symbolResultTree));
             Parent = parent;
         }
 

src/System.CommandLine/Parsing/SymbolResultTree.cs

@@ -14,7 +14,7 @@ internal sealed class SymbolResultTree : Dictionary<Symbol, SymbolResult>
 
         internal SymbolResultTree(
             Command rootCommand, 
-            List<string>? tokenizeErrors)
+            List<string>? tokenizeErrors = null)
         {
             _rootCommand = rootCommand;