@dizzlkheinz/ynab-mcpb 0.16.1 → 0.17.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dizzlkheinz/ynab-mcpb",
-  "version": "0.16.1",
+  "version": "0.17.1",
   "description": "Model Context Protocol server for YNAB (You Need A Budget) integration",
   "main": "dist/index.js",
   "type": "module",
@@ -23,9 +23,9 @@
     "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 .",
+    "lint:fix": "eslint src --ext .ts --fix && prettier --log-level silent --write .",
+    "format": "prettier --log-level silent --write .",
+    "format:check": "prettier --log-level silent --check .",
     "type-check": "tsc --noEmit",
     "test": "vitest run --project unit && npm run filter-test-results",
     "test:watch": "vitest",
@@ -66,7 +66,7 @@
   "author": "",
   "license": "AGPL-3.0",
   "dependencies": {
-    "@modelcontextprotocol/sdk": "^1.22.0",
+    "@modelcontextprotocol/sdk": "^1.24.3",
     "chrono-node": "^2.9.0",
     "csv-parse": "^6.1.0",
     "d3-array": "^3.2.4",
@@ -90,7 +90,7 @@
     "eslint": "^9.35.0",
     "eslint-config-prettier": "^10.1.8",
     "prettier": "^3.3.3",
-    "rimraf": "^6.0.1",
+    "rimraf": "^6.1.2",
     "tsx": "^4.20.6",
     "typescript": "^5.9.2",
     "typescript-eslint": "^8.42.0",

package/scripts/analyze-bundle.mjs

@@ -1,41 +1,41 @@
-import fs from 'fs';
-
-let meta;
-try {
-  meta = JSON.parse(fs.readFileSync('meta.json', 'utf-8'));
-} catch (error) {
-  console.error('❌ Error reading meta.json:', error.message);
-  process.exit(1);
-}
-if (!meta.inputs || typeof meta.inputs !== 'object') {
-  console.error('❌ Error: meta.inputs is missing or invalid in meta.json');
-  process.exit(1);
-}
-
-const inputs = Object.entries(meta.inputs)
-  .map(([path, data]) => ({
-    path,
-    bytes: data.bytes,
-  }))
-  .sort((a, b) => b.bytes - a.bytes)
-  .slice(0, 20);
-
-const totalBytes = Object.values(meta.inputs).reduce((sum, item) => sum + item.bytes, 0);
-console.log(`\n📦 Total input size: ${(totalBytes / 1024 / 1024).toFixed(2)} MB`);
-const outputKey = 'dist/bundle/index.cjs';
-if (!meta.outputs || !meta.outputs[outputKey]) {
-  console.error(`❌ Error: meta.outputs['${outputKey}'] is missing in meta.json`);
-  process.exit(1);
-}
-console.log(`📦 Output size: ${(meta.outputs[outputKey].bytes / 1024 / 1024).toFixed(2)} MB`);
-console.log('\n📊 Top 20 largest inputs in bundle:\n');
-inputs.forEach((item) => {
-  const kb = (item.bytes / 1024).toFixed(1).padStart(8);
-  console.log(`${kb} KB  ${item.path}`);
-});  const kb = (item.bytes / 1024).toFixed(1).padStart(8);
-  console.log(`${kb} KB  ${item.path}`);
-});
-
-const totalBytes = Object.values(meta.inputs).reduce((sum, item) => sum + item.bytes, 0);
-console.log(`\n📦 Total input size: ${(totalBytes / 1024 / 1024).toFixed(2)} MB`);
-console.log(`📦 Output size: ${(meta.outputs['dist/bundle/index.cjs'].bytes / 1024 / 1024).toFixed(2)} MB`);
+import fs from 'fs';
+
+let meta;
+try {
+  meta = JSON.parse(fs.readFileSync('meta.json', 'utf-8'));
+} catch (error) {
+  console.error('❌ Error reading meta.json:', error.message);
+  process.exit(1);
+}
+if (!meta.inputs || typeof meta.inputs !== 'object') {
+  console.error('❌ Error: meta.inputs is missing or invalid in meta.json');
+  process.exit(1);
+}
+
+const inputs = Object.entries(meta.inputs)
+  .map(([path, data]) => ({
+    path,
+    bytes: data.bytes,
+  }))
+  .sort((a, b) => b.bytes - a.bytes)
+  .slice(0, 20);
+
+const totalBytes = Object.values(meta.inputs).reduce((sum, item) => sum + item.bytes, 0);
+console.log(`\n📦 Total input size: ${(totalBytes / 1024 / 1024).toFixed(2)} MB`);
+const outputKey = 'dist/bundle/index.cjs';
+if (!meta.outputs || !meta.outputs[outputKey]) {
+  console.error(`❌ Error: meta.outputs['${outputKey}'] is missing in meta.json`);
+  process.exit(1);
+}
+console.log(`📦 Output size: ${(meta.outputs[outputKey].bytes / 1024 / 1024).toFixed(2)} MB`);
+console.log('\n📊 Top 20 largest inputs in bundle:\n');
+inputs.forEach((item) => {
+  const kb = (item.bytes / 1024).toFixed(1).padStart(8);
+  console.log(`${kb} KB  ${item.path}`);
+});  const kb = (item.bytes / 1024).toFixed(1).padStart(8);
+  console.log(`${kb} KB  ${item.path}`);
+});
+
+const totalBytes = Object.values(meta.inputs).reduce((sum, item) => sum + item.bytes, 0);
+console.log(`\n📦 Total input size: ${(totalBytes / 1024 / 1024).toFixed(2)} MB`);
+console.log(`📦 Output size: ${(meta.outputs['dist/bundle/index.cjs'].bytes / 1024 / 1024).toFixed(2)} MB`);

package/scripts/generate-mcpb.ps1

@@ -1,96 +1,96 @@
-# PowerShell script to generate a .mcpb file using the official Anthropic MCPB CLI
-
-param(
-    [string]$OutputDir = "dist"
-)
-
-Write-Host "Generating YNAB MCP Server .mcpb package using official CLI..." -ForegroundColor Green
-
-# Configuration
-$PackageJson = Get-Content "package.json" | ConvertFrom-Json
-$PackageName = $PackageJson.name
-$Version = $PackageJson.version
-
-# Ensure we have a built version
-if (-not (Test-Path "dist/index.js")) {
-    Write-Host "Build not found. Running build first..." -ForegroundColor Red
-    npm run build
-    if ($LASTEXITCODE -ne 0) {
-        Write-Host "Build failed!" -ForegroundColor Red
-        exit 1
-    }
-}
-
-# Check if official MCPB CLI is installed
-try {
-    $mcpbVersion = & mcpb --version
-    Write-Host "Using MCPB CLI version: $mcpbVersion" -ForegroundColor Green
-} catch {
-    Write-Host "MCPB CLI not found. Installing..." -ForegroundColor Yellow
-    npm install -g @anthropic-ai/mcpb
-    if ($LASTEXITCODE -ne 0) {
-        Write-Host "Failed to install MCPB CLI!" -ForegroundColor Red
-        exit 1
-    }
-}
-
-# Ensure manifest.json exists and is valid
-if (-not (Test-Path "manifest.json")) {
-    Write-Host "manifest.json not found. Creating one..." -ForegroundColor Yellow
-    & mcpb init -y
-    if ($LASTEXITCODE -ne 0) {
-        Write-Host "Failed to create manifest!" -ForegroundColor Red
-        exit 1
-    }
-}
-
-# Sync version from package.json to manifest.json
-Write-Host "Syncing version from package.json to manifest.json..." -ForegroundColor Yellow
-$ManifestPath = "manifest.json"
-$ManifestContent = Get-Content $ManifestPath -Raw
-$Manifest = $ManifestContent | ConvertFrom-Json
-
-if ($Manifest.version -ne $Version) {
-    Write-Host "Updating manifest version from $($Manifest.version) to $Version" -ForegroundColor Cyan
-    # Use regex to update version while preserving formatting
-    $UpdatedContent = $ManifestContent -replace '("version"\s*:\s*)"[^"]*"', "`$1`"$Version`""
-    $UpdatedContent | Set-Content $ManifestPath -NoNewline
-}
-
-# Validate the manifest
-Write-Host "Validating manifest..." -ForegroundColor Yellow
-& mcpb validate manifest.json
-if ($LASTEXITCODE -ne 0) {
-    Write-Host "Manifest validation failed!" -ForegroundColor Red
-    exit 1
-}
-
-# Pack the MCPB using official CLI
-Write-Host "Packing MCPB file..." -ForegroundColor Yellow
-$DxtFile = "$PackageName-$Version.mcpb"
-$OutputPath = Join-Path $OutputDir $DxtFile
-
-& mcpb pack . $OutputPath
-if ($LASTEXITCODE -ne 0) {
-    Write-Host "MCPB packing failed!" -ForegroundColor Red
-    exit 1
-}
-
-# Get file size
-if (Test-Path $OutputPath) {
-    $FileSize = (Get-Item $OutputPath).Length
-    $FileSizeKB = [math]::Round($FileSize / 1KB, 1)
-    $FileSizeMB = [math]::Round($FileSize / 1MB, 1)
-    
-    Write-Host "Created $OutputPath" -ForegroundColor Green
-    Write-Host "Size: $FileSizeKB KB ($FileSizeMB MB)" -ForegroundColor Cyan
-} else {
-    Write-Host "MCPB file was not created!" -ForegroundColor Red
-    exit 1
-}
-
-Write-Host ""
-Write-Host "Installation Instructions:" -ForegroundColor Yellow
-Write-Host "1. Drag and drop the .mcpb file into Claude Desktop" -ForegroundColor White
-Write-Host "2. Set YNAB_ACCESS_TOKEN environment variable" -ForegroundColor White
+# PowerShell script to generate a .mcpb file using the official Anthropic MCPB CLI
+
+param(
+    [string]$OutputDir = "dist"
+)
+
+Write-Host "Generating YNAB MCP Server .mcpb package using official CLI..." -ForegroundColor Green
+
+# Configuration
+$PackageJson = Get-Content "package.json" | ConvertFrom-Json
+$PackageName = $PackageJson.name
+$Version = $PackageJson.version
+
+# Ensure we have a built version
+if (-not (Test-Path "dist/index.js")) {
+    Write-Host "Build not found. Running build first..." -ForegroundColor Red
+    npm run build
+    if ($LASTEXITCODE -ne 0) {
+        Write-Host "Build failed!" -ForegroundColor Red
+        exit 1
+    }
+}
+
+# Check if official MCPB CLI is installed
+try {
+    $mcpbVersion = & mcpb --version
+    Write-Host "Using MCPB CLI version: $mcpbVersion" -ForegroundColor Green
+} catch {
+    Write-Host "MCPB CLI not found. Installing..." -ForegroundColor Yellow
+    npm install -g @anthropic-ai/mcpb
+    if ($LASTEXITCODE -ne 0) {
+        Write-Host "Failed to install MCPB CLI!" -ForegroundColor Red
+        exit 1
+    }
+}
+
+# Ensure manifest.json exists and is valid
+if (-not (Test-Path "manifest.json")) {
+    Write-Host "manifest.json not found. Creating one..." -ForegroundColor Yellow
+    & mcpb init -y
+    if ($LASTEXITCODE -ne 0) {
+        Write-Host "Failed to create manifest!" -ForegroundColor Red
+        exit 1
+    }
+}
+
+# Sync version from package.json to manifest.json
+Write-Host "Syncing version from package.json to manifest.json..." -ForegroundColor Yellow
+$ManifestPath = "manifest.json"
+$ManifestContent = Get-Content $ManifestPath -Raw
+$Manifest = $ManifestContent | ConvertFrom-Json
+
+if ($Manifest.version -ne $Version) {
+    Write-Host "Updating manifest version from $($Manifest.version) to $Version" -ForegroundColor Cyan
+    # Use regex to update version while preserving formatting
+    $UpdatedContent = $ManifestContent -replace '("version"\s*:\s*)"[^"]*"', "`$1`"$Version`""
+    $UpdatedContent | Set-Content $ManifestPath -NoNewline
+}
+
+# Validate the manifest
+Write-Host "Validating manifest..." -ForegroundColor Yellow
+& mcpb validate manifest.json
+if ($LASTEXITCODE -ne 0) {
+    Write-Host "Manifest validation failed!" -ForegroundColor Red
+    exit 1
+}
+
+# Pack the MCPB using official CLI
+Write-Host "Packing MCPB file..." -ForegroundColor Yellow
+$DxtFile = "$PackageName-$Version.mcpb"
+$OutputPath = Join-Path $OutputDir $DxtFile
+
+& mcpb pack . $OutputPath
+if ($LASTEXITCODE -ne 0) {
+    Write-Host "MCPB packing failed!" -ForegroundColor Red
+    exit 1
+}
+
+# Get file size
+if (Test-Path $OutputPath) {
+    $FileSize = (Get-Item $OutputPath).Length
+    $FileSizeKB = [math]::Round($FileSize / 1KB, 1)
+    $FileSizeMB = [math]::Round($FileSize / 1MB, 1)
+    
+    Write-Host "Created $OutputPath" -ForegroundColor Green
+    Write-Host "Size: $FileSizeKB KB ($FileSizeMB MB)" -ForegroundColor Cyan
+} else {
+    Write-Host "MCPB file was not created!" -ForegroundColor Red
+    exit 1
+}
+
+Write-Host ""
+Write-Host "Installation Instructions:" -ForegroundColor Yellow
+Write-Host "1. Drag and drop the .mcpb file into Claude Desktop" -ForegroundColor White
+Write-Host "2. Set YNAB_ACCESS_TOKEN environment variable" -ForegroundColor White
 Write-Host "3. Restart Claude Desktop" -ForegroundColor White

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

@@ -22,10 +22,13 @@ const env = {
 };
 
 const vitestArgs = ['vitest', 'run', '--project', 'integration:domain', ...passthroughArgs];
-const runner = process.platform === 'win32' ? 'npx.cmd' : 'npx';
+const runner = 'npx';
+const useShell = process.platform === 'win32';
+
 const child = spawn(runner, vitestArgs, {
   stdio: 'inherit',
   env,
+  shell: useShell,
 });
 
 child.on('close', (code) => {

package/scripts/watch-and-restart.ps1

@@ -1,50 +1,50 @@
-# PowerShell script to watch for file changes and restart MCP
-param(
-    [string]$Path = ".\src",
-    [string]$Filter = "*.ts",
-    [int]$RestartDelay = 3
-)
-
-Write-Host "🔍 Watching for changes in: $Path"
-Write-Host "📁 Filter: $Filter"
-
-$watcher = New-Object System.IO.FileSystemWatcher
-$watcher.Path = Resolve-Path $Path
-$watcher.Filter = $Filter
-$watcher.EnableRaisingEvents = $true
-$watcher.IncludeSubdirectories = $true
-
-$action = {
-    $path = $Event.SourceEventArgs.FullPath
-    $changeType = $Event.SourceEventArgs.ChangeType
-    Write-Host "🔄 File $changeType`: $path"
-    
-    # Build the project
-    Write-Host "🏗️  Building project..."
-    npm run build
-    
-    if ($LASTEXITCODE -eq 0) {
-        Write-Host "✅ Build successful"
-        Start-Sleep -Seconds $RestartDelay
-        
-        Write-Host "🔄 Reconnecting to YNAB MCP server..."
-        /mcp reconnect ynab-mcp-server
-        Start-Sleep -Seconds 1
-        /mcp reconnect ynab-mcp-server
-        Write-Host "✅ MCP server reconnected"
-    } else {
-        Write-Host "❌ Build failed, skipping MCP restart"
-    }
-}
-
-Register-ObjectEvent -InputObject $watcher -EventName "Changed" -Action $action
-Register-ObjectEvent -InputObject $watcher -EventName "Created" -Action $action
-
-try {
-    Write-Host "✅ File watcher started. Press Ctrl+C to stop."
-    while ($true) { Start-Sleep 1 }
-}
-finally {
-    $watcher.Dispose()
-    Write-Host "👋 File watcher stopped"
+# PowerShell script to watch for file changes and restart MCP
+param(
+    [string]$Path = ".\src",
+    [string]$Filter = "*.ts",
+    [int]$RestartDelay = 3
+)
+
+Write-Host "🔍 Watching for changes in: $Path"
+Write-Host "📁 Filter: $Filter"
+
+$watcher = New-Object System.IO.FileSystemWatcher
+$watcher.Path = Resolve-Path $Path
+$watcher.Filter = $Filter
+$watcher.EnableRaisingEvents = $true
+$watcher.IncludeSubdirectories = $true
+
+$action = {
+    $path = $Event.SourceEventArgs.FullPath
+    $changeType = $Event.SourceEventArgs.ChangeType
+    Write-Host "🔄 File $changeType`: $path"
+    
+    # Build the project
+    Write-Host "🏗️  Building project..."
+    npm run build
+    
+    if ($LASTEXITCODE -eq 0) {
+        Write-Host "✅ Build successful"
+        Start-Sleep -Seconds $RestartDelay
+        
+        Write-Host "🔄 Reconnecting to YNAB MCP server..."
+        /mcp reconnect ynab-mcp-server
+        Start-Sleep -Seconds 1
+        /mcp reconnect ynab-mcp-server
+        Write-Host "✅ MCP server reconnected"
+    } else {
+        Write-Host "❌ Build failed, skipping MCP restart"
+    }
+}
+
+Register-ObjectEvent -InputObject $watcher -EventName "Changed" -Action $action
+Register-ObjectEvent -InputObject $watcher -EventName "Created" -Action $action
+
+try {
+    Write-Host "✅ File watcher started. Press Ctrl+C to stop."
+    while ($true) { Start-Sleep 1 }
+}
+finally {
+    $watcher.Dispose()
+    Write-Host "👋 File watcher stopped"
 }

package/src/__tests__/comprehensive.integration.test.ts

@@ -748,34 +748,6 @@ describe('YNAB MCP Server - Comprehensive Integration Tests', () => {
         expect(mockYnabAPI.user.getUser).toHaveBeenCalledTimes(1);
       },
     );
-
-    it(
-      'should handle amount conversion',
-      { meta: { tier: 'domain', domain: 'workflows' } },
-      async () => {
-        // Test dollar to milliunits conversion
-        const toMilliunitsResult = await executeToolCall(server, 'ynab:convert_amount', {
-          amount: 25.75,
-          to_milliunits: true,
-        });
-        validateToolResult(toMilliunitsResult);
-
-        const toMilli = parseToolResult(toMilliunitsResult);
-        expect(toMilli.data.conversion.converted_amount).toBe(25750);
-        expect(toMilli.data.conversion.description).toBe('$25.75 = 25750 milliunits');
-
-        // Test milliunits to dollar conversion
-        const toDollarsResult = await executeToolCall(server, 'ynab:convert_amount', {
-          amount: 25750,
-          to_milliunits: false,
-        });
-        validateToolResult(toDollarsResult);
-
-        const dollars = parseToolResult(toDollarsResult);
-        expect(dollars.data.conversion.converted_amount).toBe(25.75);
-        expect(dollars.data.conversion.description).toBe('25750 milliunits = $25.75');
-      },
-    );
   });
 
   describe('Error Handling Integration', () => {

package/src/__tests__/performance.test.ts

@@ -623,12 +623,6 @@ describe('YNAB MCP Server - Performance Tests', () => {
 
       // Test multiple validation scenarios
       const validationTests = [
-        // Valid parameters
-        executeToolCall(server, 'ynab:convert_amount', {
-          amount: 25.5,
-          to_milliunits: true,
-        }),
-
         // Invalid parameters (should fail quickly)
         executeToolCall(server, 'ynab:get_budget', {
           budget_id: '', // Empty string should fail validation
@@ -648,10 +642,9 @@ describe('YNAB MCP Server - Performance Tests', () => {
 
       const totalTime = endTime - startTime;
 
-      expect(parsed).toHaveLength(3);
-      expect(parsed[0]).toBeDefined(); // Valid call should succeed
-      const firstError = parsed[1].error ?? parsed[1].data?.error;
-      const secondError = parsed[2].error ?? parsed[2].data?.error;
+      expect(parsed).toHaveLength(2);
+      const firstError = parsed[0].error ?? parsed[0].data?.error;
+      const secondError = parsed[1].error ?? parsed[1].data?.error;
       expect(firstError?.code).toBe(SecurityErrorCode.VALIDATION_ERROR); // Invalid calls should fail
       expect(secondError?.code).toBe(SecurityErrorCode.VALIDATION_ERROR);
       expect(totalTime).toBeLessThan(1000); // Validation should be fast
@@ -715,7 +708,6 @@ describe('YNAB MCP Server - Performance Tests', () => {
           executeToolCall(server, 'ynab:list_accounts', { budget_id: 'test' }),
           executeToolCall(server, 'ynab:list_transactions', { budget_id: 'test' }),
           executeToolCall(server, 'ynab:list_categories', { budget_id: 'test' }),
-          executeToolCall(server, 'ynab:convert_amount', { amount: i * 10, to_milliunits: true }),
         );
       }
 
@@ -724,7 +716,7 @@ describe('YNAB MCP Server - Performance Tests', () => {
 
       const totalTime = endTime - startTime;
 
-      expect(results).toHaveLength(100); // 20 iterations × 5 tools
+      expect(results).toHaveLength(80); // 20 iterations × 4 tools
       results.forEach((result) => expect(result).toBeDefined());
       expect(totalTime).toBeLessThan(10000); // Should complete within 10 seconds
     });

package/src/__tests__/setup.ts

@@ -12,6 +12,49 @@ const hasAccessToken = !!process.env['YNAB_ACCESS_TOKEN'];
 if (!process.env['SKIP_E2E_TESTS']) {
   process.env['SKIP_E2E_TESTS'] = hasAccessToken ? 'false' : 'true';
 }
+if (!process.env['YNAB_ACCESS_TOKEN']) {
+  process.env['YNAB_ACCESS_TOKEN'] = 'test-token-for-mocked-tests';
+}
+
+// Set test environment variables immediately
+process.env['NODE_ENV'] = 'test';
+if (!process.env['LOG_LEVEL']) {
+  process.env['LOG_LEVEL'] = 'error';
+}
+
+// Disable console output for cleaner test output unless VERBOSE_TESTS is set
+if (!process.env['VERBOSE_TESTS']) {
+  const originalConsoleError = console.error;
+
+  console.error = (...args: any[]) => {
+    const firstArg = args[0];
+    const isString = typeof firstArg === 'string';
+    // Only show errors that are part of test assertions, actual errors, or explicitly marked [ERROR]
+    if (
+      (isString &&
+        (firstArg.includes('❌') || firstArg.includes('Test') || firstArg.includes('[ERROR]'))) ||
+      firstArg instanceof Error
+    ) {
+      originalConsoleError(...args);
+    }
+  };
+
+  console.warn = () => {
+    // Suppress warnings by default
+  };
+
+  console.log = () => {
+    // Suppress logs by default
+  };
+
+  console.info = () => {
+    // Suppress info logs by default
+  };
+
+  console.debug = () => {
+    // Suppress debug logs by default
+  };
+}
 
 type TierFilter = 'core' | 'domain' | 'full';
 interface TestMeta {
@@ -47,26 +90,14 @@ const shouldRunDomain = (domain?: string): boolean => {
  * Global test setup
  */
 beforeAll(async () => {
-  // Set test environment variables
-  process.env['NODE_ENV'] = 'test';
-
   // Set default test token if not provided
   if (!process.env['YNAB_ACCESS_TOKEN']) {
     process.env['YNAB_ACCESS_TOKEN'] = 'test-token-for-mocked-tests';
   }
 
-  // Disable console.error for cleaner test output (except for specific tests)
-  if (!process.env['VERBOSE_TESTS']) {
-    const originalConsoleError = console.error;
-    console.error = (...args: any[]) => {
-      // Only show errors that are part of test assertions
-      if (args[0]?.includes?.('❌') || args[0]?.includes?.('Test')) {
-        originalConsoleError(...args);
-      }
-    };
+  if (process.env['VERBOSE_TESTS']) {
+    console.warn('🧪 Test environment initialized');
   }
-
-  console.warn('🧪 Test environment initialized');
 });
 
 /**

package/src/__tests__/workflows.e2e.test.ts

@@ -821,34 +821,6 @@ describeE2E('YNAB MCP Server - End-to-End Workflows', () => {
     });
   });
 
-  describe('Utility Tools Workflow', () => {
-    it('should convert amounts between dollars and milliunits', async () => {
-      if (testConfig.skipE2ETests) return;
-
-      // Convert dollars to milliunits
-      const toMilliunitsResult = await executeToolCall(server, 'ynab:convert_amount', {
-        amount: 25.5,
-        to_milliunits: true,
-      });
-      const milliunits = parseToolResult(toMilliunitsResult);
-
-      expect(milliunits.data?.conversion?.converted_amount).toBe(25500);
-      expect(milliunits.data?.conversion?.description).toContain('25500');
-      expect(milliunits.data?.conversion?.to_milliunits).toBe(true);
-
-      // Convert milliunits to dollars
-      const toDollarsResult = await executeToolCall(server, 'ynab:convert_amount', {
-        amount: 25500,
-        to_milliunits: false,
-      });
-      const dollars = parseToolResult(toDollarsResult);
-
-      expect(dollars.data?.conversion?.converted_amount).toBe(25.5);
-      expect(dollars.data?.conversion?.description).toContain('$25.50');
-      expect(dollars.data?.conversion?.to_milliunits).toBe(false);
-    });
-  });
-
   describe('v0.8.x Architecture Integration Tests', () => {
     describe('Cache System Verification', () => {
       it('should demonstrate cache warming after default budget set', async () => {
@@ -1065,13 +1037,7 @@ describeE2E('YNAB MCP Server - End-to-End Workflows', () => {
           expect(tool.description).toBeDefined();
           expect(tool.inputSchema).toBeDefined();
 
-          // Verify that all tools define outputSchema (as guaranteed by CHANGELOG.md and docs/reference/TOOLS.md)
-          // Note: Some utility tools like diagnostic_info or clear_cache may not define structured outputs,
-          // but most data-retrieval and CRUD tools should have output schemas.
-          expect(
-            tool.outputSchema,
-            `Tool '${tool.name}' should define an outputSchema`,
-          ).toBeDefined();
+          // Output schemas are optional; tools may omit them.
         }
       });
     });
@@ -1143,7 +1109,6 @@ describeE2E('YNAB MCP Server - End-to-End Workflows', () => {
           { name: 'ynab:list_categories', args: { budget_id: testBudgetId } },
           { name: 'ynab:list_payees', args: { budget_id: testBudgetId } },
           { name: 'ynab:get_user', args: {} },
-          { name: 'ynab:convert_amount', args: { amount: 100, to_milliunits: true } },
         ];
 
         for (const tool of v7Tools) {
@@ -1621,21 +1586,6 @@ describeE2E('YNAB MCP Server - End-to-End Workflows', () => {
       }
     });
 
-    it('should validate convert_amount output schema', async () => {
-      if (testConfig.skipE2ETests) return;
-
-      const result = await executeToolCall(server, 'ynab:convert_amount', {
-        amount: 100,
-        to_milliunits: true,
-      });
-      const validation = validateOutputSchema(server, 'convert_amount', result);
-      expect(validation.hasSchema).toBe(true);
-      expect(validation.valid).toBe(true);
-      if (!validation.valid) {
-        console.error('Schema validation errors:', validation.errors);
-      }
-    });
-
     it('should validate export_transactions output schema', async () => {
       if (testConfig.skipE2ETests) return;