update some xml docs

Author: pwelter34

src/FluentCommand/Query/AggregateFunctions.cs

@@ -1,18 +1,37 @@
 namespace FluentCommand.Query;
 
 /// <summary>
-/// Query aggregate functions 
+/// Specifies the available aggregate functions for use in SQL queries.
 /// </summary>
 public enum AggregateFunctions
 {
-    /// <summary>Average aggregate function</summary>
+    /// <summary>
+    /// Calculates the average value of a numeric column.
+    /// Corresponds to the SQL <c>AVG</c> function.
+    /// </summary>
     Average,
-    /// <summary>Count aggregate function</summary>
+
+    /// <summary>
+    /// Counts the number of rows or non-null values in a column.
+    /// Corresponds to the SQL <c>COUNT</c> function.
+    /// </summary>
     Count,
-    /// <summary>Max aggregate function</summary>
+
+    /// <summary>
+    /// Returns the maximum value from a column.
+    /// Corresponds to the SQL <c>MAX</c> function.
+    /// </summary>
     Max,
-    /// <summary>Min aggregate function</summary>
+
+    /// <summary>
+    /// Returns the minimum value from a column.
+    /// Corresponds to the SQL <c>MIN</c> function.
+    /// </summary>
     Min,
-    /// <summary>Sum aggregate function</summary>
+
+    /// <summary>
+    /// Calculates the sum of all values in a numeric column.
+    /// Corresponds to the SQL <c>SUM</c> function.
+    /// </summary>
     Sum,
 }

src/FluentCommand/Query/DeleteBuilder.cs

@@ -4,16 +4,16 @@
 namespace FluentCommand.Query;
 
 /// <summary>
-/// Delete query statement builder
+/// Provides a builder for constructing SQL DELETE statements with fluent, chainable methods.
 /// </summary>
 public class DeleteBuilder : DeleteBuilder<DeleteBuilder>
 {
     /// <summary>
     /// Initializes a new instance of the <see cref="DeleteBuilder"/> class.
     /// </summary>
-    /// <param name="queryGenerator">The query generator.</param>
-    /// <param name="parameters">The parameters.</param>
-    /// <param name="logicalOperator">The logical operator.</param>
+    /// <param name="queryGenerator">The <see cref="IQueryGenerator"/> used to generate SQL expressions.</param>
+    /// <param name="parameters">The list of <see cref="QueryParameter"/> objects for the query.</param>
+    /// <param name="logicalOperator">The logical operator (<see cref="LogicalOperators"/>) to combine WHERE expressions. Defaults to <see cref="LogicalOperators.And"/>.</param>
     public DeleteBuilder(
         IQueryGenerator queryGenerator,
         List<QueryParameter> parameters,
@@ -24,18 +24,18 @@ public DeleteBuilder(
 }
 
 /// <summary>
-/// Delete query statement builder
+/// Provides a generic base class for building SQL DELETE statements with fluent, chainable methods.
 /// </summary>
-/// <typeparam name="TBuilder">The type of the builder.</typeparam>
+/// <typeparam name="TBuilder">The type of the builder for fluent chaining.</typeparam>
 public abstract class DeleteBuilder<TBuilder> : WhereBuilder<TBuilder>
     where TBuilder : DeleteBuilder<TBuilder>
 {
     /// <summary>
     /// Initializes a new instance of the <see cref="DeleteBuilder{TBuilder}"/> class.
     /// </summary>
-    /// <param name="queryGenerator">The query generator.</param>
-    /// <param name="parameters">The parameters.</param>
-    /// <param name="logicalOperator">The logical operator.</param>
+    /// <param name="queryGenerator">The <see cref="IQueryGenerator"/> used to generate SQL expressions.</param>
+    /// <param name="parameters">The list of <see cref="QueryParameter"/> objects for the query.</param>
+    /// <param name="logicalOperator">The logical operator (<see cref="LogicalOperators"/>) to combine WHERE expressions. Defaults to <see cref="LogicalOperators.And"/>.</param>
     protected DeleteBuilder(
         IQueryGenerator queryGenerator,
         List<QueryParameter> parameters,
@@ -44,48 +44,46 @@ protected DeleteBuilder(
     {
     }
 
-
     /// <summary>
-    /// Gets the output expressions.
+    /// Gets the collection of output column expressions for the DELETE statement.
     /// </summary>
     /// <value>
-    /// The output expressions.
+    /// A <see cref="HashSet{ColumnExpression}"/> containing the output column expressions.
     /// </value>
     protected HashSet<ColumnExpression> OutputExpressions { get; } = new();
 
     /// <summary>
-    /// Gets from expressions.
+    /// Gets the collection of FROM table expressions for the DELETE statement.
     /// </summary>
     /// <value>
-    /// From expressions.
+    /// A <see cref="HashSet{TableExpression}"/> containing the FROM table expressions.
     /// </value>
     protected HashSet<TableExpression> FromExpressions { get; } = new();
 
     /// <summary>
-    /// Gets the join expressions.
+    /// Gets the collection of JOIN expressions for the DELETE statement.
     /// </summary>
     /// <value>
-    /// The join expressions.
+    /// A <see cref="HashSet{JoinExpression}"/> containing the JOIN expressions.
     /// </value>
     protected HashSet<JoinExpression> JoinExpressions { get; } = new();
 
     /// <summary>
-    /// Gets the table expression.
+    /// Gets the target table expression for the DELETE statement.
     /// </summary>
     /// <value>
-    /// The table expression.
+    /// The <see cref="TableExpression"/> representing the target table.
     /// </value>
     protected TableExpression TableExpression { get; private set; }
 
-
     /// <summary>
-    /// Set the target table to delete from.
+    /// Sets the target table to delete from.
     /// </summary>
-    /// <param name="tableName">Name of the table.</param>
-    /// <param name="tableSchema">The table schema.</param>
-    /// <param name="tableAlias">The table alias.</param>
+    /// <param name="tableName">The name of the table.</param>
+    /// <param name="tableSchema">The schema of the table (optional).</param>
+    /// <param name="tableAlias">The alias for the table (optional).</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public TBuilder Table(
         string tableName,
@@ -97,15 +95,15 @@ public TBuilder Table(
         return (TBuilder)this;
     }
 
-
     /// <summary>
-    /// Add an output clause for the specified column names.
+    /// Adds an OUTPUT clause for the specified column names.
     /// </summary>
-    /// <param name="columnNames">The column names.</param>
-    /// <param name="tableAlias">The table alias.</param>
+    /// <param name="columnNames">The collection of column names to include in the OUTPUT clause.</param>
+    /// <param name="tableAlias">The alias for the table (optional).</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
+    /// <exception cref="ArgumentNullException">Thrown if <paramref name="columnNames"/> is <c>null</c>.</exception>
     public TBuilder Output(
         IEnumerable<string> columnNames,
         string tableAlias = null)
@@ -120,13 +118,13 @@ public TBuilder Output(
     }
 
     /// <summary>
-    /// Add an output clause for the specified column name.
+    /// Adds an OUTPUT clause for the specified column name.
     /// </summary>
-    /// <param name="columnName">Name of the column.</param>
-    /// <param name="tableAlias">The table alias.</param>
-    /// <param name="columnAlias">The column alias.</param>
+    /// <param name="columnName">The name of the column to include in the OUTPUT clause.</param>
+    /// <param name="tableAlias">The alias for the table (optional).</param>
+    /// <param name="columnAlias">The alias for the column (optional).</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public TBuilder Output(
         string columnName,
@@ -141,14 +139,14 @@ public TBuilder Output(
     }
 
     /// <summary>
-    /// Conditionally add an output clause for the specified column name.
+    /// Conditionally adds an OUTPUT clause for the specified column name if the condition is met.
     /// </summary>
-    /// <param name="columnName">Name of the column.</param>
-    /// <param name="tableAlias">The table alias.</param>
-    /// <param name="columnAlias">The column alias.</param>
-    /// <param name="condition">The condition.</param>
+    /// <param name="columnName">The name of the column to include in the OUTPUT clause.</param>
+    /// <param name="tableAlias">The alias for the table (optional).</param>
+    /// <param name="columnAlias">The alias for the column (optional).</param>
+    /// <param name="condition">A function that determines whether to add the OUTPUT clause. If <c>null</c>, the clause is always added.</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public TBuilder OutputIf(
         string columnName,
@@ -162,15 +160,14 @@ public TBuilder OutputIf(
         return Output(columnName, tableAlias, columnAlias);
     }
 
-
     /// <summary>
-    /// Add a from clause to the query.
+    /// Adds a FROM clause to the DELETE statement.
     /// </summary>
-    /// <param name="tableName">Name of the table.</param>
-    /// <param name="tableSchema">The table schema.</param>
-    /// <param name="tableAlias">The table alias.</param>
+    /// <param name="tableName">The name of the table to include in the FROM clause.</param>
+    /// <param name="tableSchema">The schema of the table (optional).</param>
+    /// <param name="tableAlias">The alias for the table (optional).</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public virtual TBuilder From(
         string tableName,
@@ -185,11 +182,11 @@ public virtual TBuilder From(
     }
 
     /// <summary>
-    /// Add a raw from clause to the query.
+    /// Adds a raw FROM clause to the DELETE statement.
     /// </summary>
-    /// <param name="fromClause">From clause.</param>
+    /// <param name="fromClause">The raw SQL FROM clause.</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public TBuilder FromRaw(string fromClause)
     {
@@ -200,11 +197,11 @@ public TBuilder FromRaw(string fromClause)
     }
 
     /// <summary>
-    /// Add a join clause using the specified builder action
+    /// Adds a JOIN clause to the DELETE statement using the specified builder action.
     /// </summary>
-    /// <param name="builder">The builder.</param>
+    /// <param name="builder">An action that configures the join using a <see cref="JoinBuilder"/>.</param>
     /// <returns>
-    /// The same builder so that multiple calls can be chained.
+    /// The same builder instance for method chaining.
     /// </returns>
     public TBuilder Join(Action<JoinBuilder> builder)
     {
@@ -216,7 +213,12 @@ public TBuilder Join(Action<JoinBuilder> builder)
         return (TBuilder)this;
     }
 
-    /// <inheritdoc />
+    /// <summary>
+    /// Builds the SQL DELETE statement using the current configuration.
+    /// </summary>
+    /// <returns>
+    /// A <see cref="QueryStatement"/> containing the SQL DELETE statement and its parameters.
+    /// </returns>
     public override QueryStatement BuildStatement()
     {
         var deleteStatement = new DeleteStatement(