@nopeek/agent-bridge 0.7.16 → 0.7.17

package/dist/tool-progress.d.ts

@@ -7,7 +7,9 @@ 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 (read/write files, commands, skills, memory), end your reply with a short recap: what was going on, what you did, and the result. Skip the recap for simple chat with no tools. No em dashes.";
+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.";
+/** 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 {
     append(chunk: string): void;
     replace(full: string): void;
@@ -35,6 +37,8 @@ export type TurnFinishKind = "streamed" | "sent" | "empty";
 export declare function unpublishedTail(full: string, published: string): string;
 /** Unified diffs / `| review` 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;
 export declare function isToolDump(text: string): boolean;
 /** Drop dump lines from a live chunk without swallowing normal punctuation. */
 export declare function stripDumps(text: string): string;

package/dist/tool-progress.js

@@ -49,7 +49,9 @@ 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 (read/write files, commands, skills, memory), end your reply with a short recap: what was going on, what you did, and the result. Skip the recap for simple chat with no tools. No em dashes.";
+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.";
+/** 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. */
 export function unpublishedTail(full, published) {
     const f = full.trim();
@@ -95,8 +97,35 @@ export function isDumpLine(line) {
         return true;
     if (/^index [0-9a-f]+\.\.[0-9a-f]+/.test(l))
         return true;
+    // Unified-diff / review lines: "+    const x =" not "+1 country".
+    if (/^[+-]\s{2,}\S/.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))
+        return true;
     return false;
 }
+function isToolProgressLine(line) {
+    const l = line.trim();
+    if (!l)
+        return false;
+    return /^.+ \S+: ".+"$/.test(l) || /^.+ \S+\.\.\.$/.test(l);
+}
+/** True when the text has a user-readable recap, not just tools/dumps. */
+export function isHumanBriefing(text) {
+    const prose = text
+        .split("\n")
+        .filter((l) => {
+        const t = l.trim();
+        return Boolean(t) && !isDumpLine(t) && !isToolProgressLine(t);
+    })
+        .join("\n")
+        .trim();
+    if (!prose || isToolDump(prose))
+        return false;
+    return true;
+}
 export function isToolDump(text) {
     const t = text.trim();
     if (!t)
@@ -264,11 +293,17 @@ export class TurnPublisher {
         await this.commitText(this.held);
         this.held = "";
         const rest = unpublishedTail(reply.trim(), this.published);
-        if (!rest)
-            return this.published.trim() ? "streamed" : "empty";
-        this.opts.stopTyping();
-        const sent = await this.sendBubbles(rest);
-        return sent ? "sent" : this.published.trim() ? "streamed" : "empty";
+        if (rest) {
+            this.opts.stopTyping();
+            await this.sendBubbles(rest);
+        }
+        if (this.toolsUsed && reply.trim() && !isHumanBriefing(this.published)) {
+            this.opts.stopTyping();
+            await this.sendBubbles(MISSING_BRIEFING);
+        }
+        if (!this.published.trim())
+            return "empty";
+        return rest || this.toolsUsed ? "sent" : "streamed";
     }
     async fail(text) {
         this.cancelStart();
@@ -382,7 +417,11 @@ export class TurnPublisher {
                 if (extra)
                     s.append(extra);
                 const parts = splitBubbles(final, this.maxBubbleChars);
-                const first = parts[0] || final;
+                if (!parts.length) {
+                    await s.done("…").catch(() => { });
+                    return;
+                }
+                const first = parts[0];
                 await s.done(first).catch(() => { });
                 this.published += first;
                 for (const more of parts.slice(1)) {

package/package.json

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