Update comoss_d.yml

.editorconfig

@@ -77,8 +77,6 @@ csharp_style_expression_bodied_methods = false:suggestion
 #csharp_style_expression_bodied_operators = false:silent
 csharp_style_expression_bodied_properties = true:suggestion
 
-csharp_style_namespace_declarations = file_scoped:suggestion
-
 # Pattern matching preferences
 #csharp_style_pattern_matching_over_as_with_null_check = true:suggestion
 #csharp_style_pattern_matching_over_is_with_cast_check = true:suggestion

.github/CODEOWNERS

@@ -1,2 +1,6 @@
 # TheShuEd
 * @TheShuEd
+
+# TornadoTechnology
+*.cs @Tornado-Technology
+*.xaml @Tornado-Technology

.github/FUNDING.yml

@@ -1,14 +1,14 @@
 # These are supported funding model platforms
 
-#github: # Replace with up to 4 GitHub Sponsors-enabled usernames e.g., [user1, user2]
-#patreon: # Replace with a single Patreon username
-#open_collective: # Replace with a single Open Collective username
-#ko_fi: # Replace with a single Ko-fi username
-#tidelift: # Replace with a single Tidelift platform-name/package-name e.g., npm/babel
-#community_bridge: # Replace with a single Community Bridge project-name e.g., cloud-foundry
-#liberapay: # Replace with a single Liberapay username
-#issuehunt: # Replace with a single IssueHunt username
-#lfx_crowdfunding: # Replace with a single LFX Crowdfunding project-name e.g., cloud-foundry
-#polar: # Replace with a single Polar username
-#buy_me_a_coffee: # Replace with a single Buy Me a Coffee username
+github: # Replace with up to 4 GitHub Sponsors-enabled usernames e.g., [user1, user2]
+patreon: # Replace with a single Patreon username
+open_collective: # Replace with a single Open Collective username
+ko_fi: # Replace with a single Ko-fi username
+tidelift: # Replace with a single Tidelift platform-name/package-name e.g., npm/babel
+community_bridge: # Replace with a single Community Bridge project-name e.g., cloud-foundry
+liberapay: # Replace with a single Liberapay username
+issuehunt: # Replace with a single IssueHunt username
+lfx_crowdfunding: # Replace with a single LFX Crowdfunding project-name e.g., cloud-foundry
+polar: # Replace with a single Polar username
+buy_me_a_coffee: # Replace with a single Buy Me a Coffee username
 custom: ['https://boosty.to/theshued']

.github/ISSUE_TEMPLATE/config.yml

@@ -0,0 +1,7 @@
+contact_links:
+  - name: Report a Security Vulnerability
+    url: https://github.com/space-wizards/space-station-14/blob/master/SECURITY.md
+    about: Please report security vulnerabilities privately so we can fix them before they are publicly disclosed.
+  - name: Request a Feature
+    url: https://discord.gg/rGvu9hKffJ
+    about: Submit feature requests on our Discord server (https://discord.gg/rGvu9hKffJ).

.github/ISSUE_TEMPLATE/feature-request_en.yml

@@ -1,28 +0,0 @@
-name: "Request content or suggest an idea"
-description: "Suggest content, mechanics, or other changes that you think would benefit the project."
-labels: [suggestion]
-type: Suggestion
-body:
-  - type: textarea
-    id: concept
-    attributes:
-      label: Concept
-      placeholder: |
-        What is the nature of your proposal?
-
-        Example:
-        Add a new role ____ that performs tasks related to ____ and ____.
-    validations:
-      required: true
-
-  - type: textarea
-    id: motivation
-    attributes:
-      label: Why is this necessary?
-      placeholder: |
-        Explain how it would improve the game, or what problems it would fix. Suggestions without proper reasoning and reasons for the addition may be closed.
-
-        Example:
-        Currently, ____ often occurs in the round, which is why ____ is missing. This role could close this need by doing _____. 
-    validations:
-      required: true

.github/ISSUE_TEMPLATE/feature-request_ru.yml

@@ -1,28 +0,0 @@
-name: "Запросить контент или предложить идею"
-description: "Предложите контент, механику или другие изменения, которые вы считаете принесут пользу проекту."
-labels: [suggestion]
-type: Suggestion
-body:
-  - type: textarea
-    id: concept
-    attributes:
-      label: Концепция
-      placeholder: |
-        В чем заключается ваше предложение?
-
-        Пример:
-        Добавить новую роль ____, которая выполняет задачи связанные с ____ и ____. 
-    validations:
-      required: true
-
-  - type: textarea
-    id: motivation
-    attributes:
-      label: Зачем это нужно? 
-      placeholder: |
-        Объясните, как это улучшит игру, или какие проблемы это исправит. Предложения без должной аргументации и причины добавления могут быть закрыты.
-
-        Пример:
-        На текущий момент, в раунде часто происходит ____, из за чего не хватает ____. Эта роль могла бы закрыть эту потребность, выполняя _____. 
-    validations:
-      required: true

.github/ISSUE_TEMPLATE/issue_report.md

@@ -0,0 +1,20 @@
+---
+name: Report an Issue
+about: "..."
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+## Description
+<!-- Explain your issue in detail. Issues without proper explanation are liable to be closed by maintainers. -->
+
+**Reproduction**
+<!-- Include the steps to reproduce if applicable. -->
+
+**Screenshots**
+<!-- If applicable, add screenshots to help explain your problem. -->
+
+**Additional context**
+<!-- Add any other context about the problem here. Anything you think is related to the issue. -->

.github/ISSUE_TEMPLATE/issue_report_en.yml

@@ -1,49 +0,0 @@
-name: "Report an Issue"
-description: "Report a bug or problem you have found"
-type: Bug
-body:
-  - type: textarea
-    id: description
-    attributes:
-      label: "📄 Description"
-      description: |
-        Explain your issue in detail. Include what you expected to happen.
-      placeholder: |
-        When I click the "Save" button, nothing happens.
-    validations:
-      required: true
-
-  - type: textarea
-    id: steps
-    attributes:
-      label: "🔁 Reproduction Steps"
-      description: |
-        What steps lead to the problem? Try to list them clearly and precisely.
-      placeholder: |
-        1. Open the app
-        2. Go to Settings
-        3. Click "Save"
-    validations:
-      required: false
-
-  - type: textarea
-    id: screenshots
-    attributes:
-      label: "🖼️ Screenshots"
-      description: |
-        If applicable, add screenshots to help explain your problem.
-      placeholder: |
-        You can paste images here.
-    validations:
-      required: false
-
-  - type: textarea
-    id: context
-    attributes:
-      label: "ℹ️ Additional Context"
-      description: |
-        Any other context about the problem. Logs, version numbers, configs, etc.
-      placeholder: |
-        Occurs only on Android 11, using version 1.2.3.
-    validations:
-      required: false

.github/ISSUE_TEMPLATE/issue_report_ru.yml

@@ -1,50 +0,0 @@
-name: "Сообщить об ошибке"
-description: "Сообщите о найденной ошибке или проблеме"
-type: Bug
-body:
-  - type: textarea
-    id: description
-    attributes:
-      label: "📄 Описание"
-      description: |
-        Подробно опишите вашу проблему. Укажите, что должно было произойти.
-      placeholder: |
-        Когда я нажимаю на кнопку "Сохранить", ничего не происходит, и персонаж не сохраняется.
-    validations:
-      required: true
-
-  - type: textarea
-    id: steps
-    attributes:
-      label: "🔁 Шаги воспроизведения"
-      description: |
-        Какие шаги приводят к этой ошибке? Укажите их по порядку.
-      placeholder: |
-        1. Открыть приложение
-        2. Зайти в меню создания персонажа
-        3. Изменить цвет волос
-        4. Нажать "Сохранить"
-    validations:
-      required: false
-
-  - type: textarea
-    id: screenshots
-    attributes:
-      label: "🖼️ Скриншоты"
-      description: |
-        При наличии, добавьте скриншоты или изображения.
-      placeholder: |
-        Вы можете вставить изображения прямо сюда.
-    validations:
-      required: false
-
-  - type: textarea
-    id: context
-    attributes:
-      label: "ℹ️Дополнительная информация"
-      description: |
-        Дополнительные сведения: логи, версии, настройки и всё, что может быть связано.
-      placeholder: |
-        Возникает только на MacOS
-    validations:
-      required: false

.github/ISSUE_TEMPLATE/toolshed-feature-request.md

@@ -0,0 +1,18 @@
+---
+name: Toolshed feature request
+about: Suggest a feature for Toolshed (for game admins/developers)
+title: "[TOOLSHED REQUEST]"
+labels: Toolshed
+assignees: moonheart08
+
+---
+
+**Is your feature request related to a problem/bug? Please describe.**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the command you'd like**
+A clear and concise description of what you want and what it should do.
+If you're a technical user (i.e. programmer) including type signatures is helpful.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,23 +1,17 @@
 ## About the PR
 <!-- What did you change in this PR? -->
+<!-- Что вы изменили в своем пулл реквесте? -->
 
 ## Why / Balance
 <!-- Why was it changed? Link any discussions or issues here. Please discuss how this would affect game balance. -->
+<!-- Зачем нужно это изменение? Прикрепите любые обсуждения или проблемы здесь. Опишите, как это повлияет на текущий баланс игры. -->
 
 ## Media
 <!--
 PRs which make ingame changes (adding clothing, items, new features, etc) are required to have media attached that showcase the changes.
 Small fixes/refactors are exempt.
 -->
-
-**Changelog**
-<!-- Add a Changelog entry to make players aware of new features or changes that could affect gameplay.
-Make sure to read the guidelines and take this Changelog template out of the comment block in order for it to show up.
-Changelog must have a :cl: symbol, so the bot recognizes the changes and adds them to the game's changelog.
-
-:cl:
-- add: Added fun!
-- remove: Removed fun!
-- tweak: Changed fun!
-- fix: Fixed fun!
+<!--
+Пулл реквесты, которые несут за собой игровые изменения (добавления одежды, предметов и так далее) требуют чтобы вы прикрепили скриншоты или видеоролики, демонстрирующие эти изменения.
+Небольшие исправления не считаются.
 -->

.github/labeler.yml

@@ -1,34 +1,26 @@
-"Sprites":
+"Changes: Sprites":
 - changed-files:
   - any-glob-to-any-file: '**/*.rsi/*.png'
 
-"Mapping":
+"Changes: Map":
 - changed-files:
   - any-glob-to-any-file:
     - 'Resources/Maps/**/*.yml'
     - 'Resources/Prototypes/Maps/**/*.yml'
 
-"UI":
+"Changes: UI":
 - changed-files:
   - any-glob-to-any-file: '**/*.xaml*'
 
-"Shaders":
+"Changes: Shaders":
 - changed-files:
   - any-glob-to-any-file: '**/*.swsl'
 
-"Audio":
+"Changes: Audio":
 - changed-files:
   - any-glob-to-any-file: '**/*.ogg'
 
-"C#":
-- changed-files:
-  - any-glob-to-any-file: '**/*.cs'
-
-"YML":
-- changed-files:
-  - any-glob-to-any-file: '**/*.yml'
-
-"No C#":
+"Changes: No C#":
 - changed-files:
   # Equiv to any-glob-to-all as long as this has one matcher. If ALL changed files are not C# files, then apply label.
   - all-globs-to-all-files: "!**/*.cs"

.github/workflows/CP14Publish.yml

@@ -1,4 +1,4 @@
-name: Edge Publish
+name: Publish
 
 concurrency:
   group: publish
@@ -10,15 +10,6 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    
-    - name: Checkout repository
-      uses: actions/checkout@v4
-      with:
-        submodules: recursive
-    
-    - name: Install Python dependencies
-      run: sudo apt-get install -y python3-paramiko python3-lxml
-      
     - name: Send POST-request
       uses: appleboy/ssh-action@master
       with:
@@ -27,10 +18,3 @@ jobs:
         password: ${{ secrets.BUILD_PASS }}
         port: 22
         script: sh update.sh &> /dev/null
-
-    - name: Publish changelog (Discord)
-      continue-on-error: true
-      run: Tools/actions_changelogs_since_last_run.py
-      env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        DISCORD_WEBHOOK_URL: ${{ secrets.CHANGELOG_DISCORD_WEBHOOK }}

.github/workflows/build-docfx.yml

@@ -27,7 +27,7 @@ jobs:
       run: dotnet restore
 
     - name: Build Project
-      run: dotnet build --no-restore
+      run: dotnet build --no-restore /p:WarningsAsErrors=nullable
 
     - name: Build DocFX
       uses: nikeee/docfx-action@v1.0.0

.github/workflows/build-map-renderer.yml

@@ -42,7 +42,7 @@ jobs:
         run: dotnet restore
 
       - name: Build Project
-        run: dotnet build Content.MapRenderer --configuration Release --no-restore /m
+        run: dotnet build Content.MapRenderer --configuration Release --no-restore /p:WarningsAsErrors=nullable /m
 
       - name: Run Map Renderer
         run: dotnet run --project Content.MapRenderer Dev

.github/workflows/build-test-debug.yml

@@ -42,7 +42,7 @@ jobs:
       run: dotnet restore
 
     - name: Build Project
-      run: dotnet build --configuration DebugOpt --no-restore /m
+      run: dotnet build --configuration DebugOpt --no-restore /p:WarningsAsErrors=nullable /m
 
     - name: Run Content.Tests
       run: dotnet test --no-build --configuration DebugOpt Content.Tests/Content.Tests.csproj -- NUnit.ConsoleOut=0

.github/workflows/cla.yml

@@ -0,0 +1,38 @@
+name: "CLA Assistant"
+on:
+  issue_comment:
+    types: [created]
+  pull_request_target:
+    types: [opened,closed,synchronize]
+    paths:
+      - '**CP14**'
+
+# explicitly configure permissions, in case your GITHUB_TOKEN workflow permissions are set to read-only in repository settings
+permissions:
+  actions: write
+  contents: write # this can be 'read' if the signatures are in remote repository
+  pull-requests: write
+  statuses: write
+
+jobs:
+  CLAAssistant:
+    runs-on: ubuntu-latest
+    steps:
+      - name: "CLA Assistant"
+        if: ((github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the CLA Document and I hereby sign the CLA') || github.event_name == 'pull_request_target') && github.actor != 'TheShuEd'
+        uses: contributor-assistant/github-action@v2.6.1
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          # the below token should have repo scope and must be manually added by you in the repository's secret
+          # This token is required only if you have configured to store the signatures in a remote repository/organization
+          # PERSONAL_ACCESS_TOKEN: ${{ secrets.PERSONAL_ACCESS_TOKEN }}
+        with:
+          path-to-signatures: 'signatures/version1/cla.json'
+          path-to-document: 'https://github.com/crystallpunk-14/crystall-punk-14/blob/master/CLA.md' # e.g. a CLA or a DCO document
+          # branch should not be protected
+          branch: 'master'
+          allowlist: TheShuEd,bot*
+
+         # the followings are the optional inputs - If the optional inputs are not given, then default values will be taken
+          #custom-pr-sign-comment: 'The signature to be committed in order to sign the CLA'
+          #custom-allsigned-prcomment: 'pull request comment when all contributors has signed, defaults to **CLA Assistant Lite bot** All Contributors have signed the CLA.'

.github/workflows/labeler-needsreview.yml

@@ -2,7 +2,7 @@
 
 on:
   pull_request_target:
-    types: [review_requested, opened]
+    types: [review_requested]
 
 jobs:
   add_label:

.github/workflows/publish-testing.yml

@@ -2,7 +2,6 @@
 
 concurrency:
   group: publish-testing
-  cancel-in-progress: true
 
 on:
   workflow_dispatch:
@@ -34,7 +33,7 @@ jobs:
       run: dotnet build Content.Packaging --configuration Release --no-restore /m
 
     - name: Package server
-      run: dotnet run --project Content.Packaging server --platform win-x64 --platform win-arm64 --platform linux-x64 --platform linux-arm64 --platform osx-x64 --platform osx-arm64
+      run: dotnet run --project Content.Packaging server --platform win-x64 --platform linux-x64 --platform osx-x64 --platform linux-arm64
 
     - name: Package client
       run: dotnet run --project Content.Packaging client --no-wipe-release

.github/workflows/publish.yml

@@ -2,7 +2,6 @@ name: Publish
 
 concurrency:
   group: publish
-  cancel-in-progress: true
 
 on:
   workflow_dispatch:
@@ -14,10 +13,6 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - name: Fail if we are attempting to run on the master branch
-      if: ${{GITHUB.REF_NAME == 'master' && github.repository == 'space-wizards/space-station-14'}}
-      run: exit 1
-
     - name: Install dependencies
       run: sudo apt-get install -y python3-paramiko python3-lxml
 
@@ -41,7 +36,7 @@ jobs:
       run: dotnet build Content.Packaging --configuration Release --no-restore /m
 
     - name: Package server
-      run: dotnet run --project Content.Packaging server --platform win-x64 --platform win-arm64 --platform linux-x64 --platform linux-arm64 --platform osx-x64 --platform osx-arm64
+      run: dotnet run --project Content.Packaging server --platform win-x64 --platform linux-x64 --platform osx-x64 --platform linux-arm64
 
     - name: Package client
       run: dotnet run --project Content.Packaging client --no-wipe-release
@@ -53,14 +48,12 @@ jobs:
         GITHUB_REPOSITORY: ${{ vars.GITHUB_REPOSITORY }}
 
     - name: Publish changelog (Discord)
-      continue-on-error: true
       run: Tools/actions_changelogs_since_last_run.py
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         DISCORD_WEBHOOK_URL: ${{ secrets.CHANGELOG_DISCORD_WEBHOOK }}
 
     - name: Publish changelog (RSS)
-      continue-on-error: true
       run: Tools/actions_changelog_rss.py
       env:
         CHANGELOG_RSS_KEY: ${{ secrets.CHANGELOG_RSS_KEY }}

.github/workflows/test-packaging.yml

@@ -60,7 +60,7 @@ jobs:
         run: dotnet build Content.Packaging --configuration Release --no-restore /m
 
       - name: Package server
-        run: dotnet run --project Content.Packaging server --platform win-x64 --platform win-arm64 --platform linux-x64 --platform linux-arm64 --platform osx-x64 --platform osx-arm64
+        run: dotnet run --project Content.Packaging server --platform win-x64 --platform linux-x64 --platform osx-x64 --platform linux-arm64
 
       - name: Package client
         run: dotnet run --project Content.Packaging client --no-wipe-release

BuildChecker/git_helper.py

@@ -1,19 +1,16 @@
 #!/usr/bin/env python3
-"""
-Installs git hooks, updates them, updates submodules, that kind of thing.
-"""
+# Installs git hooks, updates them, updates submodules, that kind of thing.
 
-import os
-import shutil
 import subprocess
 import sys
-import time
+import os
+import shutil
 from pathlib import Path
 from typing import List
 
 SOLUTION_PATH = Path("..") / "SpaceStation14.sln"
 # If this doesn't match the saved version we overwrite them all.
-CURRENT_HOOKS_VERSION = "3"
+CURRENT_HOOKS_VERSION = "2"
 QUIET = len(sys.argv) == 2 and sys.argv[1] == "--quiet"
 
 
@@ -27,10 +24,12 @@ def run_command(command: List[str], capture: bool = False) -> subprocess.Complet
 
     sys.stdout.flush()
 
+    completed = None
+
     if capture:
-        completed = subprocess.run(command, stdout=subprocess.PIPE, text=True)
+        completed = subprocess.run(command, cwd="..", stdout=subprocess.PIPE)
     else:
-        completed = subprocess.run(command)
+        completed = subprocess.run(command, cwd="..")
 
     if completed.returncode != 0:
         print("Error: command exited with code {}!".format(completed.returncode))
@@ -43,7 +42,7 @@ def update_submodules():
     Updates all submodules.
     """
 
-    if 'GITHUB_ACTIONS' in os.environ:
+    if ('GITHUB_ACTIONS' in os.environ):
         return
 
     if os.path.isfile("DISABLE_SUBMODULE_AUTOUPDATE"):
@@ -76,21 +75,22 @@ def install_hooks():
                     print("No hooks change detected.")
                 return
 
+    with open("INSTALLED_HOOKS_VERSION", "w") as f:
+        f.write(CURRENT_HOOKS_VERSION)
+
     print("Hooks need updating.")
 
-    hooks_target_dir = Path(run_command(["git", "rev-parse", "--git-path", "hooks"], True).stdout.strip())
+    hooks_target_dir = Path("..")/".git"/"hooks"
     hooks_source_dir = Path("hooks")
 
     # Clear entire tree since we need to kill deleted files too.
-    for filename in os.listdir(hooks_target_dir):
-        os.remove(hooks_target_dir / filename)
+    for filename in os.listdir(str(hooks_target_dir)):
+        os.remove(str(hooks_target_dir/filename))
 
-    for filename in os.listdir(hooks_source_dir):
+    for filename in os.listdir(str(hooks_source_dir)):
         print("Copying hook {}".format(filename))
-        shutil.copy2(hooks_source_dir / filename, hooks_target_dir / filename)
-
-    with open("INSTALLED_HOOKS_VERSION", "w") as f:
-        f.write(CURRENT_HOOKS_VERSION)
+        shutil.copy2(str(hooks_source_dir/filename),
+                        str(hooks_target_dir/filename))
 
 
 def reset_solution():
@@ -104,20 +104,7 @@ def reset_solution():
     with SOLUTION_PATH.open("w") as f:
         f.write(content)
 
-def check_for_zip_download():
-    # Check if .git exists,
-    if run_command(["git", "rev-parse"]).returncode != 0:
-        print("It appears that you downloaded this repository directly from GitHub. (Using the .zip download option) \n"
-              "When downloading straight from GitHub, it leaves out important information that git needs to function. "
-              "Such as information to download the engine or even the ability to even be able to create contributions. \n"
-              "Please read and follow https://docs.spacestation14.com/en/general-development/setup/setting-up-a-development-environment.html \n"
-              "If you just want a Sandbox Server, you are following the wrong guide! You can download a premade server following the instructions here:"
-              "https://docs.spacestation14.com/en/general-development/setup/server-hosting-tutorial.html \n"
-              "Closing automatically in 30 seconds.")
-        time.sleep(30)
-        exit(1)
 
 if __name__ == '__main__':
-    check_for_zip_download()
     install_hooks()
     update_submodules()

BuildChecker/hooks/post-checkout

@@ -1,10 +1,10 @@
 #!/bin/bash
 
-gitroot=$(git rev-parse --show-toplevel)
+gitroot=`git rev-parse --show-toplevel`
 
-cd "$gitroot/BuildChecker" || exit
+cd "$gitroot/BuildChecker"
 
-if [[ $(uname) == MINGW* || $(uname) == CYGWIN* ]]; then
+if [[ `uname` == MINGW* || `uname` == CYGWIN* ]]; then
     # Windows
     py -3 git_helper.py --quiet
 else

BuildChecker/hooks/post-merge

@@ -1,5 +1,5 @@
 #!/bin/bash
 
 # Just call post-checkout since it does the same thing.
-gitroot=$(git rev-parse --git-path hooks)
-bash "$gitroot/post-checkout"
+gitroot=`git rev-parse --show-toplevel`
+bash "$gitroot/.git/hooks/post-checkout"

CODE_OF_CONDUCT.md

@@ -1,40 +0,0 @@
-# Space Station 14 Code of Conduct
-
-Space Station 14's staff and community is made up volunteers from all over the world, working on every aspect of the project - including development, teaching, and hosting integral tools.
-
-Diversity is one of our huge strengths, but it can also lead to communication issues and unhappiness. To that end, we have a few ground rules that we ask people to adhere to. This code applies equally to all levels of the project, from commenters to contributors to staff.
-
-This isn’t an exhaustive list of things that you can’t do. Rather, take it in the spirit in which it’s intended - a guide to make it easier to enrich all of us and the technical communities in which we participate.
-
-This code of conduct applies specifically to the Github repositories and its spaces managed by the Space Station 14 project or Space Wizards Federation. Some spaces, such as the Space Station 14 Discord or the official Wizard's Den game servers, have their own rules but are in spirit equal to what may be found in here.
-
-If you believe someone is violating the code of conduct, we ask that you report it by contacting a Maintainer, Project Manager or Wizard staff member through [Discord](https://discord.ss14.io/), [the forums](https://forum.spacestation14.com/), or emailing [telecommunications@spacestation14.com](mailto:telecommunications@spacestation14.com).
-
-- **Be friendly and patient.**
-- **Be welcoming.** We strive to be a community that welcomes and supports people of all backgrounds and identities. This includes, but is not limited to members of any race, ethnicity, culture, national origin, colour, immigration status, social and economic class, educational level, sex, sexual orientation, gender identity and expression, age, size, family status, political belief, religion, and mental and physical ability.
-- **Be considerate.** Your work will be used by other people, and you in turn will depend on the work of others. Any decision you take will affect users and contributors, and you should take those consequences into account when making decisions. Remember that we're a world-wide community, so you might not be communicating in someone else's primary language. We have contributors of all skill levels, some even making their first foray into a new field with this project, so keep that in mind when discussing someone's work.
-- **Be respectful.** Not all of us will agree all the time, but disagreement is no excuse for poor behavior and poor manners. We might all experience some frustration now and then, but we cannot allow that frustration to turn into a personal attack. It’s important to remember that a community where people feel uncomfortable or threatened is not a productive one. Members of the Space Station 14 community should be respectful when dealing with other members as well as with people outside the Space Station 14 community. Assume contributions to the project, even those that do not end up being included, are made in good faith.
-- **Be careful in the words that you choose.** We are a community of professionals, and we conduct ourselves professionally. Be kind to others. Do not insult or put down other participants. Harassment and other exclusionary behavior aren't acceptable. This includes, but is not limited to:
-  - Violent threats or language directed against another person.
-  - Discriminatory jokes and language.
-  - Posting sexually explicit or violent material.
-  - Posting (or threatening to post) other people's personally identifying information ("doxing").
-  - Personal insults, especially those using racist or sexist terms.
-  - Unwelcome sexual attention.
-  - Advocating for, or encouraging, any of the above behavior.
-  - Repeated harassment of others. In general, if someone asks you to stop, then stop.
-- **When we disagree, try to understand why.** Disagreements, both social and technical, happen all the time and Space Station 14 is no exception. It is important that we resolve disagreements and differing views constructively. Remember that we’re different. The strength of Space Station 14 comes from its varied community, people from a wide range of backgrounds. Different people have different perspectives on issues. Being unable to understand why someone holds a viewpoint doesn’t mean that they’re wrong. Don’t forget that it is human to make mistakes and blaming each other doesn’t get us anywhere. Instead, focus on helping to resolve issues and learning from mistakes.
-
-Original text courtesy of the [Speak Up! project](http://web.archive.org/web/20141109123859/http://speakup.io/coc.html).
-
-## On Community Moderation
-
-Deviating from the Code of Conduct on the Github repository may result in moderative actions taken by project Maintainers. This can involve your content being edited or deleted, and may result in a temporary or permanent block from the repository.
-
-This is to ensure Space Station 14 is a healthy community in which contributors feel encouraged and empowered to contribute, and to give you as a member of this community a chance to reflect on how you are interacting with it. While outright offensive and bigoted content will *always* be unacceptable on the repository, Maintainers are at liberty to take moderative actions against more ambiguous content that fail to provide constructive criticism, or that provides constructive criticism in a non-constructive manner. Examples of this include using hyperbole, bringing up PRs/changes unrelated to the discussion at hand, hostile tone, off-topic comments, creating PRs/Issues for the sole purpose of causing discussions, skirting the line of acceptable behavior, etc. Disagreeing with content or each other is fine and appreciated, but only as long as it's done with respect and in a constructive manner.
-
-Maintainers are expected to adhere to the guidelines as listed in the [Github Moderation Guidelines](https://docs.spacestation14.com/en/general-development/github-moderation-guidelines.html), though may deviate should they feel it's in the best interest of the community. If you believe you had an action incorrectly applied against you, you are encouraged to contact staff via [Discord](https://discord.ss14.io/) or [the forums](https://forum.spacestation14.com/), [appeal your Github ban](https://forum.spacestation14.com/c/ban-appeals/appeals-github/38), or make a [staff complaint](https://forum.spacestation14.com/t/staff-complaint-instructions-and-info/31).
-
-## Attribution
-
-This Code of Conduct is an edited version of the [Django Code of Conduct](https://www.djangoproject.com/conduct/), licensed under CC BY 3.0, for the Space Station 14 Github repository.

Content.Benchmarks/ComponentQueryBenchmark.cs

@@ -29,7 +29,7 @@ namespace Content.Benchmarks;
 [CategoriesColumn]
 public class ComponentQueryBenchmark
 {
-    public const string Map = "Maps/saltern.yml";
+    public const string Map = "Maps/atlas.yml";
 
     private TestPair _pair = default!;
     private IEntityManager _entMan = default!;

Content.Benchmarks/DeltaPressureBenchmark.cs

@@ -1,174 +0,0 @@
-using System.Threading.Tasks;
-using BenchmarkDotNet.Attributes;
-using BenchmarkDotNet.Diagnosers;
-using Content.IntegrationTests;
-using Content.IntegrationTests.Pair;
-using Content.Server.Atmos.Components;
-using Content.Server.Atmos.EntitySystems;
-using Content.Shared.Atmos.Components;
-using Content.Shared.CCVar;
-using Robust.Shared;
-using Robust.Shared.Analyzers;
-using Robust.Shared.Configuration;
-using Robust.Shared.GameObjects;
-using Robust.Shared.Map;
-using Robust.Shared.Map.Components;
-using Robust.Shared.Maths;
-using Robust.Shared.Prototypes;
-using Robust.Shared.Random;
-
-namespace Content.Benchmarks;
-
-/// <summary>
-/// Spawns N number of entities with a <see cref="DeltaPressureComponent"/> and
-/// simulates them for a number of ticks M.
-/// </summary>
-[Virtual]
-[GcServer(true)]
-//[MemoryDiagnoser]
-//[ThreadingDiagnoser]
-public class DeltaPressureBenchmark
-{
-    /// <summary>
-    /// Number of entities (windows, really) to spawn with a <see cref="DeltaPressureComponent"/>.
-    /// </summary>
-    [Params(1, 10, 100, 1000, 5000, 10000, 50000, 100000)]
-    public int EntityCount;
-
-    /// <summary>
-    /// Number of entities that each parallel processing job will handle.
-    /// </summary>
-    // [Params(1, 10, 100, 1000, 5000, 10000)] For testing how multithreading parameters affect performance (THESE TESTS TAKE 16+ HOURS TO RUN)
-    [Params(10)]
-    public int BatchSize;
-
-    /// <summary>
-    /// Number of entities to process per iteration in the DeltaPressure
-    /// processing loop.
-    /// </summary>
-    // [Params(100, 1000, 5000, 10000, 50000)]
-    [Params(1000)]
-    public int EntitiesPerIteration;
-
-    private readonly EntProtoId _windowProtoId = "Window";
-    private readonly EntProtoId _wallProtoId = "WallPlastitaniumIndestructible";
-
-    private TestPair _pair = default!;
-    private IEntityManager _entMan = default!;
-    private SharedMapSystem _map = default!;
-    private IRobustRandom _random = default!;
-    private IConfigurationManager _cvar = default!;
-    private ITileDefinitionManager _tileDefMan = default!;
-    private AtmosphereSystem _atmospereSystem = default!;
-
-    private Entity<GridAtmosphereComponent, GasTileOverlayComponent, MapGridComponent, TransformComponent>
-        _testEnt;
-
-    [GlobalSetup]
-    public async Task SetupAsync()
-    {
-        ProgramShared.PathOffset = "../../../../";
-        PoolManager.Startup();
-        _pair = await PoolManager.GetServerClient();
-        var server = _pair.Server;
-
-        var mapdata = await _pair.CreateTestMap();
-
-        _entMan = server.ResolveDependency<IEntityManager>();
-        _map = _entMan.System<SharedMapSystem>();
-        _random = server.ResolveDependency<IRobustRandom>();
-        _cvar = server.ResolveDependency<IConfigurationManager>();
-        _tileDefMan = server.ResolveDependency<ITileDefinitionManager>();
-        _atmospereSystem = _entMan.System<AtmosphereSystem>();
-
-        _random.SetSeed(69420); // Randomness needs to be deterministic for benchmarking.
-
-        _cvar.SetCVar(CCVars.DeltaPressureParallelToProcessPerIteration, EntitiesPerIteration);
-        _cvar.SetCVar(CCVars.DeltaPressureParallelBatchSize, BatchSize);
-
-        var plating = _tileDefMan["Plating"].TileId;
-
-        /*
-         Basically, we want to have a 5-wide grid of tiles.
-         Edges are walled, and the length of the grid is determined by N + 2.
-         Windows should only touch the top and bottom walls, and each other.
-         */
-
-        var length = EntityCount + 2; // ensures we can spawn exactly N windows between side walls
-        const int height = 5;
-
-        await server.WaitPost(() =>
-        {
-            // Fill required tiles (extend grid) with plating
-            for (var x = 0; x < length; x++)
-            {
-                for (var y = 0; y < height; y++)
-                {
-                    _map.SetTile(mapdata.Grid, mapdata.Grid, new Vector2i(x, y), new Tile(plating));
-                }
-            }
-
-            // Spawn perimeter walls and windows row in the middle (y = 2)
-            const int midY = height / 2;
-            for (var x = 0; x < length; x++)
-            {
-                for (var y = 0; y < height; y++)
-                {
-                    var coords = new EntityCoordinates(mapdata.Grid, x + 0.5f, y + 0.5f);
-
-                    var isPerimeter = x == 0 || x == length - 1 || y == 0 || y == height - 1;
-                    if (isPerimeter)
-                    {
-                        _entMan.SpawnEntity(_wallProtoId, coords);
-                        continue;
-                    }
-
-                    // Spawn windows only on the middle row, spanning interior (excluding side walls)
-                    if (y == midY)
-                    {
-                        _entMan.SpawnEntity(_windowProtoId, coords);
-                    }
-                }
-            }
-        });
-
-        // Next we run the fixgridatmos command to ensure that we have some air on our grid.
-        // Wait a little bit as well.
-        // TODO: Unhardcode command magic string when fixgridatmos is an actual command we can ref and not just
-        // a stamp-on in AtmosphereSystem.
-        await _pair.WaitCommand("fixgridatmos " + mapdata.Grid.Owner, 1);
-
-        var uid = mapdata.Grid.Owner;
-        _testEnt = new Entity<GridAtmosphereComponent, GasTileOverlayComponent, MapGridComponent, TransformComponent>(
-            uid,
-            _entMan.GetComponent<GridAtmosphereComponent>(uid),
-            _entMan.GetComponent<GasTileOverlayComponent>(uid),
-            _entMan.GetComponent<MapGridComponent>(uid),
-            _entMan.GetComponent<TransformComponent>(uid));
-    }
-
-    [Benchmark]
-    public async Task PerformFullProcess()
-    {
-        await _pair.Server.WaitPost(() =>
-        {
-            while (!_atmospereSystem.RunProcessingStage(_testEnt, AtmosphereProcessingState.DeltaPressure)) { }
-        });
-    }
-
-    [Benchmark]
-    public async Task PerformSingleRunProcess()
-    {
-        await _pair.Server.WaitPost(() =>
-        {
-            _atmospereSystem.RunProcessingStage(_testEnt, AtmosphereProcessingState.DeltaPressure);
-        });
-    }
-
-    [GlobalCleanup]
-    public async Task CleanupAsync()
-    {
-        await _pair.DisposeAsync();
-        PoolManager.Shutdown();
-    }
-}

Content.Benchmarks/MapLoadBenchmark.cs

@@ -47,7 +47,7 @@ public class MapLoadBenchmark
         PoolManager.Shutdown();
     }
 
-    public static string[] MapsSource { get; } = { "Empty", "Saltern", "Box", "Bagel", "Dev", "CentComm", "Core", "TestTeg", "Packed", "Omega", "Reach", "Meta", "Marathon", "MeteorArena", "Fland", "Oasis", "Convex"};
+    public static readonly string[] MapsSource = { "Empty", "Satlern", "Box", "Bagel", "Dev", "CentComm", "Core", "TestTeg", "Packed", "Omega", "Reach", "Meta", "Marathon", "MeteorArena", "Fland", "Oasis", "Convex"};
 
     [ParamsSource(nameof(MapsSource))]
     public string Map;

Content.Benchmarks/Program.cs

@@ -1,7 +1,15 @@
 using System;
+using System.Linq;
+using System.Threading.Tasks;
 using BenchmarkDotNet.Running;
+using Content.IntegrationTests;
+using Content.Server.Maps;
+#if DEBUG
 using BenchmarkDotNet.Configs;
+#else
 using Robust.Benchmarks.Configs;
+#endif
+using Robust.Shared.Prototypes;
 
 namespace Content.Benchmarks
 {
@@ -14,15 +22,11 @@ namespace Content.Benchmarks
             Console.ForegroundColor = ConsoleColor.Red;
             Console.WriteLine("\nWARNING: YOU ARE RUNNING A DEBUG BUILD, USE A RELEASE BUILD FOR AN ACCURATE BENCHMARK");
             Console.WriteLine("THE DEBUG BUILD IS ONLY GOOD FOR FIXING A CRASHING BENCHMARK\n");
-            var baseConfig = new DebugInProcessConfig();
+            BenchmarkSwitcher.FromAssembly(typeof(Program).Assembly).Run(args, new DebugInProcessConfig());
 #else
-            var baseConfig = Environment.GetEnvironmentVariable("ROBUST_BENCHMARKS_ENABLE_SQL") != null
-                ? DefaultSQLConfig.Instance
-                : DefaultConfig.Instance;
-#endif
-            var config = ManualConfig.Create(baseConfig);
-            config.BuildTimeout = TimeSpan.FromMinutes(5);
+            var config = Environment.GetEnvironmentVariable("ROBUST_BENCHMARKS_ENABLE_SQL") != null ? DefaultSQLConfig.Instance : null;
             BenchmarkSwitcher.FromAssembly(typeof(Program).Assembly).Run(args, config);
+#endif
         }
     }
 }

Content.Benchmarks/PvsBenchmark.cs

@@ -6,6 +6,7 @@ using BenchmarkDotNet.Attributes;
 using Content.IntegrationTests;
 using Content.IntegrationTests.Pair;
 using Content.Server.Mind;
+using Content.Server.Warps;
 using Content.Shared.Warps;
 using Robust.Shared;
 using Robust.Shared.Analyzers;

Content.Benchmarks/RaiseEventBenchmark.cs

@@ -1,126 +0,0 @@
-#nullable enable
-using System.Runtime.CompilerServices;
-using System.Threading.Tasks;
-using BenchmarkDotNet.Attributes;
-using Content.IntegrationTests;
-using Content.IntegrationTests.Pair;
-using Robust.Shared;
-using Robust.Shared.Analyzers;
-using Robust.Shared.GameObjects;
-
-namespace Content.Benchmarks;
-
-[Virtual]
-public class RaiseEventBenchmark
-{
-    private TestPair _pair = default!;
-    private BenchSystem _sys = default!;
-
-    [GlobalSetup]
-    public void Setup()
-    {
-        ProgramShared.PathOffset = "../../../../";
-        PoolManager.Startup(typeof(BenchSystem).Assembly);
-        _pair = PoolManager.GetServerClient().GetAwaiter().GetResult();
-        var entMan = _pair.Server.EntMan;
-        _sys = entMan.System<BenchSystem>();
-
-        _pair.Server.WaitPost(() =>
-        {
-            var uid = entMan.Spawn();
-            _sys.Ent = new(uid, entMan.GetComponent<TransformComponent>(uid));
-            _sys.Ent2 = new(_sys.Ent.Owner, _sys.Ent.Comp);
-        })
-            .GetAwaiter()
-            .GetResult();
-    }
-
-    [GlobalCleanup]
-    public async Task Cleanup()
-    {
-        await _pair.DisposeAsync();
-        PoolManager.Shutdown();
-    }
-
-    [Benchmark(Baseline = true)]
-    public int RaiseEvent()
-    {
-        return _sys.RaiseEvent();
-    }
-
-    [Benchmark]
-    public int RaiseCompEvent()
-    {
-        return _sys.RaiseCompEvent();
-    }
-
-    [Benchmark]
-    public int RaiseICompEvent()
-    {
-        return _sys.RaiseICompEvent();
-    }
-
-    [Benchmark]
-    public int RaiseCSharpEvent()
-    {
-        return _sys.CSharpEvent();
-    }
-
-    public sealed class BenchSystem : EntitySystem
-    {
-        public Entity<TransformComponent> Ent;
-        public Entity<IComponent> Ent2;
-
-        public delegate void EntityEventHandler(EntityUid uid, TransformComponent comp, ref BenchEv ev);
-
-        public event EntityEventHandler? OnCSharpEvent;
-
-        public override void Initialize()
-        {
-            base.Initialize();
-            SubscribeLocalEvent<TransformComponent, BenchEv>(OnEvent);
-            OnCSharpEvent += OnEvent;
-        }
-
-        public int RaiseEvent()
-        {
-            var ev = new BenchEv();
-            RaiseLocalEvent(Ent.Owner, ref ev);
-            return ev.N;
-        }
-
-        public int RaiseCompEvent()
-        {
-            var ev = new BenchEv();
-            EntityManager.EventBus.RaiseComponentEvent(Ent.Owner, Ent.Comp, ref ev);
-            return ev.N;
-        }
-
-        public int RaiseICompEvent()
-        {
-            // Raise with an IComponent instead of concrete type
-            var ev = new BenchEv();
-            EntityManager.EventBus.RaiseComponentEvent(Ent2.Owner, Ent2.Comp, ref ev);
-            return ev.N;
-        }
-
-        public int CSharpEvent()
-        {
-            var ev = new BenchEv();
-            OnCSharpEvent?.Invoke(Ent.Owner, Ent.Comp, ref ev);
-            return ev.N;
-        }
-
-        [MethodImpl(MethodImplOptions.NoInlining)]
-        private void OnEvent(EntityUid uid, TransformComponent component, ref BenchEv args)
-        {
-            args.N += uid.Id;
-        }
-
-        [ByRefEvent]
-        public struct BenchEv
-        {
-            public int N;
-        }
-    }
-}

Content.Benchmarks/SpawnEquipDeleteBenchmark.cs

@@ -8,7 +8,6 @@ using Robust.Shared;
 using Robust.Shared.Analyzers;
 using Robust.Shared.GameObjects;
 using Robust.Shared.Map;
-using Robust.Shared.Prototypes;
 
 namespace Content.Benchmarks;
 
@@ -19,11 +18,9 @@ namespace Content.Benchmarks;
 [Virtual, MemoryDiagnoser]
 public class SpawnEquipDeleteBenchmark
 {
-    private static readonly EntProtoId Mob = "MobHuman";
-    private static readonly ProtoId<StartingGearPrototype> CaptainStartingGear = "CaptainGear";
-
     private TestPair _pair = default!;
     private StationSpawningSystem _spawnSys = default!;
+    private const string Mob = "MobHuman";
     private StartingGearPrototype _gear = default!;
     private EntityUid _entity;
     private EntityCoordinates _coords;
@@ -42,7 +39,7 @@ public class SpawnEquipDeleteBenchmark
         var mapData = await _pair.CreateTestMap();
         _coords = mapData.GridCoords;
         _spawnSys = server.System<StationSpawningSystem>();
-        _gear = server.ProtoMan.Index(CaptainStartingGear);
+        _gear = server.ProtoMan.Index<StartingGearPrototype>("CaptainGear");
     }
 
     [GlobalCleanup]

Content.Client/Access/Commands/ShowAccessReadersCommand.cs

@@ -4,20 +4,39 @@ using Robust.Shared.Console;
 
 namespace Content.Client.Access.Commands;
 
-public sealed class ShowAccessReadersCommand : LocalizedEntityCommands
+public sealed class ShowAccessReadersCommand : IConsoleCommand
 {
-    [Dependency] private readonly IOverlayManager _overlay = default!;
-    [Dependency] private readonly IResourceCache _cache = default!;
-    [Dependency] private readonly SharedTransformSystem _xform = default!;
+    public string Command => "showaccessreaders";
 
-    public override string Command => "showaccessreaders";
-
-    public override void Execute(IConsoleShell shell, string argStr, string[] args)
+    public string Description => "Toggles showing access reader permissions on the map";
+    public string Help => """
+        Overlay Info:
+        -Disabled | The access reader is disabled
+        +Unrestricted | The access reader has no restrictions
+        +Set [Index]: [Tag Name]| A tag in an access set (accessor needs all tags in the set to be allowed by the set)
+        +Key [StationUid]: [StationRecordKeyId] | A StationRecordKey that is allowed
+        -Tag [Tag Name] | A tag that is not allowed (takes priority over other allows)
+        """;
+    public void Execute(IConsoleShell shell, string argStr, string[] args)
     {
-        var existing = _overlay.RemoveOverlay<AccessOverlay>();
-        if (!existing)
-            _overlay.AddOverlay(new AccessOverlay(EntityManager, _cache, _xform));
+        var collection = IoCManager.Instance;
 
-        shell.WriteLine(Loc.GetString($"cmd-showaccessreaders-status", ("status", !existing)));
+        if (collection == null)
+            return;
+
+        var overlay = collection.Resolve<IOverlayManager>();
+
+        if (overlay.RemoveOverlay<AccessOverlay>())
+        {
+            shell.WriteLine($"Set access reader debug overlay to false");
+            return;
+        }
+
+        var entManager = collection.Resolve<IEntityManager>();
+        var cache = collection.Resolve<IResourceCache>();
+        var xform = entManager.System<SharedTransformSystem>();
+
+        overlay.AddOverlay(new AccessOverlay(entManager, cache, xform));
+        shell.WriteLine($"Set access reader debug overlay to true");
     }
 }

Content.Client/Access/UI/AccessOverriderWindow.xaml.cs

@@ -29,7 +29,7 @@ namespace Content.Client.Access.UI
 
             foreach (var access in accessLevels)
             {
-                if (!protoManager.Resolve(access, out var accessLevel))
+                if (!protoManager.TryIndex(access, out var accessLevel))
                 {
                     continue;
                 }

Content.Client/Access/UI/GroupedAccessLevelChecklist.xaml

@@ -1,26 +0,0 @@
-<BoxContainer xmlns="https://spacestation14.io"
-              xmlns:gfx="clr-namespace:Robust.Client.Graphics;assembly=Robust.Client"
-              Orientation="Horizontal"
-              Margin="10 10 10 10"
-              VerticalExpand="True"
-              HorizontalExpand="True"
-              MinHeight="70">
-
-    <!-- Access groups -->
-    <BoxContainer Name="AccessGroupList" Access="Public" Orientation="Vertical" HorizontalExpand="True" SizeFlagsStretchRatio="0.5" Margin="0 0 10 0">
-        <!-- Populated with C# code -->
-    </BoxContainer>
-
-    <PanelContainer StyleClasses="LowDivider" VerticalExpand="True" Margin="0 0 0 0" SetWidth="2">
-        <PanelContainer.PanelOverride>
-            <gfx:StyleBoxFlat BackgroundColor="#FFFFFF" />
-        </PanelContainer.PanelOverride>
-    </PanelContainer>
-
-    <!-- Access levels -->
-    <ScrollContainer HorizontalExpand="True" VerticalExpand="True" Margin="10 0 0 0">
-        <BoxContainer Name="AccessLevelChecklist" Access="Public" Orientation="Vertical" HorizontalAlignment="Left">
-            <!-- Populated with C# code -->
-        </BoxContainer>
-    </ScrollContainer>
-</BoxContainer>

Content.Client/Access/UI/GroupedAccessLevelChecklist.xaml.cs

@@ -1,451 +0,0 @@
-using Content.Client.Stylesheets;
-using Content.Client.UserInterface.Controls;
-using Content.Shared.Access;
-using Robust.Client.AutoGenerated;
-using Robust.Client.Graphics;
-using Robust.Client.UserInterface;
-using Robust.Client.UserInterface.Controls;
-using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Prototypes;
-using System.Linq;
-using System.Numerics;
-
-namespace Content.Client.Access.UI;
-
-[GenerateTypedNameReferences]
-public sealed partial class GroupedAccessLevelChecklist : BoxContainer
-{
-    private static readonly ProtoId<AccessGroupPrototype> GeneralAccessGroup = "General";
-
-    [Dependency] private readonly IPrototypeManager _protoManager = default!;
-
-    private bool _isMonotone;
-    private string? _labelStyleClass;
-
-    // Access data
-    private HashSet<ProtoId<AccessGroupPrototype>> _accessGroups = new();
-    private HashSet<ProtoId<AccessLevelPrototype>> _accessLevels = new();
-    private HashSet<ProtoId<AccessLevelPrototype>> _activeAccessLevels = new();
-
-    // Button groups
-    private readonly ButtonGroup _accessGroupsButtons = new();
-
-    // Temp values
-    private int _accessGroupTabIndex = 0;
-    private bool _canInteract = false;
-    private List<AccessLevelPrototype> _accessLevelsForTab = new();
-    private readonly List<AccessLevelEntry> _accessLevelEntries = new();
-    private readonly Dictionary<AccessGroupPrototype, List<AccessLevelPrototype>> _groupedAccessLevels = new();
-
-    // Events
-    public event Action<HashSet<ProtoId<AccessLevelPrototype>>, bool>? OnAccessLevelsChangedEvent;
-
-    /// <summary>
-    /// Creates a UI control for changing access levels.
-    /// Access levels are organized under a list of tabs by their associated access group.
-    /// </summary>
-    public GroupedAccessLevelChecklist()
-    {
-        RobustXamlLoader.Load(this);
-        IoCManager.InjectDependencies(this);
-    }
-
-    private void ArrangeAccessControls()
-    {
-        // Create a list of known access groups with which to populate the UI
-        _groupedAccessLevels.Clear();
-
-        foreach (var accessGroup in _accessGroups)
-        {
-            if (!_protoManager.Resolve(accessGroup, out var accessGroupProto))
-                continue;
-
-            _groupedAccessLevels.Add(accessGroupProto, new());
-        }
-
-        // Ensure that the 'general' access group is added to handle
-        // misc. access levels that aren't associated with any group
-        if (_protoManager.Resolve(GeneralAccessGroup, out var generalAccessProto))
-            _groupedAccessLevels.TryAdd(generalAccessProto, new());
-
-        // Assign known access levels with their associated groups
-        foreach (var accessLevel in _accessLevels)
-        {
-            if (!_protoManager.Resolve(accessLevel, out var accessLevelProto))
-                continue;
-
-            var assigned = false;
-
-            foreach (var (accessGroup, accessLevels) in _groupedAccessLevels)
-            {
-                if (!accessGroup.Tags.Contains(accessLevelProto.ID))
-                    continue;
-
-                assigned = true;
-                _groupedAccessLevels[accessGroup].Add(accessLevelProto);
-            }
-
-            if (!assigned && generalAccessProto != null)
-                _groupedAccessLevels[generalAccessProto].Add(accessLevelProto);
-        }
-
-        // Remove access groups that have no assigned access levels
-        foreach (var (group, accessLevels) in _groupedAccessLevels)
-        {
-            if (accessLevels.Count == 0)
-                _groupedAccessLevels.Remove(group);
-        }
-    }
-
-    private bool TryRebuildAccessGroupControls()
-    {
-        AccessGroupList.DisposeAllChildren();
-        AccessLevelChecklist.DisposeAllChildren();
-
-        // No access level prototypes were assigned to any of the access level groups.
-        // Either the turret controller has no assigned access levels or their names were invalid.
-        if (_groupedAccessLevels.Count == 0)
-            return false;
-
-        // Reorder the access groups alphabetically
-        var orderedAccessGroups = _groupedAccessLevels.Keys.OrderBy(x => x.GetAccessGroupName()).ToList();
-
-        // Add group access buttons to the UI
-        foreach (var accessGroup in orderedAccessGroups)
-        {
-            var accessGroupButton = CreateAccessGroupButton();
-
-            // Button styling
-            if (_groupedAccessLevels.Count > 1)
-            {
-                if (AccessGroupList.ChildCount == 0)
-                    accessGroupButton.AddStyleClass(StyleBase.ButtonOpenLeft);
-                else if (_groupedAccessLevels.Count > 1 && AccessGroupList.ChildCount == (_groupedAccessLevels.Count - 1))
-                    accessGroupButton.AddStyleClass(StyleBase.ButtonOpenRight);
-                else
-                    accessGroupButton.AddStyleClass(StyleBase.ButtonOpenBoth);
-            }
-
-            accessGroupButton.Pressed = _accessGroupTabIndex == orderedAccessGroups.IndexOf(accessGroup);
-
-            // Label text and styling
-            if (_labelStyleClass != null)
-                accessGroupButton.Label.SetOnlyStyleClass(_labelStyleClass);
-
-            var accessLevelPrototypes = _groupedAccessLevels[accessGroup];
-            var prefix = accessLevelPrototypes.All(x => _activeAccessLevels.Contains(x))
-                ? "»"
-                : accessLevelPrototypes.Any(x => _activeAccessLevels.Contains(x))
-                    ? "›"
-                    : " ";
-
-            var text = Loc.GetString(
-                "turret-controls-window-access-group-label",
-                ("prefix", prefix),
-                ("label", accessGroup.GetAccessGroupName())
-            );
-
-            accessGroupButton.Text = text;
-
-            // Button events
-            accessGroupButton.OnPressed += _ => OnAccessGroupChanged(accessGroupButton.GetPositionInParent());
-
-            AccessGroupList.AddChild(accessGroupButton);
-        }
-
-        // Adjust the current tab index so it remains in range
-        if (_accessGroupTabIndex >= _groupedAccessLevels.Count)
-            _accessGroupTabIndex = _groupedAccessLevels.Count - 1;
-
-        return true;
-    }
-
-    /// <summary>
-    /// Rebuilds the checkbox list for the access level controls.
-    /// </summary>
-    public void RebuildAccessLevelsControls()
-    {
-        AccessLevelChecklist.DisposeAllChildren();
-        _accessLevelEntries.Clear();
-
-        // No access level prototypes were assigned to any of the access level groups
-        // Either turret controller has no assigned access levels, or their names were invalid
-        if (_groupedAccessLevels.Count == 0)
-            return;
-
-        // Reorder the access groups alphabetically
-        var orderedAccessGroups = _groupedAccessLevels.Keys.OrderBy(x => x.GetAccessGroupName()).ToList();
-
-        // Get the access levels associated with the current tab
-        var selectedAccessGroupTabProto = orderedAccessGroups[_accessGroupTabIndex];
-        _accessLevelsForTab = _groupedAccessLevels[selectedAccessGroupTabProto];
-        _accessLevelsForTab = _accessLevelsForTab.OrderBy(x => x.GetAccessLevelName()).ToList();
-
-        // Add an 'all' checkbox as the first child of the list if it has more than one access level
-        // Toggling this checkbox on will mark all other boxes below it on/off
-        var allCheckBox = CreateAccessLevelCheckbox();
-        allCheckBox.Text = Loc.GetString("turret-controls-window-all-checkbox");
-
-        if (_labelStyleClass != null)
-            allCheckBox.Label.SetOnlyStyleClass(_labelStyleClass);
-
-        // Add the 'all' checkbox events
-        allCheckBox.OnPressed += args =>
-        {
-            SetCheckBoxPressedState(_accessLevelEntries, allCheckBox.Pressed);
-
-            var accessLevels = new HashSet<ProtoId<AccessLevelPrototype>>();
-
-            foreach (var accessLevel in _accessLevelsForTab)
-            {
-                accessLevels.Add(accessLevel);
-            }
-
-            OnAccessLevelsChangedEvent?.Invoke(accessLevels, allCheckBox.Pressed);
-        };
-
-        AccessLevelChecklist.AddChild(allCheckBox);
-
-        // Hide the 'all' checkbox if the tab has only one access level
-        var allCheckBoxVisible = _accessLevelsForTab.Count > 1;
-
-        allCheckBox.Visible = allCheckBoxVisible;
-        allCheckBox.Disabled = !_canInteract;
-
-        // Add any remaining missing access level buttons to the UI
-        foreach (var accessLevel in _accessLevelsForTab)
-        {
-            // Create the entry
-            var accessLevelEntry = new AccessLevelEntry(_isMonotone);
-
-            accessLevelEntry.AccessLevel = accessLevel;
-            accessLevelEntry.CheckBox.Text = accessLevel.GetAccessLevelName();
-            accessLevelEntry.CheckBox.Pressed = _activeAccessLevels.Contains(accessLevel);
-            accessLevelEntry.CheckBox.Disabled = !_canInteract;
-
-            if (_labelStyleClass != null)
-                accessLevelEntry.CheckBox.Label.SetOnlyStyleClass(_labelStyleClass);
-
-            // Set the checkbox linkage lines
-            var isEndOfList = _accessLevelsForTab.IndexOf(accessLevel) == (_accessLevelsForTab.Count - 1);
-
-            var lines = new List<(Vector2, Vector2)>
-            {
-                (new Vector2(0.5f, 0f), new Vector2(0.5f, isEndOfList ? 0.5f : 1f)),
-                (new Vector2(0.5f, 0.5f), new Vector2(1f, 0.5f)),
-            };
-
-            accessLevelEntry.UpdateCheckBoxLink(lines);
-            accessLevelEntry.CheckBoxLink.Visible = allCheckBoxVisible;
-            accessLevelEntry.CheckBoxLink.Modulate = !_canInteract ? Color.Gray : Color.White;
-
-            // Add checkbox events
-            accessLevelEntry.CheckBox.OnPressed += args =>
-            {
-                // If the checkbox and its siblings are checked, check the 'all' checkbox too
-                allCheckBox.Pressed = AreAllCheckBoxesPressed(_accessLevelEntries.Select(x => x.CheckBox));
-
-                OnAccessLevelsChangedEvent?.Invoke([accessLevelEntry.AccessLevel], accessLevelEntry.CheckBox.Pressed);
-            };
-
-            AccessLevelChecklist.AddChild(accessLevelEntry);
-            _accessLevelEntries.Add(accessLevelEntry);
-        }
-
-        // Press the 'all' checkbox if all others are pressed
-        allCheckBox.Pressed = AreAllCheckBoxesPressed(_accessLevelEntries.Select(x => x.CheckBox));
-    }
-
-    private bool AreAllCheckBoxesPressed(IEnumerable<CheckBox> checkBoxes)
-    {
-        foreach (var checkBox in checkBoxes)
-        {
-            if (!checkBox.Pressed)
-                return false;
-        }
-
-        return true;
-    }
-
-    private void SetCheckBoxPressedState(List<AccessLevelEntry> accessLevelEntries, bool pressed)
-    {
-        foreach (var accessLevelEntry in accessLevelEntries)
-        {
-            accessLevelEntry.CheckBox.Pressed = pressed;
-        }
-    }
-
-
-    /// <summary>
-    /// Provides the UI with a list of access groups using which list of tabs should be populated.
-    /// </summary>
-    public void SetAccessGroups(HashSet<ProtoId<AccessGroupPrototype>> accessGroups)
-    {
-        _accessGroups = accessGroups;
-
-        ArrangeAccessControls();
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    /// <summary>
-    /// Provides the UI with a list of access levels with which it can populate the currently selected tab.
-    /// </summary>
-    public void SetAccessLevels(HashSet<ProtoId<AccessLevelPrototype>> accessLevels)
-    {
-        _accessLevels = accessLevels;
-
-        ArrangeAccessControls();
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    /// <summary>
-    /// Sets which access level checkboxes should be marked on the UI.
-    /// </summary>
-    public void SetActiveAccessLevels(HashSet<ProtoId<AccessLevelPrototype>> activeAccessLevels)
-    {
-        _activeAccessLevels = activeAccessLevels;
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    /// <summary>
-    /// Sets whether the local player can interact with the checkboxes.
-    /// </summary>
-    public void SetLocalPlayerAccessibility(bool canInteract)
-    {
-        _canInteract = canInteract;
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    /// <summary>
-    /// Sets whether the UI should use monotone buttons and checkboxes.
-    /// </summary>
-    public void SetMonotone(bool monotone)
-    {
-        _isMonotone = monotone;
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    /// <summary>
-    /// Applies the specified style to the labels on the UI buttons and checkboxes.
-    /// </summary>
-    public void SetLabelStyleClass(string? styleClass)
-    {
-        _labelStyleClass = styleClass;
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    private void OnAccessGroupChanged(int newTabIndex)
-    {
-        if (newTabIndex == _accessGroupTabIndex)
-            return;
-
-        _accessGroupTabIndex = newTabIndex;
-
-        if (TryRebuildAccessGroupControls())
-            RebuildAccessLevelsControls();
-    }
-
-    private Button CreateAccessGroupButton()
-    {
-        var button = _isMonotone ? new MonotoneButton() : new Button();
-
-        button.ToggleMode = true;
-        button.Group = _accessGroupsButtons;
-        button.Label.HorizontalAlignment = HAlignment.Left;
-
-        return button;
-    }
-
-    private CheckBox CreateAccessLevelCheckbox()
-    {
-        var checkbox = _isMonotone ? new MonotoneCheckBox() : new CheckBox();
-
-        checkbox.Margin = new Thickness(0, 0, 0, 3);
-        checkbox.ToggleMode = true;
-        checkbox.ReservesSpace = false;
-
-        return checkbox;
-    }
-
-    private sealed class AccessLevelEntry : BoxContainer
-    {
-        public ProtoId<AccessLevelPrototype> AccessLevel;
-        public readonly CheckBox CheckBox;
-        public readonly LineRenderer CheckBoxLink;
-
-        public AccessLevelEntry(bool monotone)
-        {
-            HorizontalExpand = true;
-
-            CheckBoxLink = new LineRenderer
-            {
-                SetWidth = 22,
-                VerticalExpand = true,
-                Margin = new Thickness(0, -1),
-                ReservesSpace = false,
-            };
-
-            AddChild(CheckBoxLink);
-
-            CheckBox = monotone ? new MonotoneCheckBox() : new CheckBox();
-            CheckBox.ToggleMode = true;
-            CheckBox.Margin = new Thickness(0f, 0f, 0f, 3f);
-
-            AddChild(CheckBox);
-        }
-
-        public void UpdateCheckBoxLink(List<(Vector2, Vector2)> lines)
-        {
-            CheckBoxLink.Lines = lines;
-        }
-    }
-
-    private sealed class LineRenderer : Control
-    {
-        /// <summary>
-        /// List of lines to render (their start and end x-y coordinates).
-        /// Position (0,0) is the top left corner of the control and
-        /// position (1,1) is the bottom right corner.
-        /// </summary>
-        /// <remarks>
-        /// The color of the lines is inherited from the control.
-        /// </remarks>
-        public List<(Vector2, Vector2)> Lines;
-
-        public LineRenderer()
-        {
-            Lines = new List<(Vector2, Vector2)>();
-        }
-
-        public LineRenderer(List<(Vector2, Vector2)> lines)
-        {
-            Lines = lines;
-        }
-
-        protected override void Draw(DrawingHandleScreen handle)
-        {
-            foreach (var line in Lines)
-            {
-                var start = PixelPosition +
-                            new Vector2(PixelWidth * line.Item1.X, PixelHeight * line.Item1.Y);
-
-                var end = PixelPosition +
-                          new Vector2(PixelWidth * line.Item2.X, PixelHeight * line.Item2.Y);
-
-                handle.DrawLine(start, end, ActualModulateSelf);
-            }
-        }
-    }
-}

Content.Client/Access/UI/IdCardConsoleBoundUserInterface.cs

@@ -1,11 +1,8 @@
 using Content.Shared.Access;
 using Content.Shared.Access.Components;
 using Content.Shared.Access.Systems;
-using Content.Shared.CCVar;
 using Content.Shared.Containers.ItemSlots;
 using Content.Shared.CrewManifest;
-using Content.Shared.Roles;
-using Robust.Shared.Configuration;
 using Robust.Shared.Prototypes;
 using static Content.Shared.Access.Components.IdCardConsoleComponent;
 
@@ -14,21 +11,13 @@ namespace Content.Client.Access.UI
     public sealed class IdCardConsoleBoundUserInterface : BoundUserInterface
     {
         [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
-        [Dependency] private readonly IConfigurationManager _cfgManager = default!;
         private readonly SharedIdCardConsoleSystem _idCardConsoleSystem = default!;
 
         private IdCardConsoleWindow? _window;
 
-        // CCVar.
-        private int _maxNameLength;
-        private int _maxIdJobLength;
-
         public IdCardConsoleBoundUserInterface(EntityUid owner, Enum uiKey) : base(owner, uiKey)
         {
             _idCardConsoleSystem = EntMan.System<SharedIdCardConsoleSystem>();
-
-            _maxNameLength =_cfgManager.GetCVar(CCVars.MaxNameLength);
-            _maxIdJobLength = _cfgManager.GetCVar(CCVars.MaxIdJobLength);
         }
 
         protected override void Open()
@@ -75,13 +64,13 @@ namespace Content.Client.Access.UI
             _window?.UpdateState(castState);
         }
 
-        public void SubmitData(string newFullName, string newJobTitle, List<ProtoId<AccessLevelPrototype>> newAccessList, ProtoId<JobPrototype> newJobPrototype)
+        public void SubmitData(string newFullName, string newJobTitle, List<ProtoId<AccessLevelPrototype>> newAccessList, string newJobPrototype)
         {
-            if (newFullName.Length > _maxNameLength)
-                newFullName = newFullName[.._maxNameLength];
+            if (newFullName.Length > MaxFullNameLength)
+                newFullName = newFullName[..MaxFullNameLength];
 
-            if (newJobTitle.Length > _maxIdJobLength)
-                newJobTitle = newJobTitle[.._maxIdJobLength];
+            if (newJobTitle.Length > MaxJobTitleLength)
+                newJobTitle = newJobTitle[..MaxJobTitleLength];
 
             SendMessage(new WriteToTargetIdMessage(
                 newFullName,

Content.Client/Access/UI/IdCardConsoleWindow.xaml.cs

@@ -1,13 +1,11 @@
 using System.Linq;
 using Content.Shared.Access;
 using Content.Shared.Access.Systems;
-using Content.Shared.CCVar;
 using Content.Shared.Roles;
 using Robust.Client.AutoGenerated;
 using Robust.Client.UserInterface.Controls;
 using Robust.Client.UserInterface.CustomControls;
 using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Configuration;
 using Robust.Shared.Prototypes;
 using static Content.Shared.Access.Components.IdCardConsoleComponent;
 
@@ -16,17 +14,12 @@ namespace Content.Client.Access.UI
     [GenerateTypedNameReferences]
     public sealed partial class IdCardConsoleWindow : DefaultWindow
     {
-        [Dependency] private readonly IConfigurationManager _cfgManager = default!;
         [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
         [Dependency] private readonly ILogManager _logManager = default!;
         private readonly ISawmill _logMill = default!;
 
         private readonly IdCardConsoleBoundUserInterface _owner;
 
-        // CCVar.
-        private int _maxNameLength;
-        private int _maxIdJobLength;
-
         private AccessLevelControl _accessButtons = new();
         private readonly List<string> _jobPrototypeIds = new();
 
@@ -46,11 +39,7 @@ namespace Content.Client.Access.UI
 
             _owner = owner;
 
-            _maxNameLength = _cfgManager.GetCVar(CCVars.MaxNameLength);
-            _maxIdJobLength = _cfgManager.GetCVar(CCVars.MaxIdJobLength);
-
             FullNameLineEdit.OnTextEntered += _ => SubmitData();
-            FullNameLineEdit.IsValid = s => s.Length <= _maxNameLength;
             FullNameLineEdit.OnTextChanged += _ =>
             {
                 FullNameSaveButton.Disabled = FullNameSaveButton.Text == _lastFullName;
@@ -58,7 +47,6 @@ namespace Content.Client.Access.UI
             FullNameSaveButton.OnPressed += _ => SubmitData();
 
             JobTitleLineEdit.OnTextEntered += _ => SubmitData();
-            JobTitleLineEdit.IsValid = s => s.Length <= _maxIdJobLength;
             JobTitleLineEdit.OnTextChanged += _ =>
             {
                 JobTitleSaveButton.Disabled = JobTitleLineEdit.Text == _lastJobTitle;
@@ -123,7 +111,7 @@ namespace Content.Client.Access.UI
 
             foreach (var group in job.AccessGroups)
             {
-                if (!_prototypeManager.Resolve(group, out AccessGroupPrototype? groupPrototype))
+                if (!_prototypeManager.TryIndex(group, out AccessGroupPrototype? groupPrototype))
                 {
                     continue;
                 }

Content.Client/Actions/ActionEvents.cs

@@ -1,6 +1,3 @@
-using Content.Shared.Actions.Components;
-using static Robust.Shared.Input.Binding.PointerInputCmdHandler;
-
 namespace Content.Client.Actions;
 
 /// <summary>
@@ -10,17 +7,3 @@ public sealed class FillActionSlotEvent : EntityEventArgs
 {
     public EntityUid? Action;
 }
-
-/// <summary>
-/// Client-side event used to attempt to trigger a targeted action.
-/// This only gets raised if the has <see cref="TargetActionComponent">.
-/// Handlers must set <c>Handled</c> to true, then if the action has been performed,
-/// i.e. a target is found, then FoundTarget must be set to true.
-/// </summary>
-[ByRefEvent]
-public record struct ActionTargetAttemptEvent(
-    PointerInputCmdArgs Input,
-    Entity<ActionsComponent> User,
-    ActionComponent Action,
-    bool Handled = false,
-    bool FoundTarget = false);

Content.Client/Actions/ActionsSystem.cs

@@ -1,23 +1,18 @@
 using System.IO;
 using System.Linq;
 using Content.Shared.Actions;
-using Content.Shared.Actions.Components;
 using Content.Shared.Charges.Systems;
-using Content.Shared.Mapping;
-using Content.Shared.Maps;
 using JetBrains.Annotations;
 using Robust.Client.Player;
 using Robust.Shared.ContentPack;
 using Robust.Shared.GameStates;
 using Robust.Shared.Input.Binding;
 using Robust.Shared.Player;
-using Robust.Shared.Prototypes;
 using Robust.Shared.Serialization.Manager;
 using Robust.Shared.Serialization.Markdown;
 using Robust.Shared.Serialization.Markdown.Mapping;
 using Robust.Shared.Serialization.Markdown.Sequence;
 using Robust.Shared.Serialization.Markdown.Value;
-using Robust.Shared.Timing;
 using Robust.Shared.Utility;
 using YamlDotNet.RepresentationModel;
 
@@ -30,8 +25,8 @@ namespace Content.Client.Actions
 
         [Dependency] private readonly SharedChargesSystem _sharedCharges = default!;
         [Dependency] private readonly IPlayerManager _playerManager = default!;
-        [Dependency] private readonly IPrototypeManager _proto = default!;
         [Dependency] private readonly IResourceManager _resources = default!;
+        [Dependency] private readonly ISerializationManager _serialization = default!;
         [Dependency] private readonly MetaDataSystem _metaData = default!;
 
         public event Action<EntityUid>? OnActionAdded;
@@ -43,67 +38,131 @@ namespace Content.Client.Actions
         public event Action<List<SlotAssignment>>? AssignSlot;
 
         private readonly List<EntityUid> _removed = new();
-        private readonly List<Entity<ActionComponent>> _added = new();
-
-        public static readonly EntProtoId MappingEntityAction = "BaseMappingEntityAction";
+        private readonly List<(EntityUid, BaseActionComponent?)> _added = new();
 
         public override void Initialize()
         {
             base.Initialize();
-
             SubscribeLocalEvent<ActionsComponent, LocalPlayerAttachedEvent>(OnPlayerAttached);
             SubscribeLocalEvent<ActionsComponent, LocalPlayerDetachedEvent>(OnPlayerDetached);
-            SubscribeLocalEvent<ActionsComponent, ComponentHandleState>(OnHandleState);
+            SubscribeLocalEvent<ActionsComponent, ComponentHandleState>(HandleComponentState);
 
-            SubscribeLocalEvent<ActionComponent, AfterAutoHandleStateEvent>(OnActionAutoHandleState);
-
-            SubscribeLocalEvent<EntityTargetActionComponent, ActionTargetAttemptEvent>(OnEntityTargetAttempt);
-            SubscribeLocalEvent<WorldTargetActionComponent, ActionTargetAttemptEvent>(OnWorldTargetAttempt);
+            SubscribeLocalEvent<InstantActionComponent, ComponentHandleState>(OnInstantHandleState);
+            SubscribeLocalEvent<EntityTargetActionComponent, ComponentHandleState>(OnEntityTargetHandleState);
+            SubscribeLocalEvent<WorldTargetActionComponent, ComponentHandleState>(OnWorldTargetHandleState);
+            SubscribeLocalEvent<EntityWorldTargetActionComponent, ComponentHandleState>(OnEntityWorldTargetHandleState);
         }
 
-
-        private void OnActionAutoHandleState(Entity<ActionComponent> ent, ref AfterAutoHandleStateEvent args)
+        private void OnInstantHandleState(EntityUid uid, InstantActionComponent component, ref ComponentHandleState args)
         {
-            UpdateAction(ent);
+            if (args.Current is not InstantActionComponentState state)
+                return;
+
+            BaseHandleState<InstantActionComponent>(uid, component, state);
         }
 
-        public override void UpdateAction(Entity<ActionComponent> ent)
+        private void OnEntityTargetHandleState(EntityUid uid, EntityTargetActionComponent component, ref ComponentHandleState args)
         {
+            if (args.Current is not EntityTargetActionComponentState state)
+                return;
+
+            component.Whitelist = state.Whitelist;
+            component.Blacklist = state.Blacklist;
+            component.CanTargetSelf = state.CanTargetSelf;
+            BaseHandleState<EntityTargetActionComponent>(uid, component, state);
+        }
+
+        private void OnWorldTargetHandleState(EntityUid uid, WorldTargetActionComponent component, ref ComponentHandleState args)
+        {
+            if (args.Current is not WorldTargetActionComponentState state)
+                return;
+
+            BaseHandleState<WorldTargetActionComponent>(uid, component, state);
+        }
+
+        private void OnEntityWorldTargetHandleState(EntityUid uid,
+            EntityWorldTargetActionComponent component,
+            ref ComponentHandleState args)
+        {
+            if (args.Current is not EntityWorldTargetActionComponentState state)
+                return;
+
+            component.Whitelist = state.Whitelist;
+            component.CanTargetSelf = state.CanTargetSelf;
+            BaseHandleState<EntityWorldTargetActionComponent>(uid, component, state);
+        }
+
+        private void BaseHandleState<T>(EntityUid uid, BaseActionComponent component, BaseActionComponentState state) where T : BaseActionComponent
+        {
+            // TODO ACTIONS use auto comp states
+            component.Icon = state.Icon;
+            component.IconOn = state.IconOn;
+            component.IconColor = state.IconColor;
+            component.OriginalIconColor = state.OriginalIconColor;
+            component.DisabledIconColor = state.DisabledIconColor;
+            component.Keywords.Clear();
+            component.Keywords.UnionWith(state.Keywords);
+            component.Enabled = state.Enabled;
+            component.Toggled = state.Toggled;
+            component.Cooldown = state.Cooldown;
+            component.UseDelay = state.UseDelay;
+            component.Container = EnsureEntity<T>(state.Container, uid);
+            component.EntityIcon = EnsureEntity<T>(state.EntityIcon, uid);
+            component.CheckCanInteract = state.CheckCanInteract;
+            component.CheckConsciousness = state.CheckConsciousness;
+            component.ClientExclusive = state.ClientExclusive;
+            component.Priority = state.Priority;
+            component.AttachedEntity = EnsureEntity<T>(state.AttachedEntity, uid);
+            component.RaiseOnUser = state.RaiseOnUser;
+            component.RaiseOnAction = state.RaiseOnAction;
+            component.AutoPopulate = state.AutoPopulate;
+            component.Temporary = state.Temporary;
+            component.ItemIconStyle = state.ItemIconStyle;
+            component.Sound = state.Sound;
+
+            UpdateAction(uid, component);
+        }
+
+        public override void UpdateAction(EntityUid? actionId, BaseActionComponent? action = null)
+        {
+            if (!ResolveActionData(actionId, ref action))
+                return;
+
             // TODO: Decouple this.
-            ent.Comp.IconColor = _sharedCharges.GetCurrentCharges(ent.Owner) == 0 ? ent.Comp.DisabledIconColor : ent.Comp.OriginalIconColor;
-            base.UpdateAction(ent);
-            if (_playerManager.LocalEntity != ent.Comp.AttachedEntity)
+            action.IconColor = _sharedCharges.GetCurrentCharges(actionId.Value) == 0 ? action.DisabledIconColor : action.OriginalIconColor;
+
+            base.UpdateAction(actionId, action);
+            if (_playerManager.LocalEntity != action.AttachedEntity)
                 return;
 
             ActionsUpdated?.Invoke();
         }
 
-        private void OnHandleState(Entity<ActionsComponent> ent, ref ComponentHandleState args)
+        private void HandleComponentState(EntityUid uid, ActionsComponent component, ref ComponentHandleState args)
         {
             if (args.Current is not ActionsComponentState state)
                 return;
 
-            var (uid, comp) = ent;
             _added.Clear();
             _removed.Clear();
             var stateEnts = EnsureEntitySet<ActionsComponent>(state.Actions, uid);
-            foreach (var act in comp.Actions)
+            foreach (var act in component.Actions)
             {
                 if (!stateEnts.Contains(act) && !IsClientSide(act))
                     _removed.Add(act);
             }
-            comp.Actions.ExceptWith(_removed);
+            component.Actions.ExceptWith(_removed);
 
             foreach (var actionId in stateEnts)
             {
                 if (!actionId.IsValid())
                     continue;
 
-                if (!comp.Actions.Add(actionId))
+                if (!component.Actions.Add(actionId))
                     continue;
 
-                if (GetAction(actionId) is {} action)
-                    _added.Add(action);
+                TryGetActionData(actionId, out var action);
+                _added.Add((actionId, action));
             }
 
             if (_playerManager.LocalEntity != uid)
@@ -118,46 +177,45 @@ namespace Content.Client.Actions
 
             foreach (var action in _added)
             {
-                OnActionAdded?.Invoke(action);
+                OnActionAdded?.Invoke(action.Item1);
             }
 
             ActionsUpdated?.Invoke();
         }
 
-        public static int ActionComparer(Entity<ActionComponent> a, Entity<ActionComponent> b)
+        public static int ActionComparer((EntityUid, BaseActionComponent?) a, (EntityUid, BaseActionComponent?) b)
         {
-            var priorityA = a.Comp?.Priority ?? 0;
-            var priorityB = b.Comp?.Priority ?? 0;
+            var priorityA = a.Item2?.Priority ?? 0;
+            var priorityB = b.Item2?.Priority ?? 0;
             if (priorityA != priorityB)
                 return priorityA - priorityB;
 
-            priorityA = a.Comp?.Container?.Id ?? 0;
-            priorityB = b.Comp?.Container?.Id ?? 0;
+            priorityA = a.Item2?.Container?.Id ?? 0;
+            priorityB = b.Item2?.Container?.Id ?? 0;
             return priorityA - priorityB;
         }
 
-        protected override void ActionAdded(Entity<ActionsComponent> performer, Entity<ActionComponent> action)
+        protected override void ActionAdded(EntityUid performer, EntityUid actionId, ActionsComponent comp,
+            BaseActionComponent action)
         {
-            if (_playerManager.LocalEntity != performer.Owner)
+            if (_playerManager.LocalEntity != performer)
                 return;
 
-            OnActionAdded?.Invoke(action);
-            ActionsUpdated?.Invoke();
+            OnActionAdded?.Invoke(actionId);
         }
 
-        protected override void ActionRemoved(Entity<ActionsComponent> performer, Entity<ActionComponent> action)
+        protected override void ActionRemoved(EntityUid performer, EntityUid actionId, ActionsComponent comp, BaseActionComponent action)
         {
-            if (_playerManager.LocalEntity != performer.Owner)
+            if (_playerManager.LocalEntity != performer)
                 return;
 
-            OnActionRemoved?.Invoke(action);
-            ActionsUpdated?.Invoke();
+            OnActionRemoved?.Invoke(actionId);
         }
 
-        public IEnumerable<Entity<ActionComponent>> GetClientActions()
+        public IEnumerable<(EntityUid Id, BaseActionComponent Comp)> GetClientActions()
         {
             if (_playerManager.LocalEntity is not { } user)
-                return Enumerable.Empty<Entity<ActionComponent>>();
+                return Enumerable.Empty<(EntityUid, BaseActionComponent)>();
 
             return GetActions(user);
         }
@@ -194,24 +252,25 @@ namespace Content.Client.Actions
             CommandBinds.Unregister<ActionsSystem>();
         }
 
-        public void TriggerAction(Entity<ActionComponent> action)
+        public void TriggerAction(EntityUid actionId, BaseActionComponent action)
         {
-            if (_playerManager.LocalEntity is not { } user)
-                return;
-
-            // TODO: unhardcode this somehow
-
-            if (!HasComp<InstantActionComponent>(action))
-                return;
-
-            if (action.Comp.ClientExclusive)
+            if (_playerManager.LocalEntity is not { } user ||
+                !TryComp(user, out ActionsComponent? actions))
             {
-                PerformAction(user, action);
+                return;
+            }
+
+            if (action is not InstantActionComponent instantAction)
+                return;
+
+            if (action.ClientExclusive)
+            {
+                PerformAction(user, actions, actionId, instantAction, instantAction.Event, GameTiming.CurTime);
             }
             else
             {
-                var request = new RequestPerformActionEvent(GetNetEntity(action));
-                RaisePredictiveEvent(request);
+                var request = new RequestPerformActionEvent(GetNetEntity(actionId));
+                EntityManager.RaisePredictiveEvent(request);
             }
         }
 
@@ -234,140 +293,39 @@ namespace Content.Client.Actions
             if (yamlStream.Documents[0].RootNode.ToDataNode() is not SequenceDataNode sequence)
                 return;
 
-            var actions = EnsureComp<ActionsComponent>(user);
-
             ClearAssignments?.Invoke();
 
             var assignments = new List<SlotAssignment>();
+
             foreach (var entry in sequence.Sequence)
             {
                 if (entry is not MappingDataNode map)
                     continue;
 
+                if (!map.TryGet("action", out var actionNode))
+                    continue;
+
+                var action = _serialization.Read<BaseActionComponent>(actionNode, notNullableOverride: true);
+                var actionId = Spawn();
+                AddComp(actionId, action);
+                AddActionDirect(user, actionId);
+
+                if (map.TryGet<ValueDataNode>("name", out var nameNode))
+                    _metaData.SetEntityName(actionId, nameNode.Value);
+
                 if (!map.TryGet("assignments", out var assignmentNode))
                     continue;
 
-                var actionId = EntityUid.Invalid;
-                if (map.TryGet<ValueDataNode>("action", out var actionNode))
+                var nodeAssignments = _serialization.Read<List<(byte Hotbar, byte Slot)>>(assignmentNode, notNullableOverride: true);
+
+                foreach (var index in nodeAssignments)
                 {
-                    var id = new EntProtoId(actionNode.Value);
-                    actionId = Spawn(id);
+                    var assignment = new SlotAssignment(index.Hotbar, index.Slot, actionId);
+                    assignments.Add(assignment);
                 }
-                else if (map.TryGet<ValueDataNode>("entity", out var entityNode))
-                {
-                    var id = new EntProtoId(entityNode.Value);
-                    var proto = _proto.Index(id);
-                    actionId = Spawn(MappingEntityAction);
-                    SetIcon(actionId, new SpriteSpecifier.EntityPrototype(id));
-                    SetEvent(actionId, new StartPlacementActionEvent()
-                    {
-                        PlacementOption = "SnapgridCenter",
-                        EntityType = id
-                    });
-                    _metaData.SetEntityName(actionId, proto.Name);
-                }
-                else if (map.TryGet<ValueDataNode>("tileId", out var tileNode))
-                {
-                    var id = new ProtoId<ContentTileDefinition>(tileNode.Value);
-                    var proto = _proto.Index(id);
-                    actionId = Spawn(MappingEntityAction);
-                    if (proto.Sprite is {} sprite)
-                        SetIcon(actionId, new SpriteSpecifier.Texture(sprite));
-                    SetEvent(actionId, new StartPlacementActionEvent()
-                    {
-                        PlacementOption = "AlignTileAny",
-                        TileId = id
-                    });
-                    _metaData.SetEntityName(actionId, Loc.GetString(proto.Name));
-                }
-                else
-                {
-                    Log.Error($"Mapping actions from {path} had unknown action data!");
-                    continue;
-                }
-
-                AddActionDirect((user, actions), actionId);
-            }
-        }
-
-        private void OnWorldTargetAttempt(Entity<WorldTargetActionComponent> ent, ref ActionTargetAttemptEvent args)
-        {
-            if (args.Handled)
-                return;
-
-            args.Handled = true;
-
-            var (uid, comp) = ent;
-            var action = args.Action;
-            var coords = args.Input.Coordinates;
-            var user = args.User;
-
-            if (!ValidateWorldTarget(user, coords, ent))
-                return;
-
-            // optionally send the clicked entity too, if it matches its whitelist etc
-            // this is the actual entity-world targeting magic
-            EntityUid? targetEnt = null;
-            if (TryComp<EntityTargetActionComponent>(ent, out var entity) &&
-                args.Input.EntityUid != null &&
-                ValidateEntityTarget(user, args.Input.EntityUid, (uid, entity)))
-            {
-                targetEnt = args.Input.EntityUid;
             }
 
-            if (action.ClientExclusive)
-            {
-                // TODO: abstract away from single event or maybe just RaiseLocalEvent?
-                if (comp.Event is {} ev)
-                {
-                    ev.Target = coords;
-                    ev.Entity = targetEnt;
-                }
-
-                PerformAction((user, user.Comp), (uid, action));
-            }
-            else
-                RaisePredictiveEvent(new RequestPerformActionEvent(GetNetEntity(uid), GetNetEntity(targetEnt), GetNetCoordinates(coords)));
-
-            args.FoundTarget = true;
-        }
-
-        private void OnEntityTargetAttempt(Entity<EntityTargetActionComponent> ent, ref ActionTargetAttemptEvent args)
-        {
-            if (args.Handled)
-                return;
-
-            args.Handled = true;
-
-            if (args.Input.EntityUid is not { Valid: true } entity)
-                return;
-
-            // let world target component handle it
-            var (uid, comp) = ent;
-            if (comp.Event is not {} ev)
-            {
-                DebugTools.Assert(HasComp<WorldTargetActionComponent>(ent), $"Action {ToPrettyString(ent)} requir45es WorldTargetActionComponent for entity-world targeting");
-                return;
-            }
-
-            var action = args.Action;
-            var user = args.User;
-
-            if (!ValidateEntityTarget(user, entity, ent))
-                return;
-
-            if (action.ClientExclusive)
-            {
-                ev.Target = entity;
-
-                PerformAction((user, user.Comp), (uid, action));
-            }
-            else
-            {
-                RaisePredictiveEvent(new RequestPerformActionEvent(GetNetEntity(uid), GetNetEntity(entity)));
-            }
-
-            args.FoundTarget = true;
+            AssignSlot?.Invoke(assignments);
         }
 
         public record struct SlotAssignment(byte Hotbar, byte Slot, EntityUid ActionId);

Content.Client/Actions/UI/ActionAlertTooltip.cs

@@ -21,7 +21,7 @@ namespace Content.Client.Actions.UI
         /// </summary>
         public (TimeSpan Start, TimeSpan End)? Cooldown { get; set; }
 
-        public ActionAlertTooltip(FormattedMessage name, FormattedMessage? desc, string? requires = null)
+        public ActionAlertTooltip(FormattedMessage name, FormattedMessage? desc, string? requires = null, FormattedMessage? charges = null)
         {
             _gameTiming = IoCManager.Resolve<IGameTiming>();
 
@@ -52,6 +52,17 @@ namespace Content.Client.Actions.UI
                 vbox.AddChild(description);
             }
 
+            if (charges != null && !string.IsNullOrWhiteSpace(charges.ToString()))
+            {
+                var chargesLabel = new RichTextLabel
+                {
+                    MaxWidth = TooltipTextMaxWidth,
+                    StyleClasses = { StyleNano.StyleClassTooltipActionCharges }
+                };
+                chargesLabel.SetMessage(charges);
+                vbox.AddChild(chargesLabel);
+            }
+
             vbox.AddChild(_cooldownLabel = new RichTextLabel
             {
                 MaxWidth = TooltipTextMaxWidth,

Content.Client/Administration/AdminNameOverlay.cs

@@ -1,4 +1,3 @@
-using System.Collections.Frozen;
 using System.Linq;
 using System.Numerics;
 using Content.Client.Administration.Systems;
@@ -25,7 +24,6 @@ internal sealed class AdminNameOverlay : Overlay
     private readonly EntityLookupSystem _entityLookup;
     private readonly IUserInterfaceManager _userInterfaceManager;
     private readonly SharedRoleSystem _roles;
-    private readonly IPrototypeManager _prototypeManager;
     private readonly Font _font;
     private readonly Font _fontBold;
     private AdminOverlayAntagFormat _overlayFormat;
@@ -38,9 +36,8 @@ internal sealed class AdminNameOverlay : Overlay
     private float _overlayMergeDistance;
 
     //TODO make this adjustable via GUI?
-    private static readonly FrozenSet<ProtoId<RoleTypePrototype>> Filter =
-        new ProtoId<RoleTypePrototype>[] {"SoloAntagonist", "TeamAntagonist", "SiliconAntagonist", "FreeAgent"}
-        .ToFrozenSet();
+    private readonly ProtoId<RoleTypePrototype>[] _filter =
+        ["SoloAntagonist", "TeamAntagonist", "SiliconAntagonist", "FreeAgent"];
 
     private readonly string _antagLabelClassic = Loc.GetString("admin-overlay-antag-classic");
 
@@ -52,8 +49,7 @@ internal sealed class AdminNameOverlay : Overlay
         EntityLookupSystem entityLookup,
         IUserInterfaceManager userInterfaceManager,
         IConfigurationManager config,
-        SharedRoleSystem roles,
-        IPrototypeManager prototypeManager)
+        SharedRoleSystem roles)
     {
         _system = system;
         _entityManager = entityManager;
@@ -61,7 +57,6 @@ internal sealed class AdminNameOverlay : Overlay
         _entityLookup = entityLookup;
         _userInterfaceManager = userInterfaceManager;
         _roles = roles;
-        _prototypeManager = prototypeManager;
         ZIndex = 200;
         // Setting these to a specific ttf would break the antag symbols
         _font = resourceCache.NotoStack();
@@ -130,14 +125,6 @@ internal sealed class AdminNameOverlay : Overlay
         foreach (var info in sortable.OrderBy(s => s.Item4.Y).ToList())
         {
             var playerInfo = info.Item1;
-            var rolePrototype = playerInfo.RoleProto == null
-                ? null
-                : _prototypeManager.Index(playerInfo.RoleProto.Value);
-
-            var roleName = Loc.GetString(rolePrototype?.Name ?? RoleTypePrototype.FallbackName);
-            var roleColor = rolePrototype?.Color ?? RoleTypePrototype.FallbackColor;
-            var roleSymbol = rolePrototype?.Symbol ?? RoleTypePrototype.FallbackSymbol;
-
             var aabb = info.Item2;
             var entity = info.Item3;
             var screenCoordinatesCenter = info.Item4;
@@ -222,7 +209,7 @@ internal sealed class AdminNameOverlay : Overlay
             switch (_overlaySymbolStyle)
             {
                 case AdminOverlayAntagSymbolStyle.Specific:
-                    symbol = roleSymbol;
+                    symbol = playerInfo.RoleProto.Symbol;
                     break;
                 case AdminOverlayAntagSymbolStyle.Basic:
                     symbol = Loc.GetString("player-tab-antag-prefix");
@@ -238,17 +225,17 @@ internal sealed class AdminNameOverlay : Overlay
             switch (_overlayFormat)
             {
                 case AdminOverlayAntagFormat.Roletype:
-                    color = roleColor;
-                    symbol = IsFiltered(playerInfo.RoleProto) ? symbol : string.Empty;
-                    text = IsFiltered(playerInfo.RoleProto)
-                        ? roleName.ToUpper()
+                    color = playerInfo.RoleProto.Color;
+                    symbol = _filter.Contains(playerInfo.RoleProto) ? symbol : string.Empty;
+                    text = _filter.Contains(playerInfo.RoleProto)
+                        ? Loc.GetString(playerInfo.RoleProto.Name).ToUpper()
                         : string.Empty;
                     break;
                 case AdminOverlayAntagFormat.Subtype:
-                    color = roleColor;
-                    symbol = IsFiltered(playerInfo.RoleProto) ? symbol : string.Empty;
-                    text = IsFiltered(playerInfo.RoleProto)
-                        ? _roles.GetRoleSubtypeLabel(roleName, playerInfo.Subtype).ToUpper()
+                    color = playerInfo.RoleProto.Color;
+                    symbol = _filter.Contains(playerInfo.RoleProto) ? symbol : string.Empty;
+                    text = _filter.Contains(playerInfo.RoleProto)
+                        ? _roles.GetRoleSubtypeLabel(playerInfo.RoleProto.Name, playerInfo.Subtype).ToUpper()
                         : string.Empty;
                     break;
                 default:
@@ -271,12 +258,4 @@ internal sealed class AdminNameOverlay : Overlay
             drawnOverlays.Add((screenCoordinatesCenter, currentOffset));
         }
     }
-
-    private static bool IsFiltered(ProtoId<RoleTypePrototype>? roleProtoId)
-    {
-        if (roleProtoId == null)
-            return false;
-
-        return Filter.Contains(roleProtoId.Value);
-    }
 }

Content.Client/Administration/Components/HeadstandComponent.cs

@@ -0,0 +1,10 @@
+using Content.Shared.Administration.Components;
+using Robust.Shared.GameStates;
+
+namespace Content.Client.Administration.Components;
+
+[RegisterComponent]
+public sealed partial class HeadstandComponent : SharedHeadstandComponent
+{
+
+}

Content.Client/Administration/Managers/ClientAdminManager.cs

@@ -15,7 +15,6 @@ namespace Content.Client.Administration.Managers
         [Dependency] private readonly IPlayerManager _player = default!;
         [Dependency] private readonly IClientNetManager _netMgr = default!;
         [Dependency] private readonly IClientConGroupController _conGroup = default!;
-        [Dependency] private readonly IClientConsoleHost _host = default!;
         [Dependency] private readonly IResourceManager _res = default!;
         [Dependency] private readonly ILogManager _logManager = default!;
         [Dependency] private readonly IUserInterfaceManager _userInterface = default!;
@@ -87,12 +86,12 @@ namespace Content.Client.Administration.Managers
         private void UpdateMessageRx(MsgUpdateAdminStatus message)
         {
             _availableCommands.Clear();
+            var host = IoCManager.Resolve<IClientConsoleHost>();
 
             // Anything marked as Any we'll just add even if the server doesn't know about it.
-            foreach (var (command, instance) in _host.AvailableCommands)
+            foreach (var (command, instance) in host.AvailableCommands)
             {
-                if (Attribute.GetCustomAttribute(instance.GetType(), typeof(AnyCommandAttribute)) == null)
-                    continue;
+                if (Attribute.GetCustomAttribute(instance.GetType(), typeof(AnyCommandAttribute)) == null) continue;
                 _availableCommands.Add(command);
             }
 

Content.Client/Administration/Systems/AdminFrozenSystem.cs

@@ -0,0 +1,7 @@
+using Content.Shared.Administration;
+
+namespace Content.Client.Administration.Systems;
+
+public sealed class AdminFrozenSystem : SharedAdminFrozenSystem
+{
+}

Content.Client/Administration/Systems/AdminSystem.Overlay.cs

@@ -4,7 +4,6 @@ using Robust.Client.Graphics;
 using Robust.Client.ResourceManagement;
 using Robust.Client.UserInterface;
 using Robust.Shared.Configuration;
-using Robust.Shared.Prototypes;
 
 namespace Content.Client.Administration.Systems
 {
@@ -18,7 +17,6 @@ namespace Content.Client.Administration.Systems
         [Dependency] private readonly IUserInterfaceManager _userInterfaceManager = default!;
         [Dependency] private readonly IConfigurationManager _configurationManager = default!;
         [Dependency] private readonly SharedRoleSystem _roles = default!;
-        [Dependency] private readonly IPrototypeManager _proto = default!;
 
         private AdminNameOverlay _adminNameOverlay = default!;
 
@@ -35,8 +33,7 @@ namespace Content.Client.Administration.Systems
                 _entityLookup,
                 _userInterfaceManager,
                 _configurationManager,
-                _roles,
-                _proto);
+                _roles);
             _adminManager.AdminStatusUpdated += OnAdminStatusUpdated;
         }
 

Content.Client/Administration/Systems/HeadstandSystem.cs

@@ -1,4 +1,4 @@
-using Content.Shared.Administration.Components;
+using Content.Client.Administration.Components;
 using Robust.Client.GameObjects;
 
 namespace Content.Client.Administration.Systems;

Content.Client/Administration/Systems/KillSignSystem.cs

@@ -7,8 +7,6 @@ namespace Content.Client.Administration.Systems;
 
 public sealed class KillSignSystem : EntitySystem
 {
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
     public override void Initialize()
     {
         SubscribeLocalEvent<KillSignComponent, ComponentStartup>(KillSignAdded);
@@ -20,10 +18,10 @@ public sealed class KillSignSystem : EntitySystem
         if (!TryComp<SpriteComponent>(uid, out var sprite))
             return;
 
-        if (!_sprite.LayerMapTryGet((uid, sprite), KillSignKey.Key, out var layer, false))
+        if (!sprite.LayerMapTryGet(KillSignKey.Key, out var layer))
             return;
 
-        _sprite.RemoveLayer((uid, sprite), layer);
+        sprite.RemoveLayer(layer);
     }
 
     private void KillSignAdded(EntityUid uid, KillSignComponent component, ComponentStartup args)
@@ -31,15 +29,15 @@ public sealed class KillSignSystem : EntitySystem
         if (!TryComp<SpriteComponent>(uid, out var sprite))
             return;
 
-        if (_sprite.LayerMapTryGet((uid, sprite), KillSignKey.Key, out var _, false))
+        if (sprite.LayerMapTryGet(KillSignKey.Key, out var _))
             return;
 
-        var adj = _sprite.GetLocalBounds((uid, sprite)).Height / 2 + ((1.0f / 32) * 6.0f);
+        var adj = sprite.Bounds.Height / 2 + ((1.0f/32) * 6.0f);
 
-        var layer = _sprite.AddLayer((uid, sprite), new SpriteSpecifier.Rsi(new ResPath("Objects/Misc/killsign.rsi"), "sign"));
-        _sprite.LayerMapSet((uid, sprite), KillSignKey.Key, layer);
+        var layer = sprite.AddLayer(new SpriteSpecifier.Rsi(new ResPath("Objects/Misc/killsign.rsi"), "sign"));
+        sprite.LayerMapSet(KillSignKey.Key, layer);
 
-        _sprite.LayerSetOffset((uid, sprite), layer, new Vector2(0.0f, adj));
+        sprite.LayerSetOffset(layer, new Vector2(0.0f, adj));
         sprite.LayerSetShader(layer, "unshaded");
     }
 

Content.Client/Administration/UI/AdminCamera/AdminCameraControl.xaml

@@ -1,20 +0,0 @@
-<Control
-    xmlns="https://spacestation14.io"
-    xmlns:viewport="clr-namespace:Content.Client.Viewport"
-    MouseFilter="Stop">
-    <PanelContainer StyleClasses="BackgroundDark" Name="AdminCameraWindowRoot" Access="Public">
-        <BoxContainer Orientation="Vertical" Access="Public">
-            <!-- Camera -->
-            <Control VerticalExpand="True" Name="CameraViewBox">
-                <viewport:ScalingViewport Name="CameraView"
-                                          MinSize="100 100"
-                                          MouseFilter="Ignore" />
-            </Control>
-            <!-- Controller buttons -->
-            <BoxContainer Orientation="Horizontal" Margin="5 5 5 5">
-                <Button StyleClasses="OpenRight" Name="FollowButton" HorizontalExpand="True" Access="Public" Text="{Loc 'admin-camera-window-follow'}" />
-                <Button StyleClasses="OpenLeft" Name="PopControl" HorizontalExpand="True" Access="Public" Text="{Loc 'admin-camera-window-pop-out'}" />
-            </BoxContainer>
-        </BoxContainer>
-    </PanelContainer>
-</Control>

Content.Client/Administration/UI/AdminCamera/AdminCameraControl.xaml.cs

@@ -1,101 +0,0 @@
-using System.Numerics;
-using Content.Client.Eye;
-using Content.Shared.Administration;
-using Robust.Client.AutoGenerated;
-using Robust.Client.Graphics;
-using Robust.Client.Timing;
-using Robust.Client.UserInterface;
-using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Timing;
-
-namespace Content.Client.Administration.UI.AdminCamera;
-
-[GenerateTypedNameReferences]
-public sealed partial class AdminCameraControl : Control
-{
-    [Dependency] private readonly IEntityManager _entManager = default!;
-    [Dependency] private readonly IClientGameTiming _timing = default!;
-
-    public event Action? OnFollow;
-    public event Action? OnPopoutControl;
-
-    private readonly EyeLerpingSystem _eyeLerpingSystem;
-    private readonly FixedEye _defaultEye = new();
-    private AdminCameraEuiState? _nextState;
-
-    private const float MinimumZoom = 0.1f;
-    private const float MaximumZoom = 2.0f;
-
-    public EntityUid? CurrentCamera;
-    public float Zoom = 1.0f;
-
-    public bool IsPoppedOut;
-
-    public AdminCameraControl()
-    {
-        RobustXamlLoader.Load(this);
-        IoCManager.InjectDependencies(this);
-
-        _eyeLerpingSystem = _entManager.System<EyeLerpingSystem>();
-
-        CameraView.Eye = _defaultEye;
-
-        FollowButton.OnPressed += _ => OnFollow?.Invoke();
-        PopControl.OnPressed += _ => OnPopoutControl?.Invoke();
-        CameraView.OnResized += OnResized;
-    }
-
-    private new void OnResized()
-    {
-        var width = Math.Max(CameraView.PixelWidth, (int)Math.Floor(CameraView.MinWidth));
-        var height = Math.Max(CameraView.PixelHeight, (int)Math.Floor(CameraView.MinHeight));
-
-        CameraView.ViewportSize = new Vector2i(width, height);
-    }
-
-    protected override void MouseWheel(GUIMouseWheelEventArgs args)
-    {
-        base.MouseWheel(args);
-
-        if (CameraView.Eye == null)
-            return;
-
-        Zoom = Math.Clamp(Zoom - args.Delta.Y * 0.15f * Zoom, MinimumZoom, MaximumZoom);
-        CameraView.Eye.Zoom = new Vector2(Zoom, Zoom);
-        args.Handle();
-    }
-
-    public void SetState(AdminCameraEuiState state)
-    {
-        _nextState = state;
-    }
-
-    // I know that this is awful, but I copied this from the solution editor anyways.
-    // This is needed because EUIs update before the gamestate is applied, which means it will fail to get the uid from the net entity.
-    // The suggestion from the comment in the solution editor saying to use a BUI is not ideal either:
-    // - We would need to bind the UI to an entity, but with how BUIs currently work we cannot open it in the same tick as we spawn that entity on the server.
-    // - We want the UI opened by the user session, not by their currently attached entity. Otherwise it would close in cases where admins move from one entity to another, for example when ghosting.
-    protected override void FrameUpdate(FrameEventArgs args)
-    {
-        if (_nextState == null || _timing.LastRealTick < _nextState.Tick) // make sure the last gamestate has been applied
-            return;
-
-        if (!_entManager.TryGetEntity(_nextState.Camera, out var cameraUid))
-            return;
-
-        if (CurrentCamera == null)
-        {
-            _eyeLerpingSystem.AddEye(cameraUid.Value);
-            CurrentCamera = cameraUid;
-        }
-        else if (CurrentCamera != cameraUid)
-        {
-            _eyeLerpingSystem.RemoveEye(CurrentCamera.Value);
-            _eyeLerpingSystem.AddEye(cameraUid.Value);
-            CurrentCamera = cameraUid;
-        }
-
-        if (_entManager.TryGetComponent<EyeComponent>(CurrentCamera, out var eye))
-            CameraView.Eye = eye.Eye ?? _defaultEye;
-    }
-}

Content.Client/Administration/UI/AdminCamera/AdminCameraEui.cs

@@ -1,117 +0,0 @@
-using System.Numerics;
-using Content.Client.Eui;
-using Content.Shared.Administration;
-using Content.Shared.Eui;
-using JetBrains.Annotations;
-using Robust.Client.UserInterface.Controls;
-
-namespace Content.Client.Administration.UI.AdminCamera;
-
-/// <summary>
-/// Admin Eui for opening a viewport window to observe entities.
-/// Use the "Open Camera" admin verb or the "camera" command to open.
-/// </summary>
-[UsedImplicitly]
-public sealed partial class AdminCameraEui : BaseEui
-{
-    private readonly AdminCameraWindow _window;
-    private readonly AdminCameraControl _control;
-
-    // If not null the camera is in "popped out" mode and is in an external window.
-    private OSWindow? _OSWindow;
-
-    // The last location the window was located at in game.
-    // Is used for getting knowing where to "pop in" external windows.
-    private Vector2 _lastLocation;
-
-    public AdminCameraEui()
-    {
-        _window = new AdminCameraWindow();
-        _control = new AdminCameraControl();
-
-        _window.Contents.AddChild(_control);
-
-        _control.OnFollow += () => SendMessage(new AdminCameraFollowMessage());
-        _window.OnClose += () =>
-        {
-            if (!_control.IsPoppedOut)
-                SendMessage(new CloseEuiMessage());
-        };
-
-        _control.OnPopoutControl += () =>
-        {
-            if (_control.IsPoppedOut)
-                PopIn();
-            else
-                PopOut();
-        };
-    }
-
-    // Pop the window out into an external OS window
-    private void PopOut()
-    {
-        _lastLocation = _window.Position;
-
-        // TODO: When there is a way to have a minimum window size, enforce something!
-        _OSWindow = new OSWindow
-        {
-            SetSize = _window.Size,
-            Title = _window.Title ?? Loc.GetString("admin-camera-window-title-placeholder"),
-        };
-
-        _OSWindow.Show();
-
-        if (_OSWindow.Root == null)
-            return;
-
-        _control.Orphan();
-        _OSWindow.Root.AddChild(_control);
-
-        _OSWindow.Closed += () =>
-        {
-            if (_control.IsPoppedOut)
-                SendMessage(new CloseEuiMessage());
-        };
-
-        _control.IsPoppedOut = true;
-        _control.PopControl.Text = Loc.GetString("admin-camera-window-pop-in");
-
-        _window.Close();
-    }
-
-    // Pop the window back into the in game window.
-    private void PopIn()
-    {
-        _control.Orphan();
-        _window.Contents.AddChild(_control);
-
-        _window.Open(_lastLocation);
-
-        _control.IsPoppedOut = false;
-        _control.PopControl.Text = Loc.GetString("admin-camera-window-pop-out");
-
-        _OSWindow?.Close();
-        _OSWindow = null;
-    }
-
-    public override void Opened()
-    {
-        base.Opened();
-        _window.OpenCentered();
-    }
-
-    public override void Closed()
-    {
-        base.Closed();
-        _window.Close();
-    }
-
-    public override void HandleState(EuiStateBase baseState)
-    {
-        if (baseState is not AdminCameraEuiState state)
-            return;
-
-        _window.SetState(state);
-        _control.SetState(state);
-    }
-}

Content.Client/Administration/UI/AdminCamera/AdminCameraWindow.xaml

@@ -1,6 +0,0 @@
-<DefaultWindow xmlns="https://spacestation14.io"
-    Title="{Loc admin-camera-window-title-placeholder}"
-    SetSize="425 550"
-    MinSize="200 225"
-    Name="Window">
-</DefaultWindow>

Content.Client/Administration/UI/AdminCamera/AdminCameraWindow.xaml.cs

@@ -1,23 +0,0 @@
-using Content.Shared.Administration;
-using Robust.Client.AutoGenerated;
-using Robust.Client.UserInterface.CustomControls;
-using Robust.Client.UserInterface.XAML;
-
-namespace Content.Client.Administration.UI.AdminCamera;
-
-[GenerateTypedNameReferences]
-public sealed partial class AdminCameraWindow : DefaultWindow
-{
-    public AdminCameraWindow()
-    {
-        RobustXamlLoader.Load(this);
-        IoCManager.InjectDependencies(this);
-
-        ContentsContainer.Margin = new Thickness(5, 0, 5, 0);
-    }
-
-    public void SetState(AdminCameraEuiState state)
-    {
-        Title = Loc.GetString("admin-camera-window-title", ("name", state.Name));
-    }
-}

Content.Client/Administration/UI/BanPanel/BanPanel.xaml

@@ -1,7 +1,7 @@
 <DefaultWindow
     xmlns="https://spacestation14.io"
     xmlns:cc="clr-namespace:Content.Client.Administration.UI.CustomControls"
-    Title="{Loc ban-panel-title}" MinSize="410 500">
+    Title="{Loc ban-panel-title}" MinSize="350 500">
     <BoxContainer Orientation="Vertical">
         <TabContainer Name="Tabs" VerticalExpand="True">
             <!-- Basic info -->

Content.Client/Administration/UI/BanPanel/BanPanel.xaml.cs

@@ -1,14 +1,12 @@
 using System.Linq;
 using System.Net;
 using System.Net.Sockets;
-using System.Numerics;
 using Content.Client.Administration.UI.CustomControls;
 using Content.Shared.Administration;
 using Content.Shared.CCVar;
 using Content.Shared.Database;
 using Content.Shared.Roles;
 using Robust.Client.AutoGenerated;
-using Robust.Client.GameObjects;
 using Robust.Client.Graphics;
 using Robust.Client.UserInterface;
 using Robust.Client.UserInterface.Controls;
@@ -33,21 +31,14 @@ public sealed partial class BanPanel : DefaultWindow
     private uint Multiplier { get; set; }
     private bool HasBanFlag { get; set; }
     private TimeSpan? ButtonResetOn { get; set; }
-
     // This is less efficient than just holding a reference to the root control and enumerating children, but you
     // have to know how the controls are nested, which makes the code more complicated.
-    // Role group name -> the role buttons themselves.
-    private readonly Dictionary<string, List<Button>> _roleCheckboxes = new();
+    private readonly List<CheckBox> _roleCheckboxes = new();
     private readonly ISawmill _banpanelSawmill;
 
     [Dependency] private readonly IGameTiming _gameTiming = default!;
     [Dependency] private readonly IConfigurationManager _cfg = default!;
     [Dependency] private readonly ILogManager _logManager = default!;
-    [Dependency] private readonly IEntityManager _entMan = default!;
-    [Dependency] private readonly IPrototypeManager _protoMan = default!;
-
-    private const string ExpandedArrow = "▼";
-    private const string ContractedArrow = "▶";
 
     private enum TabNumbers
     {
@@ -153,90 +144,47 @@ public sealed partial class BanPanel : DefaultWindow
 
         ReasonTextEdit.Placeholder = new Rope.Leaf(Loc.GetString("ban-panel-reason"));
 
-        var departmentJobs = _protoMan.EnumeratePrototypes<DepartmentPrototype>()
-                                      .OrderBy(x => x.Weight);
-        foreach (var proto in departmentJobs)
+        var prototypeManager = IoCManager.Resolve<IPrototypeManager>();
+        foreach (var proto in prototypeManager.EnumeratePrototypes<DepartmentPrototype>())
         {
-            var roles = proto.Roles.Select(x => _protoMan.Index(x))
-                             .OrderBy(x => x.ID);
-            CreateRoleGroup(proto.ID, proto.Color, roles);
+            CreateRoleGroup(proto.ID, proto.Roles.Select(p =>  p.Id), proto.Color);
         }
 
-        var antagRoles = _protoMan.EnumeratePrototypes<AntagPrototype>()
-                                  .OrderBy(x => x.ID);
-        CreateRoleGroup("Antagonist", Color.Red, antagRoles);
+        CreateRoleGroup("Antagonist", prototypeManager.EnumeratePrototypes<AntagPrototype>().Select(p => p.ID), Color.Red);
     }
 
-    /// <summary>
-    /// Creates a "Role group" which stores information and logic for one "group" of roll bans.
-    /// For example, all antags are one group, logi is a group, medical is a group, etc...
-    /// </summary>
-    private void CreateRoleGroup<T>(string groupName, Color color, IEnumerable<T> roles) where T : class, IPrototype
+    private void CreateRoleGroup(string roleName, IEnumerable<string> roleList, Color color)
     {
         var outerContainer = new BoxContainer
         {
-            Name = $"{groupName}GroupOuterBox",
+            Name = $"{roleName}GroupOuterBox",
             HorizontalExpand = true,
             VerticalExpand = true,
             Orientation = BoxContainer.LayoutOrientation.Vertical,
-            Margin = new Thickness(4),
+            Margin = new Thickness(4)
         };
-
-        // Stores stuff like ban all and expand buttons.
-        var roleGroupHeader = new BoxContainer
+        var departmentCheckbox = new CheckBox
         {
-            Orientation = BoxContainer.LayoutOrientation.Horizontal,
+            Name = $"{roleName}GroupCheckbox",
+            Text = roleName,
+            Modulate = color,
+            HorizontalAlignment = HAlignment.Left
         };
-
-        // Stores the role checkboxes themselves.
-        var innerContainer = new GridContainer
+        outerContainer.AddChild(departmentCheckbox);
+        var innerContainer = new BoxContainer
         {
-            Name = $"{groupName}GroupInnerBox",
+            Name = $"{roleName}GroupInnerBox",
             HorizontalExpand = true,
-            Columns = 2,
-            Visible = false,
-            Margin = new Thickness(15, 5, 0, 5),
+            Orientation = BoxContainer.LayoutOrientation.Horizontal
         };
-
-        var roleGroupCheckbox = CreateRoleGroupHeader(groupName, roleGroupHeader, color, innerContainer);
-
-        outerContainer.AddChild(roleGroupHeader);
-
-        // Add the roles themselves
-        foreach (var role in roles)
+        departmentCheckbox.OnToggled += args =>
         {
-            AddRoleCheckbox(groupName, role.ID, innerContainer, roleGroupCheckbox);
-        }
-
-        outerContainer.AddChild(innerContainer);
-
-        RolesContainer.AddChild(new PanelContainer
-        {
-            PanelOverride = new StyleBoxFlat
+            foreach (var child in innerContainer.Children)
             {
-                BackgroundColor = color
-            }
-        });
-        RolesContainer.AddChild(outerContainer);
-        RolesContainer.AddChild(new HSeparator());
-    }
-
-    private Button CreateRoleGroupHeader(string groupName, BoxContainer header, Color color, GridContainer innerContainer)
-    {
-        var roleGroupCheckbox = new Button
-        {
-            Name = $"{groupName}GroupCheckbox",
-            Text = "Ban all",
-            Margin = new Thickness(0, 0, 5, 0),
-            ToggleMode = true,
-        };
-
-        // When this is toggled, toggle all buttons in this group so they match.
-        roleGroupCheckbox.OnToggled += args =>
-        {
-            foreach (var role in _roleCheckboxes[groupName])
-            {
-                role.Pressed = args.Pressed;
+                if (child is CheckBox c)
+                {
+                    c.Pressed = args.Pressed;
+                }
             }
 
             if (args.Pressed)
@@ -251,12 +199,15 @@ public sealed partial class BanPanel : DefaultWindow
             }
             else
             {
-                foreach (var roleButtons in _roleCheckboxes.Values)
+                foreach (var childContainer in RolesContainer.Children)
                 {
-                    foreach (var button in roleButtons)
+                    if (childContainer is Container)
                     {
-                        if (button.Pressed)
-                            return;
+                        foreach (var child in childContainer.Children)
+                        {
+                            if (child is CheckBox { Pressed: true })
+                                return;
+                        }
                     }
                 }
 
@@ -269,73 +220,38 @@ public sealed partial class BanPanel : DefaultWindow
                 SeverityOption.SelectId((int) newSeverity);
             }
         };
-
-        var hideButton = new Button
+        outerContainer.AddChild(innerContainer);
+        foreach (var role in roleList)
         {
-            Text = Loc.GetString("role-bans-expand-roles") + " " + ContractedArrow,
-            ToggleMode = true,
-        };
-        hideButton.OnPressed += args =>
+            AddRoleCheckbox(role, innerContainer, departmentCheckbox);
+        }
+        RolesContainer.AddChild(new PanelContainer
         {
-            innerContainer.Visible = args.Button.Pressed;
-            ((Button)args.Button).Text = args.Button.Pressed
-                ? Loc.GetString("role-bans-contract-roles") + " " + ExpandedArrow
-                : Loc.GetString("role-bans-expand-roles") + " " + ContractedArrow;
-        };
-        header.AddChild(new Label
-        {
-            Text = groupName,
-            Modulate = color,
-            Margin = new Thickness(0, 0, 5, 0),
+            PanelOverride = new StyleBoxFlat
+            {
+                BackgroundColor = color
+            }
         });
-        header.AddChild(roleGroupCheckbox);
-        header.AddChild(hideButton);
-        return roleGroupCheckbox;
+        RolesContainer.AddChild(outerContainer);
+        RolesContainer.AddChild(new HSeparator());
     }
 
-    /// <summary>
-    /// Adds a checkbutton specifically for one "role" in a "group"
-    /// E.g. it would add the Chief Medical Officer "role" into the "Medical" group.
-    /// </summary>
-    private void AddRoleCheckbox(string group, string role, GridContainer roleGroupInnerContainer, Button roleGroupCheckbox)
+    private void AddRoleCheckbox(string role, Control container, CheckBox header)
     {
-        var roleCheckboxContainer = new BoxContainer();
-        var roleCheckButton = new Button
+        var roleCheckbox = new CheckBox
         {
             Name = $"{role}RoleCheckbox",
-            Text = role,
-            ToggleMode = true,
+            Text = role
         };
-        roleCheckButton.OnToggled += args =>
+        roleCheckbox.OnToggled += args =>
         {
-            // Checks the role group checkbox if all the children are pressed
-            if (args.Pressed && _roleCheckboxes[group].All(e => e.Pressed))
-                roleGroupCheckbox.Pressed = args.Pressed;
+            if (args is { Pressed: true, Button.Parent: { } } && args.Button.Parent.Children.Where(e => e is CheckBox).All(e => ((CheckBox) e).Pressed))
+                header.Pressed = args.Pressed;
             else
-                roleGroupCheckbox.Pressed = false;
+                header.Pressed = false;
         };
-
-        // This is adding the icon before the role name
-        // TODO: This should not be using raw strings for prototypes as it means it won't be validated at all.
-        // I know the ban manager is doing the same thing, but that should not leak into UI code.
-        if (_protoMan.TryIndex<JobPrototype>(role, out var jobPrototype) && _protoMan.Resolve(jobPrototype.Icon, out var iconProto))
-        {
-            var jobIconTexture = new TextureRect
-            {
-                Texture = _entMan.System<SpriteSystem>().Frame0(iconProto.Icon),
-                TextureScale = new Vector2(2.5f, 2.5f),
-                Stretch = TextureRect.StretchMode.KeepCentered,
-                Margin = new Thickness(5, 0, 0, 0),
-            };
-            roleCheckboxContainer.AddChild(jobIconTexture);
-        }
-
-        roleCheckboxContainer.AddChild(roleCheckButton);
-
-        roleGroupInnerContainer.AddChild(roleCheckboxContainer);
-
-        _roleCheckboxes.TryAdd(group, []);
-        _roleCheckboxes[group].Add(roleCheckButton);
+        container.AddChild(roleCheckbox);
+        _roleCheckboxes.Add(roleCheckbox);
     }
 
     public void UpdateBanFlag(bool newFlag)
@@ -553,13 +469,7 @@ public sealed partial class BanPanel : DefaultWindow
             if (_roleCheckboxes.Count == 0)
                 throw new DebugAssertException("RoleCheckboxes was empty");
 
-            foreach (var button in _roleCheckboxes.Values.SelectMany(departmentButtons => departmentButtons))
-            {
-                if (button is { Pressed: true, Text: not null })
-                {
-                    rolesList.Add(button.Text);
-                }
-            }
+            rolesList.AddRange(_roleCheckboxes.Where(c => c is { Pressed: true, Text: { } }).Select(c => c.Text!));
 
             if (rolesList.Count == 0)
             {

Content.Client/Administration/UI/Logs/AdminLogsControl.xaml

@@ -49,7 +49,6 @@
                     <Control HorizontalExpand="True"/>
                     <Label Name="Count" Access="Public"/>
                     <Control HorizontalExpand="True"/>
-                    <Button Name="ExportLogs" Access="Public" Text="{Loc admin-logs-export}"/>
                     <Button Name="PopOutButton" Access="Public" Text="{Loc admin-logs-pop-out}"/>
                 </BoxContainer>
                 <BoxContainer Orientation="Horizontal">

Content.Client/Administration/UI/Logs/AdminLogsEui.cs

@@ -1,6 +1,4 @@
-using System.IO;
-using System.Linq;
-using Content.Client.Administration.UI.CustomControls;
+using System.Linq;
 using Content.Client.Eui;
 using Content.Shared.Administration.Logs;
 using Content.Shared.Eui;
@@ -17,16 +15,6 @@ public sealed class AdminLogsEui : BaseEui
 {
     [Dependency] private readonly IClyde _clyde = default!;
     [Dependency] private readonly IUserInterfaceManager _uiManager = default!;
-    [Dependency] private readonly IFileDialogManager _dialogManager = default!;
-    [Dependency] private readonly ILogManager _log = default!;
-
-    private const char CsvSeparator = ',';
-    private const string CsvQuote = "\"";
-    private const string CsvHeader = "Date,ID,PlayerID,Severity,Type,Message";
-
-    private ISawmill _sawmill;
-
-    private bool _currentlyExportingLogs = false;
 
     public AdminLogsEui()
     {
@@ -38,9 +26,6 @@ public sealed class AdminLogsEui : BaseEui
         LogsControl.RefreshButton.OnPressed += _ => RequestLogs();
         LogsControl.NextButton.OnPressed += _ => NextLogs();
         LogsControl.PopOutButton.OnPressed += _ => PopOut();
-        LogsControl.ExportLogs.OnPressed += _ => ExportLogs();
-
-        _sawmill = _log.GetSawmill("admin.logs.ui");
     }
 
     private WindowRoot? Root { get; set; }
@@ -89,71 +74,6 @@ public sealed class AdminLogsEui : BaseEui
         SendMessage(request);
     }
 
-    private async void ExportLogs()
-    {
-        if (_currentlyExportingLogs)
-            return;
-
-        _currentlyExportingLogs = true;
-        LogsControl.ExportLogs.Disabled = true;
-
-        var file = await _dialogManager.SaveFile(new FileDialogFilters(new FileDialogFilters.Group("csv")));
-
-        if (file == null)
-            return;
-
-        try
-        {
-            // Buffer is set to 4KB for performance reasons. As the average export of 1000 logs is ~200KB
-            await using var writer = new StreamWriter(file.Value.fileStream, bufferSize: 4096);
-            await writer.WriteLineAsync(CsvHeader);
-            foreach (var child in LogsControl.LogsContainer.Children)
-            {
-                if (child is not AdminLogLabel logLabel || !child.Visible)
-                    continue;
-
-                var log = logLabel.Log;
-
-                // Date
-                // I swear to god if someone adds ,s or "s to the other fields...
-                await writer.WriteAsync(log.Date.ToString("s", System.Globalization.CultureInfo.InvariantCulture));
-                await writer.WriteAsync(CsvSeparator);
-                // ID
-                await writer.WriteAsync(log.Id.ToString());
-                await writer.WriteAsync(CsvSeparator);
-                // PlayerID
-                var players = log.Players;
-                for (var i = 0; i < players.Length; i++)
-                {
-                    await writer.WriteAsync(players[i] + (i == players.Length - 1 ? "" : " "));
-                }
-                await writer.WriteAsync(CsvSeparator);
-                // Severity
-                await writer.WriteAsync(log.Impact.ToString());
-                await writer.WriteAsync(CsvSeparator);
-                // Type
-                await writer.WriteAsync(log.Type.ToString());
-                await writer.WriteAsync(CsvSeparator);
-                // Message
-                await writer.WriteAsync(CsvQuote);
-                await writer.WriteAsync(log.Message.Replace(CsvQuote, CsvQuote + CsvQuote));
-                await writer.WriteAsync(CsvQuote);
-
-                await writer.WriteLineAsync();
-            }
-        }
-        catch (Exception exc)
-        {
-            _sawmill.Error($"Error when exporting admin log:\n{exc.StackTrace}");
-        }
-        finally
-        {
-            await file.Value.fileStream.DisposeAsync();
-            _currentlyExportingLogs = false;
-            LogsControl.ExportLogs.Disabled = false;
-        }
-    }
-
     private void PopOut()
     {
         if (LogsWindow == null)

Content.Client/Administration/UI/ManageSolutions/EditSolutionsWindow.xaml

@@ -8,21 +8,6 @@
             <OptionButton Name="SolutionOption" HorizontalExpand="True"/>
         </BoxContainer>
 
-        <BoxContainer Orientation="Horizontal" HorizontalExpand="True" Margin="0 4">
-            <Button Name="VVButton"
-                    Text="{Loc 'admin-solutions-window-vv-button'}"
-                    ToolTip="{Loc 'admin-solutions-window-vv-button-tooltip'}"
-                    Disabled="True"
-                    HorizontalExpand="True"
-                    StyleClasses="OpenRight"/>
-            <Button Name="SolutionButton"
-                    Text="{Loc 'admin-solutions-window-solution-button'}"
-                    ToolTip="{Loc 'admin-solutions-window-solution-button-tooltip'}"
-                    Disabled="True"
-                    HorizontalExpand="True"
-                    StyleClasses="OpenLeft"/>
-        </BoxContainer>
-
         <!-- The total volume / capacity of the solution -->
         <BoxContainer Name="VolumeBox" Orientation="Vertical" HorizontalExpand="True" Margin="0 4"/>
 

Content.Client/Administration/UI/ManageSolutions/EditSolutionsWindow.xaml.cs

@@ -1,4 +1,3 @@
-using Content.Client.Administration.Managers;
 using Content.Shared.Administration;
 using Content.Shared.Chemistry.Components;
 using Content.Shared.Chemistry.Reagent;
@@ -21,7 +20,6 @@ namespace Content.Client.Administration.UI.ManageSolutions
         [Dependency] private readonly IClientConsoleHost _consoleHost = default!;
         [Dependency] private readonly IEntityManager _entityManager = default!;
         [Dependency] private readonly IClientGameTiming _timing = default!;
-        [Dependency] private readonly IClientAdminManager _admin = default!;
 
         private NetEntity _target = NetEntity.Invalid;
         private string? _selectedSolution;
@@ -36,11 +34,6 @@ namespace Content.Client.Administration.UI.ManageSolutions
 
             SolutionOption.OnItemSelected += SolutionSelected;
             AddButton.OnPressed += OpenAddReagentWindow;
-            VVButton.OnPressed += OpenVVWindow;
-            SolutionButton.OnPressed += OpenSolutionWindow;
-
-            VVButton.Disabled = !_admin.CanViewVar();
-            SolutionButton.Disabled = !_admin.CanViewVar();
         }
 
         public override void Close()
@@ -278,32 +271,6 @@ namespace Content.Client.Administration.UI.ManageSolutions
             _addReagentWindow.OpenCentered();
         }
 
-        /// <summary>
-        ///     Open the corresponding solution entity in a ViewVariables window.
-        /// </summary>
-        private void OpenVVWindow(BaseButton.ButtonEventArgs obj)
-        {
-            if (_solutions == null
-                || _selectedSolution == null
-                || !_solutions.TryGetValue(_selectedSolution, out var uid)
-                || !_entityManager.TryGetNetEntity(uid, out var netEntity))
-                return;
-            _consoleHost.ExecuteCommand($"vv {netEntity}");
-        }
-
-        /// <summary>
-        ///     Open the corresponding Solution instance in a ViewVariables window.
-        /// </summary>
-        private void OpenSolutionWindow(BaseButton.ButtonEventArgs obj)
-        {
-            if (_solutions == null
-                || _selectedSolution == null
-                || !_solutions.TryGetValue(_selectedSolution, out var uid)
-                || !_entityManager.TryGetNetEntity(uid, out var netEntity))
-                return;
-            _consoleHost.ExecuteCommand($"vv /entity/{netEntity}/Solution/Solution");
-        }
-
         /// <summary>
         ///     When a new solution is selected, set _selectedSolution and update the reagent list.
         /// </summary>

Content.Client/Administration/UI/PlayerPanel/PlayerPanel.xaml

@@ -31,7 +31,6 @@
             <Button Name="FreezeAndMuteToggleButton" Text="{Loc player-panel-freeze-and-mute}" Disabled="True"/>
             <Button Name="BanButton" Text="{Loc player-panel-ban}" Disabled="True"/>
             <controls:ConfirmButton Name="RejuvenateButton" Text="{Loc player-panel-rejuvenate}" Disabled="True"/>
-            <Button Name="CameraButton" Text="{Loc player-panel-camera}" Disabled="True"/>
            </GridContainer>
        </BoxContainer>
     </BoxContainer>

Content.Client/Administration/UI/PlayerPanel/PlayerPanel.xaml.cs

@@ -18,7 +18,6 @@ public sealed partial class PlayerPanel : FancyWindow
     public event Action<NetUserId?>? OnOpenBans;
     public event Action<NetUserId?>? OnAhelp;
     public event Action<string?>? OnKick;
-    public event Action<string?>? OnCamera;
     public event Action<NetUserId?>? OnOpenBanPanel;
     public event Action<NetUserId?, bool>? OnWhitelistToggle;
     public event Action? OnFollow;
@@ -34,27 +33,26 @@ public sealed partial class PlayerPanel : FancyWindow
 
     public PlayerPanel(IClientAdminManager adminManager)
     {
-        RobustXamlLoader.Load(this);
-        _adminManager = adminManager;
+            RobustXamlLoader.Load(this);
+            _adminManager = adminManager;
 
-        UsernameCopyButton.OnPressed += _ => OnUsernameCopy?.Invoke(TargetUsername ?? "");
-        BanButton.OnPressed += _ => OnOpenBanPanel?.Invoke(TargetPlayer);
-        KickButton.OnPressed += _ => OnKick?.Invoke(TargetUsername);
-        CameraButton.OnPressed += _ => OnCamera?.Invoke(TargetUsername);
-        NotesButton.OnPressed += _ => OnOpenNotes?.Invoke(TargetPlayer);
-        ShowBansButton.OnPressed += _ => OnOpenBans?.Invoke(TargetPlayer);
-        AhelpButton.OnPressed += _ => OnAhelp?.Invoke(TargetPlayer);
-        WhitelistToggle.OnPressed += _ =>
-        {
-            OnWhitelistToggle?.Invoke(TargetPlayer, _isWhitelisted);
-            SetWhitelisted(!_isWhitelisted);
-        };
-        FollowButton.OnPressed += _ => OnFollow?.Invoke();
-        FreezeButton.OnPressed += _ => OnFreeze?.Invoke();
-        FreezeAndMuteToggleButton.OnPressed += _ => OnFreezeAndMuteToggle?.Invoke();
-        LogsButton.OnPressed += _ => OnLogs?.Invoke();
-        DeleteButton.OnPressed += _ => OnDelete?.Invoke();
-        RejuvenateButton.OnPressed += _ => OnRejuvenate?.Invoke();
+            UsernameCopyButton.OnPressed += _ => OnUsernameCopy?.Invoke(TargetUsername ?? "");
+            BanButton.OnPressed += _ => OnOpenBanPanel?.Invoke(TargetPlayer);
+            KickButton.OnPressed += _ => OnKick?.Invoke(TargetUsername);
+            NotesButton.OnPressed += _ => OnOpenNotes?.Invoke(TargetPlayer);
+            ShowBansButton.OnPressed += _ => OnOpenBans?.Invoke(TargetPlayer);
+            AhelpButton.OnPressed += _ => OnAhelp?.Invoke(TargetPlayer);
+            WhitelistToggle.OnPressed += _ =>
+            {
+                OnWhitelistToggle?.Invoke(TargetPlayer, _isWhitelisted);
+                SetWhitelisted(!_isWhitelisted);
+            };
+            FollowButton.OnPressed += _ => OnFollow?.Invoke();
+            FreezeButton.OnPressed += _ => OnFreeze?.Invoke();
+            FreezeAndMuteToggleButton.OnPressed += _ => OnFreezeAndMuteToggle?.Invoke();
+            LogsButton.OnPressed += _ => OnLogs?.Invoke();
+            DeleteButton.OnPressed += _ => OnDelete?.Invoke();
+            RejuvenateButton.OnPressed += _ => OnRejuvenate?.Invoke();
     }
 
     public void SetUsername(string player)
@@ -124,7 +122,6 @@ public sealed partial class PlayerPanel : FancyWindow
     {
         BanButton.Disabled = !_adminManager.CanCommand("banpanel");
         KickButton.Disabled = !_adminManager.CanCommand("kick");
-        CameraButton.Disabled = !_adminManager.CanCommand("camera");
         NotesButton.Disabled = !_adminManager.CanCommand("adminnotes");
         ShowBansButton.Disabled = !_adminManager.CanCommand("banlist");
         WhitelistToggle.Disabled =

Content.Client/Administration/UI/PlayerPanel/PlayerPanelEui.cs

@@ -15,7 +15,7 @@ public sealed class PlayerPanelEui : BaseEui
     [Dependency] private readonly IClientAdminManager _admin = default!;
     [Dependency] private readonly IClipboardManager _clipboard = default!;
 
-    private PlayerPanel PlayerPanel { get; }
+    private PlayerPanel PlayerPanel { get;  }
 
     public PlayerPanelEui()
     {
@@ -25,7 +25,6 @@ public sealed class PlayerPanelEui : BaseEui
         PlayerPanel.OnOpenNotes += id => _console.ExecuteCommand($"adminnotes \"{id}\"");
         // Kick command does not support GUIDs
         PlayerPanel.OnKick += username => _console.ExecuteCommand($"kick \"{username}\"");
-        PlayerPanel.OnCamera += username => _console.ExecuteCommand($"camera \"{username}\"");
         PlayerPanel.OnOpenBanPanel += id => _console.ExecuteCommand($"banpanel \"{id}\"");
         PlayerPanel.OnOpenBans += id => _console.ExecuteCommand($"banlist \"{id}\"");
         PlayerPanel.OnAhelp += id => _console.ExecuteCommand($"openahelp \"{id}\"");
@@ -38,7 +37,7 @@ public sealed class PlayerPanelEui : BaseEui
         PlayerPanel.OnFreeze += () => SendMessage(new PlayerPanelFreezeMessage());
         PlayerPanel.OnLogs += () => SendMessage(new PlayerPanelLogsMessage());
         PlayerPanel.OnRejuvenate += () => SendMessage(new PlayerPanelRejuvenationMessage());
-        PlayerPanel.OnDelete += () => SendMessage(new PlayerPanelDeleteMessage());
+        PlayerPanel.OnDelete+= () => SendMessage(new PlayerPanelDeleteMessage());
         PlayerPanel.OnFollow += () => SendMessage(new PlayerPanelFollowMessage());
 
         PlayerPanel.OnClose += () => SendMessage(new CloseEuiMessage());

Content.Client/Administration/UI/SpawnExplosion/SpawnExplosionWindow.xaml.cs

@@ -19,10 +19,10 @@ namespace Content.Client.Administration.UI.SpawnExplosion;
 public sealed partial class SpawnExplosionWindow : DefaultWindow
 {
     [Dependency] private readonly IClientConsoleHost _conHost = default!;
+    [Dependency] private readonly IMapManager _mapManager = default!;
     [Dependency] private readonly IPlayerManager _playerManager = default!;
     [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
     [Dependency] private readonly IEntityManager _entMan = default!;
-    private readonly SharedMapSystem _mapSystem;
     private readonly SharedTransformSystem _transform = default!;
 
     private readonly SpawnExplosionEui _eui;
@@ -38,7 +38,6 @@ public sealed partial class SpawnExplosionWindow : DefaultWindow
     {
         RobustXamlLoader.Load(this);
         IoCManager.InjectDependencies(this);
-        _mapSystem = _entMan.System<SharedMapSystem>();
         _transform = _entMan.System<TransformSystem>();
         _eui = eui;
 
@@ -88,7 +87,7 @@ public sealed partial class SpawnExplosionWindow : DefaultWindow
     {
         _mapData.Clear();
         MapOptions.Clear();
-        foreach (var map in _mapSystem.GetAllMapIds())
+        foreach (var map in _mapManager.GetAllMapIds())
         {
             _mapData.Add(map);
             MapOptions.AddItem(map.ToString());

Content.Client/Administration/UI/Tabs/AdminbusTab/LoadBlueprintsWindow.xaml.cs

@@ -14,9 +14,6 @@ namespace Content.Client.Administration.UI.Tabs.AdminbusTab
     [UsedImplicitly]
     public sealed partial class LoadBlueprintsWindow : DefaultWindow
     {
-        [Dependency] private readonly IEntityManager _entityManager = default!;
-        [Dependency] private readonly IPlayerManager _playerManager = default!;
-
         public LoadBlueprintsWindow()
         {
             RobustXamlLoader.Load(this);
@@ -24,9 +21,9 @@ namespace Content.Client.Administration.UI.Tabs.AdminbusTab
 
         protected override void EnteredTree()
         {
-            var mapSystem = _entityManager.System<SharedMapSystem>();
+            var mapManager = IoCManager.Resolve<IMapManager>();
 
-            foreach (var mapId in mapSystem.GetAllMapIds())
+            foreach (var mapId in mapManager.GetAllMapIds())
             {
                 MapOptions.AddItem(mapId.ToString(), (int) mapId);
             }
@@ -42,19 +39,21 @@ namespace Content.Client.Administration.UI.Tabs.AdminbusTab
 
         private void Reset()
         {
-            var xformSystem = _entityManager.System<SharedTransformSystem>();
-            var player = _playerManager.LocalEntity;
+            var entManager = IoCManager.Resolve<IEntityManager>();
+            var xformSystem = entManager.System<SharedTransformSystem>();
+            var playerManager = IoCManager.Resolve<IPlayerManager>();
+            var player = playerManager.LocalEntity;
 
             var currentMap = MapId.Nullspace;
             var position = Vector2.Zero;
             var rotation = Angle.Zero;
 
-            if (_entityManager.TryGetComponent<TransformComponent>(player, out var xform))
+            if (entManager.TryGetComponent<TransformComponent>(player, out var xform))
             {
                 currentMap = xform.MapID;
                 position = xformSystem.GetWorldPosition(xform);
 
-                if (_entityManager.TryGetComponent<TransformComponent>(xform.GridUid, out var gridXform))
+                if (entManager.TryGetComponent<TransformComponent>(xform.GridUid, out var gridXform))
                 {
                     rotation = xformSystem.GetWorldRotation(gridXform);
                 }

Content.Client/Administration/UI/Tabs/ObjectsTab/ObjectsTab.xaml.cs

@@ -57,43 +57,12 @@ public sealed partial class ObjectsTab : Control
 
     private void TeleportTo(NetEntity nent)
     {
-        var selection = _selections[ObjectTypeOptions.SelectedId];
-        switch (selection)
-        {
-            case ObjectsTabSelection.Grids:
-                {
-                    // directly teleport to the entity
-                    _console.ExecuteCommand($"tpto {nent}");
-                }
-                break;
-            case ObjectsTabSelection.Maps:
-                {
-                    // teleport to the map, not to the map entity (which is in nullspace)
-                    if (!_entityManager.TryGetEntity(nent, out var map) || !_entityManager.TryGetComponent<MapComponent>(map, out var mapComp))
-                        break;
-                    _console.ExecuteCommand($"tp 0 0 {mapComp.MapId}");
-                    break;
-                }
-            case ObjectsTabSelection.Stations:
-                {
-                    // teleport to the station's largest grid, not to the station entity (which is in nullspace)
-                    if (!_entityManager.TryGetEntity(nent, out var station))
-                        break;
-                    var largestGrid = _entityManager.EntitySysManager.GetEntitySystem<StationSystem>().GetLargestGrid(station.Value);
-                    if (largestGrid == null)
-                        break;
-                    _console.ExecuteCommand($"tpto {largestGrid.Value}");
-                    break;
-                }
-            default:
-                throw new NotImplementedException();
-        }
+        _console.ExecuteCommand($"tpto {nent}");
     }
 
     private void Delete(NetEntity nent)
     {
         _console.ExecuteCommand($"delete {nent}");
-        RefreshObjectList();
     }
 
     public void RefreshObjectList()
@@ -107,24 +76,28 @@ public sealed partial class ObjectsTab : Control
         switch (selection)
         {
             case ObjectsTabSelection.Stations:
-                entities.AddRange(_entityManager.EntitySysManager.GetEntitySystem<StationSystem>().GetStationNames());
+                entities.AddRange(_entityManager.EntitySysManager.GetEntitySystem<StationSystem>().Stations);
                 break;
             case ObjectsTabSelection.Grids:
+            {
+                var query = _entityManager.AllEntityQueryEnumerator<MapGridComponent, MetaDataComponent>();
+                while (query.MoveNext(out var uid, out _, out var metadata))
                 {
-                    var query = _entityManager.AllEntityQueryEnumerator<MapGridComponent, MetaDataComponent>();
-                    while (query.MoveNext(out var uid, out _, out var metadata))
-                        entities.Add((metadata.EntityName, _entityManager.GetNetEntity(uid)));
-
-                    break;
+                    entities.Add((metadata.EntityName, _entityManager.GetNetEntity(uid)));
                 }
+
+                break;
+            }
             case ObjectsTabSelection.Maps:
+            {
+                var query = _entityManager.AllEntityQueryEnumerator<MapComponent, MetaDataComponent>();
+                while (query.MoveNext(out var uid, out _, out var metadata))
                 {
-                    var query = _entityManager.AllEntityQueryEnumerator<MapComponent, MetaDataComponent>();
-                    while (query.MoveNext(out var uid, out _, out var metadata))
-                        entities.Add((metadata.EntityName, _entityManager.GetNetEntity(uid)));
-
-                    break;
+                    entities.Add((metadata.EntityName, _entityManager.GetNetEntity(uid)));
                 }
+
+                break;
+            }
             default:
                 throw new ArgumentOutOfRangeException(nameof(selection), selection, null);
         }
@@ -159,8 +132,8 @@ public sealed partial class ObjectsTab : Control
         entry.OnTeleport += TeleportTo;
         entry.OnDelete += Delete;
         button.ToolTip = $"{info.Name}, {info.Entity}";
+
         button.AddChild(entry);
-        button.StyleClasses.Clear();
     }
 
     private bool DataFilterCondition(string filter, ListData listData)

Content.Client/Administration/UI/Tabs/ObjectsTab/ObjectsTabEntry.xaml

@@ -1,6 +1,5 @@
 <PanelContainer xmlns="https://spacestation14.io"
                 xmlns:customControls="clr-namespace:Content.Client.Administration.UI.CustomControls"
-                xmlns:controls="clr-namespace:Content.Client.UserInterface.Controls"
                 Name="BackgroundColorPanel">
     <BoxContainer Orientation="Horizontal"
                   HorizontalExpand="True"
@@ -21,7 +20,7 @@
                HorizontalExpand="True"
                ClipText="True"/>
         <customControls:VSeparator/>
-        <controls:ConfirmButton Name="DeleteButton"
+        <Button Name="DeleteButton"
                 Text="{Loc object-tab-entity-delete}"
                 SizeFlagsStretchRatio="3"
                 HorizontalExpand="True"

Content.Client/Administration/UI/Tabs/PlayerTab/PlayerTab.xaml.cs

@@ -172,7 +172,6 @@ public sealed partial class PlayerTab : Control
             _playerTabSymbolSetting);
         button.AddChild(entry);
         button.ToolTip = $"{player.Username}, {player.CharacterName}, {player.IdentityName}, {player.StartingJob}";
-        button.StyleClasses.Clear();
     }
 
     /// <summary>

Content.Client/Administration/UI/Tabs/PlayerTab/PlayerTabEntry.xaml.cs

@@ -1,11 +1,9 @@
 using Content.Shared.Administration;
-using Content.Shared.Mind;
 using Content.Shared.Roles;
 using Robust.Client.AutoGenerated;
 using Robust.Client.Graphics;
 using Robust.Client.UserInterface.Controls;
 using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Prototypes;
 
 namespace Content.Client.Administration.UI.Tabs.PlayerTab;
 
@@ -13,7 +11,6 @@ namespace Content.Client.Administration.UI.Tabs.PlayerTab;
 public sealed partial class PlayerTabEntry : PanelContainer
 {
     [Dependency] private readonly IEntityManager _entMan = default!;
-    [Dependency] private readonly IPrototypeManager _prototype = default!;
 
     public PlayerTabEntry(
         PlayerInfo player,
@@ -26,8 +23,6 @@ public sealed partial class PlayerTabEntry : PanelContainer
         RobustXamlLoader.Load(this);
         var roles = _entMan.System<SharedRoleSystem>();
 
-        var rolePrototype = player.RoleProto == null ? null : _prototype.Index(player.RoleProto.Value);
-
         UsernameLabel.Text = player.Username;
         if (!player.Connected)
             UsernameLabel.StyleClasses.Add("Disabled");
@@ -62,19 +57,19 @@ public sealed partial class PlayerTabEntry : PanelContainer
                 break;
             default:
             case AdminPlayerTabSymbolOption.Specific:
-                symbol = player.Antag ? rolePrototype?.Symbol ?? RoleTypePrototype.FallbackSymbol : string.Empty;
+                symbol = player.Antag ? player.RoleProto.Symbol : string.Empty;
                 break;
         }
 
         CharacterLabel.Text = Loc.GetString("player-tab-character-name-antag-symbol", ("symbol", symbol), ("name", player.CharacterName));
 
         if (player.Antag && colorAntags)
-            CharacterLabel.FontColorOverride = rolePrototype?.Color ?? RoleTypePrototype.FallbackColor;
+            CharacterLabel.FontColorOverride = player.RoleProto.Color;
         if (player.IdentityName != player.CharacterName)
             CharacterLabel.Text += $" [{player.IdentityName}]";
 
-        var roletype = RoleTypeLabel.Text = Loc.GetString(rolePrototype?.Name ?? RoleTypePrototype.FallbackName);
-        var subtype = roles.GetRoleSubtypeLabel(rolePrototype?.Name ?? RoleTypePrototype.FallbackName, player.Subtype);
+        var roletype = RoleTypeLabel.Text = Loc.GetString(player.RoleProto.Name);
+        var subtype = roles.GetRoleSubtypeLabel(player.RoleProto.Name, player.Subtype);
         switch (roleSetting)
         {
             case AdminPlayerTabRoleTypeOption.RoleTypeSubtype:
@@ -97,7 +92,7 @@ public sealed partial class PlayerTabEntry : PanelContainer
         }
 
         if (colorRoles)
-            RoleTypeLabel.FontColorOverride = rolePrototype?.Color ?? RoleTypePrototype.FallbackColor;
+            RoleTypeLabel.FontColorOverride = player.RoleProto.Color;
         BackgroundColorPanel.PanelOverride = styleBoxFlat;
         OverallPlaytimeLabel.Text = player.PlaytimeString;
     }

Content.Client/Administration/UI/Tabs/RoundTab.xaml

@@ -1,13 +1,13 @@
 <Control
     xmlns="https://spacestation14.io"
-    xmlns:controls="clr-namespace:Content.Client.UserInterface.Controls"
+    xmlns:cc="clr-namespace:Content.Client.Administration.UI.CustomControls"
     Margin="4"
     MinSize="50 50">
     <GridContainer
         Columns="3">
-        <controls:ConfirmButton Name="StartRound" Text="{Loc administration-ui-round-tab-start-round}" />
-        <controls:ConfirmButton Name="EndRound" Text="{Loc administration-ui-round-tab-end-round}" />
-        <controls:ConfirmButton Name="RestartRound" Text="{Loc administration-ui-round-tab-restart-round}" />
-        <controls:ConfirmButton Name="RestartRoundNow" Text="{Loc administration-ui-round-tab-restart-round-now}" />
+        <cc:CommandButton Command="startround" Text="{Loc administration-ui-round-tab-start-round}" />
+        <cc:CommandButton Command="endround" Text="{Loc administration-ui-round-tab-end-round}" />
+        <cc:CommandButton Command="restartround" Text="{Loc administration-ui-round-tab-restart-round}" />
+        <cc:CommandButton Command="restartroundnow" Text="{Loc administration-ui-round-tab-restart-round-now}" />
     </GridContainer>
 </Control>

Content.Client/Administration/UI/Tabs/RoundTab.xaml.cs

@@ -1,24 +1,10 @@
 using Robust.Client.AutoGenerated;
-using Robust.Client.Console;
 using Robust.Client.UserInterface;
-using Robust.Client.UserInterface.XAML;
 
 namespace Content.Client.Administration.UI.Tabs
 {
     [GenerateTypedNameReferences]
     public sealed partial class RoundTab : Control
     {
-        [Dependency] private readonly IClientConsoleHost _console = default!;
-
-        public RoundTab()
-        {
-            RobustXamlLoader.Load(this);
-            IoCManager.InjectDependencies(this);
-
-            StartRound.OnPressed += _ => _console.ExecuteCommand("startround");
-            EndRound.OnPressed += _ => _console.ExecuteCommand("endround");
-            RestartRound.OnPressed += _ => _console.ExecuteCommand("restartround");
-            RestartRoundNow.OnPressed += _ => _console.ExecuteCommand("restartroundnow");
-        }
     }
 }

Content.Client/Administration/UI/Tabs/ServerTab.xaml

@@ -1,12 +1,11 @@
 <Control
     xmlns="https://spacestation14.io"
     xmlns:cc="clr-namespace:Content.Client.Administration.UI.CustomControls"
-    xmlns:controls="clr-namespace:Content.Client.UserInterface.Controls"
     Margin="4"
     MinSize="50 50">
     <GridContainer
         Columns="4" >
-        <controls:ConfirmButton Name="ServerShutdownButton" Text="{Loc server-shutdown}" />
+        <cc:CommandButton Command="shutdown" Text="{Loc server-shutdown}" />
         <cc:CommandButton Name="SetOocButton" Command="setooc" Text="{Loc server-ooc-toggle}" ToggleMode="True" />
         <cc:CommandButton Name="SetLoocButton" Command="setlooc" Text="{Loc server-looc-toggle}" ToggleMode="True" />
     </GridContainer>

Content.Client/Administration/UI/Tabs/ServerTab.xaml.cs

@@ -1,6 +1,5 @@
 using Content.Shared.CCVar;
 using Robust.Client.AutoGenerated;
-using Robust.Client.Console;
 using Robust.Client.UserInterface;
 using Robust.Client.UserInterface.XAML;
 using Robust.Shared.Configuration;
@@ -11,7 +10,6 @@ namespace Content.Client.Administration.UI.Tabs
     public sealed partial class ServerTab : Control
     {
         [Dependency] private readonly IConfigurationManager _config = default!;
-        [Dependency] private readonly IClientConsoleHost _console = default!;
 
         public ServerTab()
         {
@@ -20,8 +18,6 @@ namespace Content.Client.Administration.UI.Tabs
 
             _config.OnValueChanged(CCVars.OocEnabled, OocEnabledChanged, true);
             _config.OnValueChanged(CCVars.LoocEnabled, LoocEnabledChanged, true);
-
-            ServerShutdownButton.OnPressed += _ => _console.ExecuteCommand("shutdown");
         }
 
         private void OocEnabledChanged(bool value)

Content.Client/AlertLevel/AlertLevelDisplaySystem.cs

@@ -8,8 +8,6 @@ namespace Content.Client.AlertLevel;
 
 public sealed class AlertLevelDisplaySystem : EntitySystem
 {
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
     public override void Initialize()
     {
         base.Initialize();
@@ -23,26 +21,26 @@ public sealed class AlertLevelDisplaySystem : EntitySystem
         {
             return;
         }
-        var layer = _sprite.LayerMapReserve((uid, args.Sprite), AlertLevelDisplay.Layer);
+        var layer = args.Sprite.LayerMapReserveBlank(AlertLevelDisplay.Layer);
 
         if (args.AppearanceData.TryGetValue(AlertLevelDisplay.Powered, out var poweredObject))
         {
-            _sprite.LayerSetVisible((uid, args.Sprite), layer, poweredObject is true);
+            args.Sprite.LayerSetVisible(layer, poweredObject is true);
         }
 
         if (!args.AppearanceData.TryGetValue(AlertLevelDisplay.CurrentLevel, out var level))
         {
-            _sprite.LayerSetRsiState((uid, args.Sprite), layer, alertLevelDisplay.AlertVisuals.Values.First());
+            args.Sprite.LayerSetState(layer, alertLevelDisplay.AlertVisuals.Values.First());
             return;
         }
 
-        if (alertLevelDisplay.AlertVisuals.TryGetValue((string)level, out var visual))
+        if (alertLevelDisplay.AlertVisuals.TryGetValue((string) level, out var visual))
         {
-            _sprite.LayerSetRsiState((uid, args.Sprite), layer, visual);
+            args.Sprite.LayerSetState(layer, visual);
         }
         else
         {
-            _sprite.LayerSetRsiState((uid, args.Sprite), layer, alertLevelDisplay.AlertVisuals.Values.First());
+            args.Sprite.LayerSetState(layer, alertLevelDisplay.AlertVisuals.Values.First());
         }
     }
 }

Content.Client/Alerts/ClientAlertsSystem.cs

@@ -2,7 +2,6 @@ using System.Linq;
 using Content.Shared.Alert;
 using JetBrains.Annotations;
 using Robust.Client.Player;
-using Robust.Client.UserInterface;
 using Robust.Shared.GameStates;
 using Robust.Shared.Player;
 using Robust.Shared.Prototypes;
@@ -16,7 +15,6 @@ public sealed class ClientAlertsSystem : AlertsSystem
 
     [Dependency] private readonly IPlayerManager _playerManager = default!;
     [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
-    [Dependency] private readonly IUserInterfaceManager _ui = default!;
 
     public event EventHandler? ClearAlerts;
     public event EventHandler<IReadOnlyDictionary<AlertKey, AlertState>>? SyncAlerts;
@@ -29,12 +27,6 @@ public sealed class ClientAlertsSystem : AlertsSystem
         SubscribeLocalEvent<AlertsComponent, LocalPlayerDetachedEvent>(OnPlayerDetached);
         SubscribeLocalEvent<AlertsComponent, ComponentHandleState>(OnHandleState);
     }
-
-    protected override void HandledAlert()
-    {
-        _ui.ClickSound();
-    }
-
     protected override void LoadPrototypes()
     {
         base.LoadPrototypes();
@@ -60,24 +52,8 @@ public sealed class ClientAlertsSystem : AlertsSystem
         if (args.Current is not AlertComponentState cast)
             return;
 
-        // Save all client-sided alerts to later put back in
-        var clientAlerts = new Dictionary<AlertKey, AlertState>();
-        foreach (var alert in alerts.Comp.Alerts)
-        {
-            if (alert.Key.AlertType != null && TryGet(alert.Key.AlertType.Value, out var alertProto))
-            {
-                if (alertProto.ClientHandled)
-                    clientAlerts[alert.Key] = alert.Value;
-            }
-        }
-
         alerts.Comp.Alerts = new(cast.Alerts);
 
-        foreach (var alert in clientAlerts)
-        {
-            alerts.Comp.Alerts[alert.Key] = alert.Value;
-        }
-
         UpdateHud(alerts);
     }
 

Content.Client/Alerts/GenericCounterAlertSystem.cs

@@ -1,90 +0,0 @@
-using System.Numerics;
-using Content.Shared.Alert.Components;
-using Robust.Client.GameObjects;
-using Robust.Client.Graphics;
-
-namespace Content.Client.Alerts;
-
-/// <summary>
-/// This handles <see cref="GenericCounterAlertComponent"/>
-/// </summary>
-public sealed class GenericCounterAlertSystem : EntitySystem
-{
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
-    /// <inheritdoc/>
-    public override void Initialize()
-    {
-        SubscribeLocalEvent<GenericCounterAlertComponent, UpdateAlertSpriteEvent>(OnUpdateAlertSprite);
-    }
-
-    private void OnUpdateAlertSprite(Entity<GenericCounterAlertComponent> ent, ref UpdateAlertSpriteEvent args)
-    {
-        var sprite = args.SpriteViewEnt.Comp;
-
-        var ev = new GetGenericAlertCounterAmountEvent(args.Alert);
-        RaiseLocalEvent(args.ViewerEnt, ref ev);
-
-        if (!ev.Handled)
-            return;
-
-        // It cannot be null if its handled, but good to check to avoid ugly null ignores.
-        if (ev.Amount == null)
-            return;
-
-        // How many digits can we display
-        var maxDigitCount = GetMaxDigitCount((ent, ent, sprite));
-
-        // Clamp it to a positive number that we can actually display in full (no rollover to 0)
-        var amount = (int) Math.Clamp(ev.Amount.Value, 0, Math.Pow(10, maxDigitCount) - 1);
-
-        // This is super wack but ig it works?
-        var digitCount = ent.Comp.HideLeadingZeroes
-            ? amount.ToString().Length
-            : maxDigitCount;
-
-        if (ent.Comp.HideLeadingZeroes)
-        {
-            for (var i = 0; i < ent.Comp.DigitKeys.Count; i++)
-            {
-                if (!_sprite.LayerMapTryGet(ent.Owner, ent.Comp.DigitKeys[i], out var layer, false))
-                    continue;
-
-                _sprite.LayerSetVisible(ent.Owner, layer, i <= digitCount - 1);
-            }
-        }
-
-        // ReSharper disable once PossibleLossOfFraction
-        var baseOffset = (ent.Comp.AlertSize.X - digitCount * ent.Comp.GlyphWidth) / 2 * (1f / EyeManager.PixelsPerMeter);
-
-        for (var i = 0; i < ent.Comp.DigitKeys.Count; i++)
-        {
-            if (!_sprite.LayerMapTryGet(ent.Owner, ent.Comp.DigitKeys[i], out var layer, false))
-                continue;
-
-            var result = amount / (int) Math.Pow(10, i) % 10;
-            _sprite.LayerSetRsiState(ent.Owner, layer, result.ToString());
-
-            if (ent.Comp.CenterGlyph)
-            {
-                var offset = baseOffset + (digitCount - 1 - i) * ent.Comp.GlyphWidth * (1f / EyeManager.PixelsPerMeter);
-                _sprite.LayerSetOffset(ent.Owner, layer, new Vector2(offset, 0));
-            }
-        }
-    }
-
-    /// <summary>
-    /// Gets the number of digits that we can display.
-    /// </summary>
-    /// <returns>The number of digits.</returns>
-    private int GetMaxDigitCount(Entity<GenericCounterAlertComponent, SpriteComponent> ent)
-    {
-        for (var i = ent.Comp1.DigitKeys.Count - 1; i >= 0; i--)
-        {
-            if (_sprite.LayerExists((ent.Owner, ent.Comp2), ent.Comp1.DigitKeys[i]))
-                return i + 1;
-        }
-
-        return 0;
-    }
-}

Content.Client/Alerts/UpdateAlertSpriteEvent.cs

@@ -11,14 +11,11 @@ public record struct UpdateAlertSpriteEvent
 {
     public Entity<SpriteComponent> SpriteViewEnt;
 
-    public EntityUid ViewerEnt;
-
     public AlertPrototype Alert;
 
-    public UpdateAlertSpriteEvent(Entity<SpriteComponent> spriteViewEnt, EntityUid viewerEnt, AlertPrototype alert)
+    public UpdateAlertSpriteEvent(Entity<SpriteComponent> spriteViewEnt, AlertPrototype alert)
     {
         SpriteViewEnt = spriteViewEnt;
-        ViewerEnt = viewerEnt;
         Alert = alert;
     }
 }

Content.Client/Animals/Systems/ParrotMemorySystem.cs

@@ -1,5 +0,0 @@
-using Content.Shared.Animals.Systems;
-
-namespace Content.Client.Animals.Systems;
-
-public sealed class ParrotMemorySystem : SharedParrotMemorySystem;

Content.Client/Animations/EntityPickupAnimationSystem.cs

@@ -15,7 +15,6 @@ public sealed class EntityPickupAnimationSystem : EntitySystem
 {
     [Dependency] private readonly AnimationPlayerSystem _animations = default!;
     [Dependency] private readonly MetaDataSystem _metaData = default!;
-    [Dependency] private readonly SpriteSystem _sprite = default!;
     [Dependency] private readonly TransformSystem _transform = default!;
 
     public override void Initialize()
@@ -57,8 +56,8 @@ public sealed class EntityPickupAnimationSystem : EntitySystem
         }
 
         var sprite = Comp<SpriteComponent>(animatableClone);
-        _sprite.CopySprite((uid, sprite0), (animatableClone, sprite));
-        _sprite.SetVisible((animatableClone, sprite), true);
+        sprite.CopyFrom(sprite0);
+        sprite.Visible = true;
 
         var animations = Comp<AnimationPlayerComponent>(animatableClone);
 

Content.Client/Anomaly/AnomalyScannerScreenComponent.cs

@@ -1,40 +0,0 @@
-using Robust.Client.Graphics;
-using SixLabors.ImageSharp.PixelFormats;
-
-namespace Content.Client.Anomaly;
-
-/// <summary>
-/// This component creates and handles the drawing of a ScreenTexture to be used on the Anomaly Scanner
-/// for an indicator of Anomaly Severity.
-/// </summary>
-/// <remarks>
-/// In the future I would like to make this a more generic "DynamicTextureComponent" that can contain a dictionary
-/// of texture components like "Bar(offset, size, minimumValue, maximumValue, AppearanceKey, LayerMapKey)" that can
-/// just draw a bar or other basic drawn element that will show up on a texture layer.
-/// </remarks>
-[RegisterComponent]
-[Access(typeof(AnomalyScannerSystem))]
-public sealed partial class AnomalyScannerScreenComponent : Component
-{
-    /// <summary>
-    /// This is the texture drawn as a layer on the Anomaly Scanner device.
-    /// </summary>
-    public OwnedTexture? ScreenTexture;
-
-    /// <summary>
-    /// A small buffer that we can reuse to draw the severity bar.
-    /// </summary>
-    public Rgba32[]? BarBuf;
-
-    /// <summary>
-    /// The position of the top-left of the severity bar in pixels.
-    /// </summary>
-    [DataField(readOnly: true)]
-    public Vector2i Offset = new Vector2i(12, 17);
-
-    /// <summary>
-    /// The width and height of the severity bar in pixels.
-    /// </summary>
-    [DataField(readOnly: true)]
-    public Vector2i Size = new Vector2i(10, 3);
-}

Content.Client/Anomaly/AnomalyScannerSystem.cs

@@ -1,110 +0,0 @@
-using System.Numerics;
-using Content.Shared.Anomaly;
-using Robust.Client.GameObjects;
-using Robust.Client.Graphics;
-using Robust.Shared.Utility;
-using SixLabors.ImageSharp.PixelFormats;
-
-namespace Content.Client.Anomaly;
-
-/// <inheritdoc cref="SharedAnomalyScannerSystem"/>
-public sealed class AnomalyScannerSystem : SharedAnomalyScannerSystem
-{
-    [Dependency] private readonly IClyde _clyde = default!;
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
-    private const float MaxHueDegrees = 360f;
-    private const float GreenHueDegrees = 110f;
-    private const float RedHueDegrees = 0f;
-    private const float GreenHue = GreenHueDegrees / MaxHueDegrees;
-    private const float RedHue = RedHueDegrees / MaxHueDegrees;
-
-
-    // Just an array to initialize the pixels of a new OwnedTexture
-    private static readonly Rgba32[] EmptyTexture = new Rgba32[32*32];
-
-    public override void Initialize()
-    {
-        base.Initialize();
-
-        SubscribeLocalEvent<AnomalyScannerScreenComponent, ComponentInit>(OnComponentInit);
-        SubscribeLocalEvent<AnomalyScannerScreenComponent, ComponentStartup>(OnComponentStartup);
-        SubscribeLocalEvent<AnomalyScannerScreenComponent, AppearanceChangeEvent>(OnScannerAppearanceChanged);
-    }
-
-    private void OnComponentInit(Entity<AnomalyScannerScreenComponent> ent, ref ComponentInit args)
-    {
-        if(!_sprite.TryGetLayer(ent.Owner, AnomalyScannerVisualLayers.Base, out var layer, true))
-            return;
-
-        // Allocate the OwnedTexture
-        ent.Comp.ScreenTexture = _clyde.CreateBlankTexture<Rgba32>(layer.PixelSize);
-
-        if (layer.PixelSize.X < ent.Comp.Offset.X + ent.Comp.Size.X ||
-            layer.PixelSize.Y < ent.Comp.Offset.Y + ent.Comp.Size.Y)
-        {
-            // If the bar doesn't fit, just bail here, ScreenTexture and BarBuf will remain null, and appearance updates
-            // will do nothing.
-            DebugTools.Assert(false, "AnomalyScannerScreenComponent: Bar does not fit within sprite");
-            return;
-        }
-
-
-        // Initialize the texture
-        ent.Comp.ScreenTexture.SetSubImage((0, 0), layer.PixelSize, new ReadOnlySpan<Rgba32>(EmptyTexture));
-
-        // Initialize bar drawing buffer
-        ent.Comp.BarBuf = new Rgba32[ent.Comp.Size.X * ent.Comp.Size.Y];
-    }
-
-    private void OnComponentStartup(Entity<AnomalyScannerScreenComponent> ent, ref ComponentStartup args)
-    {
-        if (!TryComp<SpriteComponent>(ent, out var sprite))
-            return;
-
-        _sprite.LayerSetTexture((ent, sprite), AnomalyScannerVisualLayers.Screen, ent.Comp.ScreenTexture);
-    }
-
-    private void OnScannerAppearanceChanged(Entity<AnomalyScannerScreenComponent> ent, ref AppearanceChangeEvent args)
-    {
-        if (args.Sprite is null || ent.Comp.ScreenTexture is null || ent.Comp.BarBuf is null)
-            return;
-
-        args.AppearanceData.TryGetValue(AnomalyScannerVisuals.AnomalySeverity, out var severityObj);
-        if (severityObj is not float severity)
-            severity = 0;
-
-        // Get the bar length
-        var barLength = (int)(severity * ent.Comp.Size.X);
-
-        // Calculate the bar color
-        // Hue "angle" of two colors to interpolate between depending on severity
-        // Just a lerp from Green hue at severity = 0.5 to Red hue at 1.0
-        var hue = Math.Clamp(2*GreenHue * (1 - severity), RedHue, GreenHue);
-        var color = new Rgba32(Color.FromHsv(new Vector4(hue, 1f, 1f, 1f)).RGBA);
-
-        var transparent = new Rgba32(0, 0, 0, 255);
-
-        for(var y = 0; y < ent.Comp.Size.Y; y++)
-        {
-            for (var x = 0; x < ent.Comp.Size.X; x++)
-            {
-                ent.Comp.BarBuf[y*ent.Comp.Size.X + x]  = x < barLength ? color : transparent;
-            }
-        }
-
-        // Copy the buffer to the texture
-        try
-        {
-            ent.Comp.ScreenTexture.SetSubImage(
-                ent.Comp.Offset,
-                ent.Comp.Size,
-                new ReadOnlySpan<Rgba32>(ent.Comp.BarBuf)
-            );
-        }
-        catch (IndexOutOfRangeException)
-        {
-            Log.Warning($"Bar dimensions out of bounds with the texture on entity {ent.Owner}");
-        }
-    }
-}

Content.Client/Anomaly/AnomalySystem.cs

@@ -7,11 +7,10 @@ using Robust.Shared.Timing;
 
 namespace Content.Client.Anomaly;
 
-public sealed partial class AnomalySystem : SharedAnomalySystem
+public sealed class AnomalySystem : SharedAnomalySystem
 {
     [Dependency] private readonly IGameTiming _timing = default!;
     [Dependency] private readonly FloatingVisualizerSystem _floating = default!;
-    [Dependency] private readonly SpriteSystem _sprite = default!;
 
     /// <inheritdoc/>
     public override void Initialize()
@@ -24,7 +23,6 @@ public sealed partial class AnomalySystem : SharedAnomalySystem
 
         SubscribeLocalEvent<AnomalySupercriticalComponent, ComponentShutdown>(OnShutdown);
     }
-
     private void OnStartup(EntityUid uid, AnomalyComponent component, ComponentStartup args)
     {
         _floating.FloatAnimation(uid, component.FloatingOffset, component.AnimationKey, component.AnimationTime);
@@ -51,30 +49,30 @@ public sealed partial class AnomalySystem : SharedAnomalySystem
         if (HasComp<AnomalySupercriticalComponent>(uid))
             pulsing = true;
 
-        if (!_sprite.LayerMapTryGet((uid, sprite), AnomalyVisualLayers.Base, out var layer, false) ||
-            !_sprite.LayerMapTryGet((uid, sprite), AnomalyVisualLayers.Animated, out var animatedLayer, false))
+        if (!sprite.LayerMapTryGet(AnomalyVisualLayers.Base, out var layer) ||
+            !sprite.LayerMapTryGet(AnomalyVisualLayers.Animated, out var animatedLayer))
             return;
 
-        _sprite.LayerSetVisible((uid, sprite), layer, !pulsing);
-        _sprite.LayerSetVisible((uid, sprite), animatedLayer, pulsing);
+        sprite.LayerSetVisible(layer, !pulsing);
+        sprite.LayerSetVisible(animatedLayer, pulsing);
     }
 
     public override void Update(float frameTime)
     {
         base.Update(frameTime);
 
-        var query = EntityQueryEnumerator<AnomalyComponent, AnomalySupercriticalComponent, SpriteComponent>();
+        var query = EntityQueryEnumerator<AnomalySupercriticalComponent, SpriteComponent>();
 
-        while (query.MoveNext(out var uid, out var anomaly, out var super, out var sprite))
+        while (query.MoveNext(out var super, out var sprite))
         {
-            var completion = 1f - (float) ((super.EndTime - _timing.CurTime) / anomaly.SupercriticalDuration);
+            var completion = 1f - (float) ((super.EndTime - _timing.CurTime) / super.SupercriticalDuration);
             var scale = completion * (super.MaxScaleAmount - 1f) + 1f;
-            _sprite.SetScale((uid, sprite), new Vector2(scale, scale));
+            sprite.Scale = new Vector2(scale, scale);
 
-            var transparency = (byte)(65 * (1f - completion) + 190);
+            var transparency = (byte) (65 * (1f - completion) + 190);
             if (transparency < sprite.Color.AByte)
             {
-                _sprite.SetColor((uid, sprite), sprite.Color.WithAlpha(transparency));
+                sprite.Color = sprite.Color.WithAlpha(transparency);
             }
         }
     }
@@ -84,7 +82,7 @@ public sealed partial class AnomalySystem : SharedAnomalySystem
         if (!TryComp<SpriteComponent>(ent, out var sprite))
             return;
 
-        _sprite.SetScale((ent.Owner, sprite), Vector2.One);
-        _sprite.SetColor((ent.Owner, sprite), sprite.Color.WithAlpha(1f));
+        sprite.Scale = Vector2.One;
+        sprite.Color = sprite.Color.WithAlpha(1f);
     }
 }

Content.Client/Anomaly/Effects/ClientInnerBodySystem.cs

@@ -1,14 +1,12 @@
 using Content.Shared.Anomaly.Components;
 using Content.Shared.Anomaly.Effects;
-using Content.Shared.Humanoid;
+using Content.Shared.Body.Components;
 using Robust.Client.GameObjects;
 
 namespace Content.Client.Anomaly.Effects;
 
 public sealed class ClientInnerBodyAnomalySystem : SharedInnerBodyAnomalySystem
 {
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
     public override void Initialize()
     {
         SubscribeLocalEvent<InnerBodyAnomalyComponent, AfterAutoHandleStateEvent>(OnAfterHandleState);
@@ -23,19 +21,21 @@ public sealed class ClientInnerBodyAnomalySystem : SharedInnerBodyAnomalySystem
         if (ent.Comp.FallbackSprite is null)
             return;
 
-        var index = _sprite.LayerMapReserve((ent.Owner, sprite), ent.Comp.LayerMap);
+        if (!sprite.LayerMapTryGet(ent.Comp.LayerMap, out var index))
+            index = sprite.LayerMapReserveBlank(ent.Comp.LayerMap);
 
-        if (TryComp<HumanoidAppearanceComponent>(ent, out var humanoidAppearance) &&
-            ent.Comp.SpeciesSprites.TryGetValue(humanoidAppearance.Species, out var speciesSprite))
+        if (TryComp<BodyComponent>(ent, out var body) &&
+            body.Prototype is not null &&
+            ent.Comp.SpeciesSprites.TryGetValue(body.Prototype.Value, out var speciesSprite))
         {
-            _sprite.LayerSetSprite((ent.Owner, sprite), index, speciesSprite);
+            sprite.LayerSetSprite(index, speciesSprite);
         }
         else
         {
-            _sprite.LayerSetSprite((ent.Owner, sprite), index, ent.Comp.FallbackSprite);
+            sprite.LayerSetSprite(index, ent.Comp.FallbackSprite);
         }
 
-        _sprite.LayerSetVisible((ent.Owner, sprite), index, true);
+        sprite.LayerSetVisible(index, true);
         sprite.LayerSetShader(index, "unshaded");
     }
 
@@ -44,7 +44,7 @@ public sealed class ClientInnerBodyAnomalySystem : SharedInnerBodyAnomalySystem
         if (!TryComp<SpriteComponent>(ent, out var sprite))
             return;
 
-        var index = _sprite.LayerMapGet((ent.Owner, sprite), ent.Comp.LayerMap);
-        _sprite.LayerSetVisible((ent.Owner, sprite), index, false);
+        var index = sprite.LayerMapGet(ent.Comp.LayerMap);
+        sprite.LayerSetVisible(index, false);
     }
 }

Content.Client/Anomaly/Ui/AnomalyScannerBoundUserInterface.cs

@@ -20,7 +20,6 @@ public sealed class AnomalyScannerBoundUserInterface : BoundUserInterface
 
         _menu = new AnomalyScannerMenu();
         _menu.OpenCentered();
-        _menu.OnClose += Close;
     }
 
     protected override void UpdateState(BoundUserInterfaceState state)

Content.Client/Atmos/AlignAtmosPipeLayers.cs

@@ -1,203 +0,0 @@
-using Content.Client.Construction;
-using Content.Shared.Atmos.Components;
-using Content.Shared.Atmos.EntitySystems;
-using Content.Shared.Construction.Prototypes;
-using Robust.Client.GameObjects;
-using Robust.Client.Graphics;
-using Robust.Client.Placement;
-using Robust.Client.Placement.Modes;
-using Robust.Client.Utility;
-using Robust.Shared.Enums;
-using Robust.Shared.Map;
-using Robust.Shared.Map.Components;
-using Robust.Shared.Prototypes;
-using Robust.Shared.Utility;
-using System.Numerics;
-using static Robust.Client.Placement.PlacementManager;
-
-namespace Content.Client.Atmos;
-
-/// <summary>
-/// Allows users to place atmos pipes on different layers depending on how the mouse cursor is positioned within a grid tile.
-/// </summary>
-/// <remarks>
-/// This placement mode is not on the engine because it is content specific.
-/// </remarks>
-public sealed class AlignAtmosPipeLayers : SnapgridCenter
-{
-    [Dependency] private readonly IEntityManager _entityManager = default!;
-    [Dependency] private readonly IPrototypeManager _protoManager = default!;
-    [Dependency] private readonly IMapManager _mapManager = default!;
-    [Dependency] private readonly IEyeManager _eyeManager = default!;
-
-    private readonly SharedMapSystem _mapSystem;
-    private readonly SharedTransformSystem _transformSystem;
-    private readonly SharedAtmosPipeLayersSystem _pipeLayersSystem;
-    private readonly SpriteSystem _spriteSystem;
-
-    private const float SearchBoxSize = 2f;
-    private EntityCoordinates _unalignedMouseCoords = default;
-    private const float MouseDeadzoneRadius = 0.25f;
-
-    private Color _guideColor = new Color(0, 0, 0.5785f);
-    private const float GuideRadius = 0.1f;
-    private const float GuideOffset = 0.21875f;
-
-    public AlignAtmosPipeLayers(PlacementManager pMan) : base(pMan)
-    {
-        IoCManager.InjectDependencies(this);
-
-        _mapSystem = _entityManager.System<SharedMapSystem>();
-        _transformSystem = _entityManager.System<SharedTransformSystem>();
-        _pipeLayersSystem = _entityManager.System<SharedAtmosPipeLayersSystem>();
-        _spriteSystem = _entityManager.System<SpriteSystem>();
-    }
-
-    /// <inheritdoc/>
-    public override void Render(in OverlayDrawArgs args)
-    {
-        var gridUid = _entityManager.System<SharedTransformSystem>().GetGrid(MouseCoords);
-
-        if (gridUid == null || Grid == null)
-            return;
-
-        // Draw guide circles for each pipe layer if we are not in line/grid placing mode
-        if (pManager.PlacementType == PlacementTypes.None)
-        {
-            var gridRotation = _transformSystem.GetWorldRotation(gridUid.Value);
-            var worldPosition = _mapSystem.LocalToWorld(gridUid.Value, Grid, MouseCoords.Position);
-            var direction = (_eyeManager.CurrentEye.Rotation + gridRotation + Math.PI / 2).GetCardinalDir();
-            var multi = (direction == Direction.North || direction == Direction.South) ? -1f : 1f;
-
-            args.WorldHandle.DrawCircle(worldPosition, GuideRadius, _guideColor);
-            args.WorldHandle.DrawCircle(worldPosition + gridRotation.RotateVec(new Vector2(multi * GuideOffset, GuideOffset)), GuideRadius, _guideColor);
-            args.WorldHandle.DrawCircle(worldPosition - gridRotation.RotateVec(new Vector2(multi * GuideOffset, GuideOffset)), GuideRadius, _guideColor);
-        }
-
-        base.Render(args);
-    }
-
-    /// <inheritdoc/>
-    public override void AlignPlacementMode(ScreenCoordinates mouseScreen)
-    {
-        _unalignedMouseCoords = ScreenToCursorGrid(mouseScreen);
-        base.AlignPlacementMode(mouseScreen);
-
-        // Exit early if we are in line/grid placing mode
-        if (pManager.PlacementType != PlacementTypes.None)
-            return;
-
-        MouseCoords = _unalignedMouseCoords.AlignWithClosestGridTile(SearchBoxSize, _entityManager, _mapManager);
-
-        var gridId = _transformSystem.GetGrid(MouseCoords);
-
-        if (!_entityManager.TryGetComponent<MapGridComponent>(gridId, out var mapGrid))
-            return;
-
-        var gridRotation = _transformSystem.GetWorldRotation(gridId.Value);
-        CurrentTile = _mapSystem.GetTileRef(gridId.Value, mapGrid, MouseCoords);
-
-        float tileSize = mapGrid.TileSize;
-        GridDistancing = tileSize;
-
-        MouseCoords = new EntityCoordinates(MouseCoords.EntityId, new Vector2(CurrentTile.X + tileSize / 2 + pManager.PlacementOffset.X,
-            CurrentTile.Y + tileSize / 2 + pManager.PlacementOffset.Y));
-
-        // Calculate the position of the mouse cursor with respect to the center of the tile to determine which layer to use
-        var mouseCoordsDiff = _unalignedMouseCoords.Position - MouseCoords.Position;
-        var layer = AtmosPipeLayer.Primary;
-
-        if (mouseCoordsDiff.Length() > MouseDeadzoneRadius)
-        {
-            // Determine the direction of the mouse is relative to the center of the tile, adjusting for the player eye and grid rotation
-            var direction = (new Angle(mouseCoordsDiff) + _eyeManager.CurrentEye.Rotation + gridRotation + Math.PI / 2).GetCardinalDir();
-            layer = (direction == Direction.North || direction == Direction.East) ? AtmosPipeLayer.Secondary : AtmosPipeLayer.Tertiary;
-        }
-
-        // Update the construction menu placer
-        if (pManager.Hijack != null)
-            UpdateHijackedPlacer(layer, mouseScreen);
-
-        // Otherwise update the debug placer
-        else
-            UpdatePlacer(layer);
-    }
-
-    private void UpdateHijackedPlacer(AtmosPipeLayer layer, ScreenCoordinates mouseScreen)
-    {
-        // Try to get alternative prototypes from the construction prototype
-        var constructionSystem = (pManager.Hijack as ConstructionPlacementHijack)?.CurrentConstructionSystem;
-        var altPrototypes = (pManager.Hijack as ConstructionPlacementHijack)?.CurrentPrototype?.AlternativePrototypes;
-
-        if (constructionSystem == null || altPrototypes == null || (int)layer >= altPrototypes.Length)
-            return;
-
-        var newProtoId = altPrototypes[(int)layer];
-
-        if (!_protoManager.Resolve(newProtoId, out var newProto))
-            return;
-
-        if (newProto.Type != ConstructionType.Structure)
-        {
-            pManager.Clear();
-            return;
-        }
-
-        if (newProto.ID == (pManager.Hijack as ConstructionPlacementHijack)?.CurrentPrototype?.ID)
-            return;
-
-        // Start placing
-        pManager.BeginPlacing(new PlacementInformation()
-        {
-            IsTile = false,
-            PlacementOption = newProto.PlacementMode,
-        }, new ConstructionPlacementHijack(constructionSystem, newProto));
-
-        if (pManager.CurrentMode is AlignAtmosPipeLayers { } newMode)
-            newMode.RefreshGrid(mouseScreen);
-
-        // Update construction guide
-        constructionSystem.GetGuide(newProto);
-    }
-
-    private void UpdatePlacer(AtmosPipeLayer layer)
-    {
-        // Try to get alternative prototypes from the entity atmos pipe layer component
-        if (pManager.CurrentPermission?.EntityType == null)
-            return;
-
-        if (!_protoManager.TryIndex<EntityPrototype>(pManager.CurrentPermission.EntityType, out var currentProto))
-            return;
-
-        if (!currentProto.TryGetComponent<AtmosPipeLayersComponent>(out var atmosPipeLayers, _entityManager.ComponentFactory))
-            return;
-
-        if (!_pipeLayersSystem.TryGetAlternativePrototype(atmosPipeLayers, layer, out var newProtoId))
-            return;
-
-        if (_protoManager.TryIndex<EntityPrototype>(newProtoId, out var newProto))
-        {
-            // Update the placed prototype
-            pManager.CurrentPermission.EntityType = newProtoId;
-
-            // Update the appearance of the ghost sprite
-            if (newProto.TryGetComponent<SpriteComponent>(out var sprite, _entityManager.ComponentFactory))
-            {
-                var textures = new List<IDirectionalTextureProvider>();
-
-                foreach (var spriteLayer in sprite.AllLayers)
-                {
-                    if (spriteLayer.ActualRsi?.Path != null && spriteLayer.RsiState.Name != null)
-                        textures.Add(_spriteSystem.RsiStateLike(new SpriteSpecifier.Rsi(spriteLayer.ActualRsi.Path, spriteLayer.RsiState.Name)));
-                }
-
-                pManager.CurrentTextures = textures;
-            }
-        }
-    }
-
-    private void RefreshGrid(ScreenCoordinates mouseScreen)
-    {
-        base.AlignPlacementMode(mouseScreen);
-    }
-}

Content.Client/Atmos/Consoles/AtmosMonitoringConsoleNavMapControl.cs

@@ -17,10 +17,6 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
     public int? FocusNetId = null;
 
     private const int ChunkSize = 4;
-    private const float ScaleModifier = 4f;
-
-    private readonly float[] _layerFraction = { 0.5f, 0.75f, 0.25f };
-    private const float LineThickness = 0.05f;
 
     private readonly Color _basePipeNetColor = Color.LightGray;
     private readonly Color _unfocusedPipeNetColor = Color.DimGray;
@@ -99,23 +95,23 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
             foreach (var chunkedLine in atmosPipeNetwork)
             {
                 var leftTop = ScalePosition(new Vector2
-                    (Math.Min(chunkedLine.Origin.X, chunkedLine.Terminus.X) - LineThickness,
-                    Math.Min(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) - LineThickness)
+                    (Math.Min(chunkedLine.Origin.X, chunkedLine.Terminus.X) - 0.1f,
+                    Math.Min(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) - 0.1f)
                     - offset);
 
                 var rightTop = ScalePosition(new Vector2
-                    (Math.Max(chunkedLine.Origin.X, chunkedLine.Terminus.X) + LineThickness,
-                    Math.Min(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) - LineThickness)
+                    (Math.Max(chunkedLine.Origin.X, chunkedLine.Terminus.X) + 0.1f,
+                    Math.Min(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) - 0.1f)
                     - offset);
 
                 var leftBottom = ScalePosition(new Vector2
-                    (Math.Min(chunkedLine.Origin.X, chunkedLine.Terminus.X) - LineThickness,
-                    Math.Max(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) + LineThickness)
+                    (Math.Min(chunkedLine.Origin.X, chunkedLine.Terminus.X) - 0.1f,
+                    Math.Max(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) + 0.1f)
                     - offset);
 
                 var rightBottom = ScalePosition(new Vector2
-                    (Math.Max(chunkedLine.Origin.X, chunkedLine.Terminus.X) + LineThickness,
-                    Math.Max(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) + LineThickness)
+                    (Math.Max(chunkedLine.Origin.X, chunkedLine.Terminus.X) + 0.1f,
+                    Math.Max(chunkedLine.Origin.Y, chunkedLine.Terminus.Y) + 0.1f)
                     - offset);
 
                 if (!pipeVertexUVs.TryGetValue(chunkedLine.Color, out var pipeVertexUV))
@@ -146,7 +142,7 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
         if (chunks == null || grid == null)
             return decodedOutput;
 
-        // Clear stale look up table values
+        // Clear stale look up table values 
         _horizLines.Clear();
         _horizLinesReversed.Clear();
         _vertLines.Clear();
@@ -162,10 +158,10 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
         {
             var list = new List<AtmosMonitoringConsoleLine>();
 
-            foreach (var ((netId, layer, pipeColor), atmosPipeData) in chunk.AtmosPipeData)
+            foreach (var ((netId, hexColor), atmosPipeData) in chunk.AtmosPipeData)
             {
                 // Determine the correct coloration for the pipe
-                var color = pipeColor * _basePipeNetColor;
+                var color = Color.FromHex(hexColor) * _basePipeNetColor;
 
                 if (FocusNetId != null && FocusNetId != netId)
                     color *= _unfocusedPipeNetColor;
@@ -195,9 +191,6 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
                     _vertLinesReversed[color] = vertLinesReversed;
                 }
 
-                var layerFraction = _layerFraction[(int)layer];
-                var origin = new Vector2(grid.TileSize * layerFraction, -grid.TileSize * layerFraction);
-
                 // Loop over the chunk
                 for (var tileIdx = 0; tileIdx < ChunkSize * ChunkSize; tileIdx++)
                 {
@@ -215,22 +208,21 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
 
                     // Calculate the draw point offsets
                     var vertLineOrigin = (atmosPipeData & northMask << tileIdx * SharedNavMapSystem.Directions) > 0 ?
-                        new Vector2(grid.TileSize * layerFraction, -grid.TileSize * 1f) : origin;
+                        new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 1f) : new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 0.5f);
 
                     var vertLineTerminus = (atmosPipeData & southMask << tileIdx * SharedNavMapSystem.Directions) > 0 ?
-                        new Vector2(grid.TileSize * layerFraction, -grid.TileSize * 0f) : origin;
+                        new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 0f) : new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 0.5f);
 
                     var horizLineOrigin = (atmosPipeData & eastMask << tileIdx * SharedNavMapSystem.Directions) > 0 ?
-                        new Vector2(grid.TileSize * 1f, -grid.TileSize * layerFraction) : origin;
+                        new Vector2(grid.TileSize * 1f, -grid.TileSize * 0.5f) : new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 0.5f);
 
                     var horizLineTerminus = (atmosPipeData & westMask << tileIdx * SharedNavMapSystem.Directions) > 0 ?
-                        new Vector2(grid.TileSize * 0f, -grid.TileSize * layerFraction) : origin;
+                        new Vector2(grid.TileSize * 0f, -grid.TileSize * 0.5f) : new Vector2(grid.TileSize * 0.5f, -grid.TileSize * 0.5f);
 
-                    // Scale up the vectors and convert to vector2i so we can merge them
-                    AddOrUpdateNavMapLine(ConvertVector2ToVector2i(tile + horizLineOrigin, ScaleModifier),
-                        ConvertVector2ToVector2i(tile + horizLineTerminus, ScaleModifier), horizLines, horizLinesReversed);
-                    AddOrUpdateNavMapLine(ConvertVector2ToVector2i(tile + vertLineOrigin, ScaleModifier),
-                        ConvertVector2ToVector2i(tile + vertLineTerminus, ScaleModifier), vertLines, vertLinesReversed);
+                    // Since we can have pipe lines that have a length of a half tile, 
+                    // double the vectors and convert to vector2i so we can merge them
+                    AddOrUpdateNavMapLine(ConvertVector2ToVector2i(tile + horizLineOrigin, 2), ConvertVector2ToVector2i(tile + horizLineTerminus, 2), horizLines, horizLinesReversed);
+                    AddOrUpdateNavMapLine(ConvertVector2ToVector2i(tile + vertLineOrigin, 2), ConvertVector2ToVector2i(tile + vertLineTerminus, 2), vertLines, vertLinesReversed);
                 }
             }
         }
@@ -243,7 +235,7 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
 
             foreach (var (origin, terminal) in horizLines)
                 decodedOutput.Add(new AtmosMonitoringConsoleLine
-                    (ConvertVector2iToVector2(origin, 1f / ScaleModifier), ConvertVector2iToVector2(terminal, 1f / ScaleModifier), sRGB));
+                    (ConvertVector2iToVector2(origin, 0.5f), ConvertVector2iToVector2(terminal, 0.5f), sRGB));
         }
 
         foreach (var (color, vertLines) in _vertLines)
@@ -253,7 +245,7 @@ public sealed partial class AtmosMonitoringConsoleNavMapControl : NavMapControl
 
             foreach (var (origin, terminal) in vertLines)
                 decodedOutput.Add(new AtmosMonitoringConsoleLine
-                    (ConvertVector2iToVector2(origin, 1f / ScaleModifier), ConvertVector2iToVector2(terminal, 1f / ScaleModifier), sRGB));
+                    (ConvertVector2iToVector2(origin, 0.5f), ConvertVector2iToVector2(terminal, 0.5f), sRGB));
         }
 
         return decodedOutput;

Content.Client/Atmos/Consoles/AtmosMonitoringConsoleSystem.cs

@@ -15,7 +15,7 @@ public sealed class AtmosMonitoringConsoleSystem : SharedAtmosMonitoringConsoleS
 
     private void OnHandleState(EntityUid uid, AtmosMonitoringConsoleComponent component, ref ComponentHandleState args)
     {
-        Dictionary<Vector2i, Dictionary<AtmosMonitoringConsoleSubnet, ulong>> modifiedChunks;
+        Dictionary<Vector2i, Dictionary<(int, string), ulong>> modifiedChunks;
         Dictionary<NetEntity, AtmosDeviceNavMapData> atmosDevices;
 
         switch (args.Current)
@@ -54,7 +54,7 @@ public sealed class AtmosMonitoringConsoleSystem : SharedAtmosMonitoringConsoleS
         foreach (var (origin, chunk) in modifiedChunks)
         {
             var newChunk = new AtmosPipeChunk(origin);
-            newChunk.AtmosPipeData = new Dictionary<AtmosMonitoringConsoleSubnet, ulong>(chunk);
+            newChunk.AtmosPipeData = new Dictionary<(int, string), ulong>(chunk);
 
             component.AtmosPipeChunks[origin] = newChunk;
         }

Content.Client/Atmos/Consoles/AtmosMonitoringConsoleWindow.xaml.cs

@@ -13,7 +13,6 @@ using Robust.Shared.Timing;
 using Robust.Shared.Utility;
 using System.Diagnostics.CodeAnalysis;
 using System.Linq;
-using System.Numerics;
 
 namespace Content.Client.Atmos.Consoles;
 
@@ -34,8 +33,6 @@ public sealed partial class AtmosMonitoringConsoleWindow : FancyWindow
     private ProtoId<NavMapBlipPrototype> _navMapConsoleProtoId = "NavMapConsole";
     private ProtoId<NavMapBlipPrototype> _gasPipeSensorProtoId = "GasPipeSensor";
 
-    private readonly Vector2[] _pipeLayerOffsets = { new Vector2(0f, 0f), new Vector2(0.25f, 0.25f), new Vector2(-0.25f, -0.25f) };
-
     public AtmosMonitoringConsoleWindow(AtmosMonitoringConsoleBoundUserInterface userInterface, EntityUid? owner)
     {
         RobustXamlLoader.Load(this);
@@ -56,7 +53,7 @@ public sealed partial class AtmosMonitoringConsoleWindow : FancyWindow
             consoleCoords = xform.Coordinates;
             NavMap.MapUid = xform.GridUid;
 
-            // Assign station name
+            // Assign station name      
             if (_entManager.TryGetComponent<MetaDataComponent>(xform.GridUid, out var stationMetaData))
                 stationName = stationMetaData.EntityName;
 
@@ -241,10 +238,6 @@ public sealed partial class AtmosMonitoringConsoleWindow : FancyWindow
 
         var blinks = proto.Blinks || _focusEntity == metaData.NetEntity;
         var coords = _entManager.GetCoordinates(metaData.NetCoordinates);
-
-        if (proto.Placement == NavMapBlipPlacement.Offset && metaData.PipeLayer > 0)
-            coords = coords.Offset(_pipeLayerOffsets[(int)metaData.PipeLayer]);
-
         var blip = new NavMapBlip(coords, _spriteSystem.Frame0(new SpriteSpecifier.Texture(texture)), color, blinks, proto.Selectable, proto.Scale);
         NavMap.TrackedEntities[metaData.NetEntity] = blip;
     }

Content.Client/Atmos/EntitySystems/AtmosPipeAppearanceSystem.cs

@@ -1,7 +1,6 @@
 using Content.Client.SubFloor;
 using Content.Shared.Atmos;
 using Content.Shared.Atmos.Components;
-using Content.Shared.Atmos.EntitySystems;
 using Content.Shared.Atmos.Piping;
 using JetBrains.Annotations;
 using Robust.Client.GameObjects;
@@ -9,10 +8,9 @@ using Robust.Client.GameObjects;
 namespace Content.Client.Atmos.EntitySystems;
 
 [UsedImplicitly]
-public sealed partial class AtmosPipeAppearanceSystem : SharedAtmosPipeAppearanceSystem
+public sealed class AtmosPipeAppearanceSystem : EntitySystem
 {
     [Dependency] private readonly SharedAppearanceSystem _appearance = default!;
-    [Dependency] private readonly SpriteSystem _sprite = default!;
 
     public override void Initialize()
     {
@@ -27,37 +25,25 @@ public sealed partial class AtmosPipeAppearanceSystem : SharedAtmosPipeAppearanc
         if (!TryComp(uid, out SpriteComponent? sprite))
             return;
 
-        var numberOfPipeLayers = GetNumberOfPipeLayers(uid, out _);
-
-        foreach (var layerKey in Enum.GetValues<PipeConnectionLayer>())
+        foreach (PipeConnectionLayer layerKey in Enum.GetValues(typeof(PipeConnectionLayer)))
         {
-            for (byte i = 0; i < numberOfPipeLayers; i++)
-            {
-                var layerName = layerKey.ToString() + i.ToString();
-                var layer = _sprite.LayerMapReserve((uid, sprite), layerName);
-                _sprite.LayerSetRsi((uid, sprite), layer, component.Sprite[i].RsiPath);
-                _sprite.LayerSetRsiState((uid, sprite), layer, component.Sprite[i].RsiState);
-                _sprite.LayerSetDirOffset((uid, sprite), layer, ToOffset(layerKey));
-            }
+            sprite.LayerMapReserveBlank(layerKey);
+            var layer = sprite.LayerMapGet(layerKey);
+            sprite.LayerSetRSI(layer, component.Sprite.RsiPath);
+            sprite.LayerSetState(layer, component.Sprite.RsiState);
+            sprite.LayerSetDirOffset(layer, ToOffset(layerKey));
         }
     }
 
-    private void HideAllPipeConnection(Entity<SpriteComponent> entity, AtmosPipeLayersComponent? atmosPipeLayers, int numberOfPipeLayers)
+    private void HideAllPipeConnection(SpriteComponent sprite)
     {
-        var sprite = entity.Comp;
-
-        foreach (var layerKey in Enum.GetValues<PipeConnectionLayer>())
+        foreach (PipeConnectionLayer layerKey in Enum.GetValues(typeof(PipeConnectionLayer)))
         {
-            for (byte i = 0; i < numberOfPipeLayers; i++)
-            {
-                var layerName = layerKey.ToString() + i.ToString();
+            if (!sprite.LayerMapTryGet(layerKey, out var key))
+                continue;
 
-                if (!_sprite.LayerMapTryGet(entity.AsNullable(), layerName, out var key, false))
-                    continue;
-
-                var layer = sprite[key];
-                layer.Visible = false;
-            }
+            var layer = sprite[key];
+            layer.Visible = false;
         }
     }
 
@@ -73,45 +59,33 @@ public sealed partial class AtmosPipeAppearanceSystem : SharedAtmosPipeAppearanc
             return;
         }
 
-        var numberOfPipeLayers = GetNumberOfPipeLayers(uid, out var atmosPipeLayers);
-
-        if (!_appearance.TryGetData<int>(uid, PipeVisuals.VisualState, out var worldConnectedDirections, args.Component))
+        if (!_appearance.TryGetData<PipeDirection>(uid, PipeVisuals.VisualState, out var worldConnectedDirections, args.Component))
         {
-            HideAllPipeConnection((uid, args.Sprite), atmosPipeLayers, numberOfPipeLayers);
+            HideAllPipeConnection(args.Sprite);
             return;
         }
 
         if (!_appearance.TryGetData<Color>(uid, PipeColorVisuals.Color, out var color, args.Component))
             color = Color.White;
 
-        for (byte i = 0; i < numberOfPipeLayers; i++)
+        // transform connected directions to local-coordinates
+        var connectedDirections = worldConnectedDirections.RotatePipeDirection(-Transform(uid).LocalRotation);
+
+        foreach (PipeConnectionLayer layerKey in Enum.GetValues(typeof(PipeConnectionLayer)))
         {
-            // Extract the cardinal pipe orientations for the current pipe layer
-            // '15' is the four bit mask that is used to extract the pipe orientations of interest from 'worldConnectedDirections'
-            // Fun fact: a collection of four bits is called a 'nibble'! They aren't natively supported :(
-            var pipeLayerConnectedDirections = (PipeDirection)(15 & (worldConnectedDirections >> (PipeDirectionHelpers.PipeDirections * i)));
+            if (!args.Sprite.LayerMapTryGet(layerKey, out var key))
+                continue;
 
-            // Transform the connected directions to local-coordinates
-            var connectedDirections = pipeLayerConnectedDirections.RotatePipeDirection(-Transform(uid).LocalRotation);
+            var layer = args.Sprite[key];
+            var dir = (PipeDirection) layerKey;
+            var visible = connectedDirections.HasDirection(dir);
 
-            foreach (var layerKey in Enum.GetValues<PipeConnectionLayer>())
-            {
-                var layerName = layerKey.ToString() + i.ToString();
+            layer.Visible &= visible;
 
-                if (!_sprite.LayerMapTryGet((uid, args.Sprite), layerName, out var key, false))
-                    continue;
+            if (!visible)
+                continue;
 
-                var layer = args.Sprite[key];
-                var dir = (PipeDirection)layerKey;
-                var visible = connectedDirections.HasDirection(dir);
-
-                layer.Visible &= visible;
-
-                if (!visible)
-                    continue;
-
-                layer.Color = color;
-            }
+            layer.Color = color;
         }
     }
 

Content.Client/Atmos/EntitySystems/AtmosPipeLayersSystem.cs

@@ -1,56 +0,0 @@
-using Content.Shared.Atmos.Components;
-using Content.Shared.Atmos.EntitySystems;
-using Robust.Client.GameObjects;
-using Robust.Client.ResourceManagement;
-using Robust.Shared.Reflection;
-using Robust.Shared.Serialization.TypeSerializers.Implementations;
-using Robust.Shared.Utility;
-using System.Diagnostics.CodeAnalysis;
-
-namespace Content.Client.Atmos.EntitySystems;
-
-/// <summary>
-/// The system responsible for updating the appearance of layered gas pipe
-/// </summary>
-public sealed partial class AtmosPipeLayersSystem : SharedAtmosPipeLayersSystem
-{
-    [Dependency] private readonly SharedAppearanceSystem _appearance = default!;
-    [Dependency] private readonly IReflectionManager _reflection = default!;
-    [Dependency] private readonly IResourceCache _resourceCache = default!;
-    [Dependency] private readonly SpriteSystem _sprite = default!;
-
-    public override void Initialize()
-    {
-        base.Initialize();
-
-        SubscribeLocalEvent<AtmosPipeLayersComponent, AppearanceChangeEvent>(OnAppearanceChange);
-    }
-
-    private void OnAppearanceChange(Entity<AtmosPipeLayersComponent> ent, ref AppearanceChangeEvent ev)
-    {
-        if (!TryComp<SpriteComponent>(ent, out var sprite))
-            return;
-
-        if (_appearance.TryGetData<string>(ent, AtmosPipeLayerVisuals.Sprite, out var spriteRsi) &&
-            _resourceCache.TryGetResource(SpriteSpecifierSerializer.TextureRoot / spriteRsi, out RSIResource? resource))
-        {
-            _sprite.SetBaseRsi((ent, sprite), resource.RSI);
-        }
-
-        if (_appearance.TryGetData<Dictionary<string, string>>(ent, AtmosPipeLayerVisuals.SpriteLayers, out var pipeState))
-        {
-            foreach (var (layerKey, rsiPath) in pipeState)
-            {
-                if (TryParseKey(layerKey, out var @enum))
-                    _sprite.LayerSetRsi((ent, sprite), @enum, new ResPath(rsiPath));
-                else
-                    _sprite.LayerSetRsi((ent, sprite), layerKey, new ResPath(rsiPath));
-            }
-        }
-    }
-
-    private bool TryParseKey(string keyString, [NotNullWhen(true)] out Enum? @enum)
-    {
-        return _reflection.TryParseEnumReference(keyString, out @enum);
-    }
-}

Content.Client/Atmos/EntitySystems/FireVisualizerSystem.cs

@@ -2,7 +2,6 @@ using Content.Client.Atmos.Components;
 using Content.Shared.Atmos;
 using Robust.Client.GameObjects;
 using Robust.Shared.Map;
-using Robust.Shared.Utility;
 
 namespace Content.Client.Atmos.EntitySystems;
 
@@ -32,9 +31,9 @@ public sealed class FireVisualizerSystem : VisualizerSystem<FireVisualsComponent
         // Need LayerMapTryGet because Init fails if there's no existing sprite / appearancecomp
         // which means in some setups (most frequently no AppearanceComp) the layer never exists.
         if (TryComp<SpriteComponent>(uid, out var sprite) &&
-            SpriteSystem.LayerMapTryGet((uid, sprite), FireVisualLayers.Fire, out var layer, false))
+            sprite.LayerMapTryGet(FireVisualLayers.Fire, out var layer))
         {
-            SpriteSystem.RemoveLayer((uid, sprite), layer);
+            sprite.RemoveLayer(layer);
         }
     }
 
@@ -43,11 +42,11 @@ public sealed class FireVisualizerSystem : VisualizerSystem<FireVisualsComponent
         if (!TryComp<SpriteComponent>(uid, out var sprite) || !TryComp(uid, out AppearanceComponent? appearance))
             return;
 
-        SpriteSystem.LayerMapReserve((uid, sprite), FireVisualLayers.Fire);
-        SpriteSystem.LayerSetVisible((uid, sprite), FireVisualLayers.Fire, false);
+        sprite.LayerMapReserveBlank(FireVisualLayers.Fire);
+        sprite.LayerSetVisible(FireVisualLayers.Fire, false);
         sprite.LayerSetShader(FireVisualLayers.Fire, "unshaded");
         if (component.Sprite != null)
-            SpriteSystem.LayerSetRsi((uid, sprite), FireVisualLayers.Fire, new ResPath(component.Sprite));
+            sprite.LayerSetRSI(FireVisualLayers.Fire, component.Sprite);
 
         UpdateAppearance(uid, component, sprite, appearance);
     }
@@ -60,12 +59,12 @@ public sealed class FireVisualizerSystem : VisualizerSystem<FireVisualsComponent
 
     private void UpdateAppearance(EntityUid uid, FireVisualsComponent component, SpriteComponent sprite, AppearanceComponent appearance)
     {
-        if (!SpriteSystem.LayerMapTryGet((uid, sprite), FireVisualLayers.Fire, out var index, false))
+        if (!sprite.LayerMapTryGet(FireVisualLayers.Fire, out var index))
             return;
 
         AppearanceSystem.TryGetData<bool>(uid, FireVisuals.OnFire, out var onFire, appearance);
         AppearanceSystem.TryGetData<float>(uid, FireVisuals.FireStacks, out var fireStacks, appearance);
-        SpriteSystem.LayerSetVisible((uid, sprite), index, onFire);
+        sprite.LayerSetVisible(index, onFire);
 
         if (!onFire)
         {
@@ -79,9 +78,9 @@ public sealed class FireVisualizerSystem : VisualizerSystem<FireVisualsComponent
         }
 
         if (fireStacks > component.FireStackAlternateState && !string.IsNullOrEmpty(component.AlternateState))
-            SpriteSystem.LayerSetRsiState((uid, sprite), index, component.AlternateState);
+            sprite.LayerSetState(index, component.AlternateState);
         else
-            SpriteSystem.LayerSetRsiState((uid, sprite), index, component.NormalState);
+            sprite.LayerSetState(index, component.NormalState);
 
         component.LightEntity ??= Spawn(null, new EntityCoordinates(uid, default));
         var light = EnsureComp<PointLightComponent>(component.LightEntity.Value);

Content.Client/Atmos/EntitySystems/GasCanisterAppearanceSystem.cs

@@ -1,28 +0,0 @@
-using Content.Shared.Atmos.Piping.Unary.Components;
-using Content.Shared.SprayPainter.Prototypes;
-using Robust.Client.GameObjects;
-using Robust.Shared.Prototypes;
-
-namespace Content.Client.Atmos.EntitySystems;
-
-/// <summary>
-/// Used to change the appearance of gas canisters.
-/// </summary>
-public sealed class GasCanisterAppearanceSystem : VisualizerSystem<GasCanisterComponent>
-{
-    [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
-
-    protected override void OnAppearanceChange(EntityUid uid, GasCanisterComponent component, ref AppearanceChangeEvent args)
-    {
-        if (!AppearanceSystem.TryGetData<string>(uid, PaintableVisuals.Prototype, out var protoName, args.Component) || args.Sprite is not { } old)
-            return;
-
-        if (!_prototypeManager.HasIndex(protoName))
-            return;
-
-        // Create the given prototype and get its first layer.
-        var tempUid = Spawn(protoName);
-        SpriteSystem.LayerSetRsiState(uid, 0, SpriteSystem.LayerGetRsiState(tempUid, 0));
-        QueueDel(tempUid);
-    }
-}

Content.Client/Atmos/EntitySystems/GasPressureRegulatorSystem.cs

@@ -1,31 +0,0 @@
-using Content.Shared.Atmos.EntitySystems;
-using Content.Shared.Atmos.Piping.Binary.Components;
-
-namespace Content.Client.Atmos.EntitySystems;
-
-/// <summary>
-/// Represents the client system responsible for managing and updating the gas pressure regulator interface.
-/// Inherits from the shared system <see cref="SharedGasPressureRegulatorSystem"/>.
-/// </summary>
-public sealed partial class GasPressureRegulatorSystem : SharedGasPressureRegulatorSystem
-{
-    public override void Initialize()
-    {
-        base.Initialize();
-
-        SubscribeLocalEvent<GasPressureRegulatorComponent, AfterAutoHandleStateEvent>(OnValveUpdate);
-    }
-
-    private void OnValveUpdate(Entity<GasPressureRegulatorComponent> ent, ref AfterAutoHandleStateEvent args)
-    {
-        UpdateUi(ent);
-    }
-
-    protected override void UpdateUi(Entity<GasPressureRegulatorComponent> ent)
-    {
-        if (UserInterfaceSystem.TryGetOpenUi(ent.Owner, GasPressureRegulatorUiKey.Key, out var bui))
-        {
-            bui.Update();
-        }
-    }
-}

Content.Client/Atmos/EntitySystems/GasTankSystem.cs

@@ -1,29 +0,0 @@
-using Content.Shared.Atmos.Components;
-using Content.Shared.Atmos.EntitySystems;
-
-namespace Content.Client.Atmos.EntitySystems;
-
-public sealed class GasTankSystem : SharedGasTankSystem
-{
-    public override void Initialize()
-    {
-        base.Initialize();
-        SubscribeLocalEvent<GasTankComponent, AfterAutoHandleStateEvent>(OnGasTankState);
-    }
-
-    private void OnGasTankState(Entity<GasTankComponent> ent, ref AfterAutoHandleStateEvent args)
-    {
-        if (UI.TryGetOpenUi(ent.Owner, SharedGasTankUiKey.Key, out var bui))
-        {
-            bui.Update<GasTankBoundUserInterfaceState>();
-        }
-    }
-
-    public override void UpdateUserInterface(Entity<GasTankComponent> ent)
-    {
-        if (UI.TryGetOpenUi(ent.Owner, SharedGasTankUiKey.Key, out var bui))
-        {
-            bui.Update<GasTankBoundUserInterfaceState>();
-        }
-    }
-}

Content.Client/Atmos/Monitor/AtmosAlarmableVisualsSystem.cs

@@ -9,7 +9,7 @@ public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem<AtmosAlarmabl
 {
     protected override void OnAppearanceChange(EntityUid uid, AtmosAlarmableVisualsComponent component, ref AppearanceChangeEvent args)
     {
-        if (args.Sprite == null || !SpriteSystem.LayerMapTryGet((uid, args.Sprite), component.LayerMap, out var layer, false))
+        if (args.Sprite == null || !args.Sprite.LayerMapTryGet(component.LayerMap, out var layer))
             return;
 
         if (!args.AppearanceData.TryGetValue(PowerDeviceVisuals.Powered, out var poweredObject) ||
@@ -22,8 +22,8 @@ public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem<AtmosAlarmabl
         {
             foreach (var visLayer in component.HideOnDepowered)
             {
-                if (SpriteSystem.LayerMapTryGet((uid, args.Sprite), visLayer, out var powerVisibilityLayer, false))
-                    SpriteSystem.LayerSetVisible((uid, args.Sprite), powerVisibilityLayer, powered);
+                if (args.Sprite.LayerMapTryGet(visLayer, out var powerVisibilityLayer))
+                    args.Sprite.LayerSetVisible(powerVisibilityLayer, powered);
             }
         }
 
@@ -31,8 +31,8 @@ public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem<AtmosAlarmabl
         {
             foreach (var (setLayer, powerState) in component.SetOnDepowered)
             {
-                if (SpriteSystem.LayerMapTryGet((uid, args.Sprite), setLayer, out var setStateLayer, false))
-                    SpriteSystem.LayerSetRsiState((uid, args.Sprite), setStateLayer, new RSI.StateId(powerState));
+                if (args.Sprite.LayerMapTryGet(setLayer, out var setStateLayer))
+                    args.Sprite.LayerSetState(setStateLayer, new RSI.StateId(powerState));
             }
         }
 
@@ -41,7 +41,7 @@ public sealed class AtmosAlarmableVisualsSystem : VisualizerSystem<AtmosAlarmabl
             && powered
             && component.AlarmStates.TryGetValue(alarmType, out var state))
         {
-            SpriteSystem.LayerSetRsiState((uid, args.Sprite), layer, new RSI.StateId(state));
+            args.Sprite.LayerSetState(layer, new RSI.StateId(state));
         }
     }
 }

Content.Client/Atmos/Monitor/UI/AirAlarmWindow.xaml

@@ -2,7 +2,7 @@
             xmlns:x="http://schemas.microsoft.com/winfx/2007/xaml"
             xmlns:ui="clr-namespace:Content.Client.UserInterface.Controls"
             xmlns:gfx="clr-namespace:Robust.Client.Graphics;assembly=Robust.Client"
-            MinSize="500 500" Resizable="True" Title="{Loc air-alarm-ui-title}">
+            MinSize="500 500" Resizable="True" Title="Air Alarm">
     <BoxContainer Orientation="Vertical" Margin="5 5 5 5">
         <!-- Status (pressure, temperature, alarm state, device total, address, etc) -->
         <BoxContainer Orientation="Horizontal" Margin="0 0 0 2">
@@ -74,13 +74,7 @@
         <!-- Mode buttons -->
         <BoxContainer Orientation="Horizontal">
             <Label Text="{Loc 'air-alarm-ui-window-mode-label'}" Margin="0 0 2 0" />
-            <Control HorizontalExpand="True">
-                <OptionButton Name="CModeButton" />
-                <ui:StripeBack Name="CModeSelectLocked">
-                    <RichTextLabel Text="{Loc 'air-alarm-ui-window-mode-select-locked-label'}"
-                                   HorizontalAlignment="Center" />
-                </ui:StripeBack>
-            </Control>
+            <OptionButton Name="CModeButton" HorizontalExpand="True" />
             <CheckBox Name="AutoModeCheckBox" Text="{Loc 'air-alarm-ui-window-auto-mode-label'}" />
         </BoxContainer>
     </BoxContainer>

Content.Client/Atmos/Monitor/UI/AirAlarmWindow.xaml.cs

@@ -110,8 +110,6 @@ public sealed partial class AirAlarmWindow : FancyWindow
         {
             UpdateDeviceData(addr, dev);
         }
-        _modes.Visible = !state.PanicWireCut;
-        CModeSelectLocked.Visible = state.PanicWireCut;
     }
 
     public void UpdateModeSelector(AirAlarmMode mode)

Content.Client/Atmos/Monitor/UI/Widgets/PumpControl.xaml.cs

@@ -59,7 +59,7 @@ public sealed partial class PumpControl : BoxContainer
 
         foreach (var value in Enum.GetValues<VentPumpDirection>())
         {
-            _pumpDirection.AddItem(Loc.GetString($"air-alarm-ui-pump-direction-{value.ToString().ToLower()}"), (int) value);
+            _pumpDirection.AddItem(Loc.GetString($"{value}"), (int) value);
         }
 
         _pumpDirection.SelectId((int) _data.PumpDirection);
@@ -72,7 +72,7 @@ public sealed partial class PumpControl : BoxContainer
 
         foreach (var value in Enum.GetValues<VentPressureBound>())
         {
-            _pressureCheck.AddItem(Loc.GetString($"air-alarm-ui-pressure-bound-{value.ToString().ToLower()}"), (int) value);
+            _pressureCheck.AddItem(Loc.GetString($"{value}"), (int) value);
         }
 
         _pressureCheck.SelectId((int) _data.PressureChecks);

Content.Client/Atmos/Monitor/UI/Widgets/ScrubberControl.xaml

@@ -27,15 +27,9 @@
                 </BoxContainer>
                 <!-- Lower row: every single gas -->
                 <Collapsible Margin="2 2 2 2">
-                    <CollapsibleHeading Title="{Loc 'air-alarm-ui-widget-gas-filters'}" />
+                    <CollapsibleHeading Title="Gas filters" />
                     <CollapsibleBody Margin="20 0 0 0">
-                        <BoxContainer Orientation="Vertical">
-                            <BoxContainer Margin="2">
-                                <Button Name="CSelectAll" Text="{Loc 'air-alarm-ui-scrubber-select-all-gases-label'}" />
-                                <Button Name="CDeselectAll" Text="{Loc 'air-alarm-ui-scrubber-deselect-all-gases-label'}" />
-                            </BoxContainer>
-                            <GridContainer Name="CGasContainer" Columns="3" />
-                        </BoxContainer>
+                        <GridContainer HorizontalExpand="True" Name="CGasContainer" Columns="3" />
                     </CollapsibleBody>
                 </Collapsible>
             </BoxContainer>

Content.Client/Atmos/Monitor/UI/Widgets/ScrubberControl.xaml.cs

@@ -1,21 +1,15 @@
 using Content.Shared.Atmos;
-using Content.Shared.Atmos.EntitySystems;
 using Content.Shared.Atmos.Monitor.Components;
 using Content.Shared.Atmos.Piping.Unary.Components;
-using Content.Shared.Atmos.Prototypes;
 using Robust.Client.AutoGenerated;
 using Robust.Client.UserInterface.Controls;
 using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Prototypes;
 
 namespace Content.Client.Atmos.Monitor.UI.Widgets;
 
 [GenerateTypedNameReferences]
 public sealed partial class ScrubberControl : BoxContainer
 {
-    [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
-    [Dependency] private readonly IEntityManager _entMan = default!;
-
     private GasVentScrubberData _data;
     private string _address;
 
@@ -28,18 +22,12 @@ public sealed partial class ScrubberControl : BoxContainer
     private FloatSpinBox _volumeRate => CVolumeRate;
     private CheckBox _wideNet => CWideNet;
     private Button _copySettings => CCopySettings;
-    private Button _selectAll => CSelectAll;
-    private Button _deselectAll => CDeselectAll;
 
     private GridContainer _gases => CGasContainer;
     private Dictionary<Gas, Button> _gasControls = new();
 
     public ScrubberControl(GasVentScrubberData data, string address)
     {
-
-        IoCManager.InjectDependencies(this);
-        var atmosphereSystem = _entMan.System<SharedAtmosphereSystem>();
-
         RobustXamlLoader.Load(this);
 
         Name = address;
@@ -73,7 +61,7 @@ public sealed partial class ScrubberControl : BoxContainer
 
         foreach (var value in Enum.GetValues<ScrubberPumpDirection>())
         {
-            _pumpDirection.AddItem(Loc.GetString($"air-alarm-ui-pump-direction-{value.ToString().ToLower()}"), (int) value);
+            _pumpDirection.AddItem(Loc.GetString($"{value}"), (int) value);
         }
 
         _pumpDirection.SelectId((int) _data.PumpDirection);
@@ -83,35 +71,18 @@ public sealed partial class ScrubberControl : BoxContainer
             _data.PumpDirection = (ScrubberPumpDirection) args.Id;
             ScrubberDataChanged?.Invoke(_address, _data);
         };
-        _pumpDirection.Disabled = data.AirAlarmPanicWireCut;
 
         _copySettings.OnPressed += _ =>
         {
             ScrubberDataCopied?.Invoke(_data);
         };
 
-        var allGases = Enum.GetValues<Gas>();
-        _selectAll.OnPressed += _ =>
+        foreach (var value in Enum.GetValues<Gas>())
         {
-            _data.FilterGases = new HashSet<Gas>(allGases);
-            ScrubberDataChanged?.Invoke(_address, _data);
-        };
-
-        _deselectAll.OnPressed += _ =>
-        {
-            _data.FilterGases = [];
-            ScrubberDataChanged?.Invoke(_address, _data);
-        };
-
-        foreach (var value in allGases)
-        {
-            ProtoId<GasPrototype> gasProtoId = atmosphereSystem.GetGas(value);
-            var gasName = _prototypeManager.Index(gasProtoId).Name;
-
             var gasButton = new Button
             {
                 Name = value.ToString(),
-                Text = Loc.GetString(gasName),
+                Text = Loc.GetString($"{value}"),
                 ToggleMode = true,
                 HorizontalExpand = true,
                 Pressed = _data.FilterGases.Contains(value)
@@ -138,7 +109,6 @@ public sealed partial class ScrubberControl : BoxContainer
 
         _data.PumpDirection = data.PumpDirection;
         _pumpDirection.Select((int) _data.PumpDirection);
-        _pumpDirection.Disabled = data.AirAlarmPanicWireCut;
 
         _data.VolumeRate = data.VolumeRate;
         _volumeRate.Value = _data.VolumeRate;

Content.Client/Atmos/Monitor/UI/Widgets/SensorInfo.xaml.cs

@@ -1,22 +1,16 @@
 using Content.Client.Message;
 using Content.Shared.Atmos;
-using Content.Shared.Atmos.EntitySystems;
 using Content.Shared.Atmos.Monitor;
-using Content.Shared.Atmos.Prototypes;
 using Content.Shared.Temperature;
 using Robust.Client.AutoGenerated;
 using Robust.Client.UserInterface.Controls;
 using Robust.Client.UserInterface.XAML;
-using Robust.Shared.Prototypes;
 
 namespace Content.Client.Atmos.Monitor.UI.Widgets;
 
 [GenerateTypedNameReferences]
 public sealed partial class SensorInfo : BoxContainer
 {
-    [Dependency] private readonly IPrototypeManager _prototypeManager = default!;
-    [Dependency] private readonly IEntityManager _entMan = default!;
-
     public Action<string, AtmosMonitorThresholdType, AtmosAlarmThreshold, Gas?>? OnThresholdUpdate;
     public event Action<AtmosSensorData>? SensorDataCopied;
     private string _address;
@@ -29,9 +23,6 @@ public sealed partial class SensorInfo : BoxContainer
 
     public SensorInfo(AtmosSensorData data, string address)
     {
-        IoCManager.InjectDependencies(this);
-        var atmosphereSystem = _entMan.System<SharedAtmosphereSystem>();
-
         RobustXamlLoader.Load(this);
 
         _address = address;
@@ -54,12 +45,8 @@ public sealed partial class SensorInfo : BoxContainer
             var label = new RichTextLabel();
 
             var fractionGas = amount / data.TotalMoles;
-
-            ProtoId<GasPrototype> gasProtoId = atmosphereSystem.GetGas(gas);
-            var gasName = _prototypeManager.Index(gasProtoId).Name;
-
             label.SetMarkup(Loc.GetString("air-alarm-ui-gases-indicator",
-                ("gas", Loc.GetString(gasName)),
+                ("gas", $"{gas}"),
                 ("color", AirAlarmWindow.ColorForThreshold(fractionGas, data.GasThresholds[gas])),
                 ("amount", $"{amount:0.####}"),
                 ("percentage", $"{(100 * fractionGas):0.##}")));
@@ -67,7 +54,7 @@ public sealed partial class SensorInfo : BoxContainer
             _gasLabels.Add(gas, label);
 
             var threshold = data.GasThresholds[gas];
-            var gasThresholdControl = new ThresholdControl(Loc.GetString($"air-alarm-ui-thresholds-gas-title"), threshold, AtmosMonitorThresholdType.Gas, gas, 100);
+            var gasThresholdControl = new ThresholdControl(Loc.GetString($"air-alarm-ui-thresholds-gas-title", ("gas", $"{gas}")), threshold, AtmosMonitorThresholdType.Gas, gas, 100);
             gasThresholdControl.Margin = new Thickness(20, 2, 2, 2);
             gasThresholdControl.ThresholdDataChanged += (type, alarmThreshold, arg3) =>
             {
@@ -103,9 +90,6 @@ public sealed partial class SensorInfo : BoxContainer
 
     public void ChangeData(AtmosSensorData data)
     {
-        IoCManager.InjectDependencies(this);
-        var atmosphereSystem = _entMan.System<SharedAtmosphereSystem>();
-
         SensorAddress.Title = Loc.GetString("air-alarm-ui-window-listing-title", ("address", _address), ("state", data.AlarmState));
 
         AlarmStateLabel.SetMarkup(Loc.GetString("air-alarm-ui-window-alarm-state-indicator",
@@ -128,12 +112,8 @@ public sealed partial class SensorInfo : BoxContainer
             }
 
             var fractionGas = amount / data.TotalMoles;
-
-            ProtoId<GasPrototype> gasProtoId = atmosphereSystem.GetGas(gas);
-            var gasName = _prototypeManager.Index(gasProtoId).Name;
-
             label.SetMarkup(Loc.GetString("air-alarm-ui-gases-indicator",
-                ("gas", Loc.GetString(gasName)),
+                ("gas", $"{gas}"),
                 ("color", AirAlarmWindow.ColorForThreshold(fractionGas, data.GasThresholds[gas])),
                 ("amount", $"{amount:0.####}"),
                 ("percentage", $"{(100 * fractionGas):0.##}")));