@revealui/mcp 0.1.0 → 0.1.3

package/migrations/005_performance_indexes.sql

@@ -1,190 +0,0 @@
--- Performance Indexes Migration
--- Adds strategic indexes to improve query performance
-
--- ============================================================================
--- USER INDEXES
--- ============================================================================
-
--- Email lookup (used in authentication)
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_users_email
-ON users(email);
-
--- Created at for sorting recent users
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_users_created_at
-ON users(created_at DESC);
-
--- Email verification status
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_users_email_verified
-ON users(email_verified_at)
-WHERE email_verified_at IS NOT NULL;
-
--- ============================================================================
--- POST INDEXES
--- ============================================================================
-
--- Slug lookup (used in public URLs)
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_slug
-ON posts(slug);
-
--- Published posts sorted by date
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_published_at
-ON posts(published_at DESC)
-WHERE published_at IS NOT NULL;
-
--- Author foreign key
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_author_id
-ON posts(author_id);
-
--- Status filtering
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_status
-ON posts(status);
-
--- Composite index for author's posts by status
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_author_status
-ON posts(author_id, status);
-
--- Composite index for published posts list
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_published_status
-ON posts(published_at DESC, status)
-WHERE status = 'published';
-
--- Full text search on title and content
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_posts_search
-ON posts USING gin(to_tsvector('english', coalesce(title, '') || ' ' || coalesce(content, '')));
-
--- ============================================================================
--- SESSION INDEXES
--- ============================================================================
-
--- Token lookup (used in authentication)
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_sessions_token
-ON sessions(token);
-
--- User sessions lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_sessions_user_id
-ON sessions(user_id);
-
--- Expiration cleanup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_sessions_expires_at
-ON sessions(expires_at);
-
--- Active sessions by user
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_sessions_user_active
-ON sessions(user_id, expires_at)
-WHERE expires_at > NOW();
-
--- ============================================================================
--- COMMENT INDEXES (if comments table exists)
--- ============================================================================
-
--- Post comments lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_comments_post_id
-ON comments(post_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'comments');
-
--- Author comments lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_comments_author_id
-ON comments(author_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'comments');
-
--- Approved comments
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_comments_approved
-ON comments(approved_at, created_at DESC)
-WHERE approved_at IS NOT NULL
-AND EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'comments');
-
--- ============================================================================
--- CATEGORY/TAG INDEXES (if exists)
--- ============================================================================
-
--- Category slug lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_categories_slug
-ON categories(slug)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'categories');
-
--- Tag slug lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_tags_slug
-ON tags(slug)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'tags');
-
--- Post-Tag junction table
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_post_tags_post_id
-ON post_tags(post_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'post_tags');
-
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_post_tags_tag_id
-ON post_tags(tag_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'post_tags');
-
--- ============================================================================
--- MEDIA/ASSETS INDEXES (if exists)
--- ============================================================================
-
--- Media type lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_media_type
-ON media(type)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'media');
-
--- Media owner lookup
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_media_user_id
-ON media(user_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'media');
-
--- ============================================================================
--- ANALYTICS/METRICS INDEXES (if exists)
--- ============================================================================
-
--- Page views by date range
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_analytics_created_at
-ON analytics(created_at DESC)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'analytics');
-
--- Page views by resource
-CREATE INDEX CONCURRENTLY IF NOT EXISTS idx_analytics_resource
-ON analytics(resource_type, resource_id)
-WHERE EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = 'analytics');
-
--- ============================================================================
--- VERIFICATION
--- ============================================================================
-
--- View all indexes
--- SELECT
---   schemaname,
---   tablename,
---   indexname,
---   indexdef
--- FROM pg_indexes
--- WHERE schemaname = 'public'
--- ORDER BY tablename, indexname;
-
--- Check index usage
--- SELECT
---   schemaname,
---   tablename,
---   indexname,
---   idx_scan as index_scans,
---   idx_tup_read as tuples_read,
---   idx_tup_fetch as tuples_fetched
--- FROM pg_stat_user_indexes
--- ORDER BY idx_scan DESC;
-
--- Find unused indexes
--- SELECT
---   schemaname,
---   tablename,
---   indexname,
---   idx_scan
--- FROM pg_stat_user_indexes
--- WHERE idx_scan = 0
--- AND indexname NOT LIKE 'pg_toast%'
--- ORDER BY tablename, indexname;
-
--- Check index sizes
--- SELECT
---   schemaname,
---   tablename,
---   indexname,
---   pg_size_pretty(pg_relation_size(indexrelid)) as index_size
--- FROM pg_stat_user_indexes
--- ORDER BY pg_relation_size(indexrelid) DESC;

package/migrations/backfill_crdt_meta.js

@@ -1,45 +0,0 @@
-#!/usr/bin/env node
-/*
-  Backfill script to populate `_electric_meta` columns where NULL.
-  Usage: ELECTRIC_DATABASE_URL=postgres://... node backfill_crdt_meta.js
-*/
-
-const { Client } = require('pg')
-const { logger } = require('@revealui/core/observability/logger')
-
-async function main() {
-  const url = process.env.ELECTRIC_DATABASE_URL || process.env.DATABASE_URL
-  if (!url) {
-    logger.error(
-      'Set ELECTRIC_DATABASE_URL or DATABASE_URL before running.',
-      new Error('Missing database URL'),
-    )
-    process.exit(1)
-  }
-
-  const client = new Client({ connectionString: url })
-  await client.connect()
-
-  try {
-    logger.info('Backfilling documents._electric_meta where null...')
-    await client.query(
-      "UPDATE documents SET _electric_meta = '{}'::jsonb WHERE _electric_meta IS NULL",
-    )
-
-    logger.info('Backfilling subscription_state._electric_meta where null...')
-    await client.query(
-      "UPDATE subscription_state SET _electric_meta = '{}'::jsonb WHERE _electric_meta IS NULL",
-    )
-
-    logger.info('Backfill complete.')
-  } catch (err) {
-    logger.error('Backfill failed', err instanceof Error ? err : new Error(String(err)))
-    process.exitCode = 1
-  } finally {
-    await client.end()
-  }
-}
-
-if (require.main === module) {
-  main()
-}

package/src/__tests__/hypervisor.test.ts

@@ -1,212 +0,0 @@
-import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
-import { MCPHypervisor, type MCPServerConfig } from '../hypervisor.js'
-
-// ---------------------------------------------------------------------------
-// Mocks
-// ---------------------------------------------------------------------------
-
-vi.mock('@revealui/core/monitoring', () => ({
-  registerCleanupHandler: vi.fn(),
-}))
-
-vi.mock('@revealui/core/observability/logger', () => ({
-  logger: { info: vi.fn(), warn: vi.fn(), error: vi.fn() },
-}))
-
-// Mock child_process.spawn
-const mockProcess = {
-  pid: 12345,
-  exitCode: null as number | null,
-  stdin: { write: vi.fn() },
-  stdout: { on: vi.fn() },
-  stderr: { on: vi.fn() },
-  on: vi.fn(),
-  kill: vi.fn(),
-}
-
-vi.mock('node:child_process', () => ({
-  spawn: vi.fn(() => mockProcess),
-}))
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-
-const testConfig: MCPServerConfig = {
-  name: 'test-server',
-  command: 'node',
-  args: ['test-mcp-server.js'],
-}
-
-// ---------------------------------------------------------------------------
-// Tests
-// ---------------------------------------------------------------------------
-
-describe('MCPHypervisor', () => {
-  beforeEach(() => {
-    MCPHypervisor._resetForTests()
-    mockProcess.exitCode = null
-    mockProcess.stdin.write.mockClear()
-    mockProcess.stdout.on.mockClear()
-    mockProcess.stderr.on.mockClear()
-    mockProcess.on.mockClear()
-    mockProcess.kill.mockClear()
-  })
-
-  afterEach(() => {
-    MCPHypervisor._resetForTests()
-    vi.clearAllMocks()
-  })
-
-  describe('getInstance()', () => {
-    it('returns a singleton', () => {
-      const a = MCPHypervisor.getInstance()
-      const b = MCPHypervisor.getInstance()
-      expect(a).toBe(b)
-    })
-  })
-
-  describe('registerServer()', () => {
-    it('registers a server config', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-
-      const status = hv.getStatus()
-      expect(status['test-server']).toMatchObject({
-        healthy: false,
-        toolCount: 0,
-        pid: null,
-      })
-    })
-
-    it('ignores duplicate registrations', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-      hv.registerServer(testConfig) // duplicate — no throw
-
-      const status = hv.getStatus()
-      expect(Object.keys(status)).toHaveLength(1)
-    })
-  })
-
-  describe('getAllTools()', () => {
-    it('returns empty array when no servers are healthy', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-
-      expect(hv.getAllTools()).toHaveLength(0)
-    })
-
-    it('returns namespaced tools from healthy servers', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-
-      // Directly inject tools into the server entry via internal state
-      // (bypasses spawn for unit testing)
-      // Access private state via type cast — for testing only
-      const servers = (
-        hv as unknown as { servers: Map<string, { healthy: boolean; tools: unknown[] }> }
-      ).servers
-      const entry = servers.get('test-server')!
-      entry.healthy = true
-      entry.tools = [
-        {
-          name: 'create_payment',
-          description: 'Create a payment',
-          inputSchema: { type: 'object' as const, properties: {}, required: [] },
-        },
-        {
-          name: 'list_payments',
-          description: 'List payments',
-          inputSchema: { type: 'object' as const, properties: {}, required: [] },
-        },
-      ]
-
-      const tools = hv.getAllTools()
-
-      expect(tools).toHaveLength(2)
-      expect(tools[0]?.namespacedName).toBe('@@mcp_test-server_create_payment')
-      expect(tools[1]?.namespacedName).toBe('@@mcp_test-server_list_payments')
-      expect(tools[0]?.serverName).toBe('test-server')
-    })
-
-    it('skips unhealthy servers', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-
-      const servers = (
-        hv as unknown as { servers: Map<string, { healthy: boolean; tools: unknown[] }> }
-      ).servers
-      const entry = servers.get('test-server')!
-      entry.healthy = false
-      entry.tools = [
-        { name: 'some_tool', description: '', inputSchema: { type: 'object' as const } },
-      ]
-
-      expect(hv.getAllTools()).toHaveLength(0)
-    })
-
-    it('namespaces tools from multiple healthy servers', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer({ name: 'stripe', command: 'node', args: [] })
-      hv.registerServer({ name: 'vercel', command: 'node', args: [] })
-
-      const servers = (
-        hv as unknown as { servers: Map<string, { healthy: boolean; tools: unknown[] }> }
-      ).servers
-
-      const stripe = servers.get('stripe')!
-      stripe.healthy = true
-      stripe.tools = [{ name: 'charge', description: '', inputSchema: { type: 'object' as const } }]
-
-      const vercel = servers.get('vercel')!
-      vercel.healthy = true
-      vercel.tools = [{ name: 'deploy', description: '', inputSchema: { type: 'object' as const } }]
-
-      const tools = hv.getAllTools()
-      const names = tools.map((t) => t.namespacedName)
-
-      expect(names).toContain('@@mcp_stripe_charge')
-      expect(names).toContain('@@mcp_vercel_deploy')
-    })
-  })
-
-  describe('pingServer()', () => {
-    it('returns false for unknown server', async () => {
-      const hv = MCPHypervisor.getInstance()
-      const result = await hv.pingServer('nonexistent')
-      expect(result).toBe(false)
-    })
-
-    it('returns false when process is not running', async () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-
-      const result = await hv.pingServer('test-server')
-      expect(result).toBe(false)
-    })
-  })
-
-  describe('getStatus()', () => {
-    it('reflects registered servers', () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer({ name: 'alpha', command: 'node', args: [] })
-      hv.registerServer({ name: 'beta', command: 'node', args: [] })
-
-      const status = hv.getStatus()
-      expect(Object.keys(status)).toHaveLength(2)
-      expect(status['alpha']?.healthy).toBe(false)
-      expect(status['beta']?.healthy).toBe(false)
-    })
-  })
-
-  describe('unregisterServer()', () => {
-    it('removes a registered server', async () => {
-      const hv = MCPHypervisor.getInstance()
-      hv.registerServer(testConfig)
-      await hv.unregisterServer('test-server')
-
-      expect(hv.getStatus()).not.toHaveProperty('test-server')
-    })
-  })
-})

package/src/adapters/db.ts

@@ -1,180 +0,0 @@
-/**
- * DB adapter for MCP.
- * - `connectPglite()` initializes ElectricSQL/pglite client with CRDT metadata.
- * - `connectPostgres()` creates a standard Postgres client with Electric metadata.
- * - `createMcpDbClient()` factory that selects adapter based on config.
- *
- * Uses @revealui/contracts as the single source of truth for CRDT operation types.
- */
-
-import { mkdir } from 'node:fs/promises'
-import { dirname } from 'node:path'
-import getMcpConfig from '@revealui/config/mcp'
-import type { CrdtOperationsInsert, CrdtOperationsRow } from '@revealui/contracts'
-import { getSSLConfig } from '@revealui/core/database/ssl-config'
-
-export interface QueryResult<T = Record<string, unknown>> {
-  rows: T[]
-  affectedRows?: number
-}
-
-export type McpDbClient = {
-  query: <T = Record<string, unknown>>(sql: string, params?: unknown[]) => Promise<QueryResult<T>>
-  close: () => Promise<void>
-}
-
-/** Re-export contracts CRDT types for consumers */
-export type { CrdtOperationsRow, CrdtOperationsInsert }
-
-// Type for PGlite instance (minimal interface to avoid import issues)
-interface PGliteInstance {
-  waitReady: Promise<void>
-  query<T = Record<string, unknown>>(
-    sql: string,
-    params?: unknown[],
-  ): Promise<{ rows: T[]; affectedRows?: number }>
-  exec(query: string): Promise<unknown>
-  close(): Promise<void>
-}
-
-// CRDT operations table schema for conflict-free replication
-const CRDT_TABLE_SQL = `
-  CREATE TABLE IF NOT EXISTS crdt_operations (
-    id TEXT PRIMARY KEY,
-    document_id TEXT NOT NULL,
-    operation_type TEXT NOT NULL,
-    payload JSONB NOT NULL,
-    vector_clock JSONB NOT NULL,
-    node_id TEXT NOT NULL,
-    created_at TIMESTAMPTZ NOT NULL DEFAULT NOW(),
-    applied_at TIMESTAMPTZ
-  );
-  CREATE INDEX IF NOT EXISTS idx_crdt_ops_document ON crdt_operations(document_id);
-  CREATE INDEX IF NOT EXISTS idx_crdt_ops_node ON crdt_operations(node_id);
-  CREATE INDEX IF NOT EXISTS idx_crdt_ops_created ON crdt_operations(created_at DESC);
-`
-
-/**
- * Connect to PGlite (embedded PostgreSQL) for local development/testing.
- * Uses dynamic import to avoid bundling @electric-sql/pglite when not needed.
- *
- * @param options - Optional connection options
- * @param options.dataDir - Override the default data directory. Use ':memory:' for in-memory.
- */
-export async function connectPglite(options?: { dataDir?: string }): Promise<McpDbClient> {
-  const cfg = getMcpConfig()
-
-  // Determine data directory from options, config, or default
-  // Use ':memory:' for in-memory database (useful for testing)
-  const dataDir = options?.dataDir || cfg.electricDatabaseUrl || '.revealui/mcp/pglite'
-
-  let db: PGliteInstance
-
-  try {
-    // Ensure directory exists if not in-memory
-    if (dataDir !== ':memory:') {
-      await mkdir(dirname(dataDir), { recursive: true })
-    }
-
-    const { PGlite } = await import('@electric-sql/pglite')
-    db = new PGlite(dataDir) as PGliteInstance
-    await db.waitReady
-  } catch (error) {
-    const err = error as Error
-    if (err.message?.includes('Cannot find module') || err.message?.includes('MODULE_NOT_FOUND')) {
-      throw new Error(
-        '@electric-sql/pglite is not installed. Run: pnpm add -D @electric-sql/pglite',
-      )
-    }
-    throw error
-  }
-
-  // Create CRDT operations table
-  await db.exec(CRDT_TABLE_SQL)
-
-  return {
-    async query<T = Record<string, unknown>>(
-      sql: string,
-      params?: unknown[],
-    ): Promise<QueryResult<T>> {
-      const result = await db.query(sql, params)
-      return {
-        rows: result.rows as T[],
-        affectedRows: result.affectedRows,
-      }
-    },
-    async close(): Promise<void> {
-      await db.close()
-    },
-  }
-}
-
-/**
- * Connect to PostgreSQL for production use.
- * Supports Neon, Supabase, and generic PostgreSQL with SSL.
- */
-export async function connectPostgres(): Promise<McpDbClient> {
-  const cfg = getMcpConfig()
-
-  const connectionString =
-    cfg.electricDatabaseUrl ||
-    process.env.DATABASE_URL ||
-    process.env.POSTGRES_URL ||
-    process.env.SUPABASE_DATABASE_URI
-
-  if (!connectionString) {
-    throw new Error(
-      'Database connection string not found. Set ELECTRIC_DATABASE_URL, DATABASE_URL, POSTGRES_URL, or SUPABASE_DATABASE_URI.',
-    )
-  }
-
-  const { Pool } = await import('pg')
-  const pool = new Pool({
-    connectionString,
-    ssl: getSSLConfig(connectionString),
-  })
-
-  // Test connection
-  const client = await pool.connect()
-  try {
-    await client.query('SELECT 1')
-  } finally {
-    client.release()
-  }
-
-  // Create CRDT operations table
-  await pool.query(CRDT_TABLE_SQL)
-
-  return {
-    async query<T = Record<string, unknown>>(
-      sql: string,
-      params?: unknown[],
-    ): Promise<QueryResult<T>> {
-      const result = await pool.query(sql, params)
-      return {
-        rows: result.rows as T[],
-        affectedRows: result.rowCount ?? undefined,
-      }
-    },
-    async close(): Promise<void> {
-      await pool.end()
-    },
-  }
-}
-
-/**
- * Factory function that creates an MCP database client based on configuration.
- * Reads `persistenceDriver` from getMcpConfig() to determine which adapter to use.
- */
-export async function createMcpDbClient(): Promise<McpDbClient> {
-  const cfg = getMcpConfig()
-
-  if (cfg.persistenceDriver === 'postgres') {
-    return connectPostgres()
-  }
-
-  // Default to pglite for local development
-  return connectPglite()
-}
-
-export default { connectPglite, connectPostgres, createMcpDbClient }

package/src/config/config.json

@@ -1,49 +0,0 @@
-{
-  "_comment": "MCP server configuration. This config is synchronized with .cursor/mcp-config.json (source of truth). All 6 servers are included for consistency.",
-  "_source_of_truth": ".cursor/mcp-config.json",
-  "_last_synced": "2025-01",
-  "mcpServers": {
-    "vercel": {
-      "command": "pnpm",
-      "args": ["mcp:vercel"],
-      "env": {
-        "VERCEL_API_KEY": "${VERCEL_API_KEY}",
-        "VERCEL_TOKEN": "${VERCEL_TOKEN}"
-      }
-    },
-    "stripe": {
-      "command": "pnpm",
-      "args": ["mcp:stripe"],
-      "env": {
-        "STRIPE_SECRET_KEY": "${STRIPE_SECRET_KEY}"
-      }
-    },
-    "neon": {
-      "command": "pnpm",
-      "args": ["mcp:neon"],
-      "env": {
-        "NEON_API_KEY": "${NEON_API_KEY}"
-      }
-    },
-    "supabase": {
-      "command": "pnpm",
-      "args": ["mcp:supabase"],
-      "env": {
-        "SUPABASE_URL": "${SUPABASE_URL}",
-        "SUPABASE_ANON_KEY": "${SUPABASE_ANON_KEY}",
-        "SUPABASE_SERVICE_ROLE_KEY": "${SUPABASE_SERVICE_ROLE_KEY}"
-      }
-    },
-    "playwright": {
-      "command": "pnpm",
-      "args": ["mcp:playwright"]
-    },
-    "next-devtools": {
-      "command": "pnpm",
-      "args": ["mcp:next-devtools"],
-      "env": {
-        "NEXT_TELEMETRY_DISABLED": "${NEXT_TELEMETRY_DISABLED:-0}"
-      }
-    }
-  }
-}