@openclawcity/become 1.0.23 → 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");
@@ -757,9 +777,12 @@ Rules:
 - Max 3 lessons per conversation
 - If no real learning happened, return []`;
     try {
+      console.log(`[become] extracting lessons from ${agentId} (${detection.exchangeType}), text length: ${exchangeText.length}`);
       const response = await this.analyzer.analyze(prompt);
       const lessons = this.parseLessons(response);
+      console.log(`[become] LLM returned ${lessons.length} lessons`);
       for (const lesson of lessons.slice(0, 3)) {
+        console.log(`[become] saving lesson: ${lesson.skill} (confidence: ${lesson.confidence})`);
         const saved = this.store.savePending({
           name: lesson.skill,
           instruction: lesson.instruction.slice(0, 500),
@@ -769,13 +792,18 @@ Rules:
           created_at: (/* @__PURE__ */ new Date()).toISOString()
         });
         if (saved) {
+          console.log(`[become] lesson saved: ${saved.id}`);
           this.trust.recordLesson(agentId);
           if (trustLevel === "trusted") {
             this.store.approve(saved.id);
+            console.log(`[become] auto-approved (trusted agent)`);
           }
+        } else {
+          console.log(`[become] lesson NOT saved (duplicate or store error)`);
         }
       }
-    } catch {
+    } catch (err) {
+      console.log(`[become] extraction error: ${err instanceof Error ? err.message : String(err)}`);
     }
   }
   parseLessons(response) {
@@ -1895,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,