@chatpanel/events 0.18.0 → 0.19.0

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@chatpanel/events",
-  "version": "0.18.0",
-  "description": "The canonical ChatPanel event-log and capability contracts \u2014 typed durable facts, clock-free deterministic linearization, schema upcasting, and the invariants the replay harness asserts. Pure, dependency-free ESM shared by the ChatPanel extension, gateway and bridge.",
+  "version": "0.19.0",
+  "description": "The canonical ChatPanel event-log and capability contracts — typed durable facts, clock-free deterministic linearization, schema upcasting, and the invariants the replay harness asserts. Pure, dependency-free ESM shared by the ChatPanel extension, gateway and bridge.",
   "type": "module",
   "main": "index.js",
   "exports": {

package/schedule.js

@@ -152,6 +152,32 @@ export function createTriggerRegistry(triggers = []) {
 const words = (s) => String(s || '').toLowerCase().match(/[\p{L}\p{N}']+/gu) || [];
 const norm = (s) => String(s || '').trim().toLowerCase();
 
+/** Shorter than this and ordinary speech contains it constantly. */
+export const MIN_PHRASE_CHARS = 3;
+
+const escapeRe = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+
+/**
+ * Whole-word containment, not `includes`.
+ *
+ * A substring match turns "in" into a trigger that fires on interview, thing, going and
+ * finding — which is indistinguishable, from the user's chair, from a trigger that ignores
+ * its phrase entirely. That is exactly how it was reported: "for every utterance I say, it
+ * runs". Boundaries are only required at ends that are word characters, so "action item"
+ * still matches "an action item," and ":shipped" still matches ":shipped".
+ */
+export function saidIn(text, phrase) {
+  const p = norm(phrase);
+  if (p.length < MIN_PHRASE_CHARS) return false;
+  const left = /[\p{L}\p{N}]/u.test(p[0]) ? '\\b' : '';
+  const right = /[\p{L}\p{N}]/u.test(p[p.length - 1]) ? '\\b' : '';
+  try {
+    return new RegExp(`${left}${escapeRe(p)}${right}`, 'iu').test(String(text || ''));
+  } catch {
+    return norm(text).includes(p); // a phrase that will not compile still gets a plain match
+  }
+}
+
 // Whose speech a meeting trigger cares about. The default is 'anyone' because a phrase
 // trigger is usually watching for what OTHERS say — unlike a spoken command, which is only
 // ever the owner's (see voice-intents.js).
@@ -161,6 +187,60 @@ function speakerAllowed(want, speaker, ctx) {
   return true;
 }
 
+/**
+ * Words a sentence does not end on.
+ *
+ * Not a grammar: a short list of the tails that mean "the speaker is mid-thought". A caption
+ * ending on one of these is a line that has not finished arriving.
+ */
+export const DANGLING_TAILS = Object.freeze(new Set([
+  // conjunctions and subordinators — the reason, the contrast, the condition all come AFTER
+  'and', 'or', 'but', 'so', 'because', 'cause', 'cos', 'since', 'although', 'though', 'while',
+  'whereas', 'unless', 'until', 'if', 'when', 'whenever', 'that', 'which', 'who', 'whom',
+  'whose', 'than', 'as', 'like',
+  // prepositions and particles
+  'to', 'of', 'in', 'on', 'at', 'by', 'for', 'from', 'with', 'without', 'about', 'into',
+  'onto', 'over', 'under', 'between', 'through', 'during', 'against', 'per',
+  // determiners that must be followed by something. Only the ones that genuinely cannot end
+  // a sentence — 'this', 'some' and 'both' all can ("take both"), so they are not here.
+  'a', 'an', 'the', 'my', 'our', 'your', 'their', 'its',
+  // auxiliaries and copulas left hanging
+  'is', 'are', 'was', 'were', 'be', 'been', 'being', 'am', 'do', 'does', 'did', 'have', 'has',
+  'had', 'will', 'would', 'can', 'could', 'should', 'shall', 'may', 'might', 'must',
+  // fillers a speaker trails off on. Pronouns are NOT here: "we should ship it" is a whole
+  // thought, and treating every object pronoun as a trailing word made the common case wait.
+  'um', 'uh', 'er',
+]));
+
+/**
+ * Does this line read as a FINISHED thought?
+ *
+ * A live caption is delivered while it is still being spoken and GROWS across flushes, so a
+ * trigger that matched can be holding half a sentence: "the product is just amazing because"
+ * — with the reason, the only part worth acting on, still unsaid. The caller uses this to
+ * decide whether to read the transcript now or wait for the rest of it.
+ *
+ * DEFAULT TRUE, deliberately. Speech-to-text frequently emits no punctuation at all, so
+ * requiring a full stop would make every job wait every time — and the complaint this exists
+ * to fix is about missing context, not about speed. Only a line that ends the way an
+ * unfinished one ends is treated as unfinished.
+ *
+ * Pure and synchronous, like every other predicate in this file: the waiting is the caller's
+ * problem, and it is the only part that needs a platform.
+ */
+export function utteranceLooksComplete(text) {
+  const t = String(text || '').trim();
+  if (!t) return false;
+  // Sentence-final punctuation, optionally inside a closing quote or bracket.
+  if (/[.!?…]["'\u2019\u201d)\]]*$/.test(t)) return true;
+  // A comma, dash or colon at the end is a speaker who is explicitly not done.
+  if (/[,;:\-\u2013\u2014]$/.test(t)) return false;
+  const tokens = t.toLowerCase().match(/[\p{L}\p{N}']+/gu) || [];
+  const last = tokens[tokens.length - 1];
+  if (!last) return true;
+  return !DANGLING_TAILS.has(last);
+}
+
 export const timerTrigger = defineTrigger({
   id: 'timer:schedule',
   label: 'On a schedule',
@@ -208,12 +288,14 @@ export const phraseTrigger = defineTrigger({
   kind: 'meeting',
   watches: ['meeting.transcript.delta'],
   matches: (event, params = {}, ctx = {}) => {
-    const any = (params.any || []).map(norm).filter(Boolean);
-    if (!any.length) return null; // a phrase trigger with no phrase would fire on every word
+    // Both guards exist because their absence looks the same to a user: a trigger that fires
+    // on everything. An empty list would match every line; a one- or two-letter phrase
+    // effectively does too.
+    const any = (params.any || []).map(norm).filter((p) => p.length >= MIN_PHRASE_CHARS);
+    if (!any.length) return null;
     for (const seg of event.segments || []) {
       if (!speakerAllowed(params.speaker, seg.speaker, ctx)) continue;
-      const text = norm(seg.text);
-      const hit = any.find((p) => text.includes(p));
+      const hit = any.find((p) => saidIn(seg.text, p));
       if (hit) return { why: `“${hit}” said by ${seg.speaker || 'someone'}`, segment: seg, phrase: hit };
     }
     return null;
@@ -253,7 +335,11 @@ export const questionTrigger = defineTrigger({
       // Other people by default: this exists for reacting to what someone ELSE asks (an
       // interview, a customer call). The job form offers the choice, so 'anyone' and 'me' are
       // one dropdown away rather than an invisible default nobody can reach.
-      if (!speakerAllowed(params.speaker || 'others', seg.speaker, ctx)) continue;
+      // ANYONE by default, including you. 'others' read well — this exists for reacting to
+      // what someone ELSE asks — but it made the first thing anybody does (test it alone in a
+      // call, ask a question, wait) match nothing, so the feature looked dead. "Only what
+      // other people ask" is still one dropdown away in the job form.
+      if (!speakerAllowed(params.speaker || 'anyone', seg.speaker, ctx)) continue;
       const text = String(seg.text || '').trim();
       if (text.length < 8) continue; // "what?" is not a question worth waking a model for
       if (text.includes('?') || QUESTION.test(text)) return { why: `question from ${seg.speaker || 'someone'}`, segment: seg };