@open-mercato/core 0.4.2-canary-ed15f2e753 → 0.4.2-canary-e2aeb1a7bf

package/src/modules/workflows/lib/transition-handler.ts

@@ -12,6 +12,7 @@
 
 import { EntityManager } from '@mikro-orm/core'
 import type { AwilixContainer } from 'awilix'
+import type { EventBus } from '@open-mercato/events'
 import {
   WorkflowInstance,
   WorkflowDefinition,
@@ -19,6 +20,7 @@ import {
 } from '../data/entities'
 import * as ruleEvaluator from '../../business_rules/lib/rule-evaluator'
 import * as ruleEngine from '../../business_rules/lib/rule-engine'
+import type { RuleEngineContext } from '../../business_rules/lib/rule-engine'
 import * as activityExecutor from './activity-executor'
 import type { ActivityDefinition } from './activity-executor'
 import * as stepHandler from './step-handler'
@@ -193,15 +195,11 @@ export async function evaluateTransition(
 /**
  * Find all valid transitions from current step
  *
- * This function evaluates both inline conditions AND preConditions (business rules)
- * to determine which transitions are truly valid. This is important for decision
- * branching where multiple transitions exist with different preConditions.
- *
  * @param em - Entity manager
  * @param instance - Workflow instance
  * @param fromStepId - Current step ID
  * @param context - Evaluation context
- * @returns Array of evaluation results for all transitions, sorted by priority (desc)
+ * @returns Array of evaluation results for all transitions
  */
 export async function findValidTransitions(
   em: EntityManager,
@@ -219,17 +217,16 @@ export async function findValidTransitions(
       return []
     }
 
-    // Find all transitions from current step, sorted by priority (highest first)
-    const transitions = (definition.definition.transitions || [])
-      .filter((t: any) => t.fromStepId === fromStepId)
-      .sort((a: any, b: any) => (b.priority || 0) - (a.priority || 0))
+    // Find all transitions from current step
+    const transitions = (definition.definition.transitions || []).filter(
+      (t: any) => t.fromStepId === fromStepId
+    )
 
-    // Evaluate each transition including preConditions
+    // Evaluate each transition
     const results: TransitionEvaluationResult[] = []
 
     for (const transition of transitions) {
-      // First check inline condition
-      const conditionResult = await evaluateTransition(
+      const result = await evaluateTransition(
         em,
         instance,
         fromStepId,
@@ -237,42 +234,7 @@ export async function findValidTransitions(
         context
       )
 
-      if (!conditionResult.isValid) {
-        results.push(conditionResult)
-        continue
-      }
-
-      // Also evaluate preConditions if they exist
-      const preConditions = transition.preConditions || []
-      if (preConditions.length > 0) {
-        const preConditionsResult = await evaluatePreConditions(
-          em,
-          instance,
-          transition,
-          context as TransitionExecutionContext
-        )
-
-        if (!preConditionsResult.allowed) {
-          // Transition is invalid due to preConditions
-          const failedRules = preConditionsResult.executedRules
-            .filter((r) => !r.conditionResult)
-            .map((r) => r.rule.ruleId || r.rule.ruleName)
-
-          results.push({
-            isValid: false,
-            transition,
-            reason: `Pre-conditions failed: ${failedRules.join(', ')}`,
-            failedConditions: failedRules,
-          })
-          continue
-        }
-      }
-
-      // Transition is valid (both condition and preConditions passed)
-      results.push({
-        ...conditionResult,
-        transition,
-      })
+      results.push(result)
     }
 
     return results
@@ -310,6 +272,13 @@ export async function executeTransition(
   context: TransitionExecutionContext
 ): Promise<TransitionExecutionResult> {
   try {
+    let eventBus: Pick<EventBus, 'emitEvent'> | null = null
+    try {
+      eventBus = container.resolve('eventBus') as EventBus
+    } catch {
+      eventBus = null
+    }
+
     // First, evaluate if transition is valid
     const evaluation = await evaluateTransition(
       em,
@@ -333,7 +302,8 @@ export async function executeTransition(
       em,
       instance,
       transition,
-      context
+      context,
+      eventBus
     )
 
     if (!preConditionsResult.allowed) {
@@ -544,7 +514,8 @@ export async function executeTransition(
       em,
       instance,
       transition,
-      context
+      context,
+      eventBus
     )
 
     if (!postConditionsResult.allowed) {
@@ -688,10 +659,6 @@ async function evaluateTransitionConditions(
  * Pre-conditions are GUARD rules that must pass before transition can execute.
  * If any GUARD rule fails, the transition is blocked.
  *
- * If the transition defines specific preConditions with ruleIds, those are
- * executed directly via executeRuleByRuleId. Otherwise, falls back to
- * discovery-based execution via executeRules.
- *
  * @param em - Entity manager
  * @param instance - Workflow instance
  * @param transition - Transition definition
@@ -702,7 +669,8 @@ async function evaluatePreConditions(
   em: EntityManager,
   instance: WorkflowInstance,
   transition: any,
-  context: TransitionExecutionContext
+  context: TransitionExecutionContext,
+  eventBus: Pick<EventBus, 'emitEvent'> | null
 ): Promise<ruleEngine.RuleEngineResult> {
   try {
     // Load workflow definition to get workflow ID
@@ -718,88 +686,32 @@ async function evaluatePreConditions(
       }
     }
 
-    // Check if transition has specific preConditions defined
-    const preConditions = transition.preConditions || []
-
-    // If no pre-conditions defined, allow transition
-    if (preConditions.length === 0) {
-      return {
-        allowed: true,
-        executedRules: [],
-        totalExecutionTime: 0,
-      }
-    }
-
-    // Execute each pre-condition rule directly by ruleId
-    const startTime = Date.now()
-    const executedRules: ruleEngine.RuleExecutionResult[] = []
-    const errors: string[] = []
-    let allowed = true
-
-    for (const condition of preConditions) {
-      const result = await ruleEngine.executeRuleByRuleId(em, {
-        ruleId: condition.ruleId,  // String identifier
-        data: {
-          workflowInstanceId: instance.id,
-          workflowId: definition.workflowId,
-          fromStepId: transition.fromStepId,
-          toStepId: transition.toStepId,
-          workflowContext: {
-            ...instance.context,
-            ...context.workflowContext,
-          },
-          triggerData: context.triggerData,
+    // Build rule engine context
+    const ruleContext: RuleEngineContext = {
+      entityType: `workflow:${definition.workflowId}:transition`,
+      entityId: transition.transitionId || `${transition.fromStepId}->${transition.toStepId}`,
+      eventType: 'pre_transition',
+      data: {
+        workflowInstanceId: instance.id,
+        workflowId: definition.workflowId,
+        fromStepId: transition.fromStepId,
+        toStepId: transition.toStepId,
+        workflowContext: {
+          ...instance.context,
+          ...context.workflowContext,
         },
-        user: context.userId ? { id: context.userId } : undefined,
-        tenantId: instance.tenantId,
-        organizationId: instance.organizationId,
-        executedBy: context.userId,
-        entityType: `workflow:${definition.workflowId}:transition`,
-        entityId: transition.transitionId || `${transition.fromStepId}->${transition.toStepId}`,
-        eventType: 'pre_transition',
-      })
-
-      // Create a compatible RuleExecutionResult for tracking
-      // We don't have the full BusinessRule entity, but we can create a partial result
-      const ruleResult: ruleEngine.RuleExecutionResult = {
-        rule: {
-          ruleId: result.ruleId,
-          ruleName: result.ruleName,
-          ruleType: 'GUARD',
-        } as any,
-        conditionResult: result.conditionResult,
-        actionsExecuted: result.actionsExecuted,
-        executionTime: result.executionTime,
-        error: result.error,
-        logId: result.logId,
-      }
-      executedRules.push(ruleResult)
-
-      // Handle rule errors
-      if (result.error) {
-        // Rule not found, disabled, or other errors
-        const isRequired = condition.required !== false  // Default to required
-        if (isRequired) {
-          allowed = false
-          errors.push(`Rule '${result.ruleId}': ${result.error}`)
-        }
-        continue
-      }
-
-      // If required and condition failed, block transition
-      const isRequired = condition.required !== false  // Default to required
-      if (isRequired && !result.conditionResult) {
-        allowed = false
-        errors.push(`Pre-condition '${result.ruleName || result.ruleId}' failed`)
-      }
+        triggerData: context.triggerData,
+      },
+      user: context.userId ? { id: context.userId } : undefined,
+      tenantId: instance.tenantId,
+      organizationId: instance.organizationId,
+      executedBy: context.userId,
     }
 
-    return {
-      allowed,
-      executedRules,
-      totalExecutionTime: Date.now() - startTime,
-      errors: errors.length > 0 ? errors : undefined,
-    }
+    // Execute rules - only GUARD rules will affect the 'allowed' status
+    const result = await ruleEngine.executeRules(em, ruleContext, { eventBus })
+
+    return result
   } catch (error) {
     console.error('Error evaluating pre-conditions:', error)
     return {
@@ -817,9 +729,6 @@ async function evaluatePreConditions(
  * Post-conditions are GUARD rules that should pass after transition executes.
  * Unlike pre-conditions, post-condition failures are logged but don't block the transition.
  *
- * If the transition defines specific postConditions with ruleIds, those are
- * executed directly via executeRuleByRuleId. Otherwise, returns allowed: true.
- *
  * @param em - Entity manager
  * @param instance - Workflow instance
  * @param transition - Transition definition
@@ -830,7 +739,8 @@ async function evaluatePostConditions(
   em: EntityManager,
   instance: WorkflowInstance,
   transition: any,
-  context: TransitionExecutionContext
+  context: TransitionExecutionContext,
+  eventBus: Pick<EventBus, 'emitEvent'> | null
 ): Promise<ruleEngine.RuleEngineResult> {
   try {
     // Load workflow definition to get workflow ID
@@ -846,83 +756,32 @@ async function evaluatePostConditions(
       }
     }
 
-    // Check if transition has specific postConditions defined
-    const postConditions = transition.postConditions || []
-
-    // If no post-conditions defined, allow
-    if (postConditions.length === 0) {
-      return {
-        allowed: true,
-        executedRules: [],
-        totalExecutionTime: 0,
-      }
-    }
-
-    // Execute each post-condition rule directly by ruleId
-    const startTime = Date.now()
-    const executedRules: ruleEngine.RuleExecutionResult[] = []
-    const errors: string[] = []
-    let allowed = true
-
-    for (const condition of postConditions) {
-      const result = await ruleEngine.executeRuleByRuleId(em, {
-        ruleId: condition.ruleId,  // String identifier
-        data: {
-          workflowInstanceId: instance.id,
-          workflowId: definition.workflowId,
-          fromStepId: transition.fromStepId,
-          toStepId: transition.toStepId,
-          workflowContext: {
-            ...instance.context,
-            ...context.workflowContext,
-          },
-          triggerData: context.triggerData,
+    // Build rule engine context
+    const ruleContext: RuleEngineContext = {
+      entityType: `workflow:${definition.workflowId}:transition`,
+      entityId: transition.transitionId || `${transition.fromStepId}->${transition.toStepId}`,
+      eventType: 'post_transition',
+      data: {
+        workflowInstanceId: instance.id,
+        workflowId: definition.workflowId,
+        fromStepId: transition.fromStepId,
+        toStepId: transition.toStepId,
+        workflowContext: {
+          ...instance.context,
+          ...context.workflowContext,
         },
-        user: context.userId ? { id: context.userId } : undefined,
-        tenantId: instance.tenantId,
-        organizationId: instance.organizationId,
-        executedBy: context.userId,
-        entityType: `workflow:${definition.workflowId}:transition`,
-        entityId: transition.transitionId || `${transition.fromStepId}->${transition.toStepId}`,
-        eventType: 'post_transition',
-      })
-
-      // Create a compatible RuleExecutionResult for tracking
-      const ruleResult: ruleEngine.RuleExecutionResult = {
-        rule: {
-          ruleId: result.ruleId,
-          ruleName: result.ruleName,
-          ruleType: 'GUARD',
-        } as any,
-        conditionResult: result.conditionResult,
-        actionsExecuted: result.actionsExecuted,
-        executionTime: result.executionTime,
-        error: result.error,
-        logId: result.logId,
-      }
-      executedRules.push(ruleResult)
-
-      // Handle rule errors
-      if (result.error) {
-        errors.push(`Rule '${result.ruleId}': ${result.error}`)
-        // Post-conditions don't block, but track the failure
-        allowed = false
-        continue
-      }
-
-      // Track condition failures (post-conditions are warnings, not blockers)
-      if (!result.conditionResult) {
-        allowed = false
-        errors.push(`Post-condition '${result.ruleName || result.ruleId}' failed`)
-      }
+        triggerData: context.triggerData,
+      },
+      user: context.userId ? { id: context.userId } : undefined,
+      tenantId: instance.tenantId,
+      organizationId: instance.organizationId,
+      executedBy: context.userId,
     }
 
-    return {
-      allowed,
-      executedRules,
-      totalExecutionTime: Date.now() - startTime,
-      errors: errors.length > 0 ? errors : undefined,
-    }
+    // Execute rules
+    const result = await ruleEngine.executeRules(em, ruleContext, { eventBus })
+
+    return result
   } catch (error) {
     console.error('Error evaluating post-conditions:', error)
     return {

package/src/modules/workflows/migrations/Migration20251207131955.ts

@@ -2,150 +2,84 @@ import { Migration } from '@mikro-orm/migrations';
 
 export class Migration20251207131955 extends Migration {
 
-  override async up(): Promise<void> {
-    this.addSql(`alter table "rule_execution_logs" drop constraint "rule_execution_logs_rule_id_foreign";`);
-
-    this.addSql(`alter table "rule_set_members" drop constraint "rule_set_members_rule_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_prices" drop constraint "catalog_product_variant_prices_price_kind_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_category_assignments" drop constraint "catalog_product_category_assignments_category_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_prices" drop constraint "catalog_product_variant_prices_offer_id_foreign";`);
-
-    this.addSql(`alter table "catalog_products" drop constraint "catalog_products_option_schema_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_tag_assignments" drop constraint "catalog_product_tag_assignments_tag_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_option_values" drop constraint "catalog_product_variant_option_values_variant_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_prices" drop constraint "catalog_product_variant_prices_variant_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_relations" drop constraint "catalog_product_variant_relations_child_variant_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_relations" drop constraint "catalog_product_variant_relations_parent_variant_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_category_assignments" drop constraint "catalog_product_category_assignments_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_offers" drop constraint "catalog_product_offers_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_options" drop constraint "catalog_product_options_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_relations" drop constraint "catalog_product_relations_child_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_relations" drop constraint "catalog_product_relations_parent_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_tag_assignments" drop constraint "catalog_product_tag_assignments_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variant_prices" drop constraint "catalog_product_variant_prices_product_id_foreign";`);
-
-    this.addSql(`alter table "catalog_product_variants" drop constraint "catalog_product_variants_product_id_foreign";`);
-
-    this.addSql(`alter table "customer_activities" drop constraint "customer_activities_deal_id_foreign";`);
-
-    this.addSql(`alter table "customer_comments" drop constraint "customer_comments_deal_id_foreign";`);
-
-    this.addSql(`alter table "customer_deal_companies" drop constraint "customer_deal_companies_deal_id_foreign";`);
-
-    this.addSql(`alter table "customer_deal_people" drop constraint "customer_deal_people_deal_id_foreign";`);
-
-    this.addSql(`alter table "customer_activities" drop constraint "customer_activities_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_addresses" drop constraint "customer_addresses_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_comments" drop constraint "customer_comments_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_companies" drop constraint "customer_companies_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_deal_companies" drop constraint "customer_deal_companies_company_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_deal_people" drop constraint "customer_deal_people_person_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_people" drop constraint "customer_people_company_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_people" drop constraint "customer_people_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_tag_assignments" drop constraint "customer_tag_assignments_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_todo_links" drop constraint "customer_todo_links_entity_id_foreign";`);
-
-    this.addSql(`alter table "customer_tag_assignments" drop constraint "customer_tag_assignments_tag_id_foreign";`);
-
-    this.addSql(`alter table "dictionary_entries" drop constraint "dictionary_entries_dictionary_id_foreign";`);
-
-    this.addSql(`alter table "role_acls" drop constraint "role_acls_role_id_foreign";`);
-
-    this.addSql(`alter table "role_sidebar_preferences" drop constraint "role_sidebar_preferences_role_id_foreign";`);
-
-    this.addSql(`alter table "user_roles" drop constraint "user_roles_role_id_foreign";`);
-
-    this.addSql(`alter table "rule_set_members" drop constraint "rule_set_members_rule_set_id_foreign";`);
-
-    this.addSql(`alter table "sales_orders" drop constraint "sales_orders_channel_ref_id_foreign";`);
-
-    this.addSql(`alter table "sales_credit_memo_lines" drop constraint "sales_credit_memo_lines_credit_memo_id_foreign";`);
-
-    this.addSql(`alter table "sales_orders" drop constraint "sales_orders_delivery_window_ref_id_foreign";`);
-
-    this.addSql(`alter table "sales_credit_memos" drop constraint "sales_credit_memos_invoice_id_foreign";`);
-
-    this.addSql(`alter table "sales_invoice_lines" drop constraint "sales_invoice_lines_invoice_id_foreign";`);
-
-    this.addSql(`alter table "sales_payment_allocations" drop constraint "sales_payment_allocations_invoice_id_foreign";`);
-
-    this.addSql(`alter table "sales_credit_memo_lines" drop constraint "sales_credit_memo_lines_order_line_id_foreign";`);
-
-    this.addSql(`alter table "sales_invoice_lines" drop constraint "sales_invoice_lines_order_line_id_foreign";`);
-
-    this.addSql(`alter table "sales_order_adjustments" drop constraint "sales_order_adjustments_order_line_id_foreign";`);
-
-    this.addSql(`alter table "sales_shipment_items" drop constraint "sales_shipment_items_order_line_id_foreign";`);
-
-    this.addSql(`alter table "sales_credit_memos" drop constraint "sales_credit_memos_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_invoices" drop constraint "sales_invoices_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_notes" drop constraint "sales_notes_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_order_adjustments" drop constraint "sales_order_adjustments_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_order_lines" drop constraint "sales_order_lines_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_payment_allocations" drop constraint "sales_payment_allocations_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_payments" drop constraint "sales_payments_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_shipments" drop constraint "sales_shipments_order_id_foreign";`);
-
-    this.addSql(`alter table "sales_orders" drop constraint "sales_orders_payment_method_ref_id_foreign";`);
-
-    this.addSql(`alter table "sales_payments" drop constraint "sales_payments_payment_method_id_foreign";`);
-
-    this.addSql(`alter table "sales_payment_allocations" drop constraint "sales_payment_allocations_payment_id_foreign";`);
-
-    this.addSql(`alter table "sales_quote_adjustments" drop constraint "sales_quote_adjustments_quote_line_id_foreign";`);
-
-    this.addSql(`alter table "sales_notes" drop constraint "sales_notes_quote_id_foreign";`);
-
-    this.addSql(`alter table "sales_quote_adjustments" drop constraint "sales_quote_adjustments_quote_id_foreign";`);
-
-    this.addSql(`alter table "sales_quote_lines" drop constraint "sales_quote_lines_quote_id_foreign";`);
-
-    this.addSql(`alter table "sales_shipment_items" drop constraint "sales_shipment_items_shipment_id_foreign";`);
-
-    this.addSql(`alter table "sales_orders" drop constraint "sales_orders_shipping_method_ref_id_foreign";`);
-
-    this.addSql(`alter table "organizations" drop constraint "organizations_tenant_id_foreign";`);
-
-    this.addSql(`alter table "password_resets" drop constraint "password_resets_user_id_foreign";`);
-
-    this.addSql(`alter table "sessions" drop constraint "sessions_user_id_foreign";`);
-
-    this.addSql(`alter table "user_acls" drop constraint "user_acls_user_id_foreign";`);
-
-    this.addSql(`alter table "user_roles" drop constraint "user_roles_user_id_foreign";`);
+  /** Drop a foreign key constraint only when the owning table exists (safe for disabled modules). */
+  private dropConstraintIfTableExists(table: string, constraint: string): void {
+    this.addSql(`DO $$ BEGIN IF EXISTS (SELECT 1 FROM information_schema.tables WHERE table_name = '${table}') THEN ALTER TABLE "${table}" DROP CONSTRAINT IF EXISTS "${constraint}"; END IF; END $$;`)
+  }
 
-    this.addSql(`alter table "user_sidebar_preferences" drop constraint "user_sidebar_preferences_user_id_foreign";`);
+  override async up(): Promise<void> {
+    this.dropConstraintIfTableExists('rule_execution_logs', 'rule_execution_logs_rule_id_foreign');
+    this.dropConstraintIfTableExists('rule_set_members', 'rule_set_members_rule_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_prices', 'catalog_product_variant_prices_price_kind_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_category_assignments', 'catalog_product_category_assignments_category_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_prices', 'catalog_product_variant_prices_offer_id_foreign');
+    this.dropConstraintIfTableExists('catalog_products', 'catalog_products_option_schema_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_tag_assignments', 'catalog_product_tag_assignments_tag_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_option_values', 'catalog_product_variant_option_values_variant_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_prices', 'catalog_product_variant_prices_variant_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_relations', 'catalog_product_variant_relations_child_variant_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_relations', 'catalog_product_variant_relations_parent_variant_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_category_assignments', 'catalog_product_category_assignments_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_offers', 'catalog_product_offers_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_options', 'catalog_product_options_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_relations', 'catalog_product_relations_child_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_relations', 'catalog_product_relations_parent_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_tag_assignments', 'catalog_product_tag_assignments_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variant_prices', 'catalog_product_variant_prices_product_id_foreign');
+    this.dropConstraintIfTableExists('catalog_product_variants', 'catalog_product_variants_product_id_foreign');
+    this.dropConstraintIfTableExists('customer_activities', 'customer_activities_deal_id_foreign');
+    this.dropConstraintIfTableExists('customer_comments', 'customer_comments_deal_id_foreign');
+    this.dropConstraintIfTableExists('customer_deal_companies', 'customer_deal_companies_deal_id_foreign');
+    this.dropConstraintIfTableExists('customer_deal_people', 'customer_deal_people_deal_id_foreign');
+    this.dropConstraintIfTableExists('customer_activities', 'customer_activities_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_addresses', 'customer_addresses_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_comments', 'customer_comments_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_companies', 'customer_companies_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_deal_companies', 'customer_deal_companies_company_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_deal_people', 'customer_deal_people_person_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_people', 'customer_people_company_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_people', 'customer_people_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_tag_assignments', 'customer_tag_assignments_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_todo_links', 'customer_todo_links_entity_id_foreign');
+    this.dropConstraintIfTableExists('customer_tag_assignments', 'customer_tag_assignments_tag_id_foreign');
+    this.dropConstraintIfTableExists('dictionary_entries', 'dictionary_entries_dictionary_id_foreign');
+    this.dropConstraintIfTableExists('role_acls', 'role_acls_role_id_foreign');
+    this.dropConstraintIfTableExists('role_sidebar_preferences', 'role_sidebar_preferences_role_id_foreign');
+    this.dropConstraintIfTableExists('user_roles', 'user_roles_role_id_foreign');
+    this.dropConstraintIfTableExists('rule_set_members', 'rule_set_members_rule_set_id_foreign');
+    this.dropConstraintIfTableExists('sales_orders', 'sales_orders_channel_ref_id_foreign');
+    this.dropConstraintIfTableExists('sales_credit_memo_lines', 'sales_credit_memo_lines_credit_memo_id_foreign');
+    this.dropConstraintIfTableExists('sales_orders', 'sales_orders_delivery_window_ref_id_foreign');
+    this.dropConstraintIfTableExists('sales_credit_memos', 'sales_credit_memos_invoice_id_foreign');
+    this.dropConstraintIfTableExists('sales_invoice_lines', 'sales_invoice_lines_invoice_id_foreign');
+    this.dropConstraintIfTableExists('sales_payment_allocations', 'sales_payment_allocations_invoice_id_foreign');
+    this.dropConstraintIfTableExists('sales_credit_memo_lines', 'sales_credit_memo_lines_order_line_id_foreign');
+    this.dropConstraintIfTableExists('sales_invoice_lines', 'sales_invoice_lines_order_line_id_foreign');
+    this.dropConstraintIfTableExists('sales_order_adjustments', 'sales_order_adjustments_order_line_id_foreign');
+    this.dropConstraintIfTableExists('sales_shipment_items', 'sales_shipment_items_order_line_id_foreign');
+    this.dropConstraintIfTableExists('sales_credit_memos', 'sales_credit_memos_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_invoices', 'sales_invoices_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_notes', 'sales_notes_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_order_adjustments', 'sales_order_adjustments_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_order_lines', 'sales_order_lines_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_payment_allocations', 'sales_payment_allocations_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_payments', 'sales_payments_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_shipments', 'sales_shipments_order_id_foreign');
+    this.dropConstraintIfTableExists('sales_orders', 'sales_orders_payment_method_ref_id_foreign');
+    this.dropConstraintIfTableExists('sales_payments', 'sales_payments_payment_method_id_foreign');
+    this.dropConstraintIfTableExists('sales_payment_allocations', 'sales_payment_allocations_payment_id_foreign');
+    this.dropConstraintIfTableExists('sales_quote_adjustments', 'sales_quote_adjustments_quote_line_id_foreign');
+    this.dropConstraintIfTableExists('sales_notes', 'sales_notes_quote_id_foreign');
+    this.dropConstraintIfTableExists('sales_quote_adjustments', 'sales_quote_adjustments_quote_id_foreign');
+    this.dropConstraintIfTableExists('sales_quote_lines', 'sales_quote_lines_quote_id_foreign');
+    this.dropConstraintIfTableExists('sales_shipment_items', 'sales_shipment_items_shipment_id_foreign');
+    this.dropConstraintIfTableExists('sales_orders', 'sales_orders_shipping_method_ref_id_foreign');
+    this.dropConstraintIfTableExists('organizations', 'organizations_tenant_id_foreign');
+    this.dropConstraintIfTableExists('password_resets', 'password_resets_user_id_foreign');
+    this.dropConstraintIfTableExists('sessions', 'sessions_user_id_foreign');
+    this.dropConstraintIfTableExists('user_acls', 'user_acls_user_id_foreign');
+    this.dropConstraintIfTableExists('user_roles', 'user_roles_user_id_foreign');
+    this.dropConstraintIfTableExists('user_sidebar_preferences', 'user_sidebar_preferences_user_id_foreign');
 
     this.addSql(`create table "step_instances" ("id" uuid not null default gen_random_uuid(), "workflow_instance_id" uuid not null, "step_id" varchar(100) not null, "step_name" varchar(255) not null, "step_type" varchar(50) not null, "status" varchar(20) not null, "input_data" jsonb null, "output_data" jsonb null, "error_data" jsonb null, "entered_at" timestamptz null, "exited_at" timestamptz null, "execution_time_ms" int null, "retry_count" int not null default 0, "tenant_id" uuid not null, "organization_id" uuid not null, "created_at" timestamptz not null, "updated_at" timestamptz not null, constraint "step_instances_pkey" primary key ("id"));`);
     this.addSql(`create index "step_instances_tenant_org_idx" on "step_instances" ("tenant_id", "organization_id");`);

package/src/modules/workflows/notifications.ts

@@ -0,0 +1,25 @@
+import type { NotificationTypeDefinition } from '@open-mercato/shared/modules/notifications/types'
+
+export const notificationTypes: NotificationTypeDefinition[] = [
+  {
+    type: 'workflows.task.assigned',
+    module: 'workflows',
+    titleKey: 'workflows.notifications.task.assigned.title',
+    bodyKey: 'workflows.notifications.task.assigned.body',
+    icon: 'clipboard-list',
+    severity: 'info',
+    actions: [
+      {
+        id: 'view',
+        labelKey: 'common.view',
+        variant: 'outline',
+        href: '/backend/workflows/tasks/{sourceEntityId}',
+        icon: 'external-link',
+      },
+    ],
+    linkHref: '/backend/workflows/tasks/{sourceEntityId}',
+    expiresAfterHours: 168, // 7 days
+  },
+]
+
+export default notificationTypes

package/src/modules/workflows/setup.ts

@@ -0,0 +1,15 @@
+import type { ModuleSetupConfig } from '@open-mercato/shared/modules/setup'
+import { seedExampleWorkflows } from './lib/seeds'
+
+export const setup: ModuleSetupConfig = {
+  seedDefaults: async (ctx) => {
+    const scope = { tenantId: ctx.tenantId, organizationId: ctx.organizationId }
+    await seedExampleWorkflows(ctx.em, scope)
+  },
+
+  defaultRoleFeatures: {
+    admin: ['workflows.*'],
+  },
+}
+
+export default setup