@hazeljs/discovery 0.2.0-beta.16 → 0.2.0-beta.18

package/dist/client/discovery-client.js

@@ -7,10 +7,15 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DiscoveryClient = void 0;
 const memory_backend_1 = require("../backends/memory-backend");
 const strategies_1 = require("../load-balancer/strategies");
+const filter_1 = require("../utils/filter");
+const logger_1 = require("../utils/logger");
+const validation_1 = require("../utils/validation");
 class DiscoveryClient {
     constructor(config = {}, backend) {
         this.config = config;
         this.cache = new Map();
+        this.refreshIntervalHandle = null;
+        (0, validation_1.validateDiscoveryClientConfig)(config);
         this.backend = backend || new memory_backend_1.MemoryRegistryBackend();
         this.loadBalancerFactory = new strategies_1.LoadBalancerFactory();
         if (config.refreshInterval) {
@@ -28,7 +33,7 @@ class DiscoveryClient {
                 const age = Date.now() - cached.timestamp;
                 const ttl = this.config.cacheTTL || 30000;
                 if (age < ttl) {
-                    return this.applyFilter(cached.instances, filter);
+                    return (0, filter_1.applyServiceFilter)(cached.instances, filter);
                 }
             }
         }
@@ -83,41 +88,36 @@ class DiscoveryClient {
         return this.loadBalancerFactory;
     }
     /**
-     * Start cache refresh interval
+     * Close the discovery client and release all resources.
+     * Stops the cache refresh interval and clears the cache.
      */
-    startRefreshInterval() {
-        setInterval(async () => {
-            const services = await this.getAllServices();
-            for (const service of services) {
-                const instances = await this.backend.getInstances(service);
-                this.cache.set(service, {
-                    instances,
-                    timestamp: Date.now(),
-                });
-            }
-        }, this.config.refreshInterval);
+    close() {
+        if (this.refreshIntervalHandle) {
+            clearInterval(this.refreshIntervalHandle);
+            this.refreshIntervalHandle = null;
+        }
+        this.cache.clear();
     }
     /**
-     * Apply filter to instances
+     * Start cache refresh interval
      */
-    applyFilter(instances, filter) {
-        if (!filter)
-            return instances;
-        return instances.filter((instance) => {
-            if (filter.zone && instance.zone !== filter.zone)
-                return false;
-            if (filter.status && instance.status !== filter.status)
-                return false;
-            if (filter.tags && !filter.tags.every((tag) => instance.tags?.includes(tag)))
-                return false;
-            if (filter.metadata) {
-                for (const [key, value] of Object.entries(filter.metadata)) {
-                    if (instance.metadata?.[key] !== value)
-                        return false;
+    startRefreshInterval() {
+        const logger = logger_1.DiscoveryLogger.getLogger();
+        this.refreshIntervalHandle = setInterval(async () => {
+            try {
+                const services = await this.getAllServices();
+                for (const service of services) {
+                    const instances = await this.backend.getInstances(service);
+                    this.cache.set(service, {
+                        instances,
+                        timestamp: Date.now(),
+                    });
                 }
             }
-            return true;
-        });
+            catch (error) {
+                logger.error('Failed to refresh service cache', error);
+            }
+        }, this.config.refreshInterval);
     }
 }
 exports.DiscoveryClient = DiscoveryClient;

package/dist/client/service-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"service-client.d.ts","sourceRoot":"","sources":["../../src/client/service-client.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AACzC,OAAc,EAAiB,kBAAkB,EAAE,aAAa,EAAE,MAAM,OAAO,CAAC;AAEhF,MAAM,WAAW,mBAAmB;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,MAAM,CAAC,EAAE,aAAa,CAAC;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAED,qBAAa,aAAa;IAMtB,OAAO,CAAC,eAAe;IACvB,OAAO,CAAC,MAAM;IANhB,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,UAAU,CAAS;gBAGjB,eAAe,EAAE,eAAe,EAChC,MAAM,EAAE,mBAAmB;IAUrC;;OAEG;IACG,GAAG,CAAC,CAAC,GAAG,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5F;;OAEG;IACG,IAAI,CAAC,CAAC,GAAG,OAAO,EACpB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;IACG,GAAG,CAAC,CAAC,GAAG,OAAO,EACnB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;IACG,MAAM,CAAC,CAAC,GAAG,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI/F;;OAEG;IACG,KAAK,CAAC,CAAC,GAAG,OAAO,EACrB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;YACW,OAAO;IA2CrB;;OAEG;IACH,OAAO,CAAC,KAAK;CAGd"}
+{"version":3,"file":"service-client.d.ts","sourceRoot":"","sources":["../../src/client/service-client.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAIzC,OAAc,EAAiB,kBAAkB,EAAE,aAAa,EAAc,MAAM,OAAO,CAAC;AAE5F,MAAM,WAAW,mBAAmB;IAClC,WAAW,EAAE,MAAM,CAAC;IACpB,qBAAqB,CAAC,EAAE,MAAM,CAAC;IAC/B,MAAM,CAAC,EAAE,aAAa,CAAC;IACvB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,UAAU,CAAC,EAAE,MAAM,CAAC;CACrB;AAqBD,qBAAa,aAAa;IAMtB,OAAO,CAAC,eAAe;IACvB,OAAO,CAAC,MAAM;IANhB,OAAO,CAAC,aAAa,CAAgB;IACrC,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,UAAU,CAAS;gBAGjB,eAAe,EAAE,eAAe,EAChC,MAAM,EAAE,mBAAmB;IAYrC;;OAEG;IACG,GAAG,CAAC,CAAC,GAAG,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5F;;OAEG;IACG,IAAI,CAAC,CAAC,GAAG,OAAO,EACpB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;IACG,GAAG,CAAC,CAAC,GAAG,OAAO,EACnB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;IACG,MAAM,CAAC,CAAC,GAAG,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,kBAAkB,GAAG,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI/F;;OAEG;IACG,KAAK,CAAC,CAAC,GAAG,OAAO,EACrB,IAAI,EAAE,MAAM,EACZ,IAAI,CAAC,EAAE,OAAO,EACd,MAAM,CAAC,EAAE,kBAAkB,GAC1B,OAAO,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;IAI5B;;OAEG;YACW,OAAO;IAgErB;;OAEG;IACH,OAAO,CAAC,KAAK;CAGd"}

package/dist/client/service-client.js

@@ -3,18 +3,68 @@
  * Service Client
  * HTTP client with automatic service discovery and load balancing
  */
-var __importDefault = (this && this.__importDefault) || function (mod) {
-    return (mod && mod.__esModule) ? mod : { "default": mod };
-};
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ServiceClient = void 0;
-const axios_1 = __importDefault(require("axios"));
+const strategies_1 = require("../load-balancer/strategies");
+const logger_1 = require("../utils/logger");
+const validation_1 = require("../utils/validation");
+const axios_1 = __importStar(require("axios"));
+/** HTTP status codes that are safe to retry */
+const RETRYABLE_STATUS_CODES = new Set([502, 503, 504, 408, 429]);
+/**
+ * Determine whether an error is transient and worth retrying.
+ * Client errors (4xx except 408/429) are NOT retried.
+ */
+function isRetryableError(error) {
+    if (error instanceof axios_1.AxiosError) {
+        // Network / timeout errors are always retryable
+        if (!error.response)
+            return true;
+        return RETRYABLE_STATUS_CODES.has(error.response.status);
+    }
+    // Non-Axios errors (e.g. "no instances available") are retryable
+    return true;
+}
 class ServiceClient {
     constructor(discoveryClient, config) {
         this.discoveryClient = discoveryClient;
         this.config = config;
-        this.retries = config.retries || 3;
-        this.retryDelay = config.retryDelay || 1000;
+        (0, validation_1.validateServiceClientConfig)(config);
+        this.retries = config.retries ?? 3;
+        this.retryDelay = config.retryDelay ?? 1000;
         this.axiosInstance = axios_1.default.create({
             timeout: config.timeout || 5000,
         });
@@ -53,14 +103,23 @@ class ServiceClient {
      * Generic request with service discovery
      */
     async request(config) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         let lastError = null;
         for (let attempt = 0; attempt < this.retries; attempt++) {
+            // Discover service instance
+            const instance = await this.discoveryClient.getInstance(this.config.serviceName, this.config.loadBalancingStrategy, this.config.filter);
+            if (!instance) {
+                throw new Error(`No instances available for service: ${this.config.serviceName}`);
+            }
+            // Track active connections for least-connections strategy
+            const lbStrategy = this.discoveryClient
+                .getLoadBalancerFactory()
+                .get(this.config.loadBalancingStrategy || 'round-robin');
+            const isLeastConn = lbStrategy instanceof strategies_1.LeastConnectionsStrategy;
+            if (isLeastConn) {
+                lbStrategy.incrementConnections(instance.id);
+            }
             try {
-                // Discover service instance
-                const instance = await this.discoveryClient.getInstance(this.config.serviceName, this.config.loadBalancingStrategy, this.config.filter);
-                if (!instance) {
-                    throw new Error(`No instances available for service: ${this.config.serviceName}`);
-                }
                 // Build full URL
                 const baseURL = `${instance.protocol}://${instance.host}:${instance.port}`;
                 const fullConfig = {
@@ -68,20 +127,26 @@ class ServiceClient {
                     baseURL,
                 };
                 // Make request
-                return await this.axiosInstance.request(fullConfig);
+                const response = await this.axiosInstance.request(fullConfig);
+                return response;
             }
             catch (error) {
                 lastError = error;
-                // Log error only in development
-                if (process.env.NODE_ENV === 'development') {
-                    // eslint-disable-next-line no-console
-                    console.error(`Request failed (attempt ${attempt + 1}/${this.retries}):`, error);
+                logger.warn(`Request to ${this.config.serviceName} failed (attempt ${attempt + 1}/${this.retries})`, error);
+                // Only retry on transient / network errors
+                if (!isRetryableError(error)) {
+                    throw error;
                 }
-                // Wait before retry
+                // Wait before retry (skip delay on last attempt since we'll throw)
                 if (attempt < this.retries - 1) {
                     await this.sleep(this.retryDelay);
                 }
             }
+            finally {
+                if (isLeastConn) {
+                    lbStrategy.decrementConnections(instance.id);
+                }
+            }
         }
         throw lastError || new Error('Request failed after all retries');
     }

package/dist/index.d.ts

@@ -10,9 +10,12 @@ export * from './load-balancer/strategies';
 export { RegistryBackend } from './backends/registry-backend';
 export { MemoryRegistryBackend } from './backends/memory-backend';
 export { RedisRegistryBackend, RedisBackendConfig } from './backends/redis-backend';
-export { ConsulRegistryBackend, ConsulBackendConfig } from './backends/consul-backend';
+export { ConsulRegistryBackend, ConsulBackendConfig, ConsulClient, } from './backends/consul-backend';
 export { KubernetesRegistryBackend, KubernetesBackendConfig } from './backends/kubernetes-backend';
 export { ServiceRegistry as ServiceRegistryDecorator } from './decorators/service-registry.decorator';
 export { getServiceRegistryMetadata } from './decorators/service-registry.decorator';
 export * from './decorators/inject-service-client.decorator';
+export { applyServiceFilter } from './utils/filter';
+export { DiscoveryLogger, Logger } from './utils/logger';
+export { ConfigValidationError } from './utils/validation';
 //# sourceMappingURL=index.d.ts.map

package/dist/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,cAAc,SAAS,CAAC;AAGxB,cAAc,6BAA6B,CAAC;AAG5C,cAAc,2BAA2B,CAAC;AAC1C,cAAc,yBAAyB,CAAC;AAGxC,cAAc,4BAA4B,CAAC;AAG3C,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAC9D,OAAO,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAClE,OAAO,EAAE,oBAAoB,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AACpF,OAAO,EAAE,qBAAqB,EAAE,mBAAmB,EAAE,MAAM,2BAA2B,CAAC;AACvF,OAAO,EAAE,yBAAyB,EAAE,uBAAuB,EAAE,MAAM,+BAA+B,CAAC;AAGnG,OAAO,EAAE,eAAe,IAAI,wBAAwB,EAAE,MAAM,yCAAyC,CAAC;AACtG,OAAO,EAAE,0BAA0B,EAAE,MAAM,yCAAyC,CAAC;AACrF,cAAc,8CAA8C,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../src/index.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,cAAc,SAAS,CAAC;AAGxB,cAAc,6BAA6B,CAAC;AAG5C,cAAc,2BAA2B,CAAC;AAC1C,cAAc,yBAAyB,CAAC;AAGxC,cAAc,4BAA4B,CAAC;AAG3C,OAAO,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AAC9D,OAAO,EAAE,qBAAqB,EAAE,MAAM,2BAA2B,CAAC;AAClE,OAAO,EAAE,oBAAoB,EAAE,kBAAkB,EAAE,MAAM,0BAA0B,CAAC;AACpF,OAAO,EACL,qBAAqB,EACrB,mBAAmB,EACnB,YAAY,GACb,MAAM,2BAA2B,CAAC;AACnC,OAAO,EAAE,yBAAyB,EAAE,uBAAuB,EAAE,MAAM,+BAA+B,CAAC;AAGnG,OAAO,EAAE,eAAe,IAAI,wBAAwB,EAAE,MAAM,yCAAyC,CAAC;AACtG,OAAO,EAAE,0BAA0B,EAAE,MAAM,yCAAyC,CAAC;AACrF,cAAc,8CAA8C,CAAC;AAG7D,OAAO,EAAE,kBAAkB,EAAE,MAAM,gBAAgB,CAAC;AACpD,OAAO,EAAE,eAAe,EAAE,MAAM,EAAE,MAAM,gBAAgB,CAAC;AACzD,OAAO,EAAE,qBAAqB,EAAE,MAAM,oBAAoB,CAAC"}

package/dist/index.js

@@ -18,7 +18,7 @@ var __exportStar = (this && this.__exportStar) || function(m, exports) {
     for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getServiceRegistryMetadata = exports.ServiceRegistryDecorator = exports.KubernetesRegistryBackend = exports.ConsulRegistryBackend = exports.RedisRegistryBackend = exports.MemoryRegistryBackend = void 0;
+exports.ConfigValidationError = exports.DiscoveryLogger = exports.applyServiceFilter = exports.getServiceRegistryMetadata = exports.ServiceRegistryDecorator = exports.KubernetesRegistryBackend = exports.ConsulRegistryBackend = exports.RedisRegistryBackend = exports.MemoryRegistryBackend = void 0;
 // Types
 __exportStar(require("./types"), exports);
 // Registry
@@ -42,3 +42,10 @@ Object.defineProperty(exports, "ServiceRegistryDecorator", { enumerable: true, g
 var service_registry_decorator_2 = require("./decorators/service-registry.decorator");
 Object.defineProperty(exports, "getServiceRegistryMetadata", { enumerable: true, get: function () { return service_registry_decorator_2.getServiceRegistryMetadata; } });
 __exportStar(require("./decorators/inject-service-client.decorator"), exports);
+// Utilities
+var filter_1 = require("./utils/filter");
+Object.defineProperty(exports, "applyServiceFilter", { enumerable: true, get: function () { return filter_1.applyServiceFilter; } });
+var logger_1 = require("./utils/logger");
+Object.defineProperty(exports, "DiscoveryLogger", { enumerable: true, get: function () { return logger_1.DiscoveryLogger; } });
+var validation_1 = require("./utils/validation");
+Object.defineProperty(exports, "ConfigValidationError", { enumerable: true, get: function () { return validation_1.ConfigValidationError; } });

package/dist/registry/service-registry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"service-registry.d.ts","sourceRoot":"","sources":["../../src/registry/service-registry.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,qBAAqB,EAAiB,MAAM,UAAU,CAAC;AACjF,OAAO,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAI/D,qBAAa,eAAe;IAOxB,OAAO,CAAC,MAAM;IANhB,OAAO,CAAC,OAAO,CAAkB;IACjC,OAAO,CAAC,QAAQ,CAAgC;IAChD,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,eAAe,CAA+B;gBAG5C,MAAM,EAAE,qBAAqB,EACrC,OAAO,CAAC,EAAE,eAAe;IAK3B;;OAEG;IACG,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC;IAgC/B;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAiBjC;;OAEG;IACH,WAAW,IAAI,eAAe,GAAG,IAAI;IAIrC;;OAEG;IACH,UAAU,IAAI,eAAe;IAI7B;;OAEG;IACH,OAAO,CAAC,cAAc;IAYtB;;OAEG;IACH,OAAO,CAAC,YAAY;IAMpB;;OAEG;YACW,kBAAkB;IAoBhC;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAI1B;;OAEG;IACH,OAAO,CAAC,UAAU;CAgBnB"}
+{"version":3,"file":"service-registry.d.ts","sourceRoot":"","sources":["../../src/registry/service-registry.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,qBAAqB,EAAiB,MAAM,UAAU,CAAC;AACjF,OAAO,EAAE,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAM/D,qBAAa,eAAe;IAOxB,OAAO,CAAC,MAAM;IANhB,OAAO,CAAC,OAAO,CAAkB;IACjC,OAAO,CAAC,QAAQ,CAAgC;IAChD,OAAO,CAAC,iBAAiB,CAA+B;IACxD,OAAO,CAAC,eAAe,CAA+B;gBAG5C,MAAM,EAAE,qBAAqB,EACrC,OAAO,CAAC,EAAE,eAAe;IAO3B;;OAEG;IACG,QAAQ,IAAI,OAAO,CAAC,IAAI,CAAC;IAgC/B;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAiBjC;;OAEG;IACH,WAAW,IAAI,eAAe,GAAG,IAAI;IAIrC;;OAEG;IACH,UAAU,IAAI,eAAe;IAI7B;;OAEG;IACH,OAAO,CAAC,cAAc;IAYtB;;OAEG;IACH,OAAO,CAAC,YAAY;IAWpB;;OAEG;YACW,kBAAkB;IA0BhC;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAI1B;;OAEG;IACH,OAAO,CAAC,UAAU;CAgBnB"}

package/dist/registry/service-registry.js

@@ -10,6 +10,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.ServiceRegistry = void 0;
 const types_1 = require("../types");
 const memory_backend_1 = require("../backends/memory-backend");
+const logger_1 = require("../utils/logger");
+const validation_1 = require("../utils/validation");
 const axios_1 = __importDefault(require("axios"));
 class ServiceRegistry {
     constructor(config, backend) {
@@ -17,6 +19,7 @@ class ServiceRegistry {
         this.instance = null;
         this.heartbeatInterval = null;
         this.cleanupInterval = null;
+        (0, validation_1.validateServiceRegistryConfig)(config);
         this.backend = backend || new memory_backend_1.MemoryRegistryBackend();
     }
     /**
@@ -95,7 +98,13 @@ class ServiceRegistry {
      */
     startCleanup() {
         this.cleanupInterval = setInterval(async () => {
-            await this.backend.cleanup();
+            try {
+                await this.backend.cleanup();
+            }
+            catch (error) {
+                const logger = logger_1.DiscoveryLogger.getLogger();
+                logger.error('Cleanup task failed', error);
+            }
         }, 60000); // Run every minute
     }
     /**
@@ -104,6 +113,7 @@ class ServiceRegistry {
     async performHealthCheck() {
         if (!this.instance)
             return;
+        const logger = logger_1.DiscoveryLogger.getLogger();
         try {
             const url = `${this.instance.protocol}://${this.instance.host}:${this.instance.port}${this.instance.healthCheckPath}`;
             const response = await axios_1.default.get(url, { timeout: 5000 });
@@ -116,9 +126,10 @@ class ServiceRegistry {
                 await this.backend.updateStatus(this.instance.id, types_1.ServiceStatus.DOWN);
             }
         }
-        catch {
+        catch (error) {
             this.instance.status = types_1.ServiceStatus.DOWN;
             await this.backend.updateStatus(this.instance.id, types_1.ServiceStatus.DOWN);
+            logger.warn(`Health check failed for ${this.instance.name} (${this.instance.id}), marking as DOWN`, error);
         }
     }
     /**

package/dist/utils/filter.d.ts

@@ -0,0 +1,10 @@
+/**
+ * Shared filter utility for service instances
+ */
+import { ServiceInstance, ServiceFilter } from '../types';
+/**
+ * Apply a ServiceFilter to an array of ServiceInstances.
+ * Returns only instances matching all specified filter criteria.
+ */
+export declare function applyServiceFilter(instances: ServiceInstance[], filter?: ServiceFilter): ServiceInstance[];
+//# sourceMappingURL=filter.d.ts.map

package/dist/utils/filter.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"filter.d.ts","sourceRoot":"","sources":["../../src/utils/filter.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,eAAe,EAAE,aAAa,EAAE,MAAM,UAAU,CAAC;AAE1D;;;GAGG;AACH,wBAAgB,kBAAkB,CAChC,SAAS,EAAE,eAAe,EAAE,EAC5B,MAAM,CAAC,EAAE,aAAa,GACrB,eAAe,EAAE,CAgCnB"}

package/dist/utils/filter.js

@@ -0,0 +1,39 @@
+"use strict";
+/**
+ * Shared filter utility for service instances
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.applyServiceFilter = applyServiceFilter;
+/**
+ * Apply a ServiceFilter to an array of ServiceInstances.
+ * Returns only instances matching all specified filter criteria.
+ */
+function applyServiceFilter(instances, filter) {
+    if (!filter)
+        return instances;
+    return instances.filter((instance) => {
+        // Filter by zone
+        if (filter.zone && instance.zone !== filter.zone) {
+            return false;
+        }
+        // Filter by status
+        if (filter.status && instance.status !== filter.status) {
+            return false;
+        }
+        // Filter by tags
+        if (filter.tags && filter.tags.length > 0) {
+            if (!instance.tags || !filter.tags.every((tag) => instance.tags.includes(tag))) {
+                return false;
+            }
+        }
+        // Filter by metadata
+        if (filter.metadata) {
+            for (const [key, value] of Object.entries(filter.metadata)) {
+                if (!instance.metadata || instance.metadata[key] !== value) {
+                    return false;
+                }
+            }
+        }
+        return true;
+    });
+}

package/dist/utils/logger.d.ts

@@ -0,0 +1,21 @@
+/**
+ * Pluggable logger for @hazeljs/discovery
+ *
+ * Consumers can supply their own logger via DiscoveryLogger.setLogger().
+ * The default logger writes to the console.
+ */
+export interface Logger {
+    debug(message: string, ...args: unknown[]): void;
+    info(message: string, ...args: unknown[]): void;
+    warn(message: string, ...args: unknown[]): void;
+    error(message: string, ...args: unknown[]): void;
+}
+export declare const DiscoveryLogger: {
+    /** Replace the default console logger with a custom implementation */
+    setLogger(logger: Logger): void;
+    /** Reset to the default console logger */
+    resetLogger(): void;
+    /** Get the current logger instance */
+    getLogger(): Logger;
+};
+//# sourceMappingURL=logger.d.ts.map

package/dist/utils/logger.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../src/utils/logger.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,MAAM;IACrB,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IACjD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;CAClD;AAeD,eAAO,MAAM,eAAe;IAC1B,sEAAsE;sBACpD,MAAM,GAAG,IAAI;IAI/B,0CAA0C;mBAC3B,IAAI;IAInB,sCAAsC;iBACzB,MAAM;CAGpB,CAAC"}

package/dist/utils/logger.js

@@ -0,0 +1,34 @@
+"use strict";
+/**
+ * Pluggable logger for @hazeljs/discovery
+ *
+ * Consumers can supply their own logger via DiscoveryLogger.setLogger().
+ * The default logger writes to the console.
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.DiscoveryLogger = void 0;
+const defaultLogger = {
+    // eslint-disable-next-line no-console
+    debug: (msg, ...args) => console.debug(`[discovery] ${msg}`, ...args),
+    // eslint-disable-next-line no-console
+    info: (msg, ...args) => console.info(`[discovery] ${msg}`, ...args),
+    // eslint-disable-next-line no-console
+    warn: (msg, ...args) => console.warn(`[discovery] ${msg}`, ...args),
+    // eslint-disable-next-line no-console
+    error: (msg, ...args) => console.error(`[discovery] ${msg}`, ...args),
+};
+let currentLogger = defaultLogger;
+exports.DiscoveryLogger = {
+    /** Replace the default console logger with a custom implementation */
+    setLogger(logger) {
+        currentLogger = logger;
+    },
+    /** Reset to the default console logger */
+    resetLogger() {
+        currentLogger = defaultLogger;
+    },
+    /** Get the current logger instance */
+    getLogger() {
+        return currentLogger;
+    },
+};

package/dist/utils/validation.d.ts

@@ -0,0 +1,36 @@
+/**
+ * Runtime validation utilities for discovery configuration objects
+ */
+import { ServiceRegistryConfig, DiscoveryClientConfig } from '../types';
+import { RedisBackendConfig } from '../backends/redis-backend';
+import { ConsulBackendConfig } from '../backends/consul-backend';
+import { KubernetesBackendConfig } from '../backends/kubernetes-backend';
+import { ServiceClientConfig } from '../client/service-client';
+export declare class ConfigValidationError extends Error {
+    constructor(message: string);
+}
+/**
+ * Validate ServiceRegistryConfig at runtime
+ */
+export declare function validateServiceRegistryConfig(config: ServiceRegistryConfig): void;
+/**
+ * Validate DiscoveryClientConfig at runtime
+ */
+export declare function validateDiscoveryClientConfig(config: DiscoveryClientConfig): void;
+/**
+ * Validate ServiceClientConfig at runtime
+ */
+export declare function validateServiceClientConfig(config: ServiceClientConfig): void;
+/**
+ * Validate RedisBackendConfig at runtime
+ */
+export declare function validateRedisBackendConfig(config: RedisBackendConfig): void;
+/**
+ * Validate ConsulBackendConfig at runtime
+ */
+export declare function validateConsulBackendConfig(config: ConsulBackendConfig): void;
+/**
+ * Validate KubernetesBackendConfig at runtime
+ */
+export declare function validateKubernetesBackendConfig(config: KubernetesBackendConfig): void;
+//# sourceMappingURL=validation.d.ts.map

package/dist/utils/validation.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"validation.d.ts","sourceRoot":"","sources":["../../src/utils/validation.ts"],"names":[],"mappings":"AAAA;;GAEG;AAEH,OAAO,EAAE,qBAAqB,EAAE,qBAAqB,EAAE,MAAM,UAAU,CAAC;AACxE,OAAO,EAAE,kBAAkB,EAAE,MAAM,2BAA2B,CAAC;AAC/D,OAAO,EAAE,mBAAmB,EAAE,MAAM,4BAA4B,CAAC;AACjE,OAAO,EAAE,uBAAuB,EAAE,MAAM,gCAAgC,CAAC;AACzE,OAAO,EAAE,mBAAmB,EAAE,MAAM,0BAA0B,CAAC;AAE/D,qBAAa,qBAAsB,SAAQ,KAAK;gBAClC,OAAO,EAAE,MAAM;CAI5B;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAAC,MAAM,EAAE,qBAAqB,GAAG,IAAI,CAgCjF;AAED;;GAEG;AACH,wBAAgB,6BAA6B,CAAC,MAAM,EAAE,qBAAqB,GAAG,IAAI,CAejF;AAED;;GAEG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,mBAAmB,GAAG,IAAI,CAkC7E;AAED;;GAEG;AACH,wBAAgB,0BAA0B,CAAC,MAAM,EAAE,kBAAkB,GAAG,IAAI,CAe3E;AAED;;GAEG;AACH,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,mBAAmB,GAAG,IAAI,CAkB7E;AAED;;GAEG;AACH,wBAAgB,+BAA+B,CAAC,MAAM,EAAE,uBAAuB,GAAG,IAAI,CASrF"}

package/dist/utils/validation.js

@@ -0,0 +1,109 @@
+"use strict";
+/**
+ * Runtime validation utilities for discovery configuration objects
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.ConfigValidationError = void 0;
+exports.validateServiceRegistryConfig = validateServiceRegistryConfig;
+exports.validateDiscoveryClientConfig = validateDiscoveryClientConfig;
+exports.validateServiceClientConfig = validateServiceClientConfig;
+exports.validateRedisBackendConfig = validateRedisBackendConfig;
+exports.validateConsulBackendConfig = validateConsulBackendConfig;
+exports.validateKubernetesBackendConfig = validateKubernetesBackendConfig;
+class ConfigValidationError extends Error {
+    constructor(message) {
+        super(message);
+        this.name = 'ConfigValidationError';
+    }
+}
+exports.ConfigValidationError = ConfigValidationError;
+/**
+ * Validate ServiceRegistryConfig at runtime
+ */
+function validateServiceRegistryConfig(config) {
+    if (!config.name || typeof config.name !== 'string' || config.name.trim().length === 0) {
+        throw new ConfigValidationError('ServiceRegistryConfig: "name" is required and must be a non-empty string');
+    }
+    if (config.port == null ||
+        typeof config.port !== 'number' ||
+        config.port < 0 ||
+        config.port > 65535) {
+        throw new ConfigValidationError('ServiceRegistryConfig: "port" is required and must be a number between 0 and 65535');
+    }
+    if (config.healthCheckInterval != null &&
+        (typeof config.healthCheckInterval !== 'number' || config.healthCheckInterval <= 0)) {
+        throw new ConfigValidationError('ServiceRegistryConfig: "healthCheckInterval" must be a positive number (ms)');
+    }
+    if (config.protocol && !['http', 'https', 'grpc'].includes(config.protocol)) {
+        throw new ConfigValidationError('ServiceRegistryConfig: "protocol" must be one of "http", "https", "grpc"');
+    }
+}
+/**
+ * Validate DiscoveryClientConfig at runtime
+ */
+function validateDiscoveryClientConfig(config) {
+    if (config.cacheTTL != null && (typeof config.cacheTTL !== 'number' || config.cacheTTL <= 0)) {
+        throw new ConfigValidationError('DiscoveryClientConfig: "cacheTTL" must be a positive number (ms)');
+    }
+    if (config.refreshInterval != null &&
+        (typeof config.refreshInterval !== 'number' || config.refreshInterval <= 0)) {
+        throw new ConfigValidationError('DiscoveryClientConfig: "refreshInterval" must be a positive number (ms)');
+    }
+}
+/**
+ * Validate ServiceClientConfig at runtime
+ */
+function validateServiceClientConfig(config) {
+    if (!config.serviceName ||
+        typeof config.serviceName !== 'string' ||
+        config.serviceName.trim().length === 0) {
+        throw new ConfigValidationError('ServiceClientConfig: "serviceName" is required and must be a non-empty string');
+    }
+    if (config.timeout != null && (typeof config.timeout !== 'number' || config.timeout <= 0)) {
+        throw new ConfigValidationError('ServiceClientConfig: "timeout" must be a positive number (ms)');
+    }
+    if (config.retries != null &&
+        (typeof config.retries !== 'number' || config.retries < 0 || !Number.isInteger(config.retries))) {
+        throw new ConfigValidationError('ServiceClientConfig: "retries" must be a non-negative integer');
+    }
+    if (config.retryDelay != null &&
+        (typeof config.retryDelay !== 'number' || config.retryDelay < 0)) {
+        throw new ConfigValidationError('ServiceClientConfig: "retryDelay" must be a non-negative number (ms)');
+    }
+}
+/**
+ * Validate RedisBackendConfig at runtime
+ */
+function validateRedisBackendConfig(config) {
+    if (config.ttl != null && (typeof config.ttl !== 'number' || config.ttl <= 0)) {
+        throw new ConfigValidationError('RedisBackendConfig: "ttl" must be a positive number (seconds)');
+    }
+    if (config.port != null &&
+        (typeof config.port !== 'number' || config.port < 0 || config.port > 65535)) {
+        throw new ConfigValidationError('RedisBackendConfig: "port" must be a number between 0 and 65535');
+    }
+}
+/**
+ * Validate ConsulBackendConfig at runtime
+ */
+function validateConsulBackendConfig(config) {
+    if (config.ttl != null && typeof config.ttl === 'string') {
+        const match = config.ttl.match(/^(\d+)([smh])$/);
+        if (!match) {
+            throw new ConfigValidationError('ConsulBackendConfig: "ttl" must match format like "30s", "5m", or "1h"');
+        }
+    }
+    if (config.port != null &&
+        (typeof config.port !== 'number' || config.port < 0 || config.port > 65535)) {
+        throw new ConfigValidationError('ConsulBackendConfig: "port" must be a number between 0 and 65535');
+    }
+}
+/**
+ * Validate KubernetesBackendConfig at runtime
+ */
+function validateKubernetesBackendConfig(config) {
+    if (config.namespace != null &&
+        (typeof config.namespace !== 'string' || config.namespace.trim().length === 0)) {
+        throw new ConfigValidationError('KubernetesBackendConfig: "namespace" must be a non-empty string');
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hazeljs/discovery",
-  "version": "0.2.0-beta.16",
+  "version": "0.2.0-beta.18",
   "description": "Service discovery and registry for HazelJS microservices - Eureka-inspired with multiple backend support",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -18,7 +18,7 @@
     "clean": "rm -rf dist"
   },
   "dependencies": {
-    "@hazeljs/core": "^0.2.0-beta.16",
+    "@hazeljs/core": "^0.2.0-beta.18",
     "axios": "^1.6.0",
     "reflect-metadata": "^0.2.2"
   },
@@ -75,5 +75,5 @@
     "url": "https://github.com/hazeljs/hazel-js/issues"
   },
   "homepage": "https://hazeljs.com",
-  "gitHead": "e022330edc869ed8dc8d9765aefd085438088297"
+  "gitHead": "22082c1277661421cd32b3c4371c2c9d0bdf0501"
 }