@orchagent/cli 0.3.89 → 0.3.90

package/dist/lib/api.js

@@ -116,8 +116,6 @@ async function safeFetchWithRetryForCalls(url, options) {
                     parsed = JSON.parse(bodyText);
                 }
                 catch { /* ignore */ }
-                const detail = parsed?.error?.message ||
-                    parsed?.message || '';
                 const isRetryable = parsed?.error?.is_retryable;
                 // Don't retry if server explicitly says error is not retryable
                 if (isRetryable === false) {
@@ -130,8 +128,7 @@ async function safeFetchWithRetryForCalls(url, options) {
                 if (attempt < MAX_RETRIES) {
                     const delay = BASE_DELAY_MS * Math.pow(2, attempt - 1);
                     const jitter = Math.random() * 500;
-                    const detailSuffix = detail ? `: ${detail}` : '';
-                    process.stderr.write(`Request failed (${response.status}${detailSuffix}), retrying in ${Math.round((delay + jitter) / 1000)}s...\n`);
+                    process.stderr.write(`Server error (${response.status}), retrying in ${Math.round((delay + jitter) / 1000)}s...\n`);
                     await new Promise(r => setTimeout(r, delay + jitter));
                     continue;
                 }
@@ -174,8 +171,6 @@ async function safeFetchWithRetry(url, options) {
                     parsed = JSON.parse(bodyText);
                 }
                 catch { /* ignore */ }
-                const detail = parsed?.error?.message ||
-                    parsed?.message || '';
                 const isRetryable = parsed?.error?.is_retryable;
                 // Don't retry if server explicitly says error is not retryable
                 if (isRetryable === false) {
@@ -188,8 +183,7 @@ async function safeFetchWithRetry(url, options) {
                 if (attempt < MAX_RETRIES) {
                     const delay = BASE_DELAY_MS * Math.pow(2, attempt - 1);
                     const jitter = Math.random() * 500;
-                    const detailSuffix = detail ? `: ${detail}` : '';
-                    process.stderr.write(`Request failed (${response.status}${detailSuffix}), retrying in ${Math.round((delay + jitter) / 1000)}s...\n`);
+                    process.stderr.write(`Server error (${response.status}), retrying in ${Math.round((delay + jitter) / 1000)}s...\n`);
                     await new Promise(r => setTimeout(r, delay + jitter));
                     continue;
                 }

package/dist/lib/doctor/checks/environment.js

@@ -191,6 +191,37 @@ function safeRealpathSync(p) {
         return p;
     }
 }
+/**
+ * Parse a version string into a comparable numeric tuple.
+ * Returns [-1, -1, -1] for unparseable versions (sorts below all valid versions).
+ */
+function parseVersion(version) {
+    const match = version.match(/^(\d+)\.(\d+)\.(\d+)/);
+    if (!match)
+        return [-1, -1, -1];
+    return [parseInt(match[1], 10), parseInt(match[2], 10), parseInt(match[3], 10)];
+}
+/**
+ * Find the latest (highest) version from a list of version strings.
+ * Unparseable versions (e.g. "unknown") sort below all valid versions.
+ */
+function findLatestVersion(versions) {
+    let latest = versions[0];
+    let latestParts = parseVersion(latest);
+    for (let i = 1; i < versions.length; i++) {
+        const parts = parseVersion(versions[i]);
+        for (let j = 0; j < 3; j++) {
+            if (parts[j] > latestParts[j]) {
+                latest = versions[i];
+                latestParts = parts;
+                break;
+            }
+            if (parts[j] < latestParts[j])
+                break;
+        }
+    }
+    return latest;
+}
 /**
  * Detect multiple CLI installations at different paths/versions (BUG-008).
  *
@@ -248,12 +279,24 @@ async function checkDualInstallation() {
             .map((i) => `${i.path} (v${i.version})`)
             .join(', ');
         if (versionsDiffer) {
+            // Find the latest version and build a specific rm command for stale paths
+            const latestVersion = findLatestVersion(allInstalls.map((i) => i.version));
+            const stalePaths = allInstalls
+                .filter((i) => i.version !== latestVersion)
+                .map((i) => i.path);
+            const staleList = allInstalls
+                .filter((i) => i.version !== latestVersion)
+                .map((i) => `${i.path} (v${i.version})`)
+                .join(', ');
+            const fix = stalePaths.length === 1
+                ? `Remove outdated ${staleList}: run \`rm ${stalePaths[0]}\``
+                : `Remove outdated installations (${staleList}): run \`rm ${stalePaths.join(' ')}\``;
             return {
                 category: 'environment',
                 name: 'dual_installation',
                 status: 'warning',
                 message: `Multiple CLI versions found: ${pathList}`,
-                fix: 'Remove the outdated installation. Run `which -a orch orchagent` to see all paths, then remove the older binary',
+                fix,
                 details: {
                     installationCount: installations.size,
                     versionMismatch: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@orchagent/cli",
-  "version": "0.3.89",
+  "version": "0.3.90",
   "description": "Command-line interface for orchagent — deploy and run AI agents for your team",
   "license": "MIT",
   "author": "orchagent <hello@orchagent.io>",