portable-agent-layer 0.85.1 → 0.85.2

package/package.json

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

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

@@ -9,13 +9,16 @@ import { extractKeywords, similarity } from "./text-similarity";
 export type Reaction = "corrected" | "repeated" | "approved" | "new-topic" | "follow-up";
 
 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;
+  /^(?:(?: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 don'?t see (?:the|your) (?!(?:problem|issue|point|harm|difference|need|reason)\b))/i;
+
+const CLAIM_DISPUTED_RE =
+  /\byou(?:'re| are) (?:partially |partly |completely |totally )?wrong\b|\b(?:isn'?t|wasn'?t|aren'?t|weren'?t) (?:added|included|fixed|pushed|committed|applied|merged)\b/i;
 
 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;
+  /^(?:(?:understood|got it|ok(?:ay)?),?\s+)?(?: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|start|fix (?:all|them|both)|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|build|start|fix|ship)|(?:add|do|fix) (?:it )?(?:pls|please))\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;
@@ -23,11 +26,13 @@ const NEW_TOPIC_RE =
 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 FAILURE_RE = /\b(?:fail(?:ed|s|ing)?|broken|broke|errors?|crash(?:ed|es)?)\b/i;
 const REPEAT_SIMILARITY = 0.6;
 const REPEAT_MIN_KEYWORDS = 5;
 
 export function isCorrection(text: string): boolean {
-  return CORRECTION_RE.test(text.trim());
+  const trimmed = text.trim();
+  return CORRECTION_RE.test(trimmed) || CLAIM_DISPUTED_RE.test(trimmed);
 }
 
 export function isRepeat(text: string, previous: string | undefined): boolean {
@@ -39,21 +44,21 @@ 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 sentences(text: string): string[] {
+  return text
+    .split(/[.!\n]\s*/)
+    .map((s) => s.trim())
+    .filter(Boolean);
 }
 
 function isApproval(text: string): boolean {
   const trimmed = text.trim();
-  if (trimmed.includes("?") || HEDGE_RE.test(trimmed.split(/[.!?\n]/)[0])) return false;
+  if (trimmed.includes("?") || FAILURE_RE.test(trimmed)) return false;
+  if (HEDGE_RE.test(trimmed.split(/[.!?\n]/)[0])) return false;
   if (APPROVAL_RE.test(trimmed) || BARE_OK_RE.test(trimmed)) return true;
-  return isShortGoAhead(trimmed) || isShortGoAhead(lastSentence(trimmed));
+  const all = sentences(trimmed);
+  if (isShortGoAhead(all.at(-1) ?? "")) return true;
+  return isShortGoAhead(all[0] ?? "") && !HEDGE_RE.test(trimmed);
 }
 
 /** Word overlap was measured against real messages and does not separate topics; only saying so does. */