memory-journal-mcp 6.1.2 → 6.2.1

package/tests/handlers/briefing-user-message.test.ts

@@ -1,405 +0,0 @@
-/**
- * memory-journal-mcp — Briefing User Message Formatter Unit Tests
- *
- * Tests for the formatUserMessage function that builds the
- * markdown summary table displayed to the user.
- */
-
-import { describe, it, expect } from 'vitest'
-import { formatUserMessage } from '../../src/handlers/resources/core/briefing/user-message.js'
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-function baseOpts() {
-    return {
-        repoName: 'neverinfamous/memory-journal-mcp',
-        branchName: 'main',
-        ciStatus: 'passing',
-        totalEntries: 42,
-        latestPreview: '#1: Recent entry preview',
-        github: null,
-    }
-}
-
-// ============================================================================
-// Tests
-// ============================================================================
-
-describe('formatUserMessage', () => {
-    it('should produce basic markdown table without GitHub', () => {
-        const result = formatUserMessage(baseOpts())
-
-        expect(result).toContain('📋 **Session Context Loaded**')
-        expect(result).toContain('memory-journal-mcp')
-        expect(result).toContain('main')
-        expect(result).toContain('passing')
-        expect(result).toContain('42 entries')
-        expect(result).toContain('#1: Recent entry preview')
-    })
-
-    it('should include team DB row when teamTotalEntries is set', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            teamTotalEntries: 15,
-        })
-
-        expect(result).toContain('**Team DB**')
-        expect(result).toContain('15 entries')
-    })
-
-    it('should NOT include team DB row when teamTotalEntries is undefined', () => {
-        const result = formatUserMessage(baseOpts())
-        expect(result).not.toContain('**Team DB**')
-    })
-
-    // ========================================================================
-    // CI with workflow runs
-    // ========================================================================
-
-    it('should display workflow runs with icons when runs are provided', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 3,
-                openPRs: 1,
-                milestones: [],
-                workflowSummary: {
-                    passing: 2,
-                    failing: 1,
-                    pending: 0,
-                    cancelled: 0,
-                    runs: [
-                        { name: 'CI', conclusion: 'success' },
-                        { name: 'CodeQL', conclusion: 'failure' },
-                    ],
-                },
-            },
-        })
-
-        expect(result).toContain('✅ CI')
-        expect(result).toContain('❌ CodeQL')
-    })
-
-    it('should display CI counts when no individual runs', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-                workflowSummary: {
-                    passing: 3,
-                    failing: 1,
-                    pending: 2,
-                    cancelled: 0,
-                },
-            },
-        })
-
-        expect(result).toContain('3 passing')
-        expect(result).toContain('1 failing')
-        expect(result).toContain('2 pending')
-    })
-
-    // ========================================================================
-    // Issues
-    // ========================================================================
-
-    it('should include issue titles when openIssueList is provided', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 2,
-                openPRs: 0,
-                milestones: [],
-                openIssueList: [
-                    { number: 42, title: 'Fix bug' },
-                    { number: 43, title: 'Add feature' },
-                ],
-            },
-        })
-
-        expect(result).toContain('**Issues**')
-        expect(result).toContain('#42 Fix bug')
-        expect(result).toContain('#43 Add feature')
-    })
-
-    it('should show issue count only when no issue list', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 5,
-                openPRs: 0,
-                milestones: [],
-            },
-        })
-
-        expect(result).toContain('**Issues**')
-        expect(result).toContain('5 open')
-    })
-
-    // ========================================================================
-    // PRs
-    // ========================================================================
-
-    it('should include PR status summary when prStatusSummary is provided', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 3,
-                milestones: [],
-                prStatusSummary: { open: 2, merged: 5, closed: 1 },
-            },
-        })
-
-        expect(result).toContain('**PRs**')
-        expect(result).toContain('2 open')
-        expect(result).toContain('5 merged')
-        expect(result).toContain('1 closed')
-    })
-
-    it('should include PR titles when openPrList is provided (no status summary)', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 1,
-                milestones: [],
-                openPrList: [{ number: 10, title: 'Add OAuth', state: 'OPEN' }],
-            },
-        })
-
-        expect(result).toContain('**PRs**')
-        expect(result).toContain('#10 Add OAuth')
-    })
-
-    it('should show PR count only when no list or summary', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 7,
-                milestones: [],
-            },
-        })
-
-        expect(result).toContain('**PRs**')
-        expect(result).toContain('7 open')
-    })
-
-    // ========================================================================
-    // Milestones
-    // ========================================================================
-
-    it('should include milestones with progress and due dates', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [
-                    { title: 'v1.0', progress: '75%', dueOn: '2025-03-15T00:00:00Z' },
-                    { title: 'v2.0', progress: '25%', dueOn: null },
-                ],
-            },
-        })
-
-        expect(result).toContain('**Milestones**')
-        expect(result).toContain('v1.0 (75%, due 2025-03-15)')
-        expect(result).toContain('v2.0 (25%)')
-    })
-
-    it('should omit milestones row when empty', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-            },
-        })
-
-        expect(result).not.toContain('**Milestones**')
-    })
-
-    // ========================================================================
-    // Insights
-    // ========================================================================
-
-    it('should include insights row with stars, forks, clones, and views', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-                insights: {
-                    stars: 120,
-                    forks: 30,
-                    clones14d: 500,
-                    views14d: 2000,
-                },
-            },
-        })
-
-        expect(result).toContain('**Insights**')
-        expect(result).toContain('⭐ 120 stars')
-        expect(result).toContain('🍴 30 forks')
-        expect(result).toContain('📦 500 clones')
-        expect(result).toContain('👁️ 2000 views')
-        expect(result).toContain('(14d)')
-    })
-
-    it('should omit insights row when stars and forks are null', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-                insights: { stars: null, forks: null },
-            },
-        })
-
-        expect(result).not.toContain('**Insights**')
-    })
-
-    // ========================================================================
-    // Copilot Reviews
-    // ========================================================================
-
-    it('should include Copilot row with review stats', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-                copilotReviews: {
-                    reviewed: 5,
-                    approved: 3,
-                    changesRequested: 1,
-                    totalComments: 12,
-                },
-            },
-        })
-
-        expect(result).toContain('**Copilot**')
-        expect(result).toContain('5 reviewed')
-        expect(result).toContain('3 approved')
-        expect(result).toContain('1 changes requested')
-        expect(result).toContain('(12 comments)')
-    })
-
-    it('should omit changes requested when zero', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            github: {
-                repo: 'neverinfamous/memory-journal-mcp',
-                branch: 'main',
-                ci: 'passing',
-                openIssues: 0,
-                openPRs: 0,
-                milestones: [],
-                copilotReviews: {
-                    reviewed: 2,
-                    approved: 2,
-                    changesRequested: 0,
-                    totalComments: 0,
-                },
-            },
-        })
-
-        expect(result).toContain('**Copilot**')
-        expect(result).not.toContain('changes requested')
-        expect(result).not.toContain('comments')
-    })
-
-    // ========================================================================
-    // Rules & Skills
-    // ========================================================================
-
-    it('should include rules file info', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            rulesFile: {
-                path: '/path/to/.rules',
-                name: '.rules',
-                sizeKB: 4,
-                lastModified: '2h ago',
-            },
-        })
-
-        expect(result).toContain('**Rules**')
-        expect(result).toContain('.rules')
-        expect(result).toContain('4 KB')
-        expect(result).toContain('2h ago')
-    })
-
-    it('should include skills directory info', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            skillsDir: {
-                path: '/path/to/skills',
-                count: 3,
-                names: ['cloudflare', 'mcp-builder', 'next'],
-            },
-        })
-
-        expect(result).toContain('**Skills**')
-        expect(result).toContain('3 skills available')
-    })
-
-    it('should use singular "skill" for count of 1', () => {
-        const result = formatUserMessage({
-            ...baseOpts(),
-            skillsDir: {
-                path: '/path/to/skills',
-                count: 1,
-                names: ['cloudflare'],
-            },
-        })
-
-        expect(result).toContain('1 skill available')
-        expect(result).not.toContain('1 skills')
-    })
-})

package/tests/handlers/codemode-tools.test.ts

@@ -1,85 +0,0 @@
-import { describe, it, expect, beforeAll, afterAll, vi } from 'vitest'
-import { DatabaseAdapter } from '../../src/database/sqlite-adapter/index.js'
-import { callTool } from '../../src/handlers/tools/index.js'
-import * as fs from 'fs'
-
-vi.mock('../../src/codemode/sandbox-factory.js', async (importOriginal) => {
-    const actual = await importOriginal<any>()
-    return {
-        ...actual,
-        createSandboxPool: () => ({
-            execute: async (code: string) => {
-                if (code.includes('throw')) {
-                    return { success: false, error: 'test error' }
-                }
-                if (code.includes('mj.core.create_entry')) {
-                    return { success: true, result: 'undefined' }
-                }
-                return { success: true, result: 2 }
-            },
-        }),
-        setDefaultSandboxMode: vi.fn(),
-    }
-})
-
-describe('Code Mode Tool Handlers', () => {
-    let personalDb: DatabaseAdapter
-
-    const personalDbPath = './test-codemode-handlers.db'
-
-    beforeAll(async () => {
-        try {
-            if (fs.existsSync(personalDbPath)) fs.unlinkSync(personalDbPath)
-        } catch {}
-        personalDb = new DatabaseAdapter(personalDbPath)
-        await personalDb.initialize()
-    })
-
-    afterAll(() => {
-        personalDb.close()
-        try {
-            if (fs.existsSync(personalDbPath)) fs.unlinkSync(personalDbPath)
-        } catch {}
-    })
-
-    it('should execute basic code', async () => {
-        const result = (await callTool(
-            'mj_execute_code',
-            { code: 'return 1 + 1;' },
-            personalDb
-        )) as any
-        expect(result.success).toBe(true)
-        expect(result.result).toBe(2)
-    })
-
-    it('should block malicious code', async () => {
-        const result = (await callTool(
-            'mj_execute_code',
-            { code: 'process.exit(1)' },
-            personalDb
-        )) as any
-        expect(result.success).toBe(false)
-        expect(result.error).toContain('Security validation failed')
-    })
-
-    it('should respect readonly mode', async () => {
-        const result = (await callTool(
-            'mj_execute_code',
-            { code: 'return typeof mj.core.create_entry', readonly: true },
-            personalDb
-        )) as any
-        expect(result.success).toBe(true)
-        // create_entry does not have readOnlyHint: true, so it is filtered out
-        expect(result.result).toBe('undefined')
-    })
-
-    it('should capture runtime errors', async () => {
-        const result = (await callTool(
-            'mj_execute_code',
-            { code: 'throw new Error("test error")' },
-            personalDb
-        )) as any
-        expect(result.success).toBe(false)
-        expect(result.error).toContain('test error')
-    })
-})

package/tests/handlers/copilot-tools.test.ts

@@ -1,126 +0,0 @@
-/**
- * memory-journal-mcp — Copilot Tools Unit Tests
- *
- * Tests for getCopilotReviewTools handler.
- */
-
-import { describe, it, expect, vi, beforeEach } from 'vitest'
-
-// ============================================================================
-// Hoisted mocks
-// ============================================================================
-
-const { mockResolveOwnerRepo } = vi.hoisted(() => ({
-    mockResolveOwnerRepo: vi.fn(),
-}))
-
-vi.mock('../../src/utils/logger.js', () => ({
-    logger: { info: vi.fn(), warning: vi.fn(), error: vi.fn(), debug: vi.fn() },
-}))
-
-vi.mock('../../src/utils/error-helpers.js', () => ({
-    formatHandlerError: vi.fn().mockImplementation((err: Error) => ({
-        success: false,
-        error: err.message,
-    })),
-}))
-
-vi.mock('../../src/handlers/tools/github/helpers.js', () => ({
-    resolveOwnerRepo: mockResolveOwnerRepo,
-}))
-
-import { getCopilotReviewTools } from '../../src/handlers/tools/github/copilot-tools.js'
-
-// ============================================================================
-// Helpers
-// ============================================================================
-
-function createMockContext() {
-    return {
-        db: {},
-        progress: null,
-        github: {
-            getCopilotReviewSummary: vi.fn().mockResolvedValue({
-                prNumber: 42,
-                state: 'approved',
-                commentCount: 3,
-                comments: [
-                    { body: 'Use const here', path: 'src/index.ts', line: 10, isCopilot: true },
-                    { body: 'Missing null check', path: 'src/utils.ts', line: 25, isCopilot: true },
-                    { body: 'Good pattern', path: 'src/core.ts', line: 50, isCopilot: true },
-                ],
-            }),
-        },
-    }
-}
-
-// ============================================================================
-// Tests
-// ============================================================================
-
-describe('getCopilotReviewTools', () => {
-    beforeEach(() => {
-        vi.clearAllMocks()
-    })
-
-    it('should define get_copilot_reviews tool', () => {
-        const context = createMockContext()
-        const tools = getCopilotReviewTools(context as never)
-
-        expect(tools).toHaveLength(1)
-        expect(tools[0]!.name).toBe('get_copilot_reviews')
-        expect(tools[0]!.group).toBe('github')
-    })
-
-    it('should return Copilot review summary on success', async () => {
-        const context = createMockContext()
-        mockResolveOwnerRepo.mockResolvedValue({
-            owner: 'neverinfamous',
-            repo: 'memory-journal-mcp',
-            detectedOwner: 'neverinfamous',
-            detectedRepo: 'memory-journal-mcp',
-            github: context.github,
-        })
-
-        const tools = getCopilotReviewTools(context as never)
-        const handler = tools[0]!.handler
-
-        const result = (await handler({ pr_number: 42 })) as Record<string, unknown>
-
-        expect(result['prNumber']).toBe(42)
-        expect(result['state']).toBe('approved')
-        expect(result['commentCount']).toBe(3)
-        expect(result['comments']).toHaveLength(3)
-        expect((result['comments'] as Record<string, unknown>[])[0]!['body']).toBe('Use const here')
-        expect(result['owner']).toBe('neverinfamous')
-    })
-
-    it('should return error response when GitHub not available', async () => {
-        const context = createMockContext()
-        mockResolveOwnerRepo.mockResolvedValue({
-            error: true,
-            response: { success: false, error: 'GitHub integration not available' },
-        })
-
-        const tools = getCopilotReviewTools(context as never)
-        const handler = tools[0]!.handler
-
-        const result = (await handler({ pr_number: 42 })) as Record<string, unknown>
-
-        expect(result['success']).toBe(false)
-        expect(result['error']).toContain('GitHub')
-    })
-
-    it('should handle handler errors via formatHandlerError', async () => {
-        const context = createMockContext()
-        mockResolveOwnerRepo.mockRejectedValue(new Error('Network failure'))
-
-        const tools = getCopilotReviewTools(context as never)
-        const handler = tools[0]!.handler
-
-        const result = (await handler({ pr_number: 42 })) as Record<string, unknown>
-
-        expect(result['success']).toBe(false)
-        expect(result['error']).toContain('Network failure')
-    })
-})