fix(@inquirerer/utils): use semver for version comparison and add clearUpdateCache()

Author: pyramation

packages/inquirerer-utils/package.json

@@ -29,10 +29,12 @@
   },
   "dependencies": {
     "appstash": "workspace:*",
-    "inquirerer": "workspace:*"
+    "inquirerer": "workspace:*",
+    "semver": "^7.7.4"
   },
   "devDependencies": {
     "@types/minimist": "^1.2.5",
+    "@types/semver": "^7.7.1",
     "makage": "0.1.10"
   },
   "keywords": []

packages/inquirerer-utils/src/index.ts

@@ -10,5 +10,5 @@ export {
 export type { ParsedArgs, ParseArgvOptions, CliExitOptions, PackageJson } from 'inquirerer';
 
 // Update checking (requires appstash, not available in inquirerer)
-export { checkForUpdates, shouldSkipUpdateCheck } from './update-check';
+export { checkForUpdates, clearUpdateCache, shouldSkipUpdateCheck } from './update-check';
 export type { UpdateCheckOptions, UpdateCheckResult } from './update-check';

packages/inquirerer-utils/src/update-check.ts

@@ -1,6 +1,7 @@
 import { appstash } from 'appstash';
 import * as fs from 'fs';
 import * as path from 'path';
+import semver from 'semver';
 
 export interface UpdateCheckOptions {
   pkgName: string;
@@ -66,9 +67,29 @@ export function shouldSkipUpdateCheck(
   return { skip: false };
 }
 
+const CACHE_FILENAME = 'update-check.json';
 const DEFAULT_TTL = 24 * 60 * 60 * 1000; // 24 hours
 const DEFAULT_REGISTRY = 'https://registry.npmjs.org';
 
+export function clearUpdateCache(toolName: string): boolean {
+  try {
+    const dirs = appstash(toolName);
+    const cacheFile = path.join(dirs.cache, CACHE_FILENAME);
+    if (fs.existsSync(cacheFile)) {
+      fs.unlinkSync(cacheFile);
+      return true;
+    }
+  } catch {}
+  return false;
+}
+
+function isNewerVersion(latest: string, current: string): boolean {
+  if (semver.valid(latest) && semver.valid(current)) {
+    return semver.gt(latest, current);
+  }
+  return latest !== current && latest > current;
+}
+
 /**
  * Checks for updates to a package and caches the result.
  * Uses appstash for configuration storage.
@@ -108,18 +129,18 @@ export const checkForUpdates = async (
   }
 
   const dirs = appstash(toolName);
-  const cacheFile = path.join(dirs.cache, 'update-check.json');
+  const cacheFile = path.join(dirs.cache, CACHE_FILENAME);
 
   // Check cache first
   try {
     if (fs.existsSync(cacheFile)) {
       const cached = JSON.parse(fs.readFileSync(cacheFile, 'utf-8'));
       if (Date.now() - cached.timestamp < ttl) {
         return {
-          hasUpdate: cached.latestVersion !== pkgVersion && cached.latestVersion > pkgVersion,
+          hasUpdate: isNewerVersion(cached.latestVersion, pkgVersion),
           currentVersion: pkgVersion,
           latestVersion: cached.latestVersion,
-          message: cached.latestVersion > pkgVersion
+          message: isNewerVersion(cached.latestVersion, pkgVersion)
             ? `Update available: ${pkgVersion} -> ${cached.latestVersion}`
             : null
         };
@@ -158,10 +179,10 @@ export const checkForUpdates = async (
     }
 
     return {
-      hasUpdate: latestVersion !== pkgVersion && latestVersion > pkgVersion,
+      hasUpdate: isNewerVersion(latestVersion, pkgVersion),
       currentVersion: pkgVersion,
       latestVersion,
-      message: latestVersion > pkgVersion
+      message: isNewerVersion(latestVersion, pkgVersion)
         ? `Update available: ${pkgVersion} -> ${latestVersion}`
         : null
     };