Merge branch 'master' into ai-law-update

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,46 +1,62 @@
-<!-- ЭТО ШАБЛОН ВАШЕГО PULL REQUEST. Текст между стрелками - это комментарии - они не будут видны в PR. -->
-
 ## Описание PR
-<!-- Ниже опишите ваш Pull Request. Что он изменяет? На что еще это может повлиять? Постарайтесь описать все внесённые вами изменения! -->
+<!-- Что вы изменили в этом пулл реквесте? -->
 
+## Почему / Баланс
+<!-- Почему оно было изменено? Ссылайтесь на любые обсуждения или вопросы здесь. Пожалуйста, обсудите, как это повлияет на игровой баланс. -->
 **Ссылка на публикацию в Discord**
-<!-- Укажите ссылки на соответствующие обсуждения, проблемы, баги, заказы в разработку или предложения 
-
-Если ссылки отсутствуют или этот PR ваша личная инициатива, данный раздел описания можно удалить. -->
+<!-- Укажите ссылки на соответствующие обсуждения, проблемы, баги, заказы в разработку или предложения
 - [Технические проблемы](ссылка)
 - [Баги](ссылка)
 - [Заказы-разработка](ссылка)
 - [Предложения](ссылка)
-- [Перенос контента](ссылка)
+- [Перенос контента](ссылка)-->
+
+## Техническая информация
+<!-- Если речь идет об изменении кода, кратко изложите на высоком уровне принцип работы нового кода. Это облегчает рецензирование.- -->
+
+## Медиа
+<!--
+Пулл реквесты, которые вносят внутриигровые изменения (добавление одежды, предметов, новых возможностей и т.д.), должны содержать медиа, демонстрирующие изменения.
+Небольшие исправления/рефакторы не требуют медиа.
+
+Если Вы не уверены в том, что Ваш пулл реквест требует медиа, спросите мейнтейнера.
+-->
 
-**Медиа**
-<!-- Если приемлемо, добавьте скриншоты для демонстрации вашего PR. Если ваш PR представляет собой визуальное изменение, добавьте
-скриншоты, иначе он может быть закрыт. -->
+## Требования
+<!--
+В связи с наплывом ПР'ов нам необходимо убедиться, что ПР'ы следуют правильным рекомендациям.
+
+Пожалуйста, уделите время прочтению, если делаете пулл реквест (ПР) впервые.
 
-**Проверки**
-<!-- Выполнение всех следующих действий, если это приемлемо для вида изменений сильно ускорит разбор вашего PR -->
-- [ ] PR полностью завершён и мне не нужна помощь чтобы его закончить.
-- [ ] Я внимательно просмотрел все свои изменения и багов в них не нашёл.
-- [ ] Я запускал локальный сервер со своими изменениями и всё протестировал.
-- [ ] Я добавил скриншот/видео демонстрации PR в игре, **или** этот PR этого не требует.
+Отметьте поля ниже, чтобы подтвердить, что Вы действительно видели их (поставьте X в скобках, например [X]):
+-->
+- [ ] Я прочитал(а) и следую [Руководство по созданию пулл реквестов](https://docs.spacestation14.com/en/general-development/codebase-info/pull-request-guidelines.html). Я понимаю, что в противном случае мой ПР может быть закрыт по усмотрению мейнтейнера.
+- [ ] Я добавил скриншоты/видео к этому пулл реквесту, демонстрирующие его изменения в игре, **или** этот пулл реквест не требует демонстрации в игре
 
-**Изменения**
+## Критические изменения
 <!--
-Здесь вы можете написать список изменений, который будет автоматически добавлен в игру, когда ваш PR будет принят.
+Перечислите все критические изменения, включая изменения пространства имён, публичных классов/методов/полей, переименования прототипов, и предоставьте инструкции по их исправлению.
+-->
 
-В журнал изменений следует помещать только то, что действительно важно игрокам.
+**Чейнджлог**
+<!--
+Здесь Вы можете заполнить журнал изменений, который будет автоматически добавлен в игру при мердже Вашего пулл реквест.
 
-В списке изменений тип значка не является часть предложения, поэтому явно указывайте - Добавлен, Удалён, Изменён.
-плохо: - add: Новый инструмент для инженеров
-хорошо: - add: Добавлен новый инструмент для инженеров
+Чтобы игроки узнали о новых возможностях и изменениях, которые могут повлиять на их игру, добавьте запись в журнал изменений.
 
-Вы можете указать своё имя после символа :cl: именно оно будет отображаться в журнале изменений (иначе будет использоваться ваше имя на GitHub)
-Например: :cl: Ian
+Не считайте суффикс типа записи (например, add) "частью" предложения:
+плохо: - add: новый инструмент для инженеров
+хорошо: - add: добавлен новый инструмент для инженеров
 
+Помещение имени после символа :cl: изменит имя, которое будет отображаться в журнале изменений (в противном случае будет использоваться ваше имя пользователя GitHub).
+Например: :cl: AruMoon
 -->
 
+<!--
+Чтобы шаблон Чейнджлога отображался, уберите его из блока комментариев. Чейнджлог должен содержать :cl: символ, чтобы бот распознал изменения и добавил их в журнал изменений игры.
 :cl:
 - add: Добавлено веселье!
-- remove: Убрано веселье!
+- remove: Удалено веселье!
 - tweak: Изменено веселье!
 - fix: Исправлено веселье!
+-->

.github/workflows/auto-cl-update-atd.yml

@@ -1,6 +1,15 @@
 name: Auto CL update
 
 on:
+  schedule:
+    - cron: '10 * * * *'
+    - cron: '20 * * * *'
+    - cron: '30 * * * *'
+    - cron: '40 * * * *'
+    - cron: '50 * * * *'
+    - cron: '59 * * * *'
+  workflow_dispatch:
+
   push:
     branches:
       - master
@@ -34,7 +43,7 @@ jobs:
     - name: Run post-merge script
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-      run: python Tools/ADT/auto_cl.py "${{ env.GITHUB_TOKEN }}" "${{ github.repository }}"
+      run: python Tools/ADT/auto_cl.py "${{ secrets.PAT_CL_TOKEN }}" "${{ github.repository }}"
 
     - name: Configure Git
       run: |

.github/workflows/publish-publish.yml

@@ -1,12 +1,12 @@
 name: Publish Public
 
-#concurrency:
-#  group: publish
+concurrency:
+  group: publish
 
 on:
   workflow_dispatch:
   schedule:
-  - cron: '0 1 * * *'
+  - cron: '0 2 * * *'
 
 jobs:
   build:
@@ -19,6 +19,7 @@ jobs:
     - uses: actions/[email protected]
       with:
         submodules: 'recursive'
+
     - name: Setup .NET Core
       uses: actions/[email protected]
       with:
@@ -41,21 +42,10 @@ jobs:
     - name: Package client
       run: dotnet run --project Content.Packaging client --no-wipe-release
 
-    - name: Upload build artifact
-      id: artifact-upload-step
-      uses: actions/upload-artifact@v4
-      with:
-        name: build
-        path: release/*.zip
-        compression-level: 0
-        retention-days: 0
-
     - name: Publish version
-      run: Tools/publish_github_artifact.py
+      run: Tools/publish_multi_request.py
       env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
         PUBLISH_TOKEN: ${{ secrets.PUBLISH_TOKEN_PUBLIC }}
-        ARTIFACT_ID: ${{ steps.artifact-upload-step.outputs.artifact-id }}
         GITHUB_REPOSITORY: ${{ vars.GITHUB_REPOSITORY }}
         FORK_ID: ${{ vars.FORK_ID_PUBLIC }}
 
@@ -69,8 +59,3 @@ jobs:
 #      run: Tools/actions_changelog_rss.py
 #      env:
 #        CHANGELOG_RSS_KEY: ${{ secrets.CHANGELOG_RSS_KEY }}
-
-    - uses: geekyeggo/delete-artifact@v5
-      if: always()
-      with:
-        name: build

.github/workflows/publish.yml

@@ -1,7 +1,7 @@
 name: Publish
 
-#concurrency:
-#  group: publish
+concurrency:
+  group: publish
 
 on:
   workflow_dispatch:
@@ -63,58 +63,18 @@ jobs:
 
     - name: Package client
       run: dotnet run --project Content.Packaging client --no-wipe-release
-    
+
     - name: Update Build Info
       run: |
         chmod +x Tools/gen_build_info.py
         Tools/gen_build_info.py
 
-    - name: Shuffle files around
-      run: |
-        mkdir "release/${{ github.sha }}"
-        mv release/*.zip "release/${{ github.sha }}"
-  
-    - name: Upload files to mothership
-      uses: burnett01/[email protected]
-      with:
-        switches: -avzr --ignore-existing
-        path: "release/${{ github.sha }}"
-        remote_path: ${{ secrets.BUILDS_PATH }}
-        remote_host: ${{ secrets.BUILDS_HOST }}
-        remote_port: ${{ secrets.BUILDS_PORT }}
-        remote_user: ${{ secrets.BUILDS_USERNAME }}
-        remote_key: ${{ secrets.BUILDS_SSH_KEY }}
-
-    - name: Update manifest JSON
-      uses: appleboy/[email protected]
-      with:
-        host: ${{ secrets.BUILDS_HOST }}
-        username: ${{ secrets.BUILDS_USERNAME }}
-        key: ${{ secrets.BUILDS_SSH_KEY }}
-        port: ${{ secrets.BUILDS_PORT }}
-        script: python3 ~/manifest.py --version ${{ github.sha }}
-
-    - name: Update CDN repository
-      run: |
-        curl -X POST -d "" -H 'Authorization: Bearer ${{ secrets.BUILDS_TOKEN }}' "http://${{ secrets.BUILDS_HOST }}:27690/control/update"
-      
-
-    #- name: Upload build artifact
-    #  id: artifact-upload-step
-    #  uses: actions/upload-artifact@v4
-    #  with:
-    #    name: build
-    #    path: release/*.zip
-    #    compression-level: 0
-    #    retention-days: 0
-
-    #- name: Publish version
-    #  run: Tools/publish_github_artifact.py
-    #  env:
-    #    GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-    #    PUBLISH_TOKEN: ${{ secrets.PUBLISH_TOKEN }}
-    #    ARTIFACT_ID: ${{ steps.artifact-upload-step.outputs.artifact-id }}
-    #    GITHUB_REPOSITORY: ${{ vars.GITHUB_REPOSITORY }}
+    - name: Publish version
+      run: Tools/publish_multi_request.py
+      env:
+        PUBLISH_TOKEN: ${{ secrets.PUBLISH_TOKEN }}
+        GITHUB_REPOSITORY: ${{ vars.GITHUB_REPOSITORY }}
+        FORK_ID: ${{ vars.FORK_ID }}
 
     #    - name: Publish changelog (Discord)
     #      run: Tools/actions_changelogs_since_last_run.py
@@ -126,8 +86,3 @@ jobs:
     #      run: Tools/actions_changelog_rss.py
     #      env:
     #        CHANGELOG_RSS_KEY: ${{ secrets.CHANGELOG_RSS_KEY }}
-
-    #- uses: geekyeggo/delete-artifact@v5
-    #  if: always()
-    #  with:
-    #    name: build

.github/workflows/update-credits.yml

@@ -19,6 +19,8 @@ jobs:
 
       - name: Get this week's Contributors
         shell: pwsh
+        env:
+          GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
         run: Tools/dump_github_contributors.ps1 > Resources/Credits/GitHub.txt
 
       # TODO

Content.Benchmarks/MapLoadBenchmark.cs

@@ -46,7 +46,7 @@ public async Task Cleanup()
         PoolManager.Shutdown();
     }
 
-    public static readonly string[] MapsSource = { "Empty", "Satlern", "Box", "Bagel", "Dev", "CentComm", "Core", "TestTeg", "Packed", "Omega", "Reach", "Meta", "Marathon", "MeteorArena", "Fland", "Oasis" };
+    public static readonly string[] MapsSource = { "Empty", "Satlern", "Box", "Bagel", "Dev", "CentComm", "Core", "TestTeg", "Packed", "Omega", "Reach", "Meta", "Marathon", "MeteorArena", "Fland", "Oasis", "Cog" };
 
     [ParamsSource(nameof(MapsSource))]
     public string Map;

Content.Client/ADT/Heretic/HereticCombatMarkSystem.cs

@@ -0,0 +1,44 @@
+using Content.Shared.Heretic;
+using Robust.Client.GameObjects;
+using Robust.Shared.Utility;
+
+namespace Content.Client.Heretic;
+
+public sealed partial class HereticCombatMarkSystem : EntitySystem
+{
+    public override void Initialize()
+    {
+        base.Initialize();
+
+        SubscribeLocalEvent<HereticCombatMarkComponent, ComponentStartup>(OnStartup);
+        SubscribeLocalEvent<HereticCombatMarkComponent, ComponentShutdown>(OnShutdown);
+    }
+
+    private void OnStartup(Entity<HereticCombatMarkComponent> ent, ref ComponentStartup args)
+    {
+        if (!TryComp<SpriteComponent>(ent, out var sprite))
+            return;
+
+        if (sprite.LayerMapTryGet(0, out var l))
+        {
+            sprite.LayerSetState(l, ent.Comp.Path.ToLower());
+            return;
+        }
+
+        var rsi = new SpriteSpecifier.Rsi(new ResPath("ADT/Heretic/combat_marks.rsi"), ent.Comp.Path.ToLower());
+        var layer = sprite.AddLayer(rsi);
+
+        sprite.LayerMapSet(0, layer);
+        sprite.LayerSetShader(layer, "unshaded");
+    }
+    private void OnShutdown(Entity<HereticCombatMarkComponent> ent, ref ComponentShutdown args)
+    {
+        if (!TryComp<SpriteComponent>(ent, out var sprite))
+            return;
+
+        if (!sprite.LayerMapTryGet(0, out var layer))
+            return;
+
+        sprite.RemoveLayer(layer);
+    }
+}

Content.Client/ADT/Heretic/Ritual.CustomBehaviors.cs

@@ -0,0 +1,67 @@
+using Content.Shared.Heretic.Prototypes;
+
+namespace Content.Client.Heretic;
+
+// these do nothing and are there just for yaml limter to shut the fuck up.
+// make sure they stay up in sync with the server counterpart.
+// regards.
+// - john
+
+public sealed partial class RitualAshAscendBehavior : RitualSacrificeBehavior { }
+public sealed partial class RitualMuteGhoulifyBehavior : RitualSacrificeBehavior { }
+
+[Virtual] public partial class RitualSacrificeBehavior : RitualCustomBehavior
+{
+    public override bool Execute(RitualData args, out string? outstr)
+    {
+        outstr = null;
+        return true;
+    }
+
+    public override void Finalize(RitualData args)
+    {
+        // do nothing
+    }
+}
+
+public sealed partial class RitualTemperatureBehavior : RitualCustomBehavior
+{
+    public override bool Execute(RitualData args, out string? outstr)
+    {
+        outstr = null;
+        return true;
+    }
+
+    public override void Finalize(RitualData args)
+    {
+        // do nothing
+    }
+}
+
+public sealed partial class RitualReagentPuddleBehavior : RitualCustomBehavior
+{
+    public override bool Execute(RitualData args, out string? outstr)
+    {
+        outstr = null;
+        return true;
+    }
+
+    public override void Finalize(RitualData args)
+    {
+        // do nothing
+    }
+}
+
+public sealed partial class RitualKnowledgeBehavior : RitualCustomBehavior
+{
+    public override bool Execute(RitualData args, out string? outstr)
+    {
+        outstr = null;
+        return true;
+    }
+
+    public override void Finalize(RitualData args)
+    {
+        // do nothing
+    }
+}