Enable XML doc and delete obsolete IParseHelper methods.

64 files changed, 300 insertions, 49 deletions

diff --git a/src/CSharp.Build.props b/src/CSharp.Build.props
index bcd47a0c..81d24ad1 100644
--- a/src/CSharp.Build.props
+++ b/src/CSharp.Build.props
@@ -8,5 +8,6 @@
     <CheckForOverflowUnderflow>true</CheckForOverflowUnderflow>
     <SignAssembly>true</SignAssembly>
     <AssemblyOriginatorKeyFile>$([System.IO.Path]::GetFullPath($(MSBuildThisFileDirectory)wix.snk))</AssemblyOriginatorKeyFile>
+    <NBGV_EmitThisAssemblyClass>false</NBGV_EmitThisAssemblyClass>
   </PropertyGroup>
 </Project>
diff --git a/src/Directory.Build.targets b/src/Directory.Build.targets
index dac7452a..cb988931 100644
--- a/src/Directory.Build.targets
+++ b/src/Directory.Build.targets
@@ -10,6 +10,11 @@
 -->
 <Project>
   <PropertyGroup>
+    <CreateDocumentation Condition=" '$(CreateDocumentationFile)'!='true' ">false</CreateDocumentation>
+    <DocumentationFile Condition=" '$(CreateDocumentationFile)'=='true' ">$(OutputPath)\$(AssemblyName).xml</DocumentationFile>
+  </PropertyGroup>
+
+  <PropertyGroup>
     <ReplacePackageReferences>true</ReplacePackageReferences>
     <TheSolutionPath Condition=" '$(NCrunch)'=='' ">$(SolutionPath)</TheSolutionPath>
     <TheSolutionPath Condition=" '$(NCrunch)'=='1' ">$(NCrunchOriginalSolutionPath)</TheSolutionPath>
@@ -45,4 +50,7 @@
 
     </When>
   </Choose>
+
+  <Import Project="Wix.Build.targets" Condition=" Exists('Wix.Build.targets') And '$(MSBuildProjectExtension)'=='.wixproj' " />
+  <Import Project="Custom.Build.targets" Condition=" Exists('Custom.Build.targets') " />
 </Project>
diff --git a/src/WixToolset.Extensibility/BaseBurnBackendExtension.cs b/src/WixToolset.Extensibility/BaseBurnBackendExtension.cs
index 99bf5e71..0575d725 100644
--- a/src/WixToolset.Extensibility/BaseBurnBackendExtension.cs
+++ b/src/WixToolset.Extensibility/BaseBurnBackendExtension.cs
@@ -9,6 +9,9 @@ namespace WixToolset.Extensibility
     using WixToolset.Extensibility.Data;
     using WixToolset.Extensibility.Services;
 
+    /// <summary>
+    /// Base class for creating a Burn backend extension.
+    /// </summary>
     public abstract class BaseBurnBackendExtension : IBurnBackendExtension
     {
         /// <summary>
@@ -31,14 +34,25 @@ namespace WixToolset.Extensibility
         /// </summary>
         protected virtual IEnumerable<IntermediateSymbolDefinition> SymbolDefinitions => Enumerable.Empty<IntermediateSymbolDefinition>();
 
+        /// <summary>
+        /// Called after all output changes occur and right before the output is bound into its final format.
+        /// </summary>
         public virtual void BundleFinalize()
         {
         }
 
+        /// <summary>
+        /// Called after output is bound into its final format.
+        /// </summary>
+        /// <param name="result"></param>
         public virtual void PostBackendBind(IBindResult result)
         {
         }
 
+        /// <summary>
+        /// Called before binding occurs.
+        /// </summary>
+        /// <param name="context"></param>
         public virtual void PreBackendBind(IBindContext context)
         {
             this.Context = context;
@@ -46,16 +60,44 @@ namespace WixToolset.Extensibility
             this.BackendHelper = context.ServiceProvider.GetService<IBurnBackendHelper>();
         }
 
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="relatedSource"></param>
+        /// <param name="type"></param>
+        /// <param name="sourceLineNumbers"></param>
+        /// <param name="bindStage"></param>
+        /// <returns></returns>
         public virtual IResolveFileResult ResolveRelatedFile(string source, string relatedSource, string type, SourceLineNumber sourceLineNumbers, BindStage bindStage)
         {
             return null;
         }
 
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="url"></param>
+        /// <param name="fallbackUrl"></param>
+        /// <param name="packageId"></param>
+        /// <param name="payloadId"></param>
+        /// <param name="fileName"></param>
+        /// <returns></returns>
         public virtual string ResolveUrl(string url, string fallbackUrl, string packageId, string payloadId, string fileName)
         {
             return null;
         }
 
+        /// <summary>
+        /// Called for each extension symbol that hasn't been handled yet.
+        /// Use IBurnBackendHelper to add data to the appropriate data manifest.
+        /// </summary>
+        /// <param name="section">The linked section.</param>
+        /// <param name="symbol">The current symbol.</param>
+        /// <returns>
+        /// True if the extension handled the symbol, false otherwise.
+        /// The Burn backend will warn on all unhandled symbols.
+        /// </returns>
         public virtual bool TryAddSymbolToDataManifest(IntermediateSection section, IntermediateSymbol symbol)
         {
             if (this.SymbolDefinitions.Any(t => t == symbol.Definition) &&
diff --git a/src/WixToolset.Extensibility/BaseCompilerExtension.cs b/src/WixToolset.Extensibility/BaseCompilerExtension.cs
index 51fbcd0b..2eac3706 100644
--- a/src/WixToolset.Extensibility/BaseCompilerExtension.cs
+++ b/src/WixToolset.Extensibility/BaseCompilerExtension.cs
@@ -52,34 +52,24 @@ namespace WixToolset.Extensibility
         }
 
         /// <summary>
-        /// Processes an attribute for the Compiler.
+        /// See <see cref="ICompilerExtension.ParseAttribute(Intermediate, IntermediateSection, XElement, XAttribute, IDictionary{string, string})"/>
         /// </summary>
-        /// <param name="parentElement">Parent element of attribute.</param>
-        /// <param name="attribute">Attribute to process.</param>
-        /// <param name="context">Extra information about the context in which this element is being parsed.</param>
         public virtual void ParseAttribute(Intermediate intermediate, IntermediateSection section, XElement parentElement, XAttribute attribute, IDictionary<string, string> context)
         {
             this.ParseHelper.UnexpectedAttribute(parentElement, attribute);
         }
 
         /// <summary>
-        /// Processes an element for the Compiler.
+        /// See <see cref="ICompilerExtension.ParseElement(Intermediate, IntermediateSection, XElement, XElement, IDictionary{string, string})"/>
         /// </summary>
-        /// <param name="parentElement">Parent element of element to process.</param>
-        /// <param name="element">Element to process.</param>
-        /// <param name="contextValues">Extra information about the context in which this element is being parsed.</param>
         public virtual void ParseElement(Intermediate intermediate, IntermediateSection section, XElement parentElement, XElement element, IDictionary<string, string> context)
         {
             this.ParseHelper.UnexpectedElement(parentElement, element);
         }
 
         /// <summary>
-        /// Processes an element for the Compiler, with the ability to supply a component keypath.
+        /// See <see cref="ICompilerExtension.ParsePossibleKeyPathElement(Intermediate, IntermediateSection, XElement, XElement, IDictionary{string, string})"/>
         /// </summary>
-        /// <param name="parentElement">Parent element of element to process.</param>
-        /// <param name="element">Element to process.</param>
-        /// <param name="keyPath">Explicit key path.</param>
-        /// <param name="contextValues">Extra information about the context in which this element is being parsed.</param>
         public virtual IComponentKeyPath ParsePossibleKeyPathElement(Intermediate intermediate, IntermediateSection section, XElement parentElement, XElement element, IDictionary<string, string> context)
         {
             this.ParseElement(intermediate, section, parentElement, element, context);
diff --git a/src/WixToolset.Extensibility/BaseExtensionCommandLine.cs b/src/WixToolset.Extensibility/BaseExtensionCommandLine.cs
index 5f2ca109..8a336e1b 100644
--- a/src/WixToolset.Extensibility/BaseExtensionCommandLine.cs
+++ b/src/WixToolset.Extensibility/BaseExtensionCommandLine.cs
@@ -7,23 +7,41 @@ namespace WixToolset.Extensibility
     using WixToolset.Extensibility.Data;
     using WixToolset.Extensibility.Services;
 
+    /// <summary>
+    /// Base class for extensions to be able to parse the command-line.
+    /// </summary>
     public abstract class BaseExtensionCommandLine : IExtensionCommandLine
     {
+        /// <summary>
+        /// See <see cref="IExtensionCommandLine.CommandLineSwitches" />
+        /// </summary>
         public virtual IEnumerable<ExtensionCommandLineSwitch> CommandLineSwitches => Enumerable.Empty<ExtensionCommandLineSwitch>();
 
+        /// <summary>
+        /// See <see cref="IExtensionCommandLine.PostParse" />
+        /// </summary>
         public virtual void PostParse()
         {
         }
 
+        /// <summary>
+        /// See <see cref="IExtensionCommandLine.PreParse" />
+        /// </summary>
         public virtual void PreParse(ICommandLineContext context)
         {
         }
 
+        /// <summary>
+        /// See <see cref="IExtensionCommandLine.TryParseArgument" />
+        /// </summary>
         public virtual bool TryParseArgument(ICommandLineParser parser, string argument)
         {
             return false;
         }
 
+        /// <summary>
+        /// See <see cref="IExtensionCommandLine.TryParseCommand" />
+        /// </summary>
         public virtual bool TryParseCommand(ICommandLineParser parser, string argument, out ICommandLineCommand command)
         {
             command = null;
diff --git a/src/WixToolset.Extensibility/BaseExtensionData.cs b/src/WixToolset.Extensibility/BaseExtensionData.cs
index 597a8e4d..e4a10fd9 100644
--- a/src/WixToolset.Extensibility/BaseExtensionData.cs
+++ b/src/WixToolset.Extensibility/BaseExtensionData.cs
@@ -9,13 +9,22 @@ namespace WixToolset.Extensibility
     /// </summary>
     public abstract class BaseExtensionData : IExtensionData
     {
+        /// <summary>
+        /// See <see cref="IExtensionData.DefaultCulture"/>
+        /// </summary>
         public virtual string DefaultCulture => null;
 
+        /// <summary>
+        /// See <see cref="IExtensionData.GetLibrary"/>
+        /// </summary>
         public virtual Intermediate GetLibrary(ISymbolDefinitionCreator symbolDefinitions)
         {
             return null;
         }
 
+        /// <summary>
+        /// See <see cref="IExtensionData.TryGetSymbolDefinitionByName"/>
+        /// </summary>
         public virtual bool TryGetSymbolDefinitionByName(string name, out IntermediateSymbolDefinition symbolDefinition)
         {
             symbolDefinition = null;
diff --git a/src/WixToolset.Extensibility/BaseExtensionFactory.cs b/src/WixToolset.Extensibility/BaseExtensionFactory.cs
index d0686b46..2e21b51b 100644
--- a/src/WixToolset.Extensibility/BaseExtensionFactory.cs
+++ b/src/WixToolset.Extensibility/BaseExtensionFactory.cs
@@ -12,8 +12,14 @@ namespace WixToolset.Extensibility
     /// </summary>
     public abstract class BaseExtensionFactory : IExtensionFactory
     {
+        /// <summary>
+        /// The extension types of the WiX extension.
+        /// </summary>
         protected abstract IEnumerable<Type> ExtensionTypes { get; }
 
+        /// <summary>
+        /// See <see cref="IExtensionFactory.TryCreateExtension(Type, out object)"/>
+        /// </summary>
         public virtual bool TryCreateExtension(Type extensionType, out object extension)
         {
             extension = null;
diff --git a/src/WixToolset.Extensibility/BaseLayoutExtension.cs b/src/WixToolset.Extensibility/BaseLayoutExtension.cs
index 78ec8d0a..21b932ff 100644
--- a/src/WixToolset.Extensibility/BaseLayoutExtension.cs
+++ b/src/WixToolset.Extensibility/BaseLayoutExtension.cs
@@ -30,11 +30,23 @@ namespace WixToolset.Extensibility
             this.Messaging = context.ServiceProvider.GetService<IMessaging>();
         }
 
+        /// <summary>
+        /// See <see cref="ILayoutExtension.CopyFile(string, string)"/>
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="destination"></param>
+        /// <returns></returns>
         public virtual bool CopyFile(string source, string destination)
         {
             return false;
         }
 
+        /// <summary>
+        /// See <see cref="ILayoutExtension.MoveFile(string, string)"/>
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="destination"></param>
+        /// <returns></returns>
         public virtual bool MoveFile(string source, string destination)
         {
             return false;
diff --git a/src/WixToolset.Extensibility/BasePreprocessorExtension.cs b/src/WixToolset.Extensibility/BasePreprocessorExtension.cs
index cfacf97a..b9a856ec 100644
--- a/src/WixToolset.Extensibility/BasePreprocessorExtension.cs
+++ b/src/WixToolset.Extensibility/BasePreprocessorExtension.cs
@@ -70,7 +70,6 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Processes a pragma defined in the extension.
         /// </summary>
-        /// <param name="sourceLineNumbers">The location of this pragma's PI.</param>
         /// <param name="prefix">The prefix of the pragma to be processed by the extension.</param>
         /// <param name="pragma">The name of the pragma.</param>
         /// <param name="args">The pragma's arguments.</param>
diff --git a/src/WixToolset.Extensibility/BaseResolverExtension.cs b/src/WixToolset.Extensibility/BaseResolverExtension.cs
index ede903d5..72dc5c41 100644
--- a/src/WixToolset.Extensibility/BaseResolverExtension.cs
+++ b/src/WixToolset.Extensibility/BaseResolverExtension.cs
@@ -36,6 +36,14 @@ namespace WixToolset.Extensibility
             this.Messaging = context.ServiceProvider.GetService<IMessaging>();
         }
 
+        /// <summary>
+        /// See <see cref="IResolverExtension.ResolveFile(string, IntermediateSymbolDefinition, SourceLineNumber, BindStage)"/>
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="symbolDefinition"></param>
+        /// <param name="sourceLineNumbers"></param>
+        /// <param name="bindStage"></param>
+        /// <returns></returns>
         public virtual IResolveFileResult ResolveFile(string source, IntermediateSymbolDefinition symbolDefinition, SourceLineNumber sourceLineNumbers, BindStage bindStage)
         {
             return null;
diff --git a/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs b/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
index 20e02f2e..e2be63bf 100644
--- a/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
+++ b/src/WixToolset.Extensibility/BaseWindowsInstallerBackendBinderExtension.cs
@@ -40,6 +40,9 @@ namespace WixToolset.Extensibility
         /// </summary>
         protected IResolvedCabinet CreateResolvedCabinet() => this.Context.ServiceProvider.GetService<IResolvedCabinet>();
 
+        /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
+        /// </summary>
         public virtual void PreBackendBind(IBindContext context)
         {
             this.Context = context;
@@ -49,10 +52,19 @@ namespace WixToolset.Extensibility
             this.BackendHelper = context.ServiceProvider.GetService<IWindowsInstallerBackendHelper>();
         }
 
+        /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
+        /// </summary>
         public virtual IResolvedCabinet ResolveCabinet(string cabinetPath, IEnumerable<IBindFileWithPath> files) => null;
 
+        /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
+        /// </summary>
         public virtual string ResolveMedia(MediaSymbol mediaRow, string mediaLayoutDirectory, string layoutDirectory) => null;
 
+        /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
+        /// </summary>
         public virtual bool TryAddSymbolToOutput(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions)
         {
             if (this.TableDefinitions.Any(t => t.SymbolDefinition == symbol.Definition))
@@ -63,6 +75,9 @@ namespace WixToolset.Extensibility
             return false;
         }
 
+        /// <summary>
+        /// See <see cref="IWindowsInstallerBackendBinderExtension.PreBackendBind(IBindContext)"/>
+        /// </summary>
         public virtual void PostBackendBind(IBindResult result)
         {
         }
diff --git a/src/WixToolset.Extensibility/CompilerConstants.cs b/src/WixToolset.Extensibility/CompilerConstants.cs
index f26456fb..73ff685c 100644
--- a/src/WixToolset.Extensibility/CompilerConstants.cs
+++ b/src/WixToolset.Extensibility/CompilerConstants.cs
@@ -9,11 +9,34 @@ namespace WixToolset.Extensibility
     /// </summary>
     public static class CompilerConstants
     {
+        /// <summary>
+        /// 
+        /// </summary>
         public const int IntegerNotSet = int.MinValue;
+
+        /// <summary>
+        /// 
+        /// </summary>
         public const int IllegalInteger = int.MinValue + 1;
+
+        /// <summary>
+        /// 
+        /// </summary>
         public const long LongNotSet = long.MinValue;
+
+        /// <summary>
+        /// 
+        /// </summary>
         public const long IllegalLong = long.MinValue + 1;
+
+        /// <summary>
+        /// 
+        /// </summary>
         public const string IllegalGuid = "IllegalGuid";
+
+        /// <summary>
+        /// 
+        /// </summary>
         public static readonly Version IllegalVersion = new Version(Int32.MaxValue, Int32.MaxValue, Int32.MaxValue, Int32.MaxValue);
     }
 }
diff --git a/src/WixToolset.Extensibility/Data/BindStage.cs b/src/WixToolset.Extensibility/Data/BindStage.cs
index 559a5a5a..a690a896 100644
--- a/src/WixToolset.Extensibility/Data/BindStage.cs
+++ b/src/WixToolset.Extensibility/Data/BindStage.cs
@@ -2,6 +2,9 @@
 
 namespace WixToolset.Extensibility.Data
 {
+    /// <summary>
+    /// 
+    /// </summary>
     public enum BindStage
     {
         /// <summary>
diff --git a/src/WixToolset.Extensibility/Data/ExtensionCommandLineSwitch.cs b/src/WixToolset.Extensibility/Data/ExtensionCommandLineSwitch.cs
index d1d8f0c3..14b5dabb 100644
--- a/src/WixToolset.Extensibility/Data/ExtensionCommandLineSwitch.cs
+++ b/src/WixToolset.Extensibility/Data/ExtensionCommandLineSwitch.cs
@@ -1,4 +1,4 @@
-// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
+// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
 namespace WixToolset.Extensibility.Data
 {
@@ -7,8 +7,14 @@ namespace WixToolset.Extensibility.Data
     /// </summary>
     public struct ExtensionCommandLineSwitch
     {
+        /// <summary>
+        /// 
+        /// </summary>
         public string Switch { get; set; }
 
+        /// <summary>
+        /// 
+        /// </summary>
         public string Description { get; set; }
     }
 }
diff --git a/src/WixToolset.Extensibility/Data/IBindContext.cs b/src/WixToolset.Extensibility/Data/IBindContext.cs
index 650e7fc1..fd5a3ee4 100644
--- a/src/WixToolset.Extensibility/Data/IBindContext.cs
+++ b/src/WixToolset.Extensibility/Data/IBindContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IBindContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IBindFileWithPath.cs b/src/WixToolset.Extensibility/Data/IBindFileWithPath.cs
index ec78a1a0..69036113 100644
--- a/src/WixToolset.Extensibility/Data/IBindFileWithPath.cs
+++ b/src/WixToolset.Extensibility/Data/IBindFileWithPath.cs
@@ -2,6 +2,7 @@
 
 namespace WixToolset.Extensibility.Data
 {
+#pragma warning disable 1591 // TODO: add documentation
     public interface IBindFileWithPath
     {
         string Id { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IBindPath.cs b/src/WixToolset.Extensibility/Data/IBindPath.cs
index 5784a0e0..46895e89 100644
--- a/src/WixToolset.Extensibility/Data/IBindPath.cs
+++ b/src/WixToolset.Extensibility/Data/IBindPath.cs
@@ -2,6 +2,9 @@
 
 namespace WixToolset.Extensibility.Data
 {
+    /// <summary>
+    /// Interface for a bind path.
+    /// </summary>
     public interface IBindPath
     {
         /// <summary>
diff --git a/src/WixToolset.Extensibility/Data/IBindResult.cs b/src/WixToolset.Extensibility/Data/IBindResult.cs
index 0be93bce..1654f24d 100644
--- a/src/WixToolset.Extensibility/Data/IBindResult.cs
+++ b/src/WixToolset.Extensibility/Data/IBindResult.cs
@@ -6,6 +6,7 @@ namespace WixToolset.Extensibility.Data
     using System.Collections.Generic;
     using WixToolset.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IBindResult : IDisposable
     {
         IEnumerable<IFileTransfer> FileTransfers { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/ICommandLineArguments.cs b/src/WixToolset.Extensibility/Data/ICommandLineArguments.cs
index b732c648..32ee4c09 100644
--- a/src/WixToolset.Extensibility/Data/ICommandLineArguments.cs
+++ b/src/WixToolset.Extensibility/Data/ICommandLineArguments.cs
@@ -9,6 +9,7 @@ namespace WixToolset.Extensibility.Data
     /// </summary>
     public interface ICommandLineArguments
     {
+#pragma warning disable 1591 // TODO: add documentation
         string[] OriginalArguments { get; set; }
 
         string[] Arguments { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/ICommandLineContext.cs b/src/WixToolset.Extensibility/Data/ICommandLineContext.cs
index 41c913fe..fbaa84d1 100644
--- a/src/WixToolset.Extensibility/Data/ICommandLineContext.cs
+++ b/src/WixToolset.Extensibility/Data/ICommandLineContext.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility.Data
     using System;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ICommandLineContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/ICompileContext.cs b/src/WixToolset.Extensibility/Data/ICompileContext.cs
index 277b842c..1c3620b6 100644
--- a/src/WixToolset.Extensibility/Data/ICompileContext.cs
+++ b/src/WixToolset.Extensibility/Data/ICompileContext.cs
@@ -8,6 +8,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ICompileContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IComponentKeyPath.cs b/src/WixToolset.Extensibility/Data/IComponentKeyPath.cs
index 9417b836..2de9c028 100644
--- a/src/WixToolset.Extensibility/Data/IComponentKeyPath.cs
+++ b/src/WixToolset.Extensibility/Data/IComponentKeyPath.cs
@@ -2,6 +2,7 @@
 
 namespace WixToolset.Extensibility.Data
 {
+#pragma warning disable 1591 // TODO: add documentation
     public interface IComponentKeyPath
     {
         bool Explicit { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IDecompileContext.cs b/src/WixToolset.Extensibility/Data/IDecompileContext.cs
index 29d6fb3b..f783a31e 100644
--- a/src/WixToolset.Extensibility/Data/IDecompileContext.cs
+++ b/src/WixToolset.Extensibility/Data/IDecompileContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IDecompileContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IDecompileResult.cs b/src/WixToolset.Extensibility/Data/IDecompileResult.cs
index d15f8847..ad4d0841 100644
--- a/src/WixToolset.Extensibility/Data/IDecompileResult.cs
+++ b/src/WixToolset.Extensibility/Data/IDecompileResult.cs
@@ -6,6 +6,7 @@ namespace WixToolset.Extensibility.Data
     using System.Xml.Linq;
     using WixToolset.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IDecompileResult
     {
         XDocument Document { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IDelayedField.cs b/src/WixToolset.Extensibility/Data/IDelayedField.cs
index 83139413..59a578a6 100644
--- a/src/WixToolset.Extensibility/Data/IDelayedField.cs
+++ b/src/WixToolset.Extensibility/Data/IDelayedField.cs
@@ -4,6 +4,7 @@ namespace WixToolset.Extensibility.Data
 {
     using WixToolset.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IDelayedField
     {
         IntermediateField Field { get; }
diff --git a/src/WixToolset.Extensibility/Data/IExpectedExtractFile.cs b/src/WixToolset.Extensibility/Data/IExpectedExtractFile.cs
index 50015112..4bc8cd96 100644
--- a/src/WixToolset.Extensibility/Data/IExpectedExtractFile.cs
+++ b/src/WixToolset.Extensibility/Data/IExpectedExtractFile.cs
@@ -4,6 +4,7 @@ namespace WixToolset.Extensibility.Data
 {
     using System;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IExpectedExtractFile
     {
         Uri Uri { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IFileSystemContext.cs b/src/WixToolset.Extensibility/Data/IFileSystemContext.cs
index f27e16d5..321fa83c 100644
--- a/src/WixToolset.Extensibility/Data/IFileSystemContext.cs
+++ b/src/WixToolset.Extensibility/Data/IFileSystemContext.cs
@@ -6,6 +6,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IFileSystemContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IIncludedFile.cs b/src/WixToolset.Extensibility/Data/IIncludedFile.cs
index ac5e604c..e25c9f7e 100644
--- a/src/WixToolset.Extensibility/Data/IIncludedFile.cs
+++ b/src/WixToolset.Extensibility/Data/IIncludedFile.cs
@@ -4,6 +4,9 @@ namespace WixToolset.Extensibility.Data
 {
     using WixToolset.Data;
 
+    /// <summary>
+    /// Interface for an included file.
+    /// </summary>
     public interface IIncludedFile
     {
         /// <summary>
diff --git a/src/WixToolset.Extensibility/Data/IInscribeContext.cs b/src/WixToolset.Extensibility/Data/IInscribeContext.cs
index 816a0c80..79d4d1f5 100644
--- a/src/WixToolset.Extensibility/Data/IInscribeContext.cs
+++ b/src/WixToolset.Extensibility/Data/IInscribeContext.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility.Data
     using System;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IInscribeContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/ILayoutContext.cs b/src/WixToolset.Extensibility/Data/ILayoutContext.cs
index 60e5cc8d..89d61d5c 100644
--- a/src/WixToolset.Extensibility/Data/ILayoutContext.cs
+++ b/src/WixToolset.Extensibility/Data/ILayoutContext.cs
@@ -6,6 +6,7 @@ namespace WixToolset.Extensibility.Data
     using System.Threading;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ILayoutContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/ILibraryContext.cs b/src/WixToolset.Extensibility/Data/ILibraryContext.cs
index 12a400cd..68a9faa9 100644
--- a/src/WixToolset.Extensibility/Data/ILibraryContext.cs
+++ b/src/WixToolset.Extensibility/Data/ILibraryContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ILibraryContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/ILinkContext.cs b/src/WixToolset.Extensibility/Data/ILinkContext.cs
index ab698c1d..c6c9cf7d 100644
--- a/src/WixToolset.Extensibility/Data/ILinkContext.cs
+++ b/src/WixToolset.Extensibility/Data/ILinkContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ILinkContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IPreprocessContext.cs b/src/WixToolset.Extensibility/Data/IPreprocessContext.cs
index 3418242a..b07fb81f 100644
--- a/src/WixToolset.Extensibility/Data/IPreprocessContext.cs
+++ b/src/WixToolset.Extensibility/Data/IPreprocessContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IPreprocessContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IPreprocessResult.cs b/src/WixToolset.Extensibility/Data/IPreprocessResult.cs
index 955c0ced..d46c8147 100644
--- a/src/WixToolset.Extensibility/Data/IPreprocessResult.cs
+++ b/src/WixToolset.Extensibility/Data/IPreprocessResult.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility.Data
     using System.Collections.Generic;
     using System.Xml.Linq;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IPreprocessResult
     {
         XDocument Document { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IResolveContext.cs b/src/WixToolset.Extensibility/Data/IResolveContext.cs
index e84655b8..2c775932 100644
--- a/src/WixToolset.Extensibility/Data/IResolveContext.cs
+++ b/src/WixToolset.Extensibility/Data/IResolveContext.cs
@@ -7,6 +7,7 @@ namespace WixToolset.Extensibility.Data
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IResolveContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/IResolveFileResult.cs b/src/WixToolset.Extensibility/Data/IResolveFileResult.cs
index bf0af72e..c8dca81a 100644
--- a/src/WixToolset.Extensibility/Data/IResolveFileResult.cs
+++ b/src/WixToolset.Extensibility/Data/IResolveFileResult.cs
@@ -4,6 +4,7 @@ namespace WixToolset.Extensibility.Data
 {
     using System.Collections.Generic;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IResolveFileResult
     {
         IEnumerable<string> CheckedPaths { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IResolveResult.cs b/src/WixToolset.Extensibility/Data/IResolveResult.cs
index 87b9c573..7c3403d4 100644
--- a/src/WixToolset.Extensibility/Data/IResolveResult.cs
+++ b/src/WixToolset.Extensibility/Data/IResolveResult.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility.Data
     using System.Collections.Generic;
     using WixToolset.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IResolveResult
     {
         int Codepage { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IResolvedCabinet.cs b/src/WixToolset.Extensibility/Data/IResolvedCabinet.cs
index c94ff8db..0c07d387 100644
--- a/src/WixToolset.Extensibility/Data/IResolvedCabinet.cs
+++ b/src/WixToolset.Extensibility/Data/IResolvedCabinet.cs
@@ -2,6 +2,7 @@
 
 namespace WixToolset.Extensibility.Data
 {
+#pragma warning disable 1591 // TODO: add documentation
     public interface IResolvedCabinet
     {
         CabinetBuildOption BuildOption { get; set; }
diff --git a/src/WixToolset.Extensibility/Data/IUnbindContext.cs b/src/WixToolset.Extensibility/Data/IUnbindContext.cs
index bdc31240..faf77f5d 100644
--- a/src/WixToolset.Extensibility/Data/IUnbindContext.cs
+++ b/src/WixToolset.Extensibility/Data/IUnbindContext.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility.Data
     using System;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IUnbindContext
     {
         IWixToolsetServiceProvider ServiceProvider { get; }
diff --git a/src/WixToolset.Extensibility/Data/PossibleKeyPathType.cs b/src/WixToolset.Extensibility/Data/PossibleKeyPathType.cs
index 75ccb1d1..08e927e4 100644
--- a/src/WixToolset.Extensibility/Data/PossibleKeyPathType.cs
+++ b/src/WixToolset.Extensibility/Data/PossibleKeyPathType.cs
@@ -2,6 +2,9 @@
 
 namespace WixToolset.Extensibility.Data
 {
+    /// <summary>
+    /// Key path types.
+    /// </summary>
     public enum PossibleKeyPathType
     {
         /// <summary>
diff --git a/src/WixToolset.Extensibility/Data/TrackedFileType.cs b/src/WixToolset.Extensibility/Data/TrackedFileType.cs
index 195d5de9..e7f53842 100644
--- a/src/WixToolset.Extensibility/Data/TrackedFileType.cs
+++ b/src/WixToolset.Extensibility/Data/TrackedFileType.cs
@@ -1,7 +1,10 @@
-// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
+// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
 namespace WixToolset.Extensibility.Data
 {
+    /// <summary>
+    /// Tracked file types.
+    /// </summary>
     public enum TrackedFileType
     {
         /// <summary>
diff --git a/src/WixToolset.Extensibility/DecompilerConstants.cs b/src/WixToolset.Extensibility/DecompilerConstants.cs
index 83393da9..22e8530d 100644
--- a/src/WixToolset.Extensibility/DecompilerConstants.cs
+++ b/src/WixToolset.Extensibility/DecompilerConstants.cs
@@ -7,7 +7,14 @@ namespace WixToolset.Extensibility
     /// </summary>
     public static class DecompilerConstants
     {
+        /// <summary>
+        /// 
+        /// </summary>
         public const char PrimaryKeyDelimiter = '/';
+
+        /// <summary>
+        /// 
+        /// </summary>
         public const string PrimaryKeyDelimiterString = "/";
     }
 }
diff --git a/src/WixToolset.Extensibility/IBackend.cs b/src/WixToolset.Extensibility/IBackend.cs
index 12a7e834..9579c3ca 100644
--- a/src/WixToolset.Extensibility/IBackend.cs
+++ b/src/WixToolset.Extensibility/IBackend.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IBackend
     {
         IBindResult Bind(IBindContext context);
diff --git a/src/WixToolset.Extensibility/IBackendFactory.cs b/src/WixToolset.Extensibility/IBackendFactory.cs
index 1155e9b6..99a6704f 100644
--- a/src/WixToolset.Extensibility/IBackendFactory.cs
+++ b/src/WixToolset.Extensibility/IBackendFactory.cs
@@ -1,7 +1,8 @@
-// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
+// Copyright (c) .NET Foundation and contributors. All rights reserved. Licensed under the Microsoft Reciprocal License. See LICENSE.TXT file in the project root for full license information.
 
 namespace WixToolset.Extensibility
 {
+#pragma warning disable 1591 // TODO: add documentation
     public interface IBackendFactory
     {
         bool TryCreateBackend(string outputType, string outputPath, out IBackend backend);
diff --git a/src/WixToolset.Extensibility/IBurnBackendExtension.cs b/src/WixToolset.Extensibility/IBurnBackendExtension.cs
index 356d208d..769c2457 100644
--- a/src/WixToolset.Extensibility/IBurnBackendExtension.cs
+++ b/src/WixToolset.Extensibility/IBurnBackendExtension.cs
@@ -5,6 +5,9 @@ namespace WixToolset.Extensibility
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
 
+    /// <summary>
+    /// Interface all Burn backend extensions implement.
+    /// </summary>
     public interface IBurnBackendExtension
     {
         /// <summary>
@@ -12,8 +15,26 @@ namespace WixToolset.Extensibility
         /// </summary>
         void PreBackendBind(IBindContext context);
 
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="relatedSource"></param>
+        /// <param name="type"></param>
+        /// <param name="sourceLineNumbers"></param>
+        /// <param name="bindStage"></param>
+        /// <returns></returns>
         IResolveFileResult ResolveRelatedFile(string source, string relatedSource, string type, SourceLineNumber sourceLineNumbers, BindStage bindStage);
 
+        /// <summary>
+        /// 
+        /// </summary>
+        /// <param name="url"></param>
+        /// <param name="fallbackUrl"></param>
+        /// <param name="packageId"></param>
+        /// <param name="payloadId"></param>
+        /// <param name="fileName"></param>
+        /// <returns></returns>
         string ResolveUrl(string url, string fallbackUrl, string packageId, string payloadId, string fileName);
 
         /// <summary>
diff --git a/src/WixToolset.Extensibility/ICompilerExtension.cs b/src/WixToolset.Extensibility/ICompilerExtension.cs
index f2fd8193..55ef683a 100644
--- a/src/WixToolset.Extensibility/ICompilerExtension.cs
+++ b/src/WixToolset.Extensibility/ICompilerExtension.cs
@@ -26,6 +26,8 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Processes an attribute for the Compiler.
         /// </summary>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="parentElement">Parent element of attribute.</param>
         /// <param name="attribute">Attribute to process.</param>
         /// <param name="context">Extra information about the context in which this element is being parsed.</param>
@@ -34,6 +36,8 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Processes an element for the Compiler.
         /// </summary>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="parentElement">Parent element of element to process.</param>
         /// <param name="element">Element to process.</param>
         /// <param name="context">Extra information about the context in which this element is being parsed.</param>
@@ -42,9 +46,11 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Processes an element for the Compiler, with the ability to supply a component keypath.
         /// </summary>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="parentElement">Parent element of element to process.</param>
         /// <param name="element">Element to process.</param>
-        /// <param name="contextValues">Extra information about the context in which this element is being parsed.</param>
+        /// <param name="context">Extra information about the context in which this element is being parsed.</param>
         IComponentKeyPath ParsePossibleKeyPathElement(Intermediate intermediate, IntermediateSection section, XElement parentElement, XElement element, IDictionary<string, string> context);
 
         /// <summary>
diff --git a/src/WixToolset.Extensibility/IExtensionCommandLine.cs b/src/WixToolset.Extensibility/IExtensionCommandLine.cs
index 45683373..2c7d0a7e 100644
--- a/src/WixToolset.Extensibility/IExtensionCommandLine.cs
+++ b/src/WixToolset.Extensibility/IExtensionCommandLine.cs
@@ -34,11 +34,10 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Gives the extension an opportunity to provide a command.
         /// </summary>
-        /// </summary>
         /// <param name="parser">Parser to help parse the argument and additional arguments.</param>
         /// <param name="argument">Argument to parse.</param>
         /// <param name="command"></param>
-        /// <returns>True if the argument is recognized as a commond; otherwise false to allow another extension to process it.</returns>
+        /// <returns>True if the argument is recognized as a command; otherwise false to allow another extension to process it.</returns>
         bool TryParseCommand(ICommandLineParser parser, string argument, out ICommandLineCommand command);
 
         /// <summary>
diff --git a/src/WixToolset.Extensibility/IFileSystemExtension.cs b/src/WixToolset.Extensibility/IFileSystemExtension.cs
index 0a3e84b8..9807e8b9 100644
--- a/src/WixToolset.Extensibility/IFileSystemExtension.cs
+++ b/src/WixToolset.Extensibility/IFileSystemExtension.cs
@@ -9,6 +9,7 @@ namespace WixToolset.Extensibility
     /// </summary>
     public interface IFileSystemExtension
     {
+#pragma warning disable 1591 // TODO: add documentation
         void Initialize(IFileSystemContext context);
 
         bool? CompareFiles(string targetFile, string updatedFile);
diff --git a/src/WixToolset.Extensibility/IInspectorExtension.cs b/src/WixToolset.Extensibility/IInspectorExtension.cs
index 6c4be1e5..7c488a89 100644
--- a/src/WixToolset.Extensibility/IInspectorExtension.cs
+++ b/src/WixToolset.Extensibility/IInspectorExtension.cs
@@ -10,15 +10,13 @@ namespace WixToolset.Extensibility
     /// Interface for inspector extensions.
     /// </summary>
     /// <remarks>
-    /// The inspector methods are stateless, but extensions are loaded and last for the lifetime of the
-    /// containing classes like <see cref="Preprocessor"/>, <see cref="Compiler"/>, <see cref="Linker"/>,
-    /// <see cref="Differ"/>, and <see cref="Binder"/>. If you want to maintain state, you should check
+    /// The inspector methods are stateless, but extensions are loaded once. If you want to maintain state, you should check
     /// if your data is loaded for each method and, if not, load it.
     /// </remarks>
     public interface IInspectorExtension
     {
         /// <summary>
-        /// Gets or sets the <see cref="InspectorCore"/> for inspector extensions to use.
+        /// Gets or sets the <see cref="IInspectorCore"/> for inspector extensions to use.
         /// </summary>
         IInspectorCore Core { get; set; }
 
@@ -34,6 +32,7 @@ namespace WixToolset.Extensibility
         /// <param name="intermediate">The compiled output.</param>
         void InspectIntermediate(Intermediate intermediate);
 
+#if REWRITE
         /// <summary>
         /// Inspect the output.
         /// </summary>
@@ -46,13 +45,15 @@ namespace WixToolset.Extensibility
         /// transforms are the primary transforms you'll typically want to inspect
         /// and contain your changes to target products.
         /// </remarks>
+#endif
+        /// <summary />
         void InspectOutput(Intermediate output);
 
         /// <summary>
         /// Inspect the final output after binding.
         /// </summary>
         /// <param name="filePath">The file path to the final bound output.</param>
-        /// <param name="pdb">The <see cref="Pdb"/> that contains source line numbers
+        /// <param name="pdb">The <see cref="Intermediate"/> that contains source line numbers
         /// for the database and all rows.</param>
         void InspectDatabase(string filePath, Intermediate pdb);
     }
diff --git a/src/WixToolset.Extensibility/ILayoutExtension.cs b/src/WixToolset.Extensibility/ILayoutExtension.cs
index 7bcee0a8..ecd7d8f1 100644
--- a/src/WixToolset.Extensibility/ILayoutExtension.cs
+++ b/src/WixToolset.Extensibility/ILayoutExtension.cs
@@ -14,9 +14,11 @@ namespace WixToolset.Extensibility
         /// </summary>
         void PreLayout(ILayoutContext context);
 
+#pragma warning disable 1591 // TODO: add documentation
         bool CopyFile(string source, string destination);
 
         bool MoveFile(string source, string destination);
+#pragma warning restore 1591
 
         /// <summary>
         /// Called after all layout occurs.
diff --git a/src/WixToolset.Extensibility/ILibrarianExtension.cs b/src/WixToolset.Extensibility/ILibrarianExtension.cs
index e52c71c9..d9b04cd2 100644
--- a/src/WixToolset.Extensibility/ILibrarianExtension.cs
+++ b/src/WixToolset.Extensibility/ILibrarianExtension.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ILibrarianExtension
     {
         void PreCombine(ILibraryContext context);
diff --git a/src/WixToolset.Extensibility/IMessageListener.cs b/src/WixToolset.Extensibility/IMessageListener.cs
index d5baa24f..ea103538 100644
--- a/src/WixToolset.Extensibility/IMessageListener.cs
+++ b/src/WixToolset.Extensibility/IMessageListener.cs
@@ -5,6 +5,7 @@ namespace WixToolset.Extensibility
     using WixToolset.Data;
     using WixToolset.Extensibility.Services;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IMessageListener
     {
         MessageLevel CalculateMessageLevel(IMessaging messaging, Message message, MessageLevel defaultMessageLevel);
diff --git a/src/WixToolset.Extensibility/IPreprocessorExtension.cs b/src/WixToolset.Extensibility/IPreprocessorExtension.cs
index 94eb7d78..919ff1ae 100644
--- a/src/WixToolset.Extensibility/IPreprocessorExtension.cs
+++ b/src/WixToolset.Extensibility/IPreprocessorExtension.cs
@@ -41,7 +41,6 @@ namespace WixToolset.Extensibility
         /// <summary>
         /// Processes a pragma defined in the extension.
         /// </summary>
-        /// <param name="sourceLineNumbers">The location of this pragma's PI.</param>
         /// <param name="prefix">The prefix of the pragma to be processed by the extension.</param>
         /// <param name="pragma">The name of the pragma.</param>
         /// <param name="args">The pragma's arguments.</param>
diff --git a/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs b/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
index fc769da9..f257f1ec 100644
--- a/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
+++ b/src/WixToolset.Extensibility/IWindowsInstallerBackendBinderExtension.cs
@@ -13,18 +13,22 @@ namespace WixToolset.Extensibility
     /// </summary>
     public interface IWindowsInstallerBackendBinderExtension
     {
+#pragma warning disable 1591 // TODO: add documentation
         IEnumerable<TableDefinition> TableDefinitions { get; }
+#pragma warning restore 1591
 
         /// <summary>
         /// Called before binding occurs.
         /// </summary>
         void PreBackendBind(IBindContext context);
 
+#pragma warning disable 1591 // TODO: add documentation
         IResolvedCabinet ResolveCabinet(string cabinetPath, IEnumerable<IBindFileWithPath> files);
 
         string ResolveMedia(MediaSymbol mediaRow, string mediaLayoutDirectory, string layoutDirectory);
 
         bool TryAddSymbolToOutput(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions);
+#pragma warning restore 1591
 
         /// <summary>
         /// Called after all output changes occur and right before the output is bound into its final format.
diff --git a/src/WixToolset.Extensibility/Services/IBackendHelper.cs b/src/WixToolset.Extensibility/Services/IBackendHelper.cs
index bfec5256..0c8578b2 100644
--- a/src/WixToolset.Extensibility/Services/IBackendHelper.cs
+++ b/src/WixToolset.Extensibility/Services/IBackendHelper.cs
@@ -17,6 +17,7 @@ namespace WixToolset.Extensibility.Services
         /// <param name="source">Source for the file transfer.</param>
         /// <param name="destination">Destination for the file transfer.</param>
         /// <param name="move">Indicates whether to move or copy the source file.</param>
+        /// <param name="sourceLineNumbers">Optional source line numbers that requested the file transfer.</param>
         IFileTransfer CreateFileTransfer(string source, string destination, bool move, SourceLineNumber sourceLineNumbers = null);
 
         /// <summary>
diff --git a/src/WixToolset.Extensibility/Services/ICommandLineParser.cs b/src/WixToolset.Extensibility/Services/ICommandLineParser.cs
index f7e2a28f..f538e0b7 100644
--- a/src/WixToolset.Extensibility/Services/ICommandLineParser.cs
+++ b/src/WixToolset.Extensibility/Services/ICommandLineParser.cs
@@ -4,6 +4,7 @@ namespace WixToolset.Extensibility.Services
 {
     using System.Collections.Generic;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface ICommandLineParser
     {
         string ErrorArgument { get; set; }
diff --git a/src/WixToolset.Extensibility/Services/IParseHelper.cs b/src/WixToolset.Extensibility/Services/IParseHelper.cs
index 08bcd911..93144629 100644
--- a/src/WixToolset.Extensibility/Services/IParseHelper.cs
+++ b/src/WixToolset.Extensibility/Services/IParseHelper.cs
@@ -68,15 +68,6 @@ namespace WixToolset.Extensibility.Services
         /// <returns>New symbol.</returns>
         IntermediateSymbol CreateSymbol(IntermediateSection section, SourceLineNumber sourceLineNumbers, IntermediateSymbolDefinition symbolDefinition, Identifier identifier = null);
 
-        [Obsolete]
-        IntermediateSymbol CreateRow(IntermediateSection section, SourceLineNumber sourceLineNumbers, string tableName, Identifier identifier = null);
-
-        [Obsolete]
-        IntermediateSymbol CreateSymbol(IntermediateSection section, SourceLineNumber sourceLineNumbers, SymbolDefinitionType symbolType, Identifier identifier = null);
-
-        [Obsolete]
-        IntermediateSymbol CreateRow(IntermediateSection section, SourceLineNumber sourceLineNumbers, SymbolDefinitionType symbolType, Identifier identifier = null);
-
         /// <summary>
         /// Creates a directory row from a name.
         /// </summary>
@@ -106,6 +97,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Creates a Registry symbol in the active section.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source and line number of the current symbol.</param>
         /// <param name="root">The registry entry root.</param>
         /// <param name="key">The registry entry key.</param>
@@ -115,9 +107,6 @@ namespace WixToolset.Extensibility.Services
         /// <param name="escapeLeadingHash">If true, "escape" leading '#' characters so the value is written as a REG_SZ.</param>
         Identifier CreateRegistrySymbol(IntermediateSection section, SourceLineNumber sourceLineNumbers, RegistryRootType root, string key, string name, string value, string componentId, bool escapeLeadingHash);
 
-        [Obsolete]
-        Identifier CreateRegistryRow(IntermediateSection section, SourceLineNumber sourceLineNumbers, RegistryRootType root, string key, string name, string value, string componentId, bool escapeLeadingHash);
-
         /// <summary>
         /// Creates a short file/directory name using an identifier and long file/directory name as input.
         /// </summary>
@@ -131,6 +120,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Create a WixSimpleReference symbol in the active section.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line information for the row.</param>
         /// <param name="symbolName">The symbol name of the simple reference.</param>
         /// <param name="primaryKey">The primary key of the simple reference.</param>
@@ -139,6 +129,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Create a WixSimpleReference symbol in the active section.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line information for the row.</param>
         /// <param name="symbolName">The symbol name of the simple reference.</param>
         /// <param name="primaryKeys">The primary keys of the simple reference.</param>
@@ -147,6 +138,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Create a WixSimpleReference symbol in the active section.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line information for the row.</param>
         /// <param name="symbolDefinition">The symbol definition of the simple reference.</param>
         /// <param name="primaryKey">The primary key of the simple reference.</param>
@@ -155,6 +147,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Create a WixSimpleReference symbol in the active section.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line information for the row.</param>
         /// <param name="symbolDefinition">The symbol definition of the simple reference.</param>
         /// <param name="primaryKeys">The primary keys of the simple reference.</param>
@@ -167,7 +160,7 @@ namespace WixToolset.Extensibility.Services
         /// <param name="sourceLineNumbers">Source line information.</param>
         /// <param name="section">Section to create the reference in.</param>
         /// <param name="customAction">The custom action base name.</param>
-        /// <param name="currentPlatform">The platform being compiled.</param>
+        /// <param name="platform">The platform being compiled.</param>
         /// <param name="supportedPlatforms">The platforms for which there are specialized custom actions.</param>
         void CreateCustomActionReference(SourceLineNumber sourceLineNumbers, IntermediateSection section, string customAction, Platform platform, CustomActionPlatforms supportedPlatforms);
 
@@ -195,9 +188,6 @@ namespace WixToolset.Extensibility.Services
         /// <param name="childId">Id of the Child Node.</param>
         void CreateWixGroupSymbol(IntermediateSection section, SourceLineNumber sourceLineNumbers, ComplexReferenceParentType parentType, string parentId, ComplexReferenceChildType childType, string childId);
 
-        [Obsolete]
-        void CreateWixGroupRow(IntermediateSection section, SourceLineNumber sourceLineNumbers, ComplexReferenceParentType parentType, string parentId, ComplexReferenceChildType childType, string childId);
-
         /// <summary>
         /// Creates a symbol in the WixSearch table.
         /// </summary>
@@ -231,6 +221,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Add the appropriate symbols to make sure that the given table shows up in the resulting output.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line numbers.</param>
         /// <param name="tableName">Name of the table to ensure existance of.</param>
         void EnsureTable(IntermediateSection section, SourceLineNumber sourceLineNumbers, string tableName);
@@ -238,6 +229,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Add the appropriate symbols to make sure that the given table shows up in the resulting output.
         /// </summary>
+        /// <param name="section">Active section.</param>
         /// <param name="sourceLineNumbers">Source line numbers.</param>
         /// <param name="tableDefinition">Definition of the table to ensure existance of.</param>
         void EnsureTable(IntermediateSection section, SourceLineNumber sourceLineNumbers, TableDefinition tableDefinition);
@@ -408,6 +400,9 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Attempts to use an extension to parse the attribute.
         /// </summary>
+        /// <param name="extensions"></param>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="element">Element containing attribute to be parsed.</param>
         /// <param name="attribute">Attribute to be parsed.</param>
         /// <param name="context">Extra information about the context in which this element is being parsed.</param>
@@ -416,22 +411,31 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Attempts to use an extension to parse the element.
         /// </summary>
+        /// <param name="extensions"></param>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="parentElement">Element containing element to be parsed.</param>
         /// <param name="element">Element to be parsed.</param>
-        /// <param name="contextValues">Extra information about the context in which this element is being parsed.</param>
+        /// <param name="context">Extra information about the context in which this element is being parsed.</param>
         void ParseExtensionElement(IEnumerable<ICompilerExtension> extensions, Intermediate intermediate, IntermediateSection section, XElement parentElement, XElement element, IDictionary<string, string> context = null);
 
         /// <summary>
         /// Attempts to use an extension to parse the element, with support for setting component keypath.
         /// </summary>
+        /// <param name="extensions"></param>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="parentElement">Element containing element to be parsed.</param>
         /// <param name="element">Element to be parsed.</param>
-        /// <param name="contextValues">Extra information about the context in which this element is being parsed.</param>
+        /// <param name="context">Extra information about the context in which this element is being parsed.</param>
         IComponentKeyPath ParsePossibleKeyPathExtensionElement(IEnumerable<ICompilerExtension> extensions, Intermediate intermediate, IntermediateSection section, XElement parentElement, XElement element, IDictionary<string, string> context);
 
         /// <summary>
         /// Process all children of the element looking for extensions and erroring on the unexpected.
         /// </summary>
+        /// <param name="extensions"></param>
+        /// <param name="intermediate">Parent intermediate.</param>
+        /// <param name="section">Parent section.</param>
         /// <param name="element">Element to parse children.</param>
         void ParseForExtensionElements(IEnumerable<ICompilerExtension> extensions, Intermediate intermediate, IntermediateSection section, XElement element);
 
@@ -452,7 +456,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Called when the compiler encounters an unexpected attribute.
         /// </summary>
-        /// <param name="parentElement">Parent element that found unexpected attribute.</param>
+        /// <param name="element">Parent element that found unexpected attribute.</param>
         /// <param name="attribute">Unexpected attribute.</param>
         void UnexpectedAttribute(XElement element, XAttribute attribute);
 
diff --git a/src/WixToolset.Extensibility/Services/IPathResolver.cs b/src/WixToolset.Extensibility/Services/IPathResolver.cs
index 5713d2d7..5be0c4d3 100644
--- a/src/WixToolset.Extensibility/Services/IPathResolver.cs
+++ b/src/WixToolset.Extensibility/Services/IPathResolver.cs
@@ -6,7 +6,9 @@ namespace WixToolset.Extensibility.Services
     using WixToolset.Data;
     using WixToolset.Extensibility.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IPathResolver
+#pragma warning restore 1591
     {
         /// <summary>
         /// Get the canonical source path of a directory.
@@ -14,6 +16,7 @@ namespace WixToolset.Extensibility.Services
         /// <param name="directories">All cached directories.</param>
         /// <param name="componentIdGenSeeds">Hash table of Component GUID generation seeds indexed by directory id.</param>
         /// <param name="directory">Directory identifier.</param>
+        /// <param name="platform">Current platform.</param>
         /// <returns>Source path of a directory.</returns>
         string GetCanonicalDirectoryPath(Dictionary<string, IResolvedDirectory> directories, Dictionary<string, string> componentIdGenSeeds, string directory, Platform platform);
 
@@ -28,7 +31,7 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Gets the source path of a file.
         /// </summary>
-        /// <param name="directories">All cached directories in <see cref="ResolvedDirectory"/>.</param>
+        /// <param name="directories">All cached directories in <see cref="IResolvedDirectory"/>.</param>
         /// <param name="directoryId">Parent directory identifier.</param>
         /// <param name="fileName">File name (in long|source format).</param>
         /// <param name="compressed">Specifies the package is compressed.</param>
diff --git a/src/WixToolset.Extensibility/Services/IPreprocessHelper.cs b/src/WixToolset.Extensibility/Services/IPreprocessHelper.cs
index d55383db..f7973ac2 100644
--- a/src/WixToolset.Extensibility/Services/IPreprocessHelper.cs
+++ b/src/WixToolset.Extensibility/Services/IPreprocessHelper.cs
@@ -24,7 +24,7 @@ namespace WixToolset.Extensibility.Services
         /// <param name="context">The preprocess context.</param>
         /// <param name="name">The variable name.</param>
         /// <param name="value">The variable value.</param>
-        /// <param name="overwrite">Set to true to show variable overwrite warning.</param>
+        /// <param name="showWarning">Set to true to show variable overwrite warning.</param>
         void AddVariable(IPreprocessContext context, string name, string value, bool showWarning);
 
         /// <summary>
@@ -67,7 +67,7 @@ namespace WixToolset.Extensibility.Services
         /// Evaluate a Pragma.
         /// </summary>
         /// <param name="context">The preprocess context.</param>
-        /// <param name="pragmaName">The pragma's full name (<prefix>.<pragma>).</param>
+        /// <param name="pragmaName">The pragma's full name (&lt;prefix&gt;.&lt;pragma&gt;).</param>
         /// <param name="args">The arguments to the pragma.</param>
         /// <param name="parent">The parent element of the pragma.</param>
         void PreprocessPragma(IPreprocessContext context, string pragmaName, string args, XContainer parent);
diff --git a/src/WixToolset.Extensibility/Services/IVariableResolution.cs b/src/WixToolset.Extensibility/Services/IVariableResolution.cs
index efed0b1f..82bacb31 100644
--- a/src/WixToolset.Extensibility/Services/IVariableResolution.cs
+++ b/src/WixToolset.Extensibility/Services/IVariableResolution.cs
@@ -2,6 +2,7 @@
 
 namespace WixToolset.Extensibility.Services
 {
+#pragma warning disable 1591 // TODO: add documentation
     public interface IVariableResolution
     {
         bool DelayedResolve { get; set; }
diff --git a/src/WixToolset.Extensibility/Services/IVariableResolver.cs b/src/WixToolset.Extensibility/Services/IVariableResolver.cs
index a108e146..285f1fd1 100644
--- a/src/WixToolset.Extensibility/Services/IVariableResolver.cs
+++ b/src/WixToolset.Extensibility/Services/IVariableResolver.cs
@@ -4,13 +4,16 @@ namespace WixToolset.Extensibility.Services
 {
     using WixToolset.Data;
 
+#pragma warning disable 1591 // TODO: add documentation
     public interface IVariableResolver
     {
         void AddLocalization(Localization localization);
+#pragma warning restore 1591
 
         /// <summary>
         /// Add a variable.
         /// </summary>
+        /// <param name="sourceLineNumber">The source line information for the value.</param>
         /// <param name="name">The name of the variable.</param>
         /// <param name="value">The value of the variable.</param>
         /// <param name="overridable">Indicates whether the variable can be overridden by an existing variable.</param>
diff --git a/src/WixToolset.Extensibility/Services/IWindowsInstallerBackendHelper.cs b/src/WixToolset.Extensibility/Services/IWindowsInstallerBackendHelper.cs
index 008db83e..ca53ab4a 100644
--- a/src/WixToolset.Extensibility/Services/IWindowsInstallerBackendHelper.cs
+++ b/src/WixToolset.Extensibility/Services/IWindowsInstallerBackendHelper.cs
@@ -10,7 +10,26 @@ namespace WixToolset.Extensibility.Services
     /// </summary>
     public interface IWindowsInstallerBackendHelper
     {
+        /// <summary>
+        /// Creates a <see cref="Row"/> in the specified table.
+        /// </summary>
+        /// <param name="section">Parent section.</param>
+        /// <param name="symbol">Symbol with line information for the row.</param>
+        /// <param name="output">Current context.</param>
+        /// <param name="tableDefinition">Table definition for the row.</param>
+        /// <returns></returns>
         Row CreateRow(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinition tableDefinition);
+
+        /// <summary>
+        /// Looks up the registered <see cref="TableDefinition"/> for the given <see cref="IntermediateSymbol"/> and creates a <see cref="Row"/> in that table.
+        /// Goes sequentially through each field in the symbol and assigns the value to the column with the same index as the field.
+        /// If the symbol's Id is registered as the primary key then that is used for the first column and the column data is offset by 1.
+        /// </summary>
+        /// <param name="section">Parent section.</param>
+        /// <param name="symbol">Symbol to create the row from.</param>
+        /// <param name="output">Current context.</param>
+        /// <param name="tableDefinitions">Table definitions that have been registered with the binder.</param>
+        /// <returns>True if a row was created.</returns>
         bool TryAddSymbolToOutputMatchingTableDefinitions(IntermediateSection section, IntermediateSymbol symbol, WindowsInstallerData output, TableDefinitionCollection tableDefinitions);
     }
 }
diff --git a/src/WixToolset.Extensibility/Services/IWixtoolsetCoreServiceProvider.cs b/src/WixToolset.Extensibility/Services/IWixtoolsetCoreServiceProvider.cs
index 3c80ddef..2d0450dc 100644
--- a/src/WixToolset.Extensibility/Services/IWixtoolsetCoreServiceProvider.cs
+++ b/src/WixToolset.Extensibility/Services/IWixtoolsetCoreServiceProvider.cs
@@ -24,7 +24,6 @@ namespace WixToolset.Extensibility.Services
         /// <summary>
         /// Adds a service to the service locator.
         /// </summary>
-        /// <param name="serviceType">Type of the service to add.</param>
         /// <param name="creationFunction">
         /// A function that creates the service. The create function is provided the service provider
         /// itself to resolve additional services and a type dictionary that stores singleton services
diff --git a/src/WixToolset.Extensibility/WixToolset.Extensibility.csproj b/src/WixToolset.Extensibility/WixToolset.Extensibility.csproj
index 9fa4f12e..8b18c0ed 100644
--- a/src/WixToolset.Extensibility/WixToolset.Extensibility.csproj
+++ b/src/WixToolset.Extensibility/WixToolset.Extensibility.csproj
@@ -9,6 +9,7 @@
     <Description></Description>
     <DebugType>embedded</DebugType>
     <PublishRepositoryUrl>true</PublishRepositoryUrl>
+    <CreateDocumentationFile>true</CreateDocumentationFile>
   </PropertyGroup>
 
   <ItemGroup>