nothumanallowed 16.0.21 → 16.0.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "16.0.21",
+  "version": "16.0.22",
   "description": "NotHumanAllowed — 38 AI agents, 80 tools, Studio (visual agentic workflows). Email, calendar, browser automation, screen capture, canvas, cron/heartbeat, Alexandria E2E messaging, GitHub, Notion, Slack, voice chat, free AI (Liara), 28 languages. Zero-dependency CLI.",
   "type": "module",
   "bin": {

package/src/constants.mjs

@@ -5,7 +5,7 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export const VERSION = '16.0.21';
+export const VERSION = '16.0.22';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/services/message-responder.mjs

@@ -752,7 +752,9 @@ class TelegramResponder {
   async start() {
     if (!this.enabled) return;
     this.running = true;
-    this.log('[Telegram] Responder started — polling for messages');
+    // Explicit version log at boot so the user can verify what's running.
+    // Critical when chasing "bot still mentes" — answers "are you on v16.0.21?".
+    this.log(`[Telegram] Responder started — VERSION ${VERSION} — polling for messages`);
     this._pollLoop();
     // Check for npm updates after 60s, then every 24h
     this._updateCheckTimer = setTimeout(() => this._scheduleUpdateCheck(), 60 * 1000);
@@ -774,8 +776,41 @@ class TelegramResponder {
 
   async _scheduleUpdateCheck() {
     await this._checkAndNotifyUpdate();
-    // Then every 24h
+    await this._checkLocalUpdateAndRestart();
+    // Then every 24h for the npm registry check + every 5 min for local install check
     this._updateCheckTimer = setInterval(() => this._checkAndNotifyUpdate(), 24 * 60 * 60 * 1000);
+    setInterval(() => this._checkLocalUpdateAndRestart(), 5 * 60 * 1000);
+  }
+
+  /**
+   * Detect that a NEW version of nha-cli has been installed on disk while
+   * this process is still running the OLD code in memory. When detected,
+   * exit cleanly so PM2 / launchd / the dispatcher respawns us on the
+   * latest code. Without this, the user runs `npm i -g nothumanallowed@latest`
+   * but the Telegram bot keeps mentes-ing with the old logic forever.
+   */
+  async _checkLocalUpdateAndRestart() {
+    try {
+      const fileURL = await import('url');
+      const here = fileURL.fileURLToPath(import.meta.url);
+      // ../../package.json relative to this file (services/message-responder.mjs)
+      const pkgPath = path.join(path.dirname(here), '..', '..', 'package.json');
+      if (!fs.existsSync(pkgPath)) return;
+      const onDisk = JSON.parse(fs.readFileSync(pkgPath, 'utf-8')).version;
+      if (!onDisk || onDisk === VERSION) return;
+      this.log(`[Telegram] Detected new install: running v${VERSION}, on-disk v${onDisk}. Restarting to pick up new code…`);
+      // Notify any active chat that we're restarting (best-effort, fire and forget)
+      try {
+        const chatIds = getAllTelegramChatIds();
+        for (const chatId of chatIds.slice(0, 3)) {
+          this._telegramCall('sendMessage', { chat_id: parseInt(chatId, 10), text: `🔄 Aggiornamento NHA v${VERSION} → v${onDisk} in corso. Torno tra 2 secondi.` }).catch(() => {});
+        }
+      } catch {}
+      // Give the message a moment to flush, then exit. PM2 / dispatcher restarts us.
+      setTimeout(() => process.exit(0), 800);
+    } catch (e) {
+      this.log(`[Telegram] Local update check failed: ${e.message}`);
+    }
   }
 
   async _checkAndNotifyUpdate() {

package/src/services/tool-executor.mjs

@@ -2018,11 +2018,45 @@ export async function executeTool(action, params, config) {
 
     case 'calendar_move': {
       const tz = Intl.DateTimeFormat().resolvedOptions().timeZone;
-      await updateEvent(config, 'primary', params.eventId, {
-        start: { dateTime: new Date(params.newStart).toISOString(), timeZone: tz },
-        end: { dateTime: new Date(params.newEnd).toISOString(), timeZone: tz },
+      // ── Resolve eventId from optional query/oldDate (v16.0.22) ──
+      // Agent models often pass {query: 'BMW', newStart} without an eventId.
+      // Instead of failing, resolve it here via listEvents so the move
+      // actually happens. Returns honest error if multiple/no matches.
+      let eventId = params.eventId;
+      if (!eventId && (params.query || params.title || params.oldDate)) {
+        const range = (() => {
+          if (params.oldDate && /^\d{4}-\d{2}-\d{2}$/.test(params.oldDate)) {
+            const [yy, mm, dd] = params.oldDate.split('-').map(n => parseInt(n, 10));
+            const from = new Date(yy, mm - 1, dd);
+            return { from, to: new Date(from.getTime() + 86400000) };
+          }
+          const from = new Date();
+          return { from, to: new Date(from.getTime() + 90 * 86400000) };
+        })();
+        const candidates = await listEvents(config, 'primary', range.from, range.to);
+        const q = String(params.query || params.title || '').toLowerCase();
+        const matching = q
+          ? candidates.filter(e => String(e.summary || '').toLowerCase().includes(q))
+          : candidates;
+        if (matching.length === 0) {
+          return `Error: no event found matching "${q || params.oldDate}" — calendar_move did NOT execute.`;
+        }
+        if (matching.length > 1) {
+          const list = matching.slice(0, 5).map((e, i) =>
+            `${i + 1}. ${(e.start || '').slice(0, 10)} ${e.summary}`).join('\n');
+          return `Error: multiple events match "${q}" — be more specific. Candidates:\n${list}`;
+        }
+        eventId = matching[0].id;
+      }
+      if (!eventId) return 'Error: eventId or query/title required.';
+      if (!params.newStart) return 'Error: newStart required.';
+      const newStartISO = new Date(params.newStart).toISOString();
+      const newEndISO = new Date(params.newEnd || (new Date(params.newStart).getTime() + 3600000)).toISOString();
+      await updateEvent(config, 'primary', eventId, {
+        start: { dateTime: newStartISO, timeZone: tz },
+        end:   { dateTime: newEndISO,   timeZone: tz },
       });
-      return `Event rescheduled to ${formatTime(params.newStart)} - ${formatTime(params.newEnd)}.`;
+      return `Event rescheduled to ${formatTime(newStartISO)} - ${formatTime(newEndISO)}.`;
     }
 
     case 'calendar_week': {