Skip to content

Исправление отсутствие возможности лежать и передавать предметы абдукторам#912

Merged
Remuchi merged 1 commit intoWWhiteDreamProject:masterfrom
Spatison:fix-abductors
Dec 25, 2025
Merged

Исправление отсутствие возможности лежать и передавать предметы абдукторам#912
Remuchi merged 1 commit intoWWhiteDreamProject:masterfrom
Spatison:fix-abductors

Conversation

@Spatison
Copy link
Copy Markdown
Member

Описание PR

#899


Изменения

🆑

  • fix: Абдукторы научились лежать и передавать предметы

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Dec 22, 2025

Обзор

В файле конфигурации базовых видов мобов переструктурирована секция MobState: записи OfferItem, LayingDown и MobState с параметрами allowMovementWhileCrit и allowMovementWhileSoftCrit перемещены в раздел MovedByPressure, удалены из другого места конфигурации.

Изменения

Когорта / Файл(ы) Резюме изменений
Конфигурация видов мобов
Resources/Prototypes/Entities/Mobs/Species/base.yml
Переместены блоки OfferItem, LayingDown и MobState в секцию MovedByPressure; добавлены параметры allowMovementWhileCrit и allowMovementWhileSoftCrit с значением true; удалены дублирующие определения из FireVisuals раздела

Оценка трудозатрат на проверку кода

🎯 2 (Простая) | ⏱️ ~8 минут

  • Требуется проверить корректность переноса параметров MobState и их влияние на поведение движения мобов при критическом состоянии
  • Убедиться, что удаление дублирующих определений не нарушает функциональность FireVisuals

Предлагаемые метки

Changes: YML, Status: Needs Review

Предлагаемые рецензенты

  • Remuchi
  • DVOniksWyvern

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed Название четко описывает основное изменение: восстановление возможности абдукторам ложиться и передавать предметы, что полностью соответствует измененияю в конфигурации базового вида мобов.
Description check ✅ Passed Описание связано с изменениями в пул-реквесте, указывает на связанный тикет #899 и объясняет цель (восстановление функционала лежания и передачи предметов для абдукторов).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 87aec9c and acd247e.

📒 Files selected for processing (1)
  • Resources/Prototypes/Entities/Mobs/Species/base.yml
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: Lachklen
Repo: WWhiteDreamProject/wwdpublic PR: 863
File: Resources/Locale/en-US/_white/cards/hobby.ftl:5-5
Timestamp: 2025-10-22T20:13:31.389Z
Learning: В PR по EvacPod для репозитория WWhiteDreamProject/wwdpublic: если опечатка в ключе локализации (например, card-ep_hobby_mashrooms) не влияет на отображение и используется последовательно, автор предпочитает не блокировать PR; вместо этого предлагать необязательный follow-up issue.
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: Test Packaging
  • GitHub Check: YAML Linter
  • GitHub Check: build (ubuntu-latest)
  • GitHub Check: YAML map schema validator
  • GitHub Check: build (ubuntu-latest)
🔇 Additional comments (1)
Resources/Prototypes/Entities/Mobs/Species/base.yml (1)

252-260: Изменения применяют компоненты ко всем видам мобов, а не только к абдукторам.

Добавление OfferItem и LayingDown в BaseMobSpecies обеспечит абдукторам возможность ложиться и передавать предметы (поскольку MobAbductor наследует от BaseMobSpecies), но повлияет на все виды мобов, наследующие эту базовую сущность.

Эти компоненты ранее были определены только в отдельных видах (shadowkin, skeleton) и типах игроков (ipc). Если консолидация в базовую сущность намеренна и соответствует дизайну игры, то изменение корректно. Однако убедитесь, что это желаемое поведение для всех наследников BaseMobSpecies, а не только для абдукторов.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Remuchi
Copy link
Copy Markdown
Collaborator

Remuchi commented Dec 25, 2025

Test fail is unrelated to the PR

@Remuchi Remuchi merged commit 6ee9859 into WWhiteDreamProject:master Dec 25, 2025
11 of 12 checks passed
Lachklen pushed a commit to Lachklen/MYwwdpublic that referenced this pull request Dec 25, 2025
@Spatison Spatison deleted the fix-abductors branch January 8, 2026 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants