uplink-cli 0.1.0-alpha.4 → 0.1.0-alpha.6

package/cli/src/http.ts

@@ -1,4 +1,9 @@
-import fetch from "node-fetch";
+import fetch, { AbortError } from "node-fetch";
+
+// Configuration
+const REQUEST_TIMEOUT = 30000; // 30 seconds
+const MAX_RETRIES = 3;
+const INITIAL_RETRY_DELAY = 1000; // 1 second
 
 function getApiBase(): string {
   return process.env.AGENTCLOUD_API_BASE ?? "https://api.uplink.spot";
@@ -18,16 +23,40 @@ function getApiToken(apiBase: string): string | undefined {
     return process.env.AGENTCLOUD_TOKEN || undefined;
   }
 
-  // Local dev convenience:
+  // Local dev:
   // - Prefer AGENTCLOUD_TOKEN if set
-  // - Otherwise allow AGENTCLOUD_TOKEN_DEV / dev-token
+  // - Otherwise allow AGENTCLOUD_TOKEN_DEV (no hardcoded default for security)
   return (
     process.env.AGENTCLOUD_TOKEN ||
     process.env.AGENTCLOUD_TOKEN_DEV ||
-    "dev-token"
+    undefined
   );
 }
 
+// Check if error is retryable (network issues, 5xx errors)
+function isRetryable(error: unknown, statusCode?: number): boolean {
+  // Network errors are retryable
+  if (error instanceof AbortError) return true;
+  if (error instanceof Error) {
+    const msg = error.message.toLowerCase();
+    if (msg.includes("econnrefused") || msg.includes("enotfound") || 
+        msg.includes("etimedout") || msg.includes("econnreset") ||
+        msg.includes("socket hang up")) {
+      return true;
+    }
+  }
+  // 5xx server errors are retryable
+  if (statusCode && statusCode >= 500 && statusCode < 600) return true;
+  // 429 Too Many Requests is retryable
+  if (statusCode === 429) return true;
+  return false;
+}
+
+// Sleep helper
+function sleep(ms: number): Promise<void> {
+  return new Promise(resolve => setTimeout(resolve, ms));
+}
+
 export async function apiRequest(
   method: string,
   path: string,
@@ -39,22 +68,57 @@ export async function apiRequest(
     throw new Error("Missing AGENTCLOUD_TOKEN");
   }
 
-  const response = await fetch(`${apiBase}${path}`, {
-    method,
-    headers: {
-      "Content-Type": "application/json",
-      Authorization: `Bearer ${apiToken}`,
-    },
-    body: body ? JSON.stringify(body) : undefined,
-  });
-
-  const json = await response.json().catch(() => ({}));
-  if (!response.ok) {
-    throw new Error(JSON.stringify(json, null, 2));
-  }
-
-  return json;
-}
+  let lastError: Error | null = null;
+  
+  for (let attempt = 0; attempt < MAX_RETRIES; attempt++) {
+    // Create abort controller for timeout
+    const controller = new AbortController();
+    const timeoutId = setTimeout(() => controller.abort(), REQUEST_TIMEOUT);
+    
+    try {
+      const response = await fetch(`${apiBase}${path}`, {
+        method,
+        headers: {
+          "Content-Type": "application/json",
+          Authorization: `Bearer ${apiToken}`,
+        },
+        body: body ? JSON.stringify(body) : undefined,
+        signal: controller.signal,
+      });
 
+      clearTimeout(timeoutId);
 
+      const json = await response.json().catch(() => ({}));
+      
+      if (!response.ok) {
+        // Check if this error is retryable
+        if (isRetryable(null, response.status) && attempt < MAX_RETRIES - 1) {
+          const delay = INITIAL_RETRY_DELAY * Math.pow(2, attempt);
+          await sleep(delay);
+          continue;
+        }
+        throw new Error(JSON.stringify(json, null, 2));
+      }
 
+      return json;
+    } catch (error) {
+      clearTimeout(timeoutId);
+      lastError = error instanceof Error ? error : new Error(String(error));
+      
+      // Check if we should retry
+      if (isRetryable(error) && attempt < MAX_RETRIES - 1) {
+        const delay = INITIAL_RETRY_DELAY * Math.pow(2, attempt);
+        await sleep(delay);
+        continue;
+      }
+      
+      // Not retryable or out of retries
+      if (error instanceof AbortError) {
+        throw new Error(`Request timeout after ${REQUEST_TIMEOUT}ms`);
+      }
+      throw lastError;
+    }
+  }
+  
+  throw lastError || new Error("Request failed after retries");
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "uplink-cli",
-  "version": "0.1.0-alpha.4",
+  "version": "0.1.0-alpha.6",
   "description": "Localhost to public URL in seconds. No signup forms, no browser - everything in your terminal.",
   "keywords": [
     "tunnel",
@@ -50,6 +50,7 @@
     "better-sqlite3": "^11.10.0",
     "body-parser": "^1.20.3",
     "commander": "^12.1.0",
+    "compression": "^1.7.4",
     "dotenv": "^16.6.1",
     "express": "^4.19.2",
     "express-rate-limit": "^8.2.1",
@@ -62,6 +63,7 @@
     "zod": "^4.2.1"
   },
   "devDependencies": {
+    "@types/compression": "^1.7.5",
     "@types/express": "^4.17.21",
     "@types/express-rate-limit": "^5.1.3",
     "@types/node": "^22.7.4",