Felinid Thievery #1102
Felinid Thievery #1102
Conversation
…o thieving Merged new commits into branch to push
WarMechanic
changed the title
|
Felinid thiefing was removed on purpose |
|
this has no reduction and still shows doafter so seems ok balance wise |
|
What's wrong with reduction? I believe that reduction while still maintaining doafter while allowing you to successfully take something, wouldn't let you get away with said item for free. Most people respond to pickpocketing over the halfway mark so I thought it would be a fair change. Multiplicative reduction which I reverted earlier I implemented because multiplicative strip time reduction has use cases in Security to strip belts and bags off criminals to search for contraband faster. Although at the end of the day I will probably put it in a separate PR so the changes are easily revertible. |
|
I've decided that there may have been something I misunderstood regarding the PR guidelines on Robust Book since the scope of this PR was originally going to be the tweak by itself. I would greatly appreciate advice on how to treat this PR going forward. |
|
How do I fix the outdated review? |
|
holy shit |
|
in my completely one hundred percent unbiased opinion, i think vulps should get this too since they lost thieving gloves as well. |
|
huh, skub i never saw any remove thieving from vulps pr but yeah i looked at the first pr and thieving is there. i think pickpocketing fits with felinids because they have a small size, but it should be easy to copy and paste the thieving component from felinid to vulp if you want to pr it. |
|
how do i fix do not merge, is it a balance concern |
|
thanks a bunch deltanedas |
…to thieving merged update to pr
|
any word on this? slimes can turn into geras and have minor internal storage now |
|
personally from a balance standpoint im fine with this, Especially considering other species have been buffed somewhat recently |
This reverts commit 9b763f3.
About the PR
This PR updates Felinids with a soft rework to their old passive thieving gloves ability. Now, Felinids have instead learned to see hidden items when stripping other mobs (i.e. items in pockets) and strip items faster in general (100% -> 66.67%). Admin Observers get this ability too.
Why / Balance
Since the removal of passive thieving gloves (see #666), Felinids cannot so much be called pickpockets anymore since they get little to their name that benefits such. Allowing Felinids to see into players' pockets and increasing stripping speed are parts of my suggested changes to work the pickpocket identity back into the Felinid kit while having implications for both antags and non-antags.
Passive stealth was indeed removed for a very sensible and good reason, but I would like to make the distinction that passive stealth enabled uncontested shittery since Felinids could simply have gloves, magboots or even belts disappear with little retaliation and RP value by extension, this comparison honestly isn't fair. This PR addresses this concern by leaving out stealth and only introducing pocket sight and strip speed buffs. I tried to implement this new set of changes with non/antag viability in mind. While old Felinids' passive thieving gloves let you skip on 4tc to rob people, this new iteration doesn't hide doafter which allows for retaliation. Any attempts made to steal, irrespective of success can and will go noticed by players (unless thieving gloves are used). But you can even combine thieves gloves with Felinid strip speed as an antag Felinid for a fastest thief gimmick (has similar energy to species specific traitor items). Pocket sight can give any player useful information and faster stripping has implications in both the Security department and everyday stealing.
At the end of the day, Felinids could likely be used by 'shitters' to steal an item or two. But remember that stripping is an action largely restricted to roleplay and non-combat scenarios. As such, this rebalance would not affect parts of the game an Oni's stat stick with accuracy penalty would (the average greytider will not use a gun, and benefits from increased crowbar/baseball damage). This buff will likely balance itself out over time to rising awareness and an anticipated 'Felinid = thief' stigma. Subversions of this stigma provides opportunity for roleplay.
Felinid pocket sight grants them a new way to comb information that's largely befitting of their roguish archetype. While in the past the same results could have been achieved with passive stealth, this ability creates agency regarding the value of an item and whether to actually attempt stealing it in the first place (especially since you no longer have passive stealth). Having information unique to a species also creates opportunity for 'boy who cried wolf' situations wherein a set of people decide on the merit of a Felinid's argument: "i saw an e-saber in his pocket!!!", as well as providing reason for Antags to distrust Felinids (nobody should trust Felinids).
While the PR could be left with just pocket sight, pockets only pertain to two item slots where one of them likely holds an emergency oxygen tank anyway. But most importantly pocket sight is likely to go unnoticed by a majority of players that do not play Felinids or look at the github / changelogs, since pocket sight rejects an established notion that you cannot see into another players' pocket. It almost feels metagamey, like an afterthought to mention it. Adding a strip speed buff is very much complementary to the pocket sight addition since it provides diegetic ground for players to acknowledge Felinids' pickpocket potential. While pocket sight only directly affects the user, faster stealing can be witnessed by all players. As for the numbers, 66.67% strip time brings down pocket/radio/shoes/hand strip times from (3 -> 2) seconds and backpack (6 -> 4) seconds. 2 seconds I've observed to be the reaction time of most players, so Felinids will be able to nip items more consistently. I chose multiplicative strip time as faster belt stripping has better implications for Security when searching for contraband. If one were to have their belt stolen, it is likely their fault for not paying attention and would have had it stolen anyway.
I talk to myself here regarding balance choices: https://discord.com/channels/968983104247185448/1228169033547776152
Technical details
Content.Client/.../StrippableBoundUserInterface.cs checks for ThievingComponent and its IgnoreStripHidden variable on the local entity before obscuring hidden items. Because this is client-sided I do not know if this is the safest implementation of this feature.
StripTimeMultiplier variable was also added to Thieving, which simply updates a preexisting multiplier argument that's passed to an event.
Media
https://youtu.be/cAVxXPUNImI?si=fBuKBQHGrzzJn03f
Breaking changes
Changelog
🆑