opencode-self-improvement 0.2.0 → 0.2.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-self-improvement",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Durable memory for opencode: remember facts, decisions, and commands across sessions, plus a one-command per-repo setup (/repo-init).",
   "main": "index.ts",
   "exports": {

package/src/inject.ts

@@ -147,18 +147,23 @@ export async function storeVersion(): Promise<string> {
 }
 
 // Trim an injection block to `budget` characters at WHOLE-line boundaries:
-// keep lines while the joined length stays <= budget, drop the rest, never cut
-// mid-line. Returns "" when even the header line exceeds the budget (the block
-// is not worth injecting).
+// keep every line that still fits (skipping over-long ones), never cut
+// mid-line. Returns "" when even the header line exceeds the budget, and ""
+// when NOTHING but header lines survived — a dangling "## Relevant durable
+// memory" over zero memories is noise, not memory (bug seen live: a store
+// whose top matches all exceed the budget injected exactly the 26-char header).
 export function capToBudget(block: string, budget: number): string {
-  const kept: string[] = [];
-  let len = 0;
-  for (const line of block.split("\n")) {
-    const add = kept.length ? line.length + 1 : line.length;
-    if (len + add > budget) break;
+  const lines = block.split("\n");
+  // The header gives the block its meaning; if it does not fit, nothing does.
+  if (lines[0].length > budget) return "";
+  const kept: string[] = [lines[0]];
+  let len = lines[0].length;
+  for (const line of lines.slice(1)) {
+    if (len + line.length + 1 > budget) continue; // skip over-long lines, keep scanning
     kept.push(line);
-    len += add;
+    len += line.length + 1;
   }
+  if (kept.every((l) => l.startsWith("## "))) return "";
   return kept.join("\n");
 }
 

package/src/resolver.ts

@@ -54,6 +54,30 @@ export function balancedParen(text: string, openIdx: number): { inner: string; e
 
 const CALL_RE = /([A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)+)\s*\(/g;
 
+// Last dotted segment is a known file extension: `package.json(committed by…)`
+// is a file reference followed by parenthetical prose, not a call to a method
+// named `json`. This class produced 9 false drift_ref_stale_incoming blocks in
+// live use before the guard existed.
+const FILE_EXT = new Set([
+  "json", "jsonc", "jsonl", "logfmt", "md", "mdx", "ts", "tsx", "js", "jsx",
+  "mjs", "cjs", "py", "go", "rs", "rb", "java", "kt", "sh", "bash", "zsh",
+  "yaml", "yml", "toml", "ini", "conf", "cfg", "txt", "csv", "html", "htm",
+  "css", "scss", "xml", "sql", "tf", "lock", "log", "env",
+]);
+
+// Parenthetical PROSE is not a code call signature. A code-like arg list is
+// empty or contains code punctuation ({ } ( ) : = , . < > quotes); the markers
+// [ ] @ / never appear in the arg snippets drift detection cares about, and
+// bare multi-word English with no code punctuation ("main-agent review fix")
+// is prose.
+function looksProseArgs(inner: string): boolean {
+  const s = inner.trim();
+  if (!s) return false;
+  if (/[[\]@/]/.test(s)) return true;
+  if (/[{}():=",.<>'"`]/.test(s)) return false;
+  return s.split(/\s+/).length >= 2;
+}
+
 export function extractCallForms(content: string): CallForm[] {
   const forms: CallForm[] = [];
   CALL_RE.lastIndex = 0;
@@ -65,6 +89,8 @@ export function extractCallForms(content: string): CallForm[] {
     if (!paren) continue;
     const parts = symbol.split(".");
     const name = parts[parts.length - 1];
+    if (FILE_EXT.has(name.toLowerCase())) continue;
+    if (looksProseArgs(paren.inner)) continue;
     const ownerPath = parts.slice(0, -1).join(".");
     forms.push({
       symbol,