@openclawcity/become 1.0.24 → 1.0.25

package/dist/cli.cjs

@@ -146,10 +146,30 @@ function patchOpenClaw(config) {
       `Provider "${providerName}" not found in models.json or openclaw.json. Your model is "${primaryModel}" which needs a "${providerName}" provider.`
     );
   }
-  const originalUrl = provider.baseUrl;
+  let originalUrl = provider.baseUrl;
   if (!originalUrl) {
     throw new Error(`Provider "${providerName}" has no baseUrl`);
   }
+  if (originalUrl.includes("127.0.0.1") || originalUrl.includes("localhost")) {
+    if ((0, import_node_fs2.existsSync)(BACKUP_PATH)) {
+      try {
+        const backupRaw = (0, import_node_fs2.readFileSync)(BACKUP_PATH, "utf-8");
+      } catch {
+      }
+    }
+    const savedUrl = readSafe(ORIGINAL_URL_PATH);
+    if (savedUrl && !savedUrl.includes("127.0.0.1")) {
+      originalUrl = savedUrl;
+      console.log(`  recovered original URL from state: ${originalUrl}`);
+    } else {
+      throw new Error(
+        `Provider "${providerName}" baseUrl is already pointing to localhost (${originalUrl}).
+This means become was previously connected but not properly restored.
+Fix manually: set the baseUrl back to the real provider URL in:
+  ${modelsSource === "models.json" ? getModelsJsonPath(clawConfig) : OPENCLAW_CONFIG}`
+      );
+    }
+  }
   (0, import_node_fs2.writeFileSync)(BACKUP_PATH, raw, "utf-8");
   (0, import_node_fs2.writeFileSync)(ORIGINAL_URL_PATH, originalUrl, "utf-8");
   (0, import_node_fs2.writeFileSync)(PATCHED_PROVIDER_PATH, `${providerName}:${modelsSource}`, "utf-8");
@@ -1903,21 +1923,30 @@ async function start() {
     max_skills_per_call: config.max_skills_per_call,
     auto_extract: config.auto_extract
   };
-  if (config.state !== "on") {
-    try {
-      turnOn();
-    } catch (e) {
-      console.error("Warning: could not auto-connect agent:", e instanceof Error ? e.message : e);
-    }
+  try {
+    turnOn();
+  } catch (e) {
+    console.error("Warning: could not connect agent:", e instanceof Error ? e.message : e);
   }
   const originalUrlPath = (0, import_node_path7.join)((0, import_node_os5.homedir)(), ".become", "state", "original_base_url.txt");
-  let originalUpstreamUrl;
+  let upstreamUrl = config.llm_base_url;
   if ((0, import_node_fs7.existsSync)(originalUrlPath)) {
     const saved = (0, import_node_fs7.readFileSync)(originalUrlPath, "utf-8").trim();
-    if (saved) originalUpstreamUrl = saved;
+    if (saved) upstreamUrl = saved;
+  }
+  if (upstreamUrl.includes("127.0.0.1") || upstreamUrl.includes("localhost")) {
+    const port = config.proxy_port.toString();
+    if (upstreamUrl.includes(`:${port}`)) {
+      console.error(`
+FATAL: upstream URL ${upstreamUrl} points to the become proxy itself.`);
+      console.error("This would cause an infinite loop. Aborting.");
+      console.error("Run `become off` to restore your config, then `become start` again.\n");
+      process.exit(1);
+    }
   }
+  console.log(`[become] upstream: ${upstreamUrl}`);
   const analyzer = createAnalyzer(config);
-  const proxy = createProxyServer(proxyConfig, analyzer, originalUpstreamUrl);
+  const proxy = createProxyServer(proxyConfig, analyzer, upstreamUrl);
   await proxy.listen();
   const dashboard = createDashboardServer({
     store: proxy.store,