import-in-the-middle 1.11.0 → 1.11.2

package/.eslintrc.yaml

@@ -21,3 +21,6 @@ ignorePatterns:
   - test/fixtures/circular-a.js
   - test/fixtures/circular-b.js
   - test/fixtures/reexport.js
+  - test/fixtures/duplicate-explicit.mjs
+  - test/fixtures/duplicate.mjs
+  - test/fixtures/export-types/default-call-expression-renamed.mjs

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.11.0"
+  ".": "1.11.2"
 }

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.11.2](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.11.1...import-in-the-middle-v1.11.2) (2024-09-30)
+
+
+### Bug Fixes
+
+* do nothing if target does not exist in getters map ([#155](https://github.com/nodejs/import-in-the-middle/issues/155)) ([5f6be49](https://github.com/nodejs/import-in-the-middle/commit/5f6be494fc11caf8dcf900807c5b6b646fcd8d74))
+
+## [1.11.1](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.11.0...import-in-the-middle-v1.11.1) (2024-09-26)
+
+
+### Bug Fixes
+
+* Support Hooking multiple times ([#153](https://github.com/nodejs/import-in-the-middle/issues/153)) ([e0d8080](https://github.com/nodejs/import-in-the-middle/commit/e0d808041eff228f4b4519454f7eea8f0930238a))
+
 ## [1.11.0](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.10.0...import-in-the-middle-v1.11.0) (2024-07-29)
 
 

package/hook.js

@@ -259,6 +259,7 @@ async function processModule ({ srcUrl, context, parentGetSource, parentResolve,
         $${n} = v
         return true
       }
+      get.${n} = () => $${n}
       `)
     }
   }
@@ -402,10 +403,11 @@ const _ = Object.assign(
   namespace
 )
 const set = {}
+const get = {}
 
 ${Array.from(setters.values()).join('\n')}
 
-register(${JSON.stringify(realUrl)}, _, set, ${JSON.stringify(specifiers.get(realUrl))})
+register(${JSON.stringify(realUrl)}, _, set, get, ${JSON.stringify(specifiers.get(realUrl))})
 `
         }
       } catch (cause) {

package/lib/register.js

@@ -4,6 +4,7 @@
 
 const importHooks = [] // TODO should this be a Set?
 const setters = new WeakMap()
+const getters = new WeakMap()
 const specifiers = new Map()
 const toHook = []
 
@@ -12,6 +13,18 @@ const proxyHandler = {
     return setters.get(target)[name](value)
   },
 
+  get (target, name) {
+    if (name === Symbol.toStringTag) {
+      return 'Module'
+    }
+
+    const getter = getters.get(target)[name]
+
+    if (typeof getter === 'function') {
+      return getter()
+    }
+  },
+
   defineProperty (target, property, descriptor) {
     if ((!('value' in descriptor))) {
       throw new Error('Getters/setters are not supported for exports property descriptors.')
@@ -21,9 +34,10 @@ const proxyHandler = {
   }
 }
 
-function register (name, namespace, set, specifier) {
+function register (name, namespace, set, get, specifier) {
   specifiers.set(name, specifier)
   setters.set(namespace, set)
+  getters.set(namespace, get)
   const proxy = new Proxy(namespace, proxyHandler)
   importHooks.forEach(hook => hook(name, proxy))
   toHook.push([name, proxy])

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "import-in-the-middle",
-  "version": "1.11.0",
+  "version": "1.11.2",
   "description": "Intercept imports in Node.js",
   "main": "index.js",
   "scripts": {

package/test/hook/v14-double-hook.mjs

@@ -0,0 +1,33 @@
+import { fileURLToPath } from 'url'
+import { join } from 'path'
+import { strictEqual } from 'assert'
+import Hook from '../../index.js'
+
+const toWrap = join(fileURLToPath(import.meta.url), '..', '..', 'fixtures', 'foo.mjs')
+
+Hook([toWrap], (exports) => {
+  const original = exports.foo
+  exports.foo = function foo () {
+    return original() + '-first'
+  }
+})
+
+Hook([toWrap], (exports) => {
+  const original = exports.foo
+  exports.foo = function foo () {
+    return original() + '-second'
+  }
+})
+
+Hook([toWrap], (exports) => {
+  const shouldNotExist = exports.default
+  exports = function () {
+    return shouldNotExist()
+  }
+})
+
+const imp = await import('../fixtures/foo.mjs')
+
+strictEqual(imp.foo(), 'foo-first-second')
+// This should not throw!
+strictEqual(imp.default, undefined)