Finish the XML documentation in WixToolset.Mba.Core.

4623

14 files changed, 555 insertions, 624 deletions

diff --git a/src/api/burn/WixToolset.Mba.Core/BaseBootstrapperApplicationFactory.cs b/src/api/burn/WixToolset.Mba.Core/BaseBootstrapperApplicationFactory.cs
index b215681e..339bfd8f 100644
--- a/src/api/burn/WixToolset.Mba.Core/BaseBootstrapperApplicationFactory.cs
+++ b/src/api/burn/WixToolset.Mba.Core/BaseBootstrapperApplicationFactory.cs
@@ -13,8 +13,8 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Default implementation of <see cref="IBootstrapperApplicationFactory.Create(IntPtr, IntPtr)"/>
         /// </summary>
-        /// <param name="pArgs"></param>
-        /// <param name="pResults"></param>
+        /// <param name="pArgs">The args struct given by the engine when initially creating the BA.</param>
+        /// <param name="pResults">The results struct given by the engine when initially creating the BA</param>
         public void Create(IntPtr pArgs, IntPtr pResults)
         {
             InitializeFromCreateArgs(pArgs, out var engine, out var bootstrapperCommand);
diff --git a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs
index b23cb3b2..a0ec6ab9 100644
--- a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs
+++ b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplication.cs
@@ -798,7 +798,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheBegin(CacheBeginEventArgs args)
         {
             EventHandler<CacheBeginEventArgs> handler = this.CacheBegin;
@@ -811,7 +811,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePackageBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePackageBegin(CachePackageBeginEventArgs args)
         {
             EventHandler<CachePackageBeginEventArgs> handler = this.CachePackageBegin;
@@ -824,7 +824,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheAcquireBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheAcquireBegin(CacheAcquireBeginEventArgs args)
         {
             EventHandler<CacheAcquireBeginEventArgs> handler = this.CacheAcquireBegin;
@@ -837,7 +837,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheAcquireProgress"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheAcquireProgress(CacheAcquireProgressEventArgs args)
         {
             EventHandler<CacheAcquireProgressEventArgs> handler = this.CacheAcquireProgress;
@@ -863,7 +863,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheAcquireComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheAcquireComplete(CacheAcquireCompleteEventArgs args)
         {
             EventHandler<CacheAcquireCompleteEventArgs> handler = this.CacheAcquireComplete;
@@ -876,7 +876,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheVerifyBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheVerifyBegin(CacheVerifyBeginEventArgs args)
         {
             EventHandler<CacheVerifyBeginEventArgs> handler = this.CacheVerifyBegin;
@@ -889,7 +889,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheVerifyProgress"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheVerifyProgress(CacheVerifyProgressEventArgs args)
         {
             EventHandler<CacheVerifyProgressEventArgs> handler = this.CacheVerifyProgress;
@@ -902,7 +902,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheVerifyComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheVerifyComplete(CacheVerifyCompleteEventArgs args)
         {
             EventHandler<CacheVerifyCompleteEventArgs> handler = this.CacheVerifyComplete;
@@ -915,7 +915,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePackageComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePackageComplete(CachePackageCompleteEventArgs args)
         {
             EventHandler<CachePackageCompleteEventArgs> handler = this.CachePackageComplete;
@@ -1227,6 +1227,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="PlanForwardCompatibleBundle"/> event.
         /// </summary>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnPlanForwardCompatibleBundle(PlanForwardCompatibleBundleEventArgs args)
         {
             EventHandler<PlanForwardCompatibleBundleEventArgs> handler = this.PlanForwardCompatibleBundle;
@@ -1239,7 +1240,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheContainerOrPayloadVerifyBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheContainerOrPayloadVerifyBegin(CacheContainerOrPayloadVerifyBeginEventArgs args)
         {
             EventHandler<CacheContainerOrPayloadVerifyBeginEventArgs> handler = this.CacheContainerOrPayloadVerifyBegin;
@@ -1252,7 +1253,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheContainerOrPayloadVerifyProgress"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheContainerOrPayloadVerifyProgress(CacheContainerOrPayloadVerifyProgressEventArgs args)
         {
             EventHandler<CacheContainerOrPayloadVerifyProgressEventArgs> handler = this.CacheContainerOrPayloadVerifyProgress;
@@ -1265,7 +1266,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CacheContainerOrPayloadVerifyComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCacheContainerOrPayloadVerifyComplete(CacheContainerOrPayloadVerifyCompleteEventArgs args)
         {
             EventHandler<CacheContainerOrPayloadVerifyCompleteEventArgs> handler = this.CacheContainerOrPayloadVerifyComplete;
@@ -1278,7 +1279,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePayloadExtractBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePayloadExtractBegin(CachePayloadExtractBeginEventArgs args)
         {
             EventHandler<CachePayloadExtractBeginEventArgs> handler = this.CachePayloadExtractBegin;
@@ -1291,7 +1292,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePayloadExtractProgress"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePayloadExtractProgress(CachePayloadExtractProgressEventArgs args)
         {
             EventHandler<CachePayloadExtractProgressEventArgs> handler = this.CachePayloadExtractProgress;
@@ -1304,7 +1305,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePayloadExtractComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePayloadExtractComplete(CachePayloadExtractCompleteEventArgs args)
         {
             EventHandler<CachePayloadExtractCompleteEventArgs> handler = this.CachePayloadExtractComplete;
@@ -1317,7 +1318,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="SetUpdateBegin"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnSetUpdateBegin(SetUpdateBeginEventArgs args)
         {
             EventHandler<SetUpdateBeginEventArgs> handler = this.SetUpdateBegin;
@@ -1330,7 +1331,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="SetUpdateComplete"/> event.
         /// </summary>
-        /// <param name="args"></param>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnSetUpdateComplete(SetUpdateCompleteEventArgs args)
         {
             EventHandler<SetUpdateCompleteEventArgs> handler = this.SetUpdateComplete;
@@ -1382,6 +1383,7 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Called by the engine, raises the <see cref="CachePackageNonVitalValidationFailure"/> event.
         /// </summary>
+        /// <param name="args">Additional arguments for this event.</param>
         protected virtual void OnCachePackageNonVitalValidationFailure(CachePackageNonVitalValidationFailureEventArgs args)
         {
             EventHandler<CachePackageNonVitalValidationFailureEventArgs> handler = this.CachePackageNonVitalValidationFailure;
diff --git a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplicationData.cs b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplicationData.cs
index 739a08bb..13428493 100644
--- a/src/api/burn/WixToolset.Mba.Core/BootstrapperApplicationData.cs
+++ b/src/api/burn/WixToolset.Mba.Core/BootstrapperApplicationData.cs
@@ -12,12 +12,12 @@ namespace WixToolset.Mba.Core
     public class BootstrapperApplicationData : IBootstrapperApplicationData
     {
         /// <summary>
-        /// 
+        /// The default file name for BootstrapperApplicationData.
         /// </summary>
         public const string DefaultFileName = "BootstrapperApplicationData.xml";
 
         /// <summary>
-        /// 
+        /// The XML namespace for BootstrapperApplicationData.
         /// </summary>
         public const string XMLNamespace = "http://wixtoolset.org/schemas/v4/BootstrapperApplicationData";
 
diff --git a/src/api/burn/WixToolset.Mba.Core/BootstrapperCommand.cs b/src/api/burn/WixToolset.Mba.Core/BootstrapperCommand.cs
index 88a9b9bb..ed1dc191 100644
--- a/src/api/burn/WixToolset.Mba.Core/BootstrapperCommand.cs
+++ b/src/api/burn/WixToolset.Mba.Core/BootstrapperCommand.cs
@@ -13,19 +13,8 @@ namespace WixToolset.Mba.Core
     public sealed class BootstrapperCommand : IBootstrapperCommand
     {
         /// <summary>
-        /// 
+        /// See <see cref="IBootstrapperCommand"/>.
         /// </summary>
-        /// <param name="action"></param>
-        /// <param name="display"></param>
-        /// <param name="commandLine"></param>
-        /// <param name="cmdShow"></param>
-        /// <param name="resume"></param>
-        /// <param name="splashScreen"></param>
-        /// <param name="relation"></param>
-        /// <param name="passthrough"></param>
-        /// <param name="layoutDirectory"></param>
-        /// <param name="bootstrapperWorkingFolder"></param>
-        /// <param name="bootstrapperApplicationDataPath"></param>
         public BootstrapperCommand(
             LaunchAction action,
             Display display,
diff --git a/src/api/burn/WixToolset.Mba.Core/Engine.cs b/src/api/burn/WixToolset.Mba.Core/Engine.cs
index 3ff693c0..1120fb1c 100644
--- a/src/api/burn/WixToolset.Mba.Core/Engine.cs
+++ b/src/api/burn/WixToolset.Mba.Core/Engine.cs
@@ -6,21 +6,14 @@ namespace WixToolset.Mba.Core
     using System.ComponentModel;
     using System.Runtime.InteropServices;
     using System.Security;
-    using System.Text;
 
     /// <summary>
     /// Default implementation of <see cref="IEngine"/>.
     /// </summary>
     public sealed class Engine : IEngine
     {
-        private static readonly string normalizeVersionFormatString = "{0} must be less than or equal to " + UInt16.MaxValue;
-
         private IBootstrapperEngine engine;
 
-        /// <summary>
-        /// Creates a new instance of the <see cref="Engine"/> container class.
-        /// </summary>
-        /// <param name="engine">The <see cref="IBootstrapperEngine"/> to contain.</param>
         internal Engine(IBootstrapperEngine engine)
         {
             this.engine = engine;
@@ -352,81 +345,5 @@ namespace WixToolset.Mba.Core
         {
             this.engine.Quit(exitCode);
         }
-
-        /// <summary>
-        /// Utility method for converting a <see cref="Version"/> into a <see cref="long"/>.
-        /// </summary>
-        /// <param name="version"></param>
-        /// <returns></returns>
-        public static long VersionToLong(Version version)
-        {
-            // In Windows, each version component has a max value of 65535,
-            // so we truncate the version before shifting it, which will overflow if invalid.
-            long major = (long)(ushort)version.Major << 48;
-            long minor = (long)(ushort)version.Minor << 32;
-            long build = (long)(ushort)version.Build << 16;
-            long revision = (long)(ushort)version.Revision;
-
-            return major | minor | build | revision;
-        }
-
-        /// <summary>
-        /// Utility method for converting a <see cref="long"/> into a <see cref="Version"/>.
-        /// </summary>
-        /// <param name="version"></param>
-        /// <returns></returns>
-        public static Version LongToVersion(long version)
-        {
-            int major = (int)((version & ((long)0xffff << 48)) >> 48);
-            int minor = (int)((version & ((long)0xffff << 32)) >> 32);
-            int build = (int)((version & ((long)0xffff << 16)) >> 16);
-            int revision = (int)(version & 0xffff);
-
-            return new Version(major, minor, build, revision);
-        }
-
-        /// <summary>
-        /// Verifies that Version can be represented in a <see cref="long"/>.
-        /// If the Build or Revision fields are undefined, they are set to zero.
-        /// </summary>
-        public static Version NormalizeVersion(Version version)
-        {
-            if (version == null)
-            {
-                throw new ArgumentNullException("version");
-            }
-
-            int major = version.Major;
-            int minor = version.Minor;
-            int build = version.Build;
-            int revision = version.Revision;
-
-            if (major > UInt16.MaxValue)
-            {
-                throw new ArgumentOutOfRangeException("version", String.Format(normalizeVersionFormatString, "Major"));
-            }
-            if (minor > UInt16.MaxValue)
-            {
-                throw new ArgumentOutOfRangeException("version", String.Format(normalizeVersionFormatString, "Minor"));
-            }
-            if (build > UInt16.MaxValue)
-            {
-                throw new ArgumentOutOfRangeException("version", String.Format(normalizeVersionFormatString, "Build"));
-            }
-            if (build == -1)
-            {
-                build = 0;
-            }
-            if (revision > UInt16.MaxValue)
-            {
-                throw new ArgumentOutOfRangeException("version", String.Format(normalizeVersionFormatString, "Revision"));
-            }
-            if (revision == -1)
-            {
-                revision = 0;
-            }
-
-            return new Version(major, minor, build, revision);
-        }
     }
 }
diff --git a/src/api/burn/WixToolset.Mba.Core/EventArgs.cs b/src/api/burn/WixToolset.Mba.Core/EventArgs.cs
index 9d18c4eb..4b7f2245 100644
--- a/src/api/burn/WixToolset.Mba.Core/EventArgs.cs
+++ b/src/api/burn/WixToolset.Mba.Core/EventArgs.cs
@@ -14,7 +14,8 @@ namespace WixToolset.Mba.Core
     public abstract class HResultEventArgs : EventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="HResultEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public HResultEventArgs()
         {
@@ -33,7 +34,8 @@ namespace WixToolset.Mba.Core
     public abstract class CancellableHResultEventArgs : HResultEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="CancellableHResultEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public CancellableHResultEventArgs(bool cancelRecommendation)
         {
@@ -52,7 +54,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public abstract class ResultEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ResultEventArgs(Result recommendation, Result result)
         {
             this.Recommendation = recommendation;
@@ -77,9 +82,9 @@ namespace WixToolset.Mba.Core
     public abstract class StatusEventArgs : HResultEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="StatusEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
         public StatusEventArgs(int hrStatus)
         {
             this.Status = hrStatus;
@@ -96,7 +101,10 @@ namespace WixToolset.Mba.Core
     /// </summary>
     public abstract class ActionEventArgs<T> : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ActionEventArgs(int hrStatus, T recommendation, T action)
             : base(hrStatus)
         {
@@ -121,7 +129,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public abstract class CancellableActionEventArgs<T> : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CancellableActionEventArgs(bool cancelRecommendation, T recommendation, T action)
             : base(cancelRecommendation)
         {
@@ -146,7 +157,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public abstract class CacheProgressBaseEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheProgressBaseEventArgs(string packageOrContainerId, string payloadId, long progress, long total, int overallPercentage, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -184,13 +198,14 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when startup has begun.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.Startup"/>.
     /// </summary>
     [Serializable]
     public class StartupEventArgs : HResultEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="StartupEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public StartupEventArgs()
         {
@@ -198,13 +213,14 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when shutdown has begun.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.Shutdown"/>.
     /// </summary>
     [Serializable]
     public class ShutdownEventArgs : HResultEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="ShutdownEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public ShutdownEventArgs(BOOTSTRAPPER_SHUTDOWN_ACTION action)
         {
@@ -223,7 +239,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectBeginEventArgs(bool cached, RegistrationType registrationType, int packageCount, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -254,7 +273,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectForwardCompatibleBundleEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectForwardCompatibleBundleEventArgs(string bundleId, RelationType relationType, string bundleTag, bool perMachine, string version, bool missingFromCache, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -303,7 +325,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectUpdateBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectUpdateBeginEventArgs(string updateLocation, bool cancelRecommendation, bool skipRecommendation)
             : base(cancelRecommendation)
         {
@@ -328,7 +353,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectUpdateEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectUpdateEventArgs(string updateLocation, long size, string hash, UpdateHashType hashAlgorithm, string version, string title, string summary, string contentType, string content, bool cancelRecommendation, bool stopRecommendation)
             : base(cancelRecommendation)
         {
@@ -401,7 +429,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectUpdateCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectUpdateCompleteEventArgs(int hrStatus, bool ignoreRecommendation)
             : base(hrStatus)
         {
@@ -420,7 +451,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectRelatedBundleEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectRelatedBundleEventArgs(string productCode, RelationType relationType, string bundleTag, bool perMachine, string version, bool missingFromCache, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -469,7 +503,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectPackageBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectPackageBeginEventArgs(string packageId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -488,7 +525,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectCompatibleMsiPackageEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectCompatibleMsiPackageEventArgs(string packageId, string compatiblePackageId, string compatiblePackageVersion, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -519,7 +559,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectRelatedMsiPackageEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectRelatedMsiPackageEventArgs(string packageId, string upgradeCode, string productCode, bool perMachine, string version, RelatedOperation operation, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -568,12 +611,9 @@ namespace WixToolset.Mba.Core
     public class DetectPatchTargetEventArgs : CancellableHResultEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="packageId"></param>
-        /// <param name="productCode"></param>
-        /// <param name="state"></param>
-        /// <param name="cancelRecommendation"></param>
         public DetectPatchTargetEventArgs(string packageId, string productCode, PackageState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -603,7 +643,10 @@ namespace WixToolset.Mba.Core
     /// </summary>
     public class DetectMsiFeatureEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectMsiFeatureEventArgs(string packageId, string featureId, FeatureState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -629,12 +672,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments for <see cref="IDefaultBootstrapperApplication.DetectPackageComplete"/>.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.DetectPackageComplete"/>.
     /// </summary>
     [Serializable]
     public class DetectPackageCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectPackageCompleteEventArgs(string packageId, int hrStatus, PackageState state, bool cached)
             : base(hrStatus)
         {
@@ -660,16 +706,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when the detection phase has completed.
+    /// Event arguments used when the detection phase has completed.
     /// </summary>
     [Serializable]
     public class DetectCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="DetectCompleteEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
-        /// <param name="eligibleForCleanup"></param>
         public DetectCompleteEventArgs(int hrStatus, bool eligibleForCleanup)
             : base(hrStatus)
         {
@@ -688,7 +733,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanBeginEventArgs(int packageCount, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -707,7 +755,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanRelatedBundleEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanRelatedBundleEventArgs(string bundleId, RequestState recommendedState, RequestState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -738,7 +789,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanRelatedBundleTypeEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanRelatedBundleTypeEventArgs(string bundleId, RelatedBundlePlanType recommendedType, RelatedBundlePlanType type, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -769,7 +823,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanPackageBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanPackageBeginEventArgs(string packageId, PackageState currentState, bool cached, BOOTSTRAPPER_PACKAGE_CONDITION_RESULT installCondition, BOOTSTRAPPER_PACKAGE_CONDITION_RESULT repairCondition, RequestState recommendedState, BOOTSTRAPPER_CACHE_TYPE recommendedCacheType, RequestState state, BOOTSTRAPPER_CACHE_TYPE cacheType, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -836,7 +893,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanCompatibleMsiPackageBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanCompatibleMsiPackageBeginEventArgs(string packageId, string compatiblePackageId, string compatiblePackageVersion, bool recommendedRemove, bool requestRemove, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -879,7 +939,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanCompatibleMsiPackageCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanCompatibleMsiPackageCompleteEventArgs(string packageId, string compatiblePackageId, int hrStatus, bool requestedRemove)
             : base(hrStatus)
         {
@@ -910,7 +973,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanRollbackBoundaryEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanRollbackBoundaryEventArgs(string rollbackBoundaryId, bool recommendedTransaction, bool transaction, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -944,13 +1010,9 @@ namespace WixToolset.Mba.Core
     public class PlanPatchTargetEventArgs : CancellableHResultEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="packageId"></param>
-        /// <param name="productCode"></param>
-        /// <param name="recommendedState"></param>
-        /// <param name="state"></param>
-        /// <param name="cancelRecommendation"></param>
         public PlanPatchTargetEventArgs(string packageId, string productCode, RequestState recommendedState, RequestState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -987,7 +1049,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanMsiFeatureEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanMsiFeatureEventArgs(string packageId, string featureId, FeatureState recommendedState, FeatureState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1024,7 +1089,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanMsiPackageEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanMsiPackageEventArgs(string packageId, bool shouldExecute, ActionState action, BOOTSTRAPPER_MSI_FILE_VERSIONING recommendedFileVersioning, bool cancelRecommendation, BURN_MSI_PROPERTY actionMsiProperty, INSTALLUILEVEL uiLevel, bool disableExternalUiHandler, BOOTSTRAPPER_MSI_FILE_VERSIONING fileVersioning)
             : base(cancelRecommendation)
         {
@@ -1086,11 +1154,9 @@ namespace WixToolset.Mba.Core
     public class PlanPackageCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="packageId"></param>
-        /// <param name="hrStatus"></param>
-        /// <param name="requested"></param>
         public PlanPackageCompleteEventArgs(string packageId, int hrStatus, RequestState requested)
             : base(hrStatus)
         {
@@ -1115,7 +1181,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlannedCompatiblePackageEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlannedCompatiblePackageEventArgs(string packageId, string compatiblePackageId, bool remove)
         {
             this.PackageId = packageId;
@@ -1145,7 +1214,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlannedPackageEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlannedPackageEventArgs(string packageId, ActionState execute, ActionState rollback, bool cache, bool uncache)
         {
             this.PackageId = packageId;
@@ -1182,15 +1254,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when the engine has completed planning the installation.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.PlanComplete"/>.
     /// </summary>
     [Serializable]
     public class PlanCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="PlanCompleteEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
         public PlanCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1203,7 +1275,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanForwardCompatibleBundleEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanForwardCompatibleBundleEventArgs(string bundleId, RelationType relationType, string bundleTag, bool perMachine, string version, bool recommendedIgnoreBundle, bool cancelRecommendation, bool ignoreBundle)
             : base(cancelRecommendation)
         {
@@ -1258,7 +1333,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ApplyBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ApplyBeginEventArgs(int phaseCount, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1278,7 +1356,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ElevateBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ElevateBeginEventArgs(bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1286,15 +1367,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when the engine has completed starting the elevated process.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.ElevateComplete"/>.
     /// </summary>
     [Serializable]
     public class ElevateCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="ElevateCompleteEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
         public ElevateCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1307,7 +1388,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ProgressEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ProgressEventArgs(int progressPercentage, int overallPercentage, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1332,7 +1416,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ErrorEventArgs : ResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ErrorEventArgs(ErrorType errorType, string packageId, int errorCode, string errorMessage, int dwUIHint, string[] data, Result recommendation, Result result)
             : base(recommendation, result)
         {
@@ -1381,7 +1468,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class RegisterBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public RegisterBeginEventArgs(RegistrationType recommendedRegistrationType, bool cancelRecommendation, RegistrationType registrationType)
             : base(cancelRecommendation)
         {
@@ -1406,7 +1496,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class RegisterCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public RegisterCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1419,7 +1512,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class UnregisterBeginEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public UnregisterBeginEventArgs(RegistrationType recommendedRegistrationType, RegistrationType registrationType)
         {
             this.RecommendedRegistrationType = recommendedRegistrationType;
@@ -1443,7 +1539,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class UnregisterCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public UnregisterCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1456,7 +1555,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheBeginEventArgs(bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1469,7 +1571,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheAcquireBeginEventArgs : CancellableActionEventArgs<CacheOperation>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheAcquireBeginEventArgs(string packageOrContainerId, string payloadId, string source, string downloadUrl, string payloadContainerId, CacheOperation recommendation, CacheOperation action, bool cancelRecommendation)
             : base(cancelRecommendation, recommendation, action)
         {
@@ -1512,7 +1617,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheAcquireProgressEventArgs : CacheProgressBaseEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheAcquireProgressEventArgs(string packageOrContainerId, string payloadId, long progress, long total, int overallPercentage, bool cancelRecommendation)
             : base(packageOrContainerId, payloadId, progress, total, overallPercentage, cancelRecommendation)
         {
@@ -1525,7 +1633,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheAcquireCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_CACHEACQUIRECOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheAcquireCompleteEventArgs(string packageOrContainerId, string payloadId, int hrStatus, BOOTSTRAPPER_CACHEACQUIRECOMPLETE_ACTION recommendation, BOOTSTRAPPER_CACHEACQUIRECOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -1550,7 +1661,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheVerifyBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheVerifyBeginEventArgs(string packageOrContainerId, string payloadId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1575,7 +1689,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheVerifyProgressEventArgs : CacheProgressBaseEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheVerifyProgressEventArgs(string packageOrContainerId, string payloadId, long progress, long total, int overallPercentage, CacheVerifyStep verifyStep, bool cancelRecommendation)
             : base(packageOrContainerId, payloadId, progress, total, overallPercentage, cancelRecommendation)
         {
@@ -1594,7 +1711,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheVerifyCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_CACHEVERIFYCOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheVerifyCompleteEventArgs(string packageOrContainerId, string payloadId, int hrStatus, BOOTSTRAPPER_CACHEVERIFYCOMPLETE_ACTION recommendation, BOOTSTRAPPER_CACHEVERIFYCOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -1614,15 +1734,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used after the engine has cached the installation sources.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.CacheComplete"/>.
     /// </summary>
     [Serializable]
     public class CacheCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="CacheCompleteEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
         public CacheCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1635,7 +1755,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecuteBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecuteBeginEventArgs(int packageCount, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1654,7 +1777,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecutePackageBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecutePackageBeginEventArgs(string packageId, bool shouldExecute, ActionState action, INSTALLUILEVEL uiLevel, bool disableExternalUiHandler, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1697,7 +1823,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecutePatchTargetEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecutePatchTargetEventArgs(string packageId, string targetProductCode, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1722,7 +1851,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecuteMsiMessageEventArgs : ResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecuteMsiMessageEventArgs(string packageId, InstallMessage messageType, int dwUIHint, string message, string[] data, Result recommendation, Result result)
             : base(recommendation, result)
         {
@@ -1765,7 +1897,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecuteFilesInUseEventArgs : ResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecuteFilesInUseEventArgs(string packageId, string[] files, Result recommendation, FilesInUseType source, Result result)
             : base(recommendation, result)
         {
@@ -1792,12 +1927,14 @@ namespace WixToolset.Mba.Core
 
     /// <summary>
     /// Event arguments for <see cref="IDefaultBootstrapperApplication.ExecutePackageComplete"/>
-    /// Additional arguments used when the engine has completed installing a specific package.
     /// </summary>
     [Serializable]
     public class ExecutePackageCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_EXECUTEPACKAGECOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecutePackageCompleteEventArgs(string packageId, int hrStatus, ApplyRestart restart, BOOTSTRAPPER_EXECUTEPACKAGECOMPLETE_ACTION recommendation, BOOTSTRAPPER_EXECUTEPACKAGECOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -1817,15 +1954,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments used when the engine has completed installing packages.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.ExecuteComplete"/>.
     /// </summary>
     [Serializable]
     public class ExecuteCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        /// Creates a new instance of the <see cref="ExecuteCompleteEventArgs"/> class.
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus">The return code of the operation.</param>
         public ExecuteCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -1838,7 +1975,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ApplyCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_APPLYCOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ApplyCompleteEventArgs(int hrStatus, ApplyRestart restart, BOOTSTRAPPER_APPLYCOMPLETE_ACTION recommendation, BOOTSTRAPPER_APPLYCOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -1857,7 +1997,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ApplyDowngradeEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ApplyDowngradeEventArgs(int hrRecommendation, int hrStatus)
         {
             this.Recommendation = hrRecommendation;
@@ -1881,7 +2024,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheAcquireResolvingEventArgs : CancellableActionEventArgs<CacheResolveOperation>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheAcquireResolvingEventArgs(string packageOrContainerId, string payloadId, string[] searchPaths, bool foundLocal, int recommendedSearchPath, string downloadUrl, string payloadContainerId, CacheResolveOperation recommendation, int chosenSearchPath, CacheResolveOperation action, bool cancel)
             : base(cancel, recommendation, action)
         {
@@ -1942,7 +2088,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePackageBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePackageBeginEventArgs(string packageId, int cachePayloads, long packageCacheSize, bool vital, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -1979,7 +2128,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePackageCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_CACHEPACKAGECOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePackageCompleteEventArgs(string packageId, int hrStatus, BOOTSTRAPPER_CACHEPACKAGECOMPLETE_ACTION recommendation, BOOTSTRAPPER_CACHEPACKAGECOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -1998,7 +2150,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecuteProgressEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecuteProgressEventArgs(string packageId, int progressPercentage, int overallPercentage, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2024,12 +2179,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before it tries to launch the preapproved executable.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.LaunchApprovedExeBegin"/>.
     /// </summary>
     [Serializable]
     public class LaunchApprovedExeBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public LaunchApprovedExeBeginEventArgs(bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2037,12 +2195,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after it finished trying to launch the preapproved executable.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.LaunchApprovedExeComplete"/>.
     /// </summary>
     [Serializable]
     public class LaunchApprovedExeCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public LaunchApprovedExeCompleteEventArgs(int hrStatus, int processId)
             : base(hrStatus)
         {
@@ -2057,12 +2218,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before beginning an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.BeginMsiTransactionBegin"/>.
     /// </summary>
     [Serializable]
     public class BeginMsiTransactionBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public BeginMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2076,12 +2240,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after beginning an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.BeginMsiTransactionComplete"/>.
     /// </summary>
     [Serializable]
     public class BeginMsiTransactionCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public BeginMsiTransactionCompleteEventArgs(string transactionId, int hrStatus)
             : base(hrStatus)
         {
@@ -2095,12 +2262,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before committing an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.CommitMsiTransactionBegin"/>.
     /// </summary>
     [Serializable]
     public class CommitMsiTransactionBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CommitMsiTransactionBeginEventArgs(string transactionId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2114,12 +2284,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after committing an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.CommitMsiTransactionComplete"/>.
     /// </summary>
     [Serializable]
     public class CommitMsiTransactionCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CommitMsiTransactionCompleteEventArgs(string transactionId, int hrStatus, ApplyRestart restart, BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION recommendation, BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -2139,12 +2312,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before rolling back an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.RollbackMsiTransactionBegin"/>.
     /// </summary>
     [Serializable]
     public class RollbackMsiTransactionBeginEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public RollbackMsiTransactionBeginEventArgs(string transactionId)
         {
             this.TransactionId = transactionId;
@@ -2157,12 +2333,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after rolling back an MSI transaction.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.RollbackMsiTransactionComplete"/>.
     /// </summary>
     [Serializable]
     public class RollbackMsiTransactionCompleteEventArgs : ActionEventArgs<BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public RollbackMsiTransactionCompleteEventArgs(string transactionId, int hrStatus, ApplyRestart restart, BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION recommendation, BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
@@ -2182,13 +2361,14 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before pausing Windows automatic updates.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.PauseAutomaticUpdatesBegin"/>.
     /// </summary>
     [Serializable]
     public class PauseAutomaticUpdatesBeginEventArgs : HResultEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public PauseAutomaticUpdatesBeginEventArgs()
         {
@@ -2196,15 +2376,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after pausing Windows automatic updates.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.PauseAutomaticUpdatesComplete"/>.
     /// </summary>
     [Serializable]
     public class PauseAutomaticUpdatesCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus"></param>
         public PauseAutomaticUpdatesCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -2212,13 +2392,14 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine before taking a system restore point.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.SystemRestorePointBegin"/>.
     /// </summary>
     [Serializable]
     public class SystemRestorePointBeginEventArgs : HResultEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
         public SystemRestorePointBeginEventArgs()
         {
@@ -2226,15 +2407,15 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// Additional arguments passed by the engine after taking a system restore point.
+    /// Event arguments for <see cref="IDefaultBootstrapperApplication.SystemRestorePointComplete"/>.
     /// </summary>
     [Serializable]
     public class SystemRestorePointCompleteEventArgs : StatusEventArgs
     {
         /// <summary>
-        ///
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
         /// </summary>
-        /// <param name="hrStatus"></param>
         public SystemRestorePointCompleteEventArgs(int hrStatus)
             : base(hrStatus)
         {
@@ -2247,7 +2428,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheContainerOrPayloadVerifyBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheContainerOrPayloadVerifyBeginEventArgs(string packageOrContainerId, string payloadId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2272,7 +2456,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheContainerOrPayloadVerifyProgressEventArgs : CacheProgressBaseEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheContainerOrPayloadVerifyProgressEventArgs(string packageOrContainerId, string payloadId, long progress, long total, int overallPercentage, bool cancelRecommendation)
             : base(packageOrContainerId, payloadId, progress, total, overallPercentage, cancelRecommendation)
         {
@@ -2285,7 +2472,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CacheContainerOrPayloadVerifyCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CacheContainerOrPayloadVerifyCompleteEventArgs(string packageOrContainerId, string payloadId, int hrStatus)
             : base(hrStatus)
         {
@@ -2310,7 +2500,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePayloadExtractBeginEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePayloadExtractBeginEventArgs(string containerId, string payloadId, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2335,7 +2528,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePayloadExtractProgressEventArgs : CacheProgressBaseEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePayloadExtractProgressEventArgs(string containerId, string payloadId, long progress, long total, int overallPercentage, bool cancelRecommendation)
             : base(containerId, payloadId, progress, total, overallPercentage, cancelRecommendation)
         {
@@ -2348,7 +2544,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePayloadExtractCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePayloadExtractCompleteEventArgs(string containerId, string payloadId, int hrStatus)
             : base(hrStatus)
         {
@@ -2373,7 +2572,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class SetUpdateBeginEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public SetUpdateBeginEventArgs()
         {
         }
@@ -2385,7 +2587,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class SetUpdateCompleteEventArgs : StatusEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public SetUpdateCompleteEventArgs(int hrStatus, string previousPackageId, string newPackageId)
             : base(hrStatus)
         {
@@ -2410,7 +2615,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class PlanRestoreRelatedBundleEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public PlanRestoreRelatedBundleEventArgs(string bundleId, RequestState recommendedState, RequestState state, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2441,7 +2649,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class ExecuteProcessCancelEventArgs : HResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public ExecuteProcessCancelEventArgs(string packageId, int processId, BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION recommendation, BOOTSTRAPPER_EXECUTEPROCESSCANCEL_ACTION action)
         {
             this.PackageId = packageId;
@@ -2477,7 +2688,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class DetectRelatedBundlePackageEventArgs : CancellableHResultEventArgs
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public DetectRelatedBundlePackageEventArgs(string packageId, string productCode, RelationType relationType, bool perMachine, string version, bool cancelRecommendation)
             : base(cancelRecommendation)
         {
@@ -2520,7 +2734,10 @@ namespace WixToolset.Mba.Core
     [Serializable]
     public class CachePackageNonVitalValidationFailureEventArgs : ActionEventArgs<BOOTSTRAPPER_CACHEPACKAGENONVITALVALIDATIONFAILURE_ACTION>
     {
-        /// <summary />
+        /// <summary>
+        /// This class is for events raised by the engine.
+        /// It is not intended to be instantiated by user code.
+        /// </summary>
         public CachePackageNonVitalValidationFailureEventArgs(string packageId, int hrStatus, BOOTSTRAPPER_CACHEPACKAGENONVITALVALIDATIONFAILURE_ACTION recommendation, BOOTSTRAPPER_CACHEPACKAGENONVITALVALIDATIONFAILURE_ACTION action)
             : base(hrStatus, recommendation, action)
         {
diff --git a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs
index 7bfabeb3..36c7fd33 100644
--- a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs
+++ b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplication.cs
@@ -65,14 +65,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectForwardCompatibleBundle"/>.
         /// </summary>
-        /// <param name="wzBundleId"></param>
-        /// <param name="relationType"></param>
-        /// <param name="wzBundleTag"></param>
-        /// <param name="fPerMachine"></param>
-        /// <param name="wzVersion"></param>
-        /// <param name="fMissingFromCache"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectForwardCompatibleBundle(
@@ -88,10 +80,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectUpdateBegin"/>.
         /// </summary>
-        /// <param name="wzUpdateLocation"></param>
-        /// <param name="fCancel"></param>
-        /// <param name="fSkip"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectUpdateBegin(
@@ -103,18 +91,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectUpdate"/>.
         /// </summary>
-        /// <param name="wzUpdateLocation"></param>
-        /// <param name="dw64Size"></param>
-        /// <param name="wzVersion"></param>
-        /// <param name="wzHash"></param>
-        /// <param name="hashAlgorithm"></param>
-        /// <param name="wzTitle"></param>
-        /// <param name="wzSummary"></param>
-        /// <param name="wzContentType"></param>
-        /// <param name="wzContent"></param>
-        /// <param name="fCancel"></param>
-        /// <param name="fStopProcessingUpdates"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectUpdate(
@@ -134,9 +110,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectUpdateComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <param name="fIgnoreError"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectUpdateComplete(
@@ -162,9 +135,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectPackageBegin"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectPackageBegin(
@@ -202,11 +172,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectPatchTarget"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="wzProductCode"></param>
-        /// <param name="patchState"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectPatchTarget(
@@ -219,11 +184,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectMsiFeature"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="wzFeatureId"></param>
-        /// <param name="state"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectMsiFeature(
@@ -248,9 +208,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.DetectComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <param name="fEligibleForCleanup"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnDetectComplete(
@@ -261,9 +218,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanBegin"/>.
         /// </summary>
-        /// <param name="cPackages"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanBegin(
@@ -342,12 +296,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanPatchTarget"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="wzProductCode"></param>
-        /// <param name="recommendedState"></param>
-        /// <param name="pRequestedState"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanPatchTarget(
@@ -361,12 +309,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanMsiFeature"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="wzFeatureId"></param>
-        /// <param name="recommendedState"></param>
-        /// <param name="pRequestedState"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanMsiFeature(
@@ -397,10 +339,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanPackageComplete"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="hrStatus"></param>
-        /// <param name="requested"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanPackageComplete(
@@ -436,8 +374,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanComplete(
@@ -447,9 +383,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ApplyBegin"/>.
         /// </summary>
-        /// <param name="dwPhaseCount"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnApplyBegin(
@@ -460,8 +393,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ElevateBegin"/>.
         /// </summary>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnElevateBegin(
@@ -471,8 +402,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ElevateComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnElevateComplete(
@@ -482,10 +411,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.Progress"/>.
         /// </summary>
-        /// <param name="dwProgressPercentage"></param>
-        /// <param name="dwOverallPercentage"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnProgress(
@@ -497,16 +422,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.Error"/>.
         /// </summary>
-        /// <param name="errorType"></param>
-        /// <param name="wzPackageId"></param>
-        /// <param name="dwCode"></param>
-        /// <param name="wzError"></param>
-        /// <param name="dwUIHint"></param>
-        /// <param name="cData"></param>
-        /// <param name="rgwzData"></param>
-        /// <param name="nRecommendation"></param>
-        /// <param name="pResult"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnError(
@@ -535,8 +450,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.RegisterComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnRegisterComplete(
@@ -546,8 +459,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.CacheBegin"/>.
         /// </summary>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnCacheBegin(
@@ -695,9 +606,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecuteBegin"/>.
         /// </summary>
-        /// <param name="cExecutingPackages"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecuteBegin(
@@ -708,13 +616,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecutePackageBegin"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="fExecute"></param>
-        /// <param name="action"></param>
-        /// <param name="uiLevel"></param>
-        /// <param name="fDisableExternalUiHandler"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecutePackageBegin(
@@ -729,10 +630,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecutePatchTarget"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="wzTargetProductCode"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecutePatchTarget(
@@ -744,11 +641,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecuteProgress"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="dwProgressPercentage"></param>
-        /// <param name="dwOverallPercentage"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecuteProgress(
@@ -761,15 +653,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecuteMsiMessage"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="messageType"></param>
-        /// <param name="dwUIHint"></param>
-        /// <param name="wzMessage"></param>
-        /// <param name="cData"></param>
-        /// <param name="rgwzData"></param>
-        /// <param name="nRecommendation"></param>
-        /// <param name="pResult"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecuteMsiMessage(
@@ -800,12 +683,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecutePackageComplete"/>.
         /// </summary>
-        /// <param name="wzPackageId"></param>
-        /// <param name="hrStatus"></param>
-        /// <param name="restart"></param>
-        /// <param name="recommendation"></param>
-        /// <param name="pAction"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecutePackageComplete(
@@ -819,8 +696,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ExecuteComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnExecuteComplete(
@@ -840,8 +715,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.UnregisterComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnUnregisterComplete(
@@ -851,11 +724,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.ApplyComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <param name="restart"></param>
-        /// <param name="recommendation"></param>
-        /// <param name="pAction"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnApplyComplete(
@@ -868,8 +736,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.LaunchApprovedExeBegin"/>.
         /// </summary>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnLaunchApprovedExeBegin(
@@ -879,9 +745,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.LaunchApprovedExeComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <param name="processId"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnLaunchApprovedExeComplete(
@@ -892,9 +755,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.BeginMsiTransactionBegin"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnBeginMsiTransactionBegin(
@@ -905,9 +765,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.BeginMsiTransactionComplete"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnBeginMsiTransactionComplete(
@@ -918,9 +775,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.CommitMsiTransactionBegin"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <param name="fCancel"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnCommitMsiTransactionBegin(
@@ -931,12 +785,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.CommitMsiTransactionComplete"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <param name="hrStatus"></param>
-        /// <param name="restart"></param>
-        /// <param name="recommendation"></param>
-        /// <param name="pAction"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnCommitMsiTransactionComplete(
@@ -950,8 +798,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.RollbackMsiTransactionBegin"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnRollbackMsiTransactionBegin(
@@ -961,12 +807,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.RollbackMsiTransactionComplete"/>.
         /// </summary>
-        /// <param name="wzTransactionId"></param>
-        /// <param name="hrStatus"></param>
-        /// <param name="restart"></param>
-        /// <param name="recommendation"></param>
-        /// <param name="pAction"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnRollbackMsiTransactionComplete(
@@ -980,7 +820,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PauseAutomaticUpdatesBegin"/>.
         /// </summary>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPauseAutomaticUpdatesBegin(
@@ -989,8 +828,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PauseAutomaticUpdatesComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPauseAutomaticUpdatesComplete(
@@ -1000,7 +837,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.SystemRestorePointBegin"/>.
         /// </summary>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnSystemRestorePointBegin(
@@ -1009,8 +845,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.SystemRestorePointComplete"/>.
         /// </summary>
-        /// <param name="hrStatus"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnSystemRestorePointComplete(
@@ -1020,15 +854,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IDefaultBootstrapperApplication.PlanForwardCompatibleBundle"/>.
         /// </summary>
-        /// <param name="wzBundleId"></param>
-        /// <param name="relationType"></param>
-        /// <param name="wzBundleTag"></param>
-        /// <param name="fPerMachine"></param>
-        /// <param name="wzVersion"></param>
-        /// <param name="fRecommendedIgnoreBundle"></param>
-        /// <param name="fCancel"></param>
-        /// <param name="fIgnoreBundle"></param>
-        /// <returns></returns>
         [PreserveSig]
         [return: MarshalAs(UnmanagedType.I4)]
         int OnPlanForwardCompatibleBundle(
@@ -1211,27 +1036,27 @@ namespace WixToolset.Mba.Core
     public enum Display
     {
         /// <summary>
-        ///
+        /// Invalid value.
         /// </summary>
         Unknown,
 
         /// <summary>
-        ///
+        /// The bundle is being run through the Burn protocol by an external application.
         /// </summary>
         Embedded,
 
         /// <summary>
-        ///
+        /// No UI should be shown.
         /// </summary>
         None,
 
         /// <summary>
-        ///
+        /// The UI should not require any interaction from the user.
         /// </summary>
         Passive,
 
         /// <summary>
-        ///
+        /// The UI should be fully interactive.
         /// </summary>
         Full,
     }
@@ -1343,27 +1168,27 @@ namespace WixToolset.Mba.Core
     public enum Restart
     {
         /// <summary>
-        ///
+        /// Invalid value.
         /// </summary>
         Unknown,
 
         /// <summary>
-        ///
+        /// The bundle should never initiate a restart.
         /// </summary>
         Never,
 
         /// <summary>
-        ///
+        /// The bundle should prompt the user whether to restart.
         /// </summary>
         Prompt,
 
         /// <summary>
-        ///
+        /// The bundle should restart if necessary.
         /// </summary>
         Automatic,
 
         /// <summary>
-        ///
+        /// The bundle should always restart when given the option.
         /// </summary>
         Always,
     }
@@ -1396,67 +1221,67 @@ namespace WixToolset.Mba.Core
     public enum Result
     {
         /// <summary>
-        ///
+        /// An error occurred.
         /// </summary>
         Error = -1,
 
         /// <summary>
-        ///
+        /// Invalid value.
         /// </summary>
         None,
 
         /// <summary>
-        ///
+        /// IDOK
         /// </summary>
         Ok,
 
         /// <summary>
-        ///
+        /// IDCANCEL
         /// </summary>
         Cancel,
 
         /// <summary>
-        ///
+        /// IDABORT
         /// </summary>
         Abort,
 
         /// <summary>
-        ///
+        /// IDRETRY
         /// </summary>
         Retry,
 
         /// <summary>
-        ///
+        /// IDIGNORE
         /// </summary>
         Ignore,
 
         /// <summary>
-        ///
+        /// IDYES
         /// </summary>
         Yes,
 
         /// <summary>
-        ///
+        /// IDNO
         /// </summary>
         No,
 
         /// <summary>
-        /// /
+        /// IDCLOSE
         /// </summary>
         Close,
 
         /// <summary>
-        ///
+        /// IDHELP
         /// </summary>
         Help,
 
         /// <summary>
-        ///
+        /// IDTRYAGAIN
         /// </summary>
         TryAgain,
 
         /// <summary>
-        ///
+        /// IDCONTINUE
         /// </summary>
         Continue,
     }
@@ -1467,7 +1292,7 @@ namespace WixToolset.Mba.Core
     public enum ResumeType
     {
         /// <summary>
-        ///
+        /// No resume information.
         /// </summary>
         None,
 
@@ -1553,12 +1378,12 @@ namespace WixToolset.Mba.Core
     }
 
     /// <summary>
-    /// The calculated operation for the related bundle.
+    /// The calculated operation for the related MSI package.
     /// </summary>
     public enum RelatedOperation
     {
         /// <summary>
-        ///
+        /// No relation.
         /// </summary>
         None,
 
@@ -1697,47 +1522,55 @@ namespace WixToolset.Mba.Core
     public enum RelationType
     {
         /// <summary>
-        ///
+        /// No relation.
         /// </summary>
         None,
 
         /// <summary>
-        ///
+        /// The related bundle is detected by the running bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         Detect,
 
         /// <summary>
-        ///
+        /// The related bundle shares an upgrade code with the running bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         Upgrade,
 
         /// <summary>
-        ///
+        /// The related bundle is an add-on for the running bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         Addon,
 
         /// <summary>
-        ///
+        /// The related bundle is a patch for the running bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         Patch,
 
         /// <summary>
-        ///
+        /// The running bundle is an add-on for the related bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         DependentAddon,
 
         /// <summary>
-        ///
+        /// The running bundle is a patch for the related bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         DependentPatch,
 
         /// <summary>
-        ///
+        /// The related bundle is a newer version of the running bundle.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         Update,
 
         /// <summary>
-        ///
+        /// The related bundle is in the running bundle's chain.
+        /// This relationship is reversed for <see cref="IBootstrapperCommand.Relation" />
         /// </summary>
         ChainPackage,
     }
@@ -1748,37 +1581,37 @@ namespace WixToolset.Mba.Core
     public enum RelatedBundlePlanType
     {
         /// <summary>
-        ///
+        /// Don't execute the related bundle.
         /// </summary>
         None,
 
         /// <summary>
-        ///
+        /// The running bundle is a downgrade for the related bundle.
         /// </summary>
         Downgrade,
 
         /// <summary>
-        ///
+        /// The running bundle is an upgrade for the related bundle.
         /// </summary>
         Upgrade,
 
         /// <summary>
-        ///
+        /// The related bundle is an add-on of the running bundle.
         /// </summary>
         Addon,
 
         /// <summary>
-        ///
+        /// The related bundle is a patch for the running bundle.
         /// </summary>
         Patch,
 
         /// <summary>
-        ///
+        /// The running bundle is an add-on for the related bundle.
         /// </summary>
         DependentAddon,
 
         /// <summary>
-        ///
+        /// The running bundle is a patch for the related bundle.
         /// </summary>
         DependentPatch,
     }
@@ -1816,7 +1649,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_APPLYCOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1855,7 +1688,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_CACHEACQUIRECOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1872,7 +1705,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_CACHEPACKAGECOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1895,7 +1728,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_CACHEPACKAGENONVITALVALIDATIONFAILURE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1912,7 +1745,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_CACHEVERIFYCOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1933,7 +1766,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_EXECUTEPACKAGECOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -1967,7 +1800,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_EXECUTEMSITRANSACTIONCOMPLETE_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
@@ -2046,7 +1879,7 @@ namespace WixToolset.Mba.Core
     public enum BOOTSTRAPPER_SHUTDOWN_ACTION
     {
         /// <summary>
-        ///
+        /// Instructs the engine to not take any special action.
         /// </summary>
         None,
 
diff --git a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplicationFactory.cs b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplicationFactory.cs
index 64e25ff4..2e84de67 100644
--- a/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplicationFactory.cs
+++ b/src/api/burn/WixToolset.Mba.Core/IBootstrapperApplicationFactory.cs
@@ -7,7 +7,7 @@ namespace WixToolset.Mba.Core
     using System.Runtime.InteropServices;
 
     /// <summary>
-    /// Interface used by WixToolset.Mba.Host to dynamically load the BA.
+    /// Interface used by the native host to dynamically load the BA.
     /// </summary>
     [ComVisible(true)]
     [InterfaceType(ComInterfaceType.InterfaceIsIUnknown)]
diff --git a/src/api/burn/WixToolset.Mba.Core/IBootstrapperEngine.cs b/src/api/burn/WixToolset.Mba.Core/IBootstrapperEngine.cs
index eca8dda1..25ce52df 100644
--- a/src/api/burn/WixToolset.Mba.Core/IBootstrapperEngine.cs
+++ b/src/api/burn/WixToolset.Mba.Core/IBootstrapperEngine.cs
@@ -19,7 +19,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.PackageCount"/>.
         /// </summary>
-        /// <param name="pcPackages"></param>
         void GetPackageCount(
             [MarshalAs(UnmanagedType.U4)] out int pcPackages
             );
@@ -27,9 +26,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.GetVariableNumeric(string)"/>.
         /// </summary>
-        /// <param name="wzVariable"></param>
-        /// <param name="pllValue"></param>
-        /// <returns></returns>
         [PreserveSig]
         int GetVariableNumeric(
             [MarshalAs(UnmanagedType.LPWStr)] string wzVariable,
@@ -79,8 +75,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.EvaluateCondition(string)"/>.
         /// </summary>
-        /// <param name="wzCondition"></param>
-        /// <param name="pf"></param>
         void EvaluateCondition(
             [MarshalAs(UnmanagedType.LPWStr)] string wzCondition,
             [MarshalAs(UnmanagedType.Bool)] out bool pf
@@ -89,8 +83,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Log(LogLevel, string)"/>.
         /// </summary>
-        /// <param name="level"></param>
-        /// <param name="wzMessage"></param>
         void Log(
             [MarshalAs(UnmanagedType.U4)] LogLevel level,
             [MarshalAs(UnmanagedType.LPWStr)] string wzMessage
@@ -99,10 +91,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SendEmbeddedError(int, string, int)"/>.
         /// </summary>
-        /// <param name="dwErrorCode"></param>
-        /// <param name="wzMessage"></param>
-        /// <param name="dwUIHint"></param>
-        /// <param name="pnResult"></param>
         void SendEmbeddedError(
             [MarshalAs(UnmanagedType.U4)] int dwErrorCode,
             [MarshalAs(UnmanagedType.LPWStr)] string wzMessage,
@@ -113,9 +101,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SendEmbeddedProgress(int, int)"/>.
         /// </summary>
-        /// <param name="dwProgressPercentage"></param>
-        /// <param name="dwOverallProgressPercentage"></param>
-        /// <param name="pnResult"></param>
         void SendEmbeddedProgress(
             [MarshalAs(UnmanagedType.U4)] int dwProgressPercentage,
             [MarshalAs(UnmanagedType.U4)] int dwOverallProgressPercentage,
@@ -125,11 +110,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetUpdate(string, string, long, UpdateHashType, string)"/>.
         /// </summary>
-        /// <param name="wzLocalSource"></param>
-        /// <param name="wzDownloadSource"></param>
-        /// <param name="qwValue"></param>
-        /// <param name="hashType"></param>
-        /// <param name="wzHash"></param>
         void SetUpdate(
             [MarshalAs(UnmanagedType.LPWStr)] string wzLocalSource,
             [MarshalAs(UnmanagedType.LPWStr)] string wzDownloadSource,
@@ -141,9 +121,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetLocalSource(string, string, string)"/>.
         /// </summary>
-        /// <param name="wzPackageOrContainerId"></param>
-        /// <param name="wzPayloadId"></param>
-        /// <param name="wzPath"></param>
         void SetLocalSource(
             [MarshalAs(UnmanagedType.LPWStr)] string wzPackageOrContainerId,
             [MarshalAs(UnmanagedType.LPWStr)] string wzPayloadId,
@@ -153,11 +130,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetDownloadSource(string, string, string, string, string)"/>.
         /// </summary>
-        /// <param name="wzPackageOrContainerId"></param>
-        /// <param name="wzPayloadId"></param>
-        /// <param name="wzUrl"></param>
-        /// <param name="wzUser"></param>
-        /// <param name="wzPassword"></param>
         void SetDownloadSource(
             [MarshalAs(UnmanagedType.LPWStr)] string wzPackageOrContainerId,
             [MarshalAs(UnmanagedType.LPWStr)] string wzPayloadId,
@@ -169,8 +141,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetVariableNumeric(string, long)"/>.
         /// </summary>
-        /// <param name="wzVariable"></param>
-        /// <param name="llValue"></param>
         void SetVariableNumeric(
             [MarshalAs(UnmanagedType.LPWStr)] string wzVariable,
             long llValue
@@ -179,9 +149,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetVariableString(string, string, bool)"/>.
         /// </summary>
-        /// <param name="wzVariable"></param>
-        /// <param name="wzValue"></param>
-        /// <param name="fFormatted"></param>
         void SetVariableString(
             [MarshalAs(UnmanagedType.LPWStr)] string wzVariable,
                                               IntPtr wzValue,
@@ -191,8 +158,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.SetVariableVersion(string, string)"/>.
         /// </summary>
-        /// <param name="wzVariable"></param>
-        /// <param name="wzValue"></param>
         void SetVariableVersion(
             [MarshalAs(UnmanagedType.LPWStr)] string wzVariable,
                                               IntPtr wzValue
@@ -206,7 +171,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Detect(IntPtr)"/>.
         /// </summary>
-        /// <param name="hwndParent"></param>
         void Detect(
             IntPtr hwndParent
             );
@@ -214,7 +178,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Plan(LaunchAction)"/>.
         /// </summary>
-        /// <param name="action"></param>
         void Plan(
             [MarshalAs(UnmanagedType.U4)] LaunchAction action
             );
@@ -222,8 +185,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Elevate(IntPtr)"/>.
         /// </summary>
-        /// <param name="hwndParent"></param>
-        /// <returns></returns>
         [PreserveSig]
         int Elevate(
             IntPtr hwndParent
@@ -232,7 +193,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Apply(IntPtr)"/>.
         /// </summary>
-        /// <param name="hwndParent"></param>
         void Apply(
             IntPtr hwndParent
             );
@@ -240,7 +200,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.Quit(int)"/>.
         /// </summary>
-        /// <param name="dwExitCode"></param>
         void Quit(
             [MarshalAs(UnmanagedType.U4)] int dwExitCode
             );
@@ -248,10 +207,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.LaunchApprovedExe(IntPtr, string, string, int)"/>.
         /// </summary>
-        /// <param name="hwndParent"></param>
-        /// <param name="wzApprovedExeForElevationId"></param>
-        /// <param name="wzArguments"></param>
-        /// <param name="dwWaitForInputIdleTimeout"></param>
         void LaunchApprovedExe(
             IntPtr hwndParent,
             [MarshalAs(UnmanagedType.LPWStr)] string wzApprovedExeForElevationId,
@@ -262,7 +217,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// Sets the URL to the update feed.
         /// </summary>
-        /// <param name="url">URL of the update feed.</param>
         void SetUpdateSource(
             [MarshalAs(UnmanagedType.LPWStr)] string url
             );
@@ -270,9 +224,6 @@ namespace WixToolset.Mba.Core
         /// <summary>
         /// See <see cref="IEngine.CompareVersions(string, string)"/>.
         /// </summary>
-        /// <param name="wzVersion1"></param>
-        /// <param name="wzVersion2"></param>
-        /// <param name="pnResult"></param>
         void CompareVersions(
             [MarshalAs(UnmanagedType.LPWStr)] string wzVersion1,
             [MarshalAs(UnmanagedType.LPWStr)] string wzVersion2,
@@ -297,93 +248,94 @@ namespace WixToolset.Mba.Core
     public enum ActionState
     {
         /// <summary>
-        /// 
+        /// No action.
         /// </summary>
         None,
 
         /// <summary>
-        /// 
+        /// Uninstall action.
         /// </summary>
         Uninstall,
 
         /// <summary>
-        /// 
+        /// Install action.
         /// </summary>
         Install,
 
         /// <summary>
-        /// 
+        /// Modify action.
         /// </summary>
         Modify,
 
         /// <summary>
-        /// 
+        /// Repair action.
         /// </summary>
         Repair,
 
         /// <summary>
-        /// 
+        /// Minor upgrade action.
         /// </summary>
         MinorUpgrade,
     }
 
     /// <summary>
-    /// The action for the BA to perform.
+    /// The action for the bundle to perform.
     /// </summary>
     public enum LaunchAction
     {
         /// <summary>
-        /// 
+        /// Invalid action.
         /// </summary>
         Unknown,
 
         /// <summary>
-        /// 
+        /// Provide help information.
         /// </summary>
         Help,
 
         /// <summary>
-        /// 
+        /// Layout the bundle on disk, normally to prepare for offline installation.
         /// </summary>
         Layout,
 
         /// <summary>
-        /// 
+        /// Same as Uninstall, except it will always remove itself from the package cache and Add/Remove Programs.
+        /// This should only be used to remove corrupt bundles since it might not properly clean up its packages.
         /// </summary>
         UnsafeUninstall,
 
         /// <summary>
-        /// 
+        /// Uninstall the bundle.
         /// </summary>
         Uninstall,
 
         /// <summary>
-        /// 
+        /// Cache the bundle and its packages.
         /// </summary>
         Cache,
 
         /// <summary>
-        /// 
+        /// Install the bundle.
         /// </summary>
         Install,
 
         /// <summary>
-        /// 
+        /// Modify the bundle.
         /// </summary>
         Modify,
 
         /// <summary>
-        /// 
+        /// Repair the bundle
         /// </summary>
         Repair,
 
         /// <summary>
-        /// 
+        /// Launch the update registered with <see cref="IEngine.SetUpdate(string, string, long, UpdateHashType, string)"/> and then exit without waiting for it to complete.
         /// </summary>
         UpdateReplace,
 
         /// <summary>
-        /// 
+        /// Launch the update registered with <see cref="IEngine.SetUpdate(string, string, long, UpdateHashType, string)"/> as an embedded bundle.
         /// </summary>
         UpdateReplaceEmbedded,
     }
@@ -441,32 +393,32 @@ namespace WixToolset.Mba.Core
     public enum PackageState
     {
         /// <summary>
-        /// 
+        /// Invalid state.
         /// </summary>
         Unknown,
 
         /// <summary>
-        /// 
+        /// The package is not on the machine (except possibly MspPackage) and should not be installed.
         /// </summary>
         Obsolete,
 
         /// <summary>
-        /// 
+        /// The package is not installed.
         /// </summary>
         Absent,
 
         /// <summary>
-        /// 
+        /// The package is not installed but is in the package cache.
         /// </summary>
         Cached,
 
         /// <summary>
-        /// 
+        /// The package is installed.
         /// </summary>
         Present,
 
         /// <summary>
-        /// 
+        /// The package is on the machine but not active, so only uninstall operations are allowed.
         /// </summary>
         Superseded,
     }
@@ -477,68 +429,70 @@ namespace WixToolset.Mba.Core
     public enum RequestState
     {
         /// <summary>
-        /// 
+        /// No change requested.
         /// </summary>
         None,
 
         /// <summary>
-        /// /
+        /// As long as there are no dependents, the package will be uninstalled.
+        /// There are some packages that can't be uninstalled, such as an ExePackage without an UninstallCommand.
         /// </summary>
         ForceAbsent,
 
         /// <summary>
-        /// 
+        /// Request the package to not be installed on the machine.
         /// </summary>
         Absent,
 
         /// <summary>
-        /// 
+        /// Request the package to be cached and not be installed on the machine.
         /// </summary>
         Cache,
 
         /// <summary>
-        /// 
+        /// Request the package to be installed on the machine.
         /// </summary>
         Present,
 
         /// <summary>
-        /// 
+        /// Force the bundle to install the package.
         /// </summary>
         ForcePresent,
 
         /// <summary>
-        /// 
+        /// Request the package to be repaired.
         /// </summary>
         Repair,
     }
 
     /// <summary>
     /// Indicates the state of a feature.
+    /// See https://learn.microsoft.com/en-us/windows/win32/api/msi/nf-msi-msiqueryfeaturestatew.
     /// </summary>
     public enum FeatureState
     {
         /// <summary>
-        /// 
+        /// Invalid state.
         /// </summary>
         Unknown,
 
         /// <summary>
-        /// 
+        /// INSTALLSTATE_ABSENT
         /// </summary>
         Absent,
 
         /// <summary>
-        /// 
+        /// INSTALLSTATE_ADVERTISED
         /// </summary>
         Advertised,
 
         /// <summary>
-        /// 
+        /// INSTALLSTATE_LOCAL
         /// </summary>
         Local,
 
         /// <summary>
-        /// 
+        /// INSTALLSTATE_SOURCE
         /// </summary>
         Source,
     }
diff --git a/src/api/burn/WixToolset.Mba.Core/IBundleInfo.cs b/src/api/burn/WixToolset.Mba.Core/IBundleInfo.cs
index 3227b72d..951f511a 100644
--- a/src/api/burn/WixToolset.Mba.Core/IBundleInfo.cs
+++ b/src/api/burn/WixToolset.Mba.Core/IBundleInfo.cs
@@ -10,27 +10,27 @@ namespace WixToolset.Mba.Core
     public interface IBundleInfo
     {
         /// <summary>
-        /// 
+        /// The name of the variable that contains the path to the bundle's log.
         /// </summary>
         string LogVariable { get; }
 
         /// <summary>
-        /// 
+        /// Bundle/@Name
         /// </summary>
         string Name { get; }
 
         /// <summary>
-        /// 
+        /// Variables that were marked with bal:Overridable="yes".
         /// </summary>
         IOverridableVariables OverridableVariables { get; }
 
         /// <summary>
-        /// 
+        /// The packages in the bundle's chain.
         /// </summary>
         IDictionary<string, IPackageInfo> Packages { get; }
 
         /// <summary>
-        /// 
+        /// Whether the bundle is per-machine or per-user.
         /// </summary>
         bool PerMachine { get; }
 
@@ -48,4 +48,4 @@ namespace WixToolset.Mba.Core
         /// <returns>The created <see cref="IPackageInfo"/>.</returns>
         IPackageInfo AddUpdateBundleAsPackage(SetUpdateCompleteEventArgs e);
     }
-}
\ No newline at end of file
+}
diff --git a/src/api/burn/WixToolset.Mba.Core/IPackageInfo.cs b/src/api/burn/WixToolset.Mba.Core/IPackageInfo.cs
index c6285f03..b1036143 100644
--- a/src/api/burn/WixToolset.Mba.Core/IPackageInfo.cs
+++ b/src/api/burn/WixToolset.Mba.Core/IPackageInfo.cs
@@ -8,7 +8,7 @@ namespace WixToolset.Mba.Core
     public interface IPackageInfo
     {
         /// <summary>
-        /// 
+        /// The authored cache strategy for this package.
         /// </summary>
         BOOTSTRAPPER_CACHE_TYPE CacheType { get; }
 
@@ -18,82 +18,82 @@ namespace WixToolset.Mba.Core
         object CustomData { get; set; }
 
         /// <summary>
-        /// 
+        /// The package's description.
         /// </summary>
         string Description { get; }
 
         /// <summary>
-        /// 
+        /// The authored bal:DisplayInternalUICondition.
         /// </summary>
         string DisplayInternalUICondition { get; }
 
         /// <summary>
-        /// 
+        /// The package's display name.
         /// </summary>
         string DisplayName { get; }
 
         /// <summary>
-        /// 
+        /// The package's Id.
         /// </summary>
         string Id { get; }
 
         /// <summary>
-        /// 
+        /// The authored InstallCondition.
         /// </summary>
         string InstallCondition { get; }
 
         /// <summary>
-        /// 
+        /// The authored RepairCondition.
         /// </summary>
         string RepairCondition { get; }
 
         /// <summary>
-        /// 
+        /// Whether the bundle should ever recommend the package to be uninstalled.
         /// </summary>
         bool Permanent { get; }
 
         /// <summary>
-        /// 
+        /// Whether the package should be installed by the prereq BA for managed bootstrapper applications.
         /// </summary>
         bool PrereqPackage { get; }
 
         /// <summary>
-        /// 
+        /// The file name of the license file to be shown by the prereq BA.
         /// </summary>
         string PrereqLicenseFile { get; }
 
         /// <summary>
-        /// 
+        /// The URL of the license to be shown by the prereq BA.
         /// </summary>
         string PrereqLicenseUrl { get; }
 
         /// <summary>
-        /// See <see cref="PrimaryPackageType"/>
+        /// See <see cref="WixToolset.Mba.Core.PrimaryPackageType"/>
         /// </summary>
         PrimaryPackageType PrimaryPackageType { get; }
 
         /// <summary>
-        /// 
+        /// The package's ProductCode.
         /// </summary>
         string ProductCode { get; }
 
         /// <summary>
-        /// 
+        /// The type of the package.
         /// </summary>
         PackageType Type { get; }
 
         /// <summary>
-        /// 
+        /// The package's UpgradeCode.
         /// </summary>
         string UpgradeCode { get; }
 
         /// <summary>
-        /// 
+        /// The package's version.
         /// </summary>
         string Version { get; }
 
         /// <summary>
-        /// 
+        /// Whether the package's failure can be ignored while executing the chain.
         /// </summary>
         bool Vital { get; }
     }
diff --git a/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs b/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs
index 3fa1b49e..c6373647 100644
--- a/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs
+++ b/src/api/burn/WixToolset.Mba.Core/PackageInfo.cs
@@ -8,57 +8,57 @@ namespace WixToolset.Mba.Core
     using System.Xml.XPath;
 
     /// <summary>
-    /// 
+    /// The type of package.
     /// </summary>
     public enum PackageType
     {
         /// <summary>
-        /// 
+        /// Invalid type.
         /// </summary>
         Unknown,
 
         /// <summary>
-        /// 
+        /// ExePackage
         /// </summary>
         Exe,
 
         /// <summary>
-        /// 
+        /// MsiPackage
         /// </summary>
         Msi,
 
         /// <summary>
-        /// 
+        /// MspPackage
         /// </summary>
         Msp,
 
         /// <summary>
-        /// 
+        /// MsuPackage
         /// </summary>
         Msu,
 
         /// <summary>
-        /// 
+        /// Related bundle of type Upgrade
         /// </summary>
         UpgradeBundle,
 
         /// <summary>
-        /// 
+        /// Related bundle of type Addon
         /// </summary>
         AddonBundle,
 
         /// <summary>
-        /// 
+        /// Related bundle of type Patch
         /// </summary>
         PatchBundle,
 
         /// <summary>
-        /// 
+        /// Related bundle of type Update
         /// </summary>
         UpdateBundle,
 
         /// <summary>
-        /// 
+        /// BundlePackage
         /// </summary>
         ChainBundle,
     }
@@ -156,10 +156,10 @@ namespace WixToolset.Mba.Core
         internal PackageInfo() { }
 
         /// <summary>
-        /// 
+        /// Parse packages from BootstrapperApplicationData.xml.
         /// </summary>
-        /// <param name="root"></param>
-        /// <returns></returns>
+        /// <param name="root">The root node.</param>
+        /// <returns>A dictionary of the packages by Id.</returns>
         public static IDictionary<string, IPackageInfo> ParsePackagesFromXml(XPathNavigator root)
         {
             var packagesById = new Dictionary<string, IPackageInfo>();
@@ -213,6 +213,13 @@ namespace WixToolset.Mba.Core
 
                 package.RepairCondition = BootstrapperApplicationData.GetAttribute(node, "RepairCondition");
 
+                BOOTSTRAPPER_CACHE_TYPE? cacheType = GetCacheTypeAttribute(node, "Cache");
+                if (!cacheType.HasValue)
+                {
+                    throw new Exception("Failed to get cache type for package.");
+                }
+                package.CacheType = cacheType.Value;
+
                 packagesById.Add(package.Id, package);
             }
 
@@ -221,11 +228,11 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Parse the cache type attribute.
         /// </summary>
-        /// <param name="node"></param>
-        /// <param name="attributeName"></param>
-        /// <returns></returns>
+        /// <param name="node">Package node</param>
+        /// <param name="attributeName">Attribute name</param>
+        /// <returns>The cache type</returns>
         public static BOOTSTRAPPER_CACHE_TYPE? GetCacheTypeAttribute(XPathNavigator node, string attributeName)
         {
             string attributeValue = BootstrapperApplicationData.GetAttribute(node, attributeName);
@@ -250,11 +257,11 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Parse the package type attribute
         /// </summary>
-        /// <param name="node"></param>
-        /// <param name="attributeName"></param>
-        /// <returns></returns>
+        /// <param name="node">Package node</param>
+        /// <param name="attributeName">Attribute name</param>
+        /// <returns>The package type</returns>
         public static PackageType? GetPackageTypeAttribute(XPathNavigator node, string attributeName)
         {
             string attributeValue = BootstrapperApplicationData.GetAttribute(node, attributeName);
@@ -291,13 +298,13 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Create <see cref="IPackageInfo"/> from a related bundle.
         /// </summary>
-        /// <param name="id"></param>
-        /// <param name="relationType"></param>
-        /// <param name="perMachine"></param>
-        /// <param name="version"></param>
-        /// <returns></returns>
+        /// <param name="id">Package id</param>
+        /// <param name="relationType">Relation type</param>
+        /// <param name="perMachine">Whether the related bundle is per-machine</param>
+        /// <param name="version">The related bundle's version</param>
+        /// <returns>The package info</returns>
         public static IPackageInfo GetRelatedBundleAsPackage(string id, RelationType relationType, bool perMachine, string version)
         {
             PackageInfo package = new PackageInfo();
@@ -323,10 +330,10 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Create <see cref="IPackageInfo"/> from an update bundle.
         /// </summary>
-        /// <param name="id"></param>
-        /// <returns></returns>
+        /// <param name="id">Package id</param>
+        /// <returns>The package info</returns>
         public static IPackageInfo GetUpdateBundleAsPackage(string id)
         {
             PackageInfo package = new PackageInfo();
diff --git a/src/api/burn/WixToolset.Mba.Core/VerUtil.cs b/src/api/burn/WixToolset.Mba.Core/VerUtil.cs
index 14552310..3573f9d3 100644
--- a/src/api/burn/WixToolset.Mba.Core/VerUtil.cs
+++ b/src/api/burn/WixToolset.Mba.Core/VerUtil.cs
@@ -110,10 +110,10 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Clone the version.
         /// </summary>
-        /// <param name="version"></param>
-        /// <returns></returns>
+        /// <param name="version">Source version</param>
+        /// <returns>Cloned version</returns>
         public static VerUtilVersion CopyVersion(VerUtilVersion version)
         {
             var handle = VerCopyVersion(version.GetHandle());
@@ -121,11 +121,11 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Parse a version.
         /// </summary>
-        /// <param name="version"></param>
+        /// <param name="version">Source version</param>
         /// <param name="strict">Whether to throw exception on invalid version.</param>
-        /// <returns></returns>
+        /// <returns>Parsed version</returns>
         public static VerUtilVersion ParseVersion(string version, bool strict)
         {
             var handle = VerParseVersion(version, 0, strict);
@@ -133,10 +133,10 @@ namespace WixToolset.Mba.Core
         }
 
         /// <summary>
-        /// 
+        /// Parse version from qword.
         /// </summary>
-        /// <param name="version"></param>
-        /// <returns></returns>
+        /// <param name="version">Source version</param>
+        /// <returns>Parsed version</returns>
         public static VerUtilVersion VersionFromQword(long version)
         {
             var handle = VerVersionFromQword(version);
diff --git a/src/api/burn/test/WixToolsetTest.Mba.Core/VerUtilFixture.cs b/src/api/burn/test/WixToolsetTest.Mba.Core/VerUtilFixture.cs
index 6fb8aced..9c213945 100644
--- a/src/api/burn/test/WixToolsetTest.Mba.Core/VerUtilFixture.cs
+++ b/src/api/burn/test/WixToolsetTest.Mba.Core/VerUtilFixture.cs
@@ -45,7 +45,7 @@ namespace WixToolsetTest.Mba.Core
         public void CanCreateFromQword()
         {
             var version = new Version(100, 200, 300, 400);
-            var qwVersion = Engine.VersionToLong(version);
+            var qwVersion = VersionToLong(version);
 
             using var parsedVersion = VerUtil.VersionFromQword(qwVersion);
             Assert.Equal("100.200.300.400", parsedVersion.Version);
@@ -99,5 +99,17 @@ namespace WixToolsetTest.Mba.Core
             Assert.True(parsedVersion.ReleaseLabels[4].IsNumeric);
             Assert.Equal(5u, parsedVersion.ReleaseLabels[4].Value);
         }
+
+        private static long VersionToLong(Version version)
+        {
+            // In Windows, each version component has a max value of 65535,
+            // so we truncate the version before shifting it, which will overflow if invalid.
+            long major = (long)(ushort)version.Major << 48;
+            long minor = (long)(ushort)version.Minor << 32;
+            long build = (long)(ushort)version.Build << 16;
+            long revision = (long)(ushort)version.Revision;
+
+            return major | minor | build | revision;
+        }
     }
 }