@danhachuel/leadmaster 0.1.20 → 0.1.21

package/CHANGELOG.md

@@ -37,6 +37,29 @@ normaliza para a forma canônica (ex.: `0.1.0`).
    falha se `process.stdin.resume();` desaparecer, se `.unref()` aparecer, se o
    ciclo de vida vinculado for removido ou se os handlers globais sumirem.
 
+## [0.1.21] — 2026-11-09
+
+### Adicionado
+
+- **Módulo de Webhooks revisado conforme especificação:**
+  - `GET /api/webhook` agora retorna também **`ultimo_teste`** (registrado a cada
+    teste bem-sucedido no `config.json`).
+  - `PUT /api/webhook` e `POST /api/webhook/test` **validam a URL**: só
+    `http://`/`https://` são aceitos (400 com mensagem acionável).
+  - Novo evento **`campanha.iniciada`** — disparado quando uma campanha começa a
+    rodar; migração aditiva garante que ele apareça nas configurações existentes.
+  - Frontend `frontend/src/pages/Webhook.tsx` reformulado: eventos exibidos
+    diretamente e limpos (**Lead Criado**, **Campanha Iniciada**, **Campanha
+    Finalizada** — Lei de Hick), **prevenção de erros** (Nielsen): URL fora do
+    padrão http(s) bloqueia o salvamento e marca o input com **borda vermelha**,
+    botão **"Salvar Configurações"** em gradiente (Fitts) e **"Testar Webhook"**
+    em outline contrastante (von Restorff), toast de sucesso/erro no teste e card
+    **"Último teste realizado em: [data/hora]"**.
+- **7 novos testes pytest** (`backend/tests/test_webhook_routes.py`): leitura do
+  padrão, salvamento de URL/eventos com persistência, URL inválida (400), teste
+  com sucesso (mock do `httpx.post`, registra `ultimo_teste`), teste com falha de
+  conexão sem quebrar (mock de `httpx.ConnectError`) e URL de teste inválida.
+
 ## [0.1.20] — 2026-11-09
 
 ### Adicionado

package/backend/main.py

@@ -712,21 +712,47 @@ def atendimento_chat(dados: ChatIn):
 
 @app.get("/api/webhook")
 def obter_webhook():
-    return storage.get_config().get("webhook", {})
+    webhook = storage.get_config().get("webhook", {})
+    return {
+        "url": webhook.get("url", ""),
+        "eventos": webhook.get("eventos", []),
+        "ultimo_teste": webhook.get("ultimo_teste"),
+    }
+
+
+def _url_valida(url: str) -> bool:
+    return isinstance(url, str) and (url.startswith("http://") or url.startswith("https://"))
 
 
 @app.put("/api/webhook")
 def salvar_webhook(patch: dict):
     config = storage.get_config()
     webhook = config.get("webhook", {})
+    url = patch.get("url", webhook.get("url", ""))
+    if url and not _url_valida(url):
+        raise HTTPException(400, "URL inválida: deve começar com http:// ou https://.")
     webhook.update({k: v for k, v in patch.items() if k in ("url", "eventos")})
     storage.update_config({"webhook": webhook})
-    return webhook
+    return {
+        "url": webhook.get("url", ""),
+        "eventos": webhook.get("eventos", []),
+        "ultimo_teste": webhook.get("ultimo_teste"),
+    }
 
 
 @app.post("/api/webhook/test")
 def testar_webhook(dados: WebhookTestIn):
-    return agents.testar_webhook(dados.url)
+    if dados.url and not _url_valida(dados.url):
+        raise HTTPException(400, "URL inválida: deve começar com http:// ou https://.")
+    resultado = agents.testar_webhook(dados.url)
+    if resultado.get("ok"):
+        # Registra o último teste bem-sucedido no config.json.
+        config = storage.get_config()
+        webhook = config.get("webhook", {})
+        webhook["ultimo_teste"] = storage.now_iso()
+        storage.update_config({"webhook": webhook})
+        resultado["ultimo_teste"] = webhook["ultimo_teste"]
+    return resultado
 
 
 # ---------------------------------------------------------------------------

package/backend/services/agents.py

@@ -100,6 +100,7 @@ def iniciar_campanha(campaign_id: str) -> dict:
     campaign.setdefault("erros", 0)
     campaign.setdefault("eventos", [])
     storage.upsert_campaign(campaign)
+    disparar_webhook("campanha.iniciada", {"campanha_id": campaign_id, "nome": campaign["nome"]})
 
     threading.Thread(target=_executar_campanha, args=(campaign_id,), daemon=True).start()
     return {"ok": True, "mensagem": "Campanha iniciada."}

package/backend/services/storage.py

@@ -135,7 +135,7 @@ DEFAULT_CONFIG = {
     },
     "webhook": {
         "url": "",
-        "eventos": ["lead.criado", "lead.etapa", "campanha.concluida"],
+        "eventos": ["lead.criado", "campanha.iniciada", "campanha.concluida"],
     },
 }
 
@@ -189,11 +189,16 @@ def _limpar_deprecated(config: dict) -> dict:
                 mudou = True
     # Evento de webhook do recurso removido.
     webhook = config.get("webhook")
-    if isinstance(webhook, dict) and isinstance(webhook.get("eventos"), list):
-        eventos = [e for e in webhook["eventos"] if e != "video.publicado"]
-        if eventos != webhook["eventos"]:
-            webhook["eventos"] = eventos
-            mudou = True
+    if isinstance(webhook, dict):
+        if isinstance(webhook.get("eventos"), list):
+            eventos = [e for e in webhook["eventos"] if e != "video.publicado"]
+            # Migração aditiva: o evento "campanha.iniciada" passou a existir
+            # (0.1.21) — garantimos que configs antigas também o ofereçam.
+            if "campanha.iniciada" not in eventos:
+                eventos.append("campanha.iniciada")
+            if eventos != webhook["eventos"]:
+                webhook["eventos"] = eventos
+                mudou = True
     # Dados órfãos do pipeline de vídeos: o arquivo só existia para esse recurso.
     pipeline_path = storage_dir() / PIPELINE_FILE
     if pipeline_path.exists():

package/backend/tests/test_webhook_routes.py

@@ -0,0 +1,83 @@
+"""Testes das rotas de Webhook (backend/main.py).
+
+GET/PUT /api/webhook (com validação de URL) e POST /api/webhook/test com o
+HTTP externo mockado — o endpoint de teste nunca quebra com URL inválida ou
+indisponível. HOME isolado em diretório temporário.
+"""
+
+from unittest.mock import patch
+
+import httpx
+import pytest
+from fastapi.testclient import TestClient
+
+import main
+
+
+@pytest.fixture()
+def client(tmp_path, monkeypatch):
+    monkeypatch.setenv("LEADMASTER_HOME", str(tmp_path))
+    with TestClient(main.app) as c:
+        yield c
+
+
+def test_obter_padrao(client):
+    r = client.get("/api/webhook")
+    assert r.status_code == 200
+    corpo = r.json()
+    assert corpo["url"] == ""
+    assert "lead.criado" in corpo["eventos"]
+    assert corpo["ultimo_teste"] is None
+
+
+def test_salvar_url_e_eventos(client):
+    r = client.put(
+        "/api/webhook",
+        json={"url": "https://n8n.exemplo.com/webhook/abc", "eventos": ["lead.criado"]},
+    )
+    assert r.status_code == 200
+    corpo = r.json()
+    assert corpo["url"] == "https://n8n.exemplo.com/webhook/abc"
+    assert corpo["eventos"] == ["lead.criado"]
+    # Persistido no config.json
+    salvo = client.get("/api/webhook").json()
+    assert salvo["url"] == "https://n8n.exemplo.com/webhook/abc"
+
+
+def test_salvar_url_invalida_400(client):
+    r = client.put("/api/webhook", json={"url": "nao-e-uma-url"})
+    assert r.status_code == 400
+    assert "http" in r.json()["detail"]
+
+
+def _resposta_ok(*_args, **_kwargs):
+    return httpx.Response(200, json={"ok": True})
+
+
+def test_testar_webhook_sucesso(client):
+    client.put("/api/webhook", json={"url": "https://exemplo.com/hook"})
+    with patch.object(main.agents.httpx, "post", side_effect=_resposta_ok):
+        r = client.post("/api/webhook/test", json={"url": "https://exemplo.com/hook"})
+    assert r.status_code == 200
+    corpo = r.json()
+    assert corpo["ok"] is True
+    assert corpo["status_http"] == 200
+    assert corpo["ultimo_teste"]  # registrado
+
+
+def test_testar_webhook_conexao_falha_nao_quebra(client):
+    # URL indisponível: retorna erro amigável (não 500).
+    with patch.object(main.agents.httpx, "post", side_effect=httpx.ConnectError("sem rede")):
+        r = client.post("/api/webhook/test", json={"url": "https://inexistente.invalid/hook"})
+    assert r.status_code == 200
+    assert r.json()["ok"] is False
+    assert "mensagem" in r.json()
+
+
+def test_testar_webhook_url_invalida_400(client):
+    # Passa o min_length do modelo, mas falha no esquema http(s) → 400.
+    r = client.post("/api/webhook/test", json={"url": "nao-http-url.com/webhook"})
+    assert r.status_code == 400
+    # URL curta demais falha na validação do modelo.
+    r2 = client.post("/api/webhook/test", json={"url": "xyz"})
+    assert r2.status_code == 422