github/openclaw
1
1
Fork 0
mirror of https://github.com/openclaw/openclaw.git synced 2026-02-19 18:39:20 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix (memory/lancedb): require explicit opt-in for auto-capture

Browse Source
This commit is contained in:
Vignesh Natarajan
2026-02-14 18:20:48 -08:00
parent 3ca74f8e6d
commit ed7d83bcfc
2 changed files with 16 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
extensions/memory-lancedb/config.ts
View File

@@ -121,7 +121,7 @@ export const memoryConfigSchema = {
apiKey: resolveEnvVars(embedding.apiKey),
},
dbPath: typeof cfg.dbPath === "string" ? cfg.dbPath : DEFAULT_DB_PATH,
autoCapture: cfg.autoCapture !== false,
autoCapture: cfg.autoCapture === true,
autoRecall: cfg.autoRecall !== false,
captureMaxChars: captureMaxChars ?? DEFAULT_CAPTURE_MAX_CHARS,
};

15
extensions/memory-lancedb/index.test.ts
View File

@@ -120,6 +120,21 @@ describe("memory plugin e2e", () => {
expect(config?.captureMaxChars).toBe(1800);
});
test("config schema keeps autoCapture disabled by default", async () => {
const { default: memoryPlugin } = await import("./index.js");
const config = memoryPlugin.configSchema?.parse?.({
embedding: {
apiKey: OPENAI_API_KEY,
model: "text-embedding-3-small",
},
dbPath,
});
expect(config?.autoCapture).toBe(false);
expect(config?.autoRecall).toBe(true);
});
test("shouldCapture applies real capture rules", async () => {
const { shouldCapture } = await import("./index.js");