@danhachuel/leadmaster 0.1.10 → 0.1.11

package/CHANGELOG.md

@@ -6,6 +6,37 @@ O formato é baseado no [Keep a Changelog](https://keepachangelog.com/pt-BR/1.1.
 A etiqueta visual de versão usa patch de dois dígitos (ex.: `0.1.00`); o registry NPM
 normaliza para a forma canônica (ex.: `0.1.0`).
 
+## [0.1.11] — 2026-11-08
+
+### Corrigido
+
+- **Launcher ainda devolvia o prompt no MobaXterm — a 0.1.9 não resolveu** — o
+   processo Node encerrava seu event loop logo após imprimir "LeadMaster no ar...",
+   mesmo com o keep-alive `setInterval`. Sem o Node vivo, o MobaXterm fecha a
+   pseudo-console (PTY) e o uvicorn perde o terminal, encerrando junto. Correções
+   definitivas no `scripts/cli.mjs`:
+   - **Terminal preso:** `process.stdin.resume();` inserido **logo antes das
+     mensagens finais de sucesso** — o Node fica indefinidamente vivo ouvindo a
+     entrada padrão, sem depender de timers nem do wrapper do `npx`. O keep-alive
+     `setInterval` permanece como segunda camada de proteção.
+   - **Ciclo de vida vinculado nos dois sentidos:** `child.on('close')` encerra o
+     launcher quando o servidor Python cai (mensagem "Servidor encerrado"), e
+     `process.on('exit')` mata o filho quando o launcher morre — nenhum processo
+     fica órfão. O worker Python continua **sem `.unref()`** (o pai não pode
+     morrer sozinho).
+   - **Erros globais (debug):** handlers `unhandledRejection` e `uncaughtException`
+     logam o erro em vez de derrubar o launcher silenciosamente.
+   - **Sem `process.exit(0)` após o sucesso:** a saída do fluxo principal agora é
+     natural (via listeners do filho); chamadas de `process.exit` permanecem apenas
+     em caminhos de erro acionável, na parada legítima (`SIGTERM`/`SIGBREAK`) e no
+     aviso informativo "já está rodando".
+- **AGENTS.md:** regra inviolável do launcher passa de quatro para **cinco
+   proteções** (nova: *Terminal preso*, incluindo a proibição de `process.exit(0)`
+   após o sucesso e a vedação do `.unref()`).
+- **Gate de regressão reforçado:** `backend/tests/test_launcher_protections.py`
+   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.10] — 2026-11-08
 
 ### Adicionado

package/backend/tests/test_launcher_protections.py

@@ -4,12 +4,16 @@ Lê o launcher e falha se qualquer proteção contra encerramento silencioso no
 Windows (MobaXterm) for removida:
 
 1. Keep-alive com setInterval (event loop ativo sem I/O de console);
-2. Absorção de SIGINT/SIGHUP no Windows (apenas log, sem encerrar);
-3. Função guardConsoleStream (anti-EPIPE com espelho em arquivo de log);
-4. Isolamento do worker Python (detached: true / CREATE_NEW_PROCESS_GROUP).
+2. Terminal preso com `process.stdin.resume();` (solução definitiva contra a
+   devolução do prompt — o Node fica vivo ouvindo a entrada padrão);
+3. Ausência de `.unref()` no worker Python (o pai não pode morrer sozinho);
+4. Absorção de SIGINT/SIGHUP no Windows (apenas log, sem encerrar);
+5. Função guardConsoleStream (anti-EPIPE com espelho em arquivo de log);
+6. Isolamento do worker Python (detached: true / CREATE_NEW_PROCESS_GROUP).
 """
 
 from pathlib import Path
+import re
 
 CAMINHO = Path(__file__).resolve().parents[2] / "scripts" / "cli.mjs"
 
@@ -19,10 +23,38 @@ def _conteudo() -> str:
     return CAMINHO.read_text(encoding="utf-8")
 
 
+def _codigo_sem_comentarios() -> str:
+    """Código efetivo do launcher (comentários removidos).
+
+    Regras de arquitetura valem para o CÓDIGO: menções em comentários
+    (explicações, nomes de funções proibidas) não contam.
+    """
+    sem_bloco = re.sub(r"/\*.*?\*/", "", _conteudo(), flags=re.DOTALL)
+    sem_linha = re.sub(r"//[^\n]*", "", sem_bloco)
+    return sem_linha
+
+
 class TestLauncherProtections:
     def test_keepalivo(self):
         assert "setInterval(() => {}, 1000)" in _conteudo()
 
+    def test_stdin_resume_prende_terminal(self):
+        # Solução definitiva: sem esta linha o MobaXterm devolve o prompt
+        # logo após "LeadMaster no ar..." e fecha a PTY (e o uvicorn junto).
+        assert "process.stdin.resume();" in _conteudo()
+
+    def test_worker_nunca_unref(self):
+        # O filho Python referenciado mantém o pai vivo; .unref() permitiria
+        # a morte prematura do launcher. Comentários podem mencionar a regra.
+        assert ".unref()" not in _codigo_sem_comentarios()
+
+    def test_ciclo_de_vida_vinculado(self):
+        # Python cai → launcher cai (child 'close'); launcher morre → filho morre.
+        c = _conteudo()
+        assert "child.on('close'" in c
+        assert "process.on('exit'" in c
+        assert "Servidor encerrado" in c
+
     def test_absorcao_de_sinais_no_windows(self):
         c = _conteudo()
         assert "SIGINT" in c and "SIGTERM" in c and "SIGHUP" in c
@@ -44,3 +76,9 @@ class TestLauncherProtections:
         c = _conteudo()
         assert "SIGTERM" in c and "SIGBREAK" in c
         assert "naoAbsorver" in c
+
+    def test_erros_globais_logados(self):
+        # Erros silenciosos (rejeições/exceções) são logados, não matam o launcher.
+        c = _conteudo()
+        assert "unhandledRejection" in c
+        assert "uncaughtException" in c

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@danhachuel/leadmaster",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "SaaS local de prospecção e automação: CRM, extratores de leads e agentes de IA — 100% local.",
   "license": "MIT",
   "author": "Dan Hachuel",

package/scripts/cli.mjs

@@ -30,6 +30,21 @@ import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import http from 'node:http';
 
+// ---------------------------------------------------------------------------
+// Erros globais (blindagem anti-encerramento silencioso)
+// Rejeições e exceções não tratadas são APENAS LOGADAS — nunca devem derrubar
+// o launcher sem explicação (era assim que o MobaXterm perdia o processo sem
+// deixar rastro). O launcher só termina pelo fluxo normal: erro acionável,
+// parada legítima (SIGTERM/SIGBREAK) ou morte do servidor Python filho.
+// NUNCA REMOVER (regra inviolável do AGENTS.md).
+// ---------------------------------------------------------------------------
+process.on('unhandledRejection', (reason) => {
+  console.error('Unhandled Rejection:', reason);
+});
+process.on('uncaughtException', (err) => {
+  console.error('Uncaught Exception:', err);
+});
+
 const PKG_ROOT = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..');
 
 // Porta padrão da UI/API local; se estiver ocupada, o launcher avança
@@ -372,6 +387,10 @@ function cmdStart(args) {
     const url = `http://127.0.0.1:${porta}`;
     if (jaRodando) {
       console.log(`ℹ LeadMaster já está rodando em ${url} — abra o endereço no navegador.`);
+      // Saída INFORMATIVA (antes de qualquer inicialização própria): não há
+      // servidor filho para supervisionar nem terminal para prender — o
+      // launcher não deve chamar process.exit(0) no caminho de SUCESSO da
+      // inicialização (regra inviolável do AGENTS.md).
       process.exit(0);
       return;
     }
@@ -438,6 +457,14 @@ function cmdStart(args) {
       process.exit(code === null ? 1 : code);
     });
 
+    // Ciclo de vida vinculado (regra inviolável do AGENTS.md): se o Python
+    // cai, o launcher cai junto — nunca fica preso ao terminal sem servidor.
+    // O worker NUNCA é criado com .unref(): o pai não pode morrer sozinho.
+    child.on('close', (code) => {
+      console.log('Servidor encerrado');
+      process.exit(code === null ? 1 : code);
+    });
+
     const stop = () => {
       console.log('\nEncerrando LeadMaster...');
       if (platform() === 'win32' && child.pid) {
@@ -447,6 +474,16 @@ function cmdStart(args) {
       }
     };
 
+    // Morte do launcher → morte do filho (e vice-versa, via close acima):
+    // o uvicorn nunca fica órfão no terminal quando o Node é encerrado.
+    process.on('exit', () => {
+      try {
+        child.kill('SIGTERM');
+      } catch {
+        /* filho já encerrou — nada a fazer */
+      }
+    });
+
     console.log(`→ Subindo servidor local em ${url} ...`);
     const health = await waitForHealth(porta, 120000);
     if (!health) {
@@ -476,10 +513,18 @@ function cmdStart(args) {
       process.on('SIGBREAK', naoAbsorver);
     }
 
-    // Keep-alive: mantém o event loop ativo mesmo sem I/O de console (MobaXterm
-    // minimizado / sem stream ativo) — evita o encerramento silencioso.
+    // Keep-alive (regra inviolável do AGENTS.md): mantém o event loop ativo
+    // mesmo sem I/O de console (MobaXterm minimizado / sem stream ativo).
     setInterval(() => {}, 1000);
 
+    // Terminal preso (regra inviolável do AGENTS.md) — SOLUÇÃO DEFINITIVA:
+    // ao retomar a entrada padrão IMEDIATAMENTE antes das mensagens finais
+    // de sucesso, o Node permanece vivo indefinidamente aguardando qualquer
+    // digitação do usuário. O prompt do MobaXterm NUNCA é devolvido e a PTY
+    // (e com ela o uvicorn) não é fechada. Esta linha NÃO depende de timers
+    // nem de npx — NUNCA REMOVER.
+    process.stdin.resume();
+
     console.log(`✓ LeadMaster no ar em ${url} (dados: ${home})`);
     console.log('');
     console.log(`ℹ Abra ${url} no navegador quando quiser — o LeadMaster nunca abre janelas sozinho.`);