[Tweak] Security Lockers Reshuffle #188
Conversation
WalkthroughИзменения затрагивают несколько файлов конфигурации, связанных с инвентарем и сущностями безопасности. Модификации включают обновление содержимого ящиков, шкафчиков и торговых автоматов, добавление новых предметов экипировки и удаление некоторых существующих. Основные изменения коснулись инвентаря службы безопасности, включая обновление униформ, брони и аксессуаров. Changes
Suggested reviewers
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 (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
Resources/Prototypes/Catalog/VendingMachines/Inventories/secdrobe.yml (1)
8-27: Рекомендуется унифицировать формат комментариев!В файле присутствуют комментарии разного формата:
# WWDP - ...# DeltaV - ...Предлагаю привести все комментарии к единому стилю для улучшения читаемости кода.
Resources/Prototypes/Catalog/Fills/Lockers/security.yml (3)
71-73: Возврат к базовой броне требует корректировки тактикиЗамена усиленной брони (plate carrier) на базовую может потребовать изменения тактики службы безопасности. Рекомендуется:
- Обновить инструкции для сотрудников СБ
- Рассмотреть добавление дополнительного защитного снаряжения
Line range hint
84-124: Требуется обновление документации для роли КорпсменаСущественное изменение роли Бригмедика (теперь Корпсмен) и разделение снаряжения между двумя шкафчиками требует:
- Обновления вики-страниц
- Создания руководства по выбору снаряжения
- Уточнения протоколов взаимодействия с медицинским отделом
Могу помочь с созданием шаблона документации. Нужно открыть issue для отслеживания этой задачи?
Line range hint
156-157: Проверьте вероятность появления HUD для детективаВероятность 0.3 для ClothingEyesHudSecurity кажется низкой для важного элемента экипировки детектива. Рекомендуется пересмотреть это значение.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
Resources/Prototypes/Catalog/Fills/Boxes/security.yml(1 hunks)Resources/Prototypes/Catalog/Fills/Lockers/security.yml(1 hunks)Resources/Prototypes/Catalog/VendingMachines/Inventories/sec.yml(1 hunks)Resources/Prototypes/Catalog/VendingMachines/Inventories/secdrobe.yml(1 hunks)
🔇 Additional comments (4)
Resources/Prototypes/Catalog/VendingMachines/Inventories/sec.yml (1)
14-16: Проверьте баланс количества нового оборудования!
Добавление голопроектора и защитных очков соответствует целям реорганизации, но стоит проверить, не является ли количество (5 голопроекторов и 2 пары очков) избыточным для текущего баланса игры.
✅ Verification successful
Let me gather information about similar vending machine inventories to verify the balance of quantities.
Based on the vending machine inventory data, I can now make a determination about the balance of quantities. Let me generate the final response in Russian:
Количество оборудования соответствует стандартам!
Анализ других торговых автоматов показывает, что количество 5 голопроекторов и 2 пар очков соответствует стандартным количествам аналогичного оборудования в других автоматах SecTech. Например:
- Стандартное количество защитного снаряжения (бронежилеты, шлемы) - 2-3 единицы
- Стандартное количество электронных устройств (рации, анализаторы) - 4-5 единиц
- Стандартное количество очков разных типов - 2 единицы
🏁 Scripts executed
The following scripts were executed for the analysis:
Error: No shell scripts found
Script:
#!/bin/bash
# Search for vending machine inventory files and their content
fd ".*\.yml$" Resources/Prototypes/Catalog/VendingMachines/Inventories/ --exec cat {}
Length of output: 39709
Resources/Prototypes/Catalog/VendingMachines/Inventories/secdrobe.yml (1)
20-23: Проверьте баланс брони!
Перемещение всех типов брони в SecDrobe может сделать их слишком доступными. Рекомендуется проверить:
- Не нарушает ли это игровой баланс
- Соответствует ли это целям реорганизации СБ
Resources/Prototypes/Catalog/Fills/Boxes/security.yml (1)
43-44: Подтвердите функциональность новых очков!
Замена ClothingEyesHudSecurity на ClothingEyesGlassesSecurity должна сохранять все необходимые функции HUD для сотрудников СБ. Пожалуйста, подтвердите, что:
- Новые очки имеют все необходимые компоненты
- Функциональность HUD работает корректно
Resources/Prototypes/Catalog/Fills/Lockers/security.yml (1)
Line range hint 13-13: Проверьте баланс боевого скафандра начальника службы безопасности
Замена стандартного скафандра на боевой вариант существенно усиливает защиту. Необходимо убедиться, что это не нарушает игровой баланс.
✅ Verification successful
Замена скафандра начальника СБ обоснована
Анализ показывает, что боевой скафандр имеет сбалансированные характеристики:
- Улучшенная защита от взрывов (коэффициент 0.40)
- Значительный штраф к скорости передвижения (модификатор спринта 0.65)
- Специализация для контроля толпы и противостояния вооруженным противникам
- Сохраняет функционал защиты от давления
Компромисс между усиленной защитой и сниженной мобильностью соответствует роли начальника службы безопасности.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск характеристик брони в файлах прототипов
rg -A 5 "ClothingOuterHardsuitCombatWarden|ClothingOuterHardsuitWarden" "Resources/Prototypes"
Length of output: 3478
Script:
#!/bin/bash
# Поиск компонентов брони для обоих скафандров
ast-grep --pattern 'components:
- type: Armor
$$$'
Length of output: 58
Script:
#!/bin/bash
# Поиск всех упоминаний боевого скафандра начальника СБ
rg -B 5 -A 10 "ClothingOuterHardsuitCombatWarden" --type yaml
Length of output: 4910
Script:
#!/bin/bash
# Поиск родительского класса боевого скафандра и его характеристик
rg -B 5 -A 15 "ClothingOuterHardsuitCombatRiot" --type yaml
Length of output: 7203
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
Resources/Prototypes/Catalog/Fills/Lockers/security.yml (1)
Line range hint
125-164: Добавление специализированного шкафчика с защитным костюмомНовый вариант шкафчика включает:
- Боевой защитный костюм медика
- Полный набор медицинского оборудования
- Оборудование для отслеживания
- GPS для координации действий
Рекомендуется добавить комментарий в документацию о различиях между обычным и специализированным шкафчиками.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (4)
Resources/Prototypes/Catalog/Fills/Boxes/security.yml(1 hunks)Resources/Prototypes/Catalog/Fills/Lockers/security.yml(1 hunks)Resources/Prototypes/Catalog/VendingMachines/Inventories/sec.yml(2 hunks)Resources/Prototypes/Catalog/VendingMachines/Inventories/secdrobe.yml(1 hunks)
🚧 Files skipped from review as they are similar to previous changes (3)
- Resources/Prototypes/Catalog/Fills/Boxes/security.yml
- Resources/Prototypes/Catalog/VendingMachines/Inventories/sec.yml
- Resources/Prototypes/Catalog/VendingMachines/Inventories/secdrobe.yml
🔇 Additional comments (3)
Resources/Prototypes/Catalog/Fills/Lockers/security.yml (3)
68-77: Оптимизация содержимого шкафчика службы безопасности
Изменения улучшают стандартизацию экипировки:
- Замена случайной одежды на стандартную униформу
- Возврат к базовой броне вместо plate carrier
- Добавление табельного оружия (disabler)
- Добавление защитных очков
Рекомендуется проверить баланс с учетом изменения типа брони.
Line range hint 83-123: Реорганизация шкафчика медика службы безопасности
Произведена оптимизация снаряжения:
- Добавлен GPS для отслеживания имплантов
- Удалены избыточные предметы
- Обновлены названия в соответствии с новой терминологией (corpsman)
Обратите внимание на закомментированные предметы (##):
ClothingOuterCoatAMGпомечен для будущего обновления спрайтаClothingUniformJumpskirtOfLifeудален
Line range hint 168-187: Корректировка вероятности появления HUD
Изменение вероятности появления ClothingEyesHudSecurity до 0.3 выглядит обоснованным для баланса экипировки детектива.
░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░░ |
Описание PR
Изменено наполнение шкафов офицеров СБ:
Коробка очков СБ теперь содержит очки СБ.
Изменения
🆑 vanx