spindb 0.9.1 → 0.9.2

package/engines/mysql/index.ts

@@ -450,8 +450,8 @@ export class MySQLEngine extends BaseEngine {
         await this.cleanupPidFile(pidFile)
         return null
       }
-    } catch (err) {
-      const e = err as NodeJS.ErrnoException
+    } catch (error) {
+      const e = error as NodeJS.ErrnoException
       if (e.code !== 'ENOENT') {
         logWarning(`Failed to read PID file: ${e.message}`, {
           pidFile,
@@ -479,8 +479,8 @@ export class MySQLEngine extends BaseEngine {
           `"${mysqladmin}" -h 127.0.0.1 -P ${port} -u root shutdown`,
           { timeout: 5000 },
         )
-      } catch (err) {
-        const e = err as Error
+      } catch (error) {
+        const e = error as Error
         logDebug(`mysqladmin shutdown failed: ${e.message}`)
         // Continue to wait for process to die or send SIGTERM
       }
@@ -542,8 +542,8 @@ export class MySQLEngine extends BaseEngine {
         await this.cleanupPidFile(pidFile)
         return
       }
-    } catch (err) {
-      const e = err as NodeJS.ErrnoException
+    } catch (error) {
+      const e = error as NodeJS.ErrnoException
       if (e.code === 'ESRCH') {
         // Process already dead
         await this.cleanupPidFile(pidFile)
@@ -575,9 +575,9 @@ export class MySQLEngine extends BaseEngine {
         logDebug(`Process ${pid} terminated after SIGKILL`)
         await this.cleanupPidFile(pidFile)
       }
-    } catch (err) {
-      if (err instanceof SpinDBError) throw err
-      const e = err as NodeJS.ErrnoException
+    } catch (error) {
+      if (error instanceof SpinDBError) throw error
+      const e = error as NodeJS.ErrnoException
       if (e.code === 'ESRCH') {
         // Process already dead
         await this.cleanupPidFile(pidFile)
@@ -594,8 +594,8 @@ export class MySQLEngine extends BaseEngine {
     try {
       await unlink(pidFile)
       logDebug('PID file cleaned up')
-    } catch (err) {
-      const e = err as NodeJS.ErrnoException
+    } catch (error) {
+      const e = error as NodeJS.ErrnoException
       if (e.code !== 'ENOENT') {
         logDebug(`Failed to clean up PID file: ${e.message}`)
       }

package/engines/mysql/restore.ts

@@ -174,13 +174,13 @@ export async function restoreBackup(
   if (validateVersion) {
     try {
       await validateRestoreCompatibility({ dumpPath: backupPath })
-    } catch (err) {
+    } catch (error) {
       // Re-throw SpinDBError, log and continue for other errors
-      if (err instanceof Error && err.name === 'SpinDBError') {
-        throw err
+      if (error instanceof Error && error.name === 'SpinDBError') {
+        throw error
       }
       logDebug('Version validation failed, proceeding anyway', {
-        error: err instanceof Error ? err.message : String(err),
+        error: error instanceof Error ? error.message : String(error),
       })
     }
   }

package/engines/mysql/version-validator.ts

@@ -197,10 +197,10 @@ export async function parseDumpVersion(dumpPath: string): Promise<DumpInfo> {
     }
 
     return { version: null, variant }
-  } catch (err) {
+  } catch (error) {
     logDebug('Failed to parse dump version', {
       dumpPath,
-      error: err instanceof Error ? err.message : String(err),
+      error: error instanceof Error ? error.message : String(error),
     })
     return { version: null, variant: 'unknown' }
   }

package/engines/postgresql/binary-manager.ts

@@ -2,7 +2,7 @@ import { exec } from 'child_process'
 import { promisify } from 'util'
 import chalk from 'chalk'
 import { createSpinner } from '../../cli/ui/spinner'
-import { warning, error as themeError, success } from '../../cli/ui/theme'
+import { uiWarning, uiError, uiSuccess } from '../../cli/ui/theme'
 import {
   detectPackageManager as detectPM,
   installEngineDependencies,
@@ -311,7 +311,7 @@ export async function installPostgresBinaries(): Promise<boolean> {
   const packageManager = await detectPM()
   if (!packageManager) {
     spinner.fail('No supported package manager found')
-    console.log(themeError('Please install PostgreSQL client tools manually:'))
+    console.log(uiError('Please install PostgreSQL client tools manually:'))
 
     // Show platform-specific instructions from the registry
     const platform = getCurrentPlatform()
@@ -348,21 +348,21 @@ export async function installPostgresBinaries(): Promise<boolean> {
 
     if (allSuccess) {
       console.log()
-      console.log(success('PostgreSQL client tools installed successfully'))
+      console.log(uiSuccess('PostgreSQL client tools installed successfully'))
       return true
     } else {
       const failed = results.filter((r) => !r.success)
       console.log()
-      console.log(themeError('Some installations failed:'))
+      console.log(uiError('Some installations failed:'))
       for (const f of failed) {
-        console.log(themeError(`  ${f.dependency.name}: ${f.error}`))
+        console.log(uiError(`  ${f.dependency.name}: ${f.error}`))
       }
       return false
     }
   } catch (error: unknown) {
     console.log()
-    console.log(themeError('Failed to install PostgreSQL client tools'))
-    console.log(warning('Please install manually'))
+    console.log(uiError('Failed to install PostgreSQL client tools'))
+    console.log(uiWarning('Please install manually'))
     if (error instanceof Error) {
       console.log(chalk.gray(`Error details: ${error.message}`))
     }
@@ -408,7 +408,7 @@ export async function updatePostgresClientTools(): Promise<boolean> {
 
       spinner.succeed('PostgreSQL client tools updated')
       console.log(
-        success(
+        uiSuccess(
           `Client tools successfully linked to PostgreSQL ${latestMajor}`,
         ),
       )
@@ -418,13 +418,13 @@ export async function updatePostgresClientTools(): Promise<boolean> {
       // For other package managers, use the standard update
       await execWithTimeout(packageManager.updateCommand('postgresql'), 120000)
       spinner.succeed('PostgreSQL client tools updated')
-      console.log(success('Update completed successfully'))
+      console.log(uiSuccess('Update completed successfully'))
       return true
     }
   } catch (error: unknown) {
     spinner.fail('Update failed')
-    console.log(themeError('Failed to update PostgreSQL client tools'))
-    console.log(warning('Please update manually:'))
+    console.log(uiError('Failed to update PostgreSQL client tools'))
+    console.log(uiWarning('Please update manually:'))
 
     if (packageManager.name === 'brew') {
       const olderVersions = ['14', '15', '16'].filter((v) => v !== latestMajor)
@@ -473,12 +473,12 @@ export async function updatePostgresBinaries(): Promise<boolean> {
   try {
     await execWithTimeout(packageManager.updateCommand('postgresql'), 120000) // 2 minute timeout
     updateSpinner.succeed('PostgreSQL client tools updated')
-    console.log(success('Update completed successfully'))
+    console.log(uiSuccess('Update completed successfully'))
     return true
   } catch (error: unknown) {
     updateSpinner.fail('Update failed')
-    console.log(themeError('Failed to update PostgreSQL client tools'))
-    console.log(warning('Please update manually:'))
+    console.log(uiError('Failed to update PostgreSQL client tools'))
+    console.log(uiWarning('Please update manually:'))
     console.log(`  ${packageManager.updateCommand('postgresql')}`)
     if (error instanceof Error) {
       console.log(chalk.gray(`Error details: ${error.message}`))
@@ -505,7 +505,7 @@ export async function ensurePostgresBinary(
       return { success: false, info: null, action: 'install_required' }
     }
 
-    console.log(warning(`${binary} not found on your system`))
+    console.log(uiWarning(`${binary} not found on your system`))
     const success = await installPostgresBinaries()
     if (!success) {
       return { success: false, info: null, action: 'install_failed' }
@@ -527,13 +527,13 @@ export async function ensurePostgresBinary(
     }
 
     console.log(
-      warning(
+      uiWarning(
         `Your ${binary} version (${info.version}) is incompatible with the dump file`,
       ),
     )
     if (info.requiredVersion) {
       console.log(
-        warning(`Required version: ${info.requiredVersion} or compatible`),
+        uiWarning(`Required version: ${info.requiredVersion} or compatible`),
       )
     }
 

package/engines/postgresql/index.ts

@@ -155,8 +155,13 @@ export class PostgreSQLEngine extends BaseEngine {
 
     // Configure max_connections after initdb creates postgresql.conf
     const maxConnections =
-      (options.maxConnections as number) || getEngineDefaults('postgresql').maxConnections
-    await this.setConfigValue(dataDir, 'max_connections', String(maxConnections))
+      (options.maxConnections as number) ||
+      getEngineDefaults('postgresql').maxConnections
+    await this.setConfigValue(
+      dataDir,
+      'max_connections',
+      String(maxConnections),
+    )
 
     return dataDir
   }

package/engines/postgresql/restore.ts

@@ -219,8 +219,8 @@ export async function restoreBackup(
         format: detectedFormat,
         ...result,
       }
-    } catch (err) {
-      const e = err as Error & { stdout?: string; stderr?: string }
+    } catch (error) {
+      const e = error as Error & { stdout?: string; stderr?: string }
       // pg_restore often returns non-zero even on partial success
       return {
         format: detectedFormat,

package/engines/postgresql/version-validator.ts

@@ -135,11 +135,11 @@ export async function parseDumpVersion(
         }
       }
     }
-  } catch (err) {
+  } catch (error) {
     logDebug('Failed to parse dump version', {
       dumpPath,
       format,
-      error: err instanceof Error ? err.message : String(err),
+      error: error instanceof Error ? error.message : String(error),
     })
   }
 

package/engines/sqlite/index.ts

@@ -538,15 +538,28 @@ export class SQLiteEngine extends BaseEngine {
    * Download a file from HTTP/HTTPS URL
    */
   private async downloadFile(url: string, destPath: string): Promise<void> {
-    const response = await fetch(url)
-    if (!response.ok) {
-      throw new Error(
-        `Failed to download: ${response.status} ${response.statusText}`,
-      )
-    }
+    const controller = new AbortController()
+    const timeoutMs = 5 * 60 * 1000 // 5 minutes
+    const timeout = setTimeout(() => controller.abort(), timeoutMs)
+
+    try {
+      const response = await fetch(url, { signal: controller.signal })
+      if (!response.ok) {
+        throw new Error(
+          `Failed to download: ${response.status} ${response.statusText}`,
+        )
+      }
 
-    const buffer = await response.arrayBuffer()
-    await writeFile(destPath, Buffer.from(buffer))
+      const buffer = await response.arrayBuffer()
+      await writeFile(destPath, Buffer.from(buffer))
+    } catch (error) {
+      if (error instanceof Error && error.name === 'AbortError') {
+        throw new Error('Download timed out after 5 minutes')
+      }
+      throw error
+    } finally {
+      clearTimeout(timeout)
+    }
   }
 
   /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spindb",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Spin up local database containers without Docker. A DBngin-like CLI for PostgreSQL and MySQL.",
   "type": "module",
   "bin": {