@pi-unipi/updater 2.1.3 → 2.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/updater",
-  "version": "2.1.3",
+  "version": "2.2.1",
   "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": "src/index.ts",
@@ -33,7 +33,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.1.3"
+    "@pi-unipi/core": "2.2.0"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.0",

package/src/changelog.ts

@@ -6,8 +6,35 @@
  */
 
 import { existsSync, readFileSync } from "fs";
+import { dirname, join } from "node:path";
+import { fileURLToPath } from "node:url";
+import { findPackageRoot } from "@pi-unipi/core";
+import { isNewerVersion } from "./version.js";
 import type { ChangelogEntry } from "../types.js";
 
+/**
+ * Locate the shipped CHANGELOG.md.
+ *
+ * Resolving from `process.cwd()` only works when pi happens to be running
+ * inside the UniPi checkout — for everyone else the update prompt and
+ * `/unipi:changelog` came up empty. Resolve from this module's own location
+ * instead, and fall back to the working directory so a repo checkout still
+ * shows its local (possibly unreleased) notes.
+ */
+export function resolveChangelogPath(): string {
+  try {
+    const dir = dirname(fileURLToPath(import.meta.url));
+    const root = findPackageRoot(dir, "@pi-unipi/unipi");
+    if (root) {
+      const packaged = join(root, "CHANGELOG.md");
+      if (existsSync(packaged)) return packaged;
+    }
+  } catch {
+    // Fall through to the working directory.
+  }
+  return join(process.cwd(), "CHANGELOG.md");
+}
+
 /** Regex for version headers: ## [x.y.z] — YYYY-MM-DD or ## [Unreleased] */
 const VERSION_HEADER_RE = /^## \[(.+?)\](?:\s*[-—–]\s*(.+))?$/;
 
@@ -119,7 +146,11 @@ export function getNewerVersions(
       result.push(entry);
       continue;
     }
-    if (entry.version === installedVersion) break;
+    // Compare rather than test for equality. Stopping only on an exact match
+    // meant that when the installed version was absent from the changelog
+    // (a local build, a yanked release, or simply a newer version) every
+    // historical entry was reported as "new".
+    if (!isNewerVersion(entry.version, installedVersion)) break;
     result.push(entry);
   }
   return result;

package/src/tui/changelog-overlay.ts

@@ -9,9 +9,9 @@ import { existsSync } from "fs";
 import { join } from "path";
 import { Key, matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { Theme } from "@earendil-works/pi-coding-agent";
-import { parseChangelog } from "../changelog.js";
+import { parseChangelog, resolveChangelogPath } from "../changelog.js";
 import { renderMarkdown } from "../markdown.js";
-import { getInstalledPackageVersion } from "@pi-unipi/core";
+import { getInstalledPackageVersion, boxInnerWidth } from "@pi-unipi/core";
 import type { ChangelogEntry } from "../../types.js";
 
 type View = "list" | "detail";
@@ -62,7 +62,7 @@ export function renderChangelogOverlay() {
     let loaded = false;
     const ensureLoaded = () => {
       if (loaded) return;
-      const changelogPath = join(process.cwd(), "CHANGELOG.md");
+      const changelogPath = resolveChangelogPath();
       if (existsSync(changelogPath)) {
         state.entries = parseChangelog(changelogPath);
       }
@@ -72,7 +72,7 @@ export function renderChangelogOverlay() {
     const render = (width: number): string[] => {
       ensureLoaded();
 
-      const innerWidth = Math.max(22, width - 2);
+      const innerWidth = boxInnerWidth(width);
       const lines: string[] = [];
 
       // ── Header ──────────────────────────────────────────────────────

package/src/tui/readme-overlay.ts

@@ -12,6 +12,7 @@ import type { Theme } from "@earendil-works/pi-coding-agent";
 import { discoverReadmes, resolveReadmePath } from "../readme.js";
 import { renderMarkdown } from "../markdown.js";
 import type { ReadmeEntry } from "../../types.js";
+import { boxInnerWidth } from "@pi-unipi/core";
 
 type View = "list" | "content";
 
@@ -71,7 +72,7 @@ export function renderReadmeOverlay(params?: { openDirect?: string }) {
     const render = (width: number): string[] => {
       ensureLoaded();
 
-      const innerWidth = Math.max(22, width - 2);
+      const innerWidth = boxInnerWidth(width);
       const lines: string[] = [];
 
       // ── Header ──────────────────────────────────────────────────────

package/src/tui/update-overlay.ts

@@ -9,12 +9,13 @@
 import { join } from "path";
 import { Key, matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { Theme } from "@earendil-works/pi-coding-agent";
-import { parseChangelog, getNewerVersions } from "../changelog.js";
+import { parseChangelog, getNewerVersions, resolveChangelogPath } from "../changelog.js";
 import { renderMarkdown } from "../markdown.js";
 import { installUpdate } from "../installer.js";
 import { writeSkippedVersion } from "../cache.js";
 import { loadConfig } from "../settings.js";
 import type { ChangelogEntry, UpdateCheckResult } from "../../types.js";
+import { boxInnerWidth } from "@pi-unipi/core";
 
 /**
  * Pad content to exact visible width.
@@ -50,7 +51,7 @@ export function renderUpdateOverlay(checkResult: UpdateCheckResult) {
 
     // Load changelog for newer versions
     let newerVersions: ChangelogEntry[] = [];
-    const changelogPath = join(process.cwd(), "CHANGELOG.md");
+    const changelogPath = resolveChangelogPath();
     try {
       const entries = parseChangelog(changelogPath);
       newerVersions = getNewerVersions(entries, checkResult.currentVersion);
@@ -119,7 +120,7 @@ export function renderUpdateOverlay(checkResult: UpdateCheckResult) {
     };
 
     const render = (width: number): string[] => {
-      const innerWidth = Math.max(22, width - 2);
+      const innerWidth = boxInnerWidth(width);
       const lines: string[] = [];
 
       // ── Header ──────────────────────────────────────────────────────