@otto-assistant/otto 0.7.18 → 0.7.19

package/dist/session-handler/thread-session-runtime.js

@@ -1000,7 +1000,12 @@ export class ThreadSessionRuntime {
             })();
             logger.log(`[EVENT] type=${event.type} eventSessionId=${eventSessionId || 'none'} activeSessionId=${sessionId || 'none'} ${this.formatRunStateForLog()}${eventDetails}`);
         }
-        const isGlobalEvent = event.type === 'tui.toast.show';
+        // permission.asked and question.asked must always reach the handler
+        // so they get forwarded to Discord/Telegram even when the requesting
+        // session is a subagent whose metadata has been cleared from the buffer.
+        const isGlobalEvent = event.type === 'tui.toast.show' ||
+            event.type === 'permission.asked' ||
+            event.type === 'question.asked';
         const isScopedToastEvent = Boolean(toastSessionId);
         // Drop events that don't match current session (stale events from
         // previous sessions), unless it's a global event or a subtask session.
@@ -1818,8 +1823,11 @@ export class ThreadSessionRuntime {
         const isMainSession = permission.sessionID === sessionId;
         const isSubtaskSession = Boolean(subtaskInfo);
         if (!isMainSession && !isSubtaskSession) {
-            logger.log(`[PERMISSION IGNORED] Permission for unknown session (expected: ${sessionId} or subtask, got: ${permission.sessionID})`);
-            return;
+            logger.log(`[PERMISSION] Permission for non-main session (expected: ${sessionId} or subtask, got: ${permission.sessionID}) — forwarding to Discord`);
+            // Do not return — still show buttons for the Discord thread.
+            // The session mismatch usually happens when the event buffer has
+            // been cleared and a subagent's permission.asked arrives without
+            // subtask metadata in the buffer. The user needs to see these.
         }
         // Auto-deny external_directory permissions for paths that do not exist
         // on the filesystem. There is no point asking the user to approve access

package/package.json

@@ -7,7 +7,7 @@
   },
   "module": "index.ts",
   "type": "module",
-  "version": "0.7.18",
+  "version": "0.7.19",
   "scripts": {
     "dev": "tsx src/bin.ts",
     "prepublishOnly": "pnpm build",

package/src/session-handler/thread-session-runtime.ts

@@ -1452,7 +1452,13 @@ export class ThreadSessionRuntime {
       )
     }
 
-    const isGlobalEvent = event.type === 'tui.toast.show'
+    // permission.asked and question.asked must always reach the handler
+    // so they get forwarded to Discord/Telegram even when the requesting
+    // session is a subagent whose metadata has been cleared from the buffer.
+    const isGlobalEvent =
+      event.type === 'tui.toast.show' ||
+      event.type === 'permission.asked' ||
+      event.type === 'question.asked'
     const isScopedToastEvent = Boolean(toastSessionId)
 
     // Drop events that don't match current session (stale events from
@@ -2490,9 +2496,12 @@ export class ThreadSessionRuntime {
 
     if (!isMainSession && !isSubtaskSession) {
       logger.log(
-        `[PERMISSION IGNORED] Permission for unknown session (expected: ${sessionId} or subtask, got: ${permission.sessionID})`,
+        `[PERMISSION] Permission for non-main session (expected: ${sessionId} or subtask, got: ${permission.sessionID}) — forwarding to Discord`,
       )
-      return
+      // Do not return — still show buttons for the Discord thread.
+      // The session mismatch usually happens when the event buffer has
+      // been cleared and a subagent's permission.asked arrives without
+      // subtask metadata in the buffer. The user needs to see these.
     }
 
     // Auto-deny external_directory permissions for paths that do not exist