fix(plugins): include memory defaults for unlisted agents

osolmaz · osolmaz · commit b7648ca5b362 · 2026-06-06T20:02:20.000+08:00
diff --git a/src/plugins/channel-plugin-ids.test.ts b/src/plugins/channel-plugin-ids.test.ts
@@ -1177,7 +1177,7 @@ describe("resolveGatewayStartupPluginIds", () => {
       ["browser", "openai", "ollama", "memory-core"],
     ],
     [
-      "does not include unused default memory embedding providers when listed agents override memory search",
+      "includes default memory embedding providers for unlisted agents even when listed agents override memory search",
       {
         channels: {},
         agents: {
@@ -1190,7 +1190,7 @@ describe("resolveGatewayStartupPluginIds", () => {
           ],
         },
       } as OpenClawConfig,
-      ["browser", "ollama", "memory-core"],
+      ["browser", "openai", "ollama", "memory-core"],
     ],
     [
       "includes default memory embedding providers for listed agents that inherit defaults",
diff --git a/src/plugins/gateway-startup-plugin-ids.ts b/src/plugins/gateway-startup-plugin-ids.ts
@@ -629,10 +629,10 @@ export function collectConfiguredMemoryEmbeddingStartupProviderOwners(
       });
     }
   };
+  addEffectiveProviders(undefined);
   const agents = config.agents?.list;
   const agentEntries = Array.isArray(agents) ? agents.filter(isRecord) : [];
   if (agentEntries.length === 0) {
-    addEffectiveProviders(undefined);
     return [...byConfiguredIdAndSource.values()];
   }
   for (const agent of agentEntries) {

Original file line number	Diff line number	Diff line change
`@@ -1177,7 +1177,7 @@ describe("resolveGatewayStartupPluginIds", () => {`
`1177`	`1177`	`["browser", "openai", "ollama", "memory-core"],`
`1178`	`1178`	`],`
`1179`	`1179`	`[`
`1180`		`- "does not include unused default memory embedding providers when listed agents override memory search",`
	`1180`	`+ "includes default memory embedding providers for unlisted agents even when listed agents override memory search",`
`1181`	`1181`	`{`
`1182`	`1182`	`channels: {},`
`1183`	`1183`	`agents: {`
`@@ -1190,7 +1190,7 @@ describe("resolveGatewayStartupPluginIds", () => {`
`1190`	`1190`	`],`
`1191`	`1191`	`},`
`1192`	`1192`	`} as OpenClawConfig,`
`1193`		`- ["browser", "ollama", "memory-core"],`
	`1193`	`+ ["browser", "openai", "ollama", "memory-core"],`
`1194`	`1194`	`],`
`1195`	`1195`	`[`
`1196`	`1196`	`"includes default memory embedding providers for listed agents that inherit defaults",`
Original file line number	Diff line number	Diff line change
`@@ -629,10 +629,10 @@ export function collectConfiguredMemoryEmbeddingStartupProviderOwners(`
`629`	`629`	`});`
`630`	`630`	`}`
`631`	`631`	`};`
	`632`	`+ addEffectiveProviders(undefined);`
`632`	`633`	`const agents = config.agents?.list;`
`633`	`634`	`const agentEntries = Array.isArray(agents) ? agents.filter(isRecord) : [];`
`634`	`635`	`if (agentEntries.length === 0) {`
`635`		`- addEffectiveProviders(undefined);`
`636`	`636`	`return [...byConfiguredIdAndSource.values()];`
`637`	`637`	`}`
`638`	`638`	`for (const agent of agentEntries) {`