update documentation

Author: pwelter34

src/FluentCommand.SqlServer/Bulk/DataBulkCopy.cs

@@ -1,24 +1,23 @@
-using Microsoft.Data.SqlClient;
+using Microsoft.Data.SqlClient;
 
 namespace FluentCommand.Bulk;
 
 /// <summary>
-/// Bulk Copy extension methods
+/// Provides extension methods for starting and configuring SQL Server bulk copy operations.
 /// </summary>
 public static class DataBulkCopyExtensions
 {
     /// <summary>
-    /// Starts a data bulk-copy with the specified destination table name.
+    /// Starts a bulk copy operation using the specified data session and destination table name.
     /// </summary>
-    /// <param name="session">The session to use for the bulk-copy.</param>
-    /// <param name="destinationTable">Name of the destination table on the server.</param>
+    /// <param name="session">The <see cref="IDataSession"/> to use for the bulk copy operation.</param>
+    /// <param name="destinationTable">The name of the destination table on the SQL Server.</param>
     /// <returns>
-    /// A fluent <see langword="interface" /> to a <see cref="SqlBulkCopy " /> operation.
+    /// An <see cref="IDataBulkCopy"/> instance for configuring and executing the bulk copy operation.
     /// </returns>
     public static IDataBulkCopy BulkCopy(this IDataSession session, string destinationTable)
     {
         var bulkCopy = new DataBulkCopy(session, destinationTable);
         return bulkCopy;
     }
-
 }

src/FluentCommand.SqlServer/Bulk/DataBulkCopyExtensions.cs

@@ -1,34 +1,32 @@
 using System.Linq.Expressions;
 using System.Reflection;
 
-using Microsoft.Data.SqlClient;
-
 namespace FluentCommand.Bulk;
 
 /// <summary>
-/// A builder class for <see cref="IDataBulkCopy"/> column mapping.
+/// Provides a builder for configuring column mappings for <see cref="IDataBulkCopy"/> operations using strongly typed lambda expressions.
 /// </summary>
-/// <typeparam name="TEntity">The type of the entity.</typeparam>
+/// <typeparam name="TEntity">The type of the entity being mapped.</typeparam>
 public class DataBulkCopyMapping<TEntity>
 {
     private readonly IDataBulkCopy _bulkCopy;
 
     /// <summary>
     /// Initializes a new instance of the <see cref="DataBulkCopyMapping{TEntity}"/> class.
     /// </summary>
-    /// <param name="bulkCopy">The bulk copy.</param>
+    /// <param name="bulkCopy">The <see cref="IDataBulkCopy"/> instance to configure column mappings for.</param>
     internal DataBulkCopyMapping(IDataBulkCopy bulkCopy)
     {
         _bulkCopy = bulkCopy;
     }
 
     /// <summary>
-    /// Creates a new column mapping using a Lamba express to refer to source and destination columns.
+    /// Creates a new column mapping using a lambda expression to refer to both the source and destination columns by property name.
     /// </summary>
-    /// <typeparam name="TValue">The type of the value.</typeparam>
-    /// <param name="sourceProperty">A Lambda expression representing the property of the source column for the mapping.</param>
+    /// <typeparam name="TValue">The type of the property value.</typeparam>
+    /// <param name="sourceProperty">A lambda expression representing the property of the source column for the mapping.</param>
     /// <returns>
-    /// A fluent <see langword="interface" /> to a <see cref="SqlBulkCopy " /> operation.
+    /// The same <see cref="DataBulkCopyMapping{TEntity}"/> instance for fluent chaining.
     /// </returns>
     public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TValue>> sourceProperty)
     {
@@ -39,13 +37,13 @@ public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TVa
     }
 
     /// <summary>
-    /// Creates a new column mapping, using a Lamba express to refer to source column and a column name for the target column.
+    /// Creates a new column mapping using a lambda expression to refer to the source column and a column name for the destination column.
     /// </summary>
-    /// <typeparam name="TValue">The type of the value.</typeparam>
-    /// <param name="sourceProperty">A Lambda expression representing the property of the source column for the mapping.</param>
+    /// <typeparam name="TValue">The type of the property value.</typeparam>
+    /// <param name="sourceProperty">A lambda expression representing the property of the source column for the mapping.</param>
     /// <param name="destinationColumn">The name of the destination column within the destination table.</param>
     /// <returns>
-    /// A fluent <see langword="interface" /> to a <see cref="SqlBulkCopy " /> operation.
+    /// The same <see cref="DataBulkCopyMapping{TEntity}"/> instance for fluent chaining.
     /// </returns>
     public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TValue>> sourceProperty, string destinationColumn)
     {
@@ -56,13 +54,13 @@ public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TVa
     }
 
     /// <summary>
-    /// Creates a new column mapping, using a Lamba express to refer to source column and a column ordinal for the target column.
+    /// Creates a new column mapping using a lambda expression to refer to the source column and a column ordinal for the destination column.
     /// </summary>
-    /// <typeparam name="TValue">The type of the value.</typeparam>
-    /// <param name="sourceProperty">A Lambda expression representing the property of the source column for the mapping.</param>
+    /// <typeparam name="TValue">The type of the property value.</typeparam>
+    /// <param name="sourceProperty">A lambda expression representing the property of the source column for the mapping.</param>
     /// <param name="destinationOrdinal">The ordinal position of the destination column within the destination table.</param>
     /// <returns>
-    /// A fluent <see langword="interface" /> to a <see cref="SqlBulkCopy " /> operation.
+    /// The same <see cref="DataBulkCopyMapping{TEntity}"/> instance for fluent chaining.
     /// </returns>
     public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TValue>> sourceProperty, int destinationOrdinal)
     {
@@ -73,12 +71,12 @@ public DataBulkCopyMapping<TEntity> Mapping<TValue>(Expression<Func<TEntity, TVa
     }
 
     /// <summary>
-    /// Ignores the specified source column by removing it from the mapped columns collection.
+    /// Ignores the specified source column by removing it from the mapped columns collection, using a lambda expression to specify the property.
     /// </summary>
-    /// <typeparam name="TValue">The type of the value.</typeparam>
-    /// <param name="sourceProperty">A Lambda expression representing the property of the source column for the mapping.</param>
+    /// <typeparam name="TValue">The type of the property value.</typeparam>
+    /// <param name="sourceProperty">A lambda expression representing the property of the source column to ignore.</param>
     /// <returns>
-    /// A fluent <see langword="interface" /> to a <see cref="SqlBulkCopy " /> operation.
+    /// The same <see cref="DataBulkCopyMapping{TEntity}"/> instance for fluent chaining.
     /// </returns>
     public DataBulkCopyMapping<TEntity> Ignore<TValue>(Expression<Func<TEntity, TValue>> sourceProperty)
     {
@@ -88,7 +86,15 @@ public DataBulkCopyMapping<TEntity> Ignore<TValue>(Expression<Func<TEntity, TVal
         return this;
     }
 
-
+    /// <summary>
+    /// Extracts the column name from the provided lambda expression, using property attributes if available.
+    /// </summary>
+    /// <typeparam name="TValue">The type of the property value.</typeparam>
+    /// <param name="sourceProperty">A lambda expression representing the property.</param>
+    /// <returns>The resolved column name for the property.</returns>
+    /// <exception cref="ArgumentException">
+    /// Thrown if the expression does not represent a property access.
+    /// </exception>
     private string ExtractColumnName<TValue>(Expression<Func<TEntity, TValue>> sourceProperty)
     {
         var memberExpression = sourceProperty.Body as MemberExpression;