Skyrat-SS13 / Skyrat-tg

A Skyrat downstream of /tg/station SS13.
GNU Affero General Public License v3.0
122 stars 685 forks source link

[MIRROR] Full revert of #79995 (tri-ai station trait). Partal revert of #81681 (human ai) #28820

Closed SkyratBot closed 1 month ago

SkyratBot commented 1 month ago

Original PR: https://github.com/tgstation/tgstation/pull/84850

79995 removed the verb and secrets button for no reason. This means you can no longer decide to do tri-ai in the lobby, you have to do it next round or make players suffer thru another init cycle.

In general systems that require things be decided the previous round are really fucking annoying and should be minimized to only specific things that absolutely have to for technical reason.

This is not one of them.

I do not have the time to redesign the station trait to make use of the existing admin code so this is a full revert instead.

If the station trait is re-added, it should be done so without removing the secrets button or admin verb.

81681 had an undocumented and un-atmoized change, in that it removed a job related signal needlessly, that undocumented change has been reverted because this system used that signal.

@ Fikou @ JohnFulpWillard

(also cc @ tralezab and @ Ghommie as you merged the reverted prs)