import-in-the-middle 1.13.0 → 1.13.2

package/.release-please-manifest.json

@@ -1,3 +1,3 @@
 {
-  ".": "1.13.0"
+  ".": "1.13.2"
 }

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## [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)
+
+
+### Bug Fixes
+
+* handling of circular dependencies ([#181](https://github.com/nodejs/import-in-the-middle/issues/181)) ([b58092e](https://github.com/nodejs/import-in-the-middle/commit/b58092ec9becf4a14f541da4cf5bfb190f2a9a9b))
+* importing JSON files ([#182](https://github.com/nodejs/import-in-the-middle/issues/182)) ([8c52014](https://github.com/nodejs/import-in-the-middle/commit/8c52014658fcf698cc340d032b441d9e7a65be36))
+* warning from use of context.importAssertions ([#179](https://github.com/nodejs/import-in-the-middle/issues/179)) ([8e56cf1](https://github.com/nodejs/import-in-the-middle/commit/8e56cf1e89752e6c8768d648c10c12fb3178e2ae))
+
 ## [1.13.0](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.12.0...import-in-the-middle-v1.13.0) (2025-02-06)
 
 

package/hook.js

@@ -14,6 +14,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
 
@@ -253,7 +254,12 @@ async function processModule ({ srcUrl, context, parentGetSource, parentResolve,
       }
     } else {
       addSetter(n, `
-      let $${n} = _.${n}
+      let $${n}
+      try {
+        $${n} = _.${n} = namespace.${n}
+      } catch (err) {
+        if (!(err instanceof ReferenceError)) throw err
+      }
       export { $${n} as ${n} }
       set.${n} = (v) => {
         $${n} = v
@@ -344,6 +350,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
     }
@@ -362,10 +372,8 @@ function createHook (meta) {
     }
 
     // Node.js v21 renames importAssertions to importAttributes
-    if (
-      (context.importAssertions && context.importAssertions.type === 'json') ||
-      (context.importAttributes && context.importAttributes.type === 'json')
-    ) {
+    const importAttributes = context.importAttributes || context.importAssertions
+    if (importAttributes && importAttributes.type === 'json') {
       return result
     }
 
@@ -404,10 +412,7 @@ import { register } from '${iitmURL}'
 import * as namespace from ${JSON.stringify(realUrl)}
 
 // Mimic a Module object (https://tc39.es/ecma262/#sec-module-namespace-objects).
-const _ = Object.assign(
-  Object.create(null, { [Symbol.toStringTag]: { value: 'Module' } }),
-  namespace
-)
+const _ = Object.create(null, { [Symbol.toStringTag]: { value: 'Module' } })
 const set = {}
 const get = {}
 

package/lib/get-exports.js

@@ -49,7 +49,7 @@ async function getCjsExports (url, context, parentLoad, source) {
         // Resolve the re-exported module relative to the current module.
         const newUrl = pathToFileURL(require.resolve(re, { paths: [dirname(fileURLToPath(url))] })).href
 
-        if (newUrl.endsWith('.node')) {
+        if (newUrl.endsWith('.node') || newUrl.endsWith('.json')) {
           return
         }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "import-in-the-middle",
-  "version": "1.13.0",
+  "version": "1.13.2",
   "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/circular-a.mjs

@@ -0,0 +1,5 @@
+import { bar } from './circular-b.mjs'
+
+export const foo = 1
+
+export { bar }

package/test/fixtures/circular-b.mjs

@@ -0,0 +1,3 @@
+import { foo } from './circular-a.mjs'
+
+export const bar = foo + 1

package/test/fixtures/re-export-cjs-json.js

@@ -0,0 +1 @@
+module.exports = require('./something.json')

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/re-export-cjs.mjs

@@ -1,7 +1,8 @@
 import Hook from '../../index.js'
 import foo from '../fixtures/re-export-cjs-built-in.js'
 import foo2 from '../fixtures/re-export-cjs.js'
-import { strictEqual } from 'assert'
+import foo3 from '../fixtures/re-export-cjs-json.js'
+import { deepStrictEqual, strictEqual } from 'assert'
 
 Hook((exports, name) => {
   if (name.endsWith('fixtures/re-export-cjs-built-in.js')) {
@@ -13,7 +14,13 @@ Hook((exports, name) => {
     strictEqual(exports.default, 'bar')
     exports.default = '2'
   }
+
+  if (name.endsWith('fixtures/re-export-cjs-json.js')) {
+    deepStrictEqual(exports.default, { data: 'dog' })
+    exports.default = '3'
+  }
 })
 
 strictEqual(foo, '1')
 strictEqual(foo2, '2')
+strictEqual(foo3, '3')

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-circular.mjs

@@ -0,0 +1,17 @@
+import { register } from 'module'
+import Hook from '../../index.js'
+import { strictEqual } from 'assert'
+
+register('../../hook.mjs', import.meta.url)
+
+let bar
+
+Hook((exports, name) => {
+  if (name.match(/circular-b.mjs/)) {
+    bar = exports.bar
+  }
+})
+
+await import('../fixtures/circular-b.mjs')
+
+strictEqual(bar, 2)