pi-extmgr 0.1.14 → 0.1.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-extmgr",
-  "version": "0.1.14",
+  "version": "0.1.15",
   "description": "Enhanced UX for managing local Pi extensions and community packages",
   "keywords": [
     "pi-package",

package/src/packages/install.ts

@@ -8,6 +8,7 @@ import type { ExtensionAPI, ExtensionCommandContext } from "@mariozechner/pi-cod
 import { normalizePackageSource } from "../utils/format.js";
 import { clearSearchCache } from "./discovery.js";
 import { logPackageInstall } from "../utils/history.js";
+import { clearUpdatesAvailable } from "../utils/settings.js";
 import { notify, error as notifyError, success } from "../utils/notify.js";
 import { confirmAction, confirmReload, showProgress } from "../utils/ui-helpers.js";
 import { tryOperation } from "../utils/mode.js";
@@ -126,6 +127,7 @@ export async function installPackage(
   clearSearchCache();
   logPackageInstall(pi, normalized, normalized, undefined, scope, true);
   success(ctx, `Installed ${normalized} (${scope})`);
+  clearUpdatesAvailable(pi, ctx);
 
   void updateExtmgrStatus(ctx, pi);
   await confirmReload(ctx, "Package installed.");
@@ -186,6 +188,7 @@ export async function installFromUrl(
   const { fileName: name, destPath } = result;
   logPackageInstall(pi, url, name, undefined, scope, true);
   success(ctx, `Installed ${name} to:\n${destPath}`);
+  clearUpdatesAvailable(pi, ctx);
   void updateExtmgrStatus(ctx, pi);
   await confirmReload(ctx, "Extension installed.");
 }
@@ -401,6 +404,7 @@ export async function installPackageLocally(
   clearSearchCache();
   logPackageInstall(pi, `npm:${packageName}`, packageName, version, scope, true);
   success(ctx, `Installed ${packageName}@${version} locally to:\n${destResult}/index.ts`);
+  clearUpdatesAvailable(pi, ctx);
   void updateExtmgrStatus(ctx, pi);
   await confirmReload(ctx, "Extension installed.");
 }

package/src/packages/management.ts

@@ -11,6 +11,7 @@ import {
 import { formatInstalledPackageLabel, formatBytes, parseNpmSource } from "../utils/format.js";
 import { getPackageSourceKind, splitGitRepoAndRef } from "../utils/package-source.js";
 import { logPackageUpdate, logPackageRemove } from "../utils/history.js";
+import { clearUpdatesAvailable } from "../utils/settings.js";
 import { notify, error as notifyError, success } from "../utils/notify.js";
 import {
   confirmAction,
@@ -69,6 +70,7 @@ async function updatePackageInternal(
 
   logPackageUpdate(pi, source, source, undefined, true);
   success(ctx, `Updated ${source}`);
+  clearUpdatesAvailable(pi, ctx);
   void updateExtmgrStatus(ctx, pi);
 
   const reloaded = await confirmReload(ctx, "Package updated.");
@@ -97,6 +99,7 @@ async function updatePackagesInternal(
   }
 
   success(ctx, "Packages updated");
+  clearUpdatesAvailable(pi, ctx);
   void updateExtmgrStatus(ctx, pi);
 
   const reloaded = await confirmReload(ctx, "Packages updated.");
@@ -328,6 +331,9 @@ async function removePackageInternal(
   );
   notifyRemovalSummary(source, remaining, failures, ctx);
 
+  if (failures.length === 0) {
+    clearUpdatesAvailable(pi, ctx);
+  }
   void updateExtmgrStatus(ctx, pi);
 
   const restartRequested = await confirmRestart(

package/src/utils/settings.ts

@@ -265,6 +265,23 @@ export function saveAutoUpdateConfig(pi: ExtensionAPI, config: Partial<AutoUpdat
   enqueueConfigWrite(fullConfig);
 }
 
+/**
+ * Clear the updates available list after package mutations.
+ * Call this after install/update/remove to prevent stale update notifications.
+ */
+export function clearUpdatesAvailable(
+  pi: ExtensionAPI,
+  ctx: ExtensionCommandContext | ExtensionContext
+): void {
+  const config = getAutoUpdateConfig(ctx);
+  if (config.updatesAvailable && config.updatesAvailable.length > 0) {
+    saveAutoUpdateConfig(pi, {
+      ...config,
+      updatesAvailable: [],
+    });
+  }
+}
+
 /**
  * Parse duration string to milliseconds
  * Supports: 1h, 2h, 1d, 7d, 1m, 3m, etc.

package/src/utils/status.ts

@@ -8,7 +8,15 @@ import type {
 } from "@mariozechner/pi-coding-agent";
 import { getInstalledPackages } from "../packages/discovery.js";
 import { getAutoUpdateStatus } from "./auto-update.js";
-import { getAutoUpdateConfigAsync } from "./settings.js";
+import { getAutoUpdateConfigAsync, saveAutoUpdateConfig } from "./settings.js";
+
+function filterStaleUpdates(
+  knownUpdates: string[],
+  installedPackages: Awaited<ReturnType<typeof getInstalledPackages>>
+): string[] {
+  const installedNames = new Set(installedPackages.map((p) => p.name));
+  return knownUpdates.filter((name) => installedNames.has(name));
+}
 
 export async function updateExtmgrStatus(
   ctx: ExtensionCommandContext | ExtensionContext,
@@ -32,9 +40,20 @@ export async function updateExtmgrStatus(
       statusParts.push(autoUpdateStatus);
     }
 
+    // Validate updates against actually installed packages (handles external pi update)
     const knownUpdates = autoUpdateConfig.updatesAvailable ?? [];
-    if (knownUpdates.length > 0) {
-      statusParts.push(`${knownUpdates.length} update${knownUpdates.length === 1 ? "" : "s"}`);
+    const validUpdates = filterStaleUpdates(knownUpdates, packages);
+
+    // If stale updates were filtered, persist the correction
+    if (validUpdates.length !== knownUpdates.length) {
+      saveAutoUpdateConfig(pi, {
+        ...autoUpdateConfig,
+        updatesAvailable: validUpdates,
+      });
+    }
+
+    if (validUpdates.length > 0) {
+      statusParts.push(`${validUpdates.length} update${validUpdates.length === 1 ? "" : "s"}`);
     }
 
     if (statusParts.length > 0) {