npm - @qwickapps/server - Versions diffs - 1.7.1 → 1.7.2 - Mend

@qwickapps/server 1.7.1 → 1.7.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (157) hide show

package/dist-ui/index.html CHANGED Viewed

@@ -4,7 +4,7 @@
     <meta charset="UTF-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <title>Control Panel</title>
-    <script type="module" crossorigin src="/assets/index-8y0jDGcd.js"></script>
+    <script type="module" crossorigin src="/assets/index-0gzisPdy.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-DQED0KPI.css">
   </head>
   <body>

package/package.json CHANGED Viewed

@@ -1,21 +1,24 @@
 {
   "name": "@qwickapps/server",
-  "version": "1.7.1",
+  "version": "1.7.2",
   "description": "Plugin-based application server framework for building websites, APIs, admin dashboards, and full-stack products",
   "main": "dist/src/index.js",
   "types": "dist/src/index.d.ts",
   "exports": {
     ".": {
       "types": "./dist/src/index.d.ts",
-      "import": "./dist/src/index.js"
+      "import": "./dist/src/index.js",
+      "require": "./dist/src/index.js"
     },
     "./plugins": {
       "types": "./dist/src/plugins/index.d.ts",
-      "import": "./dist/src/plugins/index.js"
+      "import": "./dist/src/plugins/index.js",
+      "require": "./dist/src/plugins/index.js"
     },
     "./ui": {
       "types": "./dist-ui-lib/src/components/index.d.ts",
-      "import": "./dist-ui-lib/index.js"
+      "import": "./dist-ui-lib/index.js",
+      "require": "./dist-ui-lib/index.js"
     }
   },
   "files": [
@@ -65,7 +68,7 @@
     "@mui/material": "^7.2.0",
     "@mui/x-data-grid": "^7.29.12",
     "@playwright/test": "^1.57.0",
-    "@qwickapps/auth-client": "^1.0.0",
+    "@qwickapps/auth-client": "^1.1.0",
     "@qwickapps/react-framework": "^1.5.5",
     "@testing-library/jest-dom": "^6.0.0",
     "@testing-library/react": "^14.0.0",

package/src/plugins/api-keys/api-keys-plugin.ts CHANGED Viewed

@@ -23,22 +23,25 @@ import {
   CreateApiKeySchema,
   UpdateApiKeySchema,
 } from './types.js';
+import { hasPostgres, getPostgres } from '../postgres-plugin.js';
+import { postgresApiKeyStore } from './stores/index.js';
 // Store instance for helper access
 let currentStore: ApiKeyStore | null = null;
 /**
- * Create the API Keys plugin
+ * Create the API Keys 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 createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
-  const debug = config.debug || false;
-  // Framework automatically prefixes routes with plugin slug, so use root path
-  const apiPrefix = config.api?.prefix || '/';
-  const apiEnabled = config.api?.enabled !== false;
-  function log(message: string, data?: Record<string, unknown>) {
-    if (debug) {
-      console.log(`[ApiKeysPlugin] ${message}`, data || '');
+export function createApiKeysPlugin(config: Partial<ApiKeysPluginConfig> = {}): Plugin {
+  function log(message: string, data?: Record<string, unknown>, isError = false) {
+    const prefix = '[ApiKeysPlugin]';
+    if (isError) {
+      console.error(`${prefix} ${message}`, data || '');
+    } else if (config.debug) {
+      console.log(`${prefix} ${message}`, data || '');
     }
   }
@@ -48,15 +51,56 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
     version: '1.0.0',
     async onStart(_pluginConfig: PluginConfig, registry: PluginRegistry): Promise<void> {
-      log('Starting API keys plugin');
+      const logger = registry.getLogger('api-keys');
       // Check for users plugin dependency
       if (!registry.hasPlugin('users')) {
-        throw new Error('API Keys plugin requires Users plugin to be loaded first');
+        logger.warn('Users plugin not loaded! API Keys plugin disabled.');
+        registry.registerHealthCheck({
+          name: 'api-keys-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! API Keys plugin disabled.');
+        registry.registerHealthCheck({
+          name: 'api-keys-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 ?? postgresApiKeyStore({
+        pool: () => getPostgres().getPool(),
+        autoCreateTables: true,
+      });
+      const debug = config.debug ?? false;
+      const apiPrefix = config.api?.prefix ?? '/api-keys';
+      const apiEnabled = config.api?.enabled ?? true;
+      log('Starting API keys plugin');
       // Initialize the store (creates tables and RLS policies if needed)
-      await config.store.initialize();
+      await store.initialize();
       log('API keys store initialized');
       // Initialize optional Phase 2 stores
@@ -71,7 +115,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
       }
       // Store reference for helper access
-      currentStore = config.store;
+      currentStore = store;
       // Register health check
       registry.registerHealthCheck({
@@ -117,7 +161,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
               };
               // Create the API key
-              const apiKey = await config.store.create(params);
+              const apiKey = await store.create(params);
               // Return the key with plaintext (ONLY time plaintext is accessible)
               res.status(201).json({
@@ -152,7 +196,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
                 return res.status(401).json({ error: 'Authentication required' });
               }
-              const keys = await config.store.list(userId);
+              const keys = await store.list(userId);
               // Remove sensitive fields from response
               const sanitized = keys.map(key => ({
@@ -191,7 +235,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
               }
               const { id } = req.params;
-              const key = await config.store.get(userId, id);
+              const key = await store.get(userId, id);
               if (!key) {
                 return res.status(404).json({ error: 'API key not found' });
@@ -244,7 +288,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
               const { id } = req.params;
               const params: UpdateApiKeyParams = validation.data;
-              const updated = await config.store.update(userId, id, params);
+              const updated = await store.update(userId, id, params);
               if (!updated) {
                 return res.status(404).json({ error: 'API key not found' });
@@ -287,7 +331,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
               }
               const { id } = req.params;
-              const deleted = await config.store.delete(userId, id);
+              const deleted = await store.delete(userId, id);
               if (!deleted) {
                 return res.status(404).json({ error: 'API key not found' });
@@ -357,7 +401,7 @@ export function createApiKeysPlugin(config: ApiKeysPluginConfig): Plugin {
                 const { id: keyId } = req.params;
                 // Verify key belongs to user
-                const key = await config.store.get(userId, keyId);
+                const key = await store.get(userId, keyId);
                 if (!key) {
                   return res.status(404).json({ error: 'API key not found' });
                 }

package/src/plugins/api-keys/index.ts CHANGED Viewed

@@ -48,8 +48,5 @@ export { postgresApiKeyStore } from './stores/index.js';
 // Middleware
 export { bearerTokenAuth } from './middleware/index.js';
-// UI Components
-export { ApiKeysStatusWidget } from './ApiKeysStatusWidget.js';
-export type { ApiKeysStatusWidgetProps } from './ApiKeysStatusWidget.js';
-export { ApiKeysManagementPage } from './ApiKeysManagementPage.js';
-export type { ApiKeysManagementPageProps } from './ApiKeysManagementPage.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/api-keys/types.ts CHANGED Viewed

@@ -230,17 +230,23 @@ export interface ApiKeysApiConfig {
 /**
  * API keys plugin configuration
+ *
+ * All properties are optional - plugin will use smart defaults:
+ * - store: Postgres API key store using registry's postgres instance
+ * - api.prefix: '/api-keys'
+ * - api.enabled: true
+ * - debug: false
  */
 export interface ApiKeysPluginConfig {
-  /** API key storage backend */
-  store: ApiKeyStore;
+  /** API key storage backend (default: postgres API key store from registry) */
+  store?: ApiKeyStore;
   /** Plugin scope storage backend (optional, for Phase 2) */
   scopeStore?: import('./stores/plugin-scope-store.js').PluginScopeStore;
   /** Usage log storage backend (optional, for Phase 2) */
   usageStore?: import('./stores/usage-log-store.js').UsageLogStore;
   /** API configuration */
   api?: ApiKeysApiConfig;
-  /** Enable debug logging */
+  /** Enable debug logging (default: false) */
   debug?: boolean;
 }

package/src/plugins/auth/index.ts CHANGED Viewed

@@ -65,8 +65,6 @@ export { basicAdapter } from './adapters/basic-adapter.js';
 export { supabaseAdapter } from './adapters/supabase-adapter.js';
 export { supertokensAdapter } from './adapters/supertokens-adapter.js';
-// UI Components
-export { AuthStatusWidget } from './AuthStatusWidget.js';
-export type { AuthStatusWidgetProps } from './AuthStatusWidget.js';
-export { AuthManagementPage } from './AuthManagementPage.js';
-export type { AuthManagementPageProps } from './AuthManagementPage.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/bans-plugin.ts CHANGED Viewed

@@ -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 CHANGED Viewed

@@ -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 CHANGED Viewed

@@ -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;
 }