[Feature] New Drink Bacchus #999
Conversation
WalkthroughЭтот набор изменений вводит новый напиток под названием "Благословение Бахуса" в игровую систему. Изменения включают обновление локализации на русском языке, создание нового прототипа напитка, добавление нового вкуса, определение новой химической реакции для создания напитка и подготовку графических ресурсов. Также были удалены некоторые существующие вкусы и произведены незначительные корректировки параметров алкоголя. Changes
Poem
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
Changes: Sprites
|
ЭТО ТАКОЕ НАЧАЛО. |
![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 comments (2)
Resources/Prototypes/_Backmen/Reactions/drinks.yml (1)
Обнаружены зависимости для рецепта CondensedMilk
Удаление рецепта создания сгущённого молока повлияет на следующие игровые элементы:
- Реагент
CondensedMilkопределён вalcohol.ymlкак напиток- Банки со сгущённым молоком в
drinks_cans.ymlсодержат 30 единиц этого реагентаРекомендуется либо сохранить рецепт, либо пересмотреть связанные прототипы, чтобы избежать недоступных для крафта предметов.
🔗 Analysis chain
Line range hint
1-10: Проверьте зависимости удалённого рецептаУдаление рецепта CondensedMilk может повлиять на другие рецепты или механики игры, использующие этот ингредиент.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Поиск использования CondensedMilk в других рецептах rg -l "CondensedMilk" Resources/Prototypes/Length of output: 243
Script:
#!/bin/bash # Check how CondensedMilk is used in other files echo "=== alcohol.yml ===" rg -A 5 -B 5 "CondensedMilk" Resources/Prototypes/_Backmen/Reagents/Consumable/Drink/alcohol.yml echo -e "\n=== drinks_cans.yml ===" rg -A 5 -B 5 "CondensedMilk" Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks_cans.ymlLength of output: 790
Resources/Prototypes/_Backmen/Flavors/flavors.yml (1)
Обнаружены критические зависимости удаляемых вкусов
Следующие вкусы активно используются в других файлах и не могут быть удалены:
RedOrdenDrinkFlavour: Используется в реагенте RedOrdenDrinkenthralling: Используется в реагенте LotophagoiOilacidic: Используется в реагенте FleshAcidsublime: Используется в реагенте Ectoplasmsoju: Используется в реагенте Soju🔗 Analysis chain
Line range hint
1-89: Проверьте удаление вкусов на наличие зависимостейМассовое удаление вкусов может привести к ошибкам в существующих рецептах напитков. Необходимо убедиться, что удаленные вкусы больше нигде не используются.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Search for references to removed flavors for flavor in RedOrdenDrinkFlavour enthralling acidic sublime seeds cotton vanilla soju orangecreamcicle silverjack brainbomb atomicpunch circusjuice pinkdrink sapopicante graveyard bubbletea corncob; do echo "Checking references for flavor: $flavor" rg -l "$flavor" --type yaml doneLength of output: 9416
Script:
#!/bin/bash # Check actual usage context of referenced flavors echo "=== RedOrdenDrinkFlavour usage ===" rg -A 5 -B 5 "RedOrdenDrinkFlavour" Resources/Prototypes/_Backmen/Reagents/Consumable/Drink/alcohol.yml echo -e "\n=== enthralling usage ===" rg -A 5 -B 5 "enthralling" Resources/Prototypes/_Backmen/Reagents/psionic.yml echo -e "\n=== acidic usage ===" rg -A 5 -B 5 "acidic" Resources/Prototypes/Reagents/chemicals.yml Resources/Prototypes/Reagents/toxins.yml echo -e "\n=== sublime usage ===" rg -A 5 -B 5 "sublime" Resources/Prototypes/_Backmen/Reagents/psionic.yml echo -e "\n=== soju usage ===" rg -A 5 -B 5 "soju" Resources/Prototypes/_Backmen/Reagents/Consumable/Drink/alcohol.yml Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks_bottles.yml echo -e "\n=== vanilla usage ===" rg -A 5 -B 5 "vanilla" Resources/Prototypes/Entities/Objects/Consumable/Food/frozen.yml Resources/Prototypes/Entities/Objects/Consumable/Food/Baked/cake.ymlLength of output: 13233
🧹 Nitpick comments (2)
Resources/Locale/ru-RU/backmen/prototypes/entities/consumable/drinks/drinks.ftl (1)
4-5: Рекомендуется расширить описание напиткаТекущее описание "Вас предупредили об этом" довольно загадочное. Предлагаю добавить больше деталей о самом напитке, его эффектах и происхождении для лучшего погружения игрока.
reagent-name-bacchus-blessing = Благословение Бахуса -reagent-desc-bacchus-blessing = Вас предупредили об этом. +reagent-desc-bacchus-blessing = Легендарный напиток, названный в честь бога вина. Говорят, что даже самые стойкие не могут устоять перед его силой. Вас предупредили об этом.Resources/Locale/ru-RU/backmen/flavors/flavor-profiles.ftl (1)
2-2: Рекомендуется организовать записи в алфавитном порядкеНовый профиль вкуса добавлен в случайное место в файле. Предлагаю организовать все записи в алфавитном порядке для лучшей поддерживаемости.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
Resources/Textures/Backmen/Objects/Consumable/Drinks/bacchus.rsi/icon.pngis excluded by!**/*.png
📒 Files selected for processing (8)
Resources/Locale/ru-RU/backmen/flavors/flavor-profiles.ftl(1 hunks)Resources/Locale/ru-RU/backmen/prototypes/entities/consumable/drinks/drinks.ftl(1 hunks)Resources/Prototypes/Reagents/Consumable/Drink/alcohol.yml(3 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml(1 hunks)Resources/Prototypes/_Backmen/Flavors/flavors.yml(1 hunks)Resources/Prototypes/_Backmen/Reactions/drinks.yml(1 hunks)Resources/Prototypes/_Backmen/Reagents/Consumable/Drink/alcohol.yml(1 hunks)Resources/Textures/Backmen/Objects/Consumable/Drinks/bacchus.rsi/meta.json(1 hunks)
✅ Files skipped from review due to trivial changes (1)
- Resources/Textures/Backmen/Objects/Consumable/Drinks/bacchus.rsi/meta.json
🧰 Additional context used
🪛 GitHub Actions: YAML Linter
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml
[error] 30-30: Failed to load RSI
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
- GitHub Check: build (ubuntu-latest)
🔇 Additional comments (4)
Resources/Prototypes/_Backmen/Reactions/drinks.yml (1)
14-24: Проверьте баланс рецептаРецепт использует 4 единицы сильных алкогольных напитков для создания 3 единиц продукта. Это может быть неэффективно с точки зрения игровой механики.
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml (1)
Line range hint
1-16: Проверьте зависимости удалённого напиткаУдаление RedOrdenDrinkGlass может повлиять на другие части игры. Необходимо проверить все ссылки на этот напиток.
🧰 Tools
🪛 GitHub Actions: YAML Linter
[error] 30-30: Failed to load RSI
Resources/Prototypes/_Backmen/Flavors/flavors.yml (1)
91-94: Новый вкус добавлен корректноСтруктура нового вкуса
BacchusDrinkFlavourсоответствует существующему формату.Resources/Prototypes/Reagents/Consumable/Drink/alcohol.yml (1)
173-173: Улучшен баланс влияния алкоголя на гномовУменьшение урона от отравления для гномов (с 1.0 до 0.5 и с 0.2 до 0.1) является обоснованным изменением баланса, учитывая их расовые особенности.
Also applies to: 183-183
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml
Outdated
Show resolved
Hide resolved
спрайт сломался, это такой конец |
There was a problem hiding this comment.
Actionable comments posted: 0
🔭 Outside diff range comments (1)
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml (1)
Требуется полное удаление всех зависимостей напитка
Для корректного удаления "Кроваво-красной слезы" необходимо удалить следующие компоненты:
- Рецепт создания напитка в
Recipes/Reactions/drinks.yml- Определение реагента в
Reagents/Consumable/Drink/alcohol.yml- Профиль вкуса
RedOrdenDrinkFlavourвFlavors/flavors.yml- Локализации в:
drinks.ftlalcohol.ftlflavor-profiles.ftl🔗 Analysis chain
Line range hint
1-16: Проверьте зависимости удаляемого напиткаУдаление сущности
RedOrdenDrinkGlassможет повлиять на другие части игры. Убедитесь, что:
- Все ссылки на этот напиток удалены из рецептов
- Удалены соответствующие записи в локализации
- Обновлены все зависимые прототипы
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Check for any remaining references to RedOrdenDrinkGlass rg -l "RedOrdenDrinkGlass|RedOrdenDrink" Resources/Length of output: 535
Script:
#!/bin/bash # Check content of files referencing RedOrdenDrink echo "=== Checking flavors.yml ===" cat Resources/Prototypes/_Backmen/Flavors/flavors.yml | grep -A 5 -B 5 "RedOrden" echo -e "\n=== Checking drink recipes ===" cat Resources/Prototypes/_Backmen/Recipes/Reactions/drinks.yml | grep -A 10 -B 10 "RedOrden" echo -e "\n=== Checking alcohol reagents ===" cat Resources/Prototypes/_Backmen/Reagents/Consumable/Drink/alcohol.yml | grep -A 10 -B 10 "RedOrden" echo -e "\n=== Checking localizations ===" cat Resources/Locale/ru-RU/ss14-ru/prototypes/backmen/entities/objects/consumable/drinks/drinks.ftl | grep -A 2 -B 2 "RedOrden" cat Resources/Locale/ru-RU/backmen/reagents/meta/consumable/drink/alcohol.ftl | grep -A 2 -B 2 "RedOrden" cat Resources/Locale/ru-RU/backmen/flavors/flavor-profiles.ftl | grep -A 2 -B 2 "RedOrden"Length of output: 2393
♻️ Duplicate comments (1)
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml (1)
30-31:⚠️ Potential issueОтсутствует файл спрайта
Файл спрайта
Backmen/Objects/Consumable/Drinks/bacchus.rsiотсутствует в репозитории, что приводит к ошибкам сборки.
🧹 Nitpick comments (1)
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml (1)
21-29: Рекомендуется добавить описание напиткаВ отличие от удалённого напитка, у нового отсутствуют поля
nameиdescription. Хотя использованиеsuffixкорректно для локализации, рекомендуется добавить описание напитка в прототип для улучшения читаемости кода.id: DrinkBacchusBlessing suffix: Благословение Бахуса + name: Благословение Бахуса + description: Самый крепкий и мемный напиток в игре. components:
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (1)
Resources/Prototypes/_Backmen/Entities/Objects/Consumable/Drinks/drinks.yml (1)
19-20: Проверьте изменение родительского классаИзменение родительского класса с
DrinkGlassBaseнаDrinkGlassможет привести к неожиданному поведению:
- Убедитесь, что все необходимые компоненты присутствуют в
DrinkGlass- Проверьте, что изменение не повлияет на механику напитка
✅ Verification successful
Изменение родительского класса безопасно
Изменение с
DrinkGlassBaseнаDrinkGlassявляется безопасным, так какDrinkGlassнаследует все необходимые компоненты отDrinkGlassBase. Это изменение соответствует общей тенденции в кодовой базе.🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Compare components between DrinkGlass and DrinkGlassBase rg -A 10 "type: entity\n\s+id: DrinkGlass" Resources/ rg -A 10 "type: entity\n\s+id: DrinkGlassBase" Resources/Length of output: 499
Script:
#!/bin/bash # Search for DrinkGlass and DrinkGlassBase definitions rg -U -A 15 "^- type: entity\s+id: DrinkGlass[^a-zA-Z]" Resources/ rg -U -A 15 "^- type: entity\s+id: DrinkGlassBase" Resources/ # Search for any references to these classes rg "parent: DrinkGlass" Resources/ rg "parent: DrinkGlassBase" Resources/Length of output: 16500
Script:
#!/bin/bash # Try different pattern to find entity definitions rg -U "DrinkGlass:$" -A 10 Resources/ rg -U "DrinkGlassBase:$" -A 10 Resources/ # Look for any component differences rg -U "components:" -A 5 -B 2 "DrinkGlass" Resources/Length of output: 93316
Script:
#!/bin/bash # Try to find entity definitions with a more specific pattern rg -U "(?s)- type: entity.*?id: DrinkGlass[^a-zA-Z].*?(?=\n\n|\z)" Resources/ rg -U "(?s)- type: entity.*?id: DrinkGlassBase.*?(?=\n\n|\z)" Resources/ # Also search for any component-related files fd -g "*Glass*.yml" Resources/Length of output: 795
Script:
#!/bin/bash # Search for entity definitions with simpler patterns rg -U -A 10 "^- type: entity\s+id: DrinkGlass$" Resources/ rg -U -A 10 "^- type: entity\s+id: DrinkGlassBase$" Resources/ # Search for any component references rg "components:" -B 1 -A 3 "DrinkGlass" Resources/Length of output: 65762
Script:
#!/bin/bash # Search in Prototypes directory with simpler patterns rg -l "DrinkGlass" Resources/Prototypes/ rg -l "DrinkGlassBase" Resources/Prototypes/ # Try to find the actual definitions rg -U "id: DrinkGlass" -B 1 -A 5 Resources/Prototypes/ rg -U "id: DrinkGlassBase" -B 1 -A 5 Resources/Prototypes/Length of output: 7522
|
@KayzelW прости это в последний раз, тыкни мёрдж пожалуйста |
Бахус уже здесь - самый сильный (и мемный) алкогольный напиток в игре!
Также этанол теперь наносит в 2 раза меньше урон ядами
Summary by CodeRabbit
Новые функции
Исправления
Удаления