PetitionForm — Lead dentro do try pode transformar assinatura commitada em erro (retry duplica) #117

New issue

Closed

opened 2026-08-19 23:51:27 +00:00 by fsolla · 1 comment

fsolla commented

2026-08-19 23:51:27 +00:00

Owner

depends

serializes

priority

kind

petition-form-lead-try

chore

Origem: simplify/capture-review-debts do S10 (#94, 2026-08-19).

No PetitionForm (abaixo-assinados), trackMetaLead(pixelId, title, crypto.randomUUID()) roda DENTRO do try do submit (src/components/PetitionForm.tsx:61-63). Se crypto.randomUUID lançar (contexto não-secure — HTTP/staging; navegadores antigos), o catch mostra "Falha ao enviar assinatura" mesmo com a assinatura já commitada — o usuário tenta de novo, criando assinatura duplicada (ou recusa de telefone único) e re-disparando o Lead.

O CampaignNewsletterForm (S10) tinha o mesmo padrão e foi corrigido no S10 (guarda interna try/catch: tracking nunca falha uma captura commitada). O PetitionForm deve receber o mesmo tratamento (~4 linhas).

Ação: envolver o bloco de tracking do PetitionForm em try/catch próprio, ou mover para fora do try de submit — padrão idêntico ao aplicado em CampaignNewsletterForm.

--- id: petition-form-lead-try depends: [] serializes: [] priority: P3 kind: chore --- **Origem:** simplify/capture-review-debts do S10 (#94, 2026-08-19). No `PetitionForm` (abaixo-assinados), `trackMetaLead(pixelId, title, crypto.randomUUID())` roda DENTRO do `try` do submit (src/components/PetitionForm.tsx:61-63). Se `crypto.randomUUID` lançar (contexto não-secure — HTTP/staging; navegadores antigos), o `catch` mostra "Falha ao enviar assinatura" mesmo com a assinatura já commitada — o usuário tenta de novo, criando assinatura duplicada (ou recusa de telefone único) e re-disparando o `Lead`. O `CampaignNewsletterForm` (S10) tinha o mesmo padrão e foi corrigido no S10 (guarda interna try/catch: tracking nunca falha uma captura commitada). O `PetitionForm` deve receber o mesmo tratamento (~4 linhas). **Ação:** envolver o bloco de tracking do `PetitionForm` em try/catch próprio, ou mover para fora do try de submit — padrão idêntico ao aplicado em `CampaignNewsletterForm`.