🔧 fix(parameterComponent): remove unused errorDuplicateKey state variable

🔧 fix(parameterComponent): remove unused dict and dictArr state variables
🔧 fix(parameterComponent): remove unused useEffect hook
🔧 fix(parameterComponent): update onChange handler to setObj instead of setDict
🔧 fix(keypairListComponent): remove commented out code for future feature
🔧 fix(EditNodeModal): update condition to check for "keypairlist" type instead of "keypair" type
🔧 fix(types): update onChange type in DictComponentType to accept any value
🔧 fix(reactflowUtils): remove unnecessary comments and newline at end of file

src/frontend/src/CustomNodes/GenericNode/components/parameterComponent/index.tsx

@@ -54,7 +54,6 @@ export default function ParameterComponent({
   const updateNodeInternals = useUpdateNodeInternals();
   const [position, setPosition] = useState(0);
   const { setTabsState, tabId, save, flows } = useContext(TabsContext);
-  const [errorDuplicateKey, setErrorDuplicateKey] = useState(false);
 
   const flow = flows.find((flow) => flow.id === tabId)?.data?.nodes ?? null;
 
@@ -96,20 +95,6 @@ export default function ParameterComponent({
     renderTooltips();
   };
 
-  const [dict, setDict] = useState({
-    key1: "value1",
-    key2: "value2",
-    key3: "value3",
-    key4: "value4",
-    key5: "value5",
-    key6: "value6",
-  } as {});
-  const [dictArr, setDictArr] = useState([] as string[]);
-
-  useEffect(() => {
-    setDictArr(convertObjToArray(dict));
-  }, [dict]);
-
   
   const [obj, setObj] = useState({
     arr: ["test", 123456, false, null],
@@ -248,30 +233,12 @@ export default function ParameterComponent({
               editNode={false}
               value={obj}
               onChange={(newValue) => {
-                setDict(newValue);
+                setObj(newValue);
               }}
             />
           </div>
         )
-        : left === true && type === "keypair" ? (
+        : (
-          <div className="mt-2 w-full">
-            <KeypairListComponent
-              disabled={disabled}
-              editNode={false}
-              value={dictArr}
-              duplicateKey={errorDuplicateKey}
-              onChange={(newValue: string[]) => {
-                setErrorDuplicateKey(hasDuplicateKeys(newValue));
-                if(hasDuplicateKeys(newValue)){
-                  setDictArr(newValue);
-                }
-                else{
-                  setDict(convertArrayToObj(newValue));
-                }
-              }}
-            />
-          </div>
-        ) : (
           <ShadTooltip
             styleClasses={"tooltip-fixed-width custom-scroll nowheel"}
             delayDuration={0}

src/frontend/src/components/keypairListComponent/index.tsx

@@ -20,6 +20,42 @@ export default function KeypairListComponent({
       onChange([""]);
     }
   }, [disabled]);
+
+  //when this feature is available, this code below must be in the parent component
+  // const [errorDuplicateKey, setErrorDuplicateKey] = useState(false);
+  // const [dict, setDict] = useState({
+  //   key1: "value1",
+  //   key2: "value2",
+  //   key3: "value3",
+  //   key4: "value4",
+  //   key5: "value5",
+  //   key6: "value6",
+  // } as {});
+  // const [dictArr, setDictArr] = useState([] as string[]);
+
+  // useEffect(() => {
+  //   setDictArr(convertObjToArray(dict));
+  // }, [dict]);
+
+  // left === true && type === "keypairlist" ? (
+  //   <div className="mt-2 w-full">
+  //     <KeypairListComponent
+  //       disabled={disabled}
+  //       editNode={false}
+  //       value={dictArr}
+  //       duplicateKey={errorDuplicateKey}
+  //       onChange={(newValue: string[]) => {
+  //         setErrorDuplicateKey(hasDuplicateKeys(newValue));
+  //         if(hasDuplicateKeys(newValue)){
+  //           setDictArr(newValue);
+  //         }
+  //         else{
+  //           setDict(convertArrayToObj(newValue));
+  //         }
+  //       }}
+  //     />
+  //   </div>
+  // )
   
 
  const handleChangeKey = (event, idx) => {

src/frontend/src/modals/EditNodeModal/index.tsx

@@ -219,7 +219,7 @@ const EditNodeModal = forwardRef(
                                     </div>
                                   )
                                   : myData.node?.template[templateParam]
-                                      .type === "keypair" ? (
+                                      .type === "keypairlist" ? (
                                     <div className="mt-2 w-full">
                                       <KeypairListComponent
                                         disabled={disabled}

src/frontend/src/types/components/index.ts

@@ -64,7 +64,7 @@ export type KeyPairListComponentType = {
 
 export type DictComponentType = {
   value: any;
-  onChange: (value: string[]) => void;
+  onChange: (value) => void;
   disabled: boolean;
   editNode?: boolean;
 };

src/frontend/src/utils/reactflowUtils.ts

@@ -321,10 +321,10 @@ export function hasDuplicateKeys(array) {
   for (const obj of array) {
       for (const key in obj) {
           if (keys[key]) {
-              return true; // Duplicate key found
+              return true; 
           }
           keys[key] = true;
       }
   }
-  return false; // No duplicate keys found
+  return false; 
 }