@futdevpro/nts-dynamo 1.15.157 → 1.15.163

package/src/_services/core/request-telemetry.control-service.spec.ts

@@ -1,217 +0,0 @@
-import { EventEmitter } from 'events';
-import { NextFunction, Request, Response } from 'express';
-
-import {
-  DyNTS_RequestRouteMetric_Interface,
-  DyNTS_RequestTelemetrySnapshot_Interface
-} from '../../_models/interfaces/request-telemetry-snapshot.interface';
-import { DyNTS_RequestTelemetry_ControlService } from './request-telemetry.control-service';
-
-class ResponseStub extends EventEmitter {
-  public readonly headers: Record<string, string> = {};
-  private responseStatusCode: number = 200;
-
-  /** Returns the response status used by completion aggregation. */
-  public get statusCode(): number {
-    return this.responseStatusCode;
-  }
-
-  /** Updates the response status used by completion aggregation. */
-  public set statusCode(value: number) {
-    this.responseStatusCode = value;
-  }
-
-  /** Stores one response header for middleware assertions. */
-  public setHeader(name: string, value: string): void {
-    this.headers[name] = value;
-  }
-
-  /** Reads one stored response header for middleware composition. */
-  public getHeader(name: string): string | undefined {
-    return this.headers[name];
-  }
-}
-
-class ThrowingHeaderResponseStub extends ResponseStub {
-  /** Simulates a response implementation rejecting header mutation. */
-  public override setHeader(_name: string, _value: string): void {
-    void _name;
-    void _value;
-
-    throw new Error('simulated header failure');
-  }
-}
-
-class ThrowingStatusResponseStub extends ResponseStub {
-  /** Simulates a late response implementation failure after listener installation. */
-  public override get statusCode(): number { // eslint-disable-line @futdevpro/dynamo/no-getter-logic
-    throw new Error('simulated status failure');
-  }
-
-  /** Accepts setup writes while the read side remains deliberately unavailable. */
-  public override set statusCode(_value: number) {}
-}
-
-describe('| BFR-CTM-020 content-free request telemetry', (): void => {
-  const service: DyNTS_RequestTelemetry_ControlService = DyNTS_RequestTelemetry_ControlService.getInstance();
-
-  beforeEach((): void => service._resetForTesting());
-
-  it('| replaces caller correlation and counts finish exactly once', (): void => {
-    service.configure({ enabled: true, durationBucketUpperBoundsMs: [ 50 ], maxRouteCardinality: 4 });
-    const req: Request = createRequest('GET');
-    const res: ResponseStub = new ResponseStub();
-    const next: NextFunction = createNext();
-
-    service.createMiddleware()(req, response(res), next);
-    service.identifyRoute('/pipeline/:pipelineId')(req, response(res), next);
-    res.statusCode = 503;
-    res.emit('finish');
-    res.emit('close');
-
-    const snapshot: DyNTS_RequestTelemetrySnapshot_Interface = service.snapshot();
-
-    expect(res.headers['X-Correlation-Id']).toMatch(/^[0-9a-f-]{36}$/u);
-    expect(res.headers['X-Correlation-Id']).not.toBe('caller-value');
-    expect(res.headers['Access-Control-Expose-Headers']).toContain('X-Correlation-Id');
-    expect(snapshot.requestsStarted).toBe(1);
-    expect(snapshot.requestsCompleted).toBe(1);
-    expect(snapshot.requestsAborted).toBe(0);
-    expect(snapshot.inFlight).toBe(0);
-    expect(snapshot.statusClasses['5xx']).toBe(1);
-    expect(snapshot.routes).toEqual([ jasmine.objectContaining({
-      method: 'GET',
-      routeTemplate: '/pipeline/:pipelineId',
-      count: 1,
-      abortedCount: 0,
-      serverErrorCount: 1,
-    }) ]);
-    expect(service.getCorrelationId(req)).toBeUndefined();
-  });
-
-  it('| records close without finish as an abort and removes both listeners', (): void => {
-    service.configure({ enabled: true });
-    const req: Request = createRequest('POST');
-    const res: ResponseStub = new ResponseStub();
-
-    service.createMiddleware()(req, response(res), createNext());
-    expect(res.listenerCount('finish')).toBe(1);
-    expect(res.listenerCount('close')).toBe(1);
-    res.emit('close');
-
-    const snapshot: DyNTS_RequestTelemetrySnapshot_Interface = service.snapshot();
-
-    expect(snapshot.requestsAborted).toBe(1);
-    expect(snapshot.requestsCompleted).toBe(0);
-    expect(snapshot.inFlight).toBe(0);
-    expect(res.listenerCount('finish')).toBe(0);
-    expect(res.listenerCount('close')).toBe(0);
-  });
-
-  it('| caps declared route cardinality and never retains invalid route content', (): void => {
-    service.configure({ enabled: true, maxRouteCardinality: 1 });
-    completeRequest(service, '/first/:id');
-    completeRequest(service, '/second/:id');
-    completeRequest(service, '/unsafe?secret=:value');
-
-    const snapshot: DyNTS_RequestTelemetrySnapshot_Interface = service.snapshot();
-
-    expect(snapshot.droppedRouteTemplates).toBe(2);
-    expect(snapshot.routes.map(
-      (route: DyNTS_RequestRouteMetric_Interface): string => route.routeTemplate
-    )).toEqual([ '__other__', '/first/:id' ]);
-    expect(JSON.stringify(snapshot)).not.toContain('secret');
-    expect(snapshot.routes.length).toBe(2);
-  });
-
-  it('| caps distinct templates without collapsing allowed methods on one retained template', (): void => {
-    service.configure({ enabled: true, maxRouteCardinality: 1 });
-    completeRequest(service, '/pipeline/:id', 'GET');
-    completeRequest(service, '/pipeline/:id', 'POST');
-    completeRequest(service, '/another/:id', 'GET');
-
-    const snapshot: DyNTS_RequestTelemetrySnapshot_Interface = service.snapshot();
-
-    expect(snapshot.droppedRouteTemplates).toBe(1);
-    expect(snapshot.routes.map(
-      (route: DyNTS_RequestRouteMetric_Interface): string => `${route.method}:${route.routeTemplate}`
-    )).toEqual([ 'GET:__other__', 'GET:/pipeline/:id', 'POST:/pipeline/:id' ]);
-  });
-
-  it('| is a no-op while disabled', (): void => {
-    service.configure({ enabled: false });
-    const req: Request = createRequest('GET');
-    const res: ResponseStub = new ResponseStub();
-
-    service.createMiddleware()(req, response(res), createNext());
-
-    expect(res.headers['X-Correlation-Id']).toBeUndefined();
-    expect(res.listenerCount('finish')).toBe(0);
-    expect(service.snapshot().requestsStarted).toBe(0);
-  });
-
-  it('| cannot leak header setup failures or create phantom in-flight requests', (): void => {
-    service.configure({ enabled: true });
-    const req: Request = createRequest('GET');
-    const res: ThrowingHeaderResponseStub = new ThrowingHeaderResponseStub();
-
-    expect((): void => service.createMiddleware()(req, response(res), createNext())).not.toThrow();
-    expect(service.snapshot().requestsStarted).toBe(0);
-    expect(service.snapshot().inFlight).toBe(0);
-  });
-
-  it('| contains late telemetry failures without changing response lifecycle semantics', (): void => {
-    service.configure({ enabled: true });
-    const req: Request = createRequest('GET');
-    const res: ThrowingStatusResponseStub = new ThrowingStatusResponseStub();
-
-    service.createMiddleware()(req, response(res), createNext());
-
-    expect((): boolean => res.emit('finish')).not.toThrow();
-    expect(service.snapshot().inFlight).toBe(0);
-  });
-
-  it('| rejects conflicting or unbounded configuration with stable codes', (): void => {
-    expect((): void => service.configure({ enabled: true, maxRouteCardinality: 0 }))
-      .toThrowError('DYNTS|REQUEST_TELEMETRY|MAX_ROUTE_CARDINALITY_INVALID');
-    expect((): void => service.configure({ enabled: true, durationBucketUpperBoundsMs: [ 100, 50 ] }))
-      .toThrowError('DYNTS|REQUEST_TELEMETRY|DURATION_BUCKETS_INVALID');
-
-    service.configure({ enabled: true });
-    expect((): void => service.configure({ enabled: true, maxRouteCardinality: 5 }))
-      .toThrowError('DYNTS|REQUEST_TELEMETRY|CONFIGURATION_CONFLICT');
-  });
-});
-
-function completeRequest(
-  service: DyNTS_RequestTelemetry_ControlService,
-  routeTemplate: string,
-  method: string = 'GET'
-): void {
-  const req: Request = createRequest(method);
-  const res: ResponseStub = new ResponseStub();
-  const next: NextFunction = createNext();
-
-  service.createMiddleware()(req, response(res), next);
-  service.identifyRoute(routeTemplate)(req, response(res), next);
-  res.emit('finish');
-}
-
-function createRequest(method: string): Request {
-  // Express's runtime object is intentionally narrowed to fields the middleware is allowed to read.
-  // eslint-disable-next-line @futdevpro/dynamo/no-as-cast
-  return { method: method, headers: { 'x-correlation-id': 'caller-value' } } as unknown as Request;
-}
-
-function response(res: ResponseStub): Response {
-  // The stub implements the exact response lifecycle/header surface used by this middleware.
-  // eslint-disable-next-line @futdevpro/dynamo/no-as-cast
-  return res as unknown as Response;
-}
-
-function createNext(): NextFunction {
-  /** Deliberately empty Express continuation used by the isolated middleware tests. */
-  const next: NextFunction = (): void => undefined;
-
-  return next;
-}

package/src/_services/core/request-telemetry.control-service.ts

@@ -1,381 +0,0 @@
-import { randomUUID } from 'crypto';
-import { NextFunction, Request, RequestHandler, Response } from 'express';
-
-import { DyNTS_resolveExposeHeaders } from '../../_collections/app-version-header.util';
-import {
-  DyNTS_RequestTelemetry_Settings
-} from '../../_models/interfaces/request-telemetry-settings.interface';
-import {
-  DyNTS_RequestDurationBucket_Interface,
-  DyNTS_RequestRouteMetric_Interface,
-  DyNTS_RequestTelemetrySnapshot_Interface
-} from '../../_models/interfaces/request-telemetry-snapshot.interface';
-
-const correlationHeader: 'X-Correlation-Id' = 'X-Correlation-Id';
-const contractVersion: 'dynts-request-telemetry/1' = 'dynts-request-telemetry/1';
-const defaultDurationBuckets: number[] = [ 25, 100, 250, 1000, 5000 ];
-const defaultMaxRouteCardinality: number = 128;
-const otherRouteTemplate: string = '__other__';
-const unmatchedRouteTemplate: string = '__unmatched__';
-const allowedMethods: ReadonlySet<string> = new Set([ 'DELETE', 'GET', 'HEAD', 'OPTIONS', 'PATCH', 'POST', 'PUT' ]);
-const routeTemplatePattern: RegExp = /^\/[A-Za-z0-9_:/.*-]{0,159}$/;
-
-interface DyNTS_RequestTelemetryResolvedSettings_Interface {
-  enabled: boolean;
-  maxRouteCardinality: number;
-  durationBucketUpperBoundsMs: number[];
-}
-
-interface DyNTS_RequestTelemetryState_Interface {
-  startedAtMs: number;
-  routeTemplate: string;
-  correlationId: string;
-}
-
-interface DyNTS_RequestRouteAccumulator_Interface extends DyNTS_RequestRouteMetric_Interface {}
-
-interface DyNTS_RequestRouteObservation_Interface {
-  method: string;
-  routeTemplate: string;
-  statusCode: number;
-  durationMs: number;
-  aborted: boolean;
-}
-
-/**
- * Event-driven, content-free request correlation and aggregate metrics.
- *
- * The service never reads or stores URL, query, headers, body, IP, user agent, account identity or error content.
- * Route labels can only come from the framework's declared route templates. Caller-provided correlation values are
- * ignored and replaced with a server-generated UUID.
- */
-export class DyNTS_RequestTelemetry_ControlService {
-  private static instance: DyNTS_RequestTelemetry_ControlService;
-
-  /** Returns the process-wide request telemetry accumulator used by open and secure Express applications. */
-  public static getInstance(): DyNTS_RequestTelemetry_ControlService {
-    if (!DyNTS_RequestTelemetry_ControlService.instance) {
-      DyNTS_RequestTelemetry_ControlService.instance = new DyNTS_RequestTelemetry_ControlService();
-    }
-
-    return DyNTS_RequestTelemetry_ControlService.instance;
-  }
-
-  private settings: DyNTS_RequestTelemetryResolvedSettings_Interface = this.resolveSettings({});
-  private configurationFingerprint: string | null = null;
-  private requestStates: WeakMap<Request, DyNTS_RequestTelemetryState_Interface> = new WeakMap();
-  private readonly routeMetrics: Map<string, DyNTS_RequestRouteAccumulator_Interface> = new Map();
-  private readonly retainedRouteTemplates: Set<string> = new Set();
-  private durationCounts: number[] = this.settings.durationBucketUpperBoundsMs.map((): number => 0);
-  private overflowDurationCount: number = 0;
-  private readonly statusClassCounts: Record<string, number> = {
-    '1xx': 0,
-    '2xx': 0,
-    '3xx': 0,
-    '4xx': 0,
-    '5xx': 0,
-    other: 0,
-  };
-  private requestsStarted: number = 0;
-  private requestsCompleted: number = 0;
-  private requestsAborted: number = 0;
-  private inFlight: number = 0;
-  private droppedRouteTemplates: number = 0;
-
-  private constructor() {}
-
-  /** Installs one immutable configuration; repeated identical calls are idempotent. */
-  public configure(settings: DyNTS_RequestTelemetry_Settings): void {
-    const resolved: DyNTS_RequestTelemetryResolvedSettings_Interface = this.resolveSettings(settings);
-    const fingerprint: string = JSON.stringify(resolved);
-
-    if (this.configurationFingerprint && this.configurationFingerprint !== fingerprint) {
-      throw new Error('DYNTS|REQUEST_TELEMETRY|CONFIGURATION_CONFLICT');
-    }
-
-    if (this.configurationFingerprint === fingerprint) {
-      return;
-    }
-
-    this.settings = resolved;
-    this.configurationFingerprint = fingerprint;
-    this.durationCounts = resolved.durationBucketUpperBoundsMs.map((): number => 0);
-  }
-
-  /** Returns the early Express middleware that creates correlation and completion listeners. */
-  public createMiddleware(): RequestHandler {
-    return (req: Request, res: Response, next: NextFunction): void => {
-      if (!this.settings.enabled) {
-        next();
-
-        return;
-      }
-
-      try {
-        this.beginRequest(req, res);
-      } catch {
-        // Telemetry is observation-only and must never change application response semantics.
-      }
-
-      next();
-    };
-  }
-
-  /** Returns a route-local middleware that labels only a framework-declared route template. */
-  public identifyRoute(routeTemplate: string): RequestHandler {
-    const safeRouteTemplate: string = this.normalizeRouteTemplate(routeTemplate);
-
-    return (req: Request, _res: Response, next: NextFunction): void => {
-      const state: DyNTS_RequestTelemetryState_Interface | undefined = this.requestStates.get(req);
-
-      if (state) {
-        state.routeTemplate = safeRouteTemplate;
-      }
-
-      next();
-    };
-  }
-
-  /** Returns the generated correlation ID for internal error/audit joins without reading request headers. */
-  public getCorrelationId(req: Request): string | undefined {
-    return this.requestStates.get(req)?.correlationId;
-  }
-
-  /** Returns a stable copy of all bounded aggregate counters. */
-  public snapshot(): DyNTS_RequestTelemetrySnapshot_Interface {
-    const durationBuckets: DyNTS_RequestDurationBucket_Interface[] = this.settings.durationBucketUpperBoundsMs.map(
-      (upperBoundMs: number, index: number): DyNTS_RequestDurationBucket_Interface => ({
-        upperBoundMs: upperBoundMs,
-        count: this.durationCounts[index],
-      })
-    );
-
-    durationBuckets.push({ upperBoundMs: null, count: this.overflowDurationCount });
-
-    return {
-      contractVersion: contractVersion,
-      correlationHeader: correlationHeader,
-      requestsStarted: this.requestsStarted,
-      requestsCompleted: this.requestsCompleted,
-      requestsAborted: this.requestsAborted,
-      inFlight: this.inFlight,
-      droppedRouteTemplates: this.droppedRouteTemplates,
-      statusClasses: { ...this.statusClassCounts },
-      durationBuckets: durationBuckets,
-      routes: [ ...this.routeMetrics.values() ]
-        .map((metric: DyNTS_RequestRouteAccumulator_Interface): DyNTS_RequestRouteMetric_Interface => ({
-          ...metric,
-        }))
-        .sort((first: DyNTS_RequestRouteMetric_Interface, second: DyNTS_RequestRouteMetric_Interface): number =>
-          `${first.method}:${first.routeTemplate}`.localeCompare(`${second.method}:${second.routeTemplate}`)
-        ),
-    };
-  }
-
-  /** Test-only reset for the process-wide singleton. */
-  public _resetForTesting(): void {
-    this.settings = this.resolveSettings({});
-    this.configurationFingerprint = null;
-    this.requestStates = new WeakMap();
-    this.routeMetrics.clear();
-    this.retainedRouteTemplates.clear();
-    this.durationCounts = this.settings.durationBucketUpperBoundsMs.map((): number => 0);
-    this.overflowDurationCount = 0;
-    Object.keys(this.statusClassCounts).forEach((key: string): void => {
-      this.statusClassCounts[key] = 0;
-    });
-    this.requestsStarted = 0;
-    this.requestsCompleted = 0;
-    this.requestsAborted = 0;
-    this.inFlight = 0;
-    this.droppedRouteTemplates = 0;
-  }
-
-  private beginRequest(req: Request, res: Response): void {
-    const state: DyNTS_RequestTelemetryState_Interface = {
-      startedAtMs: Date.now(),
-      routeTemplate: unmatchedRouteTemplate,
-      correlationId: randomUUID(),
-    };
-    const existingExposeHeader: string | number | string[] | undefined =
-      res.getHeader('Access-Control-Expose-Headers');
-
-    res.setHeader(correlationHeader, state.correlationId);
-    res.setHeader(
-      'Access-Control-Expose-Headers',
-      DyNTS_resolveExposeHeaders(
-        typeof existingExposeHeader === 'string' ? existingExposeHeader : undefined,
-        correlationHeader
-      )
-    );
-
-    let settled: boolean = false;
-    /** Settles one response exactly once and removes both lifecycle listeners. */
-    const settle: (aborted: boolean) => void = (aborted: boolean): void => {
-      if (settled) {
-        return;
-      }
-      settled = true;
-      res.off('finish', onFinish);
-      res.off('close', onClose);
-
-      try {
-        this.completeRequest(req, res, aborted);
-      } catch {
-        // Telemetry is observation-only and must never surface a response-lifecycle failure.
-      }
-    };
-    /** Records a normally completed response. */
-    const onFinish: () => void = (): void => settle(false);
-    /** Records a connection closed before a normal finish. */
-    const onClose: () => void = (): void => settle(true);
-
-    try {
-      res.once('finish', onFinish);
-      res.once('close', onClose);
-    } catch (error: unknown) {
-      res.off('finish', onFinish);
-      res.off('close', onClose);
-
-      throw error;
-    }
-
-    this.requestStates.set(req, state);
-    this.requestsStarted = this.safeIncrement(this.requestsStarted);
-    this.inFlight = this.safeIncrement(this.inFlight);
-  }
-
-  private completeRequest(req: Request, res: Response, aborted: boolean): void {
-    const state: DyNTS_RequestTelemetryState_Interface | undefined = this.requestStates.get(req);
-
-    if (!state) {
-      return;
-    }
-
-    this.requestStates.delete(req);
-    this.inFlight = Math.max(0, this.inFlight - 1);
-    const durationMs: number = Math.max(0, Date.now() - state.startedAtMs);
-
-    this.recordDuration(durationMs);
-
-    if (aborted) {
-      this.requestsAborted = this.safeIncrement(this.requestsAborted);
-    } else {
-      this.requestsCompleted = this.safeIncrement(this.requestsCompleted);
-      this.recordStatusClass(res.statusCode);
-    }
-
-    this.recordRouteMetric({
-      method: req.method,
-      routeTemplate: state.routeTemplate,
-      statusCode: res.statusCode,
-      durationMs: durationMs,
-      aborted: aborted,
-    });
-  }
-
-  private recordDuration(durationMs: number): void {
-    const bucketIndex: number = this.settings.durationBucketUpperBoundsMs.findIndex(
-      (upperBoundMs: number): boolean => durationMs <= upperBoundMs
-    );
-
-    if (bucketIndex === -1) {
-      this.overflowDurationCount = this.safeIncrement(this.overflowDurationCount);
-
-      return;
-    }
-
-    this.durationCounts[bucketIndex] = this.safeIncrement(this.durationCounts[bucketIndex]);
-  }
-
-  private recordStatusClass(statusCode: number): void {
-    const statusClass: string = Number.isInteger(statusCode) && statusCode >= 100 && statusCode <= 599
-      ? `${Math.floor(statusCode / 100)}xx`
-      : 'other';
-
-    this.statusClassCounts[statusClass] = this.safeIncrement(this.statusClassCounts[statusClass]);
-  }
-
-  private recordRouteMetric(observation: DyNTS_RequestRouteObservation_Interface): void {
-    const safeMethod: string = allowedMethods.has(observation.method) ? observation.method : 'OTHER';
-    let safeRouteTemplate: string = observation.routeTemplate;
-    let routeKey: string = `${safeMethod}:${safeRouteTemplate}`;
-    let metric: DyNTS_RequestRouteAccumulator_Interface | undefined = this.routeMetrics.get(routeKey);
-
-    if (safeRouteTemplate === otherRouteTemplate) {
-      this.droppedRouteTemplates = this.safeIncrement(this.droppedRouteTemplates);
-    } else if (safeRouteTemplate !== unmatchedRouteTemplate && !this.retainedRouteTemplates.has(safeRouteTemplate)) {
-      if (this.retainedRouteTemplates.size >= this.settings.maxRouteCardinality) {
-        this.droppedRouteTemplates = this.safeIncrement(this.droppedRouteTemplates);
-        safeRouteTemplate = otherRouteTemplate;
-        routeKey = `${safeMethod}:${safeRouteTemplate}`;
-        metric = this.routeMetrics.get(routeKey);
-      } else {
-        this.retainedRouteTemplates.add(safeRouteTemplate);
-      }
-    }
-
-    if (!metric) {
-      metric = {
-        method: safeMethod,
-        routeTemplate: safeRouteTemplate,
-        count: 0,
-        abortedCount: 0,
-        serverErrorCount: 0,
-        totalDurationMs: 0,
-        maxDurationMs: 0,
-      };
-      this.routeMetrics.set(routeKey, metric);
-    }
-
-    metric.count = this.safeIncrement(metric.count);
-    metric.abortedCount = observation.aborted ? this.safeIncrement(metric.abortedCount) : metric.abortedCount;
-    metric.serverErrorCount = !observation.aborted && observation.statusCode >= 500 && observation.statusCode <= 599
-      ? this.safeIncrement(metric.serverErrorCount)
-      : metric.serverErrorCount;
-    metric.totalDurationMs = this.safeAdd(metric.totalDurationMs, observation.durationMs);
-    metric.maxDurationMs = Math.max(metric.maxDurationMs, observation.durationMs);
-  }
-
-  private normalizeRouteTemplate(routeTemplate: string): string {
-    return routeTemplatePattern.test(routeTemplate) ? routeTemplate : otherRouteTemplate;
-  }
-
-  private resolveSettings(
-    settings: DyNTS_RequestTelemetry_Settings
-  ): DyNTS_RequestTelemetryResolvedSettings_Interface {
-    const maxRouteCardinality: number = settings.maxRouteCardinality ?? defaultMaxRouteCardinality;
-    const durationBucketUpperBoundsMs: number[] = settings.durationBucketUpperBoundsMs ?? defaultDurationBuckets;
-
-    if (!Number.isInteger(maxRouteCardinality) || maxRouteCardinality < 1 || maxRouteCardinality > 1000) {
-      throw new Error('DYNTS|REQUEST_TELEMETRY|MAX_ROUTE_CARDINALITY_INVALID');
-    }
-
-    if (
-      !durationBucketUpperBoundsMs.length ||
-      durationBucketUpperBoundsMs.length > 16 ||
-      durationBucketUpperBoundsMs.some(
-        (value: number): boolean => !Number.isInteger(value) || value < 1 || value > 3_600_000
-      ) ||
-      durationBucketUpperBoundsMs.some(
-        (value: number, index: number): boolean => index > 0 && value <= durationBucketUpperBoundsMs[index - 1]
-      )
-    ) {
-      throw new Error('DYNTS|REQUEST_TELEMETRY|DURATION_BUCKETS_INVALID');
-    }
-
-    return {
-      enabled: settings.enabled === true,
-      maxRouteCardinality: maxRouteCardinality,
-      durationBucketUpperBoundsMs: [ ...durationBucketUpperBoundsMs ],
-    };
-  }
-
-  private safeIncrement(value: number): number {
-    return value >= Number.MAX_SAFE_INTEGER ? Number.MAX_SAFE_INTEGER : value + 1;
-  }
-
-  private safeAdd(first: number, second: number): number {
-    return first >= Number.MAX_SAFE_INTEGER - second ? Number.MAX_SAFE_INTEGER : first + second;
-  }
-}

package/src/_services/core/service-collection.service.spec.ts

@@ -1,46 +0,0 @@
-
-import { DyNTS_Service_Collection } from './service-collection.service';
-
-class TestService {
-  name: string;
-  constructor(name: string) {
-    this.name = name;
-  }
-}
-
-describe('| DyNTS_Service_Collection', () => {
-  it('| should be a singleton instance', () => {
-    class TestCollection extends DyNTS_Service_Collection<TestService> {
-      static getInstance(): TestCollection {
-        return TestCollection.getSingletonInstance();
-      }
-    }
-
-    const instance1 = TestCollection.getInstance();
-    const instance2 = TestCollection.getInstance();
-
-    expect(instance1).toBe(instance2);
-    expect(instance1).toBeInstanceOf(TestCollection);
-  });
-
-  it('| should allow dynamic property assignment', () => {
-    class TestCollection extends DyNTS_Service_Collection<TestService> {
-      static getInstance(): TestCollection {
-        return TestCollection.getSingletonInstance();
-      }
-    }
-
-    const collection = TestCollection.getInstance();
-    const service1 = new TestService('service1');
-    const service2 = new TestService('service2');
-
-    collection['service1'] = service1;
-    collection['service2'] = service2;
-
-    expect(collection['service1']).toBe(service1);
-    expect(collection['service2']).toBe(service2);
-    expect(collection['service1'].name).toBe('service1');
-    expect(collection['service2'].name).toBe('service2');
-  });
-});
-

package/src/_services/core/service-collection.service.ts

@@ -1,6 +0,0 @@
-
-import { DyNTS_SingletonServiceBase } from '../base/singleton.service-base';
-
-export class DyNTS_Service_Collection<T> extends DyNTS_SingletonServiceBase {
-  [service: string]: T;
-}