memory-journal-mcp 6.1.2 → 6.2.1

package/src/github/github-integration/pull-requests.ts

@@ -1,235 +0,0 @@
-import { logger } from '../../utils/logger.js'
-import type { GitHubClient } from './client.js'
-import type {
-    GitHubPullRequest,
-    GitHubReview,
-    GitHubReviewComment,
-    CopilotReviewSummary,
-} from '../../types/index.js'
-import type { PullRequestDetails } from './types.js'
-
-export class PullRequestsManager {
-    /** Known Copilot bot login patterns */
-    private static readonly COPILOT_BOT_PATTERNS = [
-        'copilot-pull-request-reviewer[bot]',
-        'github-copilot[bot]',
-        'copilot[bot]',
-    ]
-
-    constructor(private client: GitHubClient) {}
-
-    async getPullRequests(
-        owner: string,
-        repo: string,
-        state: 'open' | 'closed' | 'all' = 'open',
-        limit = 20
-    ): Promise<GitHubPullRequest[]> {
-        if (!this.client.octokit) {
-            return []
-        }
-
-        const cacheKey = `prs:${owner}:${repo}:${state}:${String(limit)}`
-        const cached = this.client.getCached(cacheKey) as GitHubPullRequest[] | undefined
-        if (cached) return cached
-
-        try {
-            const response = await this.client.octokit.pulls.list({
-                owner,
-                repo,
-                state,
-                per_page: limit,
-                sort: 'updated',
-                direction: 'desc',
-            })
-
-            const result = response.data.map((pr) => ({
-                number: pr.number,
-                title: pr.title,
-                url: pr.html_url,
-                state: pr.merged_at
-                    ? ('MERGED' as const)
-                    : pr.state === 'open'
-                      ? ('OPEN' as const)
-                      : ('CLOSED' as const),
-            }))
-
-            this.client.setCache(cacheKey, result)
-            return result
-        } catch (error) {
-            logger.error('Failed to get pull requests', {
-                module: 'GitHub',
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return []
-        }
-    }
-
-    async getPullRequest(
-        owner: string,
-        repo: string,
-        prNumber: number
-    ): Promise<PullRequestDetails | null> {
-        if (!this.client.octokit) {
-            return null
-        }
-
-        const cacheKey = `pr:${owner}:${repo}:${String(prNumber)}`
-        const cached = this.client.getCached(cacheKey) as PullRequestDetails | null | undefined
-        if (cached !== undefined) return cached
-
-        try {
-            const response = await this.client.octokit.pulls.get({
-                owner,
-                repo,
-                pull_number: prNumber,
-            })
-
-            const pr = response.data
-
-            const details: PullRequestDetails = {
-                number: pr.number,
-                title: pr.title,
-                url: pr.html_url,
-                state: pr.merged_at ? 'MERGED' : pr.state === 'open' ? 'OPEN' : 'CLOSED',
-                body: pr.body,
-                draft: pr.draft ?? false,
-                headBranch: pr.head.ref,
-                baseBranch: pr.base.ref,
-                author: pr.user?.login ?? 'unknown',
-                createdAt: pr.created_at,
-                updatedAt: pr.updated_at,
-                mergedAt: pr.merged_at,
-                closedAt: pr.closed_at,
-                additions: pr.additions,
-                deletions: pr.deletions,
-                changedFiles: pr.changed_files,
-            }
-
-            this.client.setCache(cacheKey, details)
-            return details
-        } catch (error) {
-            logger.error('Failed to get PR details', {
-                module: 'GitHub',
-                entityId: prNumber,
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return null
-        }
-    }
-
-    private static isCopilotAuthor(login: string): boolean {
-        const lower = login.toLowerCase()
-        return PullRequestsManager.COPILOT_BOT_PATTERNS.some(
-            (p) => lower === p || lower.includes('copilot')
-        )
-    }
-
-    async getReviews(owner: string, repo: string, prNumber: number): Promise<GitHubReview[]> {
-        if (!this.client.octokit) return []
-
-        const cacheKey = `reviews:${owner}:${repo}:${String(prNumber)}`
-        const cached = this.client.getCached(cacheKey) as GitHubReview[] | undefined
-        if (cached) return cached
-
-        try {
-            const response = await this.client.octokit.rest.pulls.listReviews({
-                owner,
-                repo,
-                pull_number: prNumber,
-                per_page: 100,
-            })
-
-            const reviews: GitHubReview[] = response.data.map((r) => ({
-                id: r.id,
-                author: r.user?.login ?? 'unknown',
-                state: r.state as GitHubReview['state'],
-                body: r.body ?? null,
-                submittedAt: r.submitted_at ?? r.commit_id ?? new Date().toISOString(),
-                isCopilot: PullRequestsManager.isCopilotAuthor(r.user?.login ?? ''),
-            }))
-
-            this.client.setCache(cacheKey, reviews)
-            return reviews
-        } catch (error) {
-            logger.error('Failed to get PR reviews', {
-                module: 'GitHub',
-                entityId: prNumber,
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return []
-        }
-    }
-
-    async getReviewComments(
-        owner: string,
-        repo: string,
-        prNumber: number
-    ): Promise<GitHubReviewComment[]> {
-        if (!this.client.octokit) return []
-
-        const cacheKey = `review-comments:${owner}:${repo}:${String(prNumber)}`
-        const cached = this.client.getCached(cacheKey) as GitHubReviewComment[] | undefined
-        if (cached) return cached
-
-        try {
-            const response = await this.client.octokit.rest.pulls.listReviewComments({
-                owner,
-                repo,
-                pull_number: prNumber,
-                per_page: 100,
-            })
-
-            const comments: GitHubReviewComment[] = response.data.map((c) => ({
-                id: c.id,
-                author: c.user?.login ?? 'unknown',
-                body: c.body,
-                path: c.path,
-                line: c.line ?? c.original_line ?? null,
-                side: c.side ?? 'RIGHT',
-                createdAt: c.created_at,
-                isCopilot: PullRequestsManager.isCopilotAuthor(c.user?.login ?? ''),
-            }))
-
-            this.client.setCache(cacheKey, comments)
-            return comments
-        } catch (error) {
-            logger.error('Failed to get review comments', {
-                module: 'GitHub',
-                entityId: prNumber,
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return []
-        }
-    }
-
-    async getCopilotReviewSummary(
-        owner: string,
-        repo: string,
-        prNumber: number
-    ): Promise<CopilotReviewSummary> {
-        const [reviews, comments] = await Promise.all([
-            this.getReviews(owner, repo, prNumber),
-            this.getReviewComments(owner, repo, prNumber),
-        ])
-
-        const copilotReviews = reviews.filter((r) => r.isCopilot)
-        const copilotComments = comments.filter((c) => c.isCopilot)
-
-        let state: CopilotReviewSummary['state'] = 'none'
-        if (copilotReviews.length > 0) {
-            const latest = copilotReviews[copilotReviews.length - 1]
-            if (latest !== undefined) {
-                if (latest.state === 'APPROVED') state = 'approved'
-                else if (latest.state === 'CHANGES_REQUESTED') state = 'changes_requested'
-                else if (latest.state === 'COMMENTED') state = 'commented'
-            }
-        }
-
-        return {
-            prNumber,
-            state,
-            commentCount: copilotComments.length,
-            comments: copilotComments,
-        }
-    }
-}

package/src/github/github-integration/repository.ts

@@ -1,110 +0,0 @@
-import { logger } from '../../utils/logger.js'
-import type { GitHubClient } from './client.js'
-import type { RepoInfo } from './types.js'
-import type { GitHubWorkflowRun } from '../../types/index.js'
-
-export class RepositoryManager {
-    constructor(private client: GitHubClient) {}
-
-    async getRepoInfo(): Promise<RepoInfo> {
-        try {
-            const branchResult = await this.client.git.branch()
-            const branch = branchResult.current || null
-
-            const remotes = await this.client.git.getRemotes(true)
-            const origin = remotes.find((r) => r.name === 'origin')
-            const remoteUrl = origin?.refs?.fetch || null
-
-            const { owner, repo } = this.parseRemoteUrl(remoteUrl)
-
-            const repoInfo = { owner, repo, branch, remoteUrl }
-            this.client.cachedRepoInfo = repoInfo
-            return repoInfo
-        } catch (error) {
-            logger.debug('Failed to get repo info (may not be a git repo)', {
-                module: 'GitHub',
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return { owner: null, repo: null, branch: null, remoteUrl: null }
-        }
-    }
-
-    getCachedRepoInfo(): RepoInfo | null {
-        return this.client.cachedRepoInfo
-    }
-
-    private parseRemoteUrl(remoteUrl: string | null): {
-        owner: string | null
-        repo: string | null
-    } {
-        if (!remoteUrl) return { owner: null, repo: null }
-
-        if (remoteUrl.startsWith('git@github.com:')) {
-            const pathPart = remoteUrl.replace('git@github.com:', '').replace('.git', '')
-            const parts = pathPart.split('/')
-            if (parts.length >= 2) {
-                return { owner: parts[0] ?? null, repo: parts[1] ?? null }
-            }
-        }
-
-        try {
-            const url = new URL(remoteUrl)
-            if (url.hostname === 'github.com') {
-                const path = url.pathname.replace('.git', '').replace(/^\//, '')
-                const parts = path.split('/')
-                if (parts.length >= 2) {
-                    return { owner: parts[0] ?? null, repo: parts[1] ?? null }
-                }
-            }
-        } catch {
-            // Not a valid URL
-        }
-
-        return { owner: null, repo: null }
-    }
-
-    async getWorkflowRuns(owner: string, repo: string, limit = 10): Promise<GitHubWorkflowRun[]> {
-        if (!this.client.octokit) {
-            logger.debug('GitHub API not available - no token', { module: 'GitHub' })
-            return []
-        }
-
-        const cacheKey = `workflows:${owner}:${repo}:${String(limit)}`
-        const cached = this.client.getCached(cacheKey) as GitHubWorkflowRun[] | undefined
-        if (cached) return cached
-
-        try {
-            const response = await this.client.octokit.rest.actions.listWorkflowRunsForRepo({
-                owner,
-                repo,
-                per_page: limit,
-            })
-
-            const result = response.data.workflow_runs.map((run) => ({
-                id: run.id,
-                name: run.name ?? 'Unknown Workflow',
-                status: run.status as 'queued' | 'in_progress' | 'completed',
-                conclusion: run.conclusion as
-                    | 'success'
-                    | 'failure'
-                    | 'cancelled'
-                    | 'skipped'
-                    | null,
-                url: run.html_url,
-                headBranch: run.head_branch ?? '',
-                headSha: run.head_sha,
-                createdAt: run.created_at,
-                updatedAt: run.updated_at,
-            }))
-
-            this.client.setCache(cacheKey, result)
-            return result
-        } catch (error) {
-            logger.error('Failed to get workflow runs', {
-                module: 'GitHub',
-                error: error instanceof Error ? error.message : String(error),
-            })
-            return []
-        }
-    }
-}

package/src/github/github-integration/types.ts

@@ -1,43 +0,0 @@
-import type { GitHubIssue, GitHubPullRequest } from '../../types/index.js'
-
-/**
- * Local repository information
- */
-export interface RepoInfo {
-    owner: string | null
-    repo: string | null
-    branch: string | null
-    remoteUrl: string | null
-}
-
-/**
- * GitHub issue details (extended)
- */
-export interface IssueDetails extends GitHubIssue {
-    nodeId: string
-    body: string | null
-    labels: string[]
-    assignees: string[]
-    createdAt: string
-    updatedAt: string
-    closedAt: string | null
-    commentsCount: number
-}
-
-/**
- * GitHub PR details (extended)
- */
-export interface PullRequestDetails extends GitHubPullRequest {
-    body: string | null
-    draft: boolean
-    headBranch: string
-    baseBranch: string
-    author: string
-    createdAt: string
-    updatedAt: string
-    mergedAt: string | null
-    closedAt: string | null
-    additions: number
-    deletions: number
-    changedFiles: number
-}

package/src/handlers/prompts/github.ts

@@ -1,210 +0,0 @@
-/**
- * Memory Journal MCP Server - GitHub Prompt Definitions
- *
- * Prompts: project-status-summary, pr-summary, code-review-prep,
- * pr-retrospective, actions-failure-digest, project-milestone-tracker
- */
-
-import type { IDatabaseAdapter } from '../../database/core/interfaces.js'
-import { RAW_ENTRY_COLUMNS as ENTRY_COLUMNS } from '../../database/core/entry-columns.js'
-import { ICON_PROMPT } from '../../constants/icons.js'
-import { execQuery, type InternalPromptDef } from './index.js'
-
-/**
- * Get GitHub prompt definitions
- */
-export function getGitHubPromptDefinitions(): InternalPromptDef[] {
-    return [
-        {
-            name: 'project-status-summary',
-            description: 'GitHub Project status reports',
-            icons: [ICON_PROMPT],
-            arguments: [
-                { name: 'project_number', description: 'GitHub Project number', required: true },
-            ],
-            handler: (args: Record<string, string>, db: IDatabaseAdapter) => {
-                const projectNumber = parseInt(args['project_number'] ?? '0', 10)
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE project_number = ?
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp DESC
-                    LIMIT 20
-                `,
-                    [projectNumber]
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Generate a status summary for Project #${String(projectNumber)}:\n\nEntries: ${JSON.stringify(entries, null, 2)}\n\nProvide: overview, recent activity, blockers, next steps.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-        {
-            name: 'pr-summary',
-            description: 'Pull request journal activity summary',
-            icons: [ICON_PROMPT],
-            arguments: [{ name: 'pr_number', description: 'Pull request number', required: true }],
-            handler: (args: Record<string, string>, db: IDatabaseAdapter) => {
-                const prNumber = parseInt(args['pr_number'] ?? '0', 10)
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE pr_number = ?
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp ASC
-                `,
-                    [prNumber]
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Summarize PR #${String(prNumber)} activity:\n\nJournal entries: ${JSON.stringify(entries, null, 2)}\n\nProvide: summary of changes, decisions made, testing done.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-        {
-            name: 'code-review-prep',
-            description: 'Comprehensive PR review preparation',
-            icons: [ICON_PROMPT],
-            arguments: [{ name: 'pr_number', description: 'Pull request number', required: true }],
-            handler: (args: Record<string, string>, db: IDatabaseAdapter) => {
-                const prNumber = parseInt(args['pr_number'] ?? '0', 10)
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE pr_number = ?
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp ASC
-                `,
-                    [prNumber]
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Prepare for code review of PR #${String(prNumber)}:\n\nContext entries: ${JSON.stringify(entries, null, 2)}\n\nProvide: review checklist, areas of concern, testing recommendations.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-        {
-            name: 'pr-retrospective',
-            description: 'Completed PR analysis with learnings',
-            icons: [ICON_PROMPT],
-            arguments: [{ name: 'pr_number', description: 'Pull request number', required: true }],
-            handler: (args: Record<string, string>, db: IDatabaseAdapter) => {
-                const prNumber = parseInt(args['pr_number'] ?? '0', 10)
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE pr_number = ?
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp ASC
-                `,
-                    [prNumber]
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Retrospective for PR #${String(prNumber)}:\n\nJournal entries: ${JSON.stringify(entries, null, 2)}\n\nProvide: what went well, challenges, lessons learned.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-        {
-            name: 'actions-failure-digest',
-            description: 'CI/CD failure analysis with root cause identification',
-            icons: [ICON_PROMPT],
-            arguments: [],
-            handler: (_args: Record<string, string>, db: IDatabaseAdapter) => {
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE workflow_run_id IS NOT NULL
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp DESC
-                    LIMIT 20
-                `
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Analyze CI/CD failures from these workflow entries:\n\n${JSON.stringify(entries, null, 2)}\n\nProvide: failure patterns, root causes, remediation steps.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-        {
-            name: 'project-milestone-tracker',
-            description: 'Milestone progress tracking',
-            icons: [ICON_PROMPT],
-            arguments: [
-                { name: 'project_number', description: 'GitHub Project number', required: true },
-            ],
-            handler: (args: Record<string, string>, db: IDatabaseAdapter) => {
-                const projectNumber = parseInt(args['project_number'] ?? '0', 10)
-                const entries = execQuery(
-                    db,
-                    `
-                    SELECT ${ENTRY_COLUMNS} FROM memory_journal
-                    WHERE project_number = ?
-                    AND significance_type IS NOT NULL
-                    AND deleted_at IS NULL
-                    ORDER BY timestamp DESC
-                `,
-                    [projectNumber]
-                )
-
-                return {
-                    messages: [
-                        {
-                            role: 'user',
-                            content: {
-                                type: 'text',
-                                text: `Track milestones for Project #${String(projectNumber)}:\n\nMilestone entries: ${JSON.stringify(entries, null, 2)}\n\nProvide: progress summary, upcoming milestones, timeline.`,
-                            },
-                        },
-                    ],
-                }
-            },
-        },
-    ]
-}

package/src/handlers/prompts/index.ts

@@ -1,97 +0,0 @@
-/**
- * Memory Journal MCP Server - Prompt Handlers
- *
- * Barrel file composing prompt definitions from sub-modules.
- * Exports all MCP prompts for workflow guidance.
- */
-
-import type { IDatabaseAdapter } from '../../database/core/interfaces.js'
-import type { McpIcon } from '../../types/index.js'
-import { getWorkflowPromptDefinitions } from './workflow.js'
-import { getGitHubPromptDefinitions } from './github.js'
-import { ResourceNotFoundError } from '../../types/errors.js'
-
-/**
- * Message format for MCP prompts
- */
-export interface PromptMessage {
-    role: string
-    content: {
-        type: string
-        text: string
-    }
-}
-
-/**
- * Internal prompt definition with db handler
- */
-export interface InternalPromptDef {
-    name: string
-    description: string
-    arguments?: {
-        name: string
-        description: string
-        required?: boolean
-    }[]
-    icons?: McpIcon[]
-    handler: (args: Record<string, string>, db: IDatabaseAdapter) => { messages: PromptMessage[] }
-}
-
-/**
- * Execute a raw SQL query on the database
- */
-export function execQuery(
-    db: IDatabaseAdapter,
-    sql: string,
-    params: unknown[] = []
-): Record<string, unknown>[] {
-    const result = db.executeRawQuery(sql, params)
-    if (result.length === 0) return []
-
-    const columns = result[0]?.columns ?? []
-    return (result[0]?.values ?? []).map((values: unknown[]) => {
-        const obj: Record<string, unknown> = {}
-        columns.forEach((col: string, i: number) => {
-            obj[col] = values[i]
-        })
-        return obj
-    })
-}
-
-/**
- * Get all prompt definitions for MCP list
- */
-export function getPrompts(): object[] {
-    const prompts = getAllPromptDefinitions()
-    return prompts.map((p) => ({
-        name: p.name,
-        description: p.description,
-        arguments: p.arguments,
-        icons: p.icons,
-    }))
-}
-
-/**
- * Get a prompt by name
- */
-export function getPrompt(
-    name: string,
-    args: Record<string, string>,
-    db: IDatabaseAdapter
-): { messages: PromptMessage[] } {
-    const prompts = getAllPromptDefinitions()
-    const prompt = prompts.find((p) => p.name === name)
-
-    if (!prompt) {
-        throw new ResourceNotFoundError('Prompt', name)
-    }
-
-    return prompt.handler(args, db)
-}
-
-/**
- * Get all prompt definitions by composing sub-module definitions
- */
-function getAllPromptDefinitions(): InternalPromptDef[] {
-    return [...getWorkflowPromptDefinitions(), ...getGitHubPromptDefinitions()]
-}