memory-journal-mcp 6.1.2 → 6.2.1

package/src/handlers/tools/export.ts

@@ -1,150 +0,0 @@
-/**
- * Export Tool Group - 1 tool
- *
- * Tools: export_entries
- */
-
-import { z } from 'zod'
-import type { ToolDefinition, ToolContext } from '../../types/index.js'
-import { formatHandlerError } from '../../utils/error-helpers.js'
-import { sendProgress } from '../../utils/progress-utils.js'
-import { ErrorFieldsMixin } from './error-fields-mixin.js'
-import {
-    ENTRY_TYPES,
-    DATE_FORMAT_REGEX,
-    DATE_FORMAT_MESSAGE,
-    DATE_MIN_SENTINEL,
-    DATE_MAX_SENTINEL,
-    EntryOutputSchema,
-    relaxedNumber,
-} from './schemas.js'
-
-// ============================================================================
-// Input Schemas
-// ============================================================================
-
-/** Strict schema — used inside handler for structured Zod errors */
-const ExportEntriesSchema = z.object({
-    format: z.enum(['json', 'markdown']).optional().default('json'),
-    start_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE).optional(),
-    end_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE).optional(),
-    entry_types: z.array(z.enum(ENTRY_TYPES)).optional(),
-    tags: z.array(z.string()).optional(),
-    limit: z
-        .number()
-        .max(500)
-        .optional()
-        .default(100)
-        .describe('Maximum entries to export (default: 100)'),
-})
-
-/** Relaxed schema — passed to SDK inputSchema so Zod errors reach the handler */
-const ExportEntriesSchemaMcp = z.object({
-    format: z.string().optional().default('json'),
-    start_date: z.string().optional(),
-    end_date: z.string().optional(),
-    entry_types: z.array(z.string()).optional(),
-    tags: z.array(z.string()).optional(),
-    limit: relaxedNumber()
-        .optional()
-        .default(100)
-        .describe('Maximum entries to export (default: 100)'),
-})
-
-// ============================================================================
-// Output Schemas
-// ============================================================================
-
-const ExportEntriesOutputSchema = z
-    .object({
-        format: z.enum(['json', 'markdown']).optional(),
-        entries: z.array(EntryOutputSchema).optional(),
-        count: z.number().optional(),
-        content: z.string().optional(),
-        success: z.boolean().optional(),
-        error: z.string().optional(),
-    })
-    .extend(ErrorFieldsMixin.shape)
-
-// ============================================================================
-// Tool Definitions
-// ============================================================================
-
-export function getExportTools(context: ToolContext): ToolDefinition[] {
-    const { db, progress } = context
-    return [
-        {
-            name: 'export_entries',
-            title: 'Export Entries',
-            description: 'Export journal entries to JSON or Markdown format',
-            group: 'export',
-            inputSchema: ExportEntriesSchemaMcp,
-            outputSchema: ExportEntriesOutputSchema,
-            annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: false },
-            handler: async (params: unknown) => {
-                try {
-                    const input = ExportEntriesSchema.parse(params)
-                    const limit = input.limit ?? 100
-
-                    await sendProgress(progress, 0, 2, 'Fetching entries...')
-
-                    // When entry_types filter is active, fetch a larger batch so
-                    // post-filtering doesn't silently return empty results.
-                    const hasTypeFilter = input.entry_types && input.entry_types.length > 0
-                    const fetchLimit = hasTypeFilter ? 500 : limit
-
-                    // Apply filters — use searchByDateRange when dates/tags/types present
-                    let entries
-                    if (input.start_date || input.end_date) {
-                        const startDate = input.start_date ?? DATE_MIN_SENTINEL
-                        const endDate = input.end_date ?? DATE_MAX_SENTINEL
-                        entries = db.searchByDateRange(startDate, endDate, {
-                            tags: input.tags,
-                            limit: fetchLimit,
-                        })
-                    } else if ((input.tags && input.tags.length > 0) || hasTypeFilter) {
-                        // Tags/types filter: use a wide date range to scan the full database
-                        entries = db.searchByDateRange(DATE_MIN_SENTINEL, DATE_MAX_SENTINEL, {
-                            tags: input.tags,
-                            limit: fetchLimit,
-                        })
-                    } else {
-                        entries = db.getRecentEntries(fetchLimit)
-                    }
-
-                    // Post-filter by entry_types, then cap to requested limit
-                    if (hasTypeFilter) {
-                        const allowedTypes = new Set(input.entry_types)
-                        entries = entries
-                            .filter((e) => allowedTypes.has(e.entryType))
-                            .slice(0, limit)
-                    }
-
-                    await sendProgress(
-                        progress,
-                        1,
-                        2,
-                        `Processing ${String(entries.length)} entries...`
-                    )
-
-                    if (input.format === 'markdown') {
-                        const md = entries
-                            .map(
-                                (e) =>
-                                    `## ${e.timestamp}\n\n**Type:** ${e.entryType}\n\n${e.content}\n\n---`
-                            )
-                            .join('\n\n')
-
-                        await sendProgress(progress, 2, 2, 'Export complete')
-                        return { format: 'markdown', content: md }
-                    }
-
-                    await sendProgress(progress, 2, 2, 'Export complete')
-                    return { format: 'json', entries, count: entries.length }
-                } catch (err) {
-                    return formatHandlerError(err)
-                }
-            },
-        },
-    ]
-}

package/src/handlers/tools/github/copilot-tools.ts

@@ -1,72 +0,0 @@
-/**
- * Copilot Review Tools - 1 tool
- *
- * Tools: get_copilot_reviews
- */
-
-import { z } from 'zod'
-import type { ToolDefinition, ToolContext } from '../../../types/index.js'
-import { formatHandlerError } from '../../../utils/error-helpers.js'
-import { CopilotReviewsOutputSchema } from './schemas.js'
-import { relaxedNumber } from '../schemas.js'
-import { resolveOwnerRepo } from './helpers.js'
-
-// ============================================================================
-// Tool Definitions
-// ============================================================================
-
-export function getCopilotReviewTools(context: ToolContext): ToolDefinition[] {
-    return [
-        {
-            name: 'get_copilot_reviews',
-            title: 'Get Copilot Code Reviews',
-            description:
-                'Get Copilot\'s code review findings for a pull request. Returns review state (approved/changes_requested/commented/none) and file-level comments with paths and line numbers. Use to learn from Copilot\'s review patterns and create journal entries with tag "copilot-finding".',
-            group: 'github',
-            inputSchema: z.object({
-                pr_number: relaxedNumber().describe('Pull request number'),
-                owner: z.string().optional().describe('LEAVE EMPTY to auto-detect from git'),
-                repo: z.string().optional().describe('LEAVE EMPTY to auto-detect from git'),
-            }),
-            outputSchema: CopilotReviewsOutputSchema,
-            annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
-            handler: async (params: unknown) => {
-                try {
-                    const input = z
-                        .object({
-                            pr_number: z.number(),
-                            owner: z.string().optional(),
-                            repo: z.string().optional(),
-                        })
-                        .parse(params)
-
-                    const resolved = await resolveOwnerRepo(context, input)
-                    if ('error' in resolved) return resolved.response
-
-                    const summary = await resolved.github.getCopilotReviewSummary(
-                        resolved.owner,
-                        resolved.repo,
-                        input.pr_number
-                    )
-
-                    return {
-                        prNumber: summary.prNumber,
-                        state: summary.state,
-                        commentCount: summary.commentCount,
-                        comments: summary.comments.map((c) => ({
-                            body: c.body,
-                            path: c.path,
-                            line: c.line,
-                        })),
-                        owner: resolved.owner,
-                        repo: resolved.repo,
-                        detectedOwner: resolved.detectedOwner,
-                        detectedRepo: resolved.detectedRepo,
-                    }
-                } catch (err) {
-                    return formatHandlerError(err)
-                }
-            },
-        },
-    ]
-}

package/src/handlers/tools/github/helpers.ts

@@ -1,125 +0,0 @@
-/**
- * GitHub Tool Helpers - Shared owner/repo resolution
- */
-
-import type { ToolContext } from '../../../types/index.js'
-import type { GitHubIntegration } from '../../../github/github-integration/index.js'
-
-/**
- * Resolve owner (owner-only, no repo required)
- */
-export async function resolveOwner(
-    context: ToolContext,
-    inputOwner?: string,
-    entityLabel?: string
-): Promise<
-    | {
-          owner: string
-          detectedOwner: string | null
-          repo: string | undefined
-          github: GitHubIntegration
-      }
-    | { error: true; response: Record<string, unknown> }
-> {
-    if (!context.github) {
-        return {
-            error: true,
-            response: {
-                success: false,
-                error: 'GitHub integration not available',
-                code: 'CONFIGURATION_ERROR',
-                category: 'configuration',
-                suggestion:
-                    'Set GITHUB_TOKEN and GITHUB_REPO_PATH environment variables to enable GitHub integration.',
-                recoverable: true,
-            },
-        }
-    }
-
-    const repoInfo = await context.github.getRepoInfo()
-    const detectedOwner = repoInfo.owner
-    const owner = inputOwner ?? detectedOwner ?? undefined
-    const repo = repoInfo.repo ?? undefined
-
-    if (!owner) {
-        return {
-            error: true,
-            response: {
-                success: false,
-                error: 'STOP: Could not auto-detect repository owner. DO NOT GUESS. You MUST ask the user to provide the GitHub owner.',
-                code: 'CONFIGURATION_ERROR',
-                category: 'configuration',
-                recoverable: true,
-                requiresUserInput: true,
-                detectedOwner,
-                instruction: entityLabel
-                    ? `Ask the user: "${entityLabel}"`
-                    : 'Ask the user: "What GitHub username or organization owns this project?"',
-            },
-        }
-    }
-
-    return { owner, detectedOwner, repo, github: context.github }
-}
-
-/**
- * Resolve owner + repo (both required)
- */
-export async function resolveOwnerRepo(
-    context: ToolContext,
-    input: { owner?: string; repo?: string },
-    entityLabel?: string
-): Promise<
-    | {
-          owner: string
-          repo: string
-          detectedOwner: string | null
-          detectedRepo: string | null
-          github: GitHubIntegration
-      }
-    | { error: true; response: Record<string, unknown> }
-> {
-    if (!context.github) {
-        return {
-            error: true,
-            response: {
-                success: false,
-                error: 'GitHub integration not available',
-                code: 'CONFIGURATION_ERROR',
-                category: 'configuration',
-                suggestion:
-                    'Set GITHUB_TOKEN and GITHUB_REPO_PATH environment variables to enable GitHub integration.',
-                recoverable: true,
-            },
-        }
-    }
-
-    const repoInfo = await context.github.getRepoInfo()
-    const detectedOwner = repoInfo.owner
-    const detectedRepo = repoInfo.repo
-    const owner = input.owner ?? detectedOwner ?? undefined
-    const repo = input.repo ?? detectedRepo ?? undefined
-
-    if (!owner || !repo) {
-        return {
-            error: true,
-            response: {
-                success: false,
-                error: 'STOP: Could not auto-detect repository. DO NOT GUESS. You MUST ask the user to provide the GitHub owner and repository name.',
-                code: 'CONFIGURATION_ERROR',
-                category: 'configuration',
-                recoverable: true,
-                requiresUserInput: true,
-                detectedOwner,
-                detectedRepo,
-                ...(entityLabel
-                    ? {
-                          instruction: `Ask the user: "What GitHub repository ${entityLabel}? Please provide the owner and repo name (e.g., owner/repo)."`,
-                      }
-                    : {}),
-            },
-        }
-    }
-
-    return { owner, repo, detectedOwner, detectedRepo, github: context.github }
-}

package/src/handlers/tools/github/insights-tools.ts

@@ -1,112 +0,0 @@
-/**
- * GitHub Insights Tools - 1 tool
- *
- * Tools: get_repo_insights
- */
-
-import { z } from 'zod'
-import type { ToolDefinition, ToolContext } from '../../../types/index.js'
-import { formatHandlerError } from '../../../utils/error-helpers.js'
-import { RepoInsightsOutputSchema } from './schemas.js'
-import { resolveOwnerRepo } from './helpers.js'
-
-// ============================================================================
-// Tool Definitions
-// ============================================================================
-
-export function getGitHubInsightsTools(context: ToolContext): ToolDefinition[] {
-    return [
-        {
-            name: 'get_repo_insights',
-            title: 'Repository Insights',
-            description:
-                'Get repository insights: stars, forks, traffic (clones/views), referrers, and popular paths. Use "sections" to control token usage: stars (~50 tokens), traffic (~100), referrers (~100), paths (~100), or all (~350).',
-            group: 'github',
-            inputSchema: z.object({
-                sections: z
-                    .enum(['stars', 'traffic', 'referrers', 'paths', 'all'])
-                    .optional()
-                    .default('stars')
-                    .describe(
-                        'Data section to return (default: stars). Use "all" for full payload.'
-                    ),
-                owner: z
-                    .string()
-                    .optional()
-                    .describe('Repository owner - LEAVE EMPTY to auto-detect'),
-                repo: z
-                    .string()
-                    .optional()
-                    .describe('Repository name - LEAVE EMPTY to auto-detect'),
-            }),
-            outputSchema: RepoInsightsOutputSchema,
-            annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
-            handler: async (params: unknown) => {
-                try {
-                    const input = z
-                        .object({
-                            sections: z
-                                .enum(['stars', 'traffic', 'referrers', 'paths', 'all'])
-                                .optional()
-                                .default('stars'),
-                            owner: z.string().optional(),
-                            repo: z.string().optional(),
-                        })
-                        .parse(params)
-
-                    const resolved = await resolveOwnerRepo(
-                        context,
-                        input,
-                        'should I get insights for'
-                    )
-                    if ('error' in resolved) return resolved.response
-
-                    const owner = resolved.owner
-                    const repo = resolved.repo
-
-                    const section = input.sections
-                    const result: Partial<z.infer<typeof RepoInsightsOutputSchema>> = {
-                        owner,
-                        repo,
-                        section,
-                    }
-
-                    if (section === 'stars' || section === 'all') {
-                        const stats = await resolved.github.getRepoStats(owner, repo)
-                        if (stats) {
-                            result.stars = stats.stars
-                            result.forks = stats.forks
-                            result.watchers = stats.watchers
-                            result.openIssues = stats.openIssues
-                            if (section === 'all') {
-                                result.size = stats.size
-                                result.defaultBranch = stats.defaultBranch
-                            }
-                        }
-                    }
-
-                    if (section === 'traffic' || section === 'all') {
-                        const traffic = await resolved.github.getTrafficData(owner, repo)
-                        if (traffic) {
-                            result.traffic = traffic
-                        }
-                    }
-
-                    if (section === 'referrers' || section === 'all') {
-                        const referrers = await resolved.github.getTopReferrers(owner, repo, 5)
-                        result.referrers = referrers
-                    }
-
-                    if (section === 'paths' || section === 'all') {
-                        const paths = await resolved.github.getPopularPaths(owner, repo, 5)
-                        result.paths = paths
-                    }
-
-                    return result
-                } catch (err) {
-                    return formatHandlerError(err)
-                }
-            },
-        },
-    ]
-}