@danhachuel/leadmaster 0.1.23 → 0.1.25

package/CHANGELOG.md

@@ -37,6 +37,60 @@ 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.25] — 2026-11-09
+
+### Adicionado
+
+- **Modal de Detalhes do Lead no CRM (clique no card):**
+  - Rota `PUT /api/crm/leads/{lead_id}` — edição de `nome`, `telefone` e `email`
+    (todos opcionais no payload); nome vazio → **400** (prevenção de erros),
+    lead inexistente → **404**, payload sem campos editáveis → **400**.
+  - Card do Kanban passa a abrir os detalhes no **clique simples** (antes era
+    duplo clique); flag global suprime o clique disparado no pointerup após um
+    drag, então abrir detalhes e arrastar convivem sem conflito.
+  - Modal "Detalhes do Lead" exibe **todos os dados** — inclusive `extras`
+    (Site, Endereço, Avaliação, Seguidores, Username, Bio), tags, fonte e data
+    de criação — com **inputs de edição** para Nome/Telefone/E-mail pré-preenchidos
+    (vermelho + mensagem se o nome for apagado), textarea de observações e botão
+    **"Salvar Alterações"** grande em gradiente (Fitts + von Restorff), com
+    **spinner** durante o salvamento (Zeigarnik), toast de sucesso e
+    **atualização automática** do Kanban. O "Fechar" (X) permanece sutil.
+- **5 novos testes pytest** (`backend/tests/test_lead_edit_routes.py`): edição
+  completa, edição parcial (campos omitidos preservados), 404, nome vazio (400)
+  e payload sem campos editáveis (400).
+
+## [0.1.24] — 2026-11-09
+
+### Corrigido
+
+- **Launcher — regressão do terminal preso investigada e blindagem reforçada** —
+   usuário reportou o prompt do MobaXterm voltando logo após "LeadMaster no ar"
+   na 0.1.21 (runtime permanente). A investigação mostrou que o fluxo publicado
+   **mantém** as proteções (`process.stdin.resume();`, `setInterval`,
+   `child.on('exit')`/`close`, `process.on('exit')`, sem `.unref()`) e que uma
+   morte silenciosa assim — sem nenhum log do handler de saída — só ocorre por
+   `SIGKILL`/`taskkill /F` (ex.: outra execução `install`/`start` matando a
+   instância anterior pelo kill switch, ou fechamento forçado da aba). Mesmo
+   assim, o launcher foi reforçado e os gates agora validam o **fluxo**:
+   - **Rede de segurança no `cmdStart`:** `process.on('beforeExit')` re-arma o
+     `setInterval` se o event loop esvaziar (stdin em EOF, timer perdido,
+     surpresa de PTY) — o Node jamais encerra sozinho; erros do stream de
+     stdin (`process.stdin.on('error')`) são ignorados;
+   - **Gates de fluxo (`test_launcher_protections.py`):** novo helper
+     `_corpo_cmd_start()` extrai o corpo da função para garantir que
+     `spawn`, `process.stdin.resume();`, `setInterval`, `child.on('exit')`,
+     `child.on('close')`, `process.on('exit')` e `beforeExit` existam **dentro
+     do `start`** (não no `install` nem em função morta) e que **nenhum
+     `process.exit(`** apareça após o terminal ser preso — a saída só por ciclo
+     de vida do filho ou por `SIGTERM`/`SIGBREAK`;
+   - **AGENTS.md:** regra 2 do launcher documenta a rede de segurança e a
+     exigência de posição (após o spawn, no fluxo do start).
+- **Diagnóstico rápido para o usuário:** com a regra acima, se o prompt voltar
+  sem as linhas "Encerrando LeadMaster..." (SIGTERM/SIGBREAK) ou "Servidor
+  local encerrou com código N" (Python caiu), a causa é externa ao launcher
+  (outra execução ou fechamento forçado) — o espelho
+  `storage/logs/launcher_console.log` mostra exatamente qual caminho ocorreu.
+
 ## [0.1.23] — 2026-11-09
 
 ### Adicionado

package/backend/main.py

@@ -344,6 +344,24 @@ def crm_mover_lead(lead_id: str, dados: dict):
     return atualizado
 
 
+@app.put("/api/crm/leads/{lead_id}")
+def crm_editar_lead(lead_id: str, dados: dict):
+    """Edita os dados principais do lead (modal de detalhes do CRM).
+
+    Payload aceita qualquer subconjunto de {nome, telefone, email, ...}; o nome,
+    quando presente, não pode ficar vazio (prevenção de erros).
+    """
+    if "nome" in dados and not str(dados.get("nome") or "").strip():
+        raise HTTPException(400, "O nome do lead não pode ficar vazio.")
+    patch = {k: v for k, v in dados.items() if k in ("nome", "telefone", "email")}
+    if not patch:
+        raise HTTPException(400, "Nenhum campo editável informado (nome, telefone, email).")
+    atualizado = storage.update_lead(lead_id, patch)
+    if not atualizado:
+        raise HTTPException(404, "Lead não encontrado.")
+    return atualizado
+
+
 # ---------------------------------------------------------------------------
 # Campanhas / Disparos
 # ---------------------------------------------------------------------------

package/backend/tests/test_launcher_protections.py

@@ -23,6 +23,15 @@ def _conteudo() -> str:
     return CAMINHO.read_text(encoding="utf-8")
 
 
+def _corpo_cmd_start() -> str:
+    """Corpo da função cmdStart — valida o FLUXO do start (não só strings)."""
+    c = _conteudo()
+    assert "function cmdStart(" in c, "função cmdStart não encontrada"
+    inicio = c.index("function cmdStart(")
+    fim = c.index("async function cmdDoctor")
+    return c[inicio:fim]
+
+
 class TestLauncherProtections:
     def test_keepalivo(self):
         assert "setInterval(() => {}, 1000)" in _conteudo()
@@ -44,6 +53,28 @@ class TestLauncherProtections:
         # Avisar que já existe instância no ar é proibido (impede atualizações).
         assert "já está rodando" not in c
 
+    def test_keepalive_no_fluxo_do_start(self):
+        # Gate de FLUXO (regressão da 0.1.21): as proteções precisam existir
+        # dentro do cmdStart — no install, em função morta ou em bloco errado
+        # não valem. A morte prematura do Node devolve o prompt do MobaXterm.
+        corpo = _corpo_cmd_start()
+        assert "spawn(" in corpo  # worker Python é spawnado no start
+        assert "process.stdin.resume();" in corpo  # terminal preso
+        assert "setInterval(() => {}, 1000)" in corpo  # keep-alive
+        assert "beforeExit" in corpo  # rede de segurança do event loop
+        assert "child.on('exit'" in corpo or 'child.on("exit"' in corpo
+        assert "child.on('close'" in corpo or 'child.on("close"' in corpo
+        assert "process.on('exit'" in corpo or 'process.on("exit"' in corpo
+
+    def test_start_nunca_chama_exit_apos_terminal_preso(self):
+        # Após o terminal ser preso, nenhum caminho de sucesso chama
+        # process.exit — a saída só ocorre pelo ciclo de vida do filho
+        # (child exit/close) ou por sinais de parada legítima.
+        corpo = _corpo_cmd_start()
+        pos = corpo.index("process.stdin.resume();")
+        fim_iife = corpo.index("})().catch(")  # catch de erro fatal fica de fora
+        assert "process.exit(" not in corpo[pos:fim_iife]
+
     def test_ciclo_de_vida_vinculado(self):
         c = _conteudo()
         assert "child.on('exit'" in c or "child.on(\"exit\"" in c

package/backend/tests/test_lead_edit_routes.py

@@ -0,0 +1,63 @@
+"""Testes da edição do lead — PUT /api/crm/leads/{lead_id}.
+
+Edição de nome/telefone/email (todos opcionais), persistência, 404 para ID
+inexistente e bloqueio de nome vazio (prevenção de erros).
+"""
+
+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 _criar_lead(client):
+    return client.post("/api/crm/leads", json={"nome": "Lead Original", "telefone": "(11) 90000-0000", "email": "a@b.com"}).json()
+
+
+def test_editar_campos_principais(client):
+    lead = _criar_lead(client)
+    r = client.put(
+        f"/api/crm/leads/{lead['id']}",
+        json={"nome": "Lead Editado", "telefone": "(21) 98888-7777", "email": "novo@empresa.com"},
+    )
+    assert r.status_code == 200
+    corpo = r.json()
+    assert corpo["nome"] == "Lead Editado"
+    assert corpo["telefone"] == "(21) 98888-7777"
+    assert corpo["email"] == "novo@empresa.com"
+    # Persistido no leads.json (a listagem reflete a edição).
+    salvo = [l for l in client.get("/api/crm/leads").json() if l["id"] == lead["id"]][0]
+    assert salvo["nome"] == "Lead Editado"
+
+
+def test_editar_parcial(client):
+    lead = _criar_lead(client)
+    r = client.put(f"/api/crm/leads/{lead['id']}", json={"email": "parcial@x.com"})
+    assert r.status_code == 200
+    assert r.json()["email"] == "parcial@x.com"
+    assert r.json()["nome"] == "Lead Original"  # campos omitidos permanecem
+
+
+def test_editar_inexistente_404(client):
+    r = client.put("/api/crm/leads/nao-existe", json={"nome": "X"})
+    assert r.status_code == 404
+
+
+def test_nome_vazio_bloqueado(client):
+    lead = _criar_lead(client)
+    r = client.put(f"/api/crm/leads/{lead['id']}", json={"nome": "   "})
+    assert r.status_code == 400
+    assert "vazio" in r.json()["detail"]
+
+
+def test_payload_sem_campos_editaveis(client):
+    lead = _criar_lead(client)
+    r = client.put(f"/api/crm/leads/{lead['id']}", json={"etapa": "fechado"})
+    assert r.status_code == 400