fix(memory): resolve adapter default model in plain status identity check

clawsweeper[bot] · clawsweeper[bot] · commit 9741437564f2 · 2026-06-06T02:23:59.000Z
diff --git a/extensions/memory-core/src/memory/manager-sync-ops.ts b/extensions/memory-core/src/memory/manager-sync-ops.ts
@@ -300,22 +300,18 @@ export abstract class MemoryManagerSyncOps {
     hasIndexedChunks?: boolean;
   }): MemoryIndexIdentityState {
     const hasProviderOverride = params && "provider" in params;
-    // The plain `memory status` path resolves identity before the provider is
-    // initialized, so settings.model is still the unresolved default: an empty
-    // string when no explicit model is configured. Mirror provider init's model
-    // resolution here (configured model, else adapter default) so identity does
-    // not falsely report a mismatch against meta with a blank "expected" model.
-    const configuredModel =
-      this.settings.model.trim() ||
-      resolveEmbeddingProviderFallbackModel(this.settings.provider, "", this.cfg);
+    // Plain status can compare identity before provider init. Mirror provider
+    // init's empty-model fallback so adapter defaults do not look mismatched.
     const configuredProvider =
       this.settings.provider === "none"
         ? null
         : {
             id:
               resolveEmbeddingProviderAdapterId(this.settings.provider, this.cfg) ??
               this.settings.provider,
-            model: configuredModel,
+            model:
+              this.settings.model.trim() ||
+              resolveEmbeddingProviderFallbackModel(this.settings.provider, "", this.cfg),
           };
     const provider = hasProviderOverride
       ? params.provider!
