Fixes Cocoons (Simple-Station#1169)

<!--
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]?
-->

This PR fixes the cocoon muffle issue and also add a "uncocoon" feature
for cocooners to use.
This also add icons on cocoon and fixes a butcher variable.

---

<!--
This is default collapsed, readers click to expand it and see all your
media
The PR media section can get very large at times, so this is a good way
to keep it clean
The title is written using HTML tags
The title must be within the <summary> tags or you won't see it
-->

<details><summary><h1>Media</h1></summary>
<p>


![image](https://github.com/user-attachments/assets/59c6dd7e-6b2b-45e4-bbab-e02db5d9b9ce)

![image](https://github.com/user-attachments/assets/574b9c7b-7e78-403c-82a4-cc0c3a7f2aaf)

</p>
</details>

---

# Changelog

<!--
You can add an author after the `:cl:` to change the name that appears
in the changelog (ex: `:cl: Death`)
Leaving it blank will default to your GitHub display name
This includes all available types for the changelog
-->

:cl: FoxxoTrystan
- tweak: Cocooners can now freely uncocoon any cocoons without the need
of a sharp object.
- fix: Cocoon making you stuck in muffle accent.

Content.Server/Cocoon/CocoonerSystem.cs

@@ -14,6 +14,11 @@
 using Content.Server.Speech.Components;
 using Robust.Shared.Containers;
 using Content.Shared.Mobs.Components;
+using Content.Shared.Destructible;
+using Robust.Shared.Random;
+using Content.Shared.Nutrition.Components;
+using Content.Shared.Storage;
+using Robust.Shared.Utility;
 
 namespace Content.Server.Cocoon
 {
@@ -25,17 +30,26 @@ public sealed class CocooningSystem : EntitySystem
         [Dependency] private readonly BlindableSystem _blindableSystem = default!;
         [Dependency] private readonly DamageableSystem _damageableSystem = default!;
         [Dependency] private readonly ISharedAdminLogManager _adminLogger = default!;
+        [Dependency] private readonly SharedDestructibleSystem _destructibleSystem = default!;
+        [Dependency] private readonly IRobustRandom _robustRandom = default!;
 
         private const string BodySlot = "body_slot";
 
         public override void Initialize()
         {
             base.Initialize();
-            SubscribeLocalEvent<CocoonerComponent, GetVerbsEvent<InnateVerb>>(AddCocoonVerb);
+            SubscribeLocalEvent<CocoonerComponent, GetVerbsEvent<InnateVerb>>(AddVerbs);
             SubscribeLocalEvent<CocoonComponent, EntInsertedIntoContainerMessage>(OnCocEntInserted);
             SubscribeLocalEvent<CocoonComponent, EntRemovedFromContainerMessage>(OnCocEntRemoved);
             SubscribeLocalEvent<CocoonComponent, DamageChangedEvent>(OnDamageChanged);
             SubscribeLocalEvent<CocoonerComponent, CocoonDoAfterEvent>(OnCocoonDoAfter);
+            SubscribeLocalEvent<CocoonerComponent, UnCocoonDoAfterEvent>(OnUnCocoonDoAfter);
+        }
+
+        private void AddVerbs(EntityUid uid, CocoonerComponent component, GetVerbsEvent<InnateVerb> args)
+        {
+            AddCocoonVerb(uid, component, args);
+            AddUnCocoonVerb(uid, component, args);
         }
 
         private void AddCocoonVerb(EntityUid uid, CocoonerComponent component, GetVerbsEvent<InnateVerb> args)
@@ -50,6 +64,25 @@ private void AddCocoonVerb(EntityUid uid, CocoonerComponent component, GetVerbsE
                     StartCocooning(uid, component, args.Target);
                 },
                 Text = Loc.GetString("cocoon"),
+                Icon = new SpriteSpecifier.Texture(new ResPath("/Textures/Interface/Actions/web.png")),
+                Priority = 2
+            };
+            args.Verbs.Add(verb);
+        }
+
+        private void AddUnCocoonVerb(EntityUid uid, CocoonerComponent component, GetVerbsEvent<InnateVerb> args)
+        {
+            if (!args.CanAccess || !args.CanInteract || !HasComp<CocoonComponent>(args.Target))
+                return;
+
+            InnateVerb verb = new()
+            {
+                Act = () =>
+                {
+                    StartUnCocooning(uid, component, args.Target);
+                },
+                Text = Loc.GetString("uncocoon"),
+                Icon = new SpriteSpecifier.Texture(new ResPath("/Textures/Interface/Actions/web.png")),
                 Priority = 2
             };
             args.Verbs.Add(verb);
@@ -71,9 +104,11 @@ private void OnCocEntInserted(EntityUid uid, CocoonComponent component, EntInser
         private void OnCocEntRemoved(EntityUid uid, CocoonComponent component, EntRemovedFromContainerMessage args)
         {
             if (TryComp<ReplacementAccentComponent>(args.Entity, out var replacement))
-                replacement.Accent = component.OldAccent ?? replacement.Accent;
-            else
-                RemComp<ReplacementAccentComponent>(args.Entity);
+                if (component.OldAccent is not null)
+                    replacement.Accent = component.OldAccent;
+                else
+                    RemComp(args.Entity, replacement);
+
 
             RemComp<StunnedComponent>(args.Entity);
             _blindableSystem.UpdateIsBlind(args.Entity);
@@ -107,6 +142,22 @@ private void StartCocooning(EntityUid uid, CocoonerComponent component, EntityUi
             _doAfter.TryStartDoAfter(args);
         }
 
+        private void StartUnCocooning(EntityUid uid, CocoonerComponent component, EntityUid target)
+        {
+            _popupSystem.PopupEntity(Loc.GetString("uncocoon-start-third-person", ("target", target), ("spider", Identity.Entity(uid, EntityManager))), uid,
+                Shared.Popups.PopupType.MediumCaution);
+
+            var delay = component.CocoonDelay / 2;
+
+            var args = new DoAfterArgs(EntityManager, uid, delay, new UnCocoonDoAfterEvent(), uid, target: target)
+            {
+                BreakOnUserMove = true,
+                BreakOnTargetMove = true,
+            };
+
+            _doAfter.TryStartDoAfter(args);
+        }
+
         private void OnCocoonDoAfter(EntityUid uid, CocoonerComponent component, CocoonDoAfterEvent args)
         {
             if (args.Handled || args.Cancelled || args.Args.Target == null)
@@ -128,5 +179,26 @@ private void OnCocoonDoAfter(EntityUid uid, CocoonerComponent component, CocoonD
             _adminLogger.Add(LogType.Action, impact, $"{ToPrettyString(args.Args.User):player} cocooned {ToPrettyString(args.Args.Target.Value):target}");
             args.Handled = true;
         }
+
+        private void OnUnCocoonDoAfter(EntityUid uid, CocoonerComponent component, UnCocoonDoAfterEvent args)
+        {
+            if (args.Handled || args.Cancelled || args.Args.Target == null)
+                return;
+
+            if (TryComp<ButcherableComponent>(args.Args.Target.Value, out var butcher))
+            {
+                var spawnEntities = EntitySpawnCollection.GetSpawns(butcher.SpawnedEntities, _robustRandom);
+                var coords = Transform(args.Args.Target.Value).MapPosition;
+                EntityUid popupEnt = default!;
+                foreach (var proto in spawnEntities)
+                    popupEnt = Spawn(proto, coords.Offset(_robustRandom.NextVector2(0.25f)));
+            }
+
+            _destructibleSystem.DestroyEntity(args.Args.Target.Value);
+
+            _adminLogger.Add(LogType.Action, LogImpact.Low
+            , $"{ToPrettyString(args.Args.User):player} uncocooned {ToPrettyString(args.Args.Target.Value):target}");
+            args.Handled = true;
+        }
     }
 }

Content.Shared/Cocoon/CocoonDoAfterEvent.cs

@@ -7,4 +7,9 @@ namespace Content.Shared.Cocoon
     public sealed partial class CocoonDoAfterEvent : SimpleDoAfterEvent
     {
     }
+
+    [Serializable, NetSerializable]
+    public sealed partial class UnCocoonDoAfterEvent : SimpleDoAfterEvent
+    {
+    }
 }

Resources/Locale/en-US/abilities/arachne.ftl

@@ -8,6 +8,9 @@ spin-web-start-second-person = You start spinning a web.
 spin-web-start-third-person = {CAPITALIZE(THE($spider))} starts spinning a web!
 cocoon-start-second-person = You start cocooning {THE($target)}.
 cocoon-start-third-person = {CAPITALIZE(THE($spider))} starts cocooning {THE($target)}.
+uncocoon-start-second-person = You start releasing {THE($target)}.
+uncocoon-start-third-person = {CAPITALIZE(THE($spider))} starts releasing {THE($target)}.
 spun-web-second-person = You spin up a web.
 spun-web-third-person = {CAPITALIZE(THE($spider))} spins up a web!
 cocoon = Cocoon
+uncocoon = Uncocoon

Resources/Prototypes/Nyanotrasen/Entities/Structures/Webbing/webs.yml

@@ -60,7 +60,7 @@
     butcheringType: Knife
     butcherDelay: 12
     spawned:
-    - id: MaterialCloth1
+    - id: MaterialWebSilk1
       amount: 1
       prob: 0.5 #This doesn't cost hunger so should at least make it not worth it time-wise
   - type: Appearance