import-in-the-middle 1.13.1 → 1.14.0

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.13.1"
+  ".": "1.14.0"
 }

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## [1.14.0](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.13.2...import-in-the-middle-v1.14.0) (2025-05-24)
+
+
+### Features
+
+* Optionally hook internal paths like `require-in-the-middle` ([#194](https://github.com/nodejs/import-in-the-middle/issues/194)) ([976d032](https://github.com/nodejs/import-in-the-middle/commit/976d0320426dcbf8e6260504eccbb62d83513f5a))
+
+## [1.13.2](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.13.1...import-in-the-middle-v1.13.2) (2025-05-12)
+
+
+### Bug Fixes
+
+* Don't attempt to wrap TypeScript modules ([#191](https://github.com/nodejs/import-in-the-middle/issues/191)) ([6deb87e](https://github.com/nodejs/import-in-the-middle/commit/6deb87ea069ec2ee749ce2297ea47ce071d18cf9))
+
 ## [1.13.1](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.13.0...import-in-the-middle-v1.13.1) (2025-02-28)
 
 

package/README.md

@@ -109,6 +109,23 @@ fs.readFileSync('file.txt')
 node --import=./instrument.mjs ./my-app.mjs
 ```
 
+### Experimental `experimentalPatchInternals` option
+
+It was found that `import-in-the-middle` didn't match the hooking behavior of `require-in-the-middle` in some cases: 
+https://github.com/nodejs/import-in-the-middle/issues/185
+
+The `experimentalPatchInternals` option forces the loader to match the behavior of `require-in-the-middle` in these cases.
+
+This option is experimental and may be removed or made the default in the future.
+
+```js
+import { register } from 'module'
+
+register('import-in-the-middle/hook.mjs', import.meta.url, {
+  data: { experimentalPatchInternals: true }
+})
+```
+
 ## Limitations
 
 * You cannot add new exports to a module. You can only modify existing ones.

package/hook.js

@@ -7,6 +7,7 @@ const { inspect } = require('util')
 const { builtinModules } = require('module')
 const specifiers = new Map()
 const isWin = process.platform === 'win32'
+let experimentalPatchInternals = false
 
 // FIXME: Typescript extensions are added temporarily until we find a better
 // way of supporting arbitrary extensions
@@ -14,6 +15,7 @@ const EXTENSION_RE = /\.(js|mjs|cjs|ts|mts|cts)$/
 const NODE_VERSION = process.versions.node.split('.')
 const NODE_MAJOR = Number(NODE_VERSION[0])
 const NODE_MINOR = Number(NODE_VERSION[1])
+const HANDLED_FORMATS = new Set(['builtin', 'module', 'commonjs'])
 
 let entrypoint
 
@@ -291,6 +293,10 @@ function createHook (meta) {
     global.__import_in_the_middle_initialized__ = true
 
     if (data) {
+      if (data.experimentalPatchInternals) {
+        experimentalPatchInternals = true
+      }
+
       includeModules = ensureArrayWithBareSpecifiersFileUrlsAndRegex(data.include, 'include')
       excludeModules = ensureArrayWithBareSpecifiersFileUrlsAndRegex(data.exclude, 'exclude')
 
@@ -349,6 +355,10 @@ function createHook (meta) {
       return each === specifier || each === result.url || (result.url.startsWith('file:') && each === fileURLToPath(result.url))
     }
 
+    if (result.format && !HANDLED_FORMATS.has(result.format)) {
+      return result
+    }
+
     if (includeModules && !includeModules.some(match)) {
       return result
     }
@@ -405,6 +415,7 @@ function createHook (meta) {
           source: `
 import { register } from '${iitmURL}'
 import * as namespace from ${JSON.stringify(realUrl)}
+${experimentalPatchInternals ? `import { setExperimentalPatchInternals } from '${iitmURL}'\nsetExperimentalPatchInternals(true)` : ''}
 
 // Mimic a Module object (https://tc39.es/ecma262/#sec-module-namespace-objects).
 const _ = Object.create(null, { [Symbol.toStringTag]: { value: 'Module' } })

package/index.js

@@ -10,7 +10,8 @@ const { MessageChannel } = require('worker_threads')
 const {
   importHooks,
   specifiers,
-  toHook
+  toHook,
+  getExperimentalPatchInternals
 } = require('./lib/register')
 
 function addHook (hook) {
@@ -144,7 +145,7 @@ function Hook (modules, options, hookFn) {
             if (internals) {
               name = name + path.sep + path.relative(baseDir, fileURLToPath(filename))
             } else {
-              if (!baseDir.endsWith(specifiers.get(filename))) continue
+              if (!getExperimentalPatchInternals() && !baseDir.endsWith(specifiers.get(filename))) continue
             }
           }
           callHookFn(hookFn, namespace, name, baseDir)

package/lib/register.js

@@ -43,7 +43,19 @@ function register (name, namespace, set, get, specifier) {
   toHook.push([name, proxy])
 }
 
+let experimentalPatchInternals = false
+
+function getExperimentalPatchInternals () {
+  return experimentalPatchInternals
+}
+
+function setExperimentalPatchInternals (value) {
+  experimentalPatchInternals = value
+}
+
 exports.register = register
 exports.importHooks = importHooks
 exports.specifiers = specifiers
 exports.toHook = toHook
+exports.getExperimentalPatchInternals = getExperimentalPatchInternals
+exports.setExperimentalPatchInternals = setExperimentalPatchInternals

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "import-in-the-middle",
-  "version": "1.13.1",
+  "version": "1.14.0",
   "description": "Intercept imports in Node.js",
   "main": "index.js",
   "scripts": {
@@ -50,7 +50,7 @@
     "eslint-plugin-promise": "^6.1.1",
     "got": "^14.3.0",
     "imhotap": "^2.1.0",
-    "openai": "^4.47.2",
+    "openai": "4.47.2",
     "ts-node": "^10.9.1",
     "typescript": "^4.7.4",
     "vue": "^3.4.31"

package/test/fixtures/something.mts

@@ -0,0 +1,7 @@
+export type { Debugger } from 'node:inspector'
+
+export const foo: number = 42
+
+export function bar(): number {
+  return foo
+}

package/test/hook/v24-typescript.mjs

@@ -0,0 +1,8 @@
+import { strictEqual } from 'assert'
+
+const mod = await import('../fixtures/something.mts')
+
+strictEqual(mod.foo, 42)
+strictEqual(typeof mod.bar, 'function')
+strictEqual(mod.bar(), 42)
+strictEqual(Object.keys(mod).length, 2)

package/test/register/v18.19-experimental-patch-internals.mjs

@@ -0,0 +1,14 @@
+import { register } from 'module'
+import Hook from '../../index.js'
+import { strictEqual } from 'assert'
+
+register('../../hook.mjs', import.meta.url, { data: { experimentalPatchInternals: true } })
+
+Hook(['c8'], (exports, name) => {
+  strictEqual(name, 'c8')
+  exports.Report = () => 42
+})
+
+const { Report } = await import('c8/index.js')
+
+strictEqual(Report({}), 42)