refactor(scripts): use logger instead of console throughout scripts/

Replace all console.log/console.error/console.warn calls with logger methods:

**Build Scripts** (6 files):
- build-externals/: Added logger, replaced console calls
- build/: Removed colors import, replaced console with logger methods

**Fix Scripts** (4 files):
- Removed colors imports where only used for console
- Replaced printCompletedHeader with logger.success()
- Consistent logger.log/logger.error usage

**Test Scripts** (2 files):
- Replaced all console.log with logger.log
- Updated error handlers to use logger.error

**Validate Scripts** (3 files):
- Added logger imports where missing
- Replaced console with logger.success/logger.error/logger.log
- Use logger.info() for informational messages (ℹ)

**Other** (2 files):
- lint.mjs: Full logger migration
- claude.mjs: Added logger, updated inline log.info to use logger.info()

This ensures consistent output formatting and color usage across all
scripts in the repository. Updated 17 files total.

Author: jdalton

scripts/build-externals/bundler.mjs

@@ -7,6 +7,8 @@ import { createRequire } from 'node:module'
 import path from 'node:path'
 
 import esbuild from 'esbuild'
+import { getDefaultLogger } from '#socketsecurity/lib/logger'
+
 import {
   getEsbuildConfig,
   getPackageSpecificOptions,
@@ -17,6 +19,7 @@ import {
 } from './local-packages.mjs'
 
 const require = createRequire(import.meta.url)
+const logger = getDefaultLogger()
 
 /**
  * Bundle a single package with esbuild.
@@ -32,7 +35,7 @@ export async function bundlePackage(packageName, outputPath, options = {}) {
   const { quiet = false, rootDir } = options
 
   if (!quiet) {
-    console.log(`  Bundling ${packageName}...`)
+    logger.log(`  Bundling ${packageName}...`)
   }
 
   try {
@@ -51,7 +54,7 @@ export async function bundlePackage(packageName, outputPath, options = {}) {
       await fs.access(srcExternalPath)
       packagePath = srcExternalPath
       if (!quiet) {
-        console.log(
+        logger.log(
           `  Using entry point ${path.relative(rootDir, srcExternalPath)}`,
         )
       }
@@ -60,7 +63,7 @@ export async function bundlePackage(packageName, outputPath, options = {}) {
       const localPath = await getLocalPackagePath(packageName, rootDir)
       if (localPath) {
         if (!quiet) {
-          console.log(
+          logger.log(
             `  Using local version from ${path.relative(rootDir, localPath)}`,
           )
         }
@@ -103,12 +106,12 @@ ${contentWithoutStrict}`
     const stats = await fs.stat(outputPath)
     const sizeKB = Math.round(stats.size / 1024)
     if (!quiet) {
-      console.log(`    ✓ Bundled ${packageName} (${sizeKB}KB)`)
+      logger.log(`    ✓ Bundled ${packageName} (${sizeKB}KB)`)
     }
     return sizeKB
   } catch (error) {
     if (!quiet) {
-      console.error(`    ✗ Failed to bundle ${packageName}:`, error.message)
+      logger.log(`    ✗ Failed to bundle ${packageName}: ${error.message}`)
     }
     // Create error stub.
     const stubContent = `'use strict'

scripts/build-externals/copy-files.mjs

@@ -5,6 +5,10 @@
 import { promises as fs } from 'node:fs'
 import path from 'node:path'
 
+import { getDefaultLogger } from '#socketsecurity/lib/logger'
+
+const logger = getDefaultLogger()
+
 /**
  * Ensure directory exists.
  *
@@ -37,7 +41,7 @@ export async function copyLocalFiles(srcDir, destDir, quiet = false) {
 
       await fs.copyFile(srcPath, destPath)
       if (!quiet) {
-        console.log(`  Copied ${file}`)
+        logger.log(`  Copied ${file}`)
       }
       count++
     }
@@ -82,7 +86,7 @@ export async function copyRecursive(
         // File doesn't exist, copy it.
         await fs.copyFile(srcEntry, destEntry)
         if (!quiet) {
-          console.log(`  Copied ${relPath}`)
+          logger.log(`  Copied ${relPath}`)
         }
         count++
       }

scripts/build/clean.mjs

@@ -60,7 +60,7 @@ async function cleanDirectories(tasks, options = {}) {
     } catch (error) {
       if (!quiet) {
         logger.error(`Failed to clean ${name}`)
-        console.error(error.message)
+        logger.error(error.message)
       }
       return 1
     }
@@ -211,4 +211,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(error => {
+  logger.error(error.message || error)
+  process.exitCode = 1
+})

scripts/build/externals.mjs

@@ -5,16 +5,13 @@
  * Entry point that wraps the modular build-externals system.
  */
 
-import colors from 'yoctocolors-cjs'
-
 import { isQuiet } from '#socketsecurity/lib/argv/flags'
 import { getDefaultLogger } from '#socketsecurity/lib/logger'
 import { pluralize } from '#socketsecurity/lib/words'
 
 import { buildExternals } from '../build-externals/orchestrator.mjs'
 
 const logger = getDefaultLogger()
-const printCompletedHeader = title => console.log(colors.green(`✓ ${title}`))
 
 async function main() {
   // Check for verbose mode via isVerbose or manual check
@@ -29,7 +26,7 @@ async function main() {
         bundledCount > 0
           ? `External Bundles (${bundledCount} ${pluralize('package', { count: bundledCount })})`
           : 'External Bundles (no packages)'
-      printCompletedHeader(title)
+      logger.success(title)
     }
   } catch (error) {
     logger.error(`Build failed: ${error.message || error}`)

scripts/build/js.mjs

@@ -24,7 +24,7 @@ const isWatch = process.argv.includes('--watch')
 async function buildJS() {
   try {
     if (!isQuiet) {
-      console.log('→ Building JavaScript with esbuild')
+      logger.step('Building JavaScript with esbuild')
     }
 
     const startTime = Date.now()
@@ -36,19 +36,19 @@ async function buildJS() {
     const buildTime = Date.now() - startTime
 
     if (!isQuiet) {
-      console.log(`  JavaScript built in ${buildTime}ms`)
+      logger.log(`  JavaScript built in ${buildTime}ms`)
 
       if (result?.metafile && isVerbose) {
         const analysis = analyzeMetafile(result.metafile)
-        console.log(`  Total size: ${analysis.totalSize}`)
+        logger.log(`  Total size: ${analysis.totalSize}`)
       }
     }
 
     return 0
   } catch (error) {
     if (!isQuiet) {
       logger.error('JavaScript build failed')
-      console.error(error)
+      logger.error(error)
     }
     return 1
   }
@@ -60,8 +60,8 @@ async function buildJS() {
 async function watchJS() {
   try {
     if (!isQuiet) {
-      console.log('→ Starting watch mode with incremental builds')
-      console.log('  Watching for file changes...')
+      logger.step('Starting watch mode with incremental builds')
+      logger.log('  Watching for file changes...')
     }
 
     const ctx = await context({
@@ -83,7 +83,7 @@ async function watchJS() {
 
                   if (result?.metafile && isVerbose) {
                     const analysis = analyzeMetafile(result.metafile)
-                    console.log(`  Total size: ${analysis.totalSize}`)
+                    logger.log(`  Total size: ${analysis.totalSize}`)
                   }
                 }
               }
@@ -98,7 +98,7 @@ async function watchJS() {
     // Keep process alive
     process.on('SIGINT', async () => {
       if (!isQuiet) {
-        console.log('\nStopping watch mode...')
+        logger.log('\nStopping watch mode...')
       }
       await ctx.dispose()
       process.exit(0)
@@ -109,7 +109,7 @@ async function watchJS() {
   } catch (error) {
     if (!isQuiet) {
       logger.error('Watch mode failed')
-      console.error(error)
+      logger.error(error)
     }
     return 1
   }
@@ -118,7 +118,7 @@ async function watchJS() {
 // Main
 if (isWatch) {
   watchJS().catch(error => {
-    console.error(error)
+    logger.error(error)
     process.exit(1)
   })
 } else {
@@ -127,7 +127,7 @@ if (isWatch) {
       process.exitCode = code
     })
     .catch(error => {
-      console.error(error)
+      logger.error(error)
       process.exitCode = 1
     })
 }

scripts/build/main.mjs

@@ -7,7 +7,6 @@ import path from 'node:path'
 import { fileURLToPath } from 'node:url'
 
 import { build, context } from 'esbuild'
-import colors from 'yoctocolors-cjs'
 
 import {
   analyzeMetafile,
@@ -23,9 +22,6 @@ import { runSequence } from '../utils/run-command.mjs'
 
 const logger = getDefaultLogger()
 
-// Helper for completed headers (simple wrapper)
-const printCompletedHeader = title => console.log(colors.green(`✓ ${title}`))
-
 const rootPath = path.resolve(
   path.dirname(fileURLToPath(import.meta.url)),
   '..',
@@ -69,7 +65,7 @@ async function buildSource(options = {}) {
   } catch (error) {
     if (!quiet) {
       logger.error('Source build failed')
-      console.error(error)
+      logger.error(error)
     }
     return { exitCode: 1, buildTime: 0, result: null }
   }
@@ -300,28 +296,28 @@ async function main() {
 
     // Show help if requested
     if (values.help) {
-      console.log('Build Runner')
-      console.log('\nUsage: pnpm build [options]')
-      console.log('\nOptions:')
-      console.log('  --help       Show this help message')
-      console.log('  --src        Build source code only')
-      console.log('  --types      Build TypeScript declarations only')
-      console.log(
+      logger.log('Build Runner')
+      logger.log('\nUsage: pnpm build [options]')
+      logger.log('\nOptions:')
+      logger.log('  --help       Show this help message')
+      logger.log('  --src        Build source code only')
+      logger.log('  --types      Build TypeScript declarations only')
+      logger.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')
-      console.log('  --verbose    Show detailed build output')
-      console.log('\nExamples:')
-      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(
+      logger.log('  --needed     Only build if dist files are missing')
+      logger.log('  --analyze    Show bundle size analysis')
+      logger.log('  --quiet, --silent  Suppress progress messages')
+      logger.log('  --verbose    Show detailed build output')
+      logger.log('\nExamples:')
+      logger.log('  pnpm build              # Full build (source + types)')
+      logger.log('  pnpm build --src        # Build source only')
+      logger.log('  pnpm build --types      # Build types only')
+      logger.log(
         '  pnpm build --watch      # Watch mode with incremental builds',
       )
-      console.log('  pnpm build --analyze    # Build with size analysis')
-      console.log(
+      logger.log('  pnpm build --analyze    # Build with size analysis')
+      logger.log(
         '\nNote: Watch mode uses esbuild context API for 68% faster rebuilds',
       )
       process.exitCode = 0
@@ -404,7 +400,7 @@ async function main() {
       }
 
       if (!quiet) {
-        printCompletedHeader('Build Cleaned')
+        logger.success('Build Cleaned')
       }
 
       // Run source, externals, and types builds in parallel
@@ -447,9 +443,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.error('Build failed')
       }
       printFooter()
     }
@@ -463,4 +459,7 @@ async function main() {
   }
 }
 
-main().catch(console.error)
+main().catch(error => {
+  logger.error(error.message || error)
+  process.exitCode = 1
+})