apex-dev 3.10.18 → 3.10.20

package/cli.js

@@ -145,6 +145,8 @@ function getDownloadUrl() {
   return `https://github.com/Marcus-Mok-GH/apex-dev/releases/download/v${VERSION}/apex-dev-${platform}-${arch}`;
 }
 
+const DOWNLOAD_TIMEOUT = 10 * 1000;
+
 function downloadBinary(destPath) {
   const url = getDownloadUrl();
   console.error(`Downloading apex-dev v${VERSION} for ${detectPlatform().platform}-${detectPlatform().arch}...`);
@@ -152,33 +154,80 @@ function downloadBinary(destPath) {
 
   return new Promise((resolve, reject) => {
     const file = fs.createWriteStream(destPath, { mode: 0o755 });
-    https
-      .get(url, (response) => {
-        if (response.statusCode === 302 || response.statusCode === 301) {
-          https.get(response.headers.location, (redirectRes) => {
-            if (redirectRes.statusCode !== 200) {
-              reject(new Error(`Download failed with status ${redirectRes.statusCode}`));
-              return;
-            }
-            redirectRes.pipe(file);
-            file.on("finish", () => {
-              file.close();
-              resolve();
-            });
-          }).on("error", reject);
-          return;
+    let settled = false;
+
+    function cleanupAndReject(err) {
+      if (settled) return;
+      settled = true;
+      file.destroy();
+      try { fs.unlinkSync(destPath); } catch {}
+      reject(err);
+    }
+
+    function onFileError(err) {
+      cleanupAndReject(new Error(`Write stream error: ${err.message}`));
+    }
+
+    file.on("error", onFileError);
+
+    const req = https.get(url, (response) => {
+      if (response.statusCode === 302 || response.statusCode === 301) {
+        const redirectReq = https.get(response.headers.location, (redirectRes) => {
+          if (redirectRes.statusCode !== 200) {
+            cleanupAndReject(new Error(`Download failed with status ${redirectRes.statusCode}`));
+            return;
+          }
+
+          function onFinish() {
+            file.removeListener("error", onFileError);
+            redirectReq.removeListener("error", onRedirectError);
+            redirectReq.removeListener("timeout", onRedirectTimeout);
+            file.close();
+            resolve();
+          }
+
+          file.on("finish", onFinish);
+          redirectRes.pipe(file);
+        });
+
+        function onRedirectError(err) {
+          cleanupAndReject(new Error(`Redirect request error: ${err.message}`));
         }
-        if (response.statusCode !== 200) {
-          reject(new Error(`Download failed with status ${response.statusCode}`));
-          return;
+        function onRedirectTimeout() {
+          cleanupAndReject(new Error("Download redirect timed out"));
         }
-        response.pipe(file);
-        file.on("finish", () => {
-          file.close();
-          resolve();
-        });
-      })
-      .on("error", reject);
+
+        redirectReq.on("error", onRedirectError);
+        redirectReq.setTimeout(DOWNLOAD_TIMEOUT, onRedirectTimeout);
+        return;
+      }
+
+      if (response.statusCode !== 200) {
+        cleanupAndReject(new Error(`Download failed with status ${response.statusCode}`));
+        return;
+      }
+
+      function onFinish() {
+        file.removeListener("error", onFileError);
+        req.removeListener("error", onReqError);
+        req.removeListener("timeout", onReqTimeout);
+        file.close();
+        resolve();
+      }
+
+      file.on("finish", onFinish);
+      response.pipe(file);
+    });
+
+    function onReqError(err) {
+      cleanupAndReject(new Error(`Download request error: ${err.message}`));
+    }
+    function onReqTimeout() {
+      cleanupAndReject(new Error("Download timed out"));
+    }
+
+    req.on("error", onReqError);
+    req.setTimeout(DOWNLOAD_TIMEOUT, onReqTimeout);
   });
 }
 
@@ -186,7 +235,17 @@ async function ensureBinary() {
   const localPath = getLocalBinaryPath();
 
   if (fs.existsSync(localPath)) {
-    return localPath;
+    // Check for empty/partial downloads and remove them
+    try {
+      const stat = fs.statSync(localPath);
+      if (stat.size === 0) {
+        fs.unlinkSync(localPath);
+      } else {
+        return localPath;
+      }
+    } catch {
+      try { fs.unlinkSync(localPath); } catch {}
+    }
   }
 
   try {
@@ -195,6 +254,7 @@ async function ensureBinary() {
     return localPath;
   } catch (err) {
     console.error(`Failed to download binary: ${err.message}`);
+    try { fs.unlinkSync(localPath); } catch {}
     return null;
   }
 }
@@ -221,6 +281,10 @@ function runWithBun() {
     const child = spawn(bunPath, [distPath, ...process.argv.slice(2)], {
       stdio: "inherit",
     });
+    child.on("error", (err) => {
+      console.error(`Failed to launch bun: ${err.message}`);
+      process.exit(1);
+    });
     child.on("exit", (code) => {
       process.exit(code || 0);
     });
@@ -229,6 +293,10 @@ function runWithBun() {
   const child = spawn(bunPath, [scriptPath, ...process.argv.slice(2)], {
     stdio: "inherit",
   });
+  child.on("error", (err) => {
+    console.error(`Failed to launch bun: ${err.message}`);
+    process.exit(1);
+  });
   child.on("exit", (code) => {
     process.exit(code || 0);
   });
@@ -329,6 +397,10 @@ async function main() {
     const child = spawn(binaryPath, args, {
       stdio: "inherit",
     });
+    child.on("error", (err) => {
+      console.error(`Failed to launch binary: ${err.message}`);
+      process.exit(1);
+    });
     child.on("exit", (code) => {
       process.exit(code || 0);
     });

package/dist/index.js

@@ -867,16 +867,26 @@ The user asks you to implement a new feature. You respond in multiple steps:
     deepseek: { model: "deepseek/deepseek-chat-v3", temperature: 0.1, maxTokens: 8192 },
     minimax: { model: "minimax/minimax-01", temperature: 0.1, maxTokens: 8192 }
   };
-  const _initialProvider = PROVIDERS[currentProvider];
+  const os = __require("os");
+  let savedProvider = null;
   try {
-    const configPath = path.join(__require("os").homedir(), ".apex-dev", "config.json");
+    const configPath = path.join(os.homedir(), ".apex-dev", "config.json");
     if (fs.existsSync(configPath)) {
       const savedConfig = JSON.parse(fs.readFileSync(configPath, "utf-8"));
-      if (savedConfig[currentProvider] && !process.env[_initialProvider.envKey]) {
-        process.env[_initialProvider.envKey] = savedConfig[currentProvider];
+      const hasEnvKey = Object.values(PROVIDERS).some((p) => process.env[p.envKey]);
+      if (!hasEnvKey) {
+        for (const [providerKey, provider] of Object.entries(PROVIDERS)) {
+          if (savedConfig[providerKey]) {
+            currentProvider = providerKey;
+            process.env[provider.envKey] = savedConfig[providerKey];
+            savedProvider = providerKey;
+            break;
+          }
+        }
       }
     }
   } catch (e) {}
+  const _initialProvider = PROVIDERS[currentProvider];
   const _initialKey = process.env[_initialProvider.envKey] || "no-key";
   let _internalClient = new OpenAI({
     apiKey: _initialKey,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "apex-dev",
-  "version": "3.10.18",
+  "version": "3.10.20",
   "description": "Apex AI - a friendly agentic coding assistant for the terminal",
   "main": "dist/index.js",
   "bin": {