Skip to content

Commit 1dd9bcc

Browse files
[AI] fix(memory): use fts-only fallback for unregistered adapter instead of empty string
Change resolveEmbeddingProviderFallbackModel fallbackSourceModel from empty string to fts-only so unregistered adapters produce a readable expectedModel instead of an empty string that causes permanent identity mismatch. Refs #90787
1 parent f557376 commit 1dd9bcc

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

extensions/memory-core/src/memory/manager-reindex-state.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,9 @@ describe("memory reindex state", () => {
293293
}),
294294
);
295295
expect(state.status).toBe("mismatched");
296-
expect(state.reason).toContain("expected fts-only");
296+
if (state.status === "mismatched") {
297+
expect(state.reason).toContain("expected fts-only");
298+
}
297299
});
298300

299301
it("falls back to fts-only when provider.model is whitespace-only", () => {

extensions/memory-core/src/memory/manager-sync-ops.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,7 @@ export abstract class MemoryManagerSyncOps {
517517
this.settings.provider,
518518
model:
519519
this.settings.model.trim() ||
520-
resolveEmbeddingProviderFallbackModel(this.settings.provider, "", this.cfg),
520+
resolveEmbeddingProviderFallbackModel(this.settings.provider, "fts-only", this.cfg),
521521
});
522522
const provider = hasProviderOverride
523523
? params.provider!

0 commit comments

Comments
 (0)