Skip to content

Teste: desconto com double e relógio da máquina (deve aprovar com alertas) - #12

Closed
robertosrjr wants to merge 1 commit into
mainfrom
teste/pr-com-alertas
Closed

robertosrjr wants to merge 1 commit into
mainfrom
teste/pr-com-alertas

Conversation

@robertosrjr

Copy link
Copy Markdown
Owner

Validação ponta a ponta. double para valor e LocalDate.now() no domínio. Esperado: aprovado com alertas FIN-MONEY-001 e ARCH-TIME-001 (políticas em warn). Não será mergeado.

🤖 Generated with Claude Code

@github-actions

Copy link
Copy Markdown

🛡️ Governança — ✅ Aprovado

Aprovado com 2 achado(s) não bloqueante(s).

bundle 1.5.0 · motor 1.7.1 · políticas 895466052722 · LLM google/gemini-3.5-flash-lite · Jev jev-1.13.0

Achados

Severidade Política Local Origem Descrição
⚠️ MAJOR ARCH-TIME-001 application/pedidos/src/main/java/com/robertosrjr/pedidos/domain/model/DiscountPolicy.java:12 deterministic Relógio ou fuso da máquina no domínio/aplicação. Injete Clock e use fuso explícito.
⚠️ CRITICAL FIN-MONEY-001 application/pedidos/src/main/java/com/robertosrjr/pedidos/domain/model/DiscountPolicy.java:9 deterministic Valor monetário declarado como double/float. Use BigDecimal ou um value object de dinheiro.

⛔ bloqueia · ⚠️ não bloqueia · 🔎 achado de LLM (consultivo). Exceções: waiver no repositório central, nunca por comentário.

/** Desconto de fim de semana. Versão com problemas de propósito (alertas esperados). */
public class DiscountPolicy {

private double valorDesconto = 0.05;
private double valorDesconto = 0.05;

public boolean appliesToday() {
DayOfWeek day = LocalDate.now().getDayOfWeek();
@robertosrjr
robertosrjr deleted the teste/pr-com-alertas branch October 5, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants