@revealui/mcp 0.0.1-pre.0 → 0.1.0

package/migrations/005_performance_indexes.sql

@@ -0,0 +1,190 @@
+-- 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

@@ -0,0 +1,45 @@
+#!/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/package.json

@@ -1,96 +1,32 @@
 {
   "name": "@revealui/mcp",
-  "version": "0.0.1-pre.0",
-  "description": "Reveal MCP - Unified Model Context Protocol implementation for multi-agent systems and PayloadCMS integration",
-  "type": "module",
-  "main": "./dist/index.js",
-  "module": "./dist/index.js",
-  "types": "./dist/index.d.ts",
-  "exports": {
-    ".": {
-      "types": "./dist/index.d.ts",
-      "import": "./dist/index.js"
-    },
-    "./server": {
-      "types": "./dist/server/index.d.ts",
-      "import": "./dist/server/index.js"
-    },
-    "./agents": {
-      "types": "./dist/agents/index.d.ts",
-      "import": "./dist/agents/index.js"
-    },
-    "./tools": {
-      "types": "./dist/tools/index.d.ts",
-      "import": "./dist/tools/index.js"
-    },
-    "./analytics": {
-      "types": "./dist/analytics/index.d.ts",
-      "import": "./dist/analytics/index.js"
-    },
-    "./analytics/server": {
-      "types": "./dist/analytics/server.d.ts",
-      "import": "./dist/analytics/server.js"
-    },
-    "./bridge/PayloadMCPService": {
-      "types": "./dist/bridge/PayloadMCPService.d.ts",
-      "import": "./dist/bridge/PayloadMCPService.js"
-    }
-  },
-  "files": [
-    "dist",
-    "README.md",
-    "LICENSE",
-    "CHANGELOG.md"
-  ],
-  "keywords": [
-    "mcp",
-    "model-context-protocol",
-    "multi-agent",
-    "payloadcms",
-    "ai",
-    "agents",
-    "tools",
-    "reveal"
-  ],
-  "license": "MIT",
-  "publishable": true,
-  "author": "RevealUI Team",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/RevealUIStudio/reveal.git",
-    "directory": "packages/mcp"
-  },
-  "bugs": {
-    "url": "https://github.com/RevealUIStudio/reveal/issues"
-  },
-  "homepage": "https://revealui.com",
-  "publishConfig": {
-    "access": "public"
-  },
+  "version": "0.1.0",
+  "license": "SEE LICENSE IN ../../LICENSE.commercial",
   "dependencies": {
-    "@revealui/core": "^0.0.1-pre.0",
-    "@libsql/client": "^0.15.15",
-    "@revealui/infrastructure": "0.0.1-pre.0"
+    "@modelcontextprotocol/sdk": "^1.26.0",
+    "dotenv": "^17.2.4",
+    "@revealui/config": "0.2.0",
+    "@revealui/contracts": "1.0.0",
+    "@revealui/scripts": "0.1.0",
+    "@revealui/core": "0.2.0"
   },
   "devDependencies": {
-    "@types/node": "^22.10.5",
-    "@types/react": "^19.2.2",
+    "@electric-sql/pglite": "^0.3.15",
+    "pg": "^8.18.0",
     "typescript": "^5.9.3",
-    "vitest": "^4.0.14",
-    "rimraf": "^6.1.2",
-    "tsup": "^8.3.5"
-  },
-  "peerDependencies": {
-    "payload": ">=3.60.0 <4",
-    "react": "^19.0.0"
+    "vitest": "^4.0.18"
   },
-  "engines": {
-    "node": ">=22.0.0"
+  "type": "module",
+  "publishConfig": {
+    "registry": "https://registry.npmjs.org",
+    "access": "public"
   },
   "scripts": {
-    "build": "pnpm exec tsup",
-    "clean": "rm -rf dist .tsbuildinfo",
-    "typecheck": "tsc -p tsconfig.json --noEmit",
-    "test": "vitest run"
+    "build": "tsc",
+    "clean": "rm -rf dist",
+    "dev": "tsc --watch",
+    "lint": "biome check .",
+    "test": "vitest run",
+    "typecheck": "tsc --noEmit"
   }
 }

package/src/__tests__/hypervisor.test.ts

@@ -0,0 +1,212 @@
+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')
+    })
+  })
+})