@hailer/mcp 1.1.12 → 1.1.13

package/dist/mcp/tool-registry.d.ts

@@ -10,7 +10,6 @@
  * - Enables Claude's strict mode validation with actual workspace values
  */
 import { z } from 'zod';
-import { UserContext } from './UserContextCache';
 import { McpResponse } from './utils/types';
 /**
  * Tool groups for access control
@@ -27,6 +26,12 @@ export declare enum ToolGroup {
     NUCLEAR = "nuclear",
     BOT_INTERNAL = "bot_internal"
 }
+/**
+ * Context type for tool execution
+ * - 'hailer': Requires Hailer authentication via UserContextCache (default)
+ * - 'none': No authentication needed (e.g., Weaviate tools with their own auth)
+ */
+export type ToolContextType = 'hailer' | 'none';
 /**
  * Tool definition interface
  */
@@ -35,7 +40,11 @@ export interface Tool<TSchema extends z.ZodType = z.ZodType> {
     group: ToolGroup;
     description: string;
     schema: TSchema;
-    execute: (args: z.infer<TSchema>, context: UserContext) => Promise<McpResponse>;
+    /** What authentication context this tool needs. Defaults to 'hailer'. */
+    contextType?: ToolContextType;
+    /** Context is UserContext for Hailer tools, VipunenContext for contextType:'none' tools.
+     *  Type safety is enforced at the mcp-server dispatch boundary, not here. */
+    execute: (args: z.infer<TSchema>, context: any) => Promise<McpResponse>;
 }
 /**
  * Tool definition for MCP protocol (JSON-RPC)
@@ -56,10 +65,10 @@ export interface ToolDefinition {
  */
 export declare class ToolRegistry {
     private tools;
+    private _lazyGuard?;
     private enableNuclearTools;
     private workspaceSchemas;
     private workspaceSchemasLoaded;
-    private static readonly TOOL_TO_SCHEMA_MAP;
     constructor(options?: {
         enableNuclearTools?: boolean;
     });
@@ -80,6 +89,10 @@ export declare class ToolRegistry {
      * Add a tool to the registry
      */
     addTool(tool: Tool): void;
+    /**
+     * Get tool definitions filtered by contextType — single-pass, no secondary lookups
+     */
+    getToolDefinitionsByContextType(contextType: ToolContextType): ToolDefinition[];
     /**
      * Get tool definitions with optional filtering
      *
@@ -108,7 +121,7 @@ export declare class ToolRegistry {
     /**
      * Execute tool with validation
      */
-    executeTool(name: string, args: any, context: UserContext): Promise<any>;
+    executeTool(name: string, args: any, context: any): Promise<any>;
     /**
      * Pre-transform install_workflow args to fix common LLM mistakes BEFORE validation
      */

package/dist/mcp/tool-registry.js

@@ -78,15 +78,10 @@ var ToolGroup;
  */
 class ToolRegistry {
     tools = new Map();
+    _lazyGuard;
     enableNuclearTools;
     workspaceSchemas = new Map();
     workspaceSchemasLoaded = false;
-    // Map tool names to their workspace schema files
-    static TOOL_TO_SCHEMA_MAP = {
-        'create_activity': '_workspace.schema.json', // Generic workspace schema
-        'update_activity': '_workspace.schema.json',
-        // Workflow-specific tools can be added dynamically
-    };
     constructor(options = {}) {
         this.enableNuclearTools = options.enableNuclearTools ?? false;
         logger.debug('ToolRegistry instance created', {
@@ -163,6 +158,18 @@ class ToolRegistry {
         this.tools.set(tool.name, tool);
         logger.debug('Tool registered', { name: tool.name, group: tool.group });
     }
+    /**
+     * Get tool definitions filtered by contextType — single-pass, no secondary lookups
+     */
+    getToolDefinitionsByContextType(contextType) {
+        return Array.from(this.tools.values())
+            .filter(t => (t.contextType ?? 'hailer') === contextType)
+            .map(tool => ({
+            name: tool.name,
+            description: tool.description,
+            inputSchema: this.convertZodSchemaToJsonSchema(tool.schema)
+        }));
+    }
     /**
      * Get tool definitions with optional filtering
      *
@@ -276,7 +283,9 @@ class ToolRegistry {
         }
         logger.debug('Executing tool', {
             name,
-            apiKey: context.apiKey.substring(0, 8) + '...'
+            auth: context?.apiKey
+                ? context.apiKey.substring(0, 8) + '...'
+                : `vipunen:${context?.group ?? 'unknown'}`
         });
         // Pre-transform for specific tools (fix common LLM mistakes before validation)
         let transformedArgs = args;
@@ -579,6 +588,27 @@ Then retry \`${name}\` with the correct format from the skill examples.` : `💡
                     property.type = "string";
                 }
                 break;
+            case "ZodLazy":
+                // Unwrap lazy schema (used for recursive types like WhereFilter)
+                // Guard against infinite recursion from self-referencing schemas
+                if (!this._lazyGuard)
+                    this._lazyGuard = new Set();
+                if (this._lazyGuard.has(unwrappedType)) {
+                    property.type = "object";
+                    break;
+                }
+                this._lazyGuard.add(unwrappedType);
+                if (unwrappedType._def?.getter) {
+                    const resolved = unwrappedType._def.getter();
+                    const lazyResult = this.convertZodTypeToJsonSchema(resolved);
+                    this._lazyGuard.delete(unwrappedType);
+                    if (property.description)
+                        lazyResult.description = property.description;
+                    return lazyResult;
+                }
+                this._lazyGuard.delete(unwrappedType);
+                property.type = "object";
+                break;
             default:
                 property.type = "string";
         }

package/dist/mcp/tools/activity.js

@@ -305,6 +305,46 @@ function formatFilteredActivityListResponse(activityData, workflow, init, args)
         ],
     };
 }
+/**
+ * Auto-fix date fields: convert ISO date strings (e.g. "2026-03-11") to Unix timestamps (ms).
+ * LLMs often pass date strings instead of timestamps — this catches and converts them.
+ */
+function autoFixDateFields(fields, context) {
+    if (!context.workspaceCache)
+        return;
+    for (const [fieldId, fieldValue] of Object.entries(fields)) {
+        // Only fix string values that look like ISO dates
+        if (typeof fieldValue !== 'string' || !/^\d{4}-\d{2}-\d{2}/.test(fieldValue))
+            continue;
+        // Check if this field is a date type
+        let fieldType;
+        for (const workflow of context.workspaceCache.rawInit.processes) {
+            const workflowFields = workflow.fields;
+            if (!workflowFields)
+                continue;
+            let field;
+            if (Array.isArray(workflowFields)) {
+                field = workflowFields.find((f) => f._id === fieldId || f.id === fieldId);
+            }
+            else if (typeof workflowFields === 'object') {
+                field = workflowFields[fieldId];
+            }
+            if (field) {
+                fieldType = field.type;
+                break;
+            }
+        }
+        if (fieldType === 'date') {
+            const parsed = new Date(fieldValue).getTime();
+            if (!isNaN(parsed)) {
+                logger.warn("Auto-fixing date field - converting ISO string to timestamp", {
+                    fieldId, original: fieldValue, converted: parsed,
+                });
+                fields[fieldId] = parsed;
+            }
+        }
+    }
+}
 /** Build activity update object */
 function buildActivityUpdate(args, context) {
     let parsedFields = args.fields;
@@ -345,8 +385,10 @@ function buildActivityUpdate(args, context) {
             }
         }
     }
-    // Field-type aware validation: only validate user fields as users
+    // Field-type aware validation and auto-fix
     if (parsedFields && typeof parsedFields === "object" && context.workspaceCache) {
+        // Auto-fix date strings → timestamps
+        autoFixDateFields(parsedFields, context);
         const invalidUsers = [];
         for (const [fieldId, fieldValue] of Object.entries(parsedFields)) {
             // Only check string values that look like IDs
@@ -417,10 +459,24 @@ function formatUpdateActivityResponse(args, result) {
         ],
     };
 }
+const listActivitiesDescription = `List activities (items/records/tasks) from a workflow phase.
+
+**Prerequisites:** You need workflowId, phaseId, and field IDs. Get them via:
+1. \`list_workflows_minimal\` → workflowId
+2. \`list_workflow_phases\` → phaseId
+3. \`get_workflow_schema\` → field IDs for the \`fields\` parameter
+
+**Required params:** workflowId, phaseId, fields[] (array of field IDs to return)
+
+**Example flow:** User says "show my tasks" →
+1. list_workflows_minimal → find "Tasks" workflow
+2. list_workflow_phases → find "Open" phase
+3. get_workflow_schema → get field IDs
+4. list_activities with those IDs`;
 exports.listActivitiesTool = {
     name: 'list_activities',
     group: tool_registry_1.ToolGroup.READ,
-    description: `List activities from workflow phase`,
+    description: listActivitiesDescription,
     schema: zod_1.z.object({
         workflowId: zod_1.z.string().describe("Workflow ID to list activities from"),
         phaseId: zod_1.z.string().describe("Phase ID to filter activities (required - use list_workflow_phases to get available phases)"),
@@ -560,10 +616,15 @@ exports.listActivitiesTool = {
         }
     },
 };
+const showActivityByIdDescription = `Get full details of a single activity by its ID.
+
+**When to use:** When you have an activity ID and need all its field values, or to verify an update worked.
+
+**Returns:** Complete activity with all fields, current phase, timestamps, linked items.`;
 exports.showActivityByIdTool = {
     name: 'show_activity_by_id',
     group: tool_registry_1.ToolGroup.READ,
-    description: `Get activity by ID`,
+    description: showActivityByIdDescription,
     schema: zod_1.z.object({
         activityId: zod_1.z.string().describe("Activity ID to load"),
     }),
@@ -599,11 +660,24 @@ exports.showActivityByIdTool = {
 // ============================================================================
 // TOOL 3: CREATE ACTIVITY
 // ============================================================================
-const createActivityDescription = `Create activity (single or bulk via activities array)
+const createActivityDescription = `Create a new activity (item/record/task) in a workflow.
 
-**REQUIRED:** You MUST provide either 'name' (single) or 'activities' array (bulk). Calls with only workflowId will fail.
+**Prerequisites:** Get IDs first via:
+1. \`list_workflows_minimal\` → workflowId
+2. \`list_workflow_phases\` → phaseId (use isInitial=true phase)
+3. \`get_workflow_schema\` → field IDs for the fields parameter
 
-**Date fields:** Use Unix timestamp (ms), NOT strings. Example: 1737158400000, not "2025-01-18"`;
+**Required:** workflowId + (name OR activities[])
+**Recommended:** phaseId (defaults to initial phase if omitted)
+
+**Field value formats:**
+- Text: \`"value"\`
+- Number: \`42\`
+- Date: Unix ms \`1730937600000\` (NOT "2025-01-18")
+- Dropdown: exact option \`"High"\`
+- User/Link: ID string \`"5f8a..."\`
+
+**Bulk creation:** Use activities[] array instead of name/fields for multiple items.`;
 exports.createActivityTool = {
     name: 'create_activity',
     group: tool_registry_1.ToolGroup.WRITE,
@@ -691,6 +765,8 @@ exports.createActivityTool = {
                     if (parsedFields &&
                         typeof parsedFields === "object" &&
                         Object.keys(parsedFields).length > 0) {
+                        // Auto-fix date strings → timestamps
+                        autoFixDateFields(parsedFields, context);
                         activity.fields = parsedFields;
                     }
                 }
@@ -932,7 +1008,21 @@ exports.createActivityTool = {
 // ============================================================================
 // TOOL 4: UPDATE ACTIVITY
 // ============================================================================
-const updateActivityDescription = `Update activity (single or bulk via activities array)`;
+const updateActivityDescription = `Update an existing activity - change fields or move to different phase.
+
+**To update fields:** Get field IDs from \`get_workflow_schema\` first, then pass them in the fields object.
+**To change phase:** Get phase IDs from \`list_workflow_phases\`, pass target phaseId.
+
+**Required:** activityId (for single) OR activities[] array (for bulk)
+
+**Field value formats:**
+- Text: \`"value"\`
+- Number: \`42\`
+- Date: Unix ms \`1730937600000\` (NOT strings)
+- ActivityLink: just the ID string \`"abc123..."\` (NOT an object)
+
+**Example - move to Done:** \`update_activity({ activityId: "...", phaseId: "done-phase-id" })\`
+**Example - update field:** \`update_activity({ activityId: "...", fields: { "fieldId": "new value" } })\``;
 exports.updateActivityTool = {
     name: 'update_activity',
     group: tool_registry_1.ToolGroup.WRITE,
@@ -1026,6 +1116,8 @@ exports.updateActivityTool = {
                                 }
                             }
                         }
+                        // Auto-fix date strings → timestamps
+                        autoFixDateFields(parsedFields, context);
                         activityUpdate.fields = parsedFields;
                     }
                     return activityUpdate;