portable-agent-layer 0.85.1 → 0.85.3

package/assets/schema/pal-settings.schema.json

@@ -70,6 +70,10 @@
           "type": "boolean",
           "description": "One line of the current time with every prompt, in identity.principal.timezone, so a session resumed hours later knows the day changed."
         },
+        "dueReminders": {
+          "type": "boolean",
+          "description": "Opt-in, off by default until 1.0. Due maintenance reminders (learning analysis, relationship reflect) with every prompt until a reply names their command; then quiet until the next day."
+        },
         "interactionAwareness": {
           "type": "boolean",
           "description": "Measure each user turn (pace, message length, skimming, corrections; never the text) into memory/signals/interaction/, and tell the agent when the session's pattern changes."

package/assets/templates/pal-settings.json

@@ -26,6 +26,7 @@
     "contextualSteering": true,
     "skillMatching": true,
     "wallClock": true,
+    "dueReminders": false,
     "interactionAwareness": true,
     "reactionSampling": true,
     "steeringTestReport": false

package/package.json

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

package/src/hooks/handlers/inject-retrieval.ts

@@ -8,6 +8,7 @@
  */
 
 import { promptContextResponse } from "../lib/agent";
+import { dueNudgeReminder } from "../lib/daily-nudge";
 import { observeTurn } from "../lib/interaction";
 import { logDebug, logError } from "../lib/log";
 import { parkPromptContext } from "../lib/parked-context";
@@ -81,6 +82,7 @@ export async function getPromptContext(
   const parts = [
     getWallClockReminder(),
     withinBudget(() => observeTurn(prompt, sessionId), BUDGET_MS),
+    withinBudget(() => dueNudgeReminder(), BUDGET_MS),
     getSteeringReminder(prompt),
     getSkillReminder(prompt),
     await getRetrievalReminder(prompt),

package/src/hooks/lib/context.ts

@@ -5,16 +5,12 @@
 
 import { existsSync, readFileSync } from "node:fs";
 import { resolve } from "node:path";
-import { loadReflectNudge } from "../handlers/reflect-trigger";
-import { loadAlgorithmReviewNudge } from "./algorithm-review";
-import { loadAnalyzeNudge } from "./analyze-nudge";
 import { resolveAnchor } from "./anchor";
-import { oncePerDay } from "./daily-nudge";
+import { dueNudgeSections } from "./daily-nudge";
 import { loadHandoffContext } from "./handoff-context";
 import { loadOpinionContext } from "./opinions";
 import { paths, toPath } from "./paths";
 import { loadActiveProjectsContext } from "./projects";
-import { loadReactionAuditNudge } from "./reaction-audit";
 import { loadRecentNotes } from "./relationship";
 import { loadFailurePatterns } from "./semi-static";
 import { sessionDir } from "./session-dir";
@@ -219,18 +215,11 @@ export function buildSystemReminder(
     !opts.withoutHandoff && settings.isEnabled("handoff")
       ? loadHandoffContext(sessionDir())
       : "";
-  // Maintainer-only: self-gates to a repo checkout, "" for everyone else.
-  const algoReview = oncePerDay("algorithm-review", loadAlgorithmReviewNudge());
-  const reactionAudit = oncePerDay("reaction-audit", loadReactionAuditNudge());
-  const reflectNudge = oncePerDay("reflect", loadReflectNudge());
-  const analyzeNudge = oncePerDay("analyze", loadAnalyzeNudge());
+  const dueNudges = opts.agent === "cursor" ? dueNudgeSections() : [];
   const parts: string[] = [];
   if (startup) parts.push(startup);
   if (handoff) parts.push(handoff);
-  if (algoReview) parts.push(algoReview);
-  if (reactionAudit) parts.push(reactionAudit);
-  if (reflectNudge) parts.push(reflectNudge);
-  if (analyzeNudge) parts.push(analyzeNudge);
+  parts.push(...dueNudges);
   if (selfModel) parts.push(selfModel);
   if (wisdom) parts.push(wisdom);
   if (opinions) parts.push(opinions);

package/src/hooks/lib/daily-nudge.ts

@@ -1,10 +1,32 @@
 import { readFileSync, writeFileSync } from "node:fs";
 import { resolve } from "node:path";
+import { loadReflectNudge } from "../handlers/reflect-trigger";
+import { loadAlgorithmReviewNudge } from "./algorithm-review";
+import { loadAnalyzeNudge } from "./analyze-nudge";
 import { ensureDir, paths } from "./paths";
+import { loadReactionAuditNudge } from "./reaction-audit";
+import { isOptedIn } from "./settings";
 import { localDay } from "./wall-clock";
 
 type ShownOn = Record<string, string>;
 
+interface DueNudge {
+  key: string;
+  command: string;
+  load: () => string;
+}
+
+const DUE_NUDGES: DueNudge[] = [
+  {
+    key: "algorithm-review",
+    command: "/algorithm-update",
+    load: loadAlgorithmReviewNudge,
+  },
+  { key: "reaction-audit", command: "/reaction-audit", load: loadReactionAuditNudge },
+  { key: "reflect", command: "/pal-reflect", load: loadReflectNudge },
+  { key: "analyze", command: "/pal-analyze", load: loadAnalyzeNudge },
+];
+
 function shownPath(): string {
   return resolve(ensureDir(paths.state()), "nudges-shown.json");
 }
@@ -18,18 +40,39 @@ function readShownOn(): ShownOn {
 }
 
 /**
- * A reminder asks the user for something, so it belongs in the first session
- * of the day, not in every session, resume and compaction after it.
+ * A reminder asks the user for something, so it stays until a reply has passed
+ * it on, then rests for the rest of the day.
  */
-export function oncePerDay(
+export function pendingToday(
   key: string,
   reminder: string,
   now: Date = new Date()
 ): string {
   if (!reminder) return "";
+  return readShownOn()[key] === localDay(now) ? "" : reminder;
+}
+
+export function acknowledgeMentioned(reply: string, now: Date = new Date()): void {
+  const mentioned = DUE_NUDGES.filter((nudge) => reply.includes(nudge.command));
+  if (!mentioned.length) return;
   const today = localDay(now);
   const shownOn = readShownOn();
-  if (shownOn[key] === today) return "";
-  writeFileSync(shownPath(), JSON.stringify({ ...shownOn, [key]: today }), "utf-8");
-  return reminder;
+  for (const nudge of mentioned) shownOn[nudge.key] = today;
+  writeFileSync(shownPath(), JSON.stringify(shownOn), "utf-8");
+}
+
+export function dueNudgeSections(now: Date = new Date()): string[] {
+  if (!isOptedIn("dueReminders")) return [];
+  return DUE_NUDGES.map((nudge) => pendingToday(nudge.key, nudge.load(), now)).filter(
+    Boolean
+  );
+}
+
+export function dueNudgeReminder(now: Date = new Date()): string | null {
+  const sections = dueNudgeSections(now);
+  if (!sections.length) return null;
+  return [
+    "Tell the user in one line at the end of this reply that the following is due, naming its command. This comes back every turn until a reply does.",
+    ...sections,
+  ].join("\n\n");
 }

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. */

package/src/hooks/lib/settings.ts

@@ -115,6 +115,11 @@ export function isEnabled(key: string): boolean {
   return load().dynamicContext?.[key] !== false;
 }
 
+/** Check if an opt-in dynamic context section is enabled (defaults to false) */
+export function isOptedIn(key: string): boolean {
+  return load().dynamicContext?.[key] === true;
+}
+
 /** Get the loadAtStartup file list */
 export function startupFiles(): string[] {
   return load().loadAtStartup?.files ?? [];

package/src/hooks/lib/stop.ts

@@ -19,6 +19,7 @@ import { runSynthesis } from "../handlers/synthesis";
 import { resetTab } from "../handlers/tab";
 import { updateCounts } from "../handlers/update-counts";
 import { captureWorkSession } from "../handlers/work-session";
+import { acknowledgeMentioned } from "./daily-nudge";
 import { spawnDetachedInference } from "./detached-inference";
 import {
   fileFinalReply,
@@ -274,7 +275,9 @@ function recordFinalReply(
   sessionId: string
 ): void {
   try {
-    recordReply(sessionId, extractContent(extractLastAssistant(messages)));
+    const reply = extractContent(extractLastAssistant(messages));
+    recordReply(sessionId, reply);
+    acknowledgeMentioned(reply);
   } catch (err) {
     logError("runStopHandlers:recordReply", err);
   }