@masonator/m365-mcp 0.2.0 → 0.5.0

package/dist/__tests__/auth.test.js

@@ -2,7 +2,13 @@ import { jest } from '@jest/globals';
 import { mkdtempSync, rmSync, readFileSync, writeFileSync, statSync, existsSync } from 'node:fs';
 import { join } from 'node:path';
 import { tmpdir } from 'node:os';
-import { getConfigDir, loadTokens, saveTokens, deleteTokens, isTokenExpired, loadAuthConfig, SCOPES, refreshAccessToken, getAccessToken, findAvailablePort, exchangeCodeForTokens, openBrowser, waitForAuthCallback, } from '../lib/auth.js';
+// Mock child_process to prevent browser tabs opening during tests
+const mockExecFile = jest.fn();
+jest.unstable_mockModule('node:child_process', () => ({
+    execFile: mockExecFile,
+}));
+// Dynamic import AFTER mock registration
+const { getConfigDir, loadTokens, saveTokens, deleteTokens, isTokenExpired, loadAuthConfig, SCOPES, refreshAccessToken, getAccessToken, findAvailablePort, exchangeCodeForTokens, openBrowser, waitForAuthCallback, } = await import('../lib/auth.js');
 const sampleTokens = {
     access_token: 'access-abc-123',
     refresh_token: 'refresh-xyz-789',
@@ -382,8 +388,23 @@ describe('findAvailablePort', () => {
     });
 });
 describe('openBrowser', () => {
-    it('does not throw for a URL string', () => {
-        // openBrowser calls execFile which may fail, but should not throw
+    afterEach(() => {
+        mockExecFile.mockClear();
+    });
+    it('calls execFile with the correct command for the platform', () => {
+        openBrowser('https://example.com');
+        // On macOS (darwin), should call 'open'
+        if (process.platform === 'darwin') {
+            expect(mockExecFile).toHaveBeenCalledWith('open', ['https://example.com']);
+        }
+        else {
+            expect(mockExecFile).toHaveBeenCalled();
+        }
+    });
+    it('does not throw when execFile throws', () => {
+        mockExecFile.mockImplementation(() => {
+            throw new Error('Command not found');
+        });
         expect(() => openBrowser('https://example.com')).not.toThrow();
     });
 });

package/dist/__tests__/tools/chat.test.js

@@ -4,7 +4,7 @@ jest.unstable_mockModule('../../lib/graph.js', () => ({
     graphFetch: mockGraphFetch,
 }));
 // Dynamic import AFTER the mock is registered
-const { executeChat } = await import('../../lib/tools/chat.js');
+const { executeChat, stripHtml } = await import('../../lib/tools/chat.js');
 describe('executeChat', () => {
     afterEach(() => {
         mockGraphFetch.mockReset();
@@ -159,4 +159,74 @@ describe('executeChat', () => {
         const result = await executeChat('test-token', { chat_id: 'chat-123' });
         expect(result).toContain('(empty message)');
     });
+    it('strips HTML from chat listing preview', async () => {
+        mockGraphFetch.mockResolvedValue({
+            ok: true,
+            data: {
+                value: [
+                    {
+                        id: 'chat-html',
+                        topic: 'HTML Preview Chat',
+                        chatType: 'group',
+                        lastMessagePreview: {
+                            body: { content: '<p>Hey <at id="0">Stuart</at>, check this out</p>' },
+                            createdDateTime: '2025-06-15T10:00:00Z',
+                        },
+                    },
+                ],
+            },
+        });
+        const result = await executeChat('test-token', {});
+        expect(result).toContain('Hey Stuart, check this out');
+        expect(result).not.toContain('<p>');
+        expect(result).not.toContain('<at');
+    });
+    it('handles emoji tags in messages', async () => {
+        mockGraphFetch.mockResolvedValue({
+            ok: true,
+            data: {
+                value: [
+                    {
+                        from: { user: { displayName: 'Alice' } },
+                        createdDateTime: '2025-06-15T10:30:00Z',
+                        body: {
+                            content: '<p>Great work <emoji id="thumbsup" alt="👍"></emoji></p>',
+                            contentType: 'html',
+                        },
+                    },
+                ],
+            },
+        });
+        const result = await executeChat('test-token', { chat_id: 'chat-123' });
+        expect(result).toContain('Great work 👍');
+        expect(result).not.toContain('<emoji');
+        expect(result).not.toContain('<p>');
+    });
+});
+describe('stripHtml', () => {
+    it('converts br tags to newlines', () => {
+        expect(stripHtml('line1<br>line2<br/>line3')).toBe('line1\nline2\nline3');
+    });
+    it('converts closing p tags to newlines', () => {
+        expect(stripHtml('<p>First</p><p>Second</p>')).toBe('First\nSecond');
+    });
+    it('extracts emoji alt text', () => {
+        expect(stripHtml('<emoji id="thumbsup" alt="👍"></emoji>')).toBe('👍');
+        expect(stripHtml('<emoji id="smile" alt="😊"/>')).toBe('😊');
+    });
+    it('preserves text inside at-mention tags', () => {
+        expect(stripHtml('<at id="0">Stuart Mason</at>')).toBe('Stuart Mason');
+    });
+    it('removes attachment tags and content', () => {
+        expect(stripHtml('Hello<attachment id="abc">file.pdf</attachment> world')).toBe('Hello world');
+    });
+    it('decodes HTML entities', () => {
+        expect(stripHtml('a &amp; b &lt; c &gt; d &quot;e&quot; f&#39;s')).toBe('a & b < c > d "e" f\'s');
+    });
+    it('collapses excessive newlines', () => {
+        expect(stripHtml('<p></p><p></p><p>Content</p>')).toBe('Content');
+    });
+    it('trims whitespace', () => {
+        expect(stripHtml('  <p>Hello</p>  ')).toBe('Hello');
+    });
 });

package/dist/__tests__/tools/mail.test.js

@@ -31,13 +31,25 @@ describe('executeMail', () => {
         expect(result).toContain('Importance: high | Read: Yes');
         expect(result).toContain('Here are the meeting notes from today.');
     });
-    it('handles search param', async () => {
+    it('handles search param without $orderBy', async () => {
         mockGraphFetch.mockResolvedValue({
             ok: true,
             data: { value: [] },
         });
         await executeMail('test-token', { search: 'quarterly report' });
-        expect(mockGraphFetch).toHaveBeenCalledWith(expect.stringContaining('$search="quarterly%20report"'), 'test-token', { timezone: false });
+        const calledPath = mockGraphFetch.mock.calls[0][0];
+        expect(calledPath).toContain('$search="quarterly%20report"');
+        expect(calledPath).not.toContain('$orderby');
+    });
+    it('includes $orderBy when no search is provided', async () => {
+        mockGraphFetch.mockResolvedValue({
+            ok: true,
+            data: { value: [] },
+        });
+        await executeMail('test-token', {});
+        const calledPath = mockGraphFetch.mock.calls[0][0];
+        expect(calledPath).toContain('$orderby=receivedDateTime desc');
+        expect(calledPath).not.toContain('$search');
     });
     it('handles empty results', async () => {
         mockGraphFetch.mockResolvedValue({
@@ -166,4 +178,103 @@ describe('executeMail', () => {
             timezone: false,
         });
     });
+    it('includes message ID in list results', async () => {
+        mockGraphFetch.mockResolvedValue({
+            ok: true,
+            data: {
+                value: [
+                    {
+                        id: 'AAMkAGI2',
+                        subject: 'Test Email',
+                        from: { emailAddress: { name: 'Alice', address: 'alice@example.com' } },
+                        receivedDateTime: '2025-06-15T10:00:00Z',
+                        bodyPreview: 'Preview text',
+                        isRead: true,
+                        importance: 'normal',
+                    },
+                ],
+            },
+        });
+        const result = await executeMail('test-token', {});
+        expect(result).toContain('Message ID: AAMkAGI2');
+    });
+    it('includes id in $select for list mode', async () => {
+        mockGraphFetch.mockResolvedValue({
+            ok: true,
+            data: { value: [] },
+        });
+        await executeMail('test-token', {});
+        const calledPath = mockGraphFetch.mock.calls[0][0];
+        expect(calledPath).toContain('$select=id,');
+    });
+    describe('drill-down mode', () => {
+        it('fetches full email body by message_id', async () => {
+            mockGraphFetch.mockResolvedValue({
+                ok: true,
+                data: {
+                    subject: 'Quarterly Report',
+                    from: { emailAddress: { name: 'Alice', address: 'alice@example.com' } },
+                    receivedDateTime: '2025-06-15T10:00:00Z',
+                    body: { contentType: 'text', content: 'Full body content here.' },
+                    isRead: true,
+                    importance: 'high',
+                    toRecipients: [{ emailAddress: { name: 'Bob', address: 'bob@example.com' } }],
+                    ccRecipients: [{ emailAddress: { name: 'Carol', address: 'carol@example.com' } }],
+                },
+            });
+            const result = await executeMail('test-token', { message_id: 'AAMkAGI2' });
+            expect(result).toContain('# Quarterly Report');
+            expect(result).toContain('From: Alice <alice@example.com>');
+            expect(result).toContain('To: Bob <bob@example.com>');
+            expect(result).toContain('Cc: Carol <carol@example.com>');
+            expect(result).toContain('Full body content here.');
+            expect(mockGraphFetch).toHaveBeenCalledWith(expect.stringContaining('/me/messages/AAMkAGI2'), 'test-token', { timezone: false });
+        });
+        it('strips HTML from email body', async () => {
+            mockGraphFetch.mockResolvedValue({
+                ok: true,
+                data: {
+                    subject: 'HTML Email',
+                    from: { emailAddress: { name: 'Alice', address: 'alice@example.com' } },
+                    body: {
+                        contentType: 'html',
+                        content: '<html><body><p>Hello world</p><p>Second paragraph</p></body></html>',
+                    },
+                    isRead: true,
+                    importance: 'normal',
+                },
+            });
+            const result = await executeMail('test-token', { message_id: 'msg-html' });
+            expect(result).toContain('Hello world');
+            expect(result).toContain('Second paragraph');
+            expect(result).not.toContain('<p>');
+            expect(result).not.toContain('<html>');
+        });
+        it('handles missing body gracefully', async () => {
+            mockGraphFetch.mockResolvedValue({
+                ok: true,
+                data: {
+                    subject: 'Empty Email',
+                    from: { emailAddress: { name: 'Alice', address: 'alice@example.com' } },
+                    isRead: false,
+                    importance: 'normal',
+                },
+            });
+            const result = await executeMail('test-token', { message_id: 'msg-empty' });
+            expect(result).toContain('# Empty Email');
+            expect(result).toContain('(no body)');
+        });
+        it('returns error when message not found', async () => {
+            mockGraphFetch.mockResolvedValue({
+                ok: false,
+                error: {
+                    status: 404,
+                    message: 'Resource not found. Your account may not have an Exchange Online license.',
+                },
+            });
+            const result = await executeMail('test-token', { message_id: 'nonexistent' });
+            expect(result).toContain('Error:');
+            expect(result).toContain('not found');
+        });
+    });
 });

package/dist/__tests__/tools/server-info.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/__tests__/tools/server-info.test.js

@@ -0,0 +1,60 @@
+import { executeServerInfo } from '../../lib/tools/server-info.js';
+describe('executeServerInfo', () => {
+    it('returns server version and metadata', () => {
+        const result = executeServerInfo();
+        expect(result).toContain('# m365-mcp v');
+        expect(result).toContain(`Node: ${process.version}`);
+        expect(result).toContain(`Platform: ${process.platform} ${process.arch}`);
+    });
+    it('lists all available tools', () => {
+        const result = executeServerInfo();
+        expect(result).toContain('ms_auth_status');
+        expect(result).toContain('ms_profile');
+        expect(result).toContain('ms_calendar');
+        expect(result).toContain('ms_mail');
+        expect(result).toContain('ms_chat');
+        expect(result).toContain('ms_files');
+        expect(result).toContain('ms_transcripts');
+        expect(result).toContain('ms_server_info');
+        expect(result).toContain('Tools (8)');
+    });
+    it('shows environment variable status without exposing values', () => {
+        const result = executeServerInfo();
+        expect(result).toContain('MS365_MCP_CLIENT_ID:');
+        expect(result).toContain('MS365_MCP_TENANT_ID:');
+        // Should show 'set' or 'not set', never the actual value
+        expect(result).not.toMatch(/MS365_MCP_CLIENT_ID: [a-f0-9-]{10,}/);
+    });
+    it('reports env var as set when present', () => {
+        const original = process.env['MS365_MCP_TIMEZONE'];
+        try {
+            process.env['MS365_MCP_TIMEZONE'] = 'Europe/London';
+            const result = executeServerInfo();
+            expect(result).toContain('MS365_MCP_TIMEZONE: Europe/London');
+        }
+        finally {
+            if (original === undefined) {
+                delete process.env['MS365_MCP_TIMEZONE'];
+            }
+            else {
+                process.env['MS365_MCP_TIMEZONE'] = original;
+            }
+        }
+    });
+    it('reports default for MS365_MCP_REDIRECT_URL when not set', () => {
+        const original = process.env['MS365_MCP_REDIRECT_URL'];
+        try {
+            delete process.env['MS365_MCP_REDIRECT_URL'];
+            const result = executeServerInfo();
+            expect(result).toContain('MS365_MCP_REDIRECT_URL: default (dynamic port)');
+        }
+        finally {
+            if (original === undefined) {
+                delete process.env['MS365_MCP_REDIRECT_URL'];
+            }
+            else {
+                process.env['MS365_MCP_REDIRECT_URL'] = original;
+            }
+        }
+    });
+});