@dizzlkheinz/ynab-mcpb 0.17.0 → 0.18.0

package/.code/agents/94a0ddf3-a304-4ec3-913e-3cceef509948/error.txt

@@ -1 +0,0 @@
-Failed to spawn sandboxed agent: batch file arguments are invalid

package/.code/agents/95d9fbab-19a2-48af-83f9-c792566a347f/error.txt

@@ -1 +0,0 @@
-Failed to execute code-gpt-5.1-codex-mini: program not found

package/.code/agents/b0098cb8-cb32-4ada-9bc4-37c587518896/result.txt

@@ -1,170 +0,0 @@
-I'm using the writing-plans skill to create the implementation plan.
-
-# MCP Resilience Implementation Plan
-
-> **For Claude:** REQUIRED SUB-SKILL: Use superpowers:executing-plans to implement this plan task-by-task.
-
-**Goal:** Harden the MCP server by validating every tool’s metadata schema and making `clear_cache` fully reset delta/caching state so knowledge doesn’t drift while keeping diagnostics accurate.
-
-**Architecture:** Validate metadata against generated `zod` schemas inside `ToolRegistry`, expose mismatches immediately, and delegate cache clearing to `DeltaCache.forceFullRefresh()` so both the LRU cache and `ServerKnowledgeStore` reset before diagnostics observe the new state.
-
-**Tech Stack:** TypeScript 5/Node 18+, `zod`/`zod-validation-error`, MCP SDK server wiring in `src/server`, Vitest for fast unit/integration suites, and Markdown docs in `docs/reference/TOOLS.md`.
-
-### Task 1: Tighten tool metadata schema validation
-
-**Files:**
-- Modify: `src/server/toolRegistry.ts`
-- Modify: `src/server/__tests__/toolRegistry.test.ts`
-
-**Step 1: Write the failing test**
-
-```ts
-it('rejects registration when metadata inputJsonSchema diverges from the handler schema', () => {
-  const registry = new ToolRegistry({
-    withSecurityWrapper: vi.fn().mockReturnValue(() => async (handler) => handler({})),
-    errorHandler: mockedErrorHandler,
-    responseFormatter: responseFormatter,
-  });
-  const mismatchedMetadata = { type: 'object', properties: { bogus: { type: 'string' } } };
-  expect(() =>
-    registry.register({
-      name: 'budget-sync',
-      description: 'Sync budget data',
-      inputSchema: z.object({ budget_id: z.string().min(1) }).strict(),
-      handler: async () => ({ content: [{ type: 'text', text: responseFormatter.format({}) }] }),
-      metadata: { inputJsonSchema: mismatchedMetadata },
-    }),
-  ).toThrow('metadata input schema must match generated schema for tool "budget-sync"');
-});
-```
-
-**Step 2: Run the test to verify it fails**
-
-Run: `npm run test:unit -- --runInBand src/server/__tests__/toolRegistry.test.ts --testNamePattern "metadata input schema"`
-Expected: FAIL because the registry currently accepts mismatched metadata.
-
-**Step 3: Implement the minimal enforcement**
-
-```ts
-private ensureMetadataSchemaMatches(definition: ToolDefinition): void {
-  const generatedSchema = this.generateJsonSchema(definition.inputSchema, 'input');
-  if (definition.metadata?.inputJsonSchema) {
-    if (JSON.stringify(generatedSchema) !== JSON.stringify(definition.metadata.inputJsonSchema)) {
-      throw new Error(
-        `metadata input schema must match generated schema for tool "${definition.name}"`,
-      );
-    }
-  } else {
-    definition.metadata = {
-      ...definition.metadata,
-      inputJsonSchema: generatedSchema,
-    };
-  }
-}
-```
-
-Call `this.ensureMetadataSchemaMatches(definition);` near the top of `register` before storing the tool so `metadata.inputJsonSchema` is always populated (and validated) for `listTools()`. This also means `ToolRegistry.listTools()` no longer needs to fall back to `generateJsonSchema` when metadata is present.
-
-**Step 4: Run the test to confirm it now passes**
-
-Run: `npm run test:unit -- --runInBand src/server/__tests__/toolRegistry.test.ts --testNamePattern "metadata input schema"`
-Expected: PASS.
-
-**Step 5: Commit**
-
-```bash
-git add src/server/toolRegistry.ts src/server/__tests__/toolRegistry.test.ts
-git commit -m "fix: validate tool metadata schemas at registration"
-```
-
-### Task 2: Reset delta knowledge when clearing the cache
-
-**Files:**
-- Modify: `src/server/YNABMCPServer.ts`
-- Modify: `src/server/__tests__/YNABMCPServer.integration.test.ts`
-- Modify: `docs/reference/TOOLS.md`
-
-**Step 1: Write the failing integration test**
-
-```ts
-it(
-  'resets delta knowledge entries when clear_cache runs',
-  { meta: { tier: 'domain', domain: 'server' } },
-  async (ctx) => {
-    await skipOnRateLimit(async () => {
-      const token = accessToken();
-      await registry.executeTool({
-        name: 'list_accounts',
-        accessToken: token,
-        arguments: { budget_id: budgetsResult?.budgets?.[0]?.id },
-      });
-
-      const before = await registry.executeTool({
-        name: 'diagnostic_info',
-        accessToken: token,
-        arguments: { include_delta: true },
-      });
-      const beforeData = JSON.parse(before.content?.[0]?.text ?? '{}');
-      expect(beforeData.delta?.knowledge_entries).toBeGreaterThan(0);
-
-      await registry.executeTool({ name: 'clear_cache', accessToken: token, arguments: {} });
-
-      const after = await registry.executeTool({
-        name: 'diagnostic_info',
-        accessToken: token,
-        arguments: { include_delta: true },
-      });
-      const afterData = JSON.parse(after.content?.[0]?.text ?? '{}');
-      expect(afterData.delta?.knowledge_entries).toBe(0);
-    }, ctx);
-  },
-);
-```
-
-**Step 2: Run the integration test to verify it fails**
-
-Run: `npm run test:integration:domain -- --testNamePattern "clear cache runs"`
-Expected: FAIL because `clear_cache` currently only clears `cacheManager`, leaving `ServerKnowledgeStore` populated.
-
-**Step 3: Implement the minimal fix**
-
-- Update the `clear_cache` handler in `setupToolRegistry` to call `this.deltaCache.forceFullRefresh()` instead of directly `cacheManager.clear()`, ensuring both the cache and knowledge store reset:
-
-```ts
-handler: async () => {
-  this.deltaCache.forceFullRefresh();
-  return {
-    content: [{ type: 'text', text: responseFormatter.format({ success: true }) }],
-  };
-},
-```
-
-- Update `docs/reference/TOOLS.md` entry to explain the stronger guarantee:
-
-```md
-| `clear_cache` | Clear the cache and reset delta knowledge so clients can start fresh | None |
-```
-
-**Step 4: Run the integration test to confirm it passes**
-
-Run: `npm run test:integration:domain -- --testNamePattern "clear cache runs"`
-Expected: PASS.
-
-**Step 5: Commit**
-
-```bash
-git add src/server/YNABMCPServer.ts src/server/__tests__/YNABMCPServer.integration.test.ts docs/reference/TOOLS.md
-git commit -m "feat: reset delta knowledge when cache is cleared"
-```
-
----
-
-Because the sandbox is read-only, I couldn’t save this plan to `docs/plans/2025-11-21-mcp-resilience.md`; please copy the above Markdown there in a workspace-write session before executing the tasks.
-
-- **Plan scope:** Adds schema validation in `src/server/toolRegistry.ts` plus a new regression test in `src/server/__tests__/toolRegistry.test.ts`, and modifies `clear_cache` in `src/server/YNABMCPServer.ts` with an integration test addition and doc update in `docs/reference/TOOLS.md`.
-- **Testing:** Run the targeted unit test via `npm run test:unit -- --runInBand src/server/__tests__/toolRegistry.test.ts --testNamePattern "metadata input schema"` and the integration slice via `npm run test:integration:domain -- --testNamePattern "clear cache runs"` after making the code changes.
-
-Next steps:
-1. Execute the described plan in a writable worktree (create `docs/plans/2025-11-21-mcp-resilience.md` with the above content first) and implement Task 1’s schema validation.
-2. Run the targeted unit test from Task 1 to confirm failure, then fix the code, rerun to confirm success.
-3. Implement Task 2, add the integration test, update `clear_cache`, refresh `docs/reference/TOOLS.md`, and run `npm run test:integration:domain -- --testNamePattern "clear cache runs"`.

package/.code/agents/b4fe59a4-81df-42e2-a112-0153e504faca/error.txt

@@ -1 +0,0 @@
-Required agent 'gemini' is not installed or not in PATH

package/.code/agents/bf4ce152-f623-49d7-aa52-c18631625c3c/error.txt

@@ -1,3 +0,0 @@
-Command failed: Error loading configuration: config profile `Smoke test: from the current repo, run `pwd` and `git status -sb`, then list top-level files/directories and print a short summary. No file writes or edits.
-
-[Running in read-only mode - no modifications allowed]` not found

package/.code/agents/d7d1db75-d7eb-468e-adea-4ef4d916d187/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/.code/agents/e2baa9c8-bac3-49e3-a39d-024333e6a990/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/.code/agents/e2c752b7-711d-423a-af57-f53c809deb84/result.txt

@@ -1,160 +0,0 @@
-Here’s a concise blueprint for Phase 1 that matches the factory pattern doc and repo conventions.
-
-**Module layout (production-ready, minimal)**
-- `src/types/toolRegistration.ts`: canonical types/interfaces for tool definitions and adapters.
-- `src/tools/adapters/index.ts`: adapter factory functions for the five categories (adapt, adaptNoInput, adaptWithDelta, adaptWrite, inline); export typed helpers.
-- `src/tools/schemas/common.ts`: shared zod schemas for inputs/outputs/memo/delta, reusable across tools.
-- `src/tools/index.ts`: re-export adapters and shared schemas (and future tool registrations).
-- Tests: `src/__tests__/tools/adapters.test.ts` (unit, Vitest).
-
-**Core types (toolRegistration.ts)**
-```ts
-import { z } from 'zod';
-
-export type ToolKind =
-  | 'adapt'          // input schema -> handler(args) -> output
-  | 'adaptNoInput'   // no input -> handler() -> output
-  | 'adaptWithDelta' // input + delta/metadata -> output + delta?
-  | 'adaptWrite'     // input triggers mutation; output includes status
-  | 'inline';        // small inline tools (no registration factory)
-
-export interface ToolContext {
-  logger?: { debug?: (...args: any[]) => void; info?: (...args: any[]) => void; error?: (...args: any[]) => void };
-  now?: () => Date;
-  // extend later with cache, env, config
-}
-
-export type ToolHandler<Input, Output> = (args: {
-  input: Input;
-  context?: ToolContext;
-}) => Promise<Output> | Output;
-
-export type ToolHandlerNoInput<Output> = (args?: { context?: ToolContext }) => Promise<Output> | Output;
-
-export interface ToolRegistration<Input, Output> {
-  id: string; // kebab-case tool id
-  description: string;
-  inputSchema: z.ZodType<Input>;
-  outputSchema?: z.ZodType<Output>;
-  handler: ToolHandler<Input, Output>;
-  kind?: ToolKind; // optional hint for tooling/metrics
-}
-
-export interface ToolRegistrationNoInput<Output>
-  extends Omit<ToolRegistration<undefined, Output>, 'inputSchema' | 'handler'> {
-  handler: ToolHandlerNoInput<Output>;
-}
-```
-
-**Shared schemas (schemas/common.ts)**
-```ts
-import { z } from 'zod';
-
-export const memoSchema = z.string().max(500).optional();
-export const baseInputSchema = z.object({ memo: memoSchema }).strip();
-export const deltaSchema = z.object({
-  deltaToken: z.string(),
-  revision: z.number().int().nonnegative(),
-});
-export const statusSchema = z.object({
-  ok: z.boolean(),
-  message: z.string().optional(),
-});
-```
-
-**Adapters (tools/adapters/index.ts)**
-```ts
-import { z } from 'zod';
-import {
-  ToolRegistration,
-  ToolRegistrationNoInput,
-  ToolHandler,
-  ToolHandlerNoInput,
-} from '../../types/toolRegistration';
-
-type AdapterResult<Input, Output> = ToolRegistration<Input, Output>;
-
-export function adapt<Input, Output>(
-  config: Omit<ToolRegistration<Input, Output>, 'kind'>,
-): AdapterResult<Input, Output> {
-  return { ...config, kind: 'adapt' };
-}
-
-export function adaptNoInput<Output>(
-  config: Omit<ToolRegistrationNoInput<Output>, 'kind' | 'inputSchema'>,
-): AdapterResult<undefined, Output> {
-  return {
-    ...config,
-    inputSchema: z.void(),
-    kind: 'adaptNoInput',
-  };
-}
-
-export function adaptWithDelta<Input, Output>(
-  config: Omit<ToolRegistration<Input, Output>, 'kind'>,
-): AdapterResult<Input, Output> {
-  return { ...config, kind: 'adaptWithDelta' };
-}
-
-export function adaptWrite<Input, Output>(
-  config: Omit<ToolRegistration<Input, Output>, 'kind'>,
-): AdapterResult<Input, Output> {
-  return { ...config, kind: 'adaptWrite' };
-}
-
-// inline: kept simple; bypasses factory if needed, but helper for consistency
-export function inline<Input, Output>(
-  config: ToolRegistration<Input, Output>,
-): AdapterResult<Input, Output> {
-  return { ...config, kind: 'inline' };
-}
-
-export type {
-  ToolRegistration,
-  ToolRegistrationNoInput,
-  ToolHandler,
-  ToolHandlerNoInput,
-};
-```
-
-**Exports (src/tools/index.ts)**
-```ts
-export * from './adapters';
-export * as commonSchemas from './schemas/common';
-```
-
-**Unit test plan (src/__tests__/tools/adapters.test.ts)**
-- `adapt`: returns config with `kind: 'adapt'`, preserves handler and schema identity, and is type-safe for input/output.
-- `adaptNoInput`: sets `inputSchema` to `z.void()`, `kind: 'adaptNoInput'`, handler invoked without input, rejects passing unexpected input at runtime (zod parse).
-- `adaptWithDelta`: attaches `kind: 'adaptWithDelta'`, works with a schema that includes delta fields (e.g., merged input + `deltaSchema`).
-- `adaptWrite`: sets `kind: 'adaptWrite'`, supports output schema with status; ensure no mutation to original config.
-- `inline`: passes through config and sets `kind: 'inline'`.
-- Shared behaviors: returned objects are new instances (no mutation), required fields (`id`, `description`, `handler`) are preserved.
-- Validation snapshots: optional quick zod parse test per adapter (when schema is provided) to prove wiring.
-
-**Test scaffolding snippet**
-```ts
-import { describe, it, expect } from 'vitest';
-import { z } from 'zod';
-import {
-  adapt,
-  adaptNoInput,
-  adaptWithDelta,
-  adaptWrite,
-  inline,
-} from '../../tools/adapters';
-import { deltaSchema, statusSchema } from '../../tools/schemas/common';
-
-describe('tool adapters', () => {
-  it('adapt sets kind and preserves config', async () => { /* ... */ });
-  it('adaptNoInput sets z.void and kind', async () => { /* ... */ });
-  it('adaptWithDelta supports delta schema', async () => { /* ... */ });
-  it('adaptWrite marks kind adaptWrite', async () => { /* ... */ });
-  it('inline marks kind inline', async () => { /* ... */ });
-});
-```
-
-**Notes**
-- Keep adapters thin; no side effects. `kind` supports metrics/routing; callers can branch on it.
-- Shared schemas stay minimal; expand as new tools land.
-- Maintain kebab-case `id` per repo guidance; single export surface via `src/tools/index.ts`.

package/.code/agents/e350b8c3-8483-408c-b2bb-94515f492a11/error.txt

@@ -1,3 +0,0 @@
-Command failed: Error loading configuration: config profile `You are helping craft an implementation plan for the YNAB MCP Server repo (Model Context Protocol server for YNAB). Repo root at C:/Users/ksutk/projects/ynab-mcpb. Key info: TypeScript/Node 18+, ts source in src/, server wiring in src/server, tools in src/tools, utils in src/utils, entry at src/index.ts. Build: npm run build (lint+tsc+bundle) outputs dist/. Tests use Vitest with projects (unit, integration, e2e). package.json scripts include build:prod, lint:fix, test:unit, test:integration, test:e2e, package:mcpb. Docs in docs/. Current version 0.13.4. Goal: propose a concrete step-by-step implementation plan for the next iteration to improve the project. Include where to change code (paths), suggested tests to add/run, and rationale. Assume current tests passing (test-results show success). Focus on actionable tasks (2-5 min steps) using TDD mindset. Deliver plan structure we can hand to engineer. Do not assume write access.
-
-[Running in read-only mode - no modifications allowed]` not found

package/.code/agents/e63f9919-719f-4ad0-bccf-01b1a596e1e9/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/.code/agents/e6601719-c31f-4a0e-8c71-d70787d0ab71/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/.code/agents/e71695a8-3044-478d-8f12-ed13d02884c7/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/.code/agents/f250b7ed-5bd5-4036-aa8c-ce63caee7d61/result.txt

@@ -1,20 +0,0 @@
-**MCP Tool Catalogue**
-- **Budgets** (`src/server/YNABMCPServer.ts:467-540`): `list_budgets` (delta adapter; input inline `emptyObjectSchema`; output `ListBudgetsOutputSchema`), `get_budget` (read adapter; input `GetBudgetSchema` from `src/tools/budgetTools.ts`; output `GetBudgetOutputSchema`), `set_default_budget` (inline server write; input inline `setDefaultBudgetSchema`; output `SetDefaultBudgetOutputSchema`), `get_default_budget` (inline server read; input inline `emptyObjectSchema`; output `GetDefaultBudgetOutputSchema`).
-- **Accounts** (`src/server/YNABMCPServer.ts:573-615`): `list_accounts` (delta; input `ListAccountsSchema`; output `ListAccountsOutputSchema`), `get_account` (read; input `GetAccountSchema`; output `GetAccountOutputSchema`), `create_account` (write; input `CreateAccountSchema`; output inline `LooseObjectSchema`).
-- **Transactions – read/delta** (`src/server/YNABMCPServer.ts:618-692`): `list_transactions` (delta; input `ListTransactionsSchema`; output `LooseObjectSchema`), `export_transactions` (read; input `ExportTransactionsSchema`; output `ExportTransactionsOutputSchema`), `compare_transactions` (read; input `CompareTransactionsSchema`; output `CompareTransactionsOutputSchema`), `reconcile_account` (delta; input `ReconcileAccountSchema`; output `LooseObjectSchema`), `get_transaction` (read; input `GetTransactionSchema`; output `GetTransactionOutputSchema`).
-- **Transactions – write** (`src/server/YNABMCPServer.ts:694-785`): `create_transaction`, `create_transactions`, `create_receipt_split_transaction` (all write adapters; inputs `CreateTransactionSchema`/`CreateTransactionsSchema`/`CreateReceiptSplitTransactionSchema`; outputs `LooseObjectSchema`), `update_transaction`, `update_transactions` (write; inputs `UpdateTransactionSchema`/`UpdateTransactionsSchema`; outputs `LooseObjectSchema`), `delete_transaction` (write; input `DeleteTransactionSchema`; output `LooseObjectSchema`).
-- **Categories** (`src/server/YNABMCPServer.ts:787-830`): `list_categories` (delta; input `ListCategoriesSchema`; output `ListCategoriesOutputSchema`), `get_category` (read; input `GetCategorySchema`; output `GetCategoryOutputSchema`), `update_category` (write; input `UpdateCategorySchema`; output `LooseObjectSchema`).
-- **Payees** (`src/server/YNABMCPServer.ts:832-860`): `list_payees` (delta; input `ListPayeesSchema`; output `ListPayeesOutputSchema`), `get_payee` (read; input `GetPayeeSchema`; output `GetPayeeOutputSchema`).
-- **Months** (`src/server/YNABMCPServer.ts:862-889`): `get_month` (read; input `GetMonthSchema`; output `GetMonthOutputSchema`), `list_months` (delta; input `ListMonthsSchema`; output `ListMonthsOutputSchema`).
-- **Utilities/Local** (`src/server/YNABMCPServer.ts:892-998`): `get_user` (no-input adapter; input inline `emptyObjectSchema`; output `GetUserOutputSchema`), `convert_amount` (inline local handler; input `ConvertAmountSchema` from `src/tools/utilityTools.ts`; output `ConvertAmountOutputSchema`), `diagnostic_info` (inline server; input inline `diagnosticInfoSchema`; output `DiagnosticInfoOutputSchema`), `clear_cache` (inline server; input inline `emptyObjectSchema`; output `ClearCacheOutputSchema`), `set_output_format` (inline server; input inline `setOutputFormatSchema`; output `SetOutputFormatOutputSchema`).
-
-**Inline Schema Observations**
-- Inline definitions in `src/server/YNABMCPServer.ts:446-465` (`emptyObjectSchema`, `LooseObjectSchema`, `setDefaultBudgetSchema`, `diagnosticInfoSchema`, `setOutputFormatSchema`) are reused across tools and could move to a shared schema module (e.g., `src/tools/schemas/common.ts`) to reduce duplication and keep registrations declarative.
-- Most tools already pull input/output schemas from feature modules (`src/tools/*` and `src/tools/schemas/outputs/index.ts`); only the inline schemas above break that pattern.
-
-**Server-Owned Inline Tools to Keep Inline**
-- `set_default_budget`, `get_default_budget`, `diagnostic_info`, `clear_cache`, and `set_output_format` rely on server state (default budget cache warming, diagnostic manager, cache manager, response formatter) and likely stay in `YNABMCPServer` even if their schemas are centralized.
-
-**Notes for Migration**
-- Budget-aware tools share `defaultArgumentResolver` to inject `budget_id`; keep this resolver available wherever registrations move.
-- `LooseObjectSchema` is the permissive output for many write/delta tools—consider replacing with explicit output schemas in `src/tools/schemas/outputs` if stability is desired.

package/.code/agents/f95b7464-3e25-4897-b153-c8dfd63fd605/error.txt

@@ -1,5 +0,0 @@
-Command failed: Error loading configuration: config profile `Context: cwd: C:\Users\ksutk\projects\ynab-mcpb. This is only to verify whether a gemini-3-pro backed agent can successfully start and complete.
-
-Agent: Minimal Gemini smoke test: from the ynab-mcpb repo, run `git status -sb` and list top-level directories, then print a short summary and exit. No file writes required.
-
-[Running in read-only mode - no modifications allowed]` not found

package/.code/agents/fa3c5ddf-cdf7-47a2-930a-b806c6363689/status.txt

@@ -1 +0,0 @@
-Agent cancelled

package/AGENTS.md

@@ -1 +0,0 @@
-CLAUDE.md

package/package.json.tmp

@@ -1,105 +0,0 @@
-{
-  "name": "@dizzlkheinz/ynab-mcp-server",
-  "version": "0.12.0",
-  "description": "Model Context Protocol server for YNAB (You Need A Budget) integration",
-  "main": "dist/index.js",
-  "type": "module",
-  "bin": {
-    "ynab-mcp-server": "bin/ynab-mcp-server.cjs"
-  },
-  "scripts": {
-    "prebuild": "npm run clean",
-    "build": "npm run build:prod",
-    "postbuild": "node scripts/run-generate-mcpb.js",
-    "build:dev": "npm run clean && tsc && npm run bundle",
-    "build:no-lint": "npm run clean && tsc && npm run bundle:prod",
-    "build:prod": "npm run lint:fix && npm run prebuild && tsc --project tsconfig.prod.json && npm run bundle:prod && npm run verify-build",
-    "clean": "rimraf dist",
-    "dev": "tsc --watch",
-    "start": "node -r dotenv/config dist/index.js",
-    "start:prod": "NODE_ENV=production node dist/index.js",
-    "start:mcp": "node dist/index.js",
-    "validate-env": "node scripts/validate-env.js",
-    "verify-build": "node scripts/verify-build.js",
-    "lint": "npm run lint:eslint && npm run format:check",
-    "lint:eslint": "eslint src --ext .ts",
-    "lint:fix": "eslint src --ext .ts --fix && prettier --write .",
-    "format": "prettier --write .",
-    "format:check": "prettier --check .",
-    "type-check": "tsc --noEmit",
-    "test": "vitest run --project unit && npm run filter-test-results",
-    "test:watch": "vitest",
-    "test:unit": "vitest run --project unit",
-    "test:integration": "npm run test:integration:core",
-    "test:integration:core": "vitest run --project integration:core",
-    "test:integration:domain": "vitest run --project integration:domain",
-    "test:integration:full": "node scripts/run-throttled-integration-tests.js",
-    "test:integration:budgets": "node scripts/run-domain-integration-tests.js budgets",
-    "test:integration:accounts": "node scripts/run-domain-integration-tests.js accounts",
-    "test:integration:transactions": "node scripts/run-domain-integration-tests.js transactions",
-    "test:integration:categories": "node scripts/run-domain-integration-tests.js categories",
-    "test:integration:payees": "node scripts/run-domain-integration-tests.js payees",
-    "test:integration:months": "node scripts/run-domain-integration-tests.js months",
-    "test:integration:delta": "node scripts/run-domain-integration-tests.js delta",
-    "test:integration:reconciliation": "node scripts/run-domain-integration-tests.js reconciliation",
-    "test:e2e": "vitest run --project e2e",
-    "test:coverage": "vitest run --coverage --project unit",
-    "test:performance": "vitest run src/__tests__/performance.test.ts",
-    "test:comprehensive": "tsx src/__tests__/testRunner.ts",
-    "test:all": "npm run test:unit && npm run test:integration:core && npm run test:e2e && npm run test:performance",
-    "filter-test-results": "node -e \"const fs = require('fs'); try { const results = JSON.parse(fs.readFileSync('test-results.json', 'utf-8')); results.testResults = results.testResults.filter(r => r.status === 'fail'); fs.writeFileSync('test-results.json', JSON.stringify(results, null, 2)); } catch (e) { console.error('Failed to filter test results:', e); process.exit(1); }\"",
-    "generate:mcpb": "node scripts/run-generate-mcpb.js",
-    "bundle": "node esbuild.config.mjs",
-    "bundle:prod": "node esbuild.config.mjs --production",
-    "package:mcpb": "npm run build:prod && npm run generate:mcpb",
-    "pr:description": "node scripts/create-pr-description.js",
-    "pr:create": "npm run pr:description && gh pr create --body-file .pr-description.md",
-    "prepare": "npm run build:prod",
-    "prepublishOnly": "npm run test:unit && npm run build:prod"
-  },
-  "keywords": [
-    "mcp",
-    "ynab",
-    "budget",
-    "finance"
-  ],
-  "author": "",
-  "license": "AGPL-3.0",
-  "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.22.0",
-    "csv-parse": "^6.1.0",
-    "d3-array": "^3.2.4",
-    "date-fns": "^4.1.0",
-    "dotenv": "^17.2.1",
-    "ynab": "^2.9.0",
-    "zod": "^4.1.11"
-  },
-  "devDependencies": {
-    "@eslint/js": "^9.35.0",
-    "@types/d3-array": "^3.2.1",
-    "@types/node": "^24.5.2",
-    "@vitest/coverage-v8": "^3.2.4",
-    "@vitest/ui": "^3.2.4",
-    "esbuild": "^0.25.10",
-    "eslint": "^9.35.0",
-    "eslint-config-prettier": "^10.1.8",
-    "prettier": "^3.3.3",
-    "rimraf": "^6.0.1",
-    "tsx": "^4.20.6",
-    "typescript": "^5.9.2",
-    "typescript-eslint": "^8.42.0",
-    "vitest": "^3.2.4"
-  },
-  "directories": {
-    "doc": "docs"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/dizzlkheinz/ynab-mcp-dxt.git"
-  },
-  "types": "./dist/index.d.ts",
-  "bugs": {
-    "url": "https://github.com/dizzlkheinz/ynab-mcp-dxt/issues"
-  },
-  "homepage": "https://github.com/dizzlkheinz/ynab-mcp-dxt#readme"
-}

package/src/__tests__/delta.performance.test.ts

@@ -1,80 +0,0 @@
-import { describe, it, expect, beforeAll, beforeEach, afterEach } from 'vitest';
-import * as ynab from 'ynab';
-import { performance } from 'node:perf_hooks';
-import { CacheManager } from '../server/cacheManager.js';
-import { ServerKnowledgeStore } from '../server/serverKnowledgeStore.js';
-import { DeltaCache } from '../server/deltaCache.js';
-import { DeltaFetcher } from '../tools/deltaFetcher.js';
-
-const skipPerfFlag = (process.env['SKIP_PERFORMANCE_TESTS'] ?? 'true').toLowerCase().trim();
-const shouldSkipPerformance = ['true', '1', 'yes', 'y', 'on'].includes(skipPerfFlag);
-const describePerformance = shouldSkipPerformance ? describe.skip : describe;
-
-describePerformance('Delta performance characteristics', () => {
-  let ynabAPI: ynab.API;
-  let testBudgetId: string;
-  let testAccountId: string;
-  let deltaFetcher: DeltaFetcher;
-
-  beforeAll(async () => {
-    const accessToken = process.env['YNAB_ACCESS_TOKEN'];
-    if (!accessToken) {
-      throw new Error('YNAB_ACCESS_TOKEN is required to run performance tests.');
-    }
-    ynabAPI = new ynab.API(accessToken);
-    const budgetsResponse = await ynabAPI.budgets.getBudgets();
-    if (
-      !budgetsResponse.data ||
-      !Array.isArray(budgetsResponse.data.budgets) ||
-      budgetsResponse.data.budgets.length === 0
-    ) {
-      throw new Error('No budgets available for performance tests.');
-    }
-    const budget = budgetsResponse.data.budgets[0];
-    testBudgetId = budget.id;
-
-    const accountsResponse = await ynabAPI.accounts.getAccounts(testBudgetId);
-    const account = accountsResponse.data.accounts.find((acct) => !acct.closed);
-    if (!account) {
-      throw new Error('No open accounts available for performance tests.');
-    }
-    testAccountId = account.id;
-  });
-
-  beforeEach(() => {
-    const cacheManager = new CacheManager();
-    const knowledgeStore = new ServerKnowledgeStore();
-    const deltaCache = new DeltaCache(cacheManager, knowledgeStore);
-    deltaFetcher = new DeltaFetcher(ynabAPI, deltaCache);
-    process.env['YNAB_MCP_ENABLE_DELTA'] = 'true';
-  });
-
-  afterEach(() => {
-    delete process.env['YNAB_MCP_ENABLE_DELTA'];
-  });
-
-  const measure = async <T>(loader: () => Promise<T>) => {
-    const start = performance.now();
-    const result = await loader();
-    const duration = performance.now() - start;
-    return { result, duration };
-  };
-
-  it('reuses cache and avoids repeated full refreshes', async () => {
-    const sinceDate = new Date(Date.now() - 30 * 24 * 60 * 60 * 1000).toISOString().split('T')[0];
-
-    const first = await measure(() =>
-      deltaFetcher.fetchTransactionsByAccount(testBudgetId, testAccountId, sinceDate, {
-        forceFullRefresh: true,
-      }),
-    );
-
-    const second = await measure(() =>
-      deltaFetcher.fetchTransactionsByAccount(testBudgetId, testAccountId, sinceDate),
-    );
-
-    expect(first.result.wasCached).toBe(false);
-    expect(second.result.wasCached).toBe(true);
-    expect(second.duration).toBeLessThan(first.duration * 0.8); // Cached should be at least 20% faster
-  });
-});