skydive-cli 0.2.0-beta.741 → 0.2.0-beta.742

package/dist/js/bin.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { C as setActiveWorkspace, S as listWorkspaces, T as version, _ as themes, b as getActiveWorkspaceId, o as brandHelpArt, t as installCrashHandler, w as name, x as getSessionIdentity, y as ensureActiveOrganization } from "./install-CVVefO8_.mjs";
+import { C as setActiveWorkspace, S as listWorkspaces, T as version, _ as themes, b as getActiveWorkspaceId, o as brandHelpArt, t as installCrashHandler, w as name, x as getSessionIdentity, y as ensureActiveOrganization } from "./install-Cv3sBZxr.mjs";
 import { A as resolveAppUrl, C as getPromptHistoryPath, D as getStoredApiKeyId, E as getShareMachineDefault, F as saveConfig, I as saveSession, M as resolveManagementAuth, N as resolveSession, O as getStoredApiKeyWorkspaceName, R as setLastSeenVersion, S as getLastSeenVersion, _ as API_KEY_PREFIX, b as deleteConfig, g as API_KEY_FAMILY_PREFIX, h as API_KEYS_URL, i as resolveAgent, j as resolveConfig, k as getUpdateCheckDisabled, v as DEFAULT_API_URL, x as getConfigPath } from "./print-Cd-bVSEJ.mjs";
 import { n as printError, r as printTable, t as output } from "./output-DYzzdXYV.mjs";
 import { n as createRestClient, t as HttpError } from "./rest-BlN_uWmL.mjs";
@@ -1500,7 +1500,7 @@ const importCommand = {
 				process.exit(1);
 			}
 		}
-		const { runChat } = await import("./boot-B971VDnw.mjs");
+		const { runChat } = await import("./boot-DhorgAxS.mjs");
 		await runChat({
 			appUrl,
 			sessionToken: session.value.sessionToken,
@@ -1912,7 +1912,7 @@ const chatCommand = {
 			sessionToken: session.value.sessionToken,
 			agentSelector: argv.agent ?? null
 		});
-		const { runChat } = await import("./boot-B971VDnw.mjs");
+		const { runChat } = await import("./boot-DhorgAxS.mjs");
 		await runChat({
 			appUrl,
 			sessionToken: session.value.sessionToken,
@@ -2254,7 +2254,7 @@ const switchCommand = {
 				printError("The workspace picker needs the Bun runtime and it could not be set up automatically. Pass a workspace slug instead, or install Bun and retry.");
 				process.exit(1);
 			}
-			const { runWorkspacePicker } = await import("./boot-B971VDnw.mjs");
+			const { runWorkspacePicker } = await import("./boot-DhorgAxS.mjs");
 			await runWorkspacePicker(session);
 			return;
 		}
@@ -2794,9 +2794,10 @@ function isNewerVersion(candidate, current) {
 *
 * package-manager installs are never self-mutated here — that's the install
 * mode's own tool's job, and silently reaching into a global npm/yarn/pnpm
-* prefix is exactly the footgun the existing notice policy avoids. The
-* command handles that mode by printing the upgrade command; this module is
-* binary-only.
+* prefix is exactly the footgun the existing notice policy avoids. They still
+* go through the same {@link planUpdate} version comparison (an install that
+* skips it has no way to know it is already current); only the *action*
+* differs — the command prints, or offers to run, the upgrade command.
 *
 * The dangerous part is replacing a running executable in place, so the flow
 * is split: a pure {@link planUpdate} decides *whether* and *what* (testable
@@ -2806,28 +2807,56 @@ function isNewerVersion(candidate, current) {
 /** Cap so a wrong URL or a hung CDN can't wedge `skydive update` forever. */
 const DOWNLOAD_TIMEOUT_MS = 6e4;
 /**
+* The single version comparison every install mode goes through: only a
+* strictly newer release is an update. `allowSameVersion` (`--force`) and
+* `allowDowngrade` (rollback) are the two explicit escape hatches.
+*/
+function wantsInstall(opts) {
+	const newer = isNewerVersion(opts.latestVersion, opts.currentVersion);
+	const same = opts.latestVersion === opts.currentVersion;
+	return newer || same && opts.allowSameVersion === true || !newer && !same && opts.allowDowngrade === true;
+}
+/**
 * Decide what `skydive update` should do, given the running version, the
-* install source, the fetched manifest (or null), and this host. Pure — no
+* install source, the channel's newest release, and this host. Pure — no
 * network, no filesystem — so every branch is unit-testable.
 *
+* `latestVersion` is the channel's current version for a **package-manager**
+* install (an npm dist-tag lookup). A binary install reads it off `manifest`
+* instead, because there the manifest is both the version pointer and the
+* byte source. Either way it must be a real fetched answer: a package-manager
+* install that skips the comparison is what made `skydive update` announce
+* "0.2.0 → 0.2.0" on every invocation.
+*
 * `allowSameVersion` forces a reinstall of the current version (the `--force`
 * escape hatch for a corrupt binary); `allowDowngrade` lets an explicit
 * pinned/older manifest install over a newer running binary (rollback).
 */
 function planUpdate(opts) {
 	const { currentVersion, source, manifest, platform, arch } = opts;
-	if (source !== "binary") return {
-		kind: "unsupported-source",
-		source,
-		latestVersion: manifest?.version ?? currentVersion
-	};
+	if (source !== "binary") {
+		const { latestVersion } = opts;
+		if (!latestVersion || !wantsInstall({
+			...opts,
+			latestVersion
+		})) return {
+			kind: "up-to-date",
+			currentVersion
+		};
+		return {
+			kind: "unsupported-source",
+			source,
+			latestVersion
+		};
+	}
 	if (!manifest) return {
 		kind: "up-to-date",
 		currentVersion
 	};
-	const newer = isNewerVersion(manifest.version, currentVersion);
-	const same = manifest.version === currentVersion;
-	if (!(newer || same && opts.allowSameVersion === true || !newer && !same && opts.allowDowngrade === true)) return {
+	if (!wantsInstall({
+		...opts,
+		latestVersion: manifest.version
+	})) return {
 		kind: "up-to-date",
 		currentVersion
 	};
@@ -3146,6 +3175,15 @@ async function promptYesNo(question) {
 	}
 }
 /**
+* Heading shared by the prompt and the printed instruction, so the two can't
+* disagree about what is happening. `--force` asks to reinstall the version
+* already running, and calling that an update is exactly the lie this path
+* used to tell on every invocation — so it gets its own wording.
+*/
+function renderPmUpdateHeading(opts) {
+	return opts.latestVersion === opts.currentVersion ? `Reinstalling ${opts.currentVersion}.` : `Update available: ${opts.currentVersion} \u2192 ${opts.latestVersion}`;
+}
+/**
 * Offer + run the package-manager upgrade. Returns a result the caller turns
 * into output and an exit/relaunch decision — kept separate from the actual
 * relaunch so it's testable without re-execing the test runner.
@@ -3158,7 +3196,7 @@ async function offerPackageManagerUpdate(opts) {
 	const command = installCommand(manager, opts.channel);
 	const commandStr = installCommandString(manager, opts.channel);
 	if (!opts.interactive) return { status: "skipped-noninteractive" };
-	if (!await (opts.ask ?? promptYesNo)(`Update available: ${opts.currentVersion} \u2192 ${opts.latestVersion}\nRun \`${commandStr}\` now? [Y/n] `)) return { status: "declined" };
+	if (!await (opts.ask ?? promptYesNo)(`${renderPmUpdateHeading(opts)}\nRun \`${commandStr}\` now? [Y/n] `)) return { status: "declined" };
 	const { status } = (opts.run ?? ((cmd) => {
 		const [bin, ...args] = cmd;
 		return { status: spawnSync(bin ?? "", args, { stdio: "inherit" }).status };
@@ -3210,7 +3248,10 @@ function normalizeChannel(raw, currentVersion) {
 function describePlan(plan, channel, isCheck) {
 	switch (plan.kind) {
 		case "up-to-date": return `Already on the latest ${channel} release (${plan.currentVersion}).`;
-		case "unsupported-source": return `This is a ${plan.source} install, so update through your package manager:\n  ${renderUpdateCommand(channel, plan.source)}`;
+		case "unsupported-source": return `${renderPmUpdateHeading({
+			currentVersion: version,
+			latestVersion: plan.latestVersion
+		})}\nThis is a ${plan.source} install, so update through your package manager:\n  ${renderUpdateCommand(channel, plan.source)}`;
 		case "unsupported-host": return `No Skydive binary is published for ${plan.platform}/${plan.arch}.`;
 		case "missing-from-manifest": return `The ${channel} ${plan.version} release has no binary for your platform (${plan.target}).`;
 		case "update": return isCheck ? `Update available: ${version} \u2192 ${plan.version} (${channel}, ${plan.target}).\nRun \`skydive update\` to install it.` : `Updating ${version} \u2192 ${plan.version} (${channel}, ${plan.target})...`;
@@ -3236,8 +3277,11 @@ const updateCommand = {
 		const currentVersion = version;
 		const source = resolveInstallSource();
 		const channel = normalizeChannel(argv.channel, currentVersion);
-		const manifest = await releaseSourceForInstall(source).fetchManifest(channel, { signal: AbortSignal.timeout(FETCH_TIMEOUT_MS$1) }).catch(() => null);
-		if (source === "binary" && manifest === null) {
+		const releaseSource = releaseSourceForInstall(source);
+		const signal = AbortSignal.timeout(FETCH_TIMEOUT_MS$1);
+		const manifest = source === "binary" ? await releaseSource.fetchManifest(channel, { signal }).catch(() => null) : null;
+		const latestVersion = source === "binary" ? manifest?.version ?? null : await releaseSource.fetchLatestVersion(channel, { signal }).catch(() => null);
+		if (latestVersion === null) {
 			if (argv.json) {
 				output(argv, {
 					status: "unavailable",
@@ -3246,11 +3290,12 @@ const updateCommand = {
 				});
 				return;
 			}
-			throw new UpdateError(`couldn't reach the ${channel} release channel — check your connection and try again.`);
+			throw new UpdateError(source === "binary" ? `couldn't reach the ${channel} release channel — check your connection and try again.` : `couldn't reach the npm registry to look up the ${channel} release — check your connection and try again.`);
 		}
 		const plan = planUpdate({
 			currentVersion,
 			source,
+			latestVersion,
 			manifest,
 			platform: process.platform,
 			arch: process.arch,

package/dist/js/{boot-B971VDnw.mjs → boot-DhorgAxS.mjs}

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { C as setActiveWorkspace, S as listWorkspaces, a as WORDMARK, b as getActiveWorkspaceId, c as applyTheme, d as noColorRequested, f as theme, g as themeVersion, h as themeModeFromColorFgBg, i as MARK_CELLS, l as findTheme, m as themeMode, n as buildCrashReport, p as themeForMode, r as writeCrashReport, s as DEFAULT_THEME_ID, t as installCrashHandler, u as monoTheme, v as themesForMode } from "./install-CVVefO8_.mjs";
+import { C as setActiveWorkspace, S as listWorkspaces, a as WORDMARK, b as getActiveWorkspaceId, c as applyTheme, d as noColorRequested, f as theme, g as themeVersion, h as themeModeFromColorFgBg, i as MARK_CELLS, l as findTheme, m as themeMode, n as buildCrashReport, p as themeForMode, r as writeCrashReport, s as DEFAULT_THEME_ID, t as installCrashHandler, u as monoTheme, v as themesForMode } from "./install-Cv3sBZxr.mjs";
 import { L as saveTheme, P as resolveWebUrl, T as getSavedTheme, c as cardActionErrorMessage, d as reconcileMaskedInput, f as resolveConnectUrl, i as resolveAgent, l as parseExternalOauthConnectParams, m as specKeyFor, p as parseConnectCard, s as MASK_CHAR, u as parseOauthConnectParams, v as DEFAULT_API_URL, w as getReviewStateDir, x as getConfigPath, y as DEFAULT_APP_URL } from "./print-Cd-bVSEJ.mjs";
 import { a as errorMessage, i as sendErrorMessage, n as createRestClient, o as isRecord, r as errorDetail, t as HttpError } from "./rest-BlN_uWmL.mjs";
 import { t as PortalClient } from "./client-Dc7GZ3PG.mjs";

package/dist/js/{install-CVVefO8_.mjs → install-Cv3sBZxr.mjs}

@@ -8,7 +8,7 @@ import fs from "node:fs";
 
 //#region package.json
 var name = "skydive-cli";
-var version$1 = "0.2.0-beta.741";
+var version$1 = "0.2.0-beta.742";
 
 //#endregion
 //#region src/auth/organization.ts

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skydive-cli",
-  "version": "0.2.0-beta.741",
+  "version": "0.2.0-beta.742",
   "description": "Skydive CLI — cloud agents from the command line",
   "homepage": "https://skydive.com",
   "license": "MIT",