@qwickapps/server 1.7.2 → 1.8.0

package/src/plugins/api-keys/stores/postgres-store.ts

@@ -21,6 +21,7 @@ import type {
   UpdateApiKeyParams,
   ApiKey,
   ApiKeyWithPlaintext,
+  StoreInitializationResult,
 } from '../types.js';
 
 // Pool interface (from pg package)
@@ -239,82 +240,104 @@ export function postgresApiKeyStore(config: PostgresApiKeyStoreConfig): ApiKeySt
   return {
     name: 'postgres',
 
-    async initialize(): Promise<void> {
-      if (!autoCreateTables) return;
+    async initialize(): Promise<StoreInitializationResult> {
+      if (!autoCreateTables) {
+        return { success: true };
+      }
 
-      const pool = getPool();
+      try {
+        const pool = getPool();
 
-      // Create table with foreign key to users
-      await pool.query(`
-        CREATE TABLE IF NOT EXISTS ${tableFullName} (
-          id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
-          user_id UUID NOT NULL REFERENCES "public"."users"(id) ON DELETE CASCADE,
-          name VARCHAR(255) NOT NULL,
-          key_hash VARCHAR(64) NOT NULL,
-          key_prefix VARCHAR(12) NOT NULL,
-          key_type VARCHAR(10) NOT NULL CHECK (key_type IN ('m2m', 'pat')),
-          scopes TEXT[] NOT NULL DEFAULT '{}',
-          last_used_at TIMESTAMPTZ,
-          expires_at TIMESTAMPTZ,
-          is_active BOOLEAN NOT NULL DEFAULT true,
-          created_at TIMESTAMPTZ DEFAULT NOW(),
-          updated_at TIMESTAMPTZ DEFAULT NOW()
-        );
+        // Create table with foreign key to users
+        await pool.query(`
+          CREATE TABLE IF NOT EXISTS ${tableFullName} (
+            id UUID PRIMARY KEY DEFAULT gen_random_uuid(),
+            user_id UUID NOT NULL REFERENCES "public"."users"(id) ON DELETE CASCADE,
+            name VARCHAR(255) NOT NULL,
+            key_hash VARCHAR(64) NOT NULL,
+            key_prefix VARCHAR(12) NOT NULL,
+            key_type VARCHAR(10) NOT NULL CHECK (key_type IN ('m2m', 'pat')),
+            scopes TEXT[] NOT NULL DEFAULT '{}',
+            last_used_at TIMESTAMPTZ,
+            expires_at TIMESTAMPTZ,
+            is_active BOOLEAN NOT NULL DEFAULT true,
+            created_at TIMESTAMPTZ DEFAULT NOW(),
+            updated_at TIMESTAMPTZ DEFAULT NOW()
+          );
+
+          CREATE INDEX IF NOT EXISTS idx_${tableName}_user_id ON ${tableFullName}(user_id);
+          CREATE INDEX IF NOT EXISTS idx_${tableName}_key_prefix ON ${tableFullName}(key_prefix);
+          CREATE INDEX IF NOT EXISTS idx_${tableName}_key_hash ON ${tableFullName}(key_hash);
+        `);
 
-        CREATE INDEX IF NOT EXISTS idx_${tableName}_user_id ON ${tableFullName}(user_id);
-        CREATE INDEX IF NOT EXISTS idx_${tableName}_key_prefix ON ${tableFullName}(key_prefix);
-        CREATE INDEX IF NOT EXISTS idx_${tableName}_key_hash ON ${tableFullName}(key_hash);
-      `);
-
-      // Migration: Add user_id column if it doesn't exist (for existing installations)
-      await pool.query(`
-        DO $$
-        DECLARE
-          row_count INTEGER;
-        BEGIN
-          IF NOT EXISTS (
-            SELECT 1 FROM information_schema.columns
-            WHERE table_schema = '${schema}'
-            AND table_name = '${tableName}'
-            AND column_name = 'user_id'
-          ) THEN
-            -- Check if table has data
-            EXECUTE 'SELECT COUNT(*) FROM ${tableFullName}' INTO row_count;
-
-            IF row_count > 0 THEN
-              -- If table has data, cannot add NOT NULL column
-              -- This requires manual migration or data cleanup
-              RAISE EXCEPTION 'Cannot add user_id column: table ${tableFullName} contains % rows. Please migrate data or clear the table first.', row_count;
-            ELSE
-              -- Table is empty, safe to add NOT NULL column
+        // Migration: Check if table needs user_id column migration
+        const needsMigration = await pool.query(`
+          SELECT
+            NOT EXISTS (
+              SELECT 1 FROM information_schema.columns
+              WHERE table_schema = '${schema}'
+              AND table_name = '${tableName}'
+              AND column_name = 'user_id'
+            ) AS missing_column,
+            (SELECT COUNT(*) FROM ${tableFullName}) AS row_count
+        `);
+
+        const { missing_column, row_count } = needsMigration.rows[0] as { missing_column: boolean; row_count: number };
+
+        if (missing_column) {
+          if (row_count > 0) {
+            // Table has data but is missing user_id column - requires maintenance
+            logger.warn(
+              `Table ${tableFullName} has ${row_count} rows but is missing user_id column. ` +
+              `Requires maintenance action to migrate or clear data.`
+            );
+            return {
+              success: false,
+              error: `Table ${tableFullName} contains ${row_count} rows without user_id column. Migration required.`,
+              requiresMaintenance: true,
+            };
+          } else {
+            // Table is empty, safe to add NOT NULL column
+            await pool.query(`
               ALTER TABLE ${tableFullName}
               ADD COLUMN user_id UUID NOT NULL REFERENCES "public"."users"(id) ON DELETE CASCADE;
 
               CREATE INDEX IF NOT EXISTS idx_${tableName}_user_id ON ${tableFullName}(user_id);
-            END IF;
-          END IF;
-        END $$;
-      `);
-
-      // Enable RLS if configured
-      if (enableRLS) {
-        await pool.query(`
-          ALTER TABLE ${tableFullName} ENABLE ROW LEVEL SECURITY;
-          ALTER TABLE ${tableFullName} FORCE ROW LEVEL SECURITY;
-        `);
+            `);
+            logger.info(`Added user_id column to empty table ${tableFullName}`);
+          }
+        }
 
-        // Create or replace the RLS policy
-        await pool.query(`
-          DROP POLICY IF EXISTS "${tableName}_owner" ON ${tableFullName};
-        `);
+        // Enable RLS if configured
+        if (enableRLS) {
+          await pool.query(`
+            ALTER TABLE ${tableFullName} ENABLE ROW LEVEL SECURITY;
+            ALTER TABLE ${tableFullName} FORCE ROW LEVEL SECURITY;
+          `);
+
+          // Create or replace the RLS policy
+          await pool.query(`
+            DROP POLICY IF EXISTS "${tableName}_owner" ON ${tableFullName};
+          `);
+
+          // RLS policy: users can only access their own keys
+          await pool.query(`
+            CREATE POLICY "${tableName}_owner" ON ${tableFullName}
+              FOR ALL
+              USING (user_id::text = current_setting('app.current_user_id', true))
+              WITH CHECK (user_id::text = current_setting('app.current_user_id', true));
+          `);
+        }
 
-        // RLS policy: users can only access their own keys
-        await pool.query(`
-          CREATE POLICY "${tableName}_owner" ON ${tableFullName}
-            FOR ALL
-            USING (user_id::text = current_setting('app.current_user_id', true))
-            WITH CHECK (user_id::text = current_setting('app.current_user_id', true));
-        `);
+        logger.info(`API keys store initialized successfully (table: ${tableFullName})`);
+        return { success: true };
+      } catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logger.error(`Failed to initialize API keys store: ${errorMessage}`, { error });
+        return {
+          success: false,
+          error: `Database initialization failed: ${errorMessage}`,
+        };
       }
     },
 

package/src/plugins/api-keys/types.ts

@@ -140,6 +140,18 @@ export interface ApiKeyWithPlaintext extends ApiKey {
   plaintext_key: string;
 }
 
+/**
+ * Result of store initialization
+ */
+export interface StoreInitializationResult {
+  /** Whether initialization succeeded */
+  success: boolean;
+  /** Error message if initialization failed */
+  error?: string;
+  /** Additional details about what needs to be done */
+  requiresMaintenance?: boolean;
+}
+
 /**
  * API key store interface - all storage backends must implement this
  */
@@ -149,8 +161,9 @@ export interface ApiKeyStore {
 
   /**
    * Initialize the store (create tables, RLS policies, etc.)
+   * Returns initialization status instead of throwing
    */
-  initialize(): Promise<void>;
+  initialize(): Promise<StoreInitializationResult>;
 
   /**
    * Create a new API key

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');
     },
 

package/src/plugins/index.ts

@@ -14,7 +14,7 @@ export { createLogsPlugin } from './logs-plugin.js';
 export type { LogsPluginConfig } from './logs-plugin.js';
 
 export { createMaintenancePlugin } from './maintenance-plugin.js';
-export type { MaintenancePluginConfig } from './maintenance-plugin.js';
+export type { MaintenancePluginConfig, SeedTask, SeedTaskHandler } from './maintenance-plugin.js';
 export {
   MaintenanceManagementPage,
   MaintenanceStatusWidget,

package/src/plugins/maintenance/SeedExecutor.tsx

@@ -11,6 +11,8 @@ import React, { useEffect, useRef, useState } from 'react';
 export interface SeedExecutorProps {
   apiPrefix: string;
   seedName: string;
+  seedType?: string;
+  seedOptions?: any;
   onComplete: () => void;
   onCancel: () => void;
 }
@@ -26,6 +28,8 @@ interface OutputLine {
 export const SeedExecutor: React.FC<SeedExecutorProps> = ({
   apiPrefix,
   seedName,
+  seedType = 'file',
+  seedOptions,
   onComplete,
   onCancel,
 }) => {
@@ -63,7 +67,11 @@ export const SeedExecutor: React.FC<SeedExecutorProps> = ({
         headers: {
           'Content-Type': 'application/json',
         },
-        body: JSON.stringify({ name: seedName }),
+        body: JSON.stringify({
+          name: seedName,
+          type: seedType,
+          options: seedOptions,
+        }),
       });
 
       if (!response.ok) {

package/src/plugins/maintenance/SeedList.tsx

@@ -1,7 +1,7 @@
 /**
  * Seed List Component
  *
- * Displays available seed scripts with metadata.
+ * Displays available seed scripts and custom tasks with metadata.
  *
  * Copyright (c) 2025 QwickApps.com. All rights reserved.
  */
@@ -10,10 +10,11 @@ import React, { useEffect, useState } from 'react';
 
 export interface SeedListProps {
   apiPrefix: string;
-  onExecute: (seedName: string) => void;
+  onExecute: (seedName: string, type?: string, options?: any) => void;
 }
 
 interface SeedFile {
+  type: 'file';
   name: string;
   path: string;
   size: number;
@@ -21,8 +22,18 @@ interface SeedFile {
   modifiedAt: string;
 }
 
+interface CustomTask {
+  type: 'task';
+  id: string;
+  name: string;
+  description: string;
+  options?: Record<string, any>;
+}
+
+type SeedItem = SeedFile | CustomTask;
+
 export const SeedList: React.FC<SeedListProps> = ({ apiPrefix, onExecute }) => {
-  const [seeds, setSeeds] = useState<SeedFile[]>([]);
+  const [seeds, setSeeds] = useState<SeedItem[]>([]);
   const [loading, setLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
 
@@ -57,6 +68,14 @@ export const SeedList: React.FC<SeedListProps> = ({ apiPrefix, onExecute }) => {
     return new Date(dateString).toLocaleString();
   };
 
+  const isFileType = (item: SeedItem): item is SeedFile => {
+    return item.type === 'file';
+  };
+
+  const isTaskType = (item: SeedItem): item is CustomTask => {
+    return item.type === 'task';
+  };
+
   if (loading) {
     return <div style={{ padding: '20px' }}>Loading seeds...</div>;
   }
@@ -72,55 +91,83 @@ export const SeedList: React.FC<SeedListProps> = ({ apiPrefix, onExecute }) => {
   if (seeds.length === 0) {
     return (
       <div style={{ padding: '20px', color: '#666' }}>
-        No seed scripts found in scripts directory.
+        No seed scripts or tasks found.
       </div>
     );
   }
 
   return (
     <div style={{ padding: '20px' }}>
-      <h3>Available Seed Scripts ({seeds.length})</h3>
+      <h3>Available Seeds & Tasks ({seeds.length})</h3>
       <table style={{ width: '100%', borderCollapse: 'collapse', marginTop: '16px' }}>
         <thead>
           <tr style={{ borderBottom: '2px solid #ddd', textAlign: 'left' }}>
+            <th style={{ padding: '12px' }}>Type</th>
             <th style={{ padding: '12px' }}>Name</th>
-            <th style={{ padding: '12px' }}>Size</th>
-            <th style={{ padding: '12px' }}>Modified</th>
+            <th style={{ padding: '12px' }}>Description</th>
+            <th style={{ padding: '12px' }}>Details</th>
             <th style={{ padding: '12px' }}>Action</th>
           </tr>
         </thead>
         <tbody>
-          {seeds.map((seed) => (
-            <tr key={seed.name} style={{ borderBottom: '1px solid #eee' }}>
-              <td style={{ padding: '12px', fontFamily: 'monospace' }}>
-                {seed.name}
-              </td>
-              <td style={{ padding: '12px' }}>{formatFileSize(seed.size)}</td>
-              <td style={{ padding: '12px', fontSize: '14px', color: '#666' }}>
-                {formatDate(seed.modifiedAt)}
-              </td>
-              <td style={{ padding: '12px' }}>
-                <button
-                  onClick={() => {
-                    if (confirm(`Execute ${seed.name}?`)) {
-                      onExecute(seed.name);
-                    }
-                  }}
-                  style={{
-                    padding: '6px 12px',
-                    backgroundColor: '#1976d2',
-                    color: 'white',
-                    border: 'none',
-                    borderRadius: '4px',
-                    cursor: 'pointer',
-                  }}
-                  data-testid={`execute-${seed.name}`}
-                >
-                  Execute
-                </button>
-              </td>
-            </tr>
-          ))}
+          {seeds.map((seed) => {
+            const itemKey = isFileType(seed) ? seed.name : seed.id;
+            return (
+              <tr key={itemKey} style={{ borderBottom: '1px solid #eee' }}>
+                <td style={{ padding: '12px' }}>
+                  <span
+                    style={{
+                      display: 'inline-block',
+                      padding: '4px 8px',
+                      borderRadius: '4px',
+                      fontSize: '12px',
+                      fontWeight: 'bold',
+                      backgroundColor: isFileType(seed) ? '#e3f2fd' : '#f3e5f5',
+                      color: isFileType(seed) ? '#1976d2' : '#7b1fa2',
+                    }}
+                  >
+                    {isFileType(seed) ? 'FILE' : 'TASK'}
+                  </span>
+                </td>
+                <td style={{ padding: '12px', fontFamily: 'monospace' }}>
+                  {isFileType(seed) ? seed.name : seed.name}
+                </td>
+                <td style={{ padding: '12px', fontSize: '14px', color: '#666' }}>
+                  {isTaskType(seed) ? seed.description : '-'}
+                </td>
+                <td style={{ padding: '12px', fontSize: '14px', color: '#666' }}>
+                  {isFileType(seed)
+                    ? `${formatFileSize(seed.size)} • ${formatDate(seed.modifiedAt)}`
+                    : '-'}
+                </td>
+                <td style={{ padding: '12px' }}>
+                  <button
+                    onClick={() => {
+                      const displayName = isFileType(seed) ? seed.name : seed.name;
+                      if (confirm(`Execute ${displayName}?`)) {
+                        if (isFileType(seed)) {
+                          onExecute(seed.name, 'file');
+                        } else {
+                          onExecute(seed.id, 'task', seed.options);
+                        }
+                      }
+                    }}
+                    style={{
+                      padding: '6px 12px',
+                      backgroundColor: '#1976d2',
+                      color: 'white',
+                      border: 'none',
+                      borderRadius: '4px',
+                      cursor: 'pointer',
+                    }}
+                    data-testid={`execute-${itemKey}`}
+                  >
+                    Execute
+                  </button>
+                </td>
+              </tr>
+            );
+          })}
         </tbody>
       </table>
     </div>

package/src/plugins/maintenance/SeedManagementPage.tsx

@@ -1,7 +1,7 @@
 /**
  * Seed Management Page
  *
- * Main page for managing and executing seed scripts.
+ * Main page for managing and executing seed scripts and custom tasks.
  *
  * Copyright (c) 2025 QwickApps.com. All rights reserved.
  */
@@ -23,9 +23,13 @@ export const SeedManagementPage: React.FC<SeedManagementPageProps> = ({
 }) => {
   const [activeTab, setActiveTab] = useState<Tab>('list');
   const [selectedSeed, setSelectedSeed] = useState<string | null>(null);
+  const [selectedType, setSelectedType] = useState<string>('file');
+  const [selectedOptions, setSelectedOptions] = useState<any>(undefined);
 
-  const handleExecute = (seedName: string) => {
+  const handleExecute = (seedName: string, type: string = 'file', options?: any) => {
     setSelectedSeed(seedName);
+    setSelectedType(type);
+    setSelectedOptions(options);
     setActiveTab('execute');
   };
 
@@ -36,7 +40,7 @@ export const SeedManagementPage: React.FC<SeedManagementPageProps> = ({
   return (
     <PluginManagementPage
       title="Seed Management"
-      description="Manage and execute database seed scripts"
+      description="Manage and execute database seed scripts and custom tasks"
     >
       <div style={{ marginBottom: '20px' }}>
         <button
@@ -51,7 +55,7 @@ export const SeedManagementPage: React.FC<SeedManagementPageProps> = ({
             cursor: 'pointer',
           }}
         >
-          Available Seeds
+          Available Seeds & Tasks
         </button>
         <button
           onClick={() => setActiveTab('history')}
@@ -76,6 +80,8 @@ export const SeedManagementPage: React.FC<SeedManagementPageProps> = ({
         <SeedExecutor
           apiPrefix={apiPrefix}
           seedName={selectedSeed}
+          seedType={selectedType}
+          seedOptions={selectedOptions}
           onComplete={handleExecutionComplete}
           onCancel={() => setActiveTab('list')}
         />