@dizzlkheinz/ynab-mcpb 0.18.3 → 0.19.0

package/src/types/index.ts

@@ -3,68 +3,72 @@
  */
 
 export interface AuthenticationConfig {
-  accessToken: string;
-  validateToken(): Promise<boolean>;
+	accessToken: string;
+	validateToken(): Promise<boolean>;
 }
 
 export interface ServerConfig {
-  accessToken: string;
-  defaultBudgetId?: string;
+	accessToken: string;
+	defaultBudgetId?: string;
 }
 
 export class AuthenticationError extends Error {
-  constructor(message: string) {
-    super(message);
-    this.name = 'AuthenticationError';
-  }
+	constructor(message: string) {
+		super(message);
+		this.name = "AuthenticationError";
+	}
 }
 
 export class ConfigurationError extends Error {
-  constructor(message: string) {
-    super(message);
-    this.name = 'ConfigurationError';
-  }
+	constructor(message: string) {
+		super(message);
+		this.name = "ConfigurationError";
+	}
 }
 
 // Re-export error handling types for convenience
 export {
-  ErrorHandler,
-  YNABAPIError,
-  ValidationError,
-  YNABErrorCode,
-  SecurityErrorCode,
-  type ErrorResponse,
-  handleToolError,
-  withToolErrorHandling,
-} from '../server/errorHandler.js';
+	ErrorHandler,
+	YNABAPIError,
+	ValidationError,
+	YNABErrorCode,
+	SecurityErrorCode,
+	type ErrorResponse,
+	handleToolError,
+	withToolErrorHandling,
+} from "../server/errorHandler.js";
 
 // Re-export security modules
 export {
-  RateLimiter,
-  RateLimitError,
-  globalRateLimiter,
-  type RateLimitConfig,
-  type RateLimitInfo,
-} from '../server/rateLimiter.js';
+	RateLimiter,
+	RateLimitError,
+	globalRateLimiter,
+	type RateLimitConfig,
+	type RateLimitInfo,
+} from "../server/rateLimiter.js";
 
 export {
-  RequestLogger,
-  globalRequestLogger,
-  type LogEntry,
-  type LoggerConfig,
-} from '../server/requestLogger.js';
+	RequestLogger,
+	globalRequestLogger,
+	type LogEntry,
+	type LoggerConfig,
+} from "../server/requestLogger.js";
 
 export {
-  SecurityMiddleware,
-  withSecurityWrapper,
-  type SecurityContext,
-} from '../server/securityMiddleware.js';
+	SecurityMiddleware,
+	withSecurityWrapper,
+	type SecurityContext,
+} from "../server/securityMiddleware.js";
 
 // Re-export tool annotation types
-export type { MCPToolAnnotations } from './toolAnnotations.js';
+export type { MCPToolAnnotations } from "./toolAnnotations.js";
 
 // Re-export tool registry types for convenience
-export type { ToolDefinition } from '../server/toolRegistry.js';
+export type { ToolDefinition } from "../server/toolRegistry.js";
 
 // Re-export tool registration factory types
-export type { ToolContext, ToolFactory, BudgetIdResolverFactory } from './toolRegistration.js';
+export type {
+	ToolContext,
+	ToolFactory,
+	BudgetIdResolverFactory,
+} from "./toolRegistration.js";

package/src/types/integration-tests.d.ts

@@ -1,35 +1,35 @@
-type IntegrationTestTier = 'core' | 'domain' | 'full';
+type IntegrationTestTier = "core" | "domain" | "full";
 type IntegrationTestDomain =
-  | 'budgets'
-  | 'accounts'
-  | 'transactions'
-  | 'categories'
-  | 'payees'
-  | 'months'
-  | 'delta'
-  | 'reconciliation'
-  | 'utility'
-  | 'security'
-  | 'server'
-  | 'workflows'
-  | (string & {});
+	| "budgets"
+	| "accounts"
+	| "transactions"
+	| "categories"
+	| "payees"
+	| "months"
+	| "delta"
+	| "reconciliation"
+	| "utility"
+	| "security"
+	| "server"
+	| "workflows"
+	| (string & {});
 
 export interface IntegrationTestMeta {
-  tier: IntegrationTestTier;
-  domain: IntegrationTestDomain;
+	tier: IntegrationTestTier;
+	domain: IntegrationTestDomain;
 }
 
-declare module '@vitest/runner' {
-  // eslint-disable-next-line @typescript-eslint/no-empty-object-type
-  interface TaskMeta extends Partial<IntegrationTestMeta> {}
+declare module "@vitest/runner" {
+	// eslint-disable-next-line @typescript-eslint/no-empty-object-type
+	interface TaskMeta extends Partial<IntegrationTestMeta> {}
 
-  interface TestOptions {
-    meta?: IntegrationTestMeta;
-  }
+	interface TestOptions {
+		meta?: IntegrationTestMeta;
+	}
 }
 
-declare module 'vitest' {
-  interface TestOptions {
-    meta?: IntegrationTestMeta;
-  }
+declare module "vitest" {
+	interface TestOptions {
+		meta?: IntegrationTestMeta;
+	}
 }

package/src/types/reconciliation.ts

@@ -5,26 +5,26 @@
  * This matches YNAB's native format and allows exact integer comparison.
  */
 export interface BankTransaction {
-  /** UUID generated for tracking */
-  id: string;
-  /** ISO date string YYYY-MM-DD */
-  date: string;
-  /** Amount in MILLIUNITS (negative = outflow, positive = inflow) */
-  amount: number;
-  /** Merchant/payee name from CSV */
-  payee: string;
-  /** Optional memo/description */
-  memo?: string;
-  /** Original CSV row number (1-indexed, after header) */
-  sourceRow: number;
-  /** Raw values from CSV for debugging */
-  raw: {
-    date: string;
-    amount: string;
-    description: string;
-  };
-  /** Parser warnings (e.g., ambiguous debit/credit) */
-  warnings?: string[];
+	/** UUID generated for tracking */
+	id: string;
+	/** ISO date string YYYY-MM-DD */
+	date: string;
+	/** Amount in MILLIUNITS (negative = outflow, positive = inflow) */
+	amount: number;
+	/** Merchant/payee name from CSV */
+	payee: string;
+	/** Optional memo/description */
+	memo?: string;
+	/** Original CSV row number (1-indexed, after header) */
+	sourceRow: number;
+	/** Raw values from CSV for debugging */
+	raw: {
+		date: string;
+		amount: string;
+		description: string;
+	};
+	/** Parser warnings (e.g., ambiguous debit/credit) */
+	warnings?: string[];
 }
 
 /**
@@ -37,13 +37,13 @@ export interface BankTransaction {
  * No conversion needed from the SDK.
  */
 export interface NormalizedYNABTransaction {
-  id: string;
-  date: string; // YYYY-MM-DD
-  /** Amount in MILLIUNITS (same as YNAB API) */
-  amount: number;
-  payee: string | null;
-  memo: string | null;
-  categoryName: string | null;
-  cleared: 'cleared' | 'uncleared' | 'reconciled';
-  approved: boolean;
+	id: string;
+	date: string; // YYYY-MM-DD
+	/** Amount in MILLIUNITS (same as YNAB API) */
+	amount: number;
+	payee: string | null;
+	memo: string | null;
+	categoryName: string | null;
+	cleared: "cleared" | "uncleared" | "reconciled";
+	approved: boolean;
 }

package/src/types/toolAnnotations.ts

@@ -7,38 +7,38 @@
  * @see https://blog.marcnuri.com/mcp-tool-annotations-introduction
  */
 export interface MCPToolAnnotations {
-  /**
-   * Human-readable title for UI display
-   *
-   * @example "YNAB: Delete Transaction"
-   */
-  title?: string;
+	/**
+	 * Human-readable title for UI display
+	 *
+	 * @example "YNAB: Delete Transaction"
+	 */
+	title?: string;
 
-  /**
-   * Indicates tool only reads data without modifications
-   *
-   * @example true for list_budgets, get_account
-   */
-  readOnlyHint?: boolean;
+	/**
+	 * Indicates tool only reads data without modifications
+	 *
+	 * @example true for list_budgets, get_account
+	 */
+	readOnlyHint?: boolean;
 
-  /**
-   * For non-read-only tools, indicates irreversible operations
-   *
-   * @example true for delete_transaction
-   */
-  destructiveHint?: boolean;
+	/**
+	 * For non-read-only tools, indicates irreversible operations
+	 *
+	 * @example true for delete_transaction
+	 */
+	destructiveHint?: boolean;
 
-  /**
-   * Indicates repeated identical calls have same effect
-   *
-   * @example true for set_default_budget
-   */
-  idempotentHint?: boolean;
+	/**
+	 * Indicates repeated identical calls have same effect
+	 *
+	 * @example true for set_default_budget
+	 */
+	idempotentHint?: boolean;
 
-  /**
-   * Indicates tool interacts with external systems like YNAB API
-   *
-   * @example true for all YNAB tools that call the YNAB API
-   */
-  openWorldHint?: boolean;
+	/**
+	 * Indicates tool interacts with external systems like YNAB API
+	 *
+	 * @example true for all YNAB tools that call the YNAB API
+	 */
+	openWorldHint?: boolean;
 }

package/src/types/toolRegistration.ts

@@ -5,18 +5,19 @@
  * @module types/toolRegistration
  */
 
-import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
-import * as ynab from 'ynab';
+import type { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
+import type * as ynab from "ynab";
+import type { CacheManager } from "../server/cacheManager.js";
+import type { DeltaCache } from "../server/deltaCache.js";
+import type { DiagnosticManager } from "../server/diagnostics.js";
+import type { ErrorHandler } from "../server/errorHandler.js";
+import type { ServerKnowledgeStore } from "../server/serverKnowledgeStore.js";
 import type {
-  ToolRegistry,
-  ToolExecutionPayload,
-  DefaultArgumentResolver,
-} from '../server/toolRegistry.js';
-import type { DeltaFetcher } from '../tools/deltaFetcher.js';
-import type { DeltaCache } from '../server/deltaCache.js';
-import type { ServerKnowledgeStore } from '../server/serverKnowledgeStore.js';
-import type { DiagnosticManager } from '../server/diagnostics.js';
-import type { CacheManager } from '../server/cacheManager.js';
+	DefaultArgumentResolver,
+	ToolExecutionPayload,
+	ToolRegistry,
+} from "../server/toolRegistry.js";
+import type { DeltaFetcher } from "../tools/deltaFetcher.js";
 
 /**
  * Context object passed to tool factory functions. Contains the dependencies
@@ -26,63 +27,73 @@ import type { CacheManager } from '../server/cacheManager.js';
  * Changes to this interface may affect all domain tool factories.
  */
 export interface ToolContext {
-  ynabAPI: ynab.API;
-  deltaFetcher: DeltaFetcher;
-  deltaCache: DeltaCache;
-  serverKnowledgeStore: ServerKnowledgeStore;
-  getDefaultBudgetId: () => string | undefined;
-  setDefaultBudget: (budgetId: string) => void;
-  cacheManager: CacheManager;
-  diagnosticManager?: DiagnosticManager;
+	ynabAPI: ynab.API;
+	deltaFetcher: DeltaFetcher;
+	deltaCache: DeltaCache;
+	serverKnowledgeStore: ServerKnowledgeStore;
+	getDefaultBudgetId: () => string | undefined;
+	setDefaultBudget: (budgetId: string) => void;
+	cacheManager: CacheManager;
+	errorHandler: ErrorHandler;
+	diagnosticManager?: DiagnosticManager;
 }
 
 /**
  * Factory function signature for registering a domain's tools.
  */
-export type ToolFactory = (registry: ToolRegistry, context: ToolContext) => void;
+export type ToolFactory = (
+	registry: ToolRegistry,
+	context: ToolContext,
+) => void;
 
 /**
  * Common adapter signature used within tool factories.
  */
 export type Adapter<TInput extends Record<string, unknown>> = (
-  payload: ToolExecutionPayload<TInput>,
+	payload: ToolExecutionPayload<TInput>,
 ) => Promise<CallToolResult>;
 
 /**
  * Generic handler signature used by adapter helpers.
  */
 export type Handler<TInput extends Record<string, unknown>> = (
-  api: ynab.API,
-  params: TInput,
+	api: ynab.API,
+	params: TInput,
+	errorHandler?: ErrorHandler,
 ) => Promise<CallToolResult>;
 
 /**
  * Handler signature for operations that require delta fetching.
  */
 export type DeltaHandler<TInput extends Record<string, unknown>> = (
-  api: ynab.API,
-  deltaFetcher: DeltaFetcher,
-  params: TInput,
+	api: ynab.API,
+	deltaFetcher: DeltaFetcher,
+	params: TInput,
+	errorHandler?: ErrorHandler,
 ) => Promise<CallToolResult>;
 
 /**
  * Handler signature for write operations that update caches and knowledge stores.
  */
 export type WriteHandler<TInput extends Record<string, unknown>> = (
-  api: ynab.API,
-  deltaCache: DeltaCache,
-  serverKnowledgeStore: ServerKnowledgeStore,
-  params: TInput,
+	api: ynab.API,
+	deltaCache: DeltaCache,
+	serverKnowledgeStore: ServerKnowledgeStore,
+	params: TInput,
+	errorHandler?: ErrorHandler,
 ) => Promise<CallToolResult>;
 
 /**
  * Handler signature for tools that do not accept input parameters.
  */
-export type NoInputHandler = (api: ynab.API) => Promise<CallToolResult>;
+export type NoInputHandler = (
+	api: ynab.API,
+	errorHandler?: ErrorHandler,
+) => Promise<CallToolResult>;
 
 /**
  * Helper type for default argument resolver factories.
  */
 export type BudgetIdResolverFactory = <
-  TInput extends { budget_id?: string | undefined },
+	TInput extends { budget_id?: string | undefined },
 >() => DefaultArgumentResolver<TInput>;