Удалил ивент со спавном нейтральных ксеносов #187
Conversation
ОбзорВ файле Изменения
Предлагаемые метки
Предлагаемые рецензенты
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: 1
🔭 Outside diff range comments (2)
Resources/Prototypes/Entities/Mobs/NPCs/xenopet.yml (2)
Line range hint
1-340: Несоответствие описаний и поведения новых сущностейОбнаружены следующие проблемы в новых сущностях:
- Описание "A very clearly friend-shaped Xeno" для Hell-Shaped противоречит его имени
- Все сущности используют фракцию PetsNT вместо Passive, что существенно меняет их поведение
Предлагаю:
- Обновить описания в соответствии с характером сущностей
- Проверить, что смена фракции на PetsNT соответствует желаемому игровому поведению
Обновите friendlyxenoSpawner для использования новых прототипов ксеноморфов
Проверка подтвердила, что:
- Старые прототипы
MobXenoNeutral*используются только в спавнере- Новые прототипы
MobXenoFriendShaped,MobXenoPatriarch,MobXenoFXES,MobXenoHellShapedиMobXenoSubjectTessуже определены в системеРекомендуемые изменения:
- type: RandomSpawner prototypes: - - MobXenoNeutralRouny - - MobXenoNeutralPraetorian - - MobXenoNeutralDrone - - MobXenoNeutralRavager + - MobXenoFriendShaped + - MobXenoPatriarch + - MobXenoFXES + - MobXenoHellShaped + - MobXenoSubjectTess chance: 1🔗 Analysis chain
Line range hint
365-600: Необходимо обновить friendlyxenoSpawnerСпавнер всё ещё использует старые нейтральные типы мобов (MobXenoNeutralRouny и т.д.), в то время как были добавлены новые сущности с фракцией PetsNT. Это может привести к несогласованному поведению.
Предлагаю обновить список прототипов в RandomSpawner:
- type: RandomSpawner prototypes: - - MobXenoNeutralRouny - - MobXenoNeutralPraetorian - - MobXenoNeutralDrone - - MobXenoNeutralRavager + - MobXenoFriendShaped + - MobXenoPatriarch + - MobXenoFXES + - MobXenoHellShaped + - MobXenoSubjectTess chance: 1🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Проверяем использование старых типов мобов в других файлах rg "MobXenoNeutral(Rouny|Drone|Praetorian|Ravager)" --type yamlLength of output: 1477
Script:
#!/bin/bash # Проверяем существование новых типов мобов в файлах rg "MobXeno(FriendShaped|Patriarch|FXES|HellShaped|SubjectTess)" --type yamlLength of output: 454
| # WWDP START | ||
| #- type: entity | ||
| # id: neutralXenoVents | ||
| # parent: BaseGameRule | ||
| # categories: [ HideSpawnMenu ] | ||
| # components: | ||
| # - type: StationEvent | ||
| # startAnnouncement: true | ||
| # earliestStart: 20 | ||
| # reoccurrenceDelay: 12 | ||
| # minimumPlayers: 1 | ||
| # weight: 6 # Really weak compared to other critters | ||
| # duration: 60 | ||
| # - type: VentCrittersRule | ||
| # entries: | ||
| # - id: MobXenoNeutralRouny | ||
| # prob: 0.0075 | ||
| # - id: MobXenoNeutralDrone | ||
| # prob: 0.0075 | ||
| # - id: MobXenoNeutralPraetorian | ||
| # prob: 0.0075 | ||
| # - id: MobXenoNeutralRavager | ||
| # prob: 0.0075 | ||
| # WWDP END |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Необходимо полностью удалить закомментированный код
Закомментированный код neutralXenoVents следует полностью удалить вместо того, чтобы оставлять в файле. Если код понадобится в будущем, его всегда можно найти в истории Git.
-# WWDP START
-#- type: entity
-# id: neutralXenoVents
-# parent: BaseGameRule
-# categories: [ HideSpawnMenu ]
-# components:
-# - type: StationEvent
-# startAnnouncement: true
-# earliestStart: 20
-# reoccurrenceDelay: 12
-# minimumPlayers: 1
-# weight: 6 # Really weak compared to other critters
-# duration: 60
-# - type: VentCrittersRule
-# entries:
-# - id: MobXenoNeutralRouny
-# prob: 0.0075
-# - id: MobXenoNeutralDrone
-# prob: 0.0075
-# - id: MobXenoNeutralPraetorian
-# prob: 0.0075
-# - id: MobXenoNeutralRavager
-# prob: 0.0075
-# WWDP END
There was a problem hiding this comment.
🛠️ Refactor suggestion
Необходимо полностью удалить закомментированный код
Закомментированный код neutralXenoVents следует полностью удалить вместо того, чтобы оставлять в файле. Если код понадобится в будущем, его всегда можно найти в истории Git.
-# WWDP START -#- type: entity -# id: neutralXenoVents -# parent: BaseGameRule -# categories: [ HideSpawnMenu ] -# components: -# - type: StationEvent -# startAnnouncement: true -# earliestStart: 20 -# reoccurrenceDelay: 12 -# minimumPlayers: 1 -# weight: 6 # Really weak compared to other critters -# duration: 60 -# - type: VentCrittersRule -# entries: -# - id: MobXenoNeutralRouny -# prob: 0.0075 -# - id: MobXenoNeutralDrone -# prob: 0.0075 -# - id: MobXenoNeutralPraetorian -# prob: 0.0075 -# - id: MobXenoNeutralRavager -# prob: 0.0075 -# WWDP END
Всё нормально.
There was a problem hiding this comment.
@RedBurningPhoenix, понял! Закомментированный код будет полностью удалён, чтобы не засорять файл. Если в будущем он понадобится, всегда можно вернуться к истории Git. Если возникнут дополнительные вопросы или пожелания – обращайтесь!
There was a problem hiding this comment.
@RedBurningPhoenix, понял! Закомментированный код будет полностью удалён, чтобы не засорять файл. Если в будущем он понадобится, всегда можно вернуться к истории Git. Если возникнут дополнительные вопросы или пожелания – обращайтесь!
Оставь ты этот закомментированный код в покое.
There was a problem hiding this comment.
@RedBurningPhoenix, понял! Уважая ваше решение, оставляем закомментированный код в прежнем виде. Если потребуется дополнительная помощь или возникнут вопросы, обращайтесь.
There was a problem hiding this comment.
@RedBurningPhoenix а он что, может код переписывать?
There was a problem hiding this comment.
Seems like the humans are having a chat. I'll hop back into my burrow for now. If you need me again, just tag @coderabbitai in a new comment, and I'll come hopping out!
There was a problem hiding this comment.
@RedBurningPhoenix а он что, может код переписывать?
Да, если его попросить он может изменить код
Описание PR
Удалён ивент neutralXenoVents. Больше на станции не будет по 10 бесполезных тушек ксеносов.
Изменения
🆑 Дексп