📝 CodeRabbit Chat: Implement requested code changes by coderabbitai[bot] · Pull Request #218 · frigini/MeAjudaAi

coderabbitai · 2026-04-29T13:26:21Z

Code changes was requested by @frigini.

Sprint 13.1: Implementações e otimizações extras #215 (comment)

The following files were modified:

src/Modules/Bookings/Infrastructure/Persistence/Configurations/BookingConfiguration.cs
src/Modules/Providers/Infrastructure/Persistence/Configurations/ProviderConfiguration.cs

…t roadmap documentation

…infrastructure services

…s, PII masking, and request handling middlewares

…ve unit test coverage for core components

…ysis

…are with associated tests

coderabbitai · 2026-04-29T13:26:34Z

Important

Review skipped

This PR was authored by the user configured for CodeRabbit reviews. CodeRabbit does not review PRs authored by this user. It's recommended to use a dedicated user account to post CodeRabbit review feedback.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 06853794-7638-495e-8725-6d214d58a541

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

frigini · 2026-04-29T20:56:12Z

@coderabbitai ainda necessário este PR?

coderabbitai · 2026-04-29T20:57:02Z

Tip

For best results, initiate chat on the files or code changes.

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check PR `#218` details and its diff
gh pr view 218 --repo frigini/MeAjudaAi

echo "---DIFF---"
gh pr diff 218 --repo frigini/MeAjudaAi

echo "---PR 215 comment---"
gh api repos/frigini/MeAjudaAi/issues/comments/4344011079

Length of output: 61745

@frigini: Este PR começou como implementação dos índices compostos para BookingConfiguration e ProviderConfiguration (solicitados em #215), mas cresceu bastante e agora inclui dois grupos de mudanças distintos:

1. 📌 Alterações originais solicitadas

BookingConfiguration.cs → 3 índices compostos nomeados (ix_bookings_provider_date_status, ix_bookings_client_date, ix_bookings_provider_date)
ProviderConfiguration.cs → 4 índices compostos com filtro partial (ix_providers_deleted_*)

2. 🚀 Sprint 13.1 — Performance Zero-Allocation (não solicitado originalmente)

Refatorações com Span<T> / ReadOnlySpan<char> em vários arquivos:

GeographicRestrictionMiddleware, RequestLoggingMiddleware, PermissionOptimizationMiddleware, BusinessMetricsMiddleware
RegisterCustomerCommandHandler, KeycloakService
DocumentExtensions, PiiMaskingHelper, PhoneNumberValidator, CacheMetrics
MessagingExtensions + nova classe OptionsTopicNameConvention (fix de convenção de tópico)
Docs (roadmap.md, roadmap-history.md, technical-debt.md), CI (pin azurite:3.33.0) e novos testes

Resumo: o PR ainda é necessário se você quiser os índices compostos e/ou as otimizações de zero-allocation. Caso já tenha incorporado parte dessas mudanças em outra branch, posso fechar este PR. O que prefere?

🧠 Learnings used

Learnt from: frigini
Repo: frigini/MeAjudaAi PR: 0
File: :0-0
Timestamp: 2026-01-16T21:50:23.593Z
Learning: For frigini/MeAjudaAi Web reviews, prefer commit-by-commit (newest-first) with concise verdicts and concrete follow-ups; prioritize reuse of Shared.Contracts for enums/constants to keep Web aligned with backend/shared code.

Learnt from: frigini
Repo: frigini/MeAjudaAi PR: 0
File: :0-0
Timestamp: 2026-04-20T22:04:25.820Z
Learning: MeAjudaAi (PRs futuros): para cobertura global, NÃO excluir *Configuration e *Extensions (há smoke tests exercitando-os); também não excluir *.Monitoring.*, MeAjudaAi.Shared.Jobs.* e MeAjudaAi.Shared.Mediator.*. Manter excluídos apenas Request/Response/Dto/DTO/IntegrationEvent e *DbContextFactory; Endpoints podem ficar excluídos globalmente, a menos que sejam white-listados por módulo com E2E (ex.: +MeAjudaAi.Modules.Payments.API.*Endpoint; -*Endpoint).

Learnt from: frigini
Repo: frigini/MeAjudaAi PR: 77
File: src/Modules/Providers/Tests/Unit/Application/Queries/GetProvidersByIdsQueryTests.cs:65-77
Timestamp: 2025-12-18T00:35:33.829Z
Learning: MeAjudaAi repo: Exceção à política de idioma para comentários — em testes, os cabeçalhos AAA (Arrange, Act, Assert) devem permanecer em inglês e não devem ser traduzidos. Comentários/XML continuam em português; logs em inglês; mensagens ao usuário (validações/APIs) em português.

Learnt from: frigini
Repo: frigini/MeAjudaAi PR: 0
File: :0-0
Timestamp: 2026-04-24T20:02:36.451Z
Learning: MeAjudaAi Shared coverage: Classes de infraestrutura de messaging que dependem de broker RabbitMQ real (RabbitMqDeadLetterService, DeadLetterExtensions, FailedMessageInfo, FailureAttempt, EnvironmentMetadata) são candidatas a [ExcludeFromCodeCoverage] pois não são testáveis unitariamente. Utilitários puros (PhoneNumberValidator, PiiMaskingHelper, SlugHelper, UuidGenerator) devem ter testes unitários adicionados em vez de serem excluídos.

github-actions · 2026-04-29T21:13:08Z

✅ Contrato da API Validado

Nenhuma breaking change detectada pelo oasdiff em relação à branch base.

github-actions · 2026-04-29T21:18:48Z

Code Coverage Report

Project	Package	Line Rate	Branch Rate	Health
MeAjudaAi.Web.Admin	components/layout	73%	64%	➖
MeAjudaAi.Web.Admin	components/providers	93%	92%	✔
MeAjudaAi.Web.Admin	components/ui	100%	86%	✔
MeAjudaAi.Web.Admin	hooks/admin	95%	79%	✔
MeAjudaAi.Web.Admin	lib	100%	100%	✔
MeAjudaAi.Web.Admin	Summary	94% (256 / 272)	80% (118 / 147)	-
MeAjudaAi.Web.Customer	components/auth	62%	57%	➖
MeAjudaAi.Web.Customer	components/bookings	72%	61%	➖
MeAjudaAi.Web.Customer	components/home	100%	100%	✔
MeAjudaAi.Web.Customer	components/layout	63%	36%	➖
MeAjudaAi.Web.Customer	components/profile	93%	94%	✔
MeAjudaAi.Web.Customer	components/providers	62%	63%	➖
MeAjudaAi.Web.Customer	components/reviews	64%	70%	➖
MeAjudaAi.Web.Customer	components/search	88%	70%	✔
MeAjudaAi.Web.Customer	components/service	100%	100%	✔
MeAjudaAi.Web.Customer	components/ui	97%	97%	✔
MeAjudaAi.Web.Customer	hooks	70%	52%	➖
MeAjudaAi.Web.Customer	lib	100%	100%	✔
MeAjudaAi.Web.Customer	lib/api	83%	80%	✔
MeAjudaAi.Web.Customer	lib/constants	100%	100%	✔
MeAjudaAi.Web.Customer	lib/schemas	92%	80%	✔
MeAjudaAi.Web.Customer	lib/services	94%	86%	✔
MeAjudaAi.Web.Customer	lib/utils	77%	90%	➖
MeAjudaAi.Web.Customer	Summary	77% (731 / 945)	71% (631 / 878)	-
MeAjudaAi.Web.Provider	root	53%	36%	❌
MeAjudaAi.Web.Provider	components/dashboard	73%	52%	➖
MeAjudaAi.Web.Provider	components/layout	100%	100%	✔
MeAjudaAi.Web.Provider	components/profile	100%	100%	✔
MeAjudaAi.Web.Provider	components/providers	100%	100%	✔
MeAjudaAi.Web.Provider	components/ui	100%	90%	✔
MeAjudaAi.Web.Provider	hooks	0%	0%	❌
MeAjudaAi.Web.Provider	lib	100%	100%	✔
MeAjudaAi.Web.Provider	lib/api	100%	93%	✔
MeAjudaAi.Web.Provider	lib/constants	100%	100%	✔
MeAjudaAi.Web.Provider	lib/schemas	87%	75%	✔
MeAjudaAi.Web.Provider	lib/services	100%	100%	✔
MeAjudaAi.Web.Provider	lib/utils	96%	98%	✔
MeAjudaAi.Web.Provider	Summary	75% (314 / 416)	73% (366 / 499)	-
Overall	Summary	79% (1301 / 1633)	73% (1115 / 1524)	-

github-actions · 2026-04-29T21:23:16Z

Package	Line Rate	Branch Rate	Complexity	Health
MeAjudaAi.ApiService	96%	80%	393	➖
MeAjudaAi.Modules.Bookings.Application	91%	83%	217	➖
MeAjudaAi.Modules.Bookings.Domain	97%	98%	68	➖
MeAjudaAi.Modules.Bookings.Infrastructure	96%	39%	37	➖
MeAjudaAi.Modules.Communications.API	100%	100%	4	✔
MeAjudaAi.Modules.Communications.Application	89%	69%	206	❌
MeAjudaAi.Modules.Communications.Domain	100%	100%	17	✔
MeAjudaAi.Modules.Communications.Infrastructure	92%	57%	59	➖
MeAjudaAi.Modules.Documents.API	100%	100%	3	✔
MeAjudaAi.Modules.Documents.Application	86%	72%	191	❌
MeAjudaAi.Modules.Documents.Domain	99%	100%	29	➖
MeAjudaAi.Modules.Documents.Infrastructure	88%	76%	164	❌
MeAjudaAi.Modules.Locations.API	100%	100%	11	✔
MeAjudaAi.Modules.Locations.Application	94%	95%	70	➖
MeAjudaAi.Modules.Locations.Domain	95%	82%	127	➖
MeAjudaAi.Modules.Locations.Infrastructure	92%	75%	225	➖
MeAjudaAi.Modules.Payments.API	100%	100%	3	✔
MeAjudaAi.Modules.Payments.Application	97%	92%	55	➖
MeAjudaAi.Modules.Payments.Domain	95%	86%	105	➖
MeAjudaAi.Modules.Payments.Infrastructure	90%	71%	169	➖
MeAjudaAi.Modules.Providers.API	100%	100%	17	✔
MeAjudaAi.Modules.Providers.Application	92%	82%	277	➖
MeAjudaAi.Modules.Providers.Domain	94%	87%	285	➖
MeAjudaAi.Modules.Providers.Infrastructure	94%	75%	88	➖
MeAjudaAi.Modules.Ratings.Application	91%	100%	14	➖
MeAjudaAi.Modules.Ratings.Domain	98%	100%	24	➖
MeAjudaAi.Modules.Ratings.Infrastructure	90%	75%	23	➖
MeAjudaAi.Modules.SearchProviders.API	100%	100%	2	✔
MeAjudaAi.Modules.SearchProviders.Application	94%	73%	36	➖
MeAjudaAi.Modules.SearchProviders.Domain	99%	82%	48	➖
MeAjudaAi.Modules.SearchProviders.Infrastructure	92%	89%	59	➖
MeAjudaAi.Modules.ServiceCatalogs.API	100%	100%	3	✔
MeAjudaAi.Modules.ServiceCatalogs.Application	98%	97%	170	➖
MeAjudaAi.Modules.ServiceCatalogs.Domain	98%	93%	60	➖
MeAjudaAi.Modules.ServiceCatalogs.Infrastructure	100%	78%	56	➖
MeAjudaAi.Modules.Users.API	98%	83%	13	➖
MeAjudaAi.Modules.Users.Application	94%	91%	203	➖
MeAjudaAi.Modules.Users.Domain	95%	95%	139	➖
MeAjudaAi.Modules.Users.Infrastructure	82%	59%	196	❌
MeAjudaAi.Shared	80%	78%	1383	❌
Summary	90% (14067 / 15611)	80% (3428 / 4282)	5249	➖

Minimum allowed line rate is 90%

Filipe Frigini and others added 8 commits April 28, 2026 17:38

docs: initialize technical debt tracking, roadmap history, and projec…

43a519a

…t roadmap documentation

docs: add roadmap and technical debt tracking documentation

9fffbf2

feat: implement Keycloak integration and add utility middlewares and …

80198c7

…infrastructure services

feat: implement infrastructure for messaging options, business metric…

a678f3a

…s, PII masking, and request handling middlewares

feat: implement geographic restriction middleware and add comprehensi…

660da09

…ve unit test coverage for core components

feat: add backend CI workflow for automated testing and coverage anal…

c74fb53

…ysis

feat: add document validation extensions and business metrics middlew…

d264a38

…are with associated tests

📝 CodeRabbit Chat: Implement requested code changes

f97d396

coderabbitai Bot requested a review from frigini April 29, 2026 13:26

coderabbitai Bot assigned frigini Apr 29, 2026

coderabbitai Bot mentioned this pull request Apr 29, 2026

Sprint 13.1: Implementações e otimizações extras #215

Merged

Base automatically changed from feature/extra-implementations-minimize-tech-debts to master April 29, 2026 20:47

Merge master into coderabbitai/chat/d264a38 and resolve conflicts

9f911d3

frigini merged commit faee8f2 into master Apr 29, 2026

frigini deleted the coderabbitai/chat/d264a38 branch April 29, 2026 21:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

📝 CodeRabbit Chat: Implement requested code changes#218

📝 CodeRabbit Chat: Implement requested code changes#218
frigini merged 9 commits intomasterfrom
coderabbitai/chat/d264a38

coderabbitai Bot commented Apr 29, 2026

Uh oh!

coderabbitai Bot commented Apr 29, 2026

Review skipped

Uh oh!

frigini commented Apr 29, 2026

Uh oh!

coderabbitai Bot commented Apr 29, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

coderabbitai Bot commented Apr 29, 2026

Uh oh!

coderabbitai Bot commented Apr 29, 2026

Review skipped

Uh oh!

frigini commented Apr 29, 2026

Uh oh!

coderabbitai Bot commented Apr 29, 2026

1. 📌 Alterações originais solicitadas

2. 🚀 Sprint 13.1 — Performance Zero-Allocation (não solicitado originalmente)

Uh oh!

github-actions Bot commented Apr 29, 2026

✅ Contrato da API Validado

Uh oh!

github-actions Bot commented Apr 29, 2026

Code Coverage Report

Uh oh!

github-actions Bot commented Apr 29, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant