spindb 0.7.3 → 0.8.0

package/cli/commands/edit.ts

@@ -8,16 +8,10 @@ import { promptContainerSelect } from '../ui/prompts'
 import { createSpinner } from '../ui/spinner'
 import { error, warning, success } from '../ui/theme'
 
-/**
- * Validate container name format
- */
 function isValidName(name: string): boolean {
   return /^[a-zA-Z][a-zA-Z0-9_-]*$/.test(name)
 }
 
-/**
- * Prompt for what to edit when no options provided
- */
 async function promptEditAction(): Promise<'name' | 'port' | null> {
   const { action } = await inquirer.prompt<{ action: string }>([
     {
@@ -36,9 +30,6 @@ async function promptEditAction(): Promise<'name' | 'port' | null> {
   return action as 'name' | 'port'
 }
 
-/**
- * Prompt for new container name
- */
 async function promptNewName(currentName: string): Promise<string | null> {
   const { newName } = await inquirer.prompt<{ newName: string }>([
     {
@@ -64,9 +55,6 @@ async function promptNewName(currentName: string): Promise<string | null> {
   return newName
 }
 
-/**
- * Prompt for new port
- */
 async function promptNewPort(currentPort: number): Promise<number | null> {
   const { newPort } = await inquirer.prompt<{ newPort: number }>([
     {
@@ -90,7 +78,6 @@ async function promptNewPort(currentPort: number): Promise<number | null> {
     return null
   }
 
-  // Double-check availability and warn (user already confirmed via validation)
   const portAvailable = await portManager.isPortAvailable(newPort)
   if (!portAvailable) {
     console.log(
@@ -116,7 +103,6 @@ export const editCommand = new Command('edit')
       try {
         let containerName = name
 
-        // Interactive selection if no name provided
         if (!containerName) {
           const containers = await containerManager.list()
 
@@ -133,14 +119,12 @@ export const editCommand = new Command('edit')
           containerName = selected
         }
 
-        // Get container config
         const config = await containerManager.getConfig(containerName)
         if (!config) {
           console.error(error(`Container "${containerName}" not found`))
           process.exit(1)
         }
 
-        // If no options provided, prompt for what to edit
         if (options.name === undefined && options.port === undefined) {
           const action = await promptEditAction()
           if (!action) return
@@ -162,9 +146,7 @@ export const editCommand = new Command('edit')
           }
         }
 
-        // Handle rename
         if (options.name) {
-          // Validate new name
           if (!isValidName(options.name)) {
             console.error(
               error(
@@ -174,7 +156,6 @@ export const editCommand = new Command('edit')
             process.exit(1)
           }
 
-          // Check if new name already exists
           const exists = await containerManager.exists(options.name, {
             engine: config.engine,
           })
@@ -183,7 +164,6 @@ export const editCommand = new Command('edit')
             process.exit(1)
           }
 
-          // Check if container is running
           const running = await processManager.isRunning(containerName, {
             engine: config.engine,
           })
@@ -196,7 +176,6 @@ export const editCommand = new Command('edit')
             process.exit(1)
           }
 
-          // Rename the container
           const spinner = createSpinner(
             `Renaming "${containerName}" to "${options.name}"...`,
           )
@@ -206,19 +185,15 @@ export const editCommand = new Command('edit')
 
           spinner.succeed(`Renamed "${containerName}" to "${options.name}"`)
 
-          // Update containerName for subsequent operations
           containerName = options.name
         }
 
-        // Handle port change
         if (options.port !== undefined) {
-          // Validate port
           if (options.port < 1 || options.port > 65535) {
             console.error(error('Port must be between 1 and 65535'))
             process.exit(1)
           }
 
-          // Check port availability (warning only)
           const portAvailable = await portManager.isPortAvailable(options.port)
           if (!portAvailable) {
             console.log(
@@ -228,7 +203,6 @@ export const editCommand = new Command('edit')
             )
           }
 
-          // Update the config
           const spinner = createSpinner(`Changing port to ${options.port}...`)
           spinner.start()
 

package/cli/commands/info.ts

@@ -9,17 +9,11 @@ import { error, info, header } from '../ui/theme'
 import { getEngineIcon } from '../constants'
 import type { ContainerConfig } from '../../types'
 
-/**
- * Format a date for display
- */
 function formatDate(dateString: string): string {
   const date = new Date(dateString)
   return date.toLocaleString()
 }
 
-/**
- * Get actual running status (not just config status)
- */
 async function getActualStatus(
   config: ContainerConfig,
 ): Promise<'running' | 'stopped'> {
@@ -29,9 +23,6 @@ async function getActualStatus(
   return running ? 'running' : 'stopped'
 }
 
-/**
- * Display info for a single container
- */
 async function displayContainerInfo(
   config: ContainerConfig,
   options: { json?: boolean },
@@ -109,15 +100,11 @@ async function displayContainerInfo(
   console.log()
 }
 
-/**
- * Display summary info for all containers
- */
 async function displayAllContainersInfo(
   containers: ContainerConfig[],
   options: { json?: boolean },
 ): Promise<void> {
   if (options.json) {
-    // Get actual status for all containers
     const containersWithStatus = await Promise.all(
       containers.map(async (config) => {
         const actualStatus = await getActualStatus(config)
@@ -142,7 +129,6 @@ async function displayAllContainersInfo(
   console.log(header('All Containers'))
   console.log()
 
-  // Table header
   console.log(
     chalk.gray('  ') +
       chalk.bold.white('NAME'.padEnd(18)) +
@@ -154,7 +140,6 @@ async function displayAllContainersInfo(
   )
   console.log(chalk.gray('  ' + '─'.repeat(78)))
 
-  // Table rows
   for (const container of containers) {
     const actualStatus = await getActualStatus(container)
     const statusDisplay =
@@ -178,7 +163,6 @@ async function displayAllContainersInfo(
 
   console.log()
 
-  // Summary
   const statusChecks = await Promise.all(
     containers.map((c) => getActualStatus(c)),
   )
@@ -192,7 +176,6 @@ async function displayAllContainersInfo(
   )
   console.log()
 
-  // Connection strings
   console.log(chalk.bold.white('  Connection Strings:'))
   console.log(chalk.gray('  ' + '─'.repeat(78)))
   for (const container of containers) {
@@ -221,7 +204,6 @@ export const infoCommand = new Command('info')
         return
       }
 
-      // If name provided, show single container
       if (name) {
         const config = await containerManager.getConfig(name)
         if (!config) {
@@ -232,7 +214,6 @@ export const infoCommand = new Command('info')
         return
       }
 
-      // If running interactively without name, ask if they want all or specific
       if (!options.json && process.stdout.isTTY && containers.length > 1) {
         const { choice } = await inquirer.prompt<{
           choice: string
@@ -262,7 +243,6 @@ export const infoCommand = new Command('info')
         return
       }
 
-      // Non-interactive or only one container: show all
       await displayAllContainersInfo(containers, options)
     } catch (err) {
       const e = err as Error

package/cli/commands/list.ts

@@ -6,9 +6,6 @@ import { info, error, formatBytes } from '../ui/theme'
 import { getEngineIcon } from '../constants'
 import type { ContainerConfig } from '../../types'
 
-/**
- * Get database size for a container (only if running)
- */
 async function getContainerSize(
   container: ContainerConfig,
 ): Promise<number | null> {
@@ -32,7 +29,6 @@ export const listCommand = new Command('list')
       const containers = await containerManager.list()
 
       if (options.json) {
-        // Include sizes in JSON output
         const containersWithSize = await Promise.all(
           containers.map(async (container) => ({
             ...container,
@@ -48,10 +44,8 @@ export const listCommand = new Command('list')
         return
       }
 
-      // Fetch sizes for running containers in parallel
       const sizes = await Promise.all(containers.map(getContainerSize))
 
-      // Table header
       console.log()
       console.log(
         chalk.gray('  ') +
@@ -64,7 +58,6 @@ export const listCommand = new Command('list')
       )
       console.log(chalk.gray('  ' + '─'.repeat(73)))
 
-      // Table rows
       for (let i = 0; i < containers.length; i++) {
         const container = containers[i]
         const size = sizes[i]
@@ -77,7 +70,6 @@ export const listCommand = new Command('list')
         const engineIcon = getEngineIcon(container.engine)
         const engineDisplay = `${engineIcon} ${container.engine}`
 
-        // Format size: show value if running, dash if stopped
         const sizeDisplay = size !== null ? formatBytes(size) : chalk.gray('—')
 
         console.log(
@@ -93,7 +85,6 @@ export const listCommand = new Command('list')
 
       console.log()
 
-      // Summary
       const running = containers.filter((c) => c.status === 'running').length
       const stopped = containers.filter((c) => c.status !== 'running').length
       console.log(

package/cli/commands/logs.ts

@@ -7,12 +7,8 @@ import { paths } from '../../config/paths'
 import { promptContainerSelect } from '../ui/prompts'
 import { error, warning, info } from '../ui/theme'
 
-/**
- * Get the last N lines from a file content
- */
 function getLastNLines(content: string, n: number): string {
   const lines = content.split('\n')
-  // Filter empty trailing line if present
   const nonEmptyLines =
     lines[lines.length - 1] === '' ? lines.slice(0, -1) : lines
   return nonEmptyLines.slice(-n).join('\n')
@@ -32,7 +28,6 @@ export const logsCommand = new Command('logs')
       try {
         let containerName = name
 
-        // Interactive selection if no name provided
         if (!containerName) {
           const containers = await containerManager.list()
 
@@ -49,19 +44,16 @@ export const logsCommand = new Command('logs')
           containerName = selected
         }
 
-        // Get container config
         const config = await containerManager.getConfig(containerName)
         if (!config) {
           console.error(error(`Container "${containerName}" not found`))
           process.exit(1)
         }
 
-        // Get log file path
         const logPath = paths.getContainerLogPath(config.name, {
           engine: config.engine,
         })
 
-        // Check if log file exists
         if (!existsSync(logPath)) {
           console.log(
             info(
@@ -71,7 +63,6 @@ export const logsCommand = new Command('logs')
           return
         }
 
-        // Open in editor if requested
         if (options.editor) {
           const editorCmd = process.env.EDITOR || 'vi'
           const child = spawn(editorCmd, [logPath], {
@@ -91,14 +82,12 @@ export const logsCommand = new Command('logs')
           return
         }
 
-        // Follow mode using tail -f
         if (options.follow) {
           const lineCount = parseInt(options.lines || '50', 10)
           const child = spawn('tail', ['-n', String(lineCount), '-f', logPath], {
             stdio: 'inherit',
           })
 
-          // Handle SIGINT gracefully
           process.on('SIGINT', () => {
             child.kill('SIGTERM')
             process.exit(0)
@@ -110,7 +99,6 @@ export const logsCommand = new Command('logs')
           return
         }
 
-        // Default: read and output last N lines
         const lineCount = parseInt(options.lines || '50', 10)
         const content = await readFile(logPath, 'utf-8')