🐛 Make constructor task handling more robust

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## 0.9.4 (WIP)
 
 * Fix `RequestEvents` class restarting request that is already being handled
+* Make constructor task handling more robust
 
 ## 0.9.3 (2024-08-12)
 

lib/function_inheritance.js

@@ -421,12 +421,56 @@ Blast.parseClassPath = function parseClassPath(path) {
 	return result;
 };
 
+/**
+ * Get the identifier of a constructor
+ *
+ * @author   Jelle De Loecker <[email protected]>
+ * @since    0.9.4
+ * @version  0.9.4
+ *
+ * @param    {Function}   constructor
+ *
+ * @return   {String}
+ */
+const getClassIdentifier = constructor => {
+	let result = constructor.name;
+
+	if (constructor.namespace) {
+		result = constructor.namespace + '.' + result;
+	}
+
+	return result;
+};
+
+/**
+ * Call a task for the given constructor/class
+ *
+ * @author   Jelle De Loecker <[email protected]>
+ * @since    0.9.4
+ * @version  0.9.4
+ *
+ * @param    {Function}   constructor
+ * @param    {Function}   task
+ */
+const callTaskForConstructor = (constructor, task, ...args) => {
+	try {
+		return task.call(...args);
+	} catch (err) {
+
+		let new_error = new Error('Error in class "' + getClassIdentifier(constructor) + '": ' + err.message);
+		new_error.stack = err.stack;
+		new_error.original_error = err;
+
+		throw new_error;
+	}
+}
+
 /**
  * Do the given task for a given constructor
  *
  * @author   Jelle De Loecker <[email protected]>
  * @since    0.3.6
- * @version  0.9.0
+ * @version  0.9.4
  *
  * @param    {Function}   constructor
  * @param    {Function}   task
@@ -442,7 +486,7 @@ const doConstructorTask = function doConstructorTask(constructor, task, finished
 	}
 
 	if (finished.indexOf(task) == -1) {
-		task.call(constructor);
+		callTaskForConstructor(constructor, task, constructor);
 		finished.push(task);
 	}
 };