@myvillage/cli 1.88.1 → 1.88.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@myvillage/cli",
-  "version": "1.88.1",
+  "version": "1.88.3",
   "description": "MyVillageOS CLI for community developers",
   "type": "module",
   "bin": {

package/src/agent-runtime/runner.js

@@ -171,9 +171,12 @@ async function resolveMachineIdentity(opts) {
   }
 
   if (saved?.machineKey) {
-    // Re-register with the stored credential so name/metadata/version refresh.
-    const { machineId } = await register(makeClient(saved.machineKey, null), opts);
-    persistRunnerState({ ...saved, machineId, name: opts.name || saved.name, registeredAt: new Date().toISOString() });
+    // Re-register with the stored credential so metadata/version refresh. Keep
+    // the name this machine already goes by unless a new one was asked for —
+    // restarting the runner must not rename the machine out from under the user.
+    const name = opts.name || saved.name || os.hostname();
+    const { machineId } = await register(makeClient(saved.machineKey, null), { ...opts, name });
+    persistRunnerState({ ...saved, machineId, name, registeredAt: new Date().toISOString() });
     return { machineId, machineKey: saved.machineKey };
   }
 
@@ -281,10 +284,17 @@ async function reconcileOnce(ctx) {
   }
 }
 
+// The signal here lives as long as the runner does, so the abort listener must
+// be removed when the timer wins. Leaving it attached leaked one listener per
+// loop (~4k/day at a 20s interval) and tripped MaxListenersExceededWarning.
 function sleep(ms, signal) {
   return new Promise((resolve) => {
-    const t = setTimeout(resolve, ms);
-    if (signal) signal.addEventListener('abort', () => { clearTimeout(t); resolve(); }, { once: true });
+    const onAbort = () => { clearTimeout(t); resolve(); };
+    const t = setTimeout(() => {
+      if (signal) signal.removeEventListener('abort', onAbort);
+      resolve();
+    }, ms);
+    if (signal) signal.addEventListener('abort', onAbort, { once: true });
   });
 }