clementine-agent 1.0.29 → 1.0.30

package/dist/channels/slack.js

@@ -205,32 +205,42 @@ export async function startSlack(gateway, dispatcher, slackBotManager) {
      * Returns true on success.
      *
      * Session key formats:
-     *   slack:user:{userId}                          → DM to user
+     *   slack:team:{teamId}:user:{userId}            → DM to user (workspace-namespaced, current format)
+     *   slack:team:{teamId}:dm:{userId}              → DM to user (workspace-namespaced)
+     *   slack:user:{userId}                          → DM to user (legacy, pre-namespacing)
+     *   slack:dm:{userId}                            → DM to user (legacy)
      *   slack:channel:{channelId}:{userId}           → post in channel
      *   slack:channel:{channelId}:{slug}:{userId}    → post in channel (agent-scoped chat)
-     *   slack:dm:{userId}                            → DM to user
      *   slack:agent:{slug}:{userId}                  → DM to user (agent-scoped)
      */
     async function trySlackSessionRouting(sessionKey, text) {
         const parts = sessionKey.split(':');
         if (parts[0] !== 'slack' || parts.length < 3)
             return false;
-        const kind = parts[1];
+        // Strip the `team:{teamId}:` workspace prefix if present so downstream
+        // routing logic stays format-agnostic. The current bolt app is connected
+        // to a single workspace, so we use the existing client regardless of which
+        // teamId the session names.
+        let effectiveParts = parts;
+        if (parts[1] === 'team' && parts.length >= 4) {
+            effectiveParts = ['slack', ...parts.slice(3)];
+        }
+        const kind = effectiveParts[1];
         try {
-            if ((kind === 'user' || kind === 'dm') && parts[2]) {
-                const dm = await app.client.conversations.open({ users: parts[2] });
+            if ((kind === 'user' || kind === 'dm') && effectiveParts[2]) {
+                const dm = await app.client.conversations.open({ users: effectiveParts[2] });
                 const channelId = dm.channel?.id;
                 if (!channelId)
                     return false;
                 await sendChunkedSlack(app.client, channelId, mdToSlack(text));
                 return true;
             }
-            if (kind === 'channel' && parts[2]) {
-                await sendChunkedSlack(app.client, parts[2], mdToSlack(text));
+            if (kind === 'channel' && effectiveParts[2]) {
+                await sendChunkedSlack(app.client, effectiveParts[2], mdToSlack(text));
                 return true;
             }
-            if (kind === 'agent' && parts[3]) {
-                const dm = await app.client.conversations.open({ users: parts[3] });
+            if (kind === 'agent' && effectiveParts[3]) {
+                const dm = await app.client.conversations.open({ users: effectiveParts[3] });
                 const channelId = dm.channel?.id;
                 if (!channelId)
                     return false;

package/dist/gateway/cron-scheduler.js

@@ -450,9 +450,16 @@ export class CronScheduler {
         this.watchAgentsDir();
         this.watchWorkflowDir();
         this.watchTriggers();
+        // Deep-mode jobs are owned by the router (_deliverDeepResult). The
+        // cron-scheduler callbacks below only dispatch for cron-originated runs;
+        // phase updates for deep-mode runs get routed back to the originating
+        // session instead of fanning out to every registered channel.
+        const isDeepMode = (jobName) => jobName.startsWith('deep-');
         // Wire up push notifications for unleashed task completions
         this.gateway.setUnleashedCompleteCallback((jobName, result) => {
             this.completedJobs.set(jobName, Date.now());
+            if (isDeepMode(jobName))
+                return; // router handles delivery via _deliverDeepResult
             if (result && result !== '__NOTHING__') {
                 const slug = jobName.includes(':') ? jobName.split(':')[0] : undefined;
                 // Strip system metadata for clean conversational delivery
@@ -473,7 +480,15 @@ export class CronScheduler {
             const cleanOutput = output
                 .replace(/^STATUS SUMMARY:?\s*/im, '')
                 .slice(0, 500);
-            this.dispatcher.send(`Still working on it — ${cleanOutput}`, { agentSlug: slug }).catch(err => logger.debug({ err }, 'Failed to send phase progress notification'));
+            // For deep-mode runs, target the originating session so the progress
+            // update lands in the same Discord DM / Slack thread / dashboard window.
+            const deepSessionKey = isDeepMode(jobName) ? this.gateway.findDeepTaskSessionKey(jobName) : null;
+            const ctx = {};
+            if (slug)
+                ctx.agentSlug = slug;
+            if (deepSessionKey)
+                ctx.sessionKey = deepSessionKey;
+            this.dispatcher.send(`Still working on it — ${cleanOutput}`, ctx).catch(err => logger.debug({ err }, 'Failed to send phase progress notification'));
         });
         // Wire up real-time progress summaries (throttled to max 1 per 5 minutes)
         const lastProgressSent = new Map();
@@ -484,7 +499,13 @@ export class CronScheduler {
                 return; // throttle: 1 per 5 minutes
             lastProgressSent.set(jobName, now);
             const slug = jobName.includes(':') ? jobName.split(':')[0] : undefined;
-            this.dispatcher.send(summary.slice(0, 300), { agentSlug: slug }).catch(err => logger.debug({ err }, 'Failed to send phase progress summary'));
+            const deepSessionKey = isDeepMode(jobName) ? this.gateway.findDeepTaskSessionKey(jobName) : null;
+            const ctx = {};
+            if (slug)
+                ctx.agentSlug = slug;
+            if (deepSessionKey)
+                ctx.sessionKey = deepSessionKey;
+            this.dispatcher.send(summary.slice(0, 300), ctx).catch(err => logger.debug({ err }, 'Failed to send phase progress summary'));
         });
         logger.info(`Cron scheduler started with ${this.jobs.length} jobs`);
     }

package/dist/gateway/router.d.ts

@@ -75,6 +75,13 @@ export declare class Gateway {
     constructor(assistant: PersonalAssistant);
     /** Get or create a session state entry. */
     private getSession;
+    /**
+     * Reverse-lookup the session key that owns a given deep-mode jobName.
+     * Used by the cron-scheduler callbacks so phase-progress and completion
+     * messages can be routed back to the originating channel instead of
+     * fanning out to every registered sender.
+     */
+    findDeepTaskSessionKey(jobName: string): string | null;
     getAgentManager(): AgentManager;
     getTeamRouter(): TeamRouter;
     getTeamBus(): TeamBus;

package/dist/gateway/router.js

@@ -322,6 +322,19 @@ export class Gateway {
         }
         return s;
     }
+    /**
+     * Reverse-lookup the session key that owns a given deep-mode jobName.
+     * Used by the cron-scheduler callbacks so phase-progress and completion
+     * messages can be routed back to the originating channel instead of
+     * fanning out to every registered sender.
+     */
+    findDeepTaskSessionKey(jobName) {
+        for (const [key, sess] of this.sessions) {
+            if (sess.deepTask?.jobName === jobName)
+                return key;
+        }
+        return null;
+    }
     // ── Team system accessors ──────────────────────────────────────────
     getAgentManager() {
         if (!this._agentManager) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.0.29",
+  "version": "1.0.30",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",