@fullqueso/mcp-bc-gastos 1.28.0 → 1.33.0

package/tools/payroll/payroll-lines.js

@@ -1,10 +1,11 @@
 import { resolveStores } from '../../config/company-config.js';
 import { logger } from '../../utils/logger.js';
+import { round2 } from './cost-model.js';
 
 export const payrollLinesTool = {
   name: 'get_payroll_lines',
   description:
-    'Detalle de nomina por empleado para un documento especifico. Muestra bonos, pagos por metodo (cash VES, banco VES, cash USD), totales, y balance. Requiere document_no.',
+    'Detalle de nomina por empleado para un documento. Muestra bonos, pagos por metodo (cash VES, banco VES, cash USD), y el costo por empleado en USD (cost_usd, derivado de totalVESEquivalent a la tasa del documento) → base para "costo por persona". Requiere document_no. Nota: la API de nomina NO expone dias/horas trabajados ni FTE por linea.',
   inputSchema: {
     type: 'object',
     properties: {
@@ -42,39 +43,57 @@ export async function handlePayrollLines(bcClient, args) {
   const filters = [`documentNo eq '${document_no}'`];
   if (args.employee_code) filters.push(`employeeCode eq '${args.employee_code}'`);
 
-  const url = bcClient.buildPayrollApiUrl(companyId, 'fqPayrollLines', {
+  // Header holds the exchangeRate (lines don't) → needed for per-employee cost_usd.
+  const headerUrl = bcClient.buildPayrollApiUrl(companyId, 'fqPayrollDocuments', {
+    $filter: `documentNo eq '${document_no}'`,
+    $select: 'documentNo,payrollType,status,documentDate,exchangeRate',
+  });
+
+  const linesUrl = bcClient.buildPayrollApiUrl(companyId, 'fqPayrollLines', {
     $filter: filters.join(' and '),
-    $select: 'documentNo,lineNo,employeeCode,employeeName,cedulaDeIdentidad,transportBonusVES,productivityBonusUSD,extraRedobleUSD,specialDateBonusUSD,foodBonusUSD,payCashVES,payBankVES,payCashUSD,totalBonusesVES,totalBonusesUSD,totalPayVES,totalPayUSD,balanceDifference',
+    $select:
+      'documentNo,lineNo,employeeCode,employeeName,cedulaDeIdentidad,transportBonusVES,productivityBonusUSD,extraRedobleUSD,specialDateBonusUSD,foodBonusUSD,payCashVES,payBankVES,payCashUSD,totalBonusesVES,totalBonusesUSD,totalVESEquivalent,totalPayVES,totalPayUSD,balanceDifference',
     $orderby: 'lineNo',
   });
 
-  const data = await bcClient.apiCallAllPages(url);
-  logger.info(`${storeInfo.code}: ${data.length} payroll lines for ${document_no}`);
+  const [headerRows, data] = await Promise.all([
+    bcClient.apiCallAllPages(headerUrl),
+    bcClient.apiCallAllPages(linesUrl),
+  ]);
+  const header = headerRows[0] || null;
+  const rate = header?.exchangeRate || 0;
+  logger.info(`${storeInfo.code}: ${data.length} payroll lines for ${document_no} (rate=${rate || 'n/a'})`);
 
-  let lines = data.map((line) => ({
-    document_no: line.documentNo,
-    line_no: line.lineNo,
-    employee_code: line.employeeCode,
-    employee_name: line.employeeName,
-    cedula: line.cedulaDeIdentidad || null,
-    bonuses: {
-      transport_ves: round2(line.transportBonusVES || 0),
-      productivity_usd: round2(line.productivityBonusUSD || 0),
-      extra_redoble_usd: round2(line.extraRedobleUSD || 0),
-      special_date_usd: round2(line.specialDateBonusUSD || 0),
-      food_usd: round2(line.foodBonusUSD || 0),
-    },
-    payments: {
-      cash_ves: round2(line.payCashVES || 0),
-      bank_ves: round2(line.payBankVES || 0),
-      cash_usd: round2(line.payCashUSD || 0),
-    },
-    total_bonuses_ves: round2(line.totalBonusesVES || 0),
-    total_bonuses_usd: round2(line.totalBonusesUSD || 0),
-    total_pay_ves: round2(line.totalPayVES || 0),
-    total_pay_usd: round2(line.totalPayUSD || 0),
-    balance_difference: round2(line.balanceDifference || 0),
-  }));
+  let lines = data.map((line) => {
+    const vesEquivalent = round2(line.totalVESEquivalent || 0);
+    return {
+      document_no: line.documentNo,
+      line_no: line.lineNo,
+      employee_code: line.employeeCode,
+      employee_name: line.employeeName,
+      cedula: line.cedulaDeIdentidad || null,
+      bonuses: {
+        transport_ves: round2(line.transportBonusVES || 0),
+        productivity_usd: round2(line.productivityBonusUSD || 0),
+        extra_redoble_usd: round2(line.extraRedobleUSD || 0),
+        special_date_usd: round2(line.specialDateBonusUSD || 0),
+        food_usd: round2(line.foodBonusUSD || 0),
+      },
+      payments: {
+        cash_ves: round2(line.payCashVES || 0),
+        bank_ves: round2(line.payBankVES || 0),
+        cash_usd: round2(line.payCashUSD || 0),
+      },
+      total_bonuses_ves: round2(line.totalBonusesVES || 0),
+      total_bonuses_usd: round2(line.totalBonusesUSD || 0),
+      total_ves_equivalent: vesEquivalent,
+      // Per-employee cost in USD = full pay (VES-equivalent) at the document rate.
+      cost_usd: rate ? round2(vesEquivalent / rate) : null,
+      total_pay_ves: round2(line.totalPayVES || 0),
+      total_pay_usd: round2(line.totalPayUSD || 0),
+      balance_difference: round2(line.balanceDifference || 0),
+    };
+  });
 
   // Client-side name search
   if (args.employee_search) {
@@ -85,7 +104,12 @@ export async function handlePayrollLines(bcClient, args) {
   // Summary totals
   const summary = {
     document_no,
+    payroll_type: header?.payrollType || null,
+    status: header?.status || null,
+    exchange_rate: rate || null,
     total_employees: lines.length,
+    total_cost_usd: 0,
+    total_ves_equivalent: 0,
     total_bonuses_ves: 0,
     total_bonuses_usd: 0,
     total_pay_ves: 0,
@@ -97,6 +121,8 @@ export async function handlePayrollLines(bcClient, args) {
   };
 
   for (const line of lines) {
+    summary.total_ves_equivalent += line.total_ves_equivalent;
+    if (line.cost_usd != null) summary.total_cost_usd += line.cost_usd;
     summary.total_bonuses_ves += line.total_bonuses_ves;
     summary.total_bonuses_usd += line.total_bonuses_usd;
     summary.total_pay_ves += line.total_pay_ves;
@@ -107,6 +133,8 @@ export async function handlePayrollLines(bcClient, args) {
     if (Math.abs(line.balance_difference) > 0.01) summary.employees_with_balance_diff++;
   }
 
+  summary.total_cost_usd = rate ? round2(summary.total_cost_usd) : null;
+  summary.total_ves_equivalent = round2(summary.total_ves_equivalent);
   summary.total_bonuses_ves = round2(summary.total_bonuses_ves);
   summary.total_bonuses_usd = round2(summary.total_bonuses_usd);
   summary.total_pay_ves = round2(summary.total_pay_ves);
@@ -122,7 +150,3 @@ export async function handlePayrollLines(bcClient, args) {
     lines,
   };
 }
-
-function round2(n) {
-  return Math.round(n * 100) / 100;
-}

package/tools/ventas/sales-analysis.js

@@ -99,7 +99,7 @@ export async function handleSalesAnalysis(bcClient, args) {
       data.lines.map((l) => ({ ...l, storeCode: code, storeName: data.storeName }))
     );
 
-    const storeSummary = calculateSummary(data.invoices, data.lines, exchangeRate, data.glRevenue);
+    const storeSummary = calculateSummary(data.invoices, data.lines, exchangeRate, data.glRevenue, code);
     byStoreResults.push({
       store_code: code,
       store_name: data.storeName,

package/tools/ventas/sales-store-comparison.js

@@ -59,7 +59,7 @@ export async function handleSalesStoreComparison(bcClient, args) {
   const storesSummaries = {};
 
   for (const [code, data] of Object.entries(allStoresData)) {
-    const summary = calculateSummary(data.invoices, data.lines, exchangeRate, data.glRevenue);
+    const summary = calculateSummary(data.invoices, data.lines, exchangeRate, data.glRevenue, code);
     storesSummaries[data.storeName] = { summary };
 
     const storeResult = {
@@ -69,6 +69,10 @@ export async function handleSalesStoreComparison(bcClient, args) {
 
     if (showAll || requestedMetrics.includes('revenue')) {
       storeResult.revenue_usd = summary.total_revenue_usd;
+      storeResult.revenue_store = summary.revenue_store;               // neto de interco, eventos y no-venta
+      storeResult.revenue_events = summary.revenue_events;
+      storeResult.revenue_intercompany = summary.revenue_intercompany;
+      storeResult.revenue_non_sales = summary.revenue_non_sales;
       storeResult.revenue_breakdown = summary.revenue_breakdown;
       storeResult.margin_usd = summary.total_margin_usd;
       storeResult.margin_pct = summary.margin_pct;

package/utils/sales-aggregation.js

@@ -1,3 +1,5 @@
+import { getRevenueClassification } from '../config/income-accounts.js';
+
 export function aggregateByField(lines, field) {
   const map = {};
   for (const line of lines) {
@@ -43,7 +45,7 @@ export function aggregateByDate(lines) {
     .sort((a, b) => a.date.localeCompare(b.date));
 }
 
-export function calculateSummary(invoices, lines, exchangeRate = null, glRevenue = null) {
+export function calculateSummary(invoices, lines, exchangeRate = null, glRevenue = null, storeCode = null) {
   const totalTransactions = invoices.length;
   const totalQuantity = lines.reduce((sum, l) => sum + (l.quantity || 0), 0);
   const totalCostUSD = lines.reduce((sum, l) => sum + (l.unitCostUSD || 0) * (l.quantity || 0), 0);
@@ -55,8 +57,18 @@ export function calculateSummary(invoices, lines, exchangeRate = null, glRevenue
   const marginPct = totalRevenueUSD && totalRevenueUSD !== 0
     ? (totalMarginUSD / totalRevenueUSD) * 100 : 0;
 
+  // Split revenue into the buckets that matter for the store-payroll KPI: store
+  // (own operation, the denominator) vs events (own ratio) vs intercompany (not a
+  // store sale) vs non-sales income (FX gains, purchase rebates). Store-aware.
+  const buckets = splitRevenueBuckets(glRevenue, totalRevenueUSD, storeCode);
+
   return {
     total_revenue_usd: totalRevenueUSD !== null ? Math.round(totalRevenueUSD * 100) / 100 : null,
+    revenue_total: buckets.revenue_total,               // alias of total_revenue_usd (explicit)
+    revenue_store: buckets.revenue_store,               // total − intercompany − events − non_sales
+    revenue_events: buckets.revenue_events,
+    revenue_intercompany: buckets.revenue_intercompany,
+    revenue_non_sales: buckets.revenue_non_sales,        // FX gains + purchase rebates (excluded)
     total_transactions: totalTransactions,
     total_quantity: totalQuantity,
     avg_ticket_usd: avgTicketUSD !== null ? Math.round(avgTicketUSD * 100) / 100 : null,
@@ -68,6 +80,28 @@ export function calculateSummary(invoices, lines, exchangeRate = null, glRevenue
   };
 }
 
+// revenue_store = total − intercompany − events − non_sales. Accounts per store
+// from config (not inline). Identity: total = store + interco + events + non_sales.
+export function splitRevenueBuckets(glRevenue, totalRevenueUSD, storeCode = null) {
+  if (!glRevenue || !glRevenue.breakdown || totalRevenueUSD === null) {
+    return { revenue_total: totalRevenueUSD ?? null, revenue_store: null, revenue_events: null, revenue_intercompany: null, revenue_non_sales: null };
+  }
+  const b = glRevenue.breakdown;
+  const sumAccts = (accts) => accts.reduce((s, a) => s + (b[a] || 0), 0);
+  const r2 = (n) => Math.round(n * 100) / 100;
+  const { intercompany, events, nonSales } = getRevenueClassification(storeCode);
+  const ic = sumAccts(intercompany);
+  const ev = sumAccts(events);
+  const ns = sumAccts(nonSales);
+  return {
+    revenue_total: r2(totalRevenueUSD),
+    revenue_intercompany: r2(ic),
+    revenue_events: r2(ev),
+    revenue_non_sales: r2(ns),
+    revenue_store: r2(totalRevenueUSD - ic - ev - ns),
+  };
+}
+
 export function calculateGrowth(current, previous) {
   if (!previous || previous === 0) return null;
   return ((current - previous) / Math.abs(previous)) * 100;