* Fix issue where the setOnComplete method needed to use an arrow function calling this.nextCommand, rather than passing a reference to this.nextCommand, which is a prototype and does not have access to instance data

* Improve nullish checking of this.onComplete
* Add index.js to build folder that is copied to the bin folder at build time and is the entry point to the project
* Added missing instance property declaration onComplete to AsyncMacroCommand.js

Author: cliffhall

bin/index.js

@@ -0,0 +1,6 @@
+import {AsyncCommand, AsyncMacroCommand} from './puremvc-async-command.js';
+
+export {
+    AsyncCommand,
+    AsyncMacroCommand
+}

bin/puremvc-async-command.js

@@ -161,24 +161,25 @@ class AsyncMacroCommand	extends puremvc.Notifier
      */
     nextCommand()
     {
-        if (this.subCommands.length > 0)
+        if (this.subCommands?.length > 0)
         {
             const factory = this.subCommands.shift();
             const instance	= factory();
             let isAsync	= ( instance?.isAsync === true );
-            if (isAsync) instance.setOnComplete( this.nextCommand );
+            if (isAsync) instance.setOnComplete( () => this.nextCommand() );
             instance.initializeNotifier( this.multitonKey );
             instance.execute( this.note );
             if (!isAsync) this.nextCommand();
         } else {
-            if( this.onComplete !== null ) this.onComplete();
-            this.note 		 = null;
+            if( this?.onComplete ) this.onComplete();
+            this.note = null;
             this.onComplete	= null;
         }
     }
 
     note;           // Notification
     subCommands;    // Array of command subcommand factories
+    onComplete;     // Optional function to call when the AsyncMacro completes
     isAsync = true; // simplest workaround to lack of interfaces
 }
 

bin/puremvc-async-command.min.js

@@ -1,6 +1,6 @@
 {
   "name": "@puremvc/puremvc-js-util-async-command",
-  "version": "1.0.2",
+  "version": "1.0.5",
   "description": "PureMVC JS MultiCore Async Command Utility",
   "main": "bin/index.js",
   "type": "module",
@@ -9,7 +9,7 @@
     "url": "https://github.com/PureMVC/puremvc-js-util-async-command.git"
   },
   "scripts": {
-    "build": "npm run clean && npm run build:lib",
+    "build": "npm run clean && npm run build:lib && cp build/index.js bin/",
     "build:lib": "rollup -c build/rollup.conf.mjs",
     "build:doc": "jsdoc -c build/jsdoc.json",
     "clean": "rm -rf bin",

package.json

@@ -118,23 +118,24 @@ export class AsyncMacroCommand	extends puremvc.Notifier
      */
     nextCommand()
     {
-        if (this.subCommands.length > 0)
+        if (this.subCommands?.length > 0)
         {
             const factory = this.subCommands.shift();
             const instance	= factory();
             let isAsync	= ( instance?.isAsync === true );
-            if (isAsync) instance.setOnComplete( this.nextCommand );
+            if (isAsync) instance.setOnComplete( () => this.nextCommand() );
             instance.initializeNotifier( this.multitonKey );
             instance.execute( this.note );
             if (!isAsync) this.nextCommand();
         } else {
-            if( this.onComplete !== null ) this.onComplete();
-            this.note 		 = null;
+            if( this?.onComplete ) this.onComplete();
+            this.note = null;
             this.onComplete	= null;
         }
     }
 
     note;           // Notification
     subCommands;    // Array of command subcommand factories
+    onComplete;     // Optional function to call when the AsyncMacro completes
     isAsync = true; // simplest workaround to lack of interfaces
 }