@c4t4/heyamigo 0.7.1 → 0.7.3

package/dist/memory/digest-flag.js

@@ -1,15 +1,64 @@
-const TRAILING_TAG_RE = /\[(DIGEST|JOURNAL|JOURNAL-NEW|ASYNC):\s*([^\]]+)\]\s*$/i;
+// Trailing-marker parser. Handles tags like [DIGEST: ...], [JOURNAL:...],
+// [JOURNAL-NEW:...], [ASYNC: ...] at the end of a reply.
+//
+// Uses a bracket-depth walk rather than a regex because payloads routinely
+// contain nested square brackets (e.g. [ASYNC: read a [config] file] or
+// [DIGEST: noted the [JOURNAL:x] pattern]). A naive /\[.*?\]/-style regex
+// terminates at the FIRST inner `]` and either misidentifies the tag or
+// drops it entirely. That bug leaked two real markers into user-facing
+// replies today (DIGEST ~morning, ASYNC later). This parser closes that
+// whole class of failure.
+const KINDS = ['DIGEST', 'JOURNAL', 'JOURNAL-NEW', 'ASYNC'];
+// Walk backwards from the end of the string, tracking bracket depth, to find
+// the `[` that matches the final `]`. Returns the tag kind, its payload, and
+// everything before the tag. Returns null if the tail doesn't cleanly look
+// like a supported tag — caller should stop peeling.
+function peelTrailingTag(raw) {
+    const trimmed = raw.replace(/\s+$/, '');
+    if (trimmed.length === 0)
+        return null;
+    if (trimmed[trimmed.length - 1] !== ']')
+        return null;
+    // Walk right-to-left counting depth. `]` pushes, `[` pops. The opening `[`
+    // that brings depth back to 0 is the match for the trailing `]`.
+    let depth = 0;
+    let openIdx = -1;
+    for (let i = trimmed.length - 1; i >= 0; i--) {
+        const c = trimmed[i];
+        if (c === ']') {
+            depth++;
+        }
+        else if (c === '[') {
+            depth--;
+            if (depth === 0) {
+                openIdx = i;
+                break;
+            }
+        }
+    }
+    if (openIdx < 0)
+        return null; // unbalanced — don't try to interpret
+    const inside = trimmed.slice(openIdx + 1, trimmed.length - 1);
+    const colonIdx = inside.indexOf(':');
+    if (colonIdx < 0)
+        return null;
+    const tagCandidate = inside.slice(0, colonIdx).trim().toUpperCase();
+    if (!KINDS.includes(tagCandidate))
+        return null;
+    const payload = inside.slice(colonIdx + 1).trim();
+    const remaining = trimmed.slice(0, openIdx).replace(/\s+$/, '');
+    return { kind: tagCandidate, payload, remaining };
+}
 // Peel trailing tags off the end of a reply. Supported:
 //   [DIGEST: <reason>]
 //   [JOURNAL:<slug> — <note>]         (append entry)
 //   [JOURNAL-NEW:<slug> — <purpose>]  (create journal)
 //   [ASYNC: <self-sufficient task description>]
-// Multiple tags supported, any order at the tail. Tags must be the LAST
+// Multiple tags supported in any order at the tail. Tags must be the LAST
 // thing in the reply (after trimming trailing whitespace).
 //
-// Journal pause/resume/archive is intentionally NOT a marker. If the owner
-// wants those, Claude edits the journal's index.md frontmatter directly.
-// Keeping the marker vocabulary small keeps Claude's context tight.
+// Payload can contain arbitrary characters including `[` and `]` as long as
+// the brackets are balanced within the payload.
 export function extractFlags(reply) {
     let current = reply;
     let digest = null;
@@ -17,14 +66,13 @@ export function extractFlags(reply) {
     const journalCreates = [];
     const asyncTasks = [];
     while (true) {
-        const trimmed = current.replace(/\s+$/, '');
-        const match = trimmed.match(TRAILING_TAG_RE);
-        if (!match)
+        const peeled = peelTrailingTag(current);
+        if (!peeled)
             break;
-        const kind = match[1].toUpperCase();
-        const payload = (match[2] ?? '').trim();
+        const { kind, payload, remaining } = peeled;
+        current = remaining;
         if (kind === 'DIGEST') {
-            if (digest === null)
+            if (digest === null && payload.length > 0)
                 digest = payload;
         }
         else if (kind === 'JOURNAL') {
@@ -43,7 +91,6 @@ export function extractFlags(reply) {
                 asyncTasks.unshift({ description: payload });
             }
         }
-        current = trimmed.slice(0, match.index).trimEnd();
     }
     return { clean: current, digest, journals, journalCreates, asyncTasks };
 }

package/dist/memory/preamble.js

@@ -10,6 +10,7 @@ import { ensureScaffold } from './store.js';
 import { getRoleForContext } from '../wa/whitelist.js';
 const DIGEST_REMINDER = `When something worth remembering happens (new preference, key fact, life event, changed plan), append [DIGEST: <one-line reason>] to the END of your reply. It will be stripped before sending. Flag sparingly.`;
 const JOURNAL_REMINDER = `When a message contains info for one of the journals above, append [JOURNAL:<slug> — <one-line note>] to the END of your reply. Multiple tags OK. Only use slugs listed; never invent. Full rules are in your memory instructions.`;
+const ASYNC_REMINDER = `BROWSER / SCRAPE / MULTI-STEP RESEARCH = always async. Never call browser tools (browser_navigate, browser_click, browser_take_screenshot, browser_evaluate, any mcp__*playwright*) inline — they block this chat for minutes when pages hang. Instead: send a short ack ("On it, will report back.") AND append [ASYNC: <self-sufficient task description>] at the END of your reply. The async worker has full browser access and will report back here. Even for "just one URL" — always async.`;
 function buildCriticalSection(params) {
     const { senderNumber, roleName, role, userName } = params;
     const who = userName
@@ -58,10 +59,9 @@ export function buildMemoryPreamble(params) {
         '  [AUDIO: /absolute/path/to/file.mp3]\n' +
         '  [DOCUMENT: /absolute/path/to/file.pdf]\n' +
         'The tag will be stripped from the message. Use absolute paths only.\n\n' +
-        'Browser: you have a real Chrome browser available via Playwright.\n' +
-        'You can navigate to URLs, click, fill forms, take screenshots, and read page content.\n' +
-        'Use the browser tools (mcp__playwright__*) when asked to visit websites, look something up, or take a screenshot.\n' +
-        'To send a screenshot back, take one with the browser tool, then include [IMAGE: /path/to/screenshot.png] in your reply.\n\n' +
+        'Browser (Playwright MCP): a real Chrome browser is available for navigation, clicks, forms, screenshots, page content. ' +
+        'DO NOT call browser tools inline from this main chat lane — they block the chat queue for minutes when pages stall (login walls, anti-bot, rate limits). ' +
+        'ALL browser work goes through the async lane. When a request needs browser: send a short ack AND append [ASYNC: <self-sufficient task description>] at the END of your reply. The async worker has full browser access; it will send the result back to this chat as a new message. Even a single URL check goes async. No exceptions.\n\n' +
         'File storage: if you need to save any files (screenshots, research, notes), always save them to storage/temp/. Never save files to the project root.');
     // Critical section
     sections.push(buildCriticalSection({
@@ -124,7 +124,10 @@ export function buildMemoryPreamble(params) {
     // Journals — owner-scoped, shown globally across all chats.
     const isOwner = !!config.owner.number && params.senderNumber === config.owner.number;
     const journalsBlock = isOwner ? buildJournalsPreambleBlock() : null;
-    const instructions = [DIGEST_REMINDER];
+    // ASYNC reminder goes first so it's the most prominent rule — it's the
+    // one that prevents the main chat queue from jamming on browser work.
+    // The preamble's Capabilities section also reinforces it.
+    const instructions = [ASYNC_REMINDER, DIGEST_REMINDER];
     if (journalsBlock) {
         sections.push(`[Journals: active]\n${journalsBlock}`);
         instructions.push(JOURNAL_REMINDER);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@c4t4/heyamigo",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "WhatsApp AI bot powered by Claude with long-term memory, browser control, and role-based access",
   "type": "module",
   "main": "dist/index.js",