Author: rodrigo-fournier-immutable

sample/Assets/Editor/MacBuilderUnity6.cs

@@ -65,11 +65,13 @@ private static void BuildPlayer(string defaultBuildPath, bool setupForAltTester
             // PlayerSettings, AltTester runtime scripts may be excluded from the build
             // (leading to missing-script errors and no AltTester server listening).
             // For AltTester builds, temporarily mirror the current Standalone define
-            // symbols onto the BuildProfile's scripting defines.
+            // symbols onto the BuildProfile.
             string[] originalProfileDefines = Array.Empty<string>();
+            string[] originalProfilePlayerSettingsLines = Array.Empty<string>();
             if (setupForAltTester)
             {
                 originalProfileDefines = GetBuildProfileDefines(buildProfile);
+                originalProfilePlayerSettingsLines = GetBuildProfilePlayerSettingsLines(buildProfile);
 
                 // Capture the current Standalone define symbols after SetupAltTester()
                 // has added AltTester + IMMUTABLE_E2E_TESTING.
@@ -83,6 +85,13 @@ private static void BuildPlayer(string defaultBuildPath, bool setupForAltTester
 
                 SetBuildProfileDefines(buildProfile, MergeDefines(originalProfileDefines, desired));
                 Debug.Log($"Applied {desired.Length} define symbols to BuildProfile for AltTester build.");
+
+                // IMPORTANT: macOS Profile stores Standalone define symbols inside m_PlayerSettingsYaml.
+                // If we don't patch that block, the build may still run with only the profile's defines
+                // (e.g. VUPLEX_STANDALONE), resulting in missing AltTester runtime scripts.
+                var patchedLines = WithStandaloneDefineSymbols(originalProfilePlayerSettingsLines, desired);
+                SetBuildProfilePlayerSettingsLines(buildProfile, patchedLines);
+                Debug.Log("Patched BuildProfile PlayerSettings define symbols for AltTester build.");
             }
 
             BuildPlayerWithProfileOptions options = new BuildPlayerWithProfileOptions
@@ -111,6 +120,7 @@ private static void BuildPlayer(string defaultBuildPath, bool setupForAltTester
             {
                 // Restore original profile defines to avoid dirtying the repo locally.
                 SetBuildProfileDefines(buildProfile, originalProfileDefines);
+                SetBuildProfilePlayerSettingsLines(buildProfile, originalProfilePlayerSettingsLines);
             }
 
             if (setupForAltTester)
@@ -218,6 +228,133 @@ private static string[] GetBuildProfileDefines(BuildProfile buildProfile)
         return list.ToArray();
     }
 
+    private static string[] GetBuildProfilePlayerSettingsLines(BuildProfile buildProfile)
+    {
+        var so = new SerializedObject(buildProfile);
+        var prop = so.FindProperty("m_PlayerSettingsYaml.m_Settings");
+        if (prop == null || !prop.isArray) return Array.Empty<string>();
+        var list = new List<string>(prop.arraySize);
+        for (int i = 0; i < prop.arraySize; i++)
+        {
+            var el = prop.GetArrayElementAtIndex(i);
+            var lineProp = el?.FindPropertyRelative("line");
+            if (lineProp != null)
+            {
+                list.Add(lineProp.stringValue);
+            }
+        }
+        return list.ToArray();
+    }
+
+    private static void SetBuildProfilePlayerSettingsLines(BuildProfile buildProfile, IEnumerable<string> lines)
+    {
+        var so = new SerializedObject(buildProfile);
+        var prop = so.FindProperty("m_PlayerSettingsYaml.m_Settings");
+        if (prop == null || !prop.isArray)
+        {
+            Debug.LogWarning("BuildProfile does not expose m_PlayerSettingsYaml.m_Settings; cannot patch player settings YAML.");
+            return;
+        }
+
+        prop.ClearArray();
+        int idx = 0;
+        if (lines != null)
+        {
+            foreach (var l in lines)
+            {
+                prop.InsertArrayElementAtIndex(idx);
+                var el = prop.GetArrayElementAtIndex(idx);
+                var lineProp = el.FindPropertyRelative("line");
+                if (lineProp != null)
+                {
+                    lineProp.stringValue = l ?? string.Empty;
+                }
+                idx++;
+            }
+        }
+
+        so.ApplyModifiedPropertiesWithoutUndo();
+        EditorUtility.SetDirty(buildProfile);
+        AssetDatabase.SaveAssets();
+    }
+
+    private static string[] WithStandaloneDefineSymbols(string[] originalLines, IEnumerable<string> desiredDefines)
+    {
+        // We patch the YAML lines:
+        // |   scriptingDefineSymbols:
+        // |     Standalone: <defines>
+        //
+        // Preserve existing Standalone defines (e.g. VUPLEX_STANDALONE) and append our desired ones.
+        var lines = (originalLines ?? Array.Empty<string>()).ToList();
+        if (lines.Count == 0)
+        {
+            // Unexpected, but still handle it.
+            lines.Add("| PlayerSettings:");
+        }
+
+        int symbolsIdx = lines.FindIndex(l => l != null && l.TrimEnd() == "|   scriptingDefineSymbols:");
+        if (symbolsIdx < 0)
+        {
+            // Append a minimal block near the end.
+            lines.Add("|   scriptingDefineSymbols:");
+            lines.Add("|     Standalone: " + string.Join(";", desiredDefines ?? Array.Empty<string>()));
+            return lines.ToArray();
+        }
+
+        int standaloneIdx = -1;
+        for (int i = symbolsIdx + 1; i < Math.Min(symbolsIdx + 10, lines.Count); i++)
+        {
+            var l = lines[i] ?? "";
+            if (l.TrimStart().StartsWith("|     Standalone:"))
+            {
+                standaloneIdx = i;
+                break;
+            }
+            // Stop if we hit another top-level key
+            if (l.StartsWith("|   ") && !l.StartsWith("|     "))
+            {
+                break;
+            }
+        }
+
+        string existing = "";
+        if (standaloneIdx >= 0)
+        {
+            var l = lines[standaloneIdx] ?? "";
+            existing = l.Substring(l.IndexOf(":") + 1).Trim();
+        }
+
+        var merged = new HashSet<string>(StringComparer.Ordinal);
+        foreach (var d in (existing ?? "").Split(new[] { ';' }, StringSplitOptions.RemoveEmptyEntries))
+        {
+            merged.Add(d.Trim());
+        }
+        if (desiredDefines != null)
+        {
+            foreach (var d in desiredDefines)
+            {
+                if (!string.IsNullOrEmpty(d))
+                {
+                    merged.Add(d.Trim());
+                }
+            }
+        }
+
+        string mergedStr = string.Join(";", merged.Where(s => !string.IsNullOrEmpty(s)));
+        string newLine = "|     Standalone: " + mergedStr;
+
+        if (standaloneIdx >= 0)
+        {
+            lines[standaloneIdx] = newLine;
+        }
+        else
+        {
+            lines.Insert(symbolsIdx + 1, newLine);
+        }
+
+        return lines.ToArray();
+    }
+
     private static string[] MergeDefines(IEnumerable<string> a, IEnumerable<string> b)
     {
         var set = new HashSet<string>(StringComparer.Ordinal);