fix(memory): preserve sidecar capability hooks · openclaw/openclaw@e6288ca

Recent Commits to openclaw:main

brokemac79 · 2026-05-23 · via Recent Commits to openclaw:main

Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,6 @@ import { randomUUID } from "node:crypto";`
`11`	`11`	`import type * as LanceDB from "@lancedb/lancedb";`
`12`	`12`	`import type { OpenClawConfig } from "openclaw/plugin-sdk/config-contracts";`
`13`	`13`	`import type { MemoryEmbeddingProvider } from "openclaw/plugin-sdk/memory-core-host-engine-embeddings";`
`14`		`-import { getMemoryCapabilityRegistration } from "openclaw/plugin-sdk/memory-host-core";`
`15`	`14`	`import { resolveLivePluginConfigObject } from "openclaw/plugin-sdk/plugin-config-runtime";`
`16`	`15`	`import { ensureGlobalUndiciEnvProxyDispatcher } from "openclaw/plugin-sdk/runtime-env";`
`17`	`16`	`import { normalizeLowercaseStringOrEmpty } from "openclaw/plugin-sdk/string-coerce-runtime";`
`@@ -695,9 +694,7 @@ export default definePluginEntry({`
`695`	`694`	`};`
`696`	`695`
`697`	`696`	api.logger.info(`memory-lancedb: plugin registered (db: ${resolvedDbPath}, lazy init)`);
`698`		`-const existingMemoryCapability = getMemoryCapabilityRegistration()?.capability;`
`699`	`697`	`api.registerMemoryCapability?.({`
`700`		`- ...existingMemoryCapability,`
`701`	`698`	`publicArtifacts: {`
`702`	`699`	`async listArtifacts(params) {`
`703`	`700`	`const { listMemoryHostPublicArtifacts } = await loadMemoryHostCoreModule();`

Original file line number	Diff line number	Diff line change
`@@ -204,6 +204,46 @@ describe("memory plugin state", () => {`
`204`	`204`	`]);`
`205`	`205`	`});`
`206`	`206`
	`207`	`+it("preserves sidecar runtime fields when a memory plugin adds public artifacts only", async () => {`
	`208`	`+const runtime = createMemoryRuntime();`
	`209`	`+const flushPlanResolver = () => createMemoryFlushPlan("memory/sidecar.md");`
	`210`	`+`
	`211`	`+registerMemoryCapability("memory-core", {`
	`212`	`+ flushPlanResolver,`
	`213`	`+ runtime,`
	`214`	`+});`
	`215`	`+registerMemoryCapability("memory-lancedb", {`
	`216`	`+publicArtifacts: {`
	`217`	`+async listArtifacts() {`
	`218`	`+return [`
	`219`	`+{`
	`220`	`+kind: "memory-root",`
	`221`	`+workspaceDir: "/tmp/workspace",`
	`222`	`+relativePath: "MEMORY.md",`
	`223`	`+absolutePath: "/tmp/workspace/MEMORY.md",`
	`224`	`+agentIds: ["main"],`
	`225`	`+contentType: "markdown" as const,`
	`226`	`+},`
	`227`	`+];`
	`228`	`+},`
	`229`	`+},`
	`230`	`+});`
	`231`	`+`
	`232`	`+expect(resolveMemoryFlushPlan({})?.relativePath).toBe("memory/sidecar.md");`
	`233`	`+expect(getMemoryRuntime()).toBe(runtime);`
	`234`	`+expect(getMemoryCapabilityRegistration()?.pluginId).toBe("memory-lancedb");`
	`235`	`+await expect(listActiveMemoryPublicArtifacts({ cfg: {} as never })).resolves.toEqual([`
	`236`	`+{`
	`237`	`+kind: "memory-root",`
	`238`	`+workspaceDir: "/tmp/workspace",`
	`239`	`+relativePath: "MEMORY.md",`
	`240`	`+absolutePath: "/tmp/workspace/MEMORY.md",`
	`241`	`+agentIds: ["main"],`
	`242`	`+contentType: "markdown",`
	`243`	`+},`
	`244`	`+]);`
	`245`	`+});`
	`246`	`+`
`207`	`247`	`it("passes citations mode through to the prompt builder", () => {`
`208`	`248`	`registerMemoryPromptSection(({ citationsMode }) => [`
`209`	`249`	`citations: ${citationsMode ?? "default"}`,

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,21 @@ export function registerMemoryCapability(`
`166`	`166`	`pluginId: string,`
`167`	`167`	`capability: MemoryPluginCapability,`
`168`	`168`	`): void {`
`169`		`-memoryPluginState.capability = { pluginId, capability: { ...capability } };`
	`169`	`+const existingCapability = memoryPluginState.capability?.capability;`
	`170`	`+// A selected memory plugin can add bridge artifacts while memory-core owns sidecar runtime hooks.`
	`171`	`+const shouldPreserveExisting =`
	`172`	`+existingCapability &&`
	`173`	`+Boolean(capability.publicArtifacts) &&`
	`174`	`+!capability.promptBuilder &&`
	`175`	`+!capability.flushPlanResolver &&`
	`176`	`+!capability.runtime;`
	`177`	`+memoryPluginState.capability = {`
	`178`	`+ pluginId,`
	`179`	`+capability: {`
	`180`	`+ ...(shouldPreserveExisting ? existingCapability : {}),`
	`181`	`+ ...capability,`
	`182`	`+},`
	`183`	`+};`
`170`	`184`	`}`
`171`	`185`
`172`	`186`	`function patchMemoryCapability(pluginId: string, patch: MemoryPluginCapability): void {`

此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。原文来自 — 版权归原作者所有。

推荐订阅源

Recent Commits to openclaw:main