@hkdigital/lib-core 0.5.49 → 0.5.51

package/dist/logging/internal/logger/Logger.d.ts

@@ -1,3 +1,6 @@
+/** @typedef {import('../../../generic/typedef.js').ErrorDetails} ErrorDetails */
+/** @typedef {import('../../typedef.js').LogEvent} LogEvent */
+/** @typedef {import('../../typedef.js').LogEventData} LogEventData */
 /** @typedef {import('../../typedef.js').LogLevel} LogLevel */
 /**
  * Logger class for consistent logging
@@ -63,7 +66,7 @@ export default class Logger extends EventEmitter {
      */
     error(errorOrMessage: Error | ErrorEvent | PromiseRejectionEvent | string, errorOrDetails?: Error | ErrorDetails, details?: ErrorDetails, ...args: any[]): boolean;
     /**
-     * Create a child logger with additional context
+     * New logger with same name, namespaced context, no event forwarding
      *
      * @param {string} namespace
      *   Namespace of the context (needed for chaining contexts)
@@ -72,17 +75,33 @@ export default class Logger extends EventEmitter {
      * @returns {Logger} New logger instance with merged context
      */
     context(namespace: string, additionalContext: Object): Logger;
+    /**
+     * Create a child logger that emits events on the current logger
+     *
+     * @param {string} name
+     *   Child logger name (will be appended to parent name)
+     *
+     * @param {object} [options]
+     * @param {LogLevel} [options.level] - Child logger log level
+     *
+     * @returns {Logger} New child logger instance
+     */
+    child(name: string, { level }?: {
+        level?: import("../../typedef.js").LogLevel | undefined;
+    }): Logger;
     /**
      * Log an LogEvent emitted by an event emitter
      *
      * E.g. an event that was created by another Logger instance and should be
      * forwarded to this logger.
      *
-     * @param {import('../../typedef.js').LogEventData} eventData
+     * @param {LogEventData} eventData
      */
-    logFromEvent(eventData: import("../../typedef.js").LogEventData): false | undefined;
+    logFromEvent(eventData: LogEventData): false | undefined;
     #private;
 }
 export type ErrorDetails = import("../../../generic/typedef.js").ErrorDetails;
+export type LogEvent = import("../../typedef.js").LogEvent;
+export type LogEventData = import("../../typedef.js").LogEventData;
 export type LogLevel = import("../../typedef.js").LogLevel;
 import { EventEmitter } from '../../../generic/events.js';

package/dist/logging/internal/logger/Logger.js

@@ -32,8 +32,6 @@
  * logger.setLevel(DEBUG); // Now debug messages will also be logged
  */
 
-/** @typedef {import('../../../generic/typedef.js').ErrorDetails} ErrorDetails */
-
 import * as expect from '../../../util/expect.js';
 
 import { EventEmitter } from '../../../generic/events.js';
@@ -61,6 +59,11 @@ import { exportNotNullish } from '../../../util/object.js';
 import * as is from '../../../util/is.js';
 import { HttpError } from '../../../network/errors.js';
 
+/** @typedef {import('../../../generic/typedef.js').ErrorDetails} ErrorDetails */
+
+/** @typedef {import('../../typedef.js').LogEvent} LogEvent */
+/** @typedef {import('../../typedef.js').LogEventData} LogEventData */
+
 /** @typedef {import('../../typedef.js').LogLevel} LogLevel */
 
 /**
@@ -220,7 +223,7 @@ export default class Logger extends EventEmitter {
   }
 
   /**
-   * Create a child logger with additional context
+   * New logger with same name, namespaced context, no event forwarding
    *
    * @param {string} namespace
    *   Namespace of the context (needed for chaining contexts)
@@ -241,13 +244,46 @@ export default class Logger extends EventEmitter {
     return new Logger(this.name, this.level, mergedContext);
   }
 
+  /**
+   * Create a child logger that emits events on the current logger
+   *
+   * @param {string} name
+   *   Child logger name (will be appended to parent name)
+   *
+   * @param {object} [options]
+   * @param {LogLevel} [options.level] - Child logger log level
+   *
+   * @returns {Logger} New child logger instance
+   */
+  child(name, { level } = {}) {
+    if (typeof name !== 'string') {
+      throw new Error('Invalid child logger name');
+    }
+
+    const context = {
+      ...this.#defaultContext
+    };
+
+    const childName = `${this.name}:${name}`;
+
+    const child = new Logger(childName, level ?? this.level, context);
+
+    // Forward all log events from child to parent
+    child.on(LOG, ( /** @type {LogEvent} */ logEvent) => {
+      this.emit(logEvent.level, logEvent);
+      this.emit(LOG, logEvent);
+    });
+
+    return child;
+  }
+
   /**
    * Log an LogEvent emitted by an event emitter
    *
    * E.g. an event that was created by another Logger instance and should be
    * forwarded to this logger.
    *
-   * @param {import('../../typedef.js').LogEventData} eventData
+   * @param {LogEventData} eventData
    */
   logFromEvent(eventData) {
     expect.object(eventData);
@@ -295,9 +331,9 @@ export default class Logger extends EventEmitter {
   }
 
   /**
-   * Internal event loggin method
+   * Internal event logging method
    *
-   * @param {import('../../typedef.js').LogEvent} logEvent
+   * @param {LogEvent} logEvent
    */
   #logEvent(logEvent) {
     // Emit as both specific level event and generic 'log' event
@@ -344,6 +380,7 @@ export default class Logger extends EventEmitter {
         'colno'
       ]);
 
+      // @ts-ignore
       errorEventDetails.type = 'ErrorEvent';
 
       let cause = errorEvent.error;

package/dist/state/machines/page-machine/PageMachine.svelte.js

@@ -1,6 +1,3 @@
-import { INFO } from '../../../logging/common.js';
-import { createClientLogger } from '../../../logging/client.js';
-
 /**
  * Base class for page state machines with URL route mapping
  *
@@ -220,17 +217,18 @@ export default class PageMachine {
 	 * Normalize onEnterHooks to ensure consistent format
 	 * Converts function to {onEnter: function} object
 	 *
-	 * @param {Record<string, Function|Object>} hooks - Raw hooks configuration
+	 * @param {Record<string, Function|{onEnter: Function}>} hooks - Raw hooks configuration
 	 * @returns {Record<string, {onEnter: Function}>} Normalized hooks
 	 */
 	#normalizeOnEnterHooks(hooks) {
+		/** @type {Record<string, {onEnter: Function} */
 		const normalized = {};
 
 		for (const [state, hook] of Object.entries(hooks)) {
 			if (typeof hook === 'function') {
 				// Simple function -> wrap in object
 				normalized[state] = { onEnter: hook };
-			} else if (hook && typeof hook === 'object' && hook.onEnter) {
+			} else if (hook?.onEnter) {
 				// Already an object with onEnter
 				normalized[state] = hook;
 			}
@@ -253,9 +251,14 @@ export default class PageMachine {
 		const targetState = this.#getStateFromPath(currentPath);
 
 		if (targetState && targetState !== this.#current) {
+			// const oldState = this.#current;
 			this.#current = targetState;
 			this.#visitedStates.add(targetState);
 			this.#revision++;
+
+			// Log state transition from URL sync
+			this.logger?.debug(`syncFromPath: ${currentPath} → targetState: ${targetState}`);
+
 			return true;
 		}
 
@@ -286,14 +289,15 @@ export default class PageMachine {
 		this.#visitedStates.add(newState);
 
 		// Log state transition
-		this.logger?.debug(`${oldState} → ${newState}`);
+		this.logger?.debug(`setState: ${oldState} → ${newState}`);
 
 		// Check if this state has an onEnter hook
 		const hookConfig = this.#onEnterHooks[newState];
 		if (hookConfig?.onEnter) {
 			// Create done callback for auto-transition
 			let doneCalled = false;
-			const done = (nextState) => {
+
+			const done = ( /** @type {string} */ nextState) => {
 				if (!doneCalled && nextState && nextState !== newState) {
 					doneCalled = true;
 					this.#isTransitioning = false;
@@ -322,7 +326,8 @@ export default class PageMachine {
 					await handler;
 				}
 			} catch (error) {
-				console.error(`Error in onEnter hook for state ${newState}:`, error);
+				const logger = this.logger ?? console;
+				logger.error(`Error in onEnter hook for state ${newState}:`, error);
 			}
 		}
 

package/package.json

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