@dizzlkheinz/ynab-mcpb 0.18.4 → 0.19.0

package/scripts/run-throttled-integration-tests.js

@@ -2,21 +2,25 @@
 /**
  * Runs the full integration suite with simple rate-limit-aware throttling.
  */
-import { spawn } from 'node:child_process';
-import { readdirSync } from 'node:fs';
-import path from 'node:path';
+import { spawn } from "node:child_process";
+import { readdirSync } from "node:fs";
+import path from "node:path";
 
-const RATE_LIMIT = Number(process.env['RATE_LIMIT_PER_HOUR'] ?? 200);
-const RATE_LIMIT_BUFFER = Number(process.env['RATE_LIMIT_BUFFER'] ?? 20);
-const RATE_LIMIT_WINDOW_MS = Number(process.env['RATE_LIMIT_WINDOW_MS'] ?? 60 * 60 * 1000);
-const MAX_WAIT_MS = Number(process.env['RATE_LIMIT_MAX_WAIT_MS'] ?? 60 * 60 * 1000);
+const RATE_LIMIT = Number(process.env.RATE_LIMIT_PER_HOUR ?? 200);
+const RATE_LIMIT_BUFFER = Number(process.env.RATE_LIMIT_BUFFER ?? 20);
+const RATE_LIMIT_WINDOW_MS = Number(
+	process.env.RATE_LIMIT_WINDOW_MS ?? 60 * 60 * 1000,
+);
+const MAX_WAIT_MS = Number(
+	process.env.RATE_LIMIT_MAX_WAIT_MS ?? 60 * 60 * 1000,
+);
 const projectRoot = process.cwd();
 
-const testFiles = collectIntegrationTests(path.join(projectRoot, 'src'));
+const testFiles = collectIntegrationTests(path.join(projectRoot, "src"));
 const toPosixPath = (value) => value.split(path.sep).join(path.posix.sep);
 if (testFiles.length === 0) {
-  console.error('No integration test files found.');
-  process.exit(1);
+	console.error("No integration test files found.");
+	process.exit(1);
 }
 
 const requestHistory = [];
@@ -24,99 +28,104 @@ const requestHistory = [];
 await runSequentially(testFiles);
 
 async function runSequentially(files) {
-  for (const filePath of files) {
-    const estimatedCalls = estimateCalls(filePath);
-    await throttleIfNeeded(estimatedCalls);
-    const relativePath = path.relative(projectRoot, filePath);
-    console.log(`▶️  Running ${relativePath} (estimated ${estimatedCalls} API calls)`);
-    await runVitestFile(relativePath);
-    requestHistory.push({ timestamp: Date.now(), calls: estimatedCalls });
-  }
+	for (const filePath of files) {
+		const estimatedCalls = estimateCalls(filePath);
+		await throttleIfNeeded(estimatedCalls);
+		const relativePath = path.relative(projectRoot, filePath);
+		console.log(
+			`▶️  Running ${relativePath} (estimated ${estimatedCalls} API calls)`,
+		);
+		await runVitestFile(relativePath);
+		requestHistory.push({ timestamp: Date.now(), calls: estimatedCalls });
+	}
 }
 
 function collectIntegrationTests(dir) {
-  const entries = readdirSync(dir, { withFileTypes: true });
-  const files = [];
-  for (const entry of entries) {
-    const fullPath = path.join(dir, entry.name);
-    if (entry.isDirectory()) {
-      files.push(...collectIntegrationTests(fullPath));
-    } else if (entry.isFile() && entry.name.endsWith('.integration.test.ts')) {
-      files.push(fullPath);
-    }
-  }
-  return files.sort((a, b) => a.localeCompare(b));
+	const entries = readdirSync(dir, { withFileTypes: true });
+	const files = [];
+	for (const entry of entries) {
+		const fullPath = path.join(dir, entry.name);
+		if (entry.isDirectory()) {
+			files.push(...collectIntegrationTests(fullPath));
+		} else if (entry.isFile() && entry.name.endsWith(".integration.test.ts")) {
+			files.push(fullPath);
+		}
+	}
+	return files.sort((a, b) => a.localeCompare(b));
 }
 
 async function throttleIfNeeded(nextCalls) {
-  pruneHistory();
-  while (getRecentCalls() + nextCalls > RATE_LIMIT - RATE_LIMIT_BUFFER) {
-    const nextWindowTime = requestHistory[0]?.timestamp ?? Date.now();
-    const waitUntil = nextWindowTime + RATE_LIMIT_WINDOW_MS;
-    const waitMs = Math.min(Math.max(waitUntil - Date.now(), 1000), MAX_WAIT_MS);
-    const minutes = Math.max(1, Math.round(waitMs / 60000));
-    console.warn(
-      `⏳ Approaching rate limit (${getRecentCalls()}/${RATE_LIMIT}). Waiting ${minutes} minute${
-        minutes === 1 ? '' : 's'
-      }...`,
-    );
-    await sleep(waitMs);
-    pruneHistory();
-  }
+	pruneHistory();
+	while (getRecentCalls() + nextCalls > RATE_LIMIT - RATE_LIMIT_BUFFER) {
+		const nextWindowTime = requestHistory[0]?.timestamp ?? Date.now();
+		const waitUntil = nextWindowTime + RATE_LIMIT_WINDOW_MS;
+		const waitMs = Math.min(
+			Math.max(waitUntil - Date.now(), 1000),
+			MAX_WAIT_MS,
+		);
+		const minutes = Math.max(1, Math.round(waitMs / 60000));
+		console.warn(
+			`⏳ Approaching rate limit (${getRecentCalls()}/${RATE_LIMIT}). Waiting ${minutes} minute${
+				minutes === 1 ? "" : "s"
+			}...`,
+		);
+		await sleep(waitMs);
+		pruneHistory();
+	}
 }
 
 function pruneHistory() {
-  const cutoff = Date.now() - RATE_LIMIT_WINDOW_MS;
-  while (requestHistory.length && requestHistory[0].timestamp < cutoff) {
-    requestHistory.shift();
-  }
+	const cutoff = Date.now() - RATE_LIMIT_WINDOW_MS;
+	while (requestHistory.length && requestHistory[0].timestamp < cutoff) {
+		requestHistory.shift();
+	}
 }
 
 function getRecentCalls() {
-  return requestHistory.reduce((sum, entry) => sum + entry.calls, 0);
+	return requestHistory.reduce((sum, entry) => sum + entry.calls, 0);
 }
 
 function estimateCalls(filePath) {
-  const name = filePath.toLowerCase();
-  if (name.includes('delta')) return 15;
-  if (name.includes('reconciliation')) return 25;
-  if (name.includes('transaction')) return 12;
-  if (name.includes('budget')) return 8;
-  if (name.includes('account')) return 8;
-  if (name.includes('payee')) return 10;
-  if (name.includes('category')) return 10;
-  if (name.includes('month')) return 10;
-  return 10;
+	const name = filePath.toLowerCase();
+	if (name.includes("delta")) return 15;
+	if (name.includes("reconciliation")) return 25;
+	if (name.includes("transaction")) return 12;
+	if (name.includes("budget")) return 8;
+	if (name.includes("account")) return 8;
+	if (name.includes("payee")) return 10;
+	if (name.includes("category")) return 10;
+	if (name.includes("month")) return 10;
+	return 10;
 }
 
 async function runVitestFile(testFile) {
-  const normalized = toPosixPath(testFile);
-  const vitestBin = path.join(
-    projectRoot,
-    'node_modules',
-    '.bin',
-    process.platform === 'win32' ? 'vitest.cmd' : 'vitest',
-  );
-  const vitestArgs = ['run', '--project', 'integration:full', normalized];
-  const child = spawn(vitestBin, vitestArgs, {
-    stdio: 'inherit',
-    shell: process.platform === 'win32',
-    env: {
-      ...process.env,
-      INTEGRATION_TEST_TIER: 'full',
-      INTEGRATION_TEST_DOMAINS: '',
-    },
-  });
+	const normalized = toPosixPath(testFile);
+	const vitestBin = path.join(
+		projectRoot,
+		"node_modules",
+		".bin",
+		process.platform === "win32" ? "vitest.cmd" : "vitest",
+	);
+	const vitestArgs = ["run", "--project", "integration:full", normalized];
+	const child = spawn(vitestBin, vitestArgs, {
+		stdio: "inherit",
+		shell: process.platform === "win32",
+		env: {
+			...process.env,
+			INTEGRATION_TEST_TIER: "full",
+			INTEGRATION_TEST_DOMAINS: "",
+		},
+	});
 
-  await new Promise((resolve, reject) => {
-    child.on('close', (code) => {
-      if (code === 0) resolve();
-      else reject(new Error(`Vitest exited with code ${code}`));
-    });
-    child.on('error', reject);
-  });
+	await new Promise((resolve, reject) => {
+		child.on("close", (code) => {
+			if (code === 0) resolve();
+			else reject(new Error(`Vitest exited with code ${code}`));
+		});
+		child.on("error", reject);
+	});
 }
 
 function sleep(ms) {
-  return new Promise((resolve) => setTimeout(resolve, ms));
+	return new Promise((resolve) => setTimeout(resolve, ms));
 }

package/scripts/test-delta-params.mjs

@@ -1,10 +1,10 @@
-import * as ynab from 'ynab';
+import * as ynab from "ynab";
 
 const ACCESS_TOKEN = process.env.YNAB_ACCESS_TOKEN;
 
 if (!ACCESS_TOKEN) {
-  console.error('Missing YNAB_ACCESS_TOKEN environment variable.');
-  process.exit(1);
+	console.error("Missing YNAB_ACCESS_TOKEN environment variable.");
+	process.exit(1);
 }
 
 const ynabAPI = new ynab.API(ACCESS_TOKEN);
@@ -12,129 +12,158 @@ const ynabAPI = new ynab.API(ACCESS_TOKEN);
 const results = [];
 
 function recordResult(name, signature, supported, notes, error) {
-  results.push({ name, signature, supported, notes, error: error?.message ?? null });
-  const status = supported ? '✓' : '✗';
-  const suffix = notes ? ` - ${notes}` : '';
-  console.log(`${status} ${name} (${signature})${suffix}`);
-  if (error) {
-    console.log(`   Error: ${error.message}`);
-  }
+	results.push({
+		name,
+		signature,
+		supported,
+		notes,
+		error: error?.message ?? null,
+	});
+	const status = supported ? "✓" : "✗";
+	const suffix = notes ? ` - ${notes}` : "";
+	console.log(`${status} ${name} (${signature})${suffix}`);
+	if (error) {
+		console.log(`   Error: ${error.message}`);
+	}
 }
 
-async function verifySimpleEndpoint({ name, signature, fetchFull, fetchDelta }) {
-  try {
-    const full = await fetchFull();
-    const knowledge = full?.data?.server_knowledge;
-    if (typeof knowledge !== 'number') {
-      throw new Error('Response missing server_knowledge');
-    }
-
-    await fetchDelta(knowledge);
-    recordResult(name, signature, true, `server_knowledge=${knowledge}`);
-  } catch (error) {
-    recordResult(name, signature, false, '', error);
-  }
+async function verifySimpleEndpoint({
+	name,
+	signature,
+	fetchFull,
+	fetchDelta,
+}) {
+	try {
+		const full = await fetchFull();
+		const knowledge = full?.data?.server_knowledge;
+		if (typeof knowledge !== "number") {
+			throw new Error("Response missing server_knowledge");
+		}
+
+		await fetchDelta(knowledge);
+		recordResult(name, signature, true, `server_knowledge=${knowledge}`);
+	} catch (error) {
+		recordResult(name, signature, false, "", error);
+	}
 }
 
 async function main() {
-  console.log('Verifying YNAB SDK delta request signatures...\n');
-  const budgetsResponse = await ynabAPI.budgets.getBudgets();
-  const budget = budgetsResponse?.data?.budgets?.[0];
-  if (!budget) {
-    throw new Error('No budget available to test.');
-  }
-  const budgetId = budget.id;
-  console.log(`Using budget: ${budget.name} (${budgetId})\n`);
-
-  // Budgets API
-  await verifySimpleEndpoint({
-    name: 'budgets.getBudgetById',
-    signature: 'getBudgetById(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.budgets.getBudgetById(budgetId),
-    fetchDelta: (knowledge) => ynabAPI.budgets.getBudgetById(budgetId, knowledge),
-  });
-
-  // Accounts API
-  await verifySimpleEndpoint({
-    name: 'accounts.getAccounts',
-    signature: 'getAccounts(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.accounts.getAccounts(budgetId),
-    fetchDelta: (knowledge) => ynabAPI.accounts.getAccounts(budgetId, knowledge),
-  });
-
-  // Categories API
-  await verifySimpleEndpoint({
-    name: 'categories.getCategories',
-    signature: 'getCategories(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.categories.getCategories(budgetId),
-    fetchDelta: (knowledge) => ynabAPI.categories.getCategories(budgetId, knowledge),
-  });
-
-  // Payees API
-  await verifySimpleEndpoint({
-    name: 'payees.getPayees',
-    signature: 'getPayees(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.payees.getPayees(budgetId),
-    fetchDelta: (knowledge) => ynabAPI.payees.getPayees(budgetId, knowledge),
-  });
-
-  // Months API
-  await verifySimpleEndpoint({
-    name: 'months.getBudgetMonths',
-    signature: 'getBudgetMonths(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.months.getBudgetMonths(budgetId),
-    fetchDelta: (knowledge) => ynabAPI.months.getBudgetMonths(budgetId, knowledge),
-  });
-
-  // Scheduled Transactions API
-  await verifySimpleEndpoint({
-    name: 'scheduledTransactions.getScheduledTransactions',
-    signature: 'getScheduledTransactions(budgetId, lastKnowledge?)',
-    fetchFull: () => ynabAPI.scheduledTransactions.getScheduledTransactions(budgetId),
-    fetchDelta: (knowledge) =>
-      ynabAPI.scheduledTransactions.getScheduledTransactions(budgetId, knowledge),
-  });
-
-  // Transactions API
-  try {
-    const full = await ynabAPI.transactions.getTransactions(budgetId);
-    const knowledge = full?.data?.server_knowledge;
-    if (typeof knowledge !== 'number') {
-      throw new Error('Response missing server_knowledge');
-    }
-
-    let supported = false;
-    let notes = '';
-
-    try {
-      await ynabAPI.transactions.getTransactions(budgetId, undefined, undefined, knowledge);
-      supported = true;
-      notes = 'Signature: (budgetId, sinceDate?, type?, lastKnowledge?)';
-    } catch (error) {
-      notes = 'Failed with lastKnowledge as 4th parameter';
-      throw error;
-    }
-
-    recordResult('transactions.getTransactions', notes, supported, `server_knowledge=${knowledge}`);
-  } catch (error) {
-    recordResult(
-      'transactions.getTransactions',
-      'getTransactions(budgetId, since?, type?, lastKnowledge?)',
-      false,
-      '',
-      error,
-    );
-  }
-
-  console.log('\nSummary:\n');
-  for (const row of results) {
-    console.log(
-      `${row.supported ? '✓' : '✗'} ${row.name} -> ${row.supported ? 'supports delta' : 'no delta'} (${row.signature})`,
-    );
-  }
+	console.log("Verifying YNAB SDK delta request signatures...\n");
+	const budgetsResponse = await ynabAPI.budgets.getBudgets();
+	const budget = budgetsResponse?.data?.budgets?.[0];
+	if (!budget) {
+		throw new Error("No budget available to test.");
+	}
+	const budgetId = budget.id;
+	console.log(`Using budget: ${budget.name} (${budgetId})\n`);
+
+	// Budgets API
+	await verifySimpleEndpoint({
+		name: "budgets.getBudgetById",
+		signature: "getBudgetById(budgetId, lastKnowledge?)",
+		fetchFull: () => ynabAPI.budgets.getBudgetById(budgetId),
+		fetchDelta: (knowledge) =>
+			ynabAPI.budgets.getBudgetById(budgetId, knowledge),
+	});
+
+	// Accounts API
+	await verifySimpleEndpoint({
+		name: "accounts.getAccounts",
+		signature: "getAccounts(budgetId, lastKnowledge?)",
+		fetchFull: () => ynabAPI.accounts.getAccounts(budgetId),
+		fetchDelta: (knowledge) =>
+			ynabAPI.accounts.getAccounts(budgetId, knowledge),
+	});
+
+	// Categories API
+	await verifySimpleEndpoint({
+		name: "categories.getCategories",
+		signature: "getCategories(budgetId, lastKnowledge?)",
+		fetchFull: () => ynabAPI.categories.getCategories(budgetId),
+		fetchDelta: (knowledge) =>
+			ynabAPI.categories.getCategories(budgetId, knowledge),
+	});
+
+	// Payees API
+	await verifySimpleEndpoint({
+		name: "payees.getPayees",
+		signature: "getPayees(budgetId, lastKnowledge?)",
+		fetchFull: () => ynabAPI.payees.getPayees(budgetId),
+		fetchDelta: (knowledge) => ynabAPI.payees.getPayees(budgetId, knowledge),
+	});
+
+	// Months API
+	await verifySimpleEndpoint({
+		name: "months.getBudgetMonths",
+		signature: "getBudgetMonths(budgetId, lastKnowledge?)",
+		fetchFull: () => ynabAPI.months.getBudgetMonths(budgetId),
+		fetchDelta: (knowledge) =>
+			ynabAPI.months.getBudgetMonths(budgetId, knowledge),
+	});
+
+	// Scheduled Transactions API
+	await verifySimpleEndpoint({
+		name: "scheduledTransactions.getScheduledTransactions",
+		signature: "getScheduledTransactions(budgetId, lastKnowledge?)",
+		fetchFull: () =>
+			ynabAPI.scheduledTransactions.getScheduledTransactions(budgetId),
+		fetchDelta: (knowledge) =>
+			ynabAPI.scheduledTransactions.getScheduledTransactions(
+				budgetId,
+				knowledge,
+			),
+	});
+
+	// Transactions API
+	try {
+		const full = await ynabAPI.transactions.getTransactions(budgetId);
+		const knowledge = full?.data?.server_knowledge;
+		if (typeof knowledge !== "number") {
+			throw new Error("Response missing server_knowledge");
+		}
+
+		let supported = false;
+		let notes = "";
+
+		try {
+			await ynabAPI.transactions.getTransactions(
+				budgetId,
+				undefined,
+				undefined,
+				knowledge,
+			);
+			supported = true;
+			notes = "Signature: (budgetId, sinceDate?, type?, lastKnowledge?)";
+		} catch (error) {
+			notes = "Failed with lastKnowledge as 4th parameter";
+			throw error;
+		}
+
+		recordResult(
+			"transactions.getTransactions",
+			notes,
+			supported,
+			`server_knowledge=${knowledge}`,
+		);
+	} catch (error) {
+		recordResult(
+			"transactions.getTransactions",
+			"getTransactions(budgetId, since?, type?, lastKnowledge?)",
+			false,
+			"",
+			error,
+		);
+	}
+
+	console.log("\nSummary:\n");
+	for (const row of results) {
+		console.log(
+			`${row.supported ? "✓" : "✗"} ${row.name} -> ${row.supported ? "supports delta" : "no delta"} (${row.signature})`,
+		);
+	}
 }
 
 main().catch((error) => {
-  console.error('Verification failed:', error);
-  process.exit(1);
+	console.error("Verification failed:", error);
+	process.exit(1);
 });

package/scripts/test-recommendations.ts

@@ -3,8 +3,8 @@
  * Tests the EvoCarShare scenario from user feedback
  */
 
-import { analyzeReconciliation } from '../src/tools/reconciliation/analyzer.js';
-import { DEFAULT_CONFIG as DEFAULT_MATCHING_CONFIG } from '../src/tools/reconciliation/matcher.js';
+import { analyzeReconciliation } from "../src/tools/reconciliation/analyzer.js";
+import { DEFAULT_CONFIG as DEFAULT_MATCHING_CONFIG } from "../src/tools/reconciliation/matcher.js";
 
 // Test data from user's scenario
 const csvContent = `Date,Description,Amount
@@ -13,41 +13,45 @@ const csvContent = `Date,Description,Amount
 const ynabTransactions = []; // No YNAB transactions
 
 const analysis = analyzeReconciliation(
-  csvContent,
-  undefined,
-  ynabTransactions,
-  122.22, // Statement balance
-  DEFAULT_MATCHING_CONFIG,
-  'USD',
-  'test-account',
-  'test-budget',
+	csvContent,
+	undefined,
+	ynabTransactions,
+	122.22, // Statement balance
+	DEFAULT_MATCHING_CONFIG,
+	"USD",
+	"test-account",
+	"test-budget",
 );
 
-console.log('=== RECONCILIATION ANALYSIS ===');
+console.log("=== RECONCILIATION ANALYSIS ===");
 console.log(`On Track: ${analysis.balance_info.on_track}`);
 console.log(`Discrepancy: ${analysis.balance_info.discrepancy.value_display}`);
-console.log(`\n=== RECOMMENDATIONS (${analysis.recommendations?.length || 0}) ===`);
+console.log(
+	`\n=== RECOMMENDATIONS (${analysis.recommendations?.length || 0}) ===`,
+);
 
 if (analysis.recommendations && analysis.recommendations.length > 0) {
-  for (const rec of analysis.recommendations) {
-    console.log(`\n[${rec.priority.toUpperCase()}] ${rec.message}`);
-    console.log(`  Type: ${rec.action_type}`);
-    console.log(`  Confidence: ${(rec.confidence * 100).toFixed(0)}%`);
-    console.log(`  Reason: ${rec.reason}`);
-    console.log(`  Impact: ${rec.estimated_impact.value_display}`);
-
-    if (rec.action_type === 'create_transaction') {
-      console.log(`  Parameters:`);
-      console.log(`    - Date: ${rec.parameters.date}`);
-      console.log(`    - Amount: $${(rec.parameters.amount / 1000).toFixed(2)}`);
-      console.log(`    - Payee: ${rec.parameters.payee_name}`);
-      console.log(`    - Cleared: ${rec.parameters.cleared}`);
-    }
-  }
-
-  console.log('\n✅ All checks passed!');
-  console.log('Recommendation engine is working correctly.');
+	for (const rec of analysis.recommendations) {
+		console.log(`\n[${rec.priority.toUpperCase()}] ${rec.message}`);
+		console.log(`  Type: ${rec.action_type}`);
+		console.log(`  Confidence: ${(rec.confidence * 100).toFixed(0)}%`);
+		console.log(`  Reason: ${rec.reason}`);
+		console.log(`  Impact: ${rec.estimated_impact.value_display}`);
+
+		if (rec.action_type === "create_transaction") {
+			console.log("  Parameters:");
+			console.log(`    - Date: ${rec.parameters.date}`);
+			console.log(
+				`    - Amount: $${(rec.parameters.amount / 1000).toFixed(2)}`,
+			);
+			console.log(`    - Payee: ${rec.parameters.payee_name}`);
+			console.log(`    - Cleared: ${rec.parameters.cleared}`);
+		}
+	}
+
+	console.log("\n✅ All checks passed!");
+	console.log("Recommendation engine is working correctly.");
 } else {
-  console.log('❌ No recommendations generated!');
-  process.exit(1);
+	console.log("❌ No recommendations generated!");
+	process.exit(1);
 }