spindb 0.9.1 → 0.9.3

package/cli/commands/deps.ts

@@ -1,6 +1,6 @@
 import { Command } from 'commander'
 import chalk from 'chalk'
-import { header, success, warning, error } from '../ui/theme'
+import { header, uiSuccess, uiWarning, uiError } from '../ui/theme'
 import { createSpinner } from '../ui/spinner'
 import {
   detectPackageManager,
@@ -84,7 +84,7 @@ depsCommand
       // Check specific engine
       const engineConfig = getEngineDependencies(options.engine)
       if (!engineConfig) {
-        console.error(error(`Unknown engine: ${options.engine}`))
+        console.error(uiError(`Unknown engine: ${options.engine}`))
         console.log(
           chalk.gray(
             `  Available engines: ${engineDependencies.map((e) => e.engine).join(', ')}`,
@@ -104,9 +104,9 @@ depsCommand
       const total = statuses.length
       console.log()
       if (installed === total) {
-        console.log(success(`All ${total} dependencies installed`))
+        console.log(uiSuccess(`All ${total} dependencies installed`))
       } else {
-        console.log(warning(`${installed}/${total} dependencies installed`))
+        console.log(uiWarning(`${installed}/${total} dependencies installed`))
         console.log()
         console.log(
           chalk.gray(`  Run: spindb deps install --engine ${options.engine}`),
@@ -132,7 +132,7 @@ depsCommand
     const packageManager = await detectPackageManager()
 
     if (!packageManager) {
-      console.log(error('No supported package manager detected'))
+      console.log(uiError('No supported package manager detected'))
       console.log()
 
       const platform = getCurrentPlatform()
@@ -161,7 +161,7 @@ depsCommand
       const missing = await getAllMissingDependencies()
 
       if (missing.length === 0) {
-        console.log(success('All dependencies are already installed'))
+        console.log(uiSuccess('All dependencies are already installed'))
         return
       }
 
@@ -182,14 +182,14 @@ depsCommand
         spinner.warn('Some dependencies failed to install')
         console.log()
         for (const f of failed) {
-          console.log(error(`  ${f.dependency.name}: ${f.error}`))
+          console.log(uiError(`  ${f.dependency.name}: ${f.error}`))
         }
       }
 
       if (succeeded.length > 0) {
         console.log()
         console.log(
-          success(
+          uiSuccess(
             `Installed: ${succeeded.map((r) => r.dependency.name).join(', ')}`,
           ),
         )
@@ -198,7 +198,7 @@ depsCommand
       // Install dependencies for specific engine
       const engineConfig = getEngineDependencies(options.engine)
       if (!engineConfig) {
-        console.error(error(`Unknown engine: ${options.engine}`))
+        console.error(uiError(`Unknown engine: ${options.engine}`))
         console.log(
           chalk.gray(
             `  Available engines: ${engineDependencies.map((e) => e.engine).join(', ')}`,
@@ -211,7 +211,9 @@ depsCommand
 
       if (missing.length === 0) {
         console.log(
-          success(`All ${engineConfig.displayName} dependencies are installed`),
+          uiSuccess(
+            `All ${engineConfig.displayName} dependencies are installed`,
+          ),
         )
         return
       }
@@ -241,7 +243,7 @@ depsCommand
         spinner.warn('Some dependencies failed to install')
         console.log()
         for (const f of failed) {
-          console.log(error(`  ${f.dependency.name}: ${f.error}`))
+          console.log(uiError(`  ${f.dependency.name}: ${f.error}`))
         }
 
         // Show manual instructions
@@ -259,7 +261,7 @@ depsCommand
       if (succeeded.length > 0) {
         console.log()
         console.log(
-          success(
+          uiSuccess(
             `Installed: ${succeeded.map((r) => r.dependency.name).join(', ')}`,
           ),
         )
@@ -276,7 +278,7 @@ depsCommand
       const missing = await getMissingDependencies('postgresql')
 
       if (missing.length === 0) {
-        console.log(success('All PostgreSQL dependencies are installed'))
+        console.log(uiSuccess('All PostgreSQL dependencies are installed'))
         return
       }
 
@@ -300,14 +302,14 @@ depsCommand
         spinner.warn('Some dependencies failed to install')
         console.log()
         for (const f of failed) {
-          console.log(error(`  ${f.dependency.name}: ${f.error}`))
+          console.log(uiError(`  ${f.dependency.name}: ${f.error}`))
         }
       }
 
       if (succeeded.length > 0) {
         console.log()
         console.log(
-          success(
+          uiSuccess(
             `Installed: ${succeeded.map((r) => r.dependency.name).join(', ')}`,
           ),
         )

package/cli/commands/detach.ts

@@ -0,0 +1,100 @@
+import { Command } from 'commander'
+import chalk from 'chalk'
+import { sqliteRegistry } from '../../engines/sqlite/registry'
+import { containerManager } from '../../core/container-manager'
+import { promptConfirm } from '../ui/prompts'
+import { uiSuccess, uiError, uiWarning } from '../ui/theme'
+import { Engine } from '../../types'
+
+export const detachCommand = new Command('detach')
+  .description('Unregister a SQLite database from SpinDB (keeps file on disk)')
+  .argument('<name>', 'Container name')
+  .option('-f, --force', 'Skip confirmation prompt')
+  .option('--json', 'Output as JSON')
+  .action(
+    async (
+      name: string,
+      options: { force?: boolean; json?: boolean },
+    ): Promise<void> => {
+      try {
+        // Get container config
+        const config = await containerManager.getConfig(name)
+
+        if (!config) {
+          if (options.json) {
+            console.log(
+              JSON.stringify({ success: false, error: 'Container not found' }),
+            )
+          } else {
+            console.error(uiError(`Container "${name}" not found`))
+          }
+          process.exit(1)
+        }
+
+        // Verify it's a SQLite container
+        if (config.engine !== Engine.SQLite) {
+          if (options.json) {
+            console.log(
+              JSON.stringify({
+                success: false,
+                error:
+                  'Not a SQLite container. Use "spindb delete" for server databases.',
+              }),
+            )
+          } else {
+            console.error(uiError(`"${name}" is not a SQLite container`))
+            console.log(
+              chalk.gray(
+                '  Use "spindb delete" for server databases (PostgreSQL, MySQL)',
+              ),
+            )
+          }
+          process.exit(1)
+        }
+
+        // Confirm unless --force
+        if (!options.force && !options.json) {
+          const confirmed = await promptConfirm(
+            `Detach "${name}" from SpinDB? (file will be kept on disk)`,
+            true,
+          )
+          if (!confirmed) {
+            console.log(uiWarning('Cancelled'))
+            return
+          }
+        }
+
+        const entry = await sqliteRegistry.get(name)
+        const filePath = entry?.filePath
+
+        // Remove from registry only (not the file)
+        await sqliteRegistry.remove(name)
+
+        if (options.json) {
+          console.log(
+            JSON.stringify({
+              success: true,
+              name,
+              filePath,
+            }),
+          )
+        } else {
+          console.log(uiSuccess(`Detached "${name}" from SpinDB`))
+          if (filePath) {
+            console.log(chalk.gray(`  File remains at: ${filePath}`))
+          }
+          console.log()
+          console.log(chalk.gray('  Re-attach with:'))
+          console.log(chalk.cyan(`    spindb attach ${filePath || '<path>'}`))
+        }
+      } catch (error) {
+        const e = error as Error
+        if (options.json) {
+          console.log(JSON.stringify({ success: false, error: e.message }))
+        } else {
+          console.error(uiError(e.message))
+        }
+        process.exit(1)
+      }
+    },
+  )

package/cli/commands/doctor.ts

@@ -18,7 +18,7 @@ import { sqliteRegistry } from '../../engines/sqlite/registry'
 import { paths } from '../../config/paths'
 import { getSupportedEngines } from '../../config/engine-defaults'
 import { checkEngineDependencies } from '../../core/dependency-manager'
-import { header, success } from '../ui/theme'
+import { header, uiSuccess } from '../ui/theme'
 import { Engine } from '../../types'
 
 type HealthCheckResult = {
@@ -61,7 +61,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'))
           },
         },
       }
@@ -73,12 +73,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],
     }
   }
 }
@@ -125,12 +125,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],
     }
   }
 }
@@ -141,8 +141,9 @@ async function checkContainers(): Promise<HealthCheckResult> {
 async function checkSqliteRegistry(): Promise<HealthCheckResult> {
   try {
     const entries = await sqliteRegistry.list()
+    const ignoredFolders = await sqliteRegistry.listIgnoredFolders()
 
-    if (entries.length === 0) {
+    if (entries.length === 0 && ignoredFolders.length === 0) {
       return {
         name: 'SQLite Registry',
         status: 'ok',
@@ -153,32 +154,45 @@ async function checkSqliteRegistry(): Promise<HealthCheckResult> {
     const orphans = await sqliteRegistry.findOrphans()
 
     if (orphans.length > 0) {
+      const details = [
+        ...orphans.map((o) => `"${o.name}" → ${o.filePath}`),
+        ...(ignoredFolders.length > 0
+          ? [`${ignoredFolders.length} folder(s) ignored`]
+          : []),
+      ]
+
       return {
         name: 'SQLite Registry',
         status: 'warning',
         message: `${orphans.length} orphaned entr${orphans.length === 1 ? 'y' : 'ies'} found`,
-        details: orphans.map((o) => `"${o.name}" → ${o.filePath}`),
+        details,
         action: {
           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`))
           },
         },
       }
     }
 
+    const details = [`${entries.length} database(s) registered, all files exist`]
+    if (ignoredFolders.length > 0) {
+      details.push(`${ignoredFolders.length} folder(s) ignored`)
+    }
+
     return {
       name: 'SQLite Registry',
       status: 'ok',
       message: `${entries.length} database(s) registered, all files exist`,
+      details: ignoredFolders.length > 0 ? details : undefined,
     }
-  } 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],
     }
   }
 }
@@ -211,12 +225,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],
     }
   }
 }