mirror of
https://github.com/openclaw/openclaw.git
synced 2026-08-26 12:26:38 -06:00
8b0735e89f
* refactor(memory): remove qmd backend Make builtin the sole memory-core engine, rename the retained session helper barrel, retire QMD config with doctor migrations, and remove QMD runtime/UI/policy surfaces. * docs(memory): remove qmd backend guidance Delete the QMD concept page, rewrite memory documentation for builtin retrieval, and remove QMD from navigation and taxonomy source. * refactor(memory): remove qmd-only leftovers * refactor(memory): finish qmd integration cleanup * build(deps): align root string-width types * build(deps): model root string-width tooling * refactor(memory): align qmd removal ui and docs * fix(memory): preserve qmd external paths in doctor * test(memory): remove obsolete backend probe case * test(plugin-sdk): refresh private type baseline
177 lines
5.1 KiB
TypeScript
177 lines
5.1 KiB
TypeScript
// Memory Core tests cover tools.recall tracking plugin behavior.
|
|
import type { MemorySearchResult } from "openclaw/plugin-sdk/memory-core-host-runtime-files";
|
|
import { beforeEach, describe, expect, it, vi } from "vitest";
|
|
import type { OpenClawConfig } from "../api.js";
|
|
import { resetMemoryToolMockState, setMemorySearchImpl } from "./memory-tool-manager.test-mocks.js";
|
|
import { createMemorySearchTool } from "./tools.js";
|
|
import { asOpenClawConfig } from "./tools.test-helpers.js";
|
|
|
|
type RecordShortTermRecallsFn = (params: {
|
|
workspaceDir?: string;
|
|
query: string;
|
|
results: MemorySearchResult[];
|
|
nowMs?: number;
|
|
timezone?: string;
|
|
}) => Promise<void>;
|
|
|
|
const recallTrackingMock = vi.hoisted(() => ({
|
|
recordShortTermRecalls: vi.fn<RecordShortTermRecallsFn>(async () => {}),
|
|
}));
|
|
|
|
vi.mock("./short-term-promotion.js", () => ({
|
|
recordShortTermRecalls: recallTrackingMock.recordShortTermRecalls,
|
|
}));
|
|
|
|
function createSearchTool(config: OpenClawConfig) {
|
|
const tool = createMemorySearchTool({ config });
|
|
if (!tool) {
|
|
throw new Error("memory_search tool missing");
|
|
}
|
|
return tool;
|
|
}
|
|
|
|
describe("memory_search recall tracking", () => {
|
|
beforeEach(() => {
|
|
resetMemoryToolMockState();
|
|
recallTrackingMock.recordShortTermRecalls.mockReset();
|
|
recallTrackingMock.recordShortTermRecalls.mockResolvedValue(undefined);
|
|
});
|
|
|
|
it("does not block tool results on slow best-effort recall writes", async () => {
|
|
let resolveRecall: (() => void) | undefined;
|
|
recallTrackingMock.recordShortTermRecalls.mockImplementationOnce(
|
|
async () =>
|
|
await new Promise<void>((resolve) => {
|
|
resolveRecall = resolve;
|
|
}),
|
|
);
|
|
|
|
const tool = createSearchTool(
|
|
asOpenClawConfig({
|
|
agents: { list: [{ id: "main", default: true }] },
|
|
plugins: {
|
|
entries: {
|
|
"memory-core": {
|
|
config: {
|
|
dreaming: {
|
|
enabled: true,
|
|
},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
}),
|
|
);
|
|
setMemorySearchImpl(async () => [
|
|
{
|
|
path: "memory/2026-04-03.md",
|
|
startLine: 1,
|
|
endLine: 2,
|
|
score: 0.95,
|
|
snippet: "Move backups to S3 Glacier.",
|
|
source: "memory" as const,
|
|
},
|
|
]);
|
|
|
|
let timeout: NodeJS.Timeout | undefined;
|
|
try {
|
|
const result = await Promise.race([
|
|
tool.execute("call_recall_non_blocking", { query: "glacier" }),
|
|
new Promise<never>((_, reject) => {
|
|
timeout = setTimeout(() => {
|
|
reject(new Error("memory_search waited on recall persistence"));
|
|
}, 200);
|
|
}),
|
|
]);
|
|
|
|
const details = result.details as { results: Array<{ path: string }> };
|
|
expect(details.results).toHaveLength(1);
|
|
expect(details.results[0]?.path).toBe("memory/2026-04-03.md");
|
|
expect(recallTrackingMock.recordShortTermRecalls).toHaveBeenCalledTimes(1);
|
|
} finally {
|
|
if (timeout) {
|
|
clearTimeout(timeout);
|
|
}
|
|
resolveRecall?.();
|
|
}
|
|
});
|
|
|
|
it("passes the resolved dreaming timezone into recall tracking", async () => {
|
|
setMemorySearchImpl(async () => [
|
|
{
|
|
path: "memory/2026-04-03.md",
|
|
startLine: 1,
|
|
endLine: 2,
|
|
score: 0.95,
|
|
snippet: "Move backups to S3 Glacier.",
|
|
source: "memory" as const,
|
|
},
|
|
]);
|
|
|
|
const tool = createSearchTool(
|
|
asOpenClawConfig({
|
|
agents: {
|
|
defaults: {
|
|
userTimezone: "America/Los_Angeles",
|
|
},
|
|
list: [{ id: "main", default: true }],
|
|
},
|
|
plugins: {
|
|
entries: {
|
|
"memory-core": {
|
|
config: {
|
|
dreaming: {
|
|
enabled: true,
|
|
timezone: "Europe/London",
|
|
},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
}),
|
|
);
|
|
|
|
await tool.execute("call_recall_timezone", { query: "glacier" });
|
|
|
|
expect(recallTrackingMock.recordShortTermRecalls).toHaveBeenCalledTimes(1);
|
|
const [firstCall] = recallTrackingMock.recordShortTermRecalls.mock.calls;
|
|
expect(firstCall?.[0]?.timezone).toBe("Europe/London");
|
|
});
|
|
|
|
it("skips recall tracking when dreaming is disabled", async () => {
|
|
setMemorySearchImpl(async () => [
|
|
{
|
|
path: "memory/2026-04-03.md",
|
|
startLine: 1,
|
|
endLine: 2,
|
|
score: 0.95,
|
|
snippet: "Move backups to S3 Glacier.",
|
|
source: "memory" as const,
|
|
},
|
|
]);
|
|
|
|
const tool = createSearchTool(
|
|
asOpenClawConfig({
|
|
agents: { list: [{ id: "main", default: true }] },
|
|
plugins: {
|
|
entries: {
|
|
"memory-core": {
|
|
config: {
|
|
dreaming: {
|
|
enabled: false,
|
|
},
|
|
},
|
|
},
|
|
},
|
|
},
|
|
}),
|
|
);
|
|
|
|
const result = await tool.execute("call_recall_disabled", { query: "glacier" });
|
|
const details = result.details as { results: Array<{ path: string }> };
|
|
expect(details.results).toHaveLength(1);
|
|
expect(details.results[0]?.path).toBe("memory/2026-04-03.md");
|
|
expect(recallTrackingMock.recordShortTermRecalls).not.toHaveBeenCalled();
|
|
});
|
|
});
|