@sylphx/flow 2.15.1 → 2.15.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @sylphx/flow
 
+## 2.15.2 (2025-12-17)
+
+### ⚡️ Performance
+
+- **auto-upgrade:** non-blocking version check with cache ([0550e44](https://github.com/SylphxAI/flow/commit/0550e44ea9b471ae07b2ea13c196354a7a32a605))
+
 ## 2.15.1 (2025-12-17)
 
 ### Improvements

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@sylphx/flow",
-	"version": "2.15.1",
+	"version": "2.15.2",
 	"description": "One CLI to rule them all. Unified orchestration layer for Claude Code, OpenCode, Cursor and all AI development tools. Auto-detection, auto-installation, auto-upgrade.",
 	"type": "module",
 	"bin": {

package/src/services/auto-upgrade.ts

@@ -1,10 +1,13 @@
 /**
  * Auto-Upgrade Service
- * Automatically checks and upgrades Flow and target CLI before each execution
+ * Non-blocking version check with cache
+ * Checks in background, shows result on next run
  */
 
 import { exec } from 'node:child_process';
+import { existsSync } from 'node:fs';
 import fs from 'node:fs/promises';
+import os from 'node:os';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { promisify } from 'node:util';
@@ -16,6 +19,16 @@ import { TargetInstaller } from './target-installer.js';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
+// Cache file for version checks (24 hour TTL)
+const CACHE_FILE = path.join(os.homedir(), '.sylphx-flow', 'version-cache.json');
+const CACHE_TTL_MS = 24 * 60 * 60 * 1000; // 24 hours
+
+interface VersionCache {
+  flowLatest?: string;
+  targetLatest?: Record<string, string>;
+  checkedAt: number;
+}
+
 const execAsync = promisify(exec);
 
 export interface UpgradeStatus {
@@ -43,80 +56,151 @@ export class AutoUpgrade {
   }
 
   /**
-   * Check for available upgrades for Flow and target CLI
-   * @param targetId - Optional target CLI ID to check for upgrades
-   * @returns Upgrade status indicating what needs upgrading
+   * Read version cache (instant, no network)
    */
-  async checkForUpgrades(targetId?: string): Promise<UpgradeStatus> {
-    const [flowVersion, targetVersion] = await Promise.all([
-      this.options.skipFlow ? null : this.checkFlowVersion(),
-      this.options.skipTarget || !targetId ? null : this.checkTargetVersion(targetId),
-    ]);
+  private async readCache(): Promise<VersionCache | null> {
+    try {
+      if (!existsSync(CACHE_FILE)) {
+        return null;
+      }
+      const data = await fs.readFile(CACHE_FILE, 'utf-8');
+      return JSON.parse(data);
+    } catch {
+      return null;
+    }
+  }
 
-    return {
-      flowNeedsUpgrade: !!flowVersion,
-      targetNeedsUpgrade: !!targetVersion,
-      flowVersion,
-      targetVersion,
-    };
+  /**
+   * Write version cache
+   */
+  private async writeCache(cache: VersionCache): Promise<void> {
+    try {
+      const dir = path.dirname(CACHE_FILE);
+      await fs.mkdir(dir, { recursive: true });
+      await fs.writeFile(CACHE_FILE, JSON.stringify(cache, null, 2));
+    } catch {
+      // Silent fail
+    }
   }
 
   /**
-   * Check Flow version
+   * Get current Flow version from package.json (instant, local file)
    */
-  private async checkFlowVersion(): Promise<{ current: string; latest: string } | null> {
+  private async getCurrentFlowVersion(): Promise<string> {
     try {
-      // Get current version from package.json
       const packageJsonPath = path.join(__dirname, '..', '..', 'package.json');
       const packageJson = JSON.parse(await fs.readFile(packageJsonPath, 'utf-8'));
-      const currentVersion = packageJson.version;
-
-      // Get latest version from npm
-      const { stdout } = await execAsync('npm view @sylphx/flow version');
-      const latestVersion = stdout.trim();
-
-      if (currentVersion !== latestVersion) {
-        return { current: currentVersion, latest: latestVersion };
-      }
-
-      return null;
+      return packageJson.version;
     } catch {
-      return null;
+      return 'unknown';
     }
   }
 
   /**
-   * Check target CLI version
+   * Check for available upgrades using CACHE (instant, no network)
+   * Returns cached results from previous background check
    */
-  private async checkTargetVersion(
-    targetId: string
-  ): Promise<{ current: string; latest: string } | null> {
-    const installation = this.targetInstaller.getInstallationInfo(targetId);
-    if (!installation) {
-      return null;
+  async checkForUpgrades(targetId?: string): Promise<UpgradeStatus> {
+    const cache = await this.readCache();
+    const currentVersion = await this.getCurrentFlowVersion();
+
+    // Trigger background check for next run (non-blocking)
+    this.checkInBackground(targetId);
+
+    // No cache or expired = no upgrade info yet
+    if (!cache) {
+      return {
+        flowNeedsUpgrade: false,
+        targetNeedsUpgrade: false,
+        flowVersion: null,
+        targetVersion: null,
+      };
     }
 
-    try {
-      // Get current version
-      const { stdout: currentOutput } = await execAsync(installation.checkCommand);
-      const currentMatch = currentOutput.match(/v?(\d+\.\d+\.\d+)/);
-      if (!currentMatch) {
-        return null;
+    // Check if Flow needs upgrade based on cache
+    const flowVersion =
+      cache.flowLatest && cache.flowLatest !== currentVersion
+        ? { current: currentVersion, latest: cache.flowLatest }
+        : null;
+
+    // Check if target needs upgrade based on cache
+    let targetVersion: { current: string; latest: string } | null = null;
+    if (targetId && cache.targetLatest?.[targetId]) {
+      const installation = this.targetInstaller.getInstallationInfo(targetId);
+      if (installation) {
+        try {
+          const { stdout } = await execAsync(installation.checkCommand);
+          const match = stdout.match(/v?(\d+\.\d+\.\d+)/);
+          if (match && match[1] !== cache.targetLatest[targetId]) {
+            targetVersion = { current: match[1], latest: cache.targetLatest[targetId] };
+          }
+        } catch {
+          // Silent
+        }
       }
-      const currentVersion = currentMatch[1];
+    }
 
-      // Get latest version from npm
-      const { stdout: latestOutput } = await execAsync(`npm view ${installation.package} version`);
-      const latestVersion = latestOutput.trim();
+    return {
+      flowNeedsUpgrade: !!flowVersion,
+      targetNeedsUpgrade: !!targetVersion,
+      flowVersion,
+      targetVersion,
+    };
+  }
 
-      if (currentVersion !== latestVersion) {
-        return { current: currentVersion, latest: latestVersion };
-      }
+  /**
+   * Check versions in background (non-blocking)
+   * Updates cache for next run
+   */
+  private checkInBackground(targetId?: string): void {
+    // Fire and forget - don't await
+    this.performBackgroundCheck(targetId).catch(() => {
+      // Silent fail
+    });
+  }
 
-      return null;
+  /**
+   * Perform the actual version check (called in background)
+   */
+  private async performBackgroundCheck(targetId?: string): Promise<void> {
+    const cache = await this.readCache();
+
+    // Skip if checked recently (within TTL)
+    if (cache && Date.now() - cache.checkedAt < CACHE_TTL_MS) {
+      return;
+    }
+
+    const newCache: VersionCache = {
+      checkedAt: Date.now(),
+      targetLatest: cache?.targetLatest || {},
+    };
+
+    // Check Flow version from npm (with timeout)
+    try {
+      const { stdout } = await execAsync('npm view @sylphx/flow version', { timeout: 5000 });
+      newCache.flowLatest = stdout.trim();
     } catch {
-      return null;
+      // Keep old cache value if check fails
+      newCache.flowLatest = cache?.flowLatest;
+    }
+
+    // Check target version from npm (with timeout)
+    if (targetId) {
+      const installation = this.targetInstaller.getInstallationInfo(targetId);
+      if (installation) {
+        try {
+          const { stdout } = await execAsync(`npm view ${installation.package} version`, {
+            timeout: 5000,
+          });
+          newCache.targetLatest = newCache.targetLatest || {};
+          newCache.targetLatest[targetId] = stdout.trim();
+        } catch {
+          // Keep old cache value
+        }
+      }
     }
+
+    await this.writeCache(newCache);
   }
 
   /**