@codemem/opencode-plugin 0.29.1 → 0.29.2

package/.opencode/plugins/codemem.js

@@ -15,7 +15,7 @@ import {
 
 const TRUTHY_VALUES = ["1", "true", "yes"];
 const DISABLED_VALUES = ["0", "false", "off"];
-const PINNED_BACKEND_VERSION = "0.29.1";
+const PINNED_BACKEND_VERSION = "0.29.2";
 const COMPAT_CHECK_DELAY_MS = 1500;
 const COMPAT_CHECK_CACHE_TTL_MS = 5 * 60 * 1000;
 
@@ -245,7 +245,6 @@ const applyInjectedContextToOutput = async ({
   injectEnabled,
   input,
   output,
-  injectedSessions,
   injectionToastShown,
   showToast,
   resolveInjectQuery,
@@ -255,39 +254,33 @@ const applyInjectedContextToOutput = async ({
     return false;
   }
 
+  // The previous incarnation cached by sessionID-only, so a follow-up
+  // turn whose query happened to match the cache key (e.g. because
+  // lastPromptText hadn't yet been captured by the time
+  // experimental.chat.system.transform fired) would silently re-serve
+  // the first turn's pack. Recompute on every call instead — the chat
+  // path tolerates the runCli round trip, and correctness beats the
+  // O(1) hit when the cache key isn't tied to the prompt that produced
+  // the pack.
   const query = resolveInjectQuery();
-  const cached = injectedSessions.get(input.sessionID);
-  const queryMatchesCache = cached?.query === query;
-  let contextText = queryMatchesCache ? cached?.text || "" : "";
-
-  if (!contextText || cached?.query !== query) {
-    const injected = await buildInjectedContext(query);
-    if (injected?.text) {
-      injectedSessions.set(input.sessionID, {
-        query,
-        text: injected.text,
-        metrics: injected.metrics || null,
-      });
-      contextText = injected.text;
+  const injected = await buildInjectedContext(query);
+  if (!injected?.text) {
+    return false;
+  }
 
-      if (!injectionToastShown.has(input.sessionID) && showToast) {
-        injectionToastShown.add(input.sessionID);
-        try {
-          await showToast(buildInjectionToastMessage(injected.metrics));
-        } catch {
-          // best-effort only
-        }
-      }
+  if (!injectionToastShown.has(input.sessionID) && showToast) {
+    injectionToastShown.add(input.sessionID);
+    try {
+      await showToast(buildInjectionToastMessage(injected.metrics));
+    } catch {
+      // best-effort only
     }
   }
 
-  if (!contextText) {
-    return false;
-  }
   if (!Array.isArray(output.system)) {
     output.system = [];
   }
-  output.system.push(contextText);
+  output.system.push(injected.text);
   return true;
 };
 
@@ -749,7 +742,6 @@ export const OpencodeMemPlugin = async ({
   const injectLimit = injectLimitEnv ? parseNumber(injectLimitEnv, null) : null;
   const injectTokenBudgetEnv = process.env.CODEMEM_INJECT_TOKEN_BUDGET;
   const injectTokenBudget = injectTokenBudgetEnv ? parseNumber(injectTokenBudgetEnv, null) : null;
-  const injectedSessions = new Map();
   const injectionToastShown = new Set();
   let sessionStartedAt = null;
   let activeSessionID = null;
@@ -1537,17 +1529,17 @@ export const OpencodeMemPlugin = async ({
       return "";
     }
     const metrics = parsePackMetrics(result.stdout);
-    if (debug) {
-      const itemCount = Number.isFinite(Number(metrics?.items))
-        ? Number(metrics.items)
-        : 0;
-      const packTokens = Number.isFinite(Number(metrics?.pack_tokens))
-        ? Number(metrics.pack_tokens)
-        : 0;
-      await logLine(
-        `inject.pack.ok query_len=${query ? query.length : 0} items=${itemCount} pack_tokens=${packTokens}`
-      );
-    }
+    // The pack JSON exposes the item count as `total_items`; `metrics.items`
+    // does not exist on that payload, so reading it would always log 0.
+    const itemCount = Number.isFinite(Number(metrics?.total_items))
+      ? Number(metrics.total_items)
+      : 0;
+    const packTokens = Number.isFinite(Number(metrics?.pack_tokens))
+      ? Number(metrics.pack_tokens)
+      : 0;
+    await logLine(
+      `inject.pack.ok source=opencode items=${itemCount} pack_tokens=${packTokens} query_len=${query ? query.length : 0}`
+    );
     if (metrics) {
       return {
         text: `[codemem context]\n${packText}`,
@@ -1796,25 +1788,35 @@ export const OpencodeMemPlugin = async ({
           } inject_enabled=${injectEnabled} tui_toast=${Boolean(client.tui?.showToast)} query=${JSON.stringify(safeQuery)} first_prompt_len=${firstPromptLen} last_prompt_len=${lastPromptLen} project=${JSON.stringify(projectName)} files_modified=${filesModifiedCount}`
         );
       }
-      const applied = await applyInjectedContextToOutput({
-        injectEnabled,
-        input,
-        output,
-        injectedSessions,
-        injectionToastShown,
-        showToast: client.tui?.showToast
-          ? async (message) => {
-            await client.tui.showToast({
-              body: {
-                message,
-                variant: "info",
-              },
-            });
-          }
-          : null,
-        resolveInjectQuery,
-        buildInjectedContext,
-      });
+      // Without the old per-session cache, every transform call shells out
+      // through `runCli`. Swallow rejections here so a single failed pack
+      // build (CLI crash, sqlite locked, network blip on HTTP fallback)
+      // can't take down the chat path.
+      let applied = false;
+      try {
+        applied = await applyInjectedContextToOutput({
+          injectEnabled,
+          input,
+          output,
+          injectionToastShown,
+          showToast: client.tui?.showToast
+            ? async (message) => {
+              await client.tui.showToast({
+                body: {
+                  message,
+                  variant: "info",
+                },
+              });
+            }
+            : null,
+          resolveInjectQuery,
+          buildInjectedContext,
+        });
+      } catch (err) {
+        await logLine(
+          `inject.transform.error sessionID=${input.sessionID} message=${JSON.stringify(err instanceof Error ? err.message : String(err))}`
+        );
+      }
       if (debug) {
         await logLine(
           `inject.transform.result sessionID=${input.sessionID} applied=${Boolean(applied)} system_entries=${Array.isArray(output.system) ? output.system.length : 0}`
@@ -2000,6 +2002,9 @@ export const OpencodeMemPlugin = async ({
       }
       if (eventType === "session.deleted") {
         activeSessionID = null;
+        if (sessionID) {
+          injectionToastShown.delete(sessionID);
+        }
         await stopViewer();
       }
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codemem/opencode-plugin",
-  "version": "0.29.1",
+  "version": "0.29.2",
   "description": "CodeMem plugin for OpenCode",
   "type": "module",
   "main": "./index.js",
@@ -18,7 +18,7 @@
     ".opencode/lib/"
   ],
   "dependencies": {
-    "@opencode-ai/plugin": "1.14.28"
+    "@opencode-ai/plugin": "1.14.31"
   },
   "repository": {
     "type": "git",