bereach-openclaw 1.6.8 → 1.6.9

package/openclaw.plugin.json

@@ -1,7 +1,7 @@
 {
   "id": "bereach-openclaw",
   "name": "BeReach",
-  "version": "1.6.8",
+  "version": "1.6.9",
   "description": "LinkedIn outreach automation — 75+ tools, hook-based enforcement, dynamic context",
   "configSchema": {
     "type": "object",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bereach-openclaw",
-  "version": "1.6.8",
+  "version": "1.6.9",
   "description": "BeReach LinkedIn automation plugin for OpenClaw",
   "license": "AGPL-3.0",
   "exports": {

package/src/hooks/context/formatters.ts

@@ -387,210 +387,69 @@ export function formatLiveStatus(state: SessionState, data: CacheStore, apiKey?:
     return "";
   }
 
-  const lines: string[] = ["", "## BeReach Live Status", ""];
-
   const onboardingBlock = formatOnboardingDirective(state, data, apiKey);
   const isOnboarding = !!onboardingBlock;
-  const hasCampaigns = data.activeCampaigns.length > 0;
-  const hasContacts = data.pipeline &&
-    (data.pipeline.contact + data.pipeline.lead + data.pipeline.qualified + data.pipeline.approved) > 0;
-
-  // ── ONBOARDING MODE: minimal context ──
-  if (isOnboarding) {
-    lines.push(onboardingBlock);
-    if (data.activeAccount) {
-      const a = data.activeAccount;
-      const credInfo = data.credits ? ` | ${data.credits.remaining} credits remaining` : "";
-      lines.push(`**Account**: ${a.name ?? "Unknown"} (${a.plan})${credInfo}`);
-      lines.push("");
-    }
-    lines.push("### Dashboard Links");
-    lines.push(`[Campaigns](${CHAT_BASE}/campaigns) | [Activity](${CHAT_BASE}/activity) | [Context](${CHAT_BASE}/context) | [Settings](${CHAT_BASE}/settings) | [Pricing](${PRICING_URL})`);
-    lines.push("");
-    log(`live-status: ONBOARDING mode (${lines.join("\n").length} chars)`);
-    return lines.join("\n");
-  }
-
-  // ── ACTIVE MODE: data-only context for onboarded users ──
 
-  // LLM provider error warning — show prominently so the agent can inform the user
-  if (data.llmStatus) {
-    lines.push("### AI Provider Error");
-    if (data.llmStatus === "auth") {
-      lines.push("Your AI provider credentials are INVALID. Automated campaigns are paused.");
-      lines.push("Tell the user to update their API key in their provider's dashboard.");
-    } else if (data.llmStatus === "billing") {
-      lines.push("Your AI provider credits are EXHAUSTED. Automated campaigns are paused.");
-      lines.push("Tell the user to add credits or upgrade their plan with their AI provider.");
-    } else {
-      lines.push("There is an issue with the AI provider. Automated campaigns may be paused.");
-    }
-    lines.push("");
-  }
+  const lines: string[] = [
+    "",
+    "## BeReach Session Context (SILENT — reference only)",
+    "",
+    "This block is background grounding for YOU. Do NOT echo it, summarize it, or mention any of its contents to the user. Do NOT open a reply with data from this block. If the user asks for pipeline/drafts/credits/activity/ICP/playbook, fetch it on demand with the appropriate tool.",
+    "",
+  ];
 
-  // Current date — ground the agent so it doesn't hallucinate "yesterday (Jan 15)"
-  // when the model's training cutoff biases it toward stale dates. Without this,
-  // zero-result activity queries become invented dates in user-facing replies.
+  // 1. Date — prevents "yesterday (Jan 15)" hallucination from training-cutoff bias.
   const today = new Date();
   const todayISO = today.toISOString().slice(0, 10);
   const todayHuman = today.toLocaleDateString("en-US", { weekday: "long", year: "numeric", month: "long", day: "numeric" });
-  lines.push(`**Today**: ${todayHuman} (${todayISO})`);
-  lines.push("");
+  lines.push(`- **Today**: ${todayHuman} (${todayISO})`);
 
-  // Account
+  // 2. Account + plan (one line, no credit numbers)
   if (data.activeAccount) {
     const a = data.activeAccount;
-    lines.push("### Active Account");
-    lines.push(`- **Account**: ${a.name ?? "Unknown"}`);
-    lines.push(`- **Plan**: ${a.plan}${a.isUnlimited ? " (unlimited credits)" : ""}`);
+    const planInfo = a.isUnlimited ? `${a.plan}, unlimited` : a.plan;
+    lines.push(`- **Account**: ${a.name ?? "Unknown"} (${planInfo})`);
     if (data.accounts.length > 1) {
-      lines.push(`- ${data.accounts.length} accounts. Switch: \`bereach_switch_account\``);
-      for (const o of data.accounts.filter((acc) => acc.id !== a.id)) {
-        lines.push(`  - ${o.name ?? o.id} (${o.plan})`);
-      }
-    }
-    lines.push("");
-  }
-
-  // Budget
-  if (data.credits) {
-    const c = data.credits;
-    lines.push("### Budget");
-    if (c.isUnlimited) {
-      lines.push(`- **Credits**: ${c.current} used / unlimited`);
-    } else {
-      lines.push(`- **Credits**: ${c.current}/${c.limit} used (${c.remaining} remaining, ${c.percentage}%)`);
+      lines.push(`- **Other accounts**: ${data.accounts.length - 1} (switch with \`bereach_switch_account\`)`);
     }
-    if (data.limits) {
-      const used: string[] = [];
-      let hasLimits = false;
-      for (const [action, info] of Object.entries(data.limits.limits)) {
-        const d = info.daily;
-        if (d && d.limit > 0) {
-          hasLimits = true;
-          if (d.current > 0) used.push(`${action} ${d.current}/${d.limit}`);
-        }
-      }
-      if (used.length > 0) lines.push(`- **LinkedIn Safety Limits** (daily): ${used.join(" | ")}`);
-      else if (hasLimits) lines.push(`- **LinkedIn Safety Limits**: all within safe range`);
-      if (data.limits.multiplier > 1) lines.push(`- **Account Multiplier**: ${data.limits.multiplier}x`);
-    }
-
-    // DM pacing interval (dynamic value, rule is in soul template)
-    const dmPacingCtx = data.contexts.find((c) => c.type === "dm_pacing_minutes");
-    const dmPacingMin = dmPacingCtx ? (parseInt(dmPacingCtx.content, 10) || 5) : 5;
-    lines.push(`- **DM Pacing**: 1 direct DM every ${dmPacingMin} minutes`);
-    lines.push("");
   }
 
-  // Pipeline
-  if (data.pipeline && hasContacts) {
-    const p = data.pipeline;
-    const parts = [`Contact ${p.contact}`, `Lead ${p.lead}`, `Qualified ${p.qualified}`, `Approved ${p.approved}`];
-    if (p.rejected > 0) parts.push(`Rejected ${p.rejected}`);
-    lines.push(`**Pipeline**: ${parts.join(" | ")}`);
-    lines.push("");
-  }
-
-  // Pending state (reference only — NEVER preempt with this data)
-  const pendingItems: string[] = [];
-  if (data.pendingDrafts > 0)
-    pendingItems.push(`${data.pendingDrafts} draft message${data.pendingDrafts > 1 ? "s" : ""} waiting`);
-  if (data.failedDrafts > 0)
-    pendingItems.push(`${data.failedDrafts} scheduled message(s) failed`);
-  if (data.unreadDMs > 0)
-    pendingItems.push(`${data.unreadDMs} unread LinkedIn message${data.unreadDMs > 1 ? "s" : ""}`);
-  if (data.pendingSentInvitations > 0)
-    pendingItems.push(
-      `${data.pendingSentInvitations} pending sent invitation${data.pendingSentInvitations > 1 ? "s" : ""}`,
-    );
-  if (pendingItems.length > 0) {
-    lines.push("### Pending State (reference only)");
-    for (const item of pendingItems) lines.push(`- ${item}`);
-    lines.push("");
-    lines.push("_Do NOT mention these counts unless the user explicitly asks about drafts, failed sends, unread messages, or invitations. Do NOT say \"you have N drafts waiting\" as an opener or a P.S. to an unrelated question._");
-    lines.push("");
+  // 3. DM pacing — small dynamic value the agent needs to answer pacing questions.
+  const dmPacingCtx = data.contexts.find((c) => c.type === "dm_pacing_minutes");
+  const dmPacingMin = dmPacingCtx ? (parseInt(dmPacingCtx.content, 10) || 5) : 5;
+  lines.push(`- **DM pacing**: 1 direct DM every ${dmPacingMin} minutes`);
+
+  // 4. Active campaign names ONLY — no ICP, no playbook, no funnel, no pipeline URL.
+  // Agent fetches campaign context via `bereach_context_get` when it actually needs it.
+  if (data.activeCampaigns.length > 0) {
+    const names = data.activeCampaigns
+      .slice(0, 10)
+      .map((c) => `"${c.name}" (${c.type}, id: ${c.id})`)
+      .join(", ");
+    const overflow = data.activeCampaigns.length > 10 ? ` + ${data.activeCampaigns.length - 10} more` : "";
+    lines.push(`- **Active campaigns (${data.activeCampaigns.length})**: ${names}${overflow}`);
+    lines.push(`- To read a campaign's ICP/playbook: \`bereach_context_list({ scope: "campaign:<id>" })\``);
   }
 
-  // User context entries
-  const MAX_INLINE_CONTEXTS = 10;
-  const priorityTypes = ["icp", "tone-voice", "playbook", "user-profile"];
-  const activeId = data.activeAccount?.id;
-  const meaningful = data.contexts.filter((c) => {
-    if (!c.content?.trim()) return false;
-    if (c.scope?.startsWith("campaign:")) return false; // already shown in campaign dispatch
-    if (c.scope === "user") return true;
-    if (activeId && c.scope === `account:${activeId}`) return true;
-    return false;
-  });
-
-  if (meaningful.length > 0) {
-    const sorted = [...meaningful].sort((a, b) => {
-      const aPri = priorityTypes.indexOf(a.type);
-      const bPri = priorityTypes.indexOf(b.type);
-      return (aPri === -1 ? 99 : aPri) - (bPri === -1 ? 99 : bPri);
-    });
-    const shown = sorted.slice(0, MAX_INLINE_CONTEXTS);
-    const hidden = meaningful.length - shown.length;
-
-    lines.push("### User Context");
-    for (const ctx of shown) {
-      const label = ctx.label ?? ctx.type;
-      const scopeNote = ctx.scope !== "user" ? ` [${ctx.scope}]` : "";
-      lines.push(`**${label}${scopeNote}:**`);
-      lines.push(ctx.content.trim());
-      lines.push("");
-    }
-    if (hidden > 0) {
-      lines.push(`_${hidden} more context entries available via \`bereach_context_list\`_`);
-      lines.push("");
-    }
-  }
+  lines.push("");
 
-  // Session recovery
-  const DONE_PHASES = new Set(["complete", "done", "finished", "completed"]);
-  const lgState = data.leadGenState as Record<string, unknown> | null;
-  const orState = data.outreachState as Record<string, unknown> | null;
-  const lgPhase = String(lgState?.phase ?? lgState?.currentPhase ?? "").toLowerCase();
-  const orPhase = String(orState?.phase ?? orState?.currentPhase ?? "").toLowerCase();
-  const lgIncomplete = lgState && lgPhase && !DONE_PHASES.has(lgPhase);
-  const orIncomplete = orState && orPhase && !DONE_PHASES.has(orPhase);
-
-  if (lgIncomplete || orIncomplete) {
-    lines.push("### Resume Task");
-    if (lgIncomplete) {
-      const found = lgState.totalFound ?? lgState.leadsFound ?? 0;
-      const target = lgState.target ?? lgState.totalTarget;
-      lines.push(`- Lead gen: phase=${lgPhase}, ${found}${target ? `/${target}` : ""} found. Load: \`bereach_state_get('lead-gen')\``);
-    }
-    if (orIncomplete) {
-      lines.push(`- Outreach: phase=${orPhase}. Load: \`bereach_state_get('outreach')\``);
+  // 5. LLM provider error — critical, keep it (but tight).
+  if (data.llmStatus) {
+    if (data.llmStatus === "auth") {
+      lines.push("**AI provider error**: credentials INVALID, automated campaigns paused. Tell user to update API key.");
+    } else if (data.llmStatus === "billing") {
+      lines.push("**AI provider error**: credits EXHAUSTED, automated campaigns paused. Tell user to add credits or upgrade.");
+    } else {
+      lines.push("**AI provider error**: automated campaigns may be paused.");
     }
     lines.push("");
-  } else if (lgState || orState) {
-    const keys = [lgState && "'lead-gen'", orState && "'outreach'"].filter(Boolean).join(", ");
-    lines.push(`_State available: ${keys}. Load with \`bereach_state_get(key)\` if needed._`);
-    lines.push("");
   }
 
-  // Campaign dispatch
-  const dispatchBlock = formatCampaignDispatch(data.activeCampaigns, data.campaignChecks, data.contexts);
-  if (dispatchBlock) lines.push(dispatchBlock);
-
-  // Anti-cron directive: campaigns are automated by the task scheduler, not crons
-  if (hasCampaigns) {
-    lines.push("**Scheduling**: Campaigns are automated by the task scheduler. Never suggest crons or polling — guide users to campaigns instead.");
-    lines.push("");
+  // 6. Onboarding — only fires when onboardingState.completed === false.
+  if (isOnboarding) {
+    lines.push(onboardingBlock);
   }
 
-  // Upgrade signals
-  const upgradeBlock = formatUpgradeSignals(data);
-  if (upgradeBlock) lines.push(upgradeBlock);
-
-  // Dashboard links
-  lines.push(`**Links**: [Campaigns](${CHAT_BASE}/campaigns) | [Activity](${CHAT_BASE}/activity) | [Context](${CHAT_BASE}/context) | [Settings](${CHAT_BASE}/settings) | [Pricing](${PRICING_URL})`);
-  lines.push("");
-
-  log(`live-status: ACTIVE mode, campaigns=${hasCampaigns} contacts=${hasContacts} pending=${data.pendingDrafts > 0 || data.failedDrafts > 0 || data.unreadDMs > 0} (${lines.join("\n").length} chars)`);
+  log(`live-status: minimal (${lines.join("\n").length} chars, campaigns=${data.activeCampaigns.length}, onboarding=${isOnboarding})`);
   return lines.join("\n");
 }

package/src/hooks/context/index.ts

@@ -23,7 +23,6 @@ import {
   formatLiveStatus,
   formatToneInferenceDirective,
   formatAnthropicKeyWarning,
-  formatRecentActivity,
   checkBusinessHours,
 } from "./formatters";
 import { buildTaskContext } from "./task-context";
@@ -148,67 +147,52 @@ async function autoInitProfile(state: SessionState, data: CacheStore, apiKey: st
 // ---------------------------------------------------------------------------
 
 /**
- * Build interactive context, split into static (cacheable) and dynamic (per-turn) parts.
- *
- * The OpenClaw gateway treats `appendSystemContext` as provider-cacheable and
- * `prependContext` as per-turn. By separating the soul template (static, ~8KB)
- * from the live status (dynamic), the gateway can cache the soul template across
- * all turns in a session via Anthropic prompt caching — saving ~90% on those tokens.
+ * Build the interactive system prompt: soul template + minimal live status.
+ * Everything ships via `appendSystemContext` so the gateway caches it as a
+ * single system prompt. `prependContext` is not used in interactive mode.
  */
 function buildInteractiveContext(
   state: SessionState,
   soulTemplate: string,
   liveData: CacheStore,
   apiKey: string,
-): { staticContext: string; dynamicContext: string } {
-  // Live-status block: inject on the FIRST turn of the session only. After
-  // that, the agent already has the context from turn 1 in the conversation
-  // history and doesn't need it re-injected on every turn — re-injecting
-  // every turn was causing the agent to preempt answers with dashboard
-  // summaries ("you have 12 failed drafts, 24 pending…") instead of answering
-  // the user's actual question. Same gate applies to recent activity: the
-  // agent was opening replies with "another reply came in, drafts dropped to
-  // 11" on every turn. Both blocks are background reference only — if the
-  // user wants an update, they'll ask.
-  const isFirstTurn = !state.liveStatusInjected;
-  const liveStatus = isFirstTurn ? formatLiveStatus(state, liveData, apiKey) : "";
-  const activityBlock = isFirstTurn ? formatRecentActivity(liveData.recentEvents) : "";
-  if (isFirstTurn) state.liveStatusInjected = true;
-
+): { staticContext: string } {
+  // EVERYTHING goes in staticContext → appendSystemContext → cached system prompt.
+  //
+  // Previously the live-status block was sent via `prependContext`, which the
+  // gateway renders per-turn. Haiku was echoing that block back as its visible
+  // reply ("Here's your status: ..."), polluting every conversation turn.
+  //
+  // The fix has two parts:
+  //   1. The live-status block is now MINIMAL (date + account + DM pacing +
+  //      campaign names + onboarding). Full ICP / playbook / pipeline / drafts /
+  //      activity / credit details are fetched on demand via tools.
+  //   2. It ships via `appendSystemContext` so the gateway treats it as a
+  //      system instruction (invisible to chat UI, cached across turns) instead
+  //      of content to echo.
+  const liveStatus = formatLiveStatus(state, liveData, apiKey);
   const toneDirective = formatToneInferenceDirective(state, liveData);
 
-  // Static part: soul template with rules, identity, protocols — identical every turn.
-  // Goes into appendSystemContext so the gateway can cache it.
-  const staticContext = soulTemplate;
-
-  // Dynamic part: live status (first turn only), activity, tone, warnings.
-  // Goes into prependContext so it doesn't invalidate the cached soul template.
-  let dynamicContext = liveStatus + activityBlock;
-
-  if (toneDirective) dynamicContext += toneDirective;
+  let staticContext = soulTemplate + liveStatus;
+  if (toneDirective) staticContext += toneDirective;
 
   if (!state.anthropicKeyWarningInjected) {
     const anthropicWarning = formatAnthropicKeyWarning();
-    if (anthropicWarning) dynamicContext += anthropicWarning;
+    if (anthropicWarning) staticContext += anthropicWarning;
     state.anthropicKeyWarningInjected = true;
   }
 
-  const totalLength = staticContext.length + dynamicContext.length;
-
-  // Size guard — log warning but NEVER truncate. User content (ICP, playbook, tone)
-  // must always be injected in full. Truncating can silently drop critical instructions
-  // that the agent needs for correct outreach and qualification. The LLM context window
-  // is large enough to handle the full context in practice.
+  const totalLength = staticContext.length;
   if (totalLength > MAX_CONTEXT_CHARS) {
     log(`context size WARNING: ${totalLength} chars exceeds ${MAX_CONTEXT_CHARS} soft limit (NOT truncating)`);
   }
 
   const yn = (v: unknown) => (v ? "yes" : "no");
   const ob = liveData.onboardingState;
-  log(`context: soul=${staticContext.length} live=${liveStatus.length} tone=${yn(toneDirective)} total=${totalLength} (static=${staticContext.length} dynamic=${dynamicContext.length})`);
-  log(`sections: account=${yn(liveData.activeAccount)} credits=${yn(liveData.credits)} limits=${yn(liveData.limits)} pipeline=${yn(liveData.pipeline)} contexts=${liveData.contexts.length} campaigns=${liveData.activeCampaigns.length} drafts=${liveData.pendingDrafts} failed=${liveData.failedDrafts} unread=${liveData.unreadDMs} onboarding=${ob == null ? "null" : ob.completed ? "done" : "pending"} firstSession=${yn(!liveData.sessionMeta?.lastSessionAt)}`);
+  log(`context: soul=${soulTemplate.length} live=${liveStatus.length} tone=${yn(toneDirective)} total=${totalLength}`);
+  log(`sections: account=${yn(liveData.activeAccount)} campaigns=${liveData.activeCampaigns.length} onboarding=${ob == null ? "null" : ob.completed ? "done" : "pending"} firstSession=${yn(!liveData.sessionMeta?.lastSessionAt)}`);
 
-  return { staticContext, dynamicContext };
+  return { staticContext };
 }
 
 // ---------------------------------------------------------------------------
@@ -345,7 +329,7 @@ export function registerContextHook(api: any, apiKey: string | undefined, state:
           providerMismatch = fastChanged || creativeChanged;
         }
 
-        const { staticContext, dynamicContext } = buildInteractiveContext(state, soulTemplate, liveData, key);
+        const { staticContext } = buildInteractiveContext(state, soulTemplate, liveData, key);
 
         if (providerMismatch && !state.providerMismatchWarningInjected) {
           state.providerMismatchWarningInjected = true;
@@ -370,16 +354,15 @@ export function registerContextHook(api: any, apiKey: string | undefined, state:
           ].join("\n");
           log(`provider mismatch detected — injecting /new warning (was fast=${state.initialAiFastModel} now fast=${currentFast})`);
           return {
-            appendSystemContext: staticContext,
-            prependContext: warning + dynamicContext,
+            appendSystemContext: staticContext + warning,
           };
         }
 
-        // appendSystemContext = cached by gateway (soul template, static across turns)
-        // prependContext = per-turn dynamic data (live status, activity, tone)
+        // Everything ships via appendSystemContext (cached system prompt).
+        // `prependContext` is intentionally never used in interactive mode — it
+        // rendered per-turn and Haiku was echoing it back as its visible reply.
         return {
           appendSystemContext: staticContext,
-          prependContext: dynamicContext,
         };
       } catch (err) {
         log(`error: ${errMsg(err)}`);