-
-
Notifications
You must be signed in to change notification settings - Fork 682
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Removes the "who" verb #10068
Removes the "who" verb #10068
Conversation
Wouldn't just logging the use of the verb solve 2/3 of these |
Yep. But I remember bringing this up in one of the coding channels like a year ago or so, and the answer I received by crossed among other people was that it makes 0 sense to log it. Simply because of how many false positives you'd get. On top of spamming the already fast moving admin alert chat. |
|
Okay, that doesn't mean killing it is the best option
Simple solution to that is just making a new INVESTIGATE_OOC category and making it log there. It wouldn't spam admin Chat and any records would still be accessible ingame and to outside log parsers. If 95% of the uses of this are non-malicious, the nuclear options seems overkill. |
As I said in the reply above, the issue is mostly all the false positives you'd get. People just using the who command to check who's on before they decide to play for example. It's just not worth it. Either we remove the who command, or we leave it in.
Something being possible or easy to do, doesn't mean we should enable it.
I'm aware that the total removal seems "nuclear", but like, what benefits does the who command actually give? I'm happy to be proven wrong, but I haven't been able to come up with a usecase yet that doesn't somewhat include "I want to see who's online before I play", which is something that I don't think should be encouraged by code (nor by administration for that matter). Like, we can have a long discussion here about what could be changed and a new OOC format to avoid admin chat spamming etc. , which are all decent suggestions. But, this is work and energy being put into something that brings.... what again? What does the who command give the server? Again, do players really need to know the ckey of every single connected person at all times? The crew manifest... exists... and is also IC. |
But its good for seeing if anyone you are friends with is online. Adding an optional private mode or friends list could be a nice alternative |
Well, the crew manifest still exists and can be used for that. But the actual question is do we really want that though. Your "friends" being online shouldn't be the deciding factor if you decide to play or not. Roleplay happens only if everyone is somewhat involved. Not if specific people roleplay with their specific group of friends. I might be wrong of course, but I feel like the crew manifest fullfils all the needs in this regard. And people who want to use a new character that isn't known yet to play "alone" for a bit, are also more hidden that way. (More hidden because now only their crew manifest name is shown and not their full ckey, which I know, will be shown anyways when they become an antagonist. But sometimes it's nice to just be a "blank" sheet again.) |
If you really need to play only while your friends are online, you still have the option to jab them on discord and go "Hey are you in round rn?" I don't think it hurts to remove |
Yea thats true, the only time I ever use anything like this is staffwho since it only shows me the relevant people online |
Considering we haven't really had any of these problems I see no point in removing the verb. It serves a real, but minor utility. If there is a problem with stealthmins showing just add a basic filter to hide stealthmins. |
Something not being a real issue yet, doesn't mean it can't be one in the future. I feel like that's kinda obvious.
Well, which one? The fact that it allows people to check if their friends are online? As I said in the beginning, that is something I don't think we should encourage. |
I mean, at the end of the day it's a game, and if people find people they like interacting with, so long as they aren't causing harm, then let them see if friends are on. It's really not a big deal. |
I mean what stops them from just using the ingame crew manifest? Like most of it is already there. I'm not a maintainer obviously, but I just have a hard time understanding why something that really does feel like a debug tool, is accessible to players. It has a list of potential problems and the only positive it has is already achieved by other ingame tools, that are a lot less powerful or can't potentially be abused. |
Because it's useful? I dunno, I use it sometimes to check if people are online. I don't know everyone's character names always and Who is always accessible while the manifest is not. I just don't find it to be strong enough of a push to remove it that someone might abuse it in any of the ways described. It's nothing the BYOND hub can't report and the harm is so minimal. |
Sometimes people don't want other's to know what their new character is. The crew manifest is completely sufficient in my opinion. It is what it is in the end, if you think it's fine as is then that's how it is. 🤷 |
If the alternative is to just use the crew manifest, then what's the point of removing the who verb? It would be better if byond gave us the option to respect people in invisible mode and redact their ckey |
What if we let people hide their ckey or use alias ckeys similar to how rainbow six lets you set your in-game username to be anything and numbers just get added to the end to make it unique. |
Well, all the points I listed in the first post pretty much. To me, the who verb has no positives (the crew manifest can be used as a way better replacement) and only downsides (as listed in the original PR body). That's why I don't really see why we should put in so much effort to just make the verb "good". There's just no use for the who verb. In the end you decide if you want to keep the who verb or not of course. I said what I think about it, and how the crew manifest does everything the who verb does but without all the downsides. |
I swear I have an informed argument against this PR but I haven't slept in 50 something hours so you can either try to extrapolate from my discord ranting or wait for me to do a more properly written response. Won't officially key it until then. |
I think all maintainers and a good amount of players are against this. As we discussed on discord, the who verb doesn't really come with any substantial benefits but also doesn't really come with any substantial downsides to keeping around. The majority of negative points are either inconsequential or an issue with other systems such as the stealthmin system. The only real issue would be if someone was scraping data on the lobby screen, however there are much easier ways to do that which are less noticable and easier. The who system is a nice convenience at times and I do see legitimate usecases for it since I have used it before. The reasons for removing are just simply not strong enough.
Don't get me wrong, removing it wouldn't be a terrible thing, but there are cases where I think it is handy to have and I have used it before. |
About The Pull Request
Removes the who verb completely.
Why It's Good For The Game
There's quite a couple reasons for why the "who" verb better gets removed. The who verb gives the person who uses it a complete list of every ckey currently connected to the server. This includes the ckeys stealthmins set themselves. Normal players should not have access to such a command. And admins have no use for it either.
The issues I currently see with the who command:
And all of this what for exactly? Players can already check who's on the station via the crew manifest, in the lobby. They absolutely do not need to know who's observing or just receive a full ckey list at all.
Then there's also the whole issue of "I only join when I see my metafriends being online" or "Let me see who's observing by comparing the who-list to the crew manifest, so that we can then do ghost roles together".
People joining only when their metafriends are online is an issue. And this change doesn't fix that, obviously. But I don't think it should be enabled that easily, on top of all the potential risks I pointed out above.
TLDR: The who command has a ton of downsides with virtually no benefits. Players do not need a zero cooldown full ckey list command, available from the lobby.
Testing Photographs and Procedure
Screenshots&Videos
I'm not sure what screenshot I should put here, I tested the PR and the command didn't autocomplete in the verb bar. When I typed it in manually also nothing happened.
Changelog
🆑
del: Removes the who verb
/:cl: