@1mcp/agent 0.2.0 → 0.7.0

package/build/utils/errorHandling.js

@@ -1,47 +1,11 @@
 import { ERROR_CODES } from '../constants.js';
 import logger from '../logger/logger.js';
+import { MCPError, InvalidRequestError } from './errorTypes.js';
 /**
- * Custom error types for the MCP agent
- */
-export class MCPError extends Error {
-    code;
-    data;
-    constructor(message, code, data) {
-        super(message);
-        this.name = 'MCPError';
-        this.code = code;
-        this.data = data;
-    }
-}
-export class ClientNotFoundError extends MCPError {
-    constructor(clientName) {
-        super(`Client not found: ${clientName}`, ERROR_CODES.TRANSPORT_NOT_FOUND);
-        this.name = 'ClientNotFoundError';
-    }
-}
-export class ClientConnectionError extends MCPError {
-    constructor(clientName, originalError) {
-        super(`Failed to connect to client: ${clientName}${originalError ? ` - ${originalError.message}` : ''}`, ERROR_CODES.TRANSPORT_NOT_FOUND, { originalError });
-        this.name = 'ClientConnectionError';
-    }
-}
-export class InvalidRequestError extends MCPError {
-    constructor(message, data) {
-        super(message, ERROR_CODES.INVALID_PARAMS, data);
-        this.name = 'InvalidRequestError';
-    }
-}
-export class ProxyError extends MCPError {
-    constructor(message, originalError) {
-        super(message, ERROR_CODES.INTERNAL_SERVER_ERROR, originalError ? { originalError: originalError.message } : undefined);
-        this.name = 'ProxyError';
-    }
-}
-/**
- * Handles errors in async functions by wrapping them in a try/catch block
- * @param fn The async function to wrap
- * @param errorMessage The error message to log
- * @returns A wrapped function that handles errors
+ * Wraps a function with error handling
+ * @param fn The function to wrap
+ * @param errorMessage The error message to use if the function fails
+ * @returns The wrapped function
  */
 export function withErrorHandling(fn, errorMessage) {
     return async (...args) => {
@@ -54,11 +18,70 @@ export function withErrorHandling(fn, errorMessage) {
             if (error instanceof MCPError) {
                 throw error;
             }
-            // Convert other errors to ProxyError
-            throw new ProxyError(errorMessage, error instanceof Error ? error : new Error(String(error)));
+            // Convert other errors to MCPError
+            throw new MCPError(errorMessage, ERROR_CODES.INTERNAL_SERVER_ERROR, {
+                originalError: error instanceof Error ? error : new Error(String(error)),
+            });
         }
     };
 }
+/**
+ * Normalizes an error to an MCPError type
+ * @param error The error to normalize
+ * @param errorMessage The error message to use if the error is not an MCPError
+ * @returns The normalized error
+ */
+export function normalizeError(error, errorMessage) {
+    if (error instanceof MCPError) {
+        return error;
+    }
+    if (error instanceof Error) {
+        return new MCPError(error.message || errorMessage, ERROR_CODES.INTERNAL_SERVER_ERROR);
+    }
+    return new MCPError(errorMessage, ERROR_CODES.INTERNAL_SERVER_ERROR);
+}
+/**
+ * Checks if an error is a specific MCPError type
+ * @param error The error to check
+ * @param errorType The error type to check against
+ * @returns True if the error is of the specified type
+ */
+export function isMCPError(error, errorType) {
+    return error instanceof errorType;
+}
+/**
+ * Gets the error code from an error
+ * @param error The error to get the code from
+ * @returns The error code
+ */
+export function getErrorCode(error) {
+    if (error instanceof MCPError) {
+        return error.code;
+    }
+    return ERROR_CODES.INTERNAL_SERVER_ERROR;
+}
+/**
+ * Gets the error message from an error
+ * @param error The error to get the message from
+ * @returns The error message
+ */
+export function getErrorMessage(error) {
+    if (error instanceof Error) {
+        return error.message;
+    }
+    return String(error);
+}
+/**
+ * Gets the error cause from an error
+ * @param error The error to get the cause from
+ * @returns The error cause
+ */
+export function getErrorCause(error) {
+    if (error instanceof Error && 'cause' in error) {
+        return error.cause;
+    }
+    return undefined;
+}
 /**
  * Formats an error for JSON-RPC response
  * @param error The error to format

package/build/utils/errorTypes.js

@@ -0,0 +1,47 @@
+import { ERROR_CODES } from '../constants.js';
+export class MCPError extends Error {
+    code;
+    data;
+    constructor(message, code, data) {
+        super(message);
+        this.name = 'MCPError';
+        this.code = code;
+        this.data = data;
+    }
+}
+export class ClientConnectionError extends MCPError {
+    constructor(clientName, cause) {
+        super(`Failed to connect to client ${clientName}: ${cause.message}`, ERROR_CODES.TRANSPORT_NOT_FOUND, { cause });
+        this.name = 'ClientConnectionError';
+    }
+}
+export class ClientNotFoundError extends MCPError {
+    constructor(clientName) {
+        super(`Client not found: ${clientName}`, ERROR_CODES.TRANSPORT_NOT_FOUND);
+        this.name = 'ClientNotFoundError';
+    }
+}
+export class ClientOperationError extends MCPError {
+    constructor(clientName, operation, cause, context) {
+        super(`Operation ${operation} failed on client ${clientName}: ${cause.message}`, ERROR_CODES.INTERNAL_SERVER_ERROR, { cause, context });
+        this.name = 'ClientOperationError';
+    }
+}
+export class ValidationError extends MCPError {
+    constructor(message, validationErrors) {
+        super(message, ERROR_CODES.INVALID_PARAMS, { validationErrors });
+        this.name = 'ValidationError';
+    }
+}
+export class TransportError extends MCPError {
+    constructor(transportName, cause) {
+        super(`Transport error for ${transportName}: ${cause.message}`, ERROR_CODES.CLIENT_CONNECTION_ERROR, { cause });
+        this.name = 'TransportError';
+    }
+}
+export class InvalidRequestError extends MCPError {
+    constructor(message, data) {
+        super(message, ERROR_CODES.INVALID_PARAMS, data);
+        this.name = 'InvalidRequestError';
+    }
+}

package/package.json

@@ -1,8 +1,26 @@
 {
   "name": "@1mcp/agent",
-  "version": "0.2.0",
+  "version": "0.7.0",
   "description": "One MCP server to aggregate them all - A unified Model Context Protocol server implementation",
-  "homepage": "https://1mcp.app",
+  "keywords": [
+    "mcp",
+    "mcp-server",
+    "mcp-client",
+    "model-context-protocol",
+    "agent",
+    "proxy",
+    "server",
+    "cli",
+    "ai",
+    "llm",
+    "model-aggregation",
+    "typescript"
+  ],
+  "author": "William Xu <xuzp@1mcp.app>",
+  "homepage": "https://github.com/1mcp-app/agent",
+  "bugs": {
+    "url": "https://github.com/1mcp-app/agent/issues"
+  },
   "license": "Apache-2.0",
   "repository": {
     "type": "git",
@@ -25,6 +43,7 @@
   },
   "lint-staged": {
     "*.{js,ts}": [
+      "eslint --fix",
       "prettier --write"
     ]
   },
@@ -39,20 +58,38 @@
     "zod": "^3.24.2"
   },
   "devDependencies": {
+    "@eslint/js": "^9.22.0",
+    "@testing-library/jest-dom": "^6.6.3",
     "@types/express": "^5.0.0",
+    "@types/jest": "^29.5.14",
     "@types/node": "^22.13.10",
     "@types/uuid": "^10.0.0",
     "@types/yargs": "^17.0.33",
+    "@typescript-eslint/eslint-plugin": "^8.27.0",
+    "@typescript-eslint/parser": "^8.27.0",
+    "eslint": "^9.22.0",
+    "eslint-config-prettier": "^10.1.1",
+    "eslint-plugin-prettier": "^5.2.3",
+    "globals": "^16.0.0",
     "husky": "^9.1.7",
+    "jest": "^29.7.0",
+    "jiti": "^2.4.2",
     "lint-staged": "^15.5.0",
     "prettier": "^3.5.3",
+    "ts-jest": "^29.2.6",
+    "ts-node": "^10.9.2",
     "tsc-watch": "^6.2.1",
     "typescript": "^5.3.3"
   },
   "scripts": {
-    "dev": "tsc-watch --onSuccess \"node build/index.js\"",
+    "dev": "tsc-watch --onSuccess \"node build/index.js --config ./mcp.json\"",
     "build": "tsc && node -e \"require('fs').chmodSync('build/index.js', '755')\"",
     "watch": "tsc --watch",
-    "inspector": "npx @modelcontextprotocol/inspector build/index.js"
+    "inspector": "npx -y @modelcontextprotocol/inspector",
+    "lint": "eslint . --ext .ts",
+    "lint:fix": "eslint . --ext .ts --fix",
+    "test": "jest",
+    "test:watch": "jest --watch",
+    "test:coverage": "jest --coverage"
   }
 }