@dizzlkheinz/ynab-mcpb 0.17.0 → 0.18.0

package/src/tools/toolCategories.ts

@@ -120,7 +120,6 @@ export const ToolAnnotationPresets = {
    * interacting with the YNAB API.
    *
    * Examples:
-   * - convert_amount: Converts between dollars and milliunits
    * - set_output_format: Configures local output formatting
    * - diagnostic_info: Returns local server diagnostic information
    * - clear_cache: Clears local in-memory cache

package/src/tools/utilityTools.ts

@@ -1,6 +1,5 @@
 import { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
 import * as ynab from 'ynab';
-import { z } from 'zod/v4';
 import { responseFormatter } from '../server/responseFormatter.js';
 import { withToolErrorHandling } from '../types/index.js';
 import { createAdapters } from './adapters.js';
@@ -8,18 +7,6 @@ import { emptyObjectSchema } from './schemas/common.js';
 import { ToolAnnotationPresets } from './toolCategories.js';
 import type { ToolFactory } from '../types/toolRegistration.js';
 
-/**
- * Schema for ynab:convert_amount tool parameters
- */
-export const ConvertAmountSchema = z
-  .object({
-    amount: z.number().finite(),
-    to_milliunits: z.boolean(),
-  })
-  .strict();
-
-export type ConvertAmountParams = z.infer<typeof ConvertAmountSchema>;
-
 /**
  * Handles the ynab:get_user tool call
  * Gets information about the authenticated user
@@ -49,58 +36,13 @@ export async function handleGetUser(ynabAPI: ynab.API): Promise<CallToolResult>
 }
 
 /**
- * Handles the ynab:convert_amount tool call
- * Converts between dollars and milliunits with integer arithmetic for precision
- */
-export async function handleConvertAmount(
-  _ynabAPI: ynab.API,
-  params: ConvertAmountParams,
-): Promise<CallToolResult> {
-  return await withToolErrorHandling(
-    async () => {
-      const { amount, to_milliunits } = params;
-
-      let result: number;
-      let description: string;
-
-      if (to_milliunits) {
-        // Convert from dollars to milliunits
-        // Use integer arithmetic to avoid floating-point precision issues
-        result = Math.round(amount * 1000);
-        description = `$${amount.toFixed(2)} = ${result} milliunits`;
-      } else {
-        // Convert from milliunits to dollars
-        // Assume input amount is in milliunits
-        result = amount / 1000;
-        description = `${amount} milliunits = $${result.toFixed(2)}`;
-      }
-
-      return {
-        content: [
-          {
-            type: 'text',
-            text: responseFormatter.format({
-              conversion: {
-                original_amount: amount,
-                converted_amount: result,
-                to_milliunits,
-                description,
-              },
-            }),
-          },
-        ],
-      };
-    },
-    'ynab:convert_amount',
-    'converting amount',
-  );
-}
-
-/**
- * Registers utility tools (get_user, convert_amount) using the shared factory pattern.
+ * Registers utility tools (get_user) using the shared factory pattern.
+ *
+ * Note: convert_amount was removed because all tool responses already return
+ * amounts in dollars (converted from YNAB milliunits internally).
  */
 export const registerUtilityTools: ToolFactory = (registry, context) => {
-  const { adapt, adaptNoInput } = createAdapters(context);
+  const { adaptNoInput } = createAdapters(context);
 
   registry.register({
     name: 'get_user',
@@ -114,17 +56,4 @@ export const registerUtilityTools: ToolFactory = (registry, context) => {
       },
     },
   });
-
-  registry.register({
-    name: 'convert_amount',
-    description: 'Convert between dollars and milliunits with integer arithmetic for precision',
-    inputSchema: ConvertAmountSchema,
-    handler: adapt(handleConvertAmount),
-    metadata: {
-      annotations: {
-        ...ToolAnnotationPresets.UTILITY_LOCAL,
-        title: 'YNAB: Convert Amount',
-      },
-    },
-  });
 };

package/vitest.config.ts

@@ -1,6 +1,7 @@
 import { defineConfig } from 'vitest/config';
 
 const integrationFiles = ['src/**/*.integration.test.ts'];
+const isVerbose = !!process.env['VERBOSE_TESTS'];
 
 export default defineConfig({
   test: {
@@ -17,6 +18,8 @@ export default defineConfig({
           exclude: [
             'src/**/*.integration.test.ts',
             'src/**/*.e2e.test.ts',
+            // YNABMCPServer.test.ts requires real YNAB API token and makes API calls,
+            // so it runs as part of integration tests, not unit tests
             'src/server/__tests__/YNABMCPServer.test.ts',
           ],
         },
@@ -83,7 +86,7 @@ export default defineConfig({
         },
       },
     },
-    reporters: ['verbose', 'html', './vitest-reporters/split-json-reporter.ts'],
+    reporters: [isVerbose ? 'verbose' : 'dot', 'html', './vitest-reporters/split-json-reporter.ts'],
     outputFile: {
       html: './test-results/index.html',
     },

package/.chunkhound.json

@@ -1,11 +0,0 @@
-{
-  "embedding": {
-    "provider": "voyageai",
-    "model": "voyage-3.5",
-    "api_key": "pa-Q1WewxKAEUA6QeYTmreQmY11ufEk9mKIK0DfoqqZhV6",
-    "rerank_model": "rerank-2.5"
-  },
-  "llm": {
-    "provider": "claude-code-cli"
-  }
-}

package/.code/agents/0098661e-0fa3-4990-beb9-c0cbf3f123aa/status.txt

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

package/.code/agents/01a13ef4-3f23-4f52-b33b-3585b73cfa60/error.txt

@@ -1,3 +0,0 @@
-Command failed: Error loading configuration: config profile `From the repo root, list top-level files and directories using ls or dir, then exit. No file writes or edits.
-
-[Running in read-only mode - no modifications allowed]` not found

package/.code/agents/084fd32f-e298-4728-9103-a78d7dc39613/error.txt

@@ -1,3 +0,0 @@
-Command failed: Error loading configuration: config profile `From the repo root, just run `pwd` and then exit. Do not modify any files.
-
-[Running in read-only mode - no modifications allowed]` not found

package/.code/agents/0fed51e1-a943-4b97-a2a8-a6f0f27c844d/status.txt

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

package/.code/agents/1059b6bd-5ccd-4d83-a12c-7c9d89137399/error.txt

@@ -1,5 +0,0 @@
-Command failed: ParserError: 
-Line |
-   5 |  [Running in read-only mode - no modifications allowed]
-     |          ~
-     | Missing ] at end of attribute or type literal.