spindb 0.9.1 → 0.9.3

package/cli/commands/menu/engine-handlers.ts

@@ -4,7 +4,7 @@ import { rm } from 'fs/promises'
 import { containerManager } from '../../../core/container-manager'
 import { processManager } from '../../../core/process-manager'
 import { createSpinner } from '../../ui/spinner'
-import { header, error, warning, info, formatBytes } from '../../ui/theme'
+import { header, uiError, uiWarning, uiInfo, formatBytes } from '../../ui/theme'
 import { promptConfirm } from '../../ui/prompts'
 import { getEngineIcon, ENGINE_ICONS } from '../../constants'
 import {
@@ -37,7 +37,7 @@ export async function handleEngines(): Promise<void> {
   const engines = await getInstalledEngines()
 
   if (engines.length === 0) {
-    console.log(info('No engines installed yet.'))
+    console.log(uiInfo('No engines installed yet.'))
     console.log(
       chalk.gray(
         '  PostgreSQL engines are downloaded automatically when you create a container.',
@@ -128,7 +128,9 @@ export async function handleEngines(): Promise<void> {
     console.log(chalk.gray(`  MySQL: system-installed at ${mysqlEngine.path}`))
   }
   if (sqliteEngine) {
-    console.log(chalk.gray(`  SQLite: system-installed at ${sqliteEngine.path}`))
+    console.log(
+      chalk.gray(`  SQLite: system-installed at ${sqliteEngine.path}`),
+    )
   }
   console.log()
 
@@ -212,7 +214,7 @@ async function handleDeleteEngine(
   if (usingContainers.length > 0) {
     console.log()
     console.log(
-      error(
+      uiError(
         `Cannot delete: ${usingContainers.length} container(s) are using ${engineName} ${engineVersion}`,
       ),
     )
@@ -238,7 +240,7 @@ async function handleDeleteEngine(
   )
 
   if (!confirmed) {
-    console.log(warning('Deletion cancelled'))
+    console.log(uiWarning('Deletion cancelled'))
     return
   }
 
@@ -248,8 +250,8 @@ async function handleDeleteEngine(
   try {
     await rm(enginePath, { recursive: true, force: true })
     spinner.succeed(`Deleted ${engineName} ${engineVersion}`)
-  } catch (err) {
-    const e = err as Error
+  } catch (error) {
+    const e = error as Error
     spinner.fail(`Failed to delete: ${e.message}`)
   }
 }
@@ -272,7 +274,7 @@ async function handleMysqlInfo(mysqldPath: string): Promise<void> {
 
   if (mysqlContainers.length > 0) {
     console.log(
-      warning(
+      uiWarning(
         `${mysqlContainers.length} container(s) are using ${displayName}:`,
       ),
     )
@@ -438,7 +440,9 @@ async function handleSqliteInfo(sqlitePath: string): Promise<void> {
   const sqliteContainers = containers.filter((c) => c.engine === 'sqlite')
 
   if (sqliteContainers.length > 0) {
-    console.log(info(`${sqliteContainers.length} SQLite database(s) registered:`))
+    console.log(
+      uiInfo(`${sqliteContainers.length} SQLite database(s) registered:`),
+    )
     console.log()
     for (const c of sqliteContainers) {
       const status =
@@ -471,9 +475,15 @@ async function handleSqliteInfo(sqlitePath: string): Promise<void> {
 
   console.log(chalk.white('  Notes:'))
   console.log(chalk.gray('  ' + '─'.repeat(50)))
-  console.log(chalk.gray('  • SQLite is typically pre-installed on macOS and most Linux distributions'))
+  console.log(
+    chalk.gray(
+      '  • SQLite is typically pre-installed on macOS and most Linux distributions',
+    ),
+  )
   console.log(chalk.gray('  • No server process - databases are just files'))
-  console.log(chalk.gray('  • Use "spindb delete <name>" to unregister a database'))
+  console.log(
+    chalk.gray('  • Use "spindb delete <name>" to unregister a database'),
+  )
   console.log()
 
   await inquirer.prompt([

package/cli/commands/menu/index.ts

@@ -3,7 +3,7 @@ import chalk from 'chalk'
 import inquirer from 'inquirer'
 import { containerManager } from '../../../core/container-manager'
 import { promptInstallDependencies } from '../../ui/prompts'
-import { header, error } from '../../ui/theme'
+import { header, uiError } from '../../ui/theme'
 import { getInstalledEngines } from '../../helpers'
 import {
   handleCreate,
@@ -156,8 +156,8 @@ export const menuCommand = new Command('menu')
   .action(async () => {
     try {
       await showMainMenu()
-    } catch (err) {
-      const e = err as Error
+    } catch (error) {
+      const e = error as Error
 
       // Check if this is a missing tool error
       if (
@@ -177,7 +177,7 @@ export const menuCommand = new Command('menu')
         process.exit(1)
       }
 
-      console.error(error(e.message))
+      console.error(uiError(e.message))
       process.exit(1)
     }
   })

package/cli/commands/menu/shell-handlers.ts

@@ -20,7 +20,7 @@ import {
 import { platformService } from '../../../core/platform-service'
 import { getEngine } from '../../../engines'
 import { createSpinner } from '../../ui/spinner'
-import { error, warning, info, success } from '../../ui/theme'
+import { uiError, uiWarning, uiInfo, uiSuccess } from '../../ui/theme'
 import { pressEnterToContinue } from './shared'
 
 export async function handleCopyConnectionString(
@@ -28,7 +28,7 @@ export async function handleCopyConnectionString(
 ): Promise<void> {
   const config = await containerManager.getConfig(containerName)
   if (!config) {
-    console.error(error(`Container "${containerName}" not found`))
+    console.error(uiError(`Container "${containerName}" not found`))
     return
   }
 
@@ -39,10 +39,10 @@ export async function handleCopyConnectionString(
 
   console.log()
   if (copied) {
-    console.log(success('Connection string copied to clipboard'))
+    console.log(uiSuccess('Connection string copied to clipboard'))
     console.log(chalk.gray(`  ${connectionString}`))
   } else {
-    console.log(warning('Could not copy to clipboard. Connection string:'))
+    console.log(uiWarning('Could not copy to clipboard. Connection string:'))
     console.log(chalk.cyan(`  ${connectionString}`))
   }
   console.log()
@@ -59,7 +59,7 @@ export async function handleCopyConnectionString(
 export async function handleOpenShell(containerName: string): Promise<void> {
   const config = await containerManager.getConfig(containerName)
   if (!config) {
-    console.error(error(`Container "${containerName}" not found`))
+    console.error(uiError(`Container "${containerName}" not found`))
     return
   }
 
@@ -69,12 +69,13 @@ export async function handleOpenShell(containerName: string): Promise<void> {
   const shellCheckSpinner = createSpinner('Checking available shells...')
   shellCheckSpinner.start()
 
-  const [usqlInstalled, pgcliInstalled, mycliInstalled, litecliInstalled] = await Promise.all([
-    isUsqlInstalled(),
-    isPgcliInstalled(),
-    isMycliInstalled(),
-    isLitecliInstalled(),
-  ])
+  const [usqlInstalled, pgcliInstalled, mycliInstalled, litecliInstalled] =
+    await Promise.all([
+      isUsqlInstalled(),
+      isPgcliInstalled(),
+      isMycliInstalled(),
+      isLitecliInstalled(),
+    ])
 
   shellCheckSpinner.stop()
   // Clear the spinner line
@@ -119,7 +120,9 @@ export async function handleOpenShell(containerName: string): Promise<void> {
     engineSpecificInstallValue = 'install-pgcli'
   }
 
-  const choices: Array<{ name: string; value: ShellChoice } | inquirer.Separator> = [
+  const choices: Array<
+    { name: string; value: ShellChoice } | inquirer.Separator
+  > = [
     {
       name: `>_ Use default shell (${defaultShellName})`,
       value: 'default',
@@ -173,16 +176,16 @@ export async function handleOpenShell(containerName: string): Promise<void> {
 
   if (shellChoice === 'install-pgcli') {
     console.log()
-    console.log(info('Installing pgcli for enhanced PostgreSQL shell...'))
+    console.log(uiInfo('Installing pgcli for enhanced PostgreSQL shell...'))
     const pm = await detectPackageManager()
     if (pm) {
       const result = await installPgcli(pm)
       if (result.success) {
-        console.log(success('pgcli installed successfully!'))
+        console.log(uiSuccess('pgcli installed successfully!'))
         console.log()
         await launchShell(containerName, config, connectionString, 'pgcli')
       } else {
-        console.error(error(`Failed to install pgcli: ${result.error}`))
+        console.error(uiError(`Failed to install pgcli: ${result.error}`))
         console.log()
         console.log(chalk.gray('Manual installation:'))
         for (const instruction of getPgcliManualInstructions()) {
@@ -192,7 +195,7 @@ export async function handleOpenShell(containerName: string): Promise<void> {
         await pressEnterToContinue()
       }
     } else {
-      console.error(error('No supported package manager found'))
+      console.error(uiError('No supported package manager found'))
       console.log()
       console.log(chalk.gray('Manual installation:'))
       for (const instruction of getPgcliManualInstructions()) {
@@ -206,16 +209,16 @@ export async function handleOpenShell(containerName: string): Promise<void> {
 
   if (shellChoice === 'install-mycli') {
     console.log()
-    console.log(info('Installing mycli for enhanced MySQL shell...'))
+    console.log(uiInfo('Installing mycli for enhanced MySQL shell...'))
     const pm = await detectPackageManager()
     if (pm) {
       const result = await installMycli(pm)
       if (result.success) {
-        console.log(success('mycli installed successfully!'))
+        console.log(uiSuccess('mycli installed successfully!'))
         console.log()
         await launchShell(containerName, config, connectionString, 'mycli')
       } else {
-        console.error(error(`Failed to install mycli: ${result.error}`))
+        console.error(uiError(`Failed to install mycli: ${result.error}`))
         console.log()
         console.log(chalk.gray('Manual installation:'))
         for (const instruction of getMycliManualInstructions()) {
@@ -225,7 +228,7 @@ export async function handleOpenShell(containerName: string): Promise<void> {
         await pressEnterToContinue()
       }
     } else {
-      console.error(error('No supported package manager found'))
+      console.error(uiError('No supported package manager found'))
       console.log()
       console.log(chalk.gray('Manual installation:'))
       for (const instruction of getMycliManualInstructions()) {
@@ -239,16 +242,16 @@ export async function handleOpenShell(containerName: string): Promise<void> {
 
   if (shellChoice === 'install-usql') {
     console.log()
-    console.log(info('Installing usql for enhanced shell experience...'))
+    console.log(uiInfo('Installing usql for enhanced shell experience...'))
     const pm = await detectPackageManager()
     if (pm) {
       const result = await installUsql(pm)
       if (result.success) {
-        console.log(success('usql installed successfully!'))
+        console.log(uiSuccess('usql installed successfully!'))
         console.log()
         await launchShell(containerName, config, connectionString, 'usql')
       } else {
-        console.error(error(`Failed to install usql: ${result.error}`))
+        console.error(uiError(`Failed to install usql: ${result.error}`))
         console.log()
         console.log(chalk.gray('Manual installation:'))
         for (const instruction of getUsqlManualInstructions()) {
@@ -258,7 +261,7 @@ export async function handleOpenShell(containerName: string): Promise<void> {
         await pressEnterToContinue()
       }
     } else {
-      console.error(error('No supported package manager found'))
+      console.error(uiError('No supported package manager found'))
       console.log()
       console.log(chalk.gray('Manual installation:'))
       for (const instruction of getUsqlManualInstructions()) {
@@ -272,16 +275,16 @@ export async function handleOpenShell(containerName: string): Promise<void> {
 
   if (shellChoice === 'install-litecli') {
     console.log()
-    console.log(info('Installing litecli for enhanced SQLite shell...'))
+    console.log(uiInfo('Installing litecli for enhanced SQLite shell...'))
     const pm = await detectPackageManager()
     if (pm) {
       const result = await installLitecli(pm)
       if (result.success) {
-        console.log(success('litecli installed successfully!'))
+        console.log(uiSuccess('litecli installed successfully!'))
         console.log()
         await launchShell(containerName, config, connectionString, 'litecli')
       } else {
-        console.error(error(`Failed to install litecli: ${result.error}`))
+        console.error(uiError(`Failed to install litecli: ${result.error}`))
         console.log()
         console.log(chalk.gray('Manual installation:'))
         for (const instruction of getLitecliManualInstructions()) {
@@ -291,7 +294,7 @@ export async function handleOpenShell(containerName: string): Promise<void> {
         await pressEnterToContinue()
       }
     } else {
-      console.error(error('No supported package manager found'))
+      console.error(uiError('No supported package manager found'))
       console.log()
       console.log(chalk.gray('Manual installation:'))
       for (const instruction of getLitecliManualInstructions()) {
@@ -312,7 +315,7 @@ async function launchShell(
   connectionString: string,
   shellType: 'default' | 'usql' | 'pgcli' | 'mycli' | 'litecli',
 ): Promise<void> {
-  console.log(info(`Connecting to ${containerName}...`))
+  console.log(uiInfo(`Connecting to ${containerName}...`))
   console.log()
 
   let shellCmd: string
@@ -386,7 +389,7 @@ async function launchShell(
 
     shellProcess.on('error', (err: NodeJS.ErrnoException) => {
       if (err.code === 'ENOENT') {
-        console.log(warning(`${shellCmd} not found on your system.`))
+        console.log(uiWarning(`${shellCmd} not found on your system.`))
         console.log()
         console.log(chalk.gray('  Connect manually with:'))
         console.log(chalk.cyan(`  ${connectionString}`))
@@ -394,7 +397,7 @@ async function launchShell(
         console.log(chalk.gray(`  Install ${shellCmd}:`))
         console.log(chalk.cyan(`  ${installHint}`))
       } else {
-        console.log(error(`Failed to start ${shellCmd}: ${err.message}`))
+        console.log(uiError(`Failed to start ${shellCmd}: ${err.message}`))
       }
       settle()
     })

package/cli/commands/menu/sql-handlers.ts

@@ -7,14 +7,17 @@ import { containerManager } from '../../../core/container-manager'
 import { getMissingDependencies } from '../../../core/dependency-manager'
 import { getEngine } from '../../../engines'
 import { paths } from '../../../config/paths'
-import { promptInstallDependencies, promptDatabaseSelect } from '../../ui/prompts'
-import { error, warning, info, success } from '../../ui/theme'
+import {
+  promptInstallDependencies,
+  promptDatabaseSelect,
+} from '../../ui/prompts'
+import { uiError, uiWarning, uiInfo, uiSuccess } from '../../ui/theme'
 import { pressEnterToContinue } from './shared'
 
 export async function handleRunSql(containerName: string): Promise<void> {
   const config = await containerManager.getConfig(containerName)
   if (!config) {
-    console.error(error(`Container "${containerName}" not found`))
+    console.error(uiError(`Container "${containerName}" not found`))
     return
   }
 
@@ -23,7 +26,7 @@ export async function handleRunSql(containerName: string): Promise<void> {
   let missingDeps = await getMissingDependencies(config.engine)
   if (missingDeps.length > 0) {
     console.log(
-      warning(`Missing tools: ${missingDeps.map((d) => d.name).join(', ')}`),
+      uiWarning(`Missing tools: ${missingDeps.map((d) => d.name).join(', ')}`),
     )
 
     const installed = await promptInstallDependencies(
@@ -38,7 +41,7 @@ export async function handleRunSql(containerName: string): Promise<void> {
     missingDeps = await getMissingDependencies(config.engine)
     if (missingDeps.length > 0) {
       console.log(
-        error(
+        uiError(
           `Still missing tools: ${missingDeps.map((d) => d.name).join(', ')}`,
         ),
       )
@@ -50,11 +53,14 @@ export async function handleRunSql(containerName: string): Promise<void> {
   }
 
   // Strip quotes that terminals add when drag-and-dropping files
-  const stripQuotes = (path: string) =>
-    path.replace(/^['"]|['"]$/g, '').trim()
+  const stripQuotes = (path: string) => path.replace(/^['"]|['"]$/g, '').trim()
 
   // Prompt for file path (empty input = go back)
-  console.log(chalk.gray('  Drag & drop, enter path (abs or rel), or press Enter to go back'))
+  console.log(
+    chalk.gray(
+      '  Drag & drop, enter path (abs or rel), or press Enter to go back',
+    ),
+  )
   const { filePath: rawFilePath } = await inquirer.prompt<{
     filePath: string
   }>([
@@ -90,7 +96,7 @@ export async function handleRunSql(containerName: string): Promise<void> {
   }
 
   console.log()
-  console.log(info(`Running SQL file against "${databaseName}"...`))
+  console.log(uiInfo(`Running SQL file against "${databaseName}"...`))
   console.log()
 
   try {
@@ -99,11 +105,11 @@ export async function handleRunSql(containerName: string): Promise<void> {
       database: databaseName,
     })
     console.log()
-    console.log(success('SQL file executed successfully'))
-  } catch (err) {
-    const e = err as Error
+    console.log(uiSuccess('SQL file executed successfully'))
+  } catch (error) {
+    const e = error as Error
     console.log()
-    console.log(error(`SQL execution failed: ${e.message}`))
+    console.log(uiError(`SQL execution failed: ${e.message}`))
   }
 
   console.log()
@@ -116,7 +122,7 @@ export async function handleRunSql(containerName: string): Promise<void> {
 export async function handleViewLogs(containerName: string): Promise<void> {
   const config = await containerManager.getConfig(containerName)
   if (!config) {
-    console.error(error(`Container "${containerName}" not found`))
+    console.error(uiError(`Container "${containerName}" not found`))
     return
   }
 
@@ -126,7 +132,7 @@ export async function handleViewLogs(containerName: string): Promise<void> {
 
   if (!existsSync(logPath)) {
     console.log(
-      info(
+      uiInfo(
         `No log file found for "${containerName}". The container may not have been started yet.`,
       ),
     )
@@ -194,7 +200,7 @@ export async function handleViewLogs(containerName: string): Promise<void> {
   const lineCount = action === 'tail-100' ? 100 : 50
   const content = await readFile(logPath, 'utf-8')
   if (content.trim() === '') {
-    console.log(info('Log file is empty'))
+    console.log(uiInfo('Log file is empty'))
   } else {
     const lines = content.split('\n')
     const nonEmptyLines =

package/cli/commands/menu/update-handlers.ts

@@ -9,7 +9,7 @@ import { paths } from '../../../config/paths'
 import { getSupportedEngines } from '../../../config/engine-defaults'
 import { checkEngineDependencies } from '../../../core/dependency-manager'
 import { createSpinner } from '../../ui/spinner'
-import { header, success, error, warning, info } from '../../ui/theme'
+import { header, uiSuccess, uiError, uiWarning, uiInfo } from '../../ui/theme'
 import { pressEnterToContinue } from './shared'
 import { Engine } from '../../../types'
 
@@ -26,7 +26,7 @@ export async function handleCheckUpdate(): Promise<void> {
   if (!result) {
     spinner.fail('Could not reach npm registry')
     console.log()
-    console.log(info('Check your internet connection and try again.'))
+    console.log(uiInfo('Check your internet connection and try again.'))
     console.log(chalk.gray('  Manual update: npm install -g spindb@latest'))
     console.log()
     await pressEnterToContinue()
@@ -66,27 +66,29 @@ export async function handleCheckUpdate(): Promise<void> {
         updateSpinner.succeed('Update complete')
         console.log()
         console.log(
-          success(
+          uiSuccess(
             `Updated from ${updateResult.previousVersion} to ${updateResult.newVersion}`,
           ),
         )
         console.log()
         if (updateResult.previousVersion !== updateResult.newVersion) {
-          console.log(warning('Please restart spindb to use the new version.'))
+          console.log(
+            uiWarning('Please restart spindb to use the new version.'),
+          )
           console.log()
         }
       } else {
         updateSpinner.fail('Update failed')
         console.log()
-        console.log(error(updateResult.error || 'Unknown error'))
+        console.log(uiError(updateResult.error || 'Unknown error'))
         console.log()
-        console.log(info('Manual update: npm install -g spindb@latest'))
+        console.log(uiInfo('Manual update: npm install -g spindb@latest'))
       }
       await pressEnterToContinue()
     } else if (action === 'disable') {
       await updateManager.setAutoCheckEnabled(false)
       console.log()
-      console.log(info('Update checks disabled on startup.'))
+      console.log(uiInfo('Update checks disabled on startup.'))
       console.log(chalk.gray('  Re-enable with: spindb config update-check on'))
       console.log()
       await pressEnterToContinue()
@@ -138,7 +140,7 @@ async function checkConfiguration(): Promise<HealthCheckResult> {
           label: 'Refresh binary cache',
           handler: async () => {
             await configManager.refreshAllBinaries()
-            console.log(success('Binary cache refreshed'))
+            console.log(uiSuccess('Binary cache refreshed'))
           },
         },
       }
@@ -150,12 +152,12 @@ async function checkConfiguration(): Promise<HealthCheckResult> {
       message: 'Configuration valid',
       details: [`Binary tools cached: ${binaryCount}`],
     }
-  } catch (err) {
+  } catch (error) {
     return {
       name: 'Configuration',
       status: 'error',
       message: 'Configuration file is corrupted',
-      details: [(err as Error).message],
+      details: [(error as Error).message],
     }
   }
 }
@@ -199,12 +201,12 @@ async function checkContainers(): Promise<HealthCheckResult> {
       message: `${containers.length} container(s)`,
       details,
     }
-  } catch (err) {
+  } catch (error) {
     return {
       name: 'Containers',
       status: 'error',
       message: 'Failed to list containers',
-      details: [(err as Error).message],
+      details: [(error as Error).message],
     }
   }
 }
@@ -233,7 +235,7 @@ async function checkSqliteRegistry(): Promise<HealthCheckResult> {
           label: 'Remove orphaned entries from registry',
           handler: async () => {
             const count = await sqliteRegistry.removeOrphans()
-            console.log(success(`Removed ${count} orphaned entries`))
+            console.log(uiSuccess(`Removed ${count} orphaned entries`))
           },
         },
       }
@@ -244,12 +246,12 @@ async function checkSqliteRegistry(): Promise<HealthCheckResult> {
       status: 'ok',
       message: `${entries.length} database(s) registered, all files exist`,
     }
-  } catch (err) {
+  } catch (error) {
     return {
       name: 'SQLite Registry',
       status: 'warning',
       message: 'Could not check registry',
-      details: [(err as Error).message],
+      details: [(error as Error).message],
     }
   }
 }
@@ -279,12 +281,12 @@ async function checkBinaries(): Promise<HealthCheckResult> {
       message: hasWarning ? 'Some tools missing' : 'All tools available',
       details: results,
     }
-  } catch (err) {
+  } catch (error) {
     return {
       name: 'Database Tools',
       status: 'error',
       message: 'Failed to check tools',
-      details: [(err as Error).message],
+      details: [(error as Error).message],
     }
   }
 }