@siftd/connect-agent 0.2.53 → 0.2.55

package/dist/agent.js

@@ -415,8 +415,15 @@ export async function runAgent(pollInterval = 2000) {
     });
     // Worker results - send to user when workers complete
     orchestrator.setWorkerResultCallback((workerId, result) => {
-        console.log(`[WORKER DONE] ${workerId}: ${result.slice(0, 100)}...`);
-        if (wsClient?.connected()) {
+        const trimmed = result.trim();
+        const lower = trimmed.toLowerCase();
+        const shouldSuppress = !trimmed ||
+            trimmed === '(No output)' ||
+            lower.startsWith('worker timed out') ||
+            lower.startsWith('worker error') ||
+            lower.includes('partial output: none');
+        console.log(`[WORKER DONE] ${workerId}: ${trimmed.slice(0, 100)}${shouldSuppress ? ' (suppressed)' : ''}`);
+        if (!shouldSuppress && wsClient?.connected()) {
             // Send as response with worker ID as message ID
             wsClient.sendResponse(workerId, `**Worker completed:**\n\n${result}`);
         }

package/dist/orchestrator.js

@@ -734,15 +734,18 @@ export class MasterOrchestrator {
     }
     hasCalendarMutation(message) {
         const lower = this.stripTodoSnapshot(message).toLowerCase();
-        const target = /(^|\s)\/cal\b|\/calendar\b|\bcalendar\b|\bcal\b/.test(lower);
-        const action = /\b(add|create|schedule|book|move|reschedule|update|change|cancel|delete|remove)\b/.test(lower);
+        const target = /(^|\s)\/cal\b|\/calendar\b|\bcalendar\b|\bcal\b|\bschedule\b|\bagenda\b/.test(lower);
+        const action = /\b(add|create|schedule|book|move|reschedule|update|change|cancel|delete|remove|set|put|place|remind)\b/.test(lower);
+        const dateCue = /\b(today|tomorrow|tonight|next|this\s+(week|month|year)|mon(day)?|tue(sday)?|wed(nesday)?|thu(rsday)?|fri(day)?|sat(urday)?|sun(day)?|\b\d{1,2}(:\d{2})?\s?(am|pm)\b|\b\d{1,2}\/\d{1,2}\b|\b\d{4}-\d{2}-\d{2}\b)\b/.test(lower);
+        const eventCue = /\b(meeting|call|appointment|event|reminder|session|class|lecture|review|deadline)\b/.test(lower);
+        const implicitTarget = eventCue && dateCue;
         const query = /\b(what|show|list|open|view|see)\b/.test(lower);
-        return target && action && !query;
+        return (target || implicitTarget) && action && !query;
     }
     hasFileMutation(message) {
         const lower = this.stripTodoSnapshot(message).toLowerCase();
         const target = /(^|\s)\/files\b|\bfiles?\b/.test(lower);
-        const action = /\b(create|make|write|save|generate|export|upload|attach|produce|edit|update|modify|delete|remove)\b/.test(lower);
+        const action = /\b(create|make|write|save|generate|export|upload|attach|produce|edit|update|modify|delete|remove|add|append|insert|include)\b/.test(lower);
         const query = /\b(what|show|list|open|view|see|browse|find)\b/.test(lower);
         return target && action && !query;
     }
@@ -1422,9 +1425,9 @@ ${hubContextStr}
         const toolsBase = this.getToolDefinitions();
         const last = messages[messages.length - 1];
         const lastContent = last && last.role === 'user' && typeof last.content === 'string' ? last.content : '';
-        const restrictWorkers = lastContent
-            ? (this.hasTodoMutation(lastContent) || this.hasCalendarMutation(lastContent))
-            : false;
+        const requiredTodo = lastContent ? this.hasTodoMutation(lastContent) : false;
+        const requiredCal = lastContent ? this.hasCalendarMutation(lastContent) : false;
+        const restrictWorkers = requiredTodo || requiredCal;
         const wantsTodoOrCal = restrictWorkers;
         const todoCalTools = new Set(['calendar_upsert_events', 'todo_upsert_items']);
         const blockedTools = new Set([
@@ -1444,6 +1447,7 @@ ${hubContextStr}
         const requestTimeoutMs = 60000;
         const forcedToolChoice = this.getToolChoice(currentMessages);
         let retriedForcedTool = false;
+        let retriedTodoCal = false;
         while (iterations < maxIterations) {
             iterations++;
             const toolChoice = forcedToolChoice ?? this.getToolChoice(currentMessages);
@@ -1492,13 +1496,44 @@ ${hubContextStr}
                     ];
                     continue;
                 }
+                if (wantsTodoOrCal && !retriedTodoCal && (requiredTodo || requiredCal)) {
+                    retriedTodoCal = true;
+                    const required = [
+                        requiredTodo ? 'todo_upsert_items' : null,
+                        requiredCal ? 'calendar_upsert_events' : null
+                    ].filter(Boolean).join(' and ');
+                    const followup = `You must call ${required} now. Do not spawn workers or call any other tools.`;
+                    currentMessages = [
+                        ...currentMessages,
+                        { role: 'assistant', content: response.content },
+                        { role: 'user', content: followup }
+                    ];
+                    continue;
+                }
                 return this.extractText(response.content);
             }
             const toolUseBlocks = response.content.filter((block) => block.type === 'tool_use');
             const toolNames = toolUseBlocks.map((block) => block.name);
             // Process tool calls
             const toolResults = await this.processToolCalls(response.content, sendMessage);
-            if (wantsTodoOrCal && (toolNames.includes('todo_upsert_items') || toolNames.includes('calendar_upsert_events'))) {
+            const missingTodo = requiredTodo && !toolNames.includes('todo_upsert_items');
+            const missingCal = requiredCal && !toolNames.includes('calendar_upsert_events');
+            if (wantsTodoOrCal && (missingTodo || missingCal) && !retriedTodoCal) {
+                retriedTodoCal = true;
+                const required = [
+                    missingTodo ? 'todo_upsert_items' : null,
+                    missingCal ? 'calendar_upsert_events' : null
+                ].filter(Boolean).join(' and ');
+                const followup = `You must call ${required} now. Do not repeat tools already called. Do not spawn workers.`;
+                currentMessages = [
+                    ...currentMessages,
+                    { role: 'assistant', content: response.content },
+                    { role: 'user', content: toolResults },
+                    { role: 'user', content: followup }
+                ];
+                continue;
+            }
+            if (wantsTodoOrCal && !missingTodo && !missingCal) {
                 const updates = [];
                 if (toolNames.includes('todo_upsert_items'))
                     updates.push('Updated /todo.');
@@ -2209,6 +2244,11 @@ Unlike lia_plan (internal only), this creates a VISIBLE todo list that appears i
                             try {
                                 const pathValue = String(input.path || '').trim();
                                 const content = String(input.content || '');
+                                const allowEmpty = /\b(empty|blank|placeholder)\b/i.test(this.lastUserMessage || '');
+                                if (!content.trim() && !allowEmpty) {
+                                    result = { success: false, output: '', error: 'Refusing to write empty file content.' };
+                                    break;
+                                }
                                 const targetPath = this.resolveFilesWritePath(pathValue);
                                 mkdirSync(dirname(targetPath), { recursive: true });
                                 writeFileSync(targetPath, content, 'utf8');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@siftd/connect-agent",
-  "version": "0.2.53",
+  "version": "0.2.55",
   "description": "Master orchestrator agent - control Claude Code remotely via web",
   "type": "module",
   "main": "dist/index.js",