feat!: complete FixedMathSharp v7 Unity interop migration

Author: mrdav30

.agents/AGENTS.md

@@ -7,8 +7,9 @@ engine-agnostic FixedMathSharp core library. It ships two installable package
 variants:
 
 - `com.mrdav30.fixedmathsharp` - standard package with `MemoryPack` support.
-- `com.mrdav30.fixedmathsharp.lean` - no-MemoryPack package, preferred for
-  Burst AOT-oriented Unity projects.
+- `com.mrdav30.fixedmathsharp.lean` - package without the MemoryPack
+  serializer/runtime, preferred for Burst AOT-oriented Unity projects. It
+  includes the attribute-only `Chronicler.MemoryPackShim` assembly.
 
 The actual Git repository root is `Assets/Packages`, not the outer Unity
 project root. The outer project exists so Unity can compile, sync, export, and
@@ -29,12 +30,13 @@ context.
 ## Current Package Facts
 
 - Package manifests require Unity `2022.3` or newer.
-- This Unity host project currently uses Unity `6000.3.9f1` according to
+- This Unity host project currently uses Unity `6000.5.0f1` according to
   `../../ProjectSettings/ProjectVersion.txt`.
 - Both packages include the precompiled FixedMathSharp plugin assets under
   `Plugins/`.
-- The standard package carries the MemoryPack-related dependencies; the Lean
-  package intentionally omits MemoryPack.
+- The standard package carries the MemoryPack serializer/runtime dependencies.
+  The Lean package carries only `Chronicler.MemoryPackShim` for attribute
+  compatibility and intentionally omits MemoryPack runtime serialization.
 
 ## Source Of Truth
 
@@ -110,7 +112,7 @@ or exported assets change:
 | `Build/Base` | Shared managed Unity code copied into both package variants. |
 | `Build/Editor` | Unity editor tooling for syncing and exporting packages. |
 | `com.mrdav30.fixedmathsharp` | Standard UPM package with MemoryPack support. |
-| `com.mrdav30.fixedmathsharp.lean` | Lean UPM package without MemoryPack. |
+| `com.mrdav30.fixedmathsharp.lean` | Lean UPM package without the MemoryPack serializer/runtime; includes the attribute-only Chronicler shim. |
 | `Build/Base/Runtime/Extensions` | Shared Unity adapter helpers for transforms, matrices, bounds, vectors, and quaternions. |
 | `Build/Base/Runtime/Attributes` | Shared Unity-facing attributes such as fixed-number angle and vector rotation attributes. |
 | `Build/Base/Editor/Drawers` | Shared Unity inspector drawers for FixedMathSharp types and Unity-facing attributes. |
@@ -156,10 +158,11 @@ library's deterministic design.
   path is written back.
 - Keep editor-only code under `Editor/` and runtime package code under
   `Runtime/`. Editor asmdefs must include only the `Editor` platform.
-- Do not make the Lean package depend on MemoryPack or MemoryPack-specific
-  runtime APIs.
+- Do not make the Lean package depend on the MemoryPack serializer/runtime or
+  MemoryPack-specific runtime APIs. Its attribute-only Chronicler shim is the
+  intentional exception.
 - Keep both package variants behaviorally aligned except for the intentional
-  MemoryPack dependency difference.
+  serializer/runtime integration difference.
 
 ## Sync And Verification Commands
 
@@ -224,7 +227,8 @@ When bumping the FixedMathSharp core library:
   appropriate.
 - Keep standard-package dependency DLLs aligned with the MemoryPack-enabled
   build.
-- Keep the Lean package free of MemoryPack dependencies.
+- Keep the Lean package free of the MemoryPack serializer/runtime and keep its
+  `Chronicler.MemoryPackShim` plugin aligned with the Lean core build.
 - Update both `package.json` files, package READMEs, and the root README when
   the public version, install guidance, dependencies, or variant behavior
   changes.
@@ -234,7 +238,7 @@ When bumping the FixedMathSharp core library:
 
 - Keep the root README focused on package selection and Git URL installation.
 - Keep package READMEs package-specific: standard explains MemoryPack support,
-  Lean explains the no-MemoryPack/Burst AOT rationale.
+  Lean explains the no-MemoryPack-runtime/Burst AOT rationale and attribute shim.
 - Do not duplicate long core FixedMathSharp API documentation here. Link to or
   align with the core repo instead.
 - If a Unity package behavior depends on a core FixedMathSharp convention,

.assets/ci/unity-sample-import-smoke/Assets/Tests/EditMode/FixedMathSharpSampleImportSmokeTests.cs

@@ -2,14 +2,16 @@
 using System.IO;
 using NUnit.Framework;
 using UnityEditor;
+using UnityEditor.SceneManagement;
+using UnityEngine.SceneManagement;
 
 public sealed class FixedMathSharpSampleImportSmokeTests
 {
     private const string ConfigAssetPath = "FixedMathSharpSampleImportSmokeConfig.json";
     private const string ImportedPathKey = "FixedMathSharp.SampleImportSmoke.ImportedPath";
 
     [Test]
-    public void DemoSceneSampleWasImported()
+    public void DemoSceneSampleWasImportedAndOpens()
     {
         var config = LoadConfig();
         Assert.That(config.expectedSampleAsmdef, Is.Not.Empty, $"Config must define expectedSampleAsmdef: {ConfigAssetPath}");
@@ -19,6 +21,38 @@ public void DemoSceneSampleWasImported()
 
         var importedAsmdefPath = Path.Combine(importedPath, config.expectedSampleAsmdef);
         Assert.That(File.Exists(importedAsmdefPath), Is.True, $"Imported sample asmdef was not found: {importedAsmdefPath}");
+
+        var importedScenes = Directory.GetFiles(importedPath, "*.unity", SearchOption.AllDirectories);
+        Assert.That(importedScenes, Has.Length.EqualTo(1), $"Expected one imported sample scene under: {importedPath}");
+
+        var importedScenePath = importedScenes[0];
+        AssertSerializedFieldLine(importedScenePath, "worldRotationMatrix");
+        AssertSerializedFieldLine(importedScenePath, "localFixedAsUnityMatrix");
+        AssertSerializedFieldLine(importedScenePath, "worldFixedAsUnityMatrix");
+
+        var assetsPath = Path.GetFullPath(UnityEngine.Application.dataPath);
+        var fullScenePath = Path.GetFullPath(importedScenePath);
+        Assert.That(
+            fullScenePath.StartsWith(assetsPath + Path.DirectorySeparatorChar, StringComparison.OrdinalIgnoreCase),
+            Is.True,
+            $"Imported scene is outside the Unity project: {importedScenePath}");
+        var sceneAssetPath = "Assets/" + fullScenePath[(assetsPath.Length + 1)..].Replace('\\', '/');
+
+        Scene scene = EditorSceneManager.OpenScene(sceneAssetPath, OpenSceneMode.Single);
+        Assert.That(scene.IsValid(), Is.True, $"Imported sample scene did not open: {sceneAssetPath}");
+        Assert.That(scene.rootCount, Is.GreaterThan(0), $"Imported sample scene has no roots: {sceneAssetPath}");
+    }
+
+    private static void AssertSerializedFieldLine(string scenePath, string fieldName)
+    {
+        var prefix = $"  {fieldName}:";
+        foreach (var line in File.ReadLines(scenePath))
+        {
+            if (line.StartsWith(prefix, StringComparison.Ordinal))
+                return;
+        }
+
+        Assert.Fail($"Imported sample scene is missing serialized field '{fieldName}': {scenePath}");
     }
 
     private static SmokeConfig LoadConfig()

.docs/Fixed64_Serialization.md

@@ -2,13 +2,13 @@
 
 ## Summary
 
-`Fixed64` has a Unity-compatible serialized shape. Unity can persist non-zero `Fixed64`, `Vector2d`, and `Vector3d` authoring values through ScriptableObject assets and prefabs, including nested structs and list elements.
+`Fixed64` has a Unity-compatible serialized shape. Unity can persist non-zero `Fixed64`, `Vector2d`, `Vector3d`, and `Vector4d` authoring values through ScriptableObject assets and prefabs, including nested structs and list elements.
 
 Unity's field serialization rules state: Unity serializes fields, but it does not persist `readonly` fields. Hence, `Fixed64` cannot persist its payload as `public readonly long m_rawValue` since Unity had no durable payload to write. Inspector edits could affect the live object, then reload as zero from asset or prefab YAML.
 
 The fix was to keep FixedMathSharp engine-agnostic by using a normal public `long` field instead of Unity attributes or Unity-specific wrapper types.
 
-Unity 6000.3 serialization rules: <https://docs.unity3d.com/6000.3/Documentation/Manual/script-serialization-rules.html>
+Unity 6 serialization rules: <https://docs.unity3d.com/6000.0/Documentation/Manual/script-serialization-rules.html>
 
 ## Intentional Public Field
 
@@ -23,6 +23,7 @@ The EditMode persistence tests cover:
 - `[SerializeField] Fixed64`
 - `[SerializeField] Vector2d`
 - `[SerializeField] Vector3d`
+- `[SerializeField] Vector4d`
 - a nested serializable struct containing `Fixed64`
 - a `List<T>` element containing `Fixed64` and `Vector3d`
 - ScriptableObject save/reload through `AssetDatabase`
@@ -37,7 +38,7 @@ This command produced TestRunner XML reliably without `-quit`:
 
 ```powershell
 $resultPath = 'F:\gamedevrepos\FixedMathSharp-Unity\TestResults\fixed-serialization-editmode.xml'
-$unity = 'C:\Program Files\Unity\Hub\Editor\6000.3.9f1\Editor\Unity.exe'
+$unity = 'C:\Program Files\Unity\Hub\Editor\6000.5.0f1\Editor\Unity.exe'
 $args = @(
     '-batchmode',
     '-projectPath', 'F:\gamedevrepos\FixedMathSharp-Unity',

.github/workflows/build-and-test.yml

@@ -62,7 +62,7 @@ jobs:
         uses: actions/cache@v5
         with:
           path: .ci/unity-project/Library
-          key: unity-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-${{ hashFiles('.assets/ci/scripts/prepare-unity-editmode-test-project.sh', '.assets/ci/unity-editmode-tests/**', 'Tests/EditMode/**/*.cs', 'Tests/Runtime/**/*.cs', 'com.mrdav30.fixedmathsharp/**/*.cs', 'com.mrdav30.fixedmathsharp/**/*.asmdef', 'com.mrdav30.fixedmathsharp/package.json', 'com.mrdav30.fixedmathsharp.lean/**/*.cs', 'com.mrdav30.fixedmathsharp.lean/**/*.asmdef', 'com.mrdav30.fixedmathsharp.lean/package.json') }}
+          key: unity-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-${{ hashFiles('.assets/ci/scripts/prepare-unity-editmode-test-project.sh', '.assets/ci/unity-editmode-tests/**', 'Tests/EditMode/**/*.cs', 'Tests/Runtime/**/*.cs', 'com.mrdav30.fixedmathsharp/**/*.cs', 'com.mrdav30.fixedmathsharp/**/*.asmdef', 'com.mrdav30.fixedmathsharp/Plugins/**', 'com.mrdav30.fixedmathsharp/package.json', 'com.mrdav30.fixedmathsharp.lean/**/*.cs', 'com.mrdav30.fixedmathsharp.lean/**/*.asmdef', 'com.mrdav30.fixedmathsharp.lean/Plugins/**', 'com.mrdav30.fixedmathsharp.lean/package.json') }}
           restore-keys: |
             unity-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-
 
@@ -134,7 +134,7 @@ jobs:
         uses: actions/cache@v5
         with:
           path: .ci/unity-sample-import-${{ matrix.artifact_suffix }}/Library
-          key: unity-sample-import-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-${{ hashFiles('.assets/ci/scripts/prepare-unity-sample-import-smoke.sh', '.assets/ci/unity-sample-import-smoke/**', 'com.mrdav30.fixedmathsharp/package.json', 'com.mrdav30.fixedmathsharp/Samples~/**', 'com.mrdav30.fixedmathsharp.lean/package.json', 'com.mrdav30.fixedmathsharp.lean/Samples~/**') }}
+          key: unity-sample-import-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-${{ hashFiles('.assets/ci/scripts/prepare-unity-sample-import-smoke.sh', '.assets/ci/unity-sample-import-smoke/**', 'com.mrdav30.fixedmathsharp/Plugins/**', 'com.mrdav30.fixedmathsharp/package.json', 'com.mrdav30.fixedmathsharp/Samples~/**', 'com.mrdav30.fixedmathsharp.lean/Plugins/**', 'com.mrdav30.fixedmathsharp.lean/package.json', 'com.mrdav30.fixedmathsharp.lean/Samples~/**') }}
           restore-keys: |
             unity-sample-import-library-${{ runner.os }}-${{ matrix.artifact_suffix }}-
 

Build/Base/Editor/Drawers/VectorDrawer.cs

@@ -18,11 +18,13 @@ namespace FixedMathSharp.Editor
     /// <see cref="PropertyDrawer" />
     [CustomPropertyDrawer(typeof(Vector2d))]
     [CustomPropertyDrawer(typeof(Vector3d))]
+    [CustomPropertyDrawer(typeof(Vector4d))]
     public class VectorDrawer : PropertyDrawer
     {
         private static readonly GUIContent XLabel = new("X");
         private static readonly GUIContent YLabel = new("Y");
         private static readonly GUIContent ZLabel = new("Z");
+        private static readonly GUIContent WLabel = new("W");
 
         /// <summary>
         /// Called when the GUI is drawn.
@@ -66,8 +68,13 @@ private int GetFieldCount(SerializedProperty property)
             Type type = fieldInfo?.FieldType;
             if (type == typeof(Vector2d))
                 return 2;
+            if (type == typeof(Vector4d))
+                return 4;
 
-            return property.type == nameof(Vector2d) ? 2 : 3;
+            if (property.type == nameof(Vector2d))
+                return 2;
+
+            return property.type == nameof(Vector4d) ? 4 : 3;
         }
 
         private static void DrawFixedVector(Rect contentPosition, SerializedProperty property, int fieldCount, float fieldWidth, bool hideLabels)
@@ -95,6 +102,20 @@ private static void DrawFixedVector(Rect contentPosition, SerializedProperty pro
                 if (EditorGUI.EndChangeCheck())
                     property.SetFixedPropertyValue(new Vector3d(x, y, z));
             }
+            else if (fieldCount == 4 && propertyValue is Vector4d vector4d)
+            {
+                EditorGUI.BeginChangeCheck();
+                Fixed64 x = DrawFixedComponent(contentPosition, XLabel, vector4d.X, hideLabels);
+                contentPosition.x += fieldWidth;
+                Fixed64 y = DrawFixedComponent(contentPosition, YLabel, vector4d.Y, hideLabels);
+                contentPosition.x += fieldWidth;
+                Fixed64 z = DrawFixedComponent(contentPosition, ZLabel, vector4d.Z, hideLabels);
+                contentPosition.x += fieldWidth;
+                Fixed64 w = DrawFixedComponent(contentPosition, WLabel, vector4d.W, hideLabels);
+
+                if (EditorGUI.EndChangeCheck())
+                    property.SetFixedPropertyValue(new Vector4d(x, y, z, w));
+            }
         }
 
         private static Fixed64 DrawFixedComponent(Rect position, GUIContent label, Fixed64 value, bool hideLabels)