memory-journal-mcp 6.1.2 → 6.2.1

package/tests/e2e/payloads-team.spec.ts

@@ -1,174 +0,0 @@
-/**
- * Payload Contract Tests: Team Tools (Error Path)
- *
- * Validates that all 20 team tools return well-formed structured errors
- * when TEAM_DB_PATH is not configured. Every response must include
- * { success: false, code: 'CONFIGURATION_ERROR' } — never raw MCP exceptions.
- */
-
-import { test, expect } from '@playwright/test'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { createClient, callToolAndParse } from './helpers.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('Payload Contracts: Team Tools (no TEAM_DB_PATH)', () => {
-    let client: Client
-
-    test.beforeAll(async () => {
-        client = await createClient()
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    function expectConfigError(payload: Record<string, unknown>): void {
-        expect(payload.success).toBe(false)
-        expect(payload.code).toBe('CONFIGURATION_ERROR')
-        expect(typeof payload.error).toBe('string')
-        expect(payload.category).toBe('configuration')
-        expect(typeof payload.suggestion).toBe('string')
-        expect(typeof payload.recoverable).toBe('boolean')
-    }
-
-    // --- Core ---
-    test('team_create_entry → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_create_entry', {
-            content: 'test',
-            author: 'e2e',
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_get_entry_by_id → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_get_entry_by_id', {
-            entry_id: 1,
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_get_recent → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_get_recent', {})
-        expectConfigError(payload)
-    })
-
-    test('team_list_tags → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_list_tags', {})
-        expectConfigError(payload)
-    })
-
-    // --- Search ---
-    test('team_search → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_search', {
-            query: 'test',
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_search_by_date_range → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_search_by_date_range', {
-            start_date: '2020-01-01',
-            end_date: '2030-12-31',
-        })
-        expectConfigError(payload)
-    })
-
-    // --- Admin ---
-    test('team_update_entry → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_update_entry', {
-            entry_id: 1,
-            content: 'updated',
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_delete_entry → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_delete_entry', {
-            entry_id: 1,
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_merge_tags → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_merge_tags', {
-            source_tag: 'old',
-            target_tag: 'new',
-        })
-        expectConfigError(payload)
-    })
-
-    // --- Analytics ---
-    test('team_get_statistics → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_get_statistics', {})
-        expectConfigError(payload)
-    })
-
-    test('team_get_cross_project_insights → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_get_cross_project_insights', {})
-        expectConfigError(payload)
-    })
-
-    // --- Relationships ---
-    test('team_link_entries → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_link_entries', {
-            from_entry_id: 1,
-            to_entry_id: 2,
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_visualize_relationships → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_visualize_relationships', {
-            entry_id: 1,
-        })
-        expectConfigError(payload)
-    })
-
-    // --- Export ---
-    test('team_export_entries → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_export_entries', {
-            format: 'json',
-        })
-        expectConfigError(payload)
-    })
-
-    // --- Backup ---
-    test('team_backup → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_backup', {})
-        expectConfigError(payload)
-    })
-
-    test('team_list_backups → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_list_backups', {})
-        expectConfigError(payload)
-    })
-
-    // --- Vector ---
-    test('team_semantic_search → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_semantic_search', {
-            query: 'test',
-        })
-        expectConfigError(payload)
-    })
-
-    test('team_get_vector_index_stats → { available: false } when unconfigured', async () => {
-        // This tool uses a different response shape when teamDb is null:
-        // { available: false, error: string } instead of TEAM_DB_ERROR_RESPONSE
-        const payload = await callToolAndParse(client, 'team_get_vector_index_stats', {})
-        expect(payload.available).toBe(false)
-        expect(typeof payload.error).toBe('string')
-    })
-
-    test('team_rebuild_vector_index → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_rebuild_vector_index', {})
-        expectConfigError(payload)
-    })
-
-    test('team_add_to_vector_index → CONFIGURATION_ERROR', async () => {
-        const payload = await callToolAndParse(client, 'team_add_to_vector_index', {
-            entry_id: 1,
-        })
-        expectConfigError(payload)
-    })
-})

package/tests/e2e/prompts-expanded.spec.ts

@@ -1,137 +0,0 @@
-/**
- * E2E Tests: Expanded Prompt Coverage
- *
- * Tests 13 prompts not individually exercised by prompts.spec.ts:
- * session-summary, weekly-digest, prepare-retro, analyze-period,
- * goal-tracker, get-context-bundle, get-recent-entries, confirm-briefing,
- * and 6 GitHub prompts.
- */
-
-import { test, expect } from '@playwright/test'
-import { createClient } from './helpers.js'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('E2E Prompts: Expanded Coverage', () => {
-    let client: Client
-
-    test.beforeAll(async () => {
-        client = await createClient()
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    function expectValidPrompt(response: {
-        messages: Array<{ role: string; content: unknown }>
-    }): void {
-        expect(response.messages).toBeDefined()
-        expect(Array.isArray(response.messages)).toBe(true)
-        expect(response.messages.length).toBeGreaterThan(0)
-
-        const first = response.messages[0]!
-        expect(first).toHaveProperty('role')
-        expect(first).toHaveProperty('content')
-    }
-
-    // --- Workflow prompts ---
-    test('session-summary prompt', async () => {
-        const response = await client.getPrompt({ name: 'session-summary', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    test('weekly-digest prompt', async () => {
-        const response = await client.getPrompt({ name: 'weekly-digest', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    test('prepare-retro prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'prepare-retro',
-            arguments: {},
-        })
-        expectValidPrompt(response)
-    })
-
-    test('analyze-period prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'analyze-period',
-            arguments: {
-                start_date: new Date(Date.now() - 7 * 86_400_000).toISOString().split('T')[0]!,
-                end_date: new Date().toISOString().split('T')[0]!,
-            },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('goal-tracker prompt', async () => {
-        const response = await client.getPrompt({ name: 'goal-tracker', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    test('get-context-bundle prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'get-context-bundle',
-            arguments: { query: 'architecture' },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('get-recent-entries prompt', async () => {
-        const response = await client.getPrompt({ name: 'get-recent-entries', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    test('confirm-briefing prompt', async () => {
-        const response = await client.getPrompt({ name: 'confirm-briefing', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    // --- GitHub prompts ---
-    test('project-status-summary prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'project-status-summary',
-            arguments: { project_number: '1' },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('pr-summary prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'pr-summary',
-            arguments: { pr_number: '1' },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('code-review-prep prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'code-review-prep',
-            arguments: { pr_number: '1' },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('pr-retrospective prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'pr-retrospective',
-            arguments: { pr_number: '1' },
-        })
-        expectValidPrompt(response)
-    })
-
-    test('actions-failure-digest prompt', async () => {
-        const response = await client.getPrompt({ name: 'actions-failure-digest', arguments: {} })
-        expectValidPrompt(response)
-    })
-
-    test('project-milestone-tracker prompt', async () => {
-        const response = await client.getPrompt({
-            name: 'project-milestone-tracker',
-            arguments: { project_number: '1' },
-        })
-        expectValidPrompt(response)
-    })
-})

package/tests/e2e/prompts.spec.ts

@@ -1,62 +0,0 @@
-/**
- * E2E Tests: MCP Prompts via SDK Client
- *
- * Validates prompt listing and retrieval via the Streamable HTTP transport.
- */
-
-import { test, expect } from '@playwright/test'
-import { createClient } from './helpers.js'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('E2E Prompts (via MCP SDK Client)', () => {
-    let client: Client
-
-    test.beforeAll(async () => {
-        client = await createClient()
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    test('should list all available prompts', async () => {
-        const response = await client.listPrompts()
-
-        expect(response.prompts).toBeDefined()
-        expect(Array.isArray(response.prompts)).toBe(true)
-        expect(response.prompts.length).toBeGreaterThanOrEqual(16)
-
-        const promptNames = response.prompts.map((p) => p.name)
-        expect(promptNames).toContain('prepare-standup')
-        expect(promptNames).toContain('session-summary')
-        expect(promptNames).toContain('weekly-digest')
-        expect(promptNames).toContain('find-related')
-        expect(promptNames).toContain('confirm-briefing')
-    })
-
-    test('should get a specific prompt (prepare-standup)', async () => {
-        const response = await client.getPrompt({ name: 'prepare-standup' })
-
-        expect(response.messages).toBeDefined()
-        expect(Array.isArray(response.messages)).toBe(true)
-        expect(response.messages.length).toBeGreaterThan(0)
-
-        // Each message should have role and content
-        const firstMessage = response.messages[0]!
-        expect(firstMessage).toHaveProperty('role')
-        expect(firstMessage).toHaveProperty('content')
-    })
-
-    test('should get a parameterized prompt (find-related)', async () => {
-        const response = await client.getPrompt({
-            name: 'find-related',
-            arguments: { query: 'performance optimization' },
-        })
-
-        expect(response.messages).toBeDefined()
-        expect(Array.isArray(response.messages)).toBe(true)
-        expect(response.messages.length).toBeGreaterThan(0)
-    })
-})

package/tests/e2e/protocols.spec.ts

@@ -1,134 +0,0 @@
-/**
- * E2E Tests: HTTP Transport Protocols
- *
- * Tests both Streamable HTTP (MCP 2025-03-26) and Legacy SSE (MCP 2024-11-05)
- * protocol error handling and routing.
- */
-
-import { test, expect } from '@playwright/test'
-
-test.describe('HTTP Transport Protocols', () => {
-    test.describe('Streamable HTTP (MCP 2025-03-26)', () => {
-        test('should reject non-init request without session ID', async ({ request }) => {
-            const response = await request.post('/mcp', {
-                headers: {
-                    Accept: 'application/json, text/event-stream',
-                },
-                data: {
-                    jsonrpc: '2.0',
-                    id: 1,
-                    method: 'ping',
-                },
-            })
-
-            expect(response.status()).toBe(400)
-            const body = await response.json()
-            expect(body.error).toHaveProperty(
-                'message',
-                'Bad Request: No valid session ID provided'
-            )
-        })
-
-        test('should reject malformed JSON body on /mcp', async ({ request }) => {
-            const response = await request.post('/mcp', {
-                headers: {
-                    Accept: 'application/json, text/event-stream',
-                    'Content-Type': 'application/json',
-                },
-                data: Buffer.from('{"broken": json}'),
-            })
-
-            // Express JSON parser rejects invalid JSON before reaching our handler
-            expect(response.status()).toBe(400)
-        })
-
-        test('should accept initialization and return session ID', async ({ request }) => {
-            const response = await request.post('/mcp', {
-                headers: {
-                    Accept: 'application/json, text/event-stream',
-                },
-                data: {
-                    jsonrpc: '2.0',
-                    id: 1,
-                    method: 'initialize',
-                    params: {
-                        protocolVersion: '2025-03-26',
-                        capabilities: {},
-                        clientInfo: {
-                            name: 'playwright-protocol-test',
-                            version: '1.0.0',
-                        },
-                    },
-                },
-            })
-
-            expect(response.status()).toBe(200)
-            const sessionId = response.headers()['mcp-session-id']
-            expect(sessionId).toBeDefined()
-        })
-    })
-
-    test.describe('Legacy SSE (MCP 2024-11-05)', () => {
-        test('should reject /messages POST without sessionId parameter', async ({ request }) => {
-            const response = await request.post('/messages', {
-                data: {
-                    jsonrpc: '2.0',
-                    id: 1,
-                    method: 'initialize',
-                    params: {
-                        protocolVersion: '2024-11-05',
-                        capabilities: {},
-                        clientInfo: { name: 'test', version: '1.0' },
-                    },
-                },
-            })
-
-            expect(response.status()).toBe(400)
-            const body = await response.json()
-            expect(body.error).toHaveProperty('message', 'Missing sessionId parameter')
-        })
-
-        test('should reject /messages POST with unknown sessionId', async ({ request }) => {
-            const response = await request.post('/messages?sessionId=invalid-session-uuid', {
-                data: {
-                    jsonrpc: '2.0',
-                    id: 1,
-                    method: 'ping',
-                },
-            })
-
-            expect(response.status()).toBe(404)
-            const body = await response.json()
-            expect(body.error).toHaveProperty('message', 'Session not found')
-        })
-
-        test('should complete full SDK client round-trip via Legacy SSE', async () => {
-            // Regression test: server.connect() auto-calls start() on SSEServerTransport,
-            // so a redundant start() call would throw "already started!" and break SSE entirely.
-            const { Client } = await import('@modelcontextprotocol/sdk/client/index.js')
-            const { SSEClientTransport } = await import('@modelcontextprotocol/sdk/client/sse.js')
-
-            const transport = new SSEClientTransport(new URL('http://localhost:3100/sse'))
-            const client = new Client(
-                { name: 'playwright-sse-regression', version: '1.0.0' },
-                { capabilities: {} }
-            )
-
-            try {
-                await client.connect(transport)
-
-                const response = await client.callTool({
-                    name: 'test_simple',
-                    arguments: { message: 'SSE round-trip' },
-                })
-
-                expect(response.isError).toBeUndefined()
-                expect(Array.isArray(response.content)).toBe(true)
-                const text = (response.content[0] as { type: string; text: string }).text
-                expect(text).toContain('SSE round-trip')
-            } finally {
-                await client.close()
-            }
-        })
-    })
-})