@hachej/boring-agent 0.4.62 → 0.4.64

package/dist/conversations.js

@@ -1,6 +1,7 @@
 import { defineDoc } from '@earendil-works/pi-durable';
 import { RequestGuardError, hasJsonContentType, readJsonBody } from '@hachej/boring-files/request-guard';
 import { MENTION_FILE_PREFIX } from './mentions.js';
+import { withoutTarget } from './target-block.js';
 /** The native document. `fork: 'initial'`: a fork made by `fork()` gets its own metadata in the creating commit; any other fork starts unmanaged. */
 export const conversationMetadata = defineDoc({
     kind: 'boring.conversation.metadata', checkpointWhen: () => false, version: 1, scope: 'conversation', history: 'latest', fork: 'initial',
@@ -9,7 +10,13 @@ export const conversationMetadata = defineDoc({
 const managed = (meta, owner) => meta !== undefined && meta.owner !== '' && meta.deletedAt === null && (owner === undefined || meta.owner === owner);
 /** What the person said: a leading `<target>{…}</target>` block (point-and-speak context) never becomes a title or a preview. */
 const spoken = (text) => {
-    if (!text?.startsWith('<target>'))
+    if (text === undefined)
+        return undefined;
+    const said = withoutTarget(text);
+    if (said !== text)
+        return said.trim() || undefined;
+    // Metadata boundary: older or host-written text need not be canonical, so a `<target>` start that is not a valid block still never leaks.
+    if (!text.startsWith('<target>'))
         return text;
     const end = text.indexOf('</target>');
     return end < 0 ? undefined : text.slice(end + 9).trim() || undefined;

package/dist/struggle.js

@@ -4,6 +4,7 @@
 import { createHash } from 'node:crypto';
 import { GenerationTask, ToolTask, defineExtension, hook } from '@earendil-works/pi-durable';
 import { jsonSchemaTool } from './json-schema-tool.js';
+import { withoutTarget } from './target-block.js';
 export const REPORT_BUG_TOOL = 'report_bug';
 export const BUG_CATEGORIES = ['tool_failed', 'impossible', 'person_says_wrong'];
 /** Without a host sink an event is one structured log line, so nothing is lost before a host wires one. */
@@ -21,7 +22,7 @@ const userText = (message) => message.role !== 'user' ? '' : typeof message.cont
 const asked = (message) => message?.role === 'assistant'
     && message.content.map(part => part.type === 'text' ? part.text : '').join('').trim().endsWith('?');
 /** The person's words only: the block a point-and-speak message or a mention adds is not what they said. */
-const words = (text) => new Set(text.replace(/^<target>.*?<\/target>/s, '').replace(/<file[\s\S]*?<\/file>/g, '').toLowerCase().match(/[\p{L}\p{N}]+/gu) ?? []);
+const words = (text) => new Set(withoutTarget(text).replace(/<file[\s\S]*?<\/file>/g, '').toLowerCase().match(/[\p{L}\p{N}]+/gu) ?? []);
 function rephrased(text, earlier) {
     const said = words(text);
     if (said.size < 3)

package/dist/target-block.d.ts

@@ -0,0 +1,8 @@
+/**
+ * The `<target>` block (point-and-speak) as the kit writes it: `<target>{JSON object, at most 1024 bytes}</target>`, a blank line, then what
+ * the person said. This is the one stripper packages/agent uses (conversation titles and previews, struggle detection). It repeats the kit's
+ * `withoutTarget` (`@hachej/boring-ui-kit/chat-view`) because this package has only a type edge to ui; `test/packages/target-grammar.test.mjs`
+ * runs one table through both. A block that is not valid by that rule is plain text.
+ */
+export declare function withoutTarget(text: string): string;
+//# sourceMappingURL=target-block.d.ts.map

package/dist/target-block.js

@@ -0,0 +1,25 @@
+/**
+ * The `<target>` block (point-and-speak) as the kit writes it: `<target>{JSON object, at most 1024 bytes}</target>`, a blank line, then what
+ * the person said. This is the one stripper packages/agent uses (conversation titles and previews, struggle detection). It repeats the kit's
+ * `withoutTarget` (`@hachej/boring-ui-kit/chat-view`) because this package has only a type edge to ui; `test/packages/target-grammar.test.mjs`
+ * runs one table through both. A block that is not valid by that rule is plain text.
+ */
+export function withoutTarget(text) {
+    if (!text.startsWith('<target>'))
+        return text;
+    const end = text.indexOf('</target>');
+    if (end < 0 || text.slice(end + 9, end + 11) !== '\n\n')
+        return text;
+    const json = text.slice(8, end);
+    if (new TextEncoder().encode(json).length > 1024)
+        return text;
+    try {
+        const target = JSON.parse(json);
+        if (target === null || typeof target !== 'object' || Array.isArray(target))
+            return text;
+    }
+    catch {
+        return text;
+    }
+    return text.slice(end + 11);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hachej/boring-agent",
-  "version": "0.4.62",
+  "version": "0.4.64",
   "license": "MIT",
   "private": false,
   "type": "module",
@@ -188,8 +188,8 @@
     "@earendil-works/pi-durable": "~1.4.0",
     "@earendil-works/pi-ai": "~1.4.0",
     "@earendil-works/chord": "~1.4.0",
-    "@hachej/boring-files": "^0.4.62",
-    "@hachej/boring-ui-kit": "^0.4.62",
+    "@hachej/boring-files": "^0.4.64",
+    "@hachej/boring-ui-kit": "^0.4.64",
     "typebox": "1.3.27",
     "xlsx": "^0.20.3"
   },