@qwickapps/server 1.1.9 → 1.3.0

package/src/plugins/cache-plugin.ts

@@ -48,7 +48,7 @@
  * Copyright (c) 2025 QwickApps.com. All rights reserved.
  */
 
-import type { ControlPanelPlugin, PluginContext } from '../core/types.js';
+import type { Plugin, PluginConfig, PluginRegistry } from '../core/plugin-registry.js';
 
 // Dynamic import for ioredis (optional peer dependency)
 type Redis = import('ioredis').default;
@@ -180,9 +180,19 @@ export interface CacheInstance {
    * Get all keys matching a pattern
    * @param pattern - Pattern to match (prefix is applied automatically)
    * @returns Array of matching keys (without prefix)
+   * @deprecated Use scanKeys() for production - keys() blocks Redis on large datasets
    */
   keys(pattern: string): Promise<string[]>;
 
+  /**
+   * Scan keys matching a pattern using cursor-based iteration (non-blocking)
+   * @param pattern - Pattern to match (prefix is applied automatically)
+   * @param options - Optional scan configuration
+   * @param options.count - Hint for how many keys to return per iteration (default: 100)
+   * @returns Array of matching keys (without prefix)
+   */
+  scanKeys(pattern: string, options?: { count?: number }): Promise<string[]>;
+
   /**
    * Flush all keys with the configured prefix
    * @returns Number of keys deleted
@@ -248,7 +258,7 @@ export function hasCache(name = 'default'): boolean {
  *
  * @param config - Cache configuration
  * @param instanceName - Name for this cache instance (default: 'default')
- * @returns A control panel plugin
+ * @returns A plugin
  *
  * @example
  * ```typescript
@@ -263,10 +273,11 @@ export function hasCache(name = 'default'): boolean {
 export function createCachePlugin(
   config: CachePluginConfig,
   instanceName = 'default'
-): ControlPanelPlugin {
+): Plugin {
   let client: Redis | null = null;
   const prefix = config.keyPrefix ?? '';
   const defaultTtl = config.defaultTtl ?? 3600;
+  const pluginId = `cache:${instanceName}`;
 
   const prefixKey = (key: string): string => `${prefix}${key}`;
   const unprefixKey = (key: string): string =>
@@ -382,6 +393,25 @@ export function createCachePlugin(
         return keys.map(unprefixKey);
       },
 
+      async scanKeys(pattern: string, options?: { count?: number }): Promise<string[]> {
+        if (!client) throw new Error('Cache client not initialized');
+        const results: string[] = [];
+        const stream = client.scanStream({
+          match: prefixKey(pattern),
+          count: options?.count ?? 100,
+        });
+
+        return new Promise((resolve, reject) => {
+          stream.on('data', (keys: string[]) => {
+            for (const key of keys) {
+              results.push(unprefixKey(key));
+            }
+          });
+          stream.on('end', () => resolve(results));
+          stream.on('error', (err) => reject(err));
+        });
+      },
+
       async flush(): Promise<number> {
         if (!client) throw new Error('Cache client not initialized');
         if (!prefix) {
@@ -437,11 +467,12 @@ export function createCachePlugin(
   };
 
   return {
-    name: `cache:${instanceName}`,
-    order: 5, // Initialize early, before other plugins that may need cache
+    id: pluginId,
+    name: `Redis Cache (${instanceName})`,
+    version: '1.0.0',
 
-    async onInit(context: PluginContext): Promise<void> {
-      const { registerHealthCheck, logger } = context;
+    async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
+      const logger = registry.getLogger(pluginId);
 
       // Create and register the instance
       const instance = await createInstance();
@@ -459,7 +490,7 @@ export function createCachePlugin(
 
       // Register health check if enabled
       if (config.healthCheck !== false) {
-        registerHealthCheck({
+        registry.registerHealthCheck({
           name: config.healthCheckName ?? 'redis',
           type: 'custom',
           interval: config.healthCheckInterval ?? 30000,
@@ -492,7 +523,7 @@ export function createCachePlugin(
       }
     },
 
-    async onShutdown(): Promise<void> {
+    async onStop(): Promise<void> {
       const instance = instances.get(instanceName);
       if (instance) {
         await instance.close();

package/src/plugins/config-plugin.ts

@@ -7,7 +7,8 @@
  */
 
 import type { Request, Response } from 'express';
-import type { ControlPanelPlugin, ConfigDisplayOptions, PluginContext } from '../core/types.js';
+import type { Plugin, PluginConfig, PluginRegistry } from '../core/plugin-registry.js';
+import type { ConfigDisplayOptions } from '../core/types.js';
 
 export interface ConfigPluginConfig extends ConfigDisplayOptions {}
 
@@ -20,15 +21,20 @@ interface ConfigValidationResult {
 /**
  * Create a config display plugin
  */
-export function createConfigPlugin(config: ConfigPluginConfig): ControlPanelPlugin {
+export function createConfigPlugin(config: ConfigPluginConfig): Plugin {
   return {
-    name: 'config',
-    order: 30,
+    id: 'config',
+    name: 'Config Plugin',
+    version: '1.0.0',
 
-    routes: [
-      {
+    async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
+      const logger = registry.getLogger('config');
+
+      // Register /config route
+      registry.addRoute({
         method: 'get',
         path: '/config',
+        pluginId: 'config',
         handler: (_req: Request, res: Response) => {
           const envVars: Record<string, string> = {};
 
@@ -52,10 +58,13 @@ export function createConfigPlugin(config: ConfigPluginConfig): ControlPanelPlug
             config: envVars,
           });
         },
-      },
-      {
+      });
+
+      // Register /config/validate route
+      registry.addRoute({
         method: 'get',
         path: '/config/validate',
+        pluginId: 'config',
         handler: (_req: Request, res: Response) => {
           const results: ConfigValidationResult[] = [];
           let allValid = true;
@@ -97,11 +106,13 @@ export function createConfigPlugin(config: ConfigPluginConfig): ControlPanelPlug
             results,
           });
         },
-      },
-    ],
+      });
+
+      logger.debug(`Config plugin initialized with ${config.show.length} vars`);
+    },
 
-    async onInit(context: PluginContext): Promise<void> {
-      context.logger.debug(`Config plugin initialized with ${config.show.length} vars`);
+    async onStop(): Promise<void> {
+      // Nothing to cleanup
     },
   };
 }

package/src/plugins/diagnostics-plugin.ts

@@ -9,7 +9,7 @@
 import { existsSync, readFileSync } from 'fs';
 import { resolve } from 'path';
 import type { Request, Response } from 'express';
-import type { ControlPanelPlugin, PluginContext } from '../core/types.js';
+import type { Plugin, PluginConfig, PluginRegistry } from '../core/plugin-registry.js';
 
 export interface DiagnosticsPluginConfig {
   include?: {
@@ -31,7 +31,7 @@ export interface DiagnosticsPluginConfig {
 /**
  * Create a diagnostics plugin for AI agents
  */
-export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): ControlPanelPlugin {
+export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): Plugin {
   const {
     include = { logs: { startup: 100, app: 200 }, health: true, config: true, system: true },
     logPaths = { startup: './logs/startup.log', app: './logs/app.log' },
@@ -39,13 +39,18 @@ export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): C
   } = config;
 
   return {
-    name: 'diagnostics',
-    order: 40,
+    id: 'diagnostics',
+    name: 'Diagnostics Plugin',
+    version: '1.0.0',
 
-    routes: [
-      {
+    async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
+      const logger = registry.getLogger('diagnostics');
+
+      // Register full diagnostics endpoint
+      registry.addRoute({
         method: 'get',
         path: endpoint,
+        pluginId: 'diagnostics',
         handler: (_req: Request, res: Response) => {
           try {
             const report: Record<string, unknown> = {
@@ -113,10 +118,13 @@ export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): C
             });
           }
         },
-      },
-      {
+      });
+
+      // Register summary diagnostics endpoint
+      registry.addRoute({
         method: 'get',
         path: '/diagnostics/summary',
+        pluginId: 'diagnostics',
         handler: (_req: Request, res: Response) => {
           try {
             const memUsage = process.memoryUsage();
@@ -138,11 +146,13 @@ export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): C
             });
           }
         },
-      },
-    ],
+      });
+
+      logger.debug('Diagnostics plugin initialized');
+    },
 
-    async onInit(context: PluginContext): Promise<void> {
-      context.logger.debug('Diagnostics plugin initialized');
+    async onStop(): Promise<void> {
+      // Nothing to cleanup
     },
   };
 }