@onlineapps/error-handler-core 1.0.0 → 1.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/error-handler-core",
-  "version": "1.0.0",
+  "version": "1.1.1",
   "description": "Core error handling functionality for OA Drive microservices - classification, retry, circuit breaker, DLQ routing",
   "main": "src/index.js",
   "scripts": {
@@ -21,6 +21,7 @@
   "license": "MIT",
   "dependencies": {
     "@onlineapps/monitoring-core": "^1.0.0",
+    "@onlineapps/runtime-config": "1.0.1",
     "opossum": "^7.0.0"
   },
   "devDependencies": {
@@ -31,4 +32,3 @@
     "access": "public"
   }
 }
-

package/src/DLQRouter.js

@@ -8,6 +8,7 @@ class DLQRouter {
     this.enabled = config.dlqEnabled !== false;
     this.mqClient = config.mqClient || null;
     this.dlqNamePattern = config.dlqNamePattern || '{queue}.dlq';
+    this.emitMonitoringEvents = config.emitMonitoringEvents !== false;
   }
   
   /**
@@ -51,6 +52,11 @@ class DLQRouter {
       await this.mqClient.publish(dlqName, dlqMessage, {
         persistent: true
       });
+      
+      // Emit message_dlq monitoring event
+      if (this.emitMonitoringEvents) {
+        await this._emitDLQEvent(message, dlqName, originalQueue, error, errorType, context);
+      }
     } catch (dlqError) {
       // If DLQ routing fails, log but don't throw
       // (we don't want DLQ failures to break error handling)
@@ -58,6 +64,46 @@ class DLQRouter {
     }
   }
   
+  /**
+   * Emit message_dlq event to monitoring
+   * @private
+   */
+  async _emitDLQEvent(message, dlqName, originalQueue, error, errorType, context) {
+    if (!this.mqClient) return;
+    
+    try {
+      const workflowId = message.workflow_id || context.workflow_id || 'unknown';
+      
+      await this.mqClient.publish('monitoring.workflow', {
+        event_type: 'message_dlq',
+        workflow_id: workflowId,
+        service_name: 'dlq-router',
+        step_id: 'dlq-entry',
+        timestamp: new Date().toISOString(),
+        context: {
+          _dlq: {
+            queue: dlqName,
+            originalQueue: originalQueue,
+            reason: error.message,
+            errorType: errorType,
+            retryCount: message.retryCount || context.retryCount || 0,
+            routedAt: new Date().toISOString()
+          }
+        },
+        metadata: {
+          dlq_queue: dlqName,
+          original_queue: originalQueue,
+          error_type: errorType
+        }
+      }, { persistent: true });
+      
+      console.log(`[DLQRouter] ✓ Emitted message_dlq event for ${workflowId}`);
+    } catch (emitError) {
+      // Silent fail - don't break DLQ routing
+      console.warn(`[DLQRouter] Failed to emit monitoring event:`, emitError.message);
+    }
+  }
+  
   /**
    * Generate DLQ name from original queue name
    * @private

package/src/ErrorHandlerCore.js

@@ -9,6 +9,7 @@ const RetryHandler = require('./RetryHandler');
 const CircuitBreakerManager = require('./CircuitBreaker');
 const DLQRouter = require('./DLQRouter');
 const CompensationHandler = require('./CompensationHandler');
+const runtimeCfg = require('./config');
 
 class ErrorHandlerCore {
   /**
@@ -38,8 +39,9 @@ class ErrorHandlerCore {
     }
     
     this.serviceName = config.serviceName;
-    this.serviceVersion = config.serviceVersion || '0.0.0';
-    this.environment = config.environment || process.env.NODE_ENV || 'development';
+    const resolved = runtimeCfg.resolve(config);
+    this.serviceVersion = config.serviceVersion || resolved.serviceVersion;
+    this.environment = config.environment || resolved.environment;
     this.monitoring = config.monitoring;
     
     const handling = config.handling || {};

package/src/UnifiedLogSchema.js

@@ -4,6 +4,7 @@
  */
 
 const os = require('os');
+const runtimeCfg = require('./config');
 
 class UnifiedLogSchema {
   /**
@@ -74,7 +75,7 @@ class UnifiedLogSchema {
       // Metadata
       metadata: {
         severity: this._determineSeverity(errorType),
-        environment: environment || process.env.NODE_ENV || 'development',
+        environment: runtimeCfg.get('environment', environment),
         version: serviceVersion || '0.0.0',
         hostname: os.hostname(),
         pid: process.pid

package/src/config.js

@@ -0,0 +1,25 @@
+'use strict';
+
+/**
+ * Runtime configuration schema for @onlineapps/error-handler-core.
+ *
+ * Uses @onlineapps/runtime-config for unified priority:
+ * 1. Explicit config
+ * 2. Environment variable
+ * 3. Module-owned defaults (from ./defaults.js)
+ */
+
+const { createRuntimeConfig } = require('@onlineapps/runtime-config');
+const DEFAULTS = require('./defaults');
+
+const runtimeCfg = createRuntimeConfig({
+  defaults: DEFAULTS,
+  schema: {
+    environment: { env: 'NODE_ENV', defaultKey: 'environment' },
+    serviceVersion: { env: 'SERVICE_VERSION', defaultKey: 'serviceVersion' },
+  }
+});
+
+module.exports = runtimeCfg;
+
+

package/src/defaults.js

@@ -0,0 +1,16 @@
+'use strict';
+
+/**
+ * Module-owned defaults for @onlineapps/error-handler-core.
+ *
+ * Ownership rule:
+ * - Defaults in this file belong to error-handler-core (metadata defaults).
+ * - Service identity should be provided explicitly by callers.
+ */
+
+module.exports = {
+  environment: 'development',
+  serviceVersion: '0.0.0',
+};
+
+