switchroom 0.14.51 → 0.14.53

package/dist/cli/switchroom.js

@@ -49462,8 +49462,8 @@ var {
 } = import__.default;
 
 // src/build-info.ts
-var VERSION = "0.14.51";
-var COMMIT_SHA = "53373533";
+var VERSION = "0.14.53";
+var COMMIT_SHA = "40ba8e59";
 
 // src/cli/agent.ts
 init_source();
@@ -52032,7 +52032,9 @@ function buildSettingsHooksBlock(p) {
 ` + `  - Question with a short answer \u2192 just reply with the answer.
 ` + "  - Complex tool-driven work \u2192 go straight to the tools (the " + "compose-area preview is the ambient liveness signal), then reply " + 'once with the answer or a genuine mid-work pivot ("halfway ' + 'through \u2014 found an unexpected issue, want me to continue?"). Not ' + `"still working".
 
-` + 'Do NOT send a trailing confirmation after your answer \u2014 no "Done.", ' + '"Sent.", "Hope that helps." as a separate message once you have ' + "already replied. Your answer is the last thing the user should " + `see; a follow-up "Done." is dead-air clutter (and the user's ` + "device already pinged on the answer). Stop after the answer.</turn-pacing>";
+` + 'Do NOT send a trailing confirmation after your answer \u2014 no "Done.", ' + '"Sent.", "Hope that helps." as a separate message once you have ' + "already replied. Your answer is the last thing the user should " + `see; a follow-up "Done." is dead-air clutter (and the user's ` + `device already pinged on the answer). Stop after the answer.
+
+` + 'CRITICAL: "answer" means a call to the reply tool ' + "(mcp__switchroom-telegram__reply, or stream_reply with done=true). " + "Your terminal/transcript text is NEVER delivered to Telegram \u2014 the " + "user sees only what you send through the reply tool. After a long " + "tool sequence (scheduling, multi-step research, sub-agent handback), " + "do not let your closing narration stand as the answer: end the turn " + "by passing that narration to the reply tool. No reply tool call = the " + "user got nothing, however much text you wrote.</turn-pacing>";
   const switchroomUserPromptSubmit = [
     ...useHotReloadStable ? [
       {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "switchroom",
-  "version": "0.14.51",
+  "version": "0.14.53",
   "description": "Run Claude Code 24/7 on your Claude Pro/Max subscription over Telegram. Open-source alternative to OpenClaw and NanoClaw — no API keys.",
   "type": "module",
   "bin": {

package/telegram-plugin/answer-stream-flag.ts

@@ -0,0 +1,18 @@
+/**
+ * Parse the `SWITCHROOM_VISIBLE_ANSWER_STREAM` env flag.
+ *
+ * Default **OFF** (flipped 2026-06-03, operator request) — see the rationale
+ * on the `ANSWER_STREAM_VISIBLE_ENABLED` gate in `gateway/gateway.ts`. When
+ * off, the answer lane streams to the invisible compose-box draft and the
+ * reply tool is the single canonical formatted message — no unformatted
+ * preliminary that flashes and gets deleted. Opt back IN per agent with
+ * `SWITCHROOM_VISIBLE_ANSWER_STREAM=1` (also accepts true/on/yes).
+ *
+ * Extracted as a pure function so the default + parsing are unit-testable
+ * (gateway.ts is not importable in isolation — top-level side effects).
+ */
+export function parseVisibleAnswerStreamEnabled(raw: string | undefined): boolean {
+  if (raw == null) return false
+  const v = raw.trim().toLowerCase()
+  return v === '1' || v === 'true' || v === 'on' || v === 'yes'
+}

package/telegram-plugin/dist/gateway/gateway.js

@@ -39700,6 +39700,14 @@ function createAnswerStream(config) {
   };
 }
 
+// answer-stream-flag.ts
+function parseVisibleAnswerStreamEnabled(raw) {
+  if (raw == null)
+    return false;
+  const v = raw.trim().toLowerCase();
+  return v === "1" || v === "true" || v === "on" || v === "yes";
+}
+
 // pty-tail.ts
 var import_headless = __toESM(require_xterm_headless(), 1);
 var PTY_DEBUG = process.env.SWITCHROOM_PTY_DEBUG === "1";
@@ -52158,10 +52166,10 @@ function sweepStaleTurnActiveMarker(stateDir, opts) {
 }
 
 // ../src/build-info.ts
-var VERSION = "0.14.51";
-var COMMIT_SHA = "53373533";
-var COMMIT_DATE = "2026-06-03T11:19:38Z";
-var LATEST_PR = 2127;
+var VERSION = "0.14.53";
+var COMMIT_SHA = "40ba8e59";
+var COMMIT_DATE = "2026-06-03T13:56:26Z";
+var LATEST_PR = 2131;
 var COMMITS_AHEAD_OF_TAG = 0;
 
 // gateway/boot-version.ts
@@ -54308,15 +54316,7 @@ var STREAM_THROTTLE_MS_OVERRIDE = (() => {
   return Number.isFinite(n) && n >= 0 ? n : undefined;
 })();
 var TURN_FLUSH_SAFETY_ENABLED = isTurnFlushSafetyEnabled();
-var ANSWER_STREAM_VISIBLE_ENABLED = (() => {
-  const raw = process.env.SWITCHROOM_VISIBLE_ANSWER_STREAM;
-  if (raw == null)
-    return true;
-  const v = raw.trim().toLowerCase();
-  if (v === "0" || v === "false" || v === "off" || v === "no")
-    return false;
-  return true;
-})();
+var ANSWER_STREAM_VISIBLE_ENABLED = parseVisibleAnswerStreamEnabled(process.env.SWITCHROOM_VISIBLE_ANSWER_STREAM);
 var progressDriver = null;
 var unpinProgressCardForChat = null;
 var getPinnedProgressCardMessageId = null;
@@ -57086,7 +57086,7 @@ function handleSessionEvent(ev) {
             chatId: turn.sessionChatId,
             isPrivateChat: turn.isDm,
             threadId: turn.sessionThreadId,
-            ...ANSWER_STREAM_VISIBLE_ENABLED ? { minInitialChars: 1 } : { sendMessageDraft: sendMessageDraftFn },
+            ...ANSWER_STREAM_VISIBLE_ENABLED ? { minInitialChars: 1 } : { sendMessageDraft: sendMessageDraftFn, minInitialChars: Number.MAX_SAFE_INTEGER },
             sendMessage: async (chatId, text, params) => {
               const tid = params?.message_thread_id;
               const silent = params?.purpose !== "materialize";

package/telegram-plugin/gateway/gateway.ts

@@ -96,6 +96,7 @@ import * as pendingProgress from '../pending-work-progress.js'
 import { writeSilentEndState, clearSilentEndState, recordUndeliveredTurnEnd } from '../silent-end.js'
 import { isFinalAnswerReply } from '../final-answer-detect.js'
 import { createAnswerStream, type AnswerStreamHandle } from '../answer-stream.js'
+import { parseVisibleAnswerStreamEnabled } from '../answer-stream-flag.js'
 import { type SessionEvent } from '../session-tail.js'
 import {
   shouldSuppressToolActivity,
@@ -3738,23 +3739,33 @@ const TURN_FLUSH_SAFETY_ENABLED = isTurnFlushSafetyEnabled()
 // the cross-turn pending-progress system (#1445/#1669) is the
 // canonical surface and DOES ping at the appropriate boundaries.
 //
-// 2026-05-25: default flipped ON after fleet-log audit showed
-// framework-fallback rate at ~19% of inbounds (target per
-// `reference/conversational-pacing.md`: <0.5%). Streaming the
-// model's text events live into the chat closes the
-// catastrophic-UX failure mode at the lane-of-first-defense level.
-// Aligned with the "chat IS the artifact" sub-principle (the user
-// sees a normal chat message that grows — no chrome, no parallel
-// widget). Override with SWITCHROOM_VISIBLE_ANSWER_STREAM=0 to
-// disable, e.g. for an agent that needs the legacy draft-only
-// behaviour during debugging.
-const ANSWER_STREAM_VISIBLE_ENABLED = (() => {
-  const raw = process.env.SWITCHROOM_VISIBLE_ANSWER_STREAM
-  if (raw == null) return true
-  const v = raw.trim().toLowerCase()
-  if (v === '0' || v === 'false' || v === 'off' || v === 'no') return false
-  return true
-})()
+// 2026-05-25: default flipped ON after a fleet-log audit showed a ~19%
+// framework-fallback ("still working…") rate — the visible stream gave an
+// immediate in-timeline signal that suppressed the silence-poke.
+//
+// 2026-06-03: default flipped back OFF (operator request). In practice the
+// visible stream delivered ~none of its intended benefit while imposing a
+// jarring cost:
+//   - Telegram rate-limits editMessageText to roughly once/second, so real
+//     "watch it type" streaming is impossible; and the model emits almost no
+//     interstitial assistant.text (it thinks → tool → reply), so the
+//     preliminary was a near-empty bubble (observed: 5–13 byte edits).
+//   - On every turn where the model calls the reply tool (≈always), the reply
+//     posts a SEPARATE canonical message and the stream RETRACTS (deletes) its
+//     preliminary — the user sees a raw bubble appear then vanish, replaced by
+//     the formatted reply. In supergroup topics it also mis-routed (preliminary
+//     → General, reply → topic). Net: an unformatted flash + a delete, no
+//     streaming value.
+// The anti-silence role the visible stream once filled is now covered by the
+// live ACTIVITY FEED (tool-use streaming, below), the "…typing" chat-action
+// loop, and `thinking_effort: low` (fast tool-less turns) — so off-by-default
+// no longer regresses the framework-fallback rate. With the flag off the lane
+// uses the invisible compose-box draft (the original default, #1664-compatible)
+// and the reply tool is the single canonical, formatted message.
+// Opt back IN per agent with SWITCHROOM_VISIBLE_ANSWER_STREAM=1.
+const ANSWER_STREAM_VISIBLE_ENABLED = parseVisibleAnswerStreamEnabled(
+  process.env.SWITCHROOM_VISIBLE_ANSWER_STREAM,
+)
 
 // Activity feed. The gateway streams a live "what it's doing" tool-activity
 // feed for every turn. The PreToolUse sidecar emits a `tool_label` per tool
@@ -8426,9 +8437,19 @@ function handleSessionEvent(ev: SessionEvent): void {
             // tool_use stream (case 'tool_use' above) where the real
             // signal lives. assistant.text keeps its visible-message
             // home; the reply tool stays the canonical answer.
+            // Flag OFF (default): use the compose-box draft for DMs, and set
+            // minInitialChars effectively-infinite so the lane NEVER opens a
+            // visible chat message. This matters in supergroup TOPICS, where
+            // draft transport is unsupported (gateway.ts:6422) so the lane
+            // would otherwise fall to message transport and post a visible
+            // preview once interstitial text passed the default 50-char gate
+            // — which retract() then deletes (the unformatted flash, marko
+            // General). With the gate unreachable the only posted message is
+            // the canonical reply. (The gate is bypassed for DM draft
+            // transport, so DM draft streaming is unaffected.)
             ...(ANSWER_STREAM_VISIBLE_ENABLED
               ? { minInitialChars: 1 }
-              : { sendMessageDraft: sendMessageDraftFn }),
+              : { sendMessageDraft: sendMessageDraftFn, minInitialChars: Number.MAX_SAFE_INTEGER }),
             // #1075: route through robustApiCall so flood-wait,
             // benign-400, and THREAD_NOT_FOUND are handled uniformly
             // instead of crashing the answer-stream loop on a deleted

package/telegram-plugin/tests/answer-stream-flag.test.ts

@@ -0,0 +1,27 @@
+/**
+ * Pin the SWITCHROOM_VISIBLE_ANSWER_STREAM contract: default OFF (2026-06-03),
+ * opt-in only on a truthy value. Guards against an accidental flip back to
+ * default-on (which would reintroduce the unformatted-preliminary flash +
+ * delete-on-every-reply — see the gateway gate comment).
+ */
+
+import { describe, it, expect } from 'vitest'
+import { parseVisibleAnswerStreamEnabled } from '../answer-stream-flag.js'
+
+describe('parseVisibleAnswerStreamEnabled — default OFF, opt-in', () => {
+  it('defaults OFF when unset', () => {
+    expect(parseVisibleAnswerStreamEnabled(undefined)).toBe(false)
+  })
+
+  it('stays OFF for empty / falsey / unrecognized values', () => {
+    for (const v of ['', '   ', '0', 'false', 'off', 'no', 'nope', 'enabled', 'x']) {
+      expect(parseVisibleAnswerStreamEnabled(v)).toBe(false)
+    }
+  })
+
+  it('opts IN only on explicit truthy values (case/space-insensitive)', () => {
+    for (const v of ['1', 'true', 'on', 'yes', ' TRUE ', 'On', 'YES']) {
+      expect(parseVisibleAnswerStreamEnabled(v)).toBe(true)
+    }
+  })
+})

package/telegram-plugin/tests/gateway-no-reply-single-emit.test.ts

@@ -68,6 +68,41 @@ describe('#656 — answer-stream retract() at turn_end emits nothing', () => {
     expect(deleteMessage).not.toHaveBeenCalled()
   })
 
+  // Visible-answer-stream OFF (default since 2026-06-03): the gateway passes
+  // minInitialChars=MAX_SAFE_INTEGER so a SUPERGROUP turn (message transport —
+  // draft is unsupported in forum topics) NEVER opens a visible preview, no
+  // matter how much interstitial assistant.text streams. This is the full fix
+  // for the marko-General unformatted-flash (the DM path is already covered by
+  // draft transport). Without it, message transport opens at the 50-char gate
+  // and retract() then deletes it.
+  it('flag-off supergroup: huge minInitialChars never opens a message even past the 50-char gate', async () => {
+    const sendMessage = vi.fn(async () => ({ message_id: nextMessageId++ }))
+    const editMessageText = vi.fn(async () => {})
+    const deleteMessage = vi.fn(async () => {})
+
+    const stream = createAnswerStream({
+      chatId: 'supergroup-topic',
+      isPrivateChat: false, // supergroup → message transport (no draft)
+      threadId: 4,
+      minInitialChars: Number.MAX_SAFE_INTEGER,
+      throttleMs: 250,
+      sendMessage: sendMessage as never,
+      editMessageText: editMessageText as never,
+      deleteMessage: deleteMessage as never,
+    })
+
+    // Feed 200 chars — well past the default 50-char open gate.
+    stream.update('x'.repeat(200))
+    vi.advanceTimersByTime(1000)
+    await flushMicrotasks()
+    expect(sendMessage).not.toHaveBeenCalled() // never opened a preview
+    expect(editMessageText).not.toHaveBeenCalled()
+
+    await stream.retract()
+    expect(sendMessage).not.toHaveBeenCalled()
+    expect(deleteMessage).not.toHaveBeenCalled() // nothing to delete → no flash
+  })
+
   it('retract after a preliminary send: deletes the prelim, no fresh sendMessage', async () => {
     const THROTTLE = 1000
     const sendMessage = vi.fn(async () => ({ message_id: nextMessageId++ }))