refactor: improve repository consistency and builder pattern

- Add Include(Category) to GetByIdsAsync and GetAllAsync for consistency
- Add AsNoTracking to all read queries for performance
- Fix type inference in GetAllAsync (explicit IQueryable<Service>)
- Align ServiceBuilder with domain patterns (ServiceCategoryId.New())
- Fix nullability in WithDescription to match domain model
- Document Guid.Empty validation strategy in UpdateServiceCommand
- Fix location.md heading separation (add blank line before heading)
- All 104 Catalogs tests passing

docs/modules/location.md

@@ -419,7 +419,9 @@ public class SearchableProvider
 ### **Otimizações Futuras**
 - [ ] Cache Redis para CEPs (TTL: 24h)
 - [ ] Warm-up de circuit breakers no startup
-        - [ ] Métricas customizadas (Polly telemetry)## 🚀 Próximos Passos
+        - [ ] Métricas customizadas (Polly telemetry)
+
+## 🚀 Próximos Passos
 
 ### **Fase 2 - Geocoding**
 - [ ] Implementar `GeocodingService`

src/Modules/Catalogs/Application/Commands/Service/UpdateServiceCommand.cs

@@ -7,6 +7,7 @@ namespace MeAjudaAi.Modules.Catalogs.Application.Commands.Service;
 /// <summary>
 /// Command to update an existing service's details.
 /// Validation limits must match ValidationConstants.CatalogLimits.
+/// Note: Guid.Empty validation is handled by the command handler to provide domain-specific error messages.
 /// </summary>
 public sealed record UpdateServiceCommand(
     Guid Id,

src/Modules/Catalogs/Infrastructure/Persistence/Repositories/ServiceRepository.cs

@@ -20,6 +20,7 @@ public async Task<IReadOnlyList<Service>> GetByIdsAsync(IEnumerable<ServiceId> i
         var idList = ids.ToList();
         return await context.Services
             .AsNoTracking()
+            .Include(s => s.Category)
             .Where(s => idList.Contains(s.Id))
             .ToListAsync(cancellationToken);
     }
@@ -36,7 +37,9 @@ public async Task<IReadOnlyList<Service>> GetByIdsAsync(IEnumerable<ServiceId> i
 
     public async Task<IReadOnlyList<Service>> GetAllAsync(bool activeOnly = false, CancellationToken cancellationToken = default)
     {
-        var query = context.Services.AsNoTracking().AsQueryable();
+        IQueryable<Service> query = context.Services
+            .AsNoTracking()
+            .Include(s => s.Category);
 
         if (activeOnly)
             query = query.Where(s => s.IsActive);

src/Modules/Catalogs/Tests/Builders/ServiceBuilder.cs

@@ -18,7 +18,7 @@ public ServiceBuilder()
             .CustomInstantiator(f =>
             {
                 var service = Service.Create(
-                    _categoryId ?? new ServiceCategoryId(Guid.NewGuid()),
+                    _categoryId ?? ServiceCategoryId.New(),
                     _name ?? f.Commerce.ProductName(),
                     _description ?? f.Commerce.ProductDescription(),
                     _displayOrder >= 0 ? _displayOrder : f.Random.Int(1, 100)
@@ -52,7 +52,7 @@ public ServiceBuilder WithName(string name)
         return this;
     }
 
-    public ServiceBuilder WithDescription(string description)
+    public ServiceBuilder WithDescription(string? description)
     {
         _description = description;
         return this;

tests/MeAjudaAi.E2E.Tests/Modules/Catalogs/CatalogsEndToEndTests.cs

@@ -58,7 +58,7 @@ public async Task GetServiceCategories_Should_Return_All_Categories()
         var content = await response.Content.ReadAsStringAsync();
         var result = JsonSerializer.Deserialize<JsonElement>(content, JsonOptions);
         result.TryGetProperty("data", out var data).Should().BeTrue();
-        
+
         var categories = data.Deserialize<ServiceCategoryDto[]>(JsonOptions);
         categories.Should().NotBeNull();
         categories!.Length.Should().BeGreaterThanOrEqualTo(3, "should have at least the 3 created categories");
@@ -110,7 +110,7 @@ public async Task CreateService_Should_Reject_Invalid_CategoryId()
 
         // Assert - Should reject with BadRequest or NotFound
         response.StatusCode.Should().BeOneOf(HttpStatusCode.BadRequest, HttpStatusCode.NotFound, HttpStatusCode.UnprocessableEntity);
-        
+
         var content = await response.Content.ReadAsStringAsync();
         content.Should().NotBeNullOrEmpty();
     }
@@ -132,7 +132,7 @@ public async Task GetServicesByCategory_Should_Return_Filtered_Results()
         var content = await response.Content.ReadAsStringAsync();
         var result = JsonSerializer.Deserialize<JsonElement>(content, JsonOptions);
         result.TryGetProperty("data", out var data).Should().BeTrue();
-        
+
         var services = data.Deserialize<ServiceListDto[]>(JsonOptions);
         services.Should().NotBeNull();
         services!.Length.Should().Be(3, "should return exactly 3 services for this category");
@@ -162,7 +162,7 @@ public async Task UpdateServiceCategory_Should_Modify_Existing_Category()
         // Verify the category was actually updated
         var getResponse = await ApiClient.GetAsync($"/api/v1/catalogs/categories/{category.Id.Value}");
         getResponse.StatusCode.Should().Be(HttpStatusCode.OK);
-        
+
         var updatedCategory = await ReadJsonAsync<ServiceCategoryDto>(getResponse);
         updatedCategory.Should().NotBeNull();
         updatedCategory!.Name.Should().Be(updateRequest.Name);