kafox-cli 0.9.7 → 0.9.9

package/package.json

@@ -1,7 +1,23 @@
 {
   "name": "kafox-cli",
-  "version": "0.9.7",
+  "version": "0.9.9",
   "description": "Kafox CLI: TUI com streaming, thinking, edit com diff, agent tools (files/shell), MCP e temas",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/kafox/kafox-cli.git"
+  },
+  "homepage": "https://kafox.com",
+  "bugs": {
+    "url": "https://github.com/kafox/kafox-cli/issues"
+  },
+  "keywords": [
+    "cli",
+    "tui",
+    "ai",
+    "agent",
+    "llm",
+    "mcp"
+  ],
   "bin": {
     "kafox": "bin/kafox.js"
   },
@@ -14,6 +30,7 @@
     "node": ">=22.5"
   },
   "license": "MIT",
+  "author": "Kafox <dev@kafox.com> (https://kafox.com)",
   "dependencies": {
     "chalk": "4.1.2",
     "commander": "12.1.0",

package/src/tools.js

@@ -122,9 +122,20 @@ function toolMeta(name, hub) {
 
 function resolveInside(cwd, target) {
   const root = path.resolve(cwd);
-  const full = path.resolve(root, target || ".");
+  const raw = String(target ?? ".");
+  // Caminho absoluto de outro sistema (ex.: "C:\Windows\..." no Linux) e
+  // NUL nunca valem: sem isso o `path.resolve` trata como nome de arquivo.
+  if (raw.includes("\0")) throw new Error(`caminho invalido: ${raw}`);
+  if (path.win32.isAbsolute(raw) || path.posix.isAbsolute(raw)) {
+    const full = path.isAbsolute(raw) ? path.resolve(raw) : null;
+    if (!full || (full !== root && !full.startsWith(root + path.sep))) {
+      throw new Error(`caminho fora do diretorio de trabalho: ${raw}`);
+    }
+    return full;
+  }
+  const full = path.resolve(root, raw);
   if (full !== root && !full.startsWith(root + path.sep)) {
-    throw new Error(`caminho fora do diretorio de trabalho: ${target}`);
+    throw new Error(`caminho fora do diretorio de trabalho: ${raw}`);
   }
   return full;
 }

package/src/tui.js

@@ -489,6 +489,9 @@ function initialState() {
     messages: [],
     context: [],
     streaming: null,
+    // Texto ja comitado em finishStreaming() neste turno: usado para nao
+    // duplicar a resposta quando runAgent devolve o texto no final.
+    streamedText: "",
     approval: null,
     autoApprove: false,
     busy: false,
@@ -902,8 +905,36 @@ function formatTokens(count) {
   return `${(count / 1_000_000).toFixed(1)}M`;
 }
 
-function estimateTokens(state) {
-  let chars = 0;
+/**
+ * Commita o texto que chegou pelo stream como mensagem `assistant`.
+ * Nao duplica quando o mesmo trecho ja foi commitado neste turno (retry do
+ * gateway repetindo tokens, `onToolStart` chamado duas vezes, etc.).
+ */
+function commitStreamedText(state, text) {
+  const value = String(text || "").trim();
+  if (!value) return false;
+  if (state.streamedText.includes(value)) return false;
+  const previous = [...state.messages].reverse().find((message) => message.role === "assistant");
+  if (previous && previous.text === value) return false;
+  state.messages.push({ role: "assistant", text: value });
+  state.streamedText = `${state.streamedText}${value}`;
+  return true;
+}
+
+/**
+ * Ultimo texto devolvido por `runAgent`. Quando o stream ja mostrou esse texto
+ * (commit em `finishStreaming`), nao empurra de novo: era isso que fazia a
+ * resposta aparecer duas vezes na tela.
+ */
+function commitFinalText(state, text, ms) {
+  const value = String(text || "").trim();
+  if (!value) return false;
+  if (state.streamedText.includes(value)) return false;
+  state.messages.push({ role: "assistant", text: value, ms });
+  return true;
+}
+
+function estimateTokens(state) {  let chars = 0;
   for (const message of state.context) chars += String(message.content || "").length;
   if (state.streaming) chars += String(state.streaming.text || "").length;
   return Math.round(chars / 4);
@@ -1950,13 +1981,10 @@ async function runTUI({
       if (result.messages) state.context = result.messages;
     } else if (result.ok) {
       const text = String(result.text || "").trim();
-      if (text) {
-        state.messages.push({
-          role: "assistant",
-          text,
-          ms: result.ms,
-        });
-      }
+      // O stream ja foi commitado em finishStreaming(): so empurra aqui se for
+      // texto novo (ou se nao houve streaming, ex.: resposta sem onText).
+      if (text) commitFinalText(state, text, result.ms);
+      state.streamedText = "";
       if (result.messages) state.context = result.messages;
     } else {
       state.messages.push({
@@ -1983,9 +2011,7 @@ async function runTUI({
         ms: streaming.thinkingStartedAt ? Date.now() - streaming.thinkingStartedAt : 0,
       });
     }
-    if (streaming.text.trim()) {
-      state.messages.push({ role: "assistant", text: streaming.text.trim() });
-    }
+    if (streaming.text.trim()) commitStreamedText(state, streaming.text.trim());
     state.streaming = { text: "", reasoning: "", thinkingStartedAt: 0 };
   }
 
@@ -2383,4 +2409,6 @@ module.exports = {
   planRepaint,
   APPROVAL_OPTIONS,
   layoutFor,
+  commitStreamedText,
+  commitFinalText,
 };