@hailer/mcp 1.0.29 → 1.1.2

package/dist/mcp/tools/workflow.js

@@ -6,10 +6,12 @@
  * These are PLAYGROUND tools requiring workspace administrator permissions.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.updateWorkflowPhaseTool = exports.countActivitiesTool = exports.listWorkflowsMinimalTool = exports.testFunctionFieldTool = exports.updateWorkflowFieldTool = exports.removeWorkflowTool = exports.installWorkflowTool = exports.listWorkflowsTool = exports.listWorkflowPhasesTool = exports.getWorkflowSchemaTool = void 0;
+exports.updateWorkflowPhaseTool = exports.coreInitTool = exports.countActivitiesTool = exports.listWorkflowsMinimalTool = exports.testFunctionFieldTool = exports.updateWorkflowFieldTool = exports.removeWorkflowTool = exports.installWorkflowTool = exports.listWorkflowsTool = exports.listWorkflowPhasesTool = exports.getWorkflowSchemaTool = void 0;
 const zod_1 = require("zod");
 const tool_registry_1 = require("../tool-registry");
 const logger_1 = require("../../lib/logger");
+const request_logger_1 = require("../../lib/request-logger");
+const workspace_overview_1 = require("../../bot/workspace-overview");
 const logger = (0, logger_1.createLogger)({ component: 'workflow-tools' });
 // ============================================================================
 // HELPER FUNCTIONS
@@ -166,7 +168,8 @@ exports.getWorkflowSchemaTool = {
             };
         }
         catch (error) {
-            logger.error("Failed to get workflow schema", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Failed to get workflow schema", error);
             return {
                 content: [{
                         type: "text",
@@ -244,7 +247,8 @@ exports.listWorkflowPhasesTool = {
             };
         }
         catch (error) {
-            logger.error("Failed to list workflow phases", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Failed to list workflow phases", error);
             return {
                 content: [{
                         type: "text",
@@ -345,7 +349,8 @@ exports.listWorkflowsTool = {
             };
         }
         catch (error) {
-            logger.error("Error listing workflows", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error listing workflows", error);
             return {
                 content: [{
                         type: "text",
@@ -388,7 +393,7 @@ const installWorkflowSchema = zod_1.z.object({
             ]).describe("Field type"),
             key: zod_1.z.string().optional().describe("Readable field name (like SQL column name) - RECOMMENDED"),
             required: zod_1.z.boolean().optional().describe("Whether field is required"),
-            data: zod_1.z.array(zod_1.z.string()).optional().describe("For textpredefinedoptions: string array like [\"Low\", \"Medium\", \"High\"]. For activitylink: target workflow IDs"),
+            data: zod_1.z.array(zod_1.z.string()).optional().describe("For textpredefinedoptions: string array like [\"Low\", \"Medium\", \"High\"]. For activitylink: target workflow IDs. NOTE: 'options' also accepted, will be converted to 'data'"),
             placeholder: zod_1.z.string().optional().describe("Placeholder text"),
             unit: zod_1.z.string().optional().describe("Unit for numeric/textunit fields"),
             description: zod_1.z.string().optional().describe("Field description"),
@@ -441,14 +446,61 @@ exports.installWorkflowTool = {
                         }],
                 };
             }
+            // Transform templates: fix common LLM mistakes
+            const transformedTemplates = args.workflowTemplates.map((template) => {
+                let result = { ...template };
+                // Transform fields
+                if (template.fields) {
+                    const transformedFields = {};
+                    for (const [fieldId, field] of Object.entries(template.fields)) {
+                        const f = { ...field };
+                        // Convert options/predefinedOptions/selectOptions → data
+                        if (!f.data) {
+                            if (f.options) {
+                                f.data = f.options;
+                                delete f.options;
+                            }
+                            else if (f.predefinedOptions) {
+                                f.data = f.predefinedOptions;
+                                delete f.predefinedOptions;
+                            }
+                            else if (f.selectOptions) {
+                                // Handle both array and object selectOptions
+                                f.data = Array.isArray(f.selectOptions)
+                                    ? f.selectOptions.map((o) => o.label || o.value || o)
+                                    : Object.values(f.selectOptions);
+                                delete f.selectOptions;
+                            }
+                        }
+                        // Remove disallowed properties
+                        delete f.name; // Only 'label' is allowed
+                        delete f.sequence; // Not supported
+                        delete f.order; // Not supported
+                        transformedFields[fieldId] = f;
+                    }
+                    result.fields = transformedFields;
+                }
+                // Transform phases - remove sequence/order
+                if (template.phases) {
+                    const transformedPhases = {};
+                    for (const [phaseId, phase] of Object.entries(template.phases)) {
+                        const p = { ...phase };
+                        delete p.sequence;
+                        delete p.order;
+                        transformedPhases[phaseId] = p;
+                    }
+                    result.phases = transformedPhases;
+                }
+                return result;
+            });
             logger.debug('Calling v3.workflow.install', {
                 workspaceId,
-                templatesCount: args.workflowTemplates.length
+                templatesCount: transformedTemplates.length
             });
             // Call v3.workflow.install endpoint
             const result = await context.hailer.request('v3.workflow.install', [
                 workspaceId,
-                args.workflowTemplates
+                transformedTemplates
             ]);
             logger.debug('Workflow installation successful', { result });
             // Refresh context.init.processes cache so new workflows are immediately available
@@ -456,7 +508,7 @@ exports.installWorkflowTool = {
                 const freshInit = await context.client.socket.request('v2.core.init', [['processes']]);
                 if (freshInit.processes) {
                     context.init.processes = freshInit.processes;
-                    logger.info('Refreshed workflow cache after installation', {
+                    logger.debug('Refreshed workflow cache after installation', {
                         workflowCount: freshInit.processes.length
                     });
                 }
@@ -465,46 +517,68 @@ exports.installWorkflowTool = {
                 logger.warn('Failed to refresh workflow cache after installation', { error: refreshError });
                 // Non-fatal - the workflow was created, just cache is stale
             }
-            // Build success response
-            let responseText = `✅ **Workflow Installation Successful**\n\n`;
-            responseText += `**Workspace:** ${workspaceId}\n`;
-            responseText += `**Workflows Installed:** ${args.workflowTemplates.length}\n\n`;
-            // Show workflow names and their real IDs
-            const templates = args.workflowTemplates;
-            const workflowMappings = templates
-                .filter((wf) => wf._id)
-                .map((wf) => `- ${wf.name}: \`${result[wf._id]}\` (from workflow _id \`${wf._id}\`)`)
-                .join('\n');
-            if (workflowMappings) {
-                responseText += `**Created Workflows:**\n${workflowMappings}\n\n`;
+            // Build success response - clearly separate workflow/field/phase IDs
+            const templates = transformedTemplates;
+            // Extract workflow ID (the one that starts with workflow template ID or is for the workflow)
+            // The result maps template IDs to real IDs
+            const workflowIds = [];
+            const fieldIds = {};
+            const phaseIds = {};
+            for (const [templateId, realId] of Object.entries(result)) {
+                if (templateId.startsWith('_0') || templateId.startsWith('_00')) {
+                    // Workflow ID (_0001, _0002, etc.)
+                    workflowIds.push(realId);
+                }
+                else if (templateId.startsWith('_1')) {
+                    // Field ID (_1000, _1001, etc.)
+                    fieldIds[templateId] = realId;
+                }
+                else if (templateId.startsWith('_2')) {
+                    // Phase ID (_2000, _2001, etc.)
+                    phaseIds[templateId] = realId;
+                }
+                else {
+                    // Fallback - assume workflow ID if no underscore pattern
+                    workflowIds.push(realId);
+                }
             }
-            responseText += `**Complete ID Mapping (template_id → real_id):**\n`;
-            responseText += `\`\`\`json\n${JSON.stringify(result, null, 2)}\n\`\`\`\n\n`;
-            responseText += `💡 **Next Steps:**\n`;
-            responseText += `- Use \`list_workflows\` to see the new workflows in context\n`;
-            responseText += `- Use \`create_activity\` with the real workflow IDs to create activities\n`;
-            responseText += `- Field IDs from mapping are used when setting activity field values\n\n`;
-            responseText += `**Example Activity Creation:**\n`;
-            responseText += `\`\`\`javascript\n`;
-            if (templates[0]?._id) {
-                const firstTemplate = templates[0];
-                const workflowId = result[firstTemplate._id];
-                const firstFieldId = Object.keys(firstTemplate.fields || {})[0];
-                const realFieldId = firstFieldId ? result[firstFieldId] : 'field_id';
-                responseText += `// Using workflow: ${firstTemplate.name}\n`;
-                responseText += `create_activity({\n`;
-                responseText += `  workflowId: "${workflowId}",\n`;
-                responseText += `  name: "New Activity",\n`;
-                responseText += `  fields: { "${realFieldId}": "value" }\n`;
-                responseText += `});\n`;
+            let responseText = `✅ **Workflow "${templates[0]?.name || 'Unknown'}" Created**\n\n`;
+            // MOST IMPORTANT - Workflow ID for create_activity
+            responseText += `**🎯 WORKFLOW ID (use this for create_activity):**\n`;
+            responseText += `\`${workflowIds[0] || 'check list_workflows'}\`\n\n`;
+            // Field IDs
+            if (Object.keys(fieldIds).length > 0) {
+                responseText += `**📋 Field IDs:**\n`;
+                for (const [tpl, real] of Object.entries(fieldIds)) {
+                    const fieldDef = templates[0]?.fields?.[tpl];
+                    const label = fieldDef?.label || tpl;
+                    responseText += `- ${label}: \`${real}\`\n`;
+                }
+                responseText += `\n`;
             }
-            else {
-                responseText += `create_activity({\n`;
-                responseText += `  workflowId: result._0001,\n`;
-                responseText += `  name: "New Activity",\n`;
-                responseText += `  fields: { [result._1000]: "value" }\n`;
-                responseText += `});\n`;
+            // Phase IDs
+            if (Object.keys(phaseIds).length > 0) {
+                responseText += `**🏷️ Phase IDs:**\n`;
+                for (const [tpl, real] of Object.entries(phaseIds)) {
+                    const phaseDef = templates[0]?.phases?.[tpl];
+                    const name = phaseDef?.name || tpl;
+                    responseText += `- ${name}: \`${real}\`\n`;
+                }
+                responseText += `\n`;
             }
+            // Example
+            const firstFieldId = Object.values(fieldIds)[0];
+            const firstPhaseId = Object.values(phaseIds)[0];
+            responseText += `**Example create_activity:**\n`;
+            responseText += `\`\`\`json\n`;
+            responseText += `{\n`;
+            responseText += `  "workflowId": "${workflowIds[0] || 'WORKFLOW_ID'}",\n`;
+            responseText += `  "name": "New Activity",\n`;
+            if (firstPhaseId)
+                responseText += `  "phaseId": "${firstPhaseId}",\n`;
+            if (firstFieldId)
+                responseText += `  "fields": { "${firstFieldId}": "value" }\n`;
+            responseText += `}\n`;
             responseText += `\`\`\``;
             return {
                 content: [{
@@ -515,7 +589,8 @@ exports.installWorkflowTool = {
         }
         catch (error) {
             const errorMessage = error instanceof Error ? error.message : String(error);
-            logger.error("Error installing workflow", new Error(errorMessage));
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error installing workflow", new Error(errorMessage));
             // Handle permission errors
             if (errorMessage.includes('network admin') || errorMessage.includes('PermissionDenied')) {
                 return {
@@ -646,7 +721,8 @@ exports.removeWorkflowTool = {
             };
         }
         catch (error) {
-            logger.error("Error removing workflow", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error removing workflow", error);
             const errorMessage = error instanceof Error ? error.message : String(error);
             // Handle permission errors
             if (errorMessage.includes('network admin') || errorMessage.includes('PermissionDenied') || errorMessage.includes('permission')) {
@@ -780,7 +856,8 @@ exports.updateWorkflowFieldTool = {
             };
         }
         catch (error) {
-            logger.error("Error updating workflow field", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error updating workflow field", error);
             const errorMessage = error instanceof Error ? error.message : String(error);
             // Handle permission errors
             if (errorMessage.includes('network admin') || errorMessage.includes('PermissionDenied') || errorMessage.includes('permission')) {
@@ -968,7 +1045,8 @@ functionVariables: {
             };
         }
         catch (error) {
-            logger.error("Error testing function field", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error testing function field", error);
             const errorMessage = error instanceof Error
                 ? error.message
                 : JSON.stringify(error, null, 2);
@@ -1084,7 +1162,8 @@ exports.listWorkflowsMinimalTool = {
             };
         }
         catch (error) {
-            logger.error("Error listing workflows (minimal)", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error listing workflows (minimal)", error);
             return {
                 content: [{
                         type: "text",
@@ -1180,7 +1259,8 @@ exports.countActivitiesTool = {
             };
         }
         catch (error) {
-            logger.error("Error counting activities", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error counting activities", error);
             return {
                 content: [{
                         type: "text",
@@ -1230,6 +1310,37 @@ const updateWorkflowPhaseSchema = zod_1.z.object({
     }).passthrough().describe("Phase properties to update"),
     workspaceId: zod_1.z.string().optional().describe("Optional workspace ID - defaults to current workspace"),
 });
+// ============================================================================
+// CORE INIT TOOL - Workspace overview for terminal Claude Code sessions
+// ============================================================================
+const coreInitDescription = `Load workspace context - call once at session start to get workflows, phases, teams with IDs`;
+exports.coreInitTool = {
+    name: 'core_init',
+    group: tool_registry_1.ToolGroup.READ,
+    description: coreInitDescription,
+    schema: zod_1.z.object({}),
+    async execute(_args, context) {
+        try {
+            const overview = (0, workspace_overview_1.generateWorkspaceOverview)(context.init);
+            return {
+                content: [{
+                        type: "text",
+                        text: `Use this workspace context for all subsequent tool calls. Do NOT call core_init again this session.\n\n${overview}`,
+                    }],
+            };
+        }
+        catch (error) {
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Failed to generate workspace overview", error);
+            return {
+                content: [{
+                        type: "text",
+                        text: `❌ Failed to load workspace context: ${error instanceof Error ? error.message : String(error)}`,
+                    }],
+            };
+        }
+    },
+};
 exports.updateWorkflowPhaseTool = {
     name: 'update_workflow_phase',
     group: tool_registry_1.ToolGroup.PLAYGROUND,
@@ -1301,7 +1412,8 @@ exports.updateWorkflowPhaseTool = {
             };
         }
         catch (error) {
-            logger.error("Error updating workflow phase", error);
+            if (!request_logger_1.RequestLogger.getCurrent())
+                logger.error("Error updating workflow phase", error);
             const errorMessage = error instanceof Error ? error.message : String(error);
             // Handle permission errors
             if (errorMessage.includes('network admin') || errorMessage.includes('PermissionDenied') || errorMessage.includes('permission')) {

package/dist/mcp/utils/hailer-api-client.js

@@ -6,6 +6,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.HailerApiClient = void 0;
 const logger_1 = require("../../lib/logger");
+const request_logger_1 = require("../../lib/request-logger");
 const api_errors_1 = require("./api-errors");
 const file_upload_1 = require("./file-upload");
 class HailerApiClient {
@@ -34,12 +35,14 @@ class HailerApiClient {
             return result;
         }
         catch (error) {
-            // Log with detailed context for debugging
-            this.logger.error('Socket API error', error, {
-                method,
-                args: JSON.stringify(args || []),
-                apiBaseUrl: this.clients.socket.host
-            });
+            // Log with detailed context for debugging (skip if inside a request cycle)
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                this.logger.error('Socket API error', error, {
+                    method,
+                    args: JSON.stringify(args || []),
+                    apiBaseUrl: this.clients.socket.host
+                });
+            }
             throw error;
         }
     }
@@ -105,10 +108,12 @@ class HailerApiClient {
             return result;
         }
         catch (error) {
-            this.logger.error('createActivities failed', error, {
-                workflowId,
-                activitiesCount: activities.length
-            });
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                this.logger.error('createActivities failed', error, {
+                    workflowId,
+                    activitiesCount: activities.length
+                });
+            }
             throw error;
         }
     }
@@ -188,11 +193,13 @@ class HailerApiClient {
                 }
             }
             catch (error) {
-                this.logger.error(`Error fetching batch ${i + 1}`, {
-                    error: error,
-                    currentMessageId,
-                    batchNumber: i + 1
-                });
+                if (!request_logger_1.RequestLogger.getCurrent()) {
+                    this.logger.error(`Error fetching batch ${i + 1}`, {
+                        error: error,
+                        currentMessageId,
+                        batchNumber: i + 1
+                    });
+                }
                 break;
             }
         }
@@ -351,7 +358,7 @@ class HailerApiClient {
         if (lastname) {
             await this.setUserInfo('lastname', lastname);
         }
-        this.logger.info('User display name updated', { fullName, firstname, lastname });
+        this.logger.debug('User display name updated', { fullName, firstname, lastname });
     }
     /**
      * Find user by name - searches init data for firstname/lastname match
@@ -497,7 +504,9 @@ class HailerApiClient {
             };
         }
         catch (error) {
-            this.logger.error('File download failed', error, { fileId });
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                this.logger.error('File download failed', error, { fileId });
+            }
             throw error;
         }
     }

package/dist/mcp/webhook-handler.d.ts

@@ -1,7 +1,8 @@
 /**
- * Webhook Handler Utilities
+ * Webhook Handler for Bot Config Updates
  *
- * Provides webhook token generation and verification for secure endpoints.
+ * Receives activity updates from Hailer workflow webhooks and updates
+ * local .bot-config/{workspaceId}.json files.
  */
 /**
  * Generate HMAC-SHA256 signature for webhook payload
@@ -20,7 +21,67 @@ export declare function verifyWebhookSignature(payload: string, signature: strin
  */
 export declare function getWebhookToken(): string | null;
 /**
- * Get the full webhook path with token
+ * Get the full webhook path with token (no prefix for security through obscurity)
  */
 export declare function getWebhookPath(): string | null;
+interface WebhookField {
+    id: string;
+    type: string;
+    value: any;
+    key: string;
+}
+interface WebhookPayload {
+    _id: string;
+    name: string;
+    fields: WebhookField[];
+    currentPhase: string;
+    process: string;
+    cid: string;
+    uid: string;
+    created: number;
+    updated: number;
+}
+interface BotEntry {
+    activityId: string;
+    userId: string | null;
+    email: string;
+    password: string;
+    botType: string;
+    enabled: boolean;
+    displayName?: string;
+}
+interface WorkspaceConfig {
+    workspaceId: string;
+    workspaceName: string;
+    orchestrator?: {
+        activityId: string;
+        userId: string;
+        email: string;
+        password: string;
+        displayName?: string;
+    };
+    specialists: BotEntry[];
+    lastSynced: string;
+}
+type BotUpdateCallback = (workspaceId: string, bot: BotEntry, action: 'add' | 'update' | 'remove') => void;
+export declare function onBotUpdate(callback: BotUpdateCallback): void;
+/**
+ * Process webhook payload and update workspace config
+ */
+export declare function handleBotConfigWebhook(payload: WebhookPayload): {
+    success: boolean;
+    action: string;
+    workspaceId: string;
+    botType: string | null;
+    error?: string;
+};
+/**
+ * Get workspace config (for debugging/status)
+ */
+export declare function getWorkspaceConfig(workspaceId: string): WorkspaceConfig | null;
+/**
+ * List all workspace configs
+ */
+export declare function listWorkspaceConfigs(): WorkspaceConfig[];
+export {};
 //# sourceMappingURL=webhook-handler.d.ts.map