Skip to content

Исправления после Апстрима (Лицо 096, система захвата, пробирки амнезиака) - #1154

Merged
WardexOfficial merged 1 commit into
makura-games:masterfrom
WardexOfficial:fix/fixes1
Sep 13, 2026
Merged

WardexOfficial merged 1 commit into
makura-games:masterfrom
WardexOfficial:fix/fixes1

Conversation

@WardexOfficial

@WardexOfficial WardexOfficial commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Краткое описание

Исправления после Апстрима (Лицо 096, система захвата, пробирки амнезиака)

Changelog
🆑 Wardex

  • fix: Лицо Scp-096 теперь нормально лечится
  • fix: Пробирки амнезиака более не пустые с начала раунда
  • fix: Игроки теперь могут таскать других игроков
    :end-cl:

Summary by CodeRabbit

  • Новые возможности

    • Добавлена поддержка удерживания сущностей гуманоидными персонажами при наличии подходящего профиля и отсутствии у цели компонента SCP.
    • Проверка дальности взаимодействия теперь может быть переопределена целевой сущностью.
    • Для станционного ИИ действия пользователей, не являющихся его владельцем, больше не проходят проверку дальности.
  • Исправления

    • Флаконы амнезиаков и мнезиков теперь используют контейнеры для напитков при сохранении прежнего состава и объёма.

@coderabbitai

coderabbitai Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 8569f1f3-c2eb-46eb-acf2-7c807e96e5aa

📥 Commits

Reviewing files that changed from the base of the PR and between fc10b56 and a53380c.

📒 Files selected for processing (5)
  • Content.Shared/Humanoid/Prototypes/RandomHumanoidSettingsPrototype.cs
  • Content.Shared/Interaction/SharedInteractionSystem.cs
  • Content.Shared/Silicons/StationAi/SharedStationAiSystem.cs
  • Resources/Prototypes/Body/Species/human.yml
  • Resources/Prototypes/_Scp/Entities/Objects/specific/amnesiac.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

Изменена обработка InRangeOverrideEvent для источника и цели взаимодействия. Для MobHuman добавлен ScpHolder. Для шести прототипов растворов выбран тип контейнера drink.

Changes

Взаимодействия и прототипы

Layer / File(s) Summary
Переопределение дальности взаимодействий
Content.Shared/Interaction/SharedInteractionSystem.cs, Content.Shared/Silicons/StationAi/SharedStationAiSystem.cs
Целевая сущность теперь получает InRangeOverrideEvent, если источник и цель различны. OnAiInRange пропускает пользователей, которые не являются владельцами станционного ИИ.
Ограничения компонента ScpHolder
Resources/Prototypes/Body/Species/human.yml
Прототип MobHuman требует HumanoidProfile для удерживаемых сущностей и исключает сущности с компонентом Scp.
Типы контейнеров растворов
Resources/Prototypes/_Scp/Entities/Objects/specific/amnesiac.yml, Content.Shared/Humanoid/Prototypes/RandomHumanoidSettingsPrototype.cs
Для шести прототипов тип контейнера изменён с beaker на drink. В классе настроек добавлен комментарий-маркер // Fire edit end.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested labels: 🛠️ Есть C# 🛠️, 📦 YAML | Прототип 📦

Suggested reviewers: kanopus952

Merge Risk: ⚪ Minimal · up to a5338

The changes remain mergeable with no identified current-head behavior that would disrupt interactions, carrying, or amnesiac vial use.

🚥 Pre-merge checks | ✅ 10 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 3 files. (2 skipped: 2 … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (10 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Заголовок точно описывает основные изменения: исправления для SCP-096, системы захвата и пробирок амнезиака. Формулировка краткая и понятная.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Ss14 Bridge Sync ✅ Passed Проверка пройдена. В review-scoped диапазоне изменены только 5 файлов в Content.Shared/... и Resources/Prototypes/.... Файлы в .agents/, .agent/, .claude/, .cursor/ и .github/ не изменен…
Ss14 Fork/Project Folder Selection ✅ Passed Проверка пройдена. Активный fork — Fire: remote имеет точный slug project-fire, а в репозитории присутствует _Scp. Все новые изменения в vanilla-файлах помечены Fire edit или Fire added. Fire-…
Ss14 C# Rules ✅ Passed Проверка применима: PR изменяет три .cs файла. В diff добавлены только маркер // Fire edit end, вызов target-side InRangeOverrideEvent и ранний выход для пользователей, отличных от владельца Sta…
Ss14 Yaml/Ftl Rules ✅ Passed Проверка применима к двум изменённым YAML-файлам. В diff добавлена запись ScpHolder без суффикса Component, как требует SS14 YAML. Структура holdableWhitelist и holdableBlacklist совпадает с с…
Ss14 Prototype ↔ Ftl Parity ✅ Passed Проверка применима, так как PR изменяет два файла в Resources/Prototypes/**. В human.yml существующий MobHuman получает только компонент ScpHolder; его ID и поле name не изменяются. В `amnes…
Ss14 Prediction Safety ✅ Passed Проверка применима: PR изменяет Content.Shared и общий путь InRangeUnobstructed. Добавленный вызов RaiseLocalEvent(other, ref ev) использует только детерминированную обработку события. В изменён…
Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 3 files. (2 skipped: 2 unsupported.)

  • Fix all pre-merge checks with AI
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@WardexOfficial
WardexOfficial merged commit 7186a99 into makura-games:master Sep 13, 2026
23 of 32 checks passed
@WardexOfficial
WardexOfficial deleted the fix/fixes1 branch September 13, 2026 14:36
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.

1 participant