alegra-data-agent 0.2.3 → 0.2.5

package/dist/commands/install.js

@@ -1,9 +1,9 @@
 import { AuthRequiredError } from "../api.js";
 import { hasValidAuth, readAuth } from "../auth.js";
+import { installAutoUpdate } from "../autoupdate.js";
 import { bold, cyan, dim, gray, green, log, logError, red, yellow } from "../colors.js";
 import { installAll } from "../installer.js";
-import { COMMAND_NAME, formatBytes, getClaudeSkillsDir, getPackageVersion } from "../lib.js";
-import { runHookInstall } from "./hook.js";
+import { COMMAND_NAME, formatBytes, getPackageVersion, getTargetSkillsDir } from "../lib.js";
 import { runLogin } from "./login.js";
 const BANNER_WIDTH = 58;
 const SHORT_DESCRIPTIONS = {
@@ -51,6 +51,10 @@ function shortDesc(name, manifestDesc) {
     const oneLine = manifestDesc.split(/\.|\n/)[0].trim();
     return oneLine.length > 55 ? `${oneLine.slice(0, 54)}…` : oneLine;
 }
+function formatDestinos(targets) {
+    const dirs = targets.map((t) => getTargetSkillsDir(t));
+    return dirs.length > 0 ? dirs.join("  ·  ") : "(ninguno)";
+}
 function printResultRow(result, manifest) {
     const isSkill = result.kind === "skill";
     const entry = isSkill ? manifest.skills[result.name] : manifest.mcps?.[result.name];
@@ -81,7 +85,7 @@ function printResultRow(result, manifest) {
     const padName = result.name.padEnd(22);
     log(`   ${icon} ${kindBadge}${bold(padName)} ${gray(`· ${desc}`)}`);
 }
-function printSummary(results, manifest) {
+function printSummary(results, manifest, targets) {
     const installed = results.filter((r) => r.status === "installed").length;
     const updated = results.filter((r) => r.status === "updated").length;
     const unchanged = results.filter((r) => r.status === "unchanged").length;
@@ -112,7 +116,7 @@ function printSummary(results, manifest) {
     if (failed > 0)
         parts.push(red(`${failed} fallidas`));
     log(`   ${green("✓")} ${bold(parts.join(gray(" · ")))}${gray(`  ·  ${totalFiles} archivos · ${formatBytes(totalBytes)}`)}`);
-    log(`   ${green("✓")} ${gray("destino:")} ${getClaudeSkillsDir()}`);
+    log(`   ${green("✓")} ${gray("destino:")} ${formatDestinos(targets)}`);
     log("");
     for (const r of results.filter((x) => x.status === "failed")) {
         logError(`${r.name}: ${r.error}`);
@@ -120,7 +124,7 @@ function printSummary(results, manifest) {
 }
 function printNextSteps(noHook) {
     section("Ya estás listo");
-    log(`   ${bold(yellow("1."))} ${bold("Reiniciá tu cliente")} ${gray("(Claude Code, Cursor, OpenCode o Antigravity)")}.`);
+    log(`   ${bold(yellow("1."))} ${bold("Reiniciá tu cliente")} ${gray("(Claude Code, Cursor, OpenCode, Antigravity o Codex)")}.`);
     log(`   ${bold(yellow("2."))} ${bold("Hacele tu pregunta en español.")}`);
     log(`      ${gray("Ejemplo:")} ${cyan('"cuántos nuevos clientes en mayo en Colombia?"')}`);
     log("");
@@ -137,7 +141,7 @@ function printNextSteps(noHook) {
         log(`     ${dim("$")} ${bold(`${COMMAND_NAME} hook uninstall`)}  ${gray("apagar aviso")}`);
     }
     log("");
-    log(`   ${dim("Docs · ")}${cyan("github.com/alegradev/alegra-data-agent")}`);
+    log(`   ${dim("Docs · ")}${cyan("github.com/Alegra-Team/data-agent-selfservice")}`);
     log("");
 }
 function isClaudeCode() {
@@ -176,7 +180,10 @@ export async function runInstall(opts = {}) {
         const msg = err instanceof Error ? err.message : String(err);
         logError(msg);
         if (msg.includes("EACCES") || msg.includes("permission denied")) {
-            log(dim("  Sin permisos para escribir. Verifica permisos de ~/.claude/skills/"));
+            const dirs = opts.targets?.length
+                ? formatDestinos(opts.targets)
+                : "las carpetas de skills de tus clientes AI";
+            log(dim(`  Sin permisos para escribir. Verifica permisos de ${dirs}`));
         }
         else if (msg.includes("ENOSPC") || msg.includes("no space")) {
             log(dim("  No hay espacio en disco. Libera espacio y reintenta."));
@@ -216,32 +223,34 @@ export async function runInstall(opts = {}) {
         return 1;
     }
     if (compact) {
-        printCompactSummary(result.results, result.manifest, auth?.email);
+        printCompactSummary(result.results, result.manifest, result.targets, auth?.email);
     }
     else {
         for (const r of result.results) {
             printResultRow(r, result.manifest);
         }
-        printSummary(result.results, result.manifest);
+        printSummary(result.results, result.manifest, result.targets);
     }
     const hasFails = result.results.some((r) => r.status === "failed");
-    if (hasFails)
-        return 1;
     if (opts.dryRun)
-        return 0;
+        return hasFails ? 1 : 0;
     if (!opts.noHook) {
         if (!compact)
-            log(`  ${gray("Instalando hook de notificaciones…")}`);
-        runHookInstall(opts.hookMode ?? "check");
+            log(`  ${gray("Configurando auto-update por cliente…")}`);
+        for (const t of result.targets) {
+            installAutoUpdate(t, opts.hookMode ?? "auto");
+        }
         if (!compact)
             log("");
     }
+    if (hasFails)
+        return 1;
     if (!compact) {
         printNextSteps(opts.noHook ?? false);
     }
     return 0;
 }
-function printCompactSummary(results, manifest, email) {
+function printCompactSummary(results, manifest, targets, email) {
     const ok = results.filter((r) => r.status === "installed" || r.status === "updated" || r.status === "unchanged");
     const failed = results.filter((r) => r.status === "failed");
     const installed = results.filter((r) => r.status === "installed").length;
@@ -273,7 +282,7 @@ function printCompactSummary(results, manifest, email) {
             .map((r) => r.name)
             .sort()
             .join(", "))}`);
-    log(`  ${gray("destino:")} ${cyan(getClaudeSkillsDir())}${gray("  ·  ")}${bold(yellow("reinicia tu cliente para activar skills y MCPs"))}`);
+    log(`  ${gray("destino:")} ${cyan(formatDestinos(targets))}${gray("  ·  ")}${bold(yellow("reinicia tu cliente para activar skills y MCPs"))}`);
     if (unchanged > 0 && installed === 0 && updated === 0) {
         log(`  ${dim(`(${unchanged} sin cambios desde la última instalación)`)}`);
     }

package/dist/commands/login.js

@@ -19,7 +19,7 @@ export async function runLogin(opts = {}) {
     }
     log(`  Gateway: ${dim(GATEWAY_URL)}`);
     try {
-        const record = await performBrowserLogin();
+        const record = await performBrowserLogin(opts.timeoutMs);
         writeAuth(record);
         log("");
         log(`  ${green("✓")} Autenticado como ${bold(record.email)}.`);

package/dist/commands/status.js

@@ -1,19 +1,128 @@
+import { existsSync, readFileSync } from "node:fs";
 import { AuthRequiredError, fetchManifest } from "../api.js";
 import { readAuth } from "../auth.js";
 import { bold, cyan, dim, gray, green, log, logError, logWarn, yellow } from "../colors.js";
-import { detectDrift } from "../installer.js";
-import { COMMAND_NAME, GATEWAY_URL, getClaudeSkillsDir, getLockFilePath, getPackageVersion, readLockFile, } from "../lib.js";
+import { detectDrift, inspectMcpInTarget } from "../installer.js";
+import { COMMAND_NAME, GATEWAY_URL, getAutoUpdateLogPath, getClaudeSkillsDir, getLockFilePath, getPackageVersion, getTargetSkillsDir, readLockFile, } from "../lib.js";
+import { TARGETS } from "../targets.js";
+function reportAutoUpdateLog() {
+    const logPath = getAutoUpdateLogPath();
+    if (!existsSync(logPath))
+        return;
+    try {
+        const lines = readFileSync(logPath, "utf-8").trim().split("\n").filter(Boolean);
+        if (lines.length === 0)
+            return;
+        logWarn(`El auto-update no pudo correr ${lines.length} vez/veces en un cliente GUI — última: "${lines[lines.length - 1]}". Agente: corré ${bold(`${COMMAND_NAME} update`)} (actualiza ya mismo y limpia este aviso); si reaparece, el cliente GUI no encuentra pnpm en su PATH y hay que reinstalar con ${bold(`${COMMAND_NAME} install`)} desde una terminal con pnpm disponible.`);
+        log("");
+    }
+    catch {
+    }
+}
+function reportMcps(manifest, lock) {
+    const mcps = manifest.mcps ?? {};
+    const names = Object.keys(mcps);
+    if (names.length === 0)
+        return;
+    log(bold("  MCPs:"));
+    for (const name of names) {
+        const entry = mcps[name];
+        const targets = (lock.mcps?.[name]?.targets ?? []);
+        if (targets.length === 0) {
+            log(`    ${cyan("+")} ${name} ${gray("(disponible, sin instalar)")}`);
+            continue;
+        }
+        const ok = [];
+        const missing = [];
+        const modified = [];
+        for (const t of targets) {
+            const label = TARGETS[t].label;
+            const state = inspectMcpInTarget(entry, TARGETS[t]);
+            if (state === "ok")
+                ok.push(label);
+            else if (state === "missing")
+                missing.push(label);
+            else
+                modified.push(label);
+        }
+        const notes = [];
+        if (ok.length > 0)
+            notes.push(gray(`ok en ${ok.join(", ")}`));
+        if (modified.length > 0)
+            notes.push(dim(`editado a mano en ${modified.join(", ")}`));
+        if (missing.length > 0)
+            notes.push(yellow(`falta en ${missing.join(", ")}`));
+        const icon = missing.length > 0 ? yellow("✗") : modified.length > 0 ? dim("~") : green("✓");
+        log(`    ${icon} ${name} ${gray("·")} ${notes.join(gray(" · "))}`);
+    }
+    log("");
+}
+export function findRogueMetabase(manifest, specs = Object.values(TARGETS)) {
+    const official = manifest.mcps?.metabase?.url;
+    if (!official)
+        return [];
+    const norm = (u) => u.trim().replace(/\/+$/, "").toLowerCase();
+    const officialNorm = norm(official);
+    const found = [];
+    for (const spec of specs) {
+        if (!spec.mcpConfigPath.endsWith(".json"))
+            continue;
+        if (!existsSync(spec.mcpConfigPath))
+            continue;
+        try {
+            const cfg = JSON.parse(readFileSync(spec.mcpConfigPath, "utf-8"));
+            const servers = cfg[spec.mcpRootKey];
+            if (!servers || typeof servers !== "object")
+                continue;
+            for (const [name, def] of Object.entries(servers)) {
+                if (!/metabase/i.test(name))
+                    continue;
+                const url = def?.[spec.mcpUrlKey];
+                if (typeof url === "string" && norm(url) === officialNorm)
+                    continue;
+                found.push(`${spec.label} → ${name}${typeof url === "string" ? ` (${url})` : ""}`);
+            }
+        }
+        catch {
+        }
+    }
+    return found;
+}
+function reportRogueMetabase(manifest) {
+    const official = manifest.mcps?.metabase?.url;
+    if (!official)
+        return;
+    const found = findRogueMetabase(manifest);
+    if (found.length === 0)
+        return;
+    logWarn(`Hay ${found.length} MCP(s) de Metabase que NO son el oficial de Alegra: ${found.join(" · ")}`);
+    log(gray(`                  El oficial es ${official}. Los otros son otra fuente y dan otros números.`));
+    log(gray("                  Las skills tienen instrucción de no usarlos, pero eso no los bloquea:"));
+    log(gray("                  si no los necesitás, conviene quitarlos del cliente."));
+    log("");
+}
 export async function runStatus() {
     const auth = readAuth();
+    const lockForTargets = readLockFile();
+    const installedTargets = [
+        ...new Set([
+            ...Object.values(lockForTargets.skills).flatMap((s) => s.targets ?? []),
+            ...Object.values(lockForTargets.mcps ?? {}).flatMap((m) => m.targets ?? []),
+        ]),
+    ];
+    const destino = installedTargets.length
+        ? installedTargets.map((t) => getTargetSkillsDir(t)).join("  ·  ")
+        : getClaudeSkillsDir();
     log("");
     log(bold(cyan("  Estado")));
     log(gray(`  ${"─".repeat(70)}`));
     log(`  ${gray("Paquete:")}        ${bold(COMMAND_NAME)} v${getPackageVersion()}`);
     log(`  ${gray("Gateway:")}        ${GATEWAY_URL}`);
     log(`  ${gray("Autenticado:")}    ${auth ? green(auth.email) : gray("no")}`);
-    log(`  ${gray("Destino:")}        ${getClaudeSkillsDir()}`);
+    log(`  ${gray("Destino:")}        ${destino}`);
     log(`  ${gray("Lock file:")}      ${getLockFilePath()}`);
     log("");
+    reportAutoUpdateLog();
     if (!auth) {
         logWarn(`Inicia sesión con: ${bold(`${COMMAND_NAME} login`)}`);
         log("");
@@ -37,6 +146,12 @@ export async function runStatus() {
     }
     const lock = readLockFile();
     const drift = detectDrift(manifest);
+    const cliVersion = getPackageVersion();
+    if (manifest.productVersion && manifest.productVersion !== cliVersion) {
+        log(`  ${gray("Manifest:")}       generado por v${manifest.productVersion} ${gray(`· tu CLI corre v${cliVersion}`)}`);
+        log(gray("                  No es un error: el gateway y npm se despliegan por separado."));
+        log("");
+    }
     log(bold("  Skills:"));
     log(`    ${green("✓")} ${drift.unchanged.length} al día`);
     if (drift.toInstall.length > 0)
@@ -46,9 +161,14 @@ export async function runStatus() {
     if (drift.orphaned.length > 0)
         log(`    ${dim("·")} ${drift.orphaned.length} huérfanas: ${drift.orphaned.join(", ")}`);
     log("");
-    const total = drift.toInstall.length + drift.toUpdate.length;
+    reportMcps(manifest, lock);
+    reportRogueMetabase(manifest);
+    const total = drift.toInstall.length +
+        drift.toUpdate.length +
+        drift.mcps.toInstall.length +
+        drift.mcps.toUpdate.length;
     if (total > 0) {
-        logWarn(`Hay ${total} skill(s) por actualizar. Agente: ejecutá vos mismo ${bold(`pnpm dlx ${COMMAND_NAME} update --yes`)} — el usuario no corre comandos.`);
+        logWarn(`Hay ${total} elemento(s) (skills + MCPs) por actualizar. Agente: ejecutá vos mismo ${bold(`pnpm dlx ${COMMAND_NAME} update --yes`)} — el usuario no corre comandos.`);
     }
     else if (Object.keys(lock.skills).length === 0) {
         logWarn(`Aún no hay skills instaladas. Agente: ejecutá vos mismo ${bold(`pnpm dlx ${COMMAND_NAME}`)} — el usuario no corre comandos.`);

package/dist/commands/uninstall.js

@@ -1,21 +1,30 @@
 import { existsSync, readFileSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { join } from "node:path";
+import { removeAutoUpdate } from "../autoupdate.js";
 import { dim, gray, log, logOk, logWarn, red } from "../colors.js";
 import { uninstallAll } from "../installer.js";
 import { readLockFile } from "../lib.js";
+import { TARGETS } from "../targets.js";
 import { banner, confirm } from "../ui.js";
 export async function runUninstall(opts = {}) {
     banner();
     const lock = readLockFile();
     const skills = Object.keys(lock.skills);
-    if (skills.length === 0) {
-        logWarn("No hay skills instaladas por este paquete.");
+    const mcps = Object.keys(lock.mcps ?? {});
+    if (skills.length === 0 && mcps.length === 0) {
+        logWarn("No hay skills ni MCPs instalados por este paquete.");
+        if (!opts.dryRun) {
+            for (const t of Object.keys(TARGETS))
+                removeAutoUpdate(t);
+        }
         return 0;
     }
-    log(red(`  Vas a eliminar ${skills.length} skill(s):`));
+    log(red(`  Vas a eliminar ${skills.length} skill(s) y ${mcps.length} MCP(s):`));
     for (const s of skills)
         log(gray(`    - ${s}`));
+    for (const m of mcps)
+        log(gray(`    - ${m} (MCP)`));
     log("");
     if (!opts.yes && !opts.dryRun) {
         const ok = await confirm("¿Continuar?", false);
@@ -25,8 +34,11 @@ export async function runUninstall(opts = {}) {
         }
     }
     const results = uninstallAll({ dryRun: opts.dryRun });
-    if (!opts.dryRun)
+    if (!opts.dryRun) {
+        for (const t of Object.keys(TARGETS))
+            removeAutoUpdate(t);
         removeNpmrcCooldownExclude();
+    }
     log("");
     if (opts.dryRun) {
         logOk(`${results.length} skill(s) se eliminarían (dry-run, nada borrado).`);

package/dist/commands/update.js

@@ -1,16 +1,45 @@
+import { mkdirSync, readFileSync, rmSync, writeFileSync } from "node:fs";
+import { dirname } from "node:path";
 import { AuthRequiredError, fetchManifest } from "../api.js";
 import { hasValidAuth } from "../auth.js";
+import { refreshAutoUpdate } from "../autoupdate.js";
 import { cyan, dim, gray, green, log, logError, yellow } from "../colors.js";
 import { detectDrift, installAll, summarize } from "../installer.js";
+import { getAutoUpdateLogPath, getLastCheckPath } from "../lib.js";
 import { banner } from "../ui.js";
-import { refreshHookCommand } from "./hook.js";
 import { runLogin } from "./login.js";
+const BACKGROUND_COOLDOWN_MS = 15 * 60_000;
+const BACKGROUND_LOGIN_TIMEOUT_MS = 90_000;
+function checkedRecently() {
+    try {
+        const raw = readFileSync(getLastCheckPath(), "utf-8");
+        const at = Date.parse(JSON.parse(raw).checkedAt ?? "");
+        if (Number.isNaN(at))
+            return false;
+        const age = Date.now() - at;
+        return age >= 0 && age < BACKGROUND_COOLDOWN_MS;
+    }
+    catch {
+        return false;
+    }
+}
+function markChecked() {
+    try {
+        const path = getLastCheckPath();
+        mkdirSync(dirname(path), { recursive: true });
+        writeFileSync(path, `${JSON.stringify({ checkedAt: new Date().toISOString() }, null, 2)}\n`);
+    }
+    catch {
+    }
+}
 export async function runUpdate(opts = {}) {
+    if (opts.background && !opts.dryRun && checkedRecently())
+        return 0;
     banner();
     if (!hasValidAuth()) {
-        const code = await runLogin({});
+        const code = await runLogin(opts.background ? { timeoutMs: BACKGROUND_LOGIN_TIMEOUT_MS } : {});
         if (code !== 0)
-            return code;
+            return opts.background ? 0 : code;
     }
     let reauthed = false;
     for (;;) {
@@ -51,7 +80,11 @@ async function runUpdateOnce(opts) {
     if (pending.length === 0) {
         log(green("  ✓ Todo al día (skills + MCPs).\n"));
         if (!opts.dryRun)
-            refreshHookCommand();
+            markChecked();
+        if (!opts.dryRun) {
+            refreshAutoUpdate();
+            clearAutoUpdateLog();
+        }
         return 0;
     }
     log(cyan(`  → ${pending.length} elemento(s) por actualizar:\n`));
@@ -61,20 +94,35 @@ async function runUpdateOnce(opts) {
         log(`    ${yellow("↻")} ${name} ${gray("(skill nueva versión)")}`);
     for (const name of drift.mcps.toInstall)
         log(`    ${cyan("+")} ${name} ${gray("(MCP nuevo)")}`);
-    for (const name of drift.mcps.toUpdate)
-        log(`    ${yellow("↻")} ${name} ${gray("(MCP actualizado)")}`);
+    for (const name of drift.mcps.toUpdate) {
+        const motivo = drift.mcps.missing.includes(name)
+            ? "(MCP ausente en el cliente — se repone)"
+            : "(MCP actualizado)";
+        log(`    ${yellow("↻")} ${name} ${gray(motivo)}`);
+    }
     log("");
     if (opts.dryRun) {
         log(dim("  Modo dry-run: no se modificará nada.\n"));
         return 0;
     }
-    const result = await installAll({ only: pending, targets: opts.targets, force: true });
+    const result = await installAll({ only: pending, targets: opts.targets });
     if ("error" in result) {
         logError(result.error);
         return 1;
     }
     summarize(result.results);
-    refreshHookCommand();
+    refreshAutoUpdate();
     const failed = result.results.some((r) => r.status === "failed");
+    if (!failed) {
+        clearAutoUpdateLog();
+        markChecked();
+    }
     return failed ? 1 : 0;
 }
+function clearAutoUpdateLog() {
+    try {
+        rmSync(getAutoUpdateLogPath(), { force: true });
+    }
+    catch {
+    }
+}