jeopi-cli 16.2.18 → 16.2.20

package/dist/types/cli/update-cli.d.ts

@@ -1,3 +1,25 @@
+/**
+ * npm package that ships the CLI. Exported so the startup update notice
+ * (`checkForNewVersion` in main.ts) queries the same package this command
+ * installs — the two must never drift or the notice checks a phantom name.
+ */
+export declare const NPM_PACKAGE = "jeopi-cli";
+/**
+ * Official npm registry origin.
+ *
+ * Pinned across both the version check and the bun install step so the two
+ * agree on which catalog they are talking to. A user's bun may be pointed at
+ * an unofficial mirror (corporate proxy, Taobao, etc.) that lags the upstream
+ * registry by minutes-to-hours, in which case `getLatestRelease` would resolve
+ * a version the mirror has not yet replicated and the install would fail with
+ * `No version matching "X" found for specifier "<pkg>" (but package exists)`.
+ * See #1686.
+ */
+export declare const NPM_REGISTRY = "https://registry.npmjs.org/";
+interface ReleaseInfo {
+    tag: string;
+    version: string;
+}
 /** Result from running the installed binary and parsing its reported version. */
 export interface InstalledVersionVerification {
     ok: boolean;
@@ -27,6 +49,21 @@ interface UpdateMethodResolutionOptions {
     miseDataDir?: string;
 }
 export declare function resolveUpdateMethodForTest(jeopiPath: string, bunBinDir: string | undefined, options?: UpdateMethodResolutionOptions): UpdateMethod;
+/**
+ * Get the latest release info from the npm registry.
+ * Uses npm instead of GitHub API to avoid unauthenticated rate limiting.
+ *
+ * Exported for tests: the error contract below is what a user sees when the
+ * update check dies, and it must stay diagnosable.
+ */
+export declare function getLatestRelease(): Promise<ReleaseInfo>;
+/**
+ * Resolve the latest published {@link NPM_PACKAGE} version when it is newer
+ * than `currentVersion`, else `undefined`. Backs the startup update notice
+ * (`New version X is available. Run: jeopi update`); errors and offline
+ * registries resolve to `undefined` so startup never blocks on the check.
+ */
+export declare function fetchNewerPublishedVersion(currentVersion: string): Promise<string | undefined>;
 interface BunInstallCachePruneResult {
     scannedPackages: number;
     removedEntries: number;
@@ -83,7 +120,7 @@ export declare function replaceBinaryForUpdate(options: BinaryReplacementOptions
  * (`The .node file on disk is from a different release than this loader`).
  * Listing the natives explicitly forces bun to replace them in lock-step.
  * The leaf is added only on tags the release pipeline actually publishes
- * ({@link SUPPORTED_NATIVE_TAGS}) so unsupported platforms still fail with
+ * ({@link NATIVE_LEAF_PACKAGES}) so unsupported platforms still fail with
  * the original "no matching version" message instead of `EBADPLATFORM`.
  * See #1824.
  */

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "jeopi-cli",
-	"version": "16.2.18",
+	"version": "16.2.20",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/akillness/jeopi",
 	"author": "Can Boluk",
@@ -56,17 +56,17 @@
 		"@agentclientprotocol/sdk": "0.25.0",
 		"@babel/parser": "^7.29.7",
 		"@mozilla/readability": "^0.6.0",
-		"jeopi-hashline": "16.2.18",
-		"jeopi-stats": "16.2.18",
-		"jeopi-agent-core": "16.2.18",
-		"jeopi-ai": "16.2.18",
-		"jeopi-catalog": "16.2.18",
-		"jeopi-mnemopi": "16.2.18",
-		"jeopi-natives": "16.2.18",
-		"jeopi-tui": "16.2.18",
-		"jeopi-utils": "16.2.18",
-		"jeopi-wire": "16.2.18",
-		"jeopi-snapcompact": "16.2.18",
+		"jeopi-hashline": "16.2.20",
+		"jeopi-stats": "16.2.20",
+		"jeopi-agent-core": "16.2.20",
+		"jeopi-ai": "16.2.20",
+		"jeopi-catalog": "16.2.20",
+		"jeopi-mnemopi": "16.2.20",
+		"jeopi-natives": "16.2.20",
+		"jeopi-tui": "16.2.20",
+		"jeopi-utils": "16.2.20",
+		"jeopi-wire": "16.2.20",
+		"jeopi-snapcompact": "16.2.20",
 		"@opentelemetry/api": "^1.9.1",
 		"@opentelemetry/context-async-hooks": "^2.7.1",
 		"@opentelemetry/exporter-trace-otlp-proto": "^0.218.0",

package/src/cli/update-cli.ts

@@ -14,7 +14,12 @@ import { $which, APP_NAME, isEnoent, VERSION } from "jeopi-utils";
 import { theme } from "../modes/theme/theme";
 
 const REPO = "akillness/jeopi";
-const PACKAGE = "jeopi-cli";
+/**
+ * npm package that ships the CLI. Exported so the startup update notice
+ * (`checkForNewVersion` in main.ts) queries the same package this command
+ * installs — the two must never drift or the notice checks a phantom name.
+ */
+export const NPM_PACKAGE = "jeopi-cli";
 const HOMEBREW_FORMULA = "akillness/tap/jeopi";
 const MISE_TOOL = "github:akillness/jeopi";
 /**
@@ -28,10 +33,10 @@ const MISE_TOOL = "github:akillness/jeopi";
  * `No version matching "X" found for specifier "<pkg>" (but package exists)`.
  * See #1686.
  */
-const NPM_REGISTRY = "https://registry.npmjs.org/";
+export const NPM_REGISTRY = "https://registry.npmjs.org/";
 
 /**
- * Core native addon package. Bumped in lock-step with {@link PACKAGE} so the
+ * Core native addon package. Bumped in lock-step with {@link NPM_PACKAGE} so the
  * version sentinel the loader looks up at runtime matches the `.node` on
  * disk; see {@link buildBunInstallArgs} for why this must be installed
  * explicitly rather than inherited as a transitive dependency.
@@ -39,18 +44,20 @@ const NPM_REGISTRY = "https://registry.npmjs.org/";
 const NATIVES_PACKAGE = "jeopi-natives";
 
 /**
- * Platform tags the release pipeline publishes as
- * `jeopi-natives-<tag>` leaves. Mirrors `SUPPORTED_PLATFORMS` in
- * `packages/natives/native/loader-state.js` and `LEAF_TARGETS` in
- * `packages/natives/scripts/gen-npm-packages.ts`; kept here as the local
+ * Platform tags the release pipeline publishes as native addon leaves, mapped
+ * to the npm package that carries each addon. Mirrors `SUPPORTED_PLATFORMS` in
+ * `packages/natives/native/loader-state.js` and `LEAF_TARGETS`/`leafPackageName`
+ * in `packages/natives/scripts/gen-npm-packages.ts`; kept here as the local
  * source of truth so the update path stays free of cross-package imports.
+ * `win32-x64` ships as `jeopi-natives-windows-x64` — the registry's name
+ * heuristic rejects the tag-derived name with `403 … spam detection`.
  */
-const SUPPORTED_NATIVE_TAGS: ReadonlySet<string> = new Set([
-	"linux-x64",
-	"linux-arm64",
-	"darwin-x64",
-	"darwin-arm64",
-	"win32-x64",
+const NATIVE_LEAF_PACKAGES: ReadonlyMap<string, string> = new Map([
+	["linux-x64", "jeopi-natives-linux-x64"],
+	["linux-arm64", "jeopi-natives-linux-arm64"],
+	["darwin-x64", "jeopi-natives-darwin-x64"],
+	["darwin-arm64", "jeopi-natives-darwin-arm64"],
+	["win32-x64", "jeopi-natives-windows-x64"],
 ]);
 
 function currentNativeTag(): string {
@@ -237,11 +244,29 @@ async function resolveUpdateTarget(): Promise<UpdateTarget> {
 /**
  * Get the latest release info from the npm registry.
  * Uses npm instead of GitHub API to avoid unauthenticated rate limiting.
+ *
+ * Exported for tests: the error contract below is what a user sees when the
+ * update check dies, and it must stay diagnosable.
  */
-async function getLatestRelease(): Promise<ReleaseInfo> {
-	const response = await fetch(`${NPM_REGISTRY}${PACKAGE}/latest`);
+export async function getLatestRelease(): Promise<ReleaseInfo> {
+	const url = `${NPM_REGISTRY}${NPM_PACKAGE}/latest`;
+	const response = await fetch(url);
+	if (response.status === 404) {
+		// A 404 here means the package this build was compiled to track does not
+		// exist on the registry (e.g. the CLI was renamed after this build
+		// shipped — `jeopi` → `jeopi-cli`). Self-update can never recover from
+		// this, so name the phantom package and the manual escape hatch instead
+		// of surfacing a bare "Not Found".
+		throw new Error(
+			`Package "${NPM_PACKAGE}" was not found on ${NPM_REGISTRY} (HTTP 404). ` +
+				`This build's update check targets a package that is not published; ` +
+				`reinstall manually: bun install -g ${NPM_PACKAGE}`,
+		);
+	}
 	if (!response.ok) {
-		throw new Error(`Failed to fetch release info: ${response.statusText}`);
+		throw new Error(
+			`Failed to fetch release info for ${NPM_PACKAGE} from ${NPM_REGISTRY}: HTTP ${response.status} ${response.statusText}`,
+		);
 	}
 
 	const data = (await response.json()) as { version: string };
@@ -254,6 +279,30 @@ async function getLatestRelease(): Promise<ReleaseInfo> {
 	};
 }
 
+/**
+ * Resolve the latest published {@link NPM_PACKAGE} version when it is newer
+ * than `currentVersion`, else `undefined`. Backs the startup update notice
+ * (`New version X is available. Run: jeopi update`); errors and offline
+ * registries resolve to `undefined` so startup never blocks on the check.
+ */
+export async function fetchNewerPublishedVersion(currentVersion: string): Promise<string | undefined> {
+	try {
+		const response = await fetch(`${NPM_REGISTRY}${NPM_PACKAGE}/latest`);
+		if (!response.ok) return undefined;
+
+		const data = (await response.json()) as { version?: string };
+		const latestVersion = data.version;
+
+		if (latestVersion && Bun.semver.order(latestVersion, currentVersion) > 0) {
+			return latestVersion;
+		}
+
+		return undefined;
+	} catch {
+		return undefined;
+	}
+}
+
 /**
  * Compare semver versions. Returns:
  * - negative if a < b
@@ -727,7 +776,7 @@ export async function replaceBinaryForUpdate(options: BinaryReplacementOptions):
  * (`The .node file on disk is from a different release than this loader`).
  * Listing the natives explicitly forces bun to replace them in lock-step.
  * The leaf is added only on tags the release pipeline actually publishes
- * ({@link SUPPORTED_NATIVE_TAGS}) so unsupported platforms still fail with
+ * ({@link NATIVE_LEAF_PACKAGES}) so unsupported platforms still fail with
  * the original "no matching version" message instead of `EBADPLATFORM`.
  * See #1824.
  */
@@ -737,11 +786,12 @@ export function buildBunInstallArgs(expectedVersion: string, nativeTag: string =
 		"-g",
 		"--no-cache",
 		`--registry=${NPM_REGISTRY}`,
-		`${PACKAGE}@${expectedVersion}`,
+		`${NPM_PACKAGE}@${expectedVersion}`,
 		`${NATIVES_PACKAGE}@${expectedVersion}`,
 	];
-	if (SUPPORTED_NATIVE_TAGS.has(nativeTag)) {
-		args.push(`${NATIVES_PACKAGE}-${nativeTag}@${expectedVersion}`);
+	const leafPackage = NATIVE_LEAF_PACKAGES.get(nativeTag);
+	if (leafPackage) {
+		args.push(`${leafPackage}@${expectedVersion}`);
 	}
 	return args;
 }