diff --git a/AdminStealth.cs b/AdminStealth.cs index 1351fd6..317484c 100644 --- a/AdminStealth.cs +++ b/AdminStealth.cs @@ -23,7 +23,7 @@ namespace AdminStealth; public class AdminStealth : BasePlugin { public override string ModuleName => "AdminStealth"; - public override string ModuleVersion => "0.3.15"; + public override string ModuleVersion => "0.3.13"; public override string ModuleAuthor => "astra"; public override string ModuleDescription => "Makes CS2-SimpleAdmin's css_hide look like leaving, and keeps it"; @@ -66,9 +66,6 @@ public class AdminStealth : BasePlugin // without ever calling us. foreach (var alias in _hideAliases) AddCommandListener(alias, (player, command) => OnHideTyped(player, command)); - // Players who may not hide get no reply at all, not even CSS's missing-permission one. Only - // a catch-all listener runs before every plugin's command handler, for console and chat. - AddCommandListener(null, OnAnyCommand); // Joining any team, spectators included, would make SimpleAdmin unhide them. AddCommandListener("jointeam", (player, command) => OnJoinTeam(player, command)); AddCommandListener("spectate", (player, command) => OnJoinTeam(player, command)); @@ -83,7 +80,6 @@ public class AdminStealth : BasePlugin _hideTyped.Clear(); _restoreScheduled.Clear(); _restoring.Clear(); - _closingTeamScreen.Clear(); }); } @@ -126,14 +122,6 @@ public class AdminStealth : BasePlugin return player == null ? HookResult.Continue : HookResult.Stop; } - private HookResult OnAnyCommand(CCSPlayerController? player, CommandInfo command) - { - if (player is not { IsValid: true, IsBot: false } || - !_hideAliases.Contains(command.GetArg(0), StringComparer.OrdinalIgnoreCase) || CanHide(player)) - return HookResult.Continue; - return HookResult.Stop; - } - private HookResult OnHideTyped(CCSPlayerController? player, CommandInfo command) { if (player is { IsValid: true } && !_restoring.Contains(player.Slot)) @@ -148,42 +136,20 @@ public class AdminStealth : BasePlugin player!.PrintToChat($" You're hidden, so you can't join a team. Type {ChatAlias()} to show yourself."); - CloseTeamScreen(player); - return HookResult.Stop; - } - - // M opens the team screen on the client, where the server can't block it, and a player with no - // team (as hidden admins are) can't close it without picking. Telling only their client they - // joined spectators doesn't close it (tried live), so do what css_hide does: really move them - // to spectators and back to no team, with the team screen disabled around it. Their team - // events aren't broadcast while hidden (OnPlayerTeam), and our SimpleAdmin fork doesn't unhide - // for spectators. - private readonly HashSet _closingTeamScreen = []; - - private void CloseTeamScreen(CCSPlayerController player) - { + // M opens the team screen on the client, where the server can't block it, and a player with + // no team (as hidden admins are) can't close it without picking. Answer the pick by telling + // only their client they joined spectators, with the team screen disabled around it, so + // nobody else ever sees them on a team. var slot = player.Slot; - if (!_closingTeamScreen.Add(slot)) - return; - Server.ExecuteCommand("sv_disable_teamselect_menu 1"); - Server.NextFrame(() => + AddTimer(0.3f, () => { var p = Utilities.GetPlayerFromSlot(slot); - if (p is { IsValid: true } && IsHidden(p) && p.TeamNum == (byte)CsTeam.None) - p.ChangeTeam(CsTeam.Spectator); - }); - AddTimer(0.15f, () => - { - var p = Utilities.GetPlayerFromSlot(slot); - if (p is { IsValid: true } && IsHidden(p) && p.TeamNum == (byte)CsTeam.Spectator) - p.ChangeTeam(CsTeam.None); - }, TimerFlags.STOP_ON_MAPCHANGE); - AddTimer(0.5f, () => - { - _closingTeamScreen.Remove(slot); - Server.ExecuteCommand("sv_disable_teamselect_menu 0"); + if (p is { IsValid: true } && IsHidden(p)) + SendTeamEvent(p, (int)CsTeam.Spectator, p.TeamNum); }, TimerFlags.STOP_ON_MAPCHANGE); + AddTimer(0.6f, () => Server.ExecuteCommand("sv_disable_teamselect_menu 0"), TimerFlags.STOP_ON_MAPCHANGE); + return HookResult.Stop; } private HookResult OnTeamMenu(CCSPlayerController? player, CommandInfo command) diff --git a/README.md b/README.md index edd38d6..5f82ae0 100644 --- a/README.md +++ b/README.md @@ -20,12 +20,9 @@ admin off the scoreboard. This plugin makes it look like they left the server: the team select screen doesn't open, until the restore hides them again. - The team select screen: the server's "team" menu message (`CS_UM_VGUIMenu`) is never sent to a hidden admin. M opens it on the client regardless, and a player with no team can't close it - without picking, so a pick moves them to spectators and straight back to no team, as `css_hide` - does (with `sv_disable_teamselect_menu` on around it), which closes it. The team events aren't - broadcast; for about 0.15 s they're on the spectator list of anyone looking at the scoreboard. + without picking, so a pick is answered by telling only that client it joined spectators + (with `sv_disable_teamselect_menu` on around it); nobody else sees a team change. - Restoring runs `css_hide` at most once: it toggles, so a second run could unhide them. -- Players who may not use `css_hide` (or its aliases) get no reply at all when they try it, from - console or chat: not even CSS's missing-permission message. Needs our SimpleAdmin fork (1.8.2b-zio2 or later), where moving to spectators doesn't unhide a hidden admin. Never hot-reload CS2-SimpleAdmin while this runs: CSS keeps handing out the old diff --git a/release.sh b/release.sh index 3c7228f..a3b6a11 100755 --- a/release.sh +++ b/release.sh @@ -19,13 +19,8 @@ set -euo pipefail FORGEJO_URL="https://git.zio.sh" OWNER="cs2" REPO="AdminStealth" -# Needs write:repository. Only uploading needs it; downloading from the public repo doesn't. From the -# environment, or else from .forgejo-token in the cs2/ folder above every plugin (outside their -# repos, mode 600). Don't commit a real token. -TOKEN_FILE="$(dirname "$(readlink -f "${BASH_SOURCE[0]}")")/../../.forgejo-token" -if [[ -z "${FORGEJO_TOKEN:-}" && -r "$TOKEN_FILE" ]]; then - FORGEJO_TOKEN="$(tr -d '[:space:]' < "$TOKEN_FILE")" -fi +# Placeholder - replace, or set FORGEJO_TOKEN in the environment instead. Needs write:repository. +# Only uploading needs it; downloading from the public repo doesn't. Don't commit a real token. FORGEJO_TOKEN="${FORGEJO_TOKEN:-CHANGE_ME_FORGEJO_TOKEN}" # Only these ship from the publish output. CounterStrikeSharp.API.dll is excluded by the csproj @@ -42,7 +37,7 @@ if [[ -z "$TAG" ]]; then exit 1 fi if [[ "$FORGEJO_TOKEN" == "CHANGE_ME_FORGEJO_TOKEN" ]]; then - echo "Set FORGEJO_TOKEN (export it, or put it in $TOKEN_FILE) first." >&2 + echo "Set FORGEJO_TOKEN (edit release.sh or export it) first." >&2 exit 1 fi for tool in podman jq curl git; do