@hazeljs/discovery 0.2.0-beta.8 → 0.2.0-beta.81

package/dist/__tests__/redis-backend.test.js

@@ -0,0 +1,280 @@
+"use strict";
+/**
+ * Redis Backend Tests
+ * Uses mocked ioredis client
+ */
+Object.defineProperty(exports, "__esModule", { value: true });
+const redis_backend_1 = require("../backends/redis-backend");
+const types_1 = require("../types");
+const logger_1 = require("../utils/logger");
+// Suppress console logs during tests
+beforeAll(() => {
+    logger_1.DiscoveryLogger.setLogger({
+        debug: jest.fn(),
+        info: jest.fn(),
+        warn: jest.fn(),
+        error: jest.fn(),
+    });
+});
+afterAll(() => {
+    logger_1.DiscoveryLogger.resetLogger();
+});
+/** Helper to create a mock Redis instance */
+function createMockRedis() {
+    const store = new Map();
+    const sets = new Map();
+    const ttls = new Map();
+    const listeners = new Map();
+    const mock = {
+        on: jest.fn((event, handler) => {
+            if (!listeners.has(event))
+                listeners.set(event, []);
+            listeners.get(event).push(handler);
+        }),
+        setex: jest.fn(async (key, ttl, value) => {
+            store.set(key, value);
+            ttls.set(key, ttl);
+        }),
+        get: jest.fn(async (key) => store.get(key) || null),
+        del: jest.fn(async (key) => {
+            store.delete(key);
+            ttls.delete(key);
+            return 1;
+        }),
+        mget: jest.fn(async (...keys) => {
+            // mget can receive keys as individual args or a single array
+            const flatKeys = keys.flat();
+            return flatKeys.map((k) => store.get(k) || null);
+        }),
+        sadd: jest.fn(async (key, member) => {
+            if (!sets.has(key))
+                sets.set(key, new Set());
+            sets.get(key).add(member);
+            return 1;
+        }),
+        srem: jest.fn(async (key, member) => {
+            sets.get(key)?.delete(member);
+            return 1;
+        }),
+        smembers: jest.fn(async (key) => {
+            return Array.from(sets.get(key) || []);
+        }),
+        scard: jest.fn(async (key) => {
+            return sets.get(key)?.size || 0;
+        }),
+        exists: jest.fn(async (key) => {
+            return store.has(key) ? 1 : 0;
+        }),
+        expire: jest.fn(async () => 1),
+        scan: jest.fn(async (_cursor, _match, pattern) => {
+            const keys = Array.from(sets.keys()).filter((k) => {
+                // Simple glob match: convert "prefix*" to regex
+                const regex = new RegExp('^' + pattern.replace(/\*/g, '.*') + '$');
+                return regex.test(k);
+            });
+            return ['0', keys];
+        }),
+        quit: jest.fn(async () => 'OK'),
+        // Helper to emit events during tests
+        _emit: (event, ...args) => {
+            const handlers = listeners.get(event) || [];
+            handlers.forEach((h) => h(...args));
+        },
+    };
+    return mock;
+}
+function createInstance(id, name = 'test-service', overrides = {}) {
+    return {
+        id,
+        name,
+        host: 'localhost',
+        port: 3000,
+        status: types_1.ServiceStatus.UP,
+        lastHeartbeat: new Date('2025-01-01T00:00:00Z'),
+        registeredAt: new Date('2025-01-01T00:00:00Z'),
+        ...overrides,
+    };
+}
+describe('RedisRegistryBackend', () => {
+    let redis;
+    let backend;
+    beforeEach(() => {
+        redis = createMockRedis();
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        backend = new redis_backend_1.RedisRegistryBackend(redis);
+    });
+    describe('constructor', () => {
+        it('should use default config values', () => {
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            const b = new redis_backend_1.RedisRegistryBackend(redis);
+            expect(b).toBeDefined();
+        });
+        it('should accept custom config', () => {
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            const b = new redis_backend_1.RedisRegistryBackend(redis, {
+                keyPrefix: 'custom:',
+                ttl: 120,
+            });
+            expect(b).toBeDefined();
+        });
+        it('should throw on invalid TTL', () => {
+            expect(() => {
+                // eslint-disable-next-line @typescript-eslint/no-explicit-any
+                new redis_backend_1.RedisRegistryBackend(redis, { ttl: -5 });
+            }).toThrow();
+        });
+        it('should register connection event handlers', () => {
+            expect(redis.on).toHaveBeenCalledWith('error', expect.any(Function));
+            expect(redis.on).toHaveBeenCalledWith('connect', expect.any(Function));
+            expect(redis.on).toHaveBeenCalledWith('reconnecting', expect.any(Function));
+            expect(redis.on).toHaveBeenCalledWith('close', expect.any(Function));
+        });
+    });
+    describe('register', () => {
+        it('should store the instance with TTL', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            expect(redis.setex).toHaveBeenCalledWith('hazeljs:discovery:instance:1', 90, JSON.stringify(instance));
+        });
+        it('should add instance to service set', async () => {
+            const instance = createInstance('1', 'my-service');
+            await backend.register(instance);
+            expect(redis.sadd).toHaveBeenCalledWith('hazeljs:discovery:service:my-service', '1');
+        });
+        it('should set expiration on service set', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            expect(redis.expire).toHaveBeenCalledWith('hazeljs:discovery:service:test-service', 180);
+        });
+    });
+    describe('deregister', () => {
+        it('should remove instance from Redis', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            await backend.deregister('1');
+            expect(redis.srem).toHaveBeenCalledWith('hazeljs:discovery:service:test-service', '1');
+            expect(redis.del).toHaveBeenCalledWith('hazeljs:discovery:instance:1');
+        });
+        it('should do nothing if instance does not exist', async () => {
+            await backend.deregister('non-existent');
+            expect(redis.del).not.toHaveBeenCalled();
+        });
+    });
+    describe('heartbeat', () => {
+        it('should update heartbeat timestamp and status', async () => {
+            const instance = createInstance('1');
+            instance.status = types_1.ServiceStatus.DOWN;
+            await backend.register(instance);
+            await backend.heartbeat('1');
+            // Should have been called again with updated data
+            expect(redis.setex).toHaveBeenCalledTimes(2);
+            const lastCall = redis.setex.mock.calls[1];
+            const storedInstance = JSON.parse(lastCall[2]);
+            expect(storedInstance.status).toBe(types_1.ServiceStatus.UP);
+        });
+        it('should do nothing if instance does not exist', async () => {
+            await backend.heartbeat('non-existent');
+            // setex should not be called
+            expect(redis.setex).not.toHaveBeenCalled();
+        });
+    });
+    describe('getInstances', () => {
+        it('should return all instances for a service using MGET', async () => {
+            const i1 = createInstance('1', 'svc');
+            const i2 = createInstance('2', 'svc');
+            await backend.register(i1);
+            await backend.register(i2);
+            const instances = await backend.getInstances('svc');
+            expect(instances).toHaveLength(2);
+            expect(redis.mget).toHaveBeenCalled();
+        });
+        it('should return empty array for non-existent service', async () => {
+            const instances = await backend.getInstances('non-existent');
+            expect(instances).toEqual([]);
+        });
+        it('should apply filter', async () => {
+            const i1 = createInstance('1', 'svc', { zone: 'us-east-1' });
+            const i2 = createInstance('2', 'svc', { zone: 'us-west-1' });
+            await backend.register(i1);
+            await backend.register(i2);
+            const instances = await backend.getInstances('svc', { zone: 'us-east-1' });
+            expect(instances).toHaveLength(1);
+            expect(instances[0].zone).toBe('us-east-1');
+        });
+    });
+    describe('getInstance', () => {
+        it('should return a specific instance', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            const retrieved = await backend.getInstance('1');
+            expect(retrieved).toBeDefined();
+            expect(retrieved.id).toBe('1');
+        });
+        it('should return null for non-existent instance', async () => {
+            const result = await backend.getInstance('non-existent');
+            expect(result).toBeNull();
+        });
+        it('should convert dates back from strings', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            const retrieved = await backend.getInstance('1');
+            expect(retrieved.lastHeartbeat).toBeInstanceOf(Date);
+            expect(retrieved.registeredAt).toBeInstanceOf(Date);
+        });
+    });
+    describe('getAllServices', () => {
+        it('should use SCAN instead of KEYS', async () => {
+            const i1 = createInstance('1', 'svc-a');
+            const i2 = createInstance('2', 'svc-b');
+            await backend.register(i1);
+            await backend.register(i2);
+            const services = await backend.getAllServices();
+            expect(redis.scan).toHaveBeenCalled();
+            expect(services).toHaveLength(2);
+        });
+    });
+    describe('updateStatus', () => {
+        it('should update the status of an instance', async () => {
+            const instance = createInstance('1');
+            await backend.register(instance);
+            await backend.updateStatus('1', types_1.ServiceStatus.DOWN);
+            const retrieved = await backend.getInstance('1');
+            expect(retrieved.status).toBe(types_1.ServiceStatus.DOWN);
+        });
+        it('should do nothing for non-existent instance', async () => {
+            await backend.updateStatus('non-existent', types_1.ServiceStatus.DOWN);
+            expect(redis.setex).not.toHaveBeenCalled();
+        });
+    });
+    describe('cleanup', () => {
+        it('should remove stale entries from service sets', async () => {
+            const instance = createInstance('1', 'svc');
+            await backend.register(instance);
+            // Simulate the instance key expiring in Redis
+            redis.exists.mockResolvedValueOnce(0);
+            await backend.cleanup();
+            expect(redis.srem).toHaveBeenCalledWith('hazeljs:discovery:service:svc', '1');
+        });
+    });
+    describe('connection error handling', () => {
+        it('should track connection state via events', () => {
+            redis._emit('close');
+            // Backend should mark as disconnected
+            // Next operation should throw
+            expect(backend.getInstances('svc')).rejects.toThrow('Redis backend is not connected');
+        });
+        it('should recover on connect event', async () => {
+            redis._emit('close');
+            redis._emit('connect');
+            // Should work again
+            const result = await backend.getInstances('svc');
+            expect(result).toEqual([]);
+        });
+    });
+    describe('close', () => {
+        it('should call redis.quit()', async () => {
+            await backend.close();
+            expect(redis.quit).toHaveBeenCalled();
+        });
+    });
+});

package/dist/__tests__/service-client.test.js

@@ -139,7 +139,8 @@ describe('ServiceClient', () => {
             const mockRequest = jest.fn().mockRejectedValue(new Error('Network error'));
             serviceClient.axiosInstance = { request: mockRequest };
             await expect(serviceClient.get('/api/users')).rejects.toThrow();
-            expect(mockRequest).toHaveBeenCalledTimes(3); // Default retries
+            // retries: 3 → RetryPolicy maxAttempts: 3 → 1 initial + 3 retries = 4 total
+            expect(mockRequest).toHaveBeenCalledTimes(4);
         });
         it('should respect retry delay', async () => {
             const client = new service_client_1.ServiceClient(discoveryClient, {

package/dist/backends/consul-backend.d.ts

@@ -4,7 +4,51 @@
  */
 import { RegistryBackend } from './registry-backend';
 import { ServiceInstance, ServiceFilter } from '../types';
-type Consul = any;
+/**
+ * Minimal type definitions for the Consul client API surface we use.
+ * These mirror the shapes exposed by the `consul` npm package.
+ */
+export interface ConsulClient {
+    agent: {
+        service: {
+            register(opts: Record<string, unknown>): Promise<void>;
+            deregister(serviceId: string): Promise<void>;
+            list(): Promise<Record<string, ConsulServiceEntry>>;
+        };
+        check: {
+            pass(checkId: string): Promise<void>;
+            fail(checkId: string): Promise<void>;
+            warn(checkId: string): Promise<void>;
+            list(): Promise<Record<string, ConsulCheckEntry>>;
+        };
+    };
+    health: {
+        service(opts: {
+            service: string;
+            passing?: boolean;
+        }): Promise<ConsulHealthEntry[]>;
+    };
+    catalog: {
+        service: {
+            list(): Promise<Record<string, string[]>>;
+        };
+    };
+}
+export interface ConsulServiceEntry {
+    ID: string;
+    Service: string;
+    Address: string;
+    Port: number;
+    Meta?: Record<string, string>;
+    Tags?: string[];
+}
+export interface ConsulCheckEntry {
+    Status: string;
+}
+export interface ConsulHealthEntry {
+    Service: ConsulServiceEntry;
+    Checks?: ConsulCheckEntry[];
+}
 export interface ConsulBackendConfig {
     host?: string;
     port?: number;
@@ -17,7 +61,7 @@ export declare class ConsulRegistryBackend implements RegistryBackend {
     private consul;
     private readonly ttl;
     private checkIntervals;
-    constructor(consul: Consul, config?: ConsulBackendConfig);
+    constructor(consul: ConsulClient, config?: ConsulBackendConfig);
     /**
      * Register a service instance with Consul
      */
@@ -67,10 +111,5 @@ export declare class ConsulRegistryBackend implements RegistryBackend {
      * Parse TTL string to seconds
      */
     private parseTTL;
-    /**
-     * Apply filter to instances
-     */
-    private applyFilter;
 }
-export {};
 //# sourceMappingURL=consul-backend.d.ts.map

package/dist/backends/consul-backend.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"consul-backend.d.ts","sourceRoot":"","sources":["../../src/backends/consul-backend.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAiB,MAAM,UAAU,CAAC;AAIzE,KAAK,MAAM,GAAG,GAAG,CAAC;AAElB,MAAM,WAAW,mBAAmB;IAClC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED,qBAAa,qBAAsB,YAAW,eAAe;IAC3D,OAAO,CAAC,MAAM,CAAS;IACvB,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAS;IAC7B,OAAO,CAAC,cAAc,CAA0C;gBAEpD,MAAM,EAAE,MAAM,EAAE,MAAM,GAAE,mBAAwB;IAK5D;;OAEG;IACG,QAAQ,CAAC,QAAQ,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IA0BxD;;OAEG;IACG,UAAU,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAQnD;;OAEG;IACG,SAAS,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAWlD;;OAEG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IA6D3F;;OAEG;IACG,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IAwCtE;;OAEG;IACG,cAAc,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IASzC;;OAEG;IACG,YAAY,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAgBrE;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAK9B;;OAEG;IACG,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAO5B;;OAEG;IACH,OAAO,CAAC,aAAa;IAgBrB;;OAEG;IACH,OAAO,CAAC,YAAY;IAQpB;;OAEG;IACH,OAAO,CAAC,QAAQ;IAqBhB;;OAEG;IACH,OAAO,CAAC,WAAW;CA2BpB"}
+{"version":3,"file":"consul-backend.d.ts","sourceRoot":"","sources":["../../src/backends/consul-backend.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAiB,MAAM,UAAU,CAAC;AAKzE;;;GAGG;AACH,MAAM,WAAW,YAAY;IAC3B,KAAK,EAAE;QACL,OAAO,EAAE;YACP,QAAQ,CAAC,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YACvD,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YAC7C,IAAI,IAAI,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC,CAAC;SACrD,CAAC;QACF,KAAK,EAAE;YACL,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YACrC,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YACrC,IAAI,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YACrC,IAAI,IAAI,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAC,CAAC;SACnD,CAAC;KACH,CAAC;IACF,MAAM,EAAE;QACN,OAAO,CAAC,IAAI,EAAE;YAAE,OAAO,EAAE,MAAM,CAAC;YAAC,OAAO,CAAC,EAAE,OAAO,CAAA;SAAE,GAAG,OAAO,CAAC,iBAAiB,EAAE,CAAC,CAAC;KACrF,CAAC;IACF,OAAO,EAAE;QACP,OAAO,EAAE;YACP,IAAI,IAAI,OAAO,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,EAAE,CAAC,CAAC,CAAC;SAC3C,CAAC;KACH,CAAC;CACH;AAED,MAAM,WAAW,kBAAkB;IACjC,EAAE,EAAE,MAAM,CAAC;IACX,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAC9B,IAAI,CAAC,EAAE,MAAM,EAAE,CAAC;CACjB;AAED,MAAM,WAAW,gBAAgB;IAC/B,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,MAAM,WAAW,iBAAiB;IAChC,OAAO,EAAE,kBAAkB,CAAC;IAC5B,MAAM,CAAC,EAAE,gBAAgB,EAAE,CAAC;CAC7B;AAED,MAAM,WAAW,mBAAmB;IAClC,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,GAAG,CAAC,EAAE,MAAM,CAAC;CACd;AAED,qBAAa,qBAAsB,YAAW,eAAe;IAC3D,OAAO,CAAC,MAAM,CAAe;IAC7B,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAS;IAC7B,OAAO,CAAC,cAAc,CAA0C;gBAEpD,MAAM,EAAE,YAAY,EAAE,MAAM,GAAE,mBAAwB;IAOlE;;OAEG;IACG,QAAQ,CAAC,QAAQ,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IA0BxD;;OAEG;IACG,UAAU,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAQnD;;OAEG;IACG,SAAS,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAYlD;;OAEG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IAgD3F;;OAEG;IACG,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IA2CtE;;OAEG;IACG,cAAc,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IAYzC;;OAEG;IACG,YAAY,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAiBrE;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAK9B;;OAEG;IACG,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAO5B;;OAEG;IACH,OAAO,CAAC,aAAa;IAkBrB;;OAEG;IACH,OAAO,CAAC,YAAY;IAQpB;;OAEG;IACH,OAAO,CAAC,QAAQ;CAoBjB"}

package/dist/backends/consul-backend.js

@@ -6,9 +6,13 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ConsulRegistryBackend = void 0;
 const types_1 = require("../types");
+const filter_1 = require("../utils/filter");
+const logger_1 = require("../utils/logger");
+const validation_1 = require("../utils/validation");
 class ConsulRegistryBackend {
     constructor(consul, config = {}) {
         this.checkIntervals = new Map();
+        (0, validation_1.validateConsulBackendConfig)(config);
         this.consul = consul;
         this.ttl = config.ttl || '30s';
     }
@@ -51,19 +55,21 @@ class ConsulRegistryBackend {
      * Update service instance heartbeat
      */
     async heartbeat(instanceId) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         const checkId = `service:${instanceId}`;
         try {
             // Pass TTL check
             await this.consul.agent.check.pass(checkId);
         }
-        catch {
-            // Silently fail - will be retried on next heartbeat
+        catch (error) {
+            logger.warn(`Consul heartbeat failed for ${instanceId}, will retry on next heartbeat`, error);
         }
     }
     /**
      * Get all instances of a service
      */
     async getInstances(serviceName, filter) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         try {
             const result = await this.consul.health.service({
                 service: serviceName,
@@ -99,12 +105,10 @@ class ConsulRegistryBackend {
                 };
             });
             // Apply additional filters
-            if (filter) {
-                return this.applyFilter(instances, filter);
-            }
-            return instances;
+            return (0, filter_1.applyServiceFilter)(instances, filter);
         }
-        catch {
+        catch (error) {
+            logger.error(`Failed to get instances for service "${serviceName}" from Consul`, error);
             return [];
         }
     }
@@ -112,6 +116,7 @@ class ConsulRegistryBackend {
      * Get a specific service instance
      */
     async getInstance(instanceId) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         try {
             const services = await this.consul.agent.service.list();
             const service = services[instanceId];
@@ -144,7 +149,8 @@ class ConsulRegistryBackend {
                 registeredAt: service.Meta?.registeredAt ? new Date(service.Meta.registeredAt) : new Date(),
             };
         }
-        catch {
+        catch (error) {
+            logger.error(`Failed to get instance "${instanceId}" from Consul`, error);
             return null;
         }
     }
@@ -152,11 +158,13 @@ class ConsulRegistryBackend {
      * Get all registered services
      */
     async getAllServices() {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         try {
             const services = await this.consul.catalog.service.list();
             return Object.keys(services);
         }
-        catch {
+        catch (error) {
+            logger.error('Failed to list services from Consul', error);
             return [];
         }
     }
@@ -164,6 +172,7 @@ class ConsulRegistryBackend {
      * Update service instance status
      */
     async updateStatus(instanceId, status) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         const checkId = `service:${instanceId}`;
         try {
             if (status === types_1.ServiceStatus.UP) {
@@ -176,8 +185,8 @@ class ConsulRegistryBackend {
                 await this.consul.agent.check.warn(checkId);
             }
         }
-        catch {
-            // Silently fail - status will be updated on next heartbeat
+        catch (error) {
+            logger.warn(`Failed to update status for ${instanceId} in Consul`, error);
         }
     }
     /**
@@ -201,6 +210,7 @@ class ConsulRegistryBackend {
      * Start TTL check updates for a service
      */
     startTTLCheck(instanceId, checkId) {
+        const logger = logger_1.DiscoveryLogger.getLogger();
         // Parse TTL to get interval (update at 2/3 of TTL)
         const ttlSeconds = this.parseTTL(this.ttl);
         const intervalMs = (ttlSeconds * 1000 * 2) / 3;
@@ -208,8 +218,8 @@ class ConsulRegistryBackend {
             try {
                 await this.consul.agent.check.pass(checkId);
             }
-            catch {
-                // Silently fail - will be retried on next interval
+            catch (error) {
+                logger.warn(`TTL check pass failed for ${instanceId}`, error);
             }
         }, intervalMs);
         this.checkIntervals.set(instanceId, interval);
@@ -245,31 +255,5 @@ class ConsulRegistryBackend {
                 return 30;
         }
     }
-    /**
-     * Apply filter to instances
-     */
-    applyFilter(instances, filter) {
-        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.length > 0) {
-                if (!instance.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 || instance.metadata[key] !== value) {
-                        return false;
-                    }
-                }
-            }
-            return true;
-        });
-    }
 }
 exports.ConsulRegistryBackend = ConsulRegistryBackend;

package/dist/backends/kubernetes-backend.d.ts

@@ -4,7 +4,50 @@
  */
 import { RegistryBackend } from './registry-backend';
 import { ServiceInstance, ServiceFilter } from '../types';
-type KubeConfig = any;
+/**
+ * Minimal type definitions for the Kubernetes client API surface we use.
+ * These mirror the shapes exposed by `@kubernetes/client-node`.
+ */
+export interface KubeConfig {
+    makeApiClient(apiClass: new () => any): any;
+}
+export interface K8sEndpointAddress {
+    ip: string;
+    targetRef?: {
+        name?: string;
+    };
+    nodeName?: string;
+}
+export interface K8sEndpointPort {
+    port: number;
+}
+export interface K8sEndpointSubset {
+    ports?: K8sEndpointPort[];
+    addresses?: K8sEndpointAddress[];
+    notReadyAddresses?: K8sEndpointAddress[];
+}
+export interface K8sObjectMeta {
+    name?: string;
+    annotations?: Record<string, string>;
+    labels?: Record<string, string>;
+    creationTimestamp?: string;
+}
+export interface K8sEndpoints {
+    metadata?: K8sObjectMeta;
+    subsets?: K8sEndpointSubset[];
+}
+export interface K8sService {
+    metadata?: K8sObjectMeta;
+}
+export interface K8sApiResponse<T> {
+    body: T;
+}
+export interface CoreV1ApiLike {
+    readNamespacedEndpoints(name: string, namespace: string): Promise<K8sApiResponse<K8sEndpoints>>;
+    listNamespacedService(namespace: string, pretty?: string, allowWatchBookmarks?: boolean, _continue?: string, fieldSelector?: string, labelSelector?: string): Promise<K8sApiResponse<{
+        items: K8sService[];
+    }>>;
+}
 export interface KubernetesBackendConfig {
     namespace?: string;
     labelSelector?: string;
@@ -56,10 +99,5 @@ export declare class KubernetesRegistryBackend implements RegistryBackend {
      * Create a ServiceInstance from Kubernetes endpoint data
      */
     private createServiceInstance;
-    /**
-     * Apply filter to instances
-     */
-    private applyFilter;
 }
-export {};
 //# sourceMappingURL=kubernetes-backend.d.ts.map

package/dist/backends/kubernetes-backend.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"kubernetes-backend.d.ts","sourceRoot":"","sources":["../../src/backends/kubernetes-backend.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAiB,MAAM,UAAU,CAAC;AAIzE,KAAK,UAAU,GAAG,GAAG,CAAC;AAItB,MAAM,WAAW,uBAAuB;IACtC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,qBAAa,yBAA0B,YAAW,eAAe;IAC/D,OAAO,CAAC,MAAM,CAAY;IAC1B,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAS;gBAE3B,UAAU,EAAE,UAAU,EAAE,MAAM,GAAE,uBAA4B;IASxE;;;;OAIG;IACG,QAAQ,CAAC,SAAS,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IAMzD;;;OAGG;IACG,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIpD;;;OAGG;IACG,SAAS,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAKnD;;OAEG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IA6D3F;;OAEG;IACG,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IAQtE;;OAEG;IACG,cAAc,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IAmBzC;;;OAGG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIvE;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAI9B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAsC7B;;OAEG;IACH,OAAO,CAAC,WAAW;CA2BpB"}
+{"version":3,"file":"kubernetes-backend.d.ts","sourceRoot":"","sources":["../../src/backends/kubernetes-backend.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,eAAe,EAAE,MAAM,oBAAoB,CAAC;AACrD,OAAO,EAAE,eAAe,EAAE,aAAa,EAAiB,MAAM,UAAU,CAAC;AAKzE;;;GAGG;AACH,MAAM,WAAW,UAAU;IAEzB,aAAa,CAAC,QAAQ,EAAE,UAAU,GAAG,GAAG,GAAG,CAAC;CAC7C;AAED,MAAM,WAAW,kBAAkB;IACjC,EAAE,EAAE,MAAM,CAAC;IACX,SAAS,CAAC,EAAE;QAAE,IAAI,CAAC,EAAE,MAAM,CAAA;KAAE,CAAC;IAC9B,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,WAAW,eAAe;IAC9B,IAAI,EAAE,MAAM,CAAC;CACd;AAED,MAAM,WAAW,iBAAiB;IAChC,KAAK,CAAC,EAAE,eAAe,EAAE,CAAC;IAC1B,SAAS,CAAC,EAAE,kBAAkB,EAAE,CAAC;IACjC,iBAAiB,CAAC,EAAE,kBAAkB,EAAE,CAAC;CAC1C;AAED,MAAM,WAAW,aAAa;IAC5B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,WAAW,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACrC,MAAM,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IAChC,iBAAiB,CAAC,EAAE,MAAM,CAAC;CAC5B;AAED,MAAM,WAAW,YAAY;IAC3B,QAAQ,CAAC,EAAE,aAAa,CAAC;IACzB,OAAO,CAAC,EAAE,iBAAiB,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,UAAU;IACzB,QAAQ,CAAC,EAAE,aAAa,CAAC;CAC1B;AAED,MAAM,WAAW,cAAc,CAAC,CAAC;IAC/B,IAAI,EAAE,CAAC,CAAC;CACT;AAED,MAAM,WAAW,aAAa;IAC5B,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC,YAAY,CAAC,CAAC,CAAC;IAChG,qBAAqB,CACnB,SAAS,EAAE,MAAM,EACjB,MAAM,CAAC,EAAE,MAAM,EACf,mBAAmB,CAAC,EAAE,OAAO,EAC7B,SAAS,CAAC,EAAE,MAAM,EAClB,aAAa,CAAC,EAAE,MAAM,EACtB,aAAa,CAAC,EAAE,MAAM,GACrB,OAAO,CAAC,cAAc,CAAC;QAAE,KAAK,EAAE,UAAU,EAAE,CAAA;KAAE,CAAC,CAAC,CAAC;CACrD;AAED,MAAM,WAAW,uBAAuB;IACtC,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,aAAa,CAAC,EAAE,MAAM,CAAC;CACxB;AAED,qBAAa,yBAA0B,YAAW,eAAe;IAC/D,OAAO,CAAC,MAAM,CAAgB;IAC9B,OAAO,CAAC,QAAQ,CAAC,SAAS,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAS;gBAE3B,UAAU,EAAE,UAAU,EAAE,MAAM,GAAE,uBAA4B;IAWxE;;;;OAIG;IACG,QAAQ,CAAC,SAAS,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IAMzD;;;OAGG;IACG,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIpD;;;OAGG;IACG,SAAS,CAAC,WAAW,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAKnD;;OAEG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,aAAa,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IA4D3F;;OAEG;IACG,WAAW,CAAC,UAAU,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IAQtE;;OAEG;IACG,cAAc,IAAI,OAAO,CAAC,MAAM,EAAE,CAAC;IAoBzC;;;OAGG;IACG,YAAY,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAIvE;;;OAGG;IACG,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAI9B;;OAEG;IACH,OAAO,CAAC,qBAAqB;CAiC9B"}