@basicmemory/openclaw-basic-memory 0.1.1 → 0.1.3

package/bm-client.ts

@@ -7,7 +7,6 @@ const DEFAULT_RETRY_DELAYS_MS = [500, 1000, 2000]
 
 const REQUIRED_TOOLS = [
   "search_notes",
-  "search_by_metadata",
   "read_note",
   "write_note",
   "edit_note",
@@ -153,12 +152,6 @@ export interface SchemaDiffResult {
   cardinality_changes: string[]
 }
 
-export interface MetadataSearchResult {
-  results: SearchResult[]
-  current_page: number
-  page_size: number
-}
-
 function getErrorMessage(err: unknown): string {
   return err instanceof Error ? err.message : String(err)
 }
@@ -502,25 +495,29 @@ export class BmClient {
   }
 
   async search(
-    query: string,
+    query?: string,
     limit = 10,
     project?: string,
     metadata?: {
       filters?: Record<string, unknown>
       tags?: string[]
       status?: string
+      note_types?: string[]
+      entity_types?: string[]
     },
   ): Promise<SearchResult[]> {
     const args: Record<string, unknown> = {
-      query,
       page: 1,
       page_size: limit,
       output_format: "json",
     }
+    if (query) args.query = query
     if (project) args.project = project
     if (metadata?.filters) args.metadata_filters = metadata.filters
     if (metadata?.tags) args.tags = metadata.tags
     if (metadata?.status) args.status = metadata.status
+    if (metadata?.note_types) args.note_types = metadata.note_types
+    if (metadata?.entity_types) args.entity_types = metadata.entity_types
 
     const payload = await this.callTool("search_notes", args)
 
@@ -809,27 +806,6 @@ export class BmClient {
     return payload as unknown as SchemaDiffResult
   }
 
-  async searchByMetadata(
-    filters: Record<string, unknown>,
-    limit = 20,
-    project?: string,
-  ): Promise<MetadataSearchResult> {
-    const args: Record<string, unknown> = {
-      filters,
-      limit,
-      output_format: "json",
-    }
-    if (project) args.project = project
-
-    const payload = await this.callTool("search_by_metadata", args)
-
-    if (!isRecord(payload) || !Array.isArray(payload.results)) {
-      throw new Error("invalid search_by_metadata response")
-    }
-
-    return payload as unknown as MetadataSearchResult
-  }
-
   async indexConversation(
     userMessage: string,
     assistantResponse: string,

package/hooks/recall.ts

@@ -1,8 +1,4 @@
-import type {
-  BmClient,
-  MetadataSearchResult,
-  RecentResult,
-} from "../bm-client.ts"
+import type { BmClient, RecentResult, SearchResult } from "../bm-client.ts"
 import type { BasicMemoryConfig } from "../config.ts"
 import { log } from "../logger.ts"
 
@@ -11,14 +7,14 @@ import { log } from "../logger.ts"
  * Returns empty string if nothing was found.
  */
 export function formatRecallContext(
-  tasks: MetadataSearchResult,
+  tasks: SearchResult[],
   recent: RecentResult[],
   prompt: string,
 ): string {
   const sections: string[] = []
 
-  if (tasks.results.length > 0) {
-    const taskLines = tasks.results.map((t) => {
+  if (tasks.length > 0) {
+    const taskLines = tasks.map((t) => {
       const preview =
         t.content.length > 120 ? `${t.content.slice(0, 120)}...` : t.content
       return `- **${t.title}** — ${preview}`
@@ -46,7 +42,10 @@ export function buildRecallHandler(client: BmClient, cfg: BasicMemoryConfig) {
   return async (_event: Record<string, unknown>) => {
     try {
       const [tasks, recent] = await Promise.all([
-        client.searchByMetadata({ entity_type: "Task", status: "active" }, 5),
+        client.search(undefined, 5, undefined, {
+          note_types: ["Task"],
+          status: "active",
+        }),
         client.recentActivity("1d"),
       ])
 
@@ -54,7 +53,7 @@ export function buildRecallHandler(client: BmClient, cfg: BasicMemoryConfig) {
       if (!context) return {}
 
       log.debug(
-        `recall: ${tasks.results.length} active tasks, ${recent.length} recent items`,
+        `recall: ${tasks.length} active tasks, ${recent.length} recent items`,
       )
 
       return { context }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@basicmemory/openclaw-basic-memory",
-  "version": "0.1.1",
+  "version": "0.1.3",
   "type": "module",
   "description": "Basic Memory plugin for OpenClaw — local-first knowledge graph for agent memory",
   "license": "MIT",

package/schema/task-schema.ts

@@ -8,16 +8,16 @@ type: schema
 entity: Task
 version: 1
 schema:
-  description: string, what needs to be done
-  status?(enum): [active, blocked, done, abandoned], current state
-  assigned_to?: string, who is working on this
-  steps?(array): string, ordered steps to complete
-  current_step?: integer, which step number we're on (1-indexed)
-  context?: string, key context needed to resume after memory loss
-  started?: string, when work began
-  completed?: string, when work finished
-  blockers?(array): string, what's preventing progress
-  parent_task?: Task, parent task if this is a subtask
+  description: "string, what needs to be done"
+  status?(enum): "[active, blocked, done, abandoned], current state"
+  assigned_to?: "string, who is working on this"
+  steps?(array): "string, ordered steps to complete"
+  current_step?: "integer, which step number we're on (1-indexed)"
+  context?: "string, key context needed to resume after memory loss"
+  started?: "string, when work began"
+  completed?: "string, when work finished"
+  blockers?(array): "string, what's preventing progress"
+  parent_task?: "Task, parent task if this is a subtask"
 settings:
   validation: warn
 ---

package/tools/memory-provider.ts

@@ -60,8 +60,8 @@ async function searchMemoryFile(
 
 /**
  * Search for active tasks via BM knowledge graph.
- * Uses structured metadata filtering via search_by_metadata for precise querying.
- * Falls back to filesystem scan if metadata search fails.
+ * Uses search_notes with note_types and status filters for precise querying.
+ * Falls back to filesystem scan if search fails.
  */
 async function searchActiveTasks(
   query: string,
@@ -69,44 +69,29 @@ async function searchActiveTasks(
   workspaceDir: string,
   memoryDir: string,
 ): Promise<string> {
-  // Try structured metadata search first
+  // Try structured search first
   try {
-    const filters: Record<string, unknown> = { type: "task", status: "active" }
-    const metaResults = await client.searchByMetadata(filters, 10)
-
-    if (metaResults.results.length > 0) {
-      // If the caller provided a query, filter results by keyword match
-      const queryLower = query.toLowerCase()
-      const terms = queryLower.split(/\s+/).filter((t) => t.length > 0)
-
-      const filtered =
-        terms.length > 0
-          ? metaResults.results.filter((r) => {
-              const text = `${r.title} ${r.content ?? ""}`.toLowerCase()
-              return terms.some((term) => text.includes(term))
-            })
-          : metaResults.results
-
-      if (filtered.length > 0) {
-        const matches: string[] = []
-        for (const r of filtered) {
-          const score = r.score ? ` (${r.score.toFixed(2)})` : ""
-          const preview =
-            (r.content ?? "").length > 200
-              ? `${(r.content ?? "").slice(0, 200)}…`
-              : (r.content ?? "")
-          matches.push(
-            `- **${r.title}**${score} — ${r.file_path}\n  > ${preview.replace(/\n/g, "\n  > ")}`,
-          )
-        }
-        return matches.join("\n\n")
+    const results = await client.search(query || undefined, 10, undefined, {
+      note_types: ["task"],
+      status: "active",
+    })
+
+    if (results.length > 0) {
+      const matches: string[] = []
+      for (const r of results) {
+        const score = r.score ? ` (${r.score.toFixed(2)})` : ""
+        const preview =
+          (r.content ?? "").length > 200
+            ? `${(r.content ?? "").slice(0, 200)}…`
+            : (r.content ?? "")
+        matches.push(
+          `- **${r.title}**${score} — ${r.file_path}\n  > ${preview.replace(/\n/g, "\n  > ")}`,
+        )
       }
+      return matches.join("\n\n")
     }
   } catch (err) {
-    log.debug(
-      "BM metadata task search failed, falling back to filesystem scan",
-      err,
-    )
+    log.debug("BM task search failed, falling back to filesystem scan", err)
   }
 
   // Fallback: filesystem scan for tasks not yet indexed