overwrite dynamix state instead of merging config file

pujitm · pujitm · commit 9e8d54202741 · 2025-06-25T14:45:05.000-04:00
diff --git a/api/src/store/modules/dynamix.ts b/api/src/store/modules/dynamix.ts
@@ -1,6 +1,5 @@
 import type { PayloadAction } from '@reduxjs/toolkit';
 import { createSlice } from '@reduxjs/toolkit';
-import { merge } from 'lodash-es';
 
 import { type DynamixConfig } from '@app/core/types/ini.js';
 import { loadDynamixConfigFile } from '@app/store/actions/load-dynamix-config-file.js';
@@ -29,7 +28,10 @@ export const dynamix = createSlice({
         });
 
         builder.addCase(loadDynamixConfigFile.fulfilled, (state, action) => {
-            Object.assign(state, action.payload, { status: FileLoadStatus.LOADED });
+            return {
+                ...(action.payload as DynamixConfig),
+                status: FileLoadStatus.LOADED,
+            };
         });
 
         builder.addCase(loadDynamixConfigFile.rejected, (state, action) => {
diff --git a/api/src/store/modules/emhttp.ts b/api/src/store/modules/emhttp.ts
@@ -2,7 +2,6 @@ import { join } from 'path';
 
 import type { PayloadAction } from '@reduxjs/toolkit';
 import { createAsyncThunk, createSlice } from '@reduxjs/toolkit';
-import { mergeWith } from 'lodash-es';
 
 import type { RootState } from '@app/store/index.js';
 import type { StateFileToIniParserMap } from '@app/store/types.js';
@@ -164,15 +163,6 @@ export const loadStateFiles = createAsyncThunk<
     return state;
 });
 
-// Custom merge function that replaces arrays instead of merging them element-wise
-const mergeReplaceArrays = (object: any, source: any, ...rest: any[]) =>
-    mergeWith(object, source, ...rest, (objValue: unknown, srcValue: unknown) => {
-        if (Array.isArray(srcValue)) {
-            return srcValue;
-        }
-        return undefined;
-    });
-
 export const emhttp = createSlice({
     name: 'emhttp',
     initialState,
