[PORT] add reactive armore #615
Conversation
WalkthroughВ данном запросе на изменение добавлены новые компоненты и системы, связанные с механикой телепортации при получении урона. Введены два компонента: Changes
Possibly related PRs
Suggested labels
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
RSI Diff Bot; head commit 7a1976a merging into 2377bc3 Resources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 11
🧹 Outside diff range and nitpick comments (7)
Content.Shared/_Cats/Components/DamageTeleportComponent.cs (1)
8-9: Добавьте XML-документацию для компонентаДля улучшения поддержки кода рекомендуется добавить XML-документацию, описывающую назначение компонента и его взаимодействие с системой DamageTeleportSystem.
+/// <summary> +/// Компонент, обеспечивающий телепортацию сущности при получении урона. +/// </summary> [RegisterComponent, Access(typeof(DamageTeleportSystem))] public sealed partial class DamageTeleportComponent : ComponentResources/Prototypes/_SpaceCats/Entities/Clothing/OuterClothing/vests.yml (2)
4-5: Требуется более подробное описаниеТекущее описание "Reactive armor" недостаточно информативно. Следует добавить информацию о механике телепортации при получении урона и других особенностях брони.
Предлагаю обновить описание:
- description: Reactive armor. + description: Продвинутая броня, оснащенная системой телепортации, которая активируется при получении урона. Обеспечивает хорошую защиту от различных типов повреждений.
28-28: Добавьте пустую строку в конец файлаОтсутствует пустая строка в конце файла, что может вызвать проблемы при обработке файла некоторыми инструментами.
Добавьте пустую строку в конец файла.
🧰 Tools
🪛 yamllint (1.29.0-1)
[error] 28-28: no new line character at the end of file
(new-line-at-end-of-file)
Content.Shared/_Cats/Components/ReactiveArmorComponent.cs (2)
23-28: Добавьте валидацию порога уронаПоле
DamageThresholdдолжно быть положительным числом. Рекомендуется добавить атрибут для валидации значения.[DataField("damageThreshold", required: true)] +[ViewVariables(VVAccess.ReadWrite)] public float DamageThreshold;
29-34: Исправьте документацию и добавьте валидацию накопленного урона
- В комментарии упоминается "artifact", что выглядит как ошибка копирования.
- Необходимо добавить проверку на отрицательные значения для
AccumulatedDamage./// <summary> -/// How much damage has been accumulated on the artifact so far +/// How much damage has been accumulated on the armor so far /// </summary> [DataField, ViewVariables(VVAccess.ReadWrite)] -public float AccumulatedDamage = 0; +public float AccumulatedDamage +{ + get => _accumulatedDamage; + set => _accumulatedDamage = Math.Max(0, value); +} +private float _accumulatedDamage = 0;Content.Shared/_Cats/DamageTeleportSystem.cs (2)
1-21: Необходимо очистить неиспользуемые импортыВ файле присутствуют импорты, которые не используются в коде:
Content.Shared.AlertContent.Shared.GravityContent.Shared.StandingRobust.Shared.PlayerContent.Shared.Mobs.ComponentsContent.Shared.DamageRobust.Shared.Physics.DynamicsRobust.Shared.Physics.EventsТакже заголовок файла нестандартный и может быть улучшен.
-/// Maded by Gorox for Enterprise. See CLA +// Copyright © 2024 Space Station 14 Team +// Licensed under the Space Station 14 License
34-41: Рекомендуется реорганизация обработки событийВсе три обработчика событий выполняют идентичную логику телепортации. Рекомендуется выделить общую логику в отдельный метод.
+ private void HandleTeleport(EntityUid uid, DamageTeleportComponent component) + { + var xform = Transform(uid); + _transform.SetCoordinates(uid, xform, xform.Coordinates.Offset( + _random.NextVector2(component.MinRange, component.MaxRange))); + } + public override void Initialize() { base.Initialize(); - SubscribeLocalEvent<DamageTeleportComponent, AttackedEvent>(OnReactiveDamaged); - SubscribeLocalEvent<DamageTeleportComponent, ProjectileReflectAttemptEvent>(OnReactiveDamagedProjectile); - SubscribeLocalEvent<DamageTeleportComponent, HitScanReflectAttemptEvent>(OnReactiveDamagedHitscan); + SubscribeLocalEvent<DamageTeleportComponent, AttackedEvent>((uid, comp, args) => HandleTeleport(uid, comp)); + SubscribeLocalEvent<DamageTeleportComponent, ProjectileReflectAttemptEvent>((uid, comp, args) => HandleTeleport(uid, comp)); + SubscribeLocalEvent<DamageTeleportComponent, HitScanReflectAttemptEvent>((uid, comp, args) => HandleTeleport(uid, comp)); }
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
⛔ Files ignored due to path filters (4)
Resources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/equipped-OUTERCLOTHING.pngis excluded by!**/*.pngResources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/icon.pngis excluded by!**/*.pngResources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/inhand-right.pngis excluded by!**/*.png
📒 Files selected for processing (7)
Content.Shared/_Cats/Components/DamageTeleportComponent.cs(1 hunks)Content.Shared/_Cats/Components/ReactiveArmorComponent.cs(1 hunks)Content.Shared/_Cats/DamageTeleportSystem.cs(1 hunks)Resources/Locale/ru-RU/_Cats/entities/clothing/Outer.ftl(1 hunks)Resources/Prototypes/Catalog/Fills/Lockers/heads.yml(1 hunks)Resources/Prototypes/_SpaceCats/Entities/Clothing/OuterClothing/vests.yml(1 hunks)Resources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/meta.json(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- Resources/Textures/_SpaceCats/Clothing/OuterClothing/reactive.rsi/meta.json
🧰 Additional context used
🪛 GitHub Check: build (ubuntu-latest)
Content.Shared/_Cats/Components/ReactiveArmorComponent.cs
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
🪛 GitHub Check: YAML Linter
Content.Shared/_Cats/Components/ReactiveArmorComponent.cs
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
🪛 GitHub Check: Test Packaging
Content.Shared/_Cats/Components/ReactiveArmorComponent.cs
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
🪛 yamllint (1.29.0-1)
Resources/Prototypes/_SpaceCats/Entities/Clothing/OuterClothing/vests.yml
[error] 28-28: no new line character at the end of file
(new-line-at-end-of-file)
🔇 Additional comments (4)
Resources/Locale/ru-RU/_Cats/entities/clothing/Outer.ftl (1)
1-1: Перевод названия корректный!
Перевод "reactive armor" как "реактивная броня" выполнен правильно и соответствует игровой терминологии.
Content.Shared/_Cats/Components/DamageTeleportComponent.cs (1)
1-16: Проверить корректность размещения файла
Компонент находится в директории _Cats, что выглядит нестандартно для структуры проекта. Необходимо проверить, является ли это временным размещением при портировании.
✅ Verification successful
Let me gather more information about this _Cats directory and its purpose.
Размещение файла в директории _Cats является корректным
Анализ показал, что директория _Cats является специальным модулем проекта, содержащим множество связанных компонентов, систем и ресурсов. Это не временное размещение, а полноценная часть кодовой базы, включающая:
- Компоненты и системы в
Content.Shared/_CatsиContent.Server/_Cats - Локализации в
Resources/Locale/{ru-RU,en-US}/_Cats - Аудио ресурсы в
Resources/Audio/_Cats
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check for other components in similar directory structure
echo "Checking for other components in _Cats directory:"
fd -e cs . | grep "_Cats/Components"
echo "Checking for standard component locations:"
fd -e cs . | grep "Components/" | grep -v "_Cats" | head -n 5
Length of output: 915
Script:
#!/bin/bash
# Check if there's any documentation or explanation about _Cats directory
echo "Checking for documentation mentioning _Cats:"
rg -i "_cats" "*.md"
echo -e "\nChecking for any other files in _Cats directory structure:"
fd . | grep "_Cats"
echo -e "\nChecking the git history of the _Cats directory:"
git log --oneline -- "*/_Cats/*" | head -n 5
Length of output: 10307
Resources/Prototypes/_SpaceCats/Entities/Clothing/OuterClothing/vests.yml (1)
13-21: Проверьте баланс защитных характеристик
Текущие коэффициенты брони предоставляют значительную защиту от всех типов урона. Учитывая наличие механики телепортации, возможно, следует немного снизить защитные характеристики для баланса.
Рекомендую пересмотреть коэффициенты защиты с учетом общего баланса игры.
Content.Shared/_Cats/Components/ReactiveArmorComponent.cs (1)
1-34: Проверьте интеграцию с системой урона
Компонент является частью механики телепортации при получении урона. Убедитесь, что:
- Система корректно подписана на события урона
- Обработка накопленного урона происходит атомарно
- Телепортация синхронизирована с сетевым кодом
🧰 Tools
🪛 GitHub Check: build (ubuntu-latest)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
🪛 GitHub Check: YAML Linter
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
🪛 GitHub Check: Test Packaging
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 8-8:
The type or namespace name 'ReactiveArmorSystem' could not be found (are you missing a using directive or an assembly reference?)
| @@ -0,0 +1,2 @@ | |||
| ent-ClothingOuterReactiveArmor = реактивная броня | |||
| .desc = Дорогая броня, с использованием блюспейс ядра. При получении повреждении телепортирует владельца на небольшое расстояние. | |||
There was a problem hiding this comment.
Исправьте опечатку в слове "повреждении"
В описании допущена грамматическая ошибка.
Примените следующие изменения:
- .desc = Дорогая броня, с использованием блюспейс ядра. При получении повреждении телепортирует владельца на небольшое расстояние.
+ .desc = Дорогая броня, с использованием блюспейс ядра. При получении повреждений телепортирует владельца на небольшое расстояние.📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| .desc = Дорогая броня, с использованием блюспейс ядра. При получении повреждении телепортирует владельца на небольшое расстояние. | |
| .desc = Дорогая броня, с использованием блюспейс ядра. При получении повреждений телепортирует владельца на небольшое расстояние. |
| [DataField("minRandomRadius"), ViewVariables(VVAccess.ReadWrite)] | ||
| public float MinRange = 1.0f; | ||
|
|
||
| [DataField("maxRandomRadius"), ViewVariables(VVAccess.ReadWrite)] | ||
| public float MaxRange = 2.0f; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Требуются улучшения в определении полей
Обнаружено несколько проблем с полями компонента:
- Отсутствует проверка, что MaxRange > MinRange
- Имена в DataField атрибутах не соответствуют именам полей
- Отсутствует валидация допустимых значений
- Отсутствует XML-документация полей
Предлагаемые изменения:
+ /// <summary>
+ /// Минимальный радиус случайной телепортации в метрах.
+ /// </summary>
- [DataField("minRandomRadius"), ViewVariables(VVAccess.ReadWrite)]
+ [DataField("minRange"), ViewVariables(VVAccess.ReadWrite)]
+ [DataField("validateRanges")]
public float MinRange = 1.0f;
+ /// <summary>
+ /// Максимальный радиус случайной телепортации в метрах.
+ /// </summary>
- [DataField("maxRandomRadius"), ViewVariables(VVAccess.ReadWrite)]
+ [DataField("maxRange"), ViewVariables(VVAccess.ReadWrite)]
+ [DataField("validateRanges")]
public float MaxRange = 2.0f;
+ private void ValidateRanges()
+ {
+ if (MinRange < 0)
+ MinRange = 0;
+ if (MaxRange <= MinRange)
+ MaxRange = MinRange + 1.0f;
+ }Committable suggestion skipped: line range outside the PR's diff.
| @@ -0,0 +1,16 @@ | |||
| /// Maded by Gorox for Enterprise. See CLA | |||
There was a problem hiding this comment.
💡 Codebase verification
Необходимо привести заголовки файлов к стандартному виду проекта
Обнаружено, что все файлы в директории _Cats содержат нестандартный заголовок "Maded by Gorox for Enterprise. See CLA", в то время как остальные файлы проекта используют стандартный формат без указания авторства. Необходимо:
- Удалить нестандартные заголовки CLA из следующих файлов:
- Content.Shared/_Cats/Components/DamageTeleportComponent.cs
- Content.Shared/_Cats/DamageTeleportSystem.cs
- Content.Shared/_Cats/Components/ReactiveArmorComponent.cs
Файлы должны следовать общему стилю проекта, где используются только необходимые using-директивы без указания персонального авторства.
🔗 Analysis chain
Необходимо использовать стандартный заголовок лицензии
Текущий заголовок CLA не соответствует стандартному формату лицензирования проекта. Рекомендуется использовать стандартный заголовок лицензии, принятый в проекте.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check the standard license header format used in the project
rg -l "^// <copyright" --type cs | head -n 1 | xargs cat | head -n 3
Length of output: 70
Script:
#!/bin/bash
# Let's check for common license header patterns in C# files
rg -l "^//.*[Cc]opyright|^//.*[Ll]icense|^//.*MIT|^//.*GPL|^//.*BSD" --type cs | head -n 5 | xargs -I {} sh -c 'echo "=== {} ==="; head -n 3 "{}"'
# Also check if there are other files with similar CLA mentions
rg -l "Gorox.*Enterprise|CLA" --type cs
Length of output: 354
Script:
#!/bin/bash
# Let's check the content of these files to see the pattern of CLA usage
echo "=== DamageTeleportSystem.cs ==="
head -n 3 Content.Shared/_Cats/DamageTeleportSystem.cs
echo -e "\n=== ReactiveArmorComponent.cs ==="
head -n 3 Content.Shared/_Cats/Components/ReactiveArmorComponent.cs
# Let's also check if there's a standard header pattern in other parts of the codebase
echo -e "\n=== Other C# files headers ==="
fd -e cs -E "_Cats" | head -n 5 | xargs -I {} sh -c 'echo "=== {} ==="; head -n 3 "{}"'
Length of output: 1509
| - type: Clothing | ||
| sprite: _SpaceCatsd/Clothing/OuterClothing/reactive.rsi |
There was a problem hiding this comment.
Исправьте опечатку в пути к спрайту
В компоненте Clothing указан неверный путь к спрайту (_SpaceCatsd вместо _SpaceCats).
Примените следующие изменения:
- sprite: _SpaceCatsd/Clothing/OuterClothing/reactive.rsi
+ sprite: _SpaceCats/Clothing/OuterClothing/reactive.rsi📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - type: Clothing | |
| sprite: _SpaceCatsd/Clothing/OuterClothing/reactive.rsi | |
| - type: Clothing | |
| sprite: _SpaceCats/Clothing/OuterClothing/reactive.rsi |
| component: | ||
| - type: DamageTeleport |
There was a problem hiding this comment.
Требуется конфигурация компонента DamageTeleport
Компонент DamageTeleport добавлен, но отсутствуют важные параметры, такие как MinRange и MaxRange для телепортации.
Добавьте необходимые параметры:
- type: ClothingGrant
component:
- type: DamageTeleport
+ minRange: 2.0
+ maxRange: 7.0Committable suggestion skipped: line range outside the PR's diff.
🧰 Tools
🪛 yamllint (1.29.0-1)
[error] 28-28: no new line character at the end of file
(new-line-at-end-of-file)
| /// <summary> | ||
| /// What damage types are accumulated for the trigger? | ||
| /// </summary> | ||
| [DataField("damageTypes", customTypeSerializer: typeof(PrototypeIdListSerializer<DamageTypePrototype>))] | ||
| public List<string>? DamageTypes; | ||
|
|
There was a problem hiding this comment.
🛠️ Refactor suggestion
Добавьте проверку на null для DamageTypes
Поле DamageTypes может быть null, но отсутствуют соответствующие проверки. Рекомендуется либо сделать поле non-nullable с пустым списком по умолчанию, либо добавить проверки.
-public List<string>? DamageTypes;
+[DataField("damageTypes", customTypeSerializer: typeof(PrototypeIdListSerializer<DamageTypePrototype>))]
+public List<string> DamageTypes = new();📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| /// <summary> | |
| /// What damage types are accumulated for the trigger? | |
| /// </summary> | |
| [DataField("damageTypes", customTypeSerializer: typeof(PrototypeIdListSerializer<DamageTypePrototype>))] | |
| public List<string>? DamageTypes; | |
| /// <summary> | |
| /// What damage types are accumulated for the trigger? | |
| /// </summary> | |
| [DataField("damageTypes", customTypeSerializer: typeof(PrototypeIdListSerializer<DamageTypePrototype>))] | |
| public List<string> DamageTypes = new(); |
| /// Maded by Gorox for Enterprise. See CLA | ||
| using Content.Shared.ReactiveArmor.Systems; | ||
| using Robust.Shared.Serialization.TypeSerializers.Implementations.Custom.Prototype.List; | ||
| using Content.Shared.Damage.Prototypes; | ||
|
|
||
| namespace Content.Shared.ReactiveArmor.Components; |
There was a problem hiding this comment.
💡 Codebase verification
Подтверждено отсутствие ReactiveArmorSystem
Анализ кодовой базы показал, что компонент ReactiveArmorComponent.cs действительно является единственным файлом, связанным с реактивной броней, и система ReactiveArmorSystem отсутствует. Необходимо:
- Создать файл
ReactiveArmorSystem.csв директорииContent.Shared/_Cats/Systems/ - Реализовать систему, наследующую
ISystem - Обеспечить обработку логики реактивной брони
🔗 Analysis chain
Отсутствует реализация системы ReactiveArmorSystem
Компонент ссылается на Content.Shared.ReactiveArmor.Systems, но соответствующая система не найдена в сборке. Необходимо создать ReactiveArmorSystem.cs в директории Content.Shared/_Cats/Systems/.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Check if ReactiveArmorSystem exists
fd ReactiveArmorSystem.cs
Length of output: 27
Script:
#!/bin/bash
# Search for any files containing ReactiveArmor in their name
fd -t f ReactiveArmor
# Search for any references to ReactiveArmor namespace or system
rg -l "ReactiveArmor.(Systems|Components)"
# Check for any system implementations related to ReactiveArmor
ast-grep --pattern 'class $_ : ISystem {
$$$
}'
Length of output: 236
🧰 Tools
🪛 GitHub Check: build (ubuntu-latest)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
🪛 GitHub Check: YAML Linter
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
🪛 GitHub Check: Test Packaging
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
[failure] 2-2:
The type or namespace name 'Systems' does not exist in the namespace 'Content.Shared.ReactiveArmor' (are you missing an assembly reference?)
| [Dependency] private readonly INetManager _netManager = default!; | ||
| [Dependency] private readonly EntityLookupSystem _lookup = default!; | ||
| [Dependency] private readonly IRobustRandom _random = default!; | ||
| [Dependency] private readonly IGameTiming _gameTiming = default!; | ||
| [Dependency] private readonly SharedTransformSystem _transform = default!; | ||
| [Dependency] private readonly AlertsSystem _alerts = default!; | ||
| [Dependency] private readonly SharedTransformSystem _xform = default!; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Требуется оптимизация зависимостей
В классе обнаружены следующие проблемы:
- Дублирование инъекции
SharedTransformSystem(_transform и _xform) - Неиспользуемая зависимость
AlertsSystem
- [Dependency] private readonly AlertsSystem _alerts = default!;
- [Dependency] private readonly SharedTransformSystem _xform = default!;Используйте единую ссылку на SharedTransformSystem.
Committable suggestion skipped: line range outside the PR's diff.
| private void OnReactiveDamaged(EntityUid uid, DamageTeleportComponent component, ref AttackedEvent args) | ||
| { | ||
| var xform = Transform(uid); | ||
| _xform.SetCoordinates(uid, xform, xform.Coordinates.Offset(_random.NextVector2(component.MinRange, component.MaxRange))); | ||
| } | ||
|
|
||
| private void OnReactiveDamagedProjectile(EntityUid uid, DamageTeleportComponent component, ref ProjectileReflectAttemptEvent args) | ||
| { | ||
| var xform = Transform(uid); | ||
| _xform.SetCoordinates(uid, xform, xform.Coordinates.Offset(_random.NextVector2(component.MinRange, component.MaxRange))); | ||
| } | ||
|
|
||
| private void OnReactiveDamagedHitscan(EntityUid uid, DamageTeleportComponent component, ref HitScanReflectAttemptEvent args) | ||
| { | ||
| var xform = Transform(uid); | ||
| _xform.SetCoordinates(uid, xform, xform.Coordinates.Offset(_random.NextVector2(component.MinRange, component.MaxRange))); | ||
| } |
There was a problem hiding this comment.
💡 Codebase verification
Based on the gathered information, I can now provide the final response:
Подтверждаю необходимость проверок безопасности телепортации
Анализ кодовой базы показывает, что другие системы телепортации (например, RandomTeleportArtifactSystem) также не имеют необходимых проверок безопасности. Это системная проблема, которая требует исправления.
Рекомендуемые изменения:
- Добавить проверку коллизий через PhysicsComponent (CollisionLayer/CollisionMask)
- Реализовать валидацию координат с использованием существующей системы CanAccess
- Добавить обработку случаев неудачной телепортации с возвратом в исходную позицию
Примечание: В системе уже существуют необходимые компоненты и методы для реализации этих проверок, что упростит внедрение исправлений.
🔗 Analysis chain
Отсутствуют проверки безопасности телепортации
Текущая реализация телепортации может привести к следующим проблемам:
- Телепортация в недопустимые локации
- Отсутствие проверки коллизий
- Возможность застревания в стенах или других объектах
Рекомендуется:
- Добавить проверку валидности новых координат
- Реализовать обработку коллизий
- Добавить fallback механизм при неудачной телепортации
Let's gather more information about the teleportation implementations in the found files.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка наличия подобных механизмов безопасности в других системах телепортации
# Поиск других систем телепортации
rg -l "SetCoordinates.*Offset" --type cs
# Поиск примеров проверки координат
ast-grep --pattern 'if ($COND) { $$$; SetCoordinates($$$); }'
Length of output: 342
Script:
#!/bin/bash
# Check the implementation of other teleportation systems
rg -A 10 "SetCoordinates.*Offset" Content.Server/Xenoarchaeology/XenoArtifacts/Effects/Systems/RandomTeleportArtifactSystem.cs Content.Server/Disposal/Unit/EntitySystems/DisposableSystem.cs
# Look for any coordinate validation methods
rg "ValidateCoordinates|IsValidCoordinates|CanAccess" --type cs
# Check for any collision-related methods
rg "CheckCollision|HandleCollision|CollisionLayer" --type cs
Length of output: 33595
| @@ -301,6 +301,7 @@ | |||
| - id: FlatpackOracle # backmen | |||
| - id: FlatpackSophie # backmen | |||
| - id: AnomalyPinpointer # cats | |||
| - id: ClothingOuterReactiveArmor # cats | |||
There was a problem hiding this comment.
💡 Codebase verification
Реактивная броня требует балансировки и документации
На основе проверки кодовой базы:
- Броня имеет коэффициент сопротивления взрывам 0.65, что делает её достаточно мощной
- Она доступна только в шкафчике директора исследований
- Отсутствует подробная документация о механике работы
Рекомендации:
- Добавить подробное описание механики работы брони в файл документации
- Пересмотреть значение damageCoefficient с учётом баланса игры
- Рассмотреть возможность добавления недостатков для компенсации сильной защиты
🔗 Analysis chain
Необходимо документировать возможности реактивной брони
Добавление реактивной брони в шкафчик директора исследований является значительным изменением с точки зрения игрового баланса. Рекомендуется:
- Добавить комментарий, описывающий возможности брони
- Проверить баланс относительно других глав станции
- Убедиться, что это соответствует роли директора исследований
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check for documentation and configuration of reactive armor
# Look for reactive armor documentation
rg -l "ReactiveArmor" -g "*.md"
# Check reactive armor configuration
rg -l "ReactiveArmor" -g "*.yml"
Length of output: 191
Script:
#!/bin/bash
# Check the reactive armor configuration details
rg -A 10 "ReactiveArmor" Resources/Prototypes/_SpaceCats/Entities/Clothing/OuterClothing/vests.yml
# Check if there are any other head lockers with similar armor
rg -A 5 -B 5 "ClothingOuterReactiveArmor" Resources/Prototypes/Catalog/Fills/Lockers/heads.yml
# Look for any balance-related configurations or components
ast-grep --pattern 'id: ClothingOuterReactiveArmor
parent: $_
components:
$$$'
Length of output: 975
Реактивная броня. Телепортирует своего носителя до получение урона.
Summary by CodeRabbit
Новые функции
Исправления ошибок