From dd2fee3eca371be0369fd7bab273a6233de7155e Mon Sep 17 00:00:00 2001
From: carlosmonastyrski <charly.monastyrski@gmail.com>
Date: Tue, 25 Mar 2025 08:45:38 -0300
Subject: [PATCH] Coderabbit code suggestions

---
 .../ReplicateFolderFromBoard/ReplicateFolderFromBoard.tsx   | 2 +-
 .../ActionBar/ReplicateFolderFromBoard/SecretTreeView.tsx   | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/ReplicateFolderFromBoard.tsx b/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/ReplicateFolderFromBoard.tsx
index 16eef8769..339612a89 100644
--- a/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/ReplicateFolderFromBoard.tsx
+++ b/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/ReplicateFolderFromBoard.tsx
@@ -171,7 +171,7 @@ export const ReplicateFolderFromBoard = ({
       const segment = segments[i];
       currentPath += `/${segment}`;
 
-      if (currentLevel.subFolders && currentLevel.subFolders[segment]) {
+      if (currentLevel?.subFolders?.[segment]) {
         currentLevel = currentLevel.subFolders[segment];
 
         if (currentPath === normalizedPath) {
diff --git a/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/SecretTreeView.tsx b/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/SecretTreeView.tsx
index 824dec1f5..4119f54e3 100644
--- a/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/SecretTreeView.tsx
+++ b/frontend/src/pages/secret-manager/SecretDashboardPage/components/ActionBar/ReplicateFolderFromBoard/SecretTreeView.tsx
@@ -128,7 +128,7 @@ const Folder: React.FC<FolderProps> = ({
             />
           </div>
           <Checkbox
-            id={`folder-${path}`}
+            id="folder-root"
             className="data-[state=indeterminate]:bg-secondary data-[state=checked]:bg-primary"
             isChecked={allSelected || someSelected}
             onCheckedChange={handleFolderSelect}
@@ -159,7 +159,7 @@ const Folder: React.FC<FolderProps> = ({
                   <FontAwesomeIcon icon={faKey} className="h-3 w-3" />
                 </div>
                 <Checkbox
-                  id={`folder-${path}`}
+                  id={`folder-${item.id}`}
                   className="data-[state=indeterminate]:bg-secondary data-[state=checked]:bg-primary"
                   isChecked={selectedItemIds.includes(item.id)}
                   onCheckedChange={(checked) => onItemSelect(item, !!checked)}
@@ -240,7 +240,7 @@ export const SecretTreeView: React.FC<TreeViewProps> = ({
       current = tree["/"];
 
       const targetExists = parts.every((part) => {
-        if (current?.subFolders && current.subFolders[part]) {
+        if (current?.subFolders?.[part]) {
           current = current.subFolders[part];
           return true;
         }