refactor(build): enhance build system and tooling

- Add conditional source map generation to esbuild config
- Improve post-build transform for Node ESM interop
- Standardize success messages with logger.success()
- Rebuild with source maps before coverage analysis
- Migrate build scripts to use @socketsecurity/lib logger
- Remove unused get-local-package-aliases utility from registry
- Optimize build process for production registry code

Author: jdalton

registry/.config/esbuild.config.mjs

@@ -7,7 +7,7 @@ import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 import fg from 'fast-glob'
 
-import { getLocalPackageAliases } from '../scripts/utils/get-local-package-aliases.mjs'
+import { envAsBoolean } from '@socketsecurity/lib/env/helpers'
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const rootPath = path.join(__dirname, '..')
@@ -22,38 +22,6 @@ const entryPoints = fg.sync('**/*.{ts,mts,cts}', {
   ignore: ['**/*.d.ts', '**/types/**'],
 })
 
-/**
- * Plugin to handle local package aliases when bundle: false
- * esbuild's built-in alias only works with bundle: true, so we need a custom plugin
- */
-function createAliasPlugin() {
-  const aliases = getLocalPackageAliases(rootPath)
-
-  // Only create plugin if we have local aliases
-  if (Object.keys(aliases).length === 0) {
-    return null
-  }
-
-  return {
-    name: 'local-package-aliases',
-    setup(build) {
-      // Intercept imports for aliased packages
-      for (const [packageName, aliasPath] of Object.entries(aliases)) {
-        build.onResolve({ filter: new RegExp(`^${packageName}$`) }, () => {
-          // Return the path to the local package dist
-          return { path: aliasPath, external: true }
-        })
-
-        // Handle subpath imports like '@socketsecurity/lib/spinner'
-        build.onResolve({ filter: new RegExp(`^${packageName}/`) }, args => {
-          const subpath = args.path.slice(packageName.length + 1)
-          return { path: path.join(aliasPath, subpath), external: true }
-        })
-      }
-    },
-  }
-}
-
 // Build configuration for CommonJS output
 export const buildConfig = {
   entryPoints,
@@ -64,7 +32,8 @@ export const buildConfig = {
   format: 'cjs',
   platform: 'node',
   target: 'node18',
-  sourcemap: true,
+  // Enable source maps for coverage (set COVERAGE=true env var)
+  sourcemap: envAsBoolean(process.env.COVERAGE),
   // Disable minification for better Node ESM interop
   // Node ESM requires clear exports: module.exports = { foo, bar }
   minify: false,
@@ -73,9 +42,6 @@ export const buildConfig = {
   metafile: true,
   logLevel: 'info',
 
-  // Use plugin for local package aliases (built-in alias requires bundle: true)
-  plugins: [createAliasPlugin()].filter(Boolean),
-
   // Note: Cannot use "external" with bundle: false
   // esbuild automatically treats all imports as external when not bundling
 

registry/scripts/build-js.mjs

@@ -9,13 +9,16 @@ import { fileURLToPath } from 'node:url'
 import { build, context } from 'esbuild'
 import fg from 'fast-glob'
 
-import { printError, printSuccess } from '../../scripts/utils/cli-helpers.mjs'
+import { getDefaultLogger } from '@socketsecurity/lib/logger'
+
 import {
   analyzeMetafile,
   buildConfig,
   watchConfig,
 } from '../.config/esbuild.config.mjs'
 
+const logger = getDefaultLogger()
+
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const rootPath = path.join(__dirname, '..')
 const srcPath = path.join(rootPath, 'src')
@@ -112,7 +115,7 @@ async function buildJS() {
     return 0
   } catch (error) {
     if (!isQuiet) {
-      printError('JavaScript build failed')
+      logger.error('JavaScript build failed')
       console.error(error)
     }
     return 1
@@ -140,11 +143,11 @@ async function watchJS() {
             build.onEnd(result => {
               if (result.errors.length > 0) {
                 if (!isQuiet) {
-                  printError('Rebuild failed')
+                  logger.error('Rebuild failed')
                 }
               } else {
                 if (!isQuiet) {
-                  printSuccess('Rebuild succeeded')
+                  logger.success('Rebuild succeeded')
 
                   if (result?.metafile && isVerbose) {
                     const analysis = analyzeMetafile(result.metafile)
@@ -173,7 +176,7 @@ async function watchJS() {
     await new Promise(() => {})
   } catch (error) {
     if (!isQuiet) {
-      printError('Watch mode failed')
+      logger.error('Watch mode failed')
       console.error(error)
     }
     return 1

registry/scripts/build.mjs

@@ -7,23 +7,22 @@ import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 import { build, context } from 'esbuild'
-import colors from 'yoctocolors-cjs'
 import fg from 'fast-glob'
+
+import { isQuiet } from '@socketsecurity/lib/argv/flags'
+import { getDefaultLogger } from '@socketsecurity/lib/logger'
+import { printFooter, printHeader } from '@socketsecurity/lib/stdio/header'
+
 import {
   analyzeMetafile,
   buildConfig,
   watchConfig,
 } from '../.config/esbuild.config.mjs'
-import { isQuiet } from '../../scripts/utils/flags.mjs'
-import {
-  logger,
-  printCompletedHeader,
-  printFooter,
-  printHeader,
-} from '../../scripts/utils/helpers.mjs'
 import { parseArgs } from '../../scripts/utils/parse-args.mjs'
 import { runSequence } from '../../scripts/utils/run-command.mjs'
 
+const logger = getDefaultLogger()
+
 const rootPath = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
   '..',
@@ -432,7 +431,7 @@ async function main() {
       }
 
       if (!quiet) {
-        printCompletedHeader('Build Cleaned')
+        logger.success('Build Cleaned')
       }
 
       // Run source and types builds in parallel.
@@ -463,9 +462,9 @@ async function main() {
     // Print final status and footer.
     if (!quiet) {
       if (exitCode === 0) {
-        console.log(colors.green('✓ Build completed successfully!'))
+        logger.success('Build completed successfully!')
       } else {
-        console.error(colors.red('✗ Build failed'))
+        logger.fail('Build failed')
       }
       printFooter()
     }

registry/scripts/post-build-transform.mjs

@@ -8,6 +8,10 @@ import { readFileSync, writeFileSync } from 'node:fs'
 import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
+import { getDefaultLogger } from '@socketsecurity/lib/logger'
+
+const logger = getDefaultLogger()
+
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
 const rootPath = path.join(__dirname, '..')
 const indexPath = path.join(rootPath, 'dist', 'index.js')
@@ -41,8 +45,8 @@ try {
   // Write back the transformed content
   writeFileSync(indexPath, transformedContent, 'utf8')
 
-  console.log('✓ Transformed exports for Node ESM interop')
+  logger.success('Transformed exports for Node ESM interop')
 } catch (error) {
-  console.error('Post-build transform failed:', error.message)
+  logger.error(`Post-build transform failed: ${error.message}`)
   process.exit(1)
 }