height changes and requirments #329
Conversation
github-actions
bot
added
Статус: Требуется одобрение
WalkthroughВ данном изменении добавлены новые требования к высоте для различных ролей в играх, а также обновлены атрибуты для нескольких видов существ. В частности, для ролей "Бригмедик", "Охранник", "Детектив", "Глава охраны", "Кадет безопасности" и других введено требование минимальной высоты 160 см. Также обновлены параметры высоты и ширины для видов "Вулканины", "Человека", "Мотылька", "Рептилии", "Слизня" и "Тажара". Изменения не затрагивают другие атрибуты, такие как снаряжение или доступ. Changes
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
🧹 Outside diff range comments (4)
Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml (1)
Обнаружена проблема с локализацией: английский файл содержит русский текст
В файле локализации
Resources/Locale/en-US/_LostParadise/Jobs/sec-pilot.ftlобнаружен русский текст вместо английского перевода. Необходимо исправить следующие строки:
job-name-secpilotдолжен содержать английский текст (например, "Security Pilot")job-description-secpilotдолжен содержать английский текст (например, "Patrol around the station in search of space law violators")Русский перевод корректно размещен в файле
ru-RU, но английский файл требует исправления.🔗 Analysis chain
Line range hint
1-40: Проверьте наличие русских переводов в прототипеСогласно правилам кодирования, прототипы должны содержать только английский текст или ссылки на переводы. Пожалуйста, убедитесь, что все текстовые значения соответствуют этому требованию.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check for potential Russian text in the prototype rg -i '[а-яё]' 'Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml'Length of output: 85
Script:
#!/bin/bash # Check for localization strings in the prototype file cat Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml # Search for potential localization files related to SecPilot fd -e ftl . | xargs rg -l "job-name-secpilot|job-description-secpilot"Length of output: 1282
Script:
#!/bin/bash # Check content of both localization files echo "=== English localization ===" cat Resources/Locale/en-US/_LostParadise/Jobs/sec-pilot.ftl echo -e "\n=== Russian localization ===" cat Resources/Locale/ru-RU/_LostParadise/Jobs/sec-pilot.ftlLength of output: 522
🧰 Tools
🪛 yamllint
[warning] 11-11: too few spaces before comment
(comments)
Resources/Prototypes/Roles/Jobs/Security/security_officer.yml (1)
Line range hint
1-42: Обнаружен комментарий на русском языкеВ файле присутствует комментарий на русском языке, что противоречит правилам кодирования. Согласно руководству, все прототипы должны быть только на английском языке.
Предлагаемое исправление:
- # DeltaV - ClothingOuterArmorBasic replaced in favour of plate carrier + # DeltaV - ClothingOuterArmorBasic replaced with plate carrier for better protection🧰 Tools
🪛 yamllint
[warning] 11-11: too few spaces before comment
(comments)
Resources/Prototypes/Roles/Jobs/Security/security_cadet.yml (1)
Line range hint
1-50: Обнаружены русские переводы в прототипеСогласно руководству по написанию кода, прототипы должны содержать только английский текст или ссылки на переводы. Необходимо удалить прямые русские переводы из файла.
Рекомендуется использовать ссылки на переводы вместо прямого включения русского текста. Например:
- name: "Кадет службы безопасности" + name: job-name-cadet🧰 Tools
🪛 yamllint
[warning] 10-10: too few spaces before comment
(comments)
[warning] 11-11: comment not indented like content
(comments-indentation)
Resources/Prototypes/Roles/Jobs/Security/senior_officer.yml (1)
Line range hint
47-47: Комментарии должны быть на английском языкеВ соответствии с руководством по написанию кода, все комментарии в прототипах должны быть только на английском языке. Рекомендуется перевести комментарии "Delta V" на английский язык.
Also applies to: 31-31
🧰 Tools
🪛 yamllint
[warning] 6-6: too few spaces before comment
(comments)
[warning] 12-12: too few spaces before comment
(comments)
[warning] 12-12: missing starting space in comment
(comments)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (19)
Resources/Prototypes/DeltaV/Roles/Jobs/Security/brigmedic.yml(1 hunks)Resources/Prototypes/DeltaV/Species/vulpkanin.yml(1 hunks)Resources/Prototypes/Nyanotrasen/Roles/Jobs/Security/prisonguard.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Command/captain.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/detective.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/head_of_security.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/security_cadet.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/security_officer.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/senior_officer.yml(1 hunks)Resources/Prototypes/Roles/Jobs/Security/warden.yml(1 hunks)Resources/Prototypes/Species/arachnid.yml(1 hunks)Resources/Prototypes/Species/human.yml(1 hunks)Resources/Prototypes/Species/moth.yml(1 hunks)Resources/Prototypes/Species/reptilian.yml(1 hunks)Resources/Prototypes/Species/slime.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Tajaran/Species/Tajaran.yml(1 hunks)Resources/Prototypes/_LostParadise/Roles/Jobs/CentCom/blueshieldoficer.yml(1 hunks)Resources/Prototypes/_LostParadise/Roles/Jobs/Security/SecEng.yml(1 hunks)Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (19)
Resources/Prototypes/DeltaV/Roles/Jobs/Security/brigmedic.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/DeltaV/Species/vulpkanin.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Nyanotrasen/Roles/Jobs/Security/prisonguard.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Command/captain.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/detective.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/head_of_security.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/security_cadet.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/security_officer.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/senior_officer.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Roles/Jobs/Security/warden.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Species/arachnid.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Species/human.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Species/moth.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Species/reptilian.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/Species/slime.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Tajaran/Species/Tajaran.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Roles/Jobs/CentCom/blueshieldoficer.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Roles/Jobs/Security/SecEng.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
🪛 yamllint
Resources/Prototypes/Roles/Jobs/Security/detective.yml
[error] 7-7: wrong indentation: expected at least 3
(indentation)
🔇 Additional comments (28)
Resources/Prototypes/_LostParadise/Roles/Jobs/Security/secupilot.yml (1)
7-8: Корректное добавление требования по росту
Добавленное требование минимального роста 160 см соответствует заявленным целям PR для ролей службы безопасности.
Resources/Prototypes/Roles/Jobs/Security/detective.yml (2)
7-8: Корректное добавление требования по росту
Требование минимального роста 160 см соответствует заявленным целям PR и согласуется с другими ролями службы безопасности.
🧰 Tools
🪛 yamllint
[error] 7-7: wrong indentation: expected at least 3
(indentation)
Line range hint 19-19: Проверьте влияние удаления доступа к Brig
Удаление доступа к Brig может существенно повлиять на игровой процесс детектива. Пожалуйста, подтвердите, что это намеренное изменение.
✅ Verification successful
Удаление доступа к Brig является намеренным изменением
Анализ кодовой базы показывает, что это часть более широкого изменения в DeltaV, где доступ к Brig был удален для всех ролей службы безопасности, включая:
- Detective
- Head of Security
- Security Officer
- Security Cadet
- Warden
- Prison Guard
Это подтверждается комментариями "Delta V: Removed" в нескольких файлах конфигурации ролей и явным удалением уровня доступа Brig из системы (Resources/Prototypes/Access/security.yml).
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Search for other files that might reference Detective's Brig access
rg -l "Detective.*Brig|Brig.*Detective" --type yaml
Length of output: 53
Script:
#!/bin/bash
# Search for detective role files and their access configurations
fd "detective.*\.yml" --type f --exec cat {}
# Search for any access configurations mentioning detective
rg "Detective.*access|access.*Detective" --type yaml -A 5 -B 5
# Search for brig access patterns in general to understand access structure
rg "Brig" --type yaml -A 3 -B 3
Length of output: 188288
🧰 Tools
🪛 yamllint
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[warning] 11-11: too few spaces before comment
(comments)
Resources/Prototypes/Roles/Jobs/Security/security_officer.yml (1)
7-8: Проверьте соответствие требований к росту
Добавленное требование к минимальному росту (160 см) соответствует указанным целям PR для отдела безопасности. Реализация корректна с технической точки зрения.
✅ Verification successful
Based on the search results, I can now generate the final response since I have enough information to verify the height requirement consistency across security department roles.
Требование к росту соответствует стандартам отдела
Минимальный рост 160 см установлен для всех должностей службы безопасности:
- Security Officer (security_officer.yml)
- Warden (warden.yml)
- Head of Security (head_of_security.yml)
- Security Cadet (security_cadet.yml)
- Prison Guard (prisonguard.yml)
- Brigmedic (brigmedic.yml)
- Security Engineer (SecEng.yml)
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify height requirements across security department roles
# Expected: All security roles should have 160cm minimum height requirement
rg -U "type: job[\s\S]*Security" | rg -U "CharacterHeightRequirement[\s\S]*min: \d+"
Length of output: 88261
Resources/Prototypes/Nyanotrasen/Roles/Jobs/Security/prisonguard.yml (2)
Line range hint 23-23: Корректное удаление доступа в бриг
Изменение задокументировано и соответствует общим изменениям в ролях безопасности.
7-8: Требование к росту добавлено корректно
Добавленное требование к минимальному росту 160 см соответствует указанным требованиям для других должностей службы безопасности.
Давайте проверим соответствие требований к росту в других ролях безопасности:
✅ Verification successful
Требование к росту соответствует стандартам безопасности
Проверка подтвердила, что минимальный рост 160 см для тюремного охранника соответствует требованиям, установленным для всех должностей службы безопасности:
- Офицер безопасности: 160 см
- Кадет: 160 см
- Начальник службы безопасности: 160 см
- Детектив: 160 см
- Надзиратель: 160 см
- Старший офицер: 160 см
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify height requirements across security roles
rg -U "type: job[^}]*Security" | rg -U "CharacterHeightRequirement" -A 2
Length of output: 3579
Resources/Prototypes/_LostParadise/Roles/Jobs/Security/SecEng.yml (1)
7-8: Корректное добавление требования по росту
Добавленное требование соответствует заявленным изменениям для ролей безопасности (минимум 160 см).
Давайте проверим согласованность с другими ролями безопасности:
✅ Verification successful
Требования по росту соответствуют стандартам
Проверка показала, что все роли безопасности (SecEng, Security Cadet, Security Officer, Head of Security) имеют одинаковое требование по минимальному росту в 160 единиц, что соответствует установленным стандартам.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify height requirements across security roles
# Expected: All security roles should have min: 160
rg -U "type: job[\s\S]*Security.*\n.*requirements:[\s\S]*CharacterHeightRequirement[\s\S]*min: \d+" --type yaml
Length of output: 5454
Resources/Prototypes/Roles/Jobs/Security/security_cadet.yml (1)
7-8: 🧹 Nitpick (assertive)
Проверьте соответствие минимальной высоты игровому балансу
Добавление требования минимальной высоты 160 см для кадетов службы безопасности может существенно ограничить выбор расы для этой роли. Убедитесь, что это соответствует балансу игры и лору.
Resources/Prototypes/Roles/Jobs/Security/warden.yml (2)
7-8: Требование к росту соответствует целям PR
Добавленное требование к минимальному росту в 160 см для должности Смотрителя согласуется с общими изменениями для ролей службы безопасности.
Line range hint 19-19: Подтвердите влияние удаления доступа к тюремному блоку
Удаление доступа к тюремному блоку может повлиять на игровой процесс. Необходимо убедиться, что это изменение не помешает Смотрителю выполнять свои обязанности.
✅ Verification successful
Based on the search results, I can see that the Brig access has been systematically removed across multiple security roles in the DeltaV codebase, as evidenced by the commented out lines with "# Delta V: Removed" or similar annotations in various security role files:
- warden.yml:
#- Brig # Delta V: Removed - head_of_security.yml:
#- Brig # Delta V: Removed - senior_officer.yml:
#- Brig # Delta V: Removed - detective.yml:
#- Brig # Delta V: Removed - security_cadet.yml:
#- Brig # Delta V: Removed - prisonguard.yml:
#- Brig #Delta V: Removed brig access
This appears to be an intentional design change across the entire codebase, where Brig access has been removed from all security roles. Therefore, I can generate the final response:
Удаление доступа к тюремному блоку является частью общих изменений в системе безопасности
Это изменение соответствует общей модификации системы доступа в кодовой базе DeltaV, где доступ к тюремному блоку был намеренно удален для всех ролей службы безопасности.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check other security roles and their access rights
# Look for other files that might reference Brig access
# Search for other security role definitions
rg -l "Security|Brig" Resources/Prototypes/Roles/Jobs/Security/
# Search for any references to Brig access in other files
rg "Brig" Resources/Prototypes/
Length of output: 23426
🧰 Tools
🪛 yamllint
[warning] 9-9: too few spaces before comment
(comments)
[warning] 11-11: too few spaces before comment
(comments)
Resources/Prototypes/DeltaV/Roles/Jobs/Security/brigmedic.yml (2)
7-8: Требование к росту добавлено корректно
Добавленное требование соответствует целям PR и установленным стандартам для медицинского и охранного персонала (160 см).
Line range hint 1-50: Проверка расположения прототипа
Прототип находится в папке DeltaV, что соответствует требованиям руководства по размещению новых прототипов.
🧰 Tools
🪛 yamllint
[warning] 11-11: too few spaces before comment
(comments)
Resources/Prototypes/Roles/Jobs/Security/senior_officer.yml (2)
8-9: Корректное добавление требования к росту
Требование минимального роста 160 см соответствует указанным целям PR для ролей службы безопасности.
Line range hint 47-47: Правильное обновление защитного снаряжения
Замена базовой брони на бронежилет с пластинами является логичным улучшением для данной роли.
🧰 Tools
🪛 yamllint
[warning] 6-6: too few spaces before comment
(comments)
[warning] 12-12: too few spaces before comment
(comments)
[warning] 12-12: missing starting space in comment
(comments)
Resources/Prototypes/_LostParadise/Roles/Jobs/CentCom/blueshieldoficer.yml (1)
7-8: Проверьте корректность требования к росту для данной роли
Требование минимального роста 160см добавлено корректно с технической точки зрения. Однако, так как BlueShieldOfficer является специальной ролью CentCom, нужно подтвердить, что это требование действительно должно соответствовать ограничениям для отделов Security/Medical (160см), а не Captain (150см).
Resources/Prototypes/Roles/Jobs/Security/head_of_security.yml (2)
7-8: Корректное добавление требования по росту!
Требование минимального роста 160 см соответствует указанным целям PR для должностей службы безопасности.
Line range hint 38-47: Проверить последствия удаления доступа к Brig
Удаление доступа к Brig для Главы Службы Безопасности может повлиять на его способность выполнять свои обязанности по надзору за тюремной зоной.
✅ Verification successful
The search results show that Brig access has been intentionally removed from all security roles (HoS, Warden, Senior Officer, Security Cadet, Detective) with consistent comments indicating this is a "Delta V" modification. The other matches in _LostParadise files are just related to item categories and catalogs, not actual access requirements.
Let me generate the final response in Russian:
Удаление доступа к Brig является намеренным изменением
Анализ кодовой базы показывает, что доступ к Brig был целенаправленно удален у всех ролей службы безопасности в рамках модификаций Delta V. Это системное изменение, а не ошибка:
- Изменение последовательно применено ко всем ролям безопасности (Warden, Senior Officer, Security Cadet, Detective)
- Все изменения помечены комментарием "Delta V: Removed"
- Не найдено игровых механик, требующих доступа HoS к Brig
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check if other security roles still have Brig access and if there are any references to HoS needing Brig access
# Search for Brig access in other security role definitions
echo "Checking Brig access in other security roles:"
rg "Brig" "Resources/Prototypes/Roles/Jobs/Security/"
# Search for any gameplay mechanics requiring HoS to have Brig access
echo "Checking for gameplay mechanics requiring HoS Brig access:"
rg -i "head.*security.*brig|hos.*brig" "Resources/Prototypes/"
Length of output: 4180
🧰 Tools
🪛 yamllint
[warning] 11-11: too few spaces before comment
(comments)
[warning] 11-11: missing starting space in comment
(comments)
Resources/Prototypes/Roles/Jobs/Command/captain.yml (3)
7-8: Корректное добавление требования к росту
Добавление минимального роста 150 см для роли Капитана соответствует заявленным целям PR и правильно реализовано синтаксически.
Line range hint 13-28: Хорошо документированные изменения названий отделов
Изменения названий отделов (Logistics и Epistemics) хорошо прокомментированы с пометками "DeltaV". Временные требования сохранены корректно.
Line range hint 1-85: Соответствие требованиям к коду
Файл соответствует требованиям:
- Отсутствуют переводы на русский язык
- Прототип находится в корректной папке
Resources/Prototypes/Species/slime.yml (1)
11-16: Проверьте соответствие размеров существующему лору
Добавленные параметры размеров для SlimePerson выглядят логично, но рекомендуется:
- Убедиться, что значения соответствуют существующему лору игры
- Проверить, как эти значения соотносятся с минимальными требованиями высоты для должностей (160 см для Охраны/Медицины)
✅ Verification successful
Let's examine the actual values to make an informed comparison of the dimensions across species.
Размеры слизня соответствуют общей концепции рас
Параметры размера для SlimePerson (0.7-1.1 по высоте, 0.65-1.15 по ширине) хорошо вписываются в существующую систему размеров рас:
- Они близки к параметрам гарпий (0.6-1.1) и меньше людей (0.8-1.25)
- Значения логичны для желеобразного существа, которое может менять форму
- Минимальные размеры позволяют проходить через стандартные проёмы
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Check other species files for height/width consistency
rg -l "minHeight|maxHeight|defaultHeight" Resources/Prototypes/Species/
Length of output: 352
Script:
#!/bin/bash
# Get height/width values from all species files to compare
for file in $(rg -l 'minHeight|maxHeight|defaultHeight' Resources/Prototypes/Species/); do
echo "=== $file ==="
rg "^[[:space:]]*(min|max|default)(Height|Width):" "$file"
done
Length of output: 2838
Resources/Prototypes/_LostParadise/Entities/Mobs/Species/Tajaran/Species/Tajaran.yml (1)
14-19: Проверьте соответствие размеров с игровым балансом
Добавленные параметры роста и ширины для расы Tajaran выглядят логично для гуманоидного вида. Однако, учитывая новые требования к минимальному росту для определенных ролей (160 см для Охраны и Медицины), необходимо убедиться, что указанные значения позволяют представителям этой расы занимать соответствующие должности.
Текущие значения роста:
- Минимум: 70 см
- По умолчанию: 80 см
- Максимум: 100 см
Resources/Prototypes/Species/moth.yml (1)
14-19: Проверьте соответствие минимальной высоты требованиям ролей
Текущее минимальное значение высоты (0.8) может быть слишком низким, учитывая требования для ролей Охраны и Медицины (160 см) и Капитана (150 см). Если 1.0 соответствует 100 см, то:
- Минимальная высота 0.8 = 80 см
- Максимальная высота 1.3 = 130 см
Это означает, что мотыльки не смогут занимать эти должности из-за ограничений по росту.
Resources/Prototypes/Species/reptilian.yml (1)
14-19: Параметры размеров рептилоидов обновлены корректно
Изменения высоты и ширины для расы рептилоидов выглядят логичными и соответствуют заявленным целям PR. Значения находятся в разумных пределах и согласуются с игровой механикой.
✅ Verification successful
Размеры рептилоидов соответствуют общей системе размеров рас
Анализ размеров всех рас показывает, что обновленные параметры рептилоидов (minHeight: 0.8, defaultHeight: 1.15, maxHeight: 1.35, minWidth: 0.70, defaultWidth: 1.15, maxWidth: 1.40) логично вписываются в существующую систему размеров:
- Значения находятся между параметрами малых рас (гарпии: 0.6-1.1) и крупных рас (арахниды: 0.9-1.45)
- Пропорции соответствуют другим гуманоидным расам (людям: 0.8-1.25)
- Сохраняется баланс между минимальными и максимальными значениями
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Description: Verify consistency of dimensions across species
# Check other species files for similar dimension parameters
fd -e yml -e yaml . Resources/Prototypes/Species/ --exec rg -l "(?s)type:\s+species.*?(minHeight|defaultHeight|maxHeight|minWidth|defaultWidth|maxWidth)"
Length of output: 155
Script:
#!/bin/bash
# Let's try a different approach to find species files and their dimensions
# First, find all species files
fd -e yml -e yaml . Resources/Prototypes/Species/ --exec cat {} \; | grep -A 6 "minHeight\|defaultHeight\|maxHeight\|minWidth\|defaultWidth\|maxWidth"
Length of output: 1883
Script:
#!/bin/bash
# Alternative approach using ripgrep
rg -A 6 "(minHeight|defaultHeight|maxHeight|minWidth|defaultWidth|maxWidth)" Resources/Prototypes/Species/
Length of output: 4808
Script:
#!/bin/bash
# Try to find all YAML files in Species directory
fd -e yml -e yaml . Resources/Prototypes/Species/
Length of output: 664
Resources/Prototypes/Species/arachnid.yml (2)
19-21: Параметры ширины выглядят корректно
Заданные значения ширины пропорциональны параметрам роста и выглядят логично для данного вида.
16-18: Проверьте ограничения роста для профессий
Максимальный рост арахнидов (145 см) не соответствует минимальным требованиям для:
- Медицинского персонала (160 см)
- Охраны (160 см)
- Капитана (150 см)
Пожалуйста, подтвердите, что это преднамеренное ограничение, соответствующее лору игры.
✅ Verification successful
Let's check the actual height configurations for all species and verify if there are any job-specific height requirements defined in the job configurations.
Давайте проверим, есть ли где-то в кодовой базе явные требования к росту для профессий, так как в файлах видов существ мы видим, что ограничения по росту разные, и арахниды не самые низкие среди них.
Подтверждаю ограничения для профессий службы безопасности
Проверка кодовой базы подтверждает наличие следующих требований к минимальному росту:
- Все должности службы безопасности: 160 см
- Капитан: 150 см
Максимальный рост арахнидов (145 см) действительно не позволяет им занимать эти должности. Это соответствует игровому балансу, так как:
- Другие расы также имеют ограничения по росту (например, дварфы: 60-80 см)
- В файлах локализации присутствуют строки для проверки роста
- Требования к росту явно прописаны в конфигурации должностей
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Check other species height restrictions for comparison
fd -e yml -e yaml . Resources/Prototypes/Species/ -x rg -l "minHeight|maxHeight|defaultHeight" {}
Length of output: 380
Script:
#!/bin/bash
# Get height restrictions for all species
for file in $(fd -e yml -e yaml . Resources/Prototypes/Species/); do
echo "=== $file ==="
rg "Height:" -A 1 "$file" || true
done
# Check for job height requirements
fd -e yml -e yaml . Resources/Prototypes/Jobs/ -x rg "Height|height" {} || true
Length of output: 4297
Script:
#!/bin/bash
# Search for job requirements in the entire codebase
rg -i "height.*requirement|minimum.*height|required.*height" -A 2 -B 2
# Search specifically in any job-related files
fd -e yml -e yaml -e json . -x rg -l "job|Job|occupation|role" {} | xargs rg -i "height"
Length of output: 11592
Resources/Prototypes/DeltaV/Species/vulpkanin.yml (1)
14-19:
Критические проблемы с параметрами роста и ширины
Обнаружены следующие проблемы:
- Максимальный рост (110 см) ниже минимальных требований для:
- Охраны и Медицины (160 см)
- Капитана (150 см)
- Значения роста кажутся слишком малыми для гуманоидного вида
- Значения ширины непропорциональны росту
Предлагаемые изменения:
- minHeight: 0.75
- defaultHeight: 0.85
- maxHeight: 1.1
- minWidth: 0.65
- defaultWidth: 0.85
- maxWidth: 1.15
+ minHeight: 1.6 # 160 см - минимум для соответствия требованиям
+ defaultHeight: 1.8 # 180 см - стандартный рост
+ maxHeight: 2.0 # 200 см - максимум
+ minWidth: 0.4 # Пропорционально новому росту
+ defaultWidth: 0.5
+ maxWidth: 0.6Resources/Prototypes/Species/human.yml (1)
10-15:
Некорректные значения роста и ширины для людей
Обнаружены следующие проблемы:
- Диапазон роста (80-125 см) не соответствует требованиям PR для должностей Охраны и Медицины (минимум 160 см)
- Диапазон ширины (70-125 см) кажется чрезмерно большим для человека
- Значения по умолчанию требуют корректировки
Предлагаемые изменения:
- minHeight: 0.8
- defaultHeight: 1
- maxHeight: 1.25
- minWidth: 0.7
- defaultWidth: 1
- maxWidth: 1.25
+ minHeight: 1.5 # 150 см - минимум для Капитана
+ defaultHeight: 1.7 # 170 см - средний рост человека
+ maxHeight: 2.1 # 210 см - максимальный рост
+ minWidth: 0.35 # 35 см - минимальная ширина
+ defaultWidth: 0.45 # 45 см - средняя ширина
+ maxWidth: 0.65 # 65 см - максимальная ширинаLikely invalid or redundant comment.
Описание обновления:
Изменены верхние/нижние пороги для роста у некоторых рас для соответствия имеющимся лору
так же для сб/осщ было поставлено ограничение по высоте от 160см, а капитану ограничение от 150см
Медиа
Проверки
Изменения
🆑 Kest