robot-resources 1.7.3 → 1.7.5

package/lib/tool-config.js

@@ -223,20 +223,25 @@ export function configureToolRouting() {
 
 /**
  * Restart the OpenClaw gateway so it picks up new plugin + config.
- * Spawns a detached process that waits 3s then restarts — lets the wizard
- * finish and return output to OC before the gateway dies.
+ * Spawns a detached process that waits 30s then restarts — lets the wizard
+ * exit and the OC agent deliver the install report + answer at least one
+ * follow-up before the gateway dies.
  *
- * If called from inside an OC agent session (exec: npx robot-resources),
- * a synchronous restart kills the session before output is captured.
- * The deferred approach avoids this: wizard exits → OC captures output →
- * 3s later gateway restarts → next conversation has all tools loaded.
+ * Why deferred: a synchronous restart kills the OC session before the
+ * install report is captured. The 30s window gives the agent time to
+ * respond and handle a follow-up message. After the restart, the next
+ * conversation loads the plugin and routing is active.
+ *
+ * History: PR #89 used synchronous await (worked but required user to
+ * ask for updates). Session #26 used 5s delay (too short — killed
+ * session mid-conversation). 30s balances report delivery with restart.
  */
 async function restartOpenClawGateway() {
-  spawn('sh', ['-c', 'sleep 3 && openclaw gateway restart'], {
+  spawn('sh', ['-c', 'sleep 30 && openclaw gateway restart'], {
     stdio: 'ignore',
     detached: true,
   }).unref();
-  process.stdout.write('  Restarting gateway...\n');
+  process.stdout.write('  Gateway restart scheduled (30s)...\n');
 }
 
 // Exported for testing and direct use

package/lib/wizard.js

@@ -339,23 +339,6 @@ export async function runWizard({ nonInteractive = false } = {}) {
     blank();
   }
 
-  // ── Best-effort: Single gateway restart ────────────────────────────────
-  //
-  // Merged from two previous restarts (after plugin install + after scraper
-  // registration). Everything is already installed and registered above.
-  // If the session dies during this restart, the gateway picks up changes
-  // on its next natural restart.
-
-  if (isOpenClawInstalled() && (results.tools?.some(r => r.action === 'installed') || scraperRegistered)) {
-    try {
-      await restartOpenClawGateway();
-      success('Gateway restart scheduled — tools available in next conversation');
-    } catch {
-      warn('Gateway restart failed — tools may not be available until OpenClaw restarts.');
-      warn('Run manually: openclaw gateway restart');
-    }
-  }
-
   // ── Post-install health check ──────────────────────────────────────────
 
   let healthReport = null;
@@ -398,4 +381,22 @@ export async function runWizard({ nonInteractive = false } = {}) {
       // Non-fatal — status file is a convenience, not required
     }
   }
+
+  // ── Best-effort: Deferred gateway restart (MUST be last) ───────────────
+  //
+  // Gateway restart loads the newly installed plugin + scraper MCP.
+  // Without it, tools aren't available until the next natural gateway restart.
+  //
+  // MUST be the last thing in the wizard. The deferred spawn lets the wizard
+  // exit and return output to OC before the gateway dies. Silent catch —
+  // the plugin's install message already tells the user to start a new
+  // conversation. No warn/success needed.
+  //
+  // History: PR #89 (Manuel) had this right — restart last, silent catch.
+  // Session #25 moved it earlier with 3x retries, which killed OC Telegram
+  // sessions. Reverted to deferred spawn, positioned last.
+
+  if (isOpenClawInstalled() && (results.tools?.some(r => r.action === 'installed') || scraperRegistered)) {
+    try { await restartOpenClawGateway(); } catch { /* silent — best effort */ }
+  }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "robot-resources",
-  "version": "1.7.3",
+  "version": "1.7.5",
   "description": "Robot Resources — AI agent tools. One command to install everything.",
   "type": "module",
   "bin": {