set

Author: fenndragon

Content.Shared/StatusEffect/StatusEffectsComponent.cs

@@ -15,7 +15,7 @@ public sealed partial class StatusEffectsComponent : Component
         ///     A list of status effect IDs to be allowed
         /// </summary>
         [DataField("allowed", required: true), Access(typeof(StatusEffectsSystem), Other = AccessPermissions.ReadExecute)]
-        public List<string> AllowedEffects = default!;
+        public List<string> AllowedEffects = new();
     }
 
     [RegisterComponent]

Content.Shared/StatusEffect/StatusEffectsSystem.cs

@@ -61,6 +61,8 @@ private void OnGetState(EntityUid uid, StatusEffectsComponent component, ref Com
         {
             // Using new(...) To avoid mispredictions due to MergeImplicitData. This will mean the server-side code is
             // slightly slower, and really this function should just be overridden by the client...
+            component.ActiveEffects ??= new();
+            component.AllowedEffects ??= new();
             args.State = new StatusEffectsComponentState(new(component.ActiveEffects), new(component.AllowedEffects));
         }
 
@@ -69,6 +71,11 @@ private void OnHandleState(EntityUid uid, StatusEffectsComponent component, ref
             if (args.Current is not StatusEffectsComponentState state)
                 return;
 
+            component.ActiveEffects ??= new();
+            component.AllowedEffects ??= new();
+            state.ActiveEffects ??= new();
+            state.AllowedEffects ??= new();
+
             component.AllowedEffects.Clear();
             component.AllowedEffects.AddRange(state.AllowedEffects);