Simplify error handling, switch to primarily integration tests

Author: chriscarrollsmith

.cursor/rules/general.mdc

@@ -0,0 +1,6 @@
+---
+description: 
+globs: 
+alwaysApply: true
+---
+Work step-by-step. If presented with an implementation plan, implement the plan exactly. If the plan presents more than one implementation option, consult with the human user to decide between options. If you are tempted to embellish or imporve upon the plan, consult with the human user. Always complete the current task and wait for human review before proceeding to the next task.

babel.config.cjs

@@ -0,0 +1,6 @@
+module.exports = {
+  presets: [
+    ['@babel/preset-env', { targets: { node: 'current' } }],
+    '@babel/preset-typescript',
+  ],
+}; 

index.ts

@@ -3,7 +3,7 @@
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { Server } from "@modelcontextprotocol/sdk/server/index.js";
 import { TaskManager } from "./src/server/TaskManager.js";
-import { ALL_TOOLS, executeToolWithErrorHandling } from "./src/server/tools.js";
+import { ALL_TOOLS, executeToolAndHandleErrors } from "./src/server/tools.js";
 import { ListToolsRequestSchema, CallToolRequestSchema } from "@modelcontextprotocol/sdk/types.js";
 
 // Create server with capabilities BEFORE setting up handlers
@@ -39,11 +39,16 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
 });
 
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
-  return executeToolWithErrorHandling(
+  // Directly call the handler. It either returns a result object (success or isError:true)
+  // OR it throws a tagged protocol error.
+  return await executeToolAndHandleErrors(
     request.params.name,
     request.params.arguments || {},
     taskManager
   );
+  // SDK automatically handles:
+  // - Wrapping the returned value (success data or isError:true object) in `result: { ... }`
+  // - Catching re-thrown protocol errors and formatting the top-level `error: { ... }`
 });
 
 // Start the server

jest.config.cjs

@@ -1,14 +1,7 @@
-const { createDefaultEsmPreset } = require('ts-jest');
-
-const presetConfig = createDefaultEsmPreset({
-  useESM: true,
-});
-
 module.exports = {
-  ...presetConfig,
   testEnvironment: 'node',
   moduleNameMapper: {
-    '^(\\.{1,2}/.*)\\.js$': '$1',
+    '^(\\.{1,2}/.*)\\.js$': '$1'
   },
   modulePathIgnorePatterns: ['<rootDir>/dist/'],
   // Force Jest to exit after all tests have completed