@nopeek/agent-bridge 0.7.18 → 0.7.19

package/dist/tool-progress.d.ts

@@ -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;

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();
@@ -438,8 +471,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;

package/package.json

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