[Fix] centcom door access fix #957
Conversation
WalkthroughВ данном пулл-реквесте внесены изменения в локализацию и конфигурацию доступа для различных сущностей, связанных с дверями и шлюзами. Добавлены новые записи и компоненты, такие как Changes
Poem
Warning Rate limit exceeded@KayzelW has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 5 minutes and 16 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 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
size/XL
Changes: Map
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Outside diff range and nitpick comments (1)
Resources/Prototypes/Entities/Structures/Doors/Airlocks/airlocks.yml (1)
Line range hint
1-308: Рекомендации по архитектуре системы контроля доступаРассмотрите создание отдельной категории дверей высокой безопасности с предустановленными параметрами доступа и уровнем защиты. Это упростит поддержку и обеспечит единообразие настроек безопасности для всех дверей ЦК и других критически важных зон.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/structures/doors/airlocks/access.ftl(1 hunks)Resources/Prototypes/Entities/Structures/Doors/Airlocks/access.yml(3 hunks)Resources/Prototypes/Entities/Structures/Doors/Airlocks/airlocks.yml(1 hunks)Resources/Prototypes/_Backmen/Entities/Structures/Doors/Airlocks/access.yml(11 hunks)
🔇 Additional comments (7)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/structures/doors/airlocks/access.ftl (2)
28-30: Локализация соответствует стандартам!
Новая локализация для двойного шлюза ЦК следует установленным соглашениям об именовании и форматировании. Суффикс и описание согласованы с другими записями ЦентКом.
31-31: Улучшена структурная организация файла
Добавленные пустые строки улучшают читаемость файла, логически разделяя различные секции локализации (ЦентКом, Синдикат, МТФ).
Also applies to: 41-41
Resources/Prototypes/_Backmen/Entities/Structures/Doors/Airlocks/access.yml (4)
165-166: Корректная конфигурация для окон-дверей
Права доступа соответствуют конфигурации дверей офицерского уровня.
178-179: Корректная конфигурация люков
Базовый уровень доступа CentralCommand соответствует требованиям безопасности для люков.
212-213: Корректная конфигурация двойных шлюзов
Базовый уровень доступа CentralCommand соответствует требованиям безопасности.
8-9: Проверьте иерархию прав доступа
Текущая конфигурация прав доступа для дверей ЦК выглядит логично, но рекомендуется проверить следующие моменты:
- Двери оружейной требуют доступ CentralCommand + CCOfficer + CCSecurity, что может быть избыточным
- Двери адмирала требуют только CentralCommand + CCAdmiral, возможно стоит добавить CCOfficer для административной иерархии
Also applies to: 30-31, 52-53, 74-75
✅ Verification successful
Based on the search results, I can now generate a final response since I have enough information about the access hierarchy implementation across the codebase. The results show that:
- The access levels are properly defined in
access_levels.ymlwith a clear hierarchy - Different job roles (operator, official, admiral) have appropriate access level assignments
- The door access configurations align with the security requirements across different areas
- The electronic door components are properly configured with matching access levels
Иерархия прав доступа реализована корректно
Текущая конфигурация дверей полностью соответствует иерархии должностей и уровней доступа в ЦК:
- Комбинация CentralCommand + CCOfficer + CCSecurity для оружейной оправдана, так как это обеспечивает доступ только для сотрудников службы безопасности высокого ранга
- Отсутствие CCOfficer в требованиях для дверей адмирала логично, так как CCAdmiral является высшим уровнем доступа, что подтверждается конфигурацией в
access_levels.yml
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка использования прав доступа в других файлах
rg "CCAdmiral|CCOfficer|CCSecurity" --type yaml
Length of output: 14554
Resources/Prototypes/Entities/Structures/Doors/Airlocks/access.yml (1)
293-294: Корректная реализация контроля доступа!
Изменения правильно ограничивают доступ к дверям Центрального Командования через компонент AccessReader с уровнем доступа "CentralCommand". Реализация последовательна для всех типов дверей (стандартных, стеклянных и высокозащищенных).
Давайте проверим валидность уровня доступа "CentralCommand" и убедимся, что все двери ЦК учтены:
Also applies to: 679-680, 1234-1235
✅ Verification successful
Подтверждена корректность контроля доступа к дверям ЦК
Проверка показала:
- Уровень доступа "CentralCommand" корректно определен в системе (centcomm.yml)
- Изменения охватывают все типы дверей ЦК:
- Стандартные шлюзы
- Стеклянные шлюзы
- Бронированные шлюзы
- Виндоры (включая плазменные и урановые варианты)
- Все двери используют правильную электронику (DoorElectronicsCentralCommand)
- Наследование от базовых сущностей и спрайты настроены корректно
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка определения уровня доступа CentralCommand
rg -l "CentralCommand" Resources/Prototypes/Access/
# Поиск других дверей ЦК, которые могли быть пропущены
rg -l "Central.*Command.*Door|Centcom.*Door" Resources/Prototypes/
Length of output: 161
Script:
#!/bin/bash
# Check the content of the access definition file
cat Resources/Prototypes/Access/centcomm.yml
# Search for all CentralCommand doors in the codebase
rg -l "CentralCommand" Resources/Prototypes/Entities/Structures/Doors/
# Check if there are any other door entities with Central Command in their names or descriptions
rg "Central.*Command.*door" -i Resources/Prototypes/
Length of output: 531
Script:
#!/bin/bash
# Проверим содержимое файлов, где упоминается CentralCommand
rg "CentralCommand" -A 5 Resources/Prototypes/Entities/Structures/Doors/Windoors/windoor.yml
rg "CentralCommand" -A 5 Resources/Prototypes/Entities/Structures/Doors/Airlocks/airlocks.yml
rg "CentralCommand" -A 5 Resources/Prototypes/Entities/Structures/Doors/Airlocks/assembly.yml
# Поищем другие возможные упоминания дверей ЦК с другими названиями
rg "Centcom.*door|Command.*door" -i Resources/Prototypes/
Length of output: 2511
| - type: AccessReader | ||
| access: [ [ "CentralCommand", "CCCargo"] ] |
There was a problem hiding this comment.
Несогласованность в правах доступа шлюзов
Обнаружена несогласованность в конфигурации прав доступа:
- Шлюзы инженерного и медицинского отделов требуют только CentralCommand
- Шлюзы карго и службы безопасности требуют дополнительные права доступа
Предлагаемые изменения для согласованности:
- access: [ [ "CentralCommand" ] ]
+ access: [ [ "CentralCommand", "CCEngineering" ] ]- access: [ [ "CentralCommand" ] ]
+ access: [ [ "CentralCommand", "CCMedical" ] ]Also applies to: 109-110, 123-124, 135-136
There was a problem hiding this comment.
таких ролей пока нет на станции ЦК(CentralCommand)
There was a problem hiding this comment.
@KayzelW Понятно, спасибо за уточнение. В таком случае текущая конфигурация прав доступа подходит.
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Outside diff range and nitpick comments (2)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/devices/electronics/door_access.ftl (1)
10-12: Изменения корректны, но есть предложение по улучшению согласованностиНовая запись для электроники дверей оружейной ЦентКома следует установленному формату и грамматически корректна. Однако для лучшей согласованности с другими записями ЦентКома предлагаю небольшое изменение в суффиксе.
ent-DoorElectronicsCCArmory = { ent-DoorElectronics } - .suffix = ЦентКом, Оружейная, Закрыт + .suffix = ЦентКом, Оружейный, Закрыт .desc = { ent-DoorElectronics.desc }Это сделает запись более согласованной с другими записями, где используются прилагательные мужского рода (например, "Командный", "Инженерный", "Медицинский").
Resources/Prototypes/_Backmen/Entities/Objects/Devices/Electronics/door_access.yml (1)
17-23: Необходимо уточнить суффикс для оружейной ЦКМногоуровневая система доступа реализована правильно, но суффикс
suffix: CentCom, Security, Lockedне отражает, что это оружейная.Предлагаю изменить суффикс:
- suffix: CentCom, Security, Locked + suffix: CentCom, Armory, Locked
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
📒 Files selected for processing (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/devices/electronics/door_access.ftl(1 hunks)Resources/Prototypes/Entities/Structures/Doors/Airlocks/airlocks.yml(2 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Devices/Electronics/door_access.yml(2 hunks)Resources/Prototypes/_Backmen/Entities/Structures/Doors/Airlocks/access.yml(2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
- Resources/Prototypes/_Backmen/Entities/Structures/Doors/Airlocks/access.yml
🔇 Additional comments (7)
Resources/Prototypes/_Backmen/Entities/Objects/Devices/Electronics/door_access.yml (5)
7-7: Корректное обновление прав доступа для грузового отсека ЦК
Добавление "CentralCommand" в список доступа логично структурирует иерархию доступа к грузовому отсеку Центрального Командования.
15-15: Правильное обновление прав доступа для охранного отсека ЦК
Изменение соответствует общей структуре обновления прав доступа для Центрального Командования.
39-39: Корректное обновление прав доступа для офицерского отсека ЦК
Добавление "CentralCommand" поддерживает общую иерархию доступа.
47-47: Корректное обновление прав доступа для адмиральского отсека ЦК
Изменение правильно реализует высший уровень безопасности, требуя как доступ ЦК, так и адмиральский доступ.
Line range hint 7-47: Необходима проверка совместимости обновлённой системы доступа
Убедитесь, что добавление "CentralCommand" не нарушает существующие шаблоны доступа в других частях кодовой базы.
✅ Verification successful
Based on the search results, I can generate the final response as we have comprehensive information about the access system across the codebase. The search results show:
- All files containing CentralCommand-related access references
- All access patterns in the codebase
- The hierarchy and usage of CentralCommand access levels
Final Response:
Изменения в системе доступа ЦК безопасны для использования
Анализ кодовой базы показывает, что:
- Все новые уровни доступа ЦК ("CCCargo", "CCSecurity", "CCOfficer", "CCAdmiral") корректно интегрированы в существующую иерархию
- Базовый уровень "CentralCommand" используется как основа для всех дверей ЦК
- Нет конфликтующих паттернов доступа в других частях кодовой базы
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск всех файлов, содержащих ссылки на двери ЦК
rg -l "CentralCommand|CCCargo|CCSecurity|CCOfficer|CCAdmiral" Resources/Prototypes/
# Поиск потенциальных конфликтов в системе доступа
ast-grep --pattern 'access: [[$$$]]' Resources/Prototypes/
Length of output: 28053
Resources/Prototypes/Entities/Structures/Doors/Airlocks/airlocks.yml (2)
150-151: Синхронизация контроля доступа успешно реализована
Отлично, что теперь обе двери ЦК (обычная и стеклянная) имеют одинаковые настройки контроля доступа через компонент AccessReader. Это исправляет предыдущую проблему с несогласованностью доступа.
Давайте проверим корректность реализации тега доступа:
Also applies to: 309-310
✅ Verification successful
Тег доступа CentralCommand корректно определен и используется
Проверка показала, что:
- Тег доступа "CentralCommand" определен в
Resources/Prototypes/Access/centcomm.yml - Тег корректно используется в других компонентах системы контроля доступа
- Обе двери ЦК (обычная и стеклянная) используют одинаковую конфигурацию доступа
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка определения и использования тега доступа CentralCommand
# Поиск определения тега доступа
echo "Проверка определения тега доступа:"
rg "CentralCommand" Resources/Prototypes/Access/
# Поиск других сущностей, использующих этот тег
echo -e "\nПроиск других сущностей с тем же тегом доступа:"
ast-grep --pattern 'components:
- type: AccessReader
access: [ [ "CentralCommand" ] ]'
Length of output: 33640
149-153: Проверьте унаследованные параметры безопасности
Двери ЦК наследуют средний уровень безопасности панели проводки (securityLevel: medSecurity) от дверей командования. Убедитесь, что этот уровень безопасности соответствует требованиям для дверей Центрального Командования.
Рекомендуется проверить, не требуется ли повышенный уровень безопасности:
Also applies to: 308-312
✅ Verification successful
Let me gather more information about security levels in airlocks.
Let me try one more search to find the inheritance relationships with a corrected command.
Средний уровень безопасности для дверей ЦК является обоснованным
Анализ кодовой базы показывает, что:
- Двери Центрального Командования наследуют средний уровень безопасности (
medSecurity) от дверей командования, что соответствует общей архитектуре системы - Максимальный уровень безопасности (
maxSecurity) используется только для специальных дверей повышенной безопасности в отдельном файле highsec.yml - Все остальные двери командования, включая стеклянные варианты, также используют средний уровень безопасности
Текущая реализация является корректной и соответствует общей иерархии безопасности в игре.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Проверка уровней безопасности других дверей высокого уровня доступа
echo "Поиск дверей с повышенным уровнем безопасности:"
ast-grep --pattern 'components:
- type: WiresPanelSecurity
securityLevel: highSecurity'
Length of output: 200
Script:
#!/bin/bash
# Поиск всех уровней безопасности в файлах дверей
echo "Поиск всех уровней безопасности в дверях:"
rg "securityLevel:" Resources/Prototypes/Entities/Structures/Doors/ -B2 -A1
# Поиск родительских сущностей для дверей ЦК
echo -e "\nПоиск определений дверей ЦК и их родителей:"
rg "^- type: entity\n.*id: (AirlockCentralCommand|AirlockCommand)" Resources/Prototypes/Entities/Structures/Doors/ -A 3
Length of output: 2938
Script:
#!/bin/bash
# Поиск определений дверей ЦК и их родителей с исправленным синтаксисом
echo "Поиск дверей ЦК и командования:"
rg "id: (AirlockCentralCommand|AirlockCommand)" Resources/Prototypes/Entities/Structures/Doors/ -B1 -A2
# Поиск максимального уровня безопасности
echo -e "\nПроверка дверей с максимальным уровнем безопасности:"
rg "securityLevel: maxSecurity" Resources/Prototypes/Entities/Structures/Doors/ -B3
Length of output: 3828
🆑
Summary by CodeRabbit
Новые функции
ent-DoubleGlassAirlockCentcomLockedв русской локализации для двойного стеклянного шлюза.AccessReaderдля управления доступом к сущностямAirlockCentralCommandиAirlockCentralCommandGlass.DoorElectronicsCCArmory.Исправления ошибок
ent-DoubleGlassAirlockCentcomLocked, что указывает на исправление или обновление локализации.