4756 Delete jobs child-first to avoid double-deleting recursive children

jobFacade.deleteJob recursively deletes child jobs, so iterating a flat jobIds list that contains both parents and children revisits already-deleted rows. Sort by id descending (children have larger ids since they are inserted later) and de-dup before the delete loop. Loosen the ordering assertion in the deployment test accordingly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: ivicac

server/ee/libs/embedded/embedded-configuration/embedded-configuration-service/src/main/java/com/bytechef/ee/embedded/configuration/facade/IntegrationInstanceConfigurationFacadeImpl.java

@@ -62,6 +62,7 @@
 import java.time.Instant;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -239,7 +240,12 @@ public void deleteIntegrationInstanceConfiguration(long id) {
             principalJobService.deletePrincipalJobs(jobId, PlatformType.EMBEDDED);
         }
 
-        for (long jobId : jobIds) {
+        List<Long> orderedJobIds = jobIds.stream()
+            .distinct()
+            .sorted(Comparator.reverseOrder())
+            .toList();
+
+        for (long jobId : orderedJobIds) {
             jobFacade.deleteJob(jobId);
         }
 

server/libs/automation/automation-configuration/automation-configuration-service/src/main/java/com/bytechef/automation/configuration/facade/ProjectDeploymentFacadeImpl.java

@@ -62,6 +62,7 @@
 import java.time.Instant;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
@@ -224,7 +225,12 @@ public void deleteProjectDeployment(long id) {
             principalJobService.deletePrincipalJobs(jobId, PlatformType.AUTOMATION);
         }
 
-        for (long jobId : jobIds) {
+        List<Long> orderedJobIds = jobIds.stream()
+            .distinct()
+            .sorted(Comparator.reverseOrder())
+            .toList();
+
+        for (long jobId : orderedJobIds) {
             jobFacade.deleteJob(jobId);
         }
 

server/libs/automation/automation-configuration/automation-configuration-service/src/test/java/com/bytechef/automation/configuration/facade/ProjectDeploymentFacadeIntTest.java

@@ -24,6 +24,7 @@
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.atLeastOnce;
+import static org.mockito.Mockito.calls;
 import static org.mockito.Mockito.inOrder;
 import static org.mockito.Mockito.reset;
 import static org.mockito.Mockito.verify;
@@ -211,10 +212,13 @@ public void testDeleteProjectDeploymentRemovesAllPrincipalJobsBeforeAnyJob() {
             .deletePrincipalJobs(parentJobId, PlatformType.AUTOMATION);
         inOrder.verify(principalJobService)
             .deletePrincipalJobs(childJobId, PlatformType.AUTOMATION);
-        inOrder.verify(jobFacade)
-            .deleteJob(parentJobId);
-        inOrder.verify(jobFacade)
-            .deleteJob(childJobId);
+        inOrder.verify(jobFacade, calls(1))
+            .deleteJob(anyLong());
+        inOrder.verify(jobFacade, calls(1))
+            .deleteJob(anyLong());
+
+        verify(jobFacade).deleteJob(parentJobId);
+        verify(jobFacade).deleteJob(childJobId);
     }
 
     @Disabled