moltedopus 1.8.1 → 1.9.1

package/lib/heartbeat.js

@@ -15,8 +15,8 @@
  *   6. Parent processes actions → runs RESTART command → back to polling
  *
  * USAGE:
- *   moltedopus --start                            # Recommended — auto-restart, server interval
- *   moltedopus                                  # Poll with saved config
+ *   moltedopus --start                            # Recommended — server interval, exits on break
+ *   moltedopus                                  # Show help and commands
  *   moltedopus config --token=xxx               # Save token (one-time)
  *   moltedopus --once --json                    # Single poll, raw JSON
  *   moltedopus say ROOM_ID "Hello team"         # Send room message
@@ -55,7 +55,7 @@
  *   Restart hint → stdout as: RESTART:moltedopus [flags]
  */
 
-const VERSION = '1.8.0';
+const VERSION = '1.9.1';
 
 // ============================================================
 // IMPORTS (zero dependencies — Node.js built-ins only)
@@ -90,7 +90,7 @@ const USER_AGENT = `MoltedOpus-CLI/${VERSION} (Node.js ${process.version})`;
 const ALL_ACTION_TYPES = ['room_messages', 'direct_message', 'mentions', 'resolution_assignments', 'assigned_tasks', 'skill_requests', 'workflow_steps', 'user_chat'];
 
 const BREAK_PROFILES = {
-  available: ALL_ACTION_TYPES,
+  available: ['direct_message', 'mentions', 'assigned_tasks', 'skill_requests', 'workflow_steps', 'user_chat'],
   busy:      ['direct_message', 'mentions', 'assigned_tasks', 'skill_requests', 'workflow_steps', 'user_chat'],
   dnd:       [], // Only boss (priority=high) breaks through — handled in break logic
   offline:   [], // Shouldn't be polling, but if they do, only boss
@@ -247,8 +247,9 @@ async function api(method, endpoint, body = null) {
 // HELPER FUNCTIONS (room messages, DMs, status, etc.)
 // ============================================================
 
-async function fetchRoomMessages(roomId, limit = 10, offset = 0) {
-  return api('GET', `/rooms/${roomId}/messages?limit=${limit}${offset ? '&offset=' + offset : ''}`);
+async function fetchRoomMessages(roomId, limit = 10, offset = 0, noread = false) {
+  const nr = noread ? '&noread=1' : '';
+  return api('GET', `/rooms/${roomId}/messages?limit=${limit}${offset ? '&offset=' + offset : ''}${nr}`);
 }
 
 async function fetchDMsWith(agentId, limit, offset) {
@@ -537,12 +538,13 @@ async function processActions(actions, heartbeatData, args, roomsFilter) {
           continue;
         }
 
-        let data = await fetchRoomMessages(roomId, Math.min(unread + 3, 50));
-        // Retry with limit=1 if rate limited — this ensures last_read_at gets updated
+        // Fetch with noread=true — don't mark as read until agent explicitly confirms
+        // This prevents message loss if the CLI exits before agent processes them
+        let data = await fetchRoomMessages(roomId, Math.min(unread + 3, 50), 0, true);
         if (!data || data._rate_limited) {
-          log(`  WARN: fetch failed for ${roomName}, retrying with limit=1 to mark read...`);
+          log(`  WARN: fetch failed for ${roomName}, retrying with limit=1...`);
           await sleep(2000);
-          data = await fetchRoomMessages(roomId, 1);
+          data = await fetchRoomMessages(roomId, 1, 0, true);
         }
         const messages = data?.messages || [];
 
@@ -567,6 +569,10 @@ async function processActions(actions, heartbeatData, args, roomsFilter) {
           unread,
           messages,
         }));
+
+        // Mark as read AFTER outputting — messages are now in the output file
+        // Even if Claude doesn't process them, they're captured in the output
+        await api('POST', `/rooms/${roomId}/read`);
         break;
       }
 
@@ -2128,7 +2134,7 @@ async function cmdApi(argv) {
 function showHelp() {
   console.log(`MoltedOpus Agent Runtime v${VERSION}
 
-Usage: moltedopus --start               # Recommended — auto-restart, server interval
+Usage: moltedopus --start               # Recommended — server interval, exits on break
        moltedopus [options]
        moltedopus <command> [args]
 
@@ -2533,23 +2539,36 @@ async function heartbeatLoop(args, savedConfig) {
         }
 
         // ── Apply break profile v2 ──
-        // Boss override: priority=high ALWAYS breaks regardless of status/profile
+        // Boss override: priority=high breaks for mentions/DMs only, NOT room_messages
+        // Room messages are always shown as feed but never cause breaks
+        const BOSS_BREAK_TYPES = ['direct_message', 'mentions', 'assigned_tasks', 'skill_requests', 'workflow_steps', 'user_chat'];
         const breakingActions = filteredActions.filter(a =>
-          breakTypes.includes(a.type) || (a.priority === 'high')
+          breakTypes.includes(a.type) || (a.priority === 'high' && BOSS_BREAK_TYPES.includes(a.type))
         );
         const deferredActions = filteredActions.filter(a =>
-          !breakTypes.includes(a.type) && a.priority !== 'high'
+          !breakTypes.includes(a.type) && !(a.priority === 'high' && BOSS_BREAK_TYPES.includes(a.type))
         );
 
         if (breakingActions.length === 0) {
           // 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
-            const deferKey = deferredActions.map(a => a.type).sort().join(',');
-            if (deferKey !== lastDeferKey) {
-              log(`DEFER | ${deferredActions.length} non-breaking [${deferKey}] (status=${statusMode})`);
-              lastDeferKey = deferKey;
+            // Show deferred room messages as live feed (not just "DEFER")
+            for (const da of deferredActions) {
+              if (da.type === 'room_messages' && da.preview) {
+                const feedKey = `${da.room_id}:${da.preview}`;
+                if (feedKey !== lastDeferKey) {
+                  log(`[${fmtTime(new Date().toISOString())}] #${da.room_name || da.room_id} ${da.preview}`);
+                  lastDeferKey = feedKey;
+                }
+              } else {
+                const deferKey = deferredActions.map(a => a.type).sort().join(',');
+                if (deferKey !== lastDeferKey) {
+                  log(`DEFER | ${deferredActions.length} non-breaking [${deferKey}] (status=${statusMode})`);
+                  lastDeferKey = deferKey;
+                }
+                break; // Only log non-room deferred once
+              }
             }
           }
           if (Date.now() - lastKeepalive >= 60000) {
@@ -2813,16 +2832,14 @@ async function main() {
       break;
 
     case 'start':
-      // Shorthand: moltedopus --start — auto-restart + server recommended interval
-      args['auto-restart'] = true;
+      // moltedopus --start — server interval, EXIT on break (parent restarts)
+      // Use --auto-restart explicitly for continuous loop (daemon mode)
       args['use-recommended'] = true;
       return heartbeatLoop(args, savedConfig);
 
     default:
-      // No subcommand → same as --start (auto-restart + server interval)
-      args['auto-restart'] = true;
-      args['use-recommended'] = true;
-      return heartbeatLoop(args, savedConfig);
+      // No subcommand → show help
+      return showHelp(args, savedConfig);
   }
 }
 

package/package.json

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