@onlineapps/service-common 1.0.17 → 1.0.19

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/service-common",
-  "version": "1.0.17",
+  "version": "1.0.19",
   "description": "Common utilities for both infrastructure services and business services",
   "main": "src/index.js",
   "scripts": {

package/src/errors/BusinessError.js

@@ -1,5 +1,6 @@
 'use strict';
 
+// See: docs/standards/error-handling-contract.md
 const ERROR_TYPES = {
   TRANSIENT: 'TRANSIENT',
   BUSINESS: 'BUSINESS',

package/src/errors/errorMiddleware.js

@@ -1,5 +1,6 @@
 'use strict';
 
+// See: docs/standards/error-handling-contract.md
 const { BusinessError, isBusinessError } = require('./BusinessError');
 
 function businessErrorHandler(err, req, res, next) {

package/src/index.js

@@ -24,6 +24,8 @@ const {
 } = require('./postgresClient');
 const {
   requireEnv,
+  requireNumberEnv,
+  requireBoolEnv,
   optionalEnv,
   optionalNumberEnv,
   getCriticalConfig,
@@ -61,6 +63,8 @@ module.exports = {
 
   // Configuration helpers (NO FALLBACKS for critical infrastructure)
   requireEnv,
+  requireNumberEnv,
+  requireBoolEnv,
   optionalEnv,
   optionalNumberEnv,
   getCriticalConfig,

package/src/infrastructure/waitForInfrastructureReady.js

@@ -120,7 +120,7 @@ async function waitForInfrastructureReady(options = {}) {
           
           // Optionally log individual service status
           // Note: Service names should match config.infrastructureServices in Registry
-          const serviceKeys = ['gateway', 'registry', 'validator', 'delivery', 'monitoring'];
+          const serviceKeys = ['gateway', 'registry', 'validator', 'delivery-dispatcher', 'monitoring'];
           const statuses = {};
           for (const serviceName of serviceKeys) {
             try {

package/src/runtime-config.js

@@ -29,6 +29,38 @@ function requireEnv(name, description) {
   }
 }
 
+/**
+ * Require a numeric environment variable (fail-fast, no default).
+ * @param {string} name - Environment variable name
+ * @param {string} description - Human-readable description for error messages
+ * @returns {number} Parsed number
+ * @throws {Error} If variable is missing or not a valid number
+ */
+function requireNumberEnv(name, description) {
+  const raw = requireEnv(name, description);
+  const num = Number(raw);
+  if (isNaN(num)) {
+    throw new Error(`[ServiceConfig] ${name} must be a number, got: "${raw}"`);
+  }
+  return num;
+}
+
+/**
+ * Require a boolean environment variable (fail-fast, no default).
+ * Accepts only literal "true" or "false".
+ * @param {string} name - Environment variable name
+ * @param {string} description - Human-readable description for error messages
+ * @returns {boolean} Parsed boolean
+ * @throws {Error} If variable is missing or not "true"/"false"
+ */
+function requireBoolEnv(name, description) {
+  const raw = requireEnv(name, description);
+  if (raw !== 'true' && raw !== 'false') {
+    throw new Error(`[ServiceConfig] ${name} must be "true" or "false", got: "${raw}"`);
+  }
+  return raw === 'true';
+}
+
 /**
  * Get optional environment variable with default
  * Use ONLY for non-critical values (ports, timeouts, log levels)
@@ -103,6 +135,8 @@ function getInfrastructureHealthConfig() {
 
 module.exports = {
   requireEnv,
+  requireNumberEnv,
+  requireBoolEnv,
   optionalEnv,
   optionalNumberEnv,
   getCriticalConfig,