ms365-mcp-server 1.1.10 → 1.1.12

package/dist/index.js

@@ -67,7 +67,7 @@ function parseArgs() {
 }
 const server = new Server({
     name: "ms365-mcp-server",
-    version: "1.1.10"
+    version: "1.1.12"
 }, {
     capabilities: {
         resources: {
@@ -83,6 +83,7 @@ const server = new Server({
         }
     }
 });
+logger.log('Server started with version 1.1.12');
 // Set up the resource listing request handler
 server.setRequestHandler(ListResourcesRequestSchema, async () => {
     logger.log('Received list resources request');
@@ -185,7 +186,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
         },
         {
             name: "manage_email",
-            description: "UNIFIED EMAIL MANAGEMENT: Read, search, list, mark, move, or delete emails. Combines all email operations in one powerful tool. Supports intelligent partial name matching, folder management, and advanced search criteria.",
+            description: "UNIFIED EMAIL MANAGEMENT: Read, search, list, mark, move, or delete emails. Combines all email operations in one powerful tool. Supports intelligent partial name matching, folder management, and advanced search criteria. IMPORTANT: Use 'reply_draft' or 'forward_draft' actions (not 'draft') to create threaded drafts that appear in email conversations.",
             inputSchema: {
                 type: "object",
                 properties: {
@@ -196,7 +197,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     action: {
                         type: "string",
                         enum: ["read", "search", "list", "mark", "move", "delete", "search_to_me", "draft", "update_draft", "send_draft", "list_drafts", "reply_draft", "forward_draft"],
-                        description: "Action to perform: read (get email by ID), search (find emails), list (folder contents), mark (read/unread), move (to folder), delete (permanently), search_to_me (emails addressed to you), draft (create/save draft), update_draft (modify existing draft), send_draft (send saved draft), list_drafts (list draft emails), reply_draft (create reply draft), forward_draft (create forward draft)"
+                        description: "Action to perform: read (get email by ID), search (find emails), list (folder contents), mark (read/unread), move (to folder), delete (permanently), search_to_me (emails addressed to you), draft (create standalone draft - NOTE: Use reply_draft/forward_draft for threaded drafts), update_draft (modify existing draft), send_draft (send saved draft), list_drafts (list draft emails), reply_draft (create threaded reply draft that appears in conversation), forward_draft (create threaded forward draft that appears in conversation)"
                     },
                     messageId: {
                         type: "string",
@@ -631,6 +632,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 }
             // ============ UNIFIED EMAIL MANAGEMENT TOOL ============
             case "manage_email":
+                // Debug logging for parameter validation
+                logger.log('DEBUG: manage_email called with args:', JSON.stringify(args, null, 2));
                 if (ms365Config.multiUser) {
                     const userId = args?.userId;
                     if (!userId) {
@@ -644,6 +647,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                     ms365Ops.setGraphClient(graphClient);
                 }
                 const emailAction = args?.action;
+                logger.log(`DEBUG: Email action: ${emailAction}`);
                 switch (emailAction) {
                     case "read":
                         if (!args?.messageId) {
@@ -681,6 +685,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                         };
                     case "search":
                         const searchResults = await ms365Ops.searchEmails(args);
+                        logger.log(`DEBUG: Search results count: ${searchResults.messages.length}`);
+                        logger.log(`DEBUG: First result:`, JSON.stringify(searchResults.messages[0], null, 2));
                         // Enhanced feedback for search results
                         let responseText = `🔍 Email Search Results (${searchResults.messages.length} found)`;
                         if (searchResults.messages.length === 0) {
@@ -698,7 +704,12 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                             responseText += `• Remove some search criteria to get broader results`;
                         }
                         else {
-                            responseText += `\n\n${searchResults.messages.map((email, index) => `${index + 1}. 📧 ${email.subject}\n   👤 From: ${email.from.name} <${email.from.address}>\n   📅 ${new Date(email.receivedDateTime).toLocaleDateString()}\n   ${email.isRead ? '📖 Read' : '📩 Unread'}\n   🆔 ID: ${email.id}\n`).join('\n')}`;
+                            responseText += `\n\n${searchResults.messages.map((email, index) => {
+                                // Handle missing IDs more gracefully
+                                const emailId = email.id || 'ID_MISSING';
+                                const fromDisplay = email.from?.name ? `${email.from.name} <${email.from.address}>` : email.from?.address || 'Unknown sender';
+                                return `${index + 1}. 📧 ${email.subject || 'No subject'}\n   👤 From: ${fromDisplay}\n   📅 ${email.receivedDateTime ? new Date(email.receivedDateTime).toLocaleDateString() : 'Unknown date'}\n   ${email.isRead ? '📖 Read' : '📩 Unread'}\n   🆔 ID: ${emailId}\n`;
+                            }).join('\n')}`;
                             if (searchResults.hasMore) {
                                 responseText += `\n💡 There are more results available. Use maxResults parameter to get more emails.`;
                             }
@@ -840,6 +851,11 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                         if (!args?.draftTo || !args?.draftSubject || !args?.draftBody) {
                             throw new Error("draftTo, draftSubject, and draftBody are required for draft action");
                         }
+                        // Check if this looks like a reply based on subject
+                        const isLikelyReply = args.draftSubject?.toString().toLowerCase().startsWith('re:');
+                        if (isLikelyReply) {
+                            logger.log('🔔 SUGGESTION: Consider using "reply_draft" action instead of "draft" for threaded replies that appear in conversations');
+                        }
                         const draftResult = await ms365Ops.saveDraftEmail({
                             to: args.draftTo,
                             cc: args.draftCc,
@@ -851,11 +867,16 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                             attachments: args.draftAttachments,
                             conversationId: args.conversationId
                         });
+                        let draftResponseText = `✅ Draft email saved successfully!\n📧 Subject: ${args.draftSubject}\n👥 To: ${Array.isArray(args.draftTo) ? args.draftTo.join(', ') : args.draftTo}\n🆔 Draft ID: ${draftResult.id}`;
+                        // Add helpful suggestion for threaded drafts
+                        if (isLikelyReply) {
+                            draftResponseText += `\n\n💡 TIP: This looks like a reply email. For drafts that appear in email threads, use "reply_draft" action with the original message ID instead of "draft".`;
+                        }
                         return {
                             content: [
                                 {
                                     type: "text",
-                                    text: `✅ Draft email saved successfully!\n📧 Subject: ${args.draftSubject}\n👥 To: ${Array.isArray(args.draftTo) ? args.draftTo.join(', ') : args.draftTo}\n🆔 Draft ID: ${draftResult.id}`
+                                    text: draftResponseText
                                 }
                             ]
                         };