@toolplex/client 0.1.47 → 0.2.0

package/dist/mcp-server/index.js

@@ -27,7 +27,7 @@ const bundledDependencies = {
     git: process.env.TOOLPLEX_GIT_PATH,
 };
 // Parse session resume history for restored chat sessions
-// This allows the enforcement layer to validate save_playbook and submit_feedback
+// This allows the enforcement layer to validate save_playbook
 // based on historical tool usage from the database
 let sessionResumeHistory;
 if (process.env.TOOLPLEX_SESSION_RESUME_HISTORY) {

package/dist/mcp-server/policy/policyEnforcer.d.ts

@@ -1,10 +1,9 @@
 import { ClientContext } from "../clientContext.js";
 import CallToolObserver from "./callToolObserver.js";
 import InstallObserver from "./installObserver.js";
-import { SavePlaybookParams, SubmitFeedbackParams } from "../../shared/mcpServerTypes.js";
+import { SavePlaybookParams } from "../../shared/mcpServerTypes.js";
 export declare class PolicyEnforcer {
     private playbookPolicy;
-    private feedbackPolicy;
     private serverPolicy;
     private callToolObserver;
     private installObserver;
@@ -18,11 +17,6 @@ export declare class PolicyEnforcer {
      * Throws if the playbook does not pass policy.
      */
     enforceSavePlaybookPolicy(playbook: SavePlaybookParams): void;
-    /**
-     * Enforce feedback policy validation.
-     * Throws if the feedback does not pass policy.
-     */
-    enforceFeedbackPolicy(feedback: SubmitFeedbackParams): void;
     /**
      * Enforce server call tool policy validation.
      * Throws if attempting to call a tool on a blocked server.

package/dist/mcp-server/policy/policyEnforcer.js

@@ -1,12 +1,10 @@
 import { PlaybookPolicy } from "./playbookPolicy.js";
-import { FeedbackPolicy } from "./feedbackPolicy.js";
 import CallToolObserver from "./callToolObserver.js";
 import InstallObserver from "./installObserver.js";
 import { ServerPolicy } from "./serverPolicy.js";
 export class PolicyEnforcer {
     constructor() {
         this.playbookPolicy = null;
-        this.feedbackPolicy = null;
         this.serverPolicy = null;
         this.callToolObserver = null;
         this.installObserver = null;
@@ -18,7 +16,6 @@ export class PolicyEnforcer {
         this.callToolObserver = new CallToolObserver();
         this.installObserver = new InstallObserver();
         this.playbookPolicy = new PlaybookPolicy(clientContext, this.callToolObserver);
-        this.feedbackPolicy = new FeedbackPolicy(clientContext, this.callToolObserver, this.installObserver);
         this.serverPolicy = new ServerPolicy(clientContext);
     }
     /**
@@ -31,16 +28,6 @@ export class PolicyEnforcer {
         }
         this.playbookPolicy.enforceSavePlaybookPolicy(playbook);
     }
-    /**
-     * Enforce feedback policy validation.
-     * Throws if the feedback does not pass policy.
-     */
-    enforceFeedbackPolicy(feedback) {
-        if (!this.feedbackPolicy) {
-            throw new Error("PolicyEnforcer not initialized");
-        }
-        this.feedbackPolicy.enforceFeedbackPolicy(feedback);
-    }
     /**
      * Enforce server call tool policy validation.
      * Throws if attempting to call a tool on a blocked server.

package/dist/mcp-server/toolplexApi/service.d.ts

@@ -1,4 +1,4 @@
-import { CreatePlaybookResponse, LogPlaybookUsageResponse, SubmitFeedbackResponse, SecurityAssessment, FeedbackSummaryResponse, LogTelemetryRequest, LogTelemetryBatchResponse, InitResponse, SearchResponse, NotificationRecipient } from "./types.js";
+import { CreatePlaybookResponse, LogPlaybookUsageResponse, FeedbackSummaryResponse, LogTelemetryRequest, LogTelemetryBatchResponse, InitResponse, SearchResponse, NotificationRecipient } from "./types.js";
 import { ClientContext } from "../clientContext.js";
 import { Tool } from "@modelcontextprotocol/sdk/types.js";
 import { PlaybookAction } from "../../shared/mcpServerTypes.js";
@@ -24,7 +24,6 @@ export declare class ToolplexApiService {
     search(query: string, expandedKeywords?: string[], filter?: string, size?: number, scope?: string): Promise<SearchResponse>;
     createPlaybook(playbook_name: string, description: string, actions: Array<PlaybookAction>, domain?: string, keywords?: string[], requirements?: string[], privacy?: "public" | "private" | "organization", sourcePlaybookId?: string, forkReason?: string): Promise<CreatePlaybookResponse>;
     logPlaybookUsage(playbookId: string, success: boolean, errorMessage?: string): Promise<LogPlaybookUsageResponse>;
-    submitFeedback(targetType: "server" | "playbook", targetId: string, vote: "up" | "down", message?: string, securityAssessment?: SecurityAssessment): Promise<SubmitFeedbackResponse>;
     getFeedbackSummary(): Promise<FeedbackSummaryResponse>;
     /**
      * Create an automation notification (HITL)

package/dist/mcp-server/toolplexApi/service.js

@@ -191,29 +191,6 @@ export class ToolplexApiService {
             throw err;
         }
     }
-    async submitFeedback(targetType, targetId, vote, message, securityAssessment) {
-        const requestBody = {
-            target_type: targetType,
-            target_id: targetId,
-            vote,
-            message,
-            llm_context: this.clientContext.llmContext,
-            machine_context: this.machineContext,
-            security_assessment: securityAssessment,
-        };
-        try {
-            const response = await fetch(`${this.baseUrl}/feedback/submit`, {
-                method: "POST",
-                headers: this.getHeadersWithSession(),
-                body: JSON.stringify(requestBody),
-            });
-            return this.handleFetchResponse(response);
-        }
-        catch (err) {
-            await logger.error(`Error submitting feedback: ${err}`);
-            throw err;
-        }
-    }
     async getFeedbackSummary() {
         try {
             const response = await fetch(`${this.baseUrl}/feedback/summarize`, {

package/dist/mcp-server/toolplexApi/types.d.ts

@@ -48,16 +48,6 @@ export interface InitResponse {
     announcement?: string;
     flags: ClientFlags;
 }
-export type SecurityFlag = string;
-export type RiskLevel = string;
-export interface CustomSecurityFlag {
-    custom_flag: string;
-}
-export interface SecurityAssessment {
-    security_flags: Array<SecurityFlag | CustomSecurityFlag>;
-    risk_assessment: RiskLevel;
-    context_note?: string;
-}
 export interface LlmContext {
     model_family: string;
     model_name: string;
@@ -92,24 +82,6 @@ export interface LogPlaybookUsageRequest {
 export interface LogPlaybookUsageResponse {
     success: boolean;
 }
-export interface SubmitFeedbackRequest {
-    target_type: "server" | "playbook";
-    target_id: string;
-    vote: "up" | "down";
-    message?: string;
-    llm_context: LlmContext;
-    machine_context: {
-        os: string;
-        arch: string;
-        memory_gb: number;
-        cpu_cores: string;
-    };
-    security_assessment?: SecurityAssessment;
-}
-export interface SubmitFeedbackResponse {
-    id: string;
-    success: boolean;
-}
 export interface FeedbackSummaryResponse {
     servers: Array<{
         server_id: string;

package/dist/mcp-server/toolplexServer.js

@@ -18,12 +18,10 @@ import { handleUninstallServer } from "./toolHandlers/uninstallServerHandler.js"
 import { handleSavePlaybook } from "./toolHandlers/savePlaybookHandler.js";
 import { handleLogPlaybookUsage } from "./toolHandlers/logPlaybookUsageHandler.js";
 import { handleLookupEntityTool } from "./toolHandlers/lookupEntityHandler.js";
-import { handleSubmitFeedback } from "./toolHandlers/submitFeedbackHandler.js";
-import { handleGetServerConfig } from "./toolHandlers/getServerConfigHandler.js";
 import { handleNotify } from "./toolHandlers/notifyHandler.js";
 import { StdioServerManagerClient } from "../shared/stdioServerManagerClient.js";
 import { FileLogger } from "../shared/fileLogger.js";
-import { CallToolParamsSchema, InitializeToolplexParamsSchema, InstallParamsSchema, ListToolsParamsSchema, SearchParamsSchema, UninstallParamsSchema, SavePlaybookParamsSchema, LogPlaybookUsageParamsSchema, LookupEntityParamsSchema, SubmitFeedbackParamsSchema, GetServerConfigParamsSchema, NotifyParamsSchema, } from "../shared/mcpServerTypes.js";
+import { CallToolParamsSchema, InitializeToolplexParamsSchema, InstallParamsSchema, ListToolsParamsSchema, SearchParamsSchema, UninstallParamsSchema, SavePlaybookParamsSchema, LogPlaybookUsageParamsSchema, LookupEntityParamsSchema, NotifyParamsSchema, } from "../shared/mcpServerTypes.js";
 import { version as clientVersion } from "../version.js";
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const logger = FileLogger;
@@ -223,25 +221,6 @@ export async function serve(config) {
                     result = await handleLookupEntityTool(parsed.data);
                     break;
                 }
-                case "submit_feedback": {
-                    await logger.debug("Handling submit_feedback request");
-                    const parsed = SubmitFeedbackParamsSchema.safeParse(params);
-                    if (!parsed.success)
-                        throw new Error(`Invalid submit_feedback params: ${parsed.error}`);
-                    if (!clientContext.isInitialized())
-                        throw new Error(`ToolPlex is not initialized`);
-                    result = await handleSubmitFeedback(parsed.data);
-                    break;
-                }
-                // Add get_server_config tool handler
-                case "get_server_config": {
-                    await logger.debug("Handling get_server_config request");
-                    const parsed = GetServerConfigParamsSchema.safeParse(params);
-                    if (!parsed.success)
-                        throw new Error(`Invalid get_server_config params: ${parsed.error}`);
-                    result = await handleGetServerConfig(parsed.data);
-                    break;
-                }
                 // Notify tool handler (automation mode only)
                 case "notify": {
                     await logger.debug("Handling notify request");

package/dist/server-manager/stdioServer.js

@@ -103,15 +103,6 @@ export class ServerManagerProtocol {
                 const server_name = await this.serverManager.getServerName(list_tools_params.server_id);
                 return { server_id: list_tools_params.server_id, server_name, tools };
             }
-            case "get_server_config": {
-                if (!params || typeof params.server_id !== "string") {
-                    throw new Error("Missing or invalid server_id");
-                }
-                await logger.debug(`Getting config for server ${params.server_id}`);
-                // Just return the config directly
-                const config = await this.serverManager.getServerConfig(params.server_id);
-                return config;
-            }
             case "call_tool": {
                 const call_tool_params = CallToolParamsSchema.parse(params);
                 await logger.debug(`Calling tool ${call_tool_params.tool_name} on server ${call_tool_params.server_id}`);

package/dist/shared/mcpServerTypes.d.ts

@@ -234,14 +234,6 @@ export declare const ListToolsParamsSchema: z.ZodObject<{
     server_id?: string | undefined;
 }>;
 export type ListToolplexToolsParams = z.infer<typeof ListToolsParamsSchema>;
-export declare const GetServerConfigParamsSchema: z.ZodObject<{
-    server_id: z.ZodOptional<z.ZodString>;
-}, "strip", z.ZodTypeAny, {
-    server_id?: string | undefined;
-}, {
-    server_id?: string | undefined;
-}>;
-export type GetServerConfigParams = z.infer<typeof GetServerConfigParamsSchema>;
 export declare const CallToolParamsSchema: z.ZodObject<{
     server_id: z.ZodString;
     tool_name: z.ZodString;
@@ -397,60 +389,6 @@ export declare const LogPlaybookUsageParamsSchema: z.ZodObject<{
     error_message?: string | undefined;
 }>;
 export type LogPlaybookUsageParams = z.infer<typeof LogPlaybookUsageParamsSchema>;
-export declare const SubmitFeedbackParamsSchema: z.ZodObject<{
-    target_type: z.ZodEnum<["server", "playbook"]>;
-    target_id: z.ZodString;
-    vote: z.ZodEnum<["up", "down"]>;
-    message: z.ZodOptional<z.ZodString>;
-    security_assessment: z.ZodOptional<z.ZodObject<{
-        security_flags: z.ZodArray<z.ZodUnion<[z.ZodString, z.ZodObject<{
-            custom_flag: z.ZodString;
-        }, "strip", z.ZodTypeAny, {
-            custom_flag: string;
-        }, {
-            custom_flag: string;
-        }>]>, "many">;
-        risk_assessment: z.ZodString;
-        context_note: z.ZodOptional<z.ZodString>;
-    }, "strip", z.ZodTypeAny, {
-        security_flags: (string | {
-            custom_flag: string;
-        })[];
-        risk_assessment: string;
-        context_note?: string | undefined;
-    }, {
-        security_flags: (string | {
-            custom_flag: string;
-        })[];
-        risk_assessment: string;
-        context_note?: string | undefined;
-    }>>;
-}, "strip", z.ZodTypeAny, {
-    target_type: "server" | "playbook";
-    target_id: string;
-    vote: "up" | "down";
-    message?: string | undefined;
-    security_assessment?: {
-        security_flags: (string | {
-            custom_flag: string;
-        })[];
-        risk_assessment: string;
-        context_note?: string | undefined;
-    } | undefined;
-}, {
-    target_type: "server" | "playbook";
-    target_id: string;
-    vote: "up" | "down";
-    message?: string | undefined;
-    security_assessment?: {
-        security_flags: (string | {
-            custom_flag: string;
-        })[];
-        risk_assessment: string;
-        context_note?: string | undefined;
-    } | undefined;
-}>;
-export type SubmitFeedbackParams = z.infer<typeof SubmitFeedbackParamsSchema>;
 export declare const NotifyParamsSchema: z.ZodObject<{
     title: z.ZodString;
     content: z.ZodString;

package/dist/shared/mcpServerTypes.js

@@ -70,12 +70,6 @@ export const ListToolsParamsSchema = z.object({
     server_id: z.string().optional(),
 });
 // --------------------
-// GetServerConfigParams
-// --------------------
-export const GetServerConfigParamsSchema = z.object({
-    server_id: z.string().optional(),
-});
-// --------------------
 // CallToolParams
 // --------------------
 export const CallToolParamsSchema = z.object({
@@ -141,27 +135,6 @@ export const LogPlaybookUsageParamsSchema = z.object({
     error_message: z.string().optional(),
 });
 // --------------------
-// SubmitFeedbackParams
-// --------------------
-export const SubmitFeedbackParamsSchema = z.object({
-    target_type: z.enum(["server", "playbook"]),
-    target_id: z.string(),
-    vote: z.enum(["up", "down"]),
-    message: z.string().optional(),
-    security_assessment: z
-        .object({
-        security_flags: z.array(z.union([
-            z.string(),
-            z.object({
-                custom_flag: z.string(),
-            }),
-        ])),
-        risk_assessment: z.string(),
-        context_note: z.string().optional(),
-    })
-        .optional(),
-});
-// --------------------
 // NotifyParams (for automation HITL notifications)
 // --------------------
 export const NotifyParamsSchema = z.object({

package/dist/src/mcp-server/policy/policyEnforcer.js

@@ -1,12 +1,10 @@
 import { PlaybookPolicy } from "./playbookPolicy.js";
-import { FeedbackPolicy } from "./feedbackPolicy.js";
 import CallToolObserver from "./callToolObserver.js";
 import InstallObserver from "./installObserver.js";
 import { ServerPolicy } from "./serverPolicy.js";
 export class PolicyEnforcer {
     constructor() {
         this.playbookPolicy = null;
-        this.feedbackPolicy = null;
         this.serverPolicy = null;
         this.callToolObserver = null;
         this.installObserver = null;
@@ -18,7 +16,6 @@ export class PolicyEnforcer {
         this.callToolObserver = new CallToolObserver();
         this.installObserver = new InstallObserver();
         this.playbookPolicy = new PlaybookPolicy(clientContext, this.callToolObserver);
-        this.feedbackPolicy = new FeedbackPolicy(clientContext, this.callToolObserver, this.installObserver);
         this.serverPolicy = new ServerPolicy(clientContext);
     }
     /**
@@ -31,16 +28,6 @@ export class PolicyEnforcer {
         }
         this.playbookPolicy.enforceSavePlaybookPolicy(playbook);
     }
-    /**
-     * Enforce feedback policy validation.
-     * Throws if the feedback does not pass policy.
-     */
-    enforceFeedbackPolicy(feedback) {
-        if (!this.feedbackPolicy) {
-            throw new Error("PolicyEnforcer not initialized");
-        }
-        this.feedbackPolicy.enforceFeedbackPolicy(feedback);
-    }
     /**
      * Enforce server call tool policy validation.
      * Throws if attempting to call a tool on a blocked server.

package/dist/src/mcp-server/toolplexApi/service.js

@@ -191,29 +191,6 @@ export class ToolplexApiService {
             throw err;
         }
     }
-    async submitFeedback(targetType, targetId, vote, message, securityAssessment) {
-        const requestBody = {
-            target_type: targetType,
-            target_id: targetId,
-            vote,
-            message,
-            llm_context: this.clientContext.llmContext,
-            machine_context: this.machineContext,
-            security_assessment: securityAssessment,
-        };
-        try {
-            const response = await fetch(`${this.baseUrl}/feedback/submit`, {
-                method: "POST",
-                headers: this.getHeadersWithSession(),
-                body: JSON.stringify(requestBody),
-            });
-            return this.handleFetchResponse(response);
-        }
-        catch (err) {
-            await logger.error(`Error submitting feedback: ${err}`);
-            throw err;
-        }
-    }
     async getFeedbackSummary() {
         try {
             const response = await fetch(`${this.baseUrl}/feedback/summarize`, {

package/dist/src/shared/mcpServerTypes.js

@@ -70,12 +70,6 @@ export const ListToolsParamsSchema = z.object({
     server_id: z.string().optional(),
 });
 // --------------------
-// GetServerConfigParams
-// --------------------
-export const GetServerConfigParamsSchema = z.object({
-    server_id: z.string().optional(),
-});
-// --------------------
 // CallToolParams
 // --------------------
 export const CallToolParamsSchema = z.object({
@@ -141,27 +135,6 @@ export const LogPlaybookUsageParamsSchema = z.object({
     error_message: z.string().optional(),
 });
 // --------------------
-// SubmitFeedbackParams
-// --------------------
-export const SubmitFeedbackParamsSchema = z.object({
-    target_type: z.enum(["server", "playbook"]),
-    target_id: z.string(),
-    vote: z.enum(["up", "down"]),
-    message: z.string().optional(),
-    security_assessment: z
-        .object({
-        security_flags: z.array(z.union([
-            z.string(),
-            z.object({
-                custom_flag: z.string(),
-            }),
-        ])),
-        risk_assessment: z.string(),
-        context_note: z.string().optional(),
-    })
-        .optional(),
-});
-// --------------------
 // NotifyParams (for automation HITL notifications)
 // --------------------
 export const NotifyParamsSchema = z.object({

package/package.json

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

package/dist/mcp-server/policy/feedbackPolicy.d.ts

@@ -1,27 +0,0 @@
-import CallToolObserver from "./callToolObserver.js";
-import InstallObserver from "./installObserver.js";
-import { ClientContext } from "../clientContext.js";
-import { SubmitFeedbackParams } from "../../shared/mcpServerTypes.js";
-export declare class FeedbackPolicy {
-    private callToolObserver;
-    private installObserver;
-    private clientContext;
-    private blockedMcpServersSet;
-    constructor(clientContext: ClientContext, callToolObserver: CallToolObserver, installObserver: InstallObserver);
-    /**
-     * Validates feedback by checking that:
-     * - The referenced server has been used or had an install/uninstall action
-     * - The server is not in the blocked servers list
-     *
-     * For server feedback, verifies that:
-     * - The referenced server has been connected to and used or had an install/uninstall action
-     * - The server is not blocked
-     *
-     * For playbook feedback, no validation is currently performed
-     * since playbooks can reference other playbooks.
-     *
-     * @throws Error if feedback references a server that hasn't been used or had an install/uninstall action
-     * in the current session or if the server is blocked
-     */
-    enforceFeedbackPolicy(feedback: SubmitFeedbackParams): void;
-}

package/dist/mcp-server/policy/feedbackPolicy.js

@@ -1,39 +0,0 @@
-export class FeedbackPolicy {
-    constructor(clientContext, callToolObserver, installObserver) {
-        this.callToolObserver = callToolObserver;
-        this.installObserver = installObserver;
-        this.clientContext = clientContext;
-        this.blockedMcpServersSet = new Set(clientContext.flags.blocked_mcp_servers || []);
-    }
-    /**
-     * Validates feedback by checking that:
-     * - The referenced server has been used or had an install/uninstall action
-     * - The server is not in the blocked servers list
-     *
-     * For server feedback, verifies that:
-     * - The referenced server has been connected to and used or had an install/uninstall action
-     * - The server is not blocked
-     *
-     * For playbook feedback, no validation is currently performed
-     * since playbooks can reference other playbooks.
-     *
-     * @throws Error if feedback references a server that hasn't been used or had an install/uninstall action
-     * in the current session or if the server is blocked
-     */
-    enforceFeedbackPolicy(feedback) {
-        const { target_type, target_id } = feedback;
-        if (target_type === "server") {
-            // Check if server is blocked
-            if (this.blockedMcpServersSet.has(target_id)) {
-                throw new Error(`Cannot submit feedback for blocked server "${target_id}"`);
-            }
-            // For server feedback, verify the server was actually used or had an install/uninstall action
-            if (!this.callToolObserver.wasServerCalled(target_id) &&
-                !this.installObserver.wasServerInstalled(target_id) &&
-                !this.installObserver.wasServerUninstalled(target_id)) {
-                throw new Error(`Cannot submit feedback for server "${target_id}" which has not been used or had an install/uninstall action in this session.`);
-            }
-        }
-        // For playbook feedback, we don't validate usage since playbooks can reference other playbooks
-    }
-}

package/dist/mcp-server/toolHandlers/getServerConfigHandler.d.ts

@@ -1,3 +0,0 @@
-import { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
-import { GetServerConfigParams } from "../../shared/mcpServerTypes.js";
-export declare function handleGetServerConfig(params: GetServerConfigParams): Promise<CallToolResult>;

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

@@ -1,73 +0,0 @@
-import { FileLogger } from "../../shared/fileLogger.js";
-import { findServerManagerClient } from "./serverManagerUtils.js";
-import { sanitizeServerIdForLogging, validateServerIdOrThrow, } from "../utils/serverIdValidator.js";
-import Registry from "../registry.js";
-const logger = FileLogger;
-export async function handleGetServerConfig(params) {
-    const startTime = Date.now();
-    const serverManagerClients = Registry.getServerManagerClients();
-    const telemetryLogger = Registry.getTelemetryLogger();
-    const promptsCache = Registry.getPromptsCache();
-    const policyEnforcer = Registry.getPolicyEnforcer();
-    try {
-        const server_id = params.server_id;
-        if (!server_id) {
-            throw new Error("Missing server_id");
-        }
-        // Validate server ID format
-        validateServerIdOrThrow(server_id);
-        // Check if server is blocked using policy enforcer
-        policyEnforcer.enforceUseServerPolicy(server_id);
-        await logger.debug(`Getting config for server: ${server_id}`);
-        const client = await findServerManagerClient(server_id, serverManagerClients);
-        const response_data = await client.sendRequest("get_server_config", {
-            server_id,
-        });
-        if ("error" in response_data) {
-            throw new Error(`Failed to get config for server_id ${server_id}, error message: ${response_data.error.message}`);
-        }
-        // The config is returned directly as an object
-        const config = response_data;
-        await logger.debug("Successfully retrieved server config");
-        await telemetryLogger.log("client_get_server_config", {
-            success: true,
-            log_context: {
-                server_id: sanitizeServerIdForLogging(server_id),
-            },
-            latency_ms: Date.now() - startTime,
-        });
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: JSON.stringify(config, null, 2),
-                },
-            ],
-        };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error
-            ? error.message
-            : promptsCache.getPrompt("unexpected_error");
-        await logger.error(`Failed to get server config: ${errorMessage}`);
-        await telemetryLogger.log("client_get_server_config", {
-            success: false,
-            log_context: {
-                server_id: sanitizeServerIdForLogging(params.server_id || ""),
-            },
-            pii_sanitized_error_message: errorMessage,
-            latency_ms: Date.now() - startTime,
-        });
-        return {
-            isError: true,
-            content: [
-                {
-                    type: "text",
-                    text: promptsCache
-                        .getPrompt("get_server_config_failure")
-                        .replace("{ERROR}", errorMessage),
-                },
-            ],
-        };
-    }
-}

package/dist/mcp-server/toolHandlers/submitFeedbackHandler.d.ts

@@ -1,3 +0,0 @@
-import { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
-import { SubmitFeedbackParams } from "../../shared/mcpServerTypes.js";
-export declare function handleSubmitFeedback(params: SubmitFeedbackParams): Promise<CallToolResult>;

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

@@ -1,71 +0,0 @@
-import { FileLogger } from "../../shared/fileLogger.js";
-import Registry from "../registry.js";
-const logger = FileLogger;
-export async function handleSubmitFeedback(params) {
-    const startTime = Date.now();
-    await logger.info("Handling submit feedback request");
-    await logger.debug(`Feedback params: ${JSON.stringify(params)}`);
-    const { target_type, target_id, vote, message, security_assessment } = params;
-    const apiService = Registry.getToolplexApiService();
-    const telemetryLogger = Registry.getTelemetryLogger();
-    const promptsCache = Registry.getPromptsCache();
-    const policyEnforcer = Registry.getPolicyEnforcer();
-    const clientContext = Registry.getClientContext();
-    try {
-        // Check if the client is in restricted mode
-        if (clientContext.clientMode === "restricted") {
-            throw new Error("Feedback functionality is disabled in restricted mode.");
-        }
-        // Check if read-only mode is enabled
-        if (clientContext.permissions.enable_read_only_mode) {
-            throw new Error("Saving playbooks is disabled in read-only mode");
-        }
-        // Enforce feedback policy before submitting
-        policyEnforcer.enforceFeedbackPolicy(params);
-        const response = await apiService.submitFeedback(target_type, target_id, vote, message, security_assessment);
-        await logger.info(`Feedback submitted successfully for ${target_type} ${target_id}`);
-        await telemetryLogger.log("client_submit_feedback", {
-            success: true,
-            log_context: {
-                target_type,
-                target_id,
-                feedback_id: response.id,
-            },
-            latency_ms: Date.now() - startTime,
-        });
-        return {
-            content: [
-                {
-                    type: "text",
-                    text: promptsCache
-                        .getPrompt("submit_feedback_success")
-                        .replace("{FEEDBACK_ID}", response.id),
-                },
-            ],
-        };
-    }
-    catch (error) {
-        const errorMessage = error instanceof Error ? error.message : String(error);
-        await logger.error(`Failed to submit feedback: ${errorMessage}`);
-        await telemetryLogger.log("client_submit_feedback", {
-            success: false,
-            log_context: {
-                target_type,
-                target_id,
-            },
-            pii_sanitized_error_message: errorMessage,
-            latency_ms: Date.now() - startTime,
-        });
-        return {
-            isError: true,
-            content: [
-                {
-                    type: "text",
-                    text: promptsCache
-                        .getPrompt("unexpected_error")
-                        .replace("{ERROR}", errorMessage),
-                },
-            ],
-        };
-    }
-}

package/dist/src/mcp-server/policy/feedbackPolicy.js

@@ -1,39 +0,0 @@
-export class FeedbackPolicy {
-    constructor(clientContext, callToolObserver, installObserver) {
-        this.callToolObserver = callToolObserver;
-        this.installObserver = installObserver;
-        this.clientContext = clientContext;
-        this.blockedMcpServersSet = new Set(clientContext.flags.blocked_mcp_servers || []);
-    }
-    /**
-     * Validates feedback by checking that:
-     * - The referenced server has been used or had an install/uninstall action
-     * - The server is not in the blocked servers list
-     *
-     * For server feedback, verifies that:
-     * - The referenced server has been connected to and used or had an install/uninstall action
-     * - The server is not blocked
-     *
-     * For playbook feedback, no validation is currently performed
-     * since playbooks can reference other playbooks.
-     *
-     * @throws Error if feedback references a server that hasn't been used or had an install/uninstall action
-     * in the current session or if the server is blocked
-     */
-    enforceFeedbackPolicy(feedback) {
-        const { target_type, target_id } = feedback;
-        if (target_type === "server") {
-            // Check if server is blocked
-            if (this.blockedMcpServersSet.has(target_id)) {
-                throw new Error(`Cannot submit feedback for blocked server "${target_id}"`);
-            }
-            // For server feedback, verify the server was actually used or had an install/uninstall action
-            if (!this.callToolObserver.wasServerCalled(target_id) &&
-                !this.installObserver.wasServerInstalled(target_id) &&
-                !this.installObserver.wasServerUninstalled(target_id)) {
-                throw new Error(`Cannot submit feedback for server "${target_id}" which has not been used or had an install/uninstall action in this session.`);
-            }
-        }
-        // For playbook feedback, we don't validate usage since playbooks can reference other playbooks
-    }
-}