refactor(scripts): organize validation scripts into subdirectory

Move all validate-* scripts to scripts/validation/ and remove the
validate- prefix for cleaner organization.

Changes:
- Created scripts/validation/ directory
- Moved and renamed 7 validation scripts:
  - validate-bundle-deps.mjs → validation/bundle-deps.mjs
  - validate-esbuild-minify.mjs → validation/esbuild-minify.mjs
  - validate-file-count.mjs → validation/file-count.mjs
  - validate-file-size.mjs → validation/file-size.mjs
  - validate-markdown-filenames.mjs → validation/markdown-filenames.mjs
  - validate-no-cdn-refs.mjs → validation/no-cdn-refs.mjs
  - validate-no-link-deps.mjs → validation/no-link-deps.mjs
- Updated scripts/check.mjs to reference new paths
- Updated rootPath in all validation scripts (../..)
- Updated self-reference in no-cdn-refs.mjs

Author: jdalton

scripts/bump.mjs

@@ -40,31 +40,31 @@ async function main() {
         command: 'pnpm',
       },
       {
-        args: ['scripts/validate-no-link-deps.mjs'],
+        args: ['scripts/validation/no-link-deps.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-bundle-deps.mjs'],
+        args: ['scripts/validation/bundle-deps.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-esbuild-minify.mjs'],
+        args: ['scripts/validation/esbuild-minify.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-no-cdn-refs.mjs'],
+        args: ['scripts/validation/no-cdn-refs.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-markdown-filenames.mjs'],
+        args: ['scripts/validation/markdown-filenames.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-file-size.mjs'],
+        args: ['scripts/validation/file-size.mjs'],
         command: 'node',
       },
       {
-        args: ['scripts/validate-file-count.mjs'],
+        args: ['scripts/validation/file-count.mjs'],
         command: 'node',
       },
     ]

scripts/check.mjs

@@ -10,59 +10,55 @@
  *   node scripts/generate-sdk.mjs
  */
 
-import { spawn } from 'node:child_process'
 import { readFileSync, writeFileSync } from 'node:fs'
 import { resolve } from 'node:path'
 
 import * as parser from '@babel/parser'
 import traverse from '@babel/traverse'
 import * as t from '@babel/types'
 import MagicString from 'magic-string'
+import openapiTS from 'openapi-typescript'
 
 import { getDefaultLogger } from '@socketsecurity/lib/logger'
 
 import { getRootPath } from './utils/path-helpers.mjs'
 import { runCommand } from './utils/run-command.mjs'
 
 const rootPath = getRootPath(import.meta.url)
+const openApiJsonPath = resolve(rootPath, 'openapi.json')
 const typesPath = resolve(rootPath, 'types/api.d.ts')
 
 // Initialize logger
 const logger = getDefaultLogger()
 
-async function generateTypes() {
-  return new Promise((resolve, reject) => {
-    const child = spawn('node', ['scripts/generate-types.mjs'], {
-      cwd: rootPath,
-      stdio: ['inherit', 'pipe', 'inherit'],
-    })
-
-    let output = ''
-
-    child.stdout.on('data', data => {
-      output += data.toString()
-    })
-
-    child.on('exit', code => {
-      if (code !== 0) {
-        reject(new Error(`Type generation failed with exit code ${code}`))
-        return
-      }
+/**
+ * Prettifies the OpenAPI JSON file.
+ */
+async function prettifyOpenApiJson() {
+  const openApiData = readFileSync(openApiJsonPath, 'utf8')
+  writeFileSync(
+    openApiJsonPath,
+    JSON.stringify(JSON.parse(openApiData), null, 2),
+  )
+}
 
-      try {
-        writeFileSync(typesPath, output, 'utf8')
-        // Fix array syntax after writing to disk
-        fixArraySyntax(typesPath)
-        // Add SDK v3 method name aliases
-        addSdkMethodAliases(typesPath)
-        resolve()
-      } catch (error) {
-        reject(error)
+/**
+ * Generates TypeScript types from OpenAPI schema.
+ */
+async function generateTypes() {
+  const output = await openapiTS(openApiJsonPath, {
+    transform(schemaObject) {
+      if ('format' in schemaObject && schemaObject.format === 'binary') {
+        return 'never'
       }
-    })
-
-    child.on('error', reject)
+    },
   })
+
+  writeFileSync(typesPath, output, 'utf8')
+  // Fix array syntax after writing to disk
+  fixArraySyntax(typesPath)
+  // Add SDK v3 method name aliases
+  addSdkMethodAliases(typesPath)
 }
 
 /**
@@ -213,19 +209,15 @@ async function main() {
 
     // Step 1: Prettify OpenAPI JSON
     logger.log('  1. Prettifying OpenAPI JSON...')
-    let exitCode = await runCommand('node', ['scripts/prettify-base-json.mjs'])
-    if (exitCode !== 0) {
-      process.exitCode = exitCode
-      return
-    }
+    await prettifyOpenApiJson()
 
     // Step 2: Generate types
     logger.log('  2. Generating TypeScript types...')
     await generateTypes()
 
     // Step 3: Format generated files
     logger.log('  3. Formatting generated files...')
-    exitCode = await runCommand('pnpm', [
+    let exitCode = await runCommand('pnpm', [
       'exec',
       'biome',
       'format',

scripts/generate-sdk.mjs

@@ -19,7 +19,7 @@ import { getDefaultLogger } from '@socketsecurity/lib/logger'
 const logger = getDefaultLogger()
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const rootPath = path.join(__dirname, '..')
+const rootPath = path.join(__dirname, '..', '..')
 
 // Node.js builtins to ignore (including node: prefix variants)
 const BUILTIN_MODULES = new Set([
@@ -371,12 +371,12 @@ async function main() {
     }
 
     if (violations.length > 0) {
-      logger.error('❌ Bundle dependencies validation failed\n')
+      logger.fail('Bundle dependencies validation failed\n')
 
       for (const violation of violations) {
-        logger.error(`  ${violation.message}`)
-        logger.error(`  ${violation.fix}`)
-        logger.error('')
+        logger.log(`  ${violation.message}`)
+        logger.log(`  ${violation.fix}`)
+        logger.log('')
       }
     }
 
@@ -392,7 +392,7 @@ async function main() {
     // Only fail on violations, not warnings
     process.exitCode = violations.length > 0 ? 1 : 0
   } catch (error) {
-    logger.error('Validation failed:', error.message)
+    logger.fail('Validation failed:', error.message)
     process.exitCode = 1
   }
 }

scripts/generate-types.mjs

@@ -12,7 +12,7 @@ import { getDefaultLogger } from '@socketsecurity/lib/logger'
 const logger = getDefaultLogger()
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const rootPath = path.join(__dirname, '..')
+const rootPath = path.join(__dirname, '..', '..')
 
 /**
  * Validate esbuild configuration has minify: false.
@@ -52,7 +52,7 @@ async function validateEsbuildMinify() {
 
     return violations
   } catch (error) {
-    logger.error(`Failed to load esbuild config: ${error.message}`)
+    logger.fail(`Failed to load esbuild config: ${error.message}`)
     process.exitCode = 1
     return []
   }
@@ -67,25 +67,23 @@ async function main() {
     return
   }
 
-  logger.error('❌ esbuild minify validation failed\n')
+  logger.fail('esbuild minify validation failed\n')
 
   for (const violation of violations) {
-    logger.error(`  ${violation.message}`)
-    logger.error(`  Found: minify: ${violation.value}`)
-    logger.error('  Expected: minify: false')
-    logger.error(`  Location: ${violation.location}`)
-    logger.error('')
+    logger.log(`  ${violation.message}`)
+    logger.log(`  Found: minify: ${violation.value}`)
+    logger.log('  Expected: minify: false')
+    logger.log(`  Location: ${violation.location}`)
+    logger.log('')
   }
 
-  logger.error(
-    'Minification breaks ESM/CJS interop and makes debugging harder.',
-  )
-  logger.error('')
+  logger.log('Minification breaks ESM/CJS interop and makes debugging harder.')
+  logger.log('')
 
   process.exitCode = 1
 }
 
 main().catch(error => {
-  logger.error('Validation failed:', error)
+  logger.fail('Validation failed:', error)
   process.exitCode = 1
 })

scripts/prettify-base-json.mjs

@@ -19,7 +19,7 @@ const logger = getDefaultLogger()
 const execAsync = promisify(exec)
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const rootPath = path.join(__dirname, '..')
+const rootPath = path.join(__dirname, '..', '..')
 
 // Maximum number of files in a single commit
 const MAX_FILES_PER_COMMIT = 50

scripts/validate-bundle-deps.mjs scripts/validation/bundle-deps.mjsscripts/validate-bundle-deps.mjs renamed to scripts/validation/bundle-deps.mjs

@@ -17,7 +17,7 @@ import { getDefaultLogger } from '@socketsecurity/lib/logger'
 const logger = getDefaultLogger()
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const rootPath = path.join(__dirname, '..')
+const rootPath = path.join(__dirname, '..', '..')
 
 // Maximum file size: 2MB (2,097,152 bytes)
 const MAX_FILE_SIZE = 2 * 1024 * 1024

scripts/validate-esbuild-minify.mjs scripts/validation/esbuild-minify.mjsscripts/validate-esbuild-minify.mjs renamed to scripts/validation/esbuild-minify.mjs

@@ -26,7 +26,7 @@ import { getDefaultLogger } from '@socketsecurity/lib/logger'
 const logger = getDefaultLogger()
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url))
-const rootPath = path.join(__dirname, '..')
+const rootPath = path.join(__dirname, '..', '..')
 
 // Allowed SCREAMING_CASE markdown files (without .md extension for comparison)
 const ALLOWED_SCREAMING_CASE = new Set([