Add URI to autorecall memories (volcengine#2321)

Author: wlff123

examples/openclaw-plugin/auto-recall.ts

@@ -59,8 +59,37 @@ export function estimateTokenCount(text: string): number {
 
 export type BuildMemoryLinesOptions = {
   recallPreferAbstract: boolean;
+  includeUri?: boolean;
 };
 
+function memoryCategory(item: FindResultItem): string {
+  return item.category?.trim() || "memory";
+}
+
+function indentContent(content: string): string {
+  return content
+    .split("\n")
+    .map((line) => `  ${line}`)
+    .join("\n");
+}
+
+function formatMemoryLine(
+  item: FindResultItem,
+  content: string,
+  options: BuildMemoryLinesOptions,
+): string {
+  const category = memoryCategory(item);
+  if (!options.includeUri) {
+    return `- [${category}] ${content}`;
+  }
+
+  return [
+    `- [${category}]`,
+    `  <uri>${item.uri}</uri>`,
+    indentContent(content),
+  ].join("\n");
+}
+
 async function resolveMemoryContent(
   item: FindResultItem,
   readFn: (uri: string) => Promise<string>,
@@ -95,7 +124,7 @@ export async function buildMemoryLines(
   const lines: string[] = [];
   for (const item of memories) {
     const content = await resolveMemoryContent(item, readFn, options);
-    lines.push(`- [${item.category ?? "memory"}] ${content}`);
+    lines.push(formatMemoryLine(item, content, options));
   }
   return lines;
 }
@@ -128,7 +157,7 @@ export async function buildMemoryLinesWithBudget(
     }
 
     const content = await resolveMemoryContent(item, readFn, options);
-    const line = `- [${item.category ?? "memory"}] ${content}`;
+    const line = formatMemoryLine(item, content, options);
     const separatorChars = lines.length > 0 ? 1 : 0;
     const projectedChars = totalChars + separatorChars + line.length;
 
@@ -231,6 +260,7 @@ export async function buildAutoRecallContext(params: {
         {
           recallPreferAbstract: cfg.recallPreferAbstract,
           recallMaxInjectedChars: cfg.recallMaxInjectedChars,
+          includeUri: true,
         },
       );
 

examples/openclaw-plugin/tests/ut/build-memory-lines.test.ts

@@ -52,6 +52,30 @@ describe("buildMemoryLines", () => {
     expect(lines[1]).toBe("- [facts] Works at TechCorp");
   });
 
+  it("includes uri metadata when requested", async () => {
+    const memories = [
+      makeMemory({
+        uri: "viking://user/default/memories/projects/openclaw/autorecall_filename_contract.md",
+        category: "",
+        abstract: "Filename carries semantic meaning.",
+      }),
+    ];
+    const readFn = vi.fn();
+
+    const lines = await buildMemoryLines(memories, readFn, {
+      recallPreferAbstract: true,
+      includeUri: true,
+    });
+
+    expect(lines).toEqual([
+      [
+        "- [memory]",
+        "  <uri>viking://user/default/memories/projects/openclaw/autorecall_filename_contract.md</uri>",
+        "  Filename carries semantic meaning.",
+      ].join("\n"),
+    ]);
+  });
+
   it("uses abstract when recallPreferAbstract=true", async () => {
     const memories = [makeMemory({ abstract: "The abstract text" })];
     const readFn = vi.fn();
@@ -121,7 +145,7 @@ describe("buildMemoryLines", () => {
   });
 
   it("defaults category to 'memory'", async () => {
-    const memories = [makeMemory({ category: undefined })];
+    const memories = [makeMemory({ category: "" })];
     const readFn = vi.fn();
 
     const lines = await buildMemoryLines(memories, readFn, {

examples/openclaw-plugin/tests/ut/context-engine-assemble.test.ts

@@ -147,6 +147,9 @@ describe("context-engine assemble()", () => {
       expect(result.messages[2]?.role).toBe("user");
       expect(result.messages[2]?.content).toMatch(/^<relevant-memories>/);
       expect(result.messages[2]?.content).toContain("Source: openviking-auto-recall");
+      expect(result.messages[2]?.content).toContain(
+        "<uri>viking://user/default/memories/rust-pref</uri>",
+      );
       expect(result.messages[2]?.content).toContain("User prefers Rust for backend tasks.");
       expect(result.messages[2]?.content).toContain("what backend language should we use?");
       expect(result.systemPromptAddition).toBeUndefined();