Pilot new update #1065
Pilot new update #1065
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the WalkthroughВ данном пулл-реквесте внесены изменения в систему локализации, загрузок экипировки и определений сущностей для роли пилота службы безопасности. Добавлены новые строки перевода для loadout групп (ремни, очки) в версиях en-US и ru-RU. Расширены описания экипировки, включая инструкции по установке турелей и мин, а также обновлены характеристики и стартовое снаряжение для пилотов с дополнительными уровневыми правами. Также обновлены файлы конфигураций loadout’ов, описания работы и определены новые игровые сущности с визуальными ассетами. Changes
Sequence Diagram(s)sequenceDiagram
participant P as Пилот
participant J as Система Ролей
participant L as Менеджер Loadout’ов
participant E as Система Сущностей
P->>J: Инициализация роли пилота
J->>L: Запрос конфигурации loadout’а
L-->>P: Отправка списка loadout групп (EngineerBeret, TechVisor, Hi-Viz и др.)
P->>E: Активация способности (например, установка турели или мины)
E-->>P: Подтверждение установки оборудования
Poem
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 (
|
github-actions
bot
added
Changes: Sprites
|
RSI Diff Bot; head commit 90b620b merging into b0dae5a Resources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi
Resources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi
Resources/Textures/Backmen/Objects/Weapons/temp-turret.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: 3
🧹 Nitpick comments (14)
Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/pilotsetloadouts.ftl (2)
1-2: Исправьте грамматические ошибки в описании.Текущее описание содержит грамматические ошибки.
- .desc = Здесь лежит снаряжение которые вы взяли с собой с базы. Осталось вспомнить, какие именно. + .desc = Здесь лежит снаряжение, которое вы взяли с собой с базы. Осталось вспомнить, какое именно.
4-13: Улучшите форматирование описаний.Для улучшения читаемости рекомендуется использовать более единообразное форматирование предупреждений о дезинтеграции.
-pilot-duffel-category-turret-base-name-description = Получите возможность ставить турели. !Через 60 секунд мануал дезинтегрируется! +pilot-duffel-category-turret-base-name-description = Получите возможность ставить турели. [!] Мануал дезинтегрируется через 60 секунд.Применить аналогичное форматирование ко всем описаниям.
Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl (2)
13-14: Уточните описание светоотражающего жилета.Рекомендуется добавить информацию о конкретном уровне защиты для большей ясности.
- .desc = Жилет повышенной видимости, используемый в опасных зонах. Имеет слой плотной ткани, который даёт небольшую защиту. + .desc = Жилет повышенной видимости, используемый в опасных зонах. Имеет слой плотной ткани, обеспечивающий базовую защиту от механических повреждений.
15-16: Структурируйте описание визора.Текущее описание содержит слишком много информации в одном предложении.
- .desc = Представляет из себя совокупность нескольких типов сканеров с поляризацией. Показывает состояние различных боргов и мехов, статусы ID карт и обеспечивает защиту глаз от сварки. Не имеет защиты от вспышек. + .desc = Многофункциональный визор с поляризованными сканерами. Функции: отображение состояния боргов и мехов, проверка статуса ID-карт, защита глаз при сварке. Внимание: не защищает от световых вспышек.Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/abilitiebooks.ftl (1)
1-15: Добавьте разнообразия в описания книг.Все описания книг идентичны, что может создавать впечатление копирования текста. Рекомендуется добавить уникальные детали для каждого типа турели/мины.
Пример для одной из книг:
- .desc = Это подробное руководство позволяет читателю освоить навыки установки временных турелей. Сгорает за 60 секунд! Поторопитесь и прочтите его, пока технология NT не превратила эту книгу в пыль. + .desc = Техническое руководство по развертыванию лазерных турелей в полевых условиях. Содержит схемы калибровки и настройки лазерных систем. Внимание: самоуничтожение через 60 секунд согласно протоколу безопасности NT.Рекомендуется создать уникальные описания для каждого типа книги, сохраняя предупреждение о времени дезинтеграции.
Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml (2)
51-62: Комментарий по группе SecurityOuterClothingPilot:
Добавление элемента "Hi-Viz" корректно расширило набор внешней одежды. Обратите внимание, что в конце строки 62 обнаружены лишние пробелы – рекомендуется удалить их для соответствия стандартам YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 62-62: trailing spaces
(trailing-spaces)
63-70: Комментарий по группе SecurityClothingBeltPilot:
Новая группа для поясов, включающая "EngiBelt", работает корректно. Просьба удалить лишние пробелы в конце строки 70 для соответствия требованиям YAMLlint.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 70-70: trailing spaces
(trailing-spaces)
Resources/Prototypes/_Backmen/Entities/Clothing/Eyes/glasses.yml (1)
56-82: Новый элемент "ClothingEyesSecEngineerVisors" добавлен корректно.
Новая сущность комбинирует базовые компоненты (ClothingEyesBase) с дополнительными функциональностями (ShowSecurityIcons, BaseSecurityContraband). Описание, спрайты, слои и компоненты (EyeProtection, IdentityBlocker, ShowHealthBars) заданы подробно.
Обратите внимание: рекомендуется проверить отступы и удалить лишние пробелы, чтобы удовлетворять требованиям YAMLlint.Resources/Prototypes/_Backmen/Entities/misc/pilotabilities.yml (5)
1-104: Новые определения временных турелей добавлены.
В этой части определены сущности для турелей: базовая (WeaponTurretSecBase), винтовочная (WeaponTurretSecRifle), для дизаблера (WeaponTurretSecDisabler) и лазерная (WeaponTurretSecLaser). Все сущности используют согласованные параметры (например, fireRate, capacity, lifetime) и корректно ссылаются на ресурсы спрайтов и звуков.
Рекомендуется сверить единообразие параметров между сущностями, если это предусмотрено балансом игры, а также проверить форматирование YAML.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 14-14: wrong indentation: expected 4 but found 6
(indentation)
[error] 23-23: trailing spaces
(trailing-spaces)
[warning] 33-33: wrong indentation: expected 4 but found 6
(indentation)
[warning] 41-41: wrong indentation: expected 4 but found 6
(indentation)
[error] 50-50: trailing spaces
(trailing-spaces)
[warning] 60-60: wrong indentation: expected 4 but found 6
(indentation)
[warning] 68-68: wrong indentation: expected 4 but found 6
(indentation)
[error] 77-77: trailing spaces
(trailing-spaces)
[warning] 87-87: wrong indentation: expected 4 but found 6
(indentation)
[warning] 95-95: wrong indentation: expected 4 but found 6
(indentation)
119-259: Добавлены книги-инструкции по установке турелей и мин.
Набор сущностей TurretInstallationBookBase, TurretInstallationBookRifle, TurretInstallationBookDisabler, TurretInstallationBookLaser и LandMineLayingBook содержит подробные описания, спрайты и параметры для спеллбуков. Параметры learnTime, learnPermanently, а также настройки TimedDespawn и SpawnOnDespawn выглядят последовательными.
Рекомендуется проверить, что все используемые ссылки на спрайты и описания соответствуют требованиям локализации и эстетике игры.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 127-127: wrong indentation: expected 2 but found 4
(indentation)
[warning] 155-155: wrong indentation: expected 2 but found 4
(indentation)
[error] 175-175: trailing spaces
(trailing-spaces)
[warning] 183-183: wrong indentation: expected 2 but found 4
(indentation)
[error] 203-203: trailing spaces
(trailing-spaces)
[warning] 211-211: wrong indentation: expected 2 but found 4
(indentation)
[error] 231-231: trailing spaces
(trailing-spaces)
[warning] 239-239: wrong indentation: expected 2 but found 4
(indentation)
[error] 241-241: trailing spaces
(trailing-spaces)
[error] 259-259: trailing spaces
(trailing-spaces)
336-421: Новые действия для спавна турелей и мин добавлены.
Сущности ActionSpawnTempSecTurretBase, ActionSpawnTempSecTurretRifle, ActionSpawnTempSecTurretDisabler, ActionSpawnTempSecTurretLaser и ActionSpawnSecLandMine определены с использованием WorldTargetAction с соответствующими задержками (useDelay), диапазонами (range) и иконками. Параметр event с WorldSpawnSpellEvent правильно ссылается на соответствующие эффекты.
Рекомендуется сверить согласованность задержек и проверить, что указанные состояния спрайтов соответствуют используемым ресурсам.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 370-370: trailing spaces
(trailing-spaces)
[error] 387-387: trailing spaces
(trailing-spaces)
[error] 404-404: trailing spaces
(trailing-spaces)
[error] 421-421: trailing spaces
(trailing-spaces)
422-509: Настройка снаряжения для пилотов оформлена корректно.
Сущность SecPilotSetLoadouts задаёт основные параметры рюкзака, позволяя выбирать до двух наборов. Определения наборов (thiefBackpackSet для SecTurretBase, SecTurretRifle, SecTurretDisabler, SecTurretLaser и SecLandMine) содержат описания, иконки и ссылки на соответствующие книги-инструкции.
Убедитесь, что все идентификаторы (ID) и ссылки в содержимом корректно совпадают с ранее определёнными сущностями, а также что описания соответствуют документированным требованиям игрового баланса.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 429-429: wrong indentation: expected 2 but found 4
(indentation)
[error] 441-441: trailing spaces
(trailing-spaces)
[error] 459-459: trailing spaces
(trailing-spaces)
[error] 468-468: trailing spaces
(trailing-spaces)
[error] 499-499: trailing spaces
(trailing-spaces)
1-509: Общее замечание по форматированию YAML.
Статический анализ выявил множество предупреждений по неправильным отступам и наличию пробелов в конце строк (trailing spaces). Рекомендуется провести тщательную корректировку форматирования файла, чтобы отступы соответствовали ожиданиям YAML (например, 2 или 4 пробела в зависимости от принятого стиля) и удалить лишние пробелы. Это позволит избежать потенциальных синтаксических ошибок при загрузке конфигураций.🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 14-14: wrong indentation: expected 4 but found 6
(indentation)
[error] 23-23: trailing spaces
(trailing-spaces)
[warning] 33-33: wrong indentation: expected 4 but found 6
(indentation)
[warning] 41-41: wrong indentation: expected 4 but found 6
(indentation)
[error] 50-50: trailing spaces
(trailing-spaces)
[warning] 60-60: wrong indentation: expected 4 but found 6
(indentation)
[warning] 68-68: wrong indentation: expected 4 but found 6
(indentation)
[error] 77-77: trailing spaces
(trailing-spaces)
[warning] 87-87: wrong indentation: expected 4 but found 6
(indentation)
[warning] 95-95: wrong indentation: expected 4 but found 6
(indentation)
[error] 116-116: trailing spaces
(trailing-spaces)
[error] 118-118: trailing spaces
(trailing-spaces)
[warning] 127-127: wrong indentation: expected 2 but found 4
(indentation)
[warning] 155-155: wrong indentation: expected 2 but found 4
(indentation)
[error] 175-175: trailing spaces
(trailing-spaces)
[warning] 183-183: wrong indentation: expected 2 but found 4
(indentation)
[error] 203-203: trailing spaces
(trailing-spaces)
[warning] 211-211: wrong indentation: expected 2 but found 4
(indentation)
[error] 231-231: trailing spaces
(trailing-spaces)
[warning] 239-239: wrong indentation: expected 2 but found 4
(indentation)
[error] 241-241: trailing spaces
(trailing-spaces)
[error] 259-259: trailing spaces
(trailing-spaces)
[error] 275-275: trailing spaces
(trailing-spaces)
[error] 290-290: trailing spaces
(trailing-spaces)
[error] 305-305: trailing spaces
(trailing-spaces)
[error] 370-370: trailing spaces
(trailing-spaces)
[error] 387-387: trailing spaces
(trailing-spaces)
[error] 404-404: trailing spaces
(trailing-spaces)
[error] 421-421: trailing spaces
(trailing-spaces)
[warning] 429-429: wrong indentation: expected 2 but found 4
(indentation)
[error] 441-441: trailing spaces
(trailing-spaces)
[error] 459-459: trailing spaces
(trailing-spaces)
[error] 468-468: trailing spaces
(trailing-spaces)
[error] 499-499: trailing spaces
(trailing-spaces)
Resources/Locale/en-US/_backmen/prototypes/security-pilot/pilotsetloadouts.ftl (1)
4-11: Оптимизация повторяющихся предупреждений.Предупреждение о дезинтеграции повторяется в каждом описании. Рекомендуется создать общий суффикс для всех инструкций:
+manual-disintegration-warning = !After 60 seconds the manual will disintegrate! pilot-duffel-category-turret-base-name = instructions for installing a standard turret -pilot-duffel-category-turret-base-name-description = Gain the ability to place turrets. !After 60 seconds the manual will disintegrate! +pilot-duffel-category-turret-base-name-description = Gain the ability to place turrets. { $manual-disintegration-warning } [остальные описания обновить аналогично]
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (17)
Resources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/equipped-EYES-unshaded.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/equipped-EYES.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/icon-unshaded.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/icon.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/equipped-OUTERCLOTHING-unshaded-vox.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/equipped-OUTERCLOTHING-unshaded.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/equipped-OUTERCLOTHING-vox.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/equipped-OUTERCLOTHING.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/icon-unshaded.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/icon.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/inhand-left.pngis excluded by!**/*.pngResources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/inhand-right.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/temp-turret.rsi/temp-disabler.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/temp-turret.rsi/temp-laser.pngis excluded by!**/*.pngResources/Textures/Backmen/Objects/Weapons/temp-turret.rsi/temp-rifle.pngis excluded by!**/*.png
📒 Files selected for processing (19)
Resources/Locale/en-US/_backmen/preferences/loadout-groups.ftl(1 hunks)Resources/Locale/en-US/_backmen/prototypes/security-pilot/pilotsetloadouts.ftl(1 hunks)Resources/Locale/ru-RU/_backmen/preferences/loadout-groups.ftl(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/abilitiebooks.ftl(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/abilities.ftl(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/clothing.ftl(1 hunks)Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/pilotsetloadouts.ftl(1 hunks)Resources/Locale/ru-RU/corvax/job/job-description.ftl(1 hunks)Resources/Locale/ru-RU/corvax/job/job-names.ftl(1 hunks)Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml(3 hunks)Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml(2 hunks)Resources/Prototypes/Corvax/Loadouts/role_loadouts.yml(1 hunks)Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml(2 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/Eyes/glasses.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Clothing/OuterClothing/vests.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/misc/pilotabilities.yml(1 hunks)Resources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/meta.json(1 hunks)Resources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/meta.json(1 hunks)Resources/Textures/Backmen/Objects/Weapons/temp-turret.rsi/meta.json(1 hunks)
✅ Files skipped from review due to trivial changes (4)
- Resources/Textures/Backmen/Objects/Weapons/temp-turret.rsi/meta.json
- Resources/Textures/Backmen/Clothing/OuterClothing/Vests/hazard-sec-vest.rsi/meta.json
- Resources/Locale/ru-RU/corvax/job/job-names.ftl
- Resources/Textures/Backmen/Clothing/Eyes/Glasses/security-technician-visor.rsi/meta.json
🧰 Additional context used
🪛 YAMLlint (1.35.1)
Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml
[error] 16-16: trailing spaces
(trailing-spaces)
[error] 17-17: trailing spaces
(trailing-spaces)
[error] 55-55: trailing spaces
(trailing-spaces)
Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml
[error] 62-62: trailing spaces
(trailing-spaces)
[error] 70-70: trailing spaces
(trailing-spaces)
Resources/Prototypes/_Backmen/Entities/misc/pilotabilities.yml
[warning] 14-14: wrong indentation: expected 4 but found 6
(indentation)
[error] 23-23: trailing spaces
(trailing-spaces)
[warning] 33-33: wrong indentation: expected 4 but found 6
(indentation)
[warning] 41-41: wrong indentation: expected 4 but found 6
(indentation)
[error] 50-50: trailing spaces
(trailing-spaces)
[warning] 60-60: wrong indentation: expected 4 but found 6
(indentation)
[warning] 68-68: wrong indentation: expected 4 but found 6
(indentation)
[error] 77-77: trailing spaces
(trailing-spaces)
[warning] 87-87: wrong indentation: expected 4 but found 6
(indentation)
[warning] 95-95: wrong indentation: expected 4 but found 6
(indentation)
[error] 116-116: trailing spaces
(trailing-spaces)
[error] 118-118: trailing spaces
(trailing-spaces)
[warning] 127-127: wrong indentation: expected 2 but found 4
(indentation)
[warning] 155-155: wrong indentation: expected 2 but found 4
(indentation)
[error] 175-175: trailing spaces
(trailing-spaces)
[warning] 183-183: wrong indentation: expected 2 but found 4
(indentation)
[error] 203-203: trailing spaces
(trailing-spaces)
[warning] 211-211: wrong indentation: expected 2 but found 4
(indentation)
[error] 231-231: trailing spaces
(trailing-spaces)
[warning] 239-239: wrong indentation: expected 2 but found 4
(indentation)
[error] 241-241: trailing spaces
(trailing-spaces)
[error] 259-259: trailing spaces
(trailing-spaces)
[error] 275-275: trailing spaces
(trailing-spaces)
[error] 290-290: trailing spaces
(trailing-spaces)
[error] 305-305: trailing spaces
(trailing-spaces)
[error] 370-370: trailing spaces
(trailing-spaces)
[error] 387-387: trailing spaces
(trailing-spaces)
[error] 404-404: trailing spaces
(trailing-spaces)
[error] 421-421: trailing spaces
(trailing-spaces)
[warning] 429-429: wrong indentation: expected 2 but found 4
(indentation)
[error] 441-441: trailing spaces
(trailing-spaces)
[error] 459-459: trailing spaces
(trailing-spaces)
[error] 468-468: trailing spaces
(trailing-spaces)
[error] 499-499: trailing spaces
(trailing-spaces)
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (21)
Resources/Locale/ru-RU/corvax/job/job-description.ftl (1)
2-2: Описание роли пилота обновлено корректно.Изменение улучшает описание, добавляя техническую специализацию и делая его более универсальным за счет удаления списка конкретных станций.
Resources/Locale/en-US/_backmen/preferences/loadout-groups.ftl (1)
5-6: Новые группы экипировки добавлены корректно.Добавленные строки соответствуют существующему формату именования и имеют соответствующие переводы в русской локализации.
Resources/Locale/ru-RU/_backmen/preferences/loadout-groups.ftl (1)
5-6: Переводы новых групп экипировки корректны.Добавленные переводы следуют установленному шаблону именования "Пилот СБ, ..." и соответствуют английским строкам.
Resources/Locale/ru-RU/_backmen/prototypes/security-pilot/abilities.ftl (1)
1-8: Описания способностей турелей добавлены корректно.Описания четко и профессионально объясняют функционал каждого типа турели.
Resources/Prototypes/Corvax/Loadouts/role_loadouts.yml (2)
1-15: Комментарий по удалению загрузочного набора JobIAA:
Удаление загрузочного набора JobIAA выглядит корректно. Пожалуйста, убедитесь, что все ссылки на этот набор в других частях проекта также обновлены или удалены.
16-33: Комментарий по изменению загрузочного набора JobPilot:
Изменения в наборе JobPilot отражают обновление: замена элемента "SecurityBelt" на "SecurityClothingBeltPilot" и добавление нового элемента "SecurityEyesPilot" для улучшения поддержки обновлённого оборудования. Проверьте соответствие имен загрузок с локализационными записями.Resources/Prototypes/_Backmen/Entities/Clothing/OuterClothing/vests.yml (1)
1-27: Комментарий по новой сущности ClothingOuterSecurityVestHazard:
Новый объект "hi-viz security vest" определён корректно с использованием компонентов Sprite, Clothing и Armor. Убедитесь, что все пути к ресурсам (rsi-файлы, метаданные) корректны и соответствуют игровым требованиям.Resources/Prototypes/Corvax/Roles/Jobs/Security/pilot.yml (2)
20-27: Комментарий по секции "access":
Обновлённый список прав доступа, включающий добавление "Engineering" и "Atmospherics", выглядит обоснованно. Рекомендуется перепроверить, чтобы данные изменения соответствовали требованиям баланса и логике игры.
32-46: Комментарий по разделу startingGear:
Добавление элемента "SecPilotSetLoadouts" в секцию "storage" корректно расширяет стартовое снаряжение пилота. Убедитесь, что локализационные записи для нового элемента также обновлены.Resources/Prototypes/Corvax/Loadouts/Jobs/Security/pilot.yml (5)
12-16: Комментарий по загрузке SecurityPilotEngineerBeret:
Новая загрузка "SecurityPilotEngineerBeret" для головного оборудования добавлена корректно с указанием элемента "ClothingHeadHatBeretEngineering". Проверьте, что выбранный элемент соответствует общей тематике и дизайну роли.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 16-16: trailing spaces
(trailing-spaces)
17-22: Комментарий по загрузке SecurityPilotTechVisor:
Новая загрузка для "SecurityPilotTechVisor" корректно описана, и элемент "ClothingEyesSecEngineerVisors" соответствует назначению. Рекомендуется проверить, что она интегрирована с системой визуального отображения в игре.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 17-17: trailing spaces
(trailing-spaces)
23-27: Комментарий по загрузке SecurityGlasses:
Загрузка "SecurityGlasses" для защитных очков добавлена корректно. Убедитесь, что визуальные эффекты и параметры элемента "ClothingEyesGlassesSecurity" соответствуют ожиданиям для данной роли.
40-44: Комментарий по загрузке Hi-Viz:
Добавление загрузки "Hi-Viz" для внешней одежды, ссылающейся на "ClothingOuterSecurityVestHazard", реализовано верно. Проверьте, что дизайн жилета соответствует описанию и требованиям безопасности.
56-61: Комментарий по загрузке EngiBelt:
Новая загрузка "EngiBelt" для пояса с использованием "ClothingBeltUtilityEngineering" выглядит корректно. Убедитесь, что название и функциональность элемента согласованы с общей системой загрузок.Resources/Prototypes/Corvax/Loadouts/loadout_groups.yml (5)
1-7: Комментарий по удалению группы Inventory:
Группа "Inventory" удалена корректно. Убедитесь, что связанные с ней ссылки и функциональность в остальных частях проекта также обновлены.
16-25: Комментарий по группе SecurityJumpsuitPilot:
Группа загрузок "SecurityJumpsuitPilot" оформлена корректно. Нет замечаний по изменению данного блока.
27-36: Комментарий по группе SecurityShoesPilot:
Группа загрузок для обуви представлена корректно. Проверьте, что все элементы доступны в других конфигурациях и соответствуют общей стилистике.
37-50: Комментарий по группе SecurityHeadPilot:
Добавление загрузки "SecurityPilotEngineerBeret" расширяет выбор головного оборудования. Убедитесь, что порядок элементов отвечает ожиданиям и отражён в локализационных файлах.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 50-50: trailing spaces
(trailing-spaces)
71-77: Комментарий по группе SecurityEyesPilot:
Группа загрузок для очков включает новые элементы "SecurityPilotTechVisor" и "SecurityGlasses". Убедитесь, что обновлённые локализационные записи корректно отражают изменения.Resources/Prototypes/_Backmen/Entities/misc/pilotabilities.yml (2)
105-118: Новая сущность мины "LandMineWeakExplosive" добавлена корректно.
Параметры взрыва, такие как explosionType, maxIntensity, intensitySlope и totalIntensity, а также флаг canCreateVacuum – заданы в соответствии с игровыми механиками. При необходимости можно добавить комментарии для пояснения значений, если потребуется дальнейшая настройка баланса.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 116-116: trailing spaces
(trailing-spaces)
[error] 118-118: trailing spaces
(trailing-spaces)
261-335: Добавлены технические эффекты для спавна турелей и мин.
Эффекты (EffectRCDConstructSecTurretBase, EffectRCDConstructSecTurretRifle, EffectRCDConstructSecTurretDisabler, EffectRCDConstructSecTurretLaser, EffectRCDConstructSecLandMine) реализованы с использованием Sprite, TimedDespawn, SpawnOnDespawn и EmitSoundOnSpawn. Все компоненты настроены для быстрого визуального и звукового отклика.
Убедитесь, что значения lifetime согласованы с игровым процессом и что пути к звукам и спрайтам корректны.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 275-275: trailing spaces
(trailing-spaces)
[error] 290-290: trailing spaces
(trailing-spaces)
[error] 305-305: trailing spaces
(trailing-spaces)
| ent-ActionSpawnSecLandMine = заложить мину | ||
| .desc = ЗАЛОЖИТЕ МИНУ ЗА ИМПЕРАТОРА!!! |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Требуется корректировка стиля описания мины.
Текущее описание использует неформальный стиль с заглавными буквами и чрезмерным количеством восклицательных знаков, что не соответствует стилю других описаний.
Предлагаемое изменение:
ent-ActionSpawnSecLandMine = заложить мину
-.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.
| ent-ActionSpawnSecLandMine = заложить мину | |
| .desc = ЗАЛОЖИТЕ МИНУ ЗА ИМПЕРАТОРА!!! | |
| ent-ActionSpawnSecLandMine = заложить мину | |
| .desc = Устанавливает взрывное устройство, срабатывающее при наступании. |
| pilot-duffel-category-land-mine-name = instruction for laying mines | ||
| pilot-duffel-category-land-mine-name-description = Gain the ability to lay mines. !The manual will disintegrate after 60 seconds! |
There was a problem hiding this comment.
Исправление несоответствий в описании мин.
Обнаружены следующие проблемы:
- Опечатка в названии: "instruction" вместо "instructions"
- Формулировка предупреждения отличается от других описаний
-pilot-duffel-category-land-mine-name = instruction for laying mines
+pilot-duffel-category-land-mine-name = instructions for laying mines
-pilot-duffel-category-land-mine-name-description = Gain the ability to lay mines. !The manual will disintegrate after 60 seconds!
+pilot-duffel-category-land-mine-name-description = Gain the ability to lay mines. !After 60 seconds the manual will disintegrate!📝 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.
| pilot-duffel-category-land-mine-name = instruction for laying mines | |
| pilot-duffel-category-land-mine-name-description = Gain the ability to lay mines. !The manual will disintegrate after 60 seconds! | |
| pilot-duffel-category-land-mine-name = instructions for laying mines | |
| pilot-duffel-category-land-mine-name-description = Gain the ability to lay mines. !After 60 seconds the manual will disintegrate! |
| ent-SecPilotSetLoadouts = duffel bag with technical manuals | ||
| .desc = Here is the equipment that you took with you from the base. It remains to remember which ones. |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Требуется улучшение описания сумки.
Текущее описание неясное и грамматически некорректное. Предлагаю следующие изменения:
- .desc = Here is the equipment that you took with you from the base. It remains to remember which ones.
+ .desc = A duffel bag containing technical manuals for security equipment from the base.📝 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.
| ent-SecPilotSetLoadouts = duffel bag with technical manuals | |
| .desc = Here is the equipment that you took with you from the base. It remains to remember which ones. | |
| ent-SecPilotSetLoadouts = duffel bag with technical manuals | |
| .desc = A duffel bag containing technical manuals for security equipment from the base. |
|
Как-то странно давать пилоту СБ просто актионы чтобы ставить турели, может их заменить на реальные размещаемые турели и мины? |
Ну, там не простые турели, а конкретно специальные временные турели на (30 секунд) с пониженной скоростью стрельбы (2 вместо 6). Плюсом я считаю если турели будут временными, это будет более балансно чем постоянные турели. Там можно КД у акшенов больше сделать для баланса. |
Это какой-то бред с точки зрения станционной логики, это ещё допустимо для какой-нибудь миниигры но никак не нормального РП процесса |
У тебя РСУ также работает если что. Буквально из нихуя создаёт структуры. Какая логика, о чём ты вообще? |
РСУ не из воздуха, у нее есть заряды. |
|
Убедили, сделаю по нормальному |
Описание PR
Пилоту были немного изменены лодауты, добавлено пару новых элементов одежды. Теперь пилот (техник СБ) это мультикласс, такой же как и бригмедик. Техник СБ теперь может выбрать две способности на выбор при спавне. Среди способностей есть установка турелей четырёх типов и установка мин.
Тип PR
🆑
Summary by CodeRabbit