@angular-architects/ngrx-toolkit 20.4.0 → 20.4.1

package/fesm2022/angular-architects-ngrx-toolkit.mjs

@@ -342,7 +342,7 @@ Enable automatic indexing via withDevTools('${storeName}', { indexNames: true })
         }, {});
         this.#currentState = Object.entries(this.#currentState).reduce((newState, [storeName, state]) => {
             if (storeName !== name) {
-                newState[name] = state;
+                newState[storeName] = state;
             }
             return newState;
         }, {});
@@ -350,18 +350,31 @@ Enable automatic indexing via withDevTools('${storeName}', { indexNames: true })
             tracker.removeStore(id);
         }
     }
-    renameStore(oldName, newName) {
-        const storeNames = Object.values(this.#stores).map((store) => store.name);
-        const id = throwIfNull(Object.keys(this.#stores).find((id) => this.#stores[id].name === oldName));
-        if (storeNames.includes(newName)) {
+    /**
+     * Renames a store identified by its internal id. If the store has already
+     * been removed (e.g. due to component destruction), this is a no-op.
+     */
+    renameStore(id, newName) {
+        const storeEntry = this.#stores[id];
+        if (!storeEntry) {
+            return;
+        }
+        const oldName = storeEntry.name;
+        if (oldName === newName) {
+            return;
+        }
+        const otherStoreNames = Object.entries(this.#stores)
+            .filter(([entryId]) => entryId !== id)
+            .map(([, s]) => s.name);
+        if (otherStoreNames.includes(newName)) {
             throw new Error(`NgRx Toolkit/DevTools: cannot rename from ${oldName} to ${newName}. ${newName} is already assigned to another SignalStore instance.`);
         }
-        this.#stores = Object.entries(this.#stores).reduce((newStore, [id, value]) => {
-            if (value.name === oldName) {
-                newStore[id] = { ...value, name: newName };
+        this.#stores = Object.entries(this.#stores).reduce((newStore, [entryId, value]) => {
+            if (entryId === id) {
+                newStore[entryId] = { ...value, name: newName };
             }
             else {
-                newStore[id] = value;
+                newStore[entryId] = value;
             }
             return newStore;
         }, {});
@@ -406,7 +419,7 @@ function withDevtools(name, ...features) {
         // TODO: use withProps and symbols
         return {
             [renameDevtoolsMethodName]: (newName) => {
-                syncer.renameStore(name, newName);
+                syncer.renameStore(id, newName);
             },
             [uniqueDevtoolsId]: () => id,
         };