Ask for the output directory

dotnet · jjonescz · Jul 15, 2025 · Jul 3, 2025 · Jul 3, 2025 · Jul 3, 2025
commit 4a56fc7dd1e2c8e6ea9b4237e842a779fc9957b4
diff --git a/src/Cli/dotnet/Commands/CliCommandStrings.resx b/src/Cli/dotnet/Commands/CliCommandStrings.resx
@@ -1504,6 +1504,10 @@ Tool '{1}' (version '{2}') was successfully installed. Entry is added to the man
   <data name="CmdOptionKeepSourceDescription" xml:space="preserve">
     <value>Whether to keep source files intact (otherwise, they are deleted after conversion).</value>
   </data>
+  <data name="ProjectConvertAskForOutputDirectory" xml:space="preserve">
+    <value>Specify the output directory ({0}):</value>
+    <comment>{0} is the default value</comment>
+  </data>
   <data name="ProjectConvertConfirmKeepSourceFiles" xml:space="preserve">
     <value>Keep source files?</value>
   </data>

@@ -25,14 +25,7 @@ public override int Execute()
             throw new GracefulException(CliCommandStrings.InvalidFilePath, file);
         }
 
-        // Determine the output directory.
-        string targetDirectory = _outputDirectory ?? Path.ChangeExtension(file, null);
-        if (Directory.Exists(targetDirectory))
-        {
-            throw new GracefulException(CliCommandStrings.DirectoryAlreadyExists, targetDirectory);
-        }
-
-        // Determine whether to keep the source files.
+        string targetDirectory = DetermineOutputDirectory(file);
         bool keepSourceFiles = ShouldKeepSourceFiles();
 
         // Find directives (this can fail, so do this before creating the target directory).
@@ -136,6 +129,24 @@ public override int Execute()
         }
     }
 
+    private string DetermineOutputDirectory(string file)
+    {
+        string defaultValue = Path.ChangeExtension(file, null);
+        string defaultValueRelative = Path.GetRelativePath(relativeTo: Environment.CurrentDirectory, defaultValue);
+        string targetDirectory = _outputDirectory
+            ?? InteractiveConsole.Ask(
+                string.Format(CliCommandStrings.ProjectConvertAskForOutputDirectory, defaultValueRelative),
+                _parseResult,
+                (path) => Directory.Exists(path) ? string.Format(CliCommandStrings.DirectoryAlreadyExists, Path.GetFullPath(path)) : null)
+            ?? defaultValue;
+        if (Directory.Exists(targetDirectory))
+        {
+            throw new GracefulException(CliCommandStrings.DirectoryAlreadyExists, targetDirectory);
+        }
+
+        return Path.GetFullPath(targetDirectory);
+    }
+
     private bool ShouldKeepSourceFiles()
     {
         bool? keepSourceFiles =

@@ -57,4 +57,29 @@ static bool KeyMatches(ConsoleKeyInfo pressedKey, string valueKey)
                 valueKey.ToLowerInvariant().Substring(0, 1));
         }
     }
+
+    public static string? Ask(string question, ParseResult parseResult, Func<string?, string?> validate)
+    {
+        if (!parseResult.GetValue(CommonOptions.InteractiveOption()))
+        {
+            return null;
+        }
+
+        while (true)
+        {
+            Console.Write(question);
+            Console.Write(' ');
+
+            string? answer = Console.ReadLine();
+            answer = string.IsNullOrWhiteSpace(answer) ? null : answer.Trim();
+            if (validate(answer) is { } error)
+            {
+                Console.WriteLine(error);
+            }
+            else
+            {
+                return answer;
+            }
+        }
+    }
 }