@dizzlkheinz/ynab-mcpb 0.18.4 → 0.19.0

package/dist/tools/payeeTools.js

@@ -1,24 +1,24 @@
-import { z } from 'zod/v4';
-import { withToolErrorHandling } from '../types/index.js';
-import { responseFormatter } from '../server/responseFormatter.js';
-import { cacheManager, CACHE_TTLS, CacheManager } from '../server/cacheManager.js';
-import { CacheKeys } from '../server/cacheKeys.js';
-import { resolveDeltaFetcherArgs } from './deltaSupport.js';
-import { createAdapters, createBudgetResolver } from './adapters.js';
-import { ToolAnnotationPresets } from './toolCategories.js';
+import { z } from "zod/v4";
+import { CacheKeys } from "../server/cacheKeys.js";
+import { CACHE_TTLS, CacheManager, cacheManager, } from "../server/cacheManager.js";
+import { responseFormatter } from "../server/responseFormatter.js";
+import { withToolErrorHandling } from "../types/index.js";
+import { createAdapters, createBudgetResolver } from "./adapters.js";
+import { resolveDeltaFetcherArgs } from "./deltaSupport.js";
+import { ToolAnnotationPresets } from "./toolCategories.js";
 export const ListPayeesSchema = z
     .object({
-    budget_id: z.string().min(1, 'Budget ID is required'),
+    budget_id: z.string().min(1, "Budget ID is required"),
     limit: z.number().int().positive().optional(),
 })
     .strict();
 export const GetPayeeSchema = z
     .object({
-    budget_id: z.string().min(1, 'Budget ID is required'),
-    payee_id: z.string().min(1, 'Payee ID is required'),
+    budget_id: z.string().min(1, "Budget ID is required"),
+    payee_id: z.string().min(1, "Payee ID is required"),
 })
     .strict();
-export async function handleListPayees(ynabAPI, deltaFetcherOrParams, maybeParams) {
+export async function handleListPayees(ynabAPI, deltaFetcherOrParams, maybeParams, errorHandler) {
     const { deltaFetcher, params } = resolveDeltaFetcherArgs(ynabAPI, deltaFetcherOrParams, maybeParams);
     return await withToolErrorHandling(async () => {
         const result = await deltaFetcher.fetchPayees(params.budget_id);
@@ -31,7 +31,7 @@ export async function handleListPayees(ynabAPI, deltaFetcherOrParams, maybeParam
         return {
             content: [
                 {
-                    type: 'text',
+                    type: "text",
                     text: responseFormatter.format({
                         payees: payees.map((payee) => ({
                             id: payee.id,
@@ -43,17 +43,17 @@ export async function handleListPayees(ynabAPI, deltaFetcherOrParams, maybeParam
                         returned_count: payees.length,
                         cached: wasCached,
                         cache_info: wasCached
-                            ? `Data retrieved from cache for improved performance${result.usedDelta ? ' (delta merge applied)' : ''}`
-                            : 'Fresh data retrieved from YNAB API',
+                            ? `Data retrieved from cache for improved performance${result.usedDelta ? " (delta merge applied)" : ""}`
+                            : "Fresh data retrieved from YNAB API",
                     }),
                 },
             ],
         };
-    }, 'ynab:list_payees', 'listing payees');
+    }, "ynab:list_payees", "listing payees", errorHandler);
 }
-export async function handleGetPayee(ynabAPI, params) {
+export async function handleGetPayee(ynabAPI, params, errorHandler) {
     return await withToolErrorHandling(async () => {
-        const cacheKey = CacheManager.generateKey(CacheKeys.PAYEES, 'get', params.budget_id, params.payee_id);
+        const cacheKey = CacheManager.generateKey(CacheKeys.PAYEES, "get", params.budget_id, params.payee_id);
         const wasCached = cacheManager.has(cacheKey);
         const payee = await cacheManager.wrap(cacheKey, {
             ttl: CACHE_TTLS.PAYEES,
@@ -65,7 +65,7 @@ export async function handleGetPayee(ynabAPI, params) {
         return {
             content: [
                 {
-                    type: 'text',
+                    type: "text",
                     text: responseFormatter.format({
                         payee: {
                             id: payee.id,
@@ -75,40 +75,40 @@ export async function handleGetPayee(ynabAPI, params) {
                         },
                         cached: wasCached,
                         cache_info: wasCached
-                            ? 'Data retrieved from cache for improved performance'
-                            : 'Fresh data retrieved from YNAB API',
+                            ? "Data retrieved from cache for improved performance"
+                            : "Fresh data retrieved from YNAB API",
                     }),
                 },
             ],
         };
-    }, 'ynab:get_payee', 'getting payee details');
+    }, "ynab:get_payee", "getting payee details", errorHandler);
 }
 export const registerPayeeTools = (registry, context) => {
     const { adapt, adaptWithDelta } = createAdapters(context);
     const budgetResolver = createBudgetResolver(context);
     registry.register({
-        name: 'list_payees',
-        description: 'List all payees for a specific budget',
+        name: "list_payees",
+        description: "List all payees for a specific budget",
         inputSchema: ListPayeesSchema,
         handler: adaptWithDelta(handleListPayees),
         defaultArgumentResolver: budgetResolver(),
         metadata: {
             annotations: {
                 ...ToolAnnotationPresets.READ_ONLY_EXTERNAL,
-                title: 'YNAB: List Payees',
+                title: "YNAB: List Payees",
             },
         },
     });
     registry.register({
-        name: 'get_payee',
-        description: 'Get detailed information for a specific payee',
+        name: "get_payee",
+        description: "Get detailed information for a specific payee",
         inputSchema: GetPayeeSchema,
         handler: adapt(handleGetPayee),
         defaultArgumentResolver: budgetResolver(),
         metadata: {
             annotations: {
                 ...ToolAnnotationPresets.READ_ONLY_EXTERNAL,
-                title: 'YNAB: Get Payee Details',
+                title: "YNAB: Get Payee Details",
             },
         },
     });

package/dist/tools/reconcileAdapter.d.ts

@@ -1,5 +1,5 @@
-import type { ReconciliationAnalysis } from './reconciliation/types.js';
-import type { LegacyReconciliationResult } from './reconciliation/executor.js';
+import type { LegacyReconciliationResult } from "./reconciliation/executor.js";
+import type { ReconciliationAnalysis } from "./reconciliation/types.js";
 interface AdapterOptions {
     accountName?: string;
     accountId?: string;

package/dist/tools/reconcileAdapter.js

@@ -1,7 +1,7 @@
-import { toMoneyValue, toMoneyValueFromDecimal } from '../utils/money.js';
-import { formatHumanReadableReport, } from './reconciliation/reportFormatter.js';
-const OUTPUT_VERSION = '2.0';
-const SCHEMA_URL = 'https://raw.githubusercontent.com/dizzlkheinz/ynab-mcp-mcpb/master/docs/schemas/reconciliation-v2.json';
+import { toMoneyValue, toMoneyValueFromDecimal } from "../utils/money.js";
+import { formatHumanReadableReport, } from "./reconciliation/reportFormatter.js";
+const OUTPUT_VERSION = "2.0";
+const SCHEMA_URL = "https://raw.githubusercontent.com/dizzlkheinz/ynab-mcp-mcpb/master/docs/schemas/reconciliation-v2.json";
 const toBankTransactionView = (txn, currency) => ({
     ...txn,
     amount_money: toMoneyValue(txn.amount, currency),
@@ -38,7 +38,8 @@ const convertSummary = (analysis) => ({
     statement_date_range: analysis.summary.statement_date_range,
     bank_transactions_count: analysis.summary.bank_transactions_count,
     ynab_transactions_count: analysis.summary.ynab_transactions_count,
-    ynab_in_range_count: analysis.summary.ynab_in_range_count ?? analysis.summary.ynab_transactions_count,
+    ynab_in_range_count: analysis.summary.ynab_in_range_count ??
+        analysis.summary.ynab_transactions_count,
     ynab_outside_range_count: analysis.summary.ynab_outside_range_count ?? 0,
     auto_matched: analysis.summary.auto_matched,
     suggested_matches: analysis.summary.suggested_matches,
@@ -51,7 +52,11 @@ const convertSummary = (analysis) => ({
 });
 const convertBalanceInfo = (analysis) => {
     const discrepancyMilli = analysis.balance_info.discrepancy.value_milliunits;
-    const direction = discrepancyMilli === 0 ? 'balanced' : discrepancyMilli > 0 ? 'ynab_higher' : 'bank_higher';
+    const direction = discrepancyMilli === 0
+        ? "balanced"
+        : discrepancyMilli > 0
+            ? "ynab_higher"
+            : "bank_higher";
     return {
         current_cleared: analysis.balance_info.current_cleared,
         current_uncleared: analysis.balance_info.current_uncleared,
@@ -133,8 +138,10 @@ const buildHumanNarrative = (analysis, options, execution) => {
     return formatHumanReadableReport(analysis, formatterOptions, execution);
 };
 export const buildReconciliationPayload = (analysis, options = {}, execution) => {
-    const currency = options.currencyCode ?? 'USD';
-    const executionView = execution ? convertExecution(execution, currency) : undefined;
+    const currency = options.currencyCode ?? "USD";
+    const executionView = execution
+        ? convertExecution(execution, currency)
+        : undefined;
     const structured = {
         version: OUTPUT_VERSION,
         schema_url: SCHEMA_URL,
@@ -158,16 +165,16 @@ export const buildReconciliationPayload = (analysis, options = {}, execution) =>
         },
     };
     if (analysis.recommendations && analysis.recommendations.length > 0) {
-        structured['recommendations'] = analysis.recommendations;
+        structured["recommendations"] = analysis.recommendations;
     }
     if (options.csvFormat) {
-        structured['csv_format'] = options.csvFormat;
+        structured["csv_format"] = options.csvFormat;
     }
     if (executionView) {
-        structured['execution'] = executionView;
+        structured["execution"] = executionView;
     }
     if (options.auditMetadata) {
-        structured['audit'] = options.auditMetadata;
+        structured["audit"] = options.auditMetadata;
     }
     return {
         human: buildHumanNarrative(analysis, options, execution),

package/dist/tools/reconciliation/analyzer.d.ts

@@ -1,7 +1,7 @@
-import type * as ynab from 'ynab';
-import { type ParseCSVOptions, type CSVParseResult } from './csvParser.js';
-import { type MatchingConfig } from './matcher.js';
-import type { ReconciliationAnalysis } from './types.js';
+import type * as ynab from "ynab";
+import { type CSVParseResult, type ParseCSVOptions } from "./csvParser.js";
+import { type MatchingConfig } from "./matcher.js";
+import type { ReconciliationAnalysis } from "./types.js";
 export declare function analyzeReconciliation(csvContentOrParsed: string | CSVParseResult, _csvFilePath: string | undefined, ynabTransactions: ynab.TransactionDetail[], statementBalance: number, config?: MatchingConfig, currency?: string, accountId?: string, budgetId?: string, invertBankAmounts?: boolean, csvOptions?: ParseCSVOptions, accountSnapshot?: {
     balance?: number;
     cleared_balance?: number;

package/dist/tools/reconciliation/analyzer.js

@@ -1,8 +1,8 @@
-import { parseCSV } from './csvParser.js';
-import { findMatches, normalizeConfig, DEFAULT_CONFIG } from './matcher.js';
-import { normalizeYNABTransactions } from './ynabAdapter.js';
-import { toMoneyValue } from '../../utils/money.js';
-import { generateRecommendations } from './recommendationEngine.js';
+import { parseCSV, } from "./csvParser.js";
+import { DEFAULT_CONFIG, findMatches, normalizeConfig, } from "./matcher.js";
+import { normalizeYNABTransactions } from "./ynabAdapter.js";
+import { toMoneyValue } from "../../utils/money.js";
+import { generateRecommendations } from "./recommendationEngine.js";
 function calculateDateRange(bankTransactions) {
     if (bankTransactions.length === 0) {
         return null;
@@ -14,20 +14,25 @@ function calculateDateRange(bankTransactions) {
     if (dates.length === 0) {
         return null;
     }
+    const minDate = dates[0];
+    const maxDate = dates[dates.length - 1];
+    if (!minDate || !maxDate) {
+        return null;
+    }
     return {
-        minDate: dates[0],
-        maxDate: dates[dates.length - 1],
+        minDate,
+        maxDate,
     };
 }
 function filterByDateRange(ynabTransactions, dateRange, dateToleranceDays = 7) {
+    const safeToleranceDays = dateToleranceDays < 0 ? 0 : dateToleranceDays;
     if (dateToleranceDays < 0) {
         console.warn(`[filterByDateRange] dateToleranceDays must be non-negative, got ${dateToleranceDays}. Using 0.`);
-        dateToleranceDays = 0;
     }
     const inRange = [];
     const outsideRange = [];
-    const minParts = dateRange.minDate.split('-').map(Number);
-    const maxParts = dateRange.maxDate.split('-').map(Number);
+    const minParts = dateRange.minDate.split("-").map(Number);
+    const maxParts = dateRange.maxDate.split("-").map(Number);
     if (minParts.length !== 3 ||
         maxParts.length !== 3 ||
         minParts.some((n) => !Number.isFinite(n)) ||
@@ -37,10 +42,10 @@ function filterByDateRange(ynabTransactions, dateRange, dateToleranceDays = 7) {
     }
     const [minYear, minMonth, minDay] = minParts;
     const [maxYear, maxMonth, maxDay] = maxParts;
-    const minDateWithBuffer = new Date(Date.UTC(minYear, minMonth - 1, minDay - dateToleranceDays));
-    const minDateStr = minDateWithBuffer.toISOString().split('T')[0];
-    const maxDateWithBuffer = new Date(Date.UTC(maxYear, maxMonth - 1, maxDay + dateToleranceDays));
-    const maxDateStr = maxDateWithBuffer.toISOString().split('T')[0];
+    const minDateWithBuffer = new Date(Date.UTC(minYear, minMonth - 1, minDay - safeToleranceDays));
+    const minDateStr = minDateWithBuffer.toISOString().split("T")[0] ?? "";
+    const maxDateWithBuffer = new Date(Date.UTC(maxYear, maxMonth - 1, maxDay + safeToleranceDays));
+    const maxDateStr = maxDateWithBuffer.toISOString().split("T")[0] ?? "";
     for (const txn of ynabTransactions) {
         if (txn.date >= minDateStr && txn.date <= maxDateStr) {
             inRange.push(txn);
@@ -55,16 +60,16 @@ function mapToTransactionMatch(result) {
     const candidates = result.candidates.map((c) => ({
         ynab_transaction: c.ynabTransaction,
         confidence: c.scores.combined,
-        match_reason: c.matchReasons.join(', '),
-        explanation: c.matchReasons.join(', '),
+        match_reason: c.matchReasons.join(", "),
+        explanation: c.matchReasons.join(", "),
     }));
     const match = {
         bankTransaction: result.bankTransaction,
         candidates,
         confidence: result.confidence,
         confidenceScore: result.confidenceScore,
-        matchReason: result.bestMatch?.matchReasons.join(', ') ?? 'No match found',
-        actionHint: result.confidence === 'high' ? 'approve' : 'review',
+        matchReason: result.bestMatch?.matchReasons.join(", ") ?? "No match found",
+        actionHint: result.confidence === "high" ? "approve" : "review",
     };
     if (result.bestMatch) {
         match.ynabTransaction = result.bestMatch.ynabTransaction;
@@ -72,8 +77,9 @@ function mapToTransactionMatch(result) {
     if (result.candidates[0]) {
         match.topConfidence = result.candidates[0].scores.combined;
     }
-    if (result.confidence === 'none') {
-        match.recommendation = 'This bank transaction is not in YNAB. Consider adding it.';
+    if (result.confidence === "none") {
+        match.recommendation =
+            "This bank transaction is not in YNAB. Consider adding it.";
     }
     return match;
 }
@@ -82,7 +88,7 @@ function calculateBalances(ynabTransactions, statementBalanceDecimal, currency,
     let computedUncleared = 0;
     for (const txn of ynabTransactions) {
         const amount = txn.amount;
-        if (txn.cleared === 'cleared' || txn.cleared === 'reconciled') {
+        if (txn.cleared === "cleared" || txn.cleared === "reconciled") {
             computedCleared += amount;
         }
         else {
@@ -105,11 +111,11 @@ function calculateBalances(ynabTransactions, statementBalanceDecimal, currency,
 }
 function generateSummary(bankTransactions, ynabTransactionsInRange, ynabTransactionsOutsideRange, autoMatches, suggestedMatches, unmatchedBank, unmatchedYNAB, balances) {
     const dates = bankTransactions.map((t) => t.date).sort();
-    const dateRange = dates.length > 0 ? `${dates[0]} to ${dates[dates.length - 1]}` : 'Unknown';
+    const dateRange = dates.length > 0 ? `${dates[0]} to ${dates[dates.length - 1]}` : "Unknown";
     const totalYnabCount = ynabTransactionsInRange.length + ynabTransactionsOutsideRange.length;
-    let discrepancyExplanation = '';
+    let discrepancyExplanation = "";
     if (balances.on_track) {
-        discrepancyExplanation = 'Cleared balance matches statement';
+        discrepancyExplanation = "Cleared balance matches statement";
     }
     else {
         const actionsNeeded = [];
@@ -123,7 +129,9 @@ function generateSummary(bankTransactions, ynabTransactionsInRange, ynabTransact
             actionsNeeded.push(`review ${unmatchedYNAB.length} unmatched YNAB`);
         }
         discrepancyExplanation =
-            actionsNeeded.length > 0 ? `Need to ${actionsNeeded.join(', ')}` : 'Manual review required';
+            actionsNeeded.length > 0
+                ? `Need to ${actionsNeeded.join(", ")}`
+                : "Manual review required";
     }
     return {
         statement_date_range: dateRange,
@@ -156,20 +164,20 @@ function generateNextSteps(summary) {
         steps.push(`Decide what to do with ${summary.unmatched_ynab} unmatched YNAB transactions (unclear/delete/ignore)`);
     }
     if (steps.length === 0) {
-        steps.push('All transactions matched! Review and approve to complete reconciliation');
+        steps.push("All transactions matched! Review and approve to complete reconciliation");
     }
     return steps;
 }
-function formatCurrency(amountMilli, currency = 'USD') {
-    const formatter = new Intl.NumberFormat('en-US', {
-        style: 'currency',
+function formatCurrency(amountMilli, currency = "USD") {
+    const formatter = new Intl.NumberFormat("en-US", {
+        style: "currency",
         currency: currency,
         minimumFractionDigits: 2,
         maximumFractionDigits: 2,
     });
     return formatter.format(amountMilli / 1000);
 }
-function repeatAmountInsights(unmatchedBank, currency = 'USD') {
+function repeatAmountInsights(unmatchedBank, currency = "USD") {
     const insights = [];
     if (unmatchedBank.length === 0) {
         return insights;
@@ -188,13 +196,15 @@ function repeatAmountInsights(unmatchedBank, currency = 'USD') {
         return insights;
     }
     const top = repeated[0];
+    if (!top) {
+        return insights;
+    }
     insights.push({
         id: `repeat-${top.amount}`,
-        type: 'repeat_amount',
-        severity: top.txns.length >= 4 ? 'critical' : 'warning',
+        type: "repeat_amount",
+        severity: top.txns.length >= 4 ? "critical" : "warning",
         title: `${top.txns.length} unmatched transactions at ${formatCurrency(top.amount, currency)}`,
-        description: `The bank statement shows ${top.txns.length} unmatched transaction(s) at ${formatCurrency(top.amount, currency)}. ` +
-            'Repeated amounts are usually the quickest wins — reconcile these first.',
+        description: `The bank statement shows ${top.txns.length} unmatched transaction(s) at ${formatCurrency(top.amount, currency)}. Repeated amounts are usually the quickest wins — reconcile these first.`,
         evidence: {
             amount: top.amount,
             occurrences: top.txns.length,
@@ -209,9 +219,9 @@ function anomalyInsights(balances) {
     const discrepancyAbs = Math.abs(balances.discrepancy.value_milliunits);
     if (discrepancyAbs >= 1000) {
         insights.push({
-            id: 'balance-gap',
-            type: 'anomaly',
-            severity: discrepancyAbs >= 100000 ? 'critical' : 'warning',
+            id: "balance-gap",
+            type: "anomaly",
+            severity: discrepancyAbs >= 100000 ? "critical" : "warning",
             title: `Cleared balance off by ${balances.discrepancy.value_display}`,
             description: `YNAB cleared balance is ${balances.current_cleared.value_display} but the statement expects ` +
                 `${balances.target_statement.value_display}. Focus on closing this gap.`,
@@ -235,14 +245,15 @@ function detectInsights(unmatchedBank, _summary, balances, currency, csvErrors =
     };
     if (csvErrors.length > 0) {
         addUnique({
-            id: 'csv-parse-errors',
-            type: 'anomaly',
-            severity: csvErrors.length >= 5 ? 'critical' : 'warning',
+            id: "csv-parse-errors",
+            type: "anomaly",
+            severity: csvErrors.length >= 5 ? "critical" : "warning",
             title: `${csvErrors.length} CSV parsing error(s)`,
             description: csvErrors
                 .slice(0, 3)
                 .map((e) => `Row ${e.row}: ${e.message}`)
-                .join('; ') + (csvErrors.length > 3 ? ` (+${csvErrors.length - 3} more)` : ''),
+                .join("; ") +
+                (csvErrors.length > 3 ? ` (+${csvErrors.length - 3} more)` : ""),
             evidence: {
                 error_count: csvErrors.length,
                 errors: csvErrors.slice(0, 5),
@@ -251,14 +262,15 @@ function detectInsights(unmatchedBank, _summary, balances, currency, csvErrors =
     }
     if (csvWarnings.length > 0) {
         addUnique({
-            id: 'csv-parse-warnings',
-            type: 'anomaly',
-            severity: 'info',
+            id: "csv-parse-warnings",
+            type: "anomaly",
+            severity: "info",
             title: `${csvWarnings.length} CSV parsing warning(s)`,
             description: csvWarnings
                 .slice(0, 3)
                 .map((w) => `Row ${w.row}: ${w.message}`)
-                .join('; ') + (csvWarnings.length > 3 ? ` (+${csvWarnings.length - 3} more)` : ''),
+                .join("; ") +
+                (csvWarnings.length > 3 ? ` (+${csvWarnings.length - 3} more)` : ""),
             evidence: {
                 warning_count: csvWarnings.length,
                 warnings: csvWarnings.slice(0, 5),
@@ -273,9 +285,9 @@ function detectInsights(unmatchedBank, _summary, balances, currency, csvErrors =
     }
     return insights.slice(0, 5);
 }
-export function analyzeReconciliation(csvContentOrParsed, _csvFilePath, ynabTransactions, statementBalance, config = DEFAULT_CONFIG, currency = 'USD', accountId, budgetId, invertBankAmounts = false, csvOptions, accountSnapshot) {
+export function analyzeReconciliation(csvContentOrParsed, _csvFilePath, ynabTransactions, statementBalance, config = DEFAULT_CONFIG, currency = "USD", accountId, budgetId, invertBankAmounts = false, csvOptions, accountSnapshot) {
     let parseResult;
-    if (typeof csvContentOrParsed === 'string') {
+    if (typeof csvContentOrParsed === "string") {
         parseResult = parseCSV(csvContentOrParsed, {
             ...csvOptions,
             invertAmounts: invertBankAmounts,
@@ -304,21 +316,23 @@ export function analyzeReconciliation(csvContentOrParsed, _csvFilePath, ynabTran
     const normalizedConfig = normalizeConfig(config);
     const newMatches = findMatches(newBankTransactions, ynabInRange, normalizedConfig);
     const matches = newMatches.map(mapToTransactionMatch);
-    const autoMatches = matches.filter((m) => m.confidence === 'high');
+    const autoMatches = matches.filter((m) => m.confidence === "high");
     const autoMatchedYnabIds = new Set();
-    autoMatches.forEach((m) => {
-        if (m.ynabTransaction)
-            autoMatchedYnabIds.add(m.ynabTransaction.id);
-    });
-    const suggestedMatches = matches.filter((m) => m.confidence === 'medium' &&
+    for (const match of autoMatches) {
+        if (match.ynabTransaction) {
+            autoMatchedYnabIds.add(match.ynabTransaction.id);
+        }
+    }
+    const suggestedMatches = matches.filter((m) => m.confidence === "medium" &&
         (!m.ynabTransaction || !autoMatchedYnabIds.has(m.ynabTransaction.id)));
-    const unmatchedBankMatches = matches.filter((m) => m.confidence === 'low' || m.confidence === 'none');
+    const unmatchedBankMatches = matches.filter((m) => m.confidence === "low" || m.confidence === "none");
     const unmatchedBank = unmatchedBankMatches.map((m) => m.bankTransaction);
     const matchedYnabIds = new Set();
-    matches.forEach((m) => {
-        if (m.ynabTransaction)
-            matchedYnabIds.add(m.ynabTransaction.id);
-    });
+    for (const match of matches) {
+        if (match.ynabTransaction) {
+            matchedYnabIds.add(match.ynabTransaction.id);
+        }
+    }
     const unmatchedYNAB = ynabInRange.filter((t) => !matchedYnabIds.has(t.id));
     const balances = calculateBalances(allYNABTransactions, statementBalance, currency, accountSnapshot);
     const summary = generateSummary(matches.map((m) => m.bankTransaction), ynabInRange, ynabOutsideRange, autoMatches, suggestedMatches, unmatchedBank, unmatchedYNAB, balances);
@@ -326,7 +340,7 @@ export function analyzeReconciliation(csvContentOrParsed, _csvFilePath, ynabTran
     const insights = detectInsights(unmatchedBank, summary, balances, currency, csvParseErrors, csvParseWarnings);
     const analysis = {
         success: true,
-        phase: 'analysis',
+        phase: "analysis",
         summary,
         auto_matches: autoMatches,
         suggested_matches: suggestedMatches,

package/dist/tools/reconciliation/csvParser.d.ts

@@ -1,4 +1,4 @@
-import type { BankTransaction } from '../../types/reconciliation.js';
+import type { BankTransaction } from "../../types/reconciliation.js";
 export interface CSVParseResult {
     transactions: BankTransaction[];
     errors: ParseError[];
@@ -29,7 +29,7 @@ export interface BankPreset {
     creditColumn?: string;
     descriptionColumn: string | string[];
     amountMultiplier?: number;
-    dateFormat?: 'YMD' | 'MDY' | 'DMY';
+    dateFormat?: "YMD" | "MDY" | "DMY";
     header?: boolean;
 }
 export declare const BANK_PRESETS: Record<string, BankPreset>;
@@ -46,7 +46,7 @@ export interface ParseCSVOptions {
         credit?: string;
         description?: string;
     };
-    dateFormat?: 'YMD' | 'MDY' | 'DMY';
+    dateFormat?: "YMD" | "MDY" | "DMY";
     header?: boolean;
     maxRows?: number;
     maxBytes?: number;