chore: Error if there are non-Readable Mesh

CHANGELOG-PRERELEASE.md

@@ -17,6 +17,7 @@ The format is based on [Keep a Changelog].
 ### Removed
 
 ### Fixed
+- Unclear Error with Mesh with Read/Write off `#386`
 
 ### Security
 

CHANGELOG.md

@@ -34,6 +34,7 @@ The format is based on [Keep a Changelog].
 
 ### Fixed
 - MergeBone not working well with non-restpose bones `#379`
+- Unclear Error with Mesh with Read/Write off `#386`
 
 ### Security
 

Editor/OptimizerSession.cs

@@ -55,6 +55,9 @@ public T AddToAsset<T>(T obj) where T : Object
         public T MayInstantiate<T>(T obj) where T : Object =>
             _added.Contains(obj) ? obj : AddToAsset(Object.Instantiate(obj));
 
+        public T MayCreate<T>(T obj) where T : Object, new() =>
+            _added.Contains(obj) ? obj : AddToAsset(new T { name = obj.name + "(Clone)" });
+
         public void MarkDirtyAll()
         {
             foreach (var o in _added)

Editor/Processors/EditSkinnedMeshComponentProcessor.cs

@@ -75,7 +75,7 @@ public void SaveToMesh(OptimizerSession session)
                 BuildReport.ReportingObject(targetRenderer, () =>
                 {
                     var mesh = meshFilter.sharedMesh
-                        ? session.MayInstantiate(meshFilter.sharedMesh)
+                        ? session.MayCreate(meshFilter.sharedMesh)
                         : session.AddToAsset(new Mesh());
                     meshInfo.WriteToMesh(mesh);
                     meshFilter.sharedMesh = mesh;

Editor/Processors/SkinnedMeshes/MeshInfo2.cs

@@ -34,10 +34,14 @@ internal class MeshInfo2
         public MeshInfo2(SkinnedMeshRenderer renderer)
         {
             SourceRenderer = renderer;
-            var mesh = renderer.sharedMesh
-                ? renderer.sharedMesh
-                : new Mesh { name = $"AAOGeneratedMesh({renderer.name})" };
-
+            var mesh = renderer.sharedMesh;
+            if (!mesh) return; // no mesh; make empty mesh
+            if (!mesh.isReadable)
+            {
+                BuildReport.LogFatal("The Mesh is not readable. Please Check Read/Write")?.WithContext(mesh);
+                return;
+            }
+
             BuildReport.ReportingObject(renderer, true, () =>
             {
                 ReadSkinnedMesh(mesh);
@@ -67,6 +71,12 @@ public MeshInfo2(MeshRenderer renderer)
             BuildReport.ReportingObject(renderer, true, () =>
             {
                 var mesh = renderer.GetComponent<MeshFilter>().sharedMesh;
+                if (!mesh) return; // no mesh; make empty mesh
+                if (!mesh.isReadable)
+                {
+                    BuildReport.LogFatal("The Mesh is not readable. Please Check Read/Write")?.WithContext(mesh);
+                    return;
+                }
                 ReadStaticMesh(mesh);
 
                 SetIdentityBone(renderer.transform);
@@ -438,7 +448,7 @@ public void WriteToSkinnedMeshRenderer(SkinnedMeshRenderer targetRenderer, Optim
             BuildReport.ReportingObject(targetRenderer, () =>
             {
                 var mesh = targetRenderer.sharedMesh
-                    ? session.MayInstantiate(targetRenderer.sharedMesh)
+                    ? session.MayCreate(targetRenderer.sharedMesh)
                     : session.AddToAsset(new Mesh { name = $"AAOGeneratedMesh{targetRenderer.name}" });
 
                 WriteToMesh(mesh);