refactor: code quality improvements from review feedback

Changes:
- Removed redundant unique index on composite primary key in ProviderServiceConfiguration
  (primary key already enforces uniqueness on ProviderId + ServiceId)
- Removed redundant await Task.CompletedTask in DocumentVerifiedIntegrationEventHandler
- Fixed parameter naming convention: SearchProvidersModuleApi -> searchProvidersModuleApi
  (PascalCase to camelCase for consistency)
- Added XML documentation to public API methods:
  - ProvidersModuleApi.GetProviderForIndexingAsync
  - ServiceCatalogsModuleApi.ValidateServicesAsync
- Fixed markdown formatting in documentation:
  - Converted bold emphasis to proper headings in architecture.md
  - Fixed conjunction formatting in skipped-tests-analysis.md

All changes improve code quality, maintainability, and consistency.

Refs: Code review feedback on code quality and documentation

docs/architecture.md

@@ -1246,7 +1246,8 @@ public interface IDocumentsModuleApi : IModuleApi
 }
 ```
 
-**2. Implementar em Module/Application/ModuleApi**
+##### 2. Implementar em Module/Application/ModuleApi
+
 ```csharp
 [ModuleApi("Documents", "1.0")]
 public sealed class DocumentsModuleApi(IQueryDispatcher queryDispatcher) : IDocumentsModuleApi
@@ -1260,12 +1261,14 @@ public sealed class DocumentsModuleApi(IQueryDispatcher queryDispatcher) : IDocu
 }
 ```
 
-**3. Registrar em DI (Module/Application/Extensions.cs)**
+##### 3. Registrar em DI (Module/Application/Extensions.cs)
+
 ```csharp
 services.AddScoped<IDocumentsModuleApi, DocumentsModuleApi>();
 ```
 
-**4. Injetar e Usar em Outro Módulo**
+##### 4. Injetar e Usar em Outro Módulo
+
 ```csharp
 public sealed class ActivateProviderCommandHandler(
     IDocumentsModuleApi documentsApi) // ✅ Cross-module dependency

docs/skipped-tests-analysis.md

@@ -181,7 +181,7 @@ mas conceitualmente deveria ser internal.
 **Impacto**: Violação de Onion Architecture (Infrastructure vazando para fora do módulo).
 
 **Mitigação Atual**:
-- DbContext está `public` mas **não** exposto via DI para outros módulos
+- DbContext está `public`, mas não é exposto via DI para outros módulos
 - Documentação clara que DbContext **não** deve ser usado externamente
 - Migrations controladas via CLI tools, não via código
 

src/Modules/Providers/Application/ModuleApi/ProvidersModuleApi.cs

@@ -276,6 +276,12 @@ public async Task<Result<IReadOnlyList<ModuleProviderBasicDto>>> GetProvidersByV
         );
     }
 
+    /// <summary>
+    /// Gets provider data prepared for search indexing, including coordinates obtained via geocoding.
+    /// </summary>
+    /// <param name="providerId">The unique identifier of the provider to index.</param>
+    /// <param name="cancellationToken">Cancellation token.</param>
+    /// <returns>Provider indexing DTO with geocoded location, or null if provider not found, or failure if geocoding fails.</returns>
     public async Task<Result<ModuleProviderIndexingDto?>> GetProviderForIndexingAsync(
         Guid providerId,
         CancellationToken cancellationToken = default)

src/Modules/Providers/Infrastructure/Events/Handlers/Integration/DocumentVerifiedIntegrationEventHandler.cs

@@ -61,9 +61,6 @@ public async Task HandleAsync(DocumentVerifiedIntegrationEvent integrationEvent,
                 integrationEvent.DocumentType,
                 integrationEvent.ProviderId,
                 provider.Status);
-
-            // Não precisa salvar mudanças pois não estamos modificando o provider aqui
-            await Task.CompletedTask;
         }
         catch (Exception ex)
         {

src/Modules/Providers/Infrastructure/Events/Handlers/ProviderVerificationStatusUpdatedDomainEventHandler.cs

@@ -18,7 +18,7 @@ namespace MeAjudaAi.Modules.Providers.Infrastructure.Events.Handlers;
 public sealed class ProviderVerificationStatusUpdatedDomainEventHandler(
     IMessageBus messageBus,
     ProvidersDbContext context,
-    ISearchProvidersModuleApi SearchProvidersModuleApi,
+    ISearchProvidersModuleApi searchProvidersModuleApi,
     ILogger<ProviderVerificationStatusUpdatedDomainEventHandler> logger) : IEventHandler<ProviderVerificationStatusUpdatedDomainEvent>
 {
     /// <summary>
@@ -45,7 +45,7 @@ public async Task HandleAsync(ProviderVerificationStatusUpdatedDomainEvent domai
             {
                 logger.LogInformation("Provider {ProviderId} verified, indexing in SearchProviders module", domainEvent.AggregateId);
 
-                var indexResult = await SearchProvidersModuleApi.IndexProviderAsync(provider.Id.Value, cancellationToken);
+                var indexResult = await searchProvidersModuleApi.IndexProviderAsync(provider.Id.Value, cancellationToken);
                 if (indexResult.IsFailure)
                 {
                     logger.LogError("Failed to index provider {ProviderId} in search: {Error}",
@@ -62,7 +62,7 @@ public async Task HandleAsync(ProviderVerificationStatusUpdatedDomainEvent domai
                 logger.LogInformation("Provider {ProviderId} status changed to {Status}, removing from search index", 
                     domainEvent.AggregateId, domainEvent.NewStatus);
 
-                var removeResult = await SearchProvidersModuleApi.RemoveProviderAsync(provider.Id.Value, cancellationToken);
+                var removeResult = await searchProvidersModuleApi.RemoveProviderAsync(provider.Id.Value, cancellationToken);
                 if (removeResult.IsFailure)
                 {
                     logger.LogError("Failed to remove provider {ProviderId} from search: {Error}",

src/Modules/Providers/Infrastructure/Persistence/Configurations/ProviderServiceConfiguration.cs

@@ -33,10 +33,5 @@ public void Configure(EntityTypeBuilder<ProviderService> builder)
         // Índice para busca eficiente por ServiceId
         builder.HasIndex(ps => ps.ServiceId)
             .HasDatabaseName("ix_provider_services_service_id");
-
-        // Índice composto para queries comuns
-        builder.HasIndex(ps => new { ps.ProviderId, ps.ServiceId })
-            .IsUnique()
-            .HasDatabaseName("ix_provider_services_provider_service");
     }
 }

src/Modules/ServiceCatalogs/Application/ModuleApi/ServiceCatalogsModuleApi.cs

@@ -228,6 +228,12 @@ public async Task<Result<bool>> IsServiceActiveAsync(
         }
     }
 
+    /// <summary>
+    /// Validates that the specified service IDs exist in the catalog.
+    /// </summary>
+    /// <param name="serviceIds">Collection of service IDs to validate.</param>
+    /// <param name="cancellationToken">Cancellation token.</param>
+    /// <returns>Validation result indicating which services are valid.</returns>
     public async Task<Result<ModuleServiceValidationResultDto>> ValidateServicesAsync(
         IReadOnlyCollection<Guid> serviceIds,
         CancellationToken cancellationToken = default)