update infra docs

web/backend/Infrastructure/Infrastructure.Sql/DefaultSqlConnectionFactory.cs

@@ -8,15 +8,11 @@ namespace Infrastructure.Sql;
 ///     Default <see cref="ISqlConnectionFactory" /> implementation that creates SQL Server connections,
 ///     resolving the connection string from environment variables or application configuration.
 /// </summary>
-/// <param name="configuration">The application configuration, used as a fallback source for the connection string.</param>
 public class DefaultSqlConnectionFactory(IConfiguration configuration) : ISqlConnectionFactory
 {
     private readonly string _connectionString = GetConnectionString(configuration);
 
-    /// <summary>
-    ///     Creates a new, unopened <see cref="SqlConnection" /> using the resolved connection string.
-    /// </summary>
-    /// <returns>A new <see cref="SqlConnection" /> instance.</returns>
+    /// <inheritdoc />
     public DbConnection CreateConnection()
     {
         return new SqlConnection(_connectionString);
@@ -24,15 +20,10 @@ public class DefaultSqlConnectionFactory(IConfiguration configuration) : ISqlCon
 
     /// <summary>
     ///     Resolves the SQL Server connection string, preferring (in order): the <c>DB_CONNECTION_STRING</c>
-    ///     environment variable, a connection string built from individual <c>DB_*</c> environment variables
-    ///     via <see cref="SqlConnectionStringHelper.BuildConnectionString" />, and finally the <c>"Default"</c>
-    ///     connection string from <paramref name="configuration" />.
+    ///     environment variable, a connection string built from individual <c>DB_*</c> environment variables,
+    ///     and finally the <c>"Default"</c> connection string from configuration.
     /// </summary>
-    /// <param name="configuration">The application configuration to fall back to.</param>
-    /// <returns>The resolved SQL Server connection string.</returns>
-    /// <exception cref="InvalidOperationException">
-    ///     Thrown when no connection string can be resolved from any of the supported sources.
-    /// </exception>
+    /// <exception cref="InvalidOperationException">Thrown when no connection string can be resolved from any source.</exception>
     private static string GetConnectionString(IConfiguration configuration)
     {
         // Check for full connection string first

web/backend/Infrastructure/Infrastructure.Sql/ISqlConnectionFactory.cs

@@ -10,6 +10,5 @@ public interface ISqlConnectionFactory
     /// <summary>
     ///     Creates a new, unopened database connection.
     /// </summary>
-    /// <returns>A new <see cref="DbConnection" /> instance.</returns>
     DbConnection CreateConnection();
 }

web/backend/Infrastructure/Infrastructure.Sql/Repository.cs

@@ -6,15 +6,12 @@ namespace Infrastructure.Sql;
 ///     Base class for ADO.NET-based repositories, providing shared connection creation and
 ///     entity-mapping infrastructure for derived repository implementations.
 /// </summary>
-/// <typeparam name="T">The entity type managed by the repository.</typeparam>
-/// <param name="connectionFactory">The factory used to create database connections.</param>
 public abstract class Repository<T>(ISqlConnectionFactory connectionFactory)
     where T : class
 {
     /// <summary>
     ///     Creates and opens a new database connection using the configured <see cref="ISqlConnectionFactory" />.
     /// </summary>
-    /// <returns>An open <see cref="DbConnection" /> ready for use.</returns>
     /// <exception cref="DbException">Thrown when the connection cannot be opened.</exception>
     protected async Task<DbConnection> CreateConnection()
     {
@@ -26,7 +23,5 @@ public abstract class Repository<T>(ISqlConnectionFactory connectionFactory)
     /// <summary>
     ///     Maps the current row of a data reader to an instance of <typeparamref name="T" />.
     /// </summary>
-    /// <param name="reader">The data reader positioned on the row to map.</param>
-    /// <returns>The mapped entity instance.</returns>
     protected abstract T MapToEntity(DbDataReader reader);
 }

web/backend/Infrastructure/Infrastructure.Sql/SqlConnectionStringHelper.cs

@@ -9,10 +9,9 @@ public static class SqlConnectionStringHelper
 {
     /// <summary>
     ///     Builds a SQL Server connection string from environment variables.
-    ///     Expects DB_SERVER, DB_NAME, DB_USER, DB_PASSWORD, and DB_TRUST_SERVER_CERTIFICATE.
     /// </summary>
-    /// <param name="databaseName">Optional override for the database name. If null, uses DB_NAME env var.</param>
-    /// <returns>A properly formatted SQL Server connection string.</returns>
+    /// <remarks>Throws <see cref="InvalidOperationException" /> if DB_SERVER, DB_NAME, DB_USER, or DB_PASSWORD is unset.</remarks>
+    /// <param name="databaseName">If null, falls back to the DB_NAME environment variable.</param>
     public static string BuildConnectionString(string? databaseName = null)
     {
         string server =
@@ -48,7 +47,6 @@ public static class SqlConnectionStringHelper
     /// <summary>
     ///     Builds a connection string to the master database using environment variables.
     /// </summary>
-    /// <returns>A connection string for the master database.</returns>
     public static string BuildMasterConnectionString()
     {
         return BuildConnectionString("master");