fix: recursive error causing the security page to crash (#732)

Signed-off-by: Efren Lim <[email protected]>

Author: emlimlf

frontend/app/components/modules/project/views/security.vue

@@ -220,10 +220,15 @@ const groupedData = computed(() => (securityAssessmentData.value || []).reduce((
     if (!obj[check.category]) {
       obj[check.category] = [];
     }
-    check.assessments = PROJECT_SECURITY_SERVICE.orderAssessmentsByRequirementId(
+    const tmpAssessments = PROJECT_SECURITY_SERVICE.orderAssessmentsByRequirementId(
       PROJECT_SECURITY_SERVICE.mergeDuplicateAssessments(check.assessments)
     );
-    obj[check.category]?.push(check);
+    // Create a copy of the check object to avoid mutating the original
+    const checkCopy = {
+      ...check,
+      assessments: tmpAssessments
+    };
+    obj[check.category]?.push(checkCopy);
     return obj;
   }, {} as Record<string, SecurityData[]>))
 
@@ -232,10 +237,15 @@ const groupChecksByRepository = (checks: SecurityData[]) => (checks || []).reduc
     if (!obj[check.repo]) {
       obj[check.repo] = [];
     }
-    check.assessments = PROJECT_SECURITY_SERVICE.orderAssessmentsByRequirementId(
+    const tmpAssessments = PROJECT_SECURITY_SERVICE.orderAssessmentsByRequirementId(
       PROJECT_SECURITY_SERVICE.mergeDuplicateAssessments(check.assessments)
     );
-    obj[check.repo]?.push(check);
+    // Create a copy of the check object to avoid mutating the original
+    const checkCopy = {
+      ...check,
+      assessments: tmpAssessments
+    };
+    obj[check.repo]?.push(checkCopy);
     return obj;
   }, {} as Record<string, SecurityData[]>)