@hailer/mcp 1.2.1 → 2.0.0-beta.3

package/dist/mcp/tools/discussion.js

@@ -4,12 +4,17 @@
  *
  * Tools for managing Hailer discussions/chats:
  * - List user's discussions (READ)
- * - Fetch discussion messages with pagination (WRITE)
+ * - Fetch discussion messages, latest or older via beforeMessageId cursor (READ)
  * - Join/leave discussions (WRITE)
  * - Post messages to discussions (WRITE)
+ *
+ * get_activity_from_discussion is retired as a listed tool; its lookup logic
+ * lives on as the exported lookupActivityFromDiscussion helper (used by the
+ * central alias layer and by fetch_discussion_messages enrichment).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.discussionTools = exports.getActivityFromDiscussionTool = exports.inviteDiscussionMembersTool = exports.addDiscussionMessageTool = exports.leaveDiscussionTool = exports.joinDiscussionTool = exports.fetchPreviousDiscussionMessagesTool = exports.fetchDiscussionMessagesTool = exports.listMyDiscussionsTool = void 0;
+exports.discussionTools = exports.inviteDiscussionMembersTool = exports.addDiscussionMessageTool = exports.leaveDiscussionTool = exports.joinDiscussionTool = exports.fetchDiscussionMessagesTool = exports.listMyDiscussionsTool = void 0;
+exports.lookupActivityFromDiscussion = lookupActivityFromDiscussion;
 const zod_1 = require("zod");
 const tool_registry_1 = require("../tool-registry");
 const index_1 = require("../utils/index");
@@ -24,8 +29,9 @@ const logger = (0, index_1.createLogger)({ component: 'discussion-tools' });
  * Sanitize string to remove invalid Unicode characters
  */
 function sanitizeString(str) {
-    if (!str)
+    if (!str) {
         return '';
+    }
     return str.replace(/[\ud800-\udfff]/g, '\ufffd');
 }
 /**
@@ -95,7 +101,7 @@ exports.listMyDiscussionsTool = {
                 responseText += `❌ You are not currently participating in any discussions.\n\n`;
                 responseText += `💡 **TIP**: Join activity discussions using the \`join_discussion\` tool.`;
                 return {
-                    content: [{ type: "text", text: responseText }],
+                    content: [{ type: 'text', text: responseText }],
                 };
             }
             // Collect all linked activity IDs to fetch their names
@@ -160,14 +166,16 @@ exports.listMyDiscussionsTool = {
                 other: '❓ Other Discussions'
             };
             for (const [type, typeDiscussions] of Object.entries(discussionsByType)) {
-                if (typeDiscussions.length === 0)
+                if (typeDiscussions.length === 0) {
                     continue;
+                }
                 responseText += `${typeLabels[type]} (${typeDiscussions.length}):\n\n`;
                 typeDiscussions.forEach((discussion, index) => {
                     const unreadCount = unreadCounts[discussion._id] || 0;
                     responseText += `${index + 1}. `;
-                    if (unreadCount > 0)
+                    if (unreadCount > 0) {
                         responseText += `🔴 `;
+                    }
                     // For activity discussions, prefer the activity name over discussion.name
                     let displayName = sanitizeString(discussion.name) || 'Untitled';
                     if (discussion.linked_activity && activityNames[discussion.linked_activity]) {
@@ -216,19 +224,20 @@ exports.listMyDiscussionsTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: responseText,
                     },
                 ],
             };
         }
         catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Failed to list discussions", error);
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                logger.error('Failed to list discussions', error);
+            }
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Failed to list discussions: ${error instanceof Error ? error.message : String(error)}`,
                     },
                 ],
@@ -236,10 +245,335 @@ exports.listMyDiscussionsTool = {
         }
     },
 };
+/**
+ * Reverse lookup: find the activity linked to a discussion.
+ *
+ * Replaces the retired get_activity_from_discussion tool - the central alias
+ * layer keeps the old tool name callable through this helper, and
+ * fetch_discussion_messages uses it for includeLinkedActivity enrichment.
+ *
+ * Returns null when the discussion is not found, has no linked activity, or
+ * the linked activity cannot be loaded. API errors propagate to the caller.
+ */
+async function lookupActivityFromDiscussion(discussionId, context) {
+    const syncResponse = await context.hailer.request('v2.discussion.sync', [{ timestamp: 0 }]);
+    const discussions = syncResponse.discussions || [];
+    const targetDiscussion = discussions.find((disc) => disc._id === discussionId);
+    if (!targetDiscussion?.linked_activity) {
+        logger.debug('No linked activity for discussion', {
+            discussionId,
+            discussionFound: !!targetDiscussion
+        });
+        return null;
+    }
+    const activityId = targetDiscussion.linked_activity;
+    const activity = await context.hailer.fetchActivityById(activityId);
+    if (!activity) {
+        logger.debug('Linked activity could not be loaded', { discussionId, activityId });
+        return null;
+    }
+    return {
+        activityId: activity._id,
+        name: activity.name || 'Untitled',
+        workflowId: activity.workflowId || null
+    };
+}
+/**
+ * Best-effort linked-activity lookup for response enrichment - never throws
+ */
+async function safeLinkedActivityLookup(discussionId, context) {
+    try {
+        return await lookupActivityFromDiscussion(discussionId, context);
+    }
+    catch (error) {
+        logger.debug('Linked activity enrichment failed', {
+            discussionId,
+            error: error instanceof Error ? error.message : String(error)
+        });
+        return null;
+    }
+}
 // ============================================================================
 // TOOL 2: FETCH DISCUSSION MESSAGES
 // ============================================================================
-const fetchDiscussionMessagesDescription = `Read latest messages from any Hailer discussion/chat (up to 50 messages - API limit)`;
+/**
+ * Extract a forwarded message (and any nested forwards, max depth 10) into optimized form
+ */
+function extractForwardedMessage(context, fwd, depth = 0) {
+    if (depth > 10) {
+        return null;
+    }
+    const fwdUserInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, fwd.uid);
+    let fwdUsername = `User ${fwd.uid}`;
+    if (fwdUserInfo) {
+        const fullName = `${fwdUserInfo.firstname || ''} ${fwdUserInfo.lastname || ''}`.trim();
+        fwdUsername = fullName || fwdUserInfo.fullName || `User ${fwd.uid}`;
+    }
+    const extracted = {
+        _id: fwd._id,
+        uid: fwd.uid,
+        username: fwdUsername,
+        created: new Date(fwd.created).toLocaleString(),
+        msg: fwd.msg || '',
+        type: fwd.type
+    };
+    if (fwd.forwardMessageId) {
+        extracted.forwardMessageId = fwd.forwardMessageId;
+    }
+    const nested = fwd.forwardMessage ? extractForwardedMessage(context, fwd.forwardMessage, depth + 1) : null;
+    if (nested) {
+        extracted.forwardMessage = nested;
+    }
+    return extracted;
+}
+/**
+ * Convert a raw discussion message into optimized form - strips base64 payloads, resolves usernames
+ */
+function optimizeMessage(context, msg) {
+    let cleanMsg = msg.msg || '';
+    if (cleanMsg) {
+        cleanMsg = cleanMsg.replace(/data:[^;]+;base64,[A-Za-z0-9+/=]+/g, '[FILE_REMOVED]');
+        cleanMsg = cleanMsg.replace(/!\[.*?\]\(data:[^)]+\)/g, '[IMAGE_REMOVED]');
+    }
+    const userInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, msg.uid);
+    let username = `User ${msg.uid}`;
+    if (userInfo) {
+        const fullName = `${userInfo.firstname || ''} ${userInfo.lastname || ''}`.trim();
+        username = fullName || userInfo.fullName || `User ${msg.uid}`;
+    }
+    const optimizedMsg = {
+        _id: msg._id,
+        uid: msg.uid,
+        username: username,
+        created: new Date(msg.created).toLocaleString(),
+        type: msg.type,
+        msg: cleanMsg
+    };
+    const isSystemWithoutText = msg.type !== 'user' && !msg.msg;
+    if (isSystemWithoutText) {
+        optimizedMsg.systemDescription = formatSystemMessage(msg);
+    }
+    if (isSystemWithoutText && msg.meta && Object.keys(msg.meta).length > 0) {
+        optimizedMsg.meta = msg.meta;
+    }
+    if (msg.replyTo) {
+        optimizedMsg.replyTo = msg.replyTo;
+    }
+    if (msg.forwardMessageId) {
+        optimizedMsg.forwardMessageId = msg.forwardMessageId;
+    }
+    const forwarded = msg.forwardMessage ? extractForwardedMessage(context, msg.forwardMessage, 0) : null;
+    if (forwarded) {
+        optimizedMsg.forwardMessage = forwarded;
+    }
+    return optimizedMsg;
+}
+/**
+ * Latest-messages branch of fetch_discussion_messages (no beforeMessageId)
+ */
+async function fetchLatestMessages(args, context) {
+    const limit = args.limit || 50;
+    let discussionId = args.discussionId;
+    logger.debug('Fetching discussion messages', {
+        discussionId: discussionId,
+        limit: limit,
+        includeLinkedActivity: !!args.includeLinkedActivity,
+        apiKey: context.apiKey.substring(0, 8) + '...'
+    });
+    try {
+        // Auto-detect: if this is an activity ID, get its discussion ID
+        // Try fetching as discussion first, if it fails, check if it's an activity
+        let result;
+        try {
+            result = await context.hailer.fetchDiscussionMessages(discussionId, limit);
+        }
+        catch (firstError) {
+            // If first attempt fails, try loading as activity to get discussion ID
+            logger.debug('First fetch failed, checking if this is an activity ID', {
+                providedId: discussionId,
+                error: firstError?.msg || firstError?.message
+            });
+            const activity = await context.hailer.fetchActivityById(discussionId).catch(() => null);
+            if (!activity?.discussion) {
+                // Not an activity or no discussion linked, rethrow original error
+                throw firstError;
+            }
+            logger.debug('Auto-converted activity ID to discussion ID', {
+                activityId: discussionId,
+                discussionId: activity.discussion,
+                activityName: activity.name
+            });
+            discussionId = activity.discussion;
+            result = await context.hailer.fetchDiscussionMessages(discussionId, limit);
+        }
+        const messages = result.messages || [];
+        const messageCount = messages.length;
+        const userMessages = messages.filter((msg) => msg.type === 'user');
+        const recentMessage = messages[0];
+        const optimizedMessages = messages.map((msg) => optimizeMessage(context, msg));
+        const optimizedResult = {
+            oldestLoaded: result.oldestLoaded,
+            newestLoaded: result.newestLoaded,
+            messages: optimizedMessages
+        };
+        if (args.includeLinkedActivity) {
+            optimizedResult.linkedActivity = await safeLinkedActivityLookup(discussionId, context);
+        }
+        let responseText = `📬 Successfully fetched ${messageCount} messages from discussion ${args.discussionId}`;
+        if (result.truncated) {
+            responseText += ` (limited from ${result.originalCount} available messages)`;
+        }
+        const recentSummary = recentMessage
+            ? `"${recentMessage.msg || 'System event'}" at ${new Date(recentMessage.created).toLocaleString()}`
+            : 'No messages';
+        responseText += `\n\n📊 Summary:`;
+        responseText += `\n- ${userMessages.length} user messages`;
+        responseText += `\n- ${messages.length - userMessages.length} system messages (joins, activity creation, etc.)`;
+        responseText += `\n- Most recent: ${recentSummary}`;
+        responseText += `\n- Requested limit: ${limit} messages (API max: 50)`;
+        if (result.truncated) {
+            responseText += `\n- ⚠️  More messages available: ${result.originalCount - limit} additional messages not shown`;
+        }
+        if (optimizedMessages.length > 0) {
+            const oldestMessage = optimizedMessages[optimizedMessages.length - 1];
+            responseText += `\n\n🔄 **PAGINATION**: To get older messages, call fetch_discussion_messages again with:`;
+            responseText += `\n- discussionId: "${discussionId}"`;
+            responseText += `\n- beforeMessageId: "${oldestMessage._id}"`;
+            responseText += `\n- batches: 1-10 (each batch = ~50 messages)`;
+        }
+        responseText += `\n\n💡 Use this to understand conversation context, get project updates, or analyze team communications!`;
+        responseText += `\n\nOptimized conversation data (essential fields only):\n${JSON.stringify(optimizedResult, null, 2)}`;
+        return {
+            content: [
+                {
+                    type: 'text',
+                    text: responseText,
+                },
+            ],
+        };
+    }
+    catch (error) {
+        if (!request_logger_1.RequestLogger.getCurrent()) {
+            logger.error('Error fetching discussion messages', error);
+        }
+        const errorMessage = error instanceof Error
+            ? error.message
+            : (typeof error === 'object' && error !== null)
+                ? JSON.stringify(error, null, 2)
+                : String(error);
+        const isPermissionError = errorMessage.toLowerCase().includes('permission') ||
+            errorMessage.toLowerCase().includes('access') ||
+            errorMessage.toLowerCase().includes('forbidden') ||
+            errorMessage.toLowerCase().includes('unauthorized');
+        let helpText = `❌ Error fetching discussion messages: ${errorMessage}\n\n`;
+        if (isPermissionError) {
+            helpText += `🔒 **PERMISSION REQUIRED**: You need to join this discussion before you can read messages.\n\n` +
+                `💡 **Solution**: Use the join_discussion tool first:\n` +
+                `   join_discussion(discussionId: "${args.discussionId}")\n\n`;
+        }
+        helpText += `💡 Tips:\n` +
+            `- Make sure the discussion ID is correct (24-character string)\n` +
+            `- You must be a member/follower of the discussion to read messages\n` +
+            `- Some discussions may be private or require specific permissions\n\n` +
+            `Example discussion ID: '683ef53087e3d8329abaa3ad'`;
+        return {
+            content: [
+                {
+                    type: 'text',
+                    text: helpText,
+                },
+            ],
+        };
+    }
+}
+/**
+ * Older-messages branch of fetch_discussion_messages (beforeMessageId cursor)
+ */
+async function fetchPreviousMessages(args, context) {
+    const batches = args.batches || 1;
+    logger.debug('Fetching previous discussion messages', {
+        discussionId: args.discussionId,
+        beforeMessageId: args.beforeMessageId,
+        batches: batches,
+        includeLinkedActivity: !!args.includeLinkedActivity,
+        apiKey: context.apiKey.substring(0, 8) + '...'
+    });
+    try {
+        const result = await context.hailer.fetchPreviousDiscussionMessages(args.beforeMessageId, batches);
+        const messages = result.messages || [];
+        const messageCount = messages.length;
+        const userMessages = messages.filter((msg) => msg.type === 'user');
+        const oldestMessage = messages.length > 0 ? messages[messages.length - 1] : null;
+        const optimizedMessages = messages.map((msg) => optimizeMessage(context, msg));
+        const optimizedResult = {
+            messages: optimizedMessages,
+            totalFetched: result.totalFetched,
+            batchesCompleted: result.batchesCompleted,
+            oldestMessageId: result.oldestMessageId,
+            hasMore: result.hasMore
+        };
+        if (args.includeLinkedActivity) {
+            optimizedResult.linkedActivity = await safeLinkedActivityLookup(args.discussionId, context);
+        }
+        let responseText = `📬 Successfully fetched ${messageCount} previous messages`;
+        responseText += `\n\n📊 Summary:`;
+        responseText += `\n- ${userMessages.length} user messages`;
+        responseText += `\n- ${messages.length - userMessages.length} system messages`;
+        responseText += `\n- Batches completed: ${result.batchesCompleted}/${batches}`;
+        responseText += `\n- Total messages fetched: ${result.totalFetched}`;
+        if (oldestMessage) {
+            const oldestSummary = `"${oldestMessage.msg || 'System event'}" at ${new Date(oldestMessage.created).toLocaleString()}`;
+            responseText += `\n- Oldest message: ${oldestSummary}`;
+        }
+        if (result.hasMore && optimizedMessages.length > 0) {
+            const oldestOptimizedMessage = optimizedMessages[optimizedMessages.length - 1];
+            responseText += `\n\n🔄 **CONTINUE PAGINATION**: To get even older messages, call fetch_discussion_messages with:`;
+            responseText += `\n- discussionId: "${args.discussionId}"`;
+            responseText += `\n- beforeMessageId: "${oldestOptimizedMessage._id}"`;
+            responseText += `\n- batches: 1-10 (for more history)`;
+        }
+        if (!result.hasMore) {
+            responseText += `\n\n✅ **END OF HISTORY**: No more messages available in this discussion.`;
+        }
+        responseText += `\n\n💡 Use this to explore conversation history and understand context!`;
+        responseText += `\n\nOptimized conversation data (essential fields only):\n${JSON.stringify(optimizedResult, null, 2)}`;
+        return {
+            content: [
+                {
+                    type: 'text',
+                    text: responseText,
+                },
+            ],
+        };
+    }
+    catch (error) {
+        if (!request_logger_1.RequestLogger.getCurrent()) {
+            logger.error('Error fetching previous discussion messages', error);
+        }
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        const helpText = `❌ Error fetching previous discussion messages: ${errorMessage}\n\n` +
+            `💡 **IMPORTANT**: beforeMessageId must be a MESSAGE ID, not a DISCUSSION ID!\n\n` +
+            `**How to get the correct beforeMessageId:**\n` +
+            `1. First call fetch_discussion_messages with only your discussionId\n` +
+            `2. Look at the last message in the results\n` +
+            `3. Use that message's '_id' field as beforeMessageId\n\n` +
+            `**Example:**\n` +
+            `- ❌ Wrong: "682dcfa9d36a4907a88bb279" (this is a discussion ID)\n` +
+            `- ✅ Correct: "68e5c1903b104307efdebce9" (this is a message ID)\n\n` +
+            `Message IDs are usually different from discussion IDs!`;
+        return {
+            content: [
+                {
+                    type: 'text',
+                    text: helpText,
+                },
+            ],
+        };
+    }
+}
+const fetchDiscussionMessagesDescription = 'Read messages from any Hailer discussion/chat (up to 50 per batch - API limit). ' +
+    "Latest messages by default; pass beforeMessageId (a message '_id' from a previous result) to page back through older history. " +
+    'Optionally set includeLinkedActivity to also return the activity linked to the discussion';
 exports.fetchDiscussionMessagesTool = {
     name: 'fetch_discussion_messages',
     group: tool_registry_1.ToolGroup.READ,
@@ -247,337 +581,47 @@ exports.fetchDiscussionMessagesTool = {
     schema: zod_1.z.object({
         discussionId: zod_1.z
             .string()
-            .min(24, "Discussion ID must be at least 24 characters")
-            .describe("The discussion ID to read messages from. You can get this from activity results (look for 'discussion' field) or from URLs like https://app.hailer.com/#/discussions/DISCUSSION_ID"),
+            .min(24, 'Discussion ID must be at least 24 characters')
+            .describe("The discussion ID to read messages from. You can get this from activity results (look for 'discussion' field) " +
+            'or from URLs like https://app.hailer.com/#/discussions/DISCUSSION_ID'),
         limit: zod_1.z
             .number()
             .int()
-            .min(1, "Limit must be at least 1")
-            .max(50, "Limit cannot exceed 50 messages (API maximum)")
+            .min(1, 'Limit must be at least 1')
+            .max(50, 'Limit cannot exceed 50 messages (API maximum)')
             .default(50)
             .optional()
-            .describe("Number of messages to fetch (default: 50, max: 50 due to API limit). Use fetch_previous_discussion_messages for older messages")
-    }),
-    async execute(args, context) {
-        const limit = args.limit || 50;
-        let discussionId = args.discussionId;
-        logger.debug('Fetching discussion messages', {
-            discussionId: discussionId,
-            limit: limit,
-            apiKey: context.apiKey.substring(0, 8) + '...'
-        });
-        try {
-            // Auto-detect: if this is an activity ID, get its discussion ID
-            // Try fetching as discussion first, if it fails, check if it's an activity
-            let result;
-            try {
-                result = await context.hailer.fetchDiscussionMessages(discussionId, limit);
-            }
-            catch (firstError) {
-                // If first attempt fails, try loading as activity to get discussion ID
-                logger.debug('First fetch failed, checking if this is an activity ID', {
-                    providedId: discussionId,
-                    error: firstError?.msg || firstError?.message
-                });
-                try {
-                    const activity = await context.hailer.fetchActivityById(discussionId);
-                    if (activity?.discussion) {
-                        logger.debug('Auto-converted activity ID to discussion ID', {
-                            activityId: discussionId,
-                            discussionId: activity.discussion,
-                            activityName: activity.name
-                        });
-                        discussionId = activity.discussion;
-                        result = await context.hailer.fetchDiscussionMessages(discussionId, limit);
-                    }
-                    else {
-                        // Not an activity or no discussion linked, rethrow original error
-                        throw firstError;
-                    }
-                }
-                catch (activityError) {
-                    // Activity lookup failed too, rethrow original error
-                    throw firstError;
-                }
-            }
-            const messages = result.messages || [];
-            const messageCount = messages.length;
-            const userMessages = messages.filter((m) => m.type === "user");
-            const recentMessage = messages[0];
-            function extractForwardedMessage(fwd, depth = 0) {
-                if (depth > 10)
-                    return null;
-                const fwdUserInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, fwd.uid);
-                let fwdUsername = `User ${fwd.uid}`;
-                if (fwdUserInfo) {
-                    const fullName = `${fwdUserInfo.firstname || ''} ${fwdUserInfo.lastname || ''}`.trim();
-                    fwdUsername = fullName || fwdUserInfo.fullName || `User ${fwd.uid}`;
-                }
-                const extracted = {
-                    _id: fwd._id,
-                    uid: fwd.uid,
-                    username: fwdUsername,
-                    created: new Date(fwd.created).toLocaleString(),
-                    msg: fwd.msg || '',
-                    type: fwd.type
-                };
-                if (fwd.forwardMessageId)
-                    extracted.forwardMessageId = fwd.forwardMessageId;
-                if (fwd.forwardMessage) {
-                    const nested = extractForwardedMessage(fwd.forwardMessage, depth + 1);
-                    if (nested)
-                        extracted.forwardMessage = nested;
-                }
-                return extracted;
-            }
-            const optimizedMessages = messages.map((msg) => {
-                let cleanMsg = msg.msg || '';
-                if (cleanMsg) {
-                    cleanMsg = cleanMsg.replace(/data:[^;]+;base64,[A-Za-z0-9+/=]+/g, '[FILE_REMOVED]');
-                    cleanMsg = cleanMsg.replace(/!\[.*?\]\(data:[^)]+\)/g, '[IMAGE_REMOVED]');
-                }
-                const userInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, msg.uid);
-                let username = `User ${msg.uid}`;
-                if (userInfo) {
-                    const fullName = `${userInfo.firstname || ''} ${userInfo.lastname || ''}`.trim();
-                    username = fullName || userInfo.fullName || `User ${msg.uid}`;
-                }
-                const optimizedMsg = {
-                    _id: msg._id,
-                    uid: msg.uid,
-                    username: username,
-                    created: new Date(msg.created).toLocaleString(),
-                    type: msg.type,
-                    msg: cleanMsg
-                };
-                if (msg.type !== 'user' && !msg.msg) {
-                    optimizedMsg.systemDescription = formatSystemMessage(msg);
-                    if (msg.meta && Object.keys(msg.meta).length > 0) {
-                        optimizedMsg.meta = msg.meta;
-                    }
-                }
-                if (msg.replyTo) {
-                    optimizedMsg.replyTo = msg.replyTo;
-                }
-                if (msg.forwardMessageId) {
-                    optimizedMsg.forwardMessageId = msg.forwardMessageId;
-                }
-                if (msg.forwardMessage) {
-                    const forwarded = extractForwardedMessage(msg.forwardMessage, 0);
-                    if (forwarded)
-                        optimizedMsg.forwardMessage = forwarded;
-                }
-                return optimizedMsg;
-            });
-            const optimizedResult = {
-                oldestLoaded: result.oldestLoaded,
-                newestLoaded: result.newestLoaded,
-                messages: optimizedMessages
-            };
-            let responseText = `📬 Successfully fetched ${messageCount} messages from discussion ${args.discussionId}`;
-            if (result.truncated) {
-                responseText += ` (limited from ${result.originalCount} available messages)`;
-            }
-            responseText += `\n\n📊 Summary:\n- ${userMessages.length} user messages\n- ${messages.length - userMessages.length} system messages (joins, activity creation, etc.)\n- Most recent: ${recentMessage ? `"${recentMessage.msg || "System event"}" at ${new Date(recentMessage.created).toLocaleString()}` : "No messages"}`;
-            responseText += `\n- Requested limit: ${limit} messages (API max: 50)`;
-            if (result.truncated) {
-                responseText += `\n- ⚠️  More messages available: ${result.originalCount - limit} additional messages not shown`;
-            }
-            if (optimizedMessages.length > 0) {
-                const oldestMessage = optimizedMessages[optimizedMessages.length - 1];
-                responseText += `\n\n🔄 **PAGINATION**: To get older messages, use fetch_previous_discussion_messages with:`;
-                responseText += `\n- oldestMessageId: "${oldestMessage._id}"`;
-                responseText += `\n- batches: 1-10 (each batch = ~50 messages)`;
-            }
-            responseText += `\n\n💡 Use this to understand conversation context, get project updates, or analyze team communications!\n\nOptimized conversation data (essential fields only):\n${JSON.stringify(optimizedResult, null, 2)}`;
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: responseText,
-                    },
-                ],
-            };
-        }
-        catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error fetching discussion messages", error);
-            const errorMessage = error instanceof Error
-                ? error.message
-                : (typeof error === 'object' && error !== null)
-                    ? JSON.stringify(error, null, 2)
-                    : String(error);
-            const isPermissionError = errorMessage.toLowerCase().includes('permission') ||
-                errorMessage.toLowerCase().includes('access') ||
-                errorMessage.toLowerCase().includes('forbidden') ||
-                errorMessage.toLowerCase().includes('unauthorized');
-            let helpText = `❌ Error fetching discussion messages: ${errorMessage}\n\n`;
-            if (isPermissionError) {
-                helpText += `🔒 **PERMISSION REQUIRED**: You need to join this discussion before you can read messages.\n\n` +
-                    `💡 **Solution**: Use the join_discussion tool first:\n` +
-                    `   join_discussion(discussionId: "${args.discussionId}")\n\n`;
-            }
-            helpText += `💡 Tips:\n` +
-                `- Make sure the discussion ID is correct (24-character string)\n` +
-                `- You must be a member/follower of the discussion to read messages\n` +
-                `- Some discussions may be private or require specific permissions\n\n` +
-                `Example discussion ID: '683ef53087e3d8329abaa3ad'`;
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: helpText,
-                    },
-                ],
-            };
-        }
-    },
-};
-// ============================================================================
-// TOOL 3: FETCH PREVIOUS DISCUSSION MESSAGES
-// ============================================================================
-const fetchPreviousDiscussionMessagesDescription = `Fetch older messages from a discussion using pagination - requires a message ID from previous results to continue from`;
-exports.fetchPreviousDiscussionMessagesTool = {
-    name: 'fetch_previous_discussion_messages',
-    group: tool_registry_1.ToolGroup.READ,
-    description: fetchPreviousDiscussionMessagesDescription,
-    schema: zod_1.z.object({
-        oldestMessageId: zod_1.z
+            .describe('Number of messages to fetch (default: 50, max: 50 due to API limit). Use beforeMessageId for older messages'),
+        beforeMessageId: zod_1.z
             .string()
-            .min(24, "Message ID must be at least 24 characters")
-            .describe("The ID of the oldest message from your previous fetch - use this to continue pagination. Get this from the '_id' field of the last message in previous results"),
+            .min(24, 'Message ID must be at least 24 characters')
+            .optional()
+            .describe("Pagination cursor: the '_id' of the oldest message from a previous fetch (a MESSAGE ID, not a discussion ID). " +
+            'When set, fetches messages older than this message'),
         batches: zod_1.z
             .number()
             .int()
-            .min(1, "Batches must be at least 1")
-            .max(10, "Cannot fetch more than 10 batches at once")
+            .min(1, 'Batches must be at least 1')
+            .max(10, 'Cannot fetch more than 10 batches at once')
             .default(1)
             .optional()
-            .describe("Number of previous message batches to fetch (default: 1, max: 10). Each batch contains up to 50 messages. Use higher values to get more history in one call")
+            .describe('Number of older message batches to fetch (default: 1, max: 10). Each batch contains up to 50 messages. ' +
+            'Only used together with beforeMessageId'),
+        includeLinkedActivity: zod_1.z
+            .boolean()
+            .optional()
+            .describe('Set true to include the activity linked to this discussion ({ activityId, name, workflowId } or null). ' +
+            'Costs an extra API call (default: false)')
     }),
     async execute(args, context) {
-        const batches = args.batches || 1;
-        logger.debug('Fetching previous discussion messages', {
-            oldestMessageId: args.oldestMessageId,
-            batches: batches,
-            apiKey: context.apiKey.substring(0, 8) + '...'
-        });
-        try {
-            const result = await context.hailer.fetchPreviousDiscussionMessages(args.oldestMessageId, batches);
-            const messages = result.messages || [];
-            const messageCount = messages.length;
-            const userMessages = messages.filter((m) => m.type === "user");
-            const oldestMessage = messages.length > 0 ? messages[messages.length - 1] : null;
-            function extractForwardedMessage(fwd, depth = 0) {
-                if (depth > 10)
-                    return null;
-                const fwdUserInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, fwd.uid);
-                let fwdUsername = `User ${fwd.uid}`;
-                if (fwdUserInfo) {
-                    const fullName = `${fwdUserInfo.firstname || ''} ${fwdUserInfo.lastname || ''}`.trim();
-                    fwdUsername = fullName || fwdUserInfo.fullName || `User ${fwd.uid}`;
-                }
-                const extracted = {
-                    _id: fwd._id,
-                    uid: fwd.uid,
-                    username: fwdUsername,
-                    created: new Date(fwd.created).toLocaleString(),
-                    msg: fwd.msg || '',
-                    type: fwd.type
-                };
-                if (fwd.forwardMessageId)
-                    extracted.forwardMessageId = fwd.forwardMessageId;
-                if (fwd.forwardMessage) {
-                    const nested = extractForwardedMessage(fwd.forwardMessage, depth + 1);
-                    if (nested)
-                        extracted.forwardMessage = nested;
-                }
-                return extracted;
-            }
-            const optimizedMessages = messages.map((msg) => {
-                let cleanMsg = msg.msg || '';
-                if (cleanMsg) {
-                    cleanMsg = cleanMsg.replace(/data:[^;]+;base64,[A-Za-z0-9+/=]+/g, '[FILE_REMOVED]');
-                    cleanMsg = cleanMsg.replace(/!\[.*?\]\(data:[^)]+\)/g, '[IMAGE_REMOVED]');
-                }
-                const userInfo = (0, workspace_cache_1.getUserById)(context.workspaceCache, msg.uid);
-                let username = `User ${msg.uid}`;
-                if (userInfo) {
-                    const fullName = `${userInfo.firstname || ''} ${userInfo.lastname || ''}`.trim();
-                    username = fullName || userInfo.fullName || `User ${msg.uid}`;
-                }
-                const optimizedMsg = {
-                    _id: msg._id,
-                    uid: msg.uid,
-                    username: username,
-                    created: new Date(msg.created).toLocaleString(),
-                    type: msg.type || 'user',
-                    msg: cleanMsg
-                };
-                if (msg.replyTo) {
-                    optimizedMsg.replyTo = msg.replyTo;
-                }
-                if (msg.forwardMessageId) {
-                    optimizedMsg.forwardMessageId = msg.forwardMessageId;
-                }
-                if (msg.forwardMessage) {
-                    const forwarded = extractForwardedMessage(msg.forwardMessage, 0);
-                    if (forwarded)
-                        optimizedMsg.forwardMessage = forwarded;
-                }
-                return optimizedMsg;
-            });
-            const optimizedResult = {
-                messages: optimizedMessages,
-                totalFetched: result.totalFetched,
-                batchesCompleted: result.batchesCompleted,
-                oldestMessageId: result.oldestMessageId,
-                hasMore: result.hasMore
-            };
-            let responseText = `📬 Successfully fetched ${messageCount} previous messages`;
-            responseText += `\n\n📊 Summary:`;
-            responseText += `\n- ${userMessages.length} user messages`;
-            responseText += `\n- ${messages.length - userMessages.length} system messages`;
-            responseText += `\n- Batches completed: ${result.batchesCompleted}/${batches}`;
-            responseText += `\n- Total messages fetched: ${result.totalFetched}`;
-            if (oldestMessage) {
-                responseText += `\n- Oldest message: "${oldestMessage.msg || 'System event'}" at ${new Date(oldestMessage.created).toLocaleString()}`;
-            }
-            if (result.hasMore && optimizedMessages.length > 0) {
-                const oldestOptimizedMessage = optimizedMessages[optimizedMessages.length - 1];
-                responseText += `\n\n🔄 **CONTINUE PAGINATION**: To get even older messages:`;
-                responseText += `\n- oldestMessageId: "${oldestOptimizedMessage._id}"`;
-                responseText += `\n- batches: 1-10 (for more history)`;
-            }
-            else if (!result.hasMore) {
-                responseText += `\n\n✅ **END OF HISTORY**: No more messages available in this discussion.`;
-            }
-            responseText += `\n\n💡 Use this to explore conversation history and understand context!\n\nOptimized conversation data (essential fields only):\n${JSON.stringify(optimizedResult, null, 2)}`;
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: responseText,
-                    },
-                ],
-            };
-        }
-        catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error fetching previous discussion messages", error);
-            return {
-                content: [
-                    {
-                        type: "text",
-                        text: `❌ Error fetching previous discussion messages: ${error instanceof Error ? error.message : String(error)}\n\n💡 **IMPORTANT**: Make sure you're using a MESSAGE ID, not a DISCUSSION ID!\n\n**How to get the correct oldestMessageId:**\n1. First call fetch_discussion_messages with your discussionId\n2. Look at the last message in the results\n3. Use that message's '_id' field as oldestMessageId\n\n**Example:**\n- ❌ Wrong: "682dcfa9d36a4907a88bb279" (this is a discussion ID)\n- ✅ Correct: "68e5c1903b104307efdebce9" (this is a message ID)\n\nMessage IDs are usually different from discussion IDs!`,
-                    },
-                ],
-            };
+        if (args.beforeMessageId) {
+            return fetchPreviousMessages(args, context);
         }
+        return fetchLatestMessages(args, context);
     },
 };
 // ============================================================================
-// TOOL 4: JOIN DISCUSSION
+// TOOL 3: JOIN DISCUSSION
 // ============================================================================
 const joinDiscussionDescription = `<purpose>Bot joins a discussion and optionally invites ONE user</purpose>
 <when-to-use>
@@ -599,30 +643,30 @@ exports.joinDiscussionTool = {
     schema: zod_1.z.object({
         activityId: zod_1.z
             .string()
-            .min(24, "Activity ID must be at least 24 characters")
+            .min(24, 'Activity ID must be at least 24 characters')
             .optional()
-            .describe("Activity ID to join its discussion (preferred method for activity discussions)"),
+            .describe('Activity ID to join its discussion (preferred method for activity discussions)'),
         discussionId: zod_1.z
             .string()
-            .min(24, "Discussion ID must be at least 24 characters")
+            .min(24, 'Discussion ID must be at least 24 characters')
             .optional()
-            .describe("Discussion ID to join directly (use this for non-activity discussions)"),
+            .describe('Discussion ID to join directly (use this for non-activity discussions)'),
         inviteUserId: zod_1.z
             .string()
-            .min(24, "User ID must be at least 24 characters")
+            .min(24, 'User ID must be at least 24 characters')
             .optional()
-            .describe("User ID to invite. Get from incoming message user_id attribute when user asks to be invited."),
+            .describe('User ID to invite. Get from incoming message user_id attribute when user asks to be invited.'),
         welcomeReason: zod_1.z
             .string()
             .optional()
-            .describe("Context/reason for the invite - posted as welcome message in the discussion."),
+            .describe('Context/reason for the invite - posted as welcome message in the discussion.'),
         sourceActivityId: zod_1.z
             .string()
             .min(24)
             .optional()
             .describe("IMPORTANT: The activity_id from the CURRENT discussion where the request was made. This creates a 'came from' link in the welcome message. Get this from the incoming message's activity_id attribute.")
     }).refine((data) => data.activityId || data.discussionId, {
-        message: "Either activityId or discussionId must be provided"
+        message: 'Either activityId or discussionId must be provided'
     }),
     async execute(args, context) {
         logger.debug('Joining discussion', {
@@ -694,7 +738,7 @@ exports.joinDiscussionTool = {
             else {
                 return {
                     content: [{
-                            type: "text",
+                            type: 'text',
                             text: `❌ Error: Either activityId or discussionId must be provided`,
                         }],
                 };
@@ -819,14 +863,15 @@ exports.joinDiscussionTool = {
             }
             return {
                 content: [{
-                        type: "text",
+                        type: 'text',
                         text: responseText,
                     }],
             };
         }
         catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error joining discussion", error);
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                logger.error('Error joining discussion', error);
+            }
             const errorMessage = error instanceof Error
                 ? error.message
                 : (typeof error === 'object' && error !== null)
@@ -835,7 +880,7 @@ exports.joinDiscussionTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Error joining discussion: ${errorMessage}\n\n` +
                             `💡 Troubleshooting:\n` +
                             `- For activity discussions: Use join_discussion(activityId: "...")\n` +
@@ -853,7 +898,7 @@ exports.joinDiscussionTool = {
     },
 };
 // ============================================================================
-// TOOL 5: LEAVE DISCUSSION
+// TOOL 4: LEAVE DISCUSSION
 // ============================================================================
 const leaveDiscussionDescription = `Leave a discussion - Bot removes itself from a discussion.
 
@@ -873,20 +918,20 @@ exports.leaveDiscussionTool = {
     schema: zod_1.z.object({
         activityId: zod_1.z
             .string()
-            .min(24, "Activity ID must be at least 24 characters")
+            .min(24, 'Activity ID must be at least 24 characters')
             .optional()
-            .describe("Activity ID to leave its discussion (preferred method for activity discussions)"),
+            .describe('Activity ID to leave its discussion (preferred method for activity discussions)'),
         discussionId: zod_1.z
             .string()
-            .min(24, "Discussion ID must be at least 24 characters")
+            .min(24, 'Discussion ID must be at least 24 characters')
             .optional()
-            .describe("Discussion ID to leave directly")
+            .describe('Discussion ID to leave directly')
     }).refine((data) => data.activityId || data.discussionId, {
-        message: "Either activityId or discussionId must be provided"
+        message: 'Either activityId or discussionId must be provided'
     }),
     async execute(args, context) {
         // DEBUG: Log raw input to diagnose LLM tool calling issues
-        logger.debug("leave_discussion called", {
+        logger.debug('leave_discussion called', {
             hasActivityId: !!args.activityId,
             hasDiscussionId: !!args.discussionId,
             activityId: args.activityId || '(not provided)',
@@ -906,7 +951,7 @@ exports.leaveDiscussionTool = {
                 return {
                     content: [
                         {
-                            type: "text",
+                            type: 'text',
                             text: `✅ Successfully left activity discussion!\n\n` +
                                 `📋 **Activity**: ${activity.name || args.activityId}\n` +
                                 `💬 **Discussion ID**: ${discussionId}\n` +
@@ -925,7 +970,7 @@ exports.leaveDiscussionTool = {
                 return {
                     content: [
                         {
-                            type: "text",
+                            type: 'text',
                             text: `✅ Successfully left discussion!\n\n` +
                                 `💬 **Discussion ID**: ${args.discussionId}\n\n` +
                                 `🔔 You will no longer receive notifications from this discussion.\n\n` +
@@ -938,15 +983,16 @@ exports.leaveDiscussionTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Error: Either activityId or discussionId must be provided`,
                     },
                 ],
             };
         }
         catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error leaving discussion", error);
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                logger.error('Error leaving discussion', error);
+            }
             const errorMessage = error instanceof Error
                 ? error.message
                 : (typeof error === 'object' && error !== null)
@@ -955,7 +1001,7 @@ exports.leaveDiscussionTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Error leaving discussion: ${errorMessage}\n\n` +
                             `💡 Troubleshooting:\n` +
                             `- For activity discussions: Use leave_discussion(activityId: "...")\n` +
@@ -973,7 +1019,7 @@ exports.leaveDiscussionTool = {
     },
 };
 // ============================================================================
-// TOOL 6: ADD DISCUSSION MESSAGE
+// TOOL 5: ADD DISCUSSION MESSAGE
 // ============================================================================
 const addDiscussionMessageDescription = `Post a message to a Hailer discussion with optional file attachments - AI agents can participate in conversations`;
 exports.addDiscussionMessageTool = {
@@ -981,9 +1027,9 @@ exports.addDiscussionMessageTool = {
     group: tool_registry_1.ToolGroup.WRITE,
     description: addDiscussionMessageDescription,
     schema: zod_1.z.object({
-        discussionId: zod_1.z.string().min(24, "Discussion ID must be at least 24 characters").describe("The discussion ID where to post the message"),
-        content: zod_1.z.string().min(1, "Message content cannot be empty").describe("The message text to post"),
-        fileIds: zod_1.z.array(zod_1.z.string()).optional().describe("Optional array of file IDs to attach (from upload_files tool)")
+        discussionId: zod_1.z.string().min(24, 'Discussion ID must be at least 24 characters').describe('The discussion ID where to post the message'),
+        content: zod_1.z.string().min(1, 'Message content cannot be empty').describe('The message text to post'),
+        fileIds: zod_1.z.array(zod_1.z.string()).optional().describe('Optional array of file IDs to attach (from upload_files tool)')
     }),
     async execute(args, context) {
         try {
@@ -996,7 +1042,7 @@ exports.addDiscussionMessageTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: responseText,
                     },
                 ],
@@ -1006,7 +1052,7 @@ exports.addDiscussionMessageTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Error posting message: ${error instanceof Error ? error.message : (typeof error === 'object' ? JSON.stringify(error) : String(error))}\n\n💡 Tips:\n- Make sure the discussion ID is correct (24-character string)\n- Check that you have permission to post to this discussion\n- Verify the discussion exists and is accessible\n- If attaching files, ensure file IDs are valid (from upload_files tool)\n\nExample discussion ID: '683ef53087e3d8329abaa3ad'`,
                     },
                 ],
@@ -1015,7 +1061,7 @@ exports.addDiscussionMessageTool = {
     },
 };
 // ============================================================================
-// TOOL 7: INVITE DISCUSSION MEMBERS
+// TOOL 6: INVITE DISCUSSION MEMBERS
 // ============================================================================
 const inviteDiscussionMembersDescription = `<purpose>Invite users to a discussion WITHOUT the bot joining. Use this to add people to activity discussions.</purpose>
 <when-to-use>
@@ -1031,12 +1077,12 @@ exports.inviteDiscussionMembersTool = {
     schema: zod_1.z.object({
         discussionId: zod_1.z
             .string()
-            .min(24, "Discussion ID must be at least 24 characters")
-            .describe("The discussion ID where to invite members"),
+            .min(24, 'Discussion ID must be at least 24 characters')
+            .describe('The discussion ID where to invite members'),
         userIds: zod_1.z
-            .array(zod_1.z.string().min(24, "User ID must be at least 24 characters"))
-            .min(1, "At least one user ID must be provided")
-            .describe("Array of user IDs to invite to the discussion. Use search_workspace_users to find user IDs")
+            .array(zod_1.z.string().min(24, 'User ID must be at least 24 characters'))
+            .min(1, 'At least one user ID must be provided')
+            .describe('Array of user IDs to invite to the discussion. Use search_workspace_users to find user IDs')
     }),
     async execute(args, context) {
         logger.debug('Inviting members to discussion', {
@@ -1059,7 +1105,7 @@ exports.inviteDiscussionMembersTool = {
             if (isDm) {
                 return {
                     content: [{
-                            type: "text",
+                            type: 'text',
                             text: `❌ Cannot invite members to a private chat.\n\n` +
                                 `Private chats are 1-on-1 conversations and cannot have additional members.\n` +
                                 `💡 To include more people, create a group discussion instead.`
@@ -1120,7 +1166,7 @@ exports.inviteDiscussionMembersTool = {
                 const alreadyMembersList = alreadyMembers.map((userId) => `- ${getUserName(userId)} (${userId})`).join('\n');
                 return {
                     content: [{
-                            type: "text",
+                            type: 'text',
                             text: `ℹ️ All users are already members of this discussion!\n\n` +
                                 `💬 **Discussion ID**: ${args.discussionId}\n` +
                                 `👥 **Already Members**:\n${alreadyMembersList}\n\n` +
@@ -1150,12 +1196,13 @@ exports.inviteDiscussionMembersTool = {
                 `- Receive notifications for new activity\n\n` +
                 `🔗 **Discussion Link:** https://app.hailer.com/#/discussions/${args.discussionId}`;
             return {
-                content: [{ type: "text", text: responseText }],
+                content: [{ type: 'text', text: responseText }],
             };
         }
         catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error inviting discussion members", error);
+            if (!request_logger_1.RequestLogger.getCurrent()) {
+                logger.error('Error inviting discussion members', error);
+            }
             const errorMessage = error instanceof Error
                 ? error.message
                 : (typeof error === 'object' && error !== null)
@@ -1164,7 +1211,7 @@ exports.inviteDiscussionMembersTool = {
             return {
                 content: [
                     {
-                        type: "text",
+                        type: 'text',
                         text: `❌ Error inviting members to discussion: ${errorMessage}\n\n` +
                             `💡 Troubleshooting:\n` +
                             `- Verify the discussion ID is correct (24-character string)\n` +
@@ -1184,177 +1231,13 @@ exports.inviteDiscussionMembersTool = {
 };
 // Note: Hailer API does not have a "kick" or "remove members" feature.
 // Bots can only remove themselves using the leave_discussion tool.
-// ============================================================================
-// TOOL 8: GET ACTIVITY FROM DISCUSSION
-// ============================================================================
-const getActivityFromDiscussionDescription = `🔍 Reverse lookup: Get activity details from a discussion ID
-
-**Purpose**: When you have a discussion ID but need to know what activity it belongs to.
-
-**Example**:
-\`\`\`javascript
-get_activity_from_discussion({
-  discussionId: "691ffe874217e9e8434e57f5"
-})
-\`\`\`
-
-**Returns**: Activity name, ID, workflow info, phase, and field values
-
-**Use Cases**:
-- Identify which activity a discussion belongs to
-- Get activity context when processing discussion messages
-- Navigate from chat to activity details`;
-exports.getActivityFromDiscussionTool = {
-    name: 'get_activity_from_discussion',
-    group: tool_registry_1.ToolGroup.READ,
-    description: getActivityFromDiscussionDescription,
-    schema: zod_1.z.object({
-        discussionId: zod_1.z
-            .string()
-            .min(24, "Discussion ID must be at least 24 characters")
-            .describe("The discussion ID to look up the associated activity for")
-    }),
-    async execute(args, context) {
-        logger.debug('Getting activity from discussion', {
-            discussionId: args.discussionId,
-            apiKey: context.apiKey.substring(0, 8) + '...'
-        });
-        try {
-            // First, sync discussions to find the one with this ID
-            const syncResponse = await context.hailer.request('v2.discussion.sync', [{ timestamp: 0 }]);
-            const discussions = syncResponse.discussions || [];
-            const targetDiscussion = discussions.find((d) => d._id === args.discussionId);
-            if (!targetDiscussion) {
-                return {
-                    content: [{
-                            type: "text",
-                            text: `❌ Discussion not found: ${args.discussionId}\n\n` +
-                                `💡 **Possible reasons:**\n` +
-                                `- You are not a member of this discussion\n` +
-                                `- The discussion ID is incorrect\n` +
-                                `- The discussion has been deleted\n\n` +
-                                `**Tip**: Use \`list_my_discussions\` to see discussions you have access to.`
-                        }]
-                };
-            }
-            // Check if this discussion is linked to an activity
-            if (!targetDiscussion.linked_activity) {
-                let discussionType = 'unknown';
-                if (targetDiscussion.linked_event) {
-                    discussionType = 'event';
-                }
-                else if (targetDiscussion.private === true) {
-                    discussionType = 'private';
-                }
-                else if (targetDiscussion.private === false) {
-                    discussionType = 'group';
-                }
-                return {
-                    content: [{
-                            type: "text",
-                            text: `⚠️ Discussion found but not linked to an activity\n\n` +
-                                `💬 **Discussion ID**: \`${args.discussionId}\`\n` +
-                                `📋 **Type**: ${discussionType}\n` +
-                                `📝 **Name**: ${sanitizeString(targetDiscussion.name) || 'Untitled'}\n` +
-                                (targetDiscussion.linked_event ? `📅 **Event ID**: \`${targetDiscussion.linked_event}\`\n` : '') +
-                                `\n💡 This discussion is not associated with any activity.`
-                        }]
-                };
-            }
-            // Fetch the linked activity details
-            const activityId = targetDiscussion.linked_activity;
-            const activity = await context.hailer.fetchActivityById(activityId);
-            if (!activity) {
-                return {
-                    content: [{
-                            type: "text",
-                            text: `⚠️ Discussion is linked to activity \`${activityId}\` but activity could not be loaded.\n\n` +
-                                `💡 **Possible reasons:**\n` +
-                                `- The activity has been deleted\n` +
-                                `- You don't have permission to view the activity\n` +
-                                `- The activity is in a different workspace`
-                        }]
-                };
-            }
-            // Build comprehensive response
-            let responseText = `✅ **Activity Found!**\n\n`;
-            responseText += `📋 **Activity Details:**\n`;
-            responseText += `- **Name**: ${activity.name || 'Untitled'}\n`;
-            responseText += `- **Activity ID**: \`${activity._id}\`\n`;
-            responseText += `- **Discussion ID**: \`${args.discussionId}\`\n`;
-            if (activity.workflowId) {
-                responseText += `- **Workflow ID**: \`${activity.workflowId}\`\n`;
-            }
-            if (activity.workflowName) {
-                responseText += `- **Workflow**: ${activity.workflowName}\n`;
-            }
-            if (activity.phaseName) {
-                responseText += `- **Phase**: ${activity.phaseName}\n`;
-            }
-            if (activity.phaseId) {
-                responseText += `- **Phase ID**: \`${activity.phaseId}\`\n`;
-            }
-            if (activity.created) {
-                responseText += `- **Created**: ${new Date(activity.created).toLocaleString()}\n`;
-            }
-            if (activity.updated) {
-                responseText += `- **Updated**: ${new Date(activity.updated).toLocaleString()}\n`;
-            }
-            // Add field values if present
-            if (activity.fields && Object.keys(activity.fields).length > 0) {
-                responseText += `\n📊 **Field Values:**\n`;
-                for (const [fieldId, value] of Object.entries(activity.fields)) {
-                    if (value !== null && value !== undefined && value !== '') {
-                        const displayValue = typeof value === 'object' ? JSON.stringify(value) : String(value);
-                        responseText += `- \`${fieldId}\`: ${displayValue}\n`;
-                    }
-                }
-            }
-            responseText += `\n🔗 **Links:**\n`;
-            responseText += `- Activity: https://app.hailer.com/#/activities/${activity._id}\n`;
-            responseText += `- Discussion: https://app.hailer.com/#/discussions/${args.discussionId}\n`;
-            responseText += `\n💡 **Next Steps:**\n`;
-            responseText += `- View full activity: \`show_activity_by_id({ activityId: "${activity._id}" })\`\n`;
-            responseText += `- List workflow activities: \`list_activities({ workflowId: "${activity.workflowId}", phaseId: "${activity.phaseId}", fields: [...] })\``;
-            return {
-                content: [{
-                        type: "text",
-                        text: responseText
-                    }]
-            };
-        }
-        catch (error) {
-            if (!request_logger_1.RequestLogger.getCurrent())
-                logger.error("Error getting activity from discussion", error);
-            const errorMessage = error instanceof Error
-                ? error.message
-                : (typeof error === 'object' && error !== null)
-                    ? JSON.stringify(error, null, 2)
-                    : String(error);
-            return {
-                content: [{
-                        type: "text",
-                        text: `❌ Error getting activity from discussion: ${errorMessage}\n\n` +
-                            `💡 **Troubleshooting:**\n` +
-                            `- Verify the discussion ID is correct (24-character string)\n` +
-                            `- Ensure you have access to the discussion\n` +
-                            `- Try \`list_my_discussions\` to see available discussions\n\n` +
-                            `📖 **Example:**\n` +
-                            `get_activity_from_discussion({ discussionId: "691ffe874217e9e8434e57f5" })`
-                    }]
-            };
-        }
-    }
-};
 /** All discussion tools */
 exports.discussionTools = [
     exports.listMyDiscussionsTool,
     exports.fetchDiscussionMessagesTool,
-    exports.fetchPreviousDiscussionMessagesTool,
+    exports.addDiscussionMessageTool,
     exports.joinDiscussionTool,
     exports.leaveDiscussionTool,
-    exports.addDiscussionMessageTool,
     exports.inviteDiscussionMembersTool,
-    exports.getActivityFromDiscussionTool,
 ];
 //# sourceMappingURL=discussion.js.map