@dizzlkheinz/ynab-mcpb 0.13.1 → 0.15.0

package/src/tools/reconciliation/__tests__/scenarios/repeatAmount.scenario.test.ts

@@ -1,11 +1,10 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest';
 import { analyzeReconciliation } from '../../analyzer.js';
 import type { TransactionDetail } from 'ynab';
-import * as parser from '../../../compareTransactions/parser.js';
+import * as csvParser from '../../csvParser.js';
 
-vi.mock('../../../compareTransactions/parser.js', () => ({
-  parseBankCSV: vi.fn(),
-  readCSVFile: vi.fn(),
+vi.mock('../../csvParser.js', () => ({
+  parseCSV: vi.fn(),
 }));
 
 describe('scenario: repeat amount collisions', () => {
@@ -14,19 +13,55 @@ describe('scenario: repeat amount collisions', () => {
   });
 
   it('prioritizes repeat-amount insight when multiple bank rows share totals', () => {
-    vi.mocked(parser.parseBankCSV).mockReturnValue({
+    vi.mocked(csvParser.parseCSV).mockReturnValue({
       transactions: [
-        // Three -22.22 transactions: one will match YNAB, two will remain unmatched
-        { date: '2025-10-20', amount: -22.22, payee: 'RideShare', memo: '' },
-        { date: '2025-10-21', amount: -22.22, payee: 'RideShare', memo: '' },
-        { date: '2025-10-25', amount: -22.22, payee: 'RideShare', memo: '' },
-        { date: '2025-10-23', amount: -15.0, payee: 'Cafe', memo: '' },
+        // Three -22.22 transactions in milliunits: one will match YNAB, two will remain unmatched
+        {
+          id: 'b1',
+          date: '2025-10-20',
+          amount: -22220,
+          payee: 'RideShare',
+          memo: '',
+          sourceRow: 2,
+          raw: { date: '2025-10-20', amount: '-22.22', description: 'RideShare' },
+        },
+        {
+          id: 'b2',
+          date: '2025-10-21',
+          amount: -22220,
+          payee: 'RideShare',
+          memo: '',
+          sourceRow: 3,
+          raw: { date: '2025-10-21', amount: '-22.22', description: 'RideShare' },
+        },
+        {
+          id: 'b3',
+          date: '2025-10-25',
+          amount: -22220,
+          payee: 'RideShare',
+          memo: '',
+          sourceRow: 4,
+          raw: { date: '2025-10-25', amount: '-22.22', description: 'RideShare' },
+        },
+        {
+          id: 'b4',
+          date: '2025-10-23',
+          amount: -15000,
+          payee: 'Cafe',
+          memo: '',
+          sourceRow: 5,
+          raw: { date: '2025-10-23', amount: '-15.00', description: 'Cafe' },
+        },
       ],
-      format_detected: 'standard',
-      delimiter: ',',
-      total_rows: 4,
-      valid_rows: 4,
       errors: [],
+      warnings: [],
+      meta: {
+        detectedDelimiter: ',',
+        detectedColumns: ['Date', 'Description', 'Amount'],
+        totalRows: 4,
+        validRows: 4,
+        skippedRows: 0,
+      },
     });
 
     const ynabTxns: TransactionDetail[] = [
@@ -53,7 +88,7 @@ describe('scenario: repeat amount collisions', () => {
     // Statement balance now accounts for 3 x -22.22 + 1 x -15.00 = -81.66
     const result = analyzeReconciliation('csv', undefined, ynabTxns, -81.66);
 
-    const repeatInsight = result.insights.find((insight) => insight.id.startsWith('repeat--22.22')); // Note: double dash for negative
+    const repeatInsight = result.insights.find((insight) => insight.id.startsWith('repeat--22220')); // amount in milliunits, double dash for negative
     expect(repeatInsight).toBeDefined();
     expect(repeatInsight?.severity).toBe('warning');
     expect(result.summary.unmatched_bank).toBeGreaterThan(0);

package/src/tools/reconciliation/__tests__/signDetector.test.ts

@@ -0,0 +1,211 @@
+/**
+ * Tests for smart sign detection algorithm
+ */
+
+import { describe, it, expect } from 'vitest';
+import { detectSignInversion } from '../signDetector.js';
+import type { BankTransaction, NormalizedYNABTransaction } from '../../../types/reconciliation.js';
+
+describe('detectSignInversion', () => {
+  it('detects when bank amounts need inversion (opposite signs)', () => {
+    // Bank CSV shows charges as POSITIVE (banking convention)
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: 10000, payee: 'Grocery Store' }, // +$10
+      { id: '2', date: '2025-11-21', amount: 5000, payee: 'Gas Station' }, // +$5
+      { id: '3', date: '2025-11-22', amount: 20000, payee: 'Restaurant' }, // +$20
+    ];
+
+    // YNAB shows charges as NEGATIVE
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-11-20',
+        amount: -10000, // -$10
+        payee: 'Grocery Store',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y2',
+        date: '2025-11-21',
+        amount: -5000, // -$5
+        payee: 'Gas Station',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y3',
+        date: '2025-11-22',
+        amount: -20000, // -$20
+        payee: 'Restaurant',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+
+    const result = detectSignInversion(bankTransactions, ynabTransactions);
+
+    expect(result).toBe(true); // Need inversion
+  });
+
+  it('detects when bank amounts do NOT need inversion (same signs)', () => {
+    // Bank CSV already uses YNAB convention (Wealthsimple case)
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: -10000, payee: 'Grocery Store' }, // -$10
+      { id: '2', date: '2025-11-21', amount: -5000, payee: 'Gas Station' }, // -$5
+      { id: '3', date: '2025-11-22', amount: -20000, payee: 'Restaurant' }, // -$20
+    ];
+
+    // YNAB also shows charges as NEGATIVE
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-11-20',
+        amount: -10000,
+        payee: 'Grocery Store',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y2',
+        date: '2025-11-21',
+        amount: -5000,
+        payee: 'Gas Station',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y3',
+        date: '2025-11-22',
+        amount: -20000,
+        payee: 'Restaurant',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+
+    const result = detectSignInversion(bankTransactions, ynabTransactions);
+
+    expect(result).toBe(false); // No inversion needed
+  });
+
+  it('handles mixed positive and negative amounts correctly', () => {
+    // Bank CSV with charges (positive) and payment (negative)
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: 10000, payee: 'Purchase' },
+      { id: '2', date: '2025-11-21', amount: -20000, payee: 'Payment' }, // Credit
+      { id: '3', date: '2025-11-22', amount: 5000, payee: 'Purchase' },
+    ];
+
+    // YNAB with charges (negative) and payment (positive)
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-11-20',
+        amount: -10000,
+        payee: 'Purchase',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y2',
+        date: '2025-11-21',
+        amount: 20000, // Payment (positive in YNAB)
+        payee: 'Payment',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+      {
+        id: 'y3',
+        date: '2025-11-22',
+        amount: -5000,
+        payee: 'Purchase',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+
+    const result = detectSignInversion(bankTransactions, ynabTransactions);
+
+    expect(result).toBe(true); // Need inversion (opposite signs)
+  });
+
+  it('works with small sample sizes', () => {
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: 10000, payee: 'Store' },
+    ];
+
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-11-20',
+        amount: -10000,
+        payee: 'Store',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+
+    const result = detectSignInversion(bankTransactions, ynabTransactions);
+
+    expect(result).toBe(true);
+  });
+
+  it('returns false when no matching transactions found', () => {
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: 10000, payee: 'Store A' },
+    ];
+
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-10-20', // Different month
+        amount: -50000, // Different amount
+        payee: 'Store B',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+
+    const result = detectSignInversion(bankTransactions, ynabTransactions);
+
+    // Conservative default: don't invert if we can't determine
+    expect(result).toBe(false);
+  });
+
+  it('handles empty transaction lists', () => {
+    const result1 = detectSignInversion([], []);
+    expect(result1).toBe(false);
+
+    const bankTransactions: BankTransaction[] = [
+      { id: '1', date: '2025-11-20', amount: 10000, payee: 'Store' },
+    ];
+    const result2 = detectSignInversion(bankTransactions, []);
+    expect(result2).toBe(false);
+
+    const ynabTransactions: NormalizedYNABTransaction[] = [
+      {
+        id: 'y1',
+        date: '2025-11-20',
+        amount: -10000,
+        payee: 'Store',
+        cleared: 'uncleared',
+        approved: true,
+        deleted: false,
+      },
+    ];
+    const result3 = detectSignInversion([], ynabTransactions);
+    expect(result3).toBe(false);
+  });
+});

package/src/tools/reconciliation/__tests__/ynabAdapter.test.ts

@@ -0,0 +1,61 @@
+import { describe, it, expect } from 'vitest';
+import type * as ynab from 'ynab';
+import { normalizeYNABTransaction, normalizeYNABTransactions } from '../ynabAdapter.js';
+
+describe('ynabAdapter', () => {
+  const mockTransaction: ynab.TransactionDetail = {
+    id: 'test-id-1',
+    date: '2024-01-01',
+    amount: -10000,
+    payee_name: 'Test Payee',
+    memo: 'Test Memo',
+    category_name: 'Test Category',
+    cleared: 'uncleared' as ynab.TransactionClearedStatus,
+    approved: false,
+    account_id: 'account-1',
+    deleted: false,
+    account_name: 'Test Account',
+    subtransactions: [],
+  };
+
+  it('should normalize a single transaction correctly', () => {
+    const result = normalizeYNABTransaction(mockTransaction);
+
+    expect(result).toEqual({
+      id: 'test-id-1',
+      date: '2024-01-01',
+      amount: -10000,
+      payee: 'Test Payee',
+      memo: 'Test Memo',
+      categoryName: 'Test Category',
+      cleared: 'uncleared',
+      approved: false,
+    });
+  });
+
+  it('should handle null fields correctly', () => {
+    const minimalTransaction: ynab.TransactionDetail = {
+      ...mockTransaction,
+      payee_name: null as any, // SDK typing might say string | undefined/null
+      memo: null as any,
+      category_name: null as any,
+    };
+
+    const result = normalizeYNABTransaction(minimalTransaction);
+
+    expect(result.payee).toBeNull();
+    expect(result.memo).toBeNull();
+    expect(result.categoryName).toBeNull();
+  });
+
+  it('should normalize multiple transactions', () => {
+    const transactions = [mockTransaction, { ...mockTransaction, id: 'test-id-2', amount: 5000 }];
+
+    const results = normalizeYNABTransactions(transactions);
+
+    expect(results).toHaveLength(2);
+    expect(results[0].id).toBe('test-id-1');
+    expect(results[1].id).toBe('test-id-2');
+    expect(results[1].amount).toBe(5000);
+  });
+});