Split x_conversations to return brief message to agent

- x_conversations now returns only "X conversations awaiting reply"
- x_get_conversations (app-only) returns full JSON data
- UI fetches data via x_get_conversations, agent sees minimal output
- This prevents agent from summarizing conversation data

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: nearestnabors

src/server.ts

@@ -25,7 +25,7 @@ import { fileURLToPath } from 'url';
 import { xAuthStatus } from './tools/auth-status.js';
 import { xDraftTweet } from './tools/draft-tweet.js';
 import { xPostTweet } from './tools/post-tweet.js';
-import { xConversations } from './tools/conversations.js';
+import { xConversations, xGetConversations } from './tools/conversations.js';
 import { xDismissConversation } from './tools/dismiss-conversation.js';
 
 const __filename = fileURLToPath(import.meta.url);
@@ -165,7 +165,7 @@ const toolHandlers: Record<string, ToolHandler> = {
   x_draft_tweet: xDraftTweet,
   x_post_tweet: xPostTweet,
   x_conversations: xConversations,
-  x_get_conversations: xConversations, // Same handler, UI calls this to get data
+  x_get_conversations: xGetConversations, // UI-only, returns full data
   x_dismiss_conversation: xDismissConversation,
 };
 

src/tools/conversations.ts

@@ -82,7 +82,12 @@ interface ConversationItem {
  * Tool: x_conversations
  * Fetches unreplied mentions and returns data for the conversation list UI
  */
-export async function xConversations(): Promise<unknown> {
+type FetchResult =
+  | { success: true; data: { conversations: ConversationItem[]; username: string } }
+  | { success: false; content: unknown };
+
+// Internal function that fetches and returns conversation data
+async function fetchConversations(): Promise<FetchResult> {
   // Clean up expired dismissals
   const prunedDismissals = pruneExpiredDismissals();
   if (prunedDismissals > 0) {
@@ -94,12 +99,15 @@ export async function xConversations(): Promise<unknown> {
   // Check if username is set
   if (!username) {
     return {
-      content: [
-        {
-          type: 'text',
-          text: `I need your X username to find your conversations.\n\nPlease check your auth status with x_auth_status first - your username should be detected automatically after authentication.`,
-        },
-      ],
+      success: false,
+      content: {
+        content: [
+          {
+            type: 'text',
+            text: `I need your X username to find your conversations.\n\nPlease check your auth status with x_auth_status first - your username should be detected automatically after authentication.`,
+          },
+        ],
+      },
     };
   }
 
@@ -127,19 +135,12 @@ export async function xConversations(): Promise<unknown> {
     });
 
     if (mentions.length === 0) {
-      // Return empty conversation data for UI
-      const emptyData = {
-        conversations: [],
-        username,
-      };
-
       return {
-        content: [
-          {
-            type: 'text',
-            text: JSON.stringify(emptyData),
-          },
-        ],
+        success: true,
+        data: {
+          conversations: [],
+          username,
+        },
       };
     }
 
@@ -269,33 +270,29 @@ export async function xConversations(): Promise<unknown> {
     // Update last checked timestamp
     updateLastChecked();
 
-    // Return JSON data for the conversation-list UI app
-    const conversationsData = {
-      conversations,
-      username,
-    };
-
     return {
-      content: [
-        {
-          type: 'text',
-          text: JSON.stringify(conversationsData),
-        },
-      ],
+      success: true,
+      data: {
+        conversations,
+        username,
+      },
     };
   } catch (error) {
     console.error('[ASSA] Error fetching conversations:', error);
 
     // Check for auth errors using AuthRequiredError
     if (error instanceof AuthRequiredError) {
       return {
-        content: [
-          {
-            type: 'text',
-            text: `X authorization required. Please use the x_auth_status tool to connect your account.`,
-          },
-        ],
-        isError: true,
+        success: false,
+        content: {
+          content: [
+            {
+              type: 'text',
+              text: `X authorization required. Please use the x_auth_status tool to connect your account.`,
+            },
+          ],
+          isError: true,
+        },
       };
     }
 
@@ -307,24 +304,75 @@ export async function xConversations(): Promise<unknown> {
       errorMessage.toLowerCase().includes('unauthorized')
     ) {
       return {
+        success: false,
+        content: {
+          content: [
+            {
+              type: 'text',
+              text: `X authorization required. Please use the x_auth_status tool to connect your account.`,
+            },
+          ],
+          isError: true,
+        },
+      };
+    }
+
+    return {
+      success: false,
+      content: {
         content: [
           {
             type: 'text',
-            text: `X authorization required. Please use the x_auth_status tool to connect your account.`,
+            text: `Error fetching conversations: ${errorMessage}`,
           },
         ],
         isError: true,
-      };
-    }
-
-    return {
-      content: [
-        {
-          type: 'text',
-          text: `Error fetching conversations: ${errorMessage}`,
-        },
-      ],
-      isError: true,
+      },
     };
   }
 }
+
+/**
+ * Tool: x_conversations (for agent)
+ * Returns a brief message - UI fetches full data via x_get_conversations
+ */
+export async function xConversations(): Promise<unknown> {
+  const result = await fetchConversations();
+
+  if (!result.success) {
+    return result.content;
+  }
+
+  const count = result.data.conversations.length;
+  return {
+    content: [
+      {
+        type: 'text',
+        text: count === 0
+          ? 'No conversations awaiting your reply.'
+          : `${count} conversation${count === 1 ? '' : 's'} awaiting your reply.`,
+      },
+    ],
+  };
+}
+
+/**
+ * Tool: x_get_conversations (for UI only, hidden from model)
+ * Returns full conversation data as JSON
+ */
+export async function xGetConversations(): Promise<unknown> {
+  const result = await fetchConversations();
+
+  if (!result.success) {
+    return result.content;
+  }
+
+  return {
+    content: [
+      {
+        type: 'text',
+        text: JSON.stringify(result.data),
+      },
+    ],
+  };
+}

ui-apps/conversation-list.html

@@ -11,6 +11,11 @@
         padding: 0;
       }
 
+      html, body {
+        height: fit-content;
+        min-height: 0;
+      }
+
       body {
         font-family:
           -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, sans-serif;
@@ -20,6 +25,7 @@
 
       .container {
         max-width: 600px;
+        height: fit-content;
       }