opencode-memory-pro 1.3.7 → 1.3.8

package/README.md

@@ -40,7 +40,7 @@ Published on npm — install directly (requires OpenCode ≥ 1.x and Node.js ≥
 opencode plugin opencode-memory-pro
 ```
 
-The latest release is **v1.3.7** on [npm](https://www.npmjs.com/package/opencode-memory-pro); source and releases are on [GitHub](https://github.com/tman204-50/opencode-memory-pro).
+The latest release is **v1.3.8** on [npm](https://www.npmjs.com/package/opencode-memory-pro); source and releases are on [GitHub](https://github.com/tman204-50/opencode-memory-pro).
 
 Remove the old plugin pin at the same time:
 
@@ -408,6 +408,22 @@ so your memories and graph carry over untouched.
 
 ## Changelog
 
+### v1.3.8 (2026-09-06)
+
+Fixes `memory_forget(force=true)` being unable to permanently delete a memory
+that was soft-deleted first:
+
+- **Force delete now sees hidden rows**: `softDeleteMemory` marks a row
+  `status='disabled'`, and the force path previously used `deleteById`, whose
+  `readByScopes` query filters out `status='disabled'` (and `merged`/`digested`)
+  rows — so "Use force=true for permanent deletion" silently failed and left
+  the hidden row on disk forever. The force path now uses the new
+  `deleteByIdForce` in `dist/store.js`, which tries the exact-id raw delete
+  first and otherwise scans unfiltered rows (so id prefixes still match).
+- **Tests**: integration test covers the exact scenario — soft-delete, confirm
+  the old path returns `false`, then `deleteByIdForce` removes the row and
+  reports `false` on a second attempt.
+
 ### v1.3.7 (2026-09-06)
 
 Scope normalization — fixes lost memories when a `scope` argument is

package/dist/index.js

@@ -11,7 +11,7 @@ import { requestLLMCapture, isOwnSession } from "./llm.js";
 import { createMemoryTools, createFeedbackTools, createEpisodicTools } from "./tools/index.js";
 import { sweepExpiredMemories } from "./tools/memory.js";
 import { createGraphStore } from "./graph.js";
-const PLUGIN_VERSION = "1.3.7";
+const PLUGIN_VERSION = "1.3.8";
 const SCHEMA_VERSION = 1;
 // Event-driven dedup: run consolidateDuplicates on session.idle (throttled to
 // this interval so chatty sessions aren't re-scanning the store every turn)

package/dist/store.d.ts

@@ -49,6 +49,8 @@ export declare class MemoryStore {
         globalDiscountFactor?: number;
     }): Promise<SearchResult[]>;
     deleteById(id: string, scopes: string[]): Promise<boolean>;
+    deleteByIdRaw(id: string): Promise<boolean>;
+    deleteByIdForce(id: string): Promise<boolean>;
     softDeleteMemory(id: string, scopes: string[]): Promise<boolean>;
     updateMemoryScope(id: string, newScope: string, scopes: string[]): Promise<boolean>;
     readGlobalMemories(limit?: number): Promise<MemoryRecord[]>;

package/dist/store.js

@@ -628,6 +628,28 @@ export class MemoryStore {
         this.notifyGraphRemoved(id);
         return true;
     }
+    // DELETE_BY_FORCE (1.3.8): like deleteById but sees rows the status-
+    // filtered reads hide (disabled/merged/digested). Fixes memory_forget
+    // force=true: the soft-delete path marks rows disabled, and the force
+    // path previously used deleteById, whose readByScopes filter excludes
+    // status='disabled' — so "Use force=true for permanent deletion" silently
+    // failed and left the hidden row on disk forever. Tries the exact-id raw
+    // delete first (fast path), then falls back to an unfiltered scan so
+    // prefix ids and hidden rows both work.
+    async deleteByIdForce(id) {
+        if (await this.deleteByIdRaw(id)) {
+            return true;
+        }
+        const table = this.requireTable();
+        const rows = await table.query().limit(100000).toArray();
+        const match = rows.find((row) => this.matchesId(row.id, id));
+        if (!match)
+            return false;
+        await table.delete(`id = '${escapeSql(match.id)}'`);
+        this.invalidateScope(match.scope);
+        this.notifyGraphRemoved(match.id);
+        return true;
+    }
     async softDeleteMemory(id, scopes) {
         const rows = await this.readByScopes(scopes);
         const match = rows.find((row) => this.matchesId(row.id, id));

package/dist/tools/memory.js

@@ -447,7 +447,11 @@ export function createMemoryTools(state) {
                 const activeScope = resolveScope(args.scope, context.directory || context.worktree);
                 const scopes = buildScopeFilter(activeScope, state.config.includeGlobalScope);
                 if (args.force) {
-                    const deleted = await state.store.deleteById(args.id, scopes);
+                    // FORCE_DELETE_HIDDEN (1.3.8): was deleteById, whose
+                    // readByScopes filter excludes disabled/merged rows — so
+                    // force=true could never permanently delete a memory that
+                    // was soft-deleted first.
+                    const deleted = await state.store.deleteByIdForce(args.id);
                     if (!deleted) {
                         return `Memory ${args.id} not found in current scope.`;
                     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-memory-pro",
-  "version": "1.3.7",
+  "version": "1.3.8",
   "description": "LanceDB-backed long-term memory provider for OpenCode — standalone fork of lancedb-opencode-pro with entity graph, lifecycle, and retention",
   "type": "module",
   "main": "dist/index.js",