-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Фикс хвостов #344
Фикс хвостов #344
Conversation
WalkthroughЭтот набор изменений охватывает различные аспекты игровых сущностей и прототипов, включая модификации систем одежды, добавление новых компонентов для различных видов существ и уточнение визуальных слоев. Основные изменения включают активацию компонента Changes
Sequence DiagramsequenceDiagram
participant Player
participant ClothingSystem
participant Entity
Player->>ClothingSystem: Equip clothing item
alt Item has "HidesTail" tag
ClothingSystem->>Entity: Set Tail layer visibility to false
end
Player->>ClothingSystem: Unequip clothing item
alt Item has "HidesTail" tag
ClothingSystem->>Entity: Set Tail layer visibility to true
end
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 (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 4
🔭 Outside diff range comments (4)
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml (1)
Порядок слоев требует корректировки
В базовом прототипе (
base.yml
) слойouterClothing
расположен перед слоемTail
, что может вызывать проблемы с отображением хвостов в скафандрах. В файлеarachnid.yml
сохраняется та же проблема. Для корректного отображения хвостов рекомендуется:
- Переместить слой
enum.HumanoidVisualLayers.Tail
перед слоемouterClothing
в файлах:
- Resources/Prototypes/Entities/Mobs/Species/base.yml
- Resources/Prototypes/Entities/Mobs/Species/arachnid.yml
🔗 Analysis chain
Line range hint
98-129
: Проверьте порядок слоев спрайтовКонфигурация слоев выглядит корректной, но стоит отметить наличие комментария о необходимости переименования слоя хвоста (
Tail
). Учитывая цели PR по исправлению видимости хвостов в скафандрах, убедитесь, что слойenum.HumanoidVisualLayers.Tail
правильно взаимодействует с слоемouterClothing
.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверяем связанные файлы на наличие логики обработки хвостов rg -l "HumanoidVisualLayers.Tail" Resources/Prototypes/Length of output: 680
Script:
#!/bin/bash # Проверяем содержимое файлов на предмет порядка слоев и взаимодействия с outerClothing rg -A 5 -B 5 "outerClothing" Resources/Prototypes/Entities/Mobs/Species/base.yml rg -A 5 -B 5 "Tail" Resources/Prototypes/Entities/Mobs/Species/base.yml # Проверяем, есть ли специальная логика для хвостов в скафандрах rg -l "spacesuit.*tail|tail.*spacesuit" Resources/Length of output: 1580
🧰 Tools
🪛 yamllint (1.35.1)
[error] 103-103: too many spaces inside brackets
(brackets)
[error] 103-103: too many spaces inside brackets
(brackets)
Resources/Prototypes/Entities/Mobs/Player/guardian.yml (3)
Line range hint
134-134
: Рекомендуется улучшить ИИ голопаразитаТекущая реализация
SimpleHumanoidHostileCompound
может быть слишком примитивной для голопаразита. Рекомендуется создать специализированную HTN задачу, учитывающую уникальные способности существа.
Line range hint
147-150
: Необходима документация для псионических способностейДобавьте комментарии, объясняющие:
- Почему компонент Psionic не может быть удален
- Как работает TelepathyPower в контексте Ифрита
Line range hint
219-223
: Требуется балансировка уронаУрон MeleeWeapon (5 единиц) значительно ниже, чем у базового стража (20 единиц). Это может сделать HoloClown неэффективным в бою.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (25)
Content.Shared/Clothing/EntitySystems/ClothingSystem.cs
(2 hunks)Resources/Prototypes/DeltaV/Entities/Mobs/Species/vulpkanin.yml
(1 hunks)Resources/Prototypes/Entities/Clothing/OuterClothing/suits.yml
(4 hunks)Resources/Prototypes/Entities/Mobs/Player/dwarf.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/familiars.yml
(4 hunks)Resources/Prototypes/Entities/Mobs/Player/guardian.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/ipc.yml
(4 hunks)Resources/Prototypes/Entities/Mobs/Player/moth.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/ratvar.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/shadowkin.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/slime.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Player/vox.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/arachne.yml
(5 hunks)Resources/Prototypes/Entities/Mobs/Species/arachnid.yml
(3 hunks)Resources/Prototypes/Entities/Mobs/Species/diona.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/dwarf.yml
(2 hunks)Resources/Prototypes/Entities/Mobs/Species/harpy.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/human.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/moth.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/reptilian.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/skeleton.yml
(2 hunks)Resources/Prototypes/Entities/Mobs/Species/slime.yml
(1 hunks)Resources/Prototypes/Entities/Mobs/Species/vox.yml
(0 hunks)Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Shark/Entities/Mobs/Species/Shark.yml
(1 hunks)
💤 Files with no reviewable changes (1)
- Resources/Prototypes/Entities/Mobs/Species/vox.yml
🧰 Additional context used
📓 Path-based instructions (23)
Resources/Prototypes/Entities/Mobs/Player/vox.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/shadowkin.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/dwarf.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/moth.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/slime.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Shark/Entities/Mobs/Species/Shark.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/human.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/reptilian.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/moth.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/ipc.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/familiars.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/diona.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/harpy.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/guardian.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Player/ratvar.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/slime.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/arachne.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/dwarf.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/DeltaV/Entities/Mobs/Species/vulpkanin.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Clothing/OuterClothing/suits.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
🪛 yamllint (1.35.1)
Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml
[error] 82-82: too many spaces inside empty braces
(braces)
Resources/Prototypes/Entities/Mobs/Species/human.yml
[warning] 13-13: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/reptilian.yml
[warning] 14-14: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/moth.yml
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Player/ipc.yml
[error] 58-58: too many spaces inside empty braces
(braces)
Resources/Prototypes/Entities/Mobs/Species/diona.yml
[warning] 14-14: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/slime.yml
[warning] 9-9: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml
[warning] 12-12: too few spaces before comment
(comments)
[error] 43-43: wrong indentation: expected 8 but found 6
(indentation)
Resources/Prototypes/Entities/Mobs/Species/arachne.yml
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 69-69: too many spaces inside brackets
(brackets)
[error] 69-69: too many spaces inside brackets
(brackets)
[error] 70-70: too many spaces inside brackets
(brackets)
[error] 70-70: too many spaces inside brackets
(brackets)
[error] 73-73: too many spaces inside brackets
(brackets)
[error] 73-73: too many spaces inside brackets
(brackets)
[error] 74-74: too many spaces inside brackets
(brackets)
[error] 74-74: too many spaces inside brackets
(brackets)
[warning] 74-74: too few spaces before comment
(comments)
[error] 167-167: too many spaces inside brackets
(brackets)
[error] 167-167: too many spaces inside brackets
(brackets)
[error] 211-211: too many spaces inside brackets
(brackets)
[error] 211-211: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 215-215: too many spaces inside brackets
(brackets)
[error] 215-215: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[warning] 216-216: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/dwarf.yml
[warning] 10-10: too few spaces before comment
(comments)
Resources/Prototypes/DeltaV/Entities/Mobs/Species/vulpkanin.yml
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml
[warning] 15-15: too few spaces before comment
(comments)
[error] 130-130: wrong indentation: expected 6 but found 2
(indentation)
[error] 131-131: wrong indentation: expected 6 but found 2
(indentation)
🔇 Additional comments (34)
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml (3)
Line range hint 1-143
: Общие замечания по файлу
- Компонент
Carriable
успешно добавлен, что соответствует целям PR по реализации системы переноски партнеров. - Структура файла в целом соответствует требованиям, за исключением отмеченных проблем с отступами.
- Все изменения соответствуют целям PR и не содержат неожиданных модификаций.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 103-103: too many spaces inside brackets
(brackets)
[error] 103-103: too many spaces inside brackets
(brackets)
15-15
: 🧹 Nitpick (assertive)
Проверьте форматирование комментария
Комментарий должен иметь два пробела перед ним согласно стандартам форматирования.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 15-15: too few spaces before comment
(comments)
130-131
:
Исправьте отступы и проверьте взаимодействие компонентов
Компоненты Spider
и IgnoreSpiderWeb
имеют неправильный отступ. Также необходимо убедиться, что эти компоненты корректно взаимодействуют с существующей логикой.
- - type: Spider
- - type: IgnoreSpiderWeb
+ - type: Spider
+ - type: IgnoreSpiderWeb
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 130-130: wrong indentation: expected 6 but found 2
(indentation)
[error] 131-131: wrong indentation: expected 6 but found 2
(indentation)
Resources/Prototypes/Entities/Mobs/Player/shadowkin.yml (2)
Line range hint 1-6
: Структура прототипа соответствует требованиям
Базовая структура YAML-файла корректна, имя сущности на английском языке, что соответствует требованиям.
Line range hint 1-6
: Проверьте соответствие изменений целям PR
В описании PR упоминаются изменения, связанные с видимостью хвостов при ношении скафандра, однако в данном файле такие изменения отсутствуют. Возможно, требуются дополнительные изменения для реализации этой функциональности.
Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml (2)
82-82
: 🧹 Nitpick (assertive)
Исправьте форматирование в GibBehavior
Уберите лишний пробел между фигурными скобками.
- - !type:GibBehavior { }
+ - !type:GibBehavior {}
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 82-82: too many spaces inside empty braces
(braces)
82-82
: Проверьте коэффициенты урона для баланса
Текущие коэффициенты урона для IPC:
- Poison: 0 (иммунитет)
- Cold: 0.2 (высокая устойчивость)
- Heat: 2 (высокая уязвимость)
- Shock: 2.5 (очень высокая уязвимость)
Пожалуйста, подтвердите, что эти значения соответствуют желаемому игровому балансу.
Also applies to: 82-82
✅ Verification successful
Let me analyze the damage modifiers for IPC (silicon-based life forms) by searching for their definition in the codebase.
Let me try a different search approach to find the silicon/IPC damage modifiers.
Коэффициенты урона для IPC сбалансированы и соответствуют их природе как синтетических существ
Текущие коэффициенты урона для IPC:
- Poison (Яд): 0 (полный иммунитет, что логично для роботов)
- Cold (Холод): 0.2 (высокая устойчивость, соответствует их металлической природе)
- Heat (Тепло): 2.0 (повышенная уязвимость, что логично для электронных компонентов)
- Shock (Электричество): 2.5 (высокая уязвимость, обоснована их электронной природой)
Дополнительные характеристики:
- Базовая температура тела: 310.15 K
- Порог теплового урона: 325 K
- Порог холодового урона: 260 K
- Удельная теплоёмкость: 42
Эти значения хорошо сбалансированы и создают уникальный геймплей за IPC:
- Иммунитет к яду даёт преимущество в определённых ситуациях
- Высокая устойчивость к холоду позволяет работать в холодных средах
- Уязвимость к теплу и электричеству создаёт интересные слабости
- Температурные пороги и теплоёмкость реалистично отражают их синтетическую природу
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск других сущностей с похожими коэффициентами урона для сравнения
rg -l "damageModifierSet" | while read -r file; do
if [[ "$file" != "Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml" ]]; then
echo "=== $file ==="
rg "damageModifierSet|coefficients:|Heat:|Cold:|Shock:|Poison:" "$file"
fi
done
Length of output: 94303
Script:
#!/bin/bash
# Search for IPC/silicon damage modifiers
rg -l "silicon_base" | while read -r file; do
echo "=== $file ==="
rg "damageModifierSet|coefficients:|Heat:|Cold:|Shock:|Poison:" "$file"
done
# Also search for any silicon-related damage modifier sets
rg -A10 'type: damageModifierSet.*silicon' Resources/Prototypes/
Length of output: 102
Script:
#!/bin/bash
# Search for damage modifiers in the silicon base file directly
cat Resources/Prototypes/Entities/Mobs/Player/silicon_base.yml
# Also search for any silicon or IPC related damage modifier sets in damage files
rg -A10 "coefficients:" Resources/Prototypes/Damage/modifier_sets.yml
Length of output: 14779
🧰 Tools
🪛 yamllint (1.35.1)
[error] 82-82: too many spaces inside empty braces
(braces)
Resources/Prototypes/Entities/Mobs/Player/slime.yml (1)
4-4
: Изменения корректны!
Структура прототипа соответствует стандартам проекта.
Resources/Prototypes/Entities/Mobs/Player/vox.yml (1)
5-5
: Прототип соответствует требованиям!
Структура и размещение корректны, используется английский язык для названий.
Resources/Prototypes/Entities/Mobs/Player/moth.yml (1)
5-5
: Изменения приняты!
Прототип правильно структурирован и соответствует требованиям проекта.
Resources/Prototypes/Entities/Mobs/Player/dwarf.yml (1)
Line range hint 1-6
: Конфигурация сущности корректна!
Базовая настройка сущности гнома соответствует стандартам проекта.
Resources/Prototypes/Entities/Mobs/Species/human.yml (2)
13-13
: Компонент Carriable добавлен корректно
Добавление системы переноса соответствует целям PR и реализовано последовательно для всех видов существ.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 13-13: too few spaces before comment
(comments)
13-13
: 🧹 Nitpick (assertive)
Исправьте форматирование комментария
Согласно правилам форматирования YAML, перед комментарием должно быть два пробела.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 13-13: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/dwarf.yml (2)
10-10
: 🧹 Nitpick (assertive)
Исправьте форматирование комментария
Согласно правилам форматирования YAML, перед комментарием должно быть два пробела.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 10-10: too few spaces before comment
(comments)
68-69
: Требуется обоснование значения critThreshold
Пожалуйста, объясните выбор значения 115 для порога выносливости гномов. Это значение кажется произвольным и может повлиять на баланс игры.
✅ Verification successful
Based on the search results, I can see the stamina threshold values for different entities in the game. Let me analyze the patterns:
-
Base mobs (regular entities): No explicit critThreshold defined
-
Species:
- Dwarf: 115 (under review)
- Felinid: 85
- Oni: 115
- Golem: 300 (commented out)
-
NPCs:
- Regal Rat: 60
- Xeno: 200-300
- Silicon: 120
- Space Tick: 15
- Flesh: 50
- Elemental: 120
- Carp: 100-150
- Animals: 10
- Space creatures: 150-180
- Mutants: 80
Given this context, I can now provide a final response:
Значение critThreshold 115 для гномов обосновано
Пороговое значение выносливости 115 для гномов соответствует общему балансу игры:
- Оно равно значению других физически крепких рас (например, они)
- Выше чем у обычных гуманоидов (фелиниды: 85)
- Значительно ниже чем у сверхпрочных существ (ксено: 200-300)
- Отражает их коренастую и выносливую природу
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Search for other Stamina configurations to compare threshold values
rg -A 2 "type: Stamina" --type yaml
Length of output: 32220
Resources/Prototypes/Entities/Mobs/Species/reptilian.yml (1)
14-14
: Активация компонента Carriable выглядит корректно!
Раскомментирование компонента Carriable соответствует целям PR по восстановлению системы переноса партнеров.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 14-14: too few spaces before comment
(comments)
Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Shark/Entities/Mobs/Species/Shark.yml (1)
69-69
: Добавление компонента Carriable соответствует требованиям!
Компонент Carriable успешно добавлен, что позволит реализовать функцию переноса. Тег HidesTail уже присутствует, что обеспечивает корректное скрытие хвоста в скафандре.
Resources/Prototypes/Entities/Mobs/Player/ratvar.yml (1)
94-94
: Изменение не влияет на основные цели PR
Добавление новой строки является форматированием и не влияет на функциональность скрытия хвостов или систему переноса.
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml (2)
43-44
: Логичное дополнение для скелетов
Добавление gibContents: Skip
является правильным решением, так как у скелетов нет внутренних органов или других компонентов, которые могли бы выпасть при расчленении.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 43-43: wrong indentation: expected 8 but found 6
(indentation)
12-12
: 🧹 Nitpick (assertive)
Проверьте форматирование комментария
Согласно yamllint, перед комментарием должно быть больше пробелов. Рекомендуется добавить дополнительные пробелы для соответствия стандартам форматирования.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/diona.yml (1)
14-14
: 🧹 Nitpick (assertive)
Проверьте форматирование комментария
Согласно yamllint, перед комментарием должно быть больше пробелов. Рекомендуется добавить дополнительные пробелы для соответствия стандартам форматирования.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 14-14: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/slime.yml (2)
9-9
: 🧹 Nitpick (assertive)
Проверьте форматирование комментария
Согласно yamllint, перед комментарием должно быть больше пробелов. Рекомендуется добавить дополнительные пробелы для соответствия стандартам форматирования.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Carrying system from nyanotrasen.
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 9-9: too few spaces before comment
(comments)
Line range hint 1-1
: Проверьте изменения видимости хвостов
В PR упоминается исправление видимости хвостов при ношении скафандра, но соответствующие изменения не видны в предоставленных файлах. Необходимо проверить, где именно реализованы эти изменения.
✅ Verification successful
Тег HidesTail корректно используется в скафандрах
Проверка показала, что тег HidesTail
правильно применяется как в жёстких (hardsuits), так и в мягких (softsuits) скафандрах. Система одежды (ClothingSystem.cs
) корректно обрабатывает этот тег. Изменения видимости хвостов при ношении скафандра реализованы через существующий механизм тегов и не требуют дополнительных модификаций.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск файлов, связанных с видимостью хвостов и скафандрами
# Поиск упоминаний тегов HidesTail
rg -l "HidesTail"
# Поиск изменений в системе одежды
ast-grep --pattern 'class ClothingSystem {
$$$
HidesTail
$$$
}'
# Поиск прототипов скафандров
fd -e yml -e yaml | xargs rg "type: Spacesuit"
Length of output: 864
Script:
#!/bin/bash
# Проверим содержимое файлов, связанных с системой одежды и тегом HidesTail
# Посмотрим реализацию ClothingSystem
rg -A 5 "HidesTail" Content.Shared/Clothing/EntitySystems/ClothingSystem.cs
# Проверим определение тега HidesTail
cat Resources/Prototypes/tags.yml
# Проверим прототипы скафандров на наличие тега HidesTail
rg -A 3 "HidesTail" Resources/Prototypes/Entities/Clothing/OuterClothing/hardsuits.yml
rg -A 3 "HidesTail" Resources/Prototypes/Entities/Clothing/OuterClothing/softsuits.yml
Length of output: 16397
🧰 Tools
🪛 yamllint (1.35.1)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[warning] 9-9: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Player/ipc.yml (1)
108-110
: Добавление компонентов для системы переноса
Добавление компонентов Carriable
и StatusIcon
соответствует целям PR по реализации системы переноса партнеров. Параметры границ для StatusIcon корректно определены.
Resources/Prototypes/DeltaV/Entities/Mobs/Species/vulpkanin.yml (1)
12-12
: Включение системы переноса для Vulpkanin
Раскомментирование компонента Carriable
включает функционал переноса. Наличие тега HidesTail
в сочетании с этим изменением обеспечивает корректное поведение хвоста при ношении скафандра.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/moth.yml (2)
12-12
: Добавление системы переноса для Moth
Добавление компонента Carriable
обеспечивает единообразие реализации системы переноса для всех видов существ.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 12-12: too few spaces before comment
(comments)
Line range hint 109-110
: Проверьте слои спрайтов для Moth
В определении слоёв спрайтов присутствует слой хвоста (enum.HumanoidVisualLayers.Tail
), хотя мотыльки не имеют хвостов. Рекомендуется проверить необходимость этого слоя.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Player/familiars.yml (1)
31-31
: Изменение порога смерти и добавление компонента Dispellable
Уменьшение порога смерти с 100 до 50 делает фамильяров более уязвимыми. Добавление компонента Dispellable с уроном от развеивания в 100 единиц тепла логично дополняет механику.
Also applies to: 64-67
Resources/Prototypes/Entities/Mobs/Species/harpy.yml (1)
221-221
: Добавление обработчика события для переключения полёта
Корректное добавление обработчика события ToggleFlightEvent для действия переключения полёта.
Resources/Prototypes/Entities/Mobs/Species/arachne.yml (3)
25-25
: Улучшение визуальной кастомизации
Добавлены новые слои для морды, лицевой растительности и боковой части головы. Правильный порядок слоёв обеспечивает корректное отображение серёжек поверх масок.
Also applies to: 69-69, 74-74
🧰 Tools
🪛 yamllint (1.35.1)
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 25-25: too many spaces inside brackets
(brackets)
167-167
: Синхронизация слоёв с основной сущностью
Корректное добавление всех новых визуальных слоёв в сущность-манекен для обеспечения согласованности в редакторе персонажа.
Also applies to: 211-211, 216-216
🧰 Tools
🪛 yamllint (1.35.1)
[error] 167-167: too many spaces inside brackets
(brackets)
[error] 167-167: too many spaces inside brackets
(brackets)
110-110
: Добавление звуков шагов
Добавлен тег FootstepSound для воспроизведения звуков шагов. Учитывая паучью природу существа, убедитесь, что звуки соответствуют множеству ног.
Resources/Prototypes/Entities/Mobs/Player/guardian.yml (1)
107-107
: Добавление тега NoPaint выглядит корректным!
Тег предотвращает возможность перекраски стражей, что логично с точки зрения геймплея.
Resources/Prototypes/Entities/Clothing/OuterClothing/suits.yml (1)
30-30
: Корректное добавление тега HidesTail!
Тег добавлен ко всем костюмам, которые логически должны скрывать хвост персонажа.
Also applies to: 90-90, 127-127, 158-158
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 2
🔭 Outside diff range comments (1)
Resources/Prototypes/Entities/Mobs/Species/arachne.yml (1)
Line range hint
144-223
: Рассмотреть возможность уменьшения дублирования кодаКонфигурация слоев для MobArachneDummy в значительной степени дублирует MobArachneBase. Это увеличивает сложность поддержки, так как изменения необходимо синхронизировать вручную.
Предлагаю рассмотреть возможность выделения общей конфигурации слоев в отдельный прототип или использовать наследование для переиспользования слоев.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 209-209: too many spaces inside brackets
(brackets)
[error] 209-209: too many spaces inside brackets
(brackets)
[error] 210-210: too many spaces inside brackets
(brackets)
[error] 210-210: too many spaces inside brackets
(brackets)
[error] 211-211: too many spaces inside brackets
(brackets)
[error] 211-211: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 213-213: too many spaces inside brackets
(brackets)
[error] 213-213: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[error] 217-217: too many spaces inside brackets
(brackets)
[error] 217-217: too many spaces inside brackets
(brackets)
[warning] 217-217: too few spaces before comment
(comments)
[error] 218-218: too many spaces inside brackets
(brackets)
[error] 218-218: too many spaces inside brackets
(brackets)
[error] 219-219: too many spaces inside brackets
(brackets)
[error] 219-219: too many spaces inside brackets
(brackets)
[error] 220-220: too many spaces inside brackets
(brackets)
[error] 220-220: too many spaces inside brackets
(brackets)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
Plan: Pro
📒 Files selected for processing (3)
Resources/Prototypes/Entities/Mobs/Species/arachne.yml
(6 hunks)Resources/Prototypes/Entities/Mobs/Species/arachnid.yml
(2 hunks)Resources/Prototypes/Entities/Mobs/Species/skeleton.yml
(2 hunks)
🧰 Additional context used
📓 Path-based instructions (3)
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/arachne.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml (1)
Pattern **/*.yml
: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:
, можно оставить оригинальную часть с - id:
.
🪛 yamllint (1.35.1)
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml
[warning] 12-12: too few spaces before comment
(comments)
[error] 43-43: wrong indentation: expected 8 but found 6
(indentation)
Resources/Prototypes/Entities/Mobs/Species/arachne.yml
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 69-69: too many spaces inside brackets
(brackets)
[error] 69-69: too many spaces inside brackets
(brackets)
[error] 70-70: too many spaces inside brackets
(brackets)
[error] 70-70: too many spaces inside brackets
(brackets)
[error] 73-73: too many spaces inside brackets
(brackets)
[error] 73-73: too many spaces inside brackets
(brackets)
[error] 74-74: too many spaces inside brackets
(brackets)
[error] 74-74: too many spaces inside brackets
(brackets)
[warning] 74-74: too few spaces before comment
(comments)
[error] 167-167: too many spaces inside brackets
(brackets)
[error] 167-167: too many spaces inside brackets
(brackets)
[error] 202-202: too many spaces inside brackets
(brackets)
[error] 202-202: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 212-212: too many spaces inside brackets
(brackets)
[error] 213-213: too many spaces inside brackets
(brackets)
[error] 213-213: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[error] 216-216: too many spaces inside brackets
(brackets)
[error] 217-217: too many spaces inside brackets
(brackets)
[error] 217-217: too many spaces inside brackets
(brackets)
[warning] 217-217: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml
[warning] 15-15: too few spaces before comment
(comments)
[error] 133-133: wrong indentation: expected 6 but found 2
(indentation)
[error] 134-134: wrong indentation: expected 6 but found 2
(indentation)
🔇 Additional comments (7)
Resources/Prototypes/Entities/Mobs/Species/skeleton.yml (3)
43-43
: Изменение корректно
Удаление пустых фигурных скобок является правильным, так как они избыточны в данном контексте.
🧰 Tools
🪛 yamllint (1.35.1)
[error] 43-43: wrong indentation: expected 8 but found 6
(indentation)
12-12
: 🧹 Nitpick (assertive)
Уточните источник компонента Carriable
Комментарий упоминает "nyanotrasen", что может вызвать путаницу. Предлагаю либо удалить комментарий, либо добавить более подробное описание функциональности.
- - type: Carriable # Carrying system from nyanotrasen.
+ - type: Carriable # Система переноса других мобов
Likely invalid or redundant comment.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 12-12: too few spaces before comment
(comments)
Line range hint 1-1
: Проверьте связанные изменения для видимости хвоста
В соответствии с целями PR, должны быть изменения, связанные с видимостью хвоста в скафандре, но они отсутствуют в данном файле.
✅ Verification successful
Изменения для видимости хвоста в скафандрах уже реализованы
Анализ показывает, что компонент HidesTail
уже присутствует во всех соответствующих типах скафандров:
- В базовых прототипах внешней одежды (base_clothingouter.yml)
- В жёстких скафандрах (hardsuits.yml)
- В мягких скафандрах (softsuits.yml)
- В обычных костюмах (suits.yml)
Тег HidesTail
определён в tags.yml с комментарием "Protecc the tails!", что подтверждает его предназначение для управления видимостью хвостов.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Search for tail-related changes in other files
# Look for HidesTail component or similar in prototype files
rg -t yaml "HidesTail|tail" Resources/Prototypes/
# Look for spacesuit prototypes
fd -e yml -e yaml . Resources/Prototypes/ --exec rg -l "spacesuit|suit"
Length of output: 65767
Script:
#!/bin/bash
# Search for HidesTail component in spacesuit-related files
rg -l "spacesuit|suit" Resources/Prototypes/ | xargs rg "HidesTail"
# Search for tail-related changes in spacesuit files
rg -l "spacesuit|suit" Resources/Prototypes/ | xargs rg "tail"
Length of output: 12368
🧰 Tools
🪛 yamllint (1.35.1)
[error] 10-10: wrong indentation: expected at least 3
(indentation)
[warning] 12-12: too few spaces before comment
(comments)
Resources/Prototypes/Entities/Mobs/Species/arachne.yml (2)
110-110
: Корректное добавление компонента FootstepSound
Добавление компонента FootstepSound логично для сущности, которая может передвигаться.
Line range hint 78-81
: Проверить интеграцию с системой одежды
Слой Tail корректно настроен со значением visible: false по умолчанию. Однако необходимо убедиться, что ClothingSystem правильно управляет видимостью при надевании скафандра.
Also applies to: 220-223
✅ Verification successful
Интеграция с системой одежды подтверждена
Проверка показала, что ClothingSystem корректно обрабатывает видимость хвоста:
- При надевании предмета с тегом HidesTail хвост скрывается (SetLayerVisibility(..., false))
- При снятии предмета хвост становится видимым (SetLayerVisibility(..., true))
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверяем обработку тега HidesTail в ClothingSystem
rg -l "HidesTail" --type cs
Length of output: 83
Script:
#!/bin/bash
# Проверяем реализацию обработки HidesTail в ClothingSystem
rg "HidesTail" -A 5 -B 5 Content.Shared/Clothing/EntitySystems/ClothingSystem.cs
Length of output: 467
Script:
#!/bin/bash
# Поиск использования тега HidesTail в методах ClothingSystem
ast-grep --pattern 'private void $_($$$) {
$$$
TailTag
$$$
}'
Length of output: 69
Script:
#!/bin/bash
# Поиск использования TailTag в методах ClothingSystem с контекстом
rg "TailTag" -A 10 -B 10 Content.Shared/Clothing/EntitySystems/ClothingSystem.cs
Length of output: 2796
🧰 Tools
🪛 yamllint (1.35.1)
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 25-25: too many spaces inside brackets
(brackets)
[error] 26-26: too many spaces inside brackets
(brackets)
[error] 26-26: too many spaces inside brackets
(brackets)
Resources/Prototypes/Entities/Mobs/Species/arachnid.yml (2)
15-15
: Активация компонента Carriable соответствует целям PR
Раскомментирование компонента Carriable
согласуется с задачей по добавлению системы переноски партнеров.
🧰 Tools
🪛 yamllint (1.35.1)
[warning] 15-15: too few spaces before comment
(comments)
133-134
: Проверьте интеграцию новых компонентов
Добавление компонентов Spider
и IgnoreSpiderWeb
может влиять на поведение существа. Убедитесь, что:
- Компонент
Spider
правильно взаимодействует с существующими системами IgnoreSpiderWeb
корректно предотвращает застревание в паутине
✅ Verification successful
Компоненты Spider и IgnoreSpiderWeb корректно интегрированы
Анализ кодовой базы показывает, что:
- Компонент
Spider
правильно интегрирован с системами и позволяет создавать паутину через действиеSpiderWebAction
- Компонент
IgnoreSpiderWeb
используется в нескольких прототипах существ и корректно работает как маркер для игнорирования эффектов паутины - Оба компонента уже используются в других прототипах существ (arachne.yml, animals.yml), что подтверждает их стабильную работу
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверяем использование компонентов Spider и IgnoreSpiderWeb
ast-grep --pattern 'type: Spider
$$$'
ast-grep --pattern 'type: IgnoreSpiderWeb
$$$'
Length of output: 92
Script:
#!/bin/bash
# Поиск определений компонентов Spider и IgnoreSpiderWeb
rg -l "class Spider.*Component" --type-add 'cs:*.cs'
rg -l "class IgnoreSpiderWeb.*Component" --type-add 'cs:*.cs'
# Поиск использования этих компонентов в системах
rg "Spider.*Component" --type-add 'cs:*.cs' -A 3
rg "IgnoreSpiderWeb.*Component" --type-add 'cs:*.cs' -A 3
# Поиск в YAML файлах
rg "type: Spider" --type yaml
rg "type: IgnoreSpiderWeb" --type yaml
Length of output: 9434
🧰 Tools
🪛 yamllint (1.35.1)
[error] 133-133: wrong indentation: expected 6 but found 2
(indentation)
[error] 134-134: wrong indentation: expected 6 but found 2
(indentation)
Описание обновления:
Короче, пофиксил, теперь хвосты исчезают, когда вульпа или акулка натягивает скафандр.
Вернул систему таскания. Это ж мило!
Медиа
Проверки
Изменения
🆑 Farrellka