portable-agent-layer 0.81.0 → 0.81.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "portable-agent-layer",
-  "version": "0.81.0",
+  "version": "0.81.1",
   "description": "PAL — Portable Agent Layer: persistent personal context for AI coding assistants",
   "type": "module",
   "bin": {

package/src/hooks/lib/interaction-reaction.ts

@@ -8,11 +8,6 @@ import { extractKeywords, similarity } from "./text-similarity";
 
 export type Reaction = "corrected" | "repeated" | "approved" | "new-topic" | "follow-up";
 
-export interface ReplyContext {
-  previousPrompt?: string;
-  replyKeywords: string[];
-}
-
 const CORRECTION_RE =
   /^(?:(?:no|nope|wrong)(?:[,.!]|$)|(?:that'?s|this is|it'?s) (?:wrong|not)\b|not what i\b|i said\b|i told you\b|you forgot\b|you missed\b|still (?:broken|failing|wrong)\b)/i;
 
@@ -20,15 +15,16 @@ const APPROVAL_RE =
   /^(?:good|great|nice|perfect|cool|amazing|awesome|excellent|brilliant|works|it works|thanks|thank you|thx|y+e+s+|yep|yeah|yup|sure|exactly|correct|agreed|lgtm|well done|love it|sounds good|looks good|that'?s it|(?:it|this|that) (?:is|'s) (?:fine|good|great|right))\b/i;
 
 const GO_AHEAD_RE =
-  /^(?:go ahead|go|do it|proceed|continue|merge|push|ship|commit|build|approved|let'?s (?:see|do|go))\b/i;
+  /^(?:go ahead|go|do it|proceed|continue|merge|push|ship|commit|build|start|fix (?:all|them|both)|approved|let'?s (?:see|do|go))\b/i;
+
+const NEW_TOPIC_RE =
+  /^(?:(?:ok(?:ay)?|btw|also),?\s+)?(?:new (?:\w+ ){0,2}(?:task|topic|question|thing)|(?:different|unrelated|separate|another) (?:task|topic|question|thing)|unrelated\b|switching (?:gears|topics?)|change of (?:topic|subject))/i;
 
 const GO_AHEAD_MAX_WORDS = 6;
 const BARE_OK_RE = /^(?:ok(?:ay)?|k)[,.!]?(?:\s+\S+){0,2}$/i;
 const HEDGE_RE = /\b(?:but|however|though|except|instead)\b/i;
 const REPEAT_SIMILARITY = 0.6;
 const REPEAT_MIN_KEYWORDS = 5;
-const NEW_TOPIC_MIN_KEYWORDS = 4;
-const NEW_TOPIC_MAX_OVERLAP = 0.15;
 
 export function isCorrection(text: string): boolean {
   return CORRECTION_RE.test(text.trim());
@@ -39,33 +35,36 @@ export function isRepeat(text: string, previous: string | undefined): boolean {
   return similarity(text, previous) >= REPEAT_SIMILARITY;
 }
 
+function isShortGoAhead(sentence: string): boolean {
+  return GO_AHEAD_RE.test(sentence) && sentence.split(/\s+/).length <= GO_AHEAD_MAX_WORDS;
+}
+
+function lastSentence(text: string): string {
+  return (
+    text
+      .split(/[.!\n]\s*/)
+      .filter(Boolean)
+      .at(-1)
+      ?.trim() ?? ""
+  );
+}
+
 function isApproval(text: string): boolean {
   const trimmed = text.trim();
-  if (HEDGE_RE.test(trimmed.split(/[.!?\n]/)[0])) return false;
+  if (trimmed.includes("?") || HEDGE_RE.test(trimmed.split(/[.!?\n]/)[0])) return false;
   if (APPROVAL_RE.test(trimmed) || BARE_OK_RE.test(trimmed)) return true;
-  return GO_AHEAD_RE.test(trimmed) && trimmed.split(/\s+/).length <= GO_AHEAD_MAX_WORDS;
+  return isShortGoAhead(trimmed) || isShortGoAhead(lastSentence(trimmed));
 }
 
-/** Few of the message's words appear in the exchange it follows. */
-function isNewTopic(text: string, context: ReplyContext): boolean {
-  const words = extractKeywords(text);
-  if (words.size < NEW_TOPIC_MIN_KEYWORDS) return false;
-  const exchange = new Set([
-    ...context.replyKeywords,
-    ...extractKeywords(context.previousPrompt ?? ""),
-  ]);
-  const shared = [...words].filter((w) => exchange.has(w)).length;
-  return shared / words.size <= NEW_TOPIC_MAX_OVERLAP;
+/** Word overlap was measured against real messages and does not separate topics; only saying so does. */
+function isNewTopic(text: string): boolean {
+  return NEW_TOPIC_RE.test(text.trim());
 }
 
-export function reactionTo(text: string, context: ReplyContext): Reaction {
+export function reactionTo(text: string, previousPrompt: string | undefined): Reaction {
   if (isCorrection(text)) return "corrected";
-  if (isRepeat(text, context.previousPrompt)) return "repeated";
+  if (isRepeat(text, previousPrompt)) return "repeated";
   if (isApproval(text)) return "approved";
-  if (isNewTopic(text, context)) return "new-topic";
+  if (isNewTopic(text)) return "new-topic";
   return "follow-up";
 }
-
-export function replyKeywords(reply: string): string[] {
-  return [...extractKeywords(reply)];
-}

package/src/hooks/lib/interaction.ts

@@ -19,7 +19,6 @@ import {
   isRepeat,
   type Reaction,
   reactionTo,
-  replyKeywords,
 } from "./interaction-reaction";
 import { keepSample, replyEnd } from "./interaction-samples";
 import { ensureDir, paths } from "./paths";
@@ -50,7 +49,6 @@ interface SessionTrack {
   lastPrompt?: string;
   lastPromptAt?: string;
   reply?: ReplyShape;
-  replyKeywords?: string[];
   replyEnd?: string;
   recent: TurnEvent[];
   mood?: string;
@@ -117,12 +115,7 @@ function measureTurn(
     repeated: isRepeat(text, track.lastPrompt),
     corrected: isCorrection(text),
     reply: reply ? replyFeatures(reply) : null,
-    reaction: reply
-      ? reactionTo(text, {
-          previousPrompt: track.lastPrompt,
-          replyKeywords: track.replyKeywords ?? [],
-        })
-      : null,
+    reaction: reply ? reactionTo(text, track.lastPrompt) : null,
   };
 }
 
@@ -211,7 +204,6 @@ export function recordReply(
     ...track,
     updated: now.toISOString(),
     reply: replyShape(reply, now),
-    replyKeywords: replyKeywords(reply),
     replyEnd: replyEnd(reply),
   });
 }