@fullqueso/mcp-bc-gastos 1.23.0 → 1.23.1

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # CHANGELOG
 
+## [1.23.1] — 2026-06-05
+
+### Fixed
+- **`get_cash_flow` — CapEx y Working Capital alineados al Chart of Accounts real de BC.** El CapEx sumaba el rango `15000–17999`, que en el CoA real son **activos corrientes** (cuentas por cobrar 151xx, impuestos corrientes 152xx, prepagados 16xxx, inversiones corto plazo 17xxx), **no** activos fijos. Esto inflaba CapEx y distorsionaba el FCF.
+  - **CapEx ahora = net change en PP&E `11110–12899`** (intangibles 11110–11190 + tangibles 12110–12240), **excluye `12900`** (depreciación acumulada), `17000–17999` (inversiones corto plazo) y `18000–18999` (caja/bancos).
+  - **Nueva línea WC `Δ Impuestos corrientes`** (`15210–15280`: IVA crédito fiscal, retenciones IVA/ISLR, anticipos/excedente ISLR) = `−Σ(debit−credit)`.
+  - **Nueva línea WC `Δ Prepagados`** (`16110–16900`: anticipos a proveedores, alquileres/otros prepagados) = `−Σ(debit−credit)`.
+  - `net_wc_change = ΔAR + ΔAP + ΔInv + ΔImpuestos + ΔPrepagados`. ΔAR (subledger, equivale a 151xx) y ΔInventario (`itemLedgerEntries`, equivale a 14xxx) sin cambios — no se duplican vía G/L.
+  - `fcf_bridge` agrega filas `Δ Impuestos` y `Δ Prepagados` (11 ítems). Insight de "CapEx detectado" solo aparece si hay net change real en `11110–12899`; el insight de mayor uso de caja ahora reporta cualquier driver dominante (no solo inventario).
+  - HTML: tarjeta de Capital de Trabajo muestra las dos líneas nuevas; KPI CapEx etiquetado `PP&E 11110-12899`.
+  - **Verificado contra BC real (FQ88, mayo 2026):** CapEx ANTES `$8,760.92` (cuentas 151xx/152xx) → DESPUÉS `$0` (sin movimientos PP&E). Bridge cuadra: NOPAT `$12,340.71` + ΔWC `$10,794.99` − CapEx `$0` = FCF `$23,135.70`. Cuentas 15110/15135/15210 ya no aparecen en CapEx.
+
+---
+
 ## [1.23.0] — 2026-05-31
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fullqueso/mcp-bc-gastos",
-  "version": "1.23.0",
+  "version": "1.23.1",
   "description": "MCP server for Business Central operational expense analysis, bank reconciliation, POS reconciliation, accounts receivable/payable, multi-payment draft visibility, payroll, inventory cost analysis, and manager reports - Full Queso franchise stores",
   "main": "server.js",
   "bin": {

package/tools/financials/cash-flow-html.js

@@ -210,7 +210,7 @@ function renderKpis(s) {
   ${cell('EBIT', signed(s.ebit.amount), `${s.ebit.pct}%`)}
   ${cell('NOPAT', signed(s.nopat.amount), `tax ${signed(-s.tax_estimated.amount)}`)}
   ${cell('CFO', signed(s.cfo.amount), `${s.cfo.pct}%`)}
-  ${cell('CapEx', signed(-s.capex.amount), 'activos fijos')}
+  ${cell('CapEx', signed(-s.capex.amount), 'PP&E 11110-12899')}
   ${cell('FCF', signed(s.fcf.amount), statusWord(s.fcf.status), fcfCls)}
   ${cell('FCF Margin', `${s.fcf.pct}%`, '% ingresos', fcfCls)}
 </div>`;
@@ -238,6 +238,8 @@ function renderWorkingCapital(s) {
       ${row('Δ Cobranzas (AR)', wc.delta_ar)}
       ${row('Δ Proveedores (AP)', wc.delta_ap)}
       ${row('Δ Inventario', wc.delta_inventory)}
+      ${row('Δ Impuestos corrientes', wc.delta_taxes || 0)}
+      ${row('Δ Prepagados', wc.delta_prepaid || 0)}
       <div class="wc-row total">
         <span class="wc-label">Cambio neto WC</span>
         <span class="wc-arrow ${wc.net_wc_change >= 0 ? 'up' : 'down'}">${wc.net_wc_change >= 0 ? '&#x2191;' : '&#x2193;'}</span>

package/tools/financials/cash-flow.js

@@ -6,13 +6,19 @@
 // CFO, FCF, bridge tipo waterfall e insights. READ-ONLY.
 //
 // CONVENCIÓN DE SIGNOS (impacto en caja; + = fuente de caja, − = uso de caja):
-//   delta_ar        = −Σ amountLocalCurrency(customerLedgerEntries)  → + = cobró más de lo facturado
+//   delta_ar        = −Σ amountLocalCurrency(customerLedgerEntries)  → + = cobró más de lo facturado (AR 15110–15199)
 //   delta_ap        = −Σ amountLocalCurrency(vendorLedgerEntries)    → + = debe más (fuente)
-//   delta_inventory = −Σ costAmountActual(itemLedgerEntries)         → + = inventario bajó (fuente)
-//   net_wc_change   = delta_ar + delta_ap + delta_inventory
+//   delta_inventory = −Σ costAmountActual(itemLedgerEntries)         → + = inventario bajó (fuente) (14000–14999)
+//   delta_taxes     = −Σ(debit−credit) G/L 15210–15280              → + = impuestos corrientes bajaron (fuente)
+//   delta_prepaid   = −Σ(debit−credit) G/L 16110–16900              → + = prepagados bajaron (fuente)
+//   net_wc_change   = delta_ar + delta_ap + delta_inventory + delta_taxes + delta_prepaid
 //   nopat = ebit − tax ;  cfo = nopat + net_wc_change ;  fcf = cfo − capex
 // (Resuelve la inconsistencia del spec entre los comentarios del schema y la fórmula:
-//  se adopta la convención de los comentarios — los 3 deltas ya con signo de caja.)
+//  se adopta la convención de los comentarios — los deltas ya con signo de caja.)
+//
+// CAPEX = net change (debit−credit) en PP&E 11110–12899 (intangibles 11110–11190 +
+//   tangibles 12110–12240). EXCLUYE 12900 (depreciación acumulada), 17000–17999
+//   (inversiones corto plazo) y 18000–18999 (caja y bancos = posición de caja final).
 //
 // Montos en USD (LCY = Additional Reporting Currency), igual que get_financial_statements.
 
@@ -38,8 +44,9 @@ export const cashFlowTool = {
   description:
     'Free Cash Flow (FCF) por método indirecto para una o más tiendas Full Queso (FQ01, FQ28, FQ88). ' +
     'Parte del EBIT (igual que get_financial_statements), resta impuesto estimado (ISLR 34%), ajusta por capital de trabajo ' +
-    '(Δ cuentas por cobrar, Δ cuentas por pagar, Δ inventario) para obtener el flujo de caja operativo (CFO), y resta CapEx ' +
-    '(activos fijos 15000-17999) para el FCF. Incluye NOPAT, margen FCF, status (healthy/positive_low/negative_watch/' +
+    '(Δ cuentas por cobrar, Δ cuentas por pagar, Δ inventario, Δ impuestos corrientes, Δ prepagados) para obtener el flujo de ' +
+    'caja operativo (CFO), y resta CapEx (PP&E / activos fijos 11110-12899) para el FCF. Incluye NOPAT, margen FCF, status ' +
+    '(healthy/positive_low/negative_watch/' +
     'negative_critical), bridge tipo waterfall e insights automáticos. Multi-tienda calcula consolidado. Con render_html=true ' +
     'genera un dashboard HTML (lo guarda en ~/Downloads y retorna la ruta). Úsalo cuando el usuario pida flujo de caja, ' +
     'cash flow, FCF, liquidez, "cuánta caja genera", o capital de trabajo. READ-ONLY.',
@@ -205,6 +212,8 @@ async function buildStoreCashFlow(bcClient, store, start, end) {
       delta_ar: wc.delta_ar,
       delta_ap: wc.delta_ap,
       delta_inventory: wc.delta_inventory,
+      delta_taxes: wc.delta_taxes,
+      delta_prepaid: wc.delta_prepaid,
       capex: wc.capex,
       capex_accounts: wc.capex_accounts,
       warnings: wc.warnings,
@@ -231,11 +240,20 @@ async function fetchWorkingCapitalAndCapex(bcClient, companyId, start, end) {
     $select: 'postingDate,entryType,quantity,costAmountActual',
   });
 
-  const [arEntries, apEntries, ileEntries, capexEntries] = await Promise.all([
+  // Rangos G/L del Chart of Accounts real de BC (activos 10000–19999):
+  //   CapEx → PP&E 11110–12899 (excluye 12900 depreciación acumulada)
+  //   Impuestos corrientes → 15210–15280 (IVA crédito, retenciones, anticipos ISLR)
+  //   Prepagados → 16110–16900 (anticipos a proveedores, alquileres/otros prepagados)
+  // AR (15110–15199) e Inventario (14000–14999) ya se calculan vía subledger
+  // (customer/item ledger) — NO consultarlos aquí para no duplicar.
+  // Excluidos del FCF: 17000–17999 (inversiones corto plazo) y 18000–18999 (caja/bancos).
+  const [arEntries, apEntries, ileEntries, capexEntries, taxEntries, prepaidEntries] = await Promise.all([
     bcClient.apiCallAllPages(arUrl).catch((e) => { logger.warn(`[cash-flow] AR fetch failed: ${e.message}`); return null; }),
     bcClient.apiCallAllPages(apUrl).catch((e) => { logger.warn(`[cash-flow] AP fetch failed: ${e.message}`); return null; }),
     bcClient.apiCallAllPages(ileUrl).catch((e) => { logger.warn(`[cash-flow] ILE fetch failed: ${e.message}`); return null; }),
-    bcClient.getGLEntries(companyId, start, end, '15000', '17999').catch((e) => { logger.warn(`[cash-flow] CapEx fetch failed: ${e.message}`); return null; }),
+    bcClient.getGLEntries(companyId, start, end, '11110', '12899').catch((e) => { logger.warn(`[cash-flow] CapEx fetch failed: ${e.message}`); return null; }),
+    bcClient.getGLEntries(companyId, start, end, '15210', '15280').catch((e) => { logger.warn(`[cash-flow] Impuestos fetch failed: ${e.message}`); return null; }),
+    bcClient.getGLEntries(companyId, start, end, '16110', '16900').catch((e) => { logger.warn(`[cash-flow] Prepagados fetch failed: ${e.message}`); return null; }),
   ]);
 
   const warnings = [];
@@ -259,11 +277,23 @@ async function fetchWorkingCapitalAndCapex(bcClient, companyId, start, end) {
     if (ileEntries._truncated) warnings.push('Item ledger truncado (muchas entradas) — Δinventario puede estar incompleto.');
   }
 
-  // CapEx — débitos netos en activos fijos 15000-17999 (excluye 18xxx bancos)
+  // ΔImpuestos corrientes — impacto en caja = −Σ(debit−credit) G/L 15210–15280
+  // (activo corriente: aumenta = uso de caja, baja = fuente de caja)
+  let delta_taxes = 0;
+  if (taxEntries == null) warnings.push('G/L de impuestos corrientes (15210–15280) no disponible — ΔImpuestos asumido en 0.');
+  else delta_taxes = round2(-taxEntries.reduce((s, e) => s + ((e.debitAmount || 0) - (e.creditAmount || 0)), 0));
+
+  // ΔPrepagados — impacto en caja = −Σ(debit−credit) G/L 16110–16900
+  let delta_prepaid = 0;
+  if (prepaidEntries == null) warnings.push('G/L de prepagados (16110–16900) no disponible — ΔPrepagados asumido en 0.');
+  else delta_prepaid = round2(-prepaidEntries.reduce((s, e) => s + ((e.debitAmount || 0) - (e.creditAmount || 0)), 0));
+
+  // CapEx — net change (debit−credit) en PP&E 11110–12899 (excluye 12900 depreciación,
+  // 17xxx inversiones corto plazo y 18xxx bancos)
   let capex = 0;
   const capex_accounts = {};
   if (capexEntries == null) {
-    warnings.push('G/L de activos fijos no disponible — CapEx asumido en 0.');
+    warnings.push('G/L de PP&E (activos fijos 11110–12899) no disponible — CapEx asumido en 0.');
   } else {
     for (const e of capexEntries) {
       const net = (e.debitAmount || 0) - (e.creditAmount || 0);
@@ -275,7 +305,7 @@ async function fetchWorkingCapitalAndCapex(bcClient, companyId, start, end) {
     for (const k of Object.keys(capex_accounts)) if (capex_accounts[k] === 0) delete capex_accounts[k];
   }
 
-  return { delta_ar, delta_ap, delta_inventory, capex, capex_accounts, warnings };
+  return { delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, capex, capex_accounts, warnings };
 }
 
 // ─────────────────────────────────────────────────────────────────────────────
@@ -294,7 +324,9 @@ function assembleCashFlow(meta, c) {
   const delta_ar = round2(c.delta_ar);
   const delta_ap = round2(c.delta_ap);
   const delta_inventory = round2(c.delta_inventory);
-  const net_wc_change = round2(delta_ar + delta_ap + delta_inventory);
+  const delta_taxes = round2(c.delta_taxes || 0);
+  const delta_prepaid = round2(c.delta_prepaid || 0);
+  const net_wc_change = round2(delta_ar + delta_ap + delta_inventory + delta_taxes + delta_prepaid);
 
   const cfo = round2(nopat + net_wc_change);
   const cfoPct = income > 0 ? round2((cfo / income) * 100) : 0;
@@ -304,8 +336,8 @@ function assembleCashFlow(meta, c) {
   const fcfPct = income > 0 ? round2((fcf / income) * 100) : 0;
 
   const status = fcfStatus(fcf, fcfPct, ebit);
-  const bridge = buildBridge({ ebit, tax, nopat, delta_ar, delta_ap, delta_inventory, cfo, capex, fcf });
-  const insights = buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, capex, fcf, fcfPct, status, warnings: c.warnings || [] });
+  const bridge = buildBridge({ ebit, tax, nopat, delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, cfo, capex, fcf });
+  const insights = buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, capex, fcf, fcfPct, status, warnings: c.warnings || [] });
 
   return {
     store_code: meta.store_code,
@@ -316,7 +348,7 @@ function assembleCashFlow(meta, c) {
     ebit: { amount: ebit, pct: ebitPct },
     tax_estimated: { amount: tax, rate: TAX_RATE, note: 'Estimado ISLR 34%' },
     nopat: { amount: nopat },
-    working_capital: { delta_ar, delta_ap, delta_inventory, net_wc_change },
+    working_capital: { delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, net_wc_change },
     cfo: { amount: cfo, pct: cfoPct },
     capex: { amount: capex, accounts: c.capex_accounts || {} },
     fcf: { amount: fcf, pct: fcfPct, status },
@@ -331,7 +363,7 @@ function fcfStatus(fcf, fcfPct, ebit) {
   return fcf > ebit * -0.2 ? 'negative_watch' : 'negative_critical';
 }
 
-function buildBridge({ ebit, tax, nopat, delta_ar, delta_ap, delta_inventory, cfo, capex, fcf }) {
+function buildBridge({ ebit, tax, nopat, delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, cfo, capex, fcf }) {
   const flow = (label, amount) => ({ label, amount: round2(amount), type: amount >= 0 ? 'source' : 'use' });
   return [
     { label: 'EBIT', amount: ebit, type: 'source' },
@@ -340,13 +372,15 @@ function buildBridge({ ebit, tax, nopat, delta_ar, delta_ap, delta_inventory, cf
     flow('Δ Cob. (AR)', delta_ar),
     flow('Δ Prov. (AP)', delta_ap),
     flow('Δ Inventario', delta_inventory),
+    flow('Δ Impuestos', delta_taxes),
+    flow('Δ Prepagados', delta_prepaid),
     { label: 'CFO', amount: cfo, type: 'subtotal' },
     { label: 'CapEx', amount: round2(-capex), type: 'use' },
     { label: 'FCF', amount: fcf, type: 'total' },
   ];
 }
 
-function buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, capex, fcf, fcfPct, status, warnings }) {
+function buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, delta_taxes, delta_prepaid, capex, fcf, fcfPct, status, warnings }) {
   const insights = [];
 
   // Mayor driver negativo entre los usos de capital de trabajo
@@ -354,9 +388,11 @@ function buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, capex, fcf,
     { key: 'delta_inventory', label: 'Inventario', val: delta_inventory },
     { key: 'delta_ar', label: 'Cobranza (AR)', val: delta_ar },
     { key: 'delta_ap', label: 'Pagos a proveedores (AP)', val: delta_ap },
+    { key: 'delta_taxes', label: 'Impuestos corrientes', val: delta_taxes },
+    { key: 'delta_prepaid', label: 'Prepagados', val: delta_prepaid },
   ].filter((u) => u.val < 0).sort((a, b) => a.val - b.val);
-  if (uses.length && uses[0].key === 'delta_inventory') {
-    insights.push(`Inventario es el mayor uso de caja del período ($${fmtAbs(delta_inventory)}).`);
+  if (uses.length) {
+    insights.push(`${uses[0].label} es el mayor uso de caja del período ($${fmtAbs(uses[0].val)}).`);
   }
 
   // AR consume >20% del EBIT (cobranza lenta)
@@ -364,9 +400,11 @@ function buildInsights({ ebit, delta_ar, delta_ap, delta_inventory, capex, fcf,
     insights.push(`Cobranza lenta presiona caja: ΔAR −$${fmtAbs(delta_ar)} (>20% del EBIT).`);
   }
 
-  // CapEx detectado
+  // CapEx — solo si hay net change real en PP&E (11110–12899)
   if (capex > 0) {
-    insights.push(`CapEx $${fmtAbs(capex)} en activos fijos detectado.`);
+    insights.push(`CapEx $${fmtAbs(capex)} en PP&E (activos fijos 11110–12899) detectado.`);
+  } else if (capex < 0) {
+    insights.push(`Desinversión neta en PP&E $${fmtAbs(capex)} (fuente de caja).`);
   }
 
   // Margen FCF bajo
@@ -415,6 +453,8 @@ function consolidateCashFlow(storeObjs) {
       delta_ar: sum((o) => o.working_capital.delta_ar),
       delta_ap: sum((o) => o.working_capital.delta_ap),
       delta_inventory: sum((o) => o.working_capital.delta_inventory),
+      delta_taxes: sum((o) => o.working_capital.delta_taxes || 0),
+      delta_prepaid: sum((o) => o.working_capital.delta_prepaid || 0),
       capex: sum((o) => o.capex.amount),
       capex_accounts,
       warnings: allWarnings,