@hkdigital/lib-core 0.4.42 → 0.4.43

package/dist/services/service-manager/ServiceManager.d.ts

@@ -1,3 +1,4 @@
+/** @typedef {import('../../logging/typedef.js').LogLevel} LogLevel */
 /**
  * @typedef {import('./typedef.js').ServiceConstructor} ServiceConstructor
  * @typedef {import('./typedef.js').ServiceRegistrationOptions} ServiceRegistrationOptions
@@ -174,11 +175,11 @@ export class ServiceManager extends EventEmitter {
      *   - String with service name: 'auth' (requires level parameter)
      *   - String with config: 'auth:debug,database:info'
      *   - Object: { auth: 'debug', database: 'info' }
-     * @param {string} [level] - Log level (required when nameOrConfig is service name)
+     * @param {LogLevel} [level] - Log level (required when nameOrConfig is service name)
      */
     setServiceLogLevel(nameOrConfig: string | {
         [x: string]: string;
-    }, level?: string): void;
+    }, level?: LogLevel): void;
     /**
      * Get all services with a specific tag
      *
@@ -198,6 +199,7 @@ export class ServiceManager extends EventEmitter {
     #private;
 }
 export default ServiceManager;
+export type LogLevel = import("../../logging/typedef.js").LogLevel;
 export type ServiceConstructor = import("./typedef.js").ServiceConstructor;
 export type ServiceRegistrationOptions = import("./typedef.js").ServiceRegistrationOptions;
 export type ServiceManagerConfig = import("./typedef.js").ServiceManagerConfig;

package/dist/services/service-manager/ServiceManager.js

@@ -86,6 +86,8 @@ import {
   STATE_DESTROYED
 } from '../service-base/constants.js';
 
+/** @typedef {import('../../logging/typedef.js').LogLevel} LogLevel */
+
 /**
  * @typedef {import('./typedef.js').ServiceConstructor} ServiceConstructor
  * @typedef {import('./typedef.js').ServiceRegistrationOptions} ServiceRegistrationOptions
@@ -141,7 +143,26 @@ export class ServiceManager extends EventEmitter {
     this.setManagerLogLevel(managerLogLevel);
 
     if (serviceLogLevels) {
-      this.setServiceLogLevel(serviceLogLevels);
+      // Parse and store service log levels, but don't apply them yet
+      // They will be applied when services are created in get()
+      /** @type {{[name:string]: LogLevel}} */
+      let parsedServiceLevels = {};
+
+      if (typeof serviceLogLevels === 'string') {
+        if (serviceLogLevels.includes(':')) {
+          // Parse string config: 'auth:debug,database:info'
+          parsedServiceLevels = parseServiceLogLevels(serviceLogLevels);
+        } else {
+          throw new Error(
+            'Service log levels string must include service:level pairs'
+          );
+        }
+      } else {
+        // Object config: { auth: 'debug', database: 'info' }
+        parsedServiceLevels = serviceLogLevels;
+      }
+
+      this.config.serviceLogLevels = parsedServiceLevels;
     }
   }
 
@@ -585,10 +606,10 @@ export class ServiceManager extends EventEmitter {
    *   - String with service name: 'auth' (requires level parameter)
    *   - String with config: 'auth:debug,database:info'
    *   - Object: { auth: 'debug', database: 'info' }
-   * @param {string} [level] - Log level (required when nameOrConfig is service name)
+   * @param {LogLevel} [level] - Log level (required when nameOrConfig is service name)
    */
   setServiceLogLevel(nameOrConfig, level) {
-    /** @type {{[name:string]: string}} */
+    /** @type {{[name:string]: LogLevel}} */
     let serviceLevels = {};
 
     if (typeof nameOrConfig === 'string') {
@@ -617,10 +638,10 @@ export class ServiceManager extends EventEmitter {
     for (const [name, logLevel] of Object.entries(serviceLevels)) {
       this.config.serviceLogLevels[name] = logLevel;
 
-      // Apply to existing instance
-      const instance = this.get(name);
-      if (instance) {
-        instance.setLogLevel(logLevel);
+      // Apply to existing instance if it exists and is registered
+      const entry = this.services.get(name);
+      if (entry?.instance) {
+        entry.instance.setLogLevel(logLevel);
       }
     }
   }

package/dist/services/service-manager/util.d.ts

@@ -1,18 +1,17 @@
+/** @typedef {import('../../logging/typedef.js').LogLevel} LogLevel */
 /**
  * Parse comma-separated service:level configuration string
  *
  * @param {string} configString
  *   Comma-separated string like "auth:debug,database:info,cache:warn"
  *
- * @returns {Object<string, string>} Service name to log level mapping
+ * @returns {Record<string, LogLevel>} Service name to log level mapping
  *
  * @example
  * const config = parseServiceLogLevels("auth:debug,database:info");
  * // Returns: { auth: "debug", database: "info" }
  */
-export function parseServiceLogLevels(configString: string): {
-    [x: string]: string;
-};
+export function parseServiceLogLevels(configString: string): Record<string, LogLevel>;
 /**
  * Expand log levels to include higher severity levels
  *
@@ -33,3 +32,4 @@ export function expandLogLevels(serviceLevels: {
 }): {
     [x: string]: string[];
 };
+export type LogLevel = import("../../logging/typedef.js").LogLevel;

package/dist/services/service-manager/util.js

@@ -7,13 +7,15 @@
 
 import { DEBUG, INFO, WARN, ERROR } from '../../logging/index.js';
 
+/** @typedef {import('../../logging/typedef.js').LogLevel} LogLevel */
+
 /**
  * Parse comma-separated service:level configuration string
  *
  * @param {string} configString
  *   Comma-separated string like "auth:debug,database:info,cache:warn"
  *
- * @returns {Object<string, string>} Service name to log level mapping
+ * @returns {Record<string, LogLevel>} Service name to log level mapping
  *
  * @example
  * const config = parseServiceLogLevels("auth:debug,database:info");

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@hkdigital/lib-core",
-	"version": "0.4.42",
+	"version": "0.4.43",
 	"author": {
 		"name": "HKdigital",
 		"url": "https://hkdigital.nl"