@iflow-mcp/attio-mcp 1.3.6

package/dist/handlers/tools/dispatcher/validation.js

@@ -0,0 +1,49 @@
+/**
+ * Validation utilities for tool execution
+ *
+ * Provides parameter validation and resource ID validation for tool requests
+ */
+import { createErrorResult } from '../../../utils/error-handler.js';
+import { ResourceType } from '../../../types/attio.js';
+/**
+ * Validates attribute parameters for company operations
+ *
+ * @param attributes - The attributes object to validate (key-value pairs of company attributes)
+ * @returns True if validation passes, or an error message string if validation fails
+ */
+export function validateAttributes(attributes) {
+    if (!attributes) {
+        return 'Attributes parameter is required';
+    }
+    if (typeof attributes !== 'object' || Array.isArray(attributes)) {
+        return 'Attributes parameter must be an object';
+    }
+    if (Object.keys(attributes).length === 0) {
+        return 'Attributes parameter cannot be empty';
+    }
+    return true;
+}
+/**
+ * Helper function to validate and extract resource ID based on resource type
+ *
+ * @param resourceType - The type of resource (companies, people, etc.)
+ * @param args - The request arguments containing the ID
+ * @param apiPath - The API path for error reporting
+ * @returns The validated ID or an error response
+ */
+export function validateResourceId(resourceType, args, apiPath) {
+    const idParamName = resourceType === ResourceType.COMPANIES ? 'companyId' : 'personId';
+    const id = args && typeof args === 'object'
+        ? args[idParamName]
+        : undefined;
+    if (!id) {
+        return {
+            error: createErrorResult(new Error(`${idParamName} parameter is required`), apiPath, 'GET', {
+                status: 400,
+                message: `Missing required parameter: ${idParamName}`,
+            }),
+        };
+    }
+    return id;
+}
+//# sourceMappingURL=validation.js.map

package/dist/handlers/tools/dispatcher/validation.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.js","sourceRoot":"","sources":["../../../../src/handlers/tools/dispatcher/validation.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAEH,OAAO,EAAE,iBAAiB,EAAE,MAAM,iCAAiC,CAAC;AACpE,OAAO,EAAE,YAAY,EAAE,MAAM,yBAAyB,CAAC;AAMvD;;;;;GAKG;AACH,MAAM,UAAU,kBAAkB,CAChC,UAAwD;IAExD,IAAI,CAAC,UAAU,EAAE,CAAC;QAChB,OAAO,kCAAkC,CAAC;IAC5C,CAAC;IAED,IAAI,OAAO,UAAU,KAAK,QAAQ,IAAI,KAAK,CAAC,OAAO,CAAC,UAAU,CAAC,EAAE,CAAC;QAChE,OAAO,wCAAwC,CAAC;IAClD,CAAC;IAED,IAAI,MAAM,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACzC,OAAO,sCAAsC,CAAC;IAChD,CAAC;IAED,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,kBAAkB,CAChC,YAA0B,EAC1B,IAAgD,EAChD,OAAe;IAEf,MAAM,WAAW,GACf,YAAY,KAAK,YAAY,CAAC,SAAS,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,UAAU,CAAC;IACrE,MAAM,EAAE,GACN,IAAI,IAAI,OAAO,IAAI,KAAK,QAAQ;QAC9B,CAAC,CAAE,IAAI,CAAC,WAAW,CAAwB;QAC3C,CAAC,CAAC,SAAS,CAAC;IAEhB,IAAI,CAAC,EAAE,EAAE,CAAC;QACR,OAAO;YACL,KAAK,EAAE,iBAAiB,CACtB,IAAI,KAAK,CAAC,GAAG,WAAW,wBAAwB,CAAC,EACjD,OAAO,EACP,KAAK,EACL;gBACE,MAAM,EAAE,GAAG;gBACX,OAAO,EAAE,+BAA+B,WAAW,EAAE;aACtD,CACF;SACF,CAAC;IACJ,CAAC;IAED,OAAO,EAAE,CAAC;AACZ,CAAC"}

package/dist/handlers/tools/dispatcher.d.ts

@@ -0,0 +1,11 @@
+/**
+ * Tool dispatcher module - handles tool execution dispatch and routing
+ *
+ * This module re-exports from the modular dispatcher implementation for backward compatibility.
+ * The actual implementation has been broken down into focused modules under ./dispatcher/
+ */
+export { executeToolRequest } from './dispatcher/core.js';
+export { logToolRequest, logToolError } from './dispatcher/logging.js';
+export { validateAttributes, validateResourceId, } from './dispatcher/validation.js';
+export { formatSuccessResponse } from './dispatcher/formatting.js';
+//# sourceMappingURL=dispatcher.d.ts.map

package/dist/handlers/tools/dispatcher.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"dispatcher.d.ts","sourceRoot":"","sources":["../../../src/handlers/tools/dispatcher.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAG1D,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,MAAM,yBAAyB,CAAC;AACvE,OAAO,EACL,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,4BAA4B,CAAC;AACpC,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC"}

package/dist/handlers/tools/dispatcher.js

@@ -0,0 +1,15 @@
+/**
+ * Tool dispatcher module - handles tool execution dispatch and routing
+ *
+ * This module re-exports from the modular dispatcher implementation for backward compatibility.
+ * The actual implementation has been broken down into focused modules under ./dispatcher/
+ */
+// Re-export the main executeToolRequest function from the modular implementation
+export { executeToolRequest } from './dispatcher/core.js';
+// Re-export utility functions from the modular implementation for backward compatibility
+export { logToolRequest, logToolError } from './dispatcher/logging.js';
+export { validateAttributes, validateResourceId, } from './dispatcher/validation.js';
+export { formatSuccessResponse } from './dispatcher/formatting.js';
+// Note: Legacy implementation placeholders below will be moved to dedicated operation modules in Phase 2
+// These are temporarily retained for any unimplemented operations until full modularization is complete
+//# sourceMappingURL=dispatcher.js.map

package/dist/handlers/tools/dispatcher.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"dispatcher.js","sourceRoot":"","sources":["../../../src/handlers/tools/dispatcher.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,iFAAiF;AACjF,OAAO,EAAE,kBAAkB,EAAE,MAAM,sBAAsB,CAAC;AAE1D,yFAAyF;AACzF,OAAO,EAAE,cAAc,EAAE,YAAY,EAAE,MAAM,yBAAyB,CAAC;AACvE,OAAO,EACL,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,4BAA4B,CAAC;AACpC,OAAO,EAAE,qBAAqB,EAAE,MAAM,4BAA4B,CAAC;AAEnE,yGAAyG;AACzG,wGAAwG"}

package/dist/handlers/tools/error-types.d.ts

@@ -0,0 +1,33 @@
+/**
+ * Error types for tool operations
+ */
+/**
+ * Interface for API errors with response data
+ */
+export interface ApiError extends Error {
+    response?: {
+        data: Record<string, unknown>;
+        status: number;
+    };
+    details?: Record<string, unknown>;
+}
+/**
+ * Interface for ValueMatchError with original error
+ */
+export interface ValueMatchApiError extends Error {
+    originalError?: ApiError;
+    details?: Record<string, unknown>;
+}
+/**
+ * Type guard to check if error is an ApiError
+ */
+export declare function isApiError(error: unknown): error is ApiError;
+/**
+ * Type guard to check if error has response data
+ */
+export declare function hasResponseData(error: unknown): error is {
+    response: {
+        data: Record<string, unknown>;
+    };
+};
+//# sourceMappingURL=error-types.d.ts.map

package/dist/handlers/tools/error-types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"error-types.d.ts","sourceRoot":"","sources":["../../../src/handlers/tools/error-types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH;;GAEG;AACH,MAAM,WAAW,QAAS,SAAQ,KAAK;IACrC,QAAQ,CAAC,EAAE;QACT,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;QAC9B,MAAM,EAAE,MAAM,CAAC;KAChB,CAAC;IACF,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACnC;AAED;;GAEG;AACH,MAAM,WAAW,kBAAmB,SAAQ,KAAK;IAC/C,aAAa,CAAC,EAAE,QAAQ,CAAC;IACzB,OAAO,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAC;CACnC;AAED;;GAEG;AACH,wBAAgB,UAAU,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,QAAQ,CAE5D;AAED;;GAEG;AACH,wBAAgB,eAAe,CAC7B,KAAK,EAAE,OAAO,GACb,KAAK,IAAI;IAAE,QAAQ,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;KAAE,CAAA;CAAE,CAW1D"}

package/dist/handlers/tools/error-types.js

@@ -0,0 +1,23 @@
+/**
+ * Error types for tool operations
+ */
+/**
+ * Type guard to check if error is an ApiError
+ */
+export function isApiError(error) {
+    return error instanceof Error && 'response' in error;
+}
+/**
+ * Type guard to check if error has response data
+ */
+export function hasResponseData(error) {
+    return (error !== null &&
+        typeof error === 'object' &&
+        'response' in error &&
+        error.response !== null &&
+        typeof error.response === 'object' &&
+        'data' in error.response &&
+        typeof error.response.data === 'object' &&
+        error.response.data !== null);
+}
+//# sourceMappingURL=error-types.js.map

package/dist/handlers/tools/error-types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"error-types.js","sourceRoot":"","sources":["../../../src/handlers/tools/error-types.ts"],"names":[],"mappings":"AAAA;;GAEG;AAqBH;;GAEG;AACH,MAAM,UAAU,UAAU,CAAC,KAAc;IACvC,OAAO,KAAK,YAAY,KAAK,IAAI,UAAU,IAAI,KAAK,CAAC;AACvD,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,eAAe,CAC7B,KAAc;IAEd,OAAO,CACL,KAAK,KAAK,IAAI;QACd,OAAO,KAAK,KAAK,QAAQ;QACzB,UAAU,IAAI,KAAK;QACnB,KAAK,CAAC,QAAQ,KAAK,IAAI;QACvB,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ;QAClC,MAAM,IAAI,KAAK,CAAC,QAAQ;QACxB,OAAO,KAAK,CAAC,QAAQ,CAAC,IAAI,KAAK,QAAQ;QACvC,KAAK,CAAC,QAAQ,CAAC,IAAI,KAAK,IAAI,CAC7B,CAAC;AACJ,CAAC"}

package/dist/handlers/tools/formatters.d.ts

@@ -0,0 +1,58 @@
+/**
+ * Result formatting module - handles transformation and formatting of tool results
+ */
+import { AttioRecord, AttioListEntry } from '../../types/attio.js';
+interface BatchOperationResult {
+    summary: {
+        total: number;
+        succeeded: number;
+        failed: number;
+    };
+    results: Array<{
+        id: string;
+        success: boolean;
+        error?: {
+            message?: string;
+        };
+    }>;
+}
+/**
+ * Format search results for display
+ *
+ * @param results - Array of search results
+ * @param resourceType - The type of resource being searched
+ * @returns Formatted string output
+ */
+export declare function formatSearchResults(results: AttioRecord[], resourceType: string): string;
+/**
+ * Format record details for display
+ *
+ * @param record - The record to format
+ * @returns Formatted string output
+ */
+export declare function formatRecordDetails(record: AttioRecord): string;
+/**
+ * Format list entries for display
+ *
+ * @param entries - Array of list entries
+ * @returns Formatted string output
+ */
+export declare function formatListEntries(entries: AttioListEntry[]): string;
+/**
+ * Format batch operation results for display
+ *
+ * @param result - Batch operation result
+ * @param operation - The type of batch operation
+ * @returns Formatted string output
+ */
+export declare function formatBatchResults(result: BatchOperationResult, operation: string): string;
+/**
+ * Format standard response content
+ *
+ * @param content - The content to format
+ * @param isError - Whether this is an error response
+ * @returns Formatted response object
+ */
+export declare function formatResponse(content: string | unknown, isError?: boolean): unknown;
+export {};
+//# sourceMappingURL=formatters.d.ts.map

package/dist/handlers/tools/formatters.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"formatters.d.ts","sourceRoot":"","sources":["../../../src/handlers/tools/formatters.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,OAAO,EAAE,WAAW,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AASnE,UAAU,oBAAoB;IAC5B,OAAO,EAAE;QACP,KAAK,EAAE,MAAM,CAAC;QACd,SAAS,EAAE,MAAM,CAAC;QAClB,MAAM,EAAE,MAAM,CAAC;KAChB,CAAC;IACF,OAAO,EAAE,KAAK,CAAC;QACb,EAAE,EAAE,MAAM,CAAC;QACX,OAAO,EAAE,OAAO,CAAC;QACjB,KAAK,CAAC,EAAE;YAAE,OAAO,CAAC,EAAE,MAAM,CAAA;SAAE,CAAC;KAC9B,CAAC,CAAC;CACJ;AAyDD;;;;;;GAMG;AACH,wBAAgB,mBAAmB,CACjC,OAAO,EAAE,WAAW,EAAE,EACtB,YAAY,EAAE,MAAM,GACnB,MAAM,CAYR;AAED;;;;;GAKG;AACH,wBAAgB,mBAAmB,CAAC,MAAM,EAAE,WAAW,GAAG,MAAM,CAe/D;AAED;;;;;GAKG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,cAAc,EAAE,GAAG,MAAM,CAiBnE;AAED;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAChC,MAAM,EAAE,oBAAoB,EAC5B,SAAS,EAAE,MAAM,GAChB,MAAM,CAeR;AAED;;;;;;GAMG;AAEH,wBAAgB,cAAc,CAC5B,OAAO,EAAE,MAAM,GAAG,OAAO,EACzB,OAAO,GAAE,OAAe,WAoDzB"}

package/dist/handlers/tools/formatters.js

@@ -0,0 +1,175 @@
+import { processListEntries } from '../../utils/record-utils.js';
+import { safeJsonStringify, sanitizeMcpResponse, } from '../../utils/json-serializer.js';
+import { createScopedLogger } from '../../utils/logger.js';
+/**
+ * Safely extract value from record attributes
+ *
+ * This function handles various record formats and extracts values from them,
+ * including standard AttioRecord objects and similar record-like structures
+ * that may come from different API responses or transformations.
+ *
+ * @param record - The record to extract the value from, which might be an AttioRecord or a similar structure
+ * @param fieldName - The name of the field to extract
+ * @returns The extracted value as a string, or 'Unknown' if not found
+ */
+function getAttributeValue(record, fieldName) {
+    if (!record?.values)
+        return 'Unknown';
+    const fieldValue = record.values[fieldName];
+    if (!fieldValue)
+        return 'Unknown';
+    // Handle different value formats
+    if (Array.isArray(fieldValue) && fieldValue.length > 0) {
+        const firstValue = fieldValue[0];
+        if (typeof firstValue === 'object' &&
+            firstValue !== null &&
+            'value' in firstValue) {
+            return firstValue.value || 'Unknown';
+        }
+        return String(firstValue) || 'Unknown';
+    }
+    else if (typeof fieldValue === 'object' &&
+        fieldValue !== null &&
+        'value' in fieldValue) {
+        return fieldValue.value || 'Unknown';
+    }
+    // Fallback for any other format
+    return String(fieldValue) || 'Unknown';
+}
+/**
+ * Format search results for display
+ *
+ * @param results - Array of search results
+ * @param resourceType - The type of resource being searched
+ * @returns Formatted string output
+ */
+export function formatSearchResults(results, resourceType) {
+    if (!results || results.length === 0) {
+        return `No ${resourceType} found.`;
+    }
+    return results
+        .map((record, index) => {
+        const name = getAttributeValue(record, 'name');
+        const id = record.id?.record_id || 'Unknown ID';
+        return `${index + 1}. ${name} (ID: ${id})`;
+    })
+        .join('\n');
+}
+/**
+ * Format record details for display
+ *
+ * @param record - The record to format
+ * @returns Formatted string output
+ */
+export function formatRecordDetails(record) {
+    if (!record) {
+        return 'No record found.';
+    }
+    const attributes = record.attributes || {};
+    const formattedAttrs = Object.entries(attributes)
+        .map(([key, attr]) => {
+        const attrObj = attr;
+        const value = attrObj.value || 'N/A';
+        return `${key}: ${value}`;
+    })
+        .join('\n');
+    return `Record Details:\n${formattedAttrs}`;
+}
+/**
+ * Format list entries for display
+ *
+ * @param entries - Array of list entries
+ * @returns Formatted string output
+ */
+export function formatListEntries(entries) {
+    if (!entries || entries.length === 0) {
+        return 'No entries found.';
+    }
+    const processedEntries = processListEntries(entries);
+    return processedEntries
+        .map((entry, index) => {
+        const record = entry.record;
+        const name = getAttributeValue(record, 'name');
+        const entryId = entry.id?.entry_id || 'Unknown Entry ID';
+        const recordId = record?.id?.record_id || 'Unknown ID';
+        return `${index + 1}. ${name} (Entry: ${entryId}, Record: ${recordId})`;
+    })
+        .join('\n');
+}
+/**
+ * Format batch operation results for display
+ *
+ * @param result - Batch operation result
+ * @param operation - The type of batch operation
+ * @returns Formatted string output
+ */
+export function formatBatchResults(result, operation) {
+    const summary = result.summary;
+    const details = result.results
+        .map((r) => r.success
+        ? `✅ Record ${r.id}: ${operation} successfully`
+        : `❌ Record ${r.id}: Failed - ${r.error?.message || 'Unknown error'}`)
+        .join('\n');
+    return (`Batch ${operation} operation completed:\n` +
+        `Total: ${summary.total}, Succeeded: ${summary.succeeded}, Failed: ${summary.failed}\n` +
+        details);
+}
+/**
+ * Format standard response content
+ *
+ * @param content - The content to format
+ * @param isError - Whether this is an error response
+ * @returns Formatted response object
+ */
+export function formatResponse(content, isError = false) {
+    // Handle non-string content by converting it to a string
+    let formattedContent;
+    if (typeof content === 'string') {
+        formattedContent = content;
+    }
+    else if (content === undefined || content === null) {
+        // Prevent "undefined" or "null" from being returned as response content
+        formattedContent = isError
+            ? 'An unknown error occurred'
+            : 'Operation completed successfully, but no content was returned';
+    }
+    else {
+        try {
+            // Try to convert the content to a string representation using safe serialization
+            formattedContent =
+                typeof content === 'object'
+                    ? safeJsonStringify(content, {
+                        maxDepth: 6,
+                        includeStackTraces: false,
+                    })
+                    : String(content);
+        }
+        catch (error) {
+            if (process.env.DEBUG || process.env.NODE_ENV === 'development') {
+                const log = createScopedLogger('tools.formatters', 'formatResponse');
+                log.warn('Error converting content to string', {
+                    error: String(error),
+                });
+            }
+            formattedContent = 'Error: Content could not be serialized';
+        }
+    }
+    // Ensure we never return empty content
+    if (!formattedContent) {
+        formattedContent = isError
+            ? 'An unknown error occurred'
+            : 'Operation completed successfully';
+    }
+    const response = {
+        content: [
+            {
+                type: 'text',
+                text: formattedContent,
+            },
+        ],
+        isError,
+    };
+    // Sanitize the final response to ensure it's MCP-compatible
+    return sanitizeMcpResponse(response);
+}
+//# sourceMappingURL=formatters.js.map

package/dist/handlers/tools/formatters.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"formatters.js","sourceRoot":"","sources":["../../../src/handlers/tools/formatters.ts"],"names":[],"mappings":"AAIA,OAAO,EAAE,kBAAkB,EAAE,MAAM,6BAA6B,CAAC;AACjE,OAAO,EACL,iBAAiB,EACjB,mBAAmB,GACpB,MAAM,gCAAgC,CAAC;AACxC,OAAO,EAAE,kBAAkB,EAAE,MAAM,uBAAuB,CAAC;AAsB3D;;;;;;;;;;GAUG;AACH,SAAS,iBAAiB,CACxB,MAKa,EACb,SAAiB;IAEjB,IAAI,CAAC,MAAM,EAAE,MAAM;QAAE,OAAO,SAAS,CAAC;IAEtC,MAAM,UAAU,GAAG,MAAM,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;IAE5C,IAAI,CAAC,UAAU;QAAE,OAAO,SAAS,CAAC;IAElC,iCAAiC;IACjC,IAAI,KAAK,CAAC,OAAO,CAAC,UAAU,CAAC,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACvD,MAAM,UAAU,GAAG,UAAU,CAAC,CAAC,CAAC,CAAC;QACjC,IACE,OAAO,UAAU,KAAK,QAAQ;YAC9B,UAAU,KAAK,IAAI;YACnB,OAAO,IAAI,UAAU,EACrB,CAAC;YACD,OAAQ,UAAU,CAAC,KAAgB,IAAI,SAAS,CAAC;QACnD,CAAC;QACD,OAAO,MAAM,CAAC,UAAU,CAAC,IAAI,SAAS,CAAC;IACzC,CAAC;SAAM,IACL,OAAO,UAAU,KAAK,QAAQ;QAC9B,UAAU,KAAK,IAAI;QACnB,OAAO,IAAI,UAAU,EACrB,CAAC;QACD,OAAQ,UAAU,CAAC,KAAgB,IAAI,SAAS,CAAC;IACnD,CAAC;IAED,gCAAgC;IAChC,OAAO,MAAM,CAAC,UAAU,CAAC,IAAI,SAAS,CAAC;AACzC,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,mBAAmB,CACjC,OAAsB,EACtB,YAAoB;IAEpB,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACrC,OAAO,MAAM,YAAY,SAAS,CAAC;IACrC,CAAC;IAED,OAAO,OAAO;SACX,GAAG,CAAC,CAAC,MAAM,EAAE,KAAK,EAAE,EAAE;QACrB,MAAM,IAAI,GAAG,iBAAiB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAC/C,MAAM,EAAE,GAAG,MAAM,CAAC,EAAE,EAAE,SAAS,IAAI,YAAY,CAAC;QAChD,OAAO,GAAG,KAAK,GAAG,CAAC,KAAK,IAAI,SAAS,EAAE,GAAG,CAAC;IAC7C,CAAC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAC;AAChB,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,mBAAmB,CAAC,MAAmB;IACrD,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,OAAO,kBAAkB,CAAC;IAC5B,CAAC;IAED,MAAM,UAAU,GAAG,MAAM,CAAC,UAAU,IAAK,EAA8B,CAAC;IACxE,MAAM,cAAc,GAAG,MAAM,CAAC,OAAO,CAAC,UAAU,CAAC;SAC9C,GAAG,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,CAAC,EAAE,EAAE;QACnB,MAAM,OAAO,GAAG,IAAsB,CAAC;QACvC,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,IAAI,KAAK,CAAC;QACrC,OAAO,GAAG,GAAG,KAAK,KAAK,EAAE,CAAC;IAC5B,CAAC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAC;IAEd,OAAO,oBAAoB,cAAc,EAAE,CAAC;AAC9C,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAAyB;IACzD,IAAI,CAAC,OAAO,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACrC,OAAO,mBAAmB,CAAC;IAC7B,CAAC;IAED,MAAM,gBAAgB,GAAG,kBAAkB,CAAC,OAAO,CAAC,CAAC;IAErD,OAAO,gBAAgB;SACpB,GAAG,CAAC,CAAC,KAAK,EAAE,KAAK,EAAE,EAAE;QACpB,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,CAAC;QAC5B,MAAM,IAAI,GAAG,iBAAiB,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAC/C,MAAM,OAAO,GAAG,KAAK,CAAC,EAAE,EAAE,QAAQ,IAAI,kBAAkB,CAAC;QACzD,MAAM,QAAQ,GAAG,MAAM,EAAE,EAAE,EAAE,SAAS,IAAI,YAAY,CAAC;QAEvD,OAAO,GAAG,KAAK,GAAG,CAAC,KAAK,IAAI,YAAY,OAAO,aAAa,QAAQ,GAAG,CAAC;IAC1E,CAAC,CAAC;SACD,IAAI,CAAC,IAAI,CAAC,CAAC;AAChB,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAA4B,EAC5B,SAAiB;IAEjB,MAAM,OAAO,GAAG,MAAM,CAAC,OAAO,CAAC;IAC/B,MAAM,OAAO,GAAG,MAAM,CAAC,OAAO;SAC3B,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CACT,CAAC,CAAC,OAAO;QACP,CAAC,CAAC,YAAY,CAAC,CAAC,EAAE,KAAK,SAAS,eAAe;QAC/C,CAAC,CAAC,YAAY,CAAC,CAAC,EAAE,cAAc,CAAC,CAAC,KAAK,EAAE,OAAO,IAAI,eAAe,EAAE,CACxE;SACA,IAAI,CAAC,IAAI,CAAC,CAAC;IAEd,OAAO,CACL,SAAS,SAAS,yBAAyB;QAC3C,UAAU,OAAO,CAAC,KAAK,gBAAgB,OAAO,CAAC,SAAS,aAAa,OAAO,CAAC,MAAM,IAAI;QACvF,OAAO,CACR,CAAC;AACJ,CAAC;AAED;;;;;;GAMG;AAEH,MAAM,UAAU,cAAc,CAC5B,OAAyB,EACzB,UAAmB,KAAK;IAExB,yDAAyD;IACzD,IAAI,gBAAwB,CAAC;IAE7B,IAAI,OAAO,OAAO,KAAK,QAAQ,EAAE,CAAC;QAChC,gBAAgB,GAAG,OAAO,CAAC;IAC7B,CAAC;SAAM,IAAI,OAAO,KAAK,SAAS,IAAI,OAAO,KAAK,IAAI,EAAE,CAAC;QACrD,wEAAwE;QACxE,gBAAgB,GAAG,OAAO;YACxB,CAAC,CAAC,2BAA2B;YAC7B,CAAC,CAAC,+DAA+D,CAAC;IACtE,CAAC;SAAM,CAAC;QACN,IAAI,CAAC;YACH,iFAAiF;YACjF,gBAAgB;gBACd,OAAO,OAAO,KAAK,QAAQ;oBACzB,CAAC,CAAC,iBAAiB,CAAC,OAAO,EAAE;wBACzB,QAAQ,EAAE,CAAC;wBACX,kBAAkB,EAAE,KAAK;qBAC1B,CAAC;oBACJ,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;QACxB,CAAC;QAAC,OAAO,KAAc,EAAE,CAAC;YACxB,IAAI,OAAO,CAAC,GAAG,CAAC,KAAK,IAAI,OAAO,CAAC,GAAG,CAAC,QAAQ,KAAK,aAAa,EAAE,CAAC;gBAChE,MAAM,GAAG,GAAG,kBAAkB,CAAC,kBAAkB,EAAE,gBAAgB,CAAC,CAAC;gBACrE,GAAG,CAAC,IAAI,CAAC,oCAAoC,EAAE;oBAC7C,KAAK,EAAE,MAAM,CAAC,KAAK,CAAC;iBACrB,CAAC,CAAC;YACL,CAAC;YACD,gBAAgB,GAAG,wCAAwC,CAAC;QAC9D,CAAC;IACH,CAAC;IAED,uCAAuC;IACvC,IAAI,CAAC,gBAAgB,EAAE,CAAC;QACtB,gBAAgB,GAAG,OAAO;YACxB,CAAC,CAAC,2BAA2B;YAC7B,CAAC,CAAC,kCAAkC,CAAC;IACzC,CAAC;IAED,MAAM,QAAQ,GAAG;QACf,OAAO,EAAE;YACP;gBACE,IAAI,EAAE,MAAM;gBACZ,IAAI,EAAE,gBAAgB;aACvB;SACF;QACD,OAAO;KACR,CAAC;IAEF,4DAA4D;IAC5D,OAAO,mBAAmB,CAAC,QAAQ,CAAC,CAAC;AACvC,CAAC"}

package/dist/handlers/tools/index.d.ts

@@ -0,0 +1,17 @@
+/**
+ * Main entry point for tool handlers - maintains backward compatibility
+ */
+import { Server } from '@modelcontextprotocol/sdk/server/index.js';
+import { ServerContext } from '../../server/createServer.js';
+/**
+ * Registers tool-related request handlers with the server
+ *
+ * @param server - The MCP server instance
+ * @param context - Server context with configuration
+ */
+export declare function registerToolHandlers(server: Server, context?: ServerContext): void;
+export { TOOL_DEFINITIONS, TOOL_CONFIGS } from './registry.js';
+export { findToolConfig } from './registry.js';
+export { executeToolRequest } from './dispatcher.js';
+export { formatSearchResults, formatRecordDetails, formatListEntries, formatBatchResults, formatResponse, } from './formatters.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/handlers/tools/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/handlers/tools/index.ts"],"names":[],"mappings":"AAAA;;GAEG;AACH,OAAO,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AAanE,OAAO,EAAE,aAAa,EAAE,MAAM,0BAA0B,CAAC;AAoHzD;;;;;GAKG;AACH,wBAAgB,oBAAoB,CAClC,MAAM,EAAE,MAAM,EACd,OAAO,CAAC,EAAE,aAAa,GACtB,IAAI,CAsEN;AAGD,OAAO,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAC/D,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAC/C,OAAO,EAAE,kBAAkB,EAAE,MAAM,iBAAiB,CAAC;AACrD,OAAO,EACL,mBAAmB,EACnB,mBAAmB,EACnB,iBAAiB,EACjB,kBAAkB,EAClB,cAAc,GACf,MAAM,iBAAiB,CAAC"}

package/dist/handlers/tools/index.js

@@ -0,0 +1,145 @@
+import { CallToolRequestSchema, ListToolsRequestSchema, } from '@modelcontextprotocol/sdk/types.js';
+import { warn, clearLogContext, getLogContext, OperationType, } from '../../utils/logger.js';
+import { setGlobalContext } from '../../api/lazy-client.js';
+import { getToolsListPayload } from '../../utils/mcp-discovery.js';
+import { executeToolRequest } from '../../handlers/tools/dispatcher.js';
+import { initializeToolContext } from '../../handlers/tools/dispatcher/logging.js';
+import { createSecureToolErrorResult } from '../../utils/secure-error-handler.js';
+// Constants for configuration
+const DEBUG_ENV_VAR = 'MCP_DEBUG_REQUESTS';
+const MAX_ARGUMENT_SIZE = 1024 * 1024; // 1MB limit for arguments
+/**
+ * Normalize tool request to handle missing arguments wrapper
+ * Fixes Issue #344: MCP protocol mismatch where arguments are not wrapped
+ *
+ * This function handles two cases:
+ * 1. Standard format: { name: "tool", arguments: { query: "value" } }
+ * 2. Loose format: { name: "tool", query: "value" }
+ *
+ * @param request - The incoming tool request (may have loose arguments)
+ * @returns Normalized request with proper arguments structure
+ */
+function normalizeToolRequest(request) {
+    // Validate request structure
+    if (!request.params || typeof request.params.name !== 'string') {
+        throw new Error('Invalid tool request: missing params or tool name');
+    }
+    // Type guard to check if arguments are already properly wrapped
+    if ('arguments' in request.params && request.params.arguments !== undefined) {
+        // Validate argument size to prevent DoS
+        const argSize = JSON.stringify(request.params.arguments).length;
+        if (argSize > MAX_ARGUMENT_SIZE) {
+            throw new Error(`Tool arguments too large: ${argSize} bytes (max: ${MAX_ARGUMENT_SIZE})`);
+        }
+        // Return clean request with only name and arguments
+        // This ensures no extra parameters pollute the request
+        return {
+            ...request,
+            params: {
+                name: request.params.name,
+                arguments: request.params.arguments,
+            },
+        };
+    }
+    // Handle loose arguments format
+    const params = request.params;
+    const { name, ...potentialArgs } = params;
+    // If there are additional params beyond 'name', treat them as arguments
+    const hasAdditionalParams = Object.keys(potentialArgs).length > 0;
+    if (hasAdditionalParams) {
+        // Validate argument size
+        const argSize = JSON.stringify(potentialArgs).length;
+        if (argSize > MAX_ARGUMENT_SIZE) {
+            throw new Error(`Tool arguments too large: ${argSize} bytes (max: ${MAX_ARGUMENT_SIZE})`);
+        }
+        // Use structured logging instead of console.error
+        if (process.env[DEBUG_ENV_VAR] === 'true') {
+            warn('tool:normalization', 'Normalizing loose arguments format (Issue #344)', {
+                tool: name,
+                originalParamKeys: Object.keys(request.params),
+                normalizedArgKeys: Object.keys(potentialArgs),
+                argumentCount: Object.keys(potentialArgs).length,
+            }, 'normalization');
+        }
+        // Create normalized request with wrapped arguments
+        return {
+            ...request,
+            params: {
+                name,
+                arguments: potentialArgs,
+            },
+        };
+    }
+    // No additional params, tool might not need arguments
+    return {
+        ...request,
+        params: {
+            name: request.params.name,
+            arguments: undefined,
+        },
+    };
+}
+/**
+ * Registers tool-related request handlers with the server
+ *
+ * @param server - The MCP server instance
+ * @param context - Server context with configuration
+ */
+export function registerToolHandlers(server, context) {
+    // Set the global context for lazy initialization if provided
+    if (context) {
+        setGlobalContext(context);
+    }
+    // Handler for listing available tools
+    server.setRequestHandler(ListToolsRequestSchema, async () => {
+        const { tools } = getToolsListPayload();
+        return {
+            tools,
+        };
+    });
+    // Handler for calling tools
+    server.setRequestHandler(CallToolRequestSchema, async (request) => {
+        const toolName = typeof request.params?.name === 'string'
+            ? request.params.name
+            : 'unknown_tool';
+        const correlationId = initializeToolContext(toolName);
+        try {
+            // Normalize request to handle missing arguments wrapper (Issue #344)
+            // Cast is safe because we're handling the protocol mismatch
+            const normalizedRequest = normalizeToolRequest(request);
+            const result = (await executeToolRequest(normalizedRequest));
+            return result;
+        }
+        catch (error) {
+            warn('tool:normalization', 'Tool request failed before execution', {
+                tool: toolName,
+                correlationId,
+                errorMessage: error instanceof Error ? error.message : String(error),
+            }, 'call_tool', OperationType.TOOL_EXECUTION);
+            const { requestId, userId } = getLogContext();
+            return createSecureToolErrorResult(error, {
+                module: 'handlers.tools',
+                operation: `callTool:${toolName}`,
+                resourceType: toolName,
+                correlationId,
+                requestId,
+                userId,
+                errorType: 'normalization_error',
+                clientMessage: error instanceof Error
+                    ? error.message
+                    : 'Tool request normalization failed',
+                fallbackMessage: 'Tool request normalization failed',
+                suggestion: 'Ensure the MCP request includes a tool name and wraps arguments inside the "arguments" object.',
+            });
+        }
+        finally {
+            clearLogContext();
+        }
+    });
+}
+// Re-export commonly used components for backward compatibility
+export { TOOL_DEFINITIONS, TOOL_CONFIGS } from './registry.js';
+export { findToolConfig } from './registry.js';
+export { executeToolRequest } from './dispatcher.js';
+export { formatSearchResults, formatRecordDetails, formatListEntries, formatBatchResults, formatResponse, } from './formatters.js';
+//# sourceMappingURL=index.js.map

package/dist/handlers/tools/index.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../../src/handlers/tools/index.ts"],"names":[],"mappings":"AAIA,OAAO,EACL,qBAAqB,EACrB,sBAAsB,GAGvB,MAAM,oCAAoC,CAAC;AAC5C,OAAO,EACL,IAAI,EACJ,eAAe,EACf,aAAa,EACb,aAAa,GACd,MAAM,mBAAmB,CAAC;AAE3B,OAAO,EAAE,gBAAgB,EAAE,MAAM,sBAAsB,CAAC;AAIxD,OAAO,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AAC/D,OAAO,EAAE,kBAAkB,EAAE,MAAM,gCAAgC,CAAC;AACpE,OAAO,EAAE,qBAAqB,EAAE,MAAM,wCAAwC,CAAC;AAC/E,OAAO,EAAE,2BAA2B,EAAE,MAAM,iCAAiC,CAAC;AAE9E,8BAA8B;AAC9B,MAAM,aAAa,GAAG,oBAAoB,CAAC;AAC3C,MAAM,iBAAiB,GAAG,IAAI,GAAG,IAAI,CAAC,CAAC,0BAA0B;AAajE;;;;;;;;;;GAUG;AACH,SAAS,oBAAoB,CAC3B,OAA+C;IAE/C,6BAA6B;IAC7B,IAAI,CAAC,OAAO,CAAC,MAAM,IAAI,OAAO,OAAO,CAAC,MAAM,CAAC,IAAI,KAAK,QAAQ,EAAE,CAAC;QAC/D,MAAM,IAAI,KAAK,CAAC,mDAAmD,CAAC,CAAC;IACvE,CAAC;IAED,gEAAgE;IAChE,IAAI,WAAW,IAAI,OAAO,CAAC,MAAM,IAAI,OAAO,CAAC,MAAM,CAAC,SAAS,KAAK,SAAS,EAAE,CAAC;QAC5E,wCAAwC;QACxC,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,OAAO,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,MAAM,CAAC;QAChE,IAAI,OAAO,GAAG,iBAAiB,EAAE,CAAC;YAChC,MAAM,IAAI,KAAK,CACb,6BAA6B,OAAO,gBAAgB,iBAAiB,GAAG,CACzE,CAAC;QACJ,CAAC;QAED,oDAAoD;QACpD,uDAAuD;QACvD,OAAO;YACL,GAAG,OAAO;YACV,MAAM,EAAE;gBACN,IAAI,EAAE,OAAO,CAAC,MAAM,CAAC,IAAI;gBACzB,SAAS,EAAE,OAAO,CAAC,MAAM,CAAC,SAAS;aACpC;SACF,CAAC;IACJ,CAAC;IAED,gCAAgC;IAChC,MAAM,MAAM,GAAG,OAAO,CAAC,MAAwC,CAAC;IAChE,MAAM,EAAE,IAAI,EAAE,GAAG,aAAa,EAAE,GAAG,MAAM,CAAC;IAE1C,wEAAwE;IACxE,MAAM,mBAAmB,GAAG,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;IAElE,IAAI,mBAAmB,EAAE,CAAC;QACxB,yBAAyB;QACzB,MAAM,OAAO,GAAG,IAAI,CAAC,SAAS,CAAC,aAAa,CAAC,CAAC,MAAM,CAAC;QACrD,IAAI,OAAO,GAAG,iBAAiB,EAAE,CAAC;YAChC,MAAM,IAAI,KAAK,CACb,6BAA6B,OAAO,gBAAgB,iBAAiB,GAAG,CACzE,CAAC;QACJ,CAAC;QAED,kDAAkD;QAClD,IAAI,OAAO,CAAC,GAAG,CAAC,aAAa,CAAC,KAAK,MAAM,EAAE,CAAC;YAC1C,IAAI,CACF,oBAAoB,EACpB,iDAAiD,EACjD;gBACE,IAAI,EAAE,IAAI;gBACV,iBAAiB,EAAE,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC;gBAC9C,iBAAiB,EAAE,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC;gBAC7C,aAAa,EAAE,MAAM,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC,MAAM;aACjD,EACD,eAAe,CAChB,CAAC;QACJ,CAAC;QAED,mDAAmD;QACnD,OAAO;YACL,GAAG,OAAO;YACV,MAAM,EAAE;gBACN,IAAI;gBACJ,SAAS,EAAE,aAAa;aACzB;SACF,CAAC;IACJ,CAAC;IAED,sDAAsD;IACtD,OAAO;QACL,GAAG,OAAO;QACV,MAAM,EAAE;YACN,IAAI,EAAE,OAAO,CAAC,MAAM,CAAC,IAAI;YACzB,SAAS,EAAE,SAAS;SACrB;KACF,CAAC;AACJ,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,oBAAoB,CAClC,MAAc,EACd,OAAuB;IAEvB,6DAA6D;IAC7D,IAAI,OAAO,EAAE,CAAC;QACZ,gBAAgB,CAAC,OAAO,CAAC,CAAC;IAC5B,CAAC;IAED,sCAAsC;IACtC,MAAM,CAAC,iBAAiB,CAAC,sBAAsB,EAAE,KAAK,IAAI,EAAE;QAC1D,MAAM,EAAE,KAAK,EAAE,GAAG,mBAAmB,EAAE,CAAC;QACxC,OAAO;YACL,KAAK;SACN,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,4BAA4B;IAC5B,MAAM,CAAC,iBAAiB,CACtB,qBAAqB,EACrB,KAAK,EAAE,OAAO,EAA2B,EAAE;QACzC,MAAM,QAAQ,GACZ,OAAO,OAAO,CAAC,MAAM,EAAE,IAAI,KAAK,QAAQ;YACtC,CAAC,CAAC,OAAO,CAAC,MAAM,CAAC,IAAI;YACrB,CAAC,CAAC,cAAc,CAAC;QACrB,MAAM,aAAa,GAAG,qBAAqB,CAAC,QAAQ,CAAC,CAAC;QAEtD,IAAI,CAAC;YACH,qEAAqE;YACrE,4DAA4D;YAC5D,MAAM,iBAAiB,GAAG,oBAAoB,CAC5C,OAAiD,CAClD,CAAC;YACF,MAAM,MAAM,GAAG,CAAC,MAAM,kBAAkB,CACtC,iBAAiB,CAClB,CAAmB,CAAC;YACrB,OAAO,MAAM,CAAC;QAChB,CAAC;QAAC,OAAO,KAAc,EAAE,CAAC;YACxB,IAAI,CACF,oBAAoB,EACpB,sCAAsC,EACtC;gBACE,IAAI,EAAE,QAAQ;gBACd,aAAa;gBACb,YAAY,EACV,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC;aACzD,EACD,WAAW,EACX,aAAa,CAAC,cAAc,CAC7B,CAAC;YAEF,MAAM,EAAE,SAAS,EAAE,MAAM,EAAE,GAAG,aAAa,EAAE,CAAC;YAC9C,OAAO,2BAA2B,CAAC,KAAK,EAAE;gBACxC,MAAM,EAAE,gBAAgB;gBACxB,SAAS,EAAE,YAAY,QAAQ,EAAE;gBACjC,YAAY,EAAE,QAAQ;gBACtB,aAAa;gBACb,SAAS;gBACT,MAAM;gBACN,SAAS,EAAE,qBAAqB;gBAChC,aAAa,EACX,KAAK,YAAY,KAAK;oBACpB,CAAC,CAAC,KAAK,CAAC,OAAO;oBACf,CAAC,CAAC,mCAAmC;gBACzC,eAAe,EAAE,mCAAmC;gBACpD,UAAU,EACR,gGAAgG;aACnG,CAAC,CAAC;QACL,CAAC;gBAAS,CAAC;YACT,eAAe,EAAE,CAAC;QACpB,CAAC;IACH,CAAC,CACF,CAAC;AACJ,CAAC;AAED,gEAAgE;AAChE,OAAO,EAAE,gBAAgB,EAAE,YAAY,EAAE,MAAM,eAAe,CAAC;AAC/D,OAAO,EAAE,cAAc,EAAE,MAAM,eAAe,CAAC;AAC/C,OAAO,EAAE,kBAAkB,EAAE,MAAM,iBAAiB,CAAC;AACrD,OAAO,EACL,mBAAmB,EACnB,mBAAmB,EACnB,iBAAiB,EACjB,kBAAkB,EAClB,cAAc,GACf,MAAM,iBAAiB,CAAC"}