@qwickapps/server 1.7.1 → 1.8.0

package/src/plugins/bans/bans-plugin.ts

@@ -22,6 +22,8 @@ import type {
 } from './types.js';
 import type { AuthenticatedRequest } from '../auth/types.js';
 import { getUserByEmail, getUserById, getUsersByIds } from '../users/users-plugin.js';
+import { hasPostgres, getPostgres } from '../postgres-plugin.js';
+import { postgresBanStore } from './stores/index.js';
 
 // Store instance for helper access
 let currentStore: BanStore | null = null;
@@ -29,17 +31,18 @@ let banCleanupInterval: NodeJS.Timeout | null = null;
 let pluginConfig: BansPluginConfig | null = null;
 
 /**
- * Create the Bans plugin
+ * Create the Bans plugin with smart defaults
+ *
+ * Config is optional - plugin will use defaults and get dependencies from registry.
+ * Gracefully handles missing dependencies with clear log messages.
  */
-export function createBansPlugin(config: BansPluginConfig): Plugin {
-  const debug = config.debug || false;
-  // Routes are mounted under /api by the control panel, so don't include /api in prefix
-  const apiPrefix = config.api?.prefix || '/'; // Framework adds /bans prefix automatically
-  const apiEnabled = config.api?.enabled !== false;
-
-  function log(message: string, data?: Record<string, unknown>) {
-    if (debug) {
-      console.log(`[BansPlugin] ${message}`, data || '');
+export function createBansPlugin(config: Partial<BansPluginConfig> = {}): Plugin {
+  function log(message: string, data?: Record<string, unknown>, isError = false) {
+    const prefix = '[BansPlugin]';
+    if (isError) {
+      console.error(`${prefix} ${message}`, data || '');
+    } else if (config.debug) {
+      console.log(`${prefix} ${message}`, data || '');
     }
   }
 
@@ -49,26 +52,68 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
     version: '1.0.0',
 
     async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
-      log('Starting bans plugin');
+      const logger = registry.getLogger('bans');
 
       // Check for users plugin dependency
       if (!registry.hasPlugin('users')) {
-        throw new Error('Bans plugin requires Users plugin to be loaded first');
+        logger.warn('Users plugin not loaded! Bans plugin disabled.');
+        registry.registerHealthCheck({
+          name: 'bans-store',
+          type: 'custom',
+          check: async () => ({
+            healthy: false,
+            details: {
+              error: 'Users plugin not available',
+              state: 'disabled',
+            },
+          }),
+        });
+        return;
+      }
+
+      // Check for postgres in registry
+      if (!hasPostgres()) {
+        logger.warn('No Database! Bans plugin disabled.');
+        registry.registerHealthCheck({
+          name: 'bans-store',
+          type: 'custom',
+          check: async () => ({
+            healthy: false,
+            details: {
+              error: 'PostgreSQL not available',
+              state: 'disabled',
+            },
+          }),
+        });
+        return;
       }
 
+      // Smart defaults - get dependencies from registry
+      const store = config.store ?? postgresBanStore({
+        pool: () => getPostgres().getPool(),
+        autoCreateTables: true,
+      });
+
+      const debug = config.debug ?? false;
+      const apiPrefix = config.api?.prefix ?? '/bans';
+      const apiEnabled = config.api?.enabled ?? true;
+      const supportTemporary = config.supportTemporary ?? true;
+
+      log('Starting bans plugin');
+
       // Initialize the store (creates tables if needed)
-      await config.store.initialize();
+      await store.initialize();
       log('Bans plugin migrations complete');
 
       // Store references for helper access
-      currentStore = config.store;
-      pluginConfig = config;
+      currentStore = store;
+      pluginConfig = { ...config, store, debug, supportTemporary };
 
       // Start ban cleanup interval if temporary bans are supported
-      if (config.supportTemporary) {
+      if (supportTemporary) {
         banCleanupInterval = setInterval(async () => {
           try {
-            const cleaned = await config.store.cleanupExpiredBans();
+            const cleaned = await store.cleanupExpiredBans();
             if (cleaned > 0) {
               log('Cleaned up expired bans', { count: cleaned });
             }
@@ -85,7 +130,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
         check: async () => {
           try {
             // Simple health check - list with limit 0
-            await config.store.listActiveBans({ limit: 0 });
+            await store.listActiveBans({ limit: 0 });
             return { healthy: true };
           } catch {
             return { healthy: false };
@@ -105,7 +150,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
               const limit = Math.min(parseInt(req.query.limit as string) || 50, 100);
               const offset = parseInt(req.query.offset as string) || 0;
 
-              const result = await config.store.listActiveBans({ limit, offset });
+              const result = await store.listActiveBans({ limit, offset });
 
               // Batch fetch users for all bans (single query instead of N queries)
               const userIds = [...new Set(result.bans.map((ban) => ban.user_id))];
@@ -133,7 +178,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
           pluginId: 'bans',
           handler: async (req: Request, res: Response) => {
             try {
-              const ban = await config.store.getActiveBan(req.params.userId);
+              const ban = await store.getActiveBan(req.params.userId);
               res.json({
                 isBanned: ban !== null,
                 ban,
@@ -152,7 +197,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
           pluginId: 'bans',
           handler: async (req: Request, res: Response) => {
             try {
-              const bans = await config.store.listBans(req.params.userId);
+              const bans = await store.listBans(req.params.userId);
               res.json({ bans });
             } catch (error) {
               console.error('[BansPlugin] Get ban history error:', error);
@@ -185,7 +230,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
                 return res.status(404).json({ error: 'User not found' });
               }
 
-              const ban = await config.store.createBan(input);
+              const ban = await store.createBan(input);
 
               // Call onBan callback if provided
               if (config.callbacks?.onBan) {
@@ -227,7 +272,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
                 note: req.body?.note,
               };
 
-              const removed = await config.store.removeBan(input);
+              const removed = await store.removeBan(input);
               if (!removed) {
                 return res.status(404).json({ error: 'No active ban found' });
               }
@@ -292,7 +337,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
                 metadata: req.body.metadata,
               };
 
-              const ban = await config.store.createBan(input);
+              const ban = await store.createBan(input);
 
               // Call onBan callback if provided
               if (config.callbacks?.onBan) {
@@ -335,7 +380,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
                 note: req.body?.note,
               };
 
-              const removed = await config.store.removeBan(input);
+              const removed = await store.removeBan(input);
               if (!removed) {
                 return res.status(404).json({ error: 'No active ban found' });
               }
@@ -370,7 +415,7 @@ export function createBansPlugin(config: BansPluginConfig): Plugin {
         banCleanupInterval = null;
       }
 
-      await config.store.shutdown();
+      if (currentStore) { await currentStore.shutdown(); };
       currentStore = null;
       pluginConfig = null;
 

package/src/plugins/bans/index.ts

@@ -30,8 +30,6 @@ export type {
 // Stores
 export { postgresBanStore, inMemoryBanStore } from './stores/index.js';
 
-// UI Components
-export { BansStatusWidget } from './BansStatusWidget.js';
-export type { BansStatusWidgetProps } from './BansStatusWidget.js';
-export { BansManagementPage } from './BansManagementPage.js';
-export type { BansManagementPageProps } from './BansManagementPage.js';
+// UI Components are exported from main package index (@qwickapps/server)
+// Do NOT export here to avoid loading UI dependencies when importing plugins
+

package/src/plugins/bans/types.ts

@@ -107,22 +107,29 @@ export interface BanCallbacks {
 
 /**
  * Bans plugin configuration
+ *
+ * All properties are optional - plugin will use smart defaults:
+ * - store: Postgres ban store using registry's postgres instance
+ * - supportTemporary: true
+ * - api.prefix: '/bans'
+ * - api.enabled: true
+ * - debug: false
  */
 export interface BansPluginConfig {
-  /** Ban storage backend */
-  store: BanStore;
-  /** Support temporary bans (with expiration) */
+  /** Ban storage backend (default: postgres ban store from registry) */
+  store?: BanStore;
+  /** Support temporary bans (with expiration) (default: true) */
   supportTemporary?: boolean;
   /** Callbacks */
   callbacks?: BanCallbacks;
   /** API configuration */
   api?: {
-    /** API route prefix (default: '/api/bans') */
+    /** API route prefix (default: '/bans') */
     prefix?: string;
-    /** Enable API endpoints */
+    /** Enable API endpoints (default: true) */
     enabled?: boolean;
   };
-  /** Enable debug logging */
+  /** Enable debug logging (default: false) */
   debug?: boolean;
 }
 

package/src/plugins/devices/devices-plugin.ts

@@ -28,6 +28,9 @@ import {
   isTokenExpired,
   getTokenExpiration,
 } from './token-utils.js';
+import { hasPostgres, getPostgres } from '../postgres-plugin.js';
+import { postgresDeviceStore } from './stores/index.js';
+import { computeDeviceAdapter } from './adapters/index.js';
 
 // Store instances for helper access
 let currentStore: DeviceStore | null = null;
@@ -35,16 +38,18 @@ let currentAdapter: DeviceAdapter | null = null;
 let currentConfig: DevicesPluginConfig | null = null;
 
 /**
- * Create the Devices plugin
+ * Create the Devices plugin with smart defaults
+ *
+ * Config is optional - plugin will use defaults and get dependencies from registry.
+ * Gracefully handles missing dependencies with clear log messages.
  */
-export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
-  const debug = config.debug || false;
-  const defaultTokenValidityDays = config.defaultTokenValidityDays || 90;
-  const apiPrefix = config.api?.prefix || '/'; // Framework adds /devices prefix automatically
-
-  function log(message: string, data?: Record<string, unknown>) {
-    if (debug) {
-      console.log(`[DevicesPlugin] ${message}`, data || '');
+export function createDevicesPlugin(config: Partial<DevicesPluginConfig> = {}): Plugin {
+  function log(message: string, data?: Record<string, unknown>, isError = false) {
+    const prefix = '[DevicesPlugin]';
+    if (isError) {
+      console.error(`${prefix} ${message}`, data || '');
+    } else if (config.debug) {
+      console.log(`${prefix} ${message}`, data || '');
     }
   }
 
@@ -54,16 +59,46 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
     version: '1.0.0',
 
     async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
-      log('Starting devices plugin', { adapter: config.adapter.name });
+      const logger = registry.getLogger('devices');
+
+      // Check for postgres in registry
+      if (!hasPostgres()) {
+        logger.warn('No Database! Devices plugin disabled.');
+        registry.registerHealthCheck({
+          name: 'devices-store',
+          type: 'custom',
+          check: async () => ({
+            healthy: false,
+            details: {
+              error: 'PostgreSQL not available',
+              state: 'disabled',
+            },
+          }),
+        });
+        return;
+      }
+
+      // Smart defaults - get dependencies from registry
+      const store = config.store ?? postgresDeviceStore({
+        pool: () => getPostgres().getPool(),
+        autoCreateTables: true,
+      });
+
+      const adapter = config.adapter ?? computeDeviceAdapter();
+      const debug = config.debug ?? false;
+      const defaultTokenValidityDays = config.defaultTokenValidityDays ?? 90;
+      const apiPrefix = config.api?.prefix ?? '/devices';
+
+      log('Starting devices plugin', { adapter: adapter.name });
 
       // Initialize the store (creates tables if needed)
-      await config.store.initialize();
+      await store.initialize();
       log('Devices plugin migrations complete');
 
       // Store references for helper access
-      currentStore = config.store;
-      currentAdapter = config.adapter;
-      currentConfig = config;
+      currentStore = store;
+      currentAdapter = adapter;
+      currentConfig = { ...config, store, adapter, debug, defaultTokenValidityDays };
 
       // Register health check
       registry.registerHealthCheck({
@@ -71,12 +106,12 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
         type: 'custom',
         check: async () => {
           try {
-            await config.store.search({ limit: 1 });
+            await store.search({ limit: 1 });
             return {
               healthy: true,
               details: {
-                adapter: config.adapter.name,
-                tokenPrefix: config.adapter.tokenPrefix,
+                adapter: adapter.name,
+                tokenPrefix: adapter.tokenPrefix,
               },
             };
           } catch {
@@ -106,7 +141,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
                 sortOrder: (req.query.sortOrder as DeviceSearchParams['sortOrder']) || 'desc',
               };
 
-              const result = await config.store.search(params);
+              const result = await store.search(params);
               res.json(result);
             } catch (error) {
               console.error('[DevicesPlugin] Search error:', error);
@@ -122,7 +157,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
           pluginId: 'devices',
           handler: async (req: Request, res: Response) => {
             try {
-              const device = await config.store.getById(req.params.id);
+              const device = await store.getById(req.params.id);
               if (!device) {
                 return res.status(404).json({ error: 'Device not found' });
               }
@@ -150,7 +185,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
               };
 
               // Validate using adapter
-              const validation = config.adapter.validateDeviceInput(input);
+              const validation = adapter.validateDeviceInput(input);
               if (!validation.valid) {
                 return res.status(400).json({
                   error: 'Validation failed',
@@ -181,7 +216,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
                 metadata: req.body.metadata,
               };
 
-              const device = await config.store.update(req.params.id, input);
+              const device = await store.update(req.params.id, input);
               if (!device) {
                 return res.status(404).json({ error: 'Device not found' });
               }
@@ -200,19 +235,19 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
           pluginId: 'devices',
           handler: async (req: Request, res: Response) => {
             try {
-              const device = await config.store.getById(req.params.id);
+              const device = await store.getById(req.params.id);
               if (!device) {
                 return res.status(404).json({ error: 'Device not found' });
               }
 
-              const deleted = await config.store.delete(req.params.id);
+              const deleted = await store.delete(req.params.id);
               if (!deleted) {
                 return res.status(404).json({ error: 'Device not found' });
               }
 
               // Call adapter hook
-              if (config.adapter.onDeviceDeleted) {
-                await config.adapter.onDeviceDeleted(device);
+              if (adapter.onDeviceDeleted) {
+                await adapter.onDeviceDeleted(device);
               }
 
               res.status(204).send();
@@ -230,7 +265,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
           pluginId: 'devices',
           handler: async (req: Request, res: Response) => {
             try {
-              const device = await config.store.getById(req.params.id);
+              const device = await store.getById(req.params.id);
               if (!device) {
                 return res.status(404).json({ error: 'Device not found' });
               }
@@ -295,7 +330,7 @@ export function createDevicesPlugin(config: DevicesPluginConfig): Plugin {
 
     async onStop(): Promise<void> {
       log('Stopping devices plugin');
-      await config.store.shutdown();
+      if (currentStore) { await currentStore.shutdown(); };
       currentStore = null;
       currentAdapter = null;
       currentConfig = null;

package/src/plugins/devices/index.ts

@@ -68,8 +68,6 @@ export {
 } from './token-utils.js';
 export type { DeviceTokenPair } from './token-utils.js';
 
-// UI Components
-export { DevicesStatusWidget } from './DevicesStatusWidget.js';
-export type { DevicesStatusWidgetProps } from './DevicesStatusWidget.js';
-export { DevicesManagementPage } from './DevicesManagementPage.js';
-export type { DevicesManagementPageProps } from './DevicesManagementPage.js';
+// UI Components are exported from main package index (@qwickapps/server)
+// Do NOT export here to avoid loading UI dependencies when importing plugins
+

package/src/plugins/diagnostics-plugin.ts

@@ -148,6 +148,83 @@ export function createDiagnosticsPlugin(config: DiagnosticsPluginConfig = {}): P
         },
       });
 
+      // Register maintenance listing endpoint (GET /diagnostics/maintenance)
+      registry.addRoute({
+        method: 'get',
+        path: '/maintenance',
+        pluginId: 'diagnostics',
+        handler: (_req: Request, res: Response) => {
+          try {
+            const pluginsNeedingMaintenance = registry.getPluginsNeedingMaintenance();
+
+            res.json({
+              timestamp: new Date().toISOString(),
+              count: pluginsNeedingMaintenance.length,
+              plugins: pluginsNeedingMaintenance.map(info => ({
+                pluginId: info.pluginId,
+                error: info.error,
+                recommendedAction: info.recommendedAction,
+                actions: info.actions.map(action => ({
+                  id: action.id,
+                  name: action.name,
+                  description: action.description,
+                  destructive: action.destructive,
+                })),
+              })),
+            });
+          } catch (error) {
+            res.status(500).json({
+              error: 'Failed to retrieve maintenance info',
+              message: error instanceof Error ? error.message : String(error),
+            });
+          }
+        },
+      });
+
+      // Register maintenance action endpoint (POST /diagnostics/maintenance/:pluginId/:actionId)
+      registry.addRoute({
+        method: 'post',
+        path: '/maintenance/:pluginId/:actionId',
+        pluginId: 'diagnostics',
+        handler: async (req: Request, res: Response) => {
+          try {
+            const { pluginId, actionId } = req.params;
+
+            if (!pluginId || !actionId) {
+              return res.status(400).json({
+                error: 'Missing required parameters',
+                message: 'Both pluginId and actionId are required',
+              });
+            }
+
+            logger.info(`Executing maintenance action ${actionId} for plugin ${pluginId}`);
+
+            const result = await registry.runMaintenanceAction(pluginId, actionId);
+
+            if (result.success) {
+              res.json({
+                success: true,
+                message: result.message || 'Maintenance action completed successfully',
+                pluginId,
+                actionId,
+              });
+            } else {
+              res.status(500).json({
+                success: false,
+                error: result.error || 'Maintenance action failed',
+                pluginId,
+                actionId,
+              });
+            }
+          } catch (error) {
+            res.status(500).json({
+              error: 'Failed to execute maintenance action',
+              message: error instanceof Error ? error.message : String(error),
+            });
+          }
+        },
+      });
+
       logger.debug('Diagnostics plugin initialized');
     },