Update snapshot files to remove unnecessary attributes and improve consistency in property handling

- Cleared out redundant attributes from various snapshot files, specifically removing `Rojo_Target_Value` and `Rojo_Target_PrimaryPart` where applicable.
- Ensured that attributes are consistently represented as empty objects when not needed, enhancing clarity in the snapshot structure.
- Adjusted related test cases to reflect these changes, ensuring alignment with the updated snapshot expectations.

rojo-test/serve-test-snapshots/end_to_end__tests__serve__meshpart_with_id_all.snap

@@ -36,9 +36,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: sword
       Value:
         Ref: id-5
   id-5:

rojo-test/serve-test-snapshots/end_to_end__tests__serve__meshpart_with_id_serialize_model.snap

@@ -34,7 +34,7 @@ expression: model
       <Item class="ObjectValue" referent="4">
         <Properties>
           <string name="Name">ObjectValue</string>
-          <BinaryString name="AttributesSerialize">AQAAABEAAABSb2pvX1RhcmdldF9WYWx1ZQIFAAAAc3dvcmQ=</BinaryString>
+          <BinaryString name="AttributesSerialize"></BinaryString>
           <Ref name="Value">null</Ref>
         </Properties>
       </Item>

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_all-2.snap

@@ -13,9 +13,7 @@ instances:
     Parent: id-9
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: project target
       Value:
         Ref: id-9
   id-11:
@@ -28,9 +26,7 @@ instances:
     Parent: id-7
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_PrimaryPart:
-            String: project target
       NeedsPivotMigration:
         Bool: false
       PrimaryPart:
@@ -70,9 +66,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-5:
@@ -95,9 +89,7 @@ instances:
     Parent: id-5
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-7:
@@ -112,9 +104,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Id:
-            String: model target 2
   id-8:
     Children: []
     ClassName: Model
@@ -125,9 +115,7 @@ instances:
     Parent: id-7
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_PrimaryPart:
-            String: model target 2
       NeedsPivotMigration:
         Bool: false
       PrimaryPart:

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_all.snap

@@ -13,9 +13,7 @@ instances:
     Parent: id-9
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: project target
       Value:
         Ref: id-9
   id-2:
@@ -53,9 +51,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-5:
@@ -78,9 +74,7 @@ instances:
     Parent: id-5
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-7:
@@ -103,9 +97,7 @@ instances:
     Parent: id-7
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_PrimaryPart:
-            String: model target
       NeedsPivotMigration:
         Bool: false
       PrimaryPart:

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_patch_update_all-2.snap

@@ -13,9 +13,7 @@ instances:
     Parent: id-9
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: project target
       Value:
         Ref: id-9
   id-2:
@@ -53,9 +51,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-5:
@@ -78,9 +74,7 @@ instances:
     Parent: id-5
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-7:
@@ -103,9 +97,7 @@ instances:
     Parent: id-7
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_PrimaryPart:
-            String: model target
       NeedsPivotMigration:
         Bool: false
       PrimaryPart:

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_patch_update_all.snap

@@ -13,9 +13,7 @@ instances:
     Parent: id-9
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: project target
       Value:
         Ref: id-9
   id-2:
@@ -53,9 +51,7 @@ instances:
     Parent: id-3
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-5:
@@ -78,9 +74,7 @@ instances:
     Parent: id-5
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: folder target
       Value:
         Ref: id-5
   id-7:
@@ -103,9 +97,7 @@ instances:
     Parent: id-7
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_PrimaryPart:
-            String: model target
       NeedsPivotMigration:
         Bool: false
       PrimaryPart:

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_patch_update_subscribe.snap

@@ -12,6 +12,10 @@ body:
           changedMetadata: ~
           changedName: ~
           changedProperties:
+            Attributes:
+              Attributes:
+                Rojo_Target_PrimaryPart:
+                  String: model target
             Scale:
               Float32: 1
           id: id-8

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_remove_all-2.snap

@@ -23,11 +23,8 @@ instances:
     Parent: id-2
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: test pointer
       Value:
         Ref: id-4
 messageCursor: 1
 sessionId: id-1
-

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_remove_all.snap

@@ -24,9 +24,7 @@ instances:
     Parent: id-2
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Target_Value:
-            String: test pointer
       Value:
         Ref: id-4
   id-4:
@@ -39,9 +37,6 @@ instances:
     Parent: id-2
     Properties:
       Attributes:
-        Attributes:
+        Attributes: {}
-          Rojo_Id:
-            String: test pointer
 messageCursor: 0
 sessionId: id-1
-

rojo-test/serve-test-snapshots/end_to_end__tests__serve__ref_properties_subscribe.snap

@@ -16,9 +16,7 @@ body:
           Parent: id-7
           Properties:
             Attributes:
-              Attributes:
+              Attributes: {}
-                Rojo_Target_PrimaryPart:
-                  String: project target
             NeedsPivotMigration:
               Bool: false
             PrimaryPart:

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__ignore_paths_adding-stdout.snap

@@ -2,6 +2,7 @@
 source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
+Writing src/ignored_part.model.json
 Writing src/int_value.model.json
 Writing src/subfolder/string_value.txt
 Writing src/subfolder

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

@@ -2,4 +2,4 @@
 source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
-Writing src/Message.rbxm
+Writing src/Message.model.json

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__rbxm_fallback-stdout.snap

@@ -2,8 +2,6 @@
 source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
-Writing src/ChildWithDuplicates.rbxm
 Writing src/ChildWithoutDuplicates/Child/.gitkeep
 Writing src/ChildWithoutDuplicates
 Writing src/ChildWithoutDuplicates/Child
-Removing src/ChildWithDuplicates

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__ref_properties_blank-stdout.snap

@@ -3,5 +3,4 @@ source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
 Writing src/pointer.model.json
-Writing src/target.meta.json
 Writing src/target.txt

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__ref_properties_conflict-stdout.snap

@@ -2,5 +2,7 @@
 source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
+Writing src/Pointer_1.model.json
 Writing src/Pointer_2.model.json
+Writing src/Target_1.model.json
 Writing src/Target_2.model.json

rojo-test/syncback-test-snapshots/end_to_end__rojo_test__syncback_util__ref_properties_duplicate-stdout.snap

@@ -2,4 +2,6 @@
 source: tests/rojo_test/syncback_util.rs
 expression: "String::from_utf8_lossy(&output.stdout)"
 ---
-
+Writing src/Pointer_1.model.json
+Writing src/Pointer_2.model.json
+Writing src/Target.model.json

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__duplicate_rojo_id-container.model.json.snap

@@ -7,15 +7,15 @@ expression: container.model.json
     {
       "attributes": {
         "Rojo_Id": "value_1",
-        "Rojo_Target_Value": "value_1"
+        "Rojo_Ref_Value": "ReplicatedStorage/container/value_1"
       },
       "className": "ObjectValue",
       "name": "value_1"
     },
     {
       "attributes": {
-        "Rojo_Id": "72bc28150ada2e6206442ee300004084",
+        "Rojo_Id": "value_1",
-        "Rojo_Target_Value": "72bc28150ada2e6206442ee300004084"
+        "Rojo_Ref_Value": "ReplicatedStorage/container/value_2"
       },
       "className": "ObjectValue",
       "name": "value_2"

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ignore_paths_removing-src__Message.model.json.snap

@@ -0,0 +1,10 @@
+---
+source: tests/tests/syncback.rs
+expression: src/Message.model.json
+---
+{
+  "className": "Message",
+  "properties": {
+    "Text": "This message should be written to the disk."
+  }
+}

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties-src__pointer.model.json.snap

@@ -4,7 +4,7 @@ expression: src/pointer.model.json
 ---
 {
   "attributes": {
-    "Rojo_Target_Value": "test referent id"
+    "Rojo_Ref_Value": "ReplicatedStorage/target"
   },
   "className": "ObjectValue"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_blank-src__pointer.model.json.snap

@@ -4,7 +4,7 @@ expression: src/pointer.model.json
 ---
 {
   "attributes": {
-    "Rojo_Target_Value": "62e89c49e4f800c20629c71b00003fc0"
+    "Rojo_Ref_Value": "ReplicatedStorage/target"
   },
   "className": "ObjectValue"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_conflict-src__Pointer_2.model.json.snap

@@ -4,7 +4,7 @@ expression: src/Pointer_2.model.json
 ---
 {
   "attributes": {
-    "Rojo_Target_Value": "0bd3e9e11879191708a2bc4a00000c20"
+    "Rojo_Ref_Value": "ReplicatedStorage/Target_2"
   },
   "className": "ObjectValue"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_conflict-src__Target_2.model.json.snap

@@ -4,7 +4,7 @@ expression: src/Target_2.model.json
 ---
 {
   "attributes": {
-    "Rojo_Id": "0bd3e9e11879191708a2bc4a00000c20"
+    "Rojo_Id": "identical ID"
   },
   "className": "Folder"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_pruned-src__Pointer1.model.json.snap

@@ -3,5 +3,8 @@ source: tests/tests/syncback.rs
 expression: src/Pointer1.model.json
 ---
 {
+  "attributes": {
+    "Rojo_Ref_Value": "ReplicatedStorage/Target1"
+  },
   "className": "ObjectValue"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_pruned-src__Pointer2.model.json.snap

@@ -3,5 +3,8 @@ source: tests/tests/syncback.rs
 expression: src/Pointer2.model.json
 ---
 {
+  "attributes": {
+    "Rojo_Ref_Value": "ReplicatedStorage/Target2"
+  },
   "className": "ObjectValue"
 }

rojo-test/syncback-test-snapshots/end_to_end__tests__syncback__ref_properties_pruned-src__Pointer3.model.json.snap

@@ -3,5 +3,8 @@ source: tests/tests/syncback.rs
 expression: src/Pointer3.model.json
 ---
 {
+  "attributes": {
+    "Rojo_Ref_Value": "ReplicatedStorage/Target3"
+  },
   "className": "ObjectValue"
 }

tests/tests/syncback.rs

@@ -42,7 +42,7 @@ syncback_tests! {
     // Ensures that the `ignorePaths` setting works for `init` files
     ignore_paths_init => ["src/non-init.luau", "src/init-file/init.luau"],
     // Ensures that the `ignorePaths` setting works for removals
-    ignore_paths_removing => ["src/Message.rbxm"],
+    ignore_paths_removing => ["src/Message.model.json"],
     // Ensures that `ignoreTrees` works for additions
     ignore_trees_adding => [],
     // Ensures that `ignoreTrees` works for removals
@@ -61,12 +61,12 @@ syncback_tests! {
     // default.project.json doesn't change unexpectedly.
     project_reserialize => ["attribute_mismatch.luau", "property_mismatch.project.json"],
     // Confirms that Instances that cannot serialize as directories serialize as rbxms
-    rbxm_fallback => ["src/ChildWithDuplicates.rbxm"],
+    rbxm_fallback => [],
     // Ensures that ref properties are linked properly on the file system
     ref_properties => ["src/pointer.model.json", "src/target.model.json"],
     // Ensures that ref properties are linked when no attributes are manually
     // set in the DataModel
-    ref_properties_blank => ["src/pointer.model.json", "src/target.meta.json", "src/target.txt"],
+    ref_properties_blank => ["src/pointer.model.json", "src/target.txt"],
     // Ensures that if there is a conflict in RojoRefs, one of them is rewritten.
     ref_properties_conflict => ["src/Pointer_2.model.json", "src/Target_2.model.json"],
     // Ensures that having multiple pointers that are aimed at the same target doesn't trigger ref rewrites.