fix: Prevent duplicate settings when copying workflow settings (#3204)

Author: SuaYoo

frontend/src/pages/org/workflows-new.ts

@@ -5,6 +5,8 @@ import { html } from "lit";
 import { customElement, property } from "lit/decorators.js";
 import { ifDefined } from "lit/directives/if-defined.js";
 import { when } from "lit/directives/when.js";
+import mapValues from "lodash/fp/mapValues";
+import uniq from "lodash/fp/uniq";
 import type { PartialDeep } from "type-fest";
 
 import {
@@ -26,6 +28,15 @@ import {
   type FormState as WorkflowFormState,
 } from "@/utils/workflow";
 
+// Make array values unique since mergeDeep will concatenate arrays
+const makeArrUniq = mapValues((val: unknown) => {
+  if (Array.isArray(val)) {
+    return uniq(val) as unknown[];
+  }
+
+  return val;
+});
+
 /**
  * Usage:
  * ```ts
@@ -126,7 +137,7 @@ export class WorkflowsNew extends BtrixElement {
         </sl-button>
       </header>
       ${when(this.org, (org) => {
-        const initialWorkflow = mergeDeep(
+        const mergedWorkflow = mergeDeep(
           this.defaultNewWorkflow,
           {
             profileid: org.crawlingDefaults?.profileid,
@@ -153,6 +164,11 @@ export class WorkflowsNew extends BtrixElement {
           this.initialWorkflow || {},
         );
 
+        const initialWorkflow = makeArrUniq({
+          ...mergedWorkflow,
+          config: makeArrUniq(mergedWorkflow.config),
+        }) as WorkflowParams;
+
         const scopeType = this.scopeType || initialWorkflow.config.scopeType;
 
         return html`