changebook 0.5.0 → 0.6.0

package/dist/sync.js

@@ -97,7 +97,7 @@ export async function fetchBriefSection(db, targetDir) {
     }
     const projectId = /project_id=eq\.([0-9a-f-]+)/.exec(projectFilter)?.[1] ?? null;
     const since = new Date(Date.now() - ALERT_WINDOW_DAYS * 24 * 3600 * 1000).toISOString();
-    const [moduleRows, changes, alerts, pendingTasks, healthRows] = await Promise.all([
+    const [moduleRows, changes, alerts, historial, pendingTasks, healthRows] = await Promise.all([
         db.rest('change_module?select=changelog_id,module,domain,risk,files,note,created_at&order=created_at.desc&limit=500' +
             projectFilter),
         db.rest(`changelog?select=business_impact,created_at&order=created_at.desc&limit=${MAX_CHANGES}` +
@@ -106,6 +106,12 @@ export async function fetchBriefSection(db, targetDir) {
             .rest(`regression_alerts?select=module,plain,created_at&created_at=gte.${since}&resolved_at=is.null&order=created_at.desc&limit=${MAX_ALERTS}` +
             projectFilter)
             .catch(() => []),
+        // HISTORIA de regresiones: todas, sin ventana y sin filtrar por abiertas.
+        // Es lo que sustituye al mapa de modulos en el bloque — ver `hechosCaros`.
+        db
+            .rest('regression_alerts?select=module,plain,created_at,changelog_id&order=created_at.desc&limit=500' +
+            projectFilter)
+            .catch(() => []),
         projectResolved && projectId
             ? db
                 .callRpc('list_agent_tasks', {
@@ -121,7 +127,27 @@ export async function fetchBriefSection(db, targetDir) {
             .catch(() => []),
     ]);
     const health = summarizeHealth(healthRows);
-    const section = buildSection(moduleRows, changes, alerts, projectName, pendingTasks, health.at_risk);
+    // El commit de cada regresion, para poder citarlo. Una consulta mas, y solo
+    // por los analisis que de verdad rompieron algo: es la diferencia entre «este
+    // modulo ha roto 3 veces» y «este modulo ha roto 3 veces, aqui, aqui y aqui».
+    // Sin el hash, la frase es una etiqueta; con el, es algo que se puede ir a
+    // mirar. El sync corre UNA vez por sesion, asi que la ronda extra se paga una
+    // vez y no por edicion.
+    const idsDeRegresion = [
+        ...new Set(historial.map((h) => h.changelog_id).filter((x) => Boolean(x))),
+    ].slice(0, 60);
+    const commitPorAnalisis = new Map();
+    if (idsDeRegresion.length > 0) {
+        const filas = await db
+            .rest(`changelog?select=id,commit_hash&id=in.(${idsDeRegresion.join(',')})` +
+            projectFilter)
+            .catch(() => []);
+        for (const f of filas) {
+            if (f.commit_hash)
+                commitPorAnalisis.set(f.id, f.commit_hash.slice(0, 7));
+        }
+    }
+    const section = buildSection(moduleRows, changes, alerts, projectName, pendingTasks, health.at_risk, historial, commitPorAnalisis);
     return { section, projectId, projectResolved };
 }
 export async function syncContextFiles(db, targetDir, opts = {}) {
@@ -146,7 +172,67 @@ export async function syncContextFiles(db, targetDir, opts = {}) {
     }
 }
 /** Exported for tests. */
-export function buildSection(rows, changes, alerts = [], projectName, pendingTasks = [], atRiskHealth = []) {
+/**
+ * LO QUE YA COSTO CARO AQUI: modulos que han roto algo, con fecha y commit.
+ *
+ * ── POR QUE ESTO Y NO EL MAPA DE MODULOS ────────────────────────────────────
+ *
+ * La documentacion de Claude Code lo dice de su propio `/doctor`: recorta del
+ * CLAUDE.md «directory layouts, dependency lists, and architecture overviews»
+ * y conserva «pitfalls, rationale, and conventions that differ from tool
+ * defaults». La lista de modulos con su dominio es DERIVABLE: el agente la saca
+ * leyendo el repo, y la herramienta la recorta por su cuenta.
+ *
+ * Esto no. Que un modulo haya roto tres veces sale del HISTORIAL, no del
+ * codigo: ni un grep ni un glob pueden encontrarlo, y es exactamente la
+ * categoria que /doctor conserva.
+ *
+ * ── DOS DECISIONES QUE PARECEN DETALLES ─────────────────────────────────────
+ *
+ * FECHA Y HASH POR ENTRADA, NUNCA EN LA CABECERA. Una fecha en la cabecera
+ * cambiaria a diario e invalidaria el prefijo de prompt-cache de TODO el
+ * fichero en cada sesion. Por entrada no: una linea que no cambia no invalida
+ * nada, y `upsertSection` sigue devolviendo `unchanged` cuando no ha pasado
+ * nada, asi que hechos con fecha no producen churn diario en `git status`.
+ *
+ * SE CUENTAN PROBLEMAS DISTINTOS, no filas. Mismo criterio que
+ * `computeRecidivism`: la misma regresion re-levantada tres veces es UNA, y
+ * contarla tres veces convertiria el churn del generador en falsa gravedad.
+ */
+export function hechosCaros(historial, commitPorAnalisis, tope = 6) {
+    const porModulo = new Map();
+    for (const h of historial) {
+        const modulo = (h.module ?? '').trim();
+        const texto = (h.plain ?? '').trim();
+        if (!modulo || !texto)
+            continue;
+        const entrada = porModulo.get(modulo) ?? { textos: new Set(), citas: [] };
+        if (!entrada.textos.has(texto)) {
+            entrada.textos.add(texto);
+            entrada.citas.push({
+                fecha: h.created_at.slice(5, 10).split('-').reverse().join('/'),
+                commit: h.changelog_id
+                    ? (commitPorAnalisis.get(h.changelog_id) ?? null)
+                    : null,
+            });
+        }
+        porModulo.set(modulo, entrada);
+    }
+    return [...porModulo.entries()]
+        .sort((a, b) => b[1].textos.size - a[1].textos.size || a[0].localeCompare(b[0]))
+        .slice(0, tope)
+        .map(([modulo, { textos, citas }]) => {
+        const n = textos.size;
+        // Tres citas como mucho: la linea es una pista para ir a mirar, no un
+        // informe. Con mas, la seccion se come el presupuesto del bloque.
+        const cuando = citas
+            .slice(0, 3)
+            .map((c) => (c.commit ? `${c.fecha} \`${c.commit}\`` : c.fecha))
+            .join(', ');
+        return `- **${modulo}**: ${n} regresi${n === 1 ? 'ón' : 'ones'}${cuando ? ` (${cuando})` : ''}`;
+    });
+}
+export function buildSection(rows, changes, alerts = [], projectName, pendingTasks = [], atRiskHealth = [], historial = [], commitPorAnalisis = new Map()) {
     // Newest-first rows: the first occurrence of a module is its latest state.
     const seen = new Map();
     for (const row of rows) {
@@ -210,7 +296,7 @@ export function buildSection(rows, changes, alerts = [], projectName, pendingTas
         // disparador abstracto: hay que nombrar sus verbos.
         'Disparador → tool: orientarte → `atlas_project_brief` (`intent:"orient"`) · antes de editar/cambiar/arreglar/refactorizar un archivo o función → `atlas_file_context` (dice QUIÉN DEPENDE de eso) · riesgos/acoplamientos → el brief. Tras cada commit: `atlas_record_change` (diff, hash, fecha, `summary` tuyo; reenviar es gratis).',
         '',
-        'Tools diferidas, en UNA llamada: ToolSearch `select:mcp__changebook__atlas_project_brief,mcp__changebook__atlas_file_context,mcp__changebook__atlas_symbol_lookup`.',
+        'Tools diferidas, en UNA llamada: ToolSearch `select:mcp__changebook__atlas_project_brief,mcp__changebook__atlas_file_context`.',
         '',
         // Proyecto + conducta en UNA línea: cada char de cabecera expulsa mapa.
         //
@@ -334,7 +420,18 @@ export function buildSection(rows, changes, alerts = [], projectName, pendingTas
             title: '### Encargos pendientes del dueño (proponte atacarlos)',
             lines: taskLines,
         },
-        { key: 'modules', priority: 3, title: '### Módulos', lines: moduleLines },
+        {
+            key: 'costoso',
+            priority: 1,
+            title: '### Lo que ya costó caro aquí (revisa antes de tocarlo)',
+            lines: hechosCaros(historial, commitPorAnalisis),
+        },
+        // El mapa baja de 3 a 4, y no es una degradación caprichosa: `/doctor` de
+        // Claude Code recorta por su cuenta «architecture overviews» de un
+        // CLAUDE.md porque el agente los deriva del repo. Lo que no puede derivar
+        // es qué ha roto antes. Se conserva la sección —quitarla del todo cambia
+        // más de lo necesario— pero deja de competir con los hechos.
+        { key: 'modules', priority: 4, title: '### Módulos', lines: moduleLines },
         {
             key: 'changes',
             priority: 5,
@@ -359,8 +456,17 @@ export function buildSection(rows, changes, alerts = [], projectName, pendingTas
     // reserva lo que cuestan las primeras MIN_LINEAS_MAPA líneas del mapa, y solo
     // el resto se disputa por prioridad. Si el mapa tiene menos líneas que el
     // suelo, sobra menos reserva; si no hay mapa, no se reserva nada.
-    const MIN_LINEAS_MAPA = 6;
-    const mapa = sections.find((x) => x.key === 'modules');
+    // EL SUELO CAMBIA DE DUEÑO. Lo tenía el mapa porque el estudio de 76 agentes
+    // del 2026-07-20 concluyó que un mapa expulsado era «la causa nº 1 de que el
+    // agente ignore o desconfíe del atlas». Sigue siendo cierto, pero el mapa ya
+    // no es lo irreemplazable: el agente lo deriva leyendo el repo, y la propia
+    // herramienta lo recorta. Lo que nadie puede derivar es qué rompió antes, así
+    // que el suelo protege ahora eso.
+    //
+    // Y baja de 6 líneas a 3: son entradas mucho más densas —un módulo, un
+    // número y hasta tres commits— así que tres ya dicen dónde pisar con cuidado.
+    const MIN_LINEAS_MAPA = 3;
+    const mapa = sections.find((x) => x.key === 'costoso');
     let reservaMapa = 0;
     if (mapa && mapa.lines.length > 0) {
         reservaMapa = mapa.title.length + 2;
@@ -380,7 +486,7 @@ export function buildSection(rows, changes, alerts = [], projectName, pendingTas
         if (s.lines.length === 0)
             continue;
         // El mapa gasta su reserva ADEMÁS de lo que haya quedado libre.
-        const disponible = s.key === 'modules' ? budget + reservaMapa : budget;
+        const disponible = s.key === 'costoso' ? budget + reservaMapa : budget;
         let cost = s.title.length + 2; // título + línea en blanco separadora
         let count = 0;
         for (const line of s.lines) {
@@ -391,8 +497,14 @@ export function buildSection(rows, changes, alerts = [], projectName, pendingTas
         }
         if (count > 0) {
             includedCount.set(s.key, count);
-            if (s.key === 'modules') {
+            if (s.key === 'costoso') {
                 // Lo que consumió por encima de su reserva sale del fondo común.
+                //
+                // La clave tiene que ser la MISMA que la de arriba: al mover el suelo
+                // del mapa a los hechos se me quedó aquí `modules`, así que la sección
+                // nueva gastaba la reserva Y descontaba su coste entero del fondo —
+                // doble gasto— y el mapa se llevaba luego una reserva ya usada. El
+                // bloque no reventaba el techo por casualidad, no por diseño.
                 budget -= Math.max(0, cost - reservaMapa);
                 reservaMapa = 0;
             }

package/dist/tools.js

@@ -281,76 +281,6 @@ export async function derivaContraHead(dir, hash) {
 }
 // ── Registration ──────────────────────────────────────────────────────────────
 export function registerTools(server, db) {
-    // Espejo del hospedado (T3 del benchmark 2026-07-20): definiciones
-    // exportadas por archivo, sin grep. Usos y tests siguen fuera y se dice.
-    server.registerTool("atlas_symbol_lookup", {
-        title: "Where is this symbol defined?",
-        description: `Files where an exported symbol (function/class/const/interface/type/enum) is DEFINED, with its kind and the commit that last touched it. Mechanical index built from every ingested diff.
-
-Definitions only — usages and tests are not indexed; grep for those. Exact match first, substring fallback.
-
-Args:
-  - symbol (required): the identifier to look up.
-  - project (recommended): the repo you are working in (folder name or slug).
-
-Returns (structured): { symbol, exact_match, matches: [{ symbol, file, kind, commit }] }`,
-        inputSchema: {
-            symbol: z
-                .string()
-                .min(2)
-                .max(120)
-                .describe("Identifier to look up (exported definition)"),
-            project: z
-                .string()
-                .min(1)
-                .max(120)
-                .describe("Project to scope to (repo folder name or slug)"),
-        },
-        annotations: {
-            readOnlyHint: true,
-            destructiveHint: false,
-            idempotentHint: true,
-            openWorldHint: true,
-        },
-    }, async ({ symbol, project }) => {
-        try {
-            const t0 = Date.now();
-            const pf = await db.projectFilterFor(project);
-            const base = `symbol_index?select=symbol,file,kind,commit_hash&order=symbol.asc&limit=20` +
-                pf;
-            let matches = await db.rest(`${base}&symbol=eq.${encodeURIComponent(symbol)}`);
-            let exact = true;
-            if (matches.length === 0) {
-                exact = false;
-                matches = await db.rest(`${base}&symbol=ilike.${ilikePattern(symbol)}`);
-            }
-            const lines = [`# Symbol lookup: ${symbol}`, ""];
-            if (matches.length === 0) {
-                lines.push("Not in the index. It may be unexported, renamed, or defined before the index existed — fall back to grep and SAY you did, instead of guessing.");
-            }
-            for (const m of matches) {
-                lines.push(`- ${m.symbol} (${m.kind}) — ${m.file}${m.commit_hash ? ` · commit ${m.commit_hash.slice(0, 7)}` : ""}`);
-            }
-            if (matches.length > 0) {
-                lines.push("", "Definitions only — usages and tests are not indexed; grep for those.");
-            }
-            const salida = toolResult(lines.join("\n"), {
-                symbol,
-                exact_match: exact,
-                matches: matches.map((m) => ({
-                    symbol: m.symbol,
-                    file: m.file,
-                    kind: m.kind,
-                    commit: m.commit_hash?.slice(0, 7) ?? null,
-                })),
-            });
-            recordRead(db, "atlas_symbol_lookup", pf, servedCharsOf(salida), Date.now() - t0);
-            return salida;
-        }
-        catch (error) {
-            return errorResult(error);
-        }
-    });
     server.registerTool("atlas_recent_changes", {
         title: "Recent ChangeBook changes",
         description: `List the most recent analyzed code changes from the ChangeBook changelog (newest first).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "changebook",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "mcpName": "io.github.raulbr90/changebook",
   "description": "ChangeBook for coding agents: MCP server (product memory for Claude Code/Codex) + CLI to sign in, analyze changes and sync the product map.",
   "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": "Query your product's living memory: module map + analyzed change history. Read-only MCP tools.",
-  "version": "0.5.0",
+  "version": "0.6.0",
   "websiteUrl": "https://changebook.dev",
   "remotes": [
     {
@@ -15,7 +15,7 @@
       "registryType": "npm",
       "registryBaseUrl": "https://registry.npmjs.org",
       "identifier": "changebook",
-      "version": "0.5.0",
+      "version": "0.6.0",
       "transport": {
         "type": "stdio"
       }