@nopeek/agent-bridge 0.7.17 → 0.7.19

package/dist/backends.js

@@ -25,8 +25,7 @@ const SOUL_TEMPLATE = `You are {{NAME}} (handle @{{HANDLE}}), a personal AI agen
 NoPeek end-to-end-encrypted messenger. A human messages you in a chat; you help;
 you reply. Everything you output is sent verbatim as a chat message.
 
-Voice: friendly, sharp, concise. This is a chat, not a report — short paragraphs
-or brief lists, no markdown headers. Answer the question directly.
+Voice: friendly, sharp, concise. This is a chat, not a report. Telegram markdown is ok (**bold**, lists, inline code, fenced copy blocks). Answer the question directly.
 
 You have your own memory of past conversations in this chat (each chat is a
 separate ongoing thread). Remember what the humans tell you.
@@ -293,7 +292,7 @@ you reply. Your replies are chat messages — every word you print is sent verba
 
 ## Voice
 - Friendly, sharp, practical. Concise by default — this is a chat, not a report.
-- Plain language. No markdown headers in replies; short paragraphs or brief lists only.
+- Plain language. Telegram markdown is ok: **bold** labels, lists, inline code, and fenced copy blocks. No # headers.
 
 ## Memory
 - This profile is yours alone: your soul (this file) and your memories

package/dist/tool-progress.d.ts

@@ -7,7 +7,7 @@ export type ToolProgressEvent = {
 /** One Telegram-style line. Running events only; completed is silent. */
 export declare function formatToolLine(ev: ToolProgressEvent): string | null;
 /** Hint so Hermes ends a tool-using turn with a recap, like Telegram. */
-export declare const RECAP_HINT = "When this turn used tools, your LAST message must be a short phone briefing: (1) what was wrong, (2) what you did, (3) the solution / current state. Never end on a tool list, a diff, code, or \"tasks completed\". Skip the briefing only for simple chat with no tools. No em dashes.";
+export declare const RECAP_HINT = "When this turn used tools, your LAST message must be ONE phone briefing (not split). Use **bold** labels and blank lines like Telegram: **What was wrong**, **What I did**, **Current state**. Put copyable commands in ``` fences. Never end on a tool list, a diff, or \"tasks completed\". Skip the briefing only for simple chat with no tools. No em dashes.";
 /** Posted when tools ran but the model never wrote a human recap. */
 export declare const MISSING_BRIEFING = "Done. I finished that work. Ask if you want the recap: what was wrong, what I did, and where things stand.";
 export interface StreamHandle {
@@ -35,7 +35,7 @@ export type TurnPublisherOpts = {
 export type TurnFinishKind = "streamed" | "sent" | "empty";
 /** Text in `full` that has not already been posted as commentary. */
 export declare function unpublishedTail(full: string, published: string): string;
-/** Unified diffs / `| review` dumps should never land in the chat. */
+/** Unified diffs / `┊ review diff` dumps should never land in the chat. */
 export declare function isDumpLine(line: string): boolean;
 /** True when the text has a user-readable recap, not just tools/dumps. */
 export declare function isHumanBriefing(text: string): boolean;
@@ -78,10 +78,12 @@ export declare class TurnPublisher {
     /** After `bubbleGapMs` of silence, seal the current bubble. Next text is new. */
     private armGap;
     private enqueue;
-    /** Open or append the current commentary/answer stream with this chunk only. */
+    /** After tools start, hold prose until finish() so the briefing is one bubble. */
     private appendCommentary;
     /** Finalize the current commentary so later tools land below it. */
     private commitText;
+    /** Final recap stays one Telegram-style bubble. Dumps still dropped. */
+    private sendBriefing;
     private sendBubbles;
     private addProgressLine;
     private closeProgress;

package/dist/tool-progress.js

@@ -49,7 +49,7 @@ function tidyLabel(raw) {
     return s.length > 80 ? `${s.slice(0, 77)}...` : s;
 }
 /** Hint so Hermes ends a tool-using turn with a recap, like Telegram. */
-export const RECAP_HINT = "When this turn used tools, your LAST message must be a short phone briefing: (1) what was wrong, (2) what you did, (3) the solution / current state. Never end on a tool list, a diff, code, or \"tasks completed\". Skip the briefing only for simple chat with no tools. No em dashes.";
+export const RECAP_HINT = "When this turn used tools, your LAST message must be ONE phone briefing (not split). Use **bold** labels and blank lines like Telegram: **What was wrong**, **What I did**, **Current state**. Put copyable commands in ``` fences. Never end on a tool list, a diff, or \"tasks completed\". Skip the briefing only for simple chat with no tools. No em dashes.";
 /** Posted when tools ran but the model never wrote a human recap. */
 export const MISSING_BRIEFING = "Done. I finished that work. Ask if you want the recap: what was wrong, what I did, and where things stand.";
 /** Text in `full` that has not already been posted as commentary. */
@@ -78,14 +78,18 @@ export function unpublishedTail(full, published) {
     return f;
 }
 const DEFAULT_MAX_BUBBLE = 480;
-/** Unified diffs / `| review` dumps should never land in the chat. */
+/** Hermes TUI activity prefix: ┊ review diff  /  │ read_file  /  | patch */
+function bareLine(line) {
+    return line.replace(/^[┊│|]\s*/, "").trim();
+}
+/** Unified diffs / `┊ review diff` dumps should never land in the chat. */
 export function isDumpLine(line) {
-    const l = line.trim();
+    const l = bareLine(line);
     if (!l)
         return false;
-    if (/^review diff\b/i.test(l))
+    if (/^review\s+diff\b/i.test(l))
         return true;
-    if (/^\|\s*(review|read|search|terminal|patch|write)\b/i.test(l))
+    if (/^(review|read|search|terminal|patch|write)\b/i.test(l) && line.trim().startsWith("|"))
         return true;
     if (/^@@\s+-/.test(l))
         return true;
@@ -97,9 +101,14 @@ export function isDumpLine(line) {
         return true;
     if (/^index [0-9a-f]+\.\.[0-9a-f]+/.test(l))
         return true;
+    // Diff of a markdown list item becomes "-- foo" / "+- foo".
+    if (/^[+-]{2}\s+\S/.test(l))
+        return true;
     // Unified-diff / review lines: "+    const x =" not "+1 country".
     if (/^[+-]\s{2,}\S/.test(l))
         return true;
+    if (/^[+-][A-Za-z_]/.test(l))
+        return true;
     if (/^[+-]\s*(import |from |export |const |let |function |class |def |if cmd\b|return )/.test(l))
         return true;
     if (/^[+-]\s*["'`{]/.test(l))
@@ -114,6 +123,8 @@ function isToolProgressLine(line) {
 }
 /** True when the text has a user-readable recap, not just tools/dumps. */
 export function isHumanBriefing(text) {
+    if (isToolDump(text))
+        return false;
     const prose = text
         .split("\n")
         .filter((l) => {
@@ -130,13 +141,19 @@ export function isToolDump(text) {
     const t = text.trim();
     if (!t)
         return false;
+    if (/review\s+diff/i.test(t))
+        return true;
     if (isDumpLine(t))
         return true;
     if (/^diff --git /m.test(t) && /^@@ /m.test(t))
         return true;
+    if (/^--\s+\S/m.test(t) && /^\+-\s+\S/m.test(t))
+        return true;
+    if (/^\s*#{1,3}\s+Voice\b/m.test(t) && /^\s*#{1,3}\s+Memory\b/m.test(t))
+        return true;
     const lines = t.split("\n");
     if (lines.length >= 6) {
-        const dumpy = lines.filter((l) => isDumpLine(l) || /^(@@ |[+-](?![+-])|\| )/.test(l)).length;
+        const dumpy = lines.filter((l) => isDumpLine(l) || /^(@@ |[+-](?![+-])|[┊│|]\s)/.test(l)).length;
         if (dumpy / lines.length >= 0.4)
             return true;
     }
@@ -148,8 +165,6 @@ export function stripDumps(text) {
         return text;
     if (!text.includes("\n"))
         return isDumpLine(text) ? "" : text;
-    if (isToolDump(text))
-        return "";
     return text
         .split("\n")
         .filter((l) => !isDumpLine(l))
@@ -162,9 +177,15 @@ export function splitBubbles(text, max = DEFAULT_MAX_BUBBLE) {
         return [];
     const paras = cleaned.split(/\n{2,}/).map((p) => p.trim()).filter(Boolean);
     const out = [];
+    let afterDump = false;
     for (const p of paras) {
-        if (isToolDump(p))
+        if (isToolDump(p) || /review\s+diff/i.test(p)) {
+            afterDump = true;
+            continue;
+        }
+        if (afterDump && !looksLikeCloseout(p))
             continue;
+        afterDump = false;
         if (p.length <= max) {
             out.push(p);
             continue;
@@ -194,6 +215,18 @@ export function splitBubbles(text, max = DEFAULT_MAX_BUBBLE) {
     }
     return out;
 }
+/** After a dump hunk, only keep an actual close-out — not leftover SOUL/skill text. */
+function looksLikeCloseout(p) {
+    if (/what was wrong|what i did|current state|^\*\*What /im.test(p))
+        return true;
+    if (/^Done\b/i.test(p.trim()))
+        return true;
+    if (/^\s*#{1,3}\s/m.test(p))
+        return false;
+    if (isToolDump(p))
+        return false;
+    return isHumanBriefing(p);
+}
 function hardWrap(s, max) {
     const out = [];
     let rest = s.trim();
@@ -275,13 +308,16 @@ export class TurnPublisher {
         if (this.seen.has(line) && this.progressLines.includes(line))
             return;
         this.seen.add(line);
+        const firstTool = !this.toolsUsed;
         this.toolsUsed = true;
         this.cancelStart();
         this.cancelGap();
         this.enqueue(async () => {
-            const pending = this.held;
-            this.held = "";
-            await this.commitText(pending);
+            if (firstTool) {
+                const pending = this.held;
+                this.held = "";
+                await this.commitText(pending);
+            }
             await this.addProgressLine(line);
         });
     }
@@ -290,16 +326,20 @@ export class TurnPublisher {
         this.cancelGap();
         await this.write;
         await this.closeProgress();
-        await this.commitText(this.held);
+        const pending = this.held;
         this.held = "";
+        if (!this.toolsUsed) {
+            await this.commitText(pending);
+        }
         const rest = unpublishedTail(reply.trim(), this.published);
-        if (rest) {
+        const briefing = rest || (this.toolsUsed ? pending.trim() : "");
+        if (briefing) {
             this.opts.stopTyping();
-            await this.sendBubbles(rest);
+            await this.sendBriefing(briefing);
         }
         if (this.toolsUsed && reply.trim() && !isHumanBriefing(this.published)) {
             this.opts.stopTyping();
-            await this.sendBubbles(MISSING_BRIEFING);
+            await this.sendBriefing(MISSING_BRIEFING);
         }
         if (!this.published.trim())
             return "empty";
@@ -354,26 +394,13 @@ export class TurnPublisher {
             console.error(`[turn] ${err.message}`);
         });
     }
-    /** Open or append the current commentary/answer stream with this chunk only. */
+    /** After tools start, hold prose until finish() so the briefing is one bubble. */
     async appendCommentary(chunk) {
         if (!chunk)
             return;
         if (this.toolsUsed) {
             await this.closeProgress();
             this.held += chunk;
-            const cut = this.held.lastIndexOf("\n\n");
-            if (cut >= 0) {
-                const ready = this.held.slice(0, cut);
-                this.held = this.held.slice(cut + 2);
-                this.opts.stopTyping();
-                await this.sendBubbles(ready);
-            }
-            else if (this.held.length >= this.maxBubbleChars) {
-                const ready = this.held;
-                this.held = "";
-                this.opts.stopTyping();
-                await this.sendBubbles(ready);
-            }
             return;
         }
         this.opts.stopTyping();
@@ -442,6 +469,16 @@ export class TurnPublisher {
         this.opts.stopTyping();
         await this.sendBubbles(pending);
     }
+    /** Final recap stays one Telegram-style bubble. Dumps still dropped. */
+    async sendBriefing(text) {
+        const parts = splitBubbles(text.replace(/\r\n/g, "\n"), 4000);
+        const cleaned = parts.join("\n\n").trim();
+        if (!cleaned || isToolDump(cleaned) || !isHumanBriefing(cleaned))
+            return false;
+        await this.sink.send(cleaned);
+        this.published += this.published ? `\n\n${cleaned}` : cleaned;
+        return true;
+    }
     async sendBubbles(text) {
         const parts = splitBubbles(text, this.maxBubbleChars);
         if (!parts.length)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nopeek/agent-bridge",
-  "version": "0.7.17",
+  "version": "0.7.19",
   "description": "Run your own agents as E2EE NoPeek bots. Pairs with one-time codes (multiple accounts per computer), runs every bot each account owns, and pipes messages to any command or webhook.",
   "type": "module",
   "license": "MIT",