[Fix] Huinya #1049
[Fix] Huinya #1049
Conversation
|
""" WalkthroughИзменения затрагивают несколько YAML-файлов, используемых для определения игровых объектов. Внесены корректировки в каталоги шкафов, оружейных сейфов и боеприпасов: добавлены новые сущности, обновлены существующие (например, добавлен новый предмет, изменены вероятности компонентов) и удалены устаревшие элементы. В файлах uplink_catalog.yml и uplink_nomad.yml удалены многие списки оружия, взрывчатки и боеприпасов, а также добавлены новые товары с дополнительными условиями покупки. Дополнительно стандартизирована терминология для боеприпасов, где «Bullet» заменили на «Cartridge». 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 (
|
![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 (1)
Resources/Prototypes/_Backmen/Catalog/uplink_catalog.yml (1)
156-165: 🛠️ Refactor suggestionДобавьте условия безопасности для телепортера.
Экспериментальный телепортер может быть опасным предметом. Рекомендуется добавить условия использования и ограничения для предотвращения злоупотреблений.
categories: - UplinkDeception + conditions: + - !type:BuyerWhitelistCondition + blacklist: + components: + - SurplusBundle
🧹 Nitpick comments (2)
Resources/Prototypes/_Backmen/Catalog/uplink_catalog.yml (2)
90-99: Рассмотрите возможность полного удаления закомментированного кода.Вместо сохранения закомментированного кода в репозитории, лучше полностью удалить его, так как история изменений сохранится в системе контроля версий.
173-173: Замените неформальный комментарий на подробное объяснение.Комментарий "goob" не объясняет причину высокой стоимости в 30 телекристаллов. Рекомендуется добавить понятное объяснение баланса стоимости.
- Telecrystal: 30 # goob + Telecrystal: 30 # Высокая стоимость обусловлена мощностью оружия в руках медицинского персонала
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (6)
Resources/Prototypes/Catalog/Fills/Lockers/security.yml(1 hunks)Resources/Prototypes/Catalog/uplink_catalog.yml(2 hunks)Resources/Prototypes/_Backmen/Catalog/uplink_catalog.yml(1 hunks)Resources/Prototypes/_Backmen/Catalog/uplink_nomad.yml(3 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Cartridges/pistol.yml(4 hunks)Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml(3 hunks)
🧰 Additional context used
🪛 GitHub Actions: YAML Linter
Resources/Prototypes/_Backmen/Catalog/uplink_nomad.yml
[error] 1-1: Duplicate ID: 'UplinkVectorBundle' for kind 'Content.Shared.Store.ListingPrototype'.
⏰ Context from checks skipped due to timeout of 300000ms (2)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (6)
Resources/Prototypes/_Backmen/Catalog/uplink_catalog.yml (2)
48-61: Проверьте баланс стоимости Chainsword.Стоимость в 8 телекристаллов кажется завышенной для оружия ближнего боя, доступного только капеллану. Рекомендуется пересмотреть цену с учетом других подобных предметов в каталоге.
62-79: Отличная реализация видоспецифичного предмета!Грамотно реализованы ограничения по виду и блокировка SurplusBundle для предотвращения злоупотреблений.
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Cartridges/pistol.yml (1)
45-46: Стандартизация тегов выглядит правильной!Добавление тега "Cartridge" улучшает согласованность и категоризацию сущностей боеприпасов.
Also applies to: 72-73, 100-101, 128-129
Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml (1)
110-110:✅ Verification successful
Проверьте согласованность изменений прото-полей!
Обновление полей proto с "Bullet" на "Cartridge" выглядит правильным, но убедитесь, что все соответствующие сущности и ссылки также обновлены.
Also applies to: 118-118, 126-126, 160-160, 168-168, 176-176, 243-243
🏁 Script executed:
#!/bin/bash # Проверка согласованности использования Cartridge/Bullet rg -i "bullet.*proto|cartridge.*proto" --type yamlLength of output: 1140
Обновление прото-полей выглядит корректным – проверьте, что ссылки на новые значения учтены во всех связанных сущностях!
- В файле Resources/Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml (строка 110 и аналогичные: 118, 126, 160, 168, 176, 243) значение поля
protoуспешно обновлено на"CartridgePistolACPUranium".- Результаты поиска показали, что в Resources/Maps/oasis.yml встречаются строки с
[bullet], однако, судя по формату, это элементы разметки/текста, а не ссылки на прото-поля.- Рекомендуется ещё раз убедиться, что во всех файлах, где используются ссылки на прото-поля, изменения учтены и не нарушают логику приложения.
Resources/Prototypes/Catalog/Fills/Lockers/security.yml (1)
195-195: Переименование магазина соответствует стандартизации!Изменение MagazinePistol на MagazinePistol9x17 улучшает ясность и согласованность с системой именования.
Resources/Prototypes/Catalog/uplink_catalog.yml (1)
465-475: Удаление устаревших записей каталога выглядит правильным!Очистка неиспользуемых записей оружия и боеприпасов улучшает поддерживаемость каталога.
Also applies to: 477-487, 513-522, 524-533
| - type: listing | ||
| id: UplinkVectorBundle | ||
| name: набор "MP7" | ||
| description: Модель SMG7 с архаичным дизайном, насчитывающим сотни лет, была распространена в свое время. Сегодня он находит ограниченное применение в качестве дешевых компьютерных копий или семейных реликвий, хотя каким-то образом попал в руки колониальных повстанцев. В комплетке 3 магазина. | ||
| icon: { sprite: Backmen/Objects/Weapons/Gunsx64/SMGs/mp7/big.rsi, state: icon } | ||
| productEntity: ClothingBackpackDuffelSyndicateFilledMP7 | ||
| cost: | ||
| Telecrystal: 15 | ||
| categories: | ||
| - UplinkBundles | ||
|
|
There was a problem hiding this comment.
Исправьте дублирующийся ID!
Обнаружен критический конфликт: ID 'UplinkVectorBundle' дублируется. Это вызовет проблемы при загрузке прототипов.
Примените следующие изменения:
-- type: listing
- id: UplinkVectorBundle
+ id: UplinkMP7Bundle
name: набор "MP7"📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| - type: listing | |
| id: UplinkVectorBundle | |
| name: набор "MP7" | |
| description: Модель SMG7 с архаичным дизайном, насчитывающим сотни лет, была распространена в свое время. Сегодня он находит ограниченное применение в качестве дешевых компьютерных копий или семейных реликвий, хотя каким-то образом попал в руки колониальных повстанцев. В комплетке 3 магазина. | |
| icon: { sprite: Backmen/Objects/Weapons/Gunsx64/SMGs/mp7/big.rsi, state: icon } | |
| productEntity: ClothingBackpackDuffelSyndicateFilledMP7 | |
| cost: | |
| Telecrystal: 15 | |
| categories: | |
| - UplinkBundles | |
| - type: listing | |
| id: UplinkMP7Bundle | |
| name: набор "MP7" | |
| description: Модель SMG7 с архаичным дизайном, насчитывающим сотни лет, была распространена в свое время. Сегодня он находит ограниченное применение в качестве дешевых компьютерных копий или семейных реликвий, хотя каким-то образом попал в руки колониальных повстанцев. В комплетке 3 магазина. | |
| icon: { sprite: Backmen/Objects/Weapons/Gunsx64/SMGs/mp7/big.rsi, state: icon } | |
| productEntity: ClothingBackpackDuffelSyndicateFilledMP7 | |
| cost: | |
| Telecrystal: 15 | |
| categories: | |
| - UplinkBundles |
There was a problem hiding this comment.
Error: /Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml(110,12)
No EntityPrototype found with id CartridgePistolACPUranium
Error: /Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml(118,12)
No EntityPrototype found with id CartridgePistolACPPractice
Error: /Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml(160,12)
No EntityPrototype found with id CartridgePistol5
Error: /Prototypes/_Backmen/Entities/Objects/Weapons/Guns/Ammunition/Magazine/pistol.yml(168,12)
No EntityPrototype found with id CartridgePistol57x2
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Resources/Prototypes/_Backmen/Catalog/uplink_nomad.yml (2)
1-2: Отрегулируйте неформальный комментарий в заголовке файла.Комментарий
#no_mad shitможет быть воспринят как неформальный и не соответствующий корпоративным стандартам. Рекомендуется либо удалить его, либо заменить на более нейтральное описание.
638-648: Уточните описание набора улучшений для оружия.Набор
UplinkUpgradePackсодержит обвесы для ограниченного набора моделей оружия, как указано в описании.
Рекомендуется перепроверить, что описание точно отражает функционал набора и не содержит устаревшей информации. Возможно, стоит добавить ссылку на актуальную документацию или уточнение по обновляемым моделям.🧰 Tools
🪛 YAMLlint (1.35.1)
[error] 641-641: syntax error: mapping values are not allowed here
(syntax)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
Resources/Prototypes/_Backmen/Catalog/uplink_nomad.yml(3 hunks)
🔇 Additional comments (9)
Resources/Prototypes/_Backmen/Catalog/uplink_nomad.yml (9)
3-121: Проверьте консистентность и локализацию новых лотов боеприпасов.В этом блоке добавлены новые списки для боеприпасов (например,
UplinkMosinAmmo,UplinkAKAmmo,Uplink308winAmmo,UplinkFuelTankFirethrower,UplinkMagazineMagnumPistol,UplinkMagazine50BMGMag,UplinkMagazine9x17,UplinkMagazinePistol9x17,UplinkMagazinePistol57x28,UplinkMagazineSMG57x28иUplinkMagazineSMGVector45ACP).
Проверьте, что:
- Все ключи (такие как
id,name,description,productEntity,costиcategories) оформлены в едином стиле.- Значения стоимости (например,
Telecrystal: 6или4) и тексты описаний корректны и соответствуют требованиям игры.- Локализация и терминология (например, название предмета и описание) отвечают стандартам проекта.
122-243: Проверьте корректность оформления наборов оружия и сущностей в блоке Bundles.Блок включает различные наборы (например,
UplinkP90Bundle,UplinkAKMSBundle) и связанные сущности (например,ClothingBackpackDuffelSyndicateFilledP90,ClothingBackpackDuffelSyndicateFilledAKMS).
Убедитесь, что:
- Все ссылки на
productEntityкорректны.- Описания, иконки и стоимости оформлены в соответствии с общим оформлением прототипов.
- Идентификаторы уникальны и не конфликтуют с другими объектами.
234-243:⚠️ Potential issueПроверьте дублирование идентификатора набора оружия.
В блоке с идентификатором
UplinkVectorBundleотражается набор с названием «Vector». Согласно предыдущим замечаниям и описанию изменений, этот идентификатор должен был быть переименован вUplinkMP7Bundle. Если требуется сохранить два разных объекта, необходимо удостовериться, что их идентификаторы не конфликтуют и правильно используются в системе.
255-265: Проверьте корректность замены набора на MP7.Новый блок с идентификатором
UplinkMP7Bundleвыглядит как замена для устаревшей записи сUplinkVectorBundle. Убедитесь, что во всём проекте ссылки на этот объект обновлены, и что новый набор (с названием «MP7») соответствует задумке по балансу и описанию.
266-589: Проверьте консистентность наборов оружия и игровых сущностей.В данном фрагменте добавлены различные наборы и связанные с ними сущности, включая:
- Наборы:
UplinkMP38Bundle,UplinkPPSH41Bundle,UplinkDEagle,UplinkZauerBundle,UplinkGrenadeLauncher79Bundle,UplinkRPDBundle,UplinkMG42Bundle,UplinkBR64Bundle,UplinkSVDBundle,UplinkSpas12Bundle,UplinkFlamethrowerBundle,UplinkNightVisionDevice,UplinkSniperSR127Bundleи т.д.- Сопутствующие сущности:
ClothingBackpackDuffelSyndicateFilledMP38,ClothingBackpackDuffelSyndicateFilledPPSH41,ClothingBackpackDuffelSyndicateFilledZauer,ClothingBackpackDuffelSyndicateFilledGL79,ClothingBackpackDuffelSyndicateFilledM81,ClothingBackpackDuffelSyndicateFilledRPD,ClothingBackpackDuffelSyndicateFilledMG42,ClothingBackpackDuffelSyndicateFilledBR64,ClothingBackpackDuffelSyndicateFilledSVD,ClothingBackpackDuffelSyndicateFilledSPAS12,CrateFlamethrowerBundle,BriefcaseSyndicateSniperBundleFilledи другие.Рекомендуется проверить, что:
- Для каждого набора корректно настроены поля (описание, иконка, стоимость и категории).
- Все ссылки на объекты через
productEntityактуальны.- Идентификаторы уникальны и не пересекаются с другими объектами в системе.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 533-533: wrong indentation: expected 4 but found 6
(indentation)
[warning] 583-583: wrong indentation: expected 4 but found 6
(indentation)
590-607: Проверьте параметры заполнения набора боеприпасов.Секция
ClothingBackpackDuffelSyndicateAmmoBackmenFilledобновляет содержание набора боеприпасов (например,MagazinePistolDrumSubMachineGun,MagazinePistolTopSubMachineGun,MagazinePistolSubMachineGun57x28,MagazinePistolSubMachineGunVector45ACPExt,MagazinePistol57x28Extd,MagazineRifleLight762x39Extи т.д.).
Убедитесь, что:
- Значения параметра
amountотражают задуманное количество для баланса игры.- Все ссылки на объекты (ID магазинов) корректны и соответствуют последним изменениям в системе.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 599-599: wrong indentation: expected 4 but found 6
(indentation)
620-637: Проверьте настройки набора медикаментов.В блоке
ClothingBackpackDuffelSyndicateMedicalBundleBackmenFilledопределён набор медикаментов, включающий аптечки, дефибриллятор, боевые медипены и другие предметы.
Убедитесь, что:
- Все предметы указаны верно.
- Значения количеств (например, количество аптечек и медипенов) соответствуют задумке игрового баланса.
🧰 Tools
🪛 YAMLlint (1.35.1)
[warning] 628-628: wrong indentation: expected 4 but found 6
(indentation)
649-663: Проверьте конфигурацию генератора UplinkEnergyDomeSyndie и условия его использования.Объект
UplinkEnergyDomeSyndieнастроен с указанием стоимости, категории и условиями покупки (черезStoreWhitelistConditionс тегомNukeOpsUplink).
Убедитесь, что:
- Описание и стоимость (Telecrystal) соответствуют его игровому балансу.
- Условия покупки корректно интегрированы в систему (при необходимости протестируйте логику применения условия).
664-677: Проверьте параметры энергетической батарейки UplinkGigareactorPowercell.В данном блоке указан объект
UplinkGigareactorPowercellс ценой, скидочной категорией и параметромdiscountDownTo.
Рекомендуется убедиться, что:
- Указанная цена и скидки соответствуют задуманным механикам.
- Нет конфликтов с аналогичными объектами и все данные корректно интегрируются в систему подсчёта стоимости.
Описание PR
чинит кое-что
Медиа
Тип PR
Изменения
Summary by CodeRabbit
Новые возможности
Рефакторинг / Стиль
Чистка