@cocaxcode/logbook-mcp 0.1.0 → 0.1.2

package/README.md

@@ -53,7 +53,7 @@ Your TODOs live in 5 different places: Jira, sticky notes, code comments, your h
 ### Claude Code (recommended)
 
 ```bash
-claude mcp add logbook-mcp -- npx @cocaxcode/logbook-mcp --mcp
+claude mcp add logbook-mcp -- npx @cocaxcode/logbook-mcp@latest --mcp
 ```
 
 ### Claude Desktop
@@ -65,7 +65,7 @@ Add to your `claude_desktop_config.json`:
   "mcpServers": {
     "logbook-mcp": {
       "command": "npx",
-      "args": ["@cocaxcode/logbook-mcp", "--mcp"]
+      "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]
     }
   }
 }
@@ -88,7 +88,7 @@ Add to `.cursor/mcp.json` in your project:
   "mcpServers": {
     "logbook-mcp": {
       "command": "npx",
-      "args": ["@cocaxcode/logbook-mcp", "--mcp"]
+      "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]
     }
   }
 }
@@ -103,7 +103,7 @@ Add to `.windsurf/mcp.json`:
   "mcpServers": {
     "logbook-mcp": {
       "command": "npx",
-      "args": ["@cocaxcode/logbook-mcp", "--mcp"]
+      "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]
     }
   }
 }
@@ -118,7 +118,7 @@ Add to `.vscode/mcp.json`:
   "servers": {
     "logbook-mcp": {
       "command": "npx",
-      "args": ["@cocaxcode/logbook-mcp", "--mcp"]
+      "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]
     }
   }
 }
@@ -127,7 +127,7 @@ Add to `.vscode/mcp.json`:
 ### Codex CLI
 
 ```bash
-codex mcp add logbook-mcp -- npx @cocaxcode/logbook-mcp --mcp
+codex mcp add logbook-mcp -- npx @cocaxcode/logbook-mcp@latest --mcp
 ```
 
 ### Gemini CLI
@@ -139,7 +139,7 @@ Add to `.gemini/settings.json`:
   "mcpServers": {
     "logbook-mcp": {
       "command": "npx",
-      "args": ["@cocaxcode/logbook-mcp", "--mcp"]
+      "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]
     }
   }
 }

package/dist/{cli-TYBWWQAH.js → cli-JFMNT5GT.js}

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 
 // src/cli.ts
-var VERSION = true ? "0.1.0" : "0.0.0";
+var VERSION = true ? "0.1.1" : "0.0.0";
 async function runCli(argv) {
   if (argv.includes("--version") || argv.includes("-v")) {
     console.log(`logbook-mcp v${VERSION}`);
@@ -16,7 +16,7 @@ async function runCli(argv) {
   console.log("Para usar con Claude Code, a\xF1ade a tu configuracion MCP:");
   console.log('  "logbook-mcp": {');
   console.log('    "command": "npx",');
-  console.log('    "args": ["@cocaxcode/logbook-mcp", "--mcp"]');
+  console.log('    "args": ["@cocaxcode/logbook-mcp@latest", "--mcp"]');
   console.log("  }");
 }
 export {

package/dist/index.js

@@ -6,13 +6,13 @@ async function main() {
   const hasMcpFlag = argv.includes("--mcp");
   if (hasMcpFlag) {
     const { StdioServerTransport } = await import("@modelcontextprotocol/sdk/server/stdio.js");
-    const { createServer } = await import("./server-5ZT2YKCQ.js");
+    const { createServer } = await import("./server-5CKGZQZD.js");
     const server = createServer();
     const transport = new StdioServerTransport();
     await server.connect(transport);
     console.error("logbook-mcp server running on stdio");
   } else {
-    const { runCli } = await import("./cli-TYBWWQAH.js");
+    const { runCli } = await import("./cli-JFMNT5GT.js");
     await runCli(argv);
   }
 }

package/dist/{server-5ZT2YKCQ.js → server-5CKGZQZD.js}

@@ -131,6 +131,9 @@ function insertRepo(db2, name, path) {
 function getRepoByPath(db2, path) {
   return db2.prepare("SELECT * FROM repos WHERE path = ?").get(path);
 }
+function getRepoByName(db2, name) {
+  return db2.prepare("SELECT * FROM repos WHERE name = ?").get(name);
+}
 function getTopicByName(db2, name) {
   return db2.prepare("SELECT * FROM topics WHERE name = ?").get(name);
 }
@@ -229,6 +232,7 @@ function getTodos(db2, filters = {}) {
   ).all(...params, limit);
 }
 function updateTodoStatus(db2, ids, status) {
+  if (ids.length === 0) return [];
   const completedAt = status === "done" ? (/* @__PURE__ */ new Date()).toISOString() : null;
   const placeholders = ids.map(() => "?").join(",");
   db2.prepare(
@@ -261,6 +265,7 @@ function updateTodo(db2, id, fields) {
   return db2.prepare(`${TODO_WITH_META_SQL} WHERE t.id = ?`).get(id);
 }
 function deleteTodos(db2, ids) {
+  if (ids.length === 0) return [];
   const placeholders = ids.map(() => "?").join(",");
   const existing = db2.prepare(`SELECT id FROM todos WHERE id IN (${placeholders})`).all(...ids);
   const existingIds = existing.map((r) => r.id);
@@ -345,7 +350,9 @@ function getCompletedTodos(db2, filters = {}) {
   ).all(...params);
 }
 function sanitizeFts(query) {
-  return query.split(/\s+/).filter(Boolean).map((word) => `"${word.replace(/"/g, "")}"`).join(" ");
+  const words = query.replace(/\0/g, "").split(/\s+/).filter(Boolean).map((word) => word.replace(/"/g, "")).filter(Boolean);
+  if (words.length === 0) return '""';
+  return words.map((word) => `"${word}"`).join(" ");
 }
 
 // src/tools/topics.ts
@@ -355,8 +362,8 @@ function registerTopicsTool(server) {
     "Lista o crea temas para organizar notas y TODOs. Temas predefinidos: feature, fix, chore, idea, decision, blocker.",
     {
       action: z.enum(["list", "add"]).default("list").describe("Accion: list (ver temas) o add (crear tema custom)"),
-      name: z.string().optional().describe("Nombre del nuevo tema (solo para action=add, lowercase, sin espacios)"),
-      description: z.string().optional().describe("Descripcion del nuevo tema (solo para action=add)")
+      name: z.string().min(1).max(50).regex(/^[a-z0-9-]+$/, "Solo letras minusculas, numeros y guiones").optional().describe("Nombre del nuevo tema (solo para action=add, lowercase, sin espacios)"),
+      description: z.string().max(200).optional().describe("Descripcion del nuevo tema (solo para action=add)")
     },
     async ({ action, name, description }) => {
       try {
@@ -368,8 +375,14 @@ function registerTopicsTool(server) {
               content: [{ type: "text", text: 'El parametro "name" es obligatorio para action=add' }]
             };
           }
-          const normalized = name.toLowerCase().replace(/\s+/g, "-");
-          const topic = insertTopic(db2, normalized, description);
+          const existing = getTopicByName(db2, name);
+          if (existing) {
+            return {
+              isError: true,
+              content: [{ type: "text", text: `El topic "${name}" ya existe` }]
+            };
+          }
+          const topic = insertTopic(db2, name, description);
           return {
             content: [{ type: "text", text: JSON.stringify(topic) }]
           };
@@ -381,7 +394,7 @@ function registerTopicsTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error: ${err.message}` }]
+          content: [{ type: "text", text: `Error: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -392,11 +405,11 @@ function registerTopicsTool(server) {
 import { z as z2 } from "zod";
 
 // src/git/detect-repo.ts
-import { execSync } from "child_process";
+import { execFileSync } from "child_process";
 import { basename } from "path";
 function detectRepoPath() {
   try {
-    const result = execSync("git rev-parse --show-toplevel", {
+    const result = execFileSync("git", ["rev-parse", "--show-toplevel"], {
       encoding: "utf-8",
       timeout: 5e3,
       stdio: ["pipe", "pipe", "pipe"]
@@ -411,7 +424,13 @@ function autoRegisterRepo(db2) {
   if (!repoPath) return null;
   const existing = getRepoByPath(db2, repoPath);
   if (existing) return existing;
-  const name = basename(repoPath);
+  let name = basename(repoPath);
+  const nameConflict = getRepoByName(db2, name);
+  if (nameConflict) {
+    const parts = repoPath.split("/");
+    const parent = parts.length >= 2 ? parts[parts.length - 2] : "repo";
+    name = `${parent}-${name}`;
+  }
   return insertRepo(db2, name, repoPath);
 }
 
@@ -421,7 +440,7 @@ function registerNoteTool(server) {
     "logbook_note",
     "A\xF1ade una nota al logbook. Topics: feature, fix, chore, idea, decision, blocker. Si no se pasa topic, queda sin categorizar.",
     {
-      content: z2.string().describe("Contenido de la nota"),
+      content: z2.string().min(1).max(5e3).describe("Contenido de la nota"),
       topic: z2.string().optional().describe("Topic: feature, fix, chore, idea, decision, blocker (o custom)")
     },
     async ({ content, topic }) => {
@@ -450,7 +469,7 @@ function registerNoteTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error creando nota: ${err.message}` }]
+          content: [{ type: "text", text: `Error creando nota: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -465,16 +484,16 @@ function registerTodoAddTool(server) {
     "logbook_todo_add",
     "Crea uno o varios TODOs. Para uno solo usa content. Para varios usa items (array). El topic se puede pasar o dejar que la AI lo infiera.",
     {
-      content: z3.string().optional().describe("Contenido del TODO (para crear uno solo)"),
+      content: z3.string().min(1).max(2e3).optional().describe("Contenido del TODO (para crear uno solo)"),
       topic: z3.string().optional().describe("Topic para el TODO individual"),
       priority: priorityEnum.optional().default("normal").describe("Prioridad del TODO individual"),
       items: z3.array(
         z3.object({
-          content: z3.string().describe("Contenido del TODO"),
+          content: z3.string().min(1).max(2e3).describe("Contenido del TODO"),
           topic: z3.string().optional().describe("Topic"),
           priority: priorityEnum.optional().default("normal").describe("Prioridad")
         })
-      ).optional().describe("Array de TODOs para crear varios a la vez")
+      ).max(50).optional().describe("Array de TODOs para crear varios a la vez (max 50)")
     },
     async ({ content, topic, priority, items }) => {
       try {
@@ -528,7 +547,7 @@ function registerTodoAddTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error creando TODO: ${err.message}` }]
+          content: [{ type: "text", text: `Error creando TODO: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -539,7 +558,7 @@ function registerTodoAddTool(server) {
 import { z as z4 } from "zod";
 
 // src/git/code-todos.ts
-import { execSync as execSync2 } from "child_process";
+import { execFileSync as execFileSync2 } from "child_process";
 var TAG_TO_TOPIC = {
   TODO: "feature",
   FIXME: "fix",
@@ -548,8 +567,9 @@ var TAG_TO_TOPIC = {
 };
 function scanCodeTodos(repoPath) {
   try {
-    const output = execSync2(
-      `git -C "${repoPath}" grep -n -E "(TODO|FIXME|HACK|BUG):" --no-color`,
+    const output = execFileSync2(
+      "git",
+      ["-C", repoPath, "grep", "-n", "-E", "(TODO|FIXME|HACK|BUG):", "--no-color"],
       { encoding: "utf-8", timeout: 1e4, stdio: ["pipe", "pipe", "pipe"] }
     );
     return parseGitGrepOutput(output);
@@ -646,7 +666,7 @@ function registerTodoListTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error listando TODOs: ${err.message}` }]
+          content: [{ type: "text", text: `Error listando TODOs: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -682,7 +702,7 @@ function registerTodoDoneTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error: ${err.message}` }]
+          content: [{ type: "text", text: `Error: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -732,7 +752,7 @@ function registerTodoEditTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error editando TODO: ${err.message}` }]
+          content: [{ type: "text", text: `Error editando TODO: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -765,7 +785,7 @@ function registerTodoRmTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error eliminando TODOs: ${err.message}` }]
+          content: [{ type: "text", text: `Error eliminando TODOs: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -835,7 +855,7 @@ function registerLogTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error: ${err.message}` }]
+          content: [{ type: "text", text: `Error: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -843,8 +863,9 @@ function registerLogTool(server) {
 }
 function resolveDates(period, from, to) {
   if (from) {
-    const dateTo = to ?? new Date(Date.now() + 864e5).toISOString().split("T")[0];
-    return { dateFrom: from, dateTo };
+    const endDate = to ?? from;
+    const nextDay = new Date(new Date(endDate).getTime() + 864e5).toISOString().split("T")[0];
+    return { dateFrom: from, dateTo: nextDay };
   }
   const now = /* @__PURE__ */ new Date();
   const today = now.toISOString().split("T")[0];
@@ -874,7 +895,7 @@ function registerSearchTool(server) {
     "logbook_search",
     "Busqueda full-text en notas y TODOs. Usa FTS5 para busqueda rapida.",
     {
-      query: z9.string().describe("Texto a buscar"),
+      query: z9.string().min(1).max(500).describe("Texto a buscar"),
       type: z9.enum(["all", "notes", "todos"]).optional().default("all").describe("Buscar en: all, notes, todos"),
       topic: z9.string().optional().describe("Filtrar por topic"),
       scope: z9.enum(["project", "global"]).optional().default("project").describe("Scope: project o global"),
@@ -928,7 +949,7 @@ function registerSearchTool(server) {
       } catch (err) {
         return {
           isError: true,
-          content: [{ type: "text", text: `Error buscando: ${err.message}` }]
+          content: [{ type: "text", text: `Error buscando: ${err instanceof Error ? err.message : String(err)}` }]
         };
       }
     }
@@ -936,7 +957,7 @@ function registerSearchTool(server) {
 }
 
 // src/server.ts
-var VERSION = true ? "0.1.0" : "0.0.0";
+var VERSION = true ? "0.1.1" : "0.0.0";
 function createServer() {
   const server = new McpServer({
     name: "logbook-mcp",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cocaxcode/logbook-mcp",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "MCP server — cuaderno de bitácora del developer. Notas, TODOs y code TODOs sin salir de tu AI.",
   "type": "module",
   "bin": {