feat(update): add version check to skip update when already latest

Now `nupst update` checks current version against latest release before
downloading anything.

Behavior:
- Fetches latest version from Gitea API
- Compares with current version
- Shows "Already up to date!" if versions match
- Only downloads/installs if newer version available

Example output when up to date:
  Checking for updates...
  Current version: v4.0.8
  Latest version:  v4.0.8

  ✓ Already up to date!

deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@serve.zone/nupst",
-  "version": "4.0.4",
+  "version": "4.0.8",
   "exports": "./mod.ts",
   "tasks": {
     "dev": "deno run --allow-all mod.ts",

install.sh

@@ -10,15 +10,7 @@
 #   With version specification:
 #     curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- --version v4.0.0
 #
-#   Non-interactive mode (auto-confirm):
-#     curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y
-#
-#   Downloaded script:
-#     curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh -o nupst-install.sh
-#     sudo bash nupst-install.sh
-#
 # Options:
-#   -y, --yes              Automatically answer yes to all prompts
 #   -h, --help             Show this help message
 #   --version VERSION      Install specific version (e.g., v4.0.0)
 #   --install-dir DIR      Installation directory (default: /opt/nupst)
@@ -26,7 +18,6 @@
 set -e
 
 # Default values
-AUTO_YES=0
 SHOW_HELP=0
 SPECIFIED_VERSION=""
 INSTALL_DIR="/opt/nupst"
@@ -36,10 +27,6 @@ GITEA_REPO="serve.zone/nupst"
 # Parse command line arguments
 while [[ $# -gt 0 ]]; do
   case $1 in
-    -y|--yes)
-      AUTO_YES=1
-      shift
-      ;;
     -h|--help)
       SHOW_HELP=1
       shift
@@ -67,7 +54,6 @@ if [ $SHOW_HELP -eq 1 ]; then
   echo "Usage: $0 [options]"
   echo ""
   echo "Options:"
-  echo "  -y, --yes              Automatically answer yes to all prompts"
   echo "  -h, --help             Show this help message"
   echo "  --version VERSION      Install specific version (e.g., v4.0.0)"
   echo "  --install-dir DIR      Installation directory (default: /opt/nupst)"
@@ -78,9 +64,6 @@ if [ $SHOW_HELP -eq 1 ]; then
   echo ""
   echo "  # Install specific version"
   echo "  curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- --version v4.0.0"
-  echo ""
-  echo "  # Non-interactive installation"
-  echo "  curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y"
   exit 0
 fi
 
@@ -90,36 +73,6 @@ if [ "$EUID" -ne 0 ]; then
   exit 1
 fi
 
-# Detect if script is being piped or run directly
-INTERACTIVE=1
-if [ ! -t 0 ] || [ ! -t 1 ]; then
-  # Either stdin or stdout is not a terminal
-  if [ $AUTO_YES -ne 1 ]; then
-    echo "Script detected it's running in a non-interactive environment without -y flag."
-    echo "Attempting to find a controlling terminal for interactive prompts..."
-    # Try to use a controlling terminal for user input
-    if exec < /dev/tty 2>/dev/null && [ -t 0 ]; then
-      INTERACTIVE=1
-    else
-      INTERACTIVE=0
-    fi
-
-    if [ $INTERACTIVE -eq 0 ]; then
-      echo "ERROR: No controlling terminal available for interactive prompts."
-      echo ""
-      echo "For interactive installation (RECOMMENDED):"
-      echo "  curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh -o nupst-install.sh"
-      echo "  sudo bash nupst-install.sh"
-      echo ""
-      echo "For non-interactive installation with auto-confirm:"
-      echo "  curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y"
-      exit 1
-    else
-      echo "Interactive terminal found, continuing with prompts..."
-    fi
-  fi
-fi
-
 # Helper function to detect OS and architecture
 detect_platform() {
   local os=$(uname -s)
@@ -229,22 +182,6 @@ if [ -d "$INSTALL_DIR" ]; then
     echo ""
   fi
 
-  if [ $AUTO_YES -eq 0 ] && [ $INTERACTIVE -eq 1 ]; then
-    if [ $OLD_NODE_INSTALL -eq 1 ]; then
-      echo "This will replace your Node.js installation with a pre-compiled binary."
-      echo "Your configuration in /etc/nupst/config.json will be preserved."
-      echo ""
-    fi
-    echo "Installation directory already exists: $INSTALL_DIR"
-    echo "Do you want to update/reinstall? (Y/n): "
-    read -r update_confirm
-
-    if [[ "$update_confirm" =~ ^[Nn]$ ]]; then
-      echo "Installation cancelled."
-      exit 0
-    fi
-  fi
-
   echo "Updating existing installation at $INSTALL_DIR..."
 
   # Check if service exists (enabled or running) and stop it if active
@@ -273,17 +210,6 @@ if [ -d "$INSTALL_DIR" ]; then
     echo "Old installation files removed."
   fi
 else
-  if [ $AUTO_YES -eq 0 ] && [ $INTERACTIVE -eq 1 ]; then
-    echo "NUPST will be installed to: $INSTALL_DIR"
-    echo "Continue? (Y/n): "
-    read -r install_confirm
-
-    if [[ "$install_confirm" =~ ^[Nn]$ ]]; then
-      echo "Installation cancelled."
-      exit 0
-    fi
-  fi
-
   echo "Creating installation directory: $INSTALL_DIR"
   mkdir -p "$INSTALL_DIR"
 fi
@@ -329,22 +255,8 @@ else
 fi
 
 # Create symlink for global access
-if [ $AUTO_YES -eq 0 ] && [ $INTERACTIVE -eq 1 ]; then
-  echo "Create symlink in $BIN_DIR for global access? (Y/n): "
-  read -r symlink_confirm
-
-  if [[ ! "$symlink_confirm" =~ ^[Nn]$ ]]; then
 ln -sf "$BINARY_PATH" "$BIN_DIR/nupst"
 echo "Symlink created: $BIN_DIR/nupst -> $BINARY_PATH"
-  else
-    echo "Symlink creation skipped."
-    echo "To use NUPST, run: $BINARY_PATH"
-    echo "Or manually create symlink: sudo ln -sf $BINARY_PATH $BIN_DIR/nupst"
-  fi
-else
-  ln -sf "$BINARY_PATH" "$BIN_DIR/nupst"
-  echo "Symlink created: $BIN_DIR/nupst -> $BINARY_PATH"
-fi
 
 echo ""
 

readme.md

@@ -29,15 +29,8 @@ dependencies.
 The easiest way to install NUPST is using the automated installer:
 
 ```bash
-# Download and run installer (most reliable)
-curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh -o nupst-install.sh
-sudo bash nupst-install.sh
-rm nupst-install.sh
-```
-
-```bash
-# One-line installation (non-interactive with auto-confirm)
-curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y
+# One-line installation
+curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash
 ```
 
 The installer will:
@@ -76,7 +69,6 @@ sudo mv nupst /usr/local/bin/nupst
 The installer script (`install.sh`) supports the following options:
 
 ```
--y, --yes                Automatically answer yes to all prompts
 -h, --help               Show help message
 --version VERSION        Install specific version (e.g., --version v4.0.0)
 --install-dir DIR        Custom installation directory (default: /opt/nupst)
@@ -373,7 +365,7 @@ sudo nupst service disable
 Re-run the installer to update to the latest version:
 
 ```bash
-curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y
+curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash
 ```
 
 The installer will:
@@ -461,7 +453,7 @@ The installer script automatically handles the entire migration while preserving
 
 ```bash
 # Run the installer (handles stop/update/restart automatically)
-curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash -s -- -y
+curl -sSL https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh | sudo bash
 
 # Verify
 nupst service status

ts/cli/service-handler.ts

@@ -129,81 +129,53 @@ export class ServiceHandler {
     try {
       // Check if running as root
       this.checkRootAccess(
-        'This command must be run as root to update NUPST and refresh the systemd service.',
+        'This command must be run as root to update NUPST.',
       );
 
-      const boxWidth = 45;
-      logger.logBoxTitle('NUPST Update Process', boxWidth);
-      logger.logBoxLine('Updating NUPST from repository...');
-
-      // Determine the installation directory (assuming it's either /opt/nupst or the current directory)
-      const { existsSync } = await import('fs');
-      let installDir = '/opt/nupst';
-
-      if (!existsSync(installDir)) {
-        // If not installed in /opt/nupst, use the current directory
-        const { dirname } = await import('path');
-        installDir = dirname(dirname(process.argv[1])); // Go up two levels from the executable
-        logger.logBoxLine(`Using local installation directory: ${installDir}`);
-      }
+      console.log('');
+      logger.info('Checking for updates...');
 
       try {
-        // 1. Update the repository
-        logger.logBoxLine('Pulling latest changes from git repository...');
-        execSync(`cd ${installDir} && git fetch origin && git reset --hard origin/main`, {
-          stdio: 'pipe',
+        // Get current version
+        const currentVersion = this.nupst.getVersion();
+
+        // Fetch latest version from Gitea API
+        const apiUrl = 'https://code.foss.global/api/v1/repos/serve.zone/nupst/releases/latest';
+        const response = execSync(`curl -sSL ${apiUrl}`).toString();
+        const release = JSON.parse(response);
+        const latestVersion = release.tag_name; // e.g., "v4.0.7"
+
+        logger.dim(`Current version: ${currentVersion}`);
+        logger.dim(`Latest version:  ${latestVersion}`);
+        console.log('');
+
+        // Compare versions (both are in format "v4.0.7")
+        if (currentVersion === latestVersion) {
+          logger.success('Already up to date!');
+          console.log('');
+          return;
+        }
+
+        logger.info(`New version available: ${latestVersion}`);
+        logger.dim('Downloading and installing...');
+        console.log('');
+
+        // Download and run the install script
+        // This handles everything: download binary, stop service, replace, restart
+        const installUrl = 'https://code.foss.global/serve.zone/nupst/raw/branch/main/install.sh';
+
+        execSync(`curl -sSL ${installUrl} | bash`, {
+          stdio: 'inherit', // Show install script output to user
         });
 
-        // 2. Run the install.sh script
-        logger.logBoxLine('Running install.sh to update NUPST...');
-        execSync(`cd ${installDir} && bash ./install.sh`, { stdio: 'pipe' });
-
-        // 3. Run the setup.sh script with force flag to update Node.js and dependencies
-        logger.logBoxLine('Running setup.sh to update Node.js and dependencies...');
-        execSync(`cd ${installDir} && bash ./setup.sh --force`, { stdio: 'pipe' });
-
-        // 4. Refresh the systemd service
-        logger.logBoxLine('Refreshing systemd service...');
-
-        // First check if service exists
-        let serviceExists = false;
-        try {
-          const output = execSync('systemctl list-unit-files | grep nupst.service').toString();
-          serviceExists = output.includes('nupst.service');
+        console.log('');
+        logger.success(`Updated to ${latestVersion}`);
+        console.log('');
       } catch (error) {
-          // If grep fails (service not found), serviceExists remains false
-          serviceExists = false;
-        }
-
-        if (serviceExists) {
-          // Stop the service if it's running
-          const isRunning =
-            execSync('systemctl is-active nupst.service || true').toString().trim() === 'active';
-          if (isRunning) {
-            logger.logBoxLine('Stopping nupst service...');
-            execSync('systemctl stop nupst.service');
-          }
-
-          // Reinstall the service
-          logger.logBoxLine('Reinstalling systemd service...');
-          await this.nupst.getSystemd().install();
-
-          // Restart the service if it was running
-          if (isRunning) {
-            logger.logBoxLine('Restarting nupst service...');
-            execSync('systemctl start nupst.service');
-          }
-        } else {
-          logger.logBoxLine('Systemd service not installed, skipping service refresh.');
-          logger.logBoxLine('Run "nupst enable" to install the service.');
-        }
-
-        logger.logBoxLine('Update completed successfully!');
-        logger.logBoxEnd();
-      } catch (error) {
-        logger.logBoxLine('Error during update process:');
-        logger.logBoxLine(`${error instanceof Error ? error.message : String(error)}`);
-        logger.logBoxEnd();
+        console.log('');
+        logger.error('Update failed');
+        logger.dim(`${error instanceof Error ? error.message : String(error)}`);
+        console.log('');
         process.exit(1);
       }
     } catch (error) {

ts/migrations/migration-v3-to-v4.ts

@@ -44,17 +44,30 @@ export class MigrationV3ToV4 extends BaseMigration {
   readonly toVersion = '4.0';
 
   async shouldRun(config: any): Promise<boolean> {
-    // V3 format has upsList instead of upsDevices
-    return !!config.upsList && !config.upsDevices;
+    // V3 format has upsList OR has upsDevices with flat structure (host at top level)
+    if (config.upsList && !config.upsDevices) {
+      return true; // Classic v3 with upsList
+    }
+
+    // Check if upsDevices exists but has flat structure (v3 format)
+    if (config.upsDevices && config.upsDevices.length > 0) {
+      const firstDevice = config.upsDevices[0];
+      // V3 has host at top level, v4 has it nested in snmp object
+      return !!firstDevice.host && !firstDevice.snmp;
+    }
+
+    return false;
   }
 
   async migrate(config: any): Promise<any> {
     logger.info(`${this.getName()}: Migrating v3 config to v4 format...`);
-    logger.dim(`  - Renaming upsList → upsDevices`);
     logger.dim(`  - Restructuring UPS devices (flat → nested snmp config)`);
 
+    // Get devices from either upsList or upsDevices (for partially migrated configs)
+    const sourceDevices = config.upsList || config.upsDevices;
+
     // Transform each UPS device from v3 flat structure to v4 nested structure
-    const transformedDevices = config.upsList.map((device: any) => {
+    const transformedDevices = sourceDevices.map((device: any) => {
       // Build SNMP config object
       const snmpConfig: any = {
         host: device.host,

ts/systemd.ts

@@ -3,6 +3,7 @@ import { promises as fs } from 'node:fs';
 import { execSync } from 'node:child_process';
 import { NupstDaemon } from './daemon.ts';
 import { logger } from './logger.ts';
+import { theme, symbols, getBatteryColor, getRuntimeColor, formatPowerStatus } from './colors.ts';
 
 /**
  * Class for managing systemd service
@@ -171,18 +172,50 @@ WantedBy=multi-user.target
   private displayServiceStatus(): void {
     try {
       const serviceStatus = execSync('systemctl status nupst.service').toString();
-      const boxWidth = 45;
-      logger.logBoxTitle('Service Status', boxWidth);
-      // Process each line of the status output
-      serviceStatus.split('\n').forEach((line) => {
-        logger.logBoxLine(line);
-      });
-      logger.logBoxEnd();
+      const lines = serviceStatus.split('\n');
+
+      // Parse key information from systemctl output
+      let isActive = false;
+      let pid = '';
+      let memory = '';
+      let cpu = '';
+
+      for (const line of lines) {
+        if (line.includes('Active:')) {
+          isActive = line.includes('active (running)');
+        } else if (line.includes('Main PID:')) {
+          const match = line.match(/Main PID:\s+(\d+)/);
+          if (match) pid = match[1];
+        } else if (line.includes('Memory:')) {
+          const match = line.match(/Memory:\s+([\d.]+[A-Z])/);
+          if (match) memory = match[1];
+        } else if (line.includes('CPU:')) {
+          const match = line.match(/CPU:\s+([\d.]+(?:ms|s))/);
+          if (match) cpu = match[1];
+        }
+      }
+
+      // Display beautiful status
+      console.log('');
+      if (isActive) {
+        console.log(`${symbols.running} ${theme.success('Service:')} ${theme.statusActive('active (running)')}`);
+      } else {
+        console.log(`${symbols.stopped} ${theme.dim('Service:')} ${theme.statusInactive('inactive')}`);
+      }
+
+      if (pid || memory || cpu) {
+        const details = [];
+        if (pid) details.push(`PID: ${theme.dim(pid)}`);
+        if (memory) details.push(`Memory: ${theme.dim(memory)}`);
+        if (cpu) details.push(`CPU: ${theme.dim(cpu)}`);
+        console.log(`  ${details.join('  ')}`);
+      }
+      console.log('');
+
     } catch (error) {
-      const boxWidth = 45;
-      logger.logBoxTitle('Service Status', boxWidth);
-      logger.logBoxLine('Service is not running');
-      logger.logBoxEnd();
+      console.log('');
+      console.log(`${symbols.stopped} ${theme.dim('Service:')} ${theme.statusInactive('not installed')}`);
+      console.log('');
     }
   }
 
@@ -199,7 +232,7 @@ WantedBy=multi-user.target
 
       // Check if we have the new multi-UPS config format
       if (config.upsDevices && Array.isArray(config.upsDevices) && config.upsDevices.length > 0) {
-        logger.log(`Found ${config.upsDevices.length} UPS device(s) in configuration`);
+        console.log(theme.info(`UPS Devices (${config.upsDevices.length}):`));
 
         // Show status for each UPS
         for (const ups of config.upsDevices) {
@@ -207,6 +240,7 @@ WantedBy=multi-user.target
         }
       } else if (config.snmp) {
         // Legacy single UPS configuration
+        console.log(theme.info('UPS Devices (1):'));
         const legacyUps = {
           id: 'default',
           name: 'Default UPS',
@@ -217,15 +251,16 @@ WantedBy=multi-user.target
 
         await this.displaySingleUpsStatus(legacyUps, snmp);
       } else {
-        logger.error('No UPS devices found in configuration');
+        console.log('');
+        console.log(`${symbols.warning} ${theme.warning('No UPS devices configured')}`);
+        console.log(`  ${theme.dim('Run')} ${theme.command('nupst ups add')} ${theme.dim('to add a device')}`);
+        console.log('');
       }
     } catch (error) {
-      const boxWidth = 45;
-      logger.logBoxTitle('UPS Status', boxWidth);
-      logger.logBoxLine(
-        `Failed to retrieve UPS status: ${error instanceof Error ? error.message : String(error)}`,
-      );
-      logger.logBoxEnd();
+      console.log('');
+      console.log(`${symbols.error} ${theme.error('Failed to retrieve UPS status')}`);
+      console.log(`  ${theme.dim(error instanceof Error ? error.message : String(error))}`);
+      console.log('');
     }
   }
 
@@ -235,24 +270,6 @@ WantedBy=multi-user.target
    * @param snmp SNMP manager
    */
   private async displaySingleUpsStatus(ups: any, snmp: any): Promise<void> {
-    const boxWidth = 45;
-    logger.logBoxTitle(`Connecting to UPS: ${ups.name}`, boxWidth);
-    logger.logBoxLine(`ID: ${ups.id}`);
-    logger.logBoxLine(`Host: ${ups.snmp.host}:${ups.snmp.port}`);
-    logger.logBoxLine(`UPS Model: ${ups.snmp.upsModel || 'cyberpower'}`);
-
-    if (ups.groups && ups.groups.length > 0) {
-      // Get group names if available
-      const config = this.daemon.getConfig();
-      const groupNames = ups.groups.map((groupId: string) => {
-        const group = config.groups?.find((g: { id: string }) => g.id === groupId);
-        return group ? group.name : groupId;
-      });
-      logger.logBoxLine(`Groups: ${groupNames.join(', ')}`);
-    }
-
-    logger.logBoxEnd();
-
     try {
       // Create a test config with a short timeout
       const testConfig = {
@@ -262,32 +279,43 @@ WantedBy=multi-user.target
 
       const status = await snmp.getUpsStatus(testConfig);
 
-      logger.logBoxTitle(`UPS Status: ${ups.name}`, boxWidth);
-      logger.logBoxLine(`Power Status: ${status.powerStatus}`);
-      logger.logBoxLine(`Battery Capacity: ${status.batteryCapacity}%`);
-      logger.logBoxLine(`Runtime Remaining: ${status.batteryRuntime} minutes`);
+      // Determine status symbol based on power status
+      let statusSymbol = symbols.unknown;
+      if (status.powerStatus === 'online') {
+        statusSymbol = symbols.running;
+      } else if (status.powerStatus === 'onBattery') {
+        statusSymbol = symbols.warning;
+      }
 
-      // Show threshold status
-      logger.logBoxLine('');
-      logger.logBoxLine('Thresholds:');
-      logger.logBoxLine(
-        `  Battery: ${status.batteryCapacity}% / ${ups.thresholds.battery}% ${
-          status.batteryCapacity < ups.thresholds.battery ? '⚠️' : '✓'
-        }`,
-      );
-      logger.logBoxLine(
-        `  Runtime: ${status.batteryRuntime} min / ${ups.thresholds.runtime} min ${
-          status.batteryRuntime < ups.thresholds.runtime ? '⚠️' : '✓'
-        }`,
-      );
+      // Display UPS name and power status
+      console.log(`  ${statusSymbol} ${theme.highlight(ups.name)} - ${formatPowerStatus(status.powerStatus)}`);
+
+      // Display battery with color coding
+      const batteryColor = getBatteryColor(status.batteryCapacity);
+      const batterySymbol = status.batteryCapacity >= ups.thresholds.battery ? symbols.success : symbols.warning;
+      console.log(`    Battery: ${batteryColor(status.batteryCapacity + '%')} ${batterySymbol}  Runtime: ${getRuntimeColor(status.batteryRuntime)(status.batteryRuntime + ' min')}`);
+
+      // Display host info
+      console.log(`    ${theme.dim(`Host: ${ups.snmp.host}:${ups.snmp.port}`)}`);
+
+      // Display groups if any
+      if (ups.groups && ups.groups.length > 0) {
+        const config = this.daemon.getConfig();
+        const groupNames = ups.groups.map((groupId: string) => {
+          const group = config.groups?.find((g: { id: string }) => g.id === groupId);
+          return group ? group.name : groupId;
+        });
+        console.log(`    ${theme.dim(`Groups: ${groupNames.join(', ')}`)}`);
+      }
+
+      console.log('');
 
-      logger.logBoxEnd();
     } catch (error) {
-      logger.logBoxTitle(`UPS Status: ${ups.name}`, boxWidth);
-      logger.logBoxLine(
-        `Failed to retrieve UPS status: ${error instanceof Error ? error.message : String(error)}`,
-      );
-      logger.logBoxEnd();
+      // Display error for this UPS
+      console.log(`  ${symbols.error} ${theme.highlight(ups.name)} - ${theme.error('Connection failed')}`);
+      console.log(`    ${theme.dim(error instanceof Error ? error.message : String(error))}`);
+      console.log(`    ${theme.dim(`Host: ${ups.snmp.host}:${ups.snmp.port}`)}`);
+      console.log('');
     }
   }