refactor(test): improve test infrastructure and coverage

- Increase coverage thresholds to 98% for lines and statements
- Configure type coverage to 100% with strict settings
- Optimize npm package test performance
- Standardize module identifier to import.meta.url
- Use crypto.randomUUID in temp-file-helper
- Correct assertions and remove duplicate tests
- Add c8 ignore comments with required reasons
- Update test runner with better staged file handling

Author: jdalton

.config/vitest.config.mts

@@ -171,7 +171,7 @@ export default defineConfig({
       reportsDirectory: 'coverage',
       reporter: ['text', 'json', 'html', 'lcov', 'clover'],
       // ONLY include registry/src production code (not types.ts)
-      include: ['registry/src/index.{ts,mts,cts}'],
+      include: ['registry/src/**/*.{ts,mts,cts}'],
       // Exclude everything else (must include vitest defaults)
       exclude: [
         // Vitest defaults
@@ -193,16 +193,16 @@ export default defineConfig({
         'registry/plugins/**',
         'registry/dist/**',
       ],
-      // Set to false to ONLY report files in include list that are executed
-      all: false,
+      // Set to true to report ALL files in include list, even if not executed
+      all: true,
       clean: true,
       skipFull: false,
       ignoreClassMethods: ['constructor'],
       thresholds: {
-        lines: 1,
-        functions: 80,
-        branches: 80,
-        statements: 1,
+        lines: 98,
+        functions: 100,
+        branches: 100,
+        statements: 98,
       },
     },
   },

perf/npm/json-stable-stringify.perf.mts

@@ -7,13 +7,13 @@ import { getDefaultLogger } from '@socketsecurity/lib/logger'
 import fastJsonStableStringify from 'fast-json-stable-stringify'
 import { Bench } from 'tinybench'
 
-import constants from '../../scripts/constants.mjs'
+import { PERF_NPM_FIXTURES_PATH } from '../../scripts/constants/paths.mjs'
 
 const logger = getDefaultLogger()
 
 void (async () => {
   const sampleData2MbJson = require(
-    path.join(constants.perfNpmFixturesPath, 'sample_data_2mb.json'),
+    path.join(PERF_NPM_FIXTURES_PATH, 'sample_data_2mb.json'),
   )
   const sampleData6MbJson = {
     a: sampleData2MbJson,

test/npm/array-flatten.test.mts

@@ -1,3 +1,4 @@
+/** @fileoverview Tests for @socketregistry/array-flatten npm package override. */
 import { describe, expect, it } from 'vitest'
 
 import { setupNpmPackageTest } from '../utils/npm-package-helper.mts'
@@ -7,7 +8,7 @@ const {
   module: flattenLegacy,
   skip,
   sockRegPkgName,
-} = await setupNpmPackageTest(__filename)
+} = await setupNpmPackageTest(import.meta.url)
 const flatten = flattenLegacy?.flatten
 
 describe(`${eco} > ${sockRegPkgName}`, { skip }, () => {

test/npm/assert.test.mts

@@ -14,7 +14,7 @@ const {
   pkgPath,
   skip,
   sockRegPkgName,
-} = await setupNpmPackageTest(__filename)
+} = await setupNpmPackageTest(import.meta.url)
 
 // assert package tests use old util.isError which was deprecated and removed
 // in Node.js 20+.

test/npm/date.test.mts

@@ -1,3 +1,4 @@
+/** @fileoverview Tests for @socketregistry/date npm package override. */
 import { describe, expect, it } from 'vitest'
 
 const DateImpl = require('../../packages/npm/date/Date')
@@ -92,6 +93,8 @@ describe('date', () => {
     })
   })
 
+  // Date string formatting is platform-dependent and may produce different
+  // output on Windows due to timezone and locale differences.
   describe(
     'Date string formatting',
     { skip: process.platform === 'win32' },

test/npm/deep-equal.test.mts

@@ -18,7 +18,7 @@ const {
   pkgPath,
   skip,
   sockRegPkgName,
-} = await setupNpmPackageTest(__filename)
+} = await setupNpmPackageTest(import.meta.url)
 
 // deep-equal package tests may have issues with test dependencies.
 describe(`${eco} > ${sockRegPkgName}`, { skip }, () => {

test/npm/define-properties.test.mts

@@ -1,3 +1,4 @@
+/** @fileoverview Tests for @socketregistry/define-properties npm package override. */
 import { describe, expect, it } from 'vitest'
 
 const define = require('../../packages/npm/define-properties')

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

@@ -1,3 +1,4 @@
+/** @fileoverview Tests for @socketregistry/es6-object-assign npm package override. */
 import path from 'node:path'
 // eslint-disable-next-line n/no-extraneous-import
 import { getDefaultLogger } from '@socketsecurity/lib/logger'
@@ -6,8 +7,9 @@ import { setupNpmPackageTest } from '../utils/npm-package-helper.mts'
 
 const logger = getDefaultLogger()
 
-const { eco, pkgPath, skip, sockRegPkgName } =
-  await setupNpmPackageTest(__filename)
+const { eco, pkgPath, skip, sockRegPkgName } = await setupNpmPackageTest(
+  import.meta.url,
+)
 
 // es6-object-assign has no unit tests.
 // https://github.com/rubennorte/es6-object-assign/tree/v1.1.0

test/npm/harmony-reflect.test.mts

@@ -1,11 +1,13 @@
+/** @fileoverview Tests for @socketregistry/harmony-reflect npm package override. */
 import path from 'node:path'
 
 import { describe, expect, it } from 'vitest'
 
 import { setupNpmPackageTest } from '../utils/npm-package-helper.mts'
 
-const { eco, pkgPath, skip, sockRegPkgName } =
-  await setupNpmPackageTest(__filename)
+const { eco, pkgPath, skip, sockRegPkgName } = await setupNpmPackageTest(
+  import.meta.url,
+)
 
 // 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/hasown.test.mts

@@ -1,3 +1,4 @@
+/** @fileoverview Tests for @socketregistry/hasown npm package override. */
 import { describe, expect, it } from 'vitest'
 
 const hasOwn = require('../../packages/npm/hasown')