memory-journal-mcp 4.3.0 → 4.4.0

package/tests/handlers/github-resource-handlers.test.ts

@@ -0,0 +1,473 @@
+/**
+ * GitHub Resource Handler Tests
+ *
+ * Tests GitHub-dependent resources using a mock GitHubIntegration object.
+ */
+
+import { describe, it, expect, beforeAll, afterAll, vi } from 'vitest'
+import { readResource } from '../../src/handlers/resources/index.js'
+import { SqliteAdapter } from '../../src/database/SqliteAdapter.js'
+import type { GitHubIntegration } from '../../src/github/GitHubIntegration.js'
+
+/**
+ * Creates a minimal mock GitHubIntegration with sensible defaults.
+ */
+function createMockGitHub(overrides: Partial<Record<string, unknown>> = {}): GitHubIntegration {
+    const mock = {
+        isApiAvailable: vi.fn().mockReturnValue(true),
+        getRepoInfo: vi.fn().mockResolvedValue({
+            owner: 'testowner',
+            repo: 'testrepo',
+            branch: 'main',
+            remoteUrl: 'git@github.com:testowner/testrepo.git',
+        }),
+        getRepoContext: vi.fn().mockResolvedValue({
+            repoName: 'testrepo',
+            branch: 'main',
+            commit: 'abc1234',
+            remoteUrl: 'url',
+            projects: [],
+            issues: [],
+            pullRequests: [],
+            workflowRuns: [],
+            milestones: [],
+        }),
+        getIssues: vi
+            .fn()
+            .mockResolvedValue([
+                { number: 1, title: 'Bug fix needed', url: 'url1', state: 'OPEN', milestone: null },
+            ]),
+        getPullRequests: vi
+            .fn()
+            .mockResolvedValue([{ number: 10, title: 'Feature PR', url: 'url10', state: 'OPEN' }]),
+        getWorkflowRuns: vi.fn().mockResolvedValue([
+            {
+                id: 100,
+                name: 'CI',
+                status: 'completed',
+                conclusion: 'success',
+                url: 'url',
+                headBranch: 'main',
+                headSha: 'abc1234',
+                createdAt: '2025-01-01T00:00:00Z',
+                updatedAt: '2025-01-01T01:00:00Z',
+            },
+        ]),
+        getProjectKanban: vi.fn().mockResolvedValue(null),
+        getMilestones: vi.fn().mockResolvedValue([
+            {
+                number: 1,
+                title: 'v1.0',
+                description: 'First release',
+                state: 'open',
+                url: 'url',
+                dueOn: '2025-06-01T00:00:00Z',
+                openIssues: 5,
+                closedIssues: 10,
+                createdAt: '2025-01-01T00:00:00Z',
+                updatedAt: '2025-01-02T00:00:00Z',
+                creator: 'owner1',
+            },
+        ]),
+        getMilestone: vi.fn().mockResolvedValue({
+            number: 1,
+            title: 'v1.0',
+            description: 'First release',
+            state: 'open',
+            url: 'url',
+            dueOn: '2025-06-01T00:00:00Z',
+            openIssues: 5,
+            closedIssues: 10,
+            createdAt: '2025-01-01T00:00:00Z',
+            updatedAt: '2025-01-02T00:00:00Z',
+            creator: 'owner1',
+        }),
+        getCachedRepoInfo: vi.fn().mockReturnValue({
+            owner: 'testowner',
+            repo: 'testrepo',
+            branch: 'main',
+            remoteUrl: 'git@github.com:testowner/testrepo.git',
+        }),
+        getRepoStats: vi.fn().mockResolvedValue({
+            stars: 42,
+            forks: 7,
+            watchers: 3,
+        }),
+        getTrafficData: vi.fn().mockResolvedValue({
+            clones: { total: 120, uniqueCloners: 30 },
+            views: { total: 500, uniqueVisitors: 80 },
+        }),
+        ...overrides,
+    }
+    return mock as unknown as GitHubIntegration
+}
+
+describe('GitHub Resource Handlers', () => {
+    let db: SqliteAdapter
+    const testDbPath = './test-gh-resources.db'
+
+    beforeAll(async () => {
+        db = new SqliteAdapter(testDbPath)
+        await db.initialize()
+    })
+
+    afterAll(() => {
+        db.close()
+        try {
+            const fs = require('node:fs')
+            if (fs.existsSync(testDbPath)) fs.unlinkSync(testDbPath)
+        } catch {
+            // Ignore cleanup errors
+        }
+    })
+
+    // ========================================================================
+    // memory://github/status
+    // ========================================================================
+
+    describe('memory://github/status', () => {
+        it('should return status with repo info, CI, issues, PRs', async () => {
+            const github = createMockGitHub()
+            const result = await readResource(
+                'memory://github/status',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as {
+                repository: string
+                branch: string
+                ci: { status: string }
+                issues: { openCount: number }
+                pullRequests: { openCount: number }
+            }
+
+            expect(data.repository).toBe('testowner/testrepo')
+            expect(data.branch).toBe('main')
+            expect(data.ci.status).toBe('passing')
+            expect(data.issues.openCount).toBe(1)
+            expect(data.pullRequests.openCount).toBe(1)
+        })
+
+        it('should return error when no github integration', async () => {
+            const result = await readResource(
+                'memory://github/status',
+                db,
+                undefined,
+                undefined,
+                null
+            )
+
+            const data = result.data as { error: string; hint: string }
+            expect(data.error).toContain('GitHub integration not available')
+        })
+
+        it('should handle missing owner/repo', async () => {
+            const github = createMockGitHub({
+                getRepoInfo: vi.fn().mockResolvedValue({
+                    owner: null,
+                    repo: null,
+                    branch: 'main',
+                    remoteUrl: null,
+                }),
+            })
+
+            const result = await readResource(
+                'memory://github/status',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as { error: string }
+            expect(data.error).toContain('Could not detect repository')
+        })
+    })
+
+    // ========================================================================
+    // memory://github/milestones
+    // ========================================================================
+
+    describe('memory://github/milestones', () => {
+        it('should return milestones with completion percentages', async () => {
+            const github = createMockGitHub()
+            const result = await readResource(
+                'memory://github/milestones',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as {
+                repository: string
+                milestones: { completionPercentage: number }[]
+                count: number
+            }
+
+            expect(data.repository).toBe('testowner/testrepo')
+            expect(data.count).toBe(1)
+            // 10 closed / (5 open + 10 closed) = 66.67% => 67%
+            expect(data.milestones[0]!.completionPercentage).toBe(67)
+        })
+
+        it('should return error when no github', async () => {
+            const result = await readResource(
+                'memory://github/milestones',
+                db,
+                undefined,
+                undefined,
+                null
+            )
+
+            const data = result.data as { error: string }
+            expect(data.error).toContain('GitHub integration not available')
+        })
+    })
+
+    // ========================================================================
+    // memory://milestones/{number}
+    // ========================================================================
+
+    describe('memory://milestones/{number}', () => {
+        it('should return single milestone detail', async () => {
+            const github = createMockGitHub()
+            const result = await readResource(
+                'memory://milestones/1',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as {
+                repository: string
+                milestone: { number: number; completionPercentage: number }
+            }
+
+            expect(data.repository).toBe('testowner/testrepo')
+            expect(data.milestone.number).toBe(1)
+            expect(data.milestone.completionPercentage).toBe(67)
+        })
+
+        it('should return error for not found milestone', async () => {
+            const github = createMockGitHub({
+                getMilestone: vi.fn().mockResolvedValue(null),
+            })
+            const result = await readResource(
+                'memory://milestones/999',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as { error: string }
+            expect(data.error).toContain('not found')
+        })
+    })
+
+    // ========================================================================
+    // memory://kanban/{project_number}
+    // ========================================================================
+
+    describe('memory://kanban/{project_number}', () => {
+        it('should return kanban board when available', async () => {
+            const github = createMockGitHub({
+                getProjectKanban: vi.fn().mockResolvedValue({
+                    projectId: 'PVT_1',
+                    projectTitle: 'My Board',
+                    columns: [
+                        {
+                            status: 'Todo',
+                            items: [{ id: 'I1', type: 'ISSUE', title: 'Task', number: 1 }],
+                        },
+                    ],
+                    statusOptions: [],
+                    totalItems: 1,
+                }),
+            })
+
+            const result = await readResource('memory://kanban/1', db, undefined, undefined, github)
+
+            const data = result.data as { projectTitle: string; totalItems: number }
+            expect(data.projectTitle).toBe('My Board')
+            expect(data.totalItems).toBe(1)
+        })
+
+        it('should return error when project not found', async () => {
+            const github = createMockGitHub()
+            const result = await readResource(
+                'memory://kanban/999',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as { error: string }
+            expect(data.error).toContain('not found')
+        })
+
+        it('should return error when no github', async () => {
+            const result = await readResource('memory://kanban/1', db, undefined, undefined, null)
+
+            const data = result.data as { error: string }
+            expect(data.error).toContain('GitHub integration not available')
+        })
+    })
+
+    // ========================================================================
+    // memory://kanban/{project_number}/diagram
+    // ========================================================================
+
+    describe('memory://kanban/{project_number}/diagram', () => {
+        it('should return mermaid diagram', async () => {
+            const github = createMockGitHub({
+                getProjectKanban: vi.fn().mockResolvedValue({
+                    projectId: 'PVT_1',
+                    projectTitle: 'Board',
+                    columns: [
+                        {
+                            status: 'Done',
+                            items: [
+                                {
+                                    id: 'PVTITEM_A1B2C3D4',
+                                    type: 'ISSUE',
+                                    title: 'Completed task',
+                                    number: 5,
+                                },
+                            ],
+                        },
+                    ],
+                    statusOptions: [],
+                    totalItems: 1,
+                }),
+            })
+
+            const result = await readResource(
+                'memory://kanban/1/diagram',
+                db,
+                undefined,
+                undefined,
+                github
+            )
+
+            const data = result.data as {
+                format: string
+                diagram: string
+                projectNumber: number
+                totalItems: number
+            }
+
+            expect(data.format).toBe('mermaid')
+            expect(data.diagram).toContain('graph LR')
+            expect(data.diagram).toContain('Done')
+            expect(data.totalItems).toBe(1)
+        })
+
+        it('should show fallback when no github', async () => {
+            const result = await readResource(
+                'memory://kanban/1/diagram',
+                db,
+                undefined,
+                undefined,
+                null
+            )
+
+            const data = result.data as { format: string; diagram: string }
+            expect(data.format).toBe('mermaid')
+            expect(data.diagram).toContain('NoGitHub')
+        })
+    })
+
+    // ========================================================================
+    // memory://briefing with GitHub insights
+    // ========================================================================
+
+    describe('memory://briefing with GitHub', () => {
+        it('should include insights with stars, forks, and traffic', async () => {
+            const github = createMockGitHub()
+            const result = await readResource('memory://briefing', db, undefined, undefined, github)
+
+            const data = result.data as {
+                github: {
+                    repo: string
+                    insights?: {
+                        stars: number | null
+                        forks: number | null
+                        clones14d?: number
+                        views14d?: number
+                    }
+                }
+                userMessage: string
+            }
+
+            expect(data.github.repo).toBe('testowner/testrepo')
+            expect(data.github.insights).toBeDefined()
+            expect(data.github.insights!.stars).toBe(42)
+            expect(data.github.insights!.forks).toBe(7)
+            expect(data.github.insights!.clones14d).toBe(120)
+            expect(data.github.insights!.views14d).toBe(500)
+            expect(data.userMessage).toContain('stars')
+            expect(data.userMessage).toContain('forks')
+            expect(data.userMessage).toContain('clones')
+        })
+
+        it('should include insights without traffic when getTrafficData fails', async () => {
+            const github = createMockGitHub({
+                getTrafficData: vi.fn().mockRejectedValue(new Error('403 Forbidden')),
+            })
+            const result = await readResource('memory://briefing', db, undefined, undefined, github)
+
+            const data = result.data as {
+                github: {
+                    insights?: {
+                        stars: number | null
+                        forks: number | null
+                        clones14d?: number
+                        views14d?: number
+                    }
+                }
+                userMessage: string
+            }
+
+            // Stars and forks should still be present
+            expect(data.github.insights).toBeDefined()
+            expect(data.github.insights!.stars).toBe(42)
+            expect(data.github.insights!.forks).toBe(7)
+            // Traffic should be absent
+            expect(data.github.insights!.clones14d).toBeUndefined()
+            expect(data.github.insights!.views14d).toBeUndefined()
+        })
+
+        it('should omit insights when getRepoStats fails', async () => {
+            const github = createMockGitHub({
+                getRepoStats: vi.fn().mockRejectedValue(new Error('API error')),
+            })
+            const result = await readResource('memory://briefing', db, undefined, undefined, github)
+
+            const data = result.data as {
+                github: {
+                    insights?: unknown
+                }
+            }
+
+            expect(data.github.insights).toBeUndefined()
+        })
+
+        it('should include repoInsights in more section', async () => {
+            const github = createMockGitHub()
+            const result = await readResource('memory://briefing', db, undefined, undefined, github)
+
+            const data = result.data as {
+                more: { repoInsights: string }
+            }
+
+            expect(data.more.repoInsights).toBe('memory://github/insights')
+        })
+    })
+})