@toolplex/client 0.1.7 → 0.1.9

package/dist/mcp-server/toolHandlers/callToolHandler.js

@@ -56,7 +56,6 @@ export async function handleCallTool(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: content,
         };
     }
@@ -75,7 +74,7 @@ export async function handleCallTool(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/getServerConfigHandler.js

@@ -34,7 +34,6 @@ export async function handleGetServerConfig(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -61,7 +60,7 @@ export async function handleGetServerConfig(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/initHandler.js

@@ -51,7 +51,6 @@ export async function handleInitialize(params) {
     await logger.debug("Building initialization response");
     // Safe to use prompts after init.
     const result = {
-        role: "system",
         content: [
             {
                 type: "text",

package/dist/mcp-server/toolHandlers/installServerHandler.js

@@ -2,7 +2,71 @@ import { FileLogger } from "../../shared/fileLogger.js";
 import { ServerInstallResultSchema, ListToolsResultSchema, } from "../../shared/serverManagerTypes.js";
 import Registry from "../registry.js";
 import { RuntimeCheck } from "../utils/runtimeCheck.js";
+import { isAbsolute, parse } from "path";
 const logger = FileLogger;
+/**
+ * Sanitizes ServerConfig for telemetry logging by extracting aggregate patterns
+ * while avoiding PII exposure. This function extracts useful installation patterns
+ * without logging sensitive data like file paths, API keys, or user-specific values.
+ *
+ * SECURITY: This function only logs metadata patterns, never actual values:
+ * - Command names (not paths): "npx" vs "/Users/john/bin/tool"
+ * - Argument flags (not values): "--port" vs actual port numbers
+ * - Environment variable names (not values): "API_KEY" vs actual keys
+ * - Path types for portability analysis: "absolute" vs "package_manager"
+ */
+function sanitizeServerConfig(config) {
+    // Extract command executable name without sensitive path information
+    const extractCommandType = (command) => {
+        if (!command)
+            return "none";
+        // For absolute paths, extract only the executable name (e.g., "/usr/bin/node" -> "node")
+        if (isAbsolute(command)) {
+            return parse(command).name;
+        }
+        // For relative commands, get the base command (e.g., "npx" from "npx --version")
+        return (command
+            .split(/[\s/\\]/)
+            .pop()
+            ?.split(".")[0] || "unknown");
+    };
+    // Categorize path types for portability analysis - helps identify installation reliability patterns
+    const detectPathType = (command, args) => {
+        if (!command)
+            return "system_command";
+        // Absolute paths indicate potential portability issues
+        if (isAbsolute(command) || args?.some((arg) => isAbsolute(arg))) {
+            return "absolute";
+        }
+        // Package managers are typically more reliable across systems
+        if (["npx", "uvx", "pip", "yarn", "pnpm"].includes(command)) {
+            return "package_manager";
+        }
+        return "system_command";
+    };
+    // Extract common argument flags and patterns (not values) for usage analysis
+    const extractArgPatterns = (args) => {
+        return (args?.filter((arg) => arg.startsWith("-") || // Command flags like --port, --config
+            ["stdio", "mcp", "start", "latest", "@latest"].includes(arg)) || []);
+    };
+    // Extract environment variable names (not values) to understand integration patterns
+    // SAFE: Only logs key names like "API_KEY", "DATABASE_URL" - never the actual values
+    const extractEnvKeys = (env) => {
+        if (!env)
+            return [];
+        return Object.keys(env).sort();
+    };
+    return {
+        runtime: config.runtime || "node",
+        transport: config.transport,
+        command_type: extractCommandType(config.command),
+        path_type: detectPathType(config.command, config.args),
+        arg_patterns: extractArgPatterns(config.args),
+        arg_count: config.args?.length || 0,
+        env_keys: extractEnvKeys(config.env),
+        env_count: config.env ? Object.keys(config.env).length : 0,
+    };
+}
 async function installServer(serverId, serverName, description, serverManagerClient, serverConfig) {
     await logger.info(`Starting installation of tool ${serverId}: ${serverName}`);
     await logger.debug(`Server config: ${JSON.stringify(serverConfig)}, Server ID: ${serverId}`);
@@ -91,6 +155,7 @@ export async function handleInstallServer(params) {
             success: true,
             log_context: {
                 server_id: installResult.server_id,
+                sanitized_config: sanitizeServerConfig(config),
             },
             latency_ms: Date.now() - startTime,
         });
@@ -111,7 +176,6 @@ export async function handleInstallServer(params) {
             });
         }
         return {
-            role: "system",
             content,
         };
     }
@@ -144,7 +208,7 @@ export async function handleInstallServer(params) {
             });
         }
         return {
-            role: "system",
+            isError: true,
             content,
         };
     }

package/dist/mcp-server/toolHandlers/listServersHandler.js

@@ -46,7 +46,6 @@ export async function handleListServers() {
         // Build response content
         if (allServers.length === 0) {
             return {
-                role: "system",
                 content: [
                     {
                         type: "text",
@@ -56,7 +55,6 @@ export async function handleListServers() {
             };
         }
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -83,7 +81,7 @@ export async function handleListServers() {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/listToolsHandler.js

@@ -110,7 +110,6 @@ export async function handleListTools(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content,
         };
     }
@@ -128,7 +127,7 @@ export async function handleListTools(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/logPlaybookUsageHandler.js

@@ -29,7 +29,6 @@ export async function handleLogPlaybookUsage(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -51,7 +50,7 @@ export async function handleLogPlaybookUsage(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/lookupEntityHandler.js

@@ -46,7 +46,7 @@ export async function handleLookupEntityTool(params) {
                 latency_ms: Date.now() - startTime,
             });
             return {
-                role: "system",
+                isError: true,
                 content: [
                     {
                         type: "text",
@@ -92,7 +92,6 @@ export async function handleLookupEntityTool(params) {
             });
         }
         return {
-            role: "system",
             content,
         };
     }
@@ -109,7 +108,7 @@ export async function handleLookupEntityTool(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/savePlaybookHandler.js

@@ -33,7 +33,6 @@ export async function handleSavePlaybook(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -53,7 +52,7 @@ export async function handleSavePlaybook(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/searchHandler.js

@@ -55,7 +55,6 @@ export async function handleSearchTool(params) {
         if (totalResults === 0) {
             await logger.info("No search results found");
             return {
-                role: "system",
                 content: [
                     {
                         type: "text",
@@ -91,7 +90,6 @@ export async function handleSearchTool(params) {
         ];
         await logger.info("Search completed successfully");
         return {
-            role: "system",
             content,
         };
     }
@@ -110,7 +108,7 @@ export async function handleSearchTool(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/submitFeedbackHandler.js

@@ -34,7 +34,6 @@ export async function handleSubmitFeedback(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -58,7 +57,7 @@ export async function handleSubmitFeedback(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/mcp-server/toolHandlers/uninstallServerHandler.js

@@ -41,7 +41,6 @@ export async function handleUninstallServer(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
             content: [
                 {
                     type: "text",
@@ -67,7 +66,7 @@ export async function handleUninstallServer(params) {
             latency_ms: Date.now() - startTime,
         });
         return {
-            role: "system",
+            isError: true,
             content: [
                 {
                     type: "text",

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "0.1.7";
+export declare const version = "0.1.9";

package/dist/version.js

@@ -1 +1 @@
-export const version = '0.1.7';
+export const version = '0.1.9';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@toolplex/client",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "author": "ToolPlex LLC",
   "license": "SEE LICENSE IN LICENSE",
   "description": "The official ToolPlex client for AI agent tool discovery and execution",