Make BuildCommandFromArguments reusable

Author: andyleejordan

src/PowerShellEditorServices/Services/DebugAdapter/Handlers/ConfigurationDoneHandler.cs

@@ -13,10 +13,8 @@
 using OmniSharp.Extensions.DebugAdapter.Protocol.Events;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Requests;
 using OmniSharp.Extensions.DebugAdapter.Protocol.Server;
-using System.Collections.Generic;
 using System.Management.Automation;
 using System.Management.Automation.Language;
-using System.Text;
 using System.Threading;
 using System.Threading.Tasks;
 
@@ -140,38 +138,13 @@ await _executionService
             {
                 await _executionService
                     .ExecutePSCommandAsync(
-                        BuildPSCommandFromArguments(scriptToLaunch, _debugStateService.Arguments),
+                        PSCommandHelpers.BuildCommandFromArguments(scriptToLaunch, _debugStateService.Arguments),
                         CancellationToken.None,
                         s_debuggerExecutionOptions)
                     .ConfigureAwait(false);
             }
 
             _debugAdapterServer.SendNotification(EventNames.Terminated);
         }
-
-        private static PSCommand BuildPSCommandFromArguments(string command, IReadOnlyList<string> arguments)
-        {
-            if (arguments is null or { Count: 0 })
-            {
-                return new PSCommand().AddCommand(command);
-            }
-
-            // HACK: We use AddScript instead of AddArgument/AddParameter to reuse Powershell parameter binding logic.
-            // We quote the command parameter so that expressions can still be used in the arguments.
-            var sb = new StringBuilder()
-                .Append('&')
-                .Append('"')
-                .Append(command)
-                .Append('"');
-
-            foreach (string arg in arguments)
-            {
-                sb
-                .Append(' ')
-                .Append(ArgumentEscaping.Escape(arg));
-            }
-
-            return new PSCommand().AddScript(sb.ToString());
-        }
     }
 }

src/PowerShellEditorServices/Utility/PSCommandExtensions.cs

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 
 using System;
+using System.Collections.Generic;
 using System.IO;
 using System.Linq.Expressions;
 using System.Management.Automation;
@@ -11,11 +12,11 @@
 
 namespace Microsoft.PowerShell.EditorServices.Utility
 {
-    internal static class PSCommandExtensions
+    internal static class PSCommandHelpers
     {
         private static readonly Func<CommandInfo, Command> s_commandCtor;
 
-        static PSCommandExtensions()
+        static PSCommandHelpers()
         {
             var ctor = typeof(Command).GetConstructor(
                 BindingFlags.Instance | BindingFlags.NonPublic | BindingFlags.Public,
@@ -119,5 +120,30 @@ private static StringBuilder AddCommandText(this StringBuilder sb, Command comma
 
             return sb;
         }
+
+        public static PSCommand BuildCommandFromArguments(string command, IReadOnlyList<string> arguments)
+        {
+            if (arguments is null or { Count: 0 })
+            {
+                return new PSCommand().AddCommand(command);
+            }
+
+            // HACK: We use AddScript instead of AddArgument/AddParameter to reuse Powershell parameter binding logic.
+            // We quote the command parameter so that expressions can still be used in the arguments.
+            var sb = new StringBuilder()
+                .Append('&')
+                .Append('"')
+                .Append(command)
+                .Append('"');
+
+            foreach (string arg in arguments)
+            {
+                sb
+                .Append(' ')
+                .Append(ArgumentEscaping.Escape(arg));
+            }
+
+            return new PSCommand().AddScript(sb.ToString());
+        }
     }
 }