spindb 0.9.1 → 0.9.2

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],
     }
   }
 }

package/cli/commands/restore.ts

@@ -11,7 +11,7 @@ import {
   promptInstallDependencies,
 } from '../ui/prompts'
 import { createSpinner } from '../ui/spinner'
-import { success, error, warning } from '../ui/theme'
+import { uiSuccess, uiError, uiWarning } from '../ui/theme'
 import { tmpdir } from 'os'
 import { join } from 'path'
 import { getMissingDependencies } from '../../core/dependency-manager'
@@ -50,11 +50,13 @@ export const restoreCommand = new Command('restore')
           if (running.length === 0) {
             if (containers.length === 0) {
               console.log(
-                warning('No containers found. Create one with: spindb create'),
+                uiWarning(
+                  'No containers found. Create one with: spindb create',
+                ),
               )
             } else {
               console.log(
-                warning(
+                uiWarning(
                   'No running containers. Start one first with: spindb start',
                 ),
               )
@@ -72,7 +74,7 @@ export const restoreCommand = new Command('restore')
 
         const config = await containerManager.getConfig(containerName)
         if (!config) {
-          console.error(error(`Container "${containerName}" not found`))
+          console.error(uiError(`Container "${containerName}" not found`))
           process.exit(1)
         }
 
@@ -83,7 +85,7 @@ export const restoreCommand = new Command('restore')
         })
         if (!running) {
           console.error(
-            error(
+            uiError(
               `Container "${containerName}" is not running. Start it first.`,
             ),
           )
@@ -113,7 +115,7 @@ export const restoreCommand = new Command('restore')
           missingDeps = await getMissingDependencies(config.engine)
           if (missingDeps.length > 0) {
             console.error(
-              error(
+              uiError(
                 `Still missing tools: ${missingDeps.map((d) => d.name).join(', ')}`,
               ),
             )
@@ -134,7 +136,7 @@ export const restoreCommand = new Command('restore')
 
           if (engineName === 'postgresql' && !isPgUrl) {
             console.error(
-              error(
+              uiError(
                 'Connection string must start with postgresql:// or postgres:// for PostgreSQL containers',
               ),
             )
@@ -143,7 +145,7 @@ export const restoreCommand = new Command('restore')
 
           if (engineName === 'mysql' && !isMysqlUrl) {
             console.error(
-              error(
+              uiError(
                 'Connection string must start with mysql:// for MySQL containers',
               ),
             )
@@ -152,7 +154,7 @@ export const restoreCommand = new Command('restore')
 
           if (!isPgUrl && !isMysqlUrl) {
             console.error(
-              error(
+              uiError(
                 'Connection string must start with postgresql://, postgres://, or mysql://',
               ),
             )
@@ -181,8 +183,8 @@ export const restoreCommand = new Command('restore')
               dumpSpinner.succeed('Dump created from remote database')
               backupPath = tempDumpPath
               dumpSuccess = true
-            } catch (err) {
-              const e = err as Error
+            } catch (error) {
+              const e = error as Error
               dumpSpinner.fail('Failed to create dump')
 
               const dumpTool = engineName === 'mysql' ? 'mysqldump' : 'pg_dump'
@@ -201,19 +203,19 @@ export const restoreCommand = new Command('restore')
               }
 
               console.log()
-              console.error(error(`${dumpTool} error:`))
+              console.error(uiError(`${dumpTool} error:`))
               console.log(chalk.gray(`  ${e.message}`))
               process.exit(1)
             }
           }
 
           if (!dumpSuccess) {
-            console.error(error('Failed to create dump after retries'))
+            console.error(uiError('Failed to create dump after retries'))
             process.exit(1)
           }
         } else {
           if (!backupPath) {
-            console.error(error('Backup file path is required'))
+            console.error(uiError('Backup file path is required'))
             console.log(
               chalk.gray('  Usage: spindb restore <container> <backup-file>'),
             )
@@ -226,7 +228,7 @@ export const restoreCommand = new Command('restore')
           }
 
           if (!existsSync(backupPath)) {
-            console.error(error(`Backup file not found: ${backupPath}`))
+            console.error(uiError(`Backup file not found: ${backupPath}`))
             process.exit(1)
           }
         }
@@ -237,7 +239,7 @@ export const restoreCommand = new Command('restore')
         }
 
         if (!backupPath) {
-          console.error(error('No backup path specified'))
+          console.error(uiError('No backup path specified'))
           process.exit(1)
         }
 
@@ -347,7 +349,7 @@ export const restoreCommand = new Command('restore')
           databaseName,
         )
         console.log()
-        console.log(success(`Database "${databaseName}" restored`))
+        console.log(uiSuccess(`Database "${databaseName}" restored`))
         console.log()
         console.log(chalk.gray('  Connection string:'))
         console.log(chalk.cyan(`  ${connectionString}`))
@@ -365,8 +367,8 @@ export const restoreCommand = new Command('restore')
           chalk.cyan(`  spindb connect ${containerName} -d ${databaseName}`),
         )
         console.log()
-      } catch (err) {
-        const e = err as Error
+      } catch (error) {
+        const e = error as Error
 
         const missingToolPatterns = [
           'pg_restore not found',
@@ -391,7 +393,7 @@ export const restoreCommand = new Command('restore')
           process.exit(1)
         }
 
-        console.error(error(e.message))
+        console.error(uiError(e.message))
         process.exit(1)
       } finally {
         if (tempDumpPath) {