@c4t4/heyamigo 0.7.2 → 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/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@c4t4/heyamigo",
-  "version": "0.7.2",
+  "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",