More Bug Fixes (#1311)

<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

[Mice pickup
fixed!](space-wizards/space-station-14#33602)
[Mobs can no longer spill
jugs](space-wizards/space-station-14#33602)
Rest in changelog.

# Changelog
:cl:
- fix: You can pick up mobs again with left click.
- fix: No more mob jug spill, including you revenants.
- fix: Set the Gloves of the North Star to the proper attack speed.
- fix: Medibots will no longer try to heal borgs.

---------

Co-authored-by: MilenVolf <[email protected]>
Co-authored-by: slarticodefast <[email protected]>

Content.Server/NPC/HTN/PrimitiveTasks/Operators/Specific/MedibotInjectOperator.cs

@@ -7,6 +7,7 @@
 using Content.Shared.Interaction;
 using Content.Shared.Mobs.Components;
 using Content.Shared.Popups;
+using Content.Shared.Silicon.Components;
 using Content.Shared.Silicons.Bots;
 using Robust.Shared.Audio.Systems;
 
@@ -53,9 +54,11 @@ public override HTNOperatorStatus Update(NPCBlackboard blackboard, float frameTi
         if (!blackboard.TryGetValue<EntityUid>(TargetKey, out var target, _entMan) || _entMan.Deleted(target))
             return HTNOperatorStatus.Failed;
 
-        if (!_entMan.TryGetComponent<MedibotComponent>(owner, out var botComp))
+        if (_entMan.HasComponent<SiliconComponent>(target))
             return HTNOperatorStatus.Failed;
 
+        if (!_entMan.TryGetComponent<MedibotComponent>(owner, out var botComp))
+            return HTNOperatorStatus.Failed;
 
         if (!_entMan.TryGetComponent<DamageableComponent>(target, out var damage))
             return HTNOperatorStatus.Failed;

Content.Server/NPC/HTN/PrimitiveTasks/Operators/Specific/PickNearbyInjectableOperator.cs

@@ -8,6 +8,8 @@
 using Content.Shared.Mobs.Components;
 using Content.Shared.Silicons.Bots;
 using Content.Shared.Emag.Components;
+using Content.Shared.Silicon.Components;
+
 
 namespace Content.Server.NPC.HTN.PrimitiveTasks.Operators.Specific;
 
@@ -64,6 +66,9 @@ public override void Initialize(IEntitySystemManager sysManager)
                 damageQuery.TryGetComponent(entity, out var damage) &&
                 !recentlyInjected.HasComponent(entity))
             {
+                if (_entManager.HasComponent<SiliconComponent>(entity))
+                    continue;
+
                 // no treating dead bodies
                 if (!_medibot.TryGetTreatment(medibot, state.CurrentState, out var treatment))
                     continue;

Content.Server/Nyanotrasen/Abilities/Felinid/FelinidSystem.cs

@@ -130,7 +130,11 @@ private void OnEatMouse(EntityUid uid, FelinidComponent component, EatMouseActio
 
         if (hunger.CurrentThreshold == Shared.Nutrition.Components.HungerThreshold.Overfed)
         {
-            _popupSystem.PopupEntity(Loc.GetString("food-system-you-cannot-eat-any-more"), uid, uid, Shared.Popups.PopupType.SmallCaution);
+            _popupSystem.PopupEntity(
+                Loc.GetString("food-system-you-cannot-eat-any-more"),
+                uid,
+                uid,
+                Shared.Popups.PopupType.SmallCaution);
             return;
         }
 
@@ -147,15 +151,20 @@ private void OnEatMouse(EntityUid uid, FelinidComponent component, EatMouseActio
             _actionsSystem.SetCharges(component.HairballAction, 1); // You get the charge back and that's it. Tough.
             _actionsSystem.SetEnabled(component.HairballAction, true);
         }
+
         Del(component.EatActionTarget.Value);
         component.EatActionTarget = null;
 
         _audio.PlayPvs("/Audio/DeltaV/Items/eatfood.ogg", uid, AudioHelpers.WithVariation(0.15f));
 
         _hungerSystem.ModifyHunger(uid, 50f, hunger);
 
-        if (component.EatAction != null)
+        if (component.EatAction is not null && _actionsSystem.TryGetActionData(uid, out var result))
+        {
+            if (result.AttachedEntity == null)
+                return;
             _actionsSystem.RemoveAction(uid, component.EatAction.Value);
+        }
     }
 
     private void SpawnHairball(EntityUid uid, FelinidComponent component)

Content.Shared/Buckle/SharedBuckleSystem.Interaction.cs

@@ -95,10 +95,9 @@ private void OnBuckleInteractHand(Entity<BuckleComponent> ent, ref InteractHandE
             return;
 
         if (ent.Comp.BuckledTo != null)
-            TryUnbuckle(ent!, args.User, popup: true);
+            args.Handled = TryUnbuckle(ent!, args.User, popup: true);
 
         // TODO BUCKLE add out bool for whether a pop-up was generated or not.
-        args.Handled = true;
     }
 
     private void AddStrapVerbs(EntityUid uid, StrapComponent component, GetVerbsEvent<InteractionVerb> args)

Content.Shared/Fluids/SharedPuddleSystem.Spillable.cs

@@ -34,7 +34,7 @@ private void OnExamined(Entity<SpillableComponent> entity, ref ExaminedEvent arg
 
     private void AddSpillVerb(Entity<SpillableComponent> entity, ref GetVerbsEvent<Verb> args)
     {
-        if (!args.CanAccess || !args.CanInteract)
+        if (!args.CanAccess || !args.CanInteract || args.Hands == null)
             return;
 
         if (!_solutionContainerSystem.TryGetSolution(args.Target, entity.Comp.SolutionName, out var soln, out var solution))
@@ -46,10 +46,6 @@ private void AddSpillVerb(Entity<SpillableComponent> entity, ref GetVerbsEvent<V
         if (solution.Volume == FixedPoint2.Zero)
             return;
 
-        if (HasComp<PreventSpillerComponent>(args.User))
-            return;
-
-
         Verb verb = new()
         {
             Text = Loc.GetString("spill-target-verb-get-data-text")

Resources/Prototypes/Entities/Clothing/Hands/gloves.yml

@@ -361,7 +361,7 @@
     sprite: Clothing/Hands/Gloves/northstar.rsi
   - type: MeleeWeapon
     autoAttack: true
-    attackRate: 4
+    attackRate: 0.25
     heavyStaminaCost: 2
     heavyDamageBaseModifier: 1.05
     maxTargets: 1

Resources/Prototypes/Entities/Mobs/NPCs/animals.yml

@@ -1755,10 +1755,6 @@
   - type: FelinidFood # Nyanotrasen - Felinid, ability to eat mice, see Content.Server/Nyanotrasen/Abilities/Felinid/FelinidSystem.cs
   - type: BadFood
   - type: NonSpreaderZombie
-  - type: PreventSpiller
-  - type: RandomBark
-    barkMultiplier: 0.3
-    barkType: mouse
   - type: FireVisuals
     sprite: Mobs/Effects/onfire.rsi
     normalState: Mouse_burning

Resources/Prototypes/Entities/Mobs/NPCs/space.yml

@@ -386,4 +386,4 @@
   parent: MobCobraSpace
   suffix: "Salvage Ruleset"
   components:
-    - type: SalvageMobRestrictions
+    - type: SalvageMobRestrictions