refactor: cache the return value of createRequire on the createRequire function itself

Author: Xmader

python/pythonmonkey/require.py

@@ -225,7 +225,9 @@ def load(filename: str) -> Dict:
   require = createRequire(__file__)
   require('./my-javascript-module')
 """
-def createRequire(filename):
+# We cache the return value of createRequire to always use the same require for the same filename
+@functools.lru_cache(maxsize=None) # unbounded function cache that won't remove any old values
+def createRequire(filename: str):
     createRequireInner = pm.eval("""'use strict';(
 function createRequire(filename, bootstrap_broken)
 {
@@ -248,12 +250,7 @@ def createRequire(filename):
 })""")
     return createRequireInner(filename)
 
-# We cache the return value of createRequire to always use the same require for the same filename
-@functools.lru_cache(maxsize=None) # unbounded function cache that won't remove any old values
-def _getRequire(filename: str):
-    return createRequire(filename)
-
 def require(moduleIdentifier: str):
     # Retrieve the caller’s filename from the call stack
     filename = inspect.stack()[1].filename
-    return _getRequire(filename)(moduleIdentifier)
+    return createRequire(filename)(moduleIdentifier)