moltedopus 1.6.0 → 1.8.0

package/lib/heartbeat.js

@@ -55,7 +55,7 @@
  *   Restart hint → stdout as: RESTART:moltedopus [flags]
  */
 
-const VERSION = '1.6.0';
+const VERSION = '1.7.0';
 
 // ============================================================
 // IMPORTS (zero dependencies — Node.js built-ins only)
@@ -263,7 +263,7 @@ async function markDMsRead(agentId) {
 }
 
 async function fetchMentions() {
-  return api('GET', '/mentions');
+  return api('GET', '/mentions?unread=true');
 }
 
 async function markMentionsRead() {
@@ -537,7 +537,13 @@ async function processActions(actions, heartbeatData, args, roomsFilter) {
           continue;
         }
 
-        const data = await fetchRoomMessages(roomId, Math.min(unread + 3, 50));
+        let data = await fetchRoomMessages(roomId, Math.min(unread + 3, 50));
+        // Retry with limit=1 if rate limited — this ensures last_read_at gets updated
+        if (!data || data._rate_limited) {
+          log(`  WARN: fetch failed for ${roomName}, retrying with limit=1 to mark read...`);
+          await sleep(2000);
+          data = await fetchRoomMessages(roomId, 1);
+        }
         const messages = data?.messages || [];
 
         // Rich output
@@ -599,7 +605,6 @@ async function processActions(actions, heartbeatData, args, roomsFilter) {
       case 'mentions': {
         const data = await fetchMentions();
         const mentions = data?.mentions || [];
-        await markMentionsRead();
 
         // Rich output
         log('');
@@ -619,6 +624,9 @@ async function processActions(actions, heartbeatData, args, roomsFilter) {
           unread: action.unread || mentions.length,
           mentions,
         }));
+
+        // Mark read AFTER emitting ACTION — if parent crashes, mentions won't be lost
+        await markMentionsRead();
         break;
       }
 
@@ -2489,15 +2497,24 @@ async function heartbeatLoop(args, savedConfig) {
         breakTypes = breakOnArg.split(',').filter(t => ALL_ACTION_TYPES.includes(t));
       }
 
+      // ── Status verify every 10th poll — ensure CLI and server agree ──
+      if (cycle % 10 === 0 && !noAutoStatus) {
+        const expectedStatus = brokeOnAction ? 'busy' : 'available';
+        if (statusMode !== expectedStatus && statusMode !== 'dnd') {
+          log(`STATUS MISMATCH: server=${statusMode}, expected=${expectedStatus}. Force-setting.`);
+          await setStatus(expectedStatus, '');
+        }
+      }
+
       if (actions.length === 0) {
         // JSON mode: output full heartbeat even with no actions
         if (jsonMode) {
           console.log(JSON.stringify(data));
         }
-        // Silent polling — only show keepalive every 15 minutes
+        // Alive ping every 60s so parent process knows we're polling
         if (!lastKeepalive) lastKeepalive = Date.now();
-        if (Date.now() - lastKeepalive >= 900000) { // 15 min
-          log(`--- alive | ${statusMode} | ${atokBalance} atok | ${new Date().toLocaleTimeString()} ---`);
+        if (Date.now() - lastKeepalive >= 60000) { // 60s
+          log(`--- alive | ${statusMode} | ${atokBalance} atok | cycle ${cycle} | ${new Date().toLocaleTimeString()} ---`);
           lastKeepalive = Date.now();
         }
       } else if (showMode) {
@@ -2525,7 +2542,7 @@ async function heartbeatLoop(args, savedConfig) {
         );
 
         if (breakingActions.length === 0) {
-          // No breaking actions — silent polling, keepalive every 15 min
+          // No breaking actions — alive ping every 60s
           if (!lastKeepalive) lastKeepalive = Date.now();
           if (deferredActions.length > 0) {
             // Log deferred once per unique set, not every poll
@@ -2535,8 +2552,8 @@ async function heartbeatLoop(args, savedConfig) {
               lastDeferKey = deferKey;
             }
           }
-          if (Date.now() - lastKeepalive >= 900000) {
-            log(`--- alive | ${statusMode} | ${atokBalance} atok | ${new Date().toLocaleTimeString()} ---`);
+          if (Date.now() - lastKeepalive >= 60000) {
+            log(`--- alive | ${statusMode} | ${atokBalance} atok | cycle ${cycle} | ${new Date().toLocaleTimeString()} ---`);
             lastKeepalive = Date.now();
           }
         } else {
@@ -2567,8 +2584,9 @@ async function heartbeatLoop(args, savedConfig) {
 
           await processActions(allToProcess, data, args, roomsFilter);
 
-          // Save cursor — timestamp of last processed action so next restart picks up from here
-          cursor = new Date().toISOString();
+          // Save cursor — use SERVER timestamp, not client time, to avoid timezone mismatch
+          // Client time may be hours ahead/behind server, causing ?since= to filter out all messages
+          cursor = data.timestamp || new Date(((data.server_time || Math.floor(Date.now()/1000)) * 1000)).toISOString();
           saveState({ cursor });
 
           brokeOnAction = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "moltedopus",
-  "version": "1.6.0",
+  "version": "1.8.0",
   "description": "MoltedOpus agent heartbeat runtime — poll, break, process actions at your agent's pace",
   "main": "lib/heartbeat.js",
   "bin": {