Уменьшение структурного урона & пропорций размера у Scp457

[666XxttimurkaxX666]

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

безопасное время и чуть чуть нерф 457

Ссылка на багрепорт/Предложение

Медиа (Видео/Скриншоты)

Changelog

🆑 timur

• tweak: теперь SCP-457 нужно быть гораздо меньше, чтобы входить в вентиляцию
• tweak: структурный урон SCP-457 уменьшен со 100 до 50 единиц

Summary by CodeRabbit

• Изменения игрового баланса

  • Уменьшен максимальный размер малой формы SCP-457.
  • Снижено структурное повреждение SCP-457.
  • Интервал уменьшения размера увеличен с 12 до 18 секунд.
  • SCP-457 не может входить в вентиляцию во время безопасного времени независимо от размера.
  • Малая форма не использует соответствующую маску столкновений во время безопасного времени.
  • Малая форма может проходить через гермозатворы со шлюзами.

• Документация

  • Обновлено руководство по уязвимости SCP-457 к воде и обмораживающему урону.
  • Добавлено описание безопасного времени.

[coderabbitai]

📝 Walkthrough

Walkthrough

Изменены параметры формы и распада SCP-457. Добавлен SafeTime, который блокирует вход в вентиляцию и использование маски малой формы. Руководство обновлено описаниями неуязвимости, уязвимости к воде и ограничений SafeTime.

Changes

Механики SCP-457

Layer / File(s)	Summary
Параметры формы и распада Content.Server/_Scp/Scp457/Scp457Component.cs, Content.Server/_Scp/Scp457/Scp457System.cs	SmallFormSize изменён на 0.5f, StructuralDamage — на 50f, а DecayInterval — на 18 секунд.
Ограничения SafeTime и описание механик Content.Server/_Scp/Scp457/Scp457System.cs, Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp457.yml, Resources/ServerInfo/_Scp/Guidebook/Scp457/*.xml	К сущности SCP-457 добавлен SafeTime. Во время активного SafeTime вход в вентиляцию отменяется, а маска малой формы не используется независимо от размера сущности. Руководство описывает эти ограничения, неуязвимость к урону, уязвимость к воде и проход через гермозатворы со шлюзами.

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

Possibly related PRs

• space-sunrise/project-fire#1094: изменяет те же компоненты, систему и прототип SCP-457.
• space-sunrise/project-fire#1100: содержит связанные изменения механик SCP-457 и его прототипа.
• space-sunrise/project-fire#1105: изменяет те же разделы руководства SCP-457.

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

Suggested reviewers: wardexofficial

---

Important

Pre-merge checks failed

Please resolve all errors before merging. Addressing warnings is optional.

❌ Failed checks (1 error)

Check name	Status	Explanation	Resolution
Ss14 C# Rules	❌ Error	В изменённом C# коде DecayInterval нарушает правило _ для private-полей, а SafeTime-проверка добавлена напрямую в OnVentCrawlAttempt без потока OnEvent -> TryDo -> CanDo -> Do.	Переименуйте поле в _decayInterval и вынесите проверку входа в вентиляцию в TryDo/CanDo с вызовом из обработчика события; сохраните маркеры, если код будет вынесен в vanilla-файл.

✅ Passed checks (10 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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	Проверка diff от origin/master не выявила изменений в .agents/rules, .agents/skills или bridge-каталогах; изменены только игровые и справочные файлы.
Ss14 Fork/Project Folder Selection	✅ Passed	Активная ветка — Fire: remote имеет slug project-fire, а все 5 изменённых файлов находятся в каталогах _Scp; новых файлов или Sunrise-маркеров нет.
Ss14 Yaml/Ftl Rules	✅ Passed	Изменён только scp457.yml; SafeTime записан без суффикса Component, ID Scp457 и Scp457Flame в CamelCase, кириллицы в fallback нет.
Ss14 Prototype ↔ Ftl Parity	✅ Passed	Проверка применима: PR добавляет только компонент SafeTime к существующим Scp457 и Scp457Flame. ID и поля name/description/suffix не изменены; соответствующие ent-* ключи уже существуют.
Ss14 Prediction Safety	✅ Passed	Полный diff PR не изменяет Content.Shared, Content.Client или predicted input; изменённый Scp457System находится в Content.Server и не добавляет указанные prediction hazards.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Заголовок точно описывает основные изменения параметров SCP-457: уменьшение структурного урона и размера.

✨ 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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[coderabbitai]

Actionable comments posted: 1

🤖 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 `@Content.Server/_Scp/Scp457/Scp457System.cs`:
- Line 48: Rename the private field DecayInterval to _decayInterval and update
all references to it. Since the value is never modified after initialization,
make it static readonly if compatible with its usage.

🪄 Autofix

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 Plus

Run ID: 4d361b7a-83b2-471f-98b9-513c454393ca

📥 Commits

Reviewing files that changed from the base of the PR and between edf6069 and 437f681.

📒 Files selected for processing (3)

• Content.Server/_Scp/Scp457/Scp457Component.cs
• Content.Server/_Scp/Scp457/Scp457System.cs
• Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp457.yml

[666XxttimurkaxX666]

@WardexOfficial во всё делай

[666XxttimurkaxX666]

го мержи и обновляй го

[666XxttimurkaxX666]

сервер го

[coderabbitai]

Actionable comments posted: 1

🤖 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 `@Content.Server/_Scp/Scp457/Scp457System.cs`:
- Around line 281-283: Update the SCP-457 SafeTime handling to detect the
transition from active to completed, or schedule a one-time action at TimeEnd,
and invoke UpdateAppearanceAndPhysics immediately after it ends. Ensure
collisionMask changes from SmallMobMask to MobMask even when the object size no
longer decreases and Update() skips the resize path.

🪄 Autofix

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 Plus

Run ID: 1ee9fed1-b62a-460b-ae52-6d0d5d039e8a

📥 Commits

Reviewing files that changed from the base of the PR and between 437f681 and 57a8d82.

📒 Files selected for processing (1)

• Content.Server/_Scp/Scp457/Scp457System.cs

[666XxttimurkaxX666]

@WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ @WardexOfficial СРОЧНО МЕРЖ

[WardexOfficial]

Вырежи у себя механику с SafeTime, я реализую её правильно отдельным ПРом

[WardexOfficial]

Вообще убери изменения из файлов Content.Server/_Scp/Scp457/Scp457System.cs, Resources/Prototypes/_Scp/Entities/Mobs/Player/Scp/Main/scp457.yml
Я сделаю это отдельным ПРом, более лучше