Apply Biome auto-fixes across codebase

- Use template literals instead of string concatenation
- Replace bracket notation with dot notation for literal keys
- Use Number namespace for parseFloat
- Fix import ordering and formatting
- Update test type annotations
- Fix TypeScript strict type issues

Author: jdalton

.config/esbuild.config.mjs

@@ -39,21 +39,23 @@ export const buildConfig = {
   external: [
     // Node.js built-ins
     ...builtinModules,
-    ...builtinModules.map(m => `node:${m}`)
+    ...builtinModules.map(m => `node:${m}`),
   ],
 
   // Banner for generated code
   banner: {
-    js: '/* Socket PackageURL - Built with esbuild */'
+    js: '/* Socket PackageURL - Built with esbuild */',
   },
 
   // TypeScript configuration
   tsconfig: path.join(rootPath, 'tsconfig.json'),
 
   // Define constants for optimization
   define: {
-    'process.env.NODE_ENV': JSON.stringify(process.env.NODE_ENV || 'production')
-  }
+    'process.env.NODE_ENV': JSON.stringify(
+      process.env.NODE_ENV || 'production',
+    ),
+  },
 }
 
 // Watch configuration for development with incremental builds
@@ -66,7 +68,7 @@ export const watchConfig = {
   watch: {
     // This will be extracted and not passed to context()
     // Rebuild logging is handled via plugin in build script
-  }
+  },
 }
 
 /**
@@ -81,14 +83,14 @@ function analyzeMetafile(metafile) {
     totalSize += output.bytes
     return {
       name: path.relative(rootPath, file),
-      size: (output.bytes / 1024).toFixed(2) + ' KB'
+      size: `${(output.bytes / 1024).toFixed(2)} KB`,
     }
   })
 
   return {
     files,
-    totalSize: (totalSize / 1024).toFixed(2) + ' KB'
+    totalSize: `${(totalSize / 1024).toFixed(2)} KB`,
   }
 }
 
-export { analyzeMetafile }
+export { analyzeMetafile }

.config/eslint.config.mjs

@@ -30,7 +30,7 @@ const nodeGlobalsConfig = Object.fromEntries(
 const biomeConfigPath = path.join(rootPath, 'biome.json')
 const biomeConfig = require(biomeConfigPath)
 const biomeIgnores = {
-  name: `Imported biome.json ignore patterns`,
+  name: 'Imported biome.json ignore patterns',
   ignores: biomeConfig.files.includes
     .filter(p => p.startsWith('!'))
     .map(p => convertIgnorePatternToMinimatch(p.slice(1))),
@@ -39,7 +39,7 @@ const biomeIgnores = {
 const gitignorePath = path.join(rootPath, '.gitignore')
 const gitIgnores = {
   ...includeIgnoreFile(gitignorePath),
-  name: `Imported .gitignore ignore patterns`,
+  name: 'Imported .gitignore ignore patterns',
 }
 
 export default [

.config/tsconfig.check.json

@@ -9,7 +9,9 @@
     "paths": {
       "@socketsecurity/lib": ["../../socket-lib/dist/index.d.ts"],
       "@socketsecurity/lib/*": ["../../socket-lib/dist/*"],
-      "@socketsecurity/registry": ["../../socket-registry/registry/dist/index.d.ts"],
+      "@socketsecurity/registry": [
+        "../../socket-registry/registry/dist/index.d.ts"
+      ],
       "@socketsecurity/registry/*": ["../../socket-registry/registry/dist/*"]
     },
     "skipLibCheck": true,

.config/vitest.config.isolated.mts

@@ -6,8 +6,8 @@ import { defineConfig } from 'vitest/config'
 
 // Check if coverage is enabled via CLI flags or environment.
 const isCoverageEnabled =
-  process.env['COVERAGE'] === 'true' ||
-  process.env['npm_lifecycle_event']?.includes('coverage') ||
+  process.env.COVERAGE === 'true' ||
+  process.env.npm_lifecycle_event?.includes('coverage') ||
   process.argv.some(arg => arg.includes('coverage'))
 
 export default defineConfig({

.config/vitest.config.mts

@@ -9,8 +9,8 @@ const __dirname = path.dirname(fileURLToPath(import.meta.url))
 
 // Check if coverage is enabled via CLI flags or environment.
 const isCoverageEnabled =
-  process.env['COVERAGE'] === 'true' ||
-  process.env['npm_lifecycle_event']?.includes('coverage') ||
+  process.env.COVERAGE === 'true' ||
+  process.env.npm_lifecycle_event?.includes('coverage') ||
   process.argv.some(arg => arg.includes('coverage'))
 
 export default defineConfig({

scripts/build.mjs

@@ -14,10 +14,17 @@ import { parseArgs } from '@socketsecurity/lib/argv/parse'
 import { logger } from '@socketsecurity/lib/logger'
 import { printFooter, printHeader } from '@socketsecurity/lib/stdio/header'
 
+import {
+  analyzeMetafile,
+  buildConfig,
+  watchConfig,
+} from '../.config/esbuild.config.mjs'
 import { runSequence } from './utils/run-command.mjs'
-import { analyzeMetafile, buildConfig, watchConfig } from '../.config/esbuild.config.mjs'
 
-const rootPath = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..')
+const rootPath = path.resolve(
+  path.dirname(fileURLToPath(import.meta.url)),
+  '..',
+)
 
 /**
  * Build source code with esbuild.
@@ -33,7 +40,10 @@ async function buildSource(options = {}) {
   // Clean dist directory if needed
   if (!skipClean) {
     const exitCode = await runSequence([
-      { args: ['scripts/load.cjs', 'clean', '--dist', '--quiet'], command: 'node' }
+      {
+        args: ['scripts/load.cjs', 'clean', '--dist', '--quiet'],
+        command: 'node',
+      },
     ])
     if (exitCode !== 0) {
       if (!quiet) {
@@ -49,7 +59,7 @@ async function buildSource(options = {}) {
     const logLevel = quiet ? 'silent' : verbose ? 'info' : 'warning'
     const result = await build({
       ...buildConfig,
-      logLevel
+      logLevel,
     })
     const buildTime = Date.now() - startTime
 
@@ -68,7 +78,11 @@ async function buildSource(options = {}) {
  * Returns exitCode for external logging.
  */
 async function buildTypes(options = {}) {
-  const { quiet = false, skipClean = false, verbose: _verbose = false } = options
+  const {
+    quiet = false,
+    skipClean = false,
+    verbose: _verbose = false,
+  } = options
 
   if (!quiet) {
     logger.substep('Building TypeScript declarations')
@@ -77,7 +91,10 @@ async function buildTypes(options = {}) {
   const commands = []
 
   if (!skipClean) {
-    commands.push({ args: ['scripts/load.cjs', 'clean', '--types', '--quiet'], command: 'node' })
+    commands.push({
+      args: ['scripts/load.cjs', 'clean', '--types', '--quiet'],
+      command: 'node',
+    })
   }
 
   commands.push({
@@ -122,7 +139,7 @@ async function watchBuild(options = {}) {
         {
           name: 'rebuild-logger',
           setup(build) {
-            build.onEnd((result) => {
+            build.onEnd(result => {
               if (result.errors.length > 0) {
                 if (!quiet) {
                   logger.error('Rebuild failed')
@@ -137,9 +154,9 @@ async function watchBuild(options = {}) {
                 }
               }
             })
-          }
-        }
-      ]
+          },
+        },
+      ],
     })
 
     // Enable watch mode
@@ -226,7 +243,9 @@ async function main() {
       console.log('  --help       Show this help message')
       console.log('  --src        Build source code only')
       console.log('  --types      Build TypeScript declarations only')
-      console.log('  --watch      Watch mode with incremental builds (68% faster rebuilds)')
+      console.log(
+        '  --watch      Watch mode with incremental builds (68% faster rebuilds)',
+      )
       console.log('  --needed     Only build if dist files are missing')
       console.log('  --analyze    Show bundle size analysis')
       console.log('  --quiet, --silent  Suppress progress messages')
@@ -235,9 +254,13 @@ async function main() {
       console.log('  pnpm build              # Full build (source + types)')
       console.log('  pnpm build --src        # Build source only')
       console.log('  pnpm build --types      # Build types only')
-      console.log('  pnpm build --watch      # Watch mode with incremental builds')
+      console.log(
+        '  pnpm build --watch      # Watch mode with incremental builds',
+      )
       console.log('  pnpm build --analyze    # Build with size analysis')
-      console.log('\nNote: Watch mode uses esbuild context API for 68% faster rebuilds')
+      console.log(
+        '\nNote: Watch mode uses esbuild context API for 68% faster rebuilds',
+      )
       process.exitCode = 0
       return
     }
@@ -279,18 +302,22 @@ async function main() {
       if (!quiet) {
         logger.step('Building source only')
       }
-      const { buildTime, exitCode: srcExitCode, result } = await buildSource({ quiet, verbose, analyze: values.analyze })
+      const {
+        buildTime,
+        exitCode: srcExitCode,
+        result,
+      } = await buildSource({ quiet, verbose, analyze: values.analyze })
       exitCode = srcExitCode
       if (exitCode === 0 && !quiet) {
         logger.substep(`Source build complete in ${buildTime}ms`)
 
-        if (values.analyze && result?.['metafile']) {
-          const analysis = analyzeMetafile(result['metafile'])
+        if (values.analyze && result?.metafile) {
+          const analysis = analyzeMetafile(result.metafile)
           logger.info('Build output:')
-          for (const file of analysis['files']) {
-            logger.substep(`${file['name']}: ${file['size']}`)
+          for (const file of analysis.files) {
+            logger.substep(`${file.name}: ${file.size}`)
           }
-          logger.step(`Total bundle size: ${analysis['totalSize']}`)
+          logger.step(`Total bundle size: ${analysis.totalSize}`)
         }
       }
     }
@@ -305,7 +332,10 @@ async function main() {
         logger.substep('Cleaning build directories')
       }
       exitCode = await runSequence([
-        { args: ['scripts/load.cjs', 'clean', '--dist', '--types', '--quiet'], command: 'node' }
+        {
+          args: ['scripts/load.cjs', 'clean', '--dist', '--types', '--quiet'],
+          command: 'node',
+        },
       ])
       if (exitCode !== 0) {
         if (!quiet) {
@@ -317,22 +347,27 @@ async function main() {
 
       // Run source and types builds in parallel
       const [srcResult, typesExitCode] = await Promise.all([
-        buildSource({ quiet, verbose, skipClean: true, analyze: values.analyze }),
-        buildTypes({ quiet, verbose, skipClean: true })
+        buildSource({
+          quiet,
+          verbose,
+          skipClean: true,
+          analyze: values.analyze,
+        }),
+        buildTypes({ quiet, verbose, skipClean: true }),
       ])
 
       // Log completion messages in order
       if (!quiet) {
-        if (srcResult['exitCode'] === 0) {
-          logger.substep(`Source build complete in ${srcResult['buildTime']}ms`)
+        if (srcResult.exitCode === 0) {
+          logger.substep(`Source build complete in ${srcResult.buildTime}ms`)
 
-          if (values.analyze && srcResult['result']?.['metafile']) {
-            const analysis = analyzeMetafile(srcResult['result']['metafile'])
+          if (values.analyze && srcResult.result?.metafile) {
+            const analysis = analyzeMetafile(srcResult.result.metafile)
             logger.info('Build output:')
-            for (const file of analysis['files']) {
-              logger.substep(`${file['name']}: ${file['size']}`)
+            for (const file of analysis.files) {
+              logger.substep(`${file.name}: ${file.size}`)
             }
-            logger.step(`Total bundle size: ${analysis['totalSize']}`)
+            logger.step(`Total bundle size: ${analysis.totalSize}`)
           }
         }
 
@@ -341,7 +376,7 @@ async function main() {
         }
       }
 
-      exitCode = srcResult['exitCode'] !== 0 ? srcResult['exitCode'] : typesExitCode
+      exitCode = srcResult.exitCode !== 0 ? srcResult.exitCode : typesExitCode
     }
 
     // Print final status and footer
@@ -363,4 +398,4 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(console.error)