Добавление новой способности для SCP-106 "Загрязнение"#1046
Добавление новой способности для SCP-106 "Загрязнение"#1046666XxttimurkaxX666 wants to merge 8 commits into
Conversation
|
Warning Review limit reached
More reviews will be available in 28 minutes and 23 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (9)
📝 WalkthroughWalkthroughДобавлена возможность SCP-106 создавать лужи содержащие новый реагент Scp106Reagent: новый реагент, прототип лужи, действие спауна, интеграция в сущность SCP-106 и локализации на en-US и ru-RU. ИзмененияСпособность воспроизведения лужи SCP-106
Диаграмма взаимодействияsequenceDiagram
participant Player as Игрок (SCP-106)
participant Action as Scp106PuddleSpawnAction
participant Spell as InstantSpawnSpellEvent
participant Puddle as PuddleScp106Reagent200
participant Reagent as Scp106Reagent
Player->>Action: Использовать способность
Action->>Spell: Вызвать InstantSpawnSpellEvent (prototype, TargetCasterPos)
Spell->>Puddle: Спаун сущности лужи
Puddle->>Reagent: Решение содержит 200 ед. Scp106Reagent
Reagent->>Player: (при контакте) наносит эффекты: урон, рвота, слепота, замедление
Оценка сложности рецензирования🎯 3 (Moderate) | ⏱️ ~20 minutes Suggested reviewers
Important Pre-merge checks failedPlease resolve all errors before merging. Addressing warnings is optional. ❌ Failed checks (1 error)
✅ Passed checks (9 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
![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
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Resources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftl`:
- Around line 1-2: Переименуйте строку ent-Scp106PuddleSpawnAction так, чтобы
название предмета начиналось со строчной буквы (например "тёмная субстанция") и
исправьте .desc для корректной пунктуации и орфографии (четкая фраза,
запятые/окончания: "Высвобождает среднее количество чёрной гнили, замедляющей
любого, кто наступит на неё." или похожая читабельная формулировка), обновив
соответствующие ключи ent-Scp106PuddleSpawnAction и
ent-Scp106PuddleSpawnAction.desc.
In `@Resources/Prototypes/_Scp/Reagents/scp106.yml`:
- Around line 17-53: Удалить все побочные эффекты, оставив только замедление: в
блоках, где упоминается Scp106Reagent убрать !type:HealthChange, !type:Vomit,
!type:GenericStatusEffect (TemporaryBlindness) и !type:ModifyBleed, а также
полностью удалить или заменить секцию Poison, чтобы она содержала только
!type:MovementSpeedModifier с соответствующими ReagentCondition (min thresholds)
и полями walkSpeedModifier/sprintSpeedModifier; оставьте только элементы с типом
MovementSpeedModifier и связанные ReagentCondition, удалив все остальные записи,
которые наносят урон, рвоту, слепоту или усиливают кровотечение.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 7f17c36c-3f51-47e9-bc5b-ce7fb44d432e
📒 Files selected for processing (8)
Resources/Locale/en-US/_prototypes/_scp/actions/scp106.ftlResources/Locale/en-US/_strings/_scp/reagents/reagents.ftlResources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftlResources/Locale/ru-RU/_strings/_scp/reagents/reagents.ftlResources/Prototypes/_Scp/Actions/scp106.ymlResources/Prototypes/_Scp/Entities/Effects/puddle.ymlResources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp106.ymlResources/Prototypes/_Scp/Reagents/scp106.yml
|
А так же не |
666XxttimurkaxX666
changed the title
|
@666XxttimurkaxX666 |
я эту хуйню создал и чо дальше как подписать или что там |
|
666XxttimurkaxX666
force-pushed
the
tweak2-scp-106
branch
from
a1f08e2 to
1a8325e
Compare
|
да сука всмысле unverified |
666XxttimurkaxX666
force-pushed
the
tweak2-scp-106
branch
from
1a8325e to
f0230a1
Compare
|
блять что за хуйню он запушил |
|
otkuda ono tut |
|
что за говнооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооооо |
|
оно все еще не верифицировано |
666XxttimurkaxX666
force-pushed
the
tweak2-scp-106
branch
from
f0230a1 to
27c9efe
Compare
|
наконец тооооооооооооооооо |
666XxttimurkaxX666
force-pushed
the
tweak2-scp-106
branch
from
229f092 to
8d303cf
Compare
666XxttimurkaxX666
force-pushed
the
tweak2-scp-106
branch
from
8d303cf to
ca55e46
Compare
|
@WardexOfficial а эту фигню с верификацией специально включили для этого проекта или она сама по себе в один момент появилась? |
|
просто у меня из за нее github desktop накрылся и теперь не хочет ничего принимать |
|
@WardexOfficial я починил го мерж |
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 (1)
Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml (1)
90-92:⚠️ Potential issue | 🟠 Major | ⚡ Quick winДля
ListingLimitedStockConditionне заданstock.На Line 91-92 условие ограниченного стока объявлено без значения
stock. В лучшем случае лимит не применится как ожидалось, в худшем — конфиг невалиден для этого типа.💡 Предлагаемая правка
conditions: - !type:ListingLimitedStockCondition + stock: 1🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml` around lines 90 - 92, В блоке conditions у объекта ListingLimitedStockCondition отсутствует обязательное поле stock — откройте определение условия ListingLimitedStockCondition и добавьте ключ stock с целочисленным значением или ссылкой на переменную/параметр (например stock: 10 или stock: ${maxStock}) так чтобы условие ограниченного стока имело корректное число; отредактируйте запись ListingLimitedStockCondition в файле (символ: ListingLimitedStockCondition) чтобы включить stock и сохранить валидность конфигурации.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Outside diff comments:
In `@Resources/Prototypes/_Scp/Catalog/Scp106Catalog.yml`:
- Around line 90-92: В блоке conditions у объекта ListingLimitedStockCondition
отсутствует обязательное поле stock — откройте определение условия
ListingLimitedStockCondition и добавьте ключ stock с целочисленным значением или
ссылкой на переменную/параметр (например stock: 10 или stock: ${maxStock}) так
чтобы условие ограниченного стока имело корректное число; отредактируйте запись
ListingLimitedStockCondition в файле (символ: ListingLimitedStockCondition)
чтобы включить stock и сохранить валидность конфигурации.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: 8bfadf0a-2354-45dc-8f8a-325316cb2e2b
📒 Files selected for processing (11)
Resources/Locale/en-US/_prototypes/_scp/actions/scp106.ftlResources/Locale/en-US/_strings/_scp/catalog/catalog.ftlResources/Locale/en-US/_strings/_scp/reagents/reagents.ftlResources/Locale/ru-RU/_prototypes/_scp/actions/scp106.ftlResources/Locale/ru-RU/_strings/_scp/catalog/catalog.ftlResources/Locale/ru-RU/_strings/_scp/reagents/reagents.ftlResources/Prototypes/_Scp/Actions/scp106.ymlResources/Prototypes/_Scp/Catalog/Scp106Catalog.ymlResources/Prototypes/_Scp/Entities/Effects/puddle.ymlResources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp106.ymlResources/Prototypes/_Scp/Reagents/scp106.yml
|
@WardexOfficial я изменил цену покупки фантома на 0 из за того что люди на 106 могут изначально купить любую другую способность и после не смогут купить фантома (ну то есть это будет софтлок) |
Нужно дать 106 способность становиться фантомом по умолчанию т.к сначала игры можно купить вместо фантома клинок и тоже получится софтлок. |
|
@666XxttimurkaxX666 Да, убери у 106 способность фантома из магазина и добавь её по умолчанию, так же вычти её стоимость из начальных очков магазина 106 |
3 participants
Краткое описание | Short description
теперь у сцп 106 есть новая способность спавна жижи
если что она только замедляет и всё
Ссылка на багрепорт/Предложение | Related Issue/Bug Report
Медиа (Видео/Скриншоты) | Media (Video/Screenshots)
Changelog
🆑 timur
Summary by CodeRabbit
Новые возможности
Локализация