Conflicto de fusión en adas-core.LdapLogin/LdapLoginService.cs

adas-core.Infrastructure/Repositories/MasterListRepository.cs

@@ -20,7 +20,6 @@ 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;
@@ -31,7 +30,6 @@ 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));
@@ -43,7 +41,6 @@ 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
@@ -97,7 +94,6 @@ 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
@@ -113,11 +109,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Persists the supplied <typeparamref name="T"/> entity in the collection by replacing the document identified by <see cref="T.Id"/>, or inserting a new document when no match is found (upsert). Logs and rethrows any exception raised during the operation.
+    /// Updates an existing master list entity with full replacement.
     /// </summary>
-    /// <typeparam name="T">The document type stored in the collection.</typeparam>
-    /// <param name="entity">The entity whose <see cref="T.Id"/> locates the target document and whose state is written to the collection.</param>
-    /// <!-- aidoc:v1 sig=019b691 body=863650a -->
+    /// <param name="entity">The entity with updated values.</param>
+    /// <exception cref="Exception">Throws and re-throws exceptions after logging.</exception>
     public async Task Update(T entity)
     {
         try
@@ -138,7 +133,6 @@ 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)
@@ -167,12 +161,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously retrieves an entity of type <typeparamref name="T"/> from the collection by its <see cref="ObjectId"/>, applying a projection that truncates the <c>options</c> field to 100 elements. Returns <see langword="null"/> when no matching document is found, or when an exception is caught and logged by <see cref="Log"/>.
+    /// Finds a master list entity by its ID with options projection limited to 100 items.
     /// </summary>
-    /// <typeparam name="T">The document type stored in the collection, expected to expose an <see cref="ObjectId"/> identifier.</typeparam>
-    /// <param name="id">The <see cref="ObjectId"/> of the entity to locate.</param>
-    /// <returns>A <see cref="Task{T}"/> that resolves to the matching entity, or <see langword="null"/> if the document is not found or the operation fails.</returns>
-    /// <!-- aidoc:v1 sig=81ac96a body=1519944 -->
+    /// <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>
     public async Task<T?> FindById(ObjectId id)
     {
         try
@@ -196,13 +189,14 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves a single <see cref="OptionList"/> item from a master document identified by <paramref name="masterId"/> and <paramref name="optionId"/>, applying locale-aware name resolution. When <paramref name="locale"/> is <see cref="LocaleEnum.Default"/> or matches the document's default locale, the base name is returned; otherwise, the matching entry from the <c>localeItems</c> map is used, falling back to the base name when no translation is found. Returns <see langword="null"/> when the option is not found or if an error occurs during aggregation.
+    /// 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.
     /// </summary>
-    /// <param name="masterId">The <see cref="ObjectId"/> of the master document that contains the option collection to search.</param>
-    /// <param name="optionId">The <see cref="ObjectId"/> of the specific option within the master document's options array.</param>
-    /// <param name="locale">The <see cref="LocaleEnum"/> value that drives translation selection; <see cref="LocaleEnum.Default"/> resolves to the document's default locale, while any other value attempts a locale-specific lookup.</param>
-    /// <returns>A <see cref="Task{TResult}"/> that yields the matched <see cref="OptionList"/>, or <see langword="null"/> if no document/option is found or the operation fails.</returns>
-    /// <!-- aidoc:v1 sig=fdb2628 body=0e4983e -->
+    /// <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>
     public async Task<OptionList?> FindOptionItemById(ObjectId masterId, ObjectId optionId, LocaleEnum locale)
     {
         try
@@ -348,7 +342,6 @@ 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,12 +367,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously finds an entity of type <typeparamref name="T"/> by its MongoDB <see cref="ObjectId"/>, executing an aggregation pipeline that unwinds the <c>options</c> array and resolves localized option names. When <paramref name="locale"/> is <c>null</c> or <see cref="LocaleEnum.Default"/>, the original option name is returned; otherwise, the option name is translated using the supplied locale, falling back to the original name when no matching <c>localeItems</c> entry exists. Returns <c>null</c> when no document is found or when an exception is caught and logged.
+    /// Finds a master list entity by its ID with optional locale translation for options.
+    /// Uses MongoDB aggregation to unwind options and apply translations.
     /// </summary>
-    /// <param name="id">The MongoDB <see cref="ObjectId"/> that uniquely identifies the document to retrieve.</param>
-    /// <param name="locale">The optional <see cref="LocaleEnum"/> used to resolve translated option names; pass <c>null</c> or <see cref="LocaleEnum.Default"/> to use the default (non-translated) names.</param>
-    /// <returns>A <see cref="Task{T}"/> that yields the matched entity of type <typeparamref name="T"/>, or <c>null</c> if no document matches <paramref name="id"/> or if an error occurs during aggregation.</returns>
-    /// <!-- aidoc:v1 sig=cb4a658 body=f2e88d8 -->
+    /// <param name="id">The ObjectId of the entity to retrieve.</param>
+    /// <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>
     public async Task<T?> FindById(ObjectId id, LocaleEnum? locale)
     {
         try
@@ -557,11 +551,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously finds a single <typeparamref name="T"/> entity by its <see cref="T.Name"/> using a MongoDB query, projecting only the first 100 elements of the <c>options</c> field to limit payload size. Returns <c>null</c> if no matching document is found or if an error occurs during the search.
+    /// Finds a master list entity by its name.
     /// </summary>
-    /// <param name="name">The name value used to build the equality filter against the entity's <c>Name</c> field.</param>
-    /// <returns>A <see cref="Task{T}"/> that yields the matching <typeparamref name="T"/> instance, or <c>null</c> when no document matches or the operation fails.</returns>
-    /// <!-- aidoc:v1 sig=80a1541 body=8b5b558 -->
+    /// <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>
     public async Task<T?> FindByName(string name)
     {
         try
@@ -590,7 +584,6 @@ 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);
@@ -601,7 +594,6 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// </summary>
     /// <param name="filter">The pagination and filtering parameters.</param>
     /// <returns>A fluent queryable for MasterList results.</returns>
-    /// <!-- aidoc:v1 sig=bbb39c7 body=f42ebaa -->
     public IFindFluent<T, T> GetPaginatedMasterList(PaginationFilter filter)
     {
         var filterBuilder = Builders<T>.Filter;
@@ -624,12 +616,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves the options belonging to the master list identified by <paramref name="listId"/>, optionally narrowed down by the text contained in <paramref name="filter"/>. Returns an empty collection when no master list exists for the given identifier, and when a non-empty filter text is supplied, applies a case-insensitive match against each option's name, description, or option type.
+    /// Retrieves paginated options within a master list with optional text filtering.
     /// </summary>
-    /// <param name="filter">The <see cref="PaginationFilter"/> that provides the optional text used to filter the returned options.</param>
-    /// <param name="listId">The <see cref="ObjectId"/> of the master list whose options should be retrieved.</param>
-    /// <returns>A <see cref="Task"/> that resolves to a <see cref="List{OptionList}"/> of options matching the filter, or an empty list when the master list cannot be found.</returns>
-    /// <!-- aidoc:v1 sig=2ac8d75 body=b25fb18 -->
+    /// <param name="filter">The pagination and filtering parameters.</param>
+    /// <param name="listId">The ObjectId of the master list.</param>
+    /// <returns>A list of filtered OptionList items.</returns>
     public async Task<List<OptionList>> GetPaginatedOptions(PaginationFilter filter, ObjectId listId)
     {
         //TODO: LOCALE
@@ -649,12 +640,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Adds a new <see cref="OptionList"/> entry to the master list identified by <paramref name="id"/>. Returns the created option on success, or <c>null</c> if a matching option already exists, the master list cannot be retrieved, the update does not modify any document, or the operation throws an exception that is logged.
+    /// Adds a new option to a master list.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> of the master list to update.</param>
-    /// <param name="opt">The <see cref="FilterOptionListElement"/> describing the option to add; its locale items are reused when <see cref="FilterOptionListElement.IsFromAdmPanel"/> is <c>true</c>, otherwise new locale entries are generated from the master list's <see cref="LocaleEnum"/> default.</param>
-    /// <returns>A <see cref="Task{OptionList}"/> that resolves to the newly created <see cref="OptionList"/> when the push update succeeds, or <c>null</c> when the operation is skipped, no document is modified, or the update fails.</returns>
-    /// <!-- aidoc:v1 sig=1602f0d body=e7fd138 -->
+    /// <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>
     public async Task<OptionList?> AddOptionToMasterList(ObjectId id, FilterOptionListElement opt)
     {
         var exist = await GetMasterListByIdAndSearchOptions(id, opt);
@@ -694,11 +685,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously retrieves all entities from the underlying collection.
-    /// If an exception occurs during retrieval, the error is logged and an empty collection is returned rather than propagating the failure.
+    /// Retrieves all master list entities.
     /// </summary>
-    /// <returns>A <see cref="Task"/> that yields an <see cref="IEnumerable{T}"/> containing all retrieved entities, or an empty collection if the operation fails.</returns>
-    /// <!-- aidoc:v1 sig=3a61c61 body=40d7129 -->
+    /// <returns>An enumerable of all MasterList entities.</returns>
+    /// <exception cref="Exception">Logs errors and returns empty list on failure.</exception>
     public async Task<IEnumerable<T>> GetAll()
     {
         try
@@ -714,10 +704,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves all master list entries from the underlying collection without applying any filter and projects each result into a <see cref="MasterListDto"/>, where the <see cref="MasterListDto.Options"/> property is set to the count of associated options rather than the option list itself.
+    /// Retrieves all master list entities without options, returning only metadata.
     /// </summary>
-    /// <returns>A <see cref="Task"/> that resolves to an <see cref="IEnumerable{T}"/> containing the projected <see cref="MasterListDto"/> instances, or an empty collection if an error is encountered while querying the data store.</returns>
-    /// <!-- aidoc:v1 sig=e493435 body=1c299ba -->
+    /// <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>
     public async Task<IEnumerable<MasterListDto>> GetAllWithoutOptions()
     {
         try
@@ -742,13 +732,10 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously counts all entities in the underlying <see cref="Collection"/> by invoking
-    /// <c>CountDocumentsAsync</c> with a filter that matches every document. If the operation fails,
-    /// the exception is logged via <c>Log.Error</c> and the method returns <c>0</c> as a safe fallback.
+    /// Counts the total number of master list entities in the collection.
     /// </summary>
-    /// <returns>A <see cref="Task{Int32}"/> that represents the asynchronous count operation. The result
-    /// is the total number of entities, or <c>0</c> if an error occurred while querying the collection.</returns>
-    /// <!-- aidoc:v1 sig=f1f0a98 body=01cd3d1 -->
+    /// <returns>The total count of entities.</returns>
+    /// <exception cref="Exception">Logs errors and returns 0 on failure.</exception>
     public async Task<int> Count()
     {
         try
@@ -764,12 +751,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Retrieves the options of a master list identified by <paramref name="id"/>, applying the optional <paramref name="filters"/> to restrict the results by option type, text, name, or description, and resolving the localized <c>name</c> and <c>description</c> fields for each option based on the requested locale. When <paramref name="filters"/> is <c>null</c>, no options are returned. Text searches are performed case-insensitively against both the name and description of each option using a normalized pattern, and the final result is ordered alphabetically by name. Any exception during execution is logged and an empty list is returned.
+    /// Searches for options within a master list using multiple filter criteria.
+    /// Uses MongoDB aggregation pipeline to apply filters and locale translations.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> of the master list whose options will be queried.</param>
-    /// <param name="filters">Optional <see cref="FilterOptionListElement"/> containing the search criteria (option type, text, name, description, and locale). When <c>null</c>, the method returns an empty list.</param>
-    /// <returns>A <see cref="Task{List{OptionList}}"/> containing the filtered, localized, and alphabetically ordered options; an empty list is returned when no filters are provided, no options match, or an error occurs.</returns>
-    /// <!-- aidoc:v1 sig=f6ee5be body=17d6a96 -->
+    /// <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>
     public async Task<List<OptionList>> GetMasterListByIdAndSearchOptions(ObjectId id, FilterOptionListElement? filters)
     {
         try
@@ -978,13 +966,13 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates an option within a master list, handling duplicate detection, locale-specific translations, and conditional field updates based on the master list's required field rules.
+    /// Updates a specific option within a master list with locale-aware field updates.
     /// </summary>
-    /// <param name="id">The <see cref="MongoDB.Bson.ObjectId"/> of the master list that contains the option to update.</param>
-    /// <param name="newOpt">The <see cref="OptionList"/> containing the new values to apply to the existing option.</param>
-    /// <param name="locale">The <see cref="LocaleEnum"/> indicating which locale the <paramref name="newOpt"/> applies to; when it matches the master's default locale, the main field is updated, otherwise the translation stored in <c>localeItems</c> is updated.</param>
-    /// <returns>A <see cref="Task{T}"/> containing the updated <see cref="OptionList"/>, the existing option when a duplicate is detected, or <c>null</c> when the option is not found, no fields qualify for update, or the update operation fails.</returns>
-    /// <!-- aidoc:v1 sig=e771f23 body=5520a4a -->
+    /// <param name="id">The ObjectId of the master list.</param>
+    /// <param name="newOpt">The OptionList with updated values.</param>
+    /// <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>
     public async Task<OptionList?> UpdateMasterListOption(ObjectId id, OptionList newOpt, LocaleEnum locale)
     {
         // 1. Evitar duplicados
@@ -1079,12 +1067,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates an existing option inside the master list identified by <paramref name="id"/> with the data from <paramref name="newOpt"/>. Returns <see langword="null"/> when the master list is not found, when no document is modified, or when an exception is caught and logged.
+    /// Updates a specific option within a master list with full replacement.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> of the master list that owns the option to update.</param>
-    /// <param name="newOpt">The <see cref="OptionList"/> whose <see cref="ObjectId"/> targets the option to replace and whose values are persisted to the document.</param>
-    /// <returns>A task containing the updated <see cref="OptionList"/> when the update succeeds, or <see langword="null"/> when the master list is missing, nothing was modified, or the update failed.</returns>
-    /// <!-- aidoc:v1 sig=6def54c body=9554543 -->
+    /// <param name="id">The ObjectId of the master list.</param>
+    /// <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>
     public async Task<OptionList?> UpdateMasterListOption(ObjectId id, OptionList newOpt)
     {
         var master = await FindById(id);
@@ -1120,7 +1108,6 @@ 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
@@ -1139,12 +1126,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the option details of a master list document identified by <paramref name="id"/>, persisting only the non-null fields supplied in <paramref name="opt"/>.
+    /// Updates the metadata details for a master list.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> of the master list entry to update.</param>
-    /// <param name="opt">The <see cref="UpdateMasterListDetailsDto"/> whose non-null <see cref="UpdateMasterListDetailsDto.CanAddElement"/> and <see cref="UpdateMasterListDetailsDto.OptionListDetails"/> values are written to the document.</param>
-    /// <returns>The supplied <see cref="UpdateMasterListDetailsDto"/> when at least one field is modified, or <see langword="null"/> when the document is not found, no changes are applied, or the operation fails.</returns>
-    /// <!-- aidoc:v1 sig=53d4179 body=49f50ca -->
+    /// <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>
     public async Task<UpdateMasterListDetailsDto?> UpdateOptionDetailsToMasterList(ObjectId id,
         UpdateMasterListDetailsDto opt)
     {
@@ -1174,13 +1161,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the name of the master list document identified by <paramref name="id"/>.
-    /// Returns <c>true</c> if a document was modified, <c>false</c> if no document matched the id or if an exception was caught and logged.
+    /// Updates the name of a master list.
     /// </summary>
-    /// <param name="id">The <see cref="MongoDB.Bson.ObjectId"/> of the master list entry to update.</param>
-    /// <param name="name">The new name to assign to the entry.</param>
-    /// <returns>A <see cref="Task{Boolean}"/> that resolves to <c>true</c> when the update modified a document; otherwise, <c>false</c>.</returns>
-    /// <!-- aidoc:v1 sig=5b79a51 body=36dfe99 -->
+    /// <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>
     public async Task<bool> UpdateMasterListName(ObjectId id, string name)
     {
         var filter = Builders<T>.Filter.And(
@@ -1201,13 +1187,12 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Updates the <see cref="T.Description"/> field of the master list entry identified by <paramref name="id"/>.
-    /// Returns <c>true</c> when the matched document is modified, and <c>false</c> when no document matches the filter or the underlying update operation throws, in which case the exception is logged and the call resolves to <c>false</c>.
+    /// Updates the description of a master list.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> of the master list entry to update.</param>
-    /// <param name="description">The new description value to set on the entry.</param>
-    /// <returns>A <see cref="Task{Boolean}"/> that yields <c>true</c> if the update modified a document; otherwise, <c>false</c>.</returns>
-    /// <!-- aidoc:v1 sig=5a27a55 body=3552f46 -->
+    /// <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>
     public async Task<bool> UpdateMasterListDescription(ObjectId id, string description)
     {
         var filter = Builders<T>.Filter.And(
@@ -1234,7 +1219,6 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <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>
-    /// <!-- aidoc:v1 sig=8de50cd body=d974b9b -->
     public async Task<bool> RemoveMasterListOption(ObjectId id, OptionList oldOpt)
     {
         var filter = Builders<T>.Filter.Eq("_id", id);
@@ -1263,12 +1247,9 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Creates background MongoDB indexes on the <see cref="DiagnosisList"/> collection to optimize queries against the nested <c>options</c> document fields, configuring the indexes with <c>spanish</c> as both the default and override language for text indexing.
+    /// Creates necessary indexes for the MasterList collection.
+    /// Currently creates text indexes for DiagnosisList on options.name, options.description, and options._id.
     /// </summary>
-    /// <remarks>
-    /// When the generic type parameter is not <see cref="DiagnosisList"/>, the method performs no actions and returns without creating any indexes.
-    /// </remarks>
-    /// <!-- aidoc:v1 sig=4955da2 body=2c8d1f7 -->
     public override async Task CreateIndexes()
     {
         if (typeof(T) == typeof(DiagnosisList))
@@ -1294,7 +1275,6 @@ 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);
@@ -1307,7 +1287,6 @@ 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
@@ -1380,12 +1359,11 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Creates a find query against the underlying collection by combining the provided <paramref name="filters"/> with a logical AND and applying the specified <paramref name="sort"/>. When the filter list is empty, an empty filter is used so that the query matches every document.
+    /// Creates a fluent query for paginated results with combined filters.
     /// </summary>
-    /// <param name="filters">The list of <see cref="FilterDefinition{T}"/> predicates to combine; an empty list causes no filtering to be applied.</param>
-    /// <param name="sort">The <see cref="SortDefinition{T}"/> used to order the returned documents.</param>
-    /// <returns>An <see cref="IFindFluent{T,T}"/> configured with the combined filter and sort, ready for further chaining.</returns>
-    /// <!-- aidoc:v1 sig=704d715 body=2a35652 -->
+    /// <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>
     private IFindFluent<T, T> CreateFindFluent(List<FilterDefinition<T>> filters, SortDefinition<T> sort)
     {
         var combinedFilter = filters.Any()
@@ -1401,7 +1379,6 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     /// <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>
-    /// <!-- aidoc:v1 sig=147db59 body=9b63847 -->
     private Locale GetNewItemLocale(LocaleEnum localeList, string opt)
     {
         var newLocale = new Locale();
@@ -1441,7 +1418,6 @@ 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();
@@ -1475,13 +1451,14 @@ public class MasterListRepository<T> : MongoRepository<T>, IMasterListRepository
     }
 
     /// <summary>
-    /// Asynchronously retrieves a filtered list of <see cref="OptionList"/> entries from the master collection by matching the document with the specified <paramref name="id"/> and filtering options whose resolved name equals <paramref name="newOptName"/> for the given <paramref name="locale"/>. When the requested locale differs from the document's default, the pipeline uses the translated name from <c>localeItems</c>, falling back to the original name if no translation is available. Returns an empty list if no matching options are found or if an exception is logged during execution.
+    /// Searches for options within a master list by name with locale translation.
+    /// Uses MongoDB aggregation pipeline to apply locale-aware filtering.
     /// </summary>
-    /// <param name="id">The <see cref="ObjectId"/> identifying the master document to aggregate.</param>
-    /// <param name="newOptName">The option name used to filter the options after locale-based name resolution.</param>
-    /// <param name="locale">The <see cref="LocaleEnum"/> value used to select the translated name for each option.</param>
-    /// <returns>A <see cref="Task"/> containing a <see cref="List{T}"/> of <see cref="OptionList"/> entries ordered by name, or an empty list when no match exists or an error is caught.</returns>
-    /// <!-- aidoc:v1 sig=342dc63 body=21f31fc -->
+    /// <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>
     private async Task<List<OptionList>> GetMasterListByIdAndTextSearch(
         ObjectId id, string newOptName, LocaleEnum locale)
     {

adas-core.LdapLogin/LdapLoginService.cs

@@ -20,7 +20,6 @@ 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>
@@ -52,7 +51,6 @@ 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,
@@ -85,7 +83,6 @@ public class LdapLoginService : ILoginService
     /// <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>
-    /// <!-- aidoc:v1 sig=bdfb451 body=e3fc1ac -->
     public async Task<User> Login(string username, string password)
     {
         if (_ldapConfig.Server == null)
@@ -162,7 +159,6 @@ public class LdapLoginService : ILoginService
     /// <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>
-    /// <!-- aidoc:v1 sig=851dc90 -->
     public Task<User> Login(HttpContext context)
         => throw new LoginServicesException("Not implemented");
 
@@ -173,7 +169,6 @@ public class LdapLoginService : ILoginService
     /// <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>
-    /// <!-- aidoc:v1 sig=a0b1f46 -->
     public Task<User> Authenticate(string username, string password)
         => throw new LoginServicesException("Not implemented");
 
@@ -183,7 +178,6 @@ 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");
 
@@ -193,26 +187,23 @@ 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");
 
     /// <summary>
-    /// Retrieves a <see cref="User"/> matching the specified <paramref name="username"/>.
+    /// This method is not implemented in the LdapLoginService, as the user retrieval process is handled through the Login(string username, string password) method and the GetOrCreateUser(User userEntryLdap, LdapEntry entry) method.
     /// </summary>
-    /// <param name="username">The username to look up.</param>
-    /// <returns>A <see cref="Task{TResult}"/> that resolves to the <see cref="User"/> if found, or <see langword="null"/> when no matching user exists.</returns>
-    /// <exception cref="LoginServicesException">Thrown because the operation is not yet implemented.</exception>
-    /// <!-- aidoc:v1 sig=5184c30 -->
+    /// <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>
     public Task<User?> GetByUsername(string username)
         => throw new LoginServicesException("Not implemented");
 
     /// <summary>
-    /// Retrieves all <see cref="User"/> entities from the system. This method is not yet implemented and currently throws a <see cref="LoginServicesException"/> when invoked.
+    /// This method is not implemented in the LdapLoginService, as the user retrieval process is handled through the Login(string username, string password) method and the GetOrCreateUser(User userEntryLdap, LdapEntry entry) method.
     /// </summary>
-    /// <returns>A <see cref="Task{List{User}}"/> that represents the asynchronous operation, intended to contain the full collection of <see cref="User"/> entries.</returns>
-    /// <exception cref="LoginServicesException">Thrown unconditionally because the method has not been implemented.</exception>
-    /// <!-- aidoc:v1 sig=aa066d6 -->
+    /// <returns>A task representing the asynchronous operation.</returns>
+    /// <exception cref="LoginServicesException">Thrown when the method is not implemented.</exception>
     public Task<List<User>> GetAllUsers()
         => throw new LoginServicesException("Not implemented");
 
@@ -224,7 +215,6 @@ 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)
 
     {
@@ -249,7 +239,6 @@ public class LdapLoginService : ILoginService
     /// <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>
-    /// <!-- aidoc:v1 sig=d2bafd5 body=4af15eb -->
     private async Task<List<Authorization>> CheckAuthorities(User user, LdapEntry entry)
 
     {
@@ -300,7 +289,6 @@ 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)
 
     {
@@ -339,11 +327,10 @@ public class LdapLoginService : ILoginService
 
 
     /// <summary>
-    /// Maps an <see cref="LdapEntry"/> to a <see cref="User"/> object, populating the username from the configured attribute and, when the corresponding properties are configured, composing the user's name from the first and last name attributes.
+    /// 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.
     /// </summary>
-    /// <param name="ldapEntry">The <see cref="LdapEntry"/> from which the <see cref="User"/> properties are read.</param>
-    /// <returns>A <see cref="User"/> instance populated from the LDAP entry attributes.</returns>
-    /// <!-- aidoc:v1 sig=947181a body=33ffec8 -->
+    /// <param name="ldapEntry">The LDAP entry containing the user's information.</param>
+    /// <returns>The existing or newly created user with updated authorities.</returns>
     private User GetUser(LdapEntry ldapEntry)
 
     {
@@ -378,7 +365,6 @@ 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)
 
     {