A new hosting model which is easier for Editor Services and other script analyzer hosting

Engine/Commands/GetScriptAnalyzerRuleCommand.cs

@@ -61,7 +61,6 @@ public string[] Name
         /// <summary>
         /// Severity: Array of the severity types to be enabled.
         /// </summary>
-        /// </summary>
         [ValidateSet("Warning", "Error", "Information", IgnoreCase = true)]
         [Parameter(Mandatory = false)]
         [SuppressMessage("Microsoft.Performance", "CA1819:PropertiesShouldNotReturnArrays")]

Engine/Engine.csproj

@@ -1,11 +1,12 @@
 <Project Sdk="Microsoft.NET.Sdk">
 
   <PropertyGroup>
-    <VersionPrefix>1.18.3</VersionPrefix>
+    <VersionPrefix>1.18.4</VersionPrefix>
     <TargetFrameworks>netstandard2.0;net452</TargetFrameworks>
     <AssemblyName>Microsoft.Windows.PowerShell.ScriptAnalyzer</AssemblyName>
     <PackageId>Engine</PackageId>
     <RootNamespace>Microsoft.Windows.PowerShell.ScriptAnalyzer</RootNamespace> <!-- Namespace needs to match Assembly name for ressource binding -->
+    <GenerateDocumentationFile>true</GenerateDocumentationFile>
   </PropertyGroup>
 
   <ItemGroup Condition=" '$(TargetFramework)' == 'net452' ">

Engine/Formatter.cs

@@ -12,6 +12,59 @@ namespace Microsoft.Windows.PowerShell.ScriptAnalyzer
     /// </summary>
     public class Formatter
     {
+        /// <summary>
+        /// Format a powershell script.
+        /// </summary>
+        /// <param name="scriptDefinition">A string representing a powershell script.</param>
+        /// <param name="settings">Settings to be used for formatting</param>
+        /// <param name="range">The range in which formatting should take place.</param>
+        /// <param name="runspace">The runspace entrance into the powershell engine.</param>
+        /// <param name="writer">The writer for operation message.</param>
+        /// <returns></returns>
+        public static string Format(
+            string scriptDefinition,
+            Settings settings,
+            Range range,
+            System.Management.Automation.Runspaces.Runspace runspace,
+            IOutputWriter writer)
+        {
+            // todo implement notnull attribute for such a check
+            ValidateNotNull(scriptDefinition, "scriptDefinition");
+            ValidateNotNull(settings, "settings");
+
+            Helper.Instance = new Helper(runspace.SessionStateProxy.InvokeCommand, writer);
+            Helper.Instance.Initialize();
+
+            var ruleOrder = new string[]
+            {
+                "PSPlaceCloseBrace",
+                "PSPlaceOpenBrace",
+                "PSUseConsistentWhitespace",
+                "PSUseConsistentIndentation",
+                "PSAlignAssignmentStatement",
+                "PSUseCorrectCasing"
+            };
+
+            var text = new EditableText(scriptDefinition);
+            foreach (var rule in ruleOrder)
+            {
+                if (!settings.RuleArguments.ContainsKey(rule))
+                {
+                    continue;
+                }
+
+                var currentSettings = GetCurrentSettings(settings, rule);
+                ScriptAnalyzer.Instance.UpdateSettings(currentSettings);
+                ScriptAnalyzer.Instance.Initialize(runspace, writer, null, null, null, null, true, false, null, false);
+
+                Range updatedRange;
+                bool fixesWereApplied;
+                text = ScriptAnalyzer.Instance.Fix(text, range, out updatedRange, out fixesWereApplied);
+                range = updatedRange;
+            }
+
+            return text.ToString();
+        }
         /// <summary>
         /// Format a powershell script.
         /// </summary>

Engine/Helper.cs

@@ -19,7 +19,7 @@ namespace Microsoft.Windows.PowerShell.ScriptAnalyzer
     /// <summary>
     /// This Helper class contains utility/helper functions for classes in ScriptAnalyzer.
     /// </summary>
-    public class Helper
+    public class Helper : IDisposable
     {
         #region Private members
 
@@ -67,6 +67,28 @@ internal set
 
         #endregion
 
+        private bool disposed = false;
+        public void Dispose()
+        {
+            Dispose(true);
+            GC.SuppressFinalize(this);
+        }
+
+        protected virtual void Dispose(bool disposing)
+        {
+            if ( disposed )
+            {
+                return;
+            }
+
+            if ( disposing )
+            {
+                CommandInfoCache.Dispose();
+            }
+
+            disposed = true;
+        }
+
         #region Properties
 
         /// <summary>