exprify 1.0.4 → 1.0.7

package/src/function/executor.js

@@ -1,18 +1,24 @@
+/**
+ * @param {{ get: (arg0: any) => any; has: (arg0: any) => any; }} fnRegistry
+ */
 export function createFunctionExecutor(fnRegistry, options = {}) {
   if (!fnRegistry) {
-    throw new Error("Function registry is required");
+    throw new Error('Function registry is required');
   }
 
   const config = {
-    strict: options.strict ?? true
+    strict: options.strict ?? true,
   };
 
-  /* ================= EXECUTE ================= */
-
-  function execute(name, args = [], context) {
+  /**
+   * @param {any} name
+   * @param {any} args
+   * @param {any} _context
+   */
+  function execute(name, args = [], _context) {
     const fn = fnRegistry.get(name);
 
-    /* ----- NOT FOUND ----- */
+    // not found
     if (!fn) {
       if (config.strict) {
         throw new Error(`Unknown function: ${name}`);
@@ -20,45 +26,48 @@ export function createFunctionExecutor(fnRegistry, options = {}) {
       return undefined;
     }
 
-    /* ----- VALIDATE ARGS ----- */
+    // validate args
     if (!Array.isArray(args)) {
       throw new Error(`Arguments for "${name}" must be an array`);
     }
 
-    /* ----- EXECUTE ----- */
+    // execute
     try {
       return fn(...args);
     } catch (err) {
-      throw new Error(
-        `Error in function "${name}": ${err.message}`
-      );
+      const msg = err instanceof Error ? err.message : String(err);
+      throw new Error(`Error in function "${name}": ${msg}`, { cause: err });
     }
   }
 
-  /* ================= SAFE EXECUTE ================= */
-
+  /**
+   * @param {any} name
+   * @param {any} args
+   * @param {any} context
+   */
   function safeExecute(name, args = [], context) {
     try {
       return execute(name, args, context);
     } catch (err) {
+      const msg = err instanceof Error ? err.message : String(err);
       return {
         error: true,
-        message: err.message
+        message: msg,
       };
     }
   }
 
-  /* ================= EXISTS ================= */
-
+  /**
+   * @param {any} name
+   */
   function exists(name) {
     return fnRegistry.has(name);
   }
 
-  /* ================= API ================= */
-
+  // API
   return {
     execute,
     safeExecute,
-    exists
+    exists,
   };
-}
+}