Make removing files respect ignore_paths

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__ignore_paths_removing.snap

@@ -0,0 +1,10 @@
+---
+source: tests/rojo_test/syncback_util.rs
+assertion_line: 48
+expression: "visualize_fs_snapshot(&fs_snapshot, &output_path)"
+---
+added_files: []
+added_dirs:
+  - src
+removed_files: []
+removed_dirs: []

rojo-test/syncback-tests/ignore_paths_removing/expected/default.project.json

@@ -0,0 +1,33 @@
+{
+  "name": "ignore_paths",
+  "tree": {
+    "$className": "DataModel",
+    "Workspace": {
+      "$properties": {
+        "CSGAsyncDynamicCollision": {
+          "Enum": 0
+        },
+        "DecreaseMinimumPartDensityMode": {
+          "Enum": 0
+        },
+        "MoverConstraintRootBehavior": {
+          "Enum": 0
+        },
+        "RenderingCacheOptimizations": {
+          "Enum": 0
+        },
+        "SignalBehavior": "Deferred",
+        "StreamOutBehavior": "Opportunistic",
+        "StreamingEnabled": true,
+        "StreamingIntegrityMode": "PauseOutsideLoadedArea"
+      },
+      "$path": "src"
+    }
+  },
+  "syncbackRules": {
+    "ignorePaths": [
+      "*.rbxm",
+      "*.luau"
+    ]
+  }
+}

rojo-test/syncback-tests/ignore_paths_removing/expected/src/not_in_place.luau

@@ -0,0 +1 @@
+-- This script is not in the input place file.

rojo-test/syncback-tests/ignore_paths_removing/output/default.project.json

@@ -0,0 +1,33 @@
+{
+  "name": "ignore_paths",
+  "tree": {
+    "$className": "DataModel",
+    "Workspace": {
+      "$properties": {
+        "CSGAsyncDynamicCollision": {
+          "Enum": 0
+        },
+        "DecreaseMinimumPartDensityMode": {
+          "Enum": 0
+        },
+        "MoverConstraintRootBehavior": {
+          "Enum": 0
+        },
+        "RenderingCacheOptimizations": {
+          "Enum": 0
+        },
+        "SignalBehavior": "Deferred",
+        "StreamOutBehavior": "Opportunistic",
+        "StreamingEnabled": true,
+        "StreamingIntegrityMode": "PauseOutsideLoadedArea"
+      },
+      "$path": "src"
+    }
+  },
+  "syncbackRules": {
+    "ignorePaths": [
+      "*.rbxm",
+      "*.luau"
+    ]
+  }
+}

rojo-test/syncback-tests/ignore_paths_removing/output/src/not_in_place.luau

@@ -0,0 +1 @@
+-- This script is not in the input place file.

src/syncback/mod.rs

@@ -233,6 +233,13 @@ pub fn syncback_loop(
             );
             for inst in &syncback.removed_children {
                 let path = inst.metadata().instigating_source.as_ref().unwrap().path();
+                if !is_valid_path(&ignore_patterns, project_path, path) {
+                    log::debug!(
+                        "Skipping removing {} because its matches an ignore pattern",
+                        path.display()
+                    );
+                    continue;
+                }
                 if path.is_dir() {
                     fs_snapshot.remove_dir(path)
                 } else {

tests/tests/syncback.rs

@@ -30,4 +30,5 @@ syncback_basic_test! {
     string_value_project,
     child_but_not,
     ignore_trees,
+    ignore_paths_removing,
 }