fix(api): Project hard sync existing entities deleted (#660)

Co-authored-by: Rajdip Bhattacharya <agentR47@gmail.com>

apps/api/src/project/service/project.service.ts

@@ -918,6 +918,11 @@ export class ProjectService {
     // hardCopy = false: Only add those items in the toProject that are not already present in it
     hardCopy: boolean = false
   ) {
+    // This field will be populated if hardCopy is true
+    // When we are doing a hard copy, we need to delete all the
+    // items in the toProject that are already present in it
+    const deleteOps = []
+
     // Get all the environments that belongs to the parent project
     // and replicate them for the new project
     const createEnvironmentOps = []
@@ -964,6 +969,30 @@ export class ProjectService {
       variables.forEach((variable) => {
         toProjectVariables.add(variable.name)
       })
+    } else {
+      deleteOps.push(
+        this.prisma.environment.deleteMany({
+          where: {
+            projectId: toProject.id
+          }
+        })
+      )
+
+      deleteOps.push(
+        this.prisma.secret.deleteMany({
+          where: {
+            projectId: toProject.id
+          }
+        })
+      )
+
+      deleteOps.push(
+        this.prisma.variable.deleteMany({
+          where: {
+            projectId: toProject.id
+          }
+        })
+      )
     }
 
     // We want to find all such environments in the fromProject that
@@ -1117,7 +1146,12 @@ export class ProjectService {
       )
     }
 
-    return [...createEnvironmentOps, ...createSecretOps, ...createVariableOps]
+    return [
+      ...deleteOps,
+      ...createEnvironmentOps,
+      ...createSecretOps,
+      ...createVariableOps
+    ]
   }
 
   /**

docs/cli/project.md

@@ -180,7 +180,7 @@ These are the options supported by the `sync` command:
 
 #### `-h, --hard-sync`
 
-Upserts a new copy of the parent onto the child. Defaults to soft sync.
+Force the child to match the parent by discarding all changes in the child. Defaults to soft sync.
 
 ## `unlink`