@pi-unipi/updater 3.0.0-alpha.39 → 3.0.0-alpha.40

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/updater",
-  "version": "3.0.0-alpha.39",
+  "version": "3.0.0-alpha.40",
   "description": "Auto-updater, changelog browser, and readme browser for Unipi — checks npm registry, renders CHANGELOG.md and README.md files in TUI overlays",
   "type": "module",
   "main": "index.ts",
@@ -34,7 +34,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "3.0.0-alpha.39"
+    "@pi-unipi/core": "3.0.0-alpha.40"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.87.1",

package/src/channel.ts

@@ -81,6 +81,19 @@ export function pickChannelVersion(distTags: Record<string, string> | undefined,
   return candidates.reduce((best, v) => (compareVersions(v, best) > 0 ? v : best));
 }
 
+/**
+ * UNI-272: which dist-tag the picked version came from. An alpha install
+ * offered 3.0.0 from `latest` (graduation) must install `latest` — installing
+ * `@alpha` again would reinstall the alpha it already has, and keep the
+ * install on the alpha channel forever. Ties go to the install's own channel.
+ */
+export function pickInstallTag(distTags: Record<string, string> | undefined, channel: string): string {
+  const picked = pickChannelVersion(distTags, channel);
+  if (!picked || !distTags) return channel;
+  if (distTags[channel] === picked) return channel;
+  return distTags.latest === picked ? "latest" : channel;
+}
+
 /** The `pi install` source for this channel. */
 export function installSpec(channel: string): string {
   return channel === "latest" ? `npm:${UNIPI_PACKAGE}` : `npm:${UNIPI_PACKAGE}@${channel}`;

package/src/checker.ts

@@ -12,7 +12,7 @@ import { getInstalledPackageVersion } from "@pi-unipi/core";
 import { loadConfig } from "./settings.js";
 import { readLastCheck, writeLastCheck, isCheckDue } from "./cache.js";
 import { compareVersions, isNewerVersion } from "./version.js";
-import { pickChannelVersion, readUnipiPackageSpec, resolveChannel, UNIPI_PACKAGE } from "./channel.js";
+import { pickChannelVersion, pickInstallTag, readUnipiPackageSpec, resolveChannel, UNIPI_PACKAGE } from "./channel.js";
 import type { UpdateCheckResult } from "../types.js";
 
 /** npm registry base; `UNIPI_UPDATER_REGISTRY` overrides it (tests / mirrors). */
@@ -34,12 +34,13 @@ export function currentChannel(currentVersion = getInstalledVersion()): string {
 }
 
 /** Build an update result without ever reporting downgrades as updates. */
-function toUpdateResult(latestVersion: string, currentVersion: string, channel: string): UpdateCheckResult {
+function toUpdateResult(latestVersion: string, currentVersion: string, channel: string, installTag?: string): UpdateCheckResult {
   return {
     updateAvailable: isNewerVersion(latestVersion, currentVersion),
     latestVersion,
     currentVersion,
     channel,
+    installTag: installTag ?? channel,
   };
 }
 
@@ -69,7 +70,7 @@ export async function checkForUpdates(opts: CheckOptions = {}): Promise<UpdateCh
     const cache = readLastCheck();
     if (!opts.force && cache && (cache.channel ?? "latest") === channel && !isCheckDue(config.checkIntervalMs)) {
       if (compareVersions(cache.latestVersion, currentVersion) >= 0) {
-        return toUpdateResult(cache.latestVersion, currentVersion, channel);
+        return toUpdateResult(cache.latestVersion, currentVersion, channel, cache.installTag);
       }
     }
 
@@ -87,6 +88,7 @@ export async function checkForUpdates(opts: CheckOptions = {}): Promise<UpdateCh
     // The dist-tags endpoint returns the map itself; the full packument nests it.
     const tags = (body["dist-tags"] ?? body) as Record<string, string>;
     const latestVersion = pickChannelVersion(tags, channel);
+    const installTag = pickInstallTag(tags, channel);
 
     if (!latestVersion) {
       throw new Error(`No dist-tag "${channel}" in npm response`);
@@ -96,10 +98,11 @@ export async function checkForUpdates(opts: CheckOptions = {}): Promise<UpdateCh
       lastCheck: new Date().toISOString(),
       latestVersion,
       channel,
+      installTag,
       skippedVersion: cache?.skippedVersion,
     });
 
-    return toUpdateResult(latestVersion, currentVersion, channel);
+    return toUpdateResult(latestVersion, currentVersion, channel, installTag);
   } catch (err: unknown) {
     // Network error — reuse cached info for this channel only, and never
     // suggest a downgrade from a stale cache.
@@ -110,6 +113,7 @@ export async function checkForUpdates(opts: CheckOptions = {}): Promise<UpdateCh
       latestVersion: usable?.latestVersion ?? "",
       currentVersion,
       channel,
+      installTag: usable?.installTag ?? channel,
       error: err instanceof Error ? err.message : String(err) || "Unknown error",
     };
   }

package/src/tui/update-overlay.ts

@@ -93,7 +93,8 @@ export function renderUpdateOverlay(checkResult: UpdateCheckResult, providedNewe
       state.installing = true;
       tui.requestRender();
 
-      const result = await installUpdate(state.result.channel ?? "latest");
+      // UNI-272: the tag the offered version came from (alpha → 3.0.0 stable installs `latest`).
+      const result = await installUpdate(state.result.installTag ?? state.result.channel ?? "latest");
       if (result.success) {
         done({ updated: true });
       } else {

package/types.ts

@@ -22,6 +22,8 @@ export interface LastCheckCache {
   skippedVersion?: string;
   /** npm dist-tag the check followed (absent in pre-UNI-262 caches = "latest") */
   channel?: string;
+  /** UNI-272: dist-tag to install the offered version from (≠ channel on graduation). */
+  installTag?: string;
 }
 
 /** A parsed changelog version entry */
@@ -58,6 +60,9 @@ export interface UpdateCheckResult {
   currentVersion: string;
   /** npm dist-tag this install follows ("alpha", "latest", …) */
   channel?: string;
+  /** UNI-272: the dist-tag the offered version comes from — install from this
+   * (an alpha install offered 3.0.0 stable installs `latest`, leaving alpha). */
+  installTag?: string;
   /** Error message if check failed */
   error?: string;
 }