Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (13)
🚧 Files skipped from review as they are similar to previous changes (3)
📝 WalkthroughWalkthroughВнесены возрастные требования (AgeRequirement) во множество должностей; в некоторых файлах добавлены или удалены блоки startingGear; у ряда ролей добавлены метаданные и специальные компоненты. Изменения структурны для YAML, без изменений бизнес‑логики за пределами требований и снаряжения. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can generate a title for your PR based on the changes with custom instructions.Set the |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (3)
Resources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/gastronomic_specialist.yml (1)
1-24:⚠️ Potential issue | 🔴 CriticalДобавьте ссылку на startingGear в определение должности.
В блоке job отсутствует свойство
startingGear: GastronomicSpecialistGear. Прототип экипировки определен в том же файле, но не привязан к должности. Игроки, выбравшие эту роль, не получат стартовое снаряжение.Исправление
- type: job id: GastronomicSpecialist name: job-name-gastronomic-specialist description: job-description-gastronomic-specialist playTimeTracker: JobGastronomicSpecialist requirements: - !type:AgeRequirement requiredAge: 22 - !type:AgeRequirement requiredAge: 85 inverted: true + startingGear: GastronomicSpecialistGear icon: JobGastronomicSpecialist🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/gastronomic_specialist.yml` around lines 1 - 24, The GastronomicSpecialist job lacks a startingGear link so players won't receive the equipment; update the job definition for id/Name GastronomicSpecialist by adding startingGear: GastronomicSpecialistGear at the top-level of the job block (same indentation as icon/supervisors/weight) so the job references the existing GastronomicSpecialistGear prototype and grants the gear on spawn.Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/electrical_technical_specialist.yml (1)
1-28:⚠️ Potential issue | 🟠 MajorДобавьте ссылку
startingGearв определение должности или удалите неиспользуемый прототип.Персонаж не получит стартовое снаряжение при выборе этой должности, так как поле
startingGear: ElectricalTechnicalSpecialistGearотсутствует в определении job (строки 1-28). ПрототипElectricalTechnicalSpecialistGear(строки 30-45) существует в файле, но не используется.Необходимо либо добавить
startingGear: ElectricalTechnicalSpecialistGearв определение job, либо удалить неиспользуемый прототип снаряжения.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/electrical_technical_specialist.yml` around lines 1 - 28, The job "ElectricalTechnicalSpecialist" is missing a startingGear reference so players won't receive the ElectricalTechnicalSpecialistGear prototype; either add startingGear: ElectricalTechnicalSpecialistGear to the job definition for id ElectricalTechnicalSpecialist (so the job grants the ElectricalTechnicalSpecialistGear) or remove the unused prototype named ElectricalTechnicalSpecialistGear if that gear is not needed.Resources/Prototypes/_Scp/Roles/Jobs/LogisticsService/logistics_officer.yml (1)
31-46:⚠️ Potential issue | 🟠 MajorДобавьте поле
startingGearв определение должности.Прототип
job(строки 1-29) отсутствует полеstartingGear: LogisticsOfficerGear. По сравнению с другими job-прототипами в репозитории (RRH.yml, NTF.yml, MH.yml и др.), все они явно указывают связь на стартовую экипировку. Без этого поля экипировка не будет применена к должности.Все сущности экипировки существуют и доступны в кодовой базе.
Требуется добавить
startingGear: LogisticsOfficerGearпосле поляiconв определение job.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Scp/Roles/Jobs/LogisticsService/logistics_officer.yml` around lines 31 - 46, В прототипе job в этом файле добавьте поле startingGear со значением LogisticsOfficerGear сразу после поля icon; конкретно внесите строку startingGear: LogisticsOfficerGear в блок описания job (где находятся icon и прочие свойства) чтобы связать дефиницию должности с уже существующим типом экипировки LogisticsOfficerGear.
🧹 Nitpick comments (1)
Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/technical_trainee.yml (1)
6-11: Узкий возрастной диапазон для позиции стажёра.Минимальный возраст 25 лет и максимальный 45 лет создают 20-летнее окно. Для стажёрской должности минимальный возраст кажется высоким. Возможно, стоит проверить соответствие требований в таблице.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/technical_trainee.yml` around lines 6 - 11, Текущие два требования AgeRequirement (поля requiredAge и inverted) создают узкий диапазон 25–44; исправьте их в блоке requirements: либо снизьте минимальный возраст (например, поменять первое requiredAge с 25 на 18) и/или расширьте/удалите верхнюю границу (отредактировать второе AgeRequirement или убрать inverted-правило), чтобы соответствовать принятой таблице требований — обновите значения requiredAge и/или удалите второй AgeRequirement в техничес_trainee.yml в зависимости от решения HR.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Outside diff comments:
In
`@Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/electrical_technical_specialist.yml`:
- Around line 1-28: The job "ElectricalTechnicalSpecialist" is missing a
startingGear reference so players won't receive the
ElectricalTechnicalSpecialistGear prototype; either add startingGear:
ElectricalTechnicalSpecialistGear to the job definition for id
ElectricalTechnicalSpecialist (so the job grants the
ElectricalTechnicalSpecialistGear) or remove the unused prototype named
ElectricalTechnicalSpecialistGear if that gear is not needed.
In
`@Resources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/gastronomic_specialist.yml`:
- Around line 1-24: The GastronomicSpecialist job lacks a startingGear link so
players won't receive the equipment; update the job definition for id/Name
GastronomicSpecialist by adding startingGear: GastronomicSpecialistGear at the
top-level of the job block (same indentation as icon/supervisors/weight) so the
job references the existing GastronomicSpecialistGear prototype and grants the
gear on spawn.
In `@Resources/Prototypes/_Scp/Roles/Jobs/LogisticsService/logistics_officer.yml`:
- Around line 31-46: В прототипе job в этом файле добавьте поле startingGear со
значением LogisticsOfficerGear сразу после поля icon; конкретно внесите строку
startingGear: LogisticsOfficerGear в блок описания job (где находятся icon и
прочие свойства) чтобы связать дефиницию должности с уже существующим типом
экипировки LogisticsOfficerGear.
---
Nitpick comments:
In
`@Resources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/technical_trainee.yml`:
- Around line 6-11: Текущие два требования AgeRequirement (поля requiredAge и
inverted) создают узкий диапазон 25–44; исправьте их в блоке requirements: либо
снизьте минимальный возраст (например, поменять первое requiredAge с 25 на 18)
и/или расширьте/удалите верхнюю границу (отредактировать второе AgeRequirement
или убрать inverted-правило), чтобы соответствовать принятой таблице требований
— обновите значения requiredAge и/или удалите второй AgeRequirement в
техничес_trainee.yml в зависимости от решения HR.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: a470f498-6f79-4d53-97d4-6b8361f3ecaa
📒 Files selected for processing (34)
Resources/Prototypes/_Scp/Roles/Jobs/Administration/chief_technical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/director.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/head_of_medical_sector.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/personnel_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/scientific_service_director.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/security_commander.ymlResources/Prototypes/_Scp/Roles/Jobs/Administration/senior_logistics_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/CommandantSquad/external_administrative_zone_commandant.ymlResources/Prototypes/_Scp/Roles/Jobs/CommandantSquad/external_administrative_zone_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/CommandantSquad/field_medical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/CommandantSquad/junior_external_administrative_zone_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/CommandantSquad/senior_external_administrative_zone_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/electrical_technical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/general_technical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/lead_technical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/EngineeringTechnicalService/technical_trainee.ymlResources/Prototypes/_Scp/Roles/Jobs/EthicsCommittee/ethics_committee_representative.ymlResources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/botanist_phytogeneticist.ymlResources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/gastronomic_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/FoodAndSanitationService/service_personnel.ymlResources/Prototypes/_Scp/Roles/Jobs/LogisticsService/logistics_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/MedicalService/hospital_intern.ymlResources/Prototypes/_Scp/Roles/Jobs/MedicalService/medical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/MedicalService/pharmaceutical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/MedicalService/psychotherapist.ymlResources/Prototypes/_Scp/Roles/Jobs/MedicalService/senior_medical_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/ScientificService/anomarchology_researcher.ymlResources/Prototypes/_Scp/Roles/Jobs/ScientificService/junior_researcher.ymlResources/Prototypes/_Scp/Roles/Jobs/ScientificService/lead_researcher.ymlResources/Prototypes/_Scp/Roles/Jobs/ScientificService/production_specialist.ymlResources/Prototypes/_Scp/Roles/Jobs/SpecialPurposeSquad/heavy_containment_zone_commandant.ymlResources/Prototypes/_Scp/Roles/Jobs/SpecialPurposeSquad/heavy_containment_zone_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/SpecialPurposeSquad/junior_heavy_containment_zone_officer.ymlResources/Prototypes/_Scp/Roles/Jobs/SpecialPurposeSquad/senior_heavy_containment_zone_officer.yml
|
Пишет кролик: "Добавлено новое начальное снаряжение для нескольких администратвистских должностей." |
|
Поставь максимально возможный возраст всех персонажей на 85 и убери эту границу в ролях, чтобы меньше текста было и игрокам нужно было меньше думать |
Сам же и говорил про возраст на 65 у СБ. |
|
Пусть ставят 35 лет если лень дёргать возраст. |
это по рп аномальные гиперактивные деды офицеры |
|
Я имею ввиду, что раз у нас у всех максимум 85 лет, то зачем оставлять максимально доступный в редакторе доступ в 120 лет. Нужно уменьшить максимальный возраст в редакторе до 85, а из работ это убрать. Так логика сохранится, а игрокам будет проще |
а если у кого то персонажу 86 лет? |
|
Надеюсь там есть валидация и ему сбросит до 85 |
Я выше написал про то что бред что бля старый дед дослужился лишь до кадета в 85 лет. Ну бред бля. |
быстрее ставьте всем своим персонажам 120 лет пока не поздно |
|
раритет будет |
Убери границу только у тех, кто будет иметь максимум 85 лет |
3 participants
Краткое описание | Short description
Добавление проверки на возраст всем должностям.
AgeRequirement
https://docs.google.com/spreadsheets/d/19IPltFw6aDL_wLn-AyLI5nHwxeJolM6dZ4K7nR30j54/edit?gid=1950741977#gid=1950741977
Changelog
🆑 Parashut
Summary by CodeRabbit
Примечания к выпуску
Новые функции
Изменения