Update test files and clean up legacy types

- Update test imports and configurations across all test files
- Remove obsolete external.d.ts type declarations file
- Replace eslint.config.js with eslint.config.mjs in npm tests
- Add missing rebuild-registry-exports.mjs script
- Update all test imports to use proper module paths

Author: jdalton

registry/src/types/external.d.ts

@@ -0,0 +1,163 @@
+import { readFile, writeFile } from 'node:fs/promises'
+import path from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+import fastGlob from 'fast-glob'
+
+const __filename = fileURLToPath(import.meta.url)
+const __dirname = path.dirname(__filename)
+
+const registryPath = path.join(__dirname, '..', 'registry')
+const packageJsonPath = path.join(registryPath, 'package.json')
+
+async function main() {
+  const packageJson = JSON.parse(await readFile(packageJsonPath, 'utf8'))
+
+  // Find all files in dist directory
+  const distFiles = await fastGlob.glob(['dist/**/*.{js,d.ts}'], {
+    cwd: registryPath,
+    ignore: ['node_modules/**', '**/*.test.*'],
+  })
+
+  // Also include JSON files at root.
+  const jsonFiles = [
+    'extensions.json',
+    'manifest.json',
+    'package.json',
+    'tsconfig.json',
+  ]
+
+  // Add JSON files.
+  const exports = {}
+  for (const jsonFile of jsonFiles) {
+    exports[`./${jsonFile}`] = `./${jsonFile}`
+  }
+
+  // Process dist files.
+  for (const file of distFiles) {
+    const ext = file.endsWith('.d.ts') ? '.d.ts' : path.extname(file)
+    const relativePath = file.slice('dist/'.length)
+    const exportPath = `./${relativePath.slice(0, -ext.length)}`
+    const isDts = ext === '.d.ts'
+
+    if (exports[exportPath]) {
+      exports[exportPath][isDts ? 'types' : 'default'] = `./${file}`
+    } else {
+      exports[exportPath] = {
+        types: isDts ? `./${file}` : undefined,
+        default: isDts ? undefined : `./${file}`,
+      }
+    }
+
+    // Clean up single property objects.
+    if (exports[exportPath].types === undefined) {
+      exports[exportPath] = exports[exportPath].default
+    } else if (exports[exportPath].default === undefined) {
+      delete exports[exportPath].default
+    }
+
+    // Add index exports for directories.
+    const basename = path.basename(relativePath, ext)
+    if (basename === 'index') {
+      const dirPath = `./${path.dirname(relativePath)}`
+      if (dirPath !== './.') {
+        if (exports[dirPath]) {
+          exports[dirPath][isDts ? 'types' : 'default'] = `./${file}`
+        } else {
+          exports[dirPath] = {
+            types: isDts ? `./${file}` : undefined,
+            default: isDts ? undefined : `./${file}`,
+          }
+        }
+        // Clean up single property objects.
+        if (exports[dirPath].types === undefined) {
+          exports[dirPath] = exports[dirPath].default
+        } else if (exports[dirPath].default === undefined) {
+          delete exports[dirPath].default
+        }
+      }
+    }
+  }
+
+  // Add dual exports for constants (both lowercase and uppercase).
+  const constantsExports = {}
+  for (const [exportPath, exportValue] of Object.entries(exports)) {
+    if (
+      exportPath.startsWith('./lib/constants/') &&
+      exportPath !== './lib/constants'
+    ) {
+      const pathAfterConstants = exportPath.slice('./lib/constants/'.length)
+
+      // Always add the lowercase-hyphenated version.
+      const lowercasePath = `./lib/constants/${pathAfterConstants.toLowerCase().replace(/_/g, '-')}`
+      constantsExports[lowercasePath] = exportValue
+
+      // Always add the UPPERCASE_UNDERSCORE version.
+      const uppercasePath = `./lib/constants/${pathAfterConstants.toUpperCase().replace(/-/g, '_')}`
+      constantsExports[uppercasePath] = exportValue
+    }
+  }
+
+  // Merge all exports.
+  Object.assign(exports, constantsExports)
+
+  // Sort exports.
+  const sortedExports = {}
+  const sortedKeys = Object.keys(exports).sort()
+  for (const key of sortedKeys) {
+    sortedExports[key] = exports[key]
+  }
+
+  // Add browser field for Node.js built-ins.
+  const browser = {}
+  const builtinNames = [
+    'assert',
+    'buffer',
+    'child_process',
+    'cluster',
+    'console',
+    'constants',
+    'crypto',
+    'dgram',
+    'dns',
+    'domain',
+    'events',
+    'fs',
+    'http',
+    'https',
+    'module',
+    'net',
+    'os',
+    'path',
+    'process',
+    'punycode',
+    'querystring',
+    'readline',
+    'stream',
+    'string_decoder',
+    'sys',
+    'timers',
+    'tls',
+    'tty',
+    'url',
+    'util',
+    'vm',
+    'zlib',
+  ]
+  for (const name of builtinNames) {
+    browser[name] = false
+  }
+
+  packageJson.exports = sortedExports
+  packageJson.browser = browser
+
+  await writeFile(packageJsonPath, JSON.stringify(packageJson, null, 2) + '\n')
+  console.log(
+    `✅ Created ${Object.keys(sortedExports).length} exports in registry/package.json`,
+  )
+  console.log(
+    `   Including ${Object.keys(constantsExports).length / 2} dual constant exports`,
+  )
+}
+
+main().catch(console.error)

scripts/rebuild-registry-exports.mjs

@@ -1,6 +1,6 @@
 import { describe, expect, it } from 'vitest'
 
-import { isPnpmIgnoreScriptsFlag } from '@socketsecurity/registry/lib/agent'
+import { isPnpmIgnoreScriptsFlag } from '../registry/dist/lib/agent.js'
 
 // Note: execPnpm integration testing would require mocking the spawn/execBin functions.
 // which is complex due to the module structure. The implementation can be tested.

test/npm-exec.test.mts

@@ -9,7 +9,7 @@ import { isPackageTestingSkipped } from '../../scripts/utils/tests.mjs'
 const { NPM } = constants
 
 const eco = NPM
-const sockRegPkgName = path.basename(__filename, '.test.ts')
+const sockRegPkgName = path.basename(__filename, '.test.mts')
 
 describe(
   `${eco} > ${sockRegPkgName}`,

test/npm/array-flatten.test.mts

@@ -2,16 +2,15 @@ import path from 'node:path'
 
 import { beforeAll, describe, expect, it } from 'vitest'
 
-import { logger } from '@socketsecurity/registry/lib/logger'
-
+import { logger } from '../../registry/dist/lib/logger.js'
 import constants from '../../scripts/constants.mjs'
 import { installPackageForTesting } from '../../scripts/utils/package-utils.mjs'
 import { isPackageTestingSkipped } from '../../scripts/utils/tests.mjs'
 
 const { NPM } = constants
 
 const eco = NPM
-const sockRegPkgName = path.basename(__filename, '.test.ts')
+const sockRegPkgName = path.basename(__filename, '.test.mts')
 
 // es6-object-assign has no unit tests.
 // https://github.com/rubennorte/es6-object-assign/tree/v1.1.0

test/npm/es6-object-assign.test.mts

@@ -0,0 +1,7 @@
+import constants from '@socketregistry/scripts/constants'
+
+export default [
+  {
+    extends: constants.rootEslintConfigPath,
+  },
+]

test/npm/eslint.config.js

@@ -9,7 +9,7 @@ import { isPackageTestingSkipped } from '../../scripts/utils/tests.mjs'
 const { NPM } = constants
 
 const eco = NPM
-const sockRegPkgName = path.basename(__filename, '.test.ts')
+const sockRegPkgName = path.basename(__filename, '.test.mts')
 
 // harmony-reflect has known failures in its package and requires running tests in browser.
 // https://github.com/tvcutsem/harmony-reflect/tree/v1.6.2/test

test/npm/eslint.config.mjs

@@ -10,7 +10,7 @@ import { isPackageTestingSkipped } from '../../scripts/utils/tests.mjs'
 const { NPM, UTF8 } = constants
 
 const eco = NPM
-const sockRegPkgName = path.basename(__filename, '.test.ts')
+const sockRegPkgName = path.basename(__filename, '.test.mts')
 
 // @hyrious/bun.lockb has no unit tests.
 // https://github.com/hyrious/bun.lockb/tree/v0.0.4

test/npm/harmony-reflect.test.mts

@@ -9,7 +9,7 @@ import { setupMultiEntryTest } from '../utils/test-helpers.mjs'
 const { NPM } = constants
 
 const eco = NPM
-const sockRegPkgName = path.basename(__filename, '.test.ts')
+const sockRegPkgName = path.basename(__filename, '.test.mts')
 
 // is-regex tests don't account for `is-regex` backed by.
 // `require('node:util/types).isRegExp` which triggers no proxy traps and.