changebook 0.7.0 → 0.7.1

package/dist/analyze.js

@@ -6,9 +6,37 @@
  */
 import * as path from "node:path";
 import { atlasWebUrl } from "./browser.js";
-import { commitDiff, execFileAsync, FICHEROS_GENERADOS, GIT_MAX_BUFFER_BYTES, gitErrorMessage, MAX_DIFF_CHARACTERS, projectNameFor, usableSummary, } from "./git.js";
+import { commitDiff, esFicheroGenerado, execFileAsync, FICHEROS_GENERADOS, ficherosDelCommit, GIT_MAX_BUFFER_BYTES, gitErrorMessage, MAX_DIFF_CHARACTERS, projectNameFor, usableSummary, } from "./git.js";
 import { canonicalDiffHash } from "./canonical.js";
 import { optimizeTokensForAI, truncateAtFileBoundary } from "./optimize.js";
+/**
+ * Por qué se salta un commit sin diff analizable. Tres causas, tres frases.
+ *
+ * HASTA EL 04/08 ERA UNA SOLA, Y ADIVINABA: «has no analyzable diff (merge?)».
+ * `commitDiff` excluye los ficheros que ChangeBook genera —`CLAUDE.md`,
+ * `AGENTS.md`— para no registrar sus propias regeneraciones en bucle, así que un
+ * commit normal que solo toque esos devuelve diff vacío igual que un merge. El
+ * `0bac769` de este repo era exactamente eso y el mensaje mandaba a buscar un
+ * merge que no existía.
+ *
+ * Saltar sigue siendo lo correcto en los tres casos: lo que cambia es que el
+ * instrumento sepa distinguirlos en vez de elegir uno (invariante 17 del repo).
+ *
+ * Es una función aparte y pura para que se pueda medir la ELECCIÓN sin montar
+ * un repo ni capturar `console.error`.
+ */
+export function motivoDelSalto(hash, ficheros) {
+    const corto = hash.slice(0, 8);
+    if (!ficheros.length) {
+        return `Commit ${corto} no toca ningun fichero (merge o commit vacio). Skipped.`;
+    }
+    if (ficheros.every(esFicheroGenerado)) {
+        return `Commit ${corto} solo toca ficheros que genera ChangeBook (${ficheros.join(", ")}). Saltado a proposito: regenerar el mapa no es un cambio de producto.`;
+    }
+    // Ni vacío ni todo generado: aquí no hay explicación conocida, y decirlo así
+    // es el punto. Inventar una tercera causa plausible sería repetir el bug.
+    return `Commit ${corto} toca ${ficheros.length} fichero(s) y aun asi no deja diff que analizar. Skipped. Ficheros: ${ficheros.join(", ")}`;
+}
 export async function analyze(db, options = {}) {
     const cwd = path.resolve(options.dir ?? process.cwd());
     const projectName = projectNameFor(cwd);
@@ -27,7 +55,17 @@ export async function analyze(db, options = {}) {
         agentSummary = usableSummary(meta.message) ?? undefined;
         rawDiff = await commitDiff(cwd, meta.hash);
         if (!rawDiff.trim()) {
-            console.error(`Commit ${meta.hash.slice(0, 8)} has no analyzable diff (merge?). Skipped.`);
+            // Un diff vacío tiene DOS causas y hasta el 04/08 se contaban como una:
+            // el commit no toca nada (un merge), o toca solo ficheros que ChangeBook
+            // genera y que `commitDiff` excluye a propósito. Decir «(merge?)» de lo
+            // segundo es adivinar, y adivinar mal: `0bac769` era un commit normal de
+            // CLAUDE.md y AGENTS.md, y el mensaje mandaba a buscar un merge que no
+            // existía.
+            //
+            // Saltarlo sigue siendo lo correcto en los dos casos. Lo que cambia es
+            // que ahora el instrumento sabe por qué está saltando (invariante 17).
+            const ficheros = await ficherosDelCommit(cwd, meta.hash);
+            console.error(motivoDelSalto(meta.hash, ficheros));
             return;
         }
     }

package/dist/git.js

@@ -75,8 +75,9 @@ export function usableSummary(message) {
  * exactamente eso.
  *
  * Un commit que SOLO toca estos ficheros produce un diff vacío y `analyze` lo
- * salta con su mensaje de siempre — que es la respuesta correcta: regenerar el
- * mapa no es un cambio de producto.
+ * salta, que es la respuesta correcta: regenerar el mapa no es un cambio de
+ * producto. Lo que NO era correcto era cómo lo contaba — ver
+ * `ficherosDelCommit` aquí abajo y el mensaje en `analyze.ts`.
  */
 export const FICHEROS_GENERADOS = [
     ":(exclude)CLAUDE.md",
@@ -104,6 +105,45 @@ export async function commitDiff(cwd, hash) {
         throw new Error(`Could not read the diff of ${hash.slice(0, 8)}: ${gitErrorMessage(error)}`);
     }
 }
+/**
+ * Los ficheros que toca un commit, SIN excluir los generados.
+ *
+ * POR QUÉ EXISTE. `commitDiff` excluye `CLAUDE.md`/`AGENTS.md`, así que un
+ * commit que solo toca esos devuelve diff vacío — igual que un merge, que no
+ * tiene diff propio. Dos situaciones distintas, una sola señal, y `analyze`
+ * elegía la equivocada: decía «has no analyzable diff (merge?)» de un commit
+ * normal que ni siquiera era un merge (medido el 04/08 con `0bac769`).
+ *
+ * Es la invariante 17 del repo dentro del producto: la ausencia de diff no
+ * prueba la ausencia de cambio mientras el instrumento no sepa distinguir por
+ * qué está ausente. Esta función es lo que le da esa segunda pregunta.
+ *
+ * Devuelve `[]` si el commit no toca ningún fichero — el caso del merge de
+ * verdad, y también el de un commit vacío.
+ */
+export async function ficherosDelCommit(cwd, hash) {
+    try {
+        const { stdout } = await execFileAsync("git", 
+        // Mismo `--end-of-options` que commitDiff, por el mismo motivo.
+        ["show", "--pretty=format:", "--name-only", "--end-of-options", hash], { cwd, encoding: "utf8", maxBuffer: GIT_MAX_BUFFER_BYTES });
+        return stdout
+            .split("\n")
+            .map((l) => l.trim())
+            .filter(Boolean);
+    }
+    catch (error) {
+        throw new Error(`Could not read the files of ${hash.slice(0, 8)}: ${gitErrorMessage(error)}`);
+    }
+}
+/**
+ * ¿Es un fichero de los que ChangeBook genera? Se decide con las MISMAS rutas
+ * que excluye `commitDiff`, derivadas de ella y no reescritas: dos listas que
+ * pudieran separarse harían que el mensaje explicara un salto que no ocurrió.
+ */
+export function esFicheroGenerado(ruta) {
+    const nombre = ruta.split("/").pop() ?? "";
+    return FICHEROS_GENERADOS.some((patron) => patron.replace(/^:\(exclude\)(\*\*\/)?/, "") === nombre);
+}
 /** Pulls git's stderr out of an execFile rejection for a readable message. */
 export function gitErrorMessage(error) {
     if (typeof error === "object" &&

package/dist/sync.js

@@ -54,6 +54,37 @@ const FIRMA = '_Generado por ChangeBook · changebook.app_';
 function sanitizeCell(text) {
     return text.replace(/<!--/g, '<!- -');
 }
+/**
+ * Recorta a `tope` caracteres sin partir palabras y DICIENDO que ha recortado.
+ *
+ * POR QUÉ EXISTE. Los topes de este bloque son deliberados —entra en cada
+ * sesión de agente y se reenvía en cada petición, así que es coste FIJO— pero
+ * se aplicaban con un `.slice(0, n)` pelado. El 04/08 el CLAUDE.md de este
+ * mismo repo decía «permitiendo validar q»: cortado a media palabra y sin
+ * ninguna marca. Eso no se lee como un extracto, se lee como salida rota, y
+ * este bloque es el artefacto más leído del producto.
+ *
+ * DOS COSAS, Y LAS DOS IMPORTAN:
+ *
+ *   · La elipsis SOLO aparece si de verdad se recortó. Ponerla siempre haría
+ *     que toda nota pareciera truncada, que es el error simétrico y igual de
+ *     malo: dejaría de distinguir lo completo de lo cortado.
+ *   · El hueco de la elipsis sale DE DENTRO del tope. El presupuesto está en
+ *     caracteres; un tope que se desborda por la marca que anuncia el tope es
+ *     un tope de mentira.
+ *
+ * Una palabra sola más larga que el tope se corta a lo bruto: retroceder al
+ * espacio anterior devolvería la cadena vacía, y perder la nota entera es peor
+ * que un corte feo. Ese caso está en el corpus del test.
+ */
+export function recorta(texto, tope) {
+    if (texto.length <= tope)
+        return texto;
+    const cortado = texto.slice(0, tope - 1);
+    const ultimoEspacio = cortado.lastIndexOf(' ');
+    const base = ultimoEspacio > 0 ? cortado.slice(0, ultimoEspacio) : cortado;
+    return `${base.replace(/[\s.,;:]+$/, '')}…`;
+}
 const MAX_MODULES = 15;
 const MAX_CHANGES = 5;
 const MAX_COUPLINGS = 5;
@@ -464,21 +495,21 @@ maxCitas = CITAS_GRATIS) {
         // un parrafo. Tres alertas a 200 chars se comian el 30% del presupuesto
         // y expulsaban el mapa. El texto entero sigue a una llamada de
         // `atlas_project_brief`, donde se paga solo si alguien pregunta.
-        return `- ${a.created_at.slice(0, 10)}${mod ? ` · **${mod}**` : ''} — ${sanitizeCell((a.plain ?? '').slice(0, 130))}`;
+        return `- ${a.created_at.slice(0, 10)}${mod ? ` · **${mod}**` : ''} — ${sanitizeCell(recorta(a.plain ?? '', 130))}`;
     });
     const hotspotLines = modules
         .filter((m) => m.risk === 'hotspot')
         .slice(0, 5)
         .map((m) => {
-        const note = sanitizeCell((m.note ?? '').slice(0, 110));
+        const note = sanitizeCell(recorta(m.note ?? '', 110));
         return `- **${m.module}** está marcado crítico${note ? ` — ${note}` : ''}`;
     });
-    const changeLines = changes.map((c) => `- ${c.created_at.slice(0, 10)} — ${sanitizeCell((c.business_impact ?? '').slice(0, 140))}`);
+    const changeLines = changes.map((c) => `- ${c.created_at.slice(0, 10)} — ${sanitizeCell(recorta(c.business_impact ?? '', 140))}`);
     // Salud en riesgo: los controles (auth, secretos, validación, límites…) que
     // el análisis ya marcó rotos con evidencia. Alta prioridad de presupuesto —
     // es seguridad. Solo los at_risk (lo accionable); el texto entero va a
     // `atlas_project_brief`. Cap a 130 chars como las alertas.
-    const healthLines = atRiskHealth.map((h) => `- ⚠ **${h.check}**${h.since ? ` (desde ${h.since})` : ''} — ${sanitizeCell(h.evidence.slice(0, 130))}`);
+    const healthLines = atRiskHealth.map((h) => `- ⚠ **${h.check}**${h.since ? ` (desde ${h.since})` : ''} — ${sanitizeCell(recorta(h.evidence, 130))}`);
     // Auto-remediación fase 2: la cola entra en cada sesión para que el agente
     // se OFREZCA a atacarla — proponer con plan y esperar el OK del humano,
     // nunca ejecutar por su cuenta. Títulos deduplicados (la cola real puede
@@ -489,7 +520,7 @@ maxCitas = CITAS_GRATIS) {
     const taskLines = taskTitles.length > 0
         ? [
             `- Hay ${pendingTasks.length} encargo(s) pendientes en la cola de este proyecto. Propón cuál atacarías. Cola viva: \`atlas_pending_tasks\`; cierra con \`atlas_complete_task\`.`,
-            ...taskTitles.map((t) => `- ${sanitizeCell(t.slice(0, 120))}`),
+            ...taskTitles.map((t) => `- ${sanitizeCell(recorta(t, 120))}`),
         ]
         : [];
     // El orden de renderizado (legibilidad) y la prioridad de presupuesto (qué se

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "changebook",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "mcpName": "io.github.raulbr90/changebook",
   "description": "Your agent already broke this three times. ChangeBook tells it before the fourth. MCP server + CLI: the history of what broke in your repo, served to Claude Code, Cursor or Codex before they edit.",
   "type": "module",

package/server.json

@@ -2,7 +2,7 @@
   "$schema": "https://static.modelcontextprotocol.io/schemas/2025-12-11/server.schema.json",
   "name": "io.github.raulbr90/changebook",
   "description": "Your agent already broke this three times. ChangeBook tells it before the fourth.",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "websiteUrl": "https://changebook.dev",
   "remotes": [
     {
@@ -15,7 +15,7 @@
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "changebook",
-      "version": "0.7.0",
+      "version": "0.7.1",
       "transport": {
         "type": "stdio"
       }