docs(iec62304): [REL-1.0.2] apply curated XML doc review updates

adas-core.Infrastructure/Repositories/MasterListRepository.cs

@@ -20,6 +20,7 @@ namespace adas_core.Infrastructure.Repositories;
 /// options lists, diagnoses, allergies, procedures, treatments, and other reference data.
 /// </summary>
 /// <typeparam name="T">The type of MasterList entity to manage.</typeparam>
+/// <!-- aidoc:v1 sig=f958de2 -->
 public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository<T> where T : MasterList
 {
     private readonly ApiSettings _apiSettings;
@@ -30,6 +31,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="apiSettings">API settings containing collection names configuration.</param>
     /// <param name="database">The MongoDB database instance.</param>
     /// <exception cref="ArgumentNullException">Thrown when apiSettings is null.</exception>
+    /// <!-- aidoc:v1 sig=800d538 body=d2b18a3 -->
     public MasterListRepository(IOptions<ApiSettings> apiSettings, IMongoDatabase database) : base(database)
     {
         if (apiSettings == null) throw new ArgumentNullException(nameof(apiSettings));
@@ -41,6 +43,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// Maps different MasterList subtypes to their corresponding MongoDB collection names.
     /// </summary>
     /// <returns>The collection name for the current MasterList type.</returns>
+    /// <!-- aidoc:v1 sig=94e22ff body=64e6c55 -->
     public override string GetCollectionName()
     {
         return typeof(T) switch
@@ -94,6 +97,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// </summary>
     /// <param name="id">The ObjectId of the entity to delete.</param>
     /// <exception cref="Exception">Throws and re-throws exceptions after logging.</exception>
+    /// <!-- aidoc:v1 sig=3de1ad6 body=fe0ba8c -->
     public async Task Delete(ObjectId id)
     {
         try
@@ -109,10 +113,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates an existing master list entity with full replacement.
+    /// Replaces the document in the collection matching the entity's identifier, or inserts it when no match exists (upsert behavior). Failures are logged and rethrown to the caller.
     /// </summary>
-    /// <param name="entity">The entity with updated values.</param>
-    /// <exception cref="Exception">Throws and re-throws exceptions after logging.</exception>
+    /// <param name="entity">The entity to upsert; its identifier is used as the filter and <paramref name="entity"/>'s current state replaces the matched document.</param>
+    /// <!-- aidoc:v1 sig=019b691 body=863650a -->
     public async Task Update(T entity)
     {
         try
@@ -133,6 +137,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="id">The ObjectId of the master list.</param>
     /// <param name="newOpt">The OptionList with updated values.</param>
     /// <returns>The updated OptionList if found; otherwise, null.</returns>
+    /// <!-- aidoc:v1 sig=64ca4dd body=a7714e6 -->
     public async Task<OptionList?> UpdateFullMasterListOption(ObjectId id, OptionList newOpt)
     {
         var filter = Builders<T>.Filter.Where(o => o.Id == id && o.Options.Any(opt => opt.Id == newOpt.Id)
@@ -161,11 +166,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Finds a master list entity by its ID with options projection limited to 100 items.
+    /// Asynchronously retrieves an entity of type <c>T</c> matching the specified <paramref name="id"/>, projecting the "options" array to at most 100 elements. Returns <c>null</c> when no matching document is found or when an exception is caught, with the error logged.
     /// </summary>
-    /// <param name="id">The ObjectId of the entity to retrieve.</param>
-    /// <returns>The MasterList entity if found; otherwise, null.</returns>
-    /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> identifier of the entity to locate.</param>
+    /// <returns>A <see cref="Task{T}"/> that yields the matching entity, or <c>null</c> when the entity is not found or an error occurs.</returns>
+    /// <!-- aidoc:v1 sig=81ac96a body=1519944 -->
     public async Task<T?> FindById(ObjectId id)
     {
         try
@@ -189,14 +194,15 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Finds a specific option within a master list by master and option IDs with locale translation.
-    /// Uses MongoDB aggregation to apply translations and return the translated option.
+    /// Finds a specific option item within a master document using MongoDB aggregation, returning the localized <see cref="OptionList"/> or <c>null</c> if not found.
+    /// Uses the provided <paramref name="locale"/> to resolve translations: when the locale is <see cref="LocaleEnum.Default"/> or matches the document's default locale, the option's main name is returned; otherwise, the matching entry in the option's <c>localeItems</c> is used, falling back to the main name when no translation exists.
+    /// Any exception raised during the aggregation or deserialization is logged and surfaced as a <c>null</c> result.
     /// </summary>
-    /// <param name="masterId">The ObjectId of the master list.</param>
-    /// <param name="optionId">The ObjectId of the option to retrieve.</param>
-    /// <param name="locale">The locale for translation.</param>
-    /// <returns>The OptionList with translated fields if found; otherwise, null.</returns>
-    /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <param name="masterId">The <see cref="ObjectId"/> of the master document that contains the options array.</param>
+    /// <param name="optionId">The <see cref="ObjectId"/> of the specific option to retrieve.</param>
+    /// <param name="locale">The <see cref="LocaleEnum"/> value used to select the appropriate translation.</param>
+    /// <returns>A <see cref="Task{OptionList}"/> that resolves to the matching <see cref="OptionList"/>, or <c>null</c> when the option is not found or an error occurs.</returns>
+    /// <!-- aidoc:v1 sig=fdb2628 body=0e4983e -->
     public async Task<OptionList?> FindOptionItemById(ObjectId masterId, ObjectId optionId, LocaleEnum locale)
     {
         try
@@ -342,6 +348,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="optionId">The ObjectId of the option to retrieve.</param>
     /// <returns>The OptionList if found; otherwise, null.</returns>
     /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <!-- aidoc:v1 sig=ccc34cb body=71837cc -->
     public async Task<OptionList?> FindOptionItemById(ObjectId masterId, ObjectId optionId)
     {
         try
@@ -374,6 +381,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="locale">Optional locale for translated option names.</param>
     /// <returns>The MasterList entity with translated options if found; otherwise, null.</returns>
     /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <!-- aidoc:v1 sig=cb4a658 body=f2e88d8 -->
     public async Task<T?> FindById(ObjectId id, LocaleEnum? locale)
     {
         try
@@ -551,11 +559,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Finds a master list entity by its name.
+    /// Asynchronously retrieves the first entity whose <c>Name</c> matches the supplied <paramref name="name"/>, applying a projection that caps the <c>options</c> array at 100 elements. If no document is found, or the query fails, the task resolves to <c>null</c>.
     /// </summary>
-    /// <param name="name">The name of the master list to retrieve.</param>
-    /// <returns>The MasterList entity if found; otherwise, null.</returns>
-    /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <param name="name">The value compared against the entity's <c>Name</c> property to build the equality filter.</param>
+    /// <returns>A <see cref="Task{T}"/> producing the first matching entity, or <c>null</c> when no match exists or the operation is aborted by a caught exception.</returns>
+    /// <!-- aidoc:v1 sig=80a1541 body=8b5b558 -->
     public async Task<T?> FindByName(string name)
     {
         try
@@ -584,16 +592,18 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="id">The ObjectId of the master list.</param>
     /// <param name="textSearch">Optional text to search within options.</param>
     /// <returns>A list of matching OptionList items.</returns>
+    /// <!-- aidoc:v1 sig=569d223 body=bb7f2c6 -->
     public async Task<List<OptionList>> GetMasterListByIdAndTextSearchContaining(ObjectId id, string? textSearch)
     {
         return await GetOptionsByTextSearch(textSearch, id);
     }
 
     /// <summary>
-    /// Retrieves paginated master lists with optional text filtering.
+    /// Builds a paginated query against the master collection of <typeparamref name="T"/>, applying a case-insensitive substring match on the <see cref="PaginationFilter.FilteredRequest"/> text against the <c>Name</c> property when provided. Sorts results ascending by <c>name</c> and returns a fluent find interface so callers can continue chaining pagination or projection operations.
     /// </summary>
-    /// <param name="filter">The pagination and filtering parameters.</param>
-    /// <returns>A fluent queryable for MasterList results.</returns>
+    /// <param name="filter">The <see cref="PaginationFilter"/> containing pagination options and the optional <see cref="FilteredRequest.Text"/> used to match against <c>Name</c>.</param>
+    /// <returns>An <see cref="IFindFluent{TDocument, TProjection}"/> representing the sorted (and optionally text-filtered) query, ready for further pagination configuration.</returns>
+    /// <!-- aidoc:v1 sig=bbb39c7 body=f42ebaa -->
     public IFindFluent<T, T> GetPaginatedMasterList(PaginationFilter filter)
     {
         var filterBuilder = Builders<T>.Filter;
@@ -640,12 +650,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Adds a new option to a master list.
+    /// Adds a new <see cref="OptionList"/> entry to the master list identified by <paramref name="id"/>, returning <c>null</c> when an equivalent option already exists, the database update does not modify any document, or the operation fails.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="opt">The option element to add.</param>
-    /// <returns>The newly created OptionList if successful; otherwise, null if duplicate exists.</returns>
-    /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list (document of type <c>T</c>) that will receive the new option.</param>
+    /// <param name="opt">The <see cref="FilterOptionListElement"/> describing the option to add, including its name, optional visual properties, and locale information.</param>
+    /// <returns>A <see cref="Task{T}"/> that yields the newly created <see cref="OptionList"/> when the push update succeeds, or <c>null</c> when a duplicate is detected, no document is modified, or an exception is logged and swallowed.</returns>
+    /// <!-- aidoc:v1 sig=1602f0d body=e7fd138 -->
     public async Task<OptionList?> AddOptionToMasterList(ObjectId id, FilterOptionListElement opt)
     {
         var exist = await GetMasterListByIdAndSearchOptions(id, opt);
@@ -685,10 +695,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves all master list entities.
+    /// Asynchronously retrieves all entities of type T from the underlying collection.
+    /// If an exception occurs during retrieval, the error is logged and an empty collection is returned as a fallback.
     /// </summary>
-    /// <returns>An enumerable of all MasterList entities.</returns>
-    /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
+    /// <returns>A task that represents the asynchronous operation. The task result contains an <see cref="IEnumerable{T}"/> of all entities, or an empty collection if an error was encountered.</returns>
+    /// <!-- aidoc:v1 sig=3a61c61 body=40d7129 -->
     public async Task<IEnumerable<T>> GetAll()
     {
         try
@@ -704,10 +715,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves all master list entities without options, returning only metadata.
+    /// Retrieves all master lists from the underlying collection and projects each entity into a <see cref="MasterListDto"/>,
+    /// where the <see cref="MasterListDto.Options"/> field holds the count of associated options instead of the full options collection.
+    /// If the operation fails, the exception is logged and an empty collection is returned.
     /// </summary>
-    /// <returns>An enumerable of MasterListDto containing id, name, description, listType, and options count.</returns>
-    /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
+    /// <returns>A task that represents the asynchronous operation, yielding an <see cref="IEnumerable{MasterListDto}"/> of all
+    /// projected master lists, or an empty collection if an error occurs.</returns>
+    /// <!-- aidoc:v1 sig=e493435 body=1c299ba -->
     public async Task<IEnumerable<MasterListDto>> GetAllWithoutOptions()
     {
         try
@@ -732,10 +746,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Counts the total number of master list entities in the collection.
+    /// Asynchronously counts all entities in the collection by invoking <c>CountDocumentsAsync</c> with a filter that matches every document. If the operation fails, the exception is logged and the method returns <c>0</c> as a safe fallback.
     /// </summary>
-    /// <returns>The total count of entities.</returns>
-    /// <exception cref="Exception">Logs errors and returns 0 on failure.</exception>
+    /// <returns>A <see cref="Task{Int32}"/> that yields the total number of entities, or <c>0</c> if an error occurs.</returns>
+    /// <!-- aidoc:v1 sig=f1f0a98 body=01cd3d1 -->
     public async Task<int> Count()
     {
         try
@@ -751,13 +765,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Searches for options within a master list using multiple filter criteria.
-    /// Uses MongoDB aggregation pipeline to apply filters and locale translations.
+    /// Retrieves the options of a master list identified by <paramref name="id"/>, applying the supplied <paramref name="filters"/> and resolving localized names and descriptions for the requested locale.
+    /// When <paramref name="filters"/> is <c>null</c>, or when the aggregation yields no document, an empty list is returned; any exception is logged and also results in an empty list.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="filters">The filter criteria including text, name, description, and optionType.</param>
-    /// <returns>A list of matching OptionList items ordered by name.</returns>
-    /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list to retrieve.</param>
+    /// <param name="filters">The optional <see cref="FilterOptionListElement"/> that defines the search criteria (option type, text, name, and description) and the target locale used to translate each option.</param>
+    /// <returns>A <see cref="Task{List{OptionList}}"/> of <see cref="OptionList"/> entries matching the filters, ordered by name, or an empty list when no results are found.</returns>
+    /// <!-- aidoc:v1 sig=f6ee5be body=17d6a96 -->
     public async Task<List<OptionList>> GetMasterListByIdAndSearchOptions(ObjectId id, FilterOptionListElement? filters)
     {
         try
@@ -973,6 +987,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="locale">The locale for translation updates.</param>
     /// <returns>The updated OptionList if successful; otherwise, null.</returns>
     /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <!-- aidoc:v1 sig=e771f23 body=5520a4a -->
     public async Task<OptionList?> UpdateMasterListOption(ObjectId id, OptionList newOpt, LocaleEnum locale)
     {
         // 1. Evitar duplicados
@@ -1073,6 +1088,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="newOpt">The OptionList with updated values.</param>
     /// <returns>The updated OptionList if successful; otherwise, null.</returns>
     /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <!-- aidoc:v1 sig=6def54c body=9554543 -->
     public async Task<OptionList?> UpdateMasterListOption(ObjectId id, OptionList newOpt)
     {
         var master = await FindById(id);
@@ -1108,6 +1124,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="id">The ObjectId of the master list.</param>
     /// <param name="deleteOptId">The ObjectId of the option to delete.</param>
     /// <returns>True if the option was deleted; otherwise, false.</returns>
+    /// <!-- aidoc:v1 sig=e663f16 body=75627bc -->
     public async Task<bool> DeleteMasterListOption(ObjectId id, ObjectId deleteOptId)
     {
         // Define el filtro para encontrar el documento por su _id
@@ -1126,12 +1143,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the metadata details for a master list.
+    /// Updates the option details of an existing master list entry identified by <paramref name="id"/>, applying only the fields from <paramref name="opt"/> that are not <see langword="null"/> (currently <see cref="UpdateMasterListDetailsDto.CanAddElement"/> and <see cref="UpdateMasterListDetailsDto.OptionListDetails"/>).
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="opt">The UpdateMasterListDetailsDto with updated values.</param>
-    /// <returns>The updated UpdateMasterListDetailsDto if successful; otherwise, null.</returns>
-    /// <exception cref="Exception">Logs errors and returns null on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the document to update in the collection.</param>
+    /// <param name="opt">The <see cref="UpdateMasterListDetailsDto"/> carrying the new values; <see langword="null"/> properties are left unchanged.</param>
+    /// <returns>The <paramref name="opt"/> instance if the document was found and modified; otherwise, <see langword="null"/> when the document was not found, no fields were modified, or the operation failed.</returns>
+    /// <!-- aidoc:v1 sig=53d4179 body=49f50ca -->
     public async Task<UpdateMasterListDetailsDto?> UpdateOptionDetailsToMasterList(ObjectId id,
         UpdateMasterListDetailsDto opt)
     {
@@ -1161,12 +1178,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the name of a master list.
+    /// Updates the name of a master list entry identified by <paramref name="id"/> in the underlying collection. Returns <c>true</c> when the document was modified, or <c>false</c> if no document matched the identifier or an exception was caught and logged.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="name">The new name.</param>
-    /// <returns>True if the update was successful; otherwise, false.</returns>
-    /// <exception cref="Exception">Logs errors and returns false on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list document to update.</param>
+    /// <param name="name">The new name to assign to the master list entry.</param>
+    /// <returns><c>true</c> if a document was modified; otherwise, <c>false</c>.</returns>
+    /// <!-- aidoc:v1 sig=5b79a51 body=36dfe99 -->
     public async Task<bool> UpdateMasterListName(ObjectId id, string name)
     {
         var filter = Builders<T>.Filter.And(
@@ -1187,12 +1204,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the description of a master list.
+    /// Updates the description of a master list entry identified by <paramref name="id"/> in the MongoDB collection, returning <see langword="true"/> when a document was modified and <see langword="false"/> when no document matched or a caught <see cref="Exception"/> was logged.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="description">The new description.</param>
-    /// <returns>True if the update was successful; otherwise, false.</returns>
-    /// <exception cref="Exception">Logs errors and returns false on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list entry to update.</param>
+    /// <param name="description">The new description to set on the entry.</param>
+    /// <returns>A <see cref="Task{Boolean}"/> that resolves to <see langword="true"/> if the update modified a document; otherwise, <see langword="false"/> when the entry was not found or an error was logged.</returns>
+    /// <!-- aidoc:v1 sig=5a27a55 body=3552f46 -->
     public async Task<bool> UpdateMasterListDescription(ObjectId id, string description)
     {
         var filter = Builders<T>.Filter.And(
@@ -1213,12 +1230,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Removes an option from a master list by matching all its properties.
+    /// Removes the specified option from the master list document identified by <paramref name="id"/> by issuing a pull filter against the MongoDB collection that matches all of the option's properties (name, option type, icons, colors, default flag, and description). Returns true when the document is modified, or false if no match is found or if an exception is caught and logged.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="oldOpt">The OptionList to remove.</param>
-    /// <returns>True if the option was removed; otherwise, false.</returns>
-    /// <exception cref="Exception">Logs errors and returns false on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list document to update.</param>
+    /// <param name="oldOpt">The <see cref="OptionList"/> whose property values define the filter used to locate and remove the matching element.</param>
+    /// <returns>A <see cref="Task{Boolean}"/> that resolves to true if the document was modified; otherwise, false.</returns>
+    /// <!-- aidoc:v1 sig=8de50cd body=d974b9b -->
     public async Task<bool> RemoveMasterListOption(ObjectId id, OptionList oldOpt)
     {
         var filter = Builders<T>.Filter.Eq("_id", id);
@@ -1247,9 +1264,9 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Creates necessary indexes for the MasterList collection.
-    /// Currently creates text indexes for DiagnosisList on options.name, options.description, and options._id.
+    /// Overrides the base index creation to build non-unique, background MongoDB indexes on the <c>options.name</c>, <c>options.description</c>, and <c>options._id</c> fields of the <see cref="DiagnosisList"/> collection, configuring Spanish as the default and override language for text tokenization. The logic only runs when <typeparamref name="T"/> is <see cref="DiagnosisList"/>; for any other type the method is a no-op.
     /// </summary>
+    /// <!-- aidoc:v1 sig=4955da2 body=2c8d1f7 -->
     public override async Task CreateIndexes()
     {
         if (typeof(T) == typeof(DiagnosisList))
@@ -1275,6 +1292,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// </summary>
     /// <param name="textSearch">Optional text to search within options.</param>
     /// <returns>A list of matching OptionList items.</returns>
+    /// <!-- aidoc:v1 sig=484b092 body=48e3496 -->
     public async Task<List<OptionList>> GetMasterListByTextSearch(string? textSearch)
     {
         return await GetOptionsByTextSearch(textSearch);
@@ -1287,6 +1305,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <param name="id">Optional master list ObjectId to filter results.</param>
     /// <returns>A list of matching OptionList items ordered by name.</returns>
     /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
+    /// <!-- aidoc:v1 sig=6ebeac8 body=1d7bbb7 -->
     private async Task<List<OptionList>> GetOptionsByTextSearch(string? textSearch, ObjectId? id = null)
     {
         try
@@ -1359,11 +1378,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Creates a fluent query for paginated results with combined filters.
+    /// Creates a find fluent query that applies the provided <paramref name="filters"/> combined with a logical AND, falling back to an empty filter (matching all documents) when no filters are supplied, and orders the results by the specified <paramref name="sort"/>.
     /// </summary>
-    /// <param name="filters">List of filter definitions to apply.</param>
-    /// <param name="sort">Sort definition for the query results.</param>
-    /// <returns>A fluent queryable for T results.</returns>
+    /// <param name="filters">A list of <see cref="FilterDefinition{T}"/> criteria to combine. When empty, an empty filter is used so that all documents match.</param>
+    /// <param name="sort">The <see cref="SortDefinition{T}"/> that defines the ordering of the returned documents.</param>
+    /// <returns>An <see cref="IFindFluent{T,T}"/> configured with the combined filter and the given sort.</returns>
+    /// <!-- aidoc:v1 sig=704d715 body=2a35652 -->
     private IFindFluent<T, T> CreateFindFluent(List<FilterDefinition<T>> filters, SortDefinition<T> sort)
     {
         var combinedFilter = filters.Any()
@@ -1373,12 +1393,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Generates new locale items for a master list option based on a default locale.
-    /// Creates LocaleItem entries for all locales except the specified default.
+    /// Creates a new <see cref="Locale"/> instance populated with <see cref="LocaleItem"/> entries for every <see cref="LocaleEnum"/> value except <see cref="LocaleEnum.Default"/> and the locale specified by <paramref name="localeList"/>. Each property of <see cref="Locale"/> matching an included enum name is dynamically assigned a new <see cref="LocaleItem"/> whose <see cref="LocaleItem.Name"/> is set to the value of <paramref name="opt"/>.
     /// </summary>
-    /// <param name="localeList">The default locale to exclude from translations.</param>
-    /// <param name="opt">The option name to use as default translation.</param>
-    /// <returns>A Locale object with translations for all other locales.</returns>
+    /// <param name="localeList">The locale to exclude from the generated <see cref="Locale"/> object.</param>
+    /// <param name="opt">The name assigned to every <see cref="LocaleItem"/> created in the resulting <see cref="Locale"/>.</param>
+    /// <returns>A new <see cref="Locale"/> object containing the corresponding <see cref="LocaleItem"/> entries.</returns>
+    /// <!-- aidoc:v1 sig=147db59 body=9b63847 -->
     private Locale GetNewItemLocale(LocaleEnum localeList, string opt)
     {
         var newLocale = new Locale();
@@ -1418,6 +1438,7 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// </summary>
     /// <param name="input">The input string to build the pattern from.</param>
     /// <returns>A regex-compatible pattern string.</returns>
+    /// <!-- aidoc:v1 sig=2992c00 body=ae88daf -->
     private static string BuildRegexPattern(string input)
     {
         var regexPattern = new StringBuilder();
@@ -1451,14 +1472,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Searches for options within a master list by name with locale translation.
-    /// Uses MongoDB aggregation pipeline to apply locale-aware filtering.
+    /// Retrieves the options of a master list document identified by <paramref name="id"/>, filtering them by the option name <paramref name="newOptName"/> resolved against the specified <paramref name="locale"/>. When the requested locale differs from the document's default locale, the method attempts to use the translated name; if no translation exists, it falls back to the original name. Returns an empty list when no document is found, when no options match, or when an error is logged.
     /// </summary>
-    /// <param name="id">The ObjectId of the master list.</param>
-    /// <param name="newOptName">The option name to search for.</param>
-    /// <param name="locale">The locale for translation.</param>
-    /// <returns>A list of matching OptionList items ordered by name.</returns>
-    /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
+    /// <param name="id">The <see cref="ObjectId"/> of the master list document to match in the aggregation pipeline.</param>
+    /// <param name="newOptName">The option name to filter by after the locale-based name resolution.</param>
+    /// <param name="locale">The <see cref="LocaleEnum"/> value used to select the translated name; the matching translation key is derived from its lowercased string representation.</param>
+    /// <returns>A <see cref="Task{T}"/> of <see cref="List{T}"/> of <see cref="OptionList"/> containing the matching options ordered by name, or an empty list when nothing is found or an exception occurs.</returns>
+    /// <!-- aidoc:v1 sig=342dc63 body=21f31fc -->
     private async Task<List<OptionList>> GetMasterListByIdAndTextSearch(
         ObjectId id, string newOptName, LocaleEnum locale)
     {

adas-core.LdapLogin/LdapLoginService.cs

@@ -20,6 +20,7 @@ namespace adas_core.LdapLogin;
 /// The service also handles the creation of new users in the application if they do not already exist, based on the LDAP information. 
 /// It uses configuration settings for connecting to the LDAP server and for mapping LDAP attributes to user properties and authorities.
 /// </summary>
+/// <!-- aidoc:v1 sig=4e5637f -->
 public class LdapLoginService : ILoginService
 {
     /// <summary>
@@ -51,6 +52,7 @@ public class LdapLoginService : ILoginService
     /// <param name="userService">The user service for managing user information.</param>
     /// <param name="authorityService">The authority service for managing user authorities.</param>
     /// <param name="logger">The logger for logging LDAP login operations.</param>
+    /// <!-- aidoc:v1 sig=f1b8e4e body=6c4718f -->
     public LdapLoginService(
         IOptions<LdapConfig> ldapConfig,
         IValidator<LdapConfig> validator,
@@ -76,13 +78,14 @@ public class LdapLoginService : ILoginService
     public UserEnum.LoginMethod Method => UserEnum.LoginMethod.Ldap;
 
     /// <summary>
-    /// Authenticates a user against the LDAP server using the provided username and password.
+    /// Authenticates a user against the configured LDAP server using <paramref name="username"/> and <paramref name="password"/>, and resolves the matching <see cref="User"/> entry. When an LDAP service account is configured it is used for the bind; otherwise the username is optionally prefixed with the configured domain and bound with the supplied password. Searches the directory for the user entry and creates or retrieves the local <see cref="User"/> record.
     /// </summary>
-    /// <param name="username">The username of the user to authenticate.</param>
-    /// <param name="password">The password of the user to authenticate.</param>
-    /// <returns>The authenticated user.</returns>
-    /// <exception cref="LoginServicesException">Thrown when there is an error during the login process.</exception>
-    /// <exception cref="UserNotFoundException">Thrown when the user is not found in the LDAP directory.</exception>
+    /// <param name="username">The username used to bind to LDAP and to locate the user entry in the directory.</param>
+    /// <param name="password">The password used for the LDAP bind when no service account is configured.</param>
+    /// <returns>The <see cref="User"/> resolved from the LDAP directory entry.</returns>
+    /// <exception cref="LoginServicesException">Thrown when the LDAP server is not configured, when the directory entry is not found, or when the resolved user is null.</exception>
+    /// <exception cref="UserNotFoundException">Thrown when the LDAP bind fails for the supplied credentials.</exception>
+    /// <!-- aidoc:v1 sig=bdfb451 body=e3fc1ac -->
     public async Task<User> Login(string username, string password)
     {
         if (_ldapConfig.Server == null)
@@ -154,21 +157,25 @@ public class LdapLoginService : ILoginService
     }
 
     /// <summary>
-    /// This method is not implemented in the LdapLoginService, as the login process is handled through the Login(string username, string password) method.
+    /// Authenticates a user based on the current <see cref="HttpContext"/>.
+    /// This method is not yet implemented and always throws a <see cref="LoginServicesException"/>.
     /// </summary>
-    /// <param name="context">The HTTP context of the request.</param>
-    /// <returns>A task representing the asynchronous operation.</returns>
-    /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <param name="context">The <see cref="HttpContext"/> of the incoming HTTP request used to extract authentication information.</param>
+    /// <returns>A <see cref="Task{TResult}"/> that resolves to the authenticated user.</returns>
+    /// <exception cref="LoginServicesException">Always thrown because the method is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=851dc90 -->
     public Task<User> Login(HttpContext context)
         => throw new LoginServicesException("Not implemented");
 
     /// <summary>
-    /// This method is not implemented in the LdapLoginService, as the authentication process is handled through the Login(string username, string password) method.
+    /// Authenticates a user with the provided <paramref name="username"/> and <paramref name="password"/> and returns the corresponding <see cref="User"/>.
+    /// This method is not implemented and always throws a <see cref="LoginServicesException"/> when invoked.
     /// </summary>
     /// <param name="username">The username of the user to authenticate.</param>
     /// <param name="password">The password of the user to authenticate.</param>
-    /// <returns>A task representing the asynchronous operation.</returns>
-    /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <returns>A <see cref="Task{User}"/> that yields the authenticated <see cref="User"/>.</returns>
+    /// <exception cref="LoginServicesException">Thrown because authentication is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=a0b1f46 -->
     public Task<User> Authenticate(string username, string password)
         => throw new LoginServicesException("Not implemented");
 
@@ -178,6 +185,7 @@ public class LdapLoginService : ILoginService
     /// <param name="id">The ID of the user to retrieve.</param>
     /// <returns>A task representing the asynchronous operation.</returns>
     /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=877a869 -->
     public Task<User?> GetById(ObjectId id)
         => throw new LoginServicesException("Not implemented");
 
@@ -187,6 +195,7 @@ public class LdapLoginService : ILoginService
     /// <param name="email">The email of the user to retrieve.</param>
     /// <returns>A task representing the asynchronous operation.</returns>
     /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=ba92b09 -->
     public Task<User?> GetByEmail(string email)
         => throw new LoginServicesException("Not implemented");
 
@@ -196,6 +205,7 @@ public class LdapLoginService : ILoginService
     /// <param name="username">The username of the user to retrieve.</param>
     /// <returns>A task representing the asynchronous operation.</returns>
     /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=5184c30 -->
     public Task<User?> GetByUsername(string username)
         => throw new LoginServicesException("Not implemented");
 
@@ -204,6 +214,7 @@ public class LdapLoginService : ILoginService
     /// </summary>
     /// <returns>A task representing the asynchronous operation.</returns>
     /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
+    /// <!-- aidoc:v1 sig=aa066d6 -->
     public Task<List<User>> GetAllUsers()
         => throw new LoginServicesException("Not implemented");
 
@@ -215,6 +226,7 @@ public class LdapLoginService : ILoginService
     /// <param name="userEntryLdap">The user information obtained from the LDAP entry.</param>
     /// <param name="entry">The LDAP entry containing the user's information.</param>
     /// <returns>The existing or newly created user with updated authorities.</returns>
+    /// <!-- aidoc:v1 sig=87de1c2 body=a43bebd -->
     private async Task<User?> GetOrCreateUser(User userEntryLdap, LdapEntry entry)
 
     {
@@ -234,11 +246,14 @@ public class LdapLoginService : ILoginService
 
 
     /// <summary>
-    /// This method checks the authorities of a user based on the LDAP entry and the application's configuration.
+    /// Synchronizes the authorities of a <paramref name="user"/> with the authorities defined in the supplied <paramref name="entry"/>, updating existing records or inserting new ones as appropriate.
+    /// Authorities appearing in the whitelist are preserved and excluded from the LDAP map, while remaining LDAP authorities are compared against the current persisted list; entries with <see cref="Authorization.CanUpdate"/> set to <c>true</c> have their <see cref="Authorization.Rol"/> refreshed via <see cref="AuthorityService.updateOne"/>, otherwise they are inserted with <see cref="AuthorityService.InsertOne"/>.
+    /// On any failure, the error is logged and an empty list is returned.
     /// </summary>
-    /// <param name="user">The user whose authorities are being checked.</param>
-    /// <param name="entry">The LDAP entry containing the user's information.</param>
-    /// <returns>A list of updated authorities for the user.</returns>
+    /// <param name="user">The user whose authorities are being reconciled.</param>
+    /// <param name="entry">The <see cref="LdapEntry"/> providing the source authorities map and whitelist.</param>
+    /// <returns>The updated <see cref="Authorization"/> list retrieved from the authority service.</returns>
+    /// <!-- aidoc:v1 sig=d2bafd5 body=4af15eb -->
     private async Task<List<Authorization>> CheckAuthorities(User user, LdapEntry entry)
 
     {
@@ -289,6 +304,7 @@ public class LdapLoginService : ILoginService
     /// <param name="entry">The LDAP entry containing the user's information.</param>
     /// <param name="user">The user whose authorities are being retrieved.</param>
     /// <returns>A list of authorities for the user based on the whitelist.</returns>
+    /// <!-- aidoc:v1 sig=a8263f0 body=9e6cc38 -->
     private List<Authorization> GetAuthoritiesWhiteList(LdapEntry entry, User user)
 
     {
@@ -327,10 +343,13 @@ public class LdapLoginService : ILoginService
 
 
     /// <summary>
-    /// This method retrieves a list of authorities for a user based on the LDAP entry and the application's configuration for mapping LDAP groups to authorities.
+    /// Maps the supplied <see cref="LdapEntry"/> to a <see cref="User"/> instance by reading attributes whose names are configured in the LDAP settings.
+    /// The <see cref="User.UserName"/> is taken from the configured user name property, falling back to an empty string when the attribute is absent.
+    /// When the first name and/or last name properties are configured, their attribute values are used to build the <see cref="User.Name"/>; otherwise the name is left unset, and the last name is appended to the first name when both are present.
     /// </summary>
-    /// <param name="ldapEntry">The LDAP entry containing the user's information.</param>
-    /// <returns>The existing or newly created user with updated authorities.</returns>
+    /// <param name="ldapEntry">The <see cref="LdapEntry"/> whose attributes are read to populate the resulting <see cref="User"/>.</param>
+    /// <returns>A <see cref="User"/> populated from the attributes exposed by <paramref name="ldapEntry"/> according to the current LDAP configuration.</returns>
+    /// <!-- aidoc:v1 sig=947181a body=33ffec8 -->
     private User GetUser(LdapEntry ldapEntry)
 
     {
@@ -365,6 +384,7 @@ public class LdapLoginService : ILoginService
     /// <param name="ldapEntry">The LDAP entry containing the user's information.</param>
     /// <param name="user">The user whose authorities are being retrieved.</param>
     /// <returns>A list of authorities for the user based on the LDAP entry and the application's configuration.</returns>
+    /// <!-- aidoc:v1 sig=1e1f0f2 body=18015c0 -->
     private List<Authorization> GetAuthoritiesMap(LdapEntry ldapEntry, User user)
 
     {