@agile-vibe-coding/avc 0.1.1 → 0.3.1

package/cli/update-checker.js

@@ -1,4 +1,4 @@
-import { execSync } from 'child_process';
+import { exec } from 'child_process';
 import { readFileSync, writeFileSync, existsSync, mkdirSync } from 'fs';
 import path from 'path';
 import { fileURLToPath } from 'url';
@@ -71,28 +71,30 @@ export class UpdateChecker {
       return packageJson.version;
     } catch (error) {
       updateLogger.error('Failed to read current version', error);
-      console.error('Failed to read current version:', error.message);
+      // Don't use console.error - it interferes with React Ink rendering during startup
       return null;
     }
   }
 
-  // Check npm registry for latest version
+  // Check npm registry for latest version (truly async — does not block the event loop)
   async getLatestVersion() {
-    try {
+    return new Promise((resolve) => {
       updateLogger.debug('Checking npm registry for latest version');
-      const result = execSync(`npm view ${this.packageName} version`, {
-        encoding: 'utf8',
-        timeout: 10000,
-        stdio: ['pipe', 'pipe', 'ignore'] // Suppress stderr
-      });
-      const version = result.trim();
-      updateLogger.debug(`Latest version on npm: ${version}`);
-      return version;
-    } catch (error) {
-      updateLogger.error('Failed to check npm registry', error);
-      // Silently fail - will retry on next check
-      return null;
-    }
+      exec(
+        `npm view ${this.packageName} version`,
+        { timeout: 10000 },
+        (error, stdout) => {
+          if (error) {
+            updateLogger.error('Failed to check npm registry', error);
+            resolve(null);
+          } else {
+            const version = stdout.trim();
+            updateLogger.debug(`Latest version on npm: ${version}`);
+            resolve(version);
+          }
+        }
+      );
+    });
   }
 
   // Compare versions (returns true if remote is newer)

package/cli/update-notifier.js

@@ -28,7 +28,7 @@ export const UpdateNotification = ({ onDismiss }) => {
     },
       React.createElement(Box, { flexDirection: 'column' },
         React.createElement(Text, { bold: true, color: 'blue' },
-          `⬇️  Downloading update v${state.latestVersion}...`
+          `Downloading update v${state.latestVersion}...`
         ),
         React.createElement(Text, { dimColor: true },
           'This happens in the background. Continue working!'
@@ -48,7 +48,7 @@ export const UpdateNotification = ({ onDismiss }) => {
     },
       React.createElement(Box, { flexDirection: 'column' },
         React.createElement(Text, { bold: true, color: 'green' },
-          `✅ Update v${state.downloadedVersion} ready!`
+          `Update v${state.downloadedVersion} ready!`
         ),
         React.createElement(Text, null,
           'Restart to use the new version'
@@ -95,7 +95,7 @@ export const UpdateNotification = ({ onDismiss }) => {
     },
       React.createElement(Box, { flexDirection: 'column' },
         React.createElement(Text, { bold: true, color: 'red' },
-          '❌ Update failed'
+          'Update failed'
         ),
         React.createElement(Text, null,
           state.errorMessage || 'Unknown error'
@@ -126,7 +126,7 @@ export const UpdateNotification = ({ onDismiss }) => {
     },
       React.createElement(Box, { flexDirection: 'column' },
         React.createElement(Text, { bold: true, color: 'yellow' },
-          `📦 Update available: v${state.latestVersion}`
+          `Update available: v${state.latestVersion}`
         ),
         React.createElement(Text, null,
           'Installing in background...'