switchroom 0.16.38 → 0.16.47

package/telegram-plugin/card-format.ts

@@ -28,6 +28,7 @@
  */
 
 import { escapeMarkdown, codeSpanSafe } from './format.js'
+import { normalizeDashes } from './text-voice-scrub.js'
 
 /**
  * Re-export so card surfaces have one import for the markdown escaper they
@@ -127,7 +128,12 @@ export function truncate(s: string, n: number): string {
  * Run this BEFORE `truncate` + `escapeMarkdown`: clean → measure → escape.
  */
 export function stripMarkdown(s: string): string {
-  let out = s;
+  // Close the em-dash leak on this model-authored-prose surface using the
+  // SAME dash logic as the reply-path voice scrub (no duplicated regex). Run
+  // it FIRST, while code spans (`` `…` ``) still have their backticks intact,
+  // so `normalizeDashes`'s own code-region masking preserves a dash inside a
+  // code span before the backtick-stripping below erases the fences.
+  let out = normalizeDashes(s);
   // Inline + leftover code spans → bare text.
   out = out.replace(/`+/g, '');
   // Links / images: [text](url) and ![alt](url) → the label.

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

@@ -23216,6 +23216,23 @@ function projectAssistantTextBlocks(content, make) {
   });
   return out;
 }
+function assistantLineCarriesAnswerSurface(content) {
+  if (!Array.isArray(content))
+    return false;
+  for (const c of content) {
+    const ct = c?.type ?? "";
+    if (ct === "text") {
+      const t = c.text;
+      if (typeof t === "string" && t.trim().length > 0)
+        return true;
+    } else if (ct === "tool_use") {
+      const name = c.name ?? "";
+      if (name !== "Agent" && name !== "Task")
+        return true;
+    }
+  }
+  return false;
+}
 function projectTranscriptLine(line) {
   if (line.length > MAX_JSONL_LINE_BYTES)
     return [];
@@ -23377,7 +23394,7 @@ function projectSubagentLine(line, agentId, state) {
       }
     });
     const stopReason = message?.stop_reason;
-    if (stopReason === "end_turn") {
+    if (stopReason === "end_turn" && assistantLineCarriesAnswerSurface(content)) {
       events.push({ kind: "sub_agent_turn_end", agentId });
     }
     return events;
@@ -24977,7 +24994,8 @@ var TOOL_SCHEMAS = [
         chat_id: { type: "string", description: "Chat to render the approval card in (use the chat_id of the user message that triggered the workflow)." },
         key: { type: "string", description: "Vault key the agent wants access to (matches the key shown in the VAULT-BROKER-DENIED error, e.g. `fatsecret/credentials`)." },
         scope: { type: "string", enum: ["read", "write"], description: 'Access scope: "read" (default) for `vault:<key>` references; "write" if the agent needs to put new values.' },
-        reason: { type: "string", description: `REQUIRED in practice \u2014 short human-readable rationale rendered on the card (e.g. "to look up today's food log entries"). The approval card now renders "why: not provided" when this is omitted, which signals to the operator that the agent skipped its explanation \u2014 they will usually Deny. Always supply a one-line rationale.` },
+        reason: { type: "string", description: 'REQUIRED in practice \u2014 short human-readable rationale rendered on the card (e.g. "to look up today\'s food log entries"). The approval card now renders "why: not provided" when this is omitted, which signals to the operator that the agent skipped its explanation \u2014 they will usually Deny. Always supply a one-line rationale. `why` is accepted as an alias (matching `vault_request_save`).' },
+        why: { type: "string", description: "Alias for `reason` (matches the sibling tool `vault_request_save`). If both are supplied, `reason` wins." },
         duration: { type: "string", description: 'Requested grant TTL, like "30d" or "12h". Default 30d, capped at 90d. Beyond 90d the operator should use the host CLI explicitly.' },
         message_thread_id: { type: "string", description: "Forum topic thread ID. Auto-applied from the last inbound message if not specified." }
       },