import-in-the-middle 1.8.1 → 1.9.0

package/.release-please-manifest.json

@@ -0,0 +1,3 @@
+{
+  ".": "1.9.0"
+}

package/CHANGELOG.md

@@ -0,0 +1,15 @@
+# Changelog
+
+## [1.9.0](https://github.com/nodejs/import-in-the-middle/compare/import-in-the-middle-v1.8.1...import-in-the-middle-v1.9.0) (2024-07-08)
+
+
+### Features
+
+* Allow passing of `include` or `exclude` list via `module.register()` ([#124](https://github.com/nodejs/import-in-the-middle/issues/124)) ([381f48c](https://github.com/nodejs/import-in-the-middle/commit/381f48c07ff755e88495f688c75c4912926194c7))
+
+
+### Bug Fixes
+
+* CJS `require('.')` resolution ([#108](https://github.com/nodejs/import-in-the-middle/issues/108)) ([29c77b5](https://github.com/nodejs/import-in-the-middle/commit/29c77b560aec0429154632c950923d12db36f79e))
+* Include source url for parsing failures ([#109](https://github.com/nodejs/import-in-the-middle/issues/109)) ([49d69ba](https://github.com/nodejs/import-in-the-middle/commit/49d69ba9e785d4b6a1b38d7da1293cb744b6d7e3))
+* Use `process.emitWarning` to log wrapping errors ([#114](https://github.com/nodejs/import-in-the-middle/issues/114)) ([a3778ac](https://github.com/nodejs/import-in-the-middle/commit/a3778acfbe2220ce5d521232b41da23b4383e1e3))

package/CODE_OF_CONDUCT.md

@@ -0,0 +1,4 @@
+# Code of Conduct
+
+The Code of Conduct, which applies to this project, can be found at
+https://github.com/nodejs/admin/blob/HEAD/CODE_OF_CONDUCT.md

package/CONTRIBUTING.md

@@ -1,6 +1,34 @@
-# Contributing to import-in-the-middle
+# Contributing to `import-in-the-middle`
 
-First of all, thanks for contributing!
+## Code of Conduct
 
-* If you think you've found an issue, please open a Github issue.
-* To propose improvements, feel free to submit a PR.
+Please read the
+[Code of Conduct](https://github.com/nodejs/admin/blob/main/CODE_OF_CONDUCT.md)
+which explains the minimum behavior expectations for `import-in-the-middle` contributors.
+
+<a id="developers-certificate-of-origin"></a>
+## Developer's Certificate of Origin 1.1
+
+By making a contribution to this project, I certify that:
+
+* (a) The contribution was created in whole or in part by me and I
+  have the right to submit it under the open source license
+  indicated in the file; or
+
+* (b) The contribution is based upon previous work that, to the best
+  of my knowledge, is covered under an appropriate open source
+  license and I have the right under that license to submit that
+  work with modifications, whether created in whole or in part
+  by me, under the same open source license (unless I am
+  permitted to submit under a different license), as indicated
+  in the file; or
+
+* (c) The contribution was provided directly to me by some other
+  person who certified (a), (b) or (c) and I have not modified
+  it.
+
+* (d) I understand and agree that this project and the contribution
+  are public and that a record of the contribution (including all
+  personal information I submit with it, including my sign-off) is
+  maintained indefinitely and may be redistributed consistent with
+  this project or the open source license(s) involved.

package/GOVERNANCE.md

@@ -0,0 +1,28 @@
+# `import-in-the-middle` Project Governance
+
+## Collaborators
+
+`import-in-the-middle` core collaborators maintain the
+[nodejs/import-in-the-middle](https://github.com/nodejs/import-in-the-middle)
+GitHub repository.
+The GitHub team for `import-in-the-middle` core collaborators is
+[@nodejs/import-in-the-middle-maintainers](https://github.com/orgs/nodejs/teams/import-in-the-middle-maintainers). 
+Collaborators have commit access to the
+[nodejs/import-in-the-middle](https://github.com/nodejs/import-in-the-middle)
+repository.
+
+Both collaborators and non-collaborators may propose changes to the `import-in-the-middle`
+source code. The mechanism to propose such a change is a GitHub pull request.
+Collaborators review and merge (_land_) pull requests.
+
+## PR Approval
+
+Two collaborators must approve a pull request before the pull request can land.
+Approving a pull request indicates that the collaborator accepts responsibility
+for the change. Approval must be from collaborators who are not authors of the
+change and at least one approval must be from a collaborator outside the authors
+direct team. 
+
+If a collaborator opposes a proposed change, then the change cannot land. Often,
+discussions or further changes result in collaborators removing their
+opposition. 

package/LICENSE

@@ -1,13 +1,201 @@
-Copyright 2021 Datadog, Inc.
+                                 Apache License
+                           Version 2.0, January 2004
+                        http://www.apache.org/licenses/
 
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
+   TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
 
-    http://www.apache.org/licenses/LICENSE-2.0
+   1. Definitions.
 
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
+      "License" shall mean the terms and conditions for use, reproduction,
+      and distribution as defined by Sections 1 through 9 of this document.
+
+      "Licensor" shall mean the copyright owner or entity authorized by
+      the copyright owner that is granting the License.
+
+      "Legal Entity" shall mean the union of the acting entity and all
+      other entities that control, are controlled by, or are under common
+      control with that entity. For the purposes of this definition,
+      "control" means (i) the power, direct or indirect, to cause the
+      direction or management of such entity, whether by contract or
+      otherwise, or (ii) ownership of fifty percent (50%) or more of the
+      outstanding shares, or (iii) beneficial ownership of such entity.
+
+      "You" (or "Your") shall mean an individual or Legal Entity
+      exercising permissions granted by this License.
+
+      "Source" form shall mean the preferred form for making modifications,
+      including but not limited to software source code, documentation
+      source, and configuration files.
+
+      "Object" form shall mean any form resulting from mechanical
+      transformation or translation of a Source form, including but
+      not limited to compiled object code, generated documentation,
+      and conversions to other media types.
+
+      "Work" shall mean the work of authorship, whether in Source or
+      Object form, made available under the License, as indicated by a
+      copyright notice that is included in or attached to the work
+      (an example is provided in the Appendix below).
+
+      "Derivative Works" shall mean any work, whether in Source or Object
+      form, that is based on (or derived from) the Work and for which the
+      editorial revisions, annotations, elaborations, or other modifications
+      represent, as a whole, an original work of authorship. For the purposes
+      of this License, Derivative Works shall not include works that remain
+      separable from, or merely link (or bind by name) to the interfaces of,
+      the Work and Derivative Works thereof.
+
+      "Contribution" shall mean any work of authorship, including
+      the original version of the Work and any modifications or additions
+      to that Work or Derivative Works thereof, that is intentionally
+      submitted to Licensor for inclusion in the Work by the copyright owner
+      or by an individual or Legal Entity authorized to submit on behalf of
+      the copyright owner. For the purposes of this definition, "submitted"
+      means any form of electronic, verbal, or written communication sent
+      to the Licensor or its representatives, including but not limited to
+      communication on electronic mailing lists, source code control systems,
+      and issue tracking systems that are managed by, or on behalf of, the
+      Licensor for the purpose of discussing and improving the Work, but
+      excluding communication that is conspicuously marked or otherwise
+      designated in writing by the copyright owner as "Not a Contribution."
+
+      "Contributor" shall mean Licensor and any individual or Legal Entity
+      on behalf of whom a Contribution has been received by Licensor and
+      subsequently incorporated within the Work.
+
+   2. Grant of Copyright License. Subject to the terms and conditions of
+      this License, each Contributor hereby grants to You a perpetual,
+      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
+      copyright license to reproduce, prepare Derivative Works of,
+      publicly display, publicly perform, sublicense, and distribute the
+      Work and such Derivative Works in Source or Object form.
+
+   3. Grant of Patent License. Subject to the terms and conditions of
+      this License, each Contributor hereby grants to You a perpetual,
+      worldwide, non-exclusive, no-charge, royalty-free, irrevocable
+      (except as stated in this section) patent license to make, have made,
+      use, offer to sell, sell, import, and otherwise transfer the Work,
+      where such license applies only to those patent claims licensable
+      by such Contributor that are necessarily infringed by their
+      Contribution(s) alone or by combination of their Contribution(s)
+      with the Work to which such Contribution(s) was submitted. If You
+      institute patent litigation against any entity (including a
+      cross-claim or counterclaim in a lawsuit) alleging that the Work
+      or a Contribution incorporated within the Work constitutes direct
+      or contributory patent infringement, then any patent licenses
+      granted to You under this License for that Work shall terminate
+      as of the date such litigation is filed.
+
+   4. Redistribution. You may reproduce and distribute copies of the
+      Work or Derivative Works thereof in any medium, with or without
+      modifications, and in Source or Object form, provided that You
+      meet the following conditions:
+
+      (a) You must give any other recipients of the Work or
+          Derivative Works a copy of this License; and
+
+      (b) You must cause any modified files to carry prominent notices
+          stating that You changed the files; and
+
+      (c) You must retain, in the Source form of any Derivative Works
+          that You distribute, all copyright, patent, trademark, and
+          attribution notices from the Source form of the Work,
+          excluding those notices that do not pertain to any part of
+          the Derivative Works; and
+
+      (d) If the Work includes a "NOTICE" text file as part of its
+          distribution, then any Derivative Works that You distribute must
+          include a readable copy of the attribution notices contained
+          within such NOTICE file, excluding those notices that do not
+          pertain to any part of the Derivative Works, in at least one
+          of the following places: within a NOTICE text file distributed
+          as part of the Derivative Works; within the Source form or
+          documentation, if provided along with the Derivative Works; or,
+          within a display generated by the Derivative Works, if and
+          wherever such third-party notices normally appear. The contents
+          of the NOTICE file are for informational purposes only and
+          do not modify the License. You may add Your own attribution
+          notices within Derivative Works that You distribute, alongside
+          or as an addendum to the NOTICE text from the Work, provided
+          that such additional attribution notices cannot be construed
+          as modifying the License.
+
+      You may add Your own copyright statement to Your modifications and
+      may provide additional or different license terms and conditions
+      for use, reproduction, or distribution of Your modifications, or
+      for any such Derivative Works as a whole, provided Your use,
+      reproduction, and distribution of the Work otherwise complies with
+      the conditions stated in this License.
+
+   5. Submission of Contributions. Unless You explicitly state otherwise,
+      any Contribution intentionally submitted for inclusion in the Work
+      by You to the Licensor shall be under the terms and conditions of
+      this License, without any additional terms or conditions.
+      Notwithstanding the above, nothing herein shall supersede or modify
+      the terms of any separate license agreement you may have executed
+      with Licensor regarding such Contributions.
+
+   6. Trademarks. This License does not grant permission to use the trade
+      names, trademarks, service marks, or product names of the Licensor,
+      except as required for reasonable and customary use in describing the
+      origin of the Work and reproducing the content of the NOTICE file.
+
+   7. Disclaimer of Warranty. Unless required by applicable law or
+      agreed to in writing, Licensor provides the Work (and each
+      Contributor provides its Contributions) on an "AS IS" BASIS,
+      WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+      implied, including, without limitation, any warranties or conditions
+      of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
+      PARTICULAR PURPOSE. You are solely responsible for determining the
+      appropriateness of using or redistributing the Work and assume any
+      risks associated with Your exercise of permissions under this License.
+
+   8. Limitation of Liability. In no event and under no legal theory,
+      whether in tort (including negligence), contract, or otherwise,
+      unless required by applicable law (such as deliberate and grossly
+      negligent acts) or agreed to in writing, shall any Contributor be
+      liable to You for damages, including any direct, indirect, special,
+      incidental, or consequential damages of any character arising as a
+      result of this License or out of the use or inability to use the
+      Work (including but not limited to damages for loss of goodwill,
+      work stoppage, computer failure or malfunction, or any and all
+      other commercial damages or losses), even if such Contributor
+      has been advised of the possibility of such damages.
+
+   9. Accepting Warranty or Additional Liability. While redistributing
+      the Work or Derivative Works thereof, You may choose to offer,
+      and charge a fee for, acceptance of support, warranty, indemnity,
+      or other liability obligations and/or rights consistent with this
+      License. However, in accepting such obligations, You may act only
+      on Your own behalf and on Your sole responsibility, not on behalf
+      of any other Contributor, and only if You agree to indemnify,
+      defend, and hold each Contributor harmless for any liability
+      incurred by, or claims asserted against, such Contributor by reason
+      of your accepting any such warranty or additional liability.
+
+   END OF TERMS AND CONDITIONS
+
+   APPENDIX: How to apply the Apache License to your work.
+
+      To apply the Apache License to your work, attach the following
+      boilerplate notice, with the fields enclosed by brackets "[]"
+      replaced with your own identifying information. (Don't include
+      the brackets!)  The text should be enclosed in the appropriate
+      comment syntax for the file format. We also recommend that a
+      file or class name and description of purpose be included on the
+      same "printed page" as the copyright notice for easier
+      identification within third-party archives.
+
+   Copyright [yyyy] [name of copyright owner]
+
+   Licensed under the Apache License, Version 2.0 (the "License");
+   you may not use this file except in compliance with the License.
+   You may obtain a copy of the License at
+
+       http://www.apache.org/licenses/LICENSE-2.0
+
+   Unless required by applicable law or agreed to in writing, software
+   distributed under the License is distributed on an "AS IS" BASIS,
+   WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+   See the License for the specific language governing permissions and
+   limitations under the License.

package/NOTICE

@@ -1,4 +1,34 @@
-Datadog import-in-the-middle
-Copyright 2021 Datadog, Inc.
+import-in-the-middle is licensed for use as follows:
 
-This product includes software developed at Datadog (https://www.datadoghq.com/).
+Copyright 2024 Node.js contributors. All rights reserved.
+
+    Licensed under the Apache License, Version 2.0 (the "License");
+    you may not use this file except in compliance with the License.
+    You may obtain a copy of the License at
+
+        http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.
+
+This license applies to parts of import-in-the-middle originating from the
+https://github.com/DataDog/import-in-the-middle repository:
+
+    This product includes software developed at Datadog (https://www.datadoghq.com/).
+
+    Copyright 2021 Datadog, Inc.
+
+    Licensed under the Apache License, Version 2.0 (the "License");
+    you may not use this file except in compliance with the License.
+    You may obtain a copy of the License at
+
+        http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing, software
+    distributed under the License is distributed on an "AS IS" BASIS,
+    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+    See the License for the specific language governing permissions and
+    limitations under the License.

package/README.md

@@ -38,6 +38,39 @@ command-line option.
 --loader=import-in-the-middle/hook.mjs
 ```
 
+It's also possible to register the loader hook programmatically via the Node
+[`module.register()`](https://nodejs.org/api/module.html#moduleregisterspecifier-parenturl-options)
+API. However, for this to be able to hook non-dynamic imports, it needs to be
+loaded before your app code is evaluated via the `--import` command-line option.
+
+`my-loader.mjs`
+```js
+import * as module from 'module'
+
+module.register('import-in-the-middle/hook.mjs', import.meta.url)
+```
+```shell
+node --import=./my-loader.mjs ./my-code.mjs
+```
+
+When registering the loader hook programmatically, it's possible to pass a list
+of modules or file URLs to either exclude or specifically include which modules
+are intercepted. This is useful if a module is not compatible with the loader
+hook. 
+```js
+import * as module from 'module'
+
+// Exclude intercepting a specific module by name
+module.register('import-in-the-middle/hook.mjs', import.meta.url, {
+  data: { exclude: ['package-i-want-to-exclude'] }
+})
+
+// Only intercept a specific module by name
+module.register('import-in-the-middle/hook.mjs', import.meta.url, {
+  data: { include: ['package-i-want-to-include'] }
+})
+```
+
 ## Limitations
 
 * You cannot add new exports to a module. You can only modify existing ones.

package/hook.js

@@ -3,6 +3,7 @@
 // This product includes software developed at Datadog (https://www.datadoghq.com/). Copyright 2021 Datadog, Inc.
 
 const { URL } = require('url')
+const { inspect } = require('util')
 const specifiers = new Map()
 const isWin = process.platform === 'win32'
 
@@ -115,6 +116,46 @@ function isBareSpecifier (specifier) {
   }
 }
 
+function isBareSpecifierOrFileUrl (input) {
+  // Relative and absolute paths
+  if (
+    input.startsWith('.') ||
+    input.startsWith('/')) {
+    return false
+  }
+
+  try {
+    // eslint-disable-next-line no-new
+    const url = new URL(input)
+    return url.protocol === 'file:'
+  } catch (err) {
+    // Anything that fails parsing is a bare specifier
+    return true
+  }
+}
+
+function ensureArrayWithBareSpecifiersAndFileUrls (array, type) {
+  if (!Array.isArray(array)) {
+    return undefined
+  }
+
+  const invalid = array.filter(s => !isBareSpecifierOrFileUrl(s))
+
+  if (invalid.length) {
+    throw new Error(`'${type}' option only supports bare specifiers and file URLs. Invalid entries: ${inspect(invalid)}`)
+  }
+
+  return array
+}
+
+function emitWarning (err) {
+  // Unfortunately, process.emitWarning does not output the full error
+  // with error.cause like console.warn does so we need to inspect it when
+  // tracing warnings
+  const warnMessage = process.execArgv.includes('--trace-warnings') ? inspect(err) : err
+  process.emitWarning(warnMessage)
+}
+
 /**
  * Processes a module's exports and builds a set of setter blocks.
  *
@@ -208,11 +249,19 @@ function addIitm (url) {
 function createHook (meta) {
   let cachedResolve
   const iitmURL = new URL('lib/register.js', meta.url).toString()
+  let includeModules, excludeModules
+
+  async function initialize (data) {
+    if (data) {
+      includeModules = ensureArrayWithBareSpecifiersAndFileUrls(data.include, 'include')
+      excludeModules = ensureArrayWithBareSpecifiersAndFileUrls(data.exclude, 'exclude')
+    }
+  }
 
   async function resolve (specifier, context, parentResolve) {
     cachedResolve = parentResolve
 
-    // See github.com/DataDog/import-in-the-middle/pull/76.
+    // See github.com/nodejs/import-in-the-middle/pull/76.
     if (specifier === iitmURL) {
       return {
         url: specifier,
@@ -225,14 +274,27 @@ function createHook (meta) {
     if (isWin && parentURL.indexOf('file:node') === 0) {
       context.parentURL = ''
     }
-    const url = await parentResolve(newSpecifier, context, parentResolve)
-    if (parentURL === '' && !EXTENSION_RE.test(url.url)) {
-      entrypoint = url.url
-      return { url: url.url, format: 'commonjs' }
+    const result = await parentResolve(newSpecifier, context, parentResolve)
+    if (parentURL === '' && !EXTENSION_RE.test(result.url)) {
+      entrypoint = result.url
+      return { url: result.url, format: 'commonjs' }
+    }
+
+    // For included/excluded modules, we check the specifier to match libraries
+    // that are loaded with bare specifiers from node_modules.
+    //
+    // For non-bare specifier imports, we only support matching file URL strings
+    // because using relative paths would be very error prone!
+    if (includeModules && !includeModules.some(lib => lib === specifier || lib === result.url.url)) {
+      return result
+    }
+
+    if (excludeModules && excludeModules.some(lib => lib === specifier || lib === result.url.url)) {
+      return result
     }
 
     if (isIitm(parentURL, meta) || hasIitm(parentURL)) {
-      return url
+      return result
     }
 
     // Node.js v21 renames importAssertions to importAttributes
@@ -240,24 +302,24 @@ function createHook (meta) {
       (context.importAssertions && context.importAssertions.type === 'json') ||
       (context.importAttributes && context.importAttributes.type === 'json')
     ) {
-      return url
+      return result
     }
 
     // If the file is referencing itself, we need to skip adding the iitm search params
-    if (url.url === parentURL) {
+    if (result.url === parentURL) {
       return {
-        url: url.url,
+        url: result.url,
         shortCircuit: true,
-        format: url.format
+        format: result.format
       }
     }
 
-    specifiers.set(url.url, specifier)
+    specifiers.set(result.url, specifier)
 
     return {
-      url: addIitm(url.url),
+      url: addIitm(result.url),
       shortCircuit: true,
-      format: url.format
+      format: result.format
     }
   }
 
@@ -303,7 +365,7 @@ register(${JSON.stringify(realUrl)}, _, set, ${JSON.stringify(specifiers.get(rea
         // it would be very tricky to debug
         const err = new Error(`'import-in-the-middle' failed to wrap '${realUrl}'`)
         err.cause = cause
-        console.warn(err)
+        emitWarning(err)
 
         // Revert back to the non-iitm URL
         url = realUrl
@@ -328,9 +390,10 @@ register(${JSON.stringify(realUrl)}, _, set, ${JSON.stringify(specifiers.get(rea
   }
 
   if (NODE_MAJOR >= 17 || (NODE_MAJOR === 16 && NODE_MINOR >= 12)) {
-    return { load, resolve }
+    return { initialize, load, resolve }
   } else {
     return {
+      initialize,
       load,
       resolve,
       getSource,

package/hook.mjs

@@ -4,6 +4,6 @@
 
 import { createHook } from './hook.js'
 
-const { load, resolve, getFormat, getSource } = createHook(import.meta)
+const { initialize, load, resolve, getFormat, getSource } = createHook(import.meta)
 
-export { load, resolve, getFormat, getSource }
+export { initialize, load, resolve, getFormat, getSource }

package/lib/get-exports.js

@@ -43,6 +43,9 @@ async function getCjsExports (url, context, parentLoad, source) {
           full.add(each)
         }
       } else {
+        if (re === '.') {
+          re = './'
+        }
         // Resolve the re-exported module relative to the current module.
         const newUrl = pathToFileURL(require.resolve(re, { paths: [dirname(fileURLToPath(url))] })).href
         for (const each of await getExports(newUrl, context, parentLoad)) {
@@ -92,23 +95,29 @@ async function getExports (url, context, parentLoad) {
     source = readFileSync(fileURLToPath(url), 'utf8')
   }
 
-  if (format === 'module') {
-    return getEsmExports(source)
-  }
+  try {
+    if (format === 'module') {
+      return getEsmExports(source)
+    }
 
-  if (format === 'commonjs') {
-    return getCjsExports(url, context, parentLoad, source)
-  }
+    if (format === 'commonjs') {
+      return await getCjsExports(url, context, parentLoad, source)
+    }
 
-  // At this point our `format` is either undefined or not known by us. Fall
-  // back to parsing as ESM/CJS.
-  const esmExports = getEsmExports(source)
-  if (!esmExports.length) {
-    // TODO(bengl) it's might be possible to get here if somehow the format
-    // isn't set at first and yet we have an ESM module with no exports.
-    // I couldn't construct an example that would do this, so maybe it's
-    // impossible?
-    return getCjsExports(url, context, parentLoad, source)
+    // At this point our `format` is either undefined or not known by us. Fall
+    // back to parsing as ESM/CJS.
+    const esmExports = getEsmExports(source)
+    if (!esmExports.length) {
+      // TODO(bengl) it's might be possible to get here if somehow the format
+      // isn't set at first and yet we have an ESM module with no exports.
+      // I couldn't construct an example that would do this, so maybe it's
+      // impossible?
+      return await getCjsExports(url, context, parentLoad, source)
+    }
+  } catch (cause) {
+    const err = new Error(`Failed to parse '${url}'`)
+    err.cause = cause
+    throw err
   }
 }
 

package/package.json

@@ -1,10 +1,11 @@
 {
   "name": "import-in-the-middle",
-  "version": "1.8.1",
+  "version": "1.9.0",
   "description": "Intercept imports in Node.js",
   "main": "index.js",
   "scripts": {
-    "test": "c8 --reporter lcov --check-coverage --lines 50 imhotap --files test/{hook,low-level,other,get-esm-exports}/*",
+    "test": "c8 --reporter lcov --check-coverage --lines 50 imhotap --files test/{hook,low-level,other,get-esm-exports,register}/*",
+    "test:e2e": "node test/check-exports/test.mjs",
     "test:ts": "c8 --reporter lcov imhotap --files test/typescript/*.test.mts",
     "coverage": "c8 --reporter html imhotap --files test/{hook,low-level,other,get-esm-exports}/* && echo '\nNow open coverage/index.html\n'",
     "lint": "eslint .",
@@ -12,7 +13,7 @@
   },
   "repository": {
     "type": "git",
-    "url": "git+ssh://git@github.com/DataDog/import-in-the-middle.git"
+    "url": "git+ssh://git@github.com/nodejs/import-in-the-middle.git"
   },
   "keywords": [
     "import",
@@ -25,9 +26,9 @@
   "author": "Bryan English <bryan.english@datadoghq.com>",
   "license": "Apache-2.0",
   "bugs": {
-    "url": "https://github.com/DataDog/import-in-the-middle/issues"
+    "url": "https://github.com/nodejs/import-in-the-middle/issues"
   },
-  "homepage": "https://github.com/DataDog/import-in-the-middle#readme",
+  "homepage": "https://github.com/nodejs/import-in-the-middle#readme",
   "imhotap": {
     "runner": "node",
     "test-env": "NODE_OPTIONS=--no-warnings --require ./test/version-check.js --experimental-loader ./test/generic-loader.mjs"

package/release-please-config.json

@@ -0,0 +1,10 @@
+{
+  "$schema": "https://raw.githubusercontent.com/googleapis/release-please/main/schemas/config.json",
+  "bootstrap-sha": "57d2459748492abd55e04a679d8b5752b90dfab4",
+  "packages": {
+    ".": {
+      "pull-request-title-pattern": "chore: release v${version}",
+      "release-type": "node"
+    }
+  }
+}

package/test/check-exports/test.mjs

@@ -0,0 +1,317 @@
+import { spawnSync } from 'child_process'
+import { deepStrictEqual } from 'assert'
+import { dirname, resolve } from 'path'
+import { fileURLToPath } from 'url'
+
+const cwd = dirname(fileURLToPath(import.meta.url))
+const hook = resolve(cwd, '..', '..', 'hook.mjs')
+
+const mostPopular240NpmModules = [
+  'ansi-styles',
+  'semver',
+  'supports-color',
+  'chalk', 'has-flag',
+  'debug',
+  'tslib',
+  'color-convert',
+  'ms',
+  'color-name',
+  'lru-cache',
+  'minimatch',
+  'strip-ansi',
+  'source-map',
+  'ansi-regex',
+  'glob',
+  'readable-stream',
+  'commander',
+  'yallist',
+  'string-width',
+  'escape-string-regexp',
+  'brace-expansion',
+  'find-up',
+  'p-locate',
+  'locate-path',
+  'wrap-ansi',
+  'p-limit',
+  'safe-buffer',
+  'kind-of',
+  'minipass',
+  'uuid',
+  'string_decoder',
+  'ajv',
+  'emoji-regex',
+  'isarray',
+  'react-is',
+  'fs-extra',
+  'is-fullwidth-code-point',
+  'get-stream',
+  'json-schema-traverse',
+  'yargs-parser',
+  'glob-parent',
+  'yargs',
+  'rimraf',
+  'acorn',
+  'which',
+  'estraverse',
+  'js-yaml',
+  'path-exists',
+  'argparse',
+  'pretty-format',
+  'resolve-from',
+  'cliui',
+  'schema-utils',
+  'globals',
+  'camelcase',
+  'execa',
+  'punycode',
+  'path-key',
+  'signal-exit',
+  'inherits',
+  'resolve',
+  'mkdirp',
+  'is-stream',
+  'ws',
+  'universalify',
+  'qs',
+  'slash',
+  'json5',
+  'iconv-lite',
+  'form-data',
+  'is-number',
+  'eslint-visitor-keys',
+  '@jest/types',
+  'postcss',
+  'make-dir',
+  'pify',
+  'cross-spawn',
+  'braces',
+  'whatwg-url',
+  'fill-range',
+  'eslint-scope',
+  'tr46',
+  'micromatch',
+  'convert-source-map',
+  'define-property',
+  'agent-base',
+  'shebang-regex',
+  'shebang-command',
+  'mimic-fn',
+  'globby',
+  'npm-run-path',
+  'mime',
+  '@babel/code-frame',
+  'extend-shallow',
+  'to-regex-range',
+  'onetime',
+  'https-proxy-agent',
+  'y18n',
+  'buffer',
+  'strip-bom',
+  'is-glob',
+  'doctrine',
+  'picocolors',
+  'pkg-dir',
+  '@babel/types',
+  'regenerator-runtime',
+  'human-signals',
+  '@jridgewell/trace-mapping',
+  'ignore',
+  'jsesc',
+  'parse-json',
+  'jest-worker',
+  'graceful-fs',
+  'jest-util',
+  'jsonfile',
+  'normalize-path',
+  'strip-json-comments',
+  'cosmiconfig',
+  'minimist',
+  'path-type',
+  '@babel/parser',
+  'balanced-match',
+  'picomatch',
+  'typescript',
+  'isexe',
+  'statuses',
+  'entities',
+  'bytes',
+  'node-fetch',
+  'http-errors',
+  '@babel/highlight',
+  '@babel/helper-validator-identifier',
+  'function-bind',
+  'async',
+  'sprintf-js',
+  '@babel/generator',
+  'is-extendable',
+  'get-intrinsic',
+  'lodash',
+  'mime-db',
+  'source-map-support',
+  'mime-types',
+  'is-arrayish',
+  '@babel/core',
+  'once',
+  'anymatch',
+  'depd',
+  'hosted-git-info',
+  'path-to-regexp',
+  'axios',
+  'is-core-module',
+  '@babel/template',
+  'cookie',
+  'write-file-atomic',
+  'js-tokens',
+  '@typescript-eslint/typescript-estree',
+  '@typescript-eslint/types',
+  'object-inspect',
+  'wrappy',
+  'is-extglob',
+  'chokidar',
+  '@typescript-eslint/visitor-keys',
+  'call-bind',
+  'loader-utils',
+  'browserslist',
+  'http-proxy-agent',
+  'fast-glob',
+  'concat-map',
+  'inflight',
+  'ajv-keywords',
+  'ansi-escapes',
+  'ci-info',
+  'fast-deep-equal',
+  'caniuse-lite',
+  'fs.realpath',
+  '@jridgewell/gen-mapping',
+  'setprototypeof',
+  'strip-final-newline',
+  'optionator',
+  'path-is-absolute',
+  '@babel/traverse',
+  'core-util-is',
+  'has-symbols',
+  'yocto-queue',
+  'p-try',
+  'electron-to-chromium',
+  '@smithy/smithy-client',
+  'yaml',
+  'ini',
+  '@babel/helper-plugin-utils',
+  'jest-get-type',
+  'type-check',
+  'levn',
+  'is-descriptor',
+  'prelude-ls',
+  'slice-ansi',
+  '@typescript-eslint/scope-manager',
+  'isobject',
+  'esprima',
+  '@babel/helper-split-export-declaration',
+  'callsites',
+  'readdirp',
+  'escalade',
+  'import-fresh',
+  'get-caller-file',
+  '@jridgewell/sourcemap-codec',
+  'acorn-walk',
+  'rxjs',
+  'ieee754',
+  'is-plain-obj',
+  'istanbul-lib-instrument',
+  '@babel/helper-module-imports',
+  'side-channel',
+  'normalize-package-data',
+  'is-plain-object',
+  '@jridgewell/resolve-uri',
+  'follow-redirects',
+  'array-union',
+  'json-parse-even-better-errors',
+  'path-parse',
+  'has-property-descriptors',
+  'uri-js',
+  'safer-buffer',
+  '@babel/helpers',
+  'on-finished',
+  '@babel/helper-function-name',
+  'p-map',
+  'postcss-value-parser',
+  'indent-string',
+  '@babel/helper-module-transforms',
+  'object-assign',
+  'delayed-stream',
+  '@nodelib/fs.stat',
+  'require-directory',
+  'diff',
+  'parse5',
+  'asynckit',
+  'tmp',
+  'combined-stream'
+]
+
+const otherCommonModulesUsedWithInstrumentation = [
+  'express',
+  'fastify',
+  '@hapi/hapi',
+  'connect',
+  'svelte',
+  '@sveltejs/kit',
+  'next',
+  'gatsby',
+  '@remix-run/node',
+  '@remix-run/react'
+]
+
+const modules = [...mostPopular240NpmModules, ...otherCommonModulesUsedWithInstrumentation]
+
+function installLibs (names) {
+  spawnSync('npm', ['init', '-y'], { cwd })
+  spawnSync('npm', ['install', ...names], { cwd })
+}
+
+function getExports (name, loader) {
+  const args = ['--input-type=module', '--no-warnings', '-e', `import * as lib from '${name}'; console.log(JSON.stringify(Object.keys(lib)))`]
+  if (loader) args.push(loader)
+  const out = spawnSync(process.execPath, args, { cwd })
+  if (out.status !== 0) {
+    console.error(out.stderr.toString())
+    throw new Error(`Getting exports returned non-zero exit code '${name}'`)
+  }
+  const stdout = out.stdout.toString()
+  return JSON.parse(stdout).sort()
+}
+
+const NPM_LIST_SEMVER_PARSE = /└──.*@((0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*)(?:-((?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*)(?:\.(?:0|[1-9]\d*|\d*[a-zA-Z-][0-9a-zA-Z-]*))*))?(?:\+([0-9a-zA-Z-]+(?:\.[0-9a-zA-Z-]+)*))?)/
+
+function getVersion (name) {
+  const result = spawnSync('npm', ['list', name, '--depth', '0'], { cwd })
+  const stdout = result.output.toString()
+  const [, version] = stdout.match(NPM_LIST_SEMVER_PARSE)
+  return version
+}
+
+function testLib (name) {
+  const version = getVersion(name)
+  try {
+    const expected = getExports(name)
+    const actual = getExports(name, `--experimental-loader=${hook}`)
+    deepStrictEqual(actual, expected, `Exports for ${name} are different`)
+    console.log(`✅  Exports for ${name}@${version} match`)
+    return false
+  } catch (err) {
+    console.error(`❌  Error getting exports for ${name}@${version}:`, err)
+    return true
+  }
+}
+
+console.log(`📦  Installing ${modules.length} libraries...`)
+installLibs(modules)
+
+let errored = false
+for (const mod of modules) {
+  errored += testLib(mod)
+}
+
+if (errored) {
+  console.error('❌  Some tests failed')
+  process.exit(1)
+}

package/test/fixtures/index.js

@@ -0,0 +1 @@
+module.exports.foo = 'something'

package/test/fixtures/require-root.js

@@ -0,0 +1 @@
+module.exports = { ...require('.') }

package/test/generic-loader.mjs

@@ -8,7 +8,7 @@ import path from 'path'
 
 const filename = process.env.IITM_TEST_FILE
 
-export const { load, resolve, getFormat, getSource } =
-  filename.includes('disabled')
+export const { initialize, load, resolve, getFormat, getSource } =
+  filename.includes('disabled') || filename.includes('register')
     ? {}
     : (path.extname(filename).slice(-2) === 'ts' ? tsLoader : regularLoader)

package/test/hook/require-root-cjs.mjs

@@ -0,0 +1,12 @@
+import Hook from '../../index.js'
+import { foo } from '../fixtures/require-root.js'
+import { strictEqual } from 'assert'
+
+Hook((exports, name) => {
+  if (name.endsWith('require-root.js')) {
+    strictEqual(exports.foo, 'something')
+    exports.foo += '-wrap'
+  }
+})
+
+strictEqual(foo, 'something-wrap')

package/test/register/v18.19-exclude.mjs

@@ -0,0 +1,15 @@
+import { register } from 'module'
+import Hook from '../../index.js'
+import { strictEqual } from 'assert'
+
+register('../../hook.mjs', import.meta.url, { data: { exclude: ['util'] } })
+
+const hooked = []
+
+Hook((exports, name) => {
+  hooked.push(name)
+})
+
+await import('openai')
+
+strictEqual(hooked.includes('util'), false)

package/test/register/v18.19-include.mjs

@@ -0,0 +1,16 @@
+import { register } from 'module'
+import Hook from '../../index.js'
+import { strictEqual } from 'assert'
+
+register('../../hook.mjs', import.meta.url, { data: { include: ['openai'] } })
+
+const hooked = []
+
+Hook((exports, name) => {
+  hooked.push(name)
+})
+
+await import('openai')
+
+strictEqual(hooked.length, 1)
+strictEqual(hooked[0], 'openai')