Refactored deepMerge function to improve performance (#1211)

.changeset/late-coins-beam.md

@@ -0,0 +1,5 @@
+---
+'svelte-meta-tags': patch
+---
+
+Refactored deepMerge function to improve performance

.github/workflows/ci.yml

@@ -24,5 +24,5 @@ jobs:
         run: pnpm build
       - name: Install Playwright
         run: pnpm dlx [email protected] install --with-deps
-      - name: Run Playwright
+      - name: Run Vitest and Playwright
         run: pnpm test

.github/workflows/release.yml

@@ -18,11 +18,6 @@ jobs:
       - name: Checkout Repo
         uses: actions/checkout@v4
       - uses: pnpm/[email protected]
-      - name: Setup Node.js
-        uses: actions/setup-node@v4
-        with:
-          node-version: '20.18.0'
-          cache: pnpm
       - run: pnpm install --frozen-lockfile
       - name: Create Release Pull Request or Publish to npm
         id: changesets

packages/svelte-meta-tags/src/lib/deepMerge.ts

@@ -2,26 +2,25 @@
 export const deepMerge = <X extends Record<string | symbol | number, any>>(target: X, source: X): X => {
   if (!target || !source) return target ?? source ?? ({} as X);
 
-  return Object.entries({ ...target, ...source }).reduce((acc, [key, value]) => {
-    return {
-      ...acc,
-      [key]: (() => {
-        if (target[key] instanceof Date || typeof target[key] === 'function') {
-          return target[key];
-        }
-        if (value instanceof Date || typeof value === 'function') {
-          return value;
-        }
-        if (isObject(target[key]) && isObject(value)) {
-          return deepMerge(target[key], value);
-        }
-        if (isArray(target[key]) && isArray(value)) {
-          return value;
-        }
-        return value !== undefined ? value : target[key];
-      })()
-    };
-  }, {} as X);
+  const result: Record<string | symbol | number, any> = { ...target };
+
+  for (const [key, value] of Object.entries(source)) {
+    const targetValue = target[key];
+
+    if (targetValue instanceof Date || typeof targetValue === 'function') {
+      result[key] = targetValue;
+    } else if (value instanceof Date || typeof value === 'function') {
+      result[key] = value;
+    } else if (isObject(targetValue) && isObject(value)) {
+      result[key] = deepMerge(targetValue, value);
+    } else if (isArray(targetValue) && isArray(value)) {
+      result[key] = value;
+    } else {
+      result[key] = value !== undefined ? value : targetValue;
+    }
+  }
+
+  return result as X;
 };
 
 const isObject = (obj: any): obj is Record<string | symbol | number, any> =>