remove error monitor

Author: mikeschen

src/server.ts

@@ -21,7 +21,6 @@ import localDevBypassAuthPermissions from './auth/local-dev/permissions.js'
 import MutableOrgDS from './model/MutableOrganizationDataSource.js'
 import UserDataSource from './model/UserDataSource.js'
 import BulkImportDataSource from './model/BulkImportDataSource.js'
-import { errorMonitor } from './utils/ErrorMonitor.js'
 
 /**
  * Create a GraphQL server
@@ -50,95 +49,26 @@ export async function createServer (): Promise<{ app: express.Application, serve
     schema,
     plugins: [ApolloServerPluginDrainHttpServer({ httpServer })],
     cache: new InMemoryLRUCache({
-      max: 100,
-      maxSize: 1024 * 1024 * 20, // Increased cache size
-      ttl: 300000 // 5 minutes TTL to prevent memory leaks
-    }),
-    // Enhanced error handling
-    formatError: (formattedError, _error) => {
-      // Log the error with enhanced monitoring
-      errorMonitor.logGraphQLError(formattedError, undefined, undefined)
-
-      // Don't expose internal errors in production
-      if (process.env.NODE_ENV === 'production') {
-        // Remove stack trace and internal details
-        if (formattedError.extensions?.exception != null) {
-          delete (formattedError.extensions.exception as any).stacktrace
-        }
-        if (formattedError.message.includes('internal') ||
-            formattedError.message.includes('database')) {
-          return new Error('Internal server error')
-        }
-      }
-
-      return formattedError
-    }
+      max: 50,
+      maxSize: 1024 * 1024 * 10
+    })
   })
   // server must be started before applying middleware
   await server.start()
 
   const context = process.env.LOCAL_DEV_BYPASS_AUTH === 'true' ? localDevBypassAuthContext : createContext
 
-  // Enhanced health check with memory monitoring
   app.get('/health', (req, res) => {
     const memUsage = process.memoryUsage()
-    const uptime = process.uptime()
-
-    // Check if memory usage is getting too high
-    const heapUsedMB = Math.round(memUsage.heapUsed / 1024 / 1024)
-    const heapTotalMB = Math.round(memUsage.heapTotal / 1024 / 1024)
-    const memoryUsagePercent = (heapUsedMB / heapTotalMB) * 100
-
-    const status = memoryUsagePercent > 85 ? 'warning' : 'ok'
-
     res.json({
-      status,
+      status: 'ok',
       timestamp: new Date().toISOString(),
-      uptime: `${Math.round(uptime / 60)} minutes`,
       memory: {
         rss: `${Math.round(memUsage.rss / 1024 / 1024)}MB`,
-        heapTotal: `${heapTotalMB}MB`,
-        heapUsed: `${heapUsedMB}MB`,
-        external: `${Math.round(memUsage.external / 1024 / 1024)}MB`,
-        usagePercent: `${Math.round(memoryUsagePercent)}%`
-      },
-      warnings: memoryUsagePercent > 85 ? ['High memory usage detected'] : []
-    })
-
-    // Log warning if memory usage is high
-    if (memoryUsagePercent > 85) {
-      console.warn(`High memory usage: ${Math.round(memoryUsagePercent)}% (${heapUsedMB}MB/${heapTotalMB}MB)`)
-    }
-  })
-
-  // Periodic memory cleanup and monitoring
-  setInterval(() => {
-    const memUsage = process.memoryUsage()
-    const heapUsedMB = Math.round(memUsage.heapUsed / 1024 / 1024)
-    const heapTotalMB = Math.round(memUsage.heapTotal / 1024 / 1024)
-    const memoryUsagePercent = (heapUsedMB / heapTotalMB) * 100
-
-    // Force garbage collection if memory usage is high and gc is available
-    if (memoryUsagePercent > 80 && global.gc != null) {
-      console.log('Running garbage collection due to high memory usage')
-      global.gc()
-    }
-
-    // Log memory stats every 5 minutes
-    console.log(`Memory usage: ${heapUsedMB}MB/${heapTotalMB}MB (${Math.round(memoryUsagePercent)}%)`)
-  }, 5 * 60 * 1000) // Every 5 minutes
-
-  // Error monitoring endpoint
-  app.get('/errors', (req, res) => {
-    const stats = errorMonitor.getStats()
-    const healthStatus = errorMonitor.getHealthStatus()
-
-    res.json({
-      healthStatus,
-      totalErrors: stats.totalErrors,
-      errorsByType: Object.fromEntries(stats.errorsByType),
-      recentErrors: stats.recentErrors.slice(-10), // Last 10 errors
-      lastReset: stats.lastReset
+        heapTotal: `${Math.round(memUsage.heapTotal / 1024 / 1024)}MB`,
+        heapUsed: `${Math.round(memUsage.heapUsed / 1024 / 1024)}MB`,
+        external: `${Math.round(memUsage.external / 1024 / 1024)}MB`
+      }
     })
   })