@oh-my-tool/cli 0.3.5 → 0.3.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@oh-my-tool/cli",
-  "version": "0.3.5",
+  "version": "0.3.6",
   "type": "module",
   "description": "Oh My Tool CLI - local and enterprise tools for agents",
   "keywords": [
@@ -29,7 +29,7 @@
   "dependencies": {
     "@clack/prompts": "^1.7.0",
     "@modelcontextprotocol/client": "2.0.0",
-    "@oh-my-tool/sdk": "0.3.5",
+    "@oh-my-tool/sdk": "0.3.6",
     "open": "11.0.0"
   },
   "engines": {

package/src/extension/discovery.ts

@@ -37,6 +37,7 @@ export function discoverExtensions(home: string): InstalledExtension[] {
   for (const id of readdirSync(root)) {
     const idDir = join(root, id);
     if (!statSync(idDir).isDirectory()) continue;
+    const candidates: InstalledExtension[] = [];
     for (const version of readdirSync(idDir)) {
       const versionDir = join(idDir, version);
       if (!statSync(versionDir).isDirectory()) continue;
@@ -46,7 +47,7 @@ export function discoverExtensions(home: string): InstalledExtension[] {
         const manifest = parseManifest(readFileSync(manifestPath, "utf8"));
         validateManifest(manifest);
         checkSdkCompatibility(manifest.sdkVersion);
-        out.push({
+        candidates.push({
           id,
           version,
           dir: versionDir,
@@ -57,6 +58,13 @@ export function discoverExtensions(home: string): InstalledExtension[] {
         // skip invalid or incompatible manifests
       }
     }
+    const active = candidates.reduce<InstalledExtension | undefined>((current, candidate) => {
+      if (current === undefined || Bun.semver.order(candidate.manifest.version, current.manifest.version) > 0) {
+        return candidate;
+      }
+      return current;
+    }, undefined);
+    if (active !== undefined) out.push(active);
   }
   return out;
-}
+}

package/src/extension/install.ts

@@ -1,4 +1,4 @@
-import { cp, mkdir, mkdtemp, readFile, rm, stat, writeFile } from "node:fs/promises";
+import { cp, mkdir, mkdtemp, readdir, readFile, rm, stat, writeFile } from "node:fs/promises";
 import { execFile as execFileCallback } from "node:child_process";
 import { promisify } from "node:util";
 import { tmpdir } from "node:os";
@@ -88,9 +88,39 @@ async function installExtensionDirectory(home: string, srcDir: string, expectedP
     force: true,
     filter: (s: string) => basename(s) !== "node_modules",
   });
+  await retireInactiveVersions(home, manifest.id, manifest.version);
   return { id: manifest.id, version: manifest.version, target };
 }
 
+async function retireInactiveVersions(home: string, id: string, activeVersion: string): Promise<void> {
+  const extensionRoot = join(home, "extensions", id);
+  let entries: string[];
+  try {
+    entries = await readdir(extensionRoot);
+  } catch (error) {
+    if (isMissingPathError(error)) return;
+    const message = error instanceof Error ? error.message : String(error);
+    throw new ExtensionInstallError(`failed to inspect installed versions for '${id}': ${message}`);
+  }
+
+  for (const entry of entries) {
+    if (entry === activeVersion || !EXACT_VERSION_RE.test(entry)) continue;
+    const entryPath = join(extensionRoot, entry);
+    try {
+      if ((await stat(entryPath)).isDirectory()) {
+        await rm(entryPath, { recursive: true, force: true });
+      }
+    } catch (error) {
+      const message = error instanceof Error ? error.message : String(error);
+      throw new ExtensionInstallError(`failed to retire inactive extension '${id}/${entry}': ${message}`);
+    }
+  }
+}
+
+function isMissingPathError(error: unknown): boolean {
+  return error instanceof Error && "code" in error && error.code === "ENOENT";
+}
+
 export async function installLocalExtension(home: string, srcDir: string): Promise<InstalledRef> {
   return installExtensionDirectory(home, srcDir);
 }

package/src/version.ts

@@ -1 +1 @@
-export const VERSION = "0.3.5";
+export const VERSION = "0.3.6";