test: Test 250+ popular npm packages

.github/workflows/ci.yml

@@ -103,3 +103,15 @@ jobs:
       - name: Verify Minimum Coverage Is Met
         run: >
           lcov --summary all_lcov.info | grep lines | cut -d' ' -f 4 | cut -d% -f 1 | xargs node -e "x=process.argv[1];console.log(x);assert(+x >= 90)"
+
+  integration:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      # We only test that exports match on a single node version because many of
+      # the libraries we test do not support older node versions
+      - uses: actions/setup-node@v4
+        with:
+          node-version: '22.x'
+      - run: npm install
+      - run: npm run test:e2e

.gitignore

@@ -2,4 +2,5 @@ package-lock.json
 yarn.lock
 node_modules
 coverage
-.DS_Store
+.DS_Store
+test/check-exports/package.json

package.json

@@ -5,6 +5,7 @@
   "main": "index.js",
   "scripts": {
     "test": "c8 --reporter lcov --check-coverage --lines 50 imhotap --files test/{hook,low-level,other,get-esm-exports}/*",
+    "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 .",

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)
+}