@zlink-systems/framework 0.26.0 → 0.28.0

package/src/runtime/channels/channel-socket-registry.ts

@@ -1,43 +1,58 @@
-import { ZLinkListenerRecords } from '../foundation/listener-records';
+import { RuntimeDisposal } from '../disposal';
+import { UINT64_MAX } from '@zlink-systems/stream-wire';
+import { randomBytes, randomUUID } from 'node:crypto';
 import {
   ZLinkFrameworkRuntimeState,
   type RoutingId,
   type ZLinkClientServerServerDescriptor,
   type ZLinkFanoutPublisherDescriptor
 } from '../../contracts';
-import type { ZLinkChannelOptions } from '../../contracts/Configuration/RegistrationTypes';
-import { ZLinkSocketNativeEventType } from '../diagnostics/internal-event-contracts';
+import { ZLINK_MAX_ROUTING_ID_BYTES } from '../../contracts/Common/CoreTypes';
+import type { Message } from '../../contracts/Common/Message';
 import {
-  buildAdvertisedEndpoint,
-  ZLinkConfigurationException,
-  type ZLinkFrameworkRegistration
-} from '../configuration';
+  isValidPublicWeight,
+  ZLINK_DEFAULT_PUBLIC_WEIGHT,
+  ZLINK_MAX_PUBLIC_WEIGHT
+} from '../../contracts/Configuration/RegistrationBuilderPolicy';
+import type { ZLinkChannelOptions } from '../../contracts/Configuration/RegistrationTypes';
+import { attachEndpointConnections } from '../../contracts/Configuration/RuntimeEndpointConnections';
+import { createAbortError, throwIfAborted } from '../abort';
 import type {
   ZLinkBackendContext,
   ZLinkBackendDealerSocket,
   ZLinkBackendPublisherSocket,
-  ZLinkBackendRouterSocket,
   ZLinkBackendReadablePoller,
+  ZLinkBackendRouterSocket,
   ZLinkBackendSocketMonitor,
   ZLinkBackendSocketMonitorEvent,
   ZLinkBackendSubscriberSocket,
   ZLinkChannelBackendAdapter,
   ZLinkMonitoringBackendAdapter
 } from '../backend/contracts';
-import { throwIfAborted } from '../abort';
-import { attachEndpointConnections } from '../../contracts/Configuration/RuntimeEndpointConnections';
-import { ZLinkRouteMemberSnapshot } from './route-member-snapshot';
-import {
-  ZLinkFrameworkInternalErrorKind,
-  createInternalFrameworkException
-} from '../framework-errors-internal';
-import { randomBytes, randomUUID } from 'node:crypto';
 import { ZLinkBufferMessage as RuntimeMessage } from '../backend/runtime-message';
-import type { Message } from '../../contracts/Common/Message';
-import { ServiceDiscoveryRegistry } from '../foundation/service-discovery-registry';
+import { isBackendRequestTimeoutError } from '../backend/runtime-values';
+import {
+  buildAdvertisedEndpoint,
+  ZLinkConfigurationException,
+  type ZLinkFrameworkRegistration
+} from '../configuration';
+import { ZLinkSocketNativeEventType } from '../diagnostics/internal-event-contracts';
+import { ZLinkListenerRecords } from '../foundation/listener-records';
 import { discoveryAvailabilityForRuntimeState } from '../foundation/runtime-state-projections';
+import { ServiceDiscoveryRegistry } from '../foundation/service-discovery-registry';
+import {
+  DEFAULT_SERVICE_PEER_TIMEOUT_MS,
+  DEFAULT_SERVICE_PROBE_INTERVAL_MS
+} from '../foundation/service-liveness-registry';
 import { ServiceWireProtocolError } from '../foundation/service-wire-m6a-codec';
 import {
+  createInternalFrameworkException,
+  ZLinkFrameworkInternalErrorKind
+} from '../framework-errors-internal';
+import type { ApplicationJobQueue } from '../host/application-job-queue';
+import type { ZLinkChannelEnvelopeHeader } from './channel-envelope';
+import {
+  ClientServerRejectReason,
   decodeClientServerControl,
   encodeClientServerAdmit,
   encodeClientServerHello,
@@ -46,6 +61,7 @@ import {
   encodeClientServerReject,
   encodeClientServerUpdate,
   isClientServerControlFrame,
+  normalizeClientServerMessageLimit,
   type ZLinkClientServerAdmission
 } from './client-server-service-wire';
 import {
@@ -53,16 +69,13 @@ import {
   FANOUT_LIVENESS_TOPIC,
   inspectFanoutInbound
 } from './fanout-service-wire';
-import type { ZLinkChannelEnvelopeHeader } from './channel-envelope';
-import type { ApplicationJobQueue } from '../host/application-job-queue';
-import { isBackendRequestTimeoutError } from '../backend/runtime-values';
+import { ZLinkRouteMemberSnapshot } from './route-member-snapshot';
 
 const MAX_LIFECYCLE_GENERATION = 0x7fff_ffff_ffff_ffffn;
-const CLIENT_SERVER_PROBE_INTERVAL_MS = 5_000;
-const CLIENT_SERVER_PEER_DEADLINE_MS = 15_000;
-const CLIENT_SERVER_READY_WAIT_CAP_MS = 5_000;
-const CLIENT_SERVER_READY_POLL_INTERVAL_MS = 5;
-const DEFAULT_REQUEST_TIMEOUT_MS = 30_000;
+const CLIENT_SERVER_PROBE_INTERVAL_MS = DEFAULT_SERVICE_PROBE_INTERVAL_MS;
+const CLIENT_SERVER_PEER_DEADLINE_MS = DEFAULT_SERVICE_PEER_TIMEOUT_MS;
+const CLIENT_SERVER_LIVENESS_TICK_MS = 100;
+const DEFAULT_SEND_TIMEOUT_MS = 1_000;
 
 export interface ZLinkClientServerServerSocketIdentity {
   readonly serverRid: string;
@@ -81,8 +94,8 @@ interface ClientServerPhysicalConnection {
   readonly channelName: string;
   readonly endpoint: string;
   readonly dealer: ZLinkBackendDealerSocket;
-  readonly readablePoller: ZLinkBackendReadablePoller;
-  readonly monitor: ZLinkBackendSocketMonitor;
+  readablePoller?: ZLinkBackendReadablePoller;
+  monitor?: ZLinkBackendSocketMonitor;
   readonly aliases: Set<string>;
   readonly callbacksByAlias: Map<string, ZLinkClientServerConnectionCallbacks>;
   physicalConnectionId: symbol;
@@ -103,7 +116,7 @@ interface FanoutPublisherConnection {
   readonly channelName: string;
   readonly endpoint: string;
   readonly subscriber: ZLinkBackendSubscriberSocket;
-  readonly monitor: ZLinkBackendSocketMonitor;
+  monitor?: ZLinkBackendSocketMonitor;
   readonly callbacks: ZLinkFanoutConnectionCallbacks;
   readonly physicalConnectionId: symbol;
   transportReady: boolean;
@@ -155,14 +168,20 @@ export class ZLinkChannelSocketRegistry {
       outstandingProbeId?: bigint;
     }
   >();
-  private readonly ownedMonitors = new Set<ZLinkBackendSocketMonitor>();
+  private readonly disposal = new RuntimeDisposal();
+  private readonly ownedResources = new Set<
+    | ZLinkBackendSocketMonitor
+    | ZLinkBackendReadablePoller
+    | ZLinkBackendDealerSocket
+    | ZLinkBackendSubscriberSocket
+  >();
   private readonly fanoutConnections = new Map<string, FanoutPublisherConnection>();
   private readonly fanoutPublisherNextBeacon = new Map<string, number>();
   private clientServerLivenessTimer?: NodeJS.Timeout;
   private nextClientServerProbeId = 1n;
   private readonly clientServerMonitorHandlers = new Map<
     string,
-    Set<(event: ZLinkBackendSocketMonitorEvent) => void>
+    Set<(event?: ZLinkBackendSocketMonitorEvent) => void>
   >();
   private readonly fanoutMonitorHandlers = new Map<
     string,
@@ -184,10 +203,20 @@ export class ZLinkChannelSocketRegistry {
     this.listenerRecords = listenerRecords ?? new ZLinkListenerRecords();
   }
 
-  async dispose(): Promise<void> {
+  dispose(): Promise<void> {
+    return this.disposal.run(() => this.disposeCore());
+  }
+
+  private async disposeCore(): Promise<void> {
+    this.clientServerDiscovery.dispose(
+      createInternalFrameworkException(
+        ZLinkFrameworkInternalErrorKind.RuntimeShutdown,
+        'Channel sockets are shutting down.'
+      )
+    );
     const clientServerConnections = [...new Set(this.clientServerConnections.values())];
-    const clientServerPollers = clientServerConnections.map(
-      (connection) => connection.readablePoller
+    const clientServerPollers = clientServerConnections.flatMap((connection) =>
+      connection.readablePoller === undefined ? [] : [connection.readablePoller]
     );
     const sockets = [
       ...this.clientDealers.values(),
@@ -204,8 +233,6 @@ export class ZLinkChannelSocketRegistry {
       ...this.routeRouters.values()
     ]))
       this.unregisterReceiveFlowSocket(socket);
-    this.clientDealers.clear();
-    this.clientServerConnections.clear();
     this.clientServerReadyIdentities.clear();
     this.clientServerServerDescriptors.clear();
     this.clientServerPublicWeights.clear();
@@ -215,23 +242,37 @@ export class ZLinkChannelSocketRegistry {
     this.clientServerMonitorHandlers.clear();
     this.fanoutMonitorHandlers.clear();
     this.fanoutTopologyHandlers.clear();
-    this.fanoutConnections.clear();
     this.fanoutPublisherNextBeacon.clear();
     if (this.clientServerLivenessTimer !== undefined) {
       clearInterval(this.clientServerLivenessTimer);
       this.clientServerLivenessTimer = undefined;
     }
-    this.channelRouters.clear();
-    this.publishers.clear();
-    this.routeRouters.clear();
     this.routeMembers.clear();
-    const monitors = [...this.ownedMonitors];
-    this.ownedMonitors.clear();
-    const cleanup = await Promise.allSettled([
-      ...clientServerPollers.map((poller) => poller.dispose()),
-      ...monitors.map((monitor) => monitor.dispose()),
-      ...sockets.map((socket) => socket.dispose())
-    ]);
+    const cleanup = await Promise.allSettled(
+      [...new Set([...clientServerPollers, ...this.ownedResources, ...sockets])].map(
+        async (resource) => {
+          await resource.dispose();
+          this.ownedResources.delete(resource as ZLinkBackendSocketMonitor);
+          for (const owners of [
+            this.clientDealers,
+            this.channelRouters,
+            this.publishers,
+            this.routeRouters
+          ]) {
+            for (const [name, owned] of owners) if (owned === resource) owners.delete(name);
+          }
+          for (const [name, owned] of this.clientServerConnections) {
+            if (owned.dealer === resource) this.clientServerConnections.delete(name);
+            if (owned.readablePoller === resource) owned.readablePoller = undefined;
+            if (owned.monitor === resource) owned.monitor = undefined;
+          }
+          for (const [name, owned] of this.fanoutConnections) {
+            if (owned.subscriber === resource) this.fanoutConnections.delete(name);
+            if (owned.monitor === resource) owned.monitor = undefined;
+          }
+        }
+      )
+    );
     const errors = cleanup
       .filter((result): result is PromiseRejectedResult => result.status === 'rejected')
       .map((result) => result.reason);
@@ -252,13 +293,15 @@ export class ZLinkChannelSocketRegistry {
     }
 
     const dealer = this.adapter.createDealerSocket(this.context);
+    this.clientDealers.set(channelName, dealer);
     this.registerReceiveFlowSocket(dealer);
     dealer.setChannelName(channelName);
     if (channel?.routingId !== undefined && channel.routingId.length > 0) {
       dealer.setRoutingId(deriveRoutingId(channel.routingId, 'dealer'));
     }
-    applySocketConfig(dealer, client);
-    this.clientDealers.set(channelName, dealer);
+    applySocketConfig(dealer, {
+      ...client
+    });
     return dealer;
   }
 
@@ -279,6 +322,7 @@ export class ZLinkChannelSocketRegistry {
     }
 
     const router = this.adapter.createRouterSocket(this.context);
+    this.channelRouters.set(channelName, router);
     this.registerReceiveFlowSocket(router);
     router.setChannelName(channelName);
     const identity = this.clientServerIdentities.get(channelName) ?? {
@@ -287,14 +331,14 @@ export class ZLinkChannelSocketRegistry {
     };
     this.clientServerIdentities.set(channelName, identity);
     router.setRoutingId(identity.serverRid);
-    const publicWeight = channel.server.weight ?? 100;
+    const publicWeight = channel.server.weight ?? ZLINK_DEFAULT_PUBLIC_WEIGHT;
     this.clientServerPublicWeights.set(channelName, publicWeight);
     router.peerWeight = rawAvailabilityWeight(publicWeight);
     applySocketConfig(router, channel.server);
     router.bind(channel.server.bind);
     if (this.monitoringAdapter !== undefined) {
       const monitor = this.monitoringAdapter.openSocketMonitor(router);
-      this.ownedMonitors.add(monitor);
+      this.ownedResources.add(monitor);
       monitor.onEvent((event) => {
         if (event.routingId === undefined) return;
         if (
@@ -310,7 +354,6 @@ export class ZLinkChannelSocketRegistry {
         this.clientServerAdmittedClients.get(channelName)?.delete(routingId);
       });
     }
-    this.channelRouters.set(channelName, router);
     const endpoint = advertisedEndpoint(
       router.lastEndpoint ?? channel.server.bind,
       channel.server.advertiseHost
@@ -358,7 +401,7 @@ export class ZLinkChannelSocketRegistry {
 
   clientServerServerWeight(channelName: string): number {
     this.channelRouter(channelName);
-    return this.clientServerPublicWeights.get(channelName) ?? 100;
+    return this.clientServerPublicWeights.get(channelName) ?? ZLINK_DEFAULT_PUBLIC_WEIGHT;
   }
 
   setClientServerServerWeight(channelName: string, weight: number): void {
@@ -390,46 +433,37 @@ export class ZLinkChannelSocketRegistry {
       throw new ZLinkConfigurationException(`Channel client '${channelName}' is not registered.`);
     }
     const dealer = this.adapter.createDealerSocket(this.context);
-    this.registerReceiveFlowSocket(dealer);
-    dealer.setChannelName(channelName);
-    dealer.setRoutingId(`cs-client-${randomUUID()}`);
-    applySocketConfig(dealer, client);
-    const cleanupDealer = (): void => {
-      this.unregisterReceiveFlowSocket(dealer);
-      void Promise.allSettled([dealer.dispose()]);
-    };
-    let readablePoller: ZLinkBackendReadablePoller;
-    try {
-      readablePoller = this.adapter.createReadablePoller(dealer);
-    } catch (error) {
-      cleanupDealer();
-      throw error;
-    }
-    let monitor: ZLinkBackendSocketMonitor;
-    try {
-      monitor = this.monitoringAdapter.openSocketMonitor(dealer);
-    } catch (error) {
-      try {
-        readablePoller.dispose();
-      } finally {
-        cleanupDealer();
-      }
-      throw error;
-    }
-    this.ownedMonitors.add(monitor);
     const connection: ClientServerPhysicalConnection = {
       channelName,
       endpoint,
       dealer,
-      readablePoller,
-      monitor,
       aliases: new Set([connectionId]),
       callbacksByAlias: new Map([[connectionId, callbacks]]),
       physicalConnectionId: Symbol(connectionId)
     };
-    this.clientServerConnections.set(connectionId, connection);
-    this.ensureClientServerLivenessTimer();
+    this.ownedResources.add(dealer);
+    const created: (
+      | ZLinkBackendSocketMonitor
+      | ZLinkBackendReadablePoller
+      | ZLinkBackendDealerSocket
+      | ZLinkBackendSubscriberSocket
+    )[] = [dealer];
     try {
+      this.clientServerConnections.set(connectionId, connection);
+      this.registerReceiveFlowSocket(dealer);
+      dealer.setChannelName(channelName);
+      dealer.setRoutingId(`cs-client-${randomUUID()}`);
+      applySocketConfig(dealer, {
+        ...client
+      });
+      connection.readablePoller = this.adapter.createReadablePoller(dealer);
+      created.push(connection.readablePoller);
+      this.ownedResources.add(connection.readablePoller);
+      const monitor = this.monitoringAdapter.openSocketMonitor(dealer);
+      connection.monitor = monitor;
+      created.push(monitor);
+      this.ownedResources.add(monitor);
+      this.ensureClientServerLivenessTimer();
       monitor.onEvent((event) => {
         if (
           ![...connection.aliases].some(
@@ -464,24 +498,26 @@ export class ZLinkChannelSocketRegistry {
         }
       });
       dealer.connect(endpoint);
+      return dealer;
     } catch (error) {
       this.clientServerConnections.delete(connectionId);
-      this.ownedMonitors.delete(monitor);
-      try {
-        readablePoller.dispose();
-      } finally {
-        cleanupDealer();
-        void Promise.allSettled([monitor.dispose()]);
-      }
+      this.unregisterReceiveFlowSocket(dealer);
+      void Promise.allSettled(
+        created.map(async (resource) => {
+          await resource.dispose();
+          this.ownedResources.delete(resource);
+        })
+      ).then((results) => {
+        for (const result of results)
+          if (result.status === 'rejected') this.oneWayFailureSink?.(result.reason);
+      });
       throw error;
     }
-    return dealer;
   }
 
   async closeClientServerConnection(connectionId: string): Promise<void> {
     const current = this.clientServerConnections.get(connectionId);
     if (current === undefined) return;
-    this.clientServerConnections.delete(connectionId);
     current.aliases.delete(connectionId);
     current.callbacksByAlias.delete(connectionId);
     const ready = this.clientServerReadyIdentities.get(connectionId);
@@ -507,8 +543,13 @@ export class ZLinkChannelSocketRegistry {
         });
       }
     }
-    if (current.aliases.size > 0) return;
+    this.notifyClientServerTopology(current.channelName);
+    if (current.aliases.size > 0) {
+      this.clientServerConnections.delete(connectionId);
+      return;
+    }
     await this.disposeClientServerPhysical(connectionId, current);
+    this.clientServerConnections.delete(connectionId);
   }
 
   private async disposeClientServerPhysical(
@@ -518,12 +559,21 @@ export class ZLinkChannelSocketRegistry {
     try {
       current.dealer.disconnect(current.endpoint);
     } catch {
-      // Disposal below is the terminal cleanup if the transport already closed.
+      // The socket close below releases an endpoint that is already disconnected.
     }
-    this.ownedMonitors.delete(current.monitor);
     this.unregisterReceiveFlowSocket(current.dealer);
-    current.readablePoller.dispose();
-    const results = await Promise.allSettled([current.monitor.dispose(), current.dealer.dispose()]);
+    if (current.readablePoller !== undefined) {
+      current.readablePoller.dispose();
+      this.ownedResources.delete(current.readablePoller);
+      current.readablePoller = undefined;
+    }
+    const results = await Promise.allSettled([
+      current.monitor?.dispose(),
+      (async () => {
+        await current.dealer.dispose();
+        this.ownedResources.delete(current.dealer);
+      })()
+    ]);
     const errors = results
       .filter((result): result is PromiseRejectedResult => result.status === 'rejected')
       .map((result) => result.reason);
@@ -531,6 +581,7 @@ export class ZLinkChannelSocketRegistry {
     if (errors.length > 1) {
       throw new AggregateError(errors, `ClientServer connection '${connectionId}' cleanup failed.`);
     }
+    if (current.monitor !== undefined) this.ownedResources.delete(current.monitor);
   }
 
   admitClientServerConnection(
@@ -545,6 +596,7 @@ export class ZLinkChannelSocketRegistry {
         connection.readyConnectionId
       );
       if (admitted) connection.admittedDescriptor = descriptor;
+      this.notifyClientServerTopology(descriptor.channelName);
       return admitted;
     }
     const duplicateId = [...this.clientServerReadyIdentities].find(
@@ -582,6 +634,7 @@ export class ZLinkChannelSocketRegistry {
       connection.readyConnectionId = connectionId;
       connection.admittedDescriptor = descriptor;
     }
+    this.notifyClientServerTopology(descriptor.channelName);
     return admitted;
   }
 
@@ -605,6 +658,7 @@ export class ZLinkChannelSocketRegistry {
         connection.outstandingProbeId = undefined;
       }
     }
+    this.notifyClientServerTopology(channelName);
     return removed;
   }
 
@@ -636,52 +690,52 @@ export class ZLinkChannelSocketRegistry {
     return undefined;
   }
 
-  /**
-   * Resolves a ClientServer send target, waiting when the ready candidate set is still empty, as
-   * `framework/doc/framework/common/spec/server/02-channel-transport/02-channel-messaging.ko.md` §3.2 requires: the call waits
-   * at call time for a bounded period and then fails with no-target. The bound is the shorter of
-   * the call's remaining deadline and five seconds, mirroring the .NET reference
-   * `ZLinkClientServerClientRuntime.WaitForReadyAsync`. The wait lives here because this registry
-   * owns ClientServer connections, admission and weighted selection. Sends have no per-call
-   * timeout, so they use the Channel's request timeout.
-   *
-   * The wait only observes admission that is already in flight: it never opens or reconnects a
-   * connection, so it cannot trigger the admission it waits for, and Framework startup keeps not
-   * waiting for local ClientServer admission. It applies whenever nothing is selectable, including
-   * candidate sets excluded by weight 0 or drain, because `selectClientServerDealer` filters the
-   * same way the reference `SelectReady()` does.
-   *
-   * Every attempt yields to the event loop between polls. ClientServer admission completes on the
-   * monitor drains that the backend poll timer pumps, so a synchronous wait would starve the
-   * admission it is waiting for and burn the whole bound before failing anyway. That is this
-   * execution model's form of the hazard the JVM mirror avoids by not holding the admission
-   * monitor across its sleep.
-   */
+  /** Observes discovery changes until a server is ready or the runtime closes. */
   async awaitClientDealerForOutbound(
     channelName: string,
-    signal?: AbortSignal,
-    deadlineAtMs?: number
+    signal?: AbortSignal
   ): Promise<ZLinkBackendDealerSocket | undefined> {
-    const startedAtMs = performance.now();
-    const deadline = Math.min(
-      deadlineAtMs ??
-        startedAtMs +
-          (this.registration.channels.get(channelName)?.requestTimeoutMs ??
-            this.registration.requestTimeoutMs ??
-            DEFAULT_REQUEST_TIMEOUT_MS),
-      startedAtMs + CLIENT_SERVER_READY_WAIT_CAP_MS
-    );
-    for (;;) {
-      this.drainSocketMonitors();
+    throwIfAborted(signal);
+    const select = (): ZLinkBackendDealerSocket | undefined => {
       const dealer = this.clientDealerForOutbound(channelName);
       if (dealer !== undefined) return dealer;
-      const remainingMs = deadline - performance.now();
-      if (remainingMs <= 0) return undefined;
-      throwIfAborted(signal);
-      await new Promise<void>((resolve) => {
-        setTimeout(resolve, Math.min(CLIENT_SERVER_READY_POLL_INTERVAL_MS, remainingMs));
-      });
-    }
+      if (this.clientServerDiscovery.hasReadyClientServer(channelName)) {
+        throw createInternalFrameworkException(
+          ZLinkFrameworkInternalErrorKind.RouteNotConnected,
+          `Channel '${channelName}' has no eligible ClientServer member.`
+        );
+      }
+      return undefined;
+    };
+    const dealer = select();
+    if (dealer !== undefined) return dealer;
+    return new Promise((resolve, reject) => {
+      const cleanup = (): void => {
+        unsubscribe();
+        signal?.removeEventListener('abort', abort);
+      };
+      const abort = (): void => {
+        cleanup();
+        reject(createAbortError());
+      };
+      const changed = (error?: Error): void => {
+        try {
+          if (error !== undefined) throw error;
+          const selected = select();
+          if (selected === undefined) return;
+          cleanup();
+          resolve(selected);
+        } catch (error) {
+          cleanup();
+          reject(error);
+        }
+      };
+      const unsubscribe = this.clientServerDiscovery.onClientServerChanged(changed);
+      signal?.addEventListener('abort', abort, { once: true });
+      // Registration and selection share one event-loop turn; recheck after subscribing.
+      changed();
+      if (signal?.aborted === true) abort();
+    });
   }
 
   startManualClientServerConnections(): void {
@@ -716,26 +770,41 @@ export class ZLinkChannelSocketRegistry {
     }
   }
 
-  clientServerMonitoringSource(channelName: string): ZLinkBackendSocketMonitor {
-    if (this.registration.channels.get(channelName)?.client === undefined) {
-      throw new ZLinkConfigurationException(`Channel client '${channelName}' is not registered.`);
+  clientServerMonitoringSource(channelName: string): ZLinkBackendSocketMonitor & {
+    onChange(handler: () => void): void;
+  } {
+    const channel = this.registration.channels.get(channelName);
+    if (channel?.client === undefined && channel?.server === undefined) {
+      throw new ZLinkConfigurationException(
+        `ClientServer channel '${channelName}' is not registered.`
+      );
     }
     let disposed = false;
-    let handler: ((event: ZLinkBackendSocketMonitorEvent) => void) | undefined;
+    let handler: ((event?: ZLinkBackendSocketMonitorEvent) => void) | undefined;
+    const register = (next: (event?: ZLinkBackendSocketMonitorEvent) => void): void => {
+      if (disposed) return;
+      if (handler !== undefined) {
+        this.clientServerMonitorHandlers.get(channelName)?.delete(handler);
+      }
+      handler = next;
+      let handlers = this.clientServerMonitorHandlers.get(channelName);
+      if (handlers === undefined) {
+        handlers = new Set();
+        this.clientServerMonitorHandlers.set(channelName, handlers);
+      }
+      handlers.add(next);
+    };
     return {
       nativeInstance: {},
       onEvent: (next) => {
-        if (disposed) return;
-        if (handler !== undefined) {
-          this.clientServerMonitorHandlers.get(channelName)?.delete(handler);
-        }
-        handler = next;
-        let handlers = this.clientServerMonitorHandlers.get(channelName);
-        if (handlers === undefined) {
-          handlers = new Set();
-          this.clientServerMonitorHandlers.set(channelName, handlers);
-        }
-        handlers.add(next);
+        register((event) => {
+          if (event !== undefined) next(event);
+        });
+      },
+      onChange: (next) => {
+        register((event) => {
+          if (event === undefined) next();
+        });
       },
       drain: () => 0,
       dispose: async () => {
@@ -822,6 +891,11 @@ export class ZLinkChannelSocketRegistry {
       this.clientServerServerDescriptors.set(channelName, descriptor);
       this.pushClientServerDescriptorUpdate(channelName, descriptor);
     }
+    this.notifyClientServerTopology(channelName);
+  }
+
+  private notifyClientServerTopology(channelName: string): void {
+    for (const handler of this.clientServerMonitorHandlers.get(channelName) ?? []) handler();
   }
 
   tryHandleClientServerControl(
@@ -869,7 +943,7 @@ export class ZLinkChannelSocketRegistry {
         received.parts.length !== 1 ||
         received.replyToken === null
       ) {
-        reply = encodeClientServerReject(1);
+        reply = encodeClientServerReject(ClientServerRejectReason.ProtocolVersionUnsupported);
       } else {
         const descriptor = this.clientServerServerDescriptors.get(channelName);
         if (
@@ -877,10 +951,10 @@ export class ZLinkChannelSocketRegistry {
           record.hello.channelName !== channelName ||
           record.hello.securityIdentity !== descriptor.securityIdentity
         ) {
-          reply = encodeClientServerReject(3);
+          reply = encodeClientServerReject(ClientServerRejectReason.AdmissionMismatch);
         } else {
           const normalizedEffectiveMaxMessageBytes = Math.min(
-            normalizedMessageLimit(router.maxMessageSize),
+            normalizeClientServerMessageLimit(router.maxMessageSize),
             record.hello.normalizedEffectiveMaxMessageBytes
           );
           reply = encodeClientServerAdmit(descriptor, normalizedEffectiveMaxMessageBytes);
@@ -966,23 +1040,31 @@ export class ZLinkChannelSocketRegistry {
       );
     }
     const subscriber = this.adapter.createSubscriberSocket(this.context);
-    subscriber.setChannelName(channelName);
-    setFanoutSubscriptions(subscriber, channel.subscriptions);
-    const monitor = this.monitoringAdapter.openSocketMonitor(subscriber);
     const connection: FanoutPublisherConnection = {
       channelName,
       endpoint,
       subscriber,
-      monitor,
       callbacks,
       physicalConnectionId: Symbol(connectionId),
       transportReady: false,
       ready: false
     };
-    this.fanoutConnections.set(connectionId, connection);
-    this.ownedMonitors.add(monitor);
-    this.ensureClientServerLivenessTimer();
+    this.ownedResources.add(subscriber);
+    const created: (
+      | ZLinkBackendSocketMonitor
+      | ZLinkBackendReadablePoller
+      | ZLinkBackendDealerSocket
+      | ZLinkBackendSubscriberSocket
+    )[] = [subscriber];
     try {
+      this.fanoutConnections.set(connectionId, connection);
+      subscriber.setChannelName(channelName);
+      setFanoutSubscriptions(subscriber, channel.subscriptions);
+      const monitor = this.monitoringAdapter.openSocketMonitor(subscriber);
+      connection.monitor = monitor;
+      created.push(monitor);
+      this.ownedResources.add(monitor);
+      this.ensureClientServerLivenessTimer();
       monitor.onEvent((event) => {
         if (this.fanoutConnections.get(connectionId) !== connection) return;
         for (const handler of this.fanoutMonitorHandlers.get(channelName) ?? []) {
@@ -1006,28 +1088,36 @@ export class ZLinkChannelSocketRegistry {
         }
       });
       subscriber.connect(endpoint);
+      return subscriber;
     } catch (error) {
       this.fanoutConnections.delete(connectionId);
-      this.ownedMonitors.delete(monitor);
-      void Promise.allSettled([monitor.dispose(), subscriber.dispose()]);
+      void Promise.allSettled(
+        created.map(async (resource) => {
+          await resource.dispose();
+          this.ownedResources.delete(resource);
+        })
+      ).then((results) => {
+        for (const result of results)
+          if (result.status === 'rejected') this.oneWayFailureSink?.(result.reason);
+      });
       throw error;
     }
-    return subscriber;
   }
 
   async closeFanoutSubscriberConnection(connectionId: string): Promise<void> {
     const connection = this.fanoutConnections.get(connectionId);
     if (connection === undefined) return;
-    this.fanoutConnections.delete(connectionId);
     try {
       connection.subscriber.disconnect(connection.endpoint);
     } catch {
-      // Disposal remains the terminal cleanup path.
+      // The socket close below releases an endpoint that is already disconnected.
     }
-    this.ownedMonitors.delete(connection.monitor);
     const results = await Promise.allSettled([
-      connection.monitor.dispose(),
-      connection.subscriber.dispose()
+      connection.monitor?.dispose(),
+      (async () => {
+        await connection.subscriber.dispose();
+        this.ownedResources.delete(connection.subscriber);
+      })()
     ]);
     const errors = results
       .filter((result): result is PromiseRejectedResult => result.status === 'rejected')
@@ -1036,6 +1126,8 @@ export class ZLinkChannelSocketRegistry {
     if (errors.length > 1) {
       throw new AggregateError(errors, `Fanout connection '${connectionId}' cleanup failed.`);
     }
+    this.fanoutConnections.delete(connectionId);
+    if (connection.monitor !== undefined) this.ownedResources.delete(connection.monitor);
   }
 
   handleFanoutInbound(
@@ -1156,7 +1248,7 @@ export class ZLinkChannelSocketRegistry {
     };
   }
 
-  private notifyFanoutTopology(channelName: string): void {
+  notifyFanoutTopology(channelName: string): void {
     for (const handler of this.fanoutTopologyHandlers.get(channelName) ?? []) {
       try {
         handler();
@@ -1285,7 +1377,7 @@ export class ZLinkChannelSocketRegistry {
     connection: ClientServerPhysicalConnection
   ): void {
     for (;;) {
-      if (!connection.readablePoller.wait(0)) return;
+      if (connection.readablePoller === undefined || !connection.readablePoller.wait(0)) return;
       const received = connection.dealer.recv(1);
       if (received === undefined) {
         connection.readablePoller.markDrained();
@@ -1417,12 +1509,15 @@ export class ZLinkChannelSocketRegistry {
 
   private ensureClientServerLivenessTimer(): void {
     if (this.clientServerLivenessTimer !== undefined) return;
-    this.clientServerLivenessTimer = setInterval(() => this.tickClientServerLiveness(), 100);
+    this.clientServerLivenessTimer = setInterval(
+      () => this.tickClientServerLiveness(),
+      CLIENT_SERVER_LIVENESS_TICK_MS
+    );
     this.clientServerLivenessTimer.unref();
   }
 
   private drainSocketMonitors(): void {
-    for (const monitor of this.ownedMonitors) monitor.drain();
+    for (const monitor of this.ownedResources) if ('drain' in monitor) monitor.drain();
   }
 
   private removeReadyConnection(connectionId: string): void {
@@ -1443,6 +1538,7 @@ export class ZLinkChannelSocketRegistry {
       connection.nextProbeAt = undefined;
       connection.outstandingProbeId = undefined;
     }
+    this.notifyClientServerTopology(identity.channelName);
   }
 
   private requestClientServerLiveness(
@@ -1590,7 +1686,7 @@ export class ZLinkChannelSocketRegistry {
 
   private allocateClientServerProbeId(): bigint {
     const result = this.nextClientServerProbeId;
-    this.nextClientServerProbeId = result === 0xffff_ffff_ffff_ffffn ? 1n : result + 1n;
+    this.nextClientServerProbeId = result === UINT64_MAX ? 1n : result + 1n;
     return result;
   }
 
@@ -1613,10 +1709,10 @@ export class ZLinkChannelSocketRegistry {
     }
 
     const publisher = this.adapter.createPublisherSocket(this.context);
+    this.publishers.set(channelName, publisher);
     publisher.setChannelName(channelName);
     applyFanoutPublisherSocketOptions(publisher, channel);
     publisher.bind(channel.publisher.bind);
-    this.publishers.set(channelName, publisher);
     const endpoint = buildAdvertisedEndpoint(
       publisher.lastEndpoint ?? channel.publisher.bind,
       channel.publisher.advertiseHost
@@ -1645,6 +1741,7 @@ export class ZLinkChannelSocketRegistry {
       );
     }
     const router = this.adapter.createRouterSocket(this.context);
+    this.routeRouters.set(routerChannelId, router);
     this.registerReceiveFlowSocket(router);
     router.setChannelName(routerChannelId);
     if (
@@ -1656,7 +1753,7 @@ export class ZLinkChannelSocketRegistry {
     router.setRoutingId(
       routeChannel.routingId ?? `${routeChannel.routingIdPrefix ?? routerChannelId}-${randomUUID()}`
     );
-    const publicWeight = routeChannel.weight ?? 100;
+    const publicWeight = routeChannel.weight ?? ZLINK_DEFAULT_PUBLIC_WEIGHT;
     this.routeMeshPublicWeights.set(routerChannelId, publicWeight);
     router.peerWeight = rawAvailabilityWeight(publicWeight);
     applySocketConfig(router, routeChannel);
@@ -1670,7 +1767,6 @@ export class ZLinkChannelSocketRegistry {
     if (routeChannel.bind !== undefined && routeChannel.bind.trim().length > 0) {
       router.bind(routeChannel.bind);
     }
-    this.routeRouters.set(routerChannelId, router);
     return router;
   }
 
@@ -1680,7 +1776,7 @@ export class ZLinkChannelSocketRegistry {
 
   routeMeshWeight(routerChannelId: string): number {
     this.routeRouter(routerChannelId);
-    return this.routeMeshPublicWeights.get(routerChannelId) ?? 100;
+    return this.routeMeshPublicWeights.get(routerChannelId) ?? ZLINK_DEFAULT_PUBLIC_WEIGHT;
   }
 
   setRouteMeshWeight(routerChannelId: string, weight: number): void {
@@ -1717,7 +1813,7 @@ export class ZLinkChannelSocketRegistry {
       return;
     }
     const monitor = this.monitoringAdapter.openSocketMonitor(socket);
-    this.ownedMonitors.add(monitor);
+    this.ownedResources.add(monitor);
     monitor.onEvent((event) => {
       if (
         event.nativeEvent === ZLinkSocketNativeEventType.Disconnected &&
@@ -1733,7 +1829,7 @@ export class ZLinkChannelSocketRegistry {
       return;
     }
     const monitor = this.monitoringAdapter.openSocketMonitor(router);
-    this.ownedMonitors.add(monitor);
+    this.ownedResources.add(monitor);
     monitor.onEvent((event) => {
       const routingId = event.routingId === undefined ? undefined : String(event.routingId);
       if (
@@ -1775,7 +1871,7 @@ async function requestClientServerAdmission(
     encodeClientServerHello({
       channelName,
       securityIdentity,
-      normalizedEffectiveMaxMessageBytes: normalizedMessageLimit(dealer.maxMessageSize)
+      normalizedEffectiveMaxMessageBytes: normalizeClientServerMessageLimit(dealer.maxMessageSize)
     })
   );
   try {
@@ -1853,9 +1949,9 @@ function setFanoutSubscriptions(
 
 function deriveRoutingId(baseRoutingId: string, suffix: string): string {
   const derived = `${baseRoutingId}\0${suffix}`;
-  if (Buffer.byteLength(derived, 'utf8') > 255) {
+  if (Buffer.byteLength(derived, 'utf8') > ZLINK_MAX_ROUTING_ID_BYTES) {
     throw new ZLinkConfigurationException(
-      `Derived routing id with suffix '${suffix}' exceeds the 255 byte limit.`
+      `Derived routing id with suffix '${suffix}' exceeds the ${ZLINK_MAX_ROUTING_ID_BYTES} byte limit.`
     );
   }
   return derived;
@@ -1866,6 +1962,7 @@ function applyFanoutPublisherSocketOptions(
   channel: ZLinkChannelOptions
 ): void {
   publisher.noDrop = channel.noDrop ?? false;
+  publisher.sendTimeoutMs = configuredSendTimeoutMs(channel.publisher?.sendTimeoutMs);
 }
 
 function fanoutDiscoveryConnectionId(connectionId: string): string {
@@ -1889,13 +1986,11 @@ function advertisedEndpoint(boundEndpoint: string, advertiseHost: string | undef
   return result;
 }
 
-function normalizedMessageLimit(value: number): number {
-  return Number.isSafeInteger(value) && value > 0 ? Math.min(value, 0xffff_ffff) : 0x7fff_ffff;
-}
-
 function requirePublicWeight(weight: number): void {
-  if (!Number.isInteger(weight) || weight < 0 || weight > 10_000) {
-    throw new ZLinkConfigurationException('Weight must be an integer in 0..10000.');
+  if (!isValidPublicWeight(weight)) {
+    throw new ZLinkConfigurationException(
+      `Weight must be an integer in 0..${ZLINK_MAX_PUBLIC_WEIGHT}.`
+    );
   }
 }
 
@@ -1911,7 +2006,6 @@ function applySocketConfig(
     readonly sendHighWaterMark?: number;
     readonly receiveHighWaterMark?: number;
     readonly receiveTimeoutMs?: number;
-    readonly sendTimeoutMs?: number;
     readonly maxMessageSize?: number;
   }
 ): void {
@@ -1924,10 +2018,11 @@ function applySocketConfig(
   if (config.receiveTimeoutMs !== undefined) {
     socket.receiveTimeoutMs = config.receiveTimeoutMs;
   }
-  if (config.sendTimeoutMs !== undefined) {
-    socket.sendTimeoutMs = config.sendTimeoutMs;
-  }
   if (config.maxMessageSize !== undefined) {
     socket.maxMessageSize = config.maxMessageSize;
   }
 }
+
+function configuredSendTimeoutMs(value: number | undefined): number {
+  return value ?? DEFAULT_SEND_TIMEOUT_MS;
+}