fix: actor initialization and helper config issues (v4.0.1-beta.7)

- Fixed actor not being populated with helper methods
- Fixed circular dependency in actor.js
- Moved actor creation timing to allow proper callback registration
- Fixed helpers to use this.options consistently instead of this.config
- Fixed helpers to merge defaults after _setConfig()
- Fixed container proxy getOwnPropertyDescriptor for destructuring support
- Changed createHelpers to defer await for proper callback accumulation

Author: kobenguyent

lib/actor.js

@@ -75,8 +75,9 @@ export default function (obj = {}, container) {
   if (!container) {
     container = Container
   }
-  
-  const actor = container.actor() || new Actor()
+
+  // Create a new Actor instance
+  const actor = new Actor()
 
   // load all helpers once container initialized
   container.started(() => {

lib/container.js

@@ -74,7 +74,7 @@ class Container {
     // Preload includes (so proxies can expose real objects synchronously)
     const includes = config.include || {}
 
-    // Ensure I is available for DI modules at import time
+    // Check if custom I is provided
     if (Object.prototype.hasOwnProperty.call(includes, 'I')) {
       try {
         const mod = includes.I
@@ -89,7 +89,7 @@ class Container {
         throw new Error(`Could not include object I: ${e.message}`)
       }
     } else {
-      // Create default actor if not provided via includes
+      // Create default actor - this sets up the callback in asyncHelperPromise
       createActor()
     }
 
@@ -110,7 +110,7 @@ class Container {
       }
     }
 
-    // Wait for all async helpers to finish loading and populate the actor with helper methods
+    // Wait for all async helpers to finish loading and populate the actor
     await asyncHelperPromise
 
     if (opts && opts.ai) ai.enable(config.ai) // enable AI Assistant
@@ -350,16 +350,17 @@ async function createHelpers(config) {
     }
   }
 
-  // Wait for all async helpers to be fully loaded
-  await asyncHelperPromise
-
-  // Call _init on all helpers after they're all loaded
-  for (const name in helpers) {
-    if (helpers[name]._init) {
-      await helpers[name]._init()
-      debug(`helper ${name} _init() called`)
+  // Don't await here - let Container.create() handle the await
+  // This allows actor callbacks to be registered before resolution
+  asyncHelperPromise = asyncHelperPromise.then(async () => {
+    // Call _init on all helpers after they're all loaded
+    for (const name in helpers) {
+      if (helpers[name]._init) {
+        await helpers[name]._init()
+        debug(`helper ${name} _init() called`)
+      }
     }
-  }
+  })
 
   return helpers
 }
@@ -534,10 +535,17 @@ function createSupportObjects(config) {
         return [...new Set([...keys, ...container.sharedKeys])]
       },
       getOwnPropertyDescriptor(target, prop) {
+        // For destructuring to work, we need to return the actual value from the getter
+        let value
+        if (container.sharedKeys.has(prop) && prop in container.support) {
+          value = container.support[prop]
+        } else {
+          value = lazyLoad(prop)
+        }
         return {
           enumerable: true,
           configurable: true,
-          value: target[prop],
+          value: value,
         }
       },
       get(target, key) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeceptjs",
-  "version": "4.0.0-beta.20",
+  "version": "4.0.1-beta.7",
   "type": "module",
   "description": "Supercharged End 2 End Testing Framework for NodeJS",
   "keywords": [