@inceptionstack/roundhouse 0.5.3 → 0.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inceptionstack/roundhouse",
-  "version": "0.5.3",
+  "version": "0.5.4",
   "type": "module",
   "description": "Multi-platform chat gateway that routes messages through a configured AI agent",
   "license": "MIT",

package/src/agents/pi/pi-adapter.ts

@@ -205,6 +205,23 @@ export const createPiAgentAdapter: AgentAdapterFactory = (config) => {
       } else {
         console.log(`[pi-agent] no memory extension detected — roundhouse memory will manage`);
       }
+
+      // Warn about pi extensions that bridge a chat platform directly.
+      // They hijack agent_start/message_update/agent_end and short-circuit
+      // Roundhouse's streaming pipeline — Telegram shows "typing" forever.
+      const conflicting = extNames.filter((n) => /pi-telegram(\b|[\/\\])/i.test(n));
+      if (conflicting.length > 0) {
+        const lines = [
+          "",
+          "\u26a0\ufe0f  CONFLICT: detected pi extension(s) that bridge a chat platform directly:",
+          ...conflicting.map((n) => `   - ${n}`),
+          "   Roundhouse already drives Telegram. Loading a bridge extension inside",
+          "   the pi session causes lost replies (typing indicator without text).",
+          "   Remove the extension from ~/.pi/agent/extensions or pi config and restart.",
+          "",
+        ];
+        for (const line of lines) console.warn(line);
+      }
     }
 
     return entry;
@@ -353,6 +370,13 @@ export const createPiAgentAdapter: AgentAdapterFactory = (config) => {
                   streamEvent = { type: "tool_end", toolName: event.toolName, toolCallId: event.toolCallId, isError: event.isError };
                 } else if (event.type === "turn_end") {
                   streamEvent = { type: "turn_end" };
+                } else if (event.type === "message_end") {
+                  // Pi records provider failures (auth, throttling, etc.) on the
+                  // assistant message instead of throwing — surface them.
+                  const msg = (event as any).message;
+                  if (msg?.role === "assistant" && msg.stopReason === "error" && msg.errorMessage) {
+                    streamEvent = { type: "model_error", message: msg.errorMessage };
+                  }
                 }
               }
 

package/src/gateway/streaming.ts

@@ -116,6 +116,8 @@ export async function handleStreaming(
   };
 
   let hasTextInCurrentTurn = false;
+  let hasContentThisTurn = false;
+  let modelErrorPosted = false;
   let eventCount = 0;
   let drainingNotified = false;
 
@@ -125,8 +127,9 @@ export async function handleStreaming(
       break;
     }
 
+    eventCount++;
+
     if (DEBUG_STREAM) {
-      eventCount++;
       const preview = event.type === "text_delta" ? `"${event.text.slice(0, 30)}"`
         : event.type === "custom_message" ? `${event.customType}:${event.content.slice(0, 30)}`
         : event.type === "tool_start" || event.type === "tool_end" ? event.toolName
@@ -139,12 +142,14 @@ export async function handleStreaming(
         ensureStream();
         currentPush!(event.text);
         hasTextInCurrentTurn = true;
+        hasContentThisTurn = true;
         break;
       }
 
       case "tool_start": {
         activeTools.set(event.toolCallId, event.toolName);
         if (!READ_ONLY_TOOLS.has(event.toolName)) usedFileModifyingTools = true;
+        hasContentThisTurn = true;
         if (verbose) {
           try { await thread.post(`${toolIcon(event.toolName)} Running \`${event.toolName}\`…`); } catch {}
         }
@@ -161,10 +166,22 @@ export async function handleStreaming(
           await flushCurrentStream();
           hasTextInCurrentTurn = false;
         }
+        hasContentThisTurn = true;
         await postWithFallback(thread, event.content);
         break;
       }
 
+      case "model_error": {
+        await flushCurrentStream();
+        hasTextInCurrentTurn = false;
+        hasContentThisTurn = true;
+        modelErrorPosted = true;
+        const safeMsg = event.message.split("\n")[0].slice(0, 400);
+        console.warn(`[roundhouse] model error: ${safeMsg}`);
+        try { await thread.post(`\u26a0\ufe0f Agent error: ${safeMsg}`); } catch {}
+        break;
+      }
+
       case "turn_end": {
         if (hasTextInCurrentTurn) {
           await flushCurrentStream();
@@ -207,5 +224,12 @@ export async function handleStreaming(
     await flushCurrentStream();
   }
 
+  // Safety net: if the entire turn produced no visible content and no error
+  // was already reported, notify the user so they don't stare at "typing" forever.
+  if (!hasContentThisTurn && !modelErrorPosted) {
+    console.warn(`[roundhouse] agent returned no content this turn (${eventCount} events received)`);
+    try { await thread.post("\u26a0\ufe0f Agent returned no response. Check roundhouse logs."); } catch {}
+  }
+
   return { usedTools: usedFileModifyingTools };
 }

package/src/types.ts

@@ -43,7 +43,8 @@ export type AgentStreamEvent =
   | { type: "draining" }
   | { type: "drain_complete" }
   | { type: "agent_end" }
-  | { type: "custom_message"; customType: string; content: string };
+  | { type: "custom_message"; customType: string; content: string }
+  | { type: "model_error"; message: string };
 
 // ── AdapterInfo ──────────────────────────────────────