@nopeek/agent-bridge 0.7.18 → 0.7.20

package/dist/bot.js

@@ -613,7 +613,16 @@ export class BotRunner {
                 }
             };
             const published = new TurnPublisher({
-                stream: () => ch.stream(),
+                stream: async () => {
+                    const s = await ch.stream();
+                    return {
+                        append: (chunk) => s.append(chunk),
+                        replace: (full) => s.replace(full),
+                        done: (final) => s.done(final),
+                        fail: (text) => s.fail(text),
+                        cancel: () => ch.recall(s.messageId),
+                    };
+                },
                 send: async (body) => {
                     await ch.send({ text: body });
                 },

package/dist/tool-progress.d.ts

@@ -15,6 +15,8 @@ export interface StreamHandle {
     replace(full: string): void;
     done(finalText?: string): Promise<unknown>;
     fail(text: string): Promise<void>;
+    /** Drop the placeholder bubble (NoPeek stream() starts as "…"). */
+    cancel(): Promise<void>;
 }
 export interface TurnSink {
     stream(): Promise<StreamHandle>;
@@ -35,7 +37,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;

package/dist/tool-progress.js

@@ -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,6 +101,9 @@ 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;
@@ -116,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) => {
@@ -132,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;
     }
@@ -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();
@@ -412,7 +445,13 @@ export class TurnPublisher {
                     s.append(extra);
                 const parts = splitBubbles(final, this.maxBubbleChars);
                 if (!parts.length) {
-                    await s.done("…").catch(() => { });
+                    const keep = stripDumps(final).trim();
+                    if (keep && isHumanBriefing(keep)) {
+                        await s.done(keep).catch(() => { });
+                        this.published += keep;
+                        return;
+                    }
+                    await s.cancel().catch(() => { });
                     return;
                 }
                 const first = parts[0];
@@ -438,8 +477,9 @@ export class TurnPublisher {
     }
     /** Final recap stays one Telegram-style bubble. Dumps still dropped. */
     async sendBriefing(text) {
-        const cleaned = stripDumps(text.replace(/\r\n/g, "\n")).trim();
-        if (!cleaned || isToolDump(cleaned))
+        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;
@@ -489,7 +529,11 @@ export class TurnPublisher {
         this.progress = null;
         const text = this.progressLines.join("\n");
         this.progressLines = [];
-        if (s && text)
+        if (!s)
+            return;
+        if (text)
             await s.done(text).catch(() => { });
+        else
+            await s.cancel().catch(() => { });
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nopeek/agent-bridge",
-  "version": "0.7.18",
+  "version": "0.7.20",
   "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",