refactor: update utility modules

- Update interactive-expand and command-builder
- Clean up imports and remove unused code
- Update error filtering and offline cache

Author: jdalton

src/utils/coana-spawn.mts

@@ -58,41 +58,42 @@ export async function spawnCoana(
     mixinsEnv['SOCKET_CLI_API_PROXY'] = proxyUrl
   }
 
+  // Execute Coana
   try {
     const localCoanaPath = constants.ENV['SOCKET_CLI_COANA_LOCAL_PATH']
-    // Use local Coana CLI if path is provided.
-    if (localCoanaPath) {
+      // Use local Coana CLI if path is provided.
+      if (localCoanaPath) {
+        const finalEnv = {
+          ...process.env,
+          ...constants.processEnv,
+          ...mixinsEnv,
+          ...spawnEnv,
+        }
+        const spawnResult = await spawn('node', [localCoanaPath, ...args], {
+          cwd: shadowOptions.cwd,
+          env: finalEnv,
+          stdio: spawnExtra?.['stdio'] || 'inherit',
+        })
+
+        return {
+          ok: true,
+          data: spawnResult.stdout ? spawnResult.stdout.toString() : '',
+        }
+      }
+
+      // Use npx to run coana.
+      const coanaVersion =
+        constants.ENV['INLINED_SOCKET_CLI_COANA_TECH_CLI_VERSION']
+      const packageSpec = `@coana-tech/cli@~${coanaVersion}`
+
       const finalEnv = {
         ...process.env,
         ...constants.processEnv,
         ...mixinsEnv,
         ...spawnEnv,
       }
-      const spawnResult = await spawn('node', [localCoanaPath, ...args], {
-        cwd: shadowOptions.cwd,
-        env: finalEnv,
-        stdio: spawnExtra?.['stdio'] || 'inherit',
-      })
-
-      return {
-        ok: true,
-        data: spawnResult.stdout ? spawnResult.stdout.toString() : '',
-      }
-    }
-
-    // Use npx to run coana.
-    const coanaVersion =
-      constants.ENV['INLINED_SOCKET_CLI_COANA_TECH_CLI_VERSION']
-    const packageSpec = `@coana-tech/cli@~${coanaVersion}`
-
-    const finalEnv = {
-      ...process.env,
-      ...constants.processEnv,
-      ...mixinsEnv,
-      ...spawnEnv,
-    }
 
-    const result = await runShadowCommand(packageSpec, args, {
+      const result = await runShadowCommand(packageSpec, args, {
       cwd:
         typeof shadowOptions.cwd === 'string'
           ? shadowOptions.cwd

src/utils/command-builder.mts

@@ -124,12 +124,13 @@ export interface ParentCommandOptions {
 }
 
 export function buildParentCommand(options: ParentCommandOptions): CliSubcommand {
-  const { defaultSubcommand, description, hidden = false, name, subcommands } = options
+  const { description, hidden = false, name, subcommands } = options
+  // defaultSubcommand is intentionally unused - kept for API compatibility
 
   return {
     description,
     hidden,
-    async run(argv: readonly string[], importMeta: ImportMeta, { parentName }: { parentName: string }) {
+    async run(_argv: readonly string[], _importMeta: ImportMeta, _options: { parentName: string }) {
       // This is typically handled by meowWithSubcommands
       // but we can provide a fallback
       logger.log(`Available subcommands for ${name}:`)

src/utils/error-filter.mts

@@ -28,22 +28,28 @@ const NODE_ERROR_PATTERNS = [
   /^TypeError: .* is not a function$/,
 
   // Stack trace lines
-  /^\s*at\s+.*\(/,  // "    at functionName (/path/to/file.js:123:45)"
-  /^\s*at\s+\/.*:\d+:\d+$/,  // "    at /path/to/file.js:123:45"
-  /^\s*at\s+node:/,  // Node internal modules
-  /^\s*at\s+async\s+/,  // Async stack traces
+  // "    at functionName (/path/to/file.js:123:45)"
+  /^\s*at\s+.*\(/,
+  // "    at /path/to/file.js:123:45"
+  /^\s*at\s+\/.*:\d+:\d+$/,
+  // Node internal modules
+  /^\s*at\s+node:/,
+  // Async stack traces
+  /^\s*at\s+async\s+/,
 
   // Node.js internal paths
   /node_modules\//,
   /internal\/modules\//,
   /internal\/process\//,
 
   // V8 error decoration
-  /^\s*\^+$/,  // Error position indicators like "^^^^^"
+  // Error position indicators like "^^^^^"
+  /^\s*\^+$/,
   /^SyntaxError:/,
 
   // Node warnings
-  /^\(node:\d+\)/,  // "(node:12345) Warning: ..."
+  // "(node:12345) Warning: ..."
+  /^\(node:\d+\)/,
   /^DeprecationWarning:/,
   /^ExperimentalWarning:/,
 
@@ -72,8 +78,9 @@ function cleanErrorMessage(line: string): string | null {
   }
 
   // Clean up common Node.js error prefixes
+  // Remove "Error: " prefix
   line = line
-    .replace(/^Error:\s+/, '')  // Remove "Error: " prefix
+    .replace(/^Error:\s+/, '')
     .replace(/^TypeError:\s+/, '')
     .replace(/^ReferenceError:\s+/, '')
     .replace(/^RangeError:\s+/, '')

src/utils/interactive-expand.mts

@@ -265,8 +265,8 @@ export function formatExpandableList(
     expanded?: boolean
   } = {}
 ): string {
-  const { expanded = false, maxItemsCollapsed = 3 } = options
   const theme = getTheme()
+  const { expanded = false, maxItemsCollapsed = 3 } = options
 
   if (items.length <= maxItemsCollapsed || expanded) {
     const list = items.map(item => `  • ${item}`).join('\n')

src/utils/offline-cache.mts

@@ -6,8 +6,6 @@ import { mkdir, readFile, writeFile } from 'node:fs/promises'
 import { homedir } from 'node:os'
 import { join } from 'node:path'
 
-import colors from 'yoctocolors-cjs'
-
 import { logger } from '@socketsecurity/registry/lib/logger'
 
 interface CacheEntry<T = any> {
@@ -19,14 +17,17 @@ interface CacheEntry<T = any> {
 }
 
 interface CacheOptions {
-  ttl?: number  // Time to live in milliseconds
-  offline?: boolean  // Force offline mode
-  refresh?: boolean  // Force refresh even if cached
-  namespace?: string  // Cache namespace for organization
+  // Time to live in milliseconds
+  ttl?: number
+  // Force offline mode
+  offline?: boolean
+  // Force refresh even if cached
+  refresh?: boolean
+  // Cache namespace for organization
+  namespace?: string
 }
 
 const CACHE_DIR = join(homedir(), '.socket', '_cacache')
-const CACHE_METADATA_FILE = 'cache-metadata.json'
 
 /**
  * Get cache key for a given operation
@@ -65,7 +66,8 @@ async function readFromCache<T>(
     // Check if expired
     const now = Date.now()
     if (entry.ttl > 0 && now - entry.timestamp > entry.ttl) {
-      return null  // Expired
+      // Expired
+      return null
     }
 
     return entry
@@ -88,8 +90,9 @@ async function writeToCache<T>(
     const entry: CacheEntry<T> = {
       data,
       timestamp: Date.now(),
-      ttl: options.ttl || 3600000, // Default 1 hour
-      metadata: options.namespace ? { namespace: options.namespace } : undefined,
+      // Default 1 hour
+      ttl: options.ttl || 3600000,
+      ...(options.namespace ? { metadata: { namespace: options.namespace } } : {}),
     }
 
     await writeFile(path, JSON.stringify(entry, null, 2))
@@ -209,12 +212,16 @@ export async function getCacheStats(): Promise<{
   oldest?: Date
   newest?: Date
 }> {
-  const stats = {
+  const stats: {
+    size: number
+    entries: number
+    namespaces: string[]
+    oldest?: Date
+    newest?: Date
+  } = {
     size: 0,
     entries: 0,
-    namespaces: [] as string[],
-    oldest: undefined as Date | undefined,
-    newest: undefined as Date | undefined,
+    namespaces: [],
   }
 
   if (!existsSync(CACHE_DIR)) {
@@ -232,12 +239,14 @@ export async function getCacheStats(): Promise<{
 
     for (const ns of namespaces) {
       const nsPath = join(CACHE_DIR, ns)
+      // eslint-disable-next-line no-await-in-loop
       const files = await readdir(nsPath)
 
       for (const file of files) {
         if (file.endsWith('.json')) {
           stats.entries++
           const filePath = join(nsPath, file)
+          // eslint-disable-next-line no-await-in-loop
           const fileStat = await stat(filePath)
           stats.size += fileStat.size
 

src/utils/project-context.mts

@@ -149,7 +149,7 @@ export async function getProjectContext(cwd: string = process.cwd()): Promise<Pr
     type: packageManager,
     root,
     hasLockFile,
-    framework,
+    ...(framework ? { framework } : {}),
     isMonorepo: monorepo,
   }
 }