terminal-smart-cli 0.92.0 → 0.93.0

package/bin/ts.js

@@ -913,8 +913,13 @@ async function agentCmd(words, { askFn, cwd: cwdIn = null, onCwd = null, maxIter
   // saída → o log ficava VAZIO enquanto o agente rodava. Nesses casos usa spinner NOOP: os passos
   // saem por console.log (visíveis no arquivo/pipe) e o progresso aparece de verdade.
   const _noTTY = !process.stdout.isTTY;
+  // B4: headless (não-TTY, redirect >arquivo/pipe) — o console.log é bufferizado em BLOCO e o
+  // log fica VAZIO durante a run (impossível acompanhar ao vivo; foi o que escondeu o hang de
+  // 40min no PW). _hlog escreve SÍNCRONO no fd 1 (fs.writeSync), descarregando cada linha na
+  // hora. No TTY segue o console.log normal (o spinner cuida do redesenho).
+  const _hlog = (s) => { if (_noTTY && !streamJson) { try { require('fs').writeSync(1, s + '\n'); return; } catch (_) {} } console.log(s); };
   const sp = (streamJson || _noTTY) ? { text() {}, start() {}, stop() {} } : ui.spinner(T.agent_thinking).start();
-  if (_noTTY && !streamJson) console.log('  ' + C.dim(T.agent_thinking));
+  if (_noTTY && !streamJson) _hlog('  ' + C.dim(T.agent_thinking));
   const t0 = Date.now();
   // STATUS LINE ao vivo (estilo Claude/Verboo): tempo · modelo · tokens · passo — atualiza a cada 1s.
   const _live = { steps: 0, inTok: 0, outTok: 0, model: model || 'auto' };
@@ -950,13 +955,13 @@ async function agentCmd(words, { askFn, cwd: cwdIn = null, onCwd = null, maxIter
         const tag = auto ? C.warn('▲ auto') : retry ? C.warn('⟳') : loop ? C.warn('↻ loop') : blocked ? C.err('■ ' + T.agent_blocked) : C.cyan('›');
         // nome AMIGÁVEL (Rodando no servidor…) em vez do técnico (executar_remoto); detalhe sem cortar palavra
         const _label = ui.friendlyTool(name, cfg.lang === 'en');
-        console.log('  ' + tag + ' ' + C.bold(_label) + (detail ? C.dim(' · ' + ui.wordTrunc(detail, 60)) : ''));
+        _hlog('  ' + tag + ' ' + C.bold(_label) + (detail ? C.dim(' · ' + ui.wordTrunc(detail, 60)) : ''));
         sp.start();
       },
       onStepDone: (e) => {
         if (streamJson) return; // o evento 'result' já cobre; aqui é só a UI bonita
         sp.stop();
-        console.log(stepDoneLine(e, '    '));
+        _hlog(stepDoneLine(e, '    '));
         sp.start();
       },
       askApprove: async (payload) => {
@@ -970,7 +975,7 @@ async function agentCmd(words, { askFn, cwd: cwdIn = null, onCwd = null, maxIter
       onRemote: ({ ttl }) => {
         if (streamJson) { _emit(core.AgentEvents.tool({ subtype: 'approval_remote', ttl: ttl || 120 })); return; }
         sp.stop();
-        console.log('  ' + C.warn('▲') + ' ' + C.dim(T.agent_remote_wait(ttl || 120)));
+        _hlog('  ' + C.warn('▲') + ' ' + C.dim(T.agent_remote_wait(ttl || 120)));
         sp.start();
       },
     });
@@ -1247,7 +1252,10 @@ async function vpsCmd(words) {
   const ssh = require('../lib/ssh');
   const sub = (words[0] || '').toLowerCase();
   const rest = words.slice(1);
-  const flag = (n) => { const i = rest.indexOf(n); return i >= 0 ? rest[i + 1] : null; };
+  // Lê a flag do process.argv CRU: o parser de topo tira os nomes de flag (--host…) dos
+  // positionais (POS), então `words` só traz os VALORES soltos e o pareamento se perdia
+  // (ts vps set --host X --user Y ignorava tudo e salvava o perfil velho). O argv preserva par.
+  const flag = (n) => { const i = process.argv.indexOf(n); return i >= 0 ? process.argv[i + 1] : null; };
 
   // ts vps set --host IP --user ubuntu --chave caminho.pem   (ou --login arquivo.txt / --senha ...)
   if (sub === 'set' || sub === 'config' || sub === 'add') {

package/lib/agent.js

@@ -73,6 +73,8 @@ REGRAS:
 - BINÁRIO NATIVO: ao subir/instalar um executável numa máquina, valide com file (arquitetura: 32 vs 64-bit) E ldd (as bibliotecas resolvem?) ANTES de declarar "pronto/instalado" — "está no lugar" NÃO é "roda". Se ldd mostrar "not found", instale a lib faltante e revalide.
 - WINDOWS / SHELL BASH: o shell aqui é o bash do Git, NÃO o cmd. (a) "node -e"/"python -c" com várias linhas ou com import() FALHA — escreva um arquivo .mjs/.py e rode "node arquivo.mjs"/"python arquivo.py". (b) pra ESPERAR use "sleep N" (NUNCA "timeout /t", que é do cmd e quebra no bash). (c) comandos que só existem no cmd (dir, type, if exist) rode via "cmd /c \"...\"". (d) pra matar um servidor/processo de teste, ache o PID pela PORTA ("netstat -ano | findstr :PORTA" → "taskkill /PID <pid> /F") — NUNCA por imagem ("taskkill /IM node.exe" mata processos não relacionados, inclusive os do usuário).
 - PROVE O CAMINHO REAL antes de declarar "pronto/corrigido": "testes de API passando" NÃO é "o app funciona". Se há UI ou rotas condicionais (por papel/role, por filtro), rode o FLUXO EXATO do usuário afetado — a tela/rota que quebrava — e veja o resultado; numa entrega WEB, verifique com olhos (navegador). Em fix de código com ramos (SQL com subqueries/parâmetros, condicional por role): conte placeholder-a-placeholder e rode de novo o caminho que falhava. Só diga "corrigido" COM a prova verde do caminho real — e NÃO peça pro usuário rodar a prova que você mesmo consegue rodar. Se subir um server de teste, ENCERRE-o (por PID) ao terminar — não deixe processo órfão na porta.
+- DEPLOY EM CONTAINER: edite sempre a FONTE (o diretório do projeto no host, ex /opt/projects/<app>, ou o código local), NUNCA dentro do container (/app/... é efêmero) — o "docker build" monta a imagem a partir da FONTE, então um fix feito dentro do container SOME no rebuild e você fica achando que corrigiu. Do mesmo jeito: NUNCA inclua arquivos de segredo (.env) no pacote/tar de deploy — o .env de exemplo do repo sobrescreve o .env REAL do servidor e a app volta pro modo stub. Ao recriar um container, repasse rede, portas, volumes E o --env-file que ele já tinha.
+- SEU PRÓPRIO TESTE PODE ESTAR ERRADO: antes de confiar num veredito "FALHOU", confira a ASSERÇÃO (HTTP 200/201 é SUCESSO, não falha; 401/403 numa rota protegida sem token é o comportamento CORRETO). E não re-leia/re-escreva o MESMO arquivo várias vezes: se você já leu, use o que leu.
 - ESTILO DA RESPOSTA (importante): escreva em linguagem NATURAL e direta, como explicando pra uma pessoa — evite jargão e detalhe técnico interno que não interessa a quem só quer o resultado. NÃO use emojis decorativos (nada de 🎮📡🔬🌿✅❌ etc.); pra marcar certo/errado use SÓ os símbolos ✓ (deu certo) ou ✗ (falhou). Frases curtas; comando/código/caminho sempre em bloco de código. Não repita o que já apareceu nos passos acima.
 - Termine SEMPRE com um resumo curto: o que foi feito, resultado e caminhos de arquivos criados/alterados.
 - Responda no idioma do usuário (padrão: português do Brasil).`
@@ -101,6 +103,8 @@ RULES:
 - NATIVE BINARY: when uploading/installing an executable on a machine, validate with file (architecture: 32 vs 64-bit) AND ldd (do the libraries resolve?) BEFORE declaring "done/installed" — "it's in place" is NOT "it runs". If ldd shows "not found", install the missing lib and re-validate.
 - WINDOWS / BASH SHELL: the shell here is Git bash, NOT cmd. (a) "node -e"/"python -c" with multiple lines or with import() FAILS — write a .mjs/.py file and run "node file.mjs"/"python file.py". (b) to WAIT use "sleep N" (NEVER "timeout /t", which is cmd and breaks in bash). (c) cmd-only commands (dir, type, if exist) run via "cmd /c \"...\"". (d) to kill a test server/process, find the PID by PORT ("netstat -ano | findstr :PORT" → "taskkill /PID <pid> /F") — NEVER by image ("taskkill /IM node.exe" kills unrelated processes, including the user's).
 - PROVE THE REAL PATH before declaring "done/fixed": "API tests passing" is NOT "the app works". If there's UI or conditional routes (by role, by filter), run the EXACT flow of the affected user — the screen/route that was breaking — and see the result; on a WEB deliverable, verify with eyes (browser). In a code fix with branches (SQL with subqueries/params, role conditionals): count placeholder-by-placeholder and re-run the path that was failing. Only say "fixed" WITH green proof of the real path — and do NOT ask the user to run a proof you can run yourself. If you start a test server, SHUT IT DOWN (by PID) when done — don't leave an orphan process on the port.
+- CONTAINER DEPLOY: always edit the SOURCE (the project dir on the host, e.g. /opt/projects/<app>, or the local code), NEVER inside the container (/app/... is ephemeral) — "docker build" builds the image FROM THE SOURCE, so a fix made inside the container VANISHES on rebuild while you think it's fixed. Likewise: NEVER include secret files (.env) in the deploy tar/package — the repo's sample .env overwrites the REAL server .env and the app falls back to stub mode. When recreating a container, re-pass its network, ports, volumes AND the --env-file it had.
+- YOUR OWN TEST MAY BE WRONG: before trusting a "FAILED" verdict, check the ASSERTION (HTTP 200/201 is SUCCESS, not failure; 401/403 on a protected route without a token is the CORRECT behavior). And don't re-read/re-write the SAME file repeatedly: if you already read it, use what you read.
 - RESPONSE STYLE (important): write in NATURAL, plain language, like explaining to a person — avoid jargon and internal technical detail that doesn't matter to someone who just wants the result. NO decorative emojis (no 🎮📡🔬✅❌ etc.); to mark pass/fail use ONLY the symbols ✓ (worked) or ✗ (failed). Short sentences; commands/code/paths always in a code block. Don't repeat what already showed up in the steps above.
 - ALWAYS end with a short summary: what was done, the result, and paths of created/changed files.
 - Answer in the user's language.`);

package/lib/core.js

@@ -195,8 +195,38 @@ function classifyToolResult(raw) {
   return { ok: true, status: 'ok', errorClass: null, retryable: false, evidence: String(ev || '').replace(/\s+/g, ' ').slice(0, 200) };
 }
 
+// ── SEGREDOS: nunca imprimir chave/token na saída (B18) ───────────────────────
+// O agente vaza segredo enquanto DIAGNOSTICA (cat .env, docker inspect, echo $KEY,
+// grep numa store de chaves) mesmo instruído a não fazer isso — regra sozinha não
+// segura. Aqui a saída de ferramenta passa por uma máscara determinística ANTES de
+// virar contexto/log. Mantém o prefixo (o agente ainda reconhece "é uma chave sk-…")
+// mas o valor não circula. Não altera o que é ENVIADO ao comando, só o que é EXIBIDO.
+const _SECRET_RES = [
+  // chaves com prefixo conhecido: sk-…, sk-cdc-…, ghp_…, xoxb-…, AKIA…, AIza…
+  [/\b((?:sk|pk|rk)-(?:[a-z0-9]+-)?)[A-Za-z0-9_-]{16,}/gi, (m, p) => p + '••••REDACTED'],
+  [/\b(gh[pousr]_)[A-Za-z0-9]{16,}/g, (m, p) => p + '••••REDACTED'],
+  [/\b(xox[baprs]-)[A-Za-z0-9-]{10,}/gi, (m, p) => p + '••••REDACTED'],
+  [/\b(AKIA)[0-9A-Z]{12,}/g, (m, p) => p + '••••REDACTED'],
+  [/\b(AIza)[A-Za-z0-9_-]{20,}/g, (m, p) => p + '••••REDACTED'],
+  // JWT (header.payload.signature)
+  [/\beyJ[A-Za-z0-9_-]{8,}\.[A-Za-z0-9_-]{8,}\.[A-Za-z0-9_-]{8,}/g, () => 'eyJ••••REDACTED_JWT'],
+  // atribuição de env/JSON com nome sensível: API_KEY=…, "password": "…", Bearer …
+  [/((?:api[_-]?key|apikey|secret|token|password|passwd|senha|authorization|auth[_-]?token|private[_-]?key)["']?\s*[:=]\s*["']?)([^\s"',;&|)]{8,})/gi,
+    (m, p) => p + '••••REDACTED'],
+  [/\b(Bearer\s+)[A-Za-z0-9._~+/-]{16,}=*/g, (m, p) => p + '••••REDACTED'],
+];
+function redactSecrets(input) {
+  if (input == null) return input;
+  if (typeof input !== 'string') {
+    try { return JSON.parse(redactSecrets(JSON.stringify(input))); } catch (_) { return input; }
+  }
+  let s = input;
+  for (const [re, fn] of _SECRET_RES) s = s.replace(re, fn);
+  return s;
+}
+
 module.exports = {
-  DESTRUCTIVE, isDestructive, touchesTsConfig,
+  DESTRUCTIVE, isDestructive, touchesTsConfig, redactSecrets,
   GLOBAL_STATE, globalStateWarning, describeDestructive,
   MODEL_WINDOWS, CTX_WINDOW_DEFAULT, DEFAULT_EXECUTOR, COMPACT_AT, KEEP_TAIL, winFor, estMsgsTok,
   AgentEvents, EVENT_TYPES,

package/lib/tools.js

@@ -224,6 +224,25 @@ function _guardTsHome(p) {
 // por algo que JAMAIS deve rodar é desperdício puro: recusamos NA HORA, apontando a
 // alternativa cirúrgica. Retorna a mensagem (motivo pro modelo) quando é auto-destrutivo,
 // ou null quando é seguro (aí segue o fluxo normal, inclusive o gate de aprovação comum).
+// Gotchas de shell no Windows que falham de um jeito CONFUSO (o agente perdia vários
+// passos tentando de novo). Devolve a mensagem com o CONSERTO pronto, ou null.
+function _shellGotcha(cmd) {
+  const s = String(cmd || '');
+  const multi = /[\r\n]/.test(s);
+  // python -c / node -e multilinha ou com import() dinâmico: quebra no eval / morre calado
+  if (/\bpython3?\s+-c\b/.test(s) && multi)
+    return 'python -c MULTILINHA falha em silêncio no Windows. Escreva um arquivo .py com escrever_arquivo e rode "python arquivo.py".';
+  if (/\bnode\s+-e\b/.test(s) && (multi || /\bimport\s*\(/.test(s) || /\bawait\b/.test(s)))
+    return 'node -e MULTILINHA ou com import()/await de topo falha no eval. Escreva um arquivo .mjs com escrever_arquivo e rode "node arquivo.mjs".';
+  // timeout /t é do cmd; sob git-bash pega o timeout do coreutils e dá "invalid time interval"
+  if (/\btimeout\s+\/t\b/i.test(s))
+    return 'Este shell é bash (não cmd): "timeout /t N" falha. Para esperar use "sleep N". Comandos só-do-cmd (dir, type, if exist) rode via: cmd /c "…".';
+  // matar processo por IMAGEM mata tudo com esse nome (inclusive a própria missão)
+  if (/\btaskkill\b[^\n]*\/im\b/i.test(s))
+    return 'taskkill /IM mata TODOS os processos com esse nome (inclusive os do usuário e possivelmente o próprio ts). Mate pelo PID da PORTA: netstat -ano | findstr :PORTA  →  taskkill /PID <pid> /F.';
+  return null;
+}
+
 function selfDestructiveReason(cmd, opts = {}) {
   const c = String(cmd || '');
   if (!c.trim()) return null;
@@ -345,6 +364,11 @@ async function execute(name, input, opts = {}) {
         // RECUSA INSTANTÂNEA (defesa em profundidade — o gate do agent.js já pega antes):
         // comando auto-destrutivo NUNCA roda e NUNCA espera aprovação.
         { const sd = selfDestructiveReason(comando, { cwd: baseDir }); if (sd) return { erro: sd }; }
+        // GOTCHAS DE SHELL (B2/B6/B10): padrões que FALHAM de forma confusa no Windows.
+        // Barrar ANTES de rodar (com o conserto na mão) economiza o ciclo erro→tentativa→erro
+        // que o agente repetia: python -c multilinha morria em silêncio, node -e com import()
+        // quebrava no eval, e `timeout /t` (cmd) virava o timeout do coreutils sob git-bash.
+        { const g = _shellGotcha(comando); if (g) return { erro: g }; }
         // Comandos de INSTALL/BUILD (apt/npm/pip/make/gcc/docker build…) levam muito mais que 60s —
         // sem isso o apt-get de vários pacotes MORRIA no timeout padrão (nada instalava, em silêncio).
         // Eles ganham 10min de default (até 30min se o agente pedir timeout_s); resto segue 60s.
@@ -355,7 +379,9 @@ async function execute(name, input, opts = {}) {
         return await new Promise((res) => {
           require('child_process').exec(cmd, { timeout, shell: true, windowsHide: true, maxBuffer: 4 * 1024 * 1024, cwd: fs.existsSync(baseDir) ? baseDir : undefined }, (e, out, err) => {
             const c = compactor.compact(comando, out || '', { codigo: e?.code ?? 0 });
-            const r = { stdout: c.out, stderr: compactor.stripAnsi(err || '').slice(0, 1500), codigo: e?.code ?? 0 };
+            // B18: mascara chave/token ANTES de virar contexto do modelo e linha de log.
+            const _rd = require('./core').redactSecrets;
+            const r = { stdout: _rd(c.out), stderr: _rd(compactor.stripAnsi(err || '').slice(0, 1500)), codigo: e?.code ?? 0 };
             if (c.note) r.aviso = c.note;
             if (!r.stdout.trim() && !r.stderr.trim() && r.codigo === 0 && /python3? -c/.test(comando) && comando.includes('\n'))
               r.aviso = 'stdout VAZIO: no Windows, python -c multi-linha falha em silêncio. ESCREVA um .py com escrever_arquivo e rode "python arquivo.py".';
@@ -392,7 +418,9 @@ async function execute(name, input, opts = {}) {
         const comando = String(input.comando || '');
         const out = await ssh.exec(comando, { timeoutMs: 180000 });
         const c = compactor.compact(comando, out.stdout || '', { codigo: out.code });
-        const r = { remoto: (ssh.info() && ssh.info().host) || '', stdout: c.out, stderr: compactor.stripAnsi(out.stderr || '').slice(0, 1500), codigo: out.code };
+        // B18: foi AQUI que a chave da IA vazou (cat .env / grep na store de chaves na VPS).
+        const _rd = require('./core').redactSecrets;
+        const r = { remoto: (ssh.info() && ssh.info().host) || '', stdout: _rd(c.out), stderr: _rd(compactor.stripAnsi(out.stderr || '').slice(0, 1500)), codigo: out.code };
         if (c.note) r.aviso = c.note;
         return r;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "terminal-smart-cli",
-  "version": "0.92.0",
+  "version": "0.93.0",
   "description": "Terminal Smart no seu terminal — pergunte, analise logs por pipe e orquestre agentes de IA. Comando: ts",
   "bin": {
     "ts": "bin/ts.js"