@toolplex/client 0.1.23 → 0.1.25

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

@@ -3,7 +3,8 @@ import Registry from "../registry.js";
 const logger = FileLogger;
 export async function handleSavePlaybook(params) {
     const startTime = Date.now();
-    await logger.info("Handling save playbook request");
+    const isValidationOnly = params.validate_only === true;
+    await logger.info(`Handling save playbook request${isValidationOnly ? " (validation only)" : ""}`);
     await logger.debug(`Playbook params: ${JSON.stringify(params)}`);
     const apiService = Registry.getToolplexApiService();
     const telemetryLogger = Registry.getTelemetryLogger();
@@ -19,19 +20,34 @@ export async function handleSavePlaybook(params) {
         if (clientContext.permissions.enable_read_only_mode) {
             throw new Error("Saving playbooks is disabled in read-only mode");
         }
-        // Enforce playbook policy before saving
+        // Enforce playbook policy before saving (runs in both validation and actual save)
         policyEnforcer.enforceSavePlaybookPolicy(params);
+        // If validation-only mode, return success without saving or logging telemetry
+        if (isValidationOnly) {
+            await logger.info("Playbook validation passed");
+            return {
+                content: [
+                    {
+                        type: "text",
+                        text: JSON.stringify({ validation: "passed" }),
+                    },
+                ],
+            };
+        }
         const { playbook_name, description, actions, domain, keywords, requirements, privacy, source_playbook_id, fork_reason, } = params;
         const response = await apiService.createPlaybook(playbook_name, description, actions, domain, keywords, requirements, privacy, source_playbook_id, fork_reason);
         await logger.info(`Playbook created successfully with ID: ${response.id}`);
-        await telemetryLogger.log("client_save_playbook", {
-            success: true,
-            log_context: {
-                playbook_id: response.id,
-                source_playbook_id: source_playbook_id,
-            },
-            latency_ms: Date.now() - startTime,
-        });
+        // Only log telemetry for actual saves (not validation-only calls)
+        if (!isValidationOnly) {
+            await telemetryLogger.log("client_save_playbook", {
+                success: true,
+                log_context: {
+                    playbook_id: response.id,
+                    source_playbook_id: source_playbook_id,
+                },
+                latency_ms: Date.now() - startTime,
+            });
+        }
         return {
             content: [
                 {
@@ -45,12 +61,15 @@ export async function handleSavePlaybook(params) {
     }
     catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
-        await logger.error(`Failed to create playbook: ${errorMessage}`);
-        await telemetryLogger.log("client_save_playbook", {
-            success: false,
-            pii_sanitized_error_message: errorMessage,
-            latency_ms: Date.now() - startTime,
-        });
+        await logger.error(`Failed to ${isValidationOnly ? "validate" : "create"} playbook: ${errorMessage}`);
+        // Only log telemetry for actual saves (not validation-only calls)
+        if (!isValidationOnly) {
+            await telemetryLogger.log("client_save_playbook", {
+                success: false,
+                pii_sanitized_error_message: errorMessage,
+                latency_ms: Date.now() - startTime,
+            });
+        }
         return {
             isError: true,
             content: [

package/dist/mcp-server/toolplexServer.js

@@ -219,6 +219,7 @@ export async function serve(config) {
             }
             await logger.error(`Error calling ToolPlex: ${errorMessage}`);
             result = {
+                isError: true,
                 role: "system",
                 content: [
                     {

package/dist/shared/mcpServerTypes.d.ts

@@ -316,6 +316,7 @@ export declare const SavePlaybookParamsSchema: z.ZodObject<{
     privacy: z.ZodOptional<z.ZodEnum<["public", "private"]>>;
     source_playbook_id: z.ZodOptional<z.ZodString>;
     fork_reason: z.ZodOptional<z.ZodString>;
+    validate_only: z.ZodOptional<z.ZodBoolean>;
 }, "strip", z.ZodTypeAny, {
     description: string;
     playbook_name: string;
@@ -333,6 +334,7 @@ export declare const SavePlaybookParamsSchema: z.ZodObject<{
     privacy?: "public" | "private" | undefined;
     source_playbook_id?: string | undefined;
     fork_reason?: string | undefined;
+    validate_only?: boolean | undefined;
 }, {
     description: string;
     playbook_name: string;
@@ -350,6 +352,7 @@ export declare const SavePlaybookParamsSchema: z.ZodObject<{
     privacy?: "public" | "private" | undefined;
     source_playbook_id?: string | undefined;
     fork_reason?: string | undefined;
+    validate_only?: boolean | undefined;
 }>;
 export type SavePlaybookParams = z.infer<typeof SavePlaybookParamsSchema>;
 export declare const LogPlaybookUsageParamsSchema: z.ZodObject<{

package/dist/shared/mcpServerTypes.js

@@ -126,6 +126,9 @@ export const SavePlaybookParamsSchema = z.object({
     privacy: z.enum(["public", "private"]).optional(),
     source_playbook_id: z.string().optional(),
     fork_reason: z.string().optional(),
+    // Internal parameter for validation-only mode (not exposed to agent in tool definition)
+    // Use coerce to handle both boolean and string "true"/"false" from different LLM clients
+    validate_only: z.coerce.boolean().optional(),
 });
 // --------------------
 // LogPlaybookUsageParams

package/dist/src/shared/mcpServerTypes.js

@@ -126,6 +126,9 @@ export const SavePlaybookParamsSchema = z.object({
     privacy: z.enum(["public", "private"]).optional(),
     source_playbook_id: z.string().optional(),
     fork_reason: z.string().optional(),
+    // Internal parameter for validation-only mode (not exposed to agent in tool definition)
+    // Use coerce to handle both boolean and string "true"/"false" from different LLM clients
+    validate_only: z.coerce.boolean().optional(),
 });
 // --------------------
 // LogPlaybookUsageParams

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "0.1.23";
+export declare const version = "0.1.25";

package/dist/version.js

@@ -1 +1 @@
-export const version = '0.1.23';
+export const version = '0.1.25';

package/package.json

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