]> git.smokeofanarchy.ru Git - space-station-14.git/commitdiff
Move faction exception and everything it needs to shared (#25154)
authordeltanedas <39013340+deltanedas@users.noreply.github.com>
Mon, 18 Mar 2024 07:23:25 +0000 (07:23 +0000)
committerGitHub <noreply@github.com>
Mon, 18 Mar 2024 07:23:25 +0000 (18:23 +1100)
* move faction prototype to shared

* move faction exception and member stuff to shared

* fix breaking changes for random stuff

* move pettable friend stuff to shared

* mostly fix

* final fixy

* dragonops

* final fixy II

* use querys and fix warpspeed fish (probably)

* fixer

* Rrrr!

---------

Co-authored-by: deltanedas <@deltanedas:kde.org>
Co-authored-by: metalgearsloth <comedian_vs_clown@hotmail.com>
29 files changed:
Content.Client/Entry/EntryPoint.cs
Content.Server/Dragon/Components/DragonComponent.cs
Content.Server/Dragon/DragonSystem.cs
Content.Server/Friends/Components/PettableFriendComponent.cs [deleted file]
Content.Server/Friends/Systems/PettableFriendSystem.cs [deleted file]
Content.Server/GameTicking/Rules/Components/NukeopsRuleComponent.cs
Content.Server/GameTicking/Rules/Components/TraitorRuleComponent.cs
Content.Server/GameTicking/Rules/NukeopsRuleSystem.cs
Content.Server/GameTicking/Rules/PiratesRuleSystem.cs
Content.Server/GameTicking/Rules/RevolutionaryRuleSystem.cs
Content.Server/GameTicking/Rules/TraitorRuleSystem.cs
Content.Server/NPC/Components/FactionExceptionTrackerComponent.cs [deleted file]
Content.Server/NPC/Components/NpcFactionMemberComponent.cs [deleted file]
Content.Server/NPC/Components/NpcFactionPrototype.cs [deleted file]
Content.Server/NPC/FactionData.cs [deleted file]
Content.Server/NPC/Systems/NPCRetaliationSystem.cs
Content.Server/NPC/Systems/NPCSteeringSystem.Context.cs
Content.Server/NPC/Systems/NPCSteeringSystem.cs
Content.Server/NPC/Systems/NPCUtilitySystem.cs
Content.Server/NPC/Systems/NpcFactionSystem.Exception.cs [deleted file]
Content.Server/Zombies/ZombieSystem.Transform.cs
Content.Shared/Friends/Components/PettableFriendComponent.cs [new file with mode: 0644]
Content.Shared/Friends/Systems/PettableFriendSystem.cs [new file with mode: 0644]
Content.Shared/NPC/Components/FactionExceptionComponent.cs [moved from Content.Server/NPC/Components/FactionExceptionComponent.cs with 72% similarity]
Content.Shared/NPC/Components/FactionExceptionTrackerComponent.cs [new file with mode: 0644]
Content.Shared/NPC/Components/NpcFactionMemberComponent.cs [new file with mode: 0644]
Content.Shared/NPC/Prototypes/NpcFactionPrototype.cs [new file with mode: 0644]
Content.Shared/NPC/Systems/NpcFactionSystem.Exception.cs [new file with mode: 0644]
Content.Shared/NPC/Systems/NpcFactionSystem.cs [moved from Content.Server/NPC/Systems/NpcFactionSystem.cs with 50% similarity]

index 3acc4ab18002e4f2a737bfed9299299d5cc63094..6e9d8c4021e94a2c5f55bdc4a2d4ee8dfaf6e506 100644 (file)
@@ -100,12 +100,10 @@ namespace Content.Client.Entry
             _prototypeManager.RegisterIgnore("seed"); // Seeds prototypes are server-only.
             _prototypeManager.RegisterIgnore("objective");
             _prototypeManager.RegisterIgnore("holiday");
-            _prototypeManager.RegisterIgnore("aiFaction");
             _prototypeManager.RegisterIgnore("htnCompound");
             _prototypeManager.RegisterIgnore("htnPrimitive");
             _prototypeManager.RegisterIgnore("gameMap");
             _prototypeManager.RegisterIgnore("gameMapPool");
-            _prototypeManager.RegisterIgnore("npcFaction");
             _prototypeManager.RegisterIgnore("lobbyBackground");
             _prototypeManager.RegisterIgnore("advertisementsPack");
             _prototypeManager.RegisterIgnore("gamePreset");
index f403979a0070d6f3c7571aa0359a935eebab4fc1..80461e156af172a2068e0865c45bd3ddd8456bbc 100644 (file)
@@ -1,3 +1,4 @@
+using Content.Shared.NPC.Prototypes;
 using Robust.Shared.Audio;
 using Robust.Shared.Prototypes;
 using Robust.Shared.Serialization.TypeSerializers.Implementations.Custom.Prototype;
@@ -57,5 +58,12 @@ namespace Content.Server.Dragon
             {
                 Params = AudioParams.Default.WithVolume(3f),
             };
+
+        /// <summary>
+        /// NPC faction to re-add after being zombified.
+        /// Prevents zombie dragon from being attacked by its own carp.
+        /// </summary>
+        [DataField]
+        public ProtoId<NpcFactionPrototype> Faction = "Dragon";
     }
 }
index 93d6bc8db0b3a97c376f4837129b2e7c0d5a7e07..36df6fb6f29c7636feb66eef0849827cc7c581e5 100644 (file)
@@ -10,6 +10,8 @@ using Content.Shared.Mind;
 using Content.Shared.Mind.Components;
 using Content.Shared.Mobs;
 using Content.Shared.Movement.Systems;
+using Content.Shared.NPC.Systems;
+using Content.Shared.Zombies;
 using Robust.Shared.Audio;
 using Robust.Shared.Audio.Systems;
 using Robust.Shared.GameStates;
@@ -24,6 +26,7 @@ public sealed partial class DragonSystem : EntitySystem
     [Dependency] private readonly IMapManager _mapManager = default!;
     [Dependency] private readonly ITileDefinitionManager _tileDef = default!;
     [Dependency] private readonly MovementSpeedModifierSystem _movement = default!;
+    [Dependency] private readonly NpcFactionSystem _faction = default!;
     [Dependency] private readonly PopupSystem _popup = default!;
     [Dependency] private readonly RoleSystem _role = default!;
     [Dependency] private readonly SharedActionsSystem _actions = default!;
@@ -55,6 +58,7 @@ public sealed partial class DragonSystem : EntitySystem
         SubscribeLocalEvent<DragonComponent, RefreshMovementSpeedModifiersEvent>(OnDragonMove);
         SubscribeLocalEvent<DragonComponent, MobStateChangedEvent>(OnMobStateChanged);
         SubscribeLocalEvent<DragonComponent, GenericAntagCreatedEvent>(OnCreated);
+        SubscribeLocalEvent<DragonComponent, EntityZombifiedEvent>(OnZombified);
     }
 
     public override void Update(float frameTime)
@@ -202,6 +206,12 @@ public sealed partial class DragonSystem : EntitySystem
         }, mind);
     }
 
+    private void OnZombified(Entity<DragonComponent> ent, ref EntityZombifiedEvent args)
+    {
+        // prevent carp attacking zombie dragon
+        _faction.AddFaction(ent.Owner, ent.Comp.Faction);
+    }
+
     private void Roar(EntityUid uid, DragonComponent comp)
     {
         if (comp.SoundRoar != null)
diff --git a/Content.Server/Friends/Components/PettableFriendComponent.cs b/Content.Server/Friends/Components/PettableFriendComponent.cs
deleted file mode 100644 (file)
index fe68029..0000000
+++ /dev/null
@@ -1,23 +0,0 @@
-using Content.Server.Friends.Systems;
-
-namespace Content.Server.Friends.Components;
-
-/// <summary>
-/// Pet something to become friends with it (use in hand, press Z)
-/// Uses FactionExceptionComponent behind the scenes
-/// </summary>
-[RegisterComponent, Access(typeof(PettableFriendSystem))]
-public sealed partial class PettableFriendComponent : Component
-{
-    /// <summary>
-    /// Localized popup sent when petting for the first time
-    /// </summary>
-    [DataField("successString", required: true), ViewVariables(VVAccess.ReadWrite)]
-    public string SuccessString = string.Empty;
-
-    /// <summary>
-    /// Localized popup sent when petting multiple times
-    /// </summary>
-    [DataField("failureString", required: true), ViewVariables(VVAccess.ReadWrite)]
-    public string FailureString = string.Empty;
-}
diff --git a/Content.Server/Friends/Systems/PettableFriendSystem.cs b/Content.Server/Friends/Systems/PettableFriendSystem.cs
deleted file mode 100644 (file)
index 8f70c84..0000000
+++ /dev/null
@@ -1,50 +0,0 @@
-using Content.Server.Friends.Components;
-using Content.Server.NPC.Components;
-using Content.Server.NPC.Systems;
-using Content.Shared.Chemistry.Components;
-using Content.Shared.Interaction.Events;
-using Content.Shared.Popups;
-
-namespace Content.Server.Friends.Systems;
-
-public sealed class PettableFriendSystem : EntitySystem
-{
-    [Dependency] private readonly NpcFactionSystem _factionException = default!;
-    [Dependency] private readonly SharedPopupSystem _popup = default!;
-
-    public override void Initialize()
-    {
-        base.Initialize();
-
-        SubscribeLocalEvent<PettableFriendComponent, UseInHandEvent>(OnUseInHand);
-        SubscribeLocalEvent<PettableFriendComponent, GotRehydratedEvent>(OnRehydrated);
-    }
-
-    private void OnUseInHand(EntityUid uid, PettableFriendComponent comp, UseInHandEvent args)
-    {
-        var user = args.User;
-        if (args.Handled || !TryComp<FactionExceptionComponent>(uid, out var factionException))
-            return;
-
-        if (_factionException.IsIgnored(uid, user, factionException))
-        {
-            _popup.PopupEntity(Loc.GetString(comp.FailureString, ("target", uid)), user, user);
-            return;
-        }
-
-        // you have made a new friend :)
-        _popup.PopupEntity(Loc.GetString(comp.SuccessString, ("target", uid)), user, user);
-        _factionException.IgnoreEntity(uid, user, factionException);
-        args.Handled = true;
-    }
-
-    private void OnRehydrated(EntityUid uid, PettableFriendComponent _, ref GotRehydratedEvent args)
-    {
-        // can only pet before hydrating, after that the fish cannot be negotiated with
-        if (!TryComp<FactionExceptionComponent>(uid, out var comp))
-            return;
-
-        var targetComp = AddComp<FactionExceptionComponent>(args.Target);
-        _factionException.IgnoreEntities(args.Target, comp.Ignored, targetComp);
-    }
-}
index 8efd61b4694296df56670b4dfdf897685ca52781..c66a9d12a1c5959c2467ae29d8a7f3acad37c281 100644 (file)
@@ -1,8 +1,8 @@
 using Content.Server.Maps;
-using Content.Server.NPC.Components;
 using Content.Server.RoundEnd;
 using Content.Server.StationEvents.Events;
 using Content.Shared.Dataset;
+using Content.Shared.NPC.Prototypes;
 using Content.Shared.Roles;
 using Robust.Shared.Map;
 using Robust.Shared.Prototypes;
index 62619db76a2848bf540889f3f9a733c0f9c0f3e1..e904d8a7c2abb3d120acfe399ab6e2f28cb46fe9 100644 (file)
@@ -1,5 +1,5 @@
-using Content.Server.NPC.Components;
 using Content.Shared.Dataset;
+using Content.Shared.NPC.Prototypes;
 using Content.Shared.Random;
 using Content.Shared.Roles;
 using Robust.Shared.Audio;
index b9255bcbe4fcd9f5eafa4c425ca519ee1882eac9..ea5ab9c2abe6cd6fc573b650f72f7742d1814512 100644 (file)
@@ -7,8 +7,6 @@ using Content.Server.Ghost.Roles.Components;
 using Content.Server.Ghost.Roles.Events;
 using Content.Server.Humanoid;
 using Content.Server.Mind;
-using Content.Server.NPC.Components;
-using Content.Server.NPC.Systems;
 using Content.Server.Nuke;
 using Content.Server.NukeOps;
 using Content.Server.Popups;
@@ -30,6 +28,8 @@ using Content.Shared.Humanoid.Prototypes;
 using Content.Shared.Mind.Components;
 using Content.Shared.Mobs;
 using Content.Shared.Mobs.Components;
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Nuke;
 using Content.Shared.NukeOps;
 using Content.Shared.Preferences;
@@ -474,7 +474,7 @@ public sealed class NukeopsRuleSystem : GameRuleSystem<NukeopsRuleComponent>
         var eligibleQuery = EntityQueryEnumerator<StationEventEligibleComponent, NpcFactionMemberComponent>();
         while (eligibleQuery.MoveNext(out var eligibleUid, out var eligibleComp, out var member))
         {
-            if (!_npcFaction.IsFactionHostile(component.Faction, eligibleUid, member))
+            if (!_npcFaction.IsFactionHostile(component.Faction, (eligibleUid, member)))
                 continue;
 
             eligible.Add((eligibleUid, eligibleComp, member));
index 98926536b9d641480a173c6dc6a696cb97126bfc..900554d82410e1a4f10b0fd54d4723fe4fad1eeb 100644 (file)
@@ -4,8 +4,6 @@ using Content.Server.Administration.Commands;
 using Content.Server.Cargo.Systems;
 using Content.Server.Chat.Managers;
 using Content.Server.GameTicking.Rules.Components;
-using Content.Server.NPC.Components;
-using Content.Server.NPC.Systems;
 using Content.Server.Preferences.Managers;
 using Content.Server.Spawners.Components;
 using Content.Server.Station.Components;
@@ -14,6 +12,8 @@ using Content.Shared.CCVar;
 using Content.Shared.Humanoid;
 using Content.Shared.Humanoid.Prototypes;
 using Content.Shared.Mind;
+using Content.Shared.NPC.Prototypes;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Preferences;
 using Content.Shared.Roles;
 using Robust.Server.GameObjects;
index d20775c73437881c8650d9e12a717836b14f5eb8..5caa223c9c1b9ae86852c9738425b2d13b0fe3e0 100644 (file)
@@ -4,8 +4,6 @@ using Content.Server.EUI;
 using Content.Server.Flash;
 using Content.Server.GameTicking.Rules.Components;
 using Content.Server.Mind;
-using Content.Server.NPC.Components;
-using Content.Server.NPC.Systems;
 using Content.Server.Popups;
 using Content.Server.Revolutionary;
 using Content.Server.Revolutionary.Components;
@@ -23,6 +21,8 @@ using Content.Shared.Mindshield.Components;
 using Content.Shared.Mobs;
 using Content.Shared.Mobs.Components;
 using Content.Shared.Mobs.Systems;
+using Content.Shared.NPC.Prototypes;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Revolutionary.Components;
 using Content.Shared.Roles;
 using Content.Shared.Stunnable;
index fc9f0a9a9ff13aa58f94608e2d025e5dac47fd63..769d7e0a5b53130cc412985c26998a46a6b4f7ab 100644 (file)
@@ -1,7 +1,6 @@
 using Content.Server.Antag;
 using Content.Server.GameTicking.Rules.Components;
 using Content.Server.Mind;
-using Content.Server.NPC.Systems;
 using Content.Server.Objectives;
 using Content.Server.PDA.Ringer;
 using Content.Server.Roles;
@@ -10,6 +9,7 @@ using Content.Shared.CCVar;
 using Content.Shared.Dataset;
 using Content.Shared.Mind;
 using Content.Shared.Mobs.Systems;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Objectives.Components;
 using Content.Shared.PDA;
 using Content.Shared.Roles;
diff --git a/Content.Server/NPC/Components/FactionExceptionTrackerComponent.cs b/Content.Server/NPC/Components/FactionExceptionTrackerComponent.cs
deleted file mode 100644 (file)
index 804a61b..0000000
+++ /dev/null
@@ -1,16 +0,0 @@
-using Content.Server.NPC.Systems;
-
-namespace Content.Server.NPC.Components;
-
-/// <summary>
-/// This is used for tracking entities stored in <see cref="FactionExceptionComponent"/>
-/// </summary>
-[RegisterComponent, Access(typeof(NpcFactionSystem))]
-public sealed partial class FactionExceptionTrackerComponent : Component
-{
-    /// <summary>
-    /// entities with <see cref="FactionExceptionComponent"/> that are tracking this entity.
-    /// </summary>
-    [DataField("entities")]
-    public HashSet<EntityUid> Entities = new();
-}
diff --git a/Content.Server/NPC/Components/NpcFactionMemberComponent.cs b/Content.Server/NPC/Components/NpcFactionMemberComponent.cs
deleted file mode 100644 (file)
index 72df5d0..0000000
+++ /dev/null
@@ -1,29 +0,0 @@
-using Content.Server.NPC.Systems;
-using Robust.Shared.Serialization.TypeSerializers.Implementations.Custom.Prototype.Set;
-
-namespace Content.Server.NPC.Components
-{
-    [RegisterComponent]
-    [Access(typeof(NpcFactionSystem))]
-    public sealed partial class NpcFactionMemberComponent : Component
-    {
-        /// <summary>
-        /// Factions this entity is a part of.
-        /// </summary>
-        [ViewVariables(VVAccess.ReadWrite),
-         DataField("factions", customTypeSerializer:typeof(PrototypeIdHashSetSerializer<NpcFactionPrototype>))]
-        public HashSet<string> Factions = new();
-
-        /// <summary>
-        /// Cached friendly factions.
-        /// </summary>
-        [ViewVariables]
-        public readonly HashSet<string> FriendlyFactions = new();
-
-        /// <summary>
-        /// Cached hostile factions.
-        /// </summary>
-        [ViewVariables]
-        public readonly HashSet<string> HostileFactions = new();
-    }
-}
diff --git a/Content.Server/NPC/Components/NpcFactionPrototype.cs b/Content.Server/NPC/Components/NpcFactionPrototype.cs
deleted file mode 100644 (file)
index fe57747..0000000
+++ /dev/null
@@ -1,22 +0,0 @@
-using Robust.Shared.Prototypes;
-using Robust.Shared.Serialization.TypeSerializers.Implementations.Custom.Prototype.List;
-
-namespace Content.Server.NPC.Components
-{
-    /// <summary>
-    /// Contains data about this faction's relations with other factions.
-    /// </summary>
-    [Prototype("npcFaction")]
-    public sealed partial class NpcFactionPrototype : IPrototype
-    {
-        [ViewVariables]
-        [IdDataField]
-        public string ID { get; private set; } = default!;
-
-        [ViewVariables(VVAccess.ReadWrite), DataField("friendly", customTypeSerializer:typeof(PrototypeIdListSerializer<NpcFactionPrototype>))]
-        public List<string> Friendly = new();
-
-        [ViewVariables(VVAccess.ReadWrite), DataField("hostile", customTypeSerializer:typeof(PrototypeIdListSerializer<NpcFactionPrototype>))]
-        public List<string> Hostile = new();
-    }
-}
diff --git a/Content.Server/NPC/FactionData.cs b/Content.Server/NPC/FactionData.cs
deleted file mode 100644 (file)
index b74150a..0000000
+++ /dev/null
@@ -1,13 +0,0 @@
-namespace Content.Server.NPC;
-
-/// <summary>
-/// Cached data for the faction prototype. Can be modified at runtime.
-/// </summary>
-public sealed class FactionData
-{
-    [ViewVariables]
-    public HashSet<string> Friendly = new();
-
-    [ViewVariables]
-    public HashSet<string> Hostile = new();
-}
index cde8decefc48d3bf21b89369673e09572096e2ed..d6b2000f329513bd44854d09c71c26ec955e7b8f 100644 (file)
@@ -1,7 +1,9 @@
-using Content.Server.NPC.Components;
+using Content.Server.NPC.Components;
 using Content.Shared.CombatMode;
 using Content.Shared.Damage;
 using Content.Shared.Mobs.Components;
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Systems;
 using Robust.Shared.Collections;
 using Robust.Shared.Timing;
 
@@ -22,37 +24,35 @@ public sealed class NPCRetaliationSystem : EntitySystem
         SubscribeLocalEvent<NPCRetaliationComponent, DisarmedEvent>(OnDisarmed);
     }
 
-    private void OnDamageChanged(EntityUid uid, NPCRetaliationComponent component, DamageChangedEvent args)
+    private void OnDamageChanged(Entity<NPCRetaliationComponent> ent, ref DamageChangedEvent args)
     {
         if (!args.DamageIncreased)
             return;
 
-        if (args.Origin is not { } origin)
+        if (args.Origin is not {} origin)
             return;
 
-        TryRetaliate(uid, origin, component);
+        TryRetaliate(ent, origin);
     }
 
-    private void OnDisarmed(EntityUid uid, NPCRetaliationComponent component, DisarmedEvent args)
+    private void OnDisarmed(Entity<NPCRetaliationComponent> ent, ref DisarmedEvent args)
     {
-        TryRetaliate(uid, args.Source, component);
+        TryRetaliate(ent, args.Source);
     }
 
-    public bool TryRetaliate(EntityUid uid, EntityUid target, NPCRetaliationComponent? component = null)
+    public bool TryRetaliate(Entity<NPCRetaliationComponent> ent, EntityUid target)
     {
-        if (!Resolve(uid, ref component))
-            return false;
-
         // don't retaliate against inanimate objects.
         if (!HasComp<MobStateComponent>(target))
             return false;
 
-        if (_npcFaction.IsEntityFriendly(uid, target))
+        // don't retaliate against the same faction
+        if (_npcFaction.IsEntityFriendly(ent.Owner, target))
             return false;
 
-        _npcFaction.AggroEntity(uid, target);
-        if (component.AttackMemoryLength is { } memoryLength)
-            component.AttackMemories[target] = _timing.CurTime + memoryLength;
+        _npcFaction.AggroEntity(ent.Owner, target);
+        if (ent.Comp.AttackMemoryLength is {} memoryLength)
+            ent.Comp.AttackMemories[target] = _timing.CurTime + memoryLength;
 
         return true;
     }
@@ -64,12 +64,14 @@ public sealed class NPCRetaliationSystem : EntitySystem
         var query = EntityQueryEnumerator<NPCRetaliationComponent, FactionExceptionComponent>();
         while (query.MoveNext(out var uid, out var retaliationComponent, out var factionException))
         {
+            // TODO: can probably reuse this allocation and clear it
             foreach (var entity in new ValueList<EntityUid>(retaliationComponent.AttackMemories.Keys))
             {
                 if (!TerminatingOrDeleted(entity) && _timing.CurTime < retaliationComponent.AttackMemories[entity])
                     continue;
 
-                _npcFaction.DeAggroEntity(uid, entity, factionException);
+                _npcFaction.DeAggroEntity((uid, factionException), entity);
+                // TODO: should probably remove the AttackMemory, thats the whole point of the ValueList right??
             }
         }
     }
index e7af2c91073798daaf8930e6f5cc12a85774db2c..c58dc261fefae2bbea5129b2ae1466178cbf4b40 100644 (file)
@@ -582,7 +582,7 @@ public sealed partial class NPCSteeringSystem
                 (mask & otherBody.CollisionLayer) == 0x0 &&
                 (layer & otherBody.CollisionMask) == 0x0 ||
                 !_factionQuery.TryGetComponent(ent, out var otherFaction) ||
-                !_npcFaction.IsEntityFriendly(uid, ent, ourFaction, otherFaction) ||
+                !_npcFaction.IsEntityFriendly((uid, ourFaction), (ent, otherFaction)) ||
                 // Use <= 0 so we ignore stationary friends in case.
                 Vector2.Dot(otherBody.LinearVelocity, ourVelocity) <= 0f)
             {
index c00375d6488e9d7dbba1a48b634e2ccba6feda12..f04dc56bc4d5debf30692452044f554a6c1f963a 100644 (file)
@@ -13,6 +13,8 @@ using Content.Shared.Interaction;
 using Content.Shared.Movement.Components;
 using Content.Shared.Movement.Systems;
 using Content.Shared.NPC;
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Systems;
 using Content.Shared.NPC.Events;
 using Content.Shared.Physics;
 using Content.Shared.Weapons.Melee;
index 33941be929211a4a917b24c7b40188b85dfd8878..6793161105e323421a2cbdfc84ec7e651a0966ff 100644 (file)
@@ -12,6 +12,7 @@ using Content.Shared.Fluids.Components;
 using Content.Shared.Hands.Components;
 using Content.Shared.Inventory;
 using Content.Shared.Mobs.Systems;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Nutrition.Components;
 using Content.Shared.Tools.Systems;
 using Content.Shared.Weapons.Melee;
diff --git a/Content.Server/NPC/Systems/NpcFactionSystem.Exception.cs b/Content.Server/NPC/Systems/NpcFactionSystem.Exception.cs
deleted file mode 100644 (file)
index acef900..0000000
+++ /dev/null
@@ -1,130 +0,0 @@
-using System.Linq;
-using Content.Server.NPC.Components;
-
-namespace Content.Server.NPC.Systems;
-
-/// <summary>
-/// Prevents an NPC from attacking some entities from an enemy faction.
-/// </summary>
-public sealed partial class NpcFactionSystem
-{
-    private EntityQuery<FactionExceptionComponent> _exceptionQuery;
-    private EntityQuery<FactionExceptionTrackerComponent> _trackerQuery;
-
-    public void InitializeException()
-    {
-        _exceptionQuery = GetEntityQuery<FactionExceptionComponent>();
-        _trackerQuery = GetEntityQuery<FactionExceptionTrackerComponent>();
-
-        SubscribeLocalEvent<FactionExceptionComponent, ComponentShutdown>(OnShutdown);
-        SubscribeLocalEvent<FactionExceptionTrackerComponent, ComponentShutdown>(OnTrackerShutdown);
-    }
-
-    private void OnShutdown(EntityUid uid, FactionExceptionComponent component, ComponentShutdown args)
-    {
-        foreach (var ent in component.Hostiles)
-        {
-            if (!_trackerQuery.TryGetComponent(ent, out var trackerComponent))
-                continue;
-            trackerComponent.Entities.Remove(uid);
-        }
-
-        foreach (var ent in component.Ignored)
-        {
-            if (!_trackerQuery.TryGetComponent(ent, out var trackerComponent))
-                continue;
-            trackerComponent.Entities.Remove(uid);
-        }
-    }
-
-    private void OnTrackerShutdown(EntityUid uid, FactionExceptionTrackerComponent component, ComponentShutdown args)
-    {
-        foreach (var ent in component.Entities)
-        {
-            if (!_exceptionQuery.TryGetComponent(ent, out var exceptionComponent))
-                continue;
-            exceptionComponent.Ignored.Remove(uid);
-            exceptionComponent.Hostiles.Remove(uid);
-        }
-    }
-
-    /// <summary>
-    /// Returns whether the entity from an enemy faction won't be attacked
-    /// </summary>
-    public bool IsIgnored(EntityUid uid, EntityUid target, FactionExceptionComponent? comp = null)
-    {
-        if (!Resolve(uid, ref comp, false))
-            return false;
-
-        return comp.Ignored.Contains(target);
-    }
-
-    /// <summary>
-    /// Returns the specific hostile entities for a given entity.
-    /// </summary>
-    public IEnumerable<EntityUid> GetHostiles(EntityUid uid, FactionExceptionComponent? comp = null)
-    {
-        if (!Resolve(uid, ref comp, false))
-            return Array.Empty<EntityUid>();
-
-        return comp.Hostiles;
-    }
-
-    /// <summary>
-    /// Prevents an entity from an enemy faction from being attacked
-    /// </summary>
-    public void IgnoreEntity(EntityUid uid, EntityUid target, FactionExceptionComponent? comp = null)
-    {
-        comp ??= EnsureComp<FactionExceptionComponent>(uid);
-        comp.Ignored.Add(target);
-        EnsureComp<FactionExceptionTrackerComponent>(target).Entities.Add(uid);
-    }
-
-    /// <summary>
-    /// Prevents a list of entities from an enemy faction from being attacked
-    /// </summary>
-    public void IgnoreEntities(EntityUid uid, IEnumerable<EntityUid> ignored, FactionExceptionComponent? comp = null)
-    {
-        comp ??= EnsureComp<FactionExceptionComponent>(uid);
-        foreach (var ignore in ignored)
-        {
-            IgnoreEntity(uid, ignore, comp);
-        }
-    }
-
-    /// <summary>
-    /// Makes an entity always be considered hostile.
-    /// </summary>
-    public void AggroEntity(EntityUid uid, EntityUid target, FactionExceptionComponent? comp = null)
-    {
-        comp ??= EnsureComp<FactionExceptionComponent>(uid);
-        comp.Hostiles.Add(target);
-        EnsureComp<FactionExceptionTrackerComponent>(target).Entities.Add(uid);
-    }
-
-    /// <summary>
-    /// Makes an entity no longer be considered hostile, if it was.
-    /// Doesn't apply to regular faction hostilities.
-    /// </summary>
-    public void DeAggroEntity(EntityUid uid, EntityUid target, FactionExceptionComponent? comp = null)
-    {
-        if (!Resolve(uid, ref comp, false))
-            return;
-        if (!comp.Hostiles.Remove(target) || !_trackerQuery.TryGetComponent(target, out var tracker))
-            return;
-        tracker.Entities.Remove(uid);
-    }
-
-    /// <summary>
-    /// Makes a list of entities no longer be considered hostile, if it was.
-    /// Doesn't apply to regular faction hostilities.
-    /// </summary>
-    public void AggroEntities(EntityUid uid, IEnumerable<EntityUid> entities, FactionExceptionComponent? comp = null)
-    {
-        comp ??= EnsureComp<FactionExceptionComponent>(uid);
-        foreach (var ent in entities)
-        {
-            AggroEntity(uid, ent, comp);
-        }
-    }
-}
index a4bd234307e523ac007b114a15c3565c060ccb48..8ae10d7383449d07fad075d36b46b42ed83e146c 100644 (file)
@@ -25,6 +25,8 @@ using Content.Shared.Mobs;
 using Content.Shared.Mobs.Components;
 using Content.Shared.Mobs.Systems;
 using Content.Shared.Movement.Systems;
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Systems;
 using Content.Shared.Nutrition.AnimalHusbandry;
 using Content.Shared.Nutrition.Components;
 using Content.Shared.Popups;
@@ -215,11 +217,7 @@ namespace Content.Server.Zombies
                 _damageable.SetAllDamage(target, damageablecomp, 0);
             _mobState.ChangeMobState(target, MobState.Alive);
 
-            var factionComp = EnsureComp<NpcFactionMemberComponent>(target);
-            foreach (var id in new List<string>(factionComp.Factions))
-            {
-                _faction.RemoveFaction(target, id);
-            }
+            _faction.ClearFactions(target, dirty: false);
             _faction.AddFaction(target, "Zombie");
 
             //gives it the funny "Zombie ___" name.
diff --git a/Content.Shared/Friends/Components/PettableFriendComponent.cs b/Content.Shared/Friends/Components/PettableFriendComponent.cs
new file mode 100644 (file)
index 0000000..d05e176
--- /dev/null
@@ -0,0 +1,24 @@
+using Content.Shared.Friends.Systems;
+using Robust.Shared.GameStates;
+
+namespace Content.Shared.Friends.Components;
+
+/// <summary>
+/// Pet something to become friends with it (use in hand, press Z)
+/// Requires this entity to have FactionExceptionComponent to work.
+/// </summary>
+[RegisterComponent, NetworkedComponent, Access(typeof(PettableFriendSystem))]
+public sealed partial class PettableFriendComponent : Component
+{
+    /// <summary>
+    /// Localized popup sent when petting for the first time
+    /// </summary>
+    [DataField(required: true)]
+    public LocId SuccessString = string.Empty;
+
+    /// <summary>
+    /// Localized popup sent when petting multiple times
+    /// </summary>
+    [DataField(required: true)]
+    public LocId FailureString = string.Empty;
+}
diff --git a/Content.Shared/Friends/Systems/PettableFriendSystem.cs b/Content.Shared/Friends/Systems/PettableFriendSystem.cs
new file mode 100644 (file)
index 0000000..00a4ddd
--- /dev/null
@@ -0,0 +1,62 @@
+using Content.Shared.Chemistry.Components;
+using Content.Shared.Friends.Components;
+using Content.Shared.Interaction.Events;
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Systems;
+using Content.Shared.Popups;
+using Content.Shared.Timing;
+
+namespace Content.Shared.Friends.Systems;
+
+public sealed class PettableFriendSystem : EntitySystem
+{
+    [Dependency] private readonly NpcFactionSystem _factionException = default!;
+    [Dependency] private readonly SharedPopupSystem _popup = default!;
+    [Dependency] private readonly UseDelaySystem _useDelay = default!;
+
+    private EntityQuery<FactionExceptionComponent> _exceptionQuery;
+    private EntityQuery<UseDelayComponent> _useDelayQuery;
+
+    public override void Initialize()
+    {
+        base.Initialize();
+
+        _exceptionQuery = GetEntityQuery<FactionExceptionComponent>();
+        _useDelayQuery = GetEntityQuery<UseDelayComponent>();
+
+        SubscribeLocalEvent<PettableFriendComponent, UseInHandEvent>(OnUseInHand);
+        SubscribeLocalEvent<PettableFriendComponent, GotRehydratedEvent>(OnRehydrated);
+    }
+
+    private void OnUseInHand(Entity<PettableFriendComponent> ent, ref UseInHandEvent args)
+    {
+        var (uid, comp) = ent;
+        var user = args.User;
+        if (args.Handled || !_exceptionQuery.TryGetComponent(uid, out var exceptionComp))
+            return;
+
+        if (_useDelayQuery.TryGetComponent(uid, out var useDelay) && !_useDelay.TryResetDelay((uid, useDelay), true))
+            return;
+
+        var exception = (uid, exceptionComp);
+        if (_factionException.IsIgnored(exception, user))
+        {
+            _popup.PopupClient(Loc.GetString(comp.FailureString, ("target", uid)), user, user);
+            return;
+        }
+
+        // you have made a new friend :)
+        _popup.PopupClient(Loc.GetString(comp.SuccessString, ("target", uid)), user, user);
+        _factionException.IgnoreEntity(exception, user);
+        args.Handled = true;
+    }
+
+    private void OnRehydrated(Entity<PettableFriendComponent> ent, ref GotRehydratedEvent args)
+    {
+        // can only pet before hydrating, after that the fish cannot be negotiated with
+        if (!TryComp<FactionExceptionComponent>(ent, out var comp))
+            return;
+
+        _factionException.IgnoreEntities(args.Target, comp.Ignored);
+    }
+}
similarity index 72%
rename from Content.Server/NPC/Components/FactionExceptionComponent.cs
rename to Content.Shared/NPC/Components/FactionExceptionComponent.cs
index 6abd503537c78afbb1bbd4c82984b213a5fdc85e..54de0404c2fd747d52ba168e269a4e54cdc1769f 100644 (file)
@@ -1,23 +1,24 @@
-using Content.Server.NPC.Systems;
+using Content.Shared.NPC.Systems;
+using Robust.Shared.GameStates;
 
-namespace Content.Server.NPC.Components;
+namespace Content.Shared.NPC.Components;
 
 /// <summary>
 /// Prevents an NPC from attacking ignored entities from enemy factions.
 /// Can be added to if pettable, see PettableFriendComponent.
 /// </summary>
-[RegisterComponent, Access(typeof(NpcFactionSystem))]
+[RegisterComponent, NetworkedComponent, Access(typeof(NpcFactionSystem))]
 public sealed partial class FactionExceptionComponent : Component
 {
     /// <summary>
     /// Collection of entities that this NPC will refuse to attack
     /// </summary>
-    [DataField("ignored")]
+    [DataField]
     public HashSet<EntityUid> Ignored = new();
 
     /// <summary>
     /// Collection of entities that this NPC will attack, regardless of faction.
     /// </summary>
-    [DataField("hostiles")]
+    [DataField]
     public HashSet<EntityUid> Hostiles = new();
 }
diff --git a/Content.Shared/NPC/Components/FactionExceptionTrackerComponent.cs b/Content.Shared/NPC/Components/FactionExceptionTrackerComponent.cs
new file mode 100644 (file)
index 0000000..f6eded7
--- /dev/null
@@ -0,0 +1,17 @@
+using Content.Shared.NPC.Systems;
+using Robust.Shared.GameStates;
+
+namespace Content.Shared.NPC.Components;
+
+/// <summary>
+/// This is used for tracking entities stored in <see cref="FactionExceptionComponent"/>.
+/// </summary>
+[RegisterComponent, NetworkedComponent, Access(typeof(NpcFactionSystem))]
+public sealed partial class FactionExceptionTrackerComponent : Component
+{
+    /// <summary>
+    /// Entities with <see cref="FactionExceptionComponent"/> that are tracking this entity.
+    /// </summary>
+    [DataField]
+    public HashSet<EntityUid> Entities = new();
+}
diff --git a/Content.Shared/NPC/Components/NpcFactionMemberComponent.cs b/Content.Shared/NPC/Components/NpcFactionMemberComponent.cs
new file mode 100644 (file)
index 0000000..91521e9
--- /dev/null
@@ -0,0 +1,28 @@
+using Content.Shared.NPC.Prototypes;
+using Content.Shared.NPC.Systems;
+using Robust.Shared.GameStates;
+using Robust.Shared.Prototypes;
+
+namespace Content.Shared.NPC.Components;
+
+[RegisterComponent, NetworkedComponent, Access(typeof(NpcFactionSystem))]
+public sealed partial class NpcFactionMemberComponent : Component
+{
+    /// <summary>
+    /// Factions this entity is a part of.
+    /// </summary>
+    [DataField]
+    public HashSet<ProtoId<NpcFactionPrototype>> Factions = new();
+
+    /// <summary>
+    /// Cached friendly factions.
+    /// </summary>
+    [ViewVariables]
+    public readonly HashSet<ProtoId<NpcFactionPrototype>> FriendlyFactions = new();
+
+    /// <summary>
+    /// Cached hostile factions.
+    /// </summary>
+    [ViewVariables]
+    public readonly HashSet<ProtoId<NpcFactionPrototype>> HostileFactions = new();
+}
diff --git a/Content.Shared/NPC/Prototypes/NpcFactionPrototype.cs b/Content.Shared/NPC/Prototypes/NpcFactionPrototype.cs
new file mode 100644 (file)
index 0000000..1dcdd75
--- /dev/null
@@ -0,0 +1,32 @@
+using Robust.Shared.Prototypes;
+
+namespace Content.Shared.NPC.Prototypes;
+
+/// <summary>
+/// Contains data about this faction's relations with other factions.
+/// </summary>
+[Prototype("npcFaction")]
+public sealed partial class NpcFactionPrototype : IPrototype
+{
+    [ViewVariables]
+    [IdDataField]
+    public string ID { get; private set; } = default!;
+
+    [DataField]
+    public List<ProtoId<NpcFactionPrototype>> Friendly = new();
+
+    [DataField]
+    public List<ProtoId<NpcFactionPrototype>> Hostile = new();
+}
+
+/// <summary>
+/// Cached data for the faction prototype. Is modified at runtime, whereas the prototype is not.
+/// </summary>
+public record struct FactionData
+{
+    [ViewVariables]
+    public HashSet<ProtoId<NpcFactionPrototype>> Friendly;
+
+    [ViewVariables]
+    public HashSet<ProtoId<NpcFactionPrototype>> Hostile;
+}
diff --git a/Content.Shared/NPC/Systems/NpcFactionSystem.Exception.cs b/Content.Shared/NPC/Systems/NpcFactionSystem.Exception.cs
new file mode 100644 (file)
index 0000000..e69f0c2
--- /dev/null
@@ -0,0 +1,135 @@
+using Content.Shared.NPC.Components;
+using System.Linq;
+
+namespace Content.Shared.NPC.Systems;
+
+/// <summary>
+/// Prevents an NPC from attacking some entities from an enemy faction.
+/// Also makes it attack some entities even if they are in neutral factions (retaliation).
+/// </summary>
+public sealed partial class NpcFactionSystem
+{
+    private EntityQuery<FactionExceptionComponent> _exceptionQuery;
+    private EntityQuery<FactionExceptionTrackerComponent> _trackerQuery;
+
+    public void InitializeException()
+    {
+        _exceptionQuery = GetEntityQuery<FactionExceptionComponent>();
+        _trackerQuery = GetEntityQuery<FactionExceptionTrackerComponent>();
+
+        SubscribeLocalEvent<FactionExceptionComponent, ComponentShutdown>(OnShutdown);
+        SubscribeLocalEvent<FactionExceptionTrackerComponent, ComponentShutdown>(OnTrackerShutdown);
+    }
+
+    private void OnShutdown(Entity<FactionExceptionComponent> ent, ref ComponentShutdown args)
+    {
+        foreach (var uid in ent.Comp.Hostiles)
+        {
+            if (_trackerQuery.TryGetComponent(uid, out var tracker))
+                tracker.Entities.Remove(ent);
+        }
+
+        foreach (var uid in ent.Comp.Ignored)
+        {
+            if (_trackerQuery.TryGetComponent(uid, out var tracker))
+                tracker.Entities.Remove(ent);
+        }
+    }
+
+    private void OnTrackerShutdown(Entity<FactionExceptionTrackerComponent> ent, ref ComponentShutdown args)
+    {
+        foreach (var uid in ent.Comp.Entities)
+        {
+            if (!_exceptionQuery.TryGetComponent(uid, out var exception))
+                continue;
+
+            exception.Ignored.Remove(ent);
+            exception.Hostiles.Remove(ent);
+        }
+    }
+
+    /// <summary>
+    /// Returns whether the entity from an enemy faction won't be attacked
+    /// </summary>
+    public bool IsIgnored(Entity<FactionExceptionComponent?> ent, EntityUid target)
+    {
+        if (!Resolve(ent, ref ent.Comp, false))
+            return false;
+
+        return ent.Comp.Ignored.Contains(target);
+    }
+
+    /// <summary>
+    /// Returns the specific hostile entities for a given entity.
+    /// </summary>
+    public IEnumerable<EntityUid> GetHostiles(Entity<FactionExceptionComponent?> ent)
+    {
+        if (!Resolve(ent, ref ent.Comp, false))
+            return Array.Empty<EntityUid>();
+
+        // evil c#
+        return ent.Comp!.Hostiles;
+    }
+
+    /// <summary>
+    /// Prevents an entity from an enemy faction from being attacked
+    /// </summary>
+    public void IgnoreEntity(Entity<FactionExceptionComponent?> ent, Entity<FactionExceptionTrackerComponent?> target)
+    {
+        ent.Comp ??= EnsureComp<FactionExceptionComponent>(ent);
+        ent.Comp.Ignored.Add(target);
+        target.Comp ??= EnsureComp<FactionExceptionTrackerComponent>(target);
+        target.Comp.Entities.Add(ent);
+    }
+
+    /// <summary>
+    /// Prevents a list of entities from an enemy faction from being attacked
+    /// </summary>
+    public void IgnoreEntities(Entity<FactionExceptionComponent?> ent, IEnumerable<EntityUid> ignored)
+    {
+        ent.Comp ??= EnsureComp<FactionExceptionComponent>(ent);
+        foreach (var ignore in ignored)
+        {
+            IgnoreEntity(ent, ignore);
+        }
+    }
+
+    /// <summary>
+    /// Makes an entity always be considered hostile.
+    /// </summary>
+    public void AggroEntity(Entity<FactionExceptionComponent?> ent, Entity<FactionExceptionTrackerComponent?> target)
+    {
+        ent.Comp ??= EnsureComp<FactionExceptionComponent>(ent);
+        ent.Comp.Hostiles.Add(target);
+        target.Comp ??= EnsureComp<FactionExceptionTrackerComponent>(target);
+        target.Comp.Entities.Add(ent);
+    }
+
+    /// <summary>
+    /// Makes an entity no longer be considered hostile, if it was.
+    /// Doesn't apply to regular faction hostilities.
+    /// </summary>
+    public void DeAggroEntity(Entity<FactionExceptionComponent?> ent, EntityUid target)
+    {
+        if (!Resolve(ent, ref ent.Comp, false))
+            return;
+
+        if (!ent.Comp.Hostiles.Remove(target) || !_trackerQuery.TryGetComponent(target, out var tracker))
+            return;
+
+        tracker.Entities.Remove(ent);
+    }
+
+    /// <summary>
+    /// Makes a list of entities no longer be considered hostile, if it was.
+    /// Doesn't apply to regular faction hostilities.
+    /// </summary>
+    public void AggroEntities(Entity<FactionExceptionComponent?> ent, IEnumerable<EntityUid> entities)
+    {
+        ent.Comp ??= EnsureComp<FactionExceptionComponent>(ent);
+        foreach (var uid in entities)
+        {
+            AggroEntity(ent, uid);
+        }
+    }
+}
similarity index 50%
rename from Content.Server/NPC/Systems/NpcFactionSystem.cs
rename to Content.Shared/NPC/Systems/NpcFactionSystem.cs
index 36ee3724a23bf30539c73be73a88f848a05d705b..ad81f01e1da91ffdb150ae70abce2ac5441913ed 100644 (file)
@@ -1,10 +1,10 @@
+using Content.Shared.NPC.Components;
+using Content.Shared.NPC.Prototypes;
+using Robust.Shared.Prototypes;
 using System.Collections.Frozen;
 using System.Linq;
-using Content.Server.NPC.Components;
-using JetBrains.Annotations;
-using Robust.Shared.Prototypes;
 
-namespace Content.Server.NPC.Systems;
+namespace Content.Shared.NPC.Systems;
 
 /// <summary>
 ///     Outlines faction relationships with each other.
@@ -12,7 +12,8 @@ namespace Content.Server.NPC.Systems;
 public sealed partial class NpcFactionSystem : EntitySystem
 {
     [Dependency] private readonly EntityLookupSystem _lookup = default!;
-    [Dependency] private readonly IPrototypeManager _protoManager = default!;
+    [Dependency] private readonly IPrototypeManager _proto = default!;
+    [Dependency] private readonly SharedTransformSystem _xform = default!;
 
     /// <summary>
     /// To avoid prototype mutability we store an intermediary data class that gets used instead.
@@ -36,122 +37,126 @@ public sealed partial class NpcFactionSystem : EntitySystem
             RefreshFactions();
     }
 
-    private void OnFactionStartup(EntityUid uid, NpcFactionMemberComponent memberComponent, ComponentStartup args)
+    private void OnFactionStartup(Entity<NpcFactionMemberComponent> ent, ref ComponentStartup args)
     {
-        RefreshFactions(memberComponent);
+        RefreshFactions(ent);
     }
 
     /// <summary>
     /// Refreshes the cached factions for this component.
     /// </summary>
-    private void RefreshFactions(NpcFactionMemberComponent memberComponent)
+    private void RefreshFactions(Entity<NpcFactionMemberComponent> ent)
     {
-        memberComponent.FriendlyFactions.Clear();
-        memberComponent.HostileFactions.Clear();
+        ent.Comp.FriendlyFactions.Clear();
+        ent.Comp.HostileFactions.Clear();
 
-        foreach (var faction in memberComponent.Factions)
+        foreach (var faction in ent.Comp.Factions)
         {
-            // YAML Linter already yells about this
+            // YAML Linter already yells about this, don't need to log an error here
             if (!_factions.TryGetValue(faction, out var factionData))
                 continue;
 
-            memberComponent.FriendlyFactions.UnionWith(factionData.Friendly);
-            memberComponent.HostileFactions.UnionWith(factionData.Hostile);
+            ent.Comp.FriendlyFactions.UnionWith(factionData.Friendly);
+            ent.Comp.HostileFactions.UnionWith(factionData.Hostile);
         }
     }
 
+    /// <summary>
+    /// Returns whether an entity is a member of a faction.
+    /// </summary>
+    public bool IsMember(Entity<NpcFactionMemberComponent?> ent, string faction)
+    {
+        if (!Resolve(ent, ref ent.Comp, false))
+            return false;
+
+        return ent.Comp.Factions.Contains(faction);
+    }
+
     /// <summary>
     /// Adds this entity to the particular faction.
     /// </summary>
-    public void AddFaction(EntityUid uid, string faction, bool dirty = true)
+    public void AddFaction(Entity<NpcFactionMemberComponent?> ent, string faction, bool dirty = true)
     {
-        if (!_protoManager.HasIndex<NpcFactionPrototype>(faction))
+        if (!_proto.HasIndex<NpcFactionPrototype>(faction))
         {
             Log.Error($"Unable to find faction {faction}");
             return;
         }
 
-        var comp = EnsureComp<NpcFactionMemberComponent>(uid);
-        if (!comp.Factions.Add(faction))
+        ent.Comp ??= EnsureComp<NpcFactionMemberComponent>(ent);
+        if (!ent.Comp.Factions.Add(faction))
             return;
 
         if (dirty)
-        {
-            RefreshFactions(comp);
-        }
+            RefreshFactions((ent, ent.Comp));
     }
 
     /// <summary>
     /// Removes this entity from the particular faction.
     /// </summary>
-    public void RemoveFaction(EntityUid uid, string faction, bool dirty = true)
+    public void RemoveFaction(Entity<NpcFactionMemberComponent?> ent, string faction, bool dirty = true)
     {
-        if (!_protoManager.HasIndex<NpcFactionPrototype>(faction))
+        if (!_proto.HasIndex<NpcFactionPrototype>(faction))
         {
             Log.Error($"Unable to find faction {faction}");
             return;
         }
 
-        if (!TryComp<NpcFactionMemberComponent>(uid, out var component))
+        if (!Resolve(ent, ref ent.Comp, false))
             return;
 
-        if (!component.Factions.Remove(faction))
+        if (!ent.Comp.Factions.Remove(faction))
             return;
 
         if (dirty)
-        {
-            RefreshFactions(component);
-        }
+            RefreshFactions((ent, ent.Comp));
     }
 
     /// <summary>
     /// Remove this entity from all factions.
     /// </summary>
-    public void ClearFactions(EntityUid uid, bool dirty = true)
+    public void ClearFactions(Entity<NpcFactionMemberComponent?> ent, bool dirty = true)
     {
-        if (!TryComp<NpcFactionMemberComponent>(uid, out var component))
+        if (!Resolve(ent, ref ent.Comp, false))
             return;
 
-        component.Factions.Clear();
+        ent.Comp.Factions.Clear();
 
         if (dirty)
-            RefreshFactions(component);
+            RefreshFactions((ent, ent.Comp));
     }
 
-    public IEnumerable<EntityUid> GetNearbyHostiles(EntityUid entity, float range, NpcFactionMemberComponent? component = null)
+    public IEnumerable<EntityUid> GetNearbyHostiles(Entity<NpcFactionMemberComponent?, FactionExceptionComponent?> ent, float range)
     {
-        if (!Resolve(entity, ref component, false))
+        if (!Resolve(ent, ref ent.Comp1, false))
             return Array.Empty<EntityUid>();
 
-        var hostiles = GetNearbyFactions(entity, range, component.HostileFactions);
-        if (TryComp<FactionExceptionComponent>(entity, out var factionException))
-        {
-            // ignore anything from enemy faction that we are explicitly friendly towards
-            return hostiles
-                .Union(GetHostiles(entity, factionException))
-                .Where(target => !IsIgnored(entity, target, factionException));
-        }
-
-        return hostiles;
+        var hostiles = GetNearbyFactions(ent, range, ent.Comp1.HostileFactions)
+            // ignore mobs that have both hostile faction and the same faction,
+            // otherwise having multiple factions is strictly negative
+            .Where(target => !IsEntityFriendly((ent, ent.Comp1), target));
+        if (!Resolve(ent, ref ent.Comp2, false))
+            return hostiles;
+
+        // ignore anything from enemy faction that we are explicitly friendly towards
+        var faction = (ent.Owner, ent.Comp2);
+        return hostiles
+            .Union(GetHostiles(faction))
+            .Where(target => !IsIgnored(faction, target));
     }
 
-    [PublicAPI]
-    public IEnumerable<EntityUid> GetNearbyFriendlies(EntityUid entity, float range, NpcFactionMemberComponent? component = null)
+    public IEnumerable<EntityUid> GetNearbyFriendlies(Entity<NpcFactionMemberComponent?> ent, float range)
     {
-        if (!Resolve(entity, ref component, false))
+        if (!Resolve(ent, ref ent.Comp, false))
             return Array.Empty<EntityUid>();
 
-        return GetNearbyFactions(entity, range, component.FriendlyFactions);
+        return GetNearbyFactions(ent, range, ent.Comp.FriendlyFactions);
     }
 
-    private IEnumerable<EntityUid> GetNearbyFactions(EntityUid entity, float range, HashSet<string> factions)
+    private IEnumerable<EntityUid> GetNearbyFactions(EntityUid entity, float range, HashSet<ProtoId<NpcFactionPrototype>> factions)
     {
-        var xformQuery = GetEntityQuery<TransformComponent>();
-
-        if (!xformQuery.TryGetComponent(entity, out var entityXform))
-            yield break;
-
-        foreach (var ent in _lookup.GetEntitiesInRange<NpcFactionMemberComponent>(entityXform.MapPosition, range))
+        var xform = Transform(entity);
+        foreach (var ent in _lookup.GetEntitiesInRange<NpcFactionMemberComponent>(_xform.GetMapCoordinates((entity, xform)), range))
         {
             if (ent.Owner == entity)
                 continue;
@@ -163,12 +168,15 @@ public sealed partial class NpcFactionSystem : EntitySystem
         }
     }
 
-    public bool IsEntityFriendly(EntityUid uidA, EntityUid uidB, NpcFactionMemberComponent? factionA = null, NpcFactionMemberComponent? factionB = null)
+    /// <remarks>
+    /// 1-way and purely faction based, ignores faction exception.
+    /// </remarks>
+    public bool IsEntityFriendly(Entity<NpcFactionMemberComponent?> ent, Entity<NpcFactionMemberComponent?> other)
     {
-        if (!Resolve(uidA, ref factionA, false) || !Resolve(uidB, ref factionB, false))
+        if (!Resolve(ent, ref ent.Comp, false) || !Resolve(other, ref other.Comp, false))
             return false;
 
-        return factionA.Factions.Overlaps(factionB.Factions) || factionA.FriendlyFactions.Overlaps(factionB.Factions);
+        return ent.Comp.Factions.Overlaps(other.Comp.Factions) || ent.Comp.FriendlyFactions.Overlaps(other.Comp.Factions);
     }
 
     public bool IsFactionFriendly(string target, string with)
@@ -176,13 +184,13 @@ public sealed partial class NpcFactionSystem : EntitySystem
         return _factions[target].Friendly.Contains(with) && _factions[with].Friendly.Contains(target);
     }
 
-    public bool IsFactionFriendly(string target, EntityUid with, NpcFactionMemberComponent? factionWith = null)
+    public bool IsFactionFriendly(string target, Entity<NpcFactionMemberComponent?> with)
     {
-        if (!Resolve(with, ref factionWith, false))
+        if (!Resolve(with, ref with.Comp, false))
             return false;
 
-        return factionWith.Factions.All(x => IsFactionFriendly(target, x)) ||
-               factionWith.FriendlyFactions.Contains(target);
+        return with.Comp.Factions.All(x => IsFactionFriendly(target, x)) ||
+               with.Comp.FriendlyFactions.Contains(target);
     }
 
     public bool IsFactionHostile(string target, string with)
@@ -190,13 +198,13 @@ public sealed partial class NpcFactionSystem : EntitySystem
         return _factions[target].Hostile.Contains(with) && _factions[with].Hostile.Contains(target);
     }
 
-    public bool IsFactionHostile(string target, EntityUid with, NpcFactionMemberComponent? factionWith = null)
+    public bool IsFactionHostile(string target, Entity<NpcFactionMemberComponent?> with)
     {
-        if (!Resolve(with, ref factionWith, false))
+        if (!Resolve(with, ref with.Comp, false))
             return false;
 
-        return factionWith.Factions.All(x => IsFactionHostile(target, x)) ||
-               factionWith.HostileFactions.Contains(target);
+        return with.Comp.Factions.All(x => IsFactionHostile(target, x)) ||
+               with.Comp.HostileFactions.Contains(target);
     }
 
     public bool IsFactionNeutral(string target, string with)
@@ -226,26 +234,6 @@ public sealed partial class NpcFactionSystem : EntitySystem
         RefreshFactions();
     }
 
-    private void RefreshFactions()
-    {
-
-        _factions = _protoManager.EnumeratePrototypes<NpcFactionPrototype>().ToFrozenDictionary(
-            faction => faction.ID,
-            faction =>  new FactionData
-            {
-                Friendly = faction.Friendly.ToHashSet(),
-                Hostile = faction.Hostile.ToHashSet()
-
-            });
-
-        foreach (var comp in EntityQuery<NpcFactionMemberComponent>(true))
-        {
-            comp.FriendlyFactions.Clear();
-            comp.HostileFactions.Clear();
-            RefreshFactions(comp);
-        }
-    }
-
     /// <summary>
     /// Makes the source faction hostile to the target faction, 1-way.
     /// </summary>
@@ -267,5 +255,23 @@ public sealed partial class NpcFactionSystem : EntitySystem
         sourceFaction.Hostile.Add(target);
         RefreshFactions();
     }
-}
 
+    private void RefreshFactions()
+    {
+        _factions = _proto.EnumeratePrototypes<NpcFactionPrototype>().ToFrozenDictionary(
+            faction => faction.ID,
+            faction =>  new FactionData
+            {
+                Friendly = faction.Friendly.ToHashSet(),
+                Hostile = faction.Hostile.ToHashSet()
+            });
+
+        var query = AllEntityQuery<NpcFactionMemberComponent>();
+        while (query.MoveNext(out var uid, out var comp))
+        {
+            comp.FriendlyFactions.Clear();
+            comp.HostileFactions.Clear();
+            RefreshFactions((uid, comp));
+        }
+    }
+}