memory-journal-mcp 6.1.2 → 6.2.1

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

@@ -1,176 +0,0 @@
-/**
- * Payload Contract Tests: GitHub Tools (E2E environment)
- *
- * Validates that all 16 GitHub tools return well-formed responses
- * when called in the E2E test environment. Tools will either:
- * - Return structured JSON errors (when no token or resolution fails)
- * - Return success responses with data (if git remote auto-detected + token)
- * - Return raw MCP error strings (if the tool throws before handler's catch)
- *
- * In all cases the response must be a non-empty, recognizable shape.
- */
-
-import { test, expect } from '@playwright/test'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { createClient } from './helpers.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('Payload Contracts: GitHub Tools (E2E environment)', () => {
-    let client: Client
-
-    test.beforeAll(async () => {
-        client = await createClient()
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    function expectValidGitHubResponse(payload: Record<string, unknown>): void {
-        expect(typeof payload).toBe('object')
-        expect(payload).not.toBeNull()
-        expect(Object.keys(payload).length).toBeGreaterThan(0)
-        // If it's an error, it must have at minimum an error message string
-        if (payload.success === false) {
-            expect(typeof payload.error).toBe('string')
-        }
-    }
-
-    /**
-     * Some GitHub tools throw before the handler's catch block when the GitHub
-     * API is unavailable, producing a raw non-JSON MCP error string as content.
-     * Return a synthetic error object so the test doesn't crash on JSON.parse.
-     */
-    function parseResponse(text: string): Record<string, unknown> {
-        try {
-            return JSON.parse(text) as Record<string, unknown>
-        } catch {
-            return { success: false, error: text }
-        }
-    }
-
-    function getText(response: Awaited<ReturnType<typeof client.callTool>>): string {
-        const content = response.content as Array<{ type: string; text: string }>
-        return content[0]!.text
-    }
-
-    // --- Read tools ---
-    test('get_github_issues → valid response', async () => {
-        const response = await client.callTool({ name: 'get_github_issues', arguments: {} })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('get_github_prs → valid response', async () => {
-        const response = await client.callTool({ name: 'get_github_prs', arguments: {} })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('get_github_issue → valid response', async () => {
-        const response = await client.callTool({
-            name: 'get_github_issue',
-            arguments: { issue_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('get_github_pr → valid response', async () => {
-        const response = await client.callTool({
-            name: 'get_github_pr',
-            arguments: { pr_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('get_github_context → valid response', async () => {
-        const response = await client.callTool({ name: 'get_github_context', arguments: {} })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    // --- Kanban tools ---
-    test('get_kanban_board → valid response', async () => {
-        const response = await client.callTool({
-            name: 'get_kanban_board',
-            arguments: { project_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('move_kanban_item → valid response', async () => {
-        const response = await client.callTool({
-            name: 'move_kanban_item',
-            arguments: { project_number: 1, item_id: 'PVTI_test', target_status: 'Done' },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    // --- Issue lifecycle ---
-    test('create_github_issue_with_entry → valid response', async () => {
-        const response = await client.callTool({
-            name: 'create_github_issue_with_entry',
-            arguments: { title: 'E2E test issue', body: 'test' },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('close_github_issue_with_entry → valid response', async () => {
-        const response = await client.callTool({
-            name: 'close_github_issue_with_entry',
-            arguments: { issue_number: 99999, resolution_notes: 'test' },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    // --- Milestones ---
-    test('get_github_milestones → valid response', async () => {
-        const response = await client.callTool({ name: 'get_github_milestones', arguments: {} })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('get_github_milestone → valid response', async () => {
-        const response = await client.callTool({
-            name: 'get_github_milestone',
-            arguments: { milestone_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('create_github_milestone → valid response', async () => {
-        const response = await client.callTool({
-            name: 'create_github_milestone',
-            arguments: { title: 'E2E test milestone' },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('update_github_milestone → valid response', async () => {
-        const response = await client.callTool({
-            name: 'update_github_milestone',
-            arguments: { milestone_number: 1, title: 'Updated milestone' },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    test('delete_github_milestone → valid response', async () => {
-        const response = await client.callTool({
-            name: 'delete_github_milestone',
-            arguments: { milestone_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    // --- Insights ---
-    test('get_repo_insights → valid response', async () => {
-        const response = await client.callTool({ name: 'get_repo_insights', arguments: {} })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-
-    // --- Copilot ---
-    test('get_copilot_reviews → valid response', async () => {
-        const response = await client.callTool({
-            name: 'get_copilot_reviews',
-            arguments: { pr_number: 1 },
-        })
-        expectValidGitHubResponse(parseResponse(getText(response)))
-    })
-})

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

@@ -1,56 +0,0 @@
-/**
- * Payload Contract Tests: Relationships
- *
- * Validates response shapes for 2 relationship tools:
- * link_entries, visualize_relationships.
- */
-
-import { test, expect } from '@playwright/test'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { createClient, callToolAndParse, expectSuccess } from './helpers.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('Payload Contracts: Relationships', () => {
-    let client: Client
-    let entryId1: number
-    let entryId2: number
-
-    test.beforeAll(async () => {
-        client = await createClient()
-        // Create two entries for relationship tests
-        const e1 = await callToolAndParse(client, 'create_entry', {
-            content: 'Relationship source entry',
-            entry_type: 'test_entry',
-        })
-        const e2 = await callToolAndParse(client, 'create_entry', {
-            content: 'Relationship target entry',
-            entry_type: 'project_decision',
-        })
-        entryId1 = (e1.entry as Record<string, unknown>).id as number
-        entryId2 = (e2.entry as Record<string, unknown>).id as number
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    test('link_entries returns { success, relationship }', async () => {
-        const payload = await callToolAndParse(client, 'link_entries', {
-            from_entry_id: entryId1,
-            to_entry_id: entryId2,
-            relationship_type: 'references',
-        })
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-        expect(payload.relationship).toBeDefined()
-    })
-
-    test('visualize_relationships returns Mermaid diagram', async () => {
-        const payload = await callToolAndParse(client, 'visualize_relationships', {
-            entry_id: entryId1,
-        })
-        expectSuccess(payload)
-        expect(typeof payload).toBe('object')
-    })
-})

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

@@ -1,64 +0,0 @@
-/**
- * Payload Contract Tests: Search
- *
- * Validates response shapes for 4 search tools:
- * search_entries, search_by_date_range, semantic_search, get_vector_index_stats.
- */
-
-import { test, expect } from '@playwright/test'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { createClient, callToolAndParse, expectSuccess } from './helpers.js'
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('Payload Contracts: Search', () => {
-    let client: Client
-
-    test.beforeAll(async () => {
-        client = await createClient()
-        // Seed an entry for search tests
-        await callToolAndParse(client, 'create_entry', {
-            content: 'Searchable payload contract test entry about performance',
-            entry_type: 'test_entry',
-            tags: ['search-test'],
-        })
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-    })
-
-    test('search_entries returns { entries, count }', async () => {
-        const payload = await callToolAndParse(client, 'search_entries', {
-            query: 'performance',
-        })
-        expectSuccess(payload)
-        expect(Array.isArray(payload.entries)).toBe(true)
-        expect(typeof payload.count).toBe('number')
-    })
-
-    test('search_by_date_range returns { entries, count }', async () => {
-        const payload = await callToolAndParse(client, 'search_by_date_range', {
-            start_date: '2020-01-01',
-            end_date: '2030-12-31',
-        })
-        expectSuccess(payload)
-        expect(Array.isArray(payload.entries)).toBe(true)
-        expect(typeof payload.count).toBe('number')
-    })
-
-    test('semantic_search returns results', async () => {
-        const payload = await callToolAndParse(client, 'semantic_search', {
-            query: 'performance optimization',
-            limit: 3,
-        })
-        // May return { entries, count } or { success: false } if index not ready
-        expect(typeof payload).toBe('object')
-    })
-
-    test('get_vector_index_stats returns stats', async () => {
-        const payload = await callToolAndParse(client, 'get_vector_index_stats', {})
-        expectSuccess(payload)
-        expect(payload.available).toBeDefined()
-    })
-})

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

@@ -1,231 +0,0 @@
-/**
- * Payload Contract Tests: Team Tools (Happy Path)
- *
- * Uses a dedicated server on port 3109 with TEAM_DB_PATH configured
- * to validate full team tool CRUD lifecycle:
- * create → get → search → update → link → visualize → export → backup → delete.
- */
-
-import { test, expect } from '@playwright/test'
-import type { Client } from '@modelcontextprotocol/sdk/client/index.js'
-import { Client as McpClient } from '@modelcontextprotocol/sdk/client/index.js'
-import { StreamableHTTPClientTransport } from '@modelcontextprotocol/sdk/client/streamableHttp.js'
-import { callToolAndParse, expectSuccess } from './helpers.js'
-import { mkdirSync } from 'node:fs'
-import { join } from 'node:path'
-import { spawn, type ChildProcess } from 'node:child_process'
-import { setTimeout as delay } from 'node:timers/promises'
-
-const TEAM_PORT = 3109
-const TEAM_BASE = `http://localhost:${TEAM_PORT}`
-
-test.describe.configure({ mode: 'serial' })
-
-test.describe('Payload Contracts: Team Tools (Happy Path)', () => {
-    let client: Client
-    let serverProcess: ChildProcess
-    let entryId1: number
-    let entryId2: number
-
-    test.beforeAll(async () => {
-        // Ensure test output directory exists
-        const testDir = join(process.cwd(), '.test-output', 'e2e')
-        mkdirSync(testDir, { recursive: true })
-
-        // Start server with TEAM_DB_PATH configured
-        serverProcess = spawn(
-            'node',
-            [
-                'dist/cli.js',
-                '--transport',
-                'http',
-                '--port',
-                String(TEAM_PORT),
-                '--db',
-                './.test-output/e2e/test-e2e-team-happy.db',
-            ],
-            {
-                cwd: process.cwd(),
-                stdio: 'pipe',
-                env: {
-                    ...process.env,
-                    MCP_RATE_LIMIT_MAX: '10000',
-                    TEAM_DB_PATH: './.test-output/e2e/test-e2e-team-happy-team.db',
-                },
-            }
-        )
-
-        // Wait for server readiness
-        for (let i = 0; i < 30; i++) {
-            try {
-                const res = await fetch(`${TEAM_BASE}/health`)
-                if (res.ok) break
-            } catch {
-                // Not ready
-            }
-            await delay(500)
-        }
-
-        const transport = new StreamableHTTPClientTransport(new URL(`${TEAM_BASE}/mcp`))
-        client = new McpClient({ name: 'team-happy-test', version: '1.0.0' }, { capabilities: {} })
-        await client.connect(transport)
-    })
-
-    test.afterAll(async () => {
-        await client.close()
-        serverProcess.kill('SIGTERM')
-    })
-
-    // --- Core ---
-    test('team_create_entry returns { success, entry }', async () => {
-        const payload = await callToolAndParse(client, 'team_create_entry', {
-            content: 'Team happy path test entry 1',
-            author: 'e2e-test',
-            entry_type: 'test_entry',
-            tags: ['team-test', 'happy-path'],
-        })
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-        expect(payload.entry).toBeDefined()
-        entryId1 = (payload.entry as Record<string, unknown>).id as number
-    })
-
-    test('team_create_entry (second entry)', async () => {
-        const payload = await callToolAndParse(client, 'team_create_entry', {
-            content: 'Team happy path test entry 2',
-            author: 'e2e-test',
-            entry_type: 'project_decision',
-            tags: ['team-test', 'merge-source'],
-        })
-        expectSuccess(payload)
-        entryId2 = (payload.entry as Record<string, unknown>).id as number
-    })
-
-    test('team_get_entry_by_id returns entry', async () => {
-        const payload = await callToolAndParse(client, 'team_get_entry_by_id', {
-            entry_id: entryId1,
-        })
-        expectSuccess(payload)
-        expect(payload.entry).toBeDefined()
-    })
-
-    test('team_get_recent returns entries', async () => {
-        const payload = await callToolAndParse(client, 'team_get_recent', {
-            limit: 5,
-        })
-        expectSuccess(payload)
-        expect(Array.isArray(payload.entries)).toBe(true)
-        expect(typeof payload.count).toBe('number')
-    })
-
-    test('team_list_tags returns tags', async () => {
-        const payload = await callToolAndParse(client, 'team_list_tags', {})
-        expectSuccess(payload)
-        expect(Array.isArray(payload.tags)).toBe(true)
-        expect(typeof payload.count).toBe('number')
-    })
-
-    // --- Search ---
-    test('team_search returns matches', async () => {
-        const payload = await callToolAndParse(client, 'team_search', {
-            query: 'happy path',
-        })
-        expectSuccess(payload)
-        expect(Array.isArray(payload.entries)).toBe(true)
-        expect(typeof payload.count).toBe('number')
-    })
-
-    test('team_search_by_date_range returns entries', async () => {
-        const payload = await callToolAndParse(client, 'team_search_by_date_range', {
-            start_date: '2020-01-01',
-            end_date: '2030-12-31',
-        })
-        expectSuccess(payload)
-        expect(Array.isArray(payload.entries)).toBe(true)
-    })
-
-    // --- Admin ---
-    test('team_update_entry succeeds', async () => {
-        const payload = await callToolAndParse(client, 'team_update_entry', {
-            entry_id: entryId1,
-            content: 'Updated team entry content',
-        })
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-    })
-
-    test('team_merge_tags succeeds', async () => {
-        const payload = await callToolAndParse(client, 'team_merge_tags', {
-            source_tag: 'merge-source',
-            target_tag: 'team-test',
-        })
-        expectSuccess(payload)
-    })
-
-    // --- Analytics ---
-    test('team_get_statistics returns stats', async () => {
-        const payload = await callToolAndParse(client, 'team_get_statistics', {})
-        expectSuccess(payload)
-        expect(typeof payload).toBe('object')
-    })
-
-    // --- Relationships ---
-    test('team_link_entries creates relationship', async () => {
-        const payload = await callToolAndParse(client, 'team_link_entries', {
-            from_entry_id: entryId1,
-            to_entry_id: entryId2,
-            relationship_type: 'references',
-        })
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-    })
-
-    test('team_visualize_relationships returns diagram', async () => {
-        const payload = await callToolAndParse(client, 'team_visualize_relationships', {
-            entry_id: entryId1,
-        })
-        expectSuccess(payload)
-    })
-
-    // --- Export ---
-    test('team_export_entries (json) returns entries', async () => {
-        const payload = await callToolAndParse(client, 'team_export_entries', {
-            format: 'json',
-        })
-        expectSuccess(payload)
-        expect(payload.format).toBe('json')
-    })
-
-    test('team_export_entries (markdown) returns data string', async () => {
-        const payload = await callToolAndParse(client, 'team_export_entries', {
-            format: 'markdown',
-        })
-        expectSuccess(payload)
-        expect(payload.format).toBe('markdown')
-        // Handler returns `data` key (string), not `content`
-        expect(typeof payload.data).toBe('string')
-        expect((payload.data as string).length).toBeGreaterThan(0)
-    })
-
-    // --- Backup ---
-    test('team_backup creates backup', async () => {
-        const payload = await callToolAndParse(client, 'team_backup', {})
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-    })
-
-    test('team_list_backups returns list', async () => {
-        const payload = await callToolAndParse(client, 'team_list_backups', {})
-        expectSuccess(payload)
-        expect(typeof payload).toBe('object')
-    })
-
-    // --- Delete (last to avoid breaking other tests) ---
-    test('team_delete_entry soft-deletes', async () => {
-        const payload = await callToolAndParse(client, 'team_delete_entry', {
-            entry_id: entryId2,
-        })
-        expectSuccess(payload)
-        expect(payload.success).toBe(true)
-    })
-})