@hkdigital/lib-core 0.4.76 → 0.4.78

package/dist/network/loaders/base/SceneBase.svelte.d.ts

@@ -4,7 +4,6 @@
  */
 export default class SceneBase {
     state: string;
-    _preloadVersion: number;
     initial: boolean;
     loaded: boolean;
     /** @type {SceneLoadingProgress} */

package/dist/network/loaders/base/SceneBase.svelte.js

@@ -32,9 +32,6 @@ export default class SceneBase {
   // @note this exported state is set by onenter
   state = $state(STATE_INITIAL);
 
-  // Version counter to force reactivity on new preloads
-  _preloadVersion = $state(0);
-
   initial = $derived.by(() => {
     return this.state === STATE_INITIAL;
   });
@@ -55,9 +52,6 @@ export default class SceneBase {
 
   /** @type {SceneLoadingProgress} */
   progress = $derived.by(() => {
-    // Include version to force recalculation on new preloads
-    const version = this._preloadVersion;
-    
     let totalSize = 0;
     let totalBytesLoaded = 0;
     let sourcesLoaded = 0;
@@ -96,8 +90,7 @@ export default class SceneBase {
       totalSize,
       sourcesLoaded,
       numberOfSources,
-      percentageLoaded,
-      version
+      percentageLoaded
     };
   });
 
@@ -213,9 +206,6 @@ export default class SceneBase {
    * Start loading all sources
    */
   load() {
-    // Increment version to force reactivity
-    this._preloadVersion++;
-    console.debug('SceneBase:load version incremented to', this._preloadVersion);
     this.#state.send(LOAD);
   }
 
@@ -390,11 +380,42 @@ export default class SceneBase {
       return;
     }
 
+    let completedCount = 0;
+    const totalLoaders = this.sources.length;
+
+    // Start all loaders with completion callbacks
     for (let i = 0; i < this.sources.length; i++) {
       const source = this.sources[i];
       const loader = this.getLoaderFromSource(source);
 
-      loader.load();
+      loader.load((completedLoader, finalState) => {
+        console.debug(`SceneBase:loader-finished [${completedLoader._url}] ${finalState}`);
+
+        // Check for errors
+        if (finalState === STATE_ERROR) {
+          this.#state.send(ERROR, completedLoader.error || new Error('Loader failed'));
+          return;
+        }
+
+        // Check for abort - don't count aborted loaders as completed
+        if (finalState === STATE_ABORTED) {
+          // Aborted loaders are handled by SceneBase's own abort logic
+          console.debug(`SceneBase:loader-aborted [${completedLoader._url}] - not counting toward completion`);
+          return;
+        }
+
+        // Only count successfully loaded loaders
+        if (finalState === STATE_LOADED) {
+          completedCount++;
+          console.debug(`SceneBase:loader-completed [${completedLoader._url}] (${completedCount}/${totalLoaders})`);
+
+          // Check for completion
+          if (completedCount === totalLoaders) {
+            console.debug('SceneBase:all-loaders-completed-via-callback');
+            this.#state.send(LOADED);
+          }
+        }
+      });
     }
   }
 

package/dist/network/states/NetworkLoader.svelte.d.ts

@@ -35,8 +35,11 @@ export default class NetworkLoader {
     _abortLoading: null | (() => void);
     /**
      * Start loading all network data
+     *
+     * @param {null|((loader: NetworkLoader, finalState: string) => void)} [onCompletion]
+     *   Optional callback called when loader reaches a terminal state (loaded/error/aborted)
      */
-    load(): void;
+    load(onCompletion?: null | ((loader: NetworkLoader, finalState: string) => void)): void;
     /**
      * Unoad all network data
      */

package/dist/network/states/NetworkLoader.svelte.js

@@ -8,6 +8,8 @@ import {
   STATE_UNLOADING,
   STATE_LOADED,
   STATE_ABORTING,
+  STATE_ABORTED,
+  STATE_ERROR,
   LOAD,
   ERROR,
   LOADED,
@@ -89,6 +91,9 @@ export default class NetworkLoader {
   /** @type {null|(()=>void)} */
   _abortLoading = null;
 
+  /** @type {null|((loader: NetworkLoader, finalState: string) => void)} */
+  #completionCallback = null;
+
   /**
    * Construct NetworkLoader
    *
@@ -104,6 +109,17 @@ export default class NetworkLoader {
       console.debug(`loader:onenter [${this._url}] ${currentState}`);
       this.state = currentState;
 
+      // Check if we've reached a terminal state
+      const isTerminalState = currentState === STATE_LOADED || 
+                             currentState === STATE_ERROR || 
+                             currentState === STATE_ABORTED;
+
+      if (isTerminalState && this.#completionCallback) {
+        const callback = this.#completionCallback;
+        this.#completionCallback = null; // Clean up immediately
+        callback(this, currentState); // Call with loader and final state
+      }
+
       switch (currentState) {
         case STATE_LOADING:
           {
@@ -155,8 +171,12 @@ export default class NetworkLoader {
 
   /**
    * Start loading all network data
+   *
+   * @param {null|((loader: NetworkLoader, finalState: string) => void)} [onCompletion]
+   *   Optional callback called when loader reaches a terminal state (loaded/error/aborted)
    */
-  load() {
+  load(onCompletion = null) {
+    this.#completionCallback = onCompletion;
     this.#state.send(LOAD);
   }
 
@@ -327,6 +347,12 @@ export default class NetworkLoader {
       //   this._size = this._buffer.byteLength;
       // }
 
+      // Check if we've been aborted before sending LOADED
+      if (this.#state.current === STATE_ABORTING || this.#state.current === STATE_ABORTED) {
+        console.debug(`loader:already-aborted [${this._url}] - not sending LOADED`);
+        return;
+      }
+      
       console.debug(`loader:sending-LOADED [${this._url}]`);
       this.#state.send(LOADED);
     } catch (e) {

package/package.json

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